Skip to content

Update @github/copilot to 1.0.53-2#1408

Merged
jmoseley merged 2 commits into
mainfrom
update-copilot-1.0.53-2
May 24, 2026
Merged

Update @github/copilot to 1.0.53-2#1408
jmoseley merged 2 commits into
mainfrom
update-copilot-1.0.53-2

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.53-2.

Changes

  • Updated @github/copilot in nodejs/package.json and test/harness/package.json
  • Re-ran all code generators (scripts/codegen)
  • Formatted generated output

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
@jmoseley jmoseley marked this pull request as ready for review May 24, 2026 06:52
@jmoseley jmoseley requested a review from a team as a code owner May 24, 2026 06:52
Copilot AI review requested due to automatic review settings May 24, 2026 06:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Automated dependency bump of @github/copilot to 1.0.53-2 and regeneration of the SDK’s schema-derived types/RPC surfaces across languages to match the updated wire schema.

Changes:

  • Updated @github/copilot dependency versions in Node.js packages (SDK, samples) and the test harness.
  • Regenerated and formatted schema-derived RPC + session event types for Rust, Python, Node.js/TS, Go, and .NET.
  • Introduced new schema surfaces related to session.canvas.* RPCs and session.canvas.* session events in generated outputs.
Show a summary per file
File Description
test/harness/package.json Bumps @github/copilot dev dependency to ^1.0.53-2 for the test harness.
test/harness/package-lock.json Lockfile update reflecting @github/copilot@1.0.53-2 and platform optional deps.
nodejs/package.json Bumps Node.js SDK dependency on @github/copilot to ^1.0.53-2.
nodejs/package-lock.json Lockfile update reflecting @github/copilot@1.0.53-2 and platform optional deps.
nodejs/samples/package-lock.json Updates sample lockfile dependency to @github/copilot@^1.0.53-2.
nodejs/src/generated/session-events.ts Regenerated session event typings; adds new canvas-related events/types.
nodejs/src/generated/rpc.ts Regenerated RPC typings; adds session.canvas.* RPC group and related types.
python/copilot/generated/session_events.py Regenerated session event models; adds new canvas-related events/types.
python/copilot/generated/rpc.py Regenerated RPC client/types; adds CanvasApi and canvas-related DTOs.
go/zsession_events.go Regenerated Go re-exports for new canvas session event symbols.
go/rpc/zsession_events.go Regenerated Go session-event data types/constants for canvas events.
go/rpc/zsession_encoding.go Updates event decoding to handle new canvas event types.
go/rpc/zrpc.go Regenerated Go RPC surface; adds CanvasApi and canvas-related request/response types.
rust/src/generated/session_events.rs Regenerated Rust session event enums/data; adds canvas event variants and payload structs.
rust/src/generated/api_types.rs Regenerated Rust API types and RPC method constants; adds canvas RPC constants/types.
rust/src/generated/rpc.rs Regenerated Rust RPC client; adds session.canvas.* sub-namespace and methods.
dotnet/src/Generated/SessionEvents.cs Regenerated .NET session event models; adds canvas event types and availability enum.
dotnet/src/Generated/Rpc.cs Regenerated .NET RPC surface; adds CanvasApi and canvas-related DTOs.

Copilot's findings

Files not reviewed (7)
  • go/rpc/zrpc.go: Language not supported
  • go/rpc/zsession_encoding.go: Language not supported
  • go/rpc/zsession_events.go: Language not supported
  • go/zsession_events.go: Language not supported
  • nodejs/package-lock.json: Language not supported
  • nodejs/samples/package-lock.json: Language not supported
  • test/harness/package-lock.json: Language not supported
  • Files reviewed: 2/18 changed files
  • Comments generated: 0

quicktype emits the freeform 'canvas_json_schema' field as 'Any', and the
post-processor was adding '= None' to every bare ': Any' annotation. That
turned the synthesized root RPC dataclass field into a defaulted argument
followed by required fields (canvas_list, ...), which made Python raise
'non-default argument follows default argument' at import time and broke
every Python SDK test plus the scenario import checks.

Scope the post-processor so RPC definition fields keep their required Any
annotation, matching the rest of the registry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jmoseley jmoseley enabled auto-merge May 24, 2026 07:16
@jmoseley jmoseley added this pull request to the merge queue May 24, 2026
@jmoseley jmoseley removed this pull request from the merge queue due to a manual request May 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor Author

SDK Consistency Review

Summary

This PR updates @github/copilot to 1.0.53-2 and regenerates code for Node.js, Python, Go, .NET, and Rust SDKs. The new generated types include Canvas-related APIs:

  • CanvasAction, DiscoveredCanvas, CanvasList, CanvasApi, OpenCanvasInstance, etc. (RPC types)
  • SessionCanvasOpenedEvent, SessionCanvasRegistryChangedEvent (session events)

⚠️ Consistency Gap: Java SDK not updated

The Java SDK has its own separate code generation mechanism (java/scripts/codegen/java.ts, triggered via mvn generate-sources -Pcodegen), and it was not run as part of this automated PR. As a result, Java is missing all the new Canvas-related types:

  • SessionCanvasOpenedEvent
  • SessionCanvasRegistryChangedEvent
  • Canvas RPC types (CanvasAction, CanvasList, CanvasApi, etc.)

Suggestion

The automated Update @github/copilot Dependency workflow should also trigger Java codegen (cd java && mvn generate-sources -Pcodegen) and commit the resulting changes to java/src/generated/java/. Until then, Java will be out of sync with other SDKs on Canvas functionality.

This is an automated cross-SDK consistency review.

Generated by SDK Consistency Review Agent for issue #1408 · ● 4.9M ·

@jmoseley jmoseley added this pull request to the merge queue May 24, 2026
Merged via the queue into main with commit 08f0521 May 24, 2026
49 of 50 checks passed
@jmoseley jmoseley deleted the update-copilot-1.0.53-2 branch May 24, 2026 07:48
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