@graphql-tools/delegate 12.0.2 → 12.0.3-alpha-5b28bd1c74cecaeb5ba91bb4b2bf42f00aa84ff5

package/CHANGELOG.md

@@ -1,5 +1,64 @@
 # @graphql-tools/delegate
 
+## 12.0.3-alpha-5b28bd1c74cecaeb5ba91bb4b2bf42f00aa84ff5
+### Patch Changes
+
+
+
+- [#1835](https://github.com/graphql-hive/gateway/pull/1835) [`0028015`](https://github.com/graphql-hive/gateway/commit/00280152c16b87d5e9e83a122ce3477b4f9f5b66) Thanks [@ardatan](https://github.com/ardatan)! - Delegate variable values correctly;
+  
+  When delegating requests with variables that include nested arrays, ensure that null values are preserved and passed correctly to the subschema. This fix addresses issues where null values in nested arrays were not handled properly during delegation.
+  
+  Let's say we have the following schema;
+  
+  ```ts
+  makeExecutableSchema({
+      typeDefs: /* GraphQL */ `
+          type Query {
+              test(input: InputType!): [String!]
+          }
+          input InputType {
+              value: [String!]
+          }
+      `,
+      resolvers: {
+          Query: {
+              test: (_, args) => {
+                  // Returns the incoming variable value
+                  return args.input.value;
+              },
+          },
+      }
+  });
+  ```
+  
+  When delegating a query with a variable like:
+  
+  ```json
+  {
+      "query": "query Test($value: [String!]) { test(input: { value: $value } ) }",
+      "variables": { "value": null }
+  }
+  ```
+  
+  And the result was
+  ```json
+  {
+      "data": {
+          "test": []
+      }
+  }
+  ```
+  
+  But with this fix, the result will correctly be:
+  ```json
+  {
+      "data": {
+          "test": null
+      }
+  }
+  ```
+
 ## 12.0.2
 ### Patch Changes
 

package/dist/index.cjs

@@ -1806,10 +1806,10 @@ function createRequest({
           },
           type: argAst.type
         });
-        newVariables[varName] = projectArgumentValue(
-          argValue,
-          argInstance.type
-        );
+        const varValue = projectArgumentValue(argValue, argInstance.type);
+        if (varValue != null) {
+          newVariables[varName] = varValue;
+        }
         argNodes.push({
           kind: graphql.Kind.ARGUMENT,
           name: {
@@ -1883,6 +1883,9 @@ function createRequest({
   };
 }
 function projectArgumentValue(argValue, argType) {
+  if (argValue == null) {
+    return argValue;
+  }
   if (graphql.isNonNullType(argType)) {
     return projectArgumentValue(argValue, argType.ofType);
   }
@@ -1897,7 +1900,10 @@ function projectArgumentValue(argValue, argType) {
     for (const key in argValue) {
       const field = fields[key];
       if (field) {
-        projectedValue[key] = projectArgumentValue(argValue[key], field.type);
+        const varValue = projectArgumentValue(argValue[key], field.type);
+        if (varValue != null) {
+          projectedValue[key] = varValue;
+        }
       }
     }
     return projectedValue;

package/dist/index.js

@@ -1806,10 +1806,10 @@ function createRequest({
           },
           type: argAst.type
         });
-        newVariables[varName] = projectArgumentValue(
-          argValue,
-          argInstance.type
-        );
+        const varValue = projectArgumentValue(argValue, argInstance.type);
+        if (varValue != null) {
+          newVariables[varName] = varValue;
+        }
         argNodes.push({
           kind: Kind.ARGUMENT,
           name: {
@@ -1883,6 +1883,9 @@ function createRequest({
   };
 }
 function projectArgumentValue(argValue, argType) {
+  if (argValue == null) {
+    return argValue;
+  }
   if (isNonNullType(argType)) {
     return projectArgumentValue(argValue, argType.ofType);
   }
@@ -1897,7 +1900,10 @@ function projectArgumentValue(argValue, argType) {
     for (const key in argValue) {
       const field = fields[key];
       if (field) {
-        projectedValue[key] = projectArgumentValue(argValue[key], field.type);
+        const varValue = projectArgumentValue(argValue[key], field.type);
+        if (varValue != null) {
+          projectedValue[key] = varValue;
+        }
       }
     }
     return projectedValue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-tools/delegate",
-  "version": "12.0.2",
+  "version": "12.0.3-alpha-5b28bd1c74cecaeb5ba91bb4b2bf42f00aa84ff5",
   "type": "module",
   "description": "A set of utils for faster development of GraphQL tools",
   "repository": {