@mintlify/cli 4.0.1629 → 4.0.1631

package/bin/cli.js

@@ -8,7 +8,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { buildGraph, getBrokenExternalLinks } from '@mintlify/link-rot';
+import { buildGraph, getBrokenExternalLinks, getFilesLinksDependOn } from '@mintlify/link-rot';
 import { addLog, dev, exportSite, validateBuild, ErrorLog, SpinnerLog, SuccessLog, Logs, clearLogs, BrokenLinksLog, WarningLog, } from '@mintlify/previewing';
 import { thumbnailOverrideSchema } from '@mintlify/validation';
 import { render, Text } from 'ink';
@@ -307,10 +307,26 @@ export const cli = ({ packageName = 'mint' }) => {
             const sourceFiles = fileArgs.length > 0
                 ? new Set((yield resolveExplicitFiles(CMD_EXEC_PATH, fileArgs)).map((file) => path.relative(docsRoot, path.resolve(CMD_EXEC_PATH, file))))
                 : undefined;
+            // One unparseable page shouldn't hide every other result: report it and keep going
+            const parseErrorsByFile = new Map();
             const graph = yield buildGraph(docsRoot, {
                 checkSnippets: argv['check-snippets'],
+                onParseError: (filePath, error) => parseErrorsByFile.set(filePath, error.message),
             });
             graph.precomputeFileResolutions();
+            // With --files, a failure elsewhere only matters when a selected link needs that
+            // file: an anchor pointing at it, or a snippet the selected page renders
+            const dependencies = sourceFiles
+                ? getFilesLinksDependOn(graph, sourceFiles, {
+                    checkAnchors: argv['check-anchors'],
+                    checkSnippets: argv['check-snippets'],
+                })
+                : undefined;
+            const parseErrors = [...parseErrorsByFile]
+                .filter(([filePath]) => !sourceFiles ||
+                sourceFiles.has(filePath) ||
+                (dependencies === null || dependencies === void 0 ? void 0 : dependencies.has(filePath.split(path.sep).join('/'))))
+                .map(([, message]) => message);
             const brokenInternalLinks = graph
                 .getBrokenInternalLinks({
                 checkAnchors: argv['check-anchors'],
@@ -361,13 +377,25 @@ export const cli = ({ packageName = 'mint' }) => {
                     }
                 }
             }
-            if (Object.keys(brokenLinksByFile).length === 0) {
+            const hasBrokenLinks = Object.keys(brokenLinksByFile).length > 0;
+            if (!hasBrokenLinks && parseErrors.length === 0) {
                 clearLogs();
                 addLog(_jsx(SuccessLog, { message: "no broken links found" }));
                 yield terminate(0);
             }
             clearLogs();
-            addLog(_jsx(BrokenLinksLog, { brokenLinksByFile: brokenLinksByFile }));
+            if (hasBrokenLinks) {
+                addLog(_jsx(BrokenLinksLog, { brokenLinksByFile: brokenLinksByFile }));
+            }
+            else {
+                addLog(_jsx(SuccessLog, { message: "no broken links found in the files that parsed" }));
+            }
+            for (const message of parseErrors) {
+                addLog(_jsx(ErrorLog, { message: message }));
+            }
+            if (parseErrors.length > 0) {
+                addLog(_jsx(WarningLog, { message: `skipped ${parseErrors.length} file(s) that could not be parsed; links in them were not checked` }));
+            }
         }
         catch (err) {
             addLog(_jsx(ErrorLog, { message: err instanceof Error ? err.message : 'unknown error' }));