Skip to content

Major dependency upgrades: @octokit/graphql 9, TypeScript 7, Vitest 4 #29

Description

@joaodotwork

Four majors are behind as of v1.6.0. They were held back deliberately during the release rather than bumped blind — each needs its own verification.

Package Current Latest Risk
@octokit/graphql 8.2.2 9.0.4 High — every tool calls through it
typescript 5.9.3 7.0.2 Medium — Go-based compiler rewrite
vitest 3.2.7 4.1.10 Medium — test runner migration
@types/node 22.20.1 26.2.0 See note below

Why @octokit/graphql 9 is the one that matters

It's the client every tool in this server calls through, and the injected GraphQLFn type flows into all of src/tools/*. A major there can change:

So this one wants the error-handling paths exercised against the live API, not just a green unit suite.

@types/node should stay on ^22

Not part of this upgrade, and deliberately so: engines declares node >=22, so the types should match the oldest supported runtime. Bumping to 26 would let Node 26-only APIs typecheck clean and then fail at runtime for users on 22.

Worth revisiting only if the minimum supported Node moves.

Done when

  • All three majors upgraded, lockfile committed
  • pnpm run build, pnpm run typecheck, pnpm test green on Node 22 and 24
  • isDuplicateNameError and the iteration error paths verified against the live GraphQL API, not just mocks
  • Changelogs read for breaking changes, anything relevant noted in the PR

Follows the v1.6.0 release (#27).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions