Skip to content

Update @github/copilot to 1.0.21#1039

Merged
stephentoub merged 4 commits intomainfrom
update-copilot-1.0.21
Apr 8, 2026
Merged

Update @github/copilot to 1.0.21#1039
stephentoub merged 4 commits intomainfrom
update-copilot-1.0.21

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Apr 7, 2026

Automated bump of @github/copilot to 1.0.21. The codegen renamed session.compaction.compactsession.history.compact and added new session.history.truncate and sessions.fork RPCs, requiring updates across tests and docs.

Generated code

  • Updated @github/copilot in nodejs/package.json and test/harness/package.json
  • Re-ran codegen for all 4 SDKs (Node.js, Python, Go, .NET)

E2E test fixes

  • Updated all references from compaction.compact() to history.compact() across Node.js, Python, Go, and .NET E2E tests

Documentation

  • Updated docs/troubleshooting/compatibility.md to reflect the rename and added missing entries for session.history.truncate and sessions.fork
- const result = await session.rpc.compaction.compact();
+ const result = await session.rpc.history.compact();

- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code
@github-actions github-actions bot requested a review from a team as a code owner April 7, 2026 21:43
@stephentoub
Copy link
Copy Markdown
Collaborator

@copilot is there any documentation or tests or anything that needs to be updated in reaction to these changes? Please ensure everything builds and all tests pass.

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

@copilot is there any documentation or tests or anything that needs to be updated in reaction to these changes? Please ensure everything builds and all tests pass.

Yes — the generated code renamed session.compaction.compact to session.history.compact, which broke references in E2E tests and documentation. Fixed in ce6b64d:

  • Updated E2E tests in all 4 SDKs (Node.js, Python, Go, .NET) to use history.compact() instead of compaction.compact()
  • Updated docs/troubleshooting/compatibility.md to reflect the new API path

Builds pass for all SDKs. Unit tests pass: Node.js (205 passed, 3 pre-existing vision timeout flakes), Python (87 passed), Go (all passed), .NET (all passed). Go E2E tests also compile successfully with the updated API.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/copilot_internal/user
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio (http block)
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio 2/x64/bin/git (http block)
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level debug --stdio x64/bin/bash committer.name (http block)

If you need me to access, download, or install something from one of these locations, you can either:

…ilot 1.0.21

Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/3afab108-c1a1-4942-9bce-f4c96ce1936a

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Copilot AI changed the title Update @github/copilot to 1.0.21 Fix tests and docs for compaction → history RPC rename in @github/copilot 1.0.21 Apr 7, 2026
Copilot AI requested a review from stephentoub April 7, 2026 22:08
@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/copilot_internal/user
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level info --stdio ndor/bin/as (http block)
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level info --stdio ache/go/1.24.13/-o (http block)
    • Triggering command: /opt/hostedtoolcache/node/22.22.2/x64/bin/node /opt/hostedtoolcache/node/22.22.2/x64/bin/node /home/REDACTED/work/copilot-sdk/copilot-sdk/nodejs/node_modules/@github/copilot/index.js --headless --no-auto-update --log-level info --stdio (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Fix tests and docs for compaction → history RPC rename in @github/copilot 1.0.21 Update @github/copilot to 1.0.21 Apr 7, 2026
Copilot AI requested a review from stephentoub April 7, 2026 22:16
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Apr 7, 2026

Cross-SDK Consistency Review ✅

This PR maintains excellent feature parity and consistency across all four SDK implementations. Here's a summary of the changes and their cross-SDK coverage:

Changes Reviewed

Change Node.js Python Go .NET
session.compaction.compactsession.history.compact session.rpc.history.compact() session.rpc.history.compact() session.RPC.History.Compact() session.Rpc.History.CompactAsync()
New session.history.truncate session.rpc.history.truncate(params) session.rpc.history.truncate(params) session.RPC.History.Truncate(ctx, params) session.Rpc.History.TruncateAsync(eventId)
New sessions.fork (server-level) server.rpc.sessions.fork(params) sessions_rpc.fork(params) ServerSessionsApi.Fork(ctx, params) ServerRpc.ForkAsync(sessionId, ...)

Observations

  • API placement is consistent: sessions.fork is correctly scoped as a server-level RPC in all SDKs, while history.compact and history.truncate remain session-scoped.
  • Naming conventions respected: camelCase (Node.js), snake_case (Python), PascalCase (Go/exported, .NET).
  • E2E tests updated in all four SDKs for the compact rename.
  • Documentation updated in docs/troubleshooting/compatibility.md to reflect the rename and the two new RPCs.

No inconsistencies found. 🎉

Generated by SDK Consistency Review Agent for issue #1039 ·

@stephentoub stephentoub merged commit 9351e26 into main Apr 8, 2026
36 checks passed
@stephentoub stephentoub deleted the update-copilot-1.0.21 branch April 8, 2026 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants