grafast 1.0.0-rc.1 → 1.0.0-rc.2

package/dist/interfaces.d.ts

@@ -399,9 +399,11 @@ export interface UnbatchedExecutionExtra extends ExecutionExtraBase {
 }
 export type ExecutionValue<TData = any> = BatchExecutionValue<TData> | UnaryExecutionValue<TData>;
 interface ExecutionValueBase<TData = any> {
+    /** Data at index in the batch (`0 <= batchIndex < count`) */
     at(i: number): TData;
+    /** `false` for unary execution values, `true` otherwise */
     isBatch: boolean;
-    /** Returns this.value for a unary execution value; throws if non-unary */
+    /** Returns the value for a unary execution value; throws if non-unary */
     unaryValue(): TData;
 }
 export interface BatchExecutionValue<TData = any> extends ExecutionValueBase<TData> {
@@ -422,17 +424,29 @@ export interface ExecutionDetailsStream {
     initialCount: number;
 }
 export interface ExecutionDetails<TDeps extends readonly [...any[]] = readonly [...any[]]> {
+    /** The size of the batch being processed */
     count: number;
-    indexMap: IndexMap;
-    indexForEach: IndexForEach;
+    /** An "execution value" for each dependency of the step */
     values: {
         [DepIdx in keyof TDeps]: ExecutionValue<TDeps[DepIdx]>;
     } & {
         length: TDeps["length"];
         map: ReadonlyArray<ExecutionValue<TDeps[number]>>["map"];
     };
-    extra: ExecutionExtra;
+    /** Helper; makes array from `callback(batchIndex)` for each `0 <= batchIndex < count` */
+    indexMap: IndexMap;
+    /** Helper; calls `callback` for each batchIndex in the batch; no return value */
+    indexForEach: IndexForEach;
+    /**
+     * If this step is expected to return a stream (e.g. because it's a
+     * `subscription`, or because of the `@stream` incremental delivery
+     * directive) then an object with details of the stream, such as how many
+     * records were requested up front (`initialCount`). For subscriptions,
+     * `initialCount` will always be `0`.
+     */
     stream: ExecutionDetailsStream | null;
+    /** Currently experimental, use it at your own risk (and see the source for docs) */
+    extra: ExecutionExtra;
 }
 export interface LocationDetails {
     node: ASTNode | readonly ASTNode[];

package/dist/steps/object.js

@@ -271,7 +271,7 @@ class ObjectStep extends step_js_1.UnbatchedStep {
             // Replace self with constant
             // We'll cache so that the constants can be more easily deduplicated
             const meta = opts.meta;
-            const keysJoined = this.keys.join(",");
+            const keysJoined = JSON.stringify(this.keys);
             if (!meta[keysJoined]) {
                 meta[keysJoined] = [];
             }

package/dist/utils.js

@@ -731,9 +731,16 @@ function writeableArray(a) {
  * directly or indirectly (via a chain of dependencies).
  */
 function stepADependsOnStepB(stepA, stepB, sansSideEffects = false) {
+    return _stepADependsOnStepB(stepA, stepB, sansSideEffects, new Set());
+}
+/** @internal */
+function _stepADependsOnStepB(stepA, stepB, sansSideEffects, visited) {
     if (stepA === stepB) {
         throw new Error("Invalid call to stepADependsOnStepB");
     }
+    if (visited.has(stepA))
+        return false;
+    visited.add(stepA);
     // PERF: bredth-first might be better.
     // PERF: we can stop looking once we pass a certain layerPlan boundary.
     // PERF: maybe some form of caching here would be sensible?
@@ -747,14 +754,14 @@ function stepADependsOnStepB(stepA, stepB, sansSideEffects = false) {
             dep.implicitSideEffectStep !== stepB.implicitSideEffectStep) {
             return false;
         }
-        if (stepADependsOnStepB(dep, stepB)) {
+        if (_stepADependsOnStepB(dep, stepB, false, visited)) {
             return true;
         }
     }
     if (stepA.implicitSideEffectStep) {
         if (stepA.implicitSideEffectStep === stepB)
             return true;
-        return stepADependsOnStepB(stepA.implicitSideEffectStep, stepB);
+        return _stepADependsOnStepB(stepA.implicitSideEffectStep, stepB, false, visited);
     }
     else {
         return false;

package/dist/version.d.ts

@@ -1,2 +1,2 @@
-export declare const version = "1.0.0-rc.1";
+export declare const version = "1.0.0-rc.2";
 //# sourceMappingURL=version.d.ts.map

package/dist/version.js

@@ -2,5 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.version = void 0;
 // This file is autogenerated by /scripts/postversion.mjs
-exports.version = "1.0.0-rc.1";
+exports.version = "1.0.0-rc.2";
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grafast",
-  "version": "1.0.0-rc.1",
+  "version": "1.0.0-rc.2",
   "description": "Cutting edge GraphQL planning and execution engine",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -55,11 +55,11 @@
     "node": ">=14.17"
   },
   "dependencies": {
-    "@graphile/lru": "^5.0.0-rc.1",
+    "@graphile/lru": "^5.0.0-rc.2",
     "chalk": "^4.1.2",
-    "debug": "^4.4.1",
+    "debug": "^4.4.3",
     "eventemitter3": "^5.0.1",
-    "graphile-config": "^1.0.0-rc.1",
+    "graphile-config": "^1.0.0-rc.2",
     "graphql": "^16.9.0",
     "iterall": "^1.3.0",
     "tslib": "^2.8.1"