Update @github/copilot to 1.0.51-2#1342
Merged
Merged
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the repo’s @github/copilot dependency to 1.0.51-2 and regenerates all language SDK outputs to match the updated wire schema and tooling.
Changes:
- Bumped
@github/copilotto^1.0.51-2in Node.js SDK + test harness, with corresponding lockfile updates (including newdetect-libctransitive dependency). - Regenerated Rust/Go/.NET/Python generated protocol types to reflect schema changes (notably several nano-AIU / premium-request related fields moving from integer to floating-point types).
- Reformatted generated outputs.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps @github/copilot devDependency to ^1.0.51-2. |
| test/harness/package-lock.json | Updates lockfile to @github/copilot@1.0.51-2 and adds transitive detect-libc. |
| nodejs/package.json | Bumps @github/copilot runtime dependency to ^1.0.51-2. |
| nodejs/package-lock.json | Updates lockfile to @github/copilot@1.0.51-2 and adds transitive detect-libc. |
| nodejs/samples/package-lock.json | Updates sample lockfile’s resolved @github/copilot version via workspace dependency graph. |
| rust/src/generated/session_events.rs | Regenerated session-events types; cost/request fields now use f64 in several places. |
| rust/src/generated/api_types.rs | Regenerated API types; total_nano_aiu now Option<f64> in usage metrics structures. |
| go/rpc/zsession_events.go | Regenerated session-events types; cost/request fields now use float64 in several places. |
| go/rpc/zrpc.go | Regenerated API types; total_nano_aiu now *float64 in usage metrics structures. |
| python/copilot/generated/session_events.py | Regenerated session-events types; nano-AIU and premium-request fields now parsed/typed as float. |
| python/copilot/generated/rpc.py | Regenerated API types; usage metrics nano-AIU fields now parsed/typed as float. |
| dotnet/src/Generated/SessionEvents.cs | Regenerated session-events types; nano-AIU and premium-request fields now double in several places. |
| dotnet/src/Generated/Rpc.cs | Regenerated API types; usage metrics nano-AIU fields now double?. |
Copilot's findings
Files not reviewed (5)
- go/rpc/zrpc.go: Language not supported
- go/rpc/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/13 changed files
- Comments generated: 2
The TotalNanoAiu field was changed from int to *float64 in the generated types, but the test format strings still used %d. This caused go vet to fail on all three Go CI legs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Cross-SDK Consistency Review ✅This automated codegen PR ( Summary of changes:
All 7 field instances (across
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated update of
@github/copilotto version1.0.51-2.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Next steps
When ready, click Ready for review to trigger CI checks.