Update @github/copilot to 1.0.46#1270
Merged
Merged
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
stephentoub
approved these changes
May 12, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s Node-based Copilot CLI dependency (@github/copilot) to 1.0.46 and regenerates the language SDK RPC/model bindings to match the updated schema.
Changes:
- Bumped
@github/copilotfrom^1.0.45to^1.0.46across Node packages (SDK + harness + samples lockfile). - Regenerated RPC/model types in Rust, Python, Go, .NET, and Node, including new model metadata fields (
modelPickerCategory,modelPickerPriceCategory). - Updated lockfiles and formatted regenerated outputs.
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps harness devDependency on @github/copilot to 1.0.46. |
| test/harness/package-lock.json | Updates resolved @github/copilot and platform-specific optional deps to 1.0.46. |
| nodejs/package.json | Bumps Node SDK dependency on @github/copilot to 1.0.46. |
| nodejs/package-lock.json | Updates resolved @github/copilot and optional deps to 1.0.46. |
| nodejs/samples/package-lock.json | Updates transitive lock entry to align with 1.0.46. |
| nodejs/src/generated/rpc.ts | Adds new generated ModelPickerCategory / ModelPickerPriceCategory types and model fields. |
| python/copilot/generated/rpc.py | Adds new generated enums and model fields for picker category/price category. |
| go/rpc/zrpc.go | Adds new generated model fields and corresponding string-typed enums/constants. |
| rust/src/generated/api_types.rs | Adds new generated model fields and new enums with forward-compat Unknown variants. |
| dotnet/src/Generated/Rpc.cs | Adds new generated model fields and string-backed enum structs for forward compatibility. |
Copilot's findings
Files not reviewed (4)
- go/rpc/zrpc.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/10 changed files
- Comments generated: 0
The 1.0.46 update added `model_picker_category` and `model_picker_price_category` fields to the `Model` type. Update the Rust test fixtures in lib.rs and tests/e2e/client.rs to initialize these new fields so clippy passes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Cross-SDK Consistency Review ✅This PR is an automated schema-driven update (
These are exposed as optional fields on the
All SDKs use language-appropriate naming conventions and expose the same enum values. No consistency issues found.
|
stephentoub
pushed a commit
to qmuntal/copilot-sdk
that referenced
this pull request
May 14, 2026
* Derive Default on generated Rust types When the @github/copilot schema gains an optional field on a wire type, every Rust test fixture that constructs that type by struct literal breaks until each one is hand-edited (cf. 70eb60e in github#1270). Test fixtures shouldn't have to acknowledge new wire fields they don't care about. Teach the Rust codegen to derive `Default` on every generated struct and string-style enum (with `#[default]` on the `Unknown` catch-all where one exists), and propagate non-default-ness across structs that have a required field of an untagged-enum type (which has no obvious default variant). Also derive `Default` on `RequestId` for parity with `SessionId`. With this in place, the three internal Model fixtures can spell themselves with `..Default::default()` and survive future additive schema changes without manual intervention. Production code stays explicit: the rust-coding skill already forbids `..Default::default()` outside tests, and that rule is unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Drop redundant model_picker fields after merge main added explicit None initializers for the new model_picker_* fields; the merge layered those on top of our ..Default::default() making them redundant. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Regenerate after merging main Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.46.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Next steps
When ready, click Ready for review to trigger CI checks.