Skip to content

docs(graphql): registration order and what happens when it is wrong - #132

Merged
Theauxm merged 1 commit into
mainfrom
docs/registration-order
Sep 2, 2026
Merged

docs(graphql): registration order and what happens when it is wrong#132
Theauxm merged 1 commit into
mainfrom
docs/registration-order

Conversation

@Theauxm

@Theauxm Theauxm commented Sep 2, 2026

Copy link
Copy Markdown
Member

AddTraxGraphQL() picks its subscription interceptor from the auth schemes registered by the time it runs, so authentication has to be registered before it. That was true before and undocumented; the difference now is that the wrong order fails at startup instead of quietly accepting every connection_init.

Also says plainly what is not order-dependent, since that is the more common question: @authorize is on the schema, so query and mutation gating works either way round.

Adds reference/registration-order.md as the tracked home for the contributor rule. The workspace CLAUDE.md is not in any repo, so guidance that only lives there does not reach CI or another clone.

AddTraxGraphQL() picks its subscription interceptor from the auth schemes registered
by the time it runs, so authentication has to be registered before it. That was true
before and undocumented; the difference now is that the wrong order fails at startup
instead of quietly accepting every connection_init.

Also says plainly what is not order-dependent, since that is the more common question:
@authorize is on the schema, so query and mutation gating works either way round.

Adds reference/registration-order.md as the tracked home for the contributor rule. The
workspace CLAUDE.md is not in any repo, so guidance that only lives there does not reach
CI or another clone.
@Theauxm
Theauxm merged commit 57db53d into main Sep 2, 2026
2 checks passed
@Theauxm
Theauxm deleted the docs/registration-order branch September 2, 2026 17:59
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