simplify handleNull

packages/delegate/src/proxiedResult.ts

@@ -1,4 +1,4 @@
-import { GraphQLError, responsePathAsArray, GraphQLResolveInfo } from 'graphql';
+import { GraphQLError, GraphQLResolveInfo } from 'graphql';
 
 import { mergeDeep, ERROR_SYMBOL, relocatedError, setErrors, getErrors } from '@graphql-tools/utils';
 
@@ -22,7 +22,7 @@ export function unwrapResult(parent: any, info: GraphQLResolveInfo, path: Array<
 
     const object = newParent[responseKey];
     if (object == null) {
-      return handleNull(info.fieldNodes, responsePathAsArray(info.path), errors);
+      return handleNull(errors);
     }
 
     setErrors(

packages/delegate/src/results/handleList.ts

@@ -5,7 +5,6 @@ import {
   GraphQLResolveInfo,
   getNullableType,
   GraphQLType,
-  responsePathAsArray,
   isLeafType,
   isCompositeType,
   isListType,
@@ -32,7 +31,6 @@ export function handleList(
     handleListMember(
       getNullableType(type.ofType),
       listMember,
-      index,
       index in childErrors ? childErrors[index] : [],
       subschema,
       context,
@@ -45,15 +43,14 @@ export function handleList(
 function handleListMember(
   type: GraphQLType,
   listMember: any,
-  index: number,
   errors: ReadonlyArray<GraphQLError>,
   subschema: GraphQLSchema | SubschemaConfig,
   context: Record<string, any>,
   info: GraphQLResolveInfo,
   skipTypeMerging?: boolean
 ): any {
   if (listMember == null) {
-    return handleNull(info.fieldNodes, [...responsePathAsArray(info.path), index], errors);
+    return handleNull(errors);
   }
 
   if (isLeafType(type)) {

packages/delegate/src/results/handleNull.ts

@@ -1,12 +1,8 @@
-import { FieldNode, GraphQLError } from 'graphql';
+import { GraphQLError } from 'graphql';
 
 import { getErrorsByPathSegment, CombinedError } from '@graphql-tools/utils';
 
-export function handleNull(
-  fieldNodes: ReadonlyArray<FieldNode>,
-  path: Array<string | number>,
-  errors: ReadonlyArray<GraphQLError>
-) {
+export function handleNull(errors: ReadonlyArray<GraphQLError>) {
   if (errors.length) {
     if (errors.some(error => !error.path || error.path.length < 2)) {
       if (errors.length > 1) {
@@ -20,7 +16,7 @@ export function handleNull(
 
       const result = {};
       Object.keys(childErrors).forEach(pathSegment => {
-        result[pathSegment] = handleNull(fieldNodes, [...path, pathSegment], childErrors[pathSegment]);
+        result[pathSegment] = handleNull(childErrors[pathSegment]);
       });
 
       return result;
@@ -30,7 +26,7 @@ export function handleNull(
 
     const result: Array<any> = [];
     Object.keys(childErrors).forEach(pathSegment => {
-      result.push(handleNull(fieldNodes, [...path, parseInt(pathSegment, 10)], childErrors[pathSegment]));
+      result.push(handleNull(childErrors[pathSegment]));
     });
 
     return result;

packages/delegate/src/results/handleResult.ts

@@ -1,6 +1,5 @@
 import {
   GraphQLResolveInfo,
-  responsePathAsArray,
   getNullableType,
   isCompositeType,
   isLeafType,
@@ -27,7 +26,7 @@ export function handleResult(
   const type = getNullableType(returnType);
 
   if (result == null) {
-    return handleNull(info?.fieldNodes, responsePathAsArray(info?.path), errors);
+    return handleNull(errors);
   }
 
   if (isLeafType(type)) {