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 mono-repo SDKs to @github/copilot@1.0.22 and refreshes generated RPC/session-event bindings so the Node.js, Python, Go, and .NET SDKs track the updated CLI JSON-RPC/schema surface area.
Changes:
- Bumped
@github/copilotdependency to^1.0.22in Node.js packages (SDK, samples) and the shared test harness (plus lockfiles). - Regenerated SDK bindings to reflect schema updates (notably
mcp.discover,session.usage.getMetrics, and expanded memory permission request fields). - Reformatted generated output.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps devDependency on @github/copilot to ^1.0.22. |
| test/harness/package-lock.json | Lockfile refresh for the harness dependency bump. |
| nodejs/package.json | Bumps SDK dependency on @github/copilot to ^1.0.22. |
| nodejs/package-lock.json | Lockfile refresh for the Node.js SDK dependency bump. |
| nodejs/samples/package-lock.json | Lockfile refresh for Node.js samples dependency bump. |
| nodejs/src/generated/session-events.ts | Updates generated event payload typings (memory permission request fields). |
| nodejs/src/generated/rpc.ts | Adds generated RPC types/methods for mcp.discover and session.usage.getMetrics, plus doc refinements. |
| python/copilot/generated/session_events.py | Updates generated session-event models (enum rename + memory permission request fields). |
| python/copilot/generated/rpc.py | Adds generated RPC types/methods for mcp.discover and session.usage.getMetrics, plus doc refinements. |
| go/rpc/generated_rpc.go | Adds generated RPC types/methods for mcp.discover and session.usage.getMetrics, plus some generated type renames. |
| go/generated_session_events.go | Updates generated session-event structs for expanded memory permission request fields. |
| dotnet/src/Generated/SessionEvents.cs | Updates generated session-event models (memory permission request fields + enums). |
| dotnet/src/Generated/Rpc.cs | Adds generated RPC types/methods for mcp.discover and session.usage.getMetrics (but see comment about Server type collision). |
Copilot's findings
Files not reviewed (3)
- 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: 4/13 changed files
- Comments generated: 1
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the mono-repo’s SDK bindings and generated types to align with @github/copilot@1.0.22, reflecting newly added/changed RPC methods and session event schema updates across Node.js, Python, Go, and .NET.
Changes:
- Bumped
@github/copilotdependency to^1.0.22in Node.js SDK + test harness (and refreshed lockfiles/samples lockfile). - Regenerated SDK protocol bindings (RPC + session event types) across languages, including new APIs like
mcp.discoverandsession.usage.getMetrics. - Updated C# codegen to better avoid RPC class-name collisions for array item object types.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps harness devDependency on @github/copilot to ^1.0.22. |
| test/harness/package-lock.json | Lockfile refresh for the harness dependency bump. |
| nodejs/package.json | Bumps Node.js SDK dependency on @github/copilot to ^1.0.22. |
| nodejs/package-lock.json | Lockfile refresh for Node.js dependency bump. |
| nodejs/samples/package-lock.json | Samples lockfile refresh for the dependency bump. |
| scripts/codegen/csharp.ts | Improves C# RPC type generation collision handling (stable schema key + name selection). |
| nodejs/src/generated/rpc.ts | Regenerated RPC types incl. mcp.discover, session.usage.getMetrics, richer docs. |
| nodejs/src/generated/session-events.ts | Regenerated session event types; updates memory permission request payload fields. |
| python/copilot/generated/rpc.py | Regenerated Python RPC types incl. mcp.discover, usage.get_metrics, enum rename for extensions source. |
| python/copilot/generated/session_events.py | Regenerated Python session events incl. memory permission fields; renames Action → DataAction. |
| go/rpc/generated_rpc.go | Regenerated Go RPC types incl. mcp.discover, UsageApi.GetMetrics, and some renamed types/enums. |
| go/generated_session_events.go | Regenerated Go session events; updates memory permission request payload fields. |
| dotnet/src/Generated/Rpc.cs | Regenerated .NET RPC types incl. mcp.discover, usage APIs, updated docs. |
| dotnet/src/Generated/SessionEvents.cs | Regenerated .NET session events; updates memory permission request payload fields + enums. |
Copilot's findings
Files not reviewed (3)
- 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: 5/14 changed files
- Comments generated: 2
There was a problem hiding this comment.
Pull request overview
This PR bumps the monorepo’s Node-based dependency on @github/copilot to 1.0.22 and propagates the corresponding schema-driven changes across the generated SDK surfaces (Node/TS, Python, Go, .NET), including newly generated RPC types/APIs and updated session event models.
Changes:
- Updated
@github/copilotto^1.0.22in the Node SDK, test harness, and sample lockfile(s). - Regenerated code from updated schemas across SDKs (RPC + session events), including new APIs like
mcp.discoverand experimentalsession.usage.getMetrics. - Updated .NET codegen to avoid silently reusing the same class name for different RPC schemas by tracking schema-to-class bindings.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps @github/copilot devDependency to ^1.0.22 for the harness. |
| test/harness/package-lock.json | Updates lockfile resolution/integrity and platform optional deps for @github/copilot@1.0.22. |
| scripts/codegen/csharp.ts | Enhances C# RPC type generation with schema-aware class-name collision handling for array item object types. |
| python/copilot/generated/session_events.py | Regenerates Python session event models to reflect updated permission request “memory operation” fields (store vs vote). |
| python/copilot/generated/rpc.py | Regenerates Python RPC models/APIs, adding mcp.discover and experimental session.usage.getMetrics types + client methods. |
| nodejs/src/generated/session-events.ts | Updates TS session event typings for memory permission requests (action/direction/reason; subject/citations optional). |
| nodejs/src/generated/rpc.ts | Updates TS RPC typings and client factories with mcp.discover and experimental usage.getMetrics. |
| nodejs/samples/package-lock.json | Updates samples lockfile to @github/copilot@^1.0.22. |
| nodejs/package.json | Bumps Node SDK dependency on @github/copilot to ^1.0.22. |
| nodejs/package-lock.json | Updates lockfile resolution/integrity and optional platform deps for @github/copilot@1.0.22. |
| go/rpc/generated_rpc.go | Regenerates Go RPC types/APIs, adding MCPDiscover* and experimental usage metrics support. |
| go/generated_session_events.go | Regenerates Go session events to include expanded memory permission request fields (action/direction/reason). |
| dotnet/src/Generated/SessionEvents.cs | Regenerates .NET session events for updated memory permission request shape + enums. |
| dotnet/src/Generated/Rpc.cs | Regenerates .NET RPC types/APIs, adding mcp.discover, experimental usage metrics, and schema-safe type naming. |
Copilot's findings
Files not reviewed (3)
- 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: 5/14 changed files
- Comments generated: 0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Cross-SDK Consistency Review ✅This is an automated codegen PR — all four SDK implementations (Node.js/TypeScript, Python, Go, .NET) are updated consistently from the same New features are present across all SDKs:
The No cross-SDK consistency issues found.
|
Automated update of
@github/copilotto version1.0.22.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)