don't even list a source value if there wasn't one, just like it's no…

…t in the map
yaacovCR · Sep 28, 2024 · c3e220d · c3e220d
1 parent c81c719
commit c3e220d
Showing 1 changed file with 6 additions and 10 deletions.
diff --git a/src/execution/values.ts b/src/execution/values.ts
@@ -35,7 +35,7 @@ export interface VariableValues {
 
 interface VariableValueSource {
   readonly signature: GraphQLVariableSignature;
-  readonly value: unknown;
+  readonly value?: unknown;
 }
 
 type VariableValuesOrErrors =
@@ -104,10 +104,7 @@ function coerceVariableValues(
     if (!Object.hasOwn(inputs, varName)) {
       const defaultValue = varSignature.defaultValue;
       if (defaultValue) {
-        sources[varName] = {
-          signature: varSignature,
-          value: undefined,
-        };
+        sources[varName] = { signature: varSignature };
         coerced[varName] = coerceDefaultValue(defaultValue, varType);
       } else if (isNonNullType(varType)) {
         const varTypeStr = inspect(varType);
@@ -118,10 +115,7 @@ function coerceVariableValues(
           ),
         );
       } else {
-        sources[varName] = {
-          signature: varSignature,
-          value: undefined,
-        };
+        sources[varName] = { signature: varSignature };
       }
       continue;
     }
@@ -244,7 +238,9 @@ export function experimentalGetArgumentValues(
 
     if (valueNode.kind === Kind.VARIABLE) {
       const variableName = valueNode.name.value;
-      const scopedVariableValues = fragmentVariablesValues?.sources[variableName]
+      const scopedVariableValues = fragmentVariablesValues?.sources[
+        variableName
+      ]
         ? fragmentVariablesValues
         : variableValues;
       if (