Skip to content

Update @github/copilot to 1.0.76-5 - #2140

Merged
MackinnonBuck merged 6 commits into
mainfrom
update-copilot-1.0.76-5
Jul 30, 2026
Merged

Update @github/copilot to 1.0.76-5#2140
MackinnonBuck merged 6 commits into
mainfrom
update-copilot-1.0.76-5

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.76-5.

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
@github-actions

This comment has been minimized.

@MackinnonBuck MackinnonBuck self-assigned this Jul 29, 2026
Update protocol generators and handwritten adapters for the latest RPC schema, including session filesystem SQLite transactions and newly required request fields.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b5e2763-c535-4d58-bd7d-409c5d27c448
@MackinnonBuck
MackinnonBuck marked this pull request as ready for review July 29, 2026 23:13
@MackinnonBuck
MackinnonBuck requested a review from a team as a code owner July 29, 2026 23:13
Copilot AI review requested due to automatic review settings July 29, 2026 23:13
@github-actions

This comment has been minimized.

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.76-5 and synchronizes SDK protocol bindings and adapters.

Changes:

  • Regenerates RPC and event types across all SDKs.
  • Adapts SQLite transactions and changed request fields.
  • Updates dependencies, code generators, and compatibility tests.
Show a summary per file
File Description
nodejs/** Dependency, generated bindings, SQLite adapter, exports, and tests
python/** Generated bindings, SQLite provider, exports, and E2E tests
go/** Generated bindings, SQLite provider, and E2E adaptations
dotnet/** Generated bindings, SQLite provider, session adaptation, and tests
rust/** Generated bindings, SQLite dispatch, API adaptations, and tests
java/** Codegen updates, regenerated types, handwritten adaptations, and tests
scripts/codegen/** C# generator support for optional request objects
test/harness/** Updated runtime dependency and lockfile

Review details

Files not reviewed (8)
  • go/rpc/zrpc_encoding.go: Generated file
  • go/rpc/zsession_encoding.go: Generated file
  • go/rpc/zsession_events.go: Generated file
  • go/zsession_events.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: 45/289 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread nodejs/src/sessionFsProvider.ts
Comment thread dotnet/test/E2E/InMemorySessionFsSqliteHandler.cs Fixed
Comment thread dotnet/src/SessionFsProvider.cs Fixed
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b5e2763-c535-4d58-bd7d-409c5d27c448
@github-actions

This comment has been minimized.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b5e2763-c535-4d58-bd7d-409c5d27c448
@github-actions

This comment has been minimized.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b5e2763-c535-4d58-bd7d-409c5d27c448
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b5e2763-c535-4d58-bd7d-409c5d27c448
@github-actions

Copy link
Copy Markdown
Contributor Author

Cross-SDK Consistency Review ✅

This PR adds the sqliteTransaction feature consistently across all six SDK implementations. No consistency issues found.

Summary of Changes

All SDKs receive the same sqliteTransaction / SqliteTransaction / sqlite_transaction capability, with the same three-way error classification (fatal, busyOrLocked, postCommitAmbiguous) and appropriate language-idiomatic naming:

SDK New API Error Type Pattern
Node.js Optional transaction?() on SessionFsSqliteProvider SessionFsSqliteTransactionFailure (class extending Error) Consistent ✅
Python Optional sqlite_transaction() on SessionFsSqliteProvider SessionFsSqliteTransactionFailure (exception subclass) Consistent ✅
Go Optional SessionFSSqliteTransactionProvider interface *SessionFSSqliteTransactionFailure (error type) Consistent ✅
.NET Optional ISessionFsSqliteTransactionProvider interface SessionFsSqliteTransactionException (Exception subclass) Consistent ✅
Rust Optional sqlite_transaction() on SessionFsSqliteProvider trait SessionFsSqliteTransactionError (struct impl Error) Consistent ✅
Java Generated RPC types updated; no handwritten SessionFs provider API (expected) Generated SessionFsSqliteTransactionError Consistent ✅

Design Observations (No Action Required)

  • All SDKs correctly model the transaction provider as optional (opt-in capability), with a graceful "not supported" fallback — consistent across all implementations.
  • Error classification enum values (fatal, busyOrLocked/busy_or_locked/BusyOrLocked/FATAL/BUSY_OR_LOCKED, postCommitAmbiguous) map to the same wire values with appropriate language casing conventions.
  • Java doesn't expose a handwritten SessionFsProvider API surface — this is consistent with the existing Java SDK design, not a gap.

Generated by SDK Consistency Review Agent for #2140 · sonnet46 65 AIC · ⌖ 5.49 AIC · ⊞ 6.6K ·

@MackinnonBuck
MackinnonBuck requested a review from Copilot July 30, 2026 02:26

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@MackinnonBuck
MackinnonBuck added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@MackinnonBuck
MackinnonBuck added this pull request to the merge queue Jul 30, 2026
Merged via the queue into main with commit 3854a9c Jul 30, 2026
66 checks passed
@MackinnonBuck
MackinnonBuck deleted the update-copilot-1.0.76-5 branch July 30, 2026 04:32
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.

4 participants