Fix patch duplicating schema definition nodes (#2937)

Author: jdolle

.changeset/few-beans-share.md

@@ -0,0 +1,5 @@
+---
+'@graphql-inspector/patch': patch
+---
+
+Fix duplicated schema definitions

packages/patch/__tests__/directives.test.ts

@@ -1,3 +1,6 @@
+import { parse, print } from 'graphql';
+import { Change } from '@graphql-inspector/core';
+import { errors, patch } from '../src/index.js';
 import {
   expectDiffAndPatchToMatch,
   expectDiffAndPatchToPass,
@@ -476,4 +479,25 @@ describe('repeat directives', () => {
     `;
     await expectDiffAndPatchToMatch(before, after);
   });
+
+  test('Schema Extensions can be used with directives', async () => {
+    const before = `
+      extend schema @link(url: "https://specs.apollo.dev/federation/v2.3", import: ["@key", "@external"])
+
+      type Query {
+        text: Boolean
+      }
+    `;
+
+    const changes: Change<any>[] = [];
+
+    expect(print(patch(parse(before), changes, { onError: errors.looseErrorHandler })))
+      .toMatchInlineSnapshot(`
+        "extend schema @link(url: "https://specs.apollo.dev/federation/v2.3", import: ["@key", "@external"])
+
+        type Query {
+          text: Boolean
+        }"
+      `);
+  });
 });

packages/patch/src/index.ts

@@ -530,7 +530,12 @@ export function patchCoordinatesAST(
     // filter out the non-definition nodes (e.g. field definitions)
     definitions: [
       ...schemaNodes,
-      ...Array.from(nodesByCoordinate.values()).filter(isDefinitionNode),
+      ...Array.from(nodesByCoordinate.values()).filter(
+        n =>
+          isDefinitionNode(n) &&
+          n.kind !== Kind.SCHEMA_EXTENSION &&
+          n.kind !== Kind.SCHEMA_DEFINITION,
+      ),
     ],
   };
 }