chore: bump @github/copilot-sdk to 1.0.9-preview.2 (CLI 1.0.77) - #328553
chore: bump @github/copilot-sdk to 1.0.9-preview.2 (CLI 1.0.77)#328553vs-code-engineering[bot] wants to merge 5 commits into
Conversation
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 Copilot-related dependencies to newer prerelease/preview versions across the repo.
Changes:
- Bumped
@github/copilotfrom^1.0.77to^1.0.78-2 - Bumped
@github/copilot-sdkfrom^1.0.9-preview.1to^1.0.9-preview.2
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| remote/package.json | Updates Copilot dependency versions used by the remote package. |
| package.json | Updates Copilot dependency versions used at the root/workspace level. |
Files not reviewed (1)
- remote/package-lock.json: Generated file
| "@github/copilot": "^1.0.78-2", | ||
| "@github/copilot-sdk": "^1.0.9-preview.2", |
| "@github/copilot": "^1.0.78-2", | ||
| "@github/copilot-sdk": "^1.0.9-preview.2", |
|
🏗️ Product build queued for cycle 1: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=460728 — validating the PR branch… |
|
❌ Product build against the PR branch — cycle 1:
|
|
Commit pushed:
|
|
🏗️ Product build queued for cycle 2: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=460730 — validating the PR branch… |
|
Base:
|
|
✅ Product build against the PR branch — cycle 2:
|
SDK 1.0.9-preview.2 removed managedApprovalRequired from PermissionPromptRequestPath (compile TS2353); the path prompt variant never carried it and production only reads promptRequest.autoApproval, so the redundant test field is dropped while the managed-approval semantics stay driven by permissionRequest.managedApprovalRequired. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1fff777 to
c311585
Compare
| accessKind: 'read', | ||
| paths: ['/workspace/src/file.ts'], | ||
| toolCallId: 'tc-managed', | ||
| managedApprovalRequired: true, |
There was a problem hiding this comment.
@justschen Can you look over if this is safe to remove?
It seems like SDK type for this is PermissionPromptRequestPath ?
There was a problem hiding this comment.
this is for enteprise managed approvals which is different, we probably still need this?
…elds Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cf99a5e2-4a51-4763-8aa8-0d3bbb359b33
1.0.78-2 is a prerelease outside the SDK range ^1.0.76-5, which forced a nested 1.0.77 copy. Pin the latest stable CLI instead so npm hoists one tree. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cf99a5e2-4a51-4763-8aa8-0d3bbb359b33
Version pairing noteBot originally paired SDK
So this PR is:
If we want |
SDK ShutdownType is "routine" | "error"; "normal" fails tsgo. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cf99a5e2-4a51-4763-8aa8-0d3bbb359b33
@github/copilot-sdkfrom1.0.9-preview.1to1.0.9-preview.2in root andremote/.@github/copiloton1.0.77(exact pin, no caret) so npm hoists a single CLI tree.managedApprovalRequiredfrom the pathpromptRequestfixture; that field is not onPermissionPromptRequestPath.session.shutdowntest fixture for requiredShutdownDatafields.Product build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=461048