From a1e8f6ebcb95cc51899e0d478bdf19db0d437b80 Mon Sep 17 00:00:00 2001 From: Yaacov Rydzinski Date: Thu, 30 Jan 2025 15:45:26 +0200 Subject: [PATCH] add deprecated note to assertValidExecutionArguments to notify theoretical users via deep import of imminent removal --- src/execution/execute.ts | 1 + src/execution/subscribe.ts | 2 ++ 2 files changed, 3 insertions(+) diff --git a/src/execution/execute.ts b/src/execution/execute.ts index 55c22ea9de..a965ae324f 100644 --- a/src/execution/execute.ts +++ b/src/execution/execute.ts @@ -246,6 +246,7 @@ function buildResponse( * Essential assertions before executing to provide developer feedback for * improper use of the GraphQL library. * + * @deprecated will be removed in v17 in favor of assertValidSchema() and TS checks * @internal */ export function assertValidExecutionArguments( diff --git a/src/execution/subscribe.ts b/src/execution/subscribe.ts index 8b20ec3374..78dc97916e 100644 --- a/src/execution/subscribe.ts +++ b/src/execution/subscribe.ts @@ -19,6 +19,7 @@ import type { ExecutionResult, } from './execute'; import { + // eslint-disable-next-line import/no-deprecated assertValidExecutionArguments, buildExecutionContext, buildResolveInfo, @@ -160,6 +161,7 @@ export async function createSourceEventStream( // If arguments are missing or incorrectly typed, this is an internal // developer mistake which should throw an early error. + // eslint-disable-next-line import/no-deprecated assertValidExecutionArguments(schema, document, variableValues); // If a valid execution context cannot be created due to incorrect arguments,