better-design 0.35.2 → 0.35.5

package/dist/index.js

@@ -178,8 +178,8 @@ function appSupport() {
       return configHome();
   }
 }
-function dirExists(...segments) {
-  return existsSync(path.join(...segments));
+function dirExists(...segments2) {
+  return existsSync(path.join(...segments2));
 }
 function onPath(bin) {
   const dirs = (process.env.PATH ?? "").split(path.delimiter);
@@ -28889,9 +28889,9 @@ var init_schema3 = __esm({
 });
 
 // ../shared/src/mcp/feature-map/build.ts
-function toRoute(segments) {
+function toRoute(segments2) {
   const out = [];
-  for (const seg of segments) {
+  for (const seg of segments2) {
     if (/^\(.*\)$/.test(seg) || seg.startsWith("@")) continue;
     if (seg.startsWith("_")) return null;
     const param = /^\[{1,2}(?:\.\.\.)?([^\]]+)\]{1,2}$/.exec(seg);
@@ -29613,7 +29613,7 @@ var MCP_IMPLEMENTATION_VERSION;
 var init_version2 = __esm({
   "../shared/src/mcp/version.ts"() {
     "use strict";
-    MCP_IMPLEMENTATION_VERSION = "0.35.2";
+    MCP_IMPLEMENTATION_VERSION = "0.35.5";
   }
 });
 
@@ -30550,7 +30550,10 @@ function maskTemplateImplementation(content, isMdx) {
   const mask = (value) => value.replace(/[^\r\n]/g, " ");
   let visible = content.replace(/^---\s*$[\s\S]*?^---\s*$/m, mask);
   if (isMdx) visible = maskMdxEsm(visible, mask);
-  visible = visible.replace(/<!--[\s\S]*?-->|<(script|style)\b[^>]*>[\s\S]*?<\/\1\s*>/gi, mask);
+  visible = visible.replace(
+    /<!--[\s\S]*?-->|<(script|style|pre|code|blockquote)\b[^>]*>[\s\S]*?<\/\1\s*>/gi,
+    mask
+  );
   visible = visible.replace(/\/\*[\s\S]*?\*\//g, mask);
   visible = maskTags(visible, mask);
   visible = visible.replace(/\{\{([\s\S]*?)\}\}/g, (value, expression) => mask(value.slice(0, 2)) + revealRenderedExpression(expression, mask) + mask(value.slice(-2)));
@@ -30634,6 +30637,70 @@ var init_template_rules = __esm({
   }
 });
 
+// ../shared/src/lint/path-glob.ts
+function segments(path6) {
+  const parts = [];
+  for (const part of path6.replace(/\\/g, "/").split("/")) {
+    if (part === "..") parts.pop();
+    else if (part && part !== ".") parts.push(part);
+  }
+  return parts;
+}
+function segmentMatches(segment, pattern) {
+  let s = 0;
+  let p = 0;
+  let star = -1;
+  let mark = 0;
+  while (s < segment.length) {
+    if (pattern[p] === "?" || pattern[p] === segment[s]) {
+      s += 1;
+      p += 1;
+    } else if (pattern[p] === "*") {
+      star = p;
+      p += 1;
+      mark = s;
+    } else if (star !== -1) {
+      p = star + 1;
+      mark += 1;
+      s = mark;
+    } else {
+      return false;
+    }
+  }
+  while (pattern[p] === "*") p += 1;
+  return p === pattern.length;
+}
+function matchesPathGlob(path6, glob) {
+  if (glob.length > MAX_GLOB_LENGTH || path6.length > MAX_MATCH_PATH_LENGTH) return false;
+  const parts = segments(path6);
+  let reachable = Array(parts.length + 1).fill(false);
+  reachable[0] = true;
+  for (const segment of segments(glob)) {
+    const next = reachable.map(() => false);
+    if (segment === "**") {
+      let seen = false;
+      reachable.forEach((value, j) => {
+        seen = seen || value;
+        next[j] = seen;
+      });
+    } else {
+      for (let j = 1; j <= parts.length; j += 1) {
+        next[j] = reachable[j - 1] && segmentMatches(parts[j - 1], segment);
+      }
+    }
+    reachable = next;
+  }
+  return reachable[parts.length];
+}
+var MAX_GLOB_LENGTH, MAX_MATCH_PATH_LENGTH;
+var init_path_glob = __esm({
+  "../shared/src/lint/path-glob.ts"() {
+    "use strict";
+    MAX_GLOB_LENGTH = 200;
+    MAX_MATCH_PATH_LENGTH = 512;
+  }
+});
+
 // ../shared/src/lint/engine.ts
 import ts3 from "typescript";
 function severityFromFlag(flag, s) {
@@ -30741,7 +30808,13 @@ function elementHasOnlyIconChild(element) {
   if (!tagName) return false;
   return /icon$/i.test(tagName) || /^icon/i.test(tagName) || /^(Lucide|Phosphor|Tabler|Solar)/.test(tagName);
 }
-function lintFile(input, config) {
+function withElementExemptions(path6, config) {
+  const fromConfigFolder = config.pathsFrom ? `${config.pathsFrom}/${path6}` : path6;
+  const exempt = config.rules.elementExemptions?.some((exemption) => matchesPathGlob(fromConfigFolder, exemption.path));
+  return exempt ? { ...config, rules: { ...config.rules, elements: {} } } : config;
+}
+function lintFile(input, baseConfig) {
+  const config = withElementExemptions(input.path, baseConfig);
   if (/\.css$/i.test(input.path)) return lintStylesheet(input, config);
   if (/\.(html|vue|svelte|mdx)$/i.test(input.path)) {
     return [...lintTemplate(input, config), ...lintCodeRules(input)];
@@ -31420,9 +31493,9 @@ function collectFontExports(statement, loaderBindings, exports, reExports) {
 }
 function resolveImportedModule(importerPath, specifier, modules) {
   const directory = importerPath.split("/").slice(0, -1).join("/");
-  const segments = importerPath.split("/");
-  const srcIndex = segments.lastIndexOf("src");
-  const aliasRoot = srcIndex >= 0 ? segments.slice(0, srcIndex + 1).join("/") : /^(?:apps|packages)\//.test(importerPath) ? segments.slice(0, 2).join("/") : "src";
+  const segments2 = importerPath.split("/");
+  const srcIndex = segments2.lastIndexOf("src");
+  const aliasRoot = srcIndex >= 0 ? segments2.slice(0, srcIndex + 1).join("/") : /^(?:apps|packages)\//.test(importerPath) ? segments2.slice(0, 2).join("/") : "src";
   const bases = /^(?:@|~|#)\//.test(specifier) ? [`${aliasRoot}/${specifier.slice(2)}`, `src/${specifier.slice(2)}`, specifier.slice(2)] : specifier.startsWith(".") ? [`${directory}/${specifier}`] : [];
   for (const rawBase of bases) {
     const base = normalizeSourcePath(rawBase);
@@ -31439,13 +31512,13 @@ function resolveImportedModule(importerPath, specifier, modules) {
   return void 0;
 }
 function normalizeSourcePath(path6) {
-  const segments = [];
+  const segments2 = [];
   for (const segment of path6.replace(/\\/g, "/").split("/")) {
     if (!segment || segment === ".") continue;
-    if (segment === "..") segments.pop();
-    else segments.push(segment);
+    if (segment === "..") segments2.pop();
+    else segments2.push(segment);
   }
-  return segments.join("/");
+  return segments2.join("/");
 }
 function isInsideDocumentShellClass(node) {
   let current = node.parent;
@@ -31623,6 +31696,7 @@ var init_engine = __esm({
     init_code_rules();
     init_length_rules();
     init_template_rules();
+    init_path_glob();
     NATIVE_HTML_TAGS = /* @__PURE__ */ new Set([
       "a",
       "abbr",
@@ -31784,6 +31858,10 @@ var init_engine = __esm({
 
 // ../shared/src/lint/config.ts
 import { z as z31 } from "zod";
+function parseReviewConfigOverride(value) {
+  const parsed = reviewConfigOverrideSchema.safeParse(value);
+  return parsed.success ? parsed.data : null;
+}
 function mergeReviewConfig(override) {
   if (!override) return DEFAULT_REVIEW_CONFIG;
   return {
@@ -31810,6 +31888,7 @@ var DEFAULT_VISUAL_MODEL_SCREENSHOTS, DEFAULT_ELEMENTS, DEFAULT_REVIEW_CONFIG, r
 var init_config = __esm({
   "../shared/src/lint/config.ts"() {
     "use strict";
+    init_path_glob();
     DEFAULT_VISUAL_MODEL_SCREENSHOTS = 24;
     DEFAULT_ELEMENTS = {
       "app-shell": { use: "Sidebar", from: "@/components/ui/sidebar" },
@@ -31874,6 +31953,10 @@ var init_config = __esm({
       visual: visualReviewConfigSchema.optional(),
       rules: z31.object({
         elements: z31.record(z31.string(), z31.object({ use: z31.string(), from: z31.string() })).optional(),
+        elementExemptions: z31.array(z31.object({
+          path: z31.string().min(1).max(MAX_GLOB_LENGTH),
+          reason: z31.string().trim().min(1).max(300)
+        })).max(20).optional(),
         tokens: z31.record(z31.string(), z31.string()).optional(),
         wcag: z31.object({
           enabled: z31.boolean().optional(),
@@ -31947,12 +32030,13 @@ var init_lint = __esm({
     init_engine();
     init_config();
     init_report();
+    init_path_glob();
   }
 });
 
 // src/review/config-loader.ts
 import { readFileSync as readFileSync3 } from "fs";
-import { dirname, join, parse as parse2 } from "path";
+import { dirname, join, parse as parse2, relative } from "path";
 function isNotFound(cause) {
   return cause?.code === "ENOENT" || cause?.code === "ENOTDIR";
 }
@@ -31972,13 +32056,18 @@ function loadReviewConfig(cwd) {
       }
     }
     if (raw !== null) {
+      let parsed;
       try {
-        return mergeReviewConfig(JSON.parse(raw));
+        parsed = JSON.parse(raw);
       } catch (error) {
         throw new Error(
           `Could not parse ${path6}: ${error instanceof Error ? error.message : String(error)}`
         );
       }
+      const override = parseReviewConfigOverride(parsed);
+      if (!override) throw new Error(`${path6} does not match the review config schema.`);
+      const pathsFrom = relative(dir, cwd).split("\\").join("/");
+      return { ...mergeReviewConfig(override), ...pathsFrom && { pathsFrom } };
     }
     if (dir === root) return mergeReviewConfig();
     const parent = dirname(dir);
@@ -31998,7 +32087,7 @@ var init_config_loader = __esm({
 // src/review/file-sources.ts
 import { execFileSync } from "child_process";
 import { readdirSync, statSync } from "fs";
-import { isAbsolute, join as join2, relative } from "path";
+import { isAbsolute, join as join2, relative as relative2 } from "path";
 function isReviewablePath(path6) {
   return REVIEWABLE_RE.test(path6);
 }
@@ -32020,7 +32109,7 @@ function expandPath(cwd, arg) {
   return files;
 }
 function filesFromArgs(cwd, args) {
-  return [...new Set(args.flatMap((arg) => expandPath(cwd, arg)))].map((abs) => relative(cwd, abs));
+  return [...new Set(args.flatMap((arg) => expandPath(cwd, arg)))].map((abs) => relative2(cwd, abs));
 }
 function git(cwd, args) {
   const out = execFileSync("git", args, { cwd, encoding: "utf8" });
@@ -32052,7 +32141,7 @@ __export(hook_exports, {
   runHook: () => runHook
 });
 import { readFileSync as readFileSync4 } from "fs";
-import { isAbsolute as isAbsolute2, resolve } from "path";
+import { isAbsolute as isAbsolute2, relative as relative3, resolve } from "path";
 async function runHook(stdin = process.stdin) {
   let payload;
   try {
@@ -32075,7 +32164,7 @@ async function runHook(stdin = process.stdin) {
     return 0;
   }
   try {
-    const findings = lintFile({ path: filePath, content }, loadReviewConfig(cwd));
+    const findings = lintFile({ path: relative3(cwd, absolutePath), content }, loadReviewConfig(cwd));
     const actionable = findings.filter(
       (f) => f.severity === "critical" || f.severity === "serious"
     );