Skip to content

chore: upgrade remaining major dependencies#54

Open
birdwell wants to merge 1 commit into
mainfrom
chore/remaining-major-deps
Open

chore: upgrade remaining major dependencies#54
birdwell wants to merge 1 commit into
mainfrom
chore/remaining-major-deps

Conversation

@birdwell

Copy link
Copy Markdown
Owner

Summary

  • graphql 16 → 17.0.2 (Node 22+; aligns with current runtime)
  • @testing-library/jest-dom 6 → 7.0.0 (@testing-library/dom already present; Node ≥22)
  • @types/node 25 → 26.1.1
  • Bump graphql-codegen packages so peers accept GraphQL 17 (cli 7.2, typescript/operations/schema-ast minors)

Note: graphql-request@7.4.0 still declares peer graphql@14-16. Install warns, but yarn run check / tests / build pass with current GraphQLClient + gql usage. Revisit when graffle publishes a GraphQL 17-compatible release.

Generated AniList schema drift from yarn generate was intentionally not committed.

Test plan

  • yarn run check
  • yarn test (85)
  • yarn build
  • yarn generate (smoke only; output reverted)
  • Spot-check login / AniList proxy after merge

Made with Cursor

Bump graphql 17, jest-dom 7, and @types/node 26, and align graphql-codegen peers that support GraphQL 17. graphql-request still declares a 14–16 peer range but works with the current client usage.

Co-authored-by: Cursor <cursoragent@cursor.com>
@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown

Greptile Summary

This PR upgrades the remaining major dependencies and updates their lockfile entries.

  • GraphQL 17 and compatible code-generation packages.
  • jest-dom 7 for the test environment.
  • Node 26 type declarations and undici types.

Confidence Score: 5/5

This looks mergeable with a peer-dependency compatibility caveat.

  • The updated lockfile is consistent with the declared versions.
  • Current checks cover the main build, test, and generation paths.
  • Strict installers can reject the GraphQL 17 and graphql-request 7.4 dependency combination.

package.json

Important Files Changed

Filename Overview
package.json Upgrades GraphQL, code-generation tools, jest-dom, and Node type declarations; GraphQL now falls outside graphql-request's declared peer range.
yarn.lock Updates and deduplicates the resolved packages for the new dependency ranges.

Reviews (1): Last reviewed commit: "Upgrade remaining major dependencies aft..." | Re-trigger Greptile

Comment thread package.json
Comment on lines +69 to 70
"graphql": "^17.0.2",
"graphql-request": "^7.4.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 GraphQL Peer Contract Is Violated

graphql-request@7.4.0 declares support through GraphQL 16, while this change installs GraphQL 17 for the runtime GraphQLClient path. Strict peer-dependency installers can reject this dependency tree before the application builds; align both packages or explicitly constrain the supported installation environment.

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.

1 participant