jig-ui 0.17.0 → 0.17.1

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.17.1
+
+### Fixed
+
+- **The gate judges the critiques a session touched, not every critique.**
+  A spec for one page could not finish because another page's critique
+  predated 0.17.0, whose new rules that critique had never judged, and a
+  record the project had set aside still failed for screenshots it no longer
+  had. The gate now checks a critique with a file changed since the session
+  began, and the current spec's surface after `critique`. A folder whose name
+  starts with `_` is set aside and never judged. With no transcript, as when
+  the gate is run by hand, it checks every critique as before.
+
 ## 0.17.0
 
 Thirteen more tells of a generated page, ten of them caught by `check`, and a

package/dist/index.js

@@ -6159,7 +6159,7 @@ async function ensureProbes(opts) {
 function critiquedSurfaces(projectRoot) {
   const root = join32(projectRoot, ".jig", "critique");
   if (!existsSync19(root)) return [];
-  return readdirSync12(root).filter((surface) => existsSync19(join32(root, surface, "screen.json")) || existsSync19(join32(root, surface, "code.json")));
+  return readdirSync12(root).filter((surface) => !surface.startsWith("_") && (existsSync19(join32(root, surface, "screen.json")) || existsSync19(join32(root, surface, "code.json"))));
 }
 
 // src/probe/check.ts
@@ -6575,7 +6575,7 @@ function verifyVerdicts(opts) {
 }
 
 // src/commands/gate.ts
-import { existsSync as existsSync24, mkdirSync as mkdirSync8, readdirSync as readdirSync15, readFileSync as readFileSync30, writeFileSync as writeFileSync7 } from "fs";
+import { existsSync as existsSync24, mkdirSync as mkdirSync8, readdirSync as readdirSync15, readFileSync as readFileSync30, statSync as statSync4, writeFileSync as writeFileSync7 } from "fs";
 import { createHash as createHash2 } from "crypto";
 import { join as join37 } from "path";
 var MAX_BLOCKS = 3;
@@ -6624,6 +6624,37 @@ function asksOwner(text) {
   return /\?(?=[\s)\]"'*_]|$)/.test(prose);
 }
 var ASKS_THE_OWNER = /* @__PURE__ */ new Set(["decide", "spec", "mockup"]);
+function sessionStart(transcriptPath) {
+  if (!transcriptPath || !existsSync24(transcriptPath)) return void 0;
+  try {
+    for (const line of readFileSync30(transcriptPath, "utf8").split("\n")) {
+      const m = /"timestamp"\s*:\s*"([^"]+)"/.exec(line);
+      if (m) {
+        const t = Date.parse(m[1]);
+        if (!Number.isNaN(t)) return t;
+      }
+    }
+  } catch {
+  }
+  return void 0;
+}
+function newestMtime(dir) {
+  let newest = 0;
+  for (const name of readdirSync15(dir, { withFileTypes: true })) {
+    const path = join37(dir, name.name);
+    newest = Math.max(newest, name.isDirectory() ? newestMtime(path) : statSync4(path).mtimeMs);
+  }
+  return newest;
+}
+function surfacesInPlay(root, command, transcriptPath) {
+  const critiqueDir = join37(root, ".jig", "critique");
+  if (!existsSync24(critiqueDir)) return [];
+  const all = readdirSync15(critiqueDir, { withFileTypes: true }).filter((d) => d.isDirectory() && !d.name.startsWith("_") && !d.name.startsWith(".")).map((d) => d.name);
+  const start = sessionStart(transcriptPath);
+  if (start === void 0) return all;
+  const current = command === "critique" ? /^\s*surface\s*:\s*(.+)$/im.exec(newestSpec(root)?.body.split(/^---\s*$/m)[1] ?? "")?.[1]?.trim().replace(/^["']|["']$/g, "") : void 0;
+  return all.filter((s) => s === current || newestMtime(join37(critiqueDir, s)) >= start - 1e3);
+}
 function commandProblems(root, command) {
   const problems = [];
   const spec = newestSpec(root);
@@ -6715,7 +6746,7 @@ function gate(opts) {
   problems.push(...verdictGuard(root, command));
   const critiqueDir = join37(root, ".jig", "critique");
   if (existsSync24(critiqueDir)) {
-    for (const surface of readdirSync15(critiqueDir)) {
+    for (const surface of surfacesInPlay(root, command, opts.input.transcript_path)) {
       const dir = join37(critiqueDir, surface);
       if (!existsSync24(join37(dir, "screen.json")) && !existsSync24(join37(dir, "code.json"))) continue;
       const v = verifyVerdicts({ projectRoot: root, surface });
@@ -6780,6 +6811,7 @@ function verdictGuard(root, command) {
   if (!existsSync24(critiqueDir)) return [];
   const problems = [];
   for (const surface of readdirSync15(critiqueDir)) {
+    if (surface.startsWith("_")) continue;
     const dir = join37(critiqueDir, surface);
     const now = verdictChecksum(dir);
     if (!now) continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jig-ui",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "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

@@ -1191,6 +1191,12 @@ 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.
 
+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,
+rename its folder with a leading underscore (`.jig/critique/_old-catalog`); the
+gate never judges it.
+
 A warning you are sure is right as it stands, because the detector guessed and the
 guess is wrong here, is waived on its own line or the line above, in the file's
 comment syntax: `jig-allow <ID>: <why>`. The reason is what the user reads, so write