Skip to content

Conversation

eddeee888
Copy link
Collaborator

Description

Related #10218

Name node overrides are confusing as the type doesn't match runtime behaviour. This PR fixes it for the client-side plugins (The resolvers counterpart is here)

@eddeee888 eddeee888 changed the title [typescript,typescript-operations] Remove visitor name node overrides [typescript] Remove visitor name node overrides Aug 26, 2025
Copy link

changeset-bot bot commented Aug 26, 2025

🦋 Changeset detected

Latest commit: 6adbb6d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
@graphql-codegen/visitor-plugin-common Major
@graphql-codegen/typescript Major
@graphql-codegen/typescript-document-nodes Patch
@graphql-codegen/gql-tag-operations Patch
@graphql-codegen/typescript-operations Patch
@graphql-codegen/typescript-resolvers Patch
@graphql-codegen/typed-document-node Patch
@graphql-codegen/introspection Patch
@graphql-codegen/client-preset Patch
@graphql-codegen/graphql-modules-preset Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@eddeee888 eddeee888 changed the title [typescript] Remove visitor name node overrides [typescript] Remove NameNode overrides Aug 26, 2025
@eddeee888 eddeee888 merged commit 78f7bfc into federation-fixes Aug 27, 2025
15 checks passed
@eddeee888 eddeee888 deleted the remove-visitor-name-node-override branch August 27, 2025 10:24
eddeee888 added a commit that referenced this pull request Aug 27, 2025
* Avoid NameNode and StringValue string conversion

* Add changeset
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