@graphql-inspector/action 6.2.0 → 6.2.1

package/cjs/action/src/checks.js

@@ -23,7 +23,7 @@ async function updateCheckRun(octokit, checkId, { conclusion, output }) {
         },
     });
     try {
-        await Promise.all(batches.map(async (chunk) => {
+        for (const chunk of batches) {
             await octokit.rest.checks.update({
                 check_run_id: checkId,
                 ...github.context.repo,
@@ -34,7 +34,7 @@ async function updateCheckRun(octokit, checkId, { conclusion, output }) {
                 },
             });
             core.info(`annotations sent (${chunk.length})`);
-        }));
+        }
     }
     catch (error) {
         core.error(`failed to send annotations: ${error}`);

package/cjs/checks.js

@@ -23,7 +23,7 @@ async function updateCheckRun(octokit, checkId, { conclusion, output }) {
         },
     });
     try {
-        await Promise.all(batches.map(async (chunk) => {
+        for (const chunk of batches) {
             await octokit.rest.checks.update({
                 check_run_id: checkId,
                 ...github.context.repo,
@@ -34,7 +34,7 @@ async function updateCheckRun(octokit, checkId, { conclusion, output }) {
                 },
             });
             core.info(`annotations sent (${chunk.length})`);
-        }));
+        }
     }
     catch (error) {
         core.error(`failed to send annotations: ${error}`);

package/cjs/commands/diff/src/index.js

@@ -15,13 +15,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.handler = handler;
 const fs_1 = require("fs");

package/cjs/core/src/validate/index.js

@@ -34,11 +34,8 @@ function validate(schema, sources, options) {
         }
     }
     for (const fragment of fragments) {
-        const depends = extractFragments((0, graphql_1.print)(fragment.node));
-        if (depends) {
-            for (const name of depends) {
-                graph.addDependency(fragment.node.name.value, name);
-            }
+        for (const name of extractFragments(fragment.node)) {
+            graph.addDependency(fragment.node.name.value, name);
         }
     }
     for (const doc of documents
@@ -48,7 +45,7 @@ function validate(schema, sources, options) {
             kind: graphql_1.Kind.DOCUMENT,
             definitions: doc.operations.map(d => d.node),
         };
-        const extractedFragments = (extractFragments((0, graphql_1.print)(docWithOperations)) || [])
+        const extractedFragments = extractFragments(docWithOperations)
             // resolve all nested fragments
             .map(fragmentName => resolveFragment(graph.getNodeData(fragmentName), graph))
             // flatten arrays
@@ -157,7 +154,13 @@ function resolveFragment(fragment, graph) {
         .reduce((list, current) => [...list, ...resolveFragment(graph.getNodeData(current), graph)], [fragment]);
 }
 function extractFragments(document) {
-    return (document.match(/[.]{3}[a-z0-9_]+\b/gi) || []).map(name => name.replace('...', ''));
+    const fragmentNames = [];
+    (0, graphql_1.visit)(document, {
+        FragmentSpread(node) {
+            fragmentNames.push(node.name.value);
+        },
+    });
+    return fragmentNames;
 }
 function sumLengths(...arrays) {
     return arrays.reduce((sum, { length }) => sum + length, 0);

package/esm/action/src/checks.js

@@ -19,7 +19,7 @@ export async function updateCheckRun(octokit, checkId, { conclusion, output }) {
         },
     });
     try {
-        await Promise.all(batches.map(async (chunk) => {
+        for (const chunk of batches) {
             await octokit.rest.checks.update({
                 check_run_id: checkId,
                 ...github.context.repo,
@@ -30,7 +30,7 @@ export async function updateCheckRun(octokit, checkId, { conclusion, output }) {
                 },
             });
             core.info(`annotations sent (${chunk.length})`);
-        }));
+        }
     }
     catch (error) {
         core.error(`failed to send annotations: ${error}`);

package/esm/checks.js

@@ -19,7 +19,7 @@ export async function updateCheckRun(octokit, checkId, { conclusion, output }) {
         },
     });
     try {
-        await Promise.all(batches.map(async (chunk) => {
+        for (const chunk of batches) {
             await octokit.rest.checks.update({
                 check_run_id: checkId,
                 ...github.context.repo,
@@ -30,7 +30,7 @@ export async function updateCheckRun(octokit, checkId, { conclusion, output }) {
                 },
             });
             core.info(`annotations sent (${chunk.length})`);
-        }));
+        }
     }
     catch (error) {
         core.error(`failed to send annotations: ${error}`);

package/esm/core/src/validate/index.js

@@ -1,5 +1,5 @@
 import { DepGraph } from 'dependency-graph';
-import { GraphQLError, Kind, print, validate as validateDocument, } from 'graphql';
+import { GraphQLError, Kind, print, validate as validateDocument, visit, } from 'graphql';
 import { readDocument } from '../ast/document.js';
 import { transformDocumentWithApollo, transformSchemaWithApollo } from '../utils/apollo.js';
 import { findDeprecatedUsages } from '../utils/graphql.js';
@@ -31,11 +31,8 @@ export function validate(schema, sources, options) {
         }
     }
     for (const fragment of fragments) {
-        const depends = extractFragments(print(fragment.node));
-        if (depends) {
-            for (const name of depends) {
-                graph.addDependency(fragment.node.name.value, name);
-            }
+        for (const name of extractFragments(fragment.node)) {
+            graph.addDependency(fragment.node.name.value, name);
         }
     }
     for (const doc of documents
@@ -45,7 +42,7 @@ export function validate(schema, sources, options) {
             kind: Kind.DOCUMENT,
             definitions: doc.operations.map(d => d.node),
         };
-        const extractedFragments = (extractFragments(print(docWithOperations)) || [])
+        const extractedFragments = extractFragments(docWithOperations)
             // resolve all nested fragments
             .map(fragmentName => resolveFragment(graph.getNodeData(fragmentName), graph))
             // flatten arrays
@@ -154,7 +151,13 @@ function resolveFragment(fragment, graph) {
         .reduce((list, current) => [...list, ...resolveFragment(graph.getNodeData(current), graph)], [fragment]);
 }
 function extractFragments(document) {
-    return (document.match(/[.]{3}[a-z0-9_]+\b/gi) || []).map(name => name.replace('...', ''));
+    const fragmentNames = [];
+    visit(document, {
+        FragmentSpread(node) {
+            fragmentNames.push(node.name.value);
+        },
+    });
+    return fragmentNames;
 }
 function sumLengths(...arrays) {
     return arrays.reduce((sum, { length }) => sum + length, 0);

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@graphql-inspector/action",
-  "version": "6.2.0",
+  "version": "6.2.1",
   "description": "GraphQL Inspector functionality for GitHub Actions",
   "sideEffects": false,
   "dependencies": {
     "@actions/core": "^1.10.1",
     "@actions/github": "^6.0.0",
     "@graphql-inspector/commands": "^7.0.0",
-    "@graphql-inspector/core": "^8.0.0",
+    "@graphql-inspector/core": "^8.0.1",
     "dataloader": "^2.2.2",
     "js-yaml": "^4.1.1",
     "probot": "^13.2.2",