Skip to content

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

Merged
JasonYeMSFT merged 10 commits intomainfrom
copilot/update-copilot-sdk-version
Feb 25, 2026
Merged

Update @github/copilot-sdk to ^0.1.25#1028
JasonYeMSFT merged 10 commits intomainfrom
copilot/update-copilot-sdk-version

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

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
Copilot AI requested a review from tmeschter February 25, 2026 00:53
kvenkatrajan
kvenkatrajan previously approved these changes Feb 25, 2026
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.
Update runConversations to also look for the bundled Copilot CLI.
JasonYeMSFT
JasonYeMSFT previously approved these changes Feb 25, 2026
@JasonYeMSFT JasonYeMSFT merged commit 32b4b7e into main Feb 25, 2026
11 checks passed
@JasonYeMSFT JasonYeMSFT deleted the copilot/update-copilot-sdk-version branch February 25, 2026 22:59
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.
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