Add DisableInternalDirectives option

[tobias-tengler]

No description provided.

[Copilot]

Pull request overview

Adds a new server option to control whether internal directives (e.g. @authorize) are included in schema SDL responses served via the ASP.NET Core schema download endpoints, and documents how to opt back into the legacy behavior.

Changes:

• Introduce GraphQLServerOptions.ExposeInternalDirectivesThroughSchemaRequests and pass it through schema middleware/session/formatter layers.
• Update DefaultHttpResponseFormatter schema caching to vary by (schemaName, includeInternalDirectives) so cached SDL matches the requested behavior.
• Add snapshot-based tests and migration documentation for the new option.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
website/src/docs/hotchocolate/v16/migrating/migrate-from-15-to-16.md	Adds migration guidance describing internal directives no longer being exposed by default and how to opt in.
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/HttpGetSemanticNonNullSchemaMiddlewareTests.cs	Adds test covering internal-directive inclusion when the new option is enabled (semantic-non-null endpoint).
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/HttpGetSchemaMiddlewareTests.cs	Adds test covering internal-directive inclusion when the new option is enabled (schema endpoint).
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/snapshots/HttpGetSemanticNonNullSchemaMiddlewareTests.Download_GraphQL_SemanticNonNull_Schema_Outputs_Internal_Directives_When_ExposeInternalDirectivesThroughSchemaRequests_Is_True.snap	Snapshot for the new semantic-non-null opt-in behavior.
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/snapshots/HttpGetSchemaMiddlewareTests.Download_GraphQL_Schema_Outputs_Internal_Directives_When_ExposeInternalDirectivesThroughSchemaRequests_Is_True.snap	Snapshot for the new schema opt-in behavior.
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/snapshots/HttpGetSemanticNonNullSchemaMiddlewareTests.Download_GraphQL_SemanticNonNull_Schema_Outputs_Internal_Directives_When_ExposeInternalDirectives_Is_True.snap	Adds an additional snapshot file (currently appears unused/orphaned).
src/HotChocolate/AspNetCore/test/AspNetCore.Tests/snapshots/HttpGetSchemaMiddlewareTests.Download_GraphQL_Schema_Outputs_Internal_Directives_When_ExposeInternalDirectives_Is_True.snap	Adds an additional snapshot file (currently appears unused/orphaned).
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Options/GraphQLServerOptions.cs	Adds the new configuration option and ensures it’s copied in Clone().
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/HttpGetSemanticNonNullSchemaMiddleware.cs	Threads the option into schema writing for the semantic-non-null endpoint.
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/HttpGetSchemaMiddleware.cs	Threads the option into schema writing for schema endpoint responses.
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/ExecutorSession.cs	Extends schema-writing APIs to accept an includeInternalDirectives flag.
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Formatters/IHttpResponseFormatter.cs	Extends the public formatter interface with an includeInternalDirectives parameter for schema formatting.
src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Formatters/DefaultHttpResponseFormatter.cs	Implements directive inclusion + updates schema output caching keys to incorporate the new flag.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.