jig-ui 0.18.6 → 0.18.8

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## 0.18.8 (2026-09-25)
+
+Two runs writing one manifest no longer lose an entry.
+
+### Fixed
+
+- **Two runs writing one manifest no longer lose an entry.** The merge added
+  for this in 0.4 re-read after writing and retried if its own keys were gone,
+  but a writer only checked its own keys: one that read before another's write
+  could rename over it afterwards, and both reported success. The
+  parallel-process test caught it under load in the 0.18.7 release run. The
+  read, merge and write now happen under a lock file; a lock whose process has
+  died, or older than any write takes, is taken over, so a killed run cannot
+  wedge an install.
+
+## 0.18.7 (2026-09-25)
+
+The mockup gate reads a spec's switches however they are spelled, and refuses
+icons and images in a drawing's frames.
+
+### Fixed
+
+- **`switches:` reads a switch however the spec spells it.** jig-site's
+  Versions spec wrote `switches: [breakpoint-nav]`; read as a switch named
+  that, it matched nothing, no switch was checked, and the gate passed. `nav`,
+  `breakpoint-nav` and `--breakpoint-nav` now name the same switch, and a name
+  the project does not record is reported with the ones it does.
+- **The gate refuses icons and images in a mockup's frames.** The procedure
+  says an icon is its name in brackets and an image a labelled box; jig-site's
+  Versions drawing carried the site's real icons and GitHub's mark, and nothing
+  checked. `<svg>`, `<img>`, `<picture>`, `<canvas>` and icon fonts inside a
+  frame now stop the session.
+
 ## 0.18.6 (2026-09-25)
 
 A mockup's frames must hold each size's regions, and the drawing shows either

package/dist/index.js

@@ -42,7 +42,7 @@ function isPublishedBuild(packageRoot2) {
 }
 
 // src/commands/install.ts
-import { existsSync as existsSync5, mkdirSync as mkdirSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync3 } from "fs";
+import { existsSync as existsSync5, mkdirSync as mkdirSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync2 } from "fs";
 import { join as join7 } from "path";
 
 // src/adapters/types.ts
@@ -270,7 +270,7 @@ function referenceFiles(packageRoot2) {
 
 // src/install/manifest.ts
 import { createHash } from "crypto";
-import { existsSync as existsSync3, mkdirSync as mkdirSync2, readFileSync as readFileSync2 } from "fs";
+import { closeSync, existsSync as existsSync3, mkdirSync as mkdirSync2, openSync, readFileSync as readFileSync2, statSync, unlinkSync as unlinkSync2, writeSync } from "fs";
 import { dirname as dirname3, join as join4 } from "path";
 
 // src/install/atomic.ts
@@ -342,19 +342,70 @@ function readManifest(root, manifestDir = ".jig") {
   }
   return validateManifest(raw, path);
 }
-var MERGE_ATTEMPTS = 5;
+var LOCK_STALE_MS = 1e4;
+var LOCK_WAIT_MS = 15e3;
 function writeManifest(root, m, manifestDir = ".jig") {
   const path = manifestPath(root, manifestDir);
   mkdirSync2(dirname3(path), { recursive: true });
-  for (let attempt = 0; attempt < MERGE_ATTEMPTS; attempt++) {
+  withLock(`${path}.lock`, () => {
     const onDisk = readManifestQuietly(path);
     const merged = { ...m, files: { ...onDisk?.files ?? {}, ...m.files } };
     writeFileAtomic(path, `${JSON.stringify(merged, null, 2)}
 `);
-    const after = readManifestQuietly(path);
-    if (after && Object.keys(merged.files).every((k) => k in after.files)) return;
+  });
+}
+function withLock(lock, fn) {
+  const deadline = Date.now() + LOCK_WAIT_MS;
+  for (; ; ) {
+    try {
+      const fd = openSync(lock, "wx");
+      try {
+        writeSync(fd, `${process.pid} ${Date.now()}`);
+      } finally {
+        closeSync(fd);
+      }
+      break;
+    } catch (err) {
+      if (err.code !== "EEXIST") throw err;
+      if (lockIsStale(lock) || Date.now() > deadline) {
+        try {
+          unlinkSync2(lock);
+        } catch {
+        }
+        continue;
+      }
+      sleep(10 + Math.random() * 20);
+    }
+  }
+  try {
+    fn();
+  } finally {
+    try {
+      unlinkSync2(lock);
+    } catch {
+    }
+  }
+}
+function lockIsStale(lock) {
+  try {
+    const [pid, at] = readFileSync2(lock, "utf8").trim().split(/\s+/).map(Number);
+    if (at && Date.now() - at > LOCK_STALE_MS) return true;
+    if (pid && pid !== process.pid) {
+      try {
+        process.kill(pid, 0);
+      } catch (e) {
+        if (e.code === "ESRCH") return true;
+      }
+    }
+    if (!pid || !at) return Date.now() - statSync(lock).mtimeMs > LOCK_STALE_MS;
+    return false;
+  } catch {
+    return false;
   }
 }
+function sleep(ms) {
+  Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
+}
 function readManifestQuietly(path) {
   try {
     const parsed = JSON.parse(readFileSync2(path, "utf8"));
@@ -695,7 +746,7 @@ function installStopHook(projectRoot, version2) {
   stop.push({ hooks: [{ type: "command", command }] });
   settings.hooks = { ...hooks, Stop: stop };
   mkdirSync4(join7(projectRoot, ".claude"), { recursive: true });
-  writeFileSync3(path, JSON.stringify(settings, null, 2) + "\n", "utf8");
+  writeFileSync2(path, JSON.stringify(settings, null, 2) + "\n", "utf8");
   return true;
 }
 
@@ -764,11 +815,11 @@ function readInitManifest(projectRoot) {
     return null;
   }
 }
-var MERGE_ATTEMPTS2 = 5;
+var MERGE_ATTEMPTS = 5;
 function writeInitManifest(projectRoot, m) {
   const path = join8(projectRoot, INIT_MANIFEST_REL);
   mkdirSync5(dirname5(path), { recursive: true });
-  for (let attempt = 0; attempt < MERGE_ATTEMPTS2; attempt++) {
+  for (let attempt = 0; attempt < MERGE_ATTEMPTS; attempt++) {
     const onDisk = readInitManifest(projectRoot);
     const merged = {
       ...m,
@@ -1476,7 +1527,7 @@ import { join as join24 } from "path";
 
 // src/check/files.ts
 import { execFileSync } from "child_process";
-import { readdirSync as readdirSync7, realpathSync, statSync } from "fs";
+import { readdirSync as readdirSync7, realpathSync, statSync as statSync2 } from "fs";
 import { join as join16, relative, sep } from "path";
 var EXCLUDE_DIRS = /* @__PURE__ */ new Set([
   "node_modules",
@@ -1557,7 +1608,7 @@ function walk(root, dir, out, seen) {
     let isFile = entry.isFile();
     if (entry.isSymbolicLink()) {
       try {
-        const target = statSync(abs);
+        const target = statSync2(abs);
         isDir = target.isDirectory();
         isFile = target.isFile();
       } catch {
@@ -4584,7 +4635,7 @@ function check(opts) {
 }
 
 // src/commands/init.ts
-import { existsSync as existsSync16, mkdirSync as mkdirSync6, readdirSync as readdirSync10, readFileSync as readFileSync23, rmSync as rmSync2, writeFileSync as writeFileSync5 } from "fs";
+import { existsSync as existsSync16, mkdirSync as mkdirSync6, readdirSync as readdirSync10, readFileSync as readFileSync23, rmSync as rmSync2, writeFileSync as writeFileSync4 } from "fs";
 import { dirname as dirname7, isAbsolute, join as join28, resolve as resolve2, sep as sep3 } from "path";
 
 // src/init/detect.ts
@@ -5215,7 +5266,7 @@ function wireBarrel(absPath, barrelImport) {
   if (legacy) {
     const next2 = content.slice(0, legacy.index) + `${line}
 ` + content.slice(legacy.index + legacy[0].length);
-    writeFileSync5(absPath, next2, "utf8");
+    writeFileSync4(absPath, next2, "utf8");
     return { status: "rewired", detail: "replaced the separate brand and mode imports with the barrel" };
   }
   const tailwind = /@import\s+["']tailwindcss["'];?\r?\n?/.exec(content);
@@ -5225,7 +5276,7 @@ function wireBarrel(absPath, barrelImport) {
 ` : `${line}
 `);
   mkdirSync6(dirname7(absPath), { recursive: true });
-  writeFileSync5(next === content ? absPath : absPath, next, "utf8");
+  writeFileSync4(next === content ? absPath : absPath, next, "utf8");
   return { status: "wired" };
 }
 async function init(opts) {
@@ -5385,7 +5436,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
     const defaultCssRaw = readFileSync23(join28(opts.packageRoot, "tokens", "brand.default.css"), "utf8");
     const content = renderBrandFile(defaultCssRaw, projectSlug, opts.version, proposal);
     mkdirSync6(dirname7(brandAbsPath), { recursive: true });
-    writeFileSync5(brandAbsPath, content, "utf8");
+    writeFileSync4(brandAbsPath, content, "utf8");
     files[brandRelPath] = checksum(content);
   }
   if (relocateTo && configAction !== "written") {
@@ -5393,7 +5444,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
       const existing = JSON.parse(readFileSync23(configAbsPath, "utf8"));
       const content = `${JSON.stringify({ ...existing, brand: brandRelPath }, null, 2)}
 `;
-      writeFileSync5(configAbsPath, content, "utf8");
+      writeFileSync4(configAbsPath, content, "utf8");
       files[configRelPath] = checksum(content);
       log(`  Updated ${configRelPath} to point at the new location.`);
     } catch {
@@ -5404,7 +5455,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
     const config = opts.yes ? { brand: brandRelPath } : { brand: brandRelPath, surfaces };
     const content = `${JSON.stringify(config, null, 2)}
 `;
-    writeFileSync5(configAbsPath, content, "utf8");
+    writeFileSync4(configAbsPath, content, "utf8");
     files[configRelPath] = checksum(content);
   } else if (configAction === "merged") {
     let existing = {};
@@ -5424,7 +5475,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
     };
     const content = `${JSON.stringify(merged, null, 2)}
 `;
-    writeFileSync5(configAbsPath, content, "utf8");
+    writeFileSync4(configAbsPath, content, "utf8");
     files[configRelPath] = checksum(content);
   }
   const effectiveConfig = loadEffectiveConfig(opts.projectRoot, configAbsPath, brandRelPath, surfaces);
@@ -5445,7 +5496,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
       const modeSourceRaw = readFileSync23(join28(opts.packageRoot, "tokens", modeFileName), "utf8");
       const content = vendorHeader(modeFileName, opts.version, "css", null) + modeSourceRaw;
       mkdirSync6(dirname7(modeAbsPath2), { recursive: true });
-      writeFileSync5(modeAbsPath2, content, "utf8");
+      writeFileSync4(modeAbsPath2, content, "utf8");
       files[modeRelPath] = checksum(content);
     } else {
       log(`  ${modeRelPath} exists and is not jig-tracked (or has been edited) \u2014 leaving it alone (re-run 'jig update' to refresh it).`);
@@ -5465,7 +5516,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
     }
     const content = barrelBody(barrelFor(mode), brandFileOnly, mode, opts.version);
     mkdirSync6(dirname7(abs), { recursive: true });
-    writeFileSync5(abs, content, "utf8");
+    writeFileSync4(abs, content, "utf8");
     files[rel] = checksum(content);
   }
   if (multiMode) {
@@ -5518,7 +5569,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
         } else {
           const content = utilitiesBody(aliasable, opts.version);
           mkdirSync6(dirname7(abs), { recursive: true });
-          writeFileSync5(abs, content, "utf8");
+          writeFileSync4(abs, content, "utf8");
           files[rel] = checksum(content);
           const target = findWireTarget(detection);
           const importPath = target ? relativeImportPath(
@@ -5570,7 +5621,7 @@ ${configRelPath} already exists and was not created by 'jig init' (or has been e
           return !(m && !/^[a-z]+:|^\//i.test(m[1]) && resolve2(dirname7(abs), m[1]) === bare2);
         }).join("\n");
         if (after !== before) {
-          writeFileSync5(abs, after, "utf8");
+          writeFileSync4(abs, after, "utf8");
           log(`
 Unwired ${file}: removed its import of ${relKey(...tokensRelDir, "theme.css")}.`);
           log(`  With more than one mode, the global stylesheet imports no barrel: every route would carry that mode's tokens.`);
@@ -5598,7 +5649,7 @@ Could not edit ${file}: ${err.message}. Remove its import of ${relKey(...tokensR
         const oldDirName = currentBrand.split("/").slice(0, -1).join("/");
         const before = existsSync16(abs) ? readFileSync23(abs, "utf8") : "";
         const after = before.split("\n").filter((l) => !(l.includes("@import") && l.includes(oldDirName))).join("\n");
-        if (after !== before) writeFileSync5(abs, after, "utf8");
+        if (after !== before) writeFileSync4(abs, after, "utf8");
       }
       const outcome = wireBarrel(join28(opts.projectRoot, wireTarget), barrelImport);
       wiring = { target: wireTarget, status: outcome.status, snippet };
@@ -5693,7 +5744,7 @@ import { existsSync as existsSync20, readdirSync as readdirSync13, readFileSync
 import { join as join34 } from "path";
 
 // src/probe/save.ts
-import { existsSync as existsSync19, mkdirSync as mkdirSync7, readdirSync as readdirSync12, readFileSync as readFileSync26, writeFileSync as writeFileSync6 } from "fs";
+import { existsSync as existsSync19, mkdirSync as mkdirSync7, readdirSync as readdirSync12, readFileSync as readFileSync26, writeFileSync as writeFileSync5 } from "fs";
 import { fileURLToPath as fileURLToPath2 } from "url";
 import { join as join33, relative as relative3, resolve as resolve4, sep as sep5 } from "path";
 
@@ -6042,7 +6093,7 @@ async function evaluate(endpoint, url, width, height, timeoutMs, expression) {
 
 // src/probe/serve.ts
 import { createServer } from "http";
-import { existsSync as existsSync18, readFileSync as readFileSync24, statSync as statSync2 } from "fs";
+import { existsSync as existsSync18, readFileSync as readFileSync24, statSync as statSync3 } from "fs";
 import { extname, join as join31, resolve as resolve3, sep as sep4 } from "path";
 var TYPES = {
   ".html": "text/html; charset=utf-8",
@@ -6075,7 +6126,7 @@ async function serveDirectory(dir) {
       res.writeHead(403).end();
       return;
     }
-    if (existsSync18(file) && statSync2(file).isDirectory()) file = join31(file, "index.html");
+    if (existsSync18(file) && statSync3(file).isDirectory()) file = join31(file, "index.html");
     else if (!existsSync18(file) && existsSync18(`${file}.html`)) file = `${file}.html`;
     if (!existsSync18(file)) {
       res.writeHead(404).end();
@@ -6143,7 +6194,7 @@ function saveProbe(opts) {
   const dir = join33(opts.projectRoot, ".jig", "critique", opts.surface);
   mkdirSync7(dir, { recursive: true });
   const path = join33(dir, `probe-${width}.json`);
-  writeFileSync6(path, JSON.stringify(probe), "utf8");
+  writeFileSync5(path, JSON.stringify(probe), "utf8");
   return { path: `.jig/critique/${opts.surface}/probe-${width}.json`, width, page: probe.pageFile };
 }
 function projectFile(projectRoot, path) {
@@ -6392,14 +6443,14 @@ function decisionHeadings(projectRoot) {
 }
 
 // src/check/spec-shape.ts
-import { existsSync as existsSync22, readFileSync as readFileSync29, readdirSync as readdirSync14, statSync as statSync3 } from "fs";
+import { existsSync as existsSync22, readFileSync as readFileSync29, readdirSync as readdirSync14, statSync as statSync4 } from "fs";
 import { join as join36 } from "path";
 function newestSpec(projectRoot) {
   const dir = join36(projectRoot, ".jig", "specs");
   if (!existsSync22(dir)) return void 0;
   const files = readdirSync14(dir).filter((f) => f.endsWith(".md"));
   if (files.length === 0) return void 0;
-  const newest = files.map((f) => ({ f, at: statSync3(join36(dir, f)).mtimeMs })).sort((a, b) => b.at - a.at)[0].f;
+  const newest = files.map((f) => ({ f, at: statSync4(join36(dir, f)).mtimeMs })).sort((a, b) => b.at - a.at)[0].f;
   return { path: `.jig/specs/${newest}`, slug: newest.replace(/\.spec\.md$|\.md$/, ""), body: readFileSync29(join36(dir, newest), "utf8") };
 }
 function specProblems(spec) {
@@ -6685,7 +6736,7 @@ function verifyVerdicts(opts) {
 }
 
 // src/commands/gate.ts
-import { existsSync as existsSync24, mkdirSync as mkdirSync8, readdirSync as readdirSync15, readFileSync as readFileSync32, statSync as statSync4, writeFileSync as writeFileSync7 } from "fs";
+import { existsSync as existsSync24, mkdirSync as mkdirSync8, readdirSync as readdirSync15, readFileSync as readFileSync32, statSync as statSync5, writeFileSync as writeFileSync6 } from "fs";
 import { createHash as createHash2 } from "crypto";
 import { join as join39 } from "path";
 
@@ -6708,6 +6759,12 @@ function mockupDrawingProblems(root, specBody, at) {
       `${at} has no frame for ${missing.map(([size, px2]) => `${size} (${px2}px)`).join(", ")}. A mockup draws the page once per size, side by side, in the wireframe template's \`<section class="frame" data-size="\u2026">\` boxes, so the owner sees every size at once. A single responsive page shows only the width of the window it is opened in.`
     );
   }
+  const drawn = frames.flatMap((f) => f.art.map((a) => `${f.size ?? "a"} frame: ${a}`));
+  if (drawn.length) {
+    problems.push(
+      `${at} draws icons or images in its frames (${[...new Set(drawn)].slice(0, 4).join("; ")}). A mockup is structure only: an icon is its name in brackets, \`[search]\`, and an image is a labelled box saying what goes there.`
+    );
+  }
   const regions = specRegions(specBody);
   for (const [size] of FRAMES) {
     const listed = regions[size];
@@ -6727,7 +6784,14 @@ function mockupDrawingProblems(root, specBody, at) {
   }
   const widths = new Set(frames.map((f) => f.width).filter((w) => w !== void 0));
   const crossed = specSwitches(specBody);
-  for (const s of declaredSwitches(root).filter((sw) => !crossed || crossed.includes(sw.name))) {
+  const recorded = declaredSwitches(root);
+  const unknown = (crossed ?? []).filter((name) => !recorded.some((sw) => sw.name === name));
+  if (unknown.length) {
+    problems.push(
+      `The spec's \`switches:\` names ${unknown.map((n) => `"${n}"`).join(", ")}, which the project does not record. Recorded: ${recorded.map((sw) => `${sw.name} (${sw.px}px)`).join(", ") || "none"}. Name them as \`--breakpoint-<name>\` gives them.`
+    );
+  }
+  for (const s of recorded.filter((sw) => !crossed || crossed.includes(sw.name))) {
     const absent = [s.px - 1, s.px].filter((w) => !widths.has(w));
     if (absent.length) {
       problems.push(
@@ -6742,7 +6806,7 @@ function specSwitches(specBody) {
   const value = /^switches\s*:\s*(.+)$/im.exec(front)?.[1]?.replace(/#.*$/, "").trim();
   if (value === void 0) return void 0;
   if (/^(none|\[\s*\])$/i.test(value)) return [];
-  return value.replace(/^\[|\]$/g, "").split(",").map((n) => n.trim().replace(/^["']|["']$/g, "").replace(/^--breakpoint-/, "")).filter(Boolean);
+  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)];
@@ -6752,7 +6816,8 @@ function readFrames(html) {
     const size = /\bdata-size\s*=\s*["']([\w-]+)["']/i.exec(tag)?.[1]?.toLowerCase();
     const width = Number(/\bdata-width\s*=\s*["'](\d+)["']/i.exec(tag)?.[1]) || void 0;
     const labels = [...body.matchAll(/<\w+\b[^>]*\bclass\s*=\s*["'][^"']*\bname\b[^"']*["'][^>]*>([^<]*)/gi)].map((m) => normalise(m[1])).filter(Boolean);
-    return { size, width, labels };
+    const art = [...body.matchAll(/<(svg|img|picture|canvas)\b|<i\b[^>]*\bclass\s*=\s*["'][^"']*\b(?:fa|fa-[\w-]+|material-icons|material-symbols[\w-]*|bi|bi-[\w-]+)\b/gi)].map((m) => m[1] ? `<${m[1].toLowerCase()}>` : "an icon font");
+    return { size, width, labels, art };
   });
 }
 function specRegions(specBody) {
@@ -6882,7 +6947,7 @@ function verdictsMtime(dir) {
   let newest = 0;
   for (const name of VERDICT_FILES) {
     try {
-      newest = Math.max(newest, statSync4(join39(dir, name)).mtimeMs);
+      newest = Math.max(newest, statSync5(join39(dir, name)).mtimeMs);
     } catch {
     }
   }
@@ -7061,7 +7126,7 @@ function verdictGuard(root, command, inPlay) {
     const lockPath = join39(dir, LOCK);
     if (command === "critique" && (!inPlay || inPlay.includes(surface))) {
       try {
-        writeFileSync7(lockPath, JSON.stringify({ checksum: now }) + "\n", "utf8");
+        writeFileSync6(lockPath, JSON.stringify({ checksum: now }) + "\n", "utf8");
       } catch {
       }
       continue;
@@ -7083,7 +7148,7 @@ function verdictGuard(root, command, inPlay) {
 function save(file, state) {
   try {
     mkdirSync8(join39(file, ".."), { recursive: true });
-    writeFileSync7(file, JSON.stringify(state), "utf8");
+    writeFileSync6(file, JSON.stringify(state), "utf8");
   } catch {
   }
 }

package/package.json

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

@@ -645,8 +645,8 @@ for is review: is the header taking a sixth of the phone? Is a tap target under
 down the spec's `regions:` and its `nav:` and find each one in that frame. A region
 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), and a
-frame either side of each recorded switch. This matters
+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
 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.