Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
There was a problem hiding this comment.
Pull request overview
Automated dependency bump of @github/copilot to 1.0.32-1 across the monorepo, with regenerated protocol/SDK bindings to match the updated schemas.
Changes:
- Updated
@github/copilotversion in Node packages (SDK, harness, samples) and lockfiles. - Regenerated Node/Go/Python/.NET generated protocol and session-event types from the updated schemas.
- Added newly generated protocol surface (e.g.,
session.instructions.getSources) and updated generated models/events.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps @github/copilot devDependency for the test harness. |
| test/harness/package-lock.json | Updates resolved @github/copilot artifacts for harness. |
| nodejs/package.json | Bumps @github/copilot dependency for Node SDK. |
| nodejs/package-lock.json | Updates resolved @github/copilot artifacts for Node SDK. |
| nodejs/samples/package-lock.json | Updates resolved @github/copilot artifacts for samples. |
| nodejs/src/generated/session-events.ts | Regenerated session-event union/types; adds agentId and refactors working-directory context types. |
| nodejs/src/generated/rpc.ts | Regenerated RPC schema bindings; adds instructions API and refactors various schema types. |
| python/copilot/generated/session_events.py | Regenerated Python session-event dataclasses/enums for updated schema. |
| go/rpc/generated_rpc.go | Regenerated Go RPC bindings; adds instructions API and refactors schema types. |
| go/generated_session_events.go | Regenerated Go session-event structs/enums for updated schema. |
| dotnet/src/Generated/SessionEvents.cs | Regenerated .NET session-event models (context refactor, deprecations). |
| dotnet/src/Generated/Rpc.cs | Regenerated .NET RPC models/APIs (instructions API, agent info refactor). |
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: 4
|
@copilot add a message to the obsolete attributes emitted by the C# generator about the types being deprecated and being removed in a future version |
Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/35a08e5f-ea8a-4494-9e24-084aad8ec639 Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/35a08e5f-ea8a-4494-9e24-084aad8ec639 Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Done in 793eb03. All Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
- Add shared nullable-required schema normalization in codegen utils (converts required \ properties with null descriptions to anyOf pattern) - Add Python forward-reference topological sort in codegen - Update Go type alias for renamed PurpleModelCapabilitiesOverrideLimitsVision - Add Go type conversion helper for duplicate quicktype-generated types - Add Node.js model capabilities normalization (backfill supports/limits) - Regenerate all language bindings with fixes applied Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use 'any' cast instead of 'Record<string, unknown>' to avoid TS2352 errors with strict interface types. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Import ModelCapabilitiesClass (used by generated ModelSwitchToRequest) instead of ModelCapabilitiesOverride which is now a separate duplicate type. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Client 2's event handler receives events asynchronously, so checking immediately after SendAndWait may miss them. Replace instant checks with polling (waitForEventsByType) that retries for up to 5 seconds. Applied to both 'approves' and 'rejects' subtests for consistency. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Automated update of
@github/copilotto version1.0.32-1.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Next steps
When ready, click Ready for review to trigger CI checks.