Update @github/copilot to 1.0.73#2055
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
There was a problem hiding this comment.
Pull request overview
Updates the Copilot runtime dependency to 1.0.73 and synchronizes generated RPC types across SDKs.
Changes:
- Updated package manifests and lockfiles to 1.0.73.
- Regenerated RPC models with optional per-chunk agent invocation IDs.
- Updated Java codegen metadata and generated record.
Show a summary per file
| File | Description |
|---|---|
test/harness/package.json |
Updates the harness dependency. |
test/harness/package-lock.json |
Locks 1.0.73 platform packages. |
rust/src/generated/api_types.rs |
Adds optional chunk invocation ID. |
python/copilot/generated/rpc.py |
Adds invocation ID serialization. |
nodejs/src/generated/rpc.ts |
Adds the optional RPC field. |
nodejs/samples/package-lock.json |
Updates the sample dependency. |
nodejs/package.json |
Updates the runtime dependency. |
nodejs/package-lock.json |
Locks 1.0.73 platform packages. |
java/src/generated/java/com/github/copilot/generated/rpc/LlmInferenceHttpRequestChunkRequest.java |
Adds the generated record component. |
java/scripts/codegen/package.json |
Updates the codegen dependency. |
java/scripts/codegen/package-lock.json |
Locks codegen dependencies to 1.0.73. |
java/pom.xml |
Updates the reference implementation version. |
go/rpc/zrpc.go |
Adds optional chunk invocation ID. |
dotnet/src/Generated/Rpc.cs |
Adds the nullable generated property. |
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/14 changed files
- Comments generated: 0
- Review effort level: Medium
Cross-SDK Consistency Review ✅This PR fixes the Change summary
Cross-SDK consistency
The PR explicitly acknowledges that the other five SDKs already had concurrent per-request handling and brings Rust to parity. No new public API surface is introduced that would require mirroring in other SDKs. Minor follow-up note (non-blocking)The commit message flags a potential follow-up audit: "their Overall: this PR maintains and improves cross-SDK consistency. No action required from other SDK owners. Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "awmgmcpg"See Network Configuration for more information.
|
Automated update of
@github/copilotto version1.0.73.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.