Skip to content

Update @github/copilot-sdk to ^0.1.25 - #1028

Merged
JasonYeMSFT (JasonYeMSFT) merged 10 commits into
mainfrom
copilot/update-copilot-sdk-version
Feb 25, 2026
Merged

Update @github/copilot-sdk to ^0.1.25#1028
JasonYeMSFT (JasonYeMSFT) merged 10 commits into
mainfrom
copilot/update-copilot-sdk-version

Conversation

Copilot AI commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

Updates the @github/copilot-sdk package version in the test suite.

Also update the agent-runner.ts to explicitly pass cliPath to work around these issues in Jest and Copilot SDK:

Note that we could also move to Jest 30.x but that is a larger change.

Changes Made

  • tests/package.json: Updated @github/copilot-sdk to ^0.1.25
  • tests/azure-hosted-copilot-sdk/resources/copilot-sdk-app-package.json: Updated @github/copilot-sdk to ^0.1.25
  • tests/package-lock.json: Updated to reflect the new dependency versions
Original prompt

This section details on the original issue you should resolve

<issue_title>Update @github/copilot-sdk</issue_title>
<issue_description>Update the version of the @github/copilot-sdk package we're using to 0.1.125.</issue_description>

Comments on the Issue (you are Copilot in this section)


🔒 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.

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
kvenkatrajan previously approved these changes Feb 25, 2026
Comment thread tests/azure-hosted-copilot-sdk/resources/copilot-sdk-app-package.json Outdated
Comment thread tests/package.json Outdated
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
kvenkatrajan
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
Copilot AI requested a review from kvenkatrajan February 25, 2026 01:56
…ge.json

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
kvenkatrajan
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.
Comment thread tests/utils/agent-runner.ts Outdated
Update runConversations to also look for the bundled Copilot CLI.
@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) merged commit 32b4b7e into main Feb 25, 2026
11 checks passed
@JasonYeMSFT
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.
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.
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.

Update @github/copilot-sdk

6 participants