Remove info from ExecutionParams and add operationType (#3166)

* Remove info from ExecutionParams and add operationType

* Fix tests

* Allow getBatchingExecutor to cache executor by memoizing default executor

* Memoize  for a single  so allow  to memoize  correctly.

* Rename Request to ExecutionRequest

* Remove unnecessary diff

* Remove unnecessary operationName from DelegationContext

* Add `context` in `createRequest` and `createRequestInfo` instead of `delegateToSchema`

* Get context from request

* Do not export extra types from delegate

* Add info back

* Fix typings

Author: ardatan

.changeset/lazy-turtles-dress.md

@@ -0,0 +1,25 @@
+---
+'@graphql-tools/batch-execute': major
+'@graphql-tools/delegate': major
+'@graphql-tools/links': major
+'@graphql-tools/utils': major
+'@graphql-tools/wrap': major
+---
+
+BREAKING CHANGES;
+
+- Rename `Request` to `ExecutionRequest`
+- Drop unnecessary `GraphQLResolveInfo` in `ExecutionRequest`
+- Add required `operationType: OperationTypeNode` field in `ExecutionRequest`
+- Add `context` in `createRequest` and `createRequestInfo` instead of `delegateToSchema`
+
+> It doesn't rely on info.operation.operationType to allow the user to call an operation from different root type.
+And it doesn't call getOperationAST again and again to get operation type from the document/operation because we have it in Request and ExecutionParams
+https://github.com/ardatan/graphql-tools/pull/3166/files#diff-d4824895ea613dcc1f710c3ac82e952fe0ca12391b671f70d9f2d90d5656fdceR38
+
+Improvements;
+- Memoize `defaultExecutor` for a single `GraphQLSchema` so allow `getBatchingExecutor` to memoize `batchingExecutor` correctly.
+- And there is no different `defaultExecutor` is created for `subscription` and other operation types. Only one executor is used.
+
+> Batch executor is memoized by `executor` reference but `createDefaultExecutor` didn't memoize the default executor so this memoization wasn't working correctly on `batch-execute` side.
+https://github.com/ardatan/graphql-tools/blob/remove-info-executor/packages/batch-execute/src/getBatchingExecutor.ts#L9

packages/batch-execute/src/createBatchingExecutor.ts

@@ -1,10 +1,8 @@
-import { getOperationAST } from 'graphql';
-
 import DataLoader from 'dataloader';
 
 import { ValueOrPromise } from 'value-or-promise';
 
-import { Request, Executor, ExecutionResult } from '@graphql-tools/utils';
+import { ExecutionRequest, Executor, ExecutionResult } from '@graphql-tools/utils';
 
 import { mergeRequests } from './mergeRequests';
 import { splitResult } from './splitResult';
@@ -14,32 +12,30 @@ export function createBatchingExecutor(
   dataLoaderOptions?: DataLoader.Options<any, any, any>,
   extensionsReducer: (
     mergedExtensions: Record<string, any>,
-    request: Request
+    request: ExecutionRequest
   ) => Record<string, any> = defaultExtensionsReducer
 ): Executor {
   const loader = new DataLoader(createLoadFn(executor, extensionsReducer), dataLoaderOptions);
-  return (request: Request) =>
-    request.info?.operation.operation === 'subscription' ? executor(request) : loader.load(request);
+  return (request: ExecutionRequest) => {
+    return request.operationType === 'subscription' ? executor(request) : loader.load(request);
+  };
 }
 
 function createLoadFn(
   executor: Executor,
-  extensionsReducer: (mergedExtensions: Record<string, any>, request: Request) => Record<string, any>
+  extensionsReducer: (mergedExtensions: Record<string, any>, request: ExecutionRequest) => Record<string, any>
 ) {
-  return async (requests: ReadonlyArray<Request>): Promise<Array<ExecutionResult>> => {
-    const execBatches: Array<Array<Request>> = [];
+  return async (requests: ReadonlyArray<ExecutionRequest>): Promise<Array<ExecutionResult>> => {
+    const execBatches: Array<Array<ExecutionRequest>> = [];
     let index = 0;
     const request = requests[index];
-    let currentBatch: Array<Request> = [request];
+    let currentBatch: Array<ExecutionRequest> = [request];
     execBatches.push(currentBatch);
 
-    const operationType = getOperationAST(request.document, undefined)?.operation;
-    if (operationType == null) {
-      throw new Error('Could not identify operation type of document.');
-    }
+    const operationType = request.operationType;
 
     while (++index < requests.length) {
-      const currentOperationType = getOperationAST(requests[index].document, undefined)?.operation;
+      const currentOperationType = requests[index].operationType;
       if (operationType == null) {
         throw new Error('Could not identify operation type of document.');
       }
@@ -68,7 +64,10 @@ function createLoadFn(
   };
 }
 
-function defaultExtensionsReducer(mergedExtensions: Record<string, any>, request: Request): Record<string, any> {
+function defaultExtensionsReducer(
+  mergedExtensions: Record<string, any>,
+  request: ExecutionRequest
+): Record<string, any> {
   const newExtensions = request.extensions;
   if (newExtensions != null) {
     Object.assign(mergedExtensions, newExtensions);