build(deps-dev): bump the minor group in /tests with 3 updates - #2152
build(deps-dev): bump the minor group in /tests with 3 updates#2152dependabot[bot] wants to merge 1 commit into
Conversation
Bumps the minor group in /tests with 3 updates: [@github/copilot-sdk](https://github.com/github/copilot-sdk), [eslint](https://github.com/eslint/eslint) and [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint). Updates `@github/copilot-sdk` from 0.2.0 to 0.3.0 - [Release notes](https://github.com/github/copilot-sdk/releases) - [Changelog](https://github.com/github/copilot-sdk/blob/main/CHANGELOG.md) - [Commits](github/copilot-sdk@v0.2.0...v0.3.0) Updates `eslint` from 10.2.0 to 10.3.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v10.2.0...v10.3.0) Updates `typescript-eslint` from 8.58.0 to 8.59.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.59.1/packages/typescript-eslint) --- updated-dependencies: - dependency-name: "@github/copilot-sdk" dependency-version: 0.3.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: minor - dependency-name: eslint dependency-version: 10.3.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: minor - dependency-name: typescript-eslint dependency-version: 8.59.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: minor ... Signed-off-by: dependabot[bot] <support@github.com>
Jon Gallant (jongio)
left a comment
There was a problem hiding this comment.
@github/copilot-sdk 0.2.0 -> 0.3.0 introduces a breaking type change: ToolExecutionStartData no longer satisfies Record<string, unknown> (the index signature was removed). This causes 3 typecheck failures in CI:
entra-agent-id/utils.ts:73utils/evaluate.ts:373utils/regression-detectors.ts:37
All three pass a ToolExecutionStartEvent to something expecting { data: Record<string, unknown> }.
The eslint and typescript-eslint bumps look fine (standard minor patches), but this PR can't merge until the type errors are resolved. Either:
- Update the 3 call sites to match the new
ToolExecutionStartDatashape (preferred - adapt to the new API) - Add a type assertion at the call sites as a stopgap
- Pin
@github/copilot-sdkat 0.2.x until the consuming code is updated
Option 1 is the right fix since 0.3.0 is the target.
|
Upgrading to copilot-sdk 0.3.0 is blocked by this github/copilot-sdk#1156. It doesn't export the types it's supposed to export. If we upgrade, we literally lose typing support for almost all the SDK events. |
|
Looks like these dependencies are updatable in another way, so this is no longer needed. |
Bumps the minor group in /tests with 3 updates: @github/copilot-sdk, eslint and typescript-eslint.
Updates
@github/copilot-sdkfrom 0.2.0 to 0.3.0Release notes
Sourced from @github/copilot-sdk's releases.
... (truncated)
Changelog
Sourced from @github/copilot-sdk's changelog.
... (truncated)
Commits
dd2dcbcPer-session GitHub authentication for all SDK languages, plus update runtime ...b4ef955Add configurable session idle timeout option (#1093)a3e273cSessionFs structured error contract and codegen changesb1b0df5feat: add per-agent tool visibility via defaultAgent.excludedTools (#1098)922959fExpose IncludeSubAgentStreamingEvents in all four SDKs (#1108)fd0495cUpdate@github/copilotto 1.0.32 (#1107)cf5694cUpdate@github/copilotto 1.0.32-1 (#1105)48e244dClean up redundant Python codegen lambdas (#1104)dbcea81Add deprecated schema support to all four code generators (#1099)883cc02Update@github/copilotto 1.0.30 (#1096)Updates
eslintfrom 10.2.0 to 10.3.0Release notes
Sourced from eslint's releases.
... (truncated)
Commits
788920410.3.05b69b4fBuild: changelog update for 10.3.0d32235eci: use pnpm ineslint-flat-config-utilstype integration test (#20826)b6ae5cffix: handle unavailable require cache (#20812)3ffb14echore: clean up typos in comments and JSDoc (#20821)6fb3685fix: rule suggestions cause continuation in class body (#20787)22eb58achore: add missing continue-on-error to ecosystem-tests.yml (#20818)88bf002ci: bump pnpm/action-setup from 6.0.1 to 6.0.3 (#20815)379571afeat: add suggestions for no-unused-private-class-members (#20773)97c8c33chore: update ilshidur/action-discord action to v0.4.0 (#20811)Updates
typescript-eslintfrom 8.58.0 to 8.59.1Release notes
Sourced from typescript-eslint's releases.
... (truncated)
Changelog
Sourced from typescript-eslint's changelog.
Commits
5245793chore(release): publish 8.59.1ea9ae4fchore(release): publish 8.59.090c2803chore(release): publish 8.58.2b3315fdchore: convert import eslint to import js - followup (#12100)be6b49afix: remove tsbuildinfo cache file from published packages (#12187)5311ed3chore(release): publish 8.58.1Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions