Skip to content

Accessible directive args - #330

Closed
jdolle wants to merge 2 commits into
mainfrom
accessible-directive-args
Closed

jdolle wants to merge 2 commits into
mainfrom
accessible-directive-args

Conversation

@jdolle

@jdolle jdolle commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Currently, an invalid schema can be produced because a composed directive's arguments are flagged as @inaccessible

This change takes directive arguments into account so that the schema produced is valid.

Closes #329

@jdolle
jdolle requested review from dotansimha and n1ru4l September 8, 2026 20:54
@jdolle jdolle self-assigned this Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The latest changes of this PR are not available as alpha, since there are no linked changesets for this PR.

@n1ru4l n1ru4l closed this in #329 Sep 9, 2026
kamilkisiela pushed a commit to robbin-norberg-vcc/federation-composition that referenced this pull request Sep 9, 2026
…reachable types from schema contracts (graphql-hive#329)

Preserve type referenced by directive arguments when removing
unreachable types from schema contracts

We did not traverse the directive definitions before.

Closes graphql-hive#330
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