@mintlify/cli 4.0.1630 → 4.0.1632

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mintlify/cli",
-  "version": "4.0.1630",
+  "version": "4.0.1632",
   "description": "The Mintlify CLI",
   "engines": {
     "node": ">=18.0.0"
@@ -45,12 +45,12 @@
   },
   "dependencies": {
     "@inquirer/prompts": "7.9.0",
-    "@mintlify/common": "1.0.1236",
-    "@mintlify/link-rot": "3.0.1450",
+    "@mintlify/common": "1.0.1237",
+    "@mintlify/link-rot": "3.0.1452",
     "@mintlify/models": "0.0.372",
-    "@mintlify/og": "0.0.34",
-    "@mintlify/prebuild": "1.0.1397",
-    "@mintlify/previewing": "4.0.1471",
+    "@mintlify/og": "0.0.35",
+    "@mintlify/prebuild": "1.0.1399",
+    "@mintlify/previewing": "4.0.1473",
     "@mintlify/validation": "0.1.909",
     "adm-zip": "0.6.0",
     "chalk": "5.2.0",
@@ -86,7 +86,7 @@
     "keytar": "7.9.0"
   },
   "devDependencies": {
-    "@mintlify/editor": "0.0.495",
+    "@mintlify/editor": "0.0.496",
     "@mintlify/ts-config": "2.0.2",
     "@tsconfig/recommended": "1.0.2",
     "@types/detect-port": "1.3.2",
@@ -105,5 +105,5 @@
     "vitest": "2.1.9",
     "vitest-mock-process": "1.0.4"
   },
-  "gitHead": "090fc326376b5ffda3ae2d81321ed7ede279359c"
+  "gitHead": "c84411a4bba219c08bc232df2ed90ce9d2655273"
 }

package/src/cli.tsx

@@ -1,4 +1,4 @@
-import { buildGraph, getBrokenExternalLinks } from '@mintlify/link-rot';
+import { buildGraph, getBrokenExternalLinks, getFilesLinksDependOn } from '@mintlify/link-rot';
 import {
   addLog,
   dev,
@@ -387,10 +387,29 @@ export const cli = ({ packageName = 'mint' }: { packageName?: string }) => {
                     )
                   )
                 : undefined;
+            // One unparseable page shouldn't hide every other result: report it and keep going
+            const parseErrorsByFile = new Map<string, string>();
             const graph = await 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?.has(filePath.split(path.sep).join('/'))
+              )
+              .map(([, message]) => message);
 
             const brokenInternalLinks = graph
               .getBrokenInternalLinks({
@@ -452,14 +471,29 @@ export const cli = ({ packageName = 'mint' }: { packageName?: string }) => {
               }
             }
 
-            if (Object.keys(brokenLinksByFile).length === 0) {
+            const hasBrokenLinks = Object.keys(brokenLinksByFile).length > 0;
+            if (!hasBrokenLinks && parseErrors.length === 0) {
               clearLogs();
               addLog(<SuccessLog message="no broken links found" />);
               await terminate(0);
             }
 
             clearLogs();
-            addLog(<BrokenLinksLog brokenLinksByFile={brokenLinksByFile} />);
+            if (hasBrokenLinks) {
+              addLog(<BrokenLinksLog brokenLinksByFile={brokenLinksByFile} />);
+            } else {
+              addLog(<SuccessLog message="no broken links found in the files that parsed" />);
+            }
+            for (const message of parseErrors) {
+              addLog(<ErrorLog message={message} />);
+            }
+            if (parseErrors.length > 0) {
+              addLog(
+                <WarningLog
+                  message={`skipped ${parseErrors.length} file(s) that could not be parsed; links in them were not checked`}
+                />
+              );
+            }
           } catch (err) {
             addLog(<ErrorLog message={err instanceof Error ? err.message : 'unknown error'} />);
             await terminate(1);