grafast 1.0.0-rc.4 → 1.0.0-rc.6

package/dist/args.d.ts

@@ -1,5 +1,5 @@
 import type { ExecutionArgs } from "graphql";
-import type { GrafastExecutionArgs, PromiseOrDirect } from "./interfaces.js";
+import type { GrafastExecutionArgs, PromiseOrDirect } from "./interfaces.ts";
 /** @deprecated Pass `resolvedPreset` and `requestContext` via args directly */
 export declare function hookArgs(rawArgs: ExecutionArgs, resolvedPreset: GraphileConfig.ResolvedPreset, ctx: Partial<Grafast.RequestContext>): PromiseOrDirect<Grafast.ExecutionArgs>;
 export declare function hookArgs(rawArgs: GrafastExecutionArgs): PromiseOrDirect<Grafast.ExecutionArgs>;

package/dist/args.js

@@ -1,9 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.hookArgs = hookArgs;
-const constants_js_1 = require("./constants.js");
-const middleware_js_1 = require("./middleware.js");
-const utils_js_1 = require("./utils.js");
+const constants_ts_1 = require("./constants.js");
+const middleware_ts_1 = require("./middleware.js");
+const utils_ts_1 = require("./utils.js");
 const EMPTY_OBJECT = Object.freeze(Object.create(null));
 const $$writeTest = Symbol("grafastWriteTest");
 /**
@@ -28,17 +28,17 @@ function hookArgs(rawArgs, legacyResolvedPreset, legacyCtx) {
         rawArgs.contextValue = Object.assign(Object.create(null), rawContextValue);
     }
     const middleware = rawMiddleware === undefined && resolvedPreset != null
-        ? (0, middleware_js_1.getGrafastMiddleware)(resolvedPreset)
+        ? (0, middleware_ts_1.getGrafastMiddleware)(resolvedPreset)
         : (rawMiddleware ?? null);
     if (rawMiddleware === undefined) {
         rawArgs.middleware = middleware;
     }
     const args = rawArgs;
     // Assert that args haven't already been hooked
-    if (args[constants_js_1.$$hooked]) {
+    if (args[constants_ts_1.$$hooked]) {
         throw new Error("Must not call hookArgs twice!");
     }
-    args[constants_js_1.$$hooked] = true;
+    args[constants_ts_1.$$hooked] = true;
     if (middleware != null) {
         return middleware.run("prepareArgs", { args }, finalizeWithEvent);
     }
@@ -51,7 +51,7 @@ function finalize(args) {
     const contextValue = args.contextValue;
     if (typeof userContext === "function") {
         const result = userContext(args.requestContext ?? EMPTY_OBJECT, args);
-        if ((0, utils_js_1.isPromiseLike)(result)) {
+        if ((0, utils_ts_1.isPromiseLike)(result)) {
             // Deliberately shadowed 'result'
             return result.then((result) => {
                 Object.assign(contextValue, result);

package/dist/engine/LayerPlan.d.ts

@@ -1,7 +1,7 @@
-import type { Bucket } from "../bucket.js";
-import type { Step } from "../step";
-import type { __ValueStep } from "../steps/index.js";
-import type { OperationPlan } from "./OperationPlan";
+import type { Bucket } from "../bucket.ts";
+import type { Step } from "../step.ts";
+import type { __ValueStep } from "../steps/index.ts";
+import type { OperationPlan } from "./OperationPlan.ts";
 /** Non-branching, non-deferred */
 export interface LayerPlanReasonRoot {
     type: "root";
@@ -150,8 +150,6 @@ export type LayerPlanReasonsWithParentStep = HasParent<LayerPlanReason>;
  *
  */
 export declare class LayerPlan<TReason extends LayerPlanReason = LayerPlanReason> {
-    readonly operationPlan: OperationPlan;
-    readonly reason: TReason;
     id: number;
     distributorDependencies: null | {
         [distributorStepId: number]: number[];
@@ -176,6 +174,8 @@ export declare class LayerPlan<TReason extends LayerPlanReason = LayerPlanReason
      */
     outOfBoundsLayerPlanIds: Set<number>;
     private sideEffectRootLayerPlan;
+    readonly operationPlan: OperationPlan;
+    readonly reason: TReason;
     constructor(operationPlan: OperationPlan, reason: TReason);
     toString(): string;
     print(depth?: number): string;

package/dist/engine/LayerPlan.js

@@ -5,23 +5,23 @@ exports.hasParentLayerPlan = hasParentLayerPlan;
 exports.isDeferredLayerPlan = isDeferredLayerPlan;
 const tslib_1 = require("tslib");
 const assert = tslib_1.__importStar(require("../assert.js"));
-const constants_js_1 = require("../constants.js");
-const dev_js_1 = require("../dev.js");
-const error_js_1 = require("../error.js");
-const inspect_js_1 = require("../inspect.js");
-const utils_js_1 = require("../utils.js");
-const distributor_js_1 = require("./distributor.js");
-const executeBucket_js_1 = require("./executeBucket.js");
+const constants_ts_1 = require("../constants.js");
+const dev_ts_1 = require("../dev.js");
+const error_ts_1 = require("../error.js");
+const inspect_ts_1 = require("../inspect.js");
+const utils_ts_1 = require("../utils.js");
+const distributor_ts_1 = require("./distributor.js");
+const executeBucket_ts_1 = require("./executeBucket.js");
 /**
  * If any of these flags exist on a value then it isn't the value we want to
  * include for the combo step.
  */
-const SKIP_FLAGS = constants_js_1.FLAG_ERROR | constants_js_1.FLAG_STOPPED | constants_js_1.FLAG_POLY_SKIPPED | constants_js_1.FLAG_INHIBITED;
+const SKIP_FLAGS = constants_ts_1.FLAG_ERROR | constants_ts_1.FLAG_STOPPED | constants_ts_1.FLAG_POLY_SKIPPED | constants_ts_1.FLAG_INHIBITED;
 /**
  * If any of these flags exist on the "$__typename" step, then the value
  * should resolve to null
  */
-const NO_TYPENAME_FLAGS = constants_js_1.FLAG_ERROR | constants_js_1.FLAG_STOPPED | constants_js_1.FLAG_POLY_SKIPPED | constants_js_1.FLAG_INHIBITED | constants_js_1.FLAG_NULL;
+const NO_TYPENAME_FLAGS = constants_ts_1.FLAG_ERROR | constants_ts_1.FLAG_STOPPED | constants_ts_1.FLAG_POLY_SKIPPED | constants_ts_1.FLAG_INHIBITED | constants_ts_1.FLAG_NULL;
 function hasParentLayerPlan(reason) {
     return reason.type !== "root" && reason.type !== "combined";
 }
@@ -59,11 +59,6 @@ function isDeferredLayerPlan(layerPlan) {
  */
 class LayerPlan {
     constructor(operationPlan, reason) {
-        // This layer plan is dependent on the latest side effect. Note that when
-        // we set a `rootStep` later, if the root step is dependent on this step
-        // (directly or indirectly) we will clear this property.
-        this.operationPlan = operationPlan;
-        this.reason = reason;
         /**
          * Every layer plan has a "root step" that shapes the value the layer
          * returns. Note that this step may be dependent on other steps included in
@@ -133,6 +128,11 @@ class LayerPlan {
         this.combinations = [];
         /** @internal */
         this.finalCombinations = [];
+        this.operationPlan = operationPlan;
+        this.reason = reason;
+        // This layer plan is dependent on the latest side effect. Note that when
+        // we set a `rootStep` later, if the root step is dependent on this step
+        // (directly or indirectly) we will clear this property.
         this.stepsByConstructor = new Map();
         if (reason.type === "root") {
             this.sideEffectRootLayerPlan = null;
@@ -148,7 +148,7 @@ class LayerPlan {
             const firstParentLayerPlan = reason.parentLayerPlans[0];
             this.sideEffectRootLayerPlan =
                 firstParentLayerPlan.sideEffectRootLayerPlan;
-            if (dev_js_1.isDev) {
+            if (dev_ts_1.isDev) {
                 for (const plp of reason.parentLayerPlans) {
                     assert.strictEqual(plp.sideEffectRootLayerPlan, this.sideEffectRootLayerPlan, `Expected all parentLayerPlans to have the same sideEffectRootLayerPlan`);
                 }
@@ -323,7 +323,7 @@ class LayerPlan {
                 const hasNoNullsOrErrors = false;
                 if (this.rootStep._isUnary) {
                     const fieldValue = parentBucket.store.get(itemStepId);
-                    const forbiddenFlags = fieldValue._entryFlags & constants_js_1.FORBIDDEN_BY_NULLABLE_BOUNDARY_FLAGS;
+                    const forbiddenFlags = fieldValue._entryFlags & constants_ts_1.FORBIDDEN_BY_NULLABLE_BOUNDARY_FLAGS;
                     if (forbiddenFlags) {
                         size = 0;
                     }
@@ -334,7 +334,7 @@ class LayerPlan {
                             const orig = parentBucket.store.get(stepId);
                             if (orig.isBatch) {
                                 // Prepare store with an empty list for each copyPlanId
-                                const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                                const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                                 store.set(stepId, ev);
                                 batchCopy.push({ orig, ev });
                             }
@@ -345,7 +345,7 @@ class LayerPlan {
                         const parentBucketSize = parentBucket.size;
                         for (let originalIndex = 0; originalIndex < parentBucketSize; originalIndex++) {
                             if (parentSideEffectValue === null ||
-                                !(parentSideEffectValue._flagsAt(originalIndex) & constants_js_1.FLAG_ERROR)) {
+                                !(parentSideEffectValue._flagsAt(originalIndex) & constants_ts_1.FLAG_ERROR)) {
                                 const newIndex = size++;
                                 map.set(originalIndex, newIndex);
                                 polymorphicPathList[newIndex] =
@@ -379,14 +379,14 @@ class LayerPlan {
                     }
                 }
                 else {
-                    const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                    const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                     store.set(itemStepId, ev);
                     const batchCopy = [];
                     for (const stepId of copyStepIds) {
                         const orig = parentBucket.store.get(stepId);
                         if (orig.isBatch) {
                             // Prepare store with an empty list for each copyPlanId
-                            const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                            const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                             store.set(stepId, ev);
                             batchCopy.push({ orig, ev });
                         }
@@ -403,8 +403,8 @@ class LayerPlan {
                     // we must "multiply up" (down) the store entries.
                     for (let originalIndex = 0; originalIndex < parentBucket.size; originalIndex++) {
                         if ((parentSideEffectValue === null ||
-                            !(parentSideEffectValue._flagsAt(originalIndex) & constants_js_1.FLAG_ERROR)) &&
-                            !(nullableStepStore._flagsAt(originalIndex) & constants_js_1.FLAG_NULL)) {
+                            !(parentSideEffectValue._flagsAt(originalIndex) & constants_ts_1.FLAG_ERROR)) &&
+                            !(nullableStepStore._flagsAt(originalIndex) & constants_ts_1.FLAG_NULL)) {
                             const newIndex = size++;
                             map.set(originalIndex, newIndex);
                             ev._copyResult(newIndex, nullableStepStore, originalIndex);
@@ -436,7 +436,7 @@ class LayerPlan {
                 if (this.rootStep._isUnary) {
                     throw new Error("listItem layer plan can't have a unary root step!");
                 }
-                const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                 store.set(itemStepId, ev);
                 const batchCopy = [];
                 for (const stepId of copyStepIds) {
@@ -444,7 +444,7 @@ class LayerPlan {
                     const orig = parentBucket.store.get(stepId);
                     if (orig.isBatch) {
                         // Prepare store with an empty list for each copyPlanId
-                        const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                        const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                         store.set(stepId, ev);
                         batchCopy.push({ orig, ev });
                     }
@@ -457,7 +457,7 @@ class LayerPlan {
                 for (let originalIndex = 0; originalIndex < parentBucket.size; originalIndex++) {
                     const list = listStepStore.at(originalIndex);
                     if ((parentSideEffectValue === null ||
-                        !(parentSideEffectValue._flagsAt(originalIndex) & constants_js_1.FLAG_ERROR)) &&
+                        !(parentSideEffectValue._flagsAt(originalIndex) & constants_ts_1.FLAG_ERROR)) &&
                         Array.isArray(list)) {
                         const newIndexes = [];
                         map.set(originalIndex, newIndexes);
@@ -465,15 +465,15 @@ class LayerPlan {
                             const newIndex = size++;
                             newIndexes.push(newIndex);
                             let val = list[j];
-                            let flags = constants_js_1.NO_FLAGS;
+                            let flags = constants_ts_1.NO_FLAGS;
                             if (val == null) {
-                                flags = flags | constants_js_1.FLAG_NULL;
+                                flags = flags | constants_ts_1.FLAG_NULL;
                             }
-                            else if (typeof val === "object" && (0, error_js_1.isFlaggedValue)(val)) {
+                            else if (typeof val === "object" && (0, error_ts_1.isFlaggedValue)(val)) {
                                 flags = val.flags;
                                 val = val.value;
                                 if (val == null) {
-                                    flags = flags | constants_js_1.FLAG_NULL;
+                                    flags = flags | constants_ts_1.FLAG_NULL;
                                 }
                             }
                             ev._setResult(newIndex, val, flags);
@@ -515,7 +515,7 @@ class LayerPlan {
                         throw new Error(`GrafastInternalError<548f0d84-4556-4189-8655-fb16aa3345a6>: new bucket for ${this} wants to copy ${this.operationPlan.dangerouslyGetStep(stepId)}, but bucket for ${parentBucket.layerPlan} doesn't contain that plan`);
                     }
                     if (orig.isBatch) {
-                        const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                        const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                         store.set(stepId, ev);
                         batchCopy.push({ orig, ev });
                     }
@@ -530,7 +530,7 @@ class LayerPlan {
                         continue;
                     }
                     if (parentSideEffectValue !== null &&
-                        parentSideEffectValue._flagsAt(originalIndex) & constants_js_1.FLAG_ERROR) {
+                        parentSideEffectValue._flagsAt(originalIndex) & constants_ts_1.FLAG_ERROR) {
                         skipIndex?.(originalIndex);
                         continue;
                     }
@@ -564,7 +564,7 @@ class LayerPlan {
                         throw new Error(`GrafastInternalError<548f0d84-4556-4189-8655-fb16aa3345a6>: new bucket for ${this} wants to copy ${this.operationPlan.dangerouslyGetStep(stepId)}, but bucket for ${parentBucket.layerPlan} doesn't contain that plan`);
                     }
                     if (orig.isBatch) {
-                        const ev = (0, executeBucket_js_1.batchExecutionValue)([]);
+                        const ev = (0, executeBucket_ts_1.batchExecutionValue)([]);
                         store.set(stepId, ev);
                         batchCopy.push({ orig, ev });
                     }
@@ -574,7 +574,7 @@ class LayerPlan {
                 }
                 for (let originalIndex = 0; originalIndex < parentBucket.size; originalIndex++) {
                     if (parentSideEffectValue !== null &&
-                        parentSideEffectValue._flagsAt(originalIndex) & constants_js_1.FLAG_ERROR) {
+                        parentSideEffectValue._flagsAt(originalIndex) & constants_ts_1.FLAG_ERROR) {
                         skipIndex?.(originalIndex);
                         continue;
                     }
@@ -609,12 +609,12 @@ class LayerPlan {
             }
             default: {
                 const never = this.reason;
-                throw new Error(`GrafastInternalError<8162e6c2-3d66-4d67-ba03-5310a4f9a6d4>: unhandled reason '${(0, inspect_js_1.inspect)(never)}'`);
+                throw new Error(`GrafastInternalError<8162e6c2-3d66-4d67-ba03-5310a4f9a6d4>: unhandled reason '${(0, inspect_ts_1.inspect)(never)}'`);
             }
         }
         if (size > 0) {
             if (this.distributorDependencies !== null) {
-                if (dev_js_1.isDev) {
+                if (dev_ts_1.isDev) {
                     if (this.reason.type === "listItem") {
                         throw new Error(`distributorDependencies should not be set on ${this}`);
                     }
@@ -625,7 +625,7 @@ class LayerPlan {
                         const distribEV = parentBucket.store.get(Number(distributorStepId));
                         for (const originalIndex of skippedIndicies) {
                             const v = distribEV.at(originalIndex);
-                            if ((0, distributor_js_1.isDistributor)(v)) {
+                            if ((0, distributor_ts_1.isDistributor)(v)) {
                                 for (const consumerStepId of consumerStepIds) {
                                     v.releaseIfUnused(consumerStepId);
                                 }
@@ -635,7 +635,7 @@ class LayerPlan {
                 }
             }
             // Reference
-            const childBucket = (0, executeBucket_js_1.newBucket)(parentBucket, {
+            const childBucket = (0, executeBucket_ts_1.newBucket)(parentBucket, {
                 layerPlan: this,
                 size,
                 store,
@@ -658,7 +658,7 @@ class LayerPlan {
                     const distribEV = parentBucket.store.get(Number(distributorStepId));
                     for (let originalIndex = 0; originalIndex < parentBucketSize; originalIndex++) {
                         const v = distribEV.at(originalIndex);
-                        if ((0, distributor_js_1.isDistributor)(v)) {
+                        if ((0, distributor_ts_1.isDistributor)(v)) {
                             for (const consumerStepId of consumerStepIds) {
                                 v.releaseIfUnused(consumerStepId);
                             }
@@ -680,7 +680,7 @@ class LayerPlan {
         const polymorphicPathList = [];
         const iterators = [];
         const mapByParentLayerPlanId = Object.create(null);
-        let flagUnion = constants_js_1.NO_FLAGS;
+        let flagUnion = constants_ts_1.NO_FLAGS;
         let totalSize = 0;
         const batchStepIdsToCopy = new Set();
         for (const plp of this.reason.parentLayerPlans) {
@@ -717,7 +717,7 @@ class LayerPlan {
         if (totalSize === 0)
             return null;
         for (const stepId of batchStepIdsToCopy) {
-            const newEv = (0, executeBucket_js_1.batchExecutionValue)((0, utils_js_1.arrayOfLength)(totalSize, null), (0, utils_js_1.arrayOfLength)(totalSize, constants_js_1.FLAG_NULL | constants_js_1.FLAG_STOPPED));
+            const newEv = (0, executeBucket_ts_1.batchExecutionValue)((0, utils_ts_1.arrayOfLength)(totalSize, null), (0, utils_ts_1.arrayOfLength)(totalSize, constants_ts_1.FLAG_NULL | constants_ts_1.FLAG_STOPPED));
             store.set(stepId, newEv);
             let offset = 0;
             for (const plp of this.reason.parentLayerPlans) {
@@ -744,7 +744,7 @@ class LayerPlan {
         //
         for (const { targetStepId, sourceStepsByLayerPlanId } of this
             .finalCombinations) {
-            const ev = (0, executeBucket_js_1.batchExecutionValue)((0, utils_js_1.arrayOfLength)(totalSize, null), (0, utils_js_1.arrayOfLength)(totalSize, constants_js_1.FLAG_NULL | constants_js_1.FLAG_STOPPED));
+            const ev = (0, executeBucket_ts_1.batchExecutionValue)((0, utils_ts_1.arrayOfLength)(totalSize, null), (0, utils_ts_1.arrayOfLength)(totalSize, constants_ts_1.FLAG_NULL | constants_ts_1.FLAG_STOPPED));
             store.set(targetStepId, ev);
             let offset = 0;
             for (const plp of this.reason.parentLayerPlans) {
@@ -776,7 +776,7 @@ class LayerPlan {
         //
         ////////////////////////////////////////////////////////////////////////////
         // Reference
-        const childBucket = (0, executeBucket_js_1.newBucket)({ metaByMetaKey: undefined /* use defaults */, sharedState }, {
+        const childBucket = (0, executeBucket_ts_1.newBucket)({ metaByMetaKey: undefined /* use defaults */, sharedState }, {
             layerPlan: this,
             size: totalSize,
             store,
@@ -826,8 +826,8 @@ class LayerPlan {
             // unimportant because probably the order of all subsets will be the
             // same, so we don't need to do an order-independent comparison (which
             // would be more expensiv)
-            (0, utils_js_1.arraysMatch)(c.reason.typeNames, typeNames) &&
-            (0, utils_js_1.setsMatch)(c.reason.polymorphicPaths, polymorphicPaths));
+            (0, utils_ts_1.arraysMatch)(c.reason.typeNames, typeNames) &&
+            (0, utils_ts_1.setsMatch)(c.reason.polymorphicPaths, polymorphicPaths));
         if (existing) {
             return existing;
         }

package/dist/engine/OperationPlan.d.ts

@@ -1,15 +1,15 @@
 import type { FragmentDefinitionNode, GraphQLField, GraphQLObjectType, GraphQLOutputType, GraphQLSchema, GraphQLUnionType, OperationDefinitionNode } from "graphql";
-import { $$timeout, $$ts } from "../constants.js";
-import type { Constraint } from "../constraints.js";
-import type { ErrorBehavior, GrafastPlanJSON } from "../index.js";
-import { Step } from "../index.js";
-import type { FieldPlanResolver, LocationDetails, TrackedArguments } from "../interfaces.js";
-import type { ApplyAfterModeArg } from "../operationPlan-input.js";
-import type { GrafastOperationOptions } from "../prepare.js";
-import type { LayerPlanReasonSubroutine } from "./LayerPlan.js";
-import { LayerPlan } from "./LayerPlan.js";
-import type { ProcessGroupedFieldSetDetails, StreamDetails } from "./OperationPlanTypes.js";
-import { OutputPlan } from "./OutputPlan.js";
+import { $$timeout, $$ts } from "../constants.ts";
+import type { Constraint } from "../constraints.ts";
+import type { ErrorBehavior, GrafastPlanJSON } from "../index.ts";
+import { Step } from "../index.ts";
+import type { FieldPlanResolver, LocationDetails, TrackedArguments } from "../interfaces.ts";
+import type { ApplyAfterModeArg } from "../operationPlan-input.ts";
+import type { GrafastOperationOptions } from "../prepare.ts";
+import type { LayerPlanReasonSubroutine } from "./LayerPlan.ts";
+import { LayerPlan } from "./LayerPlan.ts";
+import type { ProcessGroupedFieldSetDetails, StreamDetails } from "./OperationPlanTypes.ts";
+import { OutputPlan } from "./OutputPlan.ts";
 export declare const POLYMORPHIC_ROOT_PATH: null;
 export declare const POLYMORPHIC_ROOT_PATHS: ReadonlySet<string> | null;
 /** Beware: the list of phases may change over time... @experimental */
@@ -18,19 +18,6 @@ export interface MetaByMetaKey {
     [metaKey: string | number | symbol]: Record<string, any>;
 }
 export declare class OperationPlan {
-    readonly schema: GraphQLSchema;
-    readonly operation: OperationDefinitionNode;
-    readonly fragments: {
-        [fragmentName: string]: FragmentDefinitionNode;
-    };
-    readonly variableValues: {
-        [key: string]: any;
-    };
-    readonly context: {
-        [key: string]: any;
-    };
-    readonly rootValue: any;
-    readonly errorBehavior: ErrorBehavior;
     readonly [$$timeout]: undefined;
     readonly [$$ts]: undefined;
     readonly queryType: GraphQLObjectType;
@@ -67,6 +54,19 @@ export declare class OperationPlan {
     private optimizeMeta;
     private scalarPlanInfo;
     private frozenPlanningPaths;
+    readonly schema: GraphQLSchema;
+    readonly operation: OperationDefinitionNode;
+    readonly fragments: {
+        [fragmentName: string]: FragmentDefinitionNode;
+    };
+    readonly variableValues: {
+        [key: string]: any;
+    };
+    readonly context: {
+        [key: string]: any;
+    };
+    readonly rootValue: any;
+    readonly errorBehavior: ErrorBehavior;
     private readonly planningTimeout;
     private readonly maxPlanningDepth;
     constructor(schema: GraphQLSchema, operation: OperationDefinitionNode, fragments: {
@@ -108,12 +108,9 @@ export declare class OperationPlan {
     }>;
     /**
      *
-     * @param outputPlan - The output plan that this selection set is being added to
-     * @param path - The path within the outputPlan that we're adding stuff (only for root/object OutputPlans)
-     * @param parentStep - The step that represents the selection set root
-     * @param objectType - The object type that this selection set is being evaluated for (note polymorphic selection should already have been handled by this point)
-     * @param selections - The GraphQL selections (fields, fragment spreads, inline fragments) to evaluate
-     * @param isMutation - If true this selection set should be executed serially rather than in parallel (each field gets its own LayerPlan)
+     * @param details - planning details for this selection set. Includes the
+     * output plan, path, parent step, position type, selections, and an
+     * `isMutation` flag to enforce serial execution.
      */
     private planSelectionSet;
     private planningQueue;
@@ -143,7 +140,7 @@ export declare class OperationPlan {
         haltTree: boolean;
         latestSideEffectStep: Step<any> | null;
     } | {
-        step: import("../index.js").ErrorStep<any>;
+        step: import("../index.ts").ErrorStep<any>;
         haltTree: boolean;
         latestSideEffectStep: Step<any> | null;
     };