Another assertNotPromise

graphile · Oct 1, 2024 · 21eca19 · 21eca19
1 parent 0220a82
commit 21eca19
Showing 1 changed file with 13 additions and 7 deletions.
diff --git a/grafast/grafast/src/engine/OperationPlan.ts b/grafast/grafast/src/engine/OperationPlan.ts
@@ -79,6 +79,7 @@ import { timeSource } from "../timeSource.js";
 import type { Sudo } from "../utils.js";
 import {
   assertNotAsync,
+  assertNotPromise,
   defaultValueToValueNode,
   findVariableNamesUsed,
   hasItemPlan,
@@ -1439,18 +1440,23 @@ export class OperationPlan {
       }
     } else if (isScalarType(nullableFieldType)) {
       const scalarPlanResolver = nullableFieldType.extensions?.grafast?.plan;
-      assertNotAsync(scalarPlanResolver, `${nullableFieldType.name}.plan`);
+      const fnDescription = `${nullableFieldType.name}.plan`;
+      assertNotAsync(scalarPlanResolver, fnDescription);
       const $sideEffect = parentLayerPlan.latestSideEffectStep;
       try {
         const $leaf =
           typeof scalarPlanResolver === "function"
-            ? withGlobalLayerPlan(
-                parentLayerPlan,
-                polymorphicPaths,
+            ? assertNotPromise(
+                withGlobalLayerPlan(
+                  parentLayerPlan,
+                  polymorphicPaths,
+                  scalarPlanResolver,
+                  null,
+                  $step,
+                  this.scalarPlanInfo,
+                ),
                 scalarPlanResolver,
-                null,
-                $step,
-                this.scalarPlanInfo,
+                fnDescription,
               )
             : $step;