fix(delegate): do not use nullable prefixes if fields don't match

ardatan · Apr 26, 2024 · af7be09 · af7be09
1 parent 3d5323d
commit af7be09
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/light-mayflies-knock.md b/.changeset/light-mayflies-knock.md
@@ -0,0 +1,5 @@
+---
+"@graphql-tools/delegate": patch
+---
+
+Hotfix: do not use nullable and nonNullable prefixes if field names don't match
diff --git a/packages/delegate/src/OverlappingAliasesTransform.ts b/packages/delegate/src/OverlappingAliasesTransform.ts
@@ -48,7 +48,7 @@ export class OverlappingAliasesTransform<TContext>
                                 seenNonNullable.add(fieldName);
                                 currentNullable = false;
                               }
-                              if (seenNullable.size && seenNonNullable.size) {
+                              if (seenNullable.has(fieldName) && seenNonNullable.has(fieldName)) {
                                 transformationContext[OverlappingAliases] = true;
                                 return {
                                   ...subSelection,