jig-ui 0.24.0 → 0.26.0

package/dist/index.js

@@ -2098,6 +2098,139 @@ function isReaderText(file) {
   return hasExtension(file, STYLE_HOST_EXTENSIONS);
 }
 
+// src/check/length.ts
+var LENGTH_UNITS = [
+  "px",
+  "rem",
+  "em",
+  "pt",
+  "vh",
+  "vw",
+  "vmin",
+  "vmax",
+  "ch",
+  "ex"
+];
+var UNITS = LENGTH_UNITS.join("|");
+var LENGTH_RE = new RegExp(`^-?\\d*\\.?\\d+(?:${UNITS})$`, "i");
+var LENGTH_SCAN_RE = new RegExp(`(-?\\d*\\.?\\d+)(${UNITS})`, "gi");
+function isExcludedLength(value, unit) {
+  if (value === 0) return true;
+  return unit.toLowerCase() === "px" && (Math.abs(value) === 1 || Math.abs(value) === 2);
+}
+function hasHardCodedLength(value) {
+  LENGTH_SCAN_RE.lastIndex = 0;
+  let m;
+  while (m = LENGTH_SCAN_RE.exec(value)) {
+    if (!isExcludedLength(parseFloat(m[1]), m[2])) return true;
+  }
+  return false;
+}
+
+// src/check/tailwind.ts
+var CLASS_ATTR = /\b(?:class|className)\s*=\s*/g;
+function readBraced(source, open) {
+  let depth = 0;
+  for (let i = open; i < source.length; i++) {
+    const ch = source[i];
+    if (ch === '"' || ch === "'" || ch === "`") {
+      const quote = ch;
+      i++;
+      while (i < source.length && source[i] !== quote) {
+        if (source[i] === "\\") i++;
+        i++;
+      }
+      continue;
+    }
+    if (ch === "{") depth++;
+    else if (ch === "}" && --depth === 0) return source.slice(open + 1, i);
+  }
+  return null;
+}
+function classAttributeValues(source) {
+  const out = [];
+  const lineAt = (offset) => source.slice(0, offset).split("\n").length;
+  for (const m of source.matchAll(CLASS_ATTR)) {
+    const at = m.index + m[0].length;
+    const opener = source[at];
+    if (opener === '"' || opener === "'") {
+      const end = source.indexOf(opener, at + 1);
+      if (end !== -1) out.push({ classes: source.slice(at + 1, end), line: lineAt(at) });
+      continue;
+    }
+    if (opener !== "{") continue;
+    const raw = readBraced(source, at);
+    if (raw === null) continue;
+    const literals = [...raw.matchAll(/(["'`])([\s\S]*?)\1/g)].map(
+      (q) => q[2].replace(/\$\{[\s\S]*?\}/g, " ")
+    );
+    if (literals.length > 0) out.push({ classes: literals.join(" ").trim(), line: lineAt(at) });
+  }
+  return out;
+}
+var ARBITRARY = /(?:^|\s)(?:[\w-]+:)*([a-z][\w-]*)-\[([^\]\s]+)\]/gi;
+var COLOUR = /^(?:#[0-9a-f]{3,8}|(?:rgba?|hsla?|oklch|oklab|lab|lch|color)\()/i;
+var LENGTH_PARTS = /^(-?\d*\.?\d+)([a-z]+)$/i;
+function arbitraryValues(classes) {
+  const out = [];
+  for (const m of classes.matchAll(ARBITRARY)) {
+    const [, utility, rawValue] = m;
+    const value = rawValue.replace(/_/g, " ");
+    if (/var\(|--/.test(value)) continue;
+    if (COLOUR.test(value)) {
+      out.push({ utility, value, kind: "colour" });
+      continue;
+    }
+    if (!LENGTH_RE.test(value)) continue;
+    const parts = LENGTH_PARTS.exec(value);
+    if (isExcludedLength(parseFloat(parts[1]), parts[2])) continue;
+    out.push({ utility, value, kind: "length" });
+  }
+  return out;
+}
+var PALETTE = {
+  white: "#ffffff",
+  black: "#000000",
+  "gray-50": "#f9fafb",
+  "gray-100": "#f3f4f6",
+  "gray-200": "#e5e7eb",
+  "gray-300": "#d1d5db",
+  "gray-400": "#9ca3af",
+  "gray-500": "#6b7280",
+  "gray-600": "#4b5563",
+  "gray-700": "#374151",
+  "gray-800": "#1f2937",
+  "gray-900": "#111827",
+  "gray-950": "#030712",
+  "slate-50": "#f8fafc",
+  "slate-400": "#94a3b8",
+  "slate-500": "#64748b",
+  "slate-800": "#1e293b",
+  "slate-900": "#0f172a",
+  "slate-950": "#020617",
+  "zinc-400": "#a1a1aa",
+  "zinc-500": "#71717a",
+  "zinc-900": "#18181b",
+  "neutral-400": "#a3a3a3",
+  "neutral-500": "#737373",
+  "neutral-900": "#171717",
+  "stone-400": "#a8a29e",
+  "stone-500": "#78716c",
+  "stone-900": "#1c1917"
+};
+function palettePairs(classes) {
+  let background;
+  let foreground;
+  for (const token of classes.split(/\s+/)) {
+    const bare2 = token.replace(/^(?:[\w-]+:)*/, "");
+    const bg = /^bg-(.+)$/.exec(bare2);
+    const fg = /^text-(.+)$/.exec(bare2);
+    if (bg && PALETTE[bg[1]]) background = PALETTE[bg[1]];
+    if (fg && PALETTE[fg[1]]) foreground = PALETTE[fg[1]];
+  }
+  return background && foreground ? [{ background, foreground }] : [];
+}
+
 // src/check/run.ts
 import { readFileSync as readFileSync15 } from "fs";
 import { join as join19 } from "path";
@@ -2904,139 +3037,6 @@ var pureBlackWhite = {
   }
 };
 
-// src/check/length.ts
-var LENGTH_UNITS = [
-  "px",
-  "rem",
-  "em",
-  "pt",
-  "vh",
-  "vw",
-  "vmin",
-  "vmax",
-  "ch",
-  "ex"
-];
-var UNITS = LENGTH_UNITS.join("|");
-var LENGTH_RE = new RegExp(`^-?\\d*\\.?\\d+(?:${UNITS})$`, "i");
-var LENGTH_SCAN_RE = new RegExp(`(-?\\d*\\.?\\d+)(${UNITS})`, "gi");
-function isExcludedLength(value, unit) {
-  if (value === 0) return true;
-  return unit.toLowerCase() === "px" && (Math.abs(value) === 1 || Math.abs(value) === 2);
-}
-function hasHardCodedLength(value) {
-  LENGTH_SCAN_RE.lastIndex = 0;
-  let m;
-  while (m = LENGTH_SCAN_RE.exec(value)) {
-    if (!isExcludedLength(parseFloat(m[1]), m[2])) return true;
-  }
-  return false;
-}
-
-// src/check/tailwind.ts
-var CLASS_ATTR = /\b(?:class|className)\s*=\s*/g;
-function readBraced(source, open) {
-  let depth = 0;
-  for (let i = open; i < source.length; i++) {
-    const ch = source[i];
-    if (ch === '"' || ch === "'" || ch === "`") {
-      const quote = ch;
-      i++;
-      while (i < source.length && source[i] !== quote) {
-        if (source[i] === "\\") i++;
-        i++;
-      }
-      continue;
-    }
-    if (ch === "{") depth++;
-    else if (ch === "}" && --depth === 0) return source.slice(open + 1, i);
-  }
-  return null;
-}
-function classAttributeValues(source) {
-  const out = [];
-  const lineAt = (offset) => source.slice(0, offset).split("\n").length;
-  for (const m of source.matchAll(CLASS_ATTR)) {
-    const at = m.index + m[0].length;
-    const opener = source[at];
-    if (opener === '"' || opener === "'") {
-      const end = source.indexOf(opener, at + 1);
-      if (end !== -1) out.push({ classes: source.slice(at + 1, end), line: lineAt(at) });
-      continue;
-    }
-    if (opener !== "{") continue;
-    const raw = readBraced(source, at);
-    if (raw === null) continue;
-    const literals = [...raw.matchAll(/(["'`])([\s\S]*?)\1/g)].map(
-      (q) => q[2].replace(/\$\{[\s\S]*?\}/g, " ")
-    );
-    if (literals.length > 0) out.push({ classes: literals.join(" ").trim(), line: lineAt(at) });
-  }
-  return out;
-}
-var ARBITRARY = /(?:^|\s)(?:[\w-]+:)*([a-z][\w-]*)-\[([^\]\s]+)\]/gi;
-var COLOUR = /^(?:#[0-9a-f]{3,8}|(?:rgba?|hsla?|oklch|oklab|lab|lch|color)\()/i;
-var LENGTH_PARTS = /^(-?\d*\.?\d+)([a-z]+)$/i;
-function arbitraryValues(classes) {
-  const out = [];
-  for (const m of classes.matchAll(ARBITRARY)) {
-    const [, utility, rawValue] = m;
-    const value = rawValue.replace(/_/g, " ");
-    if (/var\(|--/.test(value)) continue;
-    if (COLOUR.test(value)) {
-      out.push({ utility, value, kind: "colour" });
-      continue;
-    }
-    if (!LENGTH_RE.test(value)) continue;
-    const parts = LENGTH_PARTS.exec(value);
-    if (isExcludedLength(parseFloat(parts[1]), parts[2])) continue;
-    out.push({ utility, value, kind: "length" });
-  }
-  return out;
-}
-var PALETTE = {
-  white: "#ffffff",
-  black: "#000000",
-  "gray-50": "#f9fafb",
-  "gray-100": "#f3f4f6",
-  "gray-200": "#e5e7eb",
-  "gray-300": "#d1d5db",
-  "gray-400": "#9ca3af",
-  "gray-500": "#6b7280",
-  "gray-600": "#4b5563",
-  "gray-700": "#374151",
-  "gray-800": "#1f2937",
-  "gray-900": "#111827",
-  "gray-950": "#030712",
-  "slate-50": "#f8fafc",
-  "slate-400": "#94a3b8",
-  "slate-500": "#64748b",
-  "slate-800": "#1e293b",
-  "slate-900": "#0f172a",
-  "slate-950": "#020617",
-  "zinc-400": "#a1a1aa",
-  "zinc-500": "#71717a",
-  "zinc-900": "#18181b",
-  "neutral-400": "#a3a3a3",
-  "neutral-500": "#737373",
-  "neutral-900": "#171717",
-  "stone-400": "#a8a29e",
-  "stone-500": "#78716c",
-  "stone-900": "#1c1917"
-};
-function palettePairs(classes) {
-  let background;
-  let foreground;
-  for (const token of classes.split(/\s+/)) {
-    const bare2 = token.replace(/^(?:[\w-]+:)*/, "");
-    const bg = /^bg-(.+)$/.exec(bare2);
-    const fg = /^text-(.+)$/.exec(bare2);
-    if (bg && PALETTE[bg[1]]) background = PALETTE[bg[1]];
-    if (fg && PALETTE[fg[1]]) foreground = PALETTE[fg[1]];
-  }
-  return background && foreground ? [{ background, foreground }] : [];
-}
-
 // src/check/detectors/contrast-floor.ts
 var FLOOR = 4.5;
 var LARGE_FLOOR = 3;
@@ -3631,7 +3631,7 @@ var EM_DASH = /—/;
 var TAG = /<\/?[A-Za-z][\w.:-]*(?:\s[^<>]*)?\/?>/g;
 function maskNonProse(source) {
   const blank3 = (m) => " ".repeat(m.length);
-  return source.replace(/^---\n[\s\S]*?\n---/, blank3).replace(/<script\b[\s\S]*?<\/script\s*>/gi, blank3).replace(/<style\b[\s\S]*?<\/style\s*>/gi, blank3).replace(/<!--[\s\S]*?-->/g, blank3).replace(/\/\*[\s\S]*?\*\//g, blank3).replace(/(^|[^:])\/\/[^\n]*/g, (m, p1) => p1 + " ".repeat(m.length - p1.length));
+  return source.replace(/^---\n[\s\S]*?\n---/, blank3).replace(/<script\b[\s\S]*?<\/script\s*>/gi, blank3).replace(/<style\b[\s\S]*?<\/style\s*>/gi, blank3).replace(/<(blockquote|q|pre|code|samp|kbd)\b[^>]*>[\s\S]*?<\/\1\s*>/gi, blank3).replace(/<!--[\s\S]*?-->/g, blank3).replace(/\/\*[\s\S]*?\*\//g, blank3).replace(/(^|[^:])\/\/[^\n]*/g, (m, p1) => p1 + " ".repeat(m.length - p1.length));
 }
 function markdownProse(source) {
   const blank3 = (m) => " ".repeat(m.length);
@@ -3914,6 +3914,8 @@ function firstGroup(match) {
   if (!value) return void 0;
   return { value, index: match.index ?? 0 };
 }
+var PREVIEW_TITLE = /property\s*=\s*["']og:title["']|(?:^|[\s{,])["']?(?:ogTitle|og:title)["']?\s*:|openGraph\s*:\s*\{[^}]*?\btitle\s*:/;
+var PREVIEW_DESCRIPTION = /property\s*=\s*["']og:description["']|(?:^|[\s{,])["']?(?:ogDescription|og:description)["']?\s*:|openGraph\s*:\s*\{[^}]*?\bdescription\s*:/;
 function readMetadata(source) {
   const htmlTitle = firstGroup(HTML_TITLE.exec(source));
   const title = htmlTitle ?? firstGroup(KEYED_TITLE.exec(source));
@@ -3931,6 +3933,9 @@ function readMetadata(source) {
     description: description && !/^\{|\$\{/.test(description.value) ? description : void 0,
     hasTitle: ANY_TITLE.test(source),
     hasDescription: ANY_DESCRIPTION.test(source),
+    hasPreviewTitle: PREVIEW_TITLE.test(source),
+    hasPreviewDescription: PREVIEW_DESCRIPTION.test(source),
+    hasPreview: PREVIEW_TITLE.test(source) && PREVIEW_DESCRIPTION.test(source),
     declares: DECLARES.test(source),
     computedTitle: COMPUTED_TITLE.test(source),
     noindex: NOINDEX.some((re) => re.test(source))
@@ -3943,6 +3948,7 @@ var TITLE_BUDGET = 60;
 var DESCRIPTION_BUDGET = 155;
 
 // src/check/detectors/metadata.ts
+var HEAD_FRAGMENT = /<svelte:head[\s>]|<head[\s>]|<Head[\s>]|<Helmet[\s>]|slot\s*=\s*["']head["']/;
 var INVENTED_DATE = /(?:lastModified|datePublished|dateModified|publishedTime|modifiedTime)\s*:\s*(?:new Date\(\s*\)|Date\.now\(\s*\))/;
 var metadata = {
   name: "metadata",
@@ -3976,9 +3982,19 @@ var metadata = {
       if (indexable && !found.hasDescription) {
         at("J-121", 0, "this page has no meta description \u2014 the search engine writes one from whatever text it finds first, usually the navigation");
       }
+      if (indexable && !found.hasPreview) {
+        const missing = [!found.hasPreviewTitle && "og:title", !found.hasPreviewDescription && "og:description"].filter(Boolean).join(" and ");
+        at("J-164", 0, found.hasPreviewTitle || found.hasPreviewDescription ? `this page has no ${missing} \u2014 a link to it, pasted into a message or a post, shows a card with that part guessed` : "this page has no link preview (og:title and og:description) \u2014 a link to it, pasted into a message or a post, shows a card built from whatever the page happens to carry");
+      }
       if (!indexable && !found.noindex) {
         at("J-123", 0, `this surface is ${ctx.mode}, which is what somebody reaches after signing in, and the page carries no noindex \u2014 robots.txt is public and advisory, and is not this`);
       }
+    } else {
+      const own = HEAD_FRAGMENT.test(raw) ? HTML_DESCRIPTION.exec(raw) ?? HTML_DESCRIPTION_REVERSED.exec(raw) : null;
+      const indexable = ctx.mode !== "product" && ctx.mode !== "operator";
+      if (indexable && own && found.description && !found.hasPreviewDescription) {
+        at("J-164", own.index, `this file writes the page's description but no og:description${found.hasPreviewTitle ? "" : " or og:title"} \u2014 a link to it, pasted into a message or a post, shows a card without the description written here`);
+      }
     }
     return findings.filter((f) => f.ruleId === ctx.ruleId);
   }
@@ -4252,7 +4268,7 @@ var HEADLINE_SELECTOR = /\bh1\b|\bh2\b|hero|display|headline|title/i;
 var SERIF = /font-family\s*:\s*([^;}]+)/i;
 function serifFamily(value) {
   const first = value.split(",")[0].trim().toLowerCase();
-  if (/sans|mono|system-ui|ui-sans|inter\b|helvetica|arial/.test(first)) return false;
+  if (/sans|mono|system-ui|ui-sans|-apple-system|blinkmacsystemfont|segoe|roboto|inter\b|helvetica|arial/.test(first)) return false;
   return /serif|georgia|garamond|playfair|fraunces|lora|merriweather|instrument serif|dm serif|cormorant|newsreader|times|baskerville|bodoni|caslon|libre caslon|source serif|pt serif|crimson/.test(value.toLowerCase());
 }
 var italicSerifDisplay = {
@@ -4837,6 +4853,9 @@ function auditTokenLayer(projectRoot) {
 }
 
 // src/commands/check.ts
+function checkExitCode(result) {
+  return result.hasError ? 1 : 0;
+}
 var LEGACY_INDEX_REL = join25(".jig", "rules.index.json");
 function resolveIndexPath(projectRoot) {
   const legacy = join25(projectRoot, LEGACY_INDEX_REL);
@@ -4913,7 +4932,9 @@ function check(opts) {
   const withStyles = files.filter((f) => {
     try {
       const src = readFileSync21(join25(opts.projectRoot, f), "utf8");
-      return hasExtension(f, CSS_EXTENSIONS) ? src.trim().length > 0 : maskNonStyleRegions(src, f).trim().length > 0;
+      if (hasExtension(f, CSS_EXTENSIONS)) return src.trim().length > 0;
+      if (maskNonStyleRegions(src, f).trim().length > 0) return true;
+      return isMarkupHost(f) && classAttributeValues(src).some((a) => a.classes.trim().length > 0);
     } catch {
       return false;
     }
@@ -6119,12 +6140,18 @@ var PROBE_SCRIPT = `(async () => {
   // the page's own copy around it still counts. So is a tool's output or a
   // command shown in <pre>, <code>, <samp> or <kbd>: jig-site's home page
   // quotes \`jig check\`'s real output, whose own wording has an em dash, and
-  // the probe blocked a page for words it had rightly not rewritten.
-  let ownText = text;
-  for (const q of document.querySelectorAll('blockquote, q, pre, code, samp, kbd')) {
-    const quoted = (q.innerText || '').trim();
-    if (quoted) ownText = ownText.split(quoted).join('\\n');
-  }
+  // the probe blocked a page for words it had rightly not rewritten. The
+  // quotations are hidden while the text is read, not cut out of it by string:
+  // an inline <code>A-01</code> earlier in the prose removed "A-01" from the
+  // output block below it, so the block no longer matched and its dash counted.
+  const quotedEls = [...document.querySelectorAll('blockquote, q, pre, code, samp, kbd')];
+  const hide = document.createElement('style');
+  hide.textContent = '[data-jig-quoted] { display: none !important; }';
+  for (const q of quotedEls) q.setAttribute('data-jig-quoted', '');
+  document.head.appendChild(hide);
+  const ownText = document.body.innerText || '';
+  hide.remove();
+  for (const q of quotedEls) q.removeAttribute('data-jig-quoted');
   const junk = [...new Set(text.match(/\\$\\{|\\{\\{|\\bundefined\\b|\\bNaN\\b|\\[object Object\\]/g) || [])];
   const unresolved = new Set();
   for (const sheet of document.styleSheets) {
@@ -6294,6 +6321,30 @@ var PROBE_SCRIPT = `(async () => {
     fauxSeen.add(key);
     fauxFaces.push({ family: first.trim().replace(/^["']|["']$/g, ''), weight, style: italic ? 'italic' : 'normal', text: (el.innerText || '').trim().slice(0, 40) });
   }
+  // M-01: \`editorial\` allows no box that scrolls sideways on a phone; a code
+  // block wraps and a wide table stacks (L-01, step 6). In a control run a
+  // spec said the code would wrap and the built page scrolled it, and nothing
+  // measured the box: the page itself was not wider than the screen.
+  const sidewaysBoxes = [];
+  for (const el of [...document.body.querySelectorAll('*')].slice(0, 4000)) {
+    if (sidewaysBoxes.length >= 5) break;
+    const ox = getComputedStyle(el).overflowX;
+    if ((ox !== 'auto' && ox !== 'scroll') || el.scrollWidth <= el.clientWidth + 1 || !vis(el)) continue;
+    sidewaysBoxes.push({ box: el.tagName.toLowerCase() + (el.classList[0] ? '.' + el.classList[0] : ''), text: (el.innerText || '').trim().replace(/\\s+/g, ' ').slice(0, 40), scrollWidth: el.scrollWidth, clientWidth: el.clientWidth });
+  }
+  // A table cell given its own display (flex, grid, block) while its table is
+  // still drawn as a table stops being a cell: it no longer takes its row's
+  // height, and its border no longer meets the row's. A team table set its
+  // name cells to flex, to line up a name and a tag, and every name drifted
+  // above its row. A stacked table (the table itself set to block) is left out.
+  const brokenCells = [];
+  for (const cell of [...document.querySelectorAll('td, th')].slice(0, 2000)) {
+    if (brokenCells.length >= 5) break;
+    const table = cell.closest('table');
+    const display = getComputedStyle(cell).display;
+    if (!table || getComputedStyle(table).display !== 'table' || display === 'table-cell' || display === 'none' || !vis(cell)) continue;
+    brokenCells.push({ cell: cell.tagName.toLowerCase() + (cell.classList[0] ? '.' + cell.classList[0] : ''), text: (cell.innerText || '').trim().replace(/\\s+/g, ' ').slice(0, 40), display });
+  }
   return JSON.stringify({
     jigProbe: ${PROBE_VERSION},
     url: location.href,
@@ -6301,6 +6352,8 @@ var PROBE_SCRIPT = `(async () => {
     scrollWidth: doc.scrollWidth,
     clientWidth: doc.clientWidth,
     sidewaysScroll: doc.scrollWidth > doc.clientWidth,
+    sidewaysBoxes,
+    brokenCells,
     bodyFont: getComputedStyle(document.body).fontFamily,
     defaultFont: getComputedStyle(document.body).fontFamily === defaultFont,
     unresolvedTokens: [...unresolved],
@@ -6317,6 +6370,7 @@ var PROBE_SCRIPT = `(async () => {
       canonical: (document.querySelector('link[rel=canonical]') || {}).href || '',
       robots: (document.querySelector('meta[name=robots]') || {}).content || '',
       ogTitle: (document.querySelector('meta[property="og:title"]') || {}).content || '',
+      ogDescription: (document.querySelector('meta[property="og:description"]') || {}).content || '',
       ogImage: (document.querySelector('meta[property="og:image"]') || {}).content || '',
     },
     contentWidth: Math.round(region.getBoundingClientRect().width),
@@ -6765,29 +6819,42 @@ function readProbes(projectRoot, dir, errors) {
 }
 var TITLE_BUDGET2 = 60;
 var DESCRIPTION_BUDGET2 = 155;
-function probeContradictions(probes, verdictOf, indexable = true) {
+function probeCheck(probes, verdictOf, indexable = true, mode) {
   const errors = [];
+  const failures = [];
   const clean = (id) => ["ok", "n/a"].includes(verdictOf(id) ?? "");
   const at = (p) => `probe-${p.width}.json`;
   for (const p of probes) {
-    if (p.sidewaysScroll && clean("D-115")) {
-      errors.push(`D-115 is "${verdictOf("D-115")}", but ${at(p)} measured the page ${p.scrollWidth}px wide in a ${p.clientWidth}px viewport \u2014 it scrolls sideways.`);
+    if (p.sidewaysScroll) {
+      if (clean("D-115")) {
+        errors.push(`D-115 is "${verdictOf("D-115")}", but ${at(p)} measured the page ${p.scrollWidth}px wide in a ${p.clientWidth}px viewport \u2014 it scrolls sideways.`);
+      } else {
+        failures.push(`${at(p)}: the page is ${p.scrollWidth}px wide in a ${p.clientWidth}px viewport, so the whole page scrolls sideways (D-115). Find what is wider than the screen and let it wrap, stack, or scroll inside its own box (L-01, step 6).`);
+      }
+    }
+    if (mode === "editorial" && p.width <= 480 && p.sidewaysBoxes?.length) {
+      const shown = p.sidewaysBoxes.map((b) => `${b.box}${b.text ? ` ("${b.text}")` : ""}, ${b.scrollWidth}px in ${b.clientWidth}px`).join("; ");
+      failures.push(`${at(p)}: ${p.sidewaysBoxes.length === 1 ? "a box scrolls" : `${p.sidewaysBoxes.length} boxes scroll`} sideways on a phone: ${shown}. \`editorial\` allows no scrolling region on mobile (M-01): a code block wraps its long lines, keeping its line breaks and indents, and a wide table stacks its rows (L-01, step 6).`);
+    }
+    if (p.brokenCells?.length) {
+      const shown = p.brokenCells.map((c) => `${c.cell}${c.text ? ` ("${c.text}")` : ""} is display: ${c.display}`).join("; ");
+      failures.push(`${at(p)}: ${p.brokenCells.length === 1 ? "a table cell has" : `${p.brokenCells.length} table cells have`} left the table: ${shown}. A cell given its own display stops taking its row's height, so its border and its text drift from the row's. Keep the cell a cell and put the flex on a wrapper inside it (P-06).`);
     }
     if (p.defaultFont) {
-      errors.push(`${at(p)}: the page renders in the browser's default font (${p.bodyFont ?? "unknown"}) \u2014 its styles are not applying. No review of this page can pass until they do.`);
+      failures.push(`${at(p)}: the page renders in the browser's default font (${p.bodyFont ?? "unknown"}) \u2014 its styles are not applying. No review of this page can pass until they do.`);
     }
     if (p.fauxFaces?.length) {
       const shown = p.fauxFaces.map((f) => `${f.family} ${f.weight}${f.style === "italic" ? " italic" : ""} ("${f.text}")`).join(", ");
-      errors.push(`${at(p)}: the page asks for ${shown}, and no loaded face covers ${p.fauxFaces.length === 1 ? "it" : "them"}, so the browser fakes the weight or slant from the nearest face it has. Load that weight, or use one the page already loads.`);
+      failures.push(`${at(p)}: the page asks for ${shown}, and no loaded face covers ${p.fauxFaces.length === 1 ? "it" : "them"}, so the browser fakes the weight or slant from the nearest face it has. Load that weight, or use one the page already loads.`);
     }
     if (p.unresolvedTokens.length) {
-      errors.push(`${at(p)}: ${p.unresolvedTokens.length} token(s) have no value in the browser (${p.unresolvedTokens.slice(0, 6).join(", ")}) \u2014 every property using them is dropped (H-117).`);
+      failures.push(`${at(p)}: ${p.unresolvedTokens.length} token(s) have no value in the browser (${p.unresolvedTokens.slice(0, 6).join(", ")}) \u2014 every property using them is dropped (H-117).`);
     }
     for (const inv of p.orderInversions ?? []) {
-      errors.push(`${at(p)}: "${inv.seenFirst}" is read first on screen but comes after "${inv.markupFirst}" in the markup. At this width the markup order is not the reading order (H-119) \u2014 reorder the document, or move it with CSS that leaves the order intact.`);
+      failures.push(`${at(p)}: "${inv.seenFirst}" is read first on screen but comes after "${inv.markupFirst}" in the markup. At this width the markup order is not the reading order (H-119) \u2014 reorder the document, or move it with CSS that leaves the order intact.`);
     }
     if (p.emDashes?.length) {
-      errors.push(`${at(p)}: the rendered page shows an em dash in ${p.emDashes.map((t) => `"${t}"`).join(", ")} (I-118) \u2014 use a full stop, a comma, a colon, or a second element.`);
+      failures.push(`${at(p)}: the rendered page shows an em dash in ${p.emDashes.map((t) => `"${t}"`).join(", ")} (I-118) \u2014 use a full stop, a comma, a colon, or a second element.`);
     }
     if (clean("B-106") && p.strandedCount) {
       const shown = (p.strandedWords ?? []).map((w) => `"${w.word}" (in "${w.text}\u2026")`).join(", ");
@@ -6801,17 +6868,21 @@ function probeContradictions(probes, verdictOf, indexable = true) {
     if (head && p.width === probes[0]?.width) {
       const noindex = /noindex/i.test(head.robots);
       if (indexable && !noindex) {
-        if (!head.title) errors.push(`${at(p)}: the page served no <title> (J-121) \u2014 a search result then shows a truncated URL.`);
-        else if (head.title.length > TITLE_BUDGET2) errors.push(`${at(p)}: the title served is ${head.title.length} characters, past the ${TITLE_BUDGET2} a search result shows (J-122): "${head.title}".`);
-        if (!head.description) errors.push(`${at(p)}: the page served no meta description (J-121) \u2014 the search engine writes one from whatever text it finds first, usually the navigation.`);
-        else if (head.description.length > DESCRIPTION_BUDGET2) errors.push(`${at(p)}: the description served is ${head.description.length} characters, past the ${DESCRIPTION_BUDGET2} (J-122).`);
+        if (!head.title) failures.push(`${at(p)}: the page served no <title> (J-121) \u2014 a search result then shows a truncated URL.`);
+        else if (head.title.length > TITLE_BUDGET2) failures.push(`${at(p)}: the title served is ${head.title.length} characters, past the ${TITLE_BUDGET2} a search result shows (J-122): "${head.title}".`);
+        if (!head.description) failures.push(`${at(p)}: the page served no meta description (J-121) \u2014 the search engine writes one from whatever text it finds first, usually the navigation.`);
+        else if (head.description.length > DESCRIPTION_BUDGET2) failures.push(`${at(p)}: the description served is ${head.description.length} characters, past the ${DESCRIPTION_BUDGET2} (J-122).`);
+        if (typeof head.ogDescription === "string" && (!head.ogTitle || !head.ogDescription)) {
+          const missing = [!head.ogTitle && "og:title", !head.ogDescription && "og:description"].filter(Boolean).join(" and ");
+          failures.push(`${at(p)}: the page served no ${missing} (J-164) \u2014 a link to it, pasted into a message or a post, shows a card built from whatever the page happens to carry.`);
+        }
       }
       if (!indexable && !noindex) {
-        errors.push(`${at(p)}: this page is not indexable, and the page served no noindex (J-123). robots.txt is public and advisory, and is not this.`);
+        failures.push(`${at(p)}: this page is not indexable, and the page served no noindex (J-123). robots.txt is public and advisory, and is not this.`);
       }
     }
     if (p.junkText.length) {
-      errors.push(`${at(p)}: the rendered text contains ${p.junkText.map((j) => `"${j}"`).join(", ")} \u2014 template code or a failed value is showing to readers.`);
+      failures.push(`${at(p)}: the rendered text contains ${p.junkText.map((j) => `"${j}"`).join(", ")} \u2014 template code or a failed value is showing to readers.`);
     }
   }
   const navClean = clean("P-14") || clean("E-61");
@@ -6842,7 +6913,7 @@ function probeContradictions(probes, verdictOf, indexable = true) {
       if (broken.length) errors.push(`P-14 is "${verdictOf("P-14")}", but ${at(phone)} operated the menu: ${broken.join("; ")}.`);
     }
   }
-  return errors;
+  return { contradictions: errors, failures };
 }
 
 // src/check/decisions.ts
@@ -7336,6 +7407,7 @@ function verifyVerdicts(opts) {
     const v = screenVerdicts.find((x) => typeof x.id === "string" && x.id.trim().toUpperCase() === id);
     return typeof v?.verdict === "string" ? v.verdict : void 0;
   };
+  const measured = [];
   let specFront = "";
   try {
     specFront = readText(join38(opts.projectRoot, ".jig", "specs", `${opts.surface}.spec.md`)).split(/^---\s*$/m)[1] ?? "";
@@ -7344,14 +7416,24 @@ function verifyVerdicts(opts) {
   if (specIndexableField(specFront) === "unreadable") {
     errors.push(`.jig/specs/${opts.surface}.spec.md: \`indexable:\` is neither true nor false, so this review cannot tell whether the page is meant to be found and does not guess. Write \`indexable: true\` or \`indexable: false\`, and put the reason in the spec's body.`);
   } else {
-    errors.push(...probeContradictions(probes, verdictOf, specIndexable(opts.projectRoot, opts.surface)));
+    const mode = /^\s*mode\s*:\s*["']?(\w+)/im.exec(specFront)?.[1]?.toLowerCase();
+    const probed = probeCheck(probes, verdictOf, specIndexable(opts.projectRoot, opts.surface), mode);
+    errors.push(...probed.contradictions);
+    const seen = /* @__PURE__ */ new Set();
+    for (const failure of probed.failures) {
+      const what = failure.replace(/^probe-\d+\.json: /, "");
+      if (!seen.has(what)) {
+        seen.add(what);
+        measured.push(failure);
+      }
+    }
   }
   if (screen.state === "ran" && !rendered) screen.state = "skipped";
   const decisions = checkDecisions(opts.projectRoot, dir, errors);
   const field = (a) => a.state === "ran" ? `ran:${a.judged}` : `${a.state}:${a.judged}${a.state === "incomplete" ? `/${a.total}` : ""}`;
-  const line = `JIG_VERDICTS: surface=${opts.surface} screen=${field(screen)} code=${field(code)} decisions=${field(decisions)} rendered=${rendered ? "yes" : "no"} findings=${screen.findings + code.findings + decisions.findings}${(screen.ruled ?? 0) + (code.ruled ?? 0) ? ` ruled=${(screen.ruled ?? 0) + (code.ruled ?? 0)}` : ""}`;
+  const line = `JIG_VERDICTS: surface=${opts.surface} screen=${field(screen)} code=${field(code)} decisions=${field(decisions)} rendered=${rendered ? "yes" : "no"} findings=${screen.findings + code.findings + decisions.findings}${(screen.ruled ?? 0) + (code.ruled ?? 0) ? ` ruled=${(screen.ruled ?? 0) + (code.ruled ?? 0)}` : ""}${measured.length ? ` measured=${measured.length}` : ""}`;
   const previous = previousFindings(opts.projectRoot, dir);
-  return { ok: errors.length === 0, errors, screen, code, decisions, rendered, line, ...previous ? { previous } : {} };
+  return { ok: errors.length === 0, errors, screen, code, decisions, rendered, measured, line, ...previous ? { previous } : {} };
 }
 function previousFindings(projectRoot, dir) {
   const git = (args) => execFileSync3("git", args, { cwd: projectRoot, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
@@ -7816,6 +7898,10 @@ function pageStatus(projectRoot, surface) {
   if (stale) return { surface, state: "reprobe", detail: `its page is as judged, but the styles or scripts it loads changed: re-probe it (\`jig verdicts ${surface} --reprobe\`)` };
   if (!v.ok) return { surface, state: "incomplete", detail: `its critique is not complete: ${v.errors[0]}` };
   const findings = v.screen.findings + v.code.findings + v.decisions.findings;
+  if (v.measured.length) {
+    const also = findings > 0 ? `, and ${findings} finding${findings === 1 ? "" : "s"} the owner has not ruled on` : "";
+    return { surface, state: "findings", detail: `the page fails ${v.measured.length} check${v.measured.length === 1 ? "" : "s"} the render probe measured (\`jig verdicts ${surface}\` lists them)${also}` };
+  }
   if (findings > 0) return { surface, state: "findings", detail: `${findings} finding${findings === 1 ? "" : "s"} the owner has not ruled on` };
   return { surface, state: "judged", detail: "judged as it stands, nothing open" };
 }
@@ -7879,7 +7965,15 @@ function ownerWordProblem(word, quoted, owner, where) {
 function specChecksum(body) {
   return createHash2("sha256").update(body.replace(/\r\n/g, "\n").replace(/^\s*(confirmed|mockup)\s*:.*$/gim, "")).digest("hex");
 }
-var mockupLine = (body) => /^\s*mockup\s*:\s*(.*)$/im.exec(body.split(/^---\s*$/m)[1] ?? "")?.[1]?.trim() ?? "";
+function mockupField(body) {
+  const raw = /^\s*mockup\s*:\s*(.*)$/im.exec(body.split(/^---\s*$/m)[1] ?? "")?.[1]?.trim() ?? "";
+  const double = /^"((?:[^"\\]|\\.)*)"\s*(?:#.*)?$/.exec(raw);
+  if (double) return double[1].replace(/\\(.)/g, "$1").trim();
+  const single = /^'((?:[^']|'')*)'\s*(?:#.*)?$/.exec(raw);
+  if (single) return single[1].replace(/''/g, "'").trim();
+  return raw;
+}
+var mockupLine = mockupField;
 function mockupWordProblems(spec, then, owner) {
   const now = mockupLine(spec.body);
   if (now === mockupLine(then)) return [];
@@ -8199,7 +8293,7 @@ function commandProblems(root, command, surface, start, owner = "") {
   }
   if (command === "mockup" && spec) {
     const front = spec.body.split(/^---\s*$/m)[1] ?? "";
-    const mockup = /^\s*mockup\s*:\s*(.+)$/im.exec(front)?.[1]?.trim() ?? "";
+    const mockup = mockupField(spec.body);
     if (/^pending/i.test(mockup) || !mockup) problems.push(`${spec.path}: \`mockup:\` is still pending. It records the user's own word \u2014 approved, or skipped with their reason.`);
     const at = /^\s*mockup_at\s*:\s*(.+)$/im.exec(front)?.[1]?.trim().replace(/^["']|["']$/g, "") ?? "";
     if (/^approved/i.test(mockup)) {
@@ -8212,7 +8306,7 @@ function commandProblems(root, command, surface, start, owner = "") {
   if (command === "tweak" && spec) problems.push(...tweakProblems(root, spec, owner, start));
   if ((command === "spec" || command === "make") && spec) {
     const front = spec.body.split(/^---\s*$/m)[1] ?? "";
-    const mockup = /^\s*mockup\s*:\s*(\S+)/im.exec(front)?.[1] ?? "";
+    const mockup = mockupField(spec.body);
     const at = /^\s*mockup_at\s*:\s*(.+)$/im.exec(front)?.[1]?.trim().replace(/^["']|["']$/g, "") ?? "";
     if (/^approved/i.test(mockup) && /\.html?$/i.test(at) && !/^https?:/i.test(at)) {
       const drift = approvedDrawingProblems(root, spec.body, at);
@@ -8233,7 +8327,7 @@ function commandProblems(root, command, surface, start, owner = "") {
   }
   if (command === "mockup" && spec) {
     const front = spec.body.split(/^---\s*$/m)[1] ?? "";
-    const mockup = /^\s*mockup\s*:\s*(\S+)/im.exec(front)?.[1] ?? "";
+    const mockup = mockupField(spec.body);
     const at = /^\s*mockup_at\s*:\s*(.+)$/im.exec(front)?.[1]?.trim().replace(/^["']|["']$/g, "") ?? "";
     if (/^approved/i.test(mockup) && at && !/^https?:/i.test(at) && existsSync27(join43(root, at))) {
       const changedAfter = drawingChangedAfterApproval(root, spec.path, at);
@@ -8361,8 +8455,8 @@ function tweakProblems(root, spec, owner = "", start) {
   const problems = [];
   const front = spec.body.split(/^---\s*$/m)[1] ?? "";
   if (!/^\s*confirmed\s*:\s*true\b/im.test(front)) problems.push(`${spec.path} is not confirmed. A tweak changes a page the owner has confirmed; an unconfirmed spec goes through \`spec\`.`);
-  const mockup = /^\s*mockup\s*:\s*(\S+)/im.exec(front)?.[1] ?? "";
-  if (!/^(approved|skipped)/i.test(mockup)) problems.push(`${spec.path}: \`mockup:\` is ${mockup || "empty"}. A tweak changes a page whose drawing the owner has approved (or skipped); take a new page through \`mockup\` and \`make\`.`);
+  const mockup = mockupField(spec.body);
+  if (!/^(approved|skipped)/i.test(mockup)) problems.push(`${spec.path}: \`mockup:\` is ${mockup.split(/\s/)[0] || "empty"}. A tweak changes a page whose drawing the owner has approved (or skipped); take a new page through \`mockup\` and \`make\`.`);
   else problems.push(...structureSinceApproval(root, spec, front));
   const surface = spec.slug;
   const record = readTweak(join43(root, ".jig", "critique", surface));
@@ -8536,6 +8630,13 @@ ${result.report}`);
 ` + shown.join("\n") + (v.errors.length > shown.length ? `
   \u2026 and ${v.errors.length - shown.length} more` : "")
         );
+      } else if (v.measured.length && command !== "critique") {
+        const shown = v.measured.slice(0, 6).map((e) => `  ${e}`);
+        problems.push(
+          `jig verdicts ${surface}: the page fails what the render probe measured. Fix the page, then re-take the probes (\`jig verdicts ${surface} --reprobe\`).
+` + shown.join("\n") + (v.measured.length > shown.length ? `
+  \u2026 and ${v.measured.length - shown.length} more` : "")
+        );
       }
     }
   }
@@ -8874,6 +8975,10 @@ program.command("verdicts").description("Verify a critique's verdict files and c
       console.error(`  The page changed after these probes. \`jig verdicts ${surface} --reprobe\` re-takes them on the page as it is now.`);
     }
     if (result.ok) console.log(`  Every rule in both passes has a verdict.`);
+    if (result.measured.length) {
+      console.log(`  The page fails ${result.measured.length} check(s) the render probe measured. They are findings on the page, for make or tweak to fix; a critique reports them and leaves the page as it is:`);
+      for (const failure of result.measured) console.log(`  ! ${failure}`);
+    }
     const since = result.decisions.since ?? [];
     if (since.length) console.log(`  ${since.length} decision(s) recorded after this critique, for the next one to judge: ${since.join(", ")}.`);
     const p = result.previous;
@@ -8894,7 +8999,7 @@ program.command("check").description("Check the repo against Jig's mechanical +
   // this as widening the RULES and attested a clean diff as a clean project.
   "scan every file in the repo, not just those changed since HEAD (same rules either way)",
   false
-).option("--ci", "mechanical bucket only; exits non-zero on any error, deterministic", false).option("--json", "emit findings as JSON", false).action((opts) => {
+).option("--ci", "mechanical bucket only, so the run is deterministic", false).option("--json", "emit findings as JSON", false).action((opts) => {
   const projectRoot = findProjectRoot(process.cwd());
   try {
     const result = check({
@@ -8905,7 +9010,8 @@ program.command("check").description("Check the repo against Jig's mechanical +
       ci: opts.ci
     });
     console.log(opts.json ? JSON.stringify(result.findings, null, 2) : result.report);
-    if (opts.ci && result.hasError) process.exit(1);
+    const code = checkExitCode(result);
+    if (code !== 0) process.exit(code);
   } catch (err) {
     console.error(err.message);
     process.exit(1);