chore: bump @github/copilot-sdk to 1.0.10-preview.0 and @github/copilot to 1.0.79-7 - #329631
chore: bump @github/copilot-sdk to 1.0.10-preview.0 and @github/copilot to 1.0.79-7#329631vs-code-engineering[bot] wants to merge 2 commits into
Conversation
vs-code-engineering
Bot
added
the
copilot-sdk-auto-upgrade
vs-code-engineering
Bot
added
the
copilot-sdk-auto-upgrade
There was a problem hiding this comment.
Pull request overview
Note
Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.
Updates GitHub Copilot dependencies across the root and remote/ packages to align on newer preview builds.
Changes:
- Bumps
@github/copilotto1.0.79-7 - Bumps
@github/copilot-sdkto1.0.10-preview.0and pins it (removes^range)
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Updates Copilot/Copilot SDK dependency versions in the root package. |
| remote/package.json | Mirrors the same Copilot/Copilot SDK version updates for the remote package. |
Files not reviewed (1)
- remote/package-lock.json: Generated file
| "@github/copilot": "1.0.79-7", | ||
| "@github/copilot-sdk": "1.0.10-preview.0", |
| "@github/copilot": "1.0.79-7", | ||
| "@github/copilot-sdk": "1.0.10-preview.0", |
|
🏗️ Product build queued for cycle 1: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=462510 — validating the PR branch… |
|
❌ Product build against the PR branch — cycle 1:
|
The @github/copilot-sdk 1.0.10-preview.0 bump widened ManagedSettingsResolvedData.source to add 'client' and 'mixed', and added the optional clientManaged field. Mirror those in IAgentHostManagedSettingsSnapshot so getManagedSettingsDiagnostics() type-checks. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Commit pushed:
|
|
looking to adopt the change in #328082 |
Automated signed Copilot dependency bump.
@github/copilot-sdk:^1.0.9-preview.3→1.0.10-preview.0@github/copilot:1.0.79-2→1.0.79-7main@55876d4576bf644ae7b69909e3f78bd0dbae83c6Draft until validation passes. The
sdk-vscode-bump-fixagent will prime packaging changes from the canary insights and iterate on failures.Each product build triggered for this PR is posted as a comment below with a link.
Published package layout changes
Copilot published package layout diff
Best-effort path comparison from
npm pack --dry-run; CLI platform paths use representative x64 packages for macOS, Linux, and Windows. Content-only and architecture-specific changes are left to the product build. This is not an availability gate.@github/copilot-sdk1.0.9-preview.3 → 1.0.10-preview.0No published path changes.
@github/copilot1.0.79-2 → 1.0.79-7No published path changes.
@github/copilot-darwin-x641.0.79-2 → 1.0.79-7No published path changes.
@github/copilot-linux-x641.0.79-2 → 1.0.79-7No published path changes.
@github/copilot-win32-x641.0.79-2 → 1.0.79-7No published path changes.
🔧 Integration fix — cycle 1
Failure: Product-build Compile failed on all platforms/arches with
TS2322atsrc/vs/platform/agentHost/node/copilot/copilotAgent.ts(1040,3)— the object returned bygetManagedSettingsDiagnostics()was not assignable toIAgentHostManagedSettingsSnapshot.Root cause (intentional upstream contract change):
@github/copilot-sdk@1.0.10-preview.0widenedManagedSettingsResolvedData.sourcefrom"server" | "device" | "none"toManagedSettingsResolvedSource = "server" | "device" | "client" | "mixed" | "none"and added the optionalclientManaged?: booleanfield. VS Codes snapshot interface still declared the narrow union and lackedclientManaged.SDK+CLI correlation: The change ships in this exact SDK (
1.0.10-preview.0) — verified in both the public SDK d.ts (dist/generated/session-events.d.ts) and the runtime SDK d.ts bundled in@github/copilot(sdk/index.d.ts). Pure type migration, not a native-payload/koffi/glibc issue, so the CLI (1.0.79-7) match is not load-bearing here.Fix: Widened
IAgentHostManagedSettingsSnapshot.sourceto the new 5-value union and addedreadonly clientManaged?: booleaninsrc/vs/platform/agentHost/common/agentService.ts. The snapshot is only serialized to a JSON diagnostics block downstream (developerActions.ts), so widening is safe for consumers.Validation: Interface now exactly mirrors the SDK
ManagedSettingsResolvedDatacontract field-for-field. Localtsccould not be run (checkout is write-protected to the agent and has no localnode_modules), so correctness was confirmed by direct comparison against the installed SDK type definitions.State: Fix pushed to branch — awaiting re-dispatched product build.