chore: bump @github/copilot to 1.0.84-4 - #335518
Conversation
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
There was a problem hiding this comment.
🟡 Changes recommended
Required Linux x64 and Windows x64 platform packages are unpublished, leaving supported installations without a Copilot executable.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Bumps @github/copilot from 1.0.84-3 to 1.0.84-4 across local and remote dependencies.
Changes:
- Updates dependency manifests.
- Regenerates lockfiles and platform package metadata.
File summaries
| File | Description |
|---|---|
package.json |
Updates the root dependency. |
package-lock.json |
Locks the updated root packages. |
remote/package.json |
Updates the remote dependency. |
remote/package-lock.json |
Locks the updated remote packages. |
Review details
Files not reviewed (1)
- remote/package-lock.json: Generated file
- Files reviewed: 2/4 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
🤖 sdk-vscode-bump-fix — blocked (publish-race lockfile defect)Stage: Completed automated work
Remaining minimal human taskRegenerate the two lockfiles now that the packages exist, then push to this branch: npm install --package-lock-only --ignore-scripts --no-audit --no-fund
npm install --package-lock-only --ignore-scripts --no-audit --no-fund --prefix remoteThis re-adds the 4 missing entries ( Why this could not be automated here: the fix needs npm-registry access to obtain the Warning Firewall blocked 2 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "github.com"
- "registry.npmjs.org"See Network Configuration for more information.
|
(Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
roblourens
left a comment
There was a problem hiding this comment.
Lockfiles now include every published Copilot platform package and pass npm lockfile validation. (Written by Copilot)
The merge-base changed after approval.
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: Ulugbek Abdullaev (@ulugbekna)Matched files:
Sandeep Somavarapu (@sandy081)Matched files:
Ladislau Szomoru (@lszomoru)Matched files:
Kyle Cutler (@kycutler)Matched files:
Joaquín Ruales (@jruales)Matched files:
|
Pull request was converted to draft
The platform packages were not yet published when the automated bump generated the lockfiles, causing npm ci to fail across CI jobs. (Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bb96ec2 to
bf801e5
Compare
roblourens
left a comment
There was a problem hiding this comment.
Verified the corrected PR contains only the Copilot dependency bump and regenerated platform lockfile entries. (Written by Copilot)
Automated signed Copilot dependency bump.
@github/copilot:1.0.84-3→1.0.84-4main@45744b3a87b664a8b16a45a5078493f9b681a1dcDraft until validation passes. The
sdk-vscode-bump-fixagent first iterates on Linux PR checks (up to 10 cycles), then on the authoritative full product build (up to 5 cycles).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/copilot1.0.84-3 → 1.0.84-4No published path changes.
@github/copilot-darwin-x641.0.84-3 → 1.0.84-4No published path changes.
@github/copilot-linux-x641.0.84-3 → 1.0.84-4_Could not inspect this package: Command failed: npm pack @github/copilot-linux-x64@1.0.84-4 --dry-run --json --ignore-scripts
npm warn Unknown project config "disturl". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "target". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "ms_build_id". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "runtime". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "build_from_source". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "timeout". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm error code ETARGET
npm error notarget No matching version found for @github/copilot-linux-x64@1.0.84-4.
npm error notarget In most cases you or one of your dependencies are requesting a package version that doesn't exist.
npm error A complete log of this run can be found in: /home/runner/.npm/_logs/2026-09-10T17_26_12_108Z-debug-0.log
_
@github/copilot-win32-x641.0.84-3 → 1.0.84-4_Could not inspect this package: Command failed: npm pack @github/copilot-win32-x64@1.0.84-4 --dry-run --json --ignore-scripts
npm warn Unknown project config "disturl". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "target". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "ms_build_id". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "runtime". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "build_from_source". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm warn Unknown project config "timeout". This will stop working in the next major version of npm. See
npm help npmrcfor supported config options.npm error code ETARGET
npm error notarget No matching version found for @github/copilot-win32-x64@1.0.84-4.
npm error notarget In most cases you or one of your dependencies are requesting a package version that doesn't exist.
npm error A complete log of this run can be found in: /home/runner/.npm/_logs/2026-09-10T17_26_15_243Z-debug-0.log
_
sdk-vscode-bump-fix status
Stage:⚠️ two decorrelated flaky PR-check jobs; no integration change warranted
pr-checkscycle 1 · SDK1.0.13· CLI1.0.84-4· State:Prior blocker resolved
The previous cycle's blocker — the publish-race lockfile omission (4 of 8
@github/copilotplatform sub-packages missing frompackage-lock.json/remote/package-lock.json) — has been fixed on the branch by commitbf801e5("chore: restore Copilot platform packages to lockfiles"). Both lockfiles now list all 8 platform entries (darwin/linux/linuxmusl/win32×arm64/x64), matching the published@github/copilot@1.0.84-4optionalDependencies. As a result everynpm ci-dependent job now passes: Copilot - Test (Linux/Windows), Copilot - Check Test Cache, Compile & Hygiene, and the native-binary inventory test are all ✅.Remaining failures are cross-platform flake, not a contract change
Only 2 of ~26 checks fail, and each is contradicted by the same test type passing on other platforms:
A genuine SDK/CLI native-payload or runtime regression would fail deterministically for that binary's tests across platforms (the shared Agent Host spawns the same bumped
@github/copilotbinary everywhere). Single-job, single-platform failures whose sibling-platform equivalents pass are the signature of ordinary VS Code CI flake, not a bump-owned failure.SDK+CLI correlation / compatibility
1.0.84-3→1.0.84-4; SDK stays1.0.13. The branch code diff vsmainis lockfiles only — all integration surfaces (agent.tsmanaged-settings snapshot,mock-llm-server.tsnon-streaming Responses,build/.moduleignoreSDK platform-package strips) already landed inmainand require no change here.1.0.14-canary.*/ CLI1.0.84-canary.*.unsigned— a different CLI from this bump's signed1.0.84-4; treated as non-applicable hypotheses.sdk-compatibility.md: SDK1.0.13is the pinned/published SDK; no SDK bump is needed for this CLI-only PR. The upstream SDK dependency PR Update Copilot CLI to 1.0.84-4 github/copilot-sdk#2609 is merged.Work completed / not done
bf801e5) and allnpm cijobs are green.mainis lockfiles only;extensions/copilot(maintenance-mode, frozen) left untouched.github.com/ADO are firewall-blocked from this sandbox), so the flake classification rests on the cross-platform check matrix above.Smallest human/next task
Re-run the two failed jobs (Linux / Electron, macOS / Electron-Smoke). Given every sibling-platform equivalent is green, a plain re-run is expected to clear them and turn the PR-check stage green, advancing to the product build.
Warning
Firewall blocked 2 domains
The following domains were blocked by the firewall during workflow execution:
api.github.comgithub.com[!TIP]
api.github.comis blocked because GitHub API access uses the built-in GitHub tools by default. Instead of addingapi.github.comtonetwork.allowed, usetools.github.mode: gh-proxyfor direct pre-authenticated GitHub CLI access without requiring network access toapi.github.com:See GitHub Tools for more information on
gh-proxymode.To allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.