Skip to content

Update @github/copilot to 1.0.73#2055

Merged
stephentoub merged 1 commit into
mainfrom
update-copilot-1.0.73
Jul 22, 2026
Merged

Update @github/copilot to 1.0.73#2055
stephentoub merged 1 commit into
mainfrom
update-copilot-1.0.73

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.73.

Changes

  • Updated @github/copilot in nodejs/package.json and test/harness/package.json
  • Re-ran all code generators (scripts/codegen)
  • Formatted generated output
  • Updated Java codegen dependency, POM property, and regenerated Java types

Java Handwritten Code Adaptation Plan

If java-sdk-tests CI fails on this PR, follow these steps:

  1. Identify failures: Run mvn clean, mvn verify from java/ locally or check the java-sdk-tests workflow run logs.
  2. Categorize errors:
    • Constructor signature changes (new fields added to generated records)
    • Enum value additions/renames in generated types
    • New event types requiring handler registration
    • Removed or renamed generated types
  3. Fix handwritten source (java/src/main/java/com/github/copilot/sdk/):
    • Update call sites passing positional constructor args to include new fields (typically null for optional new fields).
    • Update switch/if-else over enum values to handle new cases.
    • Register handlers for new event types in CopilotSession.java if applicable.
  4. Fix handwritten tests (java/src/test/java/com/github/copilot/sdk/):
    • Same constructor/enum fixes as above.
    • Add new test methods for new functionality if the change adds user-facing API surface.
  5. Validate: cd java && mvn clean test-compile jar:jar && mvn verify -Dskip.test.harness=true
  6. Format: cd java && mvn spotless:apply
  7. Push fixes to this PR branch.

To automate this, trigger the java-adapt-handwritten-code-to-accept-upgrade-changes agentic workflow instead.

Next steps

When ready, click Ready for review to trigger CI checks.

Created by the Update @github/copilot Dependency workflow.

- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code
@stephentoub
stephentoub marked this pull request as ready for review July 22, 2026 20:11
@stephentoub
stephentoub requested a review from a team as a code owner July 22, 2026 20:11
Copilot AI review requested due to automatic review settings July 22, 2026 20:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@github-actions

Copy link
Copy Markdown
Contributor Author

Cross-SDK Consistency Review ✅

This PR fixes the ask_user / userInput.request event-loop starvation bug in the Rust SDK and adds an E2E regression test. Here is the consistency assessment:

Change summary

  • Rust SDK (rust/src/session.rs): Moves handle_request dispatch from inline await to a tokio::spawn'd task — each inbound JSON-RPC request now runs concurrently instead of serializing on the event loop. This matches the behavior of all other five SDKs.
  • E2E regression test (rust/tests/e2e/ask_user.rs): Adds ask_user_does_not_block_sibling_tool_call_in_same_turn, covering the exact starvation scenario.
  • Codegen updates: agentInvocationId added to LlmInferenceHttpRequestChunkRequest — consistently applied across all languages (TypeScript, Python, Go, .NET, Java, Rust generated files).

Cross-SDK consistency

SDK Per-request concurrency ask_user E2E test
Node.js onRequest handler (JSON-RPC library dispatches concurrently) nodejs/test/e2e/ask_user.e2e.test.ts
Python ✅ Per-request spawn python/e2e/test_ask_user_e2e.py
Go ✅ Per-request goroutine go/internal/e2e/ask_user_e2e_test.go
.NET ✅ StreamJsonRpc dispatches concurrently
Java ✅ Per-request async
Rust Fixed in this PR Added in this PR

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 userInput.request handlers are worth a follow-up audit for the same inline-await asymmetry" for dotnet/go/python/nodejs. This is a separate concern and not a blocking issue for this PR — the other SDKs' RPC frameworks handle concurrency at a higher level than Rust's manual select! loop.

Overall: this PR maintains and improves cross-SDK consistency. No action required from other SDK owners.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

Generated by SDK Consistency Review Agent for #2055 · 41.4 AIC · ⌖ 5.25 AIC · ⊞ 5K ·

@stephentoub
stephentoub added this pull request to the merge queue Jul 22, 2026
Merged via the queue into main with commit a54b0b5 Jul 22, 2026
108 of 111 checks passed
@stephentoub
stephentoub deleted the update-copilot-1.0.73 branch July 22, 2026 23:12
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