@domandigital/craft 0.20.4 → 0.21.0

package/dist/phone.d.ts

@@ -1,9 +1,3 @@
-interface SourceFile {
-    /** Repo-relative path. The extension decides how the file is read. */
-    path: string;
-    text: string;
-}
-
 /**
  * The in-page half of the phone-first gate.
  *
@@ -23,6 +17,12 @@ interface PhoneFloors {
     targetPx: number;
     fieldFontPx: number;
     textPx: number;
+    /**
+     * The height of a text field, select or textarea (portal standard rule 5:
+     * 48px inputs). Left out, 48. Below `targetPx` the field also fails the
+     * target check; this floor is what catches 44 to 47px.
+     */
+    fieldHeightPx?: number;
 }
 /**
  * What a measure can find. The first five are the 375px floors (DOM-775); the
@@ -44,6 +44,12 @@ declare function measurePhoneInPage(floors: PhoneFloors & {
     only?: PhoneCheck[];
 }): RawPhoneFinding[];
 
+interface SourceFile {
+    /** Repo-relative path. The extension decides how the file is read. */
+    path: string;
+    text: string;
+}
+
 /**
  * The judging half of the phone-first gate: pure text work, tested without a
  * browser or a disk.
@@ -62,6 +68,11 @@ declare function measurePhoneInPage(floors: PhoneFloors & {
  * the day it goes in while the existing ones stay listed in the open. The
  * baseline only shrinks: a count higher than what is found fails as well
  * ("lower the baseline"), so a fix cannot leave room for the next breach.
+ *
+ * Shrinking is also enforced against the base branch (DOM-1053): a baseline
+ * entry, or a higher count, that main's copy does not have fails, unless the
+ * entry carries a `reason` (a named exception: third-party UI, a
+ * destructive-only form).
  */
 
 interface StaticFinding {
@@ -86,18 +97,30 @@ interface MeasuredRoute {
 interface Baseline {
     /** Why the baseline exists and which issue burns it down. Free text, for a person. */
     note?: string;
+    /**
+     * `reason` makes an entry a named exception: it may be added, or raised,
+     * after the baseline went in. Absent on the faults that were there first.
+     */
     static: {
         file: string;
         count: number;
         found?: string[];
+        reason?: string;
     }[];
     measured: {
         route: string;
         check: PhoneCheck;
         element: string;
         count: number;
+        reason?: string;
     }[];
 }
+/** An entry is a named exception when its `reason` says something. */
+declare const hasReason: (entry: {
+    reason?: unknown;
+}) => boolean;
+/** A baseline file, read. Old files (no `reason`, no `note`) read as before; a `reason` that is not text is an error. */
+declare function parseBaseline(raw: unknown, where: string): Baseline;
 declare const EMPTY_BASELINE: Baseline;
 /** The reason in a `mobile-first-ok: <reason>` comment, "" for the marker with no reason, null for none. */
 declare function reasonOf(line: string | undefined): string | null;
@@ -106,7 +129,10 @@ declare function reasonOf(line: string | undefined): string | null;
  * `mobile-first-ok: <reason>` on the line or the line above, or the whole file
  * in `exceptions` with a reason. A marker with no reason is itself a finding.
  */
-declare function scanPhoneFirst(files: SourceFile[], exceptions?: FileException[]): {
+declare function scanPhoneFirst(files: SourceFile[], exceptions?: FileException[], options?: {
+    /** True for a file in an app whose tailwindcss is v3: it is also scanned for `/NN` opacity on a `var()` colour. */
+    tailwind3?: (path: string) => boolean;
+}): {
     findings: StaticFinding[];
     excused: number;
 };
@@ -128,7 +154,48 @@ interface Judgement {
 declare function judgeStatic(findings: StaticFinding[], baseline: Baseline): Judgement;
 declare function judgeMeasured(routes: MeasuredRoute[], baseline: Baseline): Judgement;
 /** The baseline that makes this run pass: what is there now, counted. */
-declare function writeBaseline(findings: StaticFinding[], routes: MeasuredRoute[], note?: string): Baseline;
+declare function writeBaseline(findings: StaticFinding[], routes: MeasuredRoute[], note?: string, previous?: Baseline): Baseline;
+/**
+ * Entries the baseline holds that the base branch's copy does not, or holds
+ * at a lower count, and that name no reason. The baseline lists the faults
+ * that were there when the gate arrived; a new breach is fixed where it is.
+ * An entry with a `reason` is a named exception and may be added or raised.
+ */
+declare function judgeGrowth(current: Baseline, base: Baseline, baseName?: string): string[];
+/** Whether the baseline holds anything a reason does not name: only then is the base branch's copy needed. */
+declare const hasUnexcusedEntries: (b: Baseline) => boolean;
+/**
+ * Page routes from the page files of a Next app directory, as paths relative
+ * to it (`portal/(shell)/invoices/[id]/page.tsx`). Route groups `(x)`, parallel
+ * slots `@x` and private folders `_x` are not part of the URL; an intercepting
+ * route `(.)x` is a second way into a page, not a page of its own.
+ */
+declare function routesFromPages(files: string[]): string[];
+/** Page routes from Next's `app-path-routes-manifest.json` (`{ "/portal/page": "/portal" }`). Route handlers and internal `_` routes are not pages. */
+declare function routesFromManifest(manifest: unknown): string[];
+interface ExcludedRoute {
+    route: string;
+    reason: string;
+}
+/** A route as the gate names it: no query, no hash, no trailing slash. */
+declare const cleanRoute: (route: string) => string;
+/** Whether a measured route (`/portal/projects/p1`, or the pattern itself) is a page route (`/portal/projects/[id]`). */
+declare function routeMatches(pattern: string, route: string): boolean;
+interface RouteJudgement {
+    /** Page routes nothing measured and no exclusion covers: the gate fails on these. */
+    missing: string[];
+    /** Problems with `excludeRoutes` itself: no reason, or names no page. */
+    problems: string[];
+    /** Pages measured or excluded with a reason. */
+    covered: number;
+}
+/**
+ * Every page route of the app is in the gate's route list. A measured route
+ * counts for the most specific page it matches, so `/portal/projects/new` does
+ * not cover `/portal/projects/[id]`. A page that only redirects measures
+ * nothing: it is excluded with that reason.
+ */
+declare function judgeRoutes(pages: string[], measuredRoutes: string[], excluded?: ExcludedRoute[]): RouteJudgement;
 
 /**
  * What "every viewport" means (STANDARD.md section 10, DOM-787): the ladder
@@ -263,8 +330,13 @@ declare const PHONE: {
     readonly isMobile: true;
     readonly hasTouch: true;
 };
-/** The house floors at phone width. Not per-repo settings. */
-declare const PHONE_FLOORS: PhoneFloors;
+/**
+ * The house floors at phone width. A site may raise `textPx`, or lower
+ * `fieldHeightPx` with a reason; they are not per-repo taste. The field height
+ * is portal standard rule 5 (48px inputs); until DOM-1053 the gate held fields
+ * to the 44px target floor only.
+ */
+declare const PHONE_FLOORS: Required<PhoneFloors>;
 /** Where `measurePhone` writes and `craft phone-first --measured` reads, unless told otherwise. */
 declare const DEFAULT_MEASURED_DIR = "phone-first/measured";
 /** A route as a file name: `/portal/invoices` is `portal-invoices.json`, `/` is `home.json`. */
@@ -309,7 +381,7 @@ interface MeasureViewportsOptions {
     scope?: "all" | "phone" | "wide";
     /** Override house thresholds. */
     thresholds?: Partial<Thresholds>;
-    /** Override the 12 / 16 / 44px floors. The defaults are the house standard; a site may set `textPx` higher. */
+    /** Override the 12 / 16 / 44px floors and the 48px field height (`fieldHeightPx`). The defaults are the house standard; a site may set `textPx` higher. */
     floors?: Partial<PhoneFloors>;
     /** The main action's selector: one for all routes, or per route with `*` as the fallback. Absent: not checked. */
     mainAction?: MainAction;
@@ -440,6 +512,8 @@ interface MeasurePhoneOptions {
     route: string;
     /** The folder to write `<route>.json` into. Default `phone-first/measured`, from the working directory. */
     out?: string;
+    /** Override the 12 / 16 / 44 / 48px floors (text, field text, target, field height). The defaults are the house standard. */
+    floors?: Partial<PhoneFloors>;
 }
 /**
  * Measure the page as it is now and write the result for `craft phone-first`.
@@ -448,4 +522,4 @@ interface MeasurePhoneOptions {
  */
 declare function measurePhone(page: MinimalPage, options: MeasurePhoneOptions): Promise<MeasuredRoute>;
 
-export { BANDS, type Band, type Baseline, DEFAULT_MEASURED_DIR, DEFAULT_SWEEP, DEFAULT_THRESHOLDS, DESKTOP, EMPTY_BASELINE, type FileException, type InPageGroup, type Judgement, LADDER, type MainAction, type MeasurePhoneOptions, type MeasureSiteOptions, type MeasureViewportsOptions, type MeasuredRoute, type OpenedPage, PHONE, PHONE_FLOORS, type PhoneCheck, type PhoneFloors, type RawPhoneFinding, SHORT_PHONE, type SiteBrowser, type SitePage, type SiteResult, type StaticFinding, type Sweep, type Thresholds, type Viewport, type ViewportArg, type ViewportPage, type ViewportsResult, WIDE_WIDTHS, bandOf, checkExceptions, formatWidths, judgeMeasured, judgeStatic, mainActionFor, measurePhone, measurePhoneInPage, measureRoute, measureSite, measureViewportInPage, measureViewports, mergeViewportFindings, normaliseElement, reasonOf, routeFile, scanPhoneFirst, sweepWidths, writeBaseline };
+export { BANDS, type Band, type Baseline, DEFAULT_MEASURED_DIR, DEFAULT_SWEEP, DEFAULT_THRESHOLDS, DESKTOP, EMPTY_BASELINE, type ExcludedRoute, type FileException, type InPageGroup, type Judgement, LADDER, type MainAction, type MeasurePhoneOptions, type MeasureSiteOptions, type MeasureViewportsOptions, type MeasuredRoute, type OpenedPage, PHONE, PHONE_FLOORS, type PhoneCheck, type PhoneFloors, type RawPhoneFinding, type RouteJudgement, SHORT_PHONE, type SiteBrowser, type SitePage, type SiteResult, type StaticFinding, type Sweep, type Thresholds, type Viewport, type ViewportArg, type ViewportPage, type ViewportsResult, WIDE_WIDTHS, bandOf, checkExceptions, cleanRoute, formatWidths, hasReason, hasUnexcusedEntries, judgeGrowth, judgeMeasured, judgeRoutes, judgeStatic, mainActionFor, measurePhone, measurePhoneInPage, measureRoute, measureSite, measureViewportInPage, measureViewports, mergeViewportFindings, normaliseElement, parseBaseline, reasonOf, routeFile, routeMatches, routesFromManifest, routesFromPages, scanPhoneFirst, sweepWidths, writeBaseline };

package/dist/phone.js

@@ -9,13 +9,14 @@ var PHONE = {
   isMobile: true,
   hasTouch: true
 };
-var PHONE_FLOORS = { targetPx: 44, fieldFontPx: 16, textPx: 12 };
+var PHONE_FLOORS = { targetPx: 44, fieldFontPx: 16, textPx: 12, fieldHeightPx: 48 };
 var DEFAULT_MEASURED_DIR = "phone-first/measured";
 var routeFile = (route) => `${route === "/" ? "home" : route.replace(/^\/+|\/+$/g, "").replace(/[^\w.-]+/g, "-")}.json`;
 
 // src/phone/measure-in-page.ts
 function measurePhoneInPage(floors) {
   const out = [];
+  const fieldHeightPx = floors.fieldHeightPx ?? 48;
   const only = floors.only;
   const on = (c) => !only || only.includes(c);
   const vw = document.documentElement.clientWidth;
@@ -59,6 +60,9 @@ function measurePhoneInPage(floors) {
     if (!shown(e) || quiet(e) || e.tagName === "INPUT" && NOT_TEXT.has(e.type)) continue;
     const px = parseFloat(getComputedStyle(e).fontSize);
     if (px < floors.fieldFontPx) out.push({ check: "field", element: describe(e), detail: `set at ${px}px, under ${floors.fieldFontPx}px, so Safari on iOS zooms on focus` });
+    const box = e.getBoundingClientRect();
+    const height = Math.round(box.height);
+    if (box.width > 1 && height > 1 && height < fieldHeightPx) out.push({ check: "field", element: describe(e), detail: `${height}px tall, under ${fieldHeightPx}px (use min-h-12)` });
   }
   for (const e of on("text") ? Array.from(document.body.querySelectorAll("*")) : []) {
     if (!shown(e) || quiet(e) || ["SCRIPT", "STYLE", "NOSCRIPT", "TEMPLATE"].includes(e.tagName)) continue;
@@ -366,17 +370,26 @@ function measureViewportInPage(arg) {
 }
 
 // src/phone/judge.ts
+var hasReason = (entry) => typeof entry.reason === "string" && entry.reason.trim() !== "";
+function parseBaseline(raw, where) {
+  const b = raw ?? {};
+  const entries = [...b.static ?? [], ...b.measured ?? []];
+  const bad = entries.find((e) => e.reason !== void 0 && typeof e.reason !== "string");
+  if (bad) throw new Error(`${where}: an entry has a "reason" that is not text: ${JSON.stringify(bad)}`);
+  return { ...b.note ? { note: b.note } : {}, static: b.static ?? [], measured: b.measured ?? [] };
+}
 var EMPTY_BASELINE = { static: [], measured: [] };
 var MEDIA = /@(?:custom-)?media\b([^{;]*)/g;
 var CAPS_WIDTH = /\bmax-(?:device-)?width\s*:|\bwidth\s*<|>=?\s*(?:device-)?width\b/i;
 var VARIANT = /(?<![\w@-])max-(\[[^\]\s]+\]|[a-z0-9]+):/g;
 var NOT_VARIANTS = /* @__PURE__ */ new Set(["width", "height", "lines", "resolution", "zoom", "inline-size", "block-size"]);
 var OK = /mobile-first-ok\b:?(.*)/;
+var VAR_OPACITY = /[^\s"'`]*var\([^)]*\)\]\/[0-9]+/g;
 function reasonOf(line) {
   const m = line?.match(OK);
   return m ? m[1].replace(/\*\/|-->|[{}]/g, "").trim() : null;
 }
-function scanPhoneFirst(files, exceptions = []) {
+function scanPhoneFirst(files, exceptions = [], options = {}) {
   const excusedFiles = new Set(exceptions.filter((e) => e.reason?.trim()).map((e) => e.file));
   const findings = [];
   let excused = 0;
@@ -391,7 +404,18 @@ function scanPhoneFirst(files, exceptions = []) {
     for (const m of text.matchAll(VARIANT)) {
       if (!NOT_VARIANTS.has(m[1])) found.push({ line: lineOf(m.index), what: m[0] });
     }
-    for (const { line, what } of found.sort((a, b) => a.line - b.line)) {
+    const silent = /* @__PURE__ */ new Set();
+    if (options.tailwind3?.(path)) {
+      for (const m of text.matchAll(VAR_OPACITY)) {
+        silent.add(found.push({ line: lineOf(m.index), what: `${m[0]} (Tailwind 3 emits no CSS for opacity on a var() colour)` }) - 1);
+      }
+    }
+    const order = found.map((f, i) => ({ ...f, silent: silent.has(i) })).sort((a, b) => a.line - b.line);
+    for (const { line, what, silent: cannotExcuse } of order) {
+      if (cannotExcuse) {
+        findings.push({ file: path, line, what });
+        continue;
+      }
       const here = reasonOf(lines[line - 1]);
       const above = reasonOf(lines[line - 2]);
       if (here || above) {
@@ -496,17 +520,101 @@ var LABEL = {
   tel: "phone number is not a link",
   linelength: "line too long"
 };
-function writeBaseline(findings, routes, note) {
+function writeBaseline(findings, routes, note, previous) {
   const byFile = tally(findings, (f) => f.file);
   const measured = routes.flatMap(
     ({ route, findings: findings2 }) => [...tally(findings2, (f) => measuredKey(route, f.check, normaliseElement(f.element))).values()].map(({ item, count }) => ({ route, check: item.check, element: normaliseElement(item.element), count }))
   );
+  const staticReasons = new Map((previous?.static ?? []).filter(hasReason).map((b) => [b.file, b.reason]));
+  const measuredReasons = new Map((previous?.measured ?? []).filter(hasReason).map((b) => [measuredKey(b.route, b.check, b.element), b.reason]));
   return {
     ...note ? { note } : {},
-    static: [...byFile.values()].map(({ item, count }) => ({ file: item.file, count, found: findings.filter((f) => f.file === item.file).map((f) => f.what) })).sort((a, b) => a.file.localeCompare(b.file)),
-    measured: measured.sort((a, b) => a.route.localeCompare(b.route) || a.check.localeCompare(b.check) || a.element.localeCompare(b.element))
+    static: [...byFile.values()].map(({ item, count }) => ({ file: item.file, count, found: findings.filter((f) => f.file === item.file).map((f) => f.what), ...staticReasons.has(item.file) ? { reason: staticReasons.get(item.file) } : {} })).sort((a, b) => a.file.localeCompare(b.file)),
+    measured: measured.map((m) => measuredReasons.has(measuredKey(m.route, m.check, m.element)) ? { ...m, reason: measuredReasons.get(measuredKey(m.route, m.check, m.element)) } : m).sort((a, b) => a.route.localeCompare(b.route) || a.check.localeCompare(b.check) || a.element.localeCompare(b.element))
   };
 }
+function judgeGrowth(current, base, baseName = "main") {
+  const out = [];
+  const baseStatic = new Map(base.static.map((b) => [b.file, b.count]));
+  for (const b of current.static) {
+    const was = baseStatic.get(b.file) ?? 0;
+    if (b.count > was && !hasReason(b)) out.push(`${b.file}: ${was ? `${b.count} desktop-first rules, ${baseName} allows ${was}` : `${b.count} desktop-first rule${b.count === 1 ? "" : "s"} added to the baseline`}`);
+  }
+  const baseMeasured = new Map(base.measured.map((b) => [measuredKey(b.route, b.check, b.element), b.count]));
+  for (const b of current.measured) {
+    const was = baseMeasured.get(measuredKey(b.route, b.check, b.element)) ?? 0;
+    if (b.count > was && !hasReason(b)) out.push(`${b.route}: ${LABEL[b.check]} ${b.element}: ${was ? `${b.count} in the baseline, ${baseName} allows ${was}` : `added to the baseline (${b.count})`}`);
+  }
+  return out;
+}
+var hasUnexcusedEntries = (b) => [...b.static, ...b.measured].some((e) => !hasReason(e));
+var PAGE_FILE = /(?:^|\/)page\.(?:tsx|ts|jsx|js|mdx)$/;
+var INTERCEPTING = /^\(\.{1,3}\)/;
+function routesFromPages(files) {
+  const routes = /* @__PURE__ */ new Set();
+  for (const file of files) {
+    const rel = file.replace(/\\/g, "/");
+    if (!PAGE_FILE.test(rel)) continue;
+    const dirs = rel.split("/").slice(0, -1);
+    if (dirs.some((d) => INTERCEPTING.test(d) || d.startsWith("_"))) continue;
+    const url = dirs.filter((d) => !/^\(.*\)$/.test(d) && !d.startsWith("@"));
+    routes.add(`/${url.join("/")}`);
+  }
+  return [...routes].sort();
+}
+function routesFromManifest(manifest) {
+  const routes = /* @__PURE__ */ new Set();
+  for (const [key, route] of Object.entries(manifest ?? {})) {
+    if (!/(?:^|\/)page$/.test(key) || typeof route !== "string") continue;
+    if (route.split("/").some((d) => d.startsWith("_"))) continue;
+    routes.add(route);
+  }
+  return [...routes].sort();
+}
+var segmentsOf = (route) => route.split("/").filter(Boolean);
+var rank = (seg) => seg.startsWith("[[...") ? 3 : seg.startsWith("[...") ? 2 : seg.startsWith("[") ? 1 : 0;
+var cleanRoute = (route) => {
+  const path = route.replace(/[?#].*$/, "").replace(/\/+$/, "");
+  return path === "" ? "/" : path;
+};
+function routeMatches(pattern, route) {
+  if (pattern === route) return true;
+  const want = segmentsOf(pattern);
+  const have = segmentsOf(route);
+  for (let i = 0; i < want.length; i++) {
+    const seg = want[i];
+    if (seg.startsWith("[[...")) return true;
+    if (seg.startsWith("[...")) return have.length > i;
+    if (have[i] === void 0) return false;
+    if (!seg.startsWith("[") && seg !== have[i]) return false;
+  }
+  return want.length === have.length;
+}
+var moreSpecific = (a, b) => {
+  const x = segmentsOf(a).map(rank);
+  const y = segmentsOf(b).map(rank);
+  for (let i = 0; i < Math.min(x.length, y.length); i++) if (x[i] !== y[i]) return x[i] < y[i];
+  return x.length > y.length;
+};
+var excludes = (entry, page) => entry.endsWith("/*") ? page === entry.slice(0, -2) || page.startsWith(entry.slice(0, -1)) : entry === page;
+function judgeRoutes(pages, measuredRoutes, excluded = []) {
+  const problems = [];
+  excluded.forEach((e, i) => {
+    const at = `excludeRoutes[${i}]`;
+    const x = e;
+    if (typeof x?.route !== "string" || !x.route.startsWith("/")) problems.push(`${at}: names no route (start it with /)`);
+    else if (typeof x.reason !== "string" || !x.reason.trim()) problems.push(`${at} (${x.route}): no reason. Say why the gate cannot measure this page, or remove the entry.`);
+    else if (!pages.some((p) => excludes(x.route, p))) problems.push(`${at}: ${x.route} is not a page route of the app. Remove the entry.`);
+  });
+  const named = excluded.filter((e) => typeof e?.route === "string" && typeof e.reason === "string" && e.reason.trim());
+  const covered = /* @__PURE__ */ new Set();
+  for (const route of measuredRoutes.map(cleanRoute)) {
+    const hit = pages.filter((p) => routeMatches(p, route)).sort((a, b) => moreSpecific(a, b) ? -1 : moreSpecific(b, a) ? 1 : 0)[0];
+    if (hit) covered.add(hit);
+  }
+  for (const page of pages) if (named.some((e) => excludes(e.route, page))) covered.add(page);
+  return { missing: pages.filter((p) => !covered.has(p)), problems, covered: covered.size };
+}
 
 // src/phone/viewports.ts
 var BANDS = [
@@ -550,16 +658,16 @@ var DEFAULT_THRESHOLDS = {
 var mainActionFor = (config, route) => typeof config === "string" ? config : config ? config[route] ?? config["*"] : void 0;
 function formatWidths(failing, tested) {
   const plain = tested.filter((n) => /^\d+$/.test(n)).sort((a, b) => Number(a) - Number(b));
-  const rank = new Map(plain.map((name, i) => [name, i]));
+  const rank2 = new Map(plain.map((name, i) => [name, i]));
   const names = [...new Set(failing)];
   const runs = [];
-  for (const name of names.filter((n) => rank.has(n)).sort((a, b) => rank.get(a) - rank.get(b))) {
+  for (const name of names.filter((n) => rank2.has(n)).sort((a, b) => rank2.get(a) - rank2.get(b))) {
     const last = runs.at(-1);
-    if (last && rank.get(name) === rank.get(last.at(-1)) + 1) last.push(name);
+    if (last && rank2.get(name) === rank2.get(last.at(-1)) + 1) last.push(name);
     else runs.push([name]);
   }
   const out = runs.map((r) => r.length > 1 ? `${r[0]}-${r.at(-1)}` : r[0]);
-  return [...out, ...names.filter((n) => !rank.has(n))].join(", ");
+  return [...out, ...names.filter((n) => !rank2.has(n))].join(", ");
 }
 var keyOf = (f) => `${f.check}\0${f.element.replace(/("[^]*)$/, (t) => t.replace(/\d+/g, "#"))}`;
 function mergeViewportFindings(perViewport, tested) {
@@ -724,7 +832,7 @@ async function measureSite(browser, options) {
 async function measurePhone(page, options) {
   if (!options.route.startsWith("/")) throw new Error(`measurePhone: route "${options.route}" should start with /`);
   await page.evaluate(() => document.fonts?.ready.then(() => void 0), void 0);
-  const findings = await page.evaluate(measurePhoneInPage, PHONE_FLOORS);
+  const findings = await page.evaluate(measurePhoneInPage, { ...PHONE_FLOORS, ...options.floors ?? {} });
   const viewport = page.viewportSize?.() ?? void 0;
   const result = { route: options.route, ...viewport ? { viewport } : {}, findings };
   const dir = options.out ?? DEFAULT_MEASURED_DIR;
@@ -747,8 +855,13 @@ export {
   WIDE_WIDTHS,
   bandOf,
   checkExceptions,
+  cleanRoute,
   formatWidths,
+  hasReason,
+  hasUnexcusedEntries,
+  judgeGrowth,
   judgeMeasured,
+  judgeRoutes,
   judgeStatic,
   mainActionFor,
   measurePhone,
@@ -759,8 +872,12 @@ export {
   measureViewports,
   mergeViewportFindings,
   normaliseElement,
+  parseBaseline,
   reasonOf,
   routeFile,
+  routeMatches,
+  routesFromManifest,
+  routesFromPages,
   scanPhoneFirst,
   sweepWidths,
   writeBaseline

package/dist/{viewports-cli-WWWWEBKI.js → viewports-cli-XYYX4SKV.js}

@@ -1,17 +1,18 @@
 import {
   EMPTY_BASELINE,
   judgeMeasured,
+  parseBaseline,
   writeBaseline
-} from "./chunk-IJUZ7RV4.js";
+} from "./chunk-BQ7H23QP.js";
 import {
   loadPlaywright
-} from "./chunk-YOZLPNJS.js";
+} from "./chunk-6DRBFDWT.js";
 import "./chunk-JYNXGCYZ.js";
 import {
   toJson
 } from "./chunk-QIEDBL4J.js";
-import "./chunk-JGWOA4LI.js";
-import "./chunk-7DQDF3VL.js";
+import "./chunk-DE2ZAV4G.js";
+import "./chunk-ONA73THJ.js";
 import "./chunk-QYBAVU3P.js";
 
 // src/phone/viewports-cli.ts
@@ -25,7 +26,7 @@ var PHONE = {
   isMobile: true,
   hasTouch: true
 };
-var PHONE_FLOORS = { targetPx: 44, fieldFontPx: 16, textPx: 12 };
+var PHONE_FLOORS = { targetPx: 44, fieldFontPx: 16, textPx: 12, fieldHeightPx: 48 };
 var DEFAULT_MEASURED_DIR = "phone-first/measured";
 var routeFile = (route) => `${route === "/" ? "home" : route.replace(/^\/+|\/+$/g, "").replace(/[^\w.-]+/g, "-")}.json`;
 
@@ -36,6 +37,7 @@ import { join } from "path";
 // src/phone/measure-in-page.ts
 function measurePhoneInPage(floors) {
   const out = [];
+  const fieldHeightPx = floors.fieldHeightPx ?? 48;
   const only = floors.only;
   const on = (c) => !only || only.includes(c);
   const vw = document.documentElement.clientWidth;
@@ -79,6 +81,9 @@ function measurePhoneInPage(floors) {
     if (!shown(e) || quiet(e) || e.tagName === "INPUT" && NOT_TEXT.has(e.type)) continue;
     const px = parseFloat(getComputedStyle(e).fontSize);
     if (px < floors.fieldFontPx) out.push({ check: "field", element: describe(e), detail: `set at ${px}px, under ${floors.fieldFontPx}px, so Safari on iOS zooms on focus` });
+    const box = e.getBoundingClientRect();
+    const height = Math.round(box.height);
+    if (box.width > 1 && height > 1 && height < fieldHeightPx) out.push({ check: "field", element: describe(e), detail: `${height}px tall, under ${fieldHeightPx}px (use min-h-12)` });
   }
   for (const e of on("text") ? Array.from(document.body.querySelectorAll("*")) : []) {
     if (!shown(e) || quiet(e) || ["SCRIPT", "STYLE", "NOSCRIPT", "TEMPLATE"].includes(e.tagName)) continue;
@@ -596,8 +601,7 @@ function loadConfig(path, cwd) {
 }
 function loadBaseline(path) {
   if (!existsSync(path)) return EMPTY_BASELINE;
-  const b = JSON.parse(readFileSync(path, "utf8"));
-  return { ...b.note ? { note: b.note } : {}, static: b.static ?? [], measured: b.measured ?? [] };
+  return parseBaseline(JSON.parse(readFileSync(path, "utf8")), path);
 }
 async function runViewports(args, io) {
   const flags = parse(args);
@@ -642,7 +646,7 @@ ${USAGE}`);
     const baseline = loadBaseline(baselinePath);
     const measuredRoutes = new Set(site.routes.map((r) => r.route));
     if (flags.update) {
-      const next = writeBaseline([], site.routes, baseline.note);
+      const next = writeBaseline([], site.routes, baseline.note, baseline);
       next.static = baseline.static;
       next.measured = [...baseline.measured.filter((m) => !measuredRoutes.has(m.route)), ...next.measured].sort((a, b) => a.route.localeCompare(b.route) || a.check.localeCompare(b.check) || a.element.localeCompare(b.element));
       writeFileSync2(baselinePath, `${JSON.stringify(next, null, 2)}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@domandigital/craft",
-  "version": "0.20.4",
+  "version": "0.21.0",
   "private": false,
   "description": "The Doman Digital house craft standard as numbers: OKLCh colour ramps that snap to existing anchors, derived semantic tokens with measured WCAG and APCA contrast, one shared motion vocabulary for CSS and JS, and a base stylesheet. Zero dependencies.",
   "license": "Apache-2.0",

package/word-lists.json

@@ -1,5 +1,5 @@
 {
-  "catalogueVersion": "2026.10.2",
+  "catalogueVersion": "2026.10.4",
   "lists": {
     "aiWords": [
       "tapestry",

package/dist/chunk-IJUZ7RV4.js

@@ -1,151 +0,0 @@
-// src/phone/judge.ts
-var EMPTY_BASELINE = { static: [], measured: [] };
-var MEDIA = /@(?:custom-)?media\b([^{;]*)/g;
-var CAPS_WIDTH = /\bmax-(?:device-)?width\s*:|\bwidth\s*<|>=?\s*(?:device-)?width\b/i;
-var VARIANT = /(?<![\w@-])max-(\[[^\]\s]+\]|[a-z0-9]+):/g;
-var NOT_VARIANTS = /* @__PURE__ */ new Set(["width", "height", "lines", "resolution", "zoom", "inline-size", "block-size"]);
-var OK = /mobile-first-ok\b:?(.*)/;
-function reasonOf(line) {
-  const m = line?.match(OK);
-  return m ? m[1].replace(/\*\/|-->|[{}]/g, "").trim() : null;
-}
-function scanPhoneFirst(files, exceptions = []) {
-  const excusedFiles = new Set(exceptions.filter((e) => e.reason?.trim()).map((e) => e.file));
-  const findings = [];
-  let excused = 0;
-  for (const { path, text } of files) {
-    if (excusedFiles.has(path)) continue;
-    const lines = text.split("\n");
-    const lineOf = (index) => text.slice(0, index).split("\n").length;
-    const found = [];
-    for (const m of text.matchAll(MEDIA)) {
-      if (CAPS_WIDTH.test(m[1])) found.push({ line: lineOf(m.index), what: `${m[0].slice(0, m[0].length - m[1].length)}${m[1].replace(/\s+/g, " ").trimEnd()}` });
-    }
-    for (const m of text.matchAll(VARIANT)) {
-      if (!NOT_VARIANTS.has(m[1])) found.push({ line: lineOf(m.index), what: m[0] });
-    }
-    for (const { line, what } of found.sort((a, b) => a.line - b.line)) {
-      const here = reasonOf(lines[line - 1]);
-      const above = reasonOf(lines[line - 2]);
-      if (here || above) {
-        excused++;
-        continue;
-      }
-      findings.push({ file: path, line, what: here === "" || above === "" ? `${what} (mobile-first-ok with no reason)` : what });
-    }
-  }
-  return { findings, excused };
-}
-function checkExceptions(exceptions, exists) {
-  const problems = [];
-  exceptions.forEach((e, i) => {
-    const at = `exceptions[${i}]`;
-    const x = e;
-    if (typeof x?.file !== "string" || !x.file.trim()) problems.push(`${at}: names no file`);
-    else if (typeof x.reason !== "string" || !x.reason.trim()) problems.push(`${at} (${x.file}): no reason. Say why this file needs a max-width query, or remove the exception.`);
-    else if (!exists(x.file)) problems.push(`${at}: ${x.file} is not there. Remove the exception.`);
-  });
-  return problems;
-}
-var normaliseElement = (element) => element.replace(/("[^]*)$/, (text) => text.replace(/\d+/g, "#"));
-var tally = (items, key) => {
-  const m = /* @__PURE__ */ new Map();
-  for (const item of items) {
-    const k = key(item);
-    const had = m.get(k);
-    if (had) had.count++;
-    else m.set(k, { item, count: 1 });
-  }
-  return m;
-};
-function judgeStatic(findings, baseline) {
-  const byFile = tally(findings, (f) => f.file);
-  const allowed = new Map(baseline.static.map((b) => [b.file, b.count]));
-  const fresh = [];
-  const stale = [];
-  let known = 0;
-  for (const [file, { count }] of byFile) {
-    const allow = allowed.get(file) ?? 0;
-    known += Math.min(count, allow);
-    if (count > allow) {
-      const here = findings.filter((f) => f.file === file);
-      const lead = allow ? `${file}: ${count} desktop-first rules, the baseline allows ${allow}. One of these is new:` : `${file}:`;
-      fresh.push(`${lead}
-${here.map((f) => `    line ${f.line}: ${f.what}`).join("\n")}`);
-    }
-  }
-  for (const { file, count } of baseline.static) {
-    const now = byFile.get(file)?.count ?? 0;
-    if (now < count) stale.push(`${file}: the baseline allows ${count}, there ${now === 1 ? "is" : "are"} ${now} now. Lower it to ${now}${now ? "" : " (remove the line)"}.`);
-  }
-  return { fresh, stale, known };
-}
-var measuredKey = (route, check, element) => `${route}\0${check}\0${element}`;
-function judgeMeasured(routes, baseline) {
-  const fresh = [];
-  const stale = [];
-  let known = 0;
-  const allowed = new Map(baseline.measured.map((b) => [measuredKey(b.route, b.check, b.element), b.count]));
-  const seen = /* @__PURE__ */ new Map();
-  const measured = new Set(routes.map((r) => r.route));
-  for (const { route, findings } of routes) {
-    const counted = tally(findings, (f) => measuredKey(route, f.check, normaliseElement(f.element)));
-    for (const [key, { item, count }] of counted) {
-      seen.set(key, count);
-      const allow = allowed.get(key) ?? 0;
-      known += Math.min(count, allow);
-      if (count > allow) {
-        const times = allow ? ` (${count} found, the baseline allows ${allow})` : count > 1 ? ` (${count} of them)` : "";
-        fresh.push(`${route}: ${LABEL[item.check]} ${item.element}: ${item.detail}${times}`);
-      }
-    }
-  }
-  const missing = /* @__PURE__ */ new Set();
-  for (const b of baseline.measured) {
-    const key = measuredKey(b.route, b.check, b.element);
-    if (!measured.has(b.route)) {
-      if (!missing.has(b.route)) stale.push(`${b.route}: in the baseline but not measured this run. Measure it, or remove its lines.`);
-      missing.add(b.route);
-      continue;
-    }
-    const now = seen.get(key) ?? 0;
-    if (now < b.count) stale.push(`${b.route}: ${LABEL[b.check]} ${b.element}: the baseline allows ${b.count}, ${now} found. Lower it to ${now}${now ? "" : " (remove the line)"}.`);
-  }
-  return { fresh, stale, known };
-}
-var LABEL = {
-  scroll: "scrolls sideways",
-  target: "small target",
-  field: "small field",
-  text: "small text",
-  title: "title cut off",
-  overlap: "text overlaps text",
-  action: "main action out of reach",
-  layers: "fixed layers cover too much",
-  landscape: "landscape screen eaten by fixed layers",
-  image: "image not sized for its place",
-  input: "wrong keyboard",
-  autocomplete: "field the phone cannot fill in",
-  tel: "phone number is not a link",
-  linelength: "line too long"
-};
-function writeBaseline(findings, routes, note) {
-  const byFile = tally(findings, (f) => f.file);
-  const measured = routes.flatMap(
-    ({ route, findings: findings2 }) => [...tally(findings2, (f) => measuredKey(route, f.check, normaliseElement(f.element))).values()].map(({ item, count }) => ({ route, check: item.check, element: normaliseElement(item.element), count }))
-  );
-  return {
-    ...note ? { note } : {},
-    static: [...byFile.values()].map(({ item, count }) => ({ file: item.file, count, found: findings.filter((f) => f.file === item.file).map((f) => f.what) })).sort((a, b) => a.file.localeCompare(b.file)),
-    measured: measured.sort((a, b) => a.route.localeCompare(b.route) || a.check.localeCompare(b.check) || a.element.localeCompare(b.element))
-  };
-}
-
-export {
-  EMPTY_BASELINE,
-  scanPhoneFirst,
-  checkExceptions,
-  judgeStatic,
-  judgeMeasured,
-  writeBaseline
-};