no need to save ancestors

we only use the parent because masking analysis done differently
yaacovCR · Nov 22, 2023 · 5aefe08 · 5aefe08
1 parent a2adbc1
commit 5aefe08
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/src/execution/collectFields.ts b/src/execution/collectFields.ts
@@ -28,7 +28,7 @@ import { getDirectiveValues } from './values.js';
 
 export interface DeferUsage {
   label: string | undefined;
-  ancestors: ReadonlyArray<DeferUsage | undefined>;
+  parent: DeferUsage | undefined;
 }
 
 export type GroupedFieldSet = Map<string, ReadonlyArray<FieldDetail>>;
@@ -244,10 +244,7 @@ function getDeferUsage(
 
   return {
     label: typeof defer.label === 'string' ? defer.label : undefined,
-    ancestors:
-      parentDeferUsage === undefined
-        ? [parentDeferUsage]
-        : [parentDeferUsage, ...parentDeferUsage.ancestors],
+    parent: parentDeferUsage,
   };
 }
 

diff --git a/src/execution/execute.ts b/src/execution/execute.ts
@@ -1454,8 +1454,7 @@ function addNewDeferredFragments(
 
   // For each new deferUsage object:
   for (const newDeferUsage of newDeferUsages) {
-    // DeferUsage objects track their parent targets; the immediate parent is always the first member of this list.
-    const parentTarget = newDeferUsage.ancestors[0];
+    const parentTarget = newDeferUsage.parent;
 
     // If the parent target is defined, the parent target is a DeferUsage object and
     // the parent result record is the DeferredFragmentRecord corresponding to that DeferUsage.