@fringetools/cli 0.2.1 → 0.2.2

package/README.md

@@ -119,6 +119,8 @@ After each edit the hook runs; with errors it returns, for example:
 {"decision":"block","reason":"Fringe found 2 accessibility error(s). Fix them before finishing:\n- FR-203 src/Signup.tsx:9 — input not associated with a label (add id + <label htmlFor>, or aria-label)\n…"}
 ```
 
+The hook also keeps `.fringe/last.json` (the previous run's errors) and tells the agent what its **last edit broke**: "Your last edit introduced 1 new error(s) and fixed 2; 3 failing in total", with the new ones listed first and marked `NEW`. If that file is missing or unreadable it simply skips the comparison. It still blocks on every remaining error; on a legacy repo use the baseline so old debt is not nagged about.
+
 This was verified end to end with real agent runs: the agent receives the findings and fixes them (see [docs/architecture.md](../../docs/architecture.md)). Why `--claude-hook` and not `--hook`: Claude Code only passes a hook's output to the agent as `decision`/`reason` JSON on exit 0 (or stderr on exit 2); the CI contract `--hook` exits 1 and is ignored.
 
 **Skills** — `fringe init` installs five into `.claude/skills/` (sources in [skill/](skill/), one folder each); the Claude Code plugin ships the same folders, namespaced `/fringe:…`: `fringe` (below), `fringe-review` ("review this page": ranked, plain-English findings with fixes), `fringe-audit` (whole-repo plan: top rules and files, a fix order, offers `--update-baseline`), `fringe-debt` (lists `fringe-ignore` suppressions and the baseline) and `fringe-help` (quick reference). Invoke as `/fringe-review` etc. Reviews run `fringe check`, so Claude Code will ask you to approve that command once (or allow `Bash(fringe:*)` / `Bash(npx:*)`).

package/dist/bin.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   runCli
-} from "./chunk-Z4UN7OMW.js";
+} from "./chunk-TNZKMKKQ.js";
 
 // src/bin.ts
 process.exitCode = await runCli(process.argv.slice(2));

package/dist/{chunk-Z4UN7OMW.js → chunk-TNZKMKKQ.js}

@@ -1,10 +1,10 @@
 // src/index.ts
 import { randomUUID } from "crypto";
-import { existsSync as existsSync3, readFileSync as readFileSync8, statSync } from "fs";
-import { dirname as dirname2, join as join8, resolve } from "path";
+import { existsSync as existsSync3, readFileSync as readFileSync9, statSync } from "fs";
+import { dirname as dirname2, join as join9, resolve } from "path";
 
 // src/version.ts
-var VERSION = "0.2.1";
+var VERSION = "0.2.2";
 
 // src/check.ts
 import { readFileSync as readFileSync5 } from "fs";
@@ -144,7 +144,7 @@ function collectText(node) {
   }
   return t.trim();
 }
-function parseJsx(file, source) {
+function parseJsx(file2, source) {
   let ast;
   try {
     ast = parse(source, { sourceType: "unambiguous", plugins: ["jsx", "typescript"], errorRecovery: false });
@@ -162,7 +162,7 @@ function parseJsx(file, source) {
       for (const a of node.openingElement.attributes) {
         if (a.type === "JSXAttribute") attrs[a.name.name] = attrValue(a.value);
       }
-      const el = { tag: tag2, attrs, text: collectText(node), loc: { file, line: node.loc?.start.line ?? 0, col: node.loc?.start.column ?? 0 } };
+      const el = { tag: tag2, attrs, text: collectText(node), loc: { file: file2, line: node.loc?.start.line ?? 0, col: node.loc?.start.column ?? 0 } };
       if (parent) el.parentTag = parent.tag;
       if (parent?.inLabel) el.inLabel = true;
       if (parent?.attrs?.className !== void 0) el.parentClass = parent.attrs.className;
@@ -181,7 +181,7 @@ function parseJsx(file, source) {
 // src/parse/html.ts
 import { parse as parse2 } from "parse5";
 var textOf = (n) => n.nodeName === "#text" ? n.value : (n.childNodes ?? []).map(textOf).join("");
-function parseHtml(file, source) {
+function parseHtml(file2, source) {
   const doc = parse2(source, { sourceCodeLocationInfo: true });
   const elems = [];
   (function walkNode(node, parent) {
@@ -189,7 +189,7 @@ function parseHtml(file, source) {
       const attrs = {};
       for (const a of node.attrs ?? []) attrs[a.name] = a.value;
       const text = textOf(node).trim();
-      const el = { tag: node.tagName, attrs, text, loc: { file, line: node.sourceCodeLocation?.startLine ?? 0, col: node.sourceCodeLocation?.startCol ?? 0 } };
+      const el = { tag: node.tagName, attrs, text, loc: { file: file2, line: node.sourceCodeLocation?.startLine ?? 0, col: node.sourceCodeLocation?.startCol ?? 0 } };
       if (parent) el.parentTag = parent.tag;
       if (parent?.inLabel) el.inLabel = true;
       if (parent?.attrs?.class !== void 0) el.parentClass = parent.attrs.class;
@@ -204,12 +204,12 @@ function parseHtml(file, source) {
 
 // src/parse/css.ts
 import postcss from "postcss";
-function parseCss(file, source) {
+function parseCss(file2, source) {
   const decls = [];
   const rec = (node, selector) => {
     for (const child of node.nodes ?? []) {
       if (child.type === "decl") {
-        decls.push({ prop: child.prop, value: String(child.value), selector, loc: { file, line: child.source?.start?.line ?? 0, col: child.source?.start?.column ?? 0 } });
+        decls.push({ prop: child.prop, value: String(child.value), selector, loc: { file: file2, line: child.source?.start?.line ?? 0, col: child.source?.start?.column ?? 0 } });
       } else if (child.type === "rule" || child.type === "atrule") {
         rec(child, child.type === "rule" ? child.selector : selector);
       }
@@ -1056,10 +1056,10 @@ function gatherTokenColors(root, cfg, cssSources) {
   const kind = cfg.tokenSource?.kind;
   if (kind === "design-tokens-json") {
     const rel = cfg.tokenSource?.path ?? "";
-    const file = isAbsolute(rel) ? rel : join4(root, rel);
+    const file2 = isAbsolute(rel) ? rel : join4(root, rel);
     let json;
     try {
-      json = JSON.parse(readFileSync3(file, "utf8"));
+      json = JSON.parse(readFileSync3(file2, "utf8"));
     } catch {
       return out;
     }
@@ -1386,25 +1386,25 @@ async function runCheck(root, cfg = loadConfig(root), useBaseline = true, only)
   const skipped = [];
   const cssSources = [];
   const contexts = [];
-  for (const file of files) {
+  for (const file2 of files) {
     let source = "";
     try {
-      source = readFileSync5(file, "utf8");
+      source = readFileSync5(file2, "utf8");
     } catch {
-      skipped.push(file);
+      skipped.push(file2);
       continue;
     }
-    const ctx = { file, source, elems: [], css: [], tokenColors: [] };
+    const ctx = { file: file2, source, elems: [], css: [], tokenColors: [] };
     try {
-      if (/\.(jsx|tsx|js|ts)$/.test(file)) ctx.elems = parseJsx(file, source);
-      else if (/\.html$/.test(file)) ctx.elems = parseHtml(file, source);
-      else if (/\.(css|scss)$/.test(file)) {
-        ctx.css = parseCss(file, source);
-        cssSources.push({ file, source });
+      if (/\.(jsx|tsx|js|ts)$/.test(file2)) ctx.elems = parseJsx(file2, source);
+      else if (/\.html$/.test(file2)) ctx.elems = parseHtml(file2, source);
+      else if (/\.(css|scss)$/.test(file2)) {
+        ctx.css = parseCss(file2, source);
+        cssSources.push({ file: file2, source });
       }
     } catch (e) {
       if (e.message.startsWith("parse-skip")) {
-        skipped.push(file);
+        skipped.push(file2);
         continue;
       }
       throw e;
@@ -1454,10 +1454,14 @@ function toHook(r) {
     reason
   });
 }
-function toClaudeHook(r, root) {
-  const errs = r.findings.filter((f) => f.severity === "error");
-  if (errs.length === 0) return "";
-  const shown = errs.slice(0, 8).map((f) => `- ${f.rule} ${relative4(root, f.loc.file)}:${f.loc.line} \u2014 ${f.message} (${f.hint})`);
+function toClaudeHook(r, root, history) {
+  const all = r.findings.filter((f) => f.severity === "error");
+  if (all.length === 0) return "";
+  const fresh = new Set(history?.introduced);
+  const errs = [...all.filter((f) => fresh.has(f)), ...all.filter((f) => !fresh.has(f))];
+  const note = history && (history.introduced.length > 0 || history.fixed > 0) ? `Your last edit introduced ${history.introduced.length} new error(s)${history.fixed ? ` and fixed ${history.fixed}` : ""}; ${errs.length} failing in total. New ones are marked NEW.
+` : "";
+  const shown = errs.slice(0, 8).map((f) => `- ${fresh.has(f) ? "NEW " : ""}${f.rule} ${relative4(root, f.loc.file)}:${f.loc.line} \u2014 ${f.message} (${f.hint})`);
   const shownRules = [...new Set(errs.slice(0, 8).map((f) => f.rule))];
   const recipes = shownRules.map((id) => {
     const g = GUIDANCE[id];
@@ -1467,7 +1471,7 @@ function toClaudeHook(r, root) {
 \u2026and ${errs.length - shown.length} more (run \`fringe check\`).` : "";
   return JSON.stringify({
     decision: "block",
-    reason: `Fringe found ${errs.length} accessibility error(s). Fix them before finishing:
+    reason: `${note}Fringe found ${errs.length} accessibility error(s). Fix them before finishing:
 ${shown.join("\n")}${more}
 How to fix them well:
 ${recipes.join("\n")}
@@ -1562,19 +1566,51 @@ function startBoard(root, port) {
   });
 }
 
+// src/receipts.ts
+import { mkdirSync as mkdirSync2, readFileSync as readFileSync7, writeFileSync as writeFileSync2 } from "fs";
+import { join as join7 } from "path";
+var file = (root) => join7(root, ".fringe", "last.json");
+function receipts(root, findings) {
+  const errs = findings.filter((f) => f.severity === "error");
+  const keys = errs.map((f) => keyOf(root, f));
+  let prev;
+  try {
+    const p = JSON.parse(readFileSync7(file(root), "utf8"));
+    if (Array.isArray(p)) prev = p;
+  } catch {
+  }
+  try {
+    mkdirSync2(join7(root, ".fringe"), { recursive: true });
+    writeFileSync2(file(root), JSON.stringify(keys));
+  } catch {
+  }
+  if (!prev) return void 0;
+  const left = /* @__PURE__ */ new Map();
+  for (const k of prev) left.set(k, (left.get(k) ?? 0) + 1);
+  const introduced = errs.filter((f, i) => {
+    const n = left.get(keys[i]) ?? 0;
+    if (n > 0) {
+      left.set(keys[i], n - 1);
+      return false;
+    }
+    return true;
+  });
+  return { introduced, fixed: [...left.values()].reduce((a, b) => a + b, 0) };
+}
+
 // src/init.ts
-import { existsSync as existsSync2, mkdirSync as mkdirSync2, readFileSync as readFileSync7, readdirSync as readdirSync3, writeFileSync as writeFileSync2, copyFileSync } from "fs";
-import { dirname, join as join7 } from "path";
+import { existsSync as existsSync2, mkdirSync as mkdirSync3, readFileSync as readFileSync8, readdirSync as readdirSync3, writeFileSync as writeFileSync3, copyFileSync } from "fs";
+import { dirname, join as join8 } from "path";
 import { fileURLToPath } from "url";
 var HOOK = "npx -y @fringetools/cli check --claude-hook";
 var SKILL_DIR = fileURLToPath(new URL("../skill/", import.meta.url));
 function initProject(root) {
   const done = [];
-  const settingsPath = join7(root, ".claude", "settings.json");
+  const settingsPath = join8(root, ".claude", "settings.json");
   let settings = {};
   if (existsSync2(settingsPath)) {
     try {
-      settings = JSON.parse(readFileSync7(settingsPath, "utf8"));
+      settings = JSON.parse(readFileSync8(settingsPath, "utf8"));
     } catch {
       throw new Error(`${settingsPath} is not valid JSON \u2014 fix it and rerun`);
     }
@@ -1583,16 +1619,16 @@ function initProject(root) {
   if (/(fringe|@fringetools\/cli) check/.test(JSON.stringify(post))) done.push("hook already present");
   else {
     post.push({ matcher: "Edit|Write", hooks: [{ type: "command", command: HOOK }] });
-    mkdirSync2(dirname(settingsPath), { recursive: true });
-    writeFileSync2(settingsPath, JSON.stringify(settings, null, 2) + "\n");
+    mkdirSync3(dirname(settingsPath), { recursive: true });
+    writeFileSync3(settingsPath, JSON.stringify(settings, null, 2) + "\n");
     done.push(`hook added to ${settingsPath}`);
   }
   for (const name of readdirSync3(SKILL_DIR)) {
-    const skillPath = join7(root, ".claude", "skills", name, "SKILL.md");
+    const skillPath = join8(root, ".claude", "skills", name, "SKILL.md");
     if (existsSync2(skillPath)) done.push(`skill ${name} already present`);
     else {
-      mkdirSync2(dirname(skillPath), { recursive: true });
-      copyFileSync(join7(SKILL_DIR, name, "SKILL.md"), skillPath);
+      mkdirSync3(dirname(skillPath), { recursive: true });
+      copyFileSync(join8(SKILL_DIR, name, "SKILL.md"), skillPath);
       done.push(`skill ${name} installed`);
     }
   }
@@ -1722,7 +1758,7 @@ var err = (o) => o.err ?? ((s) => process.stderr.write(s));
 function readManifest(path2, warn) {
   let raw;
   try {
-    raw = readFileSync8(path2, "utf8");
+    raw = readFileSync9(path2, "utf8");
   } catch {
     warn(`fringe: warning: manifest not found at ${path2} \u2014 continuing without
 `);
@@ -1749,7 +1785,7 @@ function resolveTarget(arg) {
   if (!statSync(arg).isFile()) return { root: arg };
   let dir = dirname2(arg);
   for (let d = dir, i = 0; i < 12; i++, d = dirname2(d)) {
-    if (existsSync3(join8(d, "package.json")) || existsSync3(join8(d, "fringe.config.json"))) {
+    if (existsSync3(join9(d, "package.json")) || existsSync3(join9(d, "fringe.config.json"))) {
       dir = d;
       break;
     }
@@ -1805,7 +1841,7 @@ async function runCli(argv, opts = {}) {
       }
       const result = await runCheck(root, cfg, true, only);
       if (rest.includes("--claude-hook")) {
-        const out = toClaudeHook(result, root);
+        const out = toClaudeHook(result, root, receipts(root, result.findings));
         if (out) log(opts)(out);
         return 0;
       }

package/dist/index.js

@@ -1,6 +1,6 @@
 import {
   runCli
-} from "./chunk-Z4UN7OMW.js";
+} from "./chunk-TNZKMKKQ.js";
 export {
   runCli
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fringetools/cli",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "license": "MIT",
   "type": "module",
   "bin": {