jig-ui 0.20.1 → 0.21.0

package/CHANGELOG.md

@@ -1,5 +1,75 @@
 # Changelog
 
+## 0.21.0 (2026-09-28)
+
+Dev builds, so a fix is tried on a real project before it ships, and the gate
+fixes that trial on jig-site turned up.
+
+### Added
+
+- **Dev builds.** A version carrying `-dev` (`0.21.0-dev.1`) is a local build
+  of a fix, tried on a real project before it is released. npm has no such
+  version, so its skill, its commands and its Stop hook run the `jig` on PATH
+  rather than `npx jig-ui@<version>`; a hook that could not run would let every
+  agent stop unchecked. `scripts/pack-dev.mjs` packs one. Installing a release
+  afterwards puts the `npx` hook back in place of the dev one.
+
+### Fixed
+
+- **The Stop hook refreshes only the probes it judges.** It re-rendered every
+  critiqued page on every stop, so a link added to jig-site's header left sixty
+  probe files changed on pages no one was working on, and three sessions
+  committed them. A page's probes are refreshed when its own critique or tweak
+  runs.
+- **A mockup is checked before it goes to the owner.** The pause for approval
+  skipped the drawing check, so on jig-site two mockup sessions put a drawing
+  the check refused to the owner, reported the gate's blocks as "waiting on
+  owner review", and the owner approved a drawing the gate then refused `make`
+  for.
+- **A region label may say its name with spaces.** "on this page" now labels the
+  spec's `on-this-page`; hyphens, underscores and spaces compare as one.
+- **A comment after a size is a comment.** `phone:   # judged at 360px` over a
+  full composition was refused as a one-line size.
+- **A critique's report names every finding its verdict files hold.** jig-site's
+  Guide critique held seven and listed six; make reads the report. Checked
+  where the project keeps a `REPORT.md` beside the verdicts.
+- **A session cannot change Jig's mode file.** A jig-site make round narrowed
+  `--size-rail` in `mode.editorial.css` to make three columns fit; `update`
+  owns that file, and a project's own values go in its brand file or stylesheet.
+- **A decide amendment answers for its own reason.** Like a tweak's, only the
+  `**Why…:**` paragraphs it adds must quote the owner; three jig-site amendments
+  had to relabel an earlier round's reason before the gate let them stop.
+- **A finding is fixed only where the source it cited changed.** `verdicts`
+  told jig-site's fourth home critique that seven findings were fixed; four had
+  flipped because its readers read the same unchanged lines differently. A
+  finding judged ok is now fixed when a line it cited (`file:line`) changed, or
+  its file when it cited no line, or any source when it cited nothing. The rest
+  are reported as judged ok though the lines they cited did not change: a
+  different reading, or a fix made where the finding did not point.
+- **The gate asks for the verdict lock to be committed.** It writes
+  `verdicts.lock` when a critique or tweak stops, after the agent has committed,
+  so every such session on jig-site left it behind. When the verdicts are
+  committed and the lock is not, it stops the session once to commit it.
+- **A tweak's decision quotes the owner's recorded words.** A jig-site tweak
+  shown a screenshot wrote an exception to `E-51` "given directly by the owner
+  … by reference rather than words". A tweak's new `**Why:**` now meets
+  `decide`'s rule (the owner's words in quotation marks, `not given`, or
+  `**Why (inferred):**`), and its quotations must be in tweak.json's `change`.
+  A reason `decide` wrote earlier is not the tweak's to answer for.
+- **`spec`, `mockup` and `critique` write under `.jig/` only.** On jig-site a
+  critique swapped the rule its page demonstrates to get past a block, then
+  re-judged the page it had changed, and a mockup session wrote the site's
+  stylesheet. The gate stops either.
+- **An approved drawing changed afterwards goes back to pending.** The session
+  that recorded a jig-site approval then redrew frames and moved a switch.
+- **Quoted tool output is not the page's copy.** The probe's em-dash scan skips
+  `<pre>`, `<code>`, `<samp>` and `<kbd>`, as it skips quotations: jig-site's
+  home page quotes `jig check`'s real output, whose wording has one. Probe
+  version 9.
+- **A drawing's size frames are the class `frame`, not any class containing
+  it.** jig-site's home drawing captioned its rule specimens `spec-frame`, and
+  the gate split every size frame at each one and reported regions missing.
+
 ## 0.20.1 (2026-09-27)
 
 `jig gate` run by hand no longer tells an agent that a passing tweak failed.

package/dist/index.js

@@ -40,6 +40,14 @@ function assetRoot(startDir = getPackageRoot()) {
 function isPublishedBuild(packageRoot2) {
   return packageRoot2.split(/[\\/]/).includes("node_modules");
 }
+function isDevVersion(version2) {
+  return !!version2 && /-dev(\.|$)/.test(version2);
+}
+function cliInvocation(version2) {
+  if (!version2) return "npx jig-ui";
+  return isDevVersion(version2) ? "jig" : `npx jig-ui@${version2}`;
+}
+var JIG_GATE_HOOK = /(?:\bjig-ui@[^\s"]+|^jig) gate\b/;
 
 // src/commands/install.ts
 import { existsSync as existsSync5, mkdirSync as mkdirSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync2 } from "fs";
@@ -578,7 +586,7 @@ function buildSkillBody(packageRoot2, rulesPath, version2) {
   );
   return render(template, {
     command_prefix: "/jig ",
-    scripts_path: version2 ? `npx jig-ui@${version2}` : "npx jig-ui",
+    scripts_path: cliInvocation(version2),
     // `update` must NOT carry the pin. Pinned, it refreshes to the version
     // already installed — reporting "Updated Jig → <same version>" for a no-op
     // — so a reader following the skill could never upgrade.
@@ -597,7 +605,7 @@ function buildCommandBody(packageRoot2, rulesPath, version2, argsPlaceholder) {
   const templatePath = join7(packageRoot2, "templates", "COMMAND.md.tmpl");
   if (!existsSync5(templatePath)) return { body: "", subcommands };
   const template = readFileSync4(templatePath, "utf8");
-  const body = template.replace(/\{\{command_prefix\}\}/g, "/jig ").replace(/\{\{args_placeholder\}\}/g, argsPlaceholder).replace(/\{\{subcommand_list\}\}/g, subcommands.join(", ")).replace(/\{\{scripts_path\}\}/g, `npx jig-ui@${version2}`).replace(/\{\{update_path\}\}/g, "npx jig-ui@latest").replace(/\{\{rules_path\}\}/g, rulesPath).replace(/\{\{config_file\}\}/g, "jig.config.json");
+  const body = template.replace(/\{\{command_prefix\}\}/g, "/jig ").replace(/\{\{args_placeholder\}\}/g, argsPlaceholder).replace(/\{\{subcommand_list\}\}/g, subcommands.join(", ")).replace(/\{\{scripts_path\}\}/g, cliInvocation(version2)).replace(/\{\{update_path\}\}/g, "npx jig-ui@latest").replace(/\{\{rules_path\}\}/g, rulesPath).replace(/\{\{config_file\}\}/g, "jig.config.json");
   const unresolved = [...new Set(body.match(/\{\{[a-z_]+\}\}/g) ?? [])];
   if (unresolved.length > 0) {
     throw new Error(
@@ -724,7 +732,8 @@ function install(opts) {
 function hasStopHook(projectRoot) {
   try {
     const settings = readFileSync4(join7(projectRoot, ".claude", "settings.json"), "utf8");
-    return /\bjig-ui@[^\s"]+ gate\b/.test(settings);
+    const hooks = JSON.parse(settings).hooks?.Stop ?? [];
+    return hooks.some((group) => (group.hooks ?? []).some((h) => JIG_GATE_HOOK.test(h.command ?? "")));
   } catch {
     return false;
   }
@@ -739,9 +748,9 @@ function installStopHook(projectRoot, version2) {
       return false;
     }
   }
-  const command = `npx --yes jig-ui@${version2} gate`;
+  const command = isDevVersion(version2) ? "jig gate" : `npx --yes jig-ui@${version2} gate`;
   const hooks = settings.hooks ?? {};
-  const isJig = (h) => /\bjig-ui@[^\s]+ gate\b/.test(h.command ?? "");
+  const isJig = (h) => JIG_GATE_HOOK.test(h.command ?? "");
   const stop = (hooks.Stop ?? []).map((group) => ({ ...group, hooks: group.hooks.filter((h) => !isJig(h)) })).filter((group) => group.hooks.length > 0);
   stop.push({ hooks: [{ type: "command", command }] });
   settings.hooks = { ...hooks, Stop: stop };
@@ -5771,7 +5780,7 @@ import { fileURLToPath as fileURLToPath2 } from "url";
 import { join as join33, relative as relative3, resolve as resolve4, sep as sep5 } from "path";
 
 // src/probe/script.ts
-var PROBE_VERSION = 8;
+var PROBE_VERSION = 9;
 var PROBE_SCRIPT = `(async () => {
   const doc = document.documentElement;
   // checkVisibility also sees content a closed <details> hides: Chromium hides
@@ -5786,9 +5795,12 @@ var PROBE_SCRIPT = `(async () => {
   const text = document.body.innerText || '';
   // I-118: a verbatim quotation keeps its own punctuation. Text marked as a
   // quotation is the source's words, so it is taken out before the dash scan;
-  // the page's own copy around it still counts.
+  // 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')) {
+  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');
   }
@@ -6557,12 +6569,22 @@ function decisionSections(body) {
   close();
   return sections;
 }
-function unsourcedReasons(current, before) {
+function changedWhys(current, before, newOnly = false) {
   const then = decisionSections(before);
-  const problems = [];
+  const paragraphs = (text) => text.split(/\n\s*\n/).map((p) => p.trim()).filter((p) => /^\*\*Why\b[^*]*:\*\*/.test(p));
+  const out = [];
   for (const [name, text] of decisionSections(current)) {
-    if (then.get(name) === text) continue;
-    const whys = text.split(/\n\s*\n/).filter((p) => /^\s*\*\*Why\b[^*]*:\*\*/.test(p));
+    const old = then.get(name);
+    if (old === text) continue;
+    const kept = newOnly && old !== void 0 ? new Set(paragraphs(old)) : /* @__PURE__ */ new Set();
+    const whys = paragraphs(text).filter((p) => !kept.has(p));
+    if (whys.length) out.push([name, whys]);
+  }
+  return out;
+}
+function unsourcedReasons(current, before, opts = {}) {
+  const problems = [];
+  for (const [name, whys] of changedWhys(current, before, opts.newWhysOnly)) {
     for (const why of whys) {
       const label = /^\s*\*\*(Why\b[^*]*):\*\*/.exec(why)[1];
       const said = why.replace(/^\s*\*\*Why\b[^*]*:\*\*/, "").trim();
@@ -6575,6 +6597,23 @@ function unsourcedReasons(current, before) {
   }
   return problems;
 }
+function quotesNotFrom(current, before, words) {
+  const norm = (t) => t.toLowerCase().replace(/[‘’]/g, "'").replace(/\s+/g, " ").trim();
+  const said = norm(words);
+  const problems = [];
+  for (const [name, whys] of changedWhys(current, before, true)) {
+    for (const why of whys) {
+      const label = /^\s*\*\*(Why\b[^*]*):\*\*/.exec(why)[1];
+      if (/inferred/i.test(label)) continue;
+      const missing = [...why.matchAll(/["“]([^"”]{3,})["”]/g)].map((m) => m[1]).filter((q) => q.split(/…|\.\.\./).map((part) => norm(part).replace(/^[\s.,;:]+|[\s.,;:!?]+$/g, "")).filter(Boolean).some((part) => !said.includes(part)));
+      if (missing.length) {
+        problems.push(`"${name}": its \`**${label}:**\` quotes "${missing[0]}", which the owner's words in tweak.json (\`change\`) do not say. Quote the owner as tweak.json records them, or put your reading under \`**Why (inferred):**\`.`);
+        break;
+      }
+    }
+  }
+  return problems;
+}
 
 // src/check/spec-shape.ts
 import { existsSync as existsSync22, readFileSync as readFileSync29, readdirSync as readdirSync14, statSync as statSync4 } from "fs";
@@ -6615,7 +6654,7 @@ function specProblems(spec) {
       problems.push(`${spec.path} has no \`${size}:\` composition under \`sizes:\`. Every size is written in full, phone first; \`same-as:\` needs a \`why:\`.`);
       continue;
     }
-    if (line[1].trim()) {
+    if (line[1].replace(/(^|\s)#.*$/, "").trim()) {
       problems.push(`${spec.path}: \`${size}:\` is a one-line description. A size is a composition \u2014 \`regions:\` in order, \`hierarchy:\`, \`nav:\` \u2014 each on its own line, or \`same-as:\` with a \`why:\`.`);
       continue;
     }
@@ -6918,10 +6957,12 @@ function previousFindings(projectRoot, dir) {
   const files = ["screen.json", "code.json", "decisions.json"];
   const paths = files.map((f) => `${rel}/${f}`);
   let commit;
+  let target;
   try {
     const dirty = git(["status", "--porcelain", "--", ...paths]).trim() !== "";
     const touched = git(["log", "--format=%H", "-2", "--", ...paths]).split("\n").filter(Boolean);
     commit = dirty ? touched[0] ?? "" : touched[1] ?? "";
+    target = dirty ? void 0 : touched[0];
   } catch {
     return void 0;
   }
@@ -6942,7 +6983,7 @@ function previousFindings(projectRoot, dir) {
     }
     for (const v of Array.isArray(body.verdicts) ? body.verdicts : []) {
       const key = String(v.id ?? v.decision ?? "").trim();
-      if (key && typeof v.verdict === "string") out.set(f === "decisions.json" ? `"${key}"` : key.toUpperCase(), v.verdict);
+      if (key && typeof v.verdict === "string") out.set(f === "decisions.json" ? `"${key}"` : key.toUpperCase(), { verdict: v.verdict, reason: String(v.reason ?? "") });
     }
     return out;
   };
@@ -6952,17 +6993,64 @@ function previousFindings(projectRoot, dir) {
     for (const [k, v] of read(f, commit)) before.set(k, v);
     for (const [k, v] of read(f)) now.set(k, v);
   }
-  const result = { commit: commit.slice(0, 7), fixed: [], open: [], added: [], ruled: [] };
+  const changed = sourceChanges(git, commit, target);
+  const result = { commit: commit.slice(0, 7), fixed: [], unchanged: [], open: [], added: [], ruled: [] };
   for (const [k, v] of before) {
-    if (v !== "finding") continue;
-    const after = now.get(k);
+    if (v.verdict !== "finding") continue;
+    const after = now.get(k)?.verdict;
     if (after === "finding") result.open.push(k);
     else if (after === "ruled") result.ruled.push(k);
-    else if (after === "ok" || after === "n/a") result.fixed.push(k);
+    else if (after === "ok" || after === "n/a") (changed(v.reason) ? result.fixed : result.unchanged).push(k);
   }
-  for (const [k, v] of now) if (v === "finding" && before.get(k) !== "finding") result.added.push(k);
+  for (const [k, v] of now) if (v.verdict === "finding" && before.get(k)?.verdict !== "finding") result.added.push(k);
   return result;
 }
+var CITATION = /([\w@.\-[\]/]*[\w\]-]\.[a-z][a-z0-9]{0,5})(?::(\d+(?:-\d+)?(?:,\s*\d+(?:-\d+)?)*))?/gi;
+function sourceChanges(git, from, to) {
+  const range = to ? [from, to] : [from];
+  let tracked = [];
+  let changedFiles = /* @__PURE__ */ new Set();
+  try {
+    tracked = git(["ls-tree", "-r", "--name-only", from]).split("\n").filter((f) => f && !f.startsWith(".jig/"));
+    changedFiles = new Set(git(["diff", "--name-only", ...range, "--", ".", ":(exclude).jig"]).split("\n").filter(Boolean));
+  } catch {
+  }
+  const hunks = /* @__PURE__ */ new Map();
+  const oldRanges = (file) => {
+    let out = hunks.get(file);
+    if (out) return out;
+    out = [];
+    try {
+      for (const m of git(["diff", "-U0", ...range, "--", file]).matchAll(/^@@ -(\d+)(?:,(\d+))? /gm)) {
+        const start = Number(m[1]);
+        const count = m[2] === void 0 ? 1 : Number(m[2]);
+        out.push(count === 0 ? [start, start + 1] : [start, start + count - 1]);
+      }
+    } catch {
+    }
+    hunks.set(file, out);
+    return out;
+  };
+  return (reason) => {
+    let cited = false;
+    for (const m of reason.matchAll(CITATION)) {
+      const path = m[1].replace(/^\.\//, "");
+      const matches2 = tracked.filter((f) => f === path || f.endsWith(`/${path}`));
+      if (!matches2.length) continue;
+      cited = true;
+      const lines = (m[2] ?? "").split(",").map((r) => r.trim()).filter(Boolean).map((r) => {
+        const [a, b] = r.split("-").map(Number);
+        return [a, b ?? a];
+      });
+      for (const file of matches2) {
+        if (!changedFiles.has(file)) continue;
+        if (!lines.length) return true;
+        if (oldRanges(file).some(([hs, he]) => lines.some(([s, e]) => hs <= e && he >= s))) return true;
+      }
+    }
+    return cited ? false : changedFiles.size > 0;
+  };
+}
 
 // src/commands/gate.ts
 import { existsSync as existsSync24, mkdirSync as mkdirSync8, readdirSync as readdirSync15, readFileSync as readFileSync32, statSync as statSync5, writeFileSync as writeFileSync6 } from "fs";
@@ -7053,7 +7141,7 @@ function specSwitches(specBody) {
   return value.replace(/^\[|\]$/g, "").split(",").map((n) => n.trim().replace(/^["']|["']$/g, "").replace(/^(--)?breakpoint-/, "")).filter(Boolean);
 }
 function readFrames(html) {
-  const opens = [...html.matchAll(/<\w+\b[^>]*\bclass\s*=\s*["'][^"']*\bframe\b[^"']*["'][^>]*>/gi)];
+  const opens = [...html.matchAll(/<\w+\b[^>]*\bclass\s*=\s*["']([^"']*)["'][^>]*>/gi)].filter((m) => m[1].split(/\s+/).includes("frame"));
   return opens.map((open, i) => {
     const tag = open[0];
     const body = html.slice(open.index + tag.length, opens[i + 1]?.index ?? html.length);
@@ -7119,7 +7207,9 @@ function regionDrawn(labels, region) {
   const found = (name) => labels.some((label) => labelMatches(label, name) || labelMatches(stripQualifiers(label), name));
   return found(region.name) || !!region.parts && region.parts.every(found);
 }
-function labelMatches(label, name) {
+function labelMatches(rawLabel, rawName) {
+  const label = rawLabel.replace(/[-_]+/g, " ").replace(/\s+/g, " ").trim();
+  const name = rawName.replace(/[-_]+/g, " ").replace(/\s+/g, " ").trim();
   return label === name || label.startsWith(`${name} `) || label.startsWith(`${name}:`) || new RegExp(`\\b${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`).test(label);
 }
 function normalise(text) {
@@ -7228,6 +7318,10 @@ function surfacesInPlay(root, command, transcriptPath, surface) {
   const current = command === "critique" || command === "tweak" ? specFor(root, surface)?.slug : void 0;
   return all.filter((s) => s === current || verdictsMtime(join39(critiqueDir, s)) >= start - 1e3);
 }
+function surfacesToProbe(root, input) {
+  const invocation = lastJigInvocation(input.transcript_path);
+  return surfacesInPlay(root, invocation?.command, input.transcript_path, invocation?.surface);
+}
 function fileAtSessionStart(root, path, start) {
   const git = (args) => execFileSync4("git", args, { cwd: root, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
   try {
@@ -7237,6 +7331,35 @@ function fileAtSessionStart(root, path, start) {
     return "";
   }
 }
+function changedOutsideRecords(root, start) {
+  const git = (args) => execFileSync4("git", args, { cwd: root, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
+  try {
+    const base = start === void 0 ? "HEAD" : git(["rev-list", "-1", `--before=@${Math.floor(start / 1e3)}`, "HEAD"]).trim();
+    if (!base) return [];
+    const changed = git(["diff", "--name-only", base, "--", ".", ":(exclude).jig"]).split("\n").filter(Boolean);
+    const created = git(["ls-files", "--others", "--exclude-standard", "--", ".", ":(exclude).jig"]).split("\n").filter(Boolean).filter((f) => {
+      try {
+        return start === void 0 || statSync5(join39(root, f)).mtimeMs >= start - 1e3;
+      } catch {
+        return false;
+      }
+    });
+    return [.../* @__PURE__ */ new Set([...changed, ...created])].sort();
+  } catch {
+    return [];
+  }
+}
+function drawingChangedAfterApproval(root, specPath, at) {
+  const git = (args) => execFileSync4("git", args, { cwd: root, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
+  try {
+    const approvedIn = git(["log", "-1", "--format=%H", "-G", "^mockup[[:space:]]*:[[:space:]]*approved", "--", specPath]).trim();
+    if (!approvedIn) return void 0;
+    const then = git(["show", `${approvedIn}:./${at}`]);
+    return then !== readFileSync32(join39(root, at), "utf8") ? approvedIn.slice(0, 7) : void 0;
+  } catch {
+    return void 0;
+  }
+}
 function commandProblems(root, command, surface, start) {
   const problems = [];
   const spec = specFor(root, surface);
@@ -7249,7 +7372,7 @@ function commandProblems(root, command, surface, start) {
         problems.push("DECISIONS.md has no `## Unresolved` section. Round 3 asks by name what is still undecided; write what the owner named, or `None named by the owner.`");
       }
       if (/\[TODO\]/.test(body)) problems.push("DECISIONS.md still contains [TODO] markers.");
-      problems.push(...unsourcedReasons(body, fileAtSessionStart(root, found, start)).map((p) => `DECISIONS.md: ${p}`));
+      problems.push(...unsourcedReasons(body, fileAtSessionStart(root, found, start), { newWhysOnly: true }).map((p) => `DECISIONS.md: ${p}`));
     }
   }
   if (command === "spec" || command === "mockup" || command === "make" || command === "critique" || command === "tweak") {
@@ -7285,6 +7408,25 @@ function commandProblems(root, command, surface, start) {
       }
     }
   }
+  if (command === "spec" || command === "mockup" || command === "critique") {
+    const outside = changedOutsideRecords(root, start);
+    if (outside.length) {
+      problems.push(
+        `${command} writes under .jig/ only, and this session changed ${outside.slice(0, 6).join(", ")}${outside.length > 6 ? ` and ${outside.length - 6} more` : ""}. Put them back as they were when the session began, and hand the change to \`/jig make\` (or \`/jig tweak\` for a small one to a built page). ` + (command === "critique" ? "A critique that changes the page it judges is grading its own work; report the finding instead." : "The build is make's.")
+      );
+    }
+  }
+  if (command === "mockup" && spec) {
+    const front = spec.body.split(/^---\s*$/m)[1] ?? "";
+    const mockup = /^\s*mockup\s*:\s*(\S+)/im.exec(front)?.[1] ?? "";
+    const at = /^\s*mockup_at\s*:\s*(.+)$/im.exec(front)?.[1]?.trim().replace(/^["']|["']$/g, "") ?? "";
+    if (/^approved/i.test(mockup) && at && !/^https?:/i.test(at) && existsSync24(join39(root, at))) {
+      const changedAfter = drawingChangedAfterApproval(root, spec.path, at);
+      if (changedAfter) {
+        problems.push(`${at} has changed since the owner approved it (${changedAfter}). An approval is of the drawing as the owner saw it: set \`mockup: pending\` in ${spec.path} and ask again.`);
+      }
+    }
+  }
   if (command === "make") {
     const found = decisionsFile(root);
     if (found) {
@@ -7299,6 +7441,36 @@ function commandProblems(root, command, surface, start) {
       }
     }
   }
+  if (command === "tweak" && spec) {
+    const found = decisionsFile(root);
+    if (found) {
+      let now = "";
+      try {
+        now = readFileSync32(join39(root, found), "utf8");
+      } catch {
+      }
+      const then = fileAtSessionStart(root, found, start);
+      if (now && now !== then) {
+        problems.push(...unsourcedReasons(now, then, { newWhysOnly: true }).map((p) => `${found}: ${p}`));
+        const change = readTweak(join39(root, ".jig", "critique", spec.slug))?.change;
+        if (change) problems.push(...quotesNotFrom(now, then, change).map((p) => `${found}: ${p}`));
+      }
+    }
+  }
+  if (command !== "update" && command !== "init" && command !== "install") {
+    for (const file of modeFiles(root)) {
+      let now = "";
+      try {
+        now = readFileSync32(join39(root, file), "utf8");
+      } catch {
+        continue;
+      }
+      const then = fileAtSessionStart(root, file, start);
+      if (then && now !== then) {
+        problems.push(`${file} changed in this session. A mode file is Jig's, refreshed by \`update\`; a value this project needs of its own goes in its brand file or its own stylesheet. Restore it (\`git checkout -- ${file}\`, or \`git show <commit>:${file}\` if the change is committed).`);
+      }
+    }
+  }
   if (command === "critique") {
     const dir = join39(root, ".jig", "critique");
     const surfaces = existsSync24(dir) ? readdirSync15(dir).filter((s) => existsSync24(join39(dir, s, "screen.json")) || existsSync24(join39(dir, s, "code.json"))) : [];
@@ -7311,12 +7483,56 @@ function commandProblems(root, command, surface, start) {
         problems.push(`${surface2}: the screen pass judged ${v.screen.judged} rules with rendered: false. Those rules are judged on a render \u2014 open the page at 360, 768 and 1280, run \`jig probe\` at each, and judge them there.`);
       }
     }
+    const own = specFor(root, surface)?.slug;
+    if (own && surfaces.includes(own)) problems.push(...reportOmissions(join39(dir, own), own));
     if (surfaces.length === 0) {
       problems.push("critique wrote no verdict files. Each reader arm writes .jig/critique/<surface>/screen.json or code.json, and `jig verdicts <surface>` \u2014 not your own count \u2014 decides whether the review is complete. A report without them is not a review.");
     }
   }
   return problems;
 }
+function drawingBeforeAsking(root, surface) {
+  const spec = specFor(root, surface);
+  if (!spec) return [];
+  const front = spec.body.split(/^---\s*$/m)[1] ?? "";
+  const named = /^\s*mockup_at\s*:\s*(.+)$/im.exec(front)?.[1]?.trim().replace(/^["']|["']$/g, "") ?? "";
+  const at = named || `.jig/mockups/${spec.slug}.html`;
+  if (/^https?:/i.test(at) || !/\.html?$/i.test(at) || !existsSync24(join39(root, at))) return [];
+  const found = mockupDrawingProblems(root, spec.body, at);
+  return found.length ? [`Fix the drawing before you put it to the owner; they approve what the spec lists, drawn. ${found.join(" ")}`] : [];
+}
+function modeFiles(root) {
+  try {
+    const state = JSON.parse(readFileSync32(join39(root, ".jig", "state.json"), "utf8"));
+    return Object.keys(state.files ?? {}).filter((f) => /(^|\/)mode\.[\w-]+\.css$/.test(f));
+  } catch {
+    return [];
+  }
+}
+function reportOmissions(dir, surface) {
+  let report2;
+  try {
+    report2 = readFileSync32(join39(dir, "REPORT.md"), "utf8").toLowerCase();
+  } catch {
+    return [];
+  }
+  const missing = [];
+  for (const f of VERDICT_FILES) {
+    let file;
+    try {
+      file = JSON.parse(readFileSync32(join39(dir, f), "utf8"));
+    } catch {
+      continue;
+    }
+    for (const v of Array.isArray(file.verdicts) ? file.verdicts : []) {
+      if (v.verdict !== "finding") continue;
+      const key = String(v.id ?? v.decision ?? "").trim();
+      if (key && !report2.includes(key.toLowerCase())) missing.push(v.id ? key : `"${key}"`);
+    }
+  }
+  if (!missing.length) return [];
+  return [`.jig/critique/${surface}/REPORT.md leaves out ${missing.length === 1 ? "a finding" : `${missing.length} findings`} the verdict files hold: ${missing.slice(0, 8).join(", ")}. The report is what the owner and \`make\` read; name every finding in it, by id (a decision by its name).`];
+}
 function tweakProblems(root, spec) {
   const problems = [];
   const front = spec.body.split(/^---\s*$/m)[1] ?? "";
@@ -7404,6 +7620,7 @@ function gate(opts) {
   const command = invocation?.command;
   const waiting = command !== void 0 && ASKS_THE_OWNER.has(command) && asksOwner(lastAssistantText(opts.input.transcript_path));
   const problems = command && !waiting ? commandProblems(root, command, invocation?.surface, sessionStart(opts.input.transcript_path)).map((p) => `/jig ${command}: ${p}`) : [];
+  if (waiting && command === "mockup") problems.push(...drawingBeforeAsking(root, invocation?.surface).map((p) => `/jig mockup: ${p}`));
   const selection = selectFiles(root, false);
   const changedUi = selection.mode === "changed" && selection.files.some((f) => isStyleBearing(f) || isReaderText(f));
   if (changedUi) {
@@ -7530,6 +7747,7 @@ function verdictGuard(root, command, inPlay) {
     const lockPath = join39(dir, LOCK);
     if (command === "critique" && (!inPlay || inPlay.includes(surface))) {
       writeLock(lockPath, dir, now);
+      problems.push(...lockLeftBehind(root, surface));
       continue;
     }
     let lock = {};
@@ -7542,7 +7760,10 @@ function verdictGuard(root, command, inPlay) {
     if (command === "tweak" && (!inPlay || inPlay.includes(surface)) && locked !== now) {
       const own = tweakVerdictProblems(dir, surface, lock.verdicts);
       problems.push(...own);
-      if (own.length === 0) writeLock(lockPath, dir, now);
+      if (own.length === 0) {
+        writeLock(lockPath, dir, now);
+        problems.push(...lockLeftBehind(root, surface));
+      }
       continue;
     }
     if (locked && locked !== now) {
@@ -7553,6 +7774,17 @@ function verdictGuard(root, command, inPlay) {
   }
   return problems;
 }
+function lockLeftBehind(root, surface) {
+  const rel = `.jig/critique/${surface}`;
+  try {
+    const status = (paths) => execFileSync4("git", ["status", "--porcelain", "--", ...paths], { cwd: root, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim();
+    if (!status([`${rel}/${LOCK}`])) return [];
+    if (status(VERDICT_FILES.map((f) => `${rel}/${f}`))) return [];
+  } catch {
+    return [];
+  }
+  return [`${rel}/${LOCK} was written when you stopped, after the verdicts it records were committed. Commit it on its own (\`git add ${rel}/${LOCK} && git commit -m "chore: record ${surface}'s verdict lock"\`), and nothing else.`];
+}
 function tweakVerdictProblems(dir, surface, lockedVerdicts) {
   const record = readTweak(dir);
   if (!record) {
@@ -7696,6 +7928,12 @@ function seo(opts) {
 var packageRoot = getPackageRoot();
 var { version } = JSON.parse(readFileSync34(join41(packageRoot, "package.json"), "utf8"));
 function warnIfUnpublishedPin() {
+  if (isDevVersion(version)) {
+    console.warn(
+      `Dev build ${version}: the skill, its commands and the Stop hook run the \`jig\` on your PATH, not npm, which has no such version. Keep this build installed while the project uses it, and install a published version to go back.`
+    );
+    return;
+  }
   if (isPublishedBuild(packageRoot)) return;
   console.warn(
     `Note: this is a source build, so the skill pins 'npx jig-ui@${version}' \u2014 a version that may not be published. Agents reading it will not be able to run the CLI until it is. Re-run install or update from a published build to correct the pin.`
@@ -7819,7 +8057,7 @@ program.command("verdicts").description("Verify a critique's verdict files and c
     const p = result.previous;
     if (p) {
       const list = (ids) => ids.length ? ` (${ids.slice(0, 8).join(", ")}${ids.length > 8 ? ", \u2026" : ""})` : "";
-      console.log(`  Since the critique before this one (${p.commit}): ${p.fixed.length} fixed${list(p.fixed)}, ${p.open.length} still open${list(p.open)}, ${p.ruled.length} ruled by the owner${list(p.ruled)}, ${p.added.length} new${list(p.added)}.`);
+      console.log(`  Since the critique before this one (${p.commit}): ${p.fixed.length} fixed${list(p.fixed)}, ${p.unchanged.length} judged ok though the lines they cited did not change${list(p.unchanged)}, ${p.open.length} still open${list(p.open)}, ${p.ruled.length} ruled by the owner${list(p.ruled)}, ${p.added.length} new${list(p.added)}.`);
     }
     console.log(`  ${result.line}`);
     process.exit(result.ok ? 0 : 1);
@@ -7917,7 +8155,8 @@ program.command("gate").description("Run by the Claude Code Stop hook: block sto
   const cwd = input.cwd ?? process.cwd();
   try {
     const projectRoot = findProjectRoot(cwd);
-    for (const surface of critiquedSurfaces(projectRoot)) {
+    const critiqued = critiquedSurfaces(projectRoot);
+    for (const surface of surfacesToProbe(projectRoot, input).filter((s) => critiqued.includes(s))) {
       const page = surfacePage(projectRoot, surface) ?? recordedPage(projectRoot, surface);
       if (!page) continue;
       try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jig-ui",
-  "version": "0.20.1",
+  "version": "0.21.0",
   "description": "A design system for coding agents. 143 numbered UI rules, brand x mode design tokens, and an installer for Claude Code, Codex, Cursor and opencode.",
   "license": "Apache-2.0",
   "type": "module",

package/templates/COMMAND.md.tmpl

@@ -657,7 +657,10 @@ down the spec's `regions:` and its `nav:` and find each one in that frame. A reg
 the spec names and the frame lacks is drawn now, not raised in review. The gate
 checks an HTML drawing when you stop: a frame for each of the four sizes, each
 size's regions labelled in its frame (by name where the spec names them), a
-frame either side of each recorded switch, and no icon or image in any frame. This matters
+frame either side of each recorded switch, and no icon or image in any frame. It checks
+the same while you wait for the user's word, so a drawing it refuses cannot be put to
+them. When it stops you, say what it refused and fix it; never report a gate block as
+waiting on the user. This matters
 most on the phone, where a region dropped for space is easy to miss and the reviewer
 is looking at what is there, not at what is not.
 
@@ -1159,11 +1162,16 @@ preserve while fixing them.
 
 **Say what happened to the last round's findings.** `verdicts` compares this
 critique with the one before it in git and prints `Since the critique before
-this one: <n> fixed, <n> still open, <n> ruled by the owner, <n> new`, with the
-ids. Put that line in the report, as `verdicts` printed it, above the findings.
-The owner should not have to set two reports side by side to learn whether the
-make round worked. The comparison is made after both arms have written, so it
-tells the arms nothing.
+this one: <n> fixed, <n> judged ok though the lines they cited did not change, <n>
+still open, <n> ruled by the owner, <n> new`, with the ids. Put that line in the
+report, as `verdicts` printed it, above the findings. The owner should not have to set two reports side
+by side to learn whether the make round worked. The comparison is made after both
+arms have written, so it tells the arms nothing.
+
+A finding counts as fixed only where the lines it cited changed. One judged ok
+where nothing it pointed at changed is either two readers disagreeing about the
+same code, or a fix made somewhere the finding did not point. Look at which, and
+say so: the reading you stand by and why, or where the fix landed.
 
 List `ruled` verdicts after the findings, under their own heading, each with the
 decision it cites. They are not findings, and they are not hidden either.
@@ -1256,6 +1264,14 @@ colour for a mark, how versions are written), record it in `DECISIONS.md` the wa
 applies a rule, or answers a finding the rules already settle (a word stranded on
 its own line, `B-106`), there is nothing to decide; do not invent a decision.
 
+The owner's words are the ones tweak.json records as `change`. A `**Why:**` you
+add quotes them, in quotation marks, and nothing else; what you read into them,
+or into a screenshot the owner pointed at, goes under `**Why (inferred):**`. On
+jig-site a tweak shown a picture of a copy button wrote an exception to `E-51`
+"given directly by the owner … by reference rather than words": a ruling nobody
+gave, which every later critique would have judged the page by. The gate stops a
+tweak whose new `**Why:**` quotes words tweak.json does not hold.
+
 ### 2. Bring the spec in line, if it disagrees
 
 If the spec says something the change contradicts, or the page will no longer
@@ -1341,6 +1357,11 @@ blocking. It also refuses when a critique's verdict files changed after `critiqu
 wrote them, in a session that did not run `critique`: a fix is judged by the next
 review, never by the builder editing the last one.
 
+When a `critique` or `tweak` session stops, the gate records its verdicts in
+`.jig/critique/<surface>/verdicts.lock`. That happens after you have committed, so
+if your verdicts are committed it stops you once more to commit the lock on its
+own; the next stop passes.
+
 It judges the critiques this session touched: any with a file changed since the
 session began, and the current spec's surface when you ran `critique`. Another
 page's older critique does not hold you back. To set a critique aside for good,