@mintlify/cli 4.0.1744 → 4.0.1746

package/bin/validate/rules/build.js

@@ -17,20 +17,28 @@ const IMPORT_TYPES = new Set([
 ]);
 const POSITION = /^:(\d+)(?::(\d+))?/;
 const QUOTED = /"([^"]+)"/;
-const toFinding = (site, { type, message }) => {
-    var _a;
-    const clean = stripVTControlCharacters(message).trim();
-    const importer = IMPORT_TYPES.has(type)
+const toFinding = (site, warning) => {
+    var _a, _b;
+    const clean = stripVTControlCharacters(warning.message).trim();
+    const importer = warning.file == null && IMPORT_TYPES.has(warning.type)
         ? IMPORTERS.map((pattern) => { var _a; return (_a = pattern.exec(clean)) === null || _a === void 0 ? void 0 : _a[1]; }).find(Boolean)
         : undefined;
     // Missing nav pages are always about the config, even when the page looks like a file
-    const match = importer || type === 'missing-nav-page' ? null : FILE_IN_MESSAGE.exec(clean);
+    const match = warning.file != null || importer || warning.type === 'missing-nav-page'
+        ? null
+        : FILE_IN_MESSAGE.exec(clean);
     const found = importer !== null && importer !== void 0 ? importer : match === null || match === void 0 ? void 0 : match[0];
-    const file = found ? normalizeRelativePath(toPosixPath(found)) : site.configFile;
+    const file = (_a = warning.file) !== null && _a !== void 0 ? _a : (found ? normalizeRelativePath(toPosixPath(found)) : site.configFile);
     const position = match ? POSITION.exec(clean.slice(match.index + match[0].length)) : null;
     // Config warnings quote the entry they're about, such as a navigation page
-    const quoted = file === site.configFile ? (_a = QUOTED.exec(clean)) === null || _a === void 0 ? void 0 : _a[0] : undefined;
-    return Object.assign(Object.assign(Object.assign({ file }, (position && Object.assign({ line: Number(position[1]) }, (position[2] && { column: Number(position[2]) })))), (quoted && site.locate(file, quoted))), { message: clean });
+    const quoted = warning.file == null && file === site.configFile ? (_b = QUOTED.exec(clean)) === null || _b === void 0 ? void 0 : _b[0] : undefined;
+    const located = warning.ref != null && warning.line == null
+        ? site.locate(file, JSON.stringify(warning.ref))
+        : quoted
+            ? site.locate(file, quoted)
+            : undefined;
+    return Object.assign(Object.assign(Object.assign({ file }, (warning.line != null
+        ? Object.assign({ line: warning.line }, (warning.column != null && { column: warning.column })) : position && Object.assign({ line: Number(position[1]) }, (position[2] && { column: Number(position[2]) })))), located), { message: clean });
 };
 /** Warning types some rule turns into diagnostics, so a strict-build failure is explained. */
 const COVERED_TYPES = new Set(['parse-error']);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mintlify/cli",
-  "version": "4.0.1744",
+  "version": "4.0.1746",
   "description": "The Mintlify CLI",
   "engines": {
     "node": ">=18.0.0"
@@ -45,12 +45,12 @@
   },
   "dependencies": {
     "@inquirer/prompts": "7.9.0",
-    "@mintlify/common": "1.0.1309",
-    "@mintlify/link-rot": "3.0.1540",
+    "@mintlify/common": "1.0.1310",
+    "@mintlify/link-rot": "3.0.1542",
     "@mintlify/models": "0.0.379",
-    "@mintlify/og": "0.0.108",
-    "@mintlify/prebuild": "1.0.1476",
-    "@mintlify/previewing": "4.0.1558",
+    "@mintlify/og": "0.0.109",
+    "@mintlify/prebuild": "1.0.1478",
+    "@mintlify/previewing": "4.0.1560",
     "@mintlify/validation": "0.1.944",
     "adm-zip": "0.6.0",
     "chalk": "5.2.0",
@@ -84,7 +84,7 @@
     "keytar": "7.9.0"
   },
   "devDependencies": {
-    "@mintlify/editor": "0.0.582",
+    "@mintlify/editor": "0.0.583",
     "@mintlify/ts-config": "2.0.2",
     "@tsconfig/recommended": "1.0.2",
     "@types/detect-port": "1.3.2",
@@ -103,5 +103,5 @@
     "vitest": "2.1.9",
     "vitest-mock-process": "1.0.4"
   },
-  "gitHead": "c32639354792b7678f023b42e6e85cfd1d942d42"
+  "gitHead": "942f3d5baac187dc0f64ed7234d98201e6703353"
 }

package/src/validate/rules/build.ts

@@ -27,25 +27,39 @@ const POSITION = /^:(\d+)(?::(\d+))?/;
 
 const QUOTED = /"([^"]+)"/;
 
-const toFinding = (site: Site, { type, message }: Warning): Finding => {
-  const clean = stripVTControlCharacters(message).trim();
-  const importer = IMPORT_TYPES.has(type)
-    ? IMPORTERS.map((pattern) => pattern.exec(clean)?.[1]).find(Boolean)
-    : undefined;
+const toFinding = (site: Site, warning: Warning): Finding => {
+  const clean = stripVTControlCharacters(warning.message).trim();
+  const importer =
+    warning.file == null && IMPORT_TYPES.has(warning.type)
+      ? IMPORTERS.map((pattern) => pattern.exec(clean)?.[1]).find(Boolean)
+      : undefined;
   // Missing nav pages are always about the config, even when the page looks like a file
-  const match = importer || type === 'missing-nav-page' ? null : FILE_IN_MESSAGE.exec(clean);
+  const match =
+    warning.file != null || importer || warning.type === 'missing-nav-page'
+      ? null
+      : FILE_IN_MESSAGE.exec(clean);
   const found = importer ?? match?.[0];
-  const file = found ? normalizeRelativePath(toPosixPath(found)) : site.configFile;
+  const file =
+    warning.file ?? (found ? normalizeRelativePath(toPosixPath(found)) : site.configFile);
   const position = match ? POSITION.exec(clean.slice(match.index + match[0].length)) : null;
   // Config warnings quote the entry they're about, such as a navigation page
-  const quoted = file === site.configFile ? QUOTED.exec(clean)?.[0] : undefined;
+  const quoted =
+    warning.file == null && file === site.configFile ? QUOTED.exec(clean)?.[0] : undefined;
+  const located =
+    warning.ref != null && warning.line == null
+      ? site.locate(file, JSON.stringify(warning.ref))
+      : quoted
+        ? site.locate(file, quoted)
+        : undefined;
   return {
     file,
-    ...(position && {
-      line: Number(position[1]),
-      ...(position[2] && { column: Number(position[2]) }),
-    }),
-    ...(quoted && site.locate(file, quoted)),
+    ...(warning.line != null
+      ? { line: warning.line, ...(warning.column != null && { column: warning.column }) }
+      : position && {
+          line: Number(position[1]),
+          ...(position[2] && { column: Number(position[2]) }),
+        }),
+    ...located,
     message: clean,
   };
 };