Update @github/copilot to 1.0.69#1941
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s pinned @github/copilot dependency to 1.0.69 and regenerates SDK wire-protocol types to match the updated schema across multiple languages.
Changes:
- Bumped
@github/copilotversion from^1.0.69-3to^1.0.69in Node.js packages, the test harness, and Java codegen. - Regenerated RPC/schema-derived types across SDKs, including adding
reloadExtensionstoPluginsReloadRequest. - Updated dependency lockfiles to resolve to
@github/copilot@1.0.69and matching platform optional dependencies.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps harness devDependency @github/copilot to ^1.0.69. |
| test/harness/package-lock.json | Updates resolved @github/copilot tarball/integrity and optional platform packages to 1.0.69. |
| rust/src/generated/api_types.rs | Regenerated Rust API types; adds reload_extensions to PluginsReloadRequest. |
| python/copilot/generated/rpc.py | Regenerated Python RPC dataclasses; adds reload_extensions and (de)serialization support. |
| nodejs/src/generated/rpc.ts | Regenerated TS RPC interfaces; adds reloadExtensions to PluginsReloadRequest. |
| nodejs/samples/package-lock.json | Updates samples lockfile to pull @github/copilot@^1.0.69. |
| nodejs/package.json | Bumps Node.js SDK dependency @github/copilot to ^1.0.69. |
| nodejs/package-lock.json | Updates resolved @github/copilot tarball/integrity and optional platform packages to 1.0.69. |
| java/scripts/codegen/package.json | Bumps Java codegen dependency @github/copilot to ^1.0.69. |
| java/scripts/codegen/package-lock.json | Updates Java codegen lockfile to resolve @github/copilot@1.0.69. |
| java/pom.xml | Updates the tracked reference-impl version property from ^1.0.69-3 to ^1.0.69. |
| go/rpc/zrpc.go | Regenerated Go RPC types; adds ReloadExtensions and includes it in PluginsAPI.Reload request building. |
| dotnet/src/Generated/Rpc.cs | Regenerated .NET RPC types; adds ReloadExtensions and wires it into ReloadAsync. |
Review details
Files not reviewed (5)
- go/rpc/zrpc.go: Generated file
- java/scripts/codegen/package-lock.json: Generated file
- nodejs/package-lock.json: Generated file
- nodejs/samples/package-lock.json: Generated file
- test/harness/package-lock.json: Generated file
- Files reviewed: 4/13 changed files
- Comments generated: 2
- Review effort level: Low
This comment has been minimized.
This comment has been minimized.
Use the pending permission hydration RPC after resuming a session instead of reusing the pre-suspend request ID, which can be stale after continuePendingWork resumes the prompt. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
Preserve the original permission request ID across resume, matching runtime behavior, but wait until the resumed session exposes that ID through pendingRequests before responding. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
The runtime intends original permission request IDs to remain durable, but CI shows handling the pre-suspend ID can still return false after resume. Keep the SDK test on the respondable hydrated pending request ID and track the durable-ID mismatch as runtime follow-up. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Cross-SDK Consistency ReviewThis PR adds the new
🔍 Java InconsistencyThe Java SDK's The generated This PR's description says Java types were regenerated, but the diff shows no changes to
Suggested FixInvestigate the Java codegen ( This is a pre-existing inconsistency, but since this PR is specifically updating the
|
Automated update of
@github/copilotto version1.0.69.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Java Handwritten Code Adaptation Plan
If
java-sdk-testsCI fails on this PR, follow these steps:mvn clean,mvn verifyfromjava/locally or check thejava-sdk-testsworkflow run logs.java/src/main/java/com/github/copilot/sdk/):nullfor optional new fields).CopilotSession.javaif applicable.java/src/test/java/com/github/copilot/sdk/):cd java && mvn clean test-compile jar:jar && mvn verify -Dskip.test.harness=truecd java && mvn spotless:applyNext steps
When ready, click Ready for review to trigger CI checks.