Skip to content

[visitor-plugin-common] Make ClientSideBaseVisitor fragments protected#10874

Merged
eddeee888 merged 2 commits into
masterfrom
make-fragments-protected
Jun 18, 2026
Merged

[visitor-plugin-common] Make ClientSideBaseVisitor fragments protected#10874
eddeee888 merged 2 commits into
masterfrom
make-fragments-protected

Conversation

@eddeee888

@eddeee888 eddeee888 commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Description

Plugins such as typescript-react-query may need access to the parsed and validated fragments.

For example, in the mentioned community package, we need to check if there are fragments to add certain imports.

Related dotansimha/graphql-code-generator-community#1410 (comment)

dotansimha/graphql-code-generator-community#1516

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@changeset-bot

changeset-bot Bot commented Jun 18, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 412a5ad

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

This PR includes changesets to release 1 package
Name Type
@graphql-codegen/visitor-plugin-common 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

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphql-codegen/visitor-plugin-common 7.1.1-alpha-20260618134842-412a5adeb9179a03dcd3dadccc0f91dc062df16e npm ↗︎ unpkg ↗︎

@github-actions

Copy link
Copy Markdown
Contributor

💻 Website Preview

The latest changes are available as preview in: https://pr-10874.graphql-code-generator.pages.dev

@eddeee888 eddeee888 merged commit 6d37a7d into master Jun 18, 2026
18 checks passed
@eddeee888 eddeee888 deleted the make-fragments-protected branch June 18, 2026 13:55
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