Skip to content

Update @github/copilot to 1.0.57-4#1522

Merged
stephentoub merged 6 commits into
mainfrom
update-copilot-1.0.57-4
May 31, 2026
Merged

Update @github/copilot to 1.0.57-4#1522
stephentoub merged 6 commits into
mainfrom
update-copilot-1.0.57-4

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.57-4.

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
@stephentoub stephentoub marked this pull request as ready for review May 31, 2026 16:55
@stephentoub stephentoub requested a review from a team as a code owner May 31, 2026 16:55
Copilot AI review requested due to automatic review settings May 31, 2026 16:55
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 from 1.0.57-3 to 1.0.57-4, with regenerated SDK types reflecting an upstream doc/schema tweak to ModelSwitchToRequest.contextTier.

Changes:

  • Bump @github/copilot to ^1.0.57-4 in nodejs/package.json and test/harness/package.json (plus lockfiles).
  • Regenerated ModelSwitchToRequest.contextTier docs across Node/Python/Go/Rust/.NET, with the Node type narrowing from ContextTier | null to ContextTier.
Show a summary per file
File Description
nodejs/package.json Bumps @github/copilot to ^1.0.57-4.
nodejs/package-lock.json Lockfile refresh for the new version and platform packages.
nodejs/samples/package-lock.json Transitive lockfile refresh for samples.
test/harness/package.json Bumps test-harness @github/copilot to ^1.0.57-4.
test/harness/package-lock.json Lockfile refresh for the harness.
nodejs/src/generated/rpc.ts Regenerated ModelSwitchToRequest; contextTier is no longer nullable and doc removed.
python/copilot/generated/rpc.py Updated context_tier docstring.
go/rpc/zrpc.go Updated ContextTier field doc comment.
rust/src/generated/api_types.rs Updated context_tier doc comment.
dotnet/src/Generated/Rpc.cs Updated ContextTier summary on type and RPC wrapper.

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

@stephentoub stephentoub enabled auto-merge May 31, 2026 17:18
@stephentoub stephentoub disabled auto-merge May 31, 2026 17:18
@stephentoub stephentoub marked this pull request as draft May 31, 2026 17:18
@stephentoub stephentoub marked this pull request as ready for review May 31, 2026 17:19
@github-actions github-actions Bot mentioned this pull request May 31, 2026
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor Author

Cross-SDK Consistency Review

The contextTier support added to setModel/SetModel is consistently implemented across Node.js, Python, Go, .NET, and Rust — great work! 🎉

However, the Java SDK appears to be missing this update:

Java gaps found

  1. SessionModelSwitchToParams.java (generated) — missing contextTier field:

    // java/src/generated/java/com/github/copilot/generated/rpc/SessionModelSwitchToParams.java
    public record SessionModelSwitchToParams(
        String sessionId,
        String modelId,
        String reasoningEffort,
        ReasoningSummary reasoningSummary,
        ModelCapabilitiesOverride modelCapabilities
        // missing: ContextTier contextTier
    ) {}
  2. CopilotSession.setModel() — no contextTier parameter or SetModelOptions equivalent. The most complete overload today is:

    setModel(String model, String reasoningEffort, String reasoningSummary, ModelCapabilitiesOverride modelCapabilities)

    Compared to other SDKs that now support:

    • .NET: SetModelAsync(string model, SetModelOptions options) where SetModelOptions includes ContextTier
    • Rust: SetModelOptions::with_context_tier(ContextTier)
    • Node.js: setModel(model, { contextTier })
    • Python: set_model(model, context_tier=...)
    • Go: SetModelOptions{ ContextTier: &tier }

Suggested next steps

  • Re-run the Java codegen (cd java && mvn generate-sources -Pcodegen) once the schema update is picked up — this should regenerate SessionModelSwitchToParams with the contextTier field.
  • Add a setModel overload (or a SetModelOptions class) in CopilotSession.java that exposes contextTier, following the same pattern as the recently added reasoningSummary parameter.

This is a consistency gap, not a blocker for this PR — but it would be good to track as a follow-up to keep the Java SDK in parity with the others.

Generated by SDK Consistency Review Agent for issue #1522 · ● 7.2M ·

@stephentoub stephentoub added this pull request to the merge queue May 31, 2026
Merged via the queue into main with commit 4018d92 May 31, 2026
46 checks passed
@stephentoub stephentoub deleted the update-copilot-1.0.57-4 branch May 31, 2026 17:58
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.

3 participants