fix(api): type ResolverWrapper against graphql's native GraphQLResolveInfo

[smartive-nicolai]

Branched off #555 and contains that PR's commit, so merging this lands the @graphql-tools/utils v12 bump and its fix together. #555 can be closed in favour of this. If you'd rather keep them separate, retarget this at renovate/major-graphql-tools-monorepo.

The problem

@graphql-tools/utils v12 augments GraphQLResolveInfo with two required members, getAbortSignal and getAsyncHelpers. graphql 17 declares both natively; graphql 16 declares neither.

ResolverWrapper was expressed through that package's IFieldResolver, so it inherited the augmented resolve info. graphql-codegen types its ResolverFn against graphql's own GraphQLResolveInfo — so on graphql 16, every consumer that wraps codegen-shaped resolvers stops compiling:

error TS2322: Type '(resolver: ResolverFn<...>) => ResolverFn<...>' is not assignable to type 'ResolverWrapper'.
  Types of parameters 'resolver' and 'resolver' are incompatible.
    Types of parameters 'info' and 'info' are incompatible.
      Type 'GraphQLResolveInfo' is missing the following properties from type 'GraphQLResolveInfo': getAbortSignal, getAsyncHelpers

graphql 16 is inside our declared peer range (^16.12.0 || ^17.0.0), and Renovate's commit is a fix: so semantic-release ships it as a patch — this would have gone out silently in 29.4.1.

Runtime is unaffected; this is type-level only.

The fix

Express ResolverWrapper with graphql's own GraphQLResolveInfo, which is what every other resolver signature in the codebase (context.ts, resolvers/resolver.ts, resolvers/mutations.ts, api/execute.ts) already uses. The utils import was the odd one out.

additionalResolvers keeps using IResolvers — I checked that direction separately and it stays assignable on graphql 16, so it needs no change.

Internals are unaffected: getResolvers returns Record<string, any> and the wrapper is applied through lodash.mapValues.

Why CI didn't catch it

Two reasons, and the second is worth knowing independently:

1. The only wrapper the suite exercises is an inferred (resolver) => resolver (tests/api/execute.spec.ts), which never pins down the parameter type.
2. ts-jest runs transpile-only — tsconfig.jest.json sets isolatedModules: true, so no test file is ever type-checked. A type error anywhere under tests/ is currently invisible.

So this adds tests/types/resolver-wrapper.ts, type-checked by tsc through a scoped tsconfig.type-tests.json and wired into npm test as test:types. It runs on both graphql majors of the existing matrix. Scoped deliberately rather than type-checking all of tests/: tsconfig.jest.json's **/*.ts include sweeps in the separate docs/ docusaurus project, which does not type-check standalone.

knip.json gains a tests/types/** ignore, since the file is consumed by tsc rather than imported.

Verification

Ran locally against both graphql majors (npm install graphql@^16.12.0 / ^17.0.0, as the matrix does):

	graphql 16.14.2	graphql 17.0.2
lint	pass	pass
test:types	pass	pass
test:unit (157 tests)	pass	pass
build	pass	pass
knip	pass	pass

And confirmed the new test is load-bearing: with the src/api/execute.ts change reverted, test:types fails on graphql 16 with exactly the TS2322 above, while passing on graphql 17.

tests/api needs postgres and was not run locally — CI covers it.

Follow-up not included here

The --legacy-peer-deps comment in .github/workflows/testing.yml is stale: it blames @graphql-codegen/* for capping the graphql peer at ^16, but those packages all allow ^17 now. The actual remaining holdout is graphql-config (pulled in by @graphql-codegen/cli), whose latest release 5.1.6 still caps at ^16 — so the flag does have to stay, just for a different reason. I had this as a comment fix but the pushing GitHub App lacks workflows permission, so it needs a human commit.

🤖 Generated with Claude Code

[smartive-nicolai]

Superseded by #561, which is the same branch and the same commit (44c521b) targeted at renovate/major-graphql-tools-monorepo instead of main, so the fix merges into #555 and the two land together.

This PR can be closed. I could not close or retarget it myself — the GitHub App used here can create and comment on pull requests but has no update permission.

For the record, CI on 44c521b was fully green here: all 21 Test Package matrix jobs plus Lint project.

• 🐝