Fix type error with arguments being undefined (#2078)

* Fix type error with `arguments` being undefined * Use the Coalesce Nullish operator * add changeset Co-authored-by: Arda TANRIKULU <ardatanrikulu@gmail.com>
ardatan · Oct 27, 2020 · 294dedd · 294dedd
1 parent 895e2e7
commit 294dedd
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/.changeset/cool-ads-sin.md b/.changeset/cool-ads-sin.md
@@ -0,0 +1,5 @@
+---
+'@graphql-tools/delegate': patch
+---
+
+fix(delegate): Fix type error with `arguments` being undefined
diff --git a/.changeset/itchy-pillows-trade.md b/.changeset/itchy-pillows-trade.md
@@ -0,0 +1,5 @@
+---
+'@graphql-tools/url-loader': patch
+---
+
+fix(url-loader): fix typing mismatch
diff --git a/packages/delegate/src/transforms/AddArgumentsAsVariables.ts b/packages/delegate/src/transforms/AddArgumentsAsVariables.ts
@@ -83,7 +83,7 @@ function addVariablesToRootField(
 
     operation.selectionSet.selections.forEach((selection: SelectionNode) => {
       if (selection.kind === Kind.FIELD) {
-        const argumentNodes = selection.arguments;
+        const argumentNodes = selection.arguments ?? [];
         const argumentNodeMap: Record<string, ArgumentNode> = argumentNodes.reduce(
           (prev, argument) => ({
             ...prev,