Update @github/copilot-sdk to ^0.1.25 - #1028
Merged
JasonYeMSFT (JasonYeMSFT) merged 10 commits intoFeb 25, 2026
Merged
Conversation
Co-authored-by: tmeschter <10506730+tmeschter@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update version of @github/copilot-sdk to 0.1.125
Update @github/copilot-sdk to 0.1.125
Feb 25, 2026
kvenkatrajan
previously approved these changes
Feb 25, 2026
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
kvenkatrajan
previously approved these changes
Feb 25, 2026
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
Copilot
AI
changed the title
Update @github/copilot-sdk to 0.1.125
Update @github/copilot-sdk to ^0.1.25
Feb 25, 2026
…ge.json Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
kvenkatrajan
previously approved these changes
Feb 25, 2026
The Copilot SDK package uses `const sdkUrl = import.meta.resolve("@github/copilot/sdk");` to find its location on disk, and from there finds the location of the Copilot CLI from the @github/copilot package (or really, a .js file that wraps the CLI, but that isn't relevant).
The `import.meta.resolve` just does not work when running test code through Jest, so the test run quickly dies with an error like:
```
● azure-deploy_ - Integration Tests › static-web-apps-deploy › creates whiteboard application
TypeError: (intermediate value).resolve is not a function
368 | }
369 |
> 370 | const client = new CopilotClient({
| ^
371 | logLevel: process.env.DEBUG ? "all" : "error",
372 | cwd: testWorkspace,
373 | cliArgs: cliArgs,
at getBundledCliPath (node_modules/@github/copilot-sdk/dist/client.js:31:30)
at new CopilotClient (node_modules/@github/copilot-sdk/dist/client.js:105:35)
at Object.run (utils/agent-runner.ts:370:22)
at async Object.<anonymous> (azure-deploy/integration.test.ts:104:29)
```
Here we hack around this by finding the path to the CLI ourselves and passing it in via the `cliPath` property.
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Tom Meschter (tmeschter)
temporarily deployed
to
cideploytest
February 25, 2026 22:27 — with
GitHub Actions
Inactive
Update runConversations to also look for the bundled Copilot CLI.
JasonYeMSFT (JasonYeMSFT)
previously approved these changes
Feb 25, 2026
JasonYeMSFT (JasonYeMSFT)
approved these changes
Feb 25, 2026
Tom Meschter (tmeschter)
approved these changes
Feb 25, 2026
JasonYeMSFT (JasonYeMSFT)
deleted the
copilot/update-copilot-sdk-version
branch
February 25, 2026 22:59
Tom Meschter (tmeschter)
added a commit
to tmeschter/GitHub-Copilot-for-Azure
that referenced
this pull request
Feb 26, 2026
PR microsoft#1028 updated the Copilot SDK package we use for testing to at least version 0.1.25. This brought along a [breaking change](github/copilot-sdk#509) where all tool operations are denied by default. Since our integration tests needs to run in a fully automated manner, this commit adds `onPermissionRequest: approveAll` to restore the previous behavior.
kvenkatrajan
pushed a commit
that referenced
this pull request
Feb 26, 2026
PR #1028 updated the Copilot SDK package we use for testing to at least version 0.1.25. This brought along a [breaking change](github/copilot-sdk#509) where all tool operations are denied by default. Since our integration tests needs to run in a fully automated manner, this commit adds `onPermissionRequest: approveAll` to restore the previous behavior.
This was referenced Feb 27, 2026
Barbara 4bes (Ba4bes)
pushed a commit
to Ba4bes/GitHub-Copilot-for-Azure
that referenced
this pull request
Apr 24, 2026
* Initial plan
* chore: update @github/copilot-sdk to 0.1.125
Co-authored-by: tmeschter <10506730+tmeschter@users.noreply.github.com>
* chore: use ^0.1.125 for @github/copilot-sdk
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
* chore: update @github/copilot-sdk to ^0.1.25 in tests/package.json
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
* chore: update @github/copilot-sdk to ^0.1.25 in copilot-sdk-app-package.json
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
* chore: override @github/copilot to 0.0.411
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
* chore: remove @github/copilot overrides from package.json
Co-authored-by: tmeschter <10506730+tmeschter@users.noreply.github.com>
* Set the `cliPath` in the SDK
The Copilot SDK package uses `const sdkUrl = import.meta.resolve("@github/copilot/sdk");` to find its location on disk, and from there finds the location of the Copilot CLI from the @github/copilot package (or really, a .js file that wraps the CLI, but that isn't relevant).
The `import.meta.resolve` just does not work when running test code through Jest, so the test run quickly dies with an error like:
```
● azure-deploy_ - Integration Tests › static-web-apps-deploy › creates whiteboard application
TypeError: (intermediate value).resolve is not a function
368 | }
369 |
> 370 | const client = new CopilotClient({
| ^
371 | logLevel: process.env.DEBUG ? "all" : "error",
372 | cwd: testWorkspace,
373 | cliArgs: cliArgs,
at getBundledCliPath (node_modules/@github/copilot-sdk/dist/client.js:31:30)
at new CopilotClient (node_modules/@github/copilot-sdk/dist/client.js:105:35)
at Object.run (utils/agent-runner.ts:370:22)
at async Object.<anonymous> (azure-deploy/integration.test.ts:104:29)
```
Here we hack around this by finding the path to the CLI ourselves and passing it in via the `cliPath` property.
* Update `runConversations`
Update runConversations to also look for the bundled Copilot CLI.
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tmeschter <10506730+tmeschter@users.noreply.github.com>
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
Co-authored-by: Tom Meschter (from Dev Box) <tomescht@microsoft.com>
Barbara 4bes (Ba4bes)
pushed a commit
to Ba4bes/GitHub-Copilot-for-Azure
that referenced
this pull request
Apr 24, 2026
PR microsoft#1028 updated the Copilot SDK package we use for testing to at least version 0.1.25. This brought along a [breaking change](github/copilot-sdk#509) where all tool operations are denied by default. Since our integration tests needs to run in a fully automated manner, this commit adds `onPermissionRequest: approveAll` to restore the previous behavior.
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.
Updates the
@github/copilot-sdkpackage version in the test suite.Also update the agent-runner.ts to explicitly pass
cliPathto work around these issues in Jest and Copilot SDK:import.meta.resolvein ESM context jestjs/jest#15330Note that we could also move to Jest 30.x but that is a larger change.
Changes Made
tests/package.json: Updated@github/copilot-sdkto^0.1.25tests/azure-hosted-copilot-sdk/resources/copilot-sdk-app-package.json: Updated@github/copilot-sdkto^0.1.25tests/package-lock.json: Updated to reflect the new dependency versionsOriginal prompt
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.