kaizen-agent 1.40.0 → 1.40.2

package/cli/board.test.ts

@@ -157,7 +157,7 @@ test("card menu items: appropriate actions and distinct icons across phases", ()
   const menuSnippet = (cards.slice(from, cards.indexOf('el.innerHTML += `<div class="menu-pop"', from)) + "}").replace("const items = [];", "items = [];");
   const a0 = cards.indexOf("const actionOf =");
   const actionOf = cards.slice(a0, cards.indexOf("\n\n", a0));
-  const getItems = new Function("c", "archive", "S", `const ico = ${ico.toString()};\n${actionOf}\nconst store = { S, menuKey: 1 };\nlet k = 1, items = [];\n${menuSnippet}\nreturn items;`);
+  const getItems = new Function("c", "archive", "S", `const ico = ${ico.toString()};\nconst decided = new Set(), answer = () => "";\n${actionOf}\nconst store = { S, menuKey: 1 };\nlet k = 1, items = [];\n${menuSnippet}\nreturn items;`);
 
   // Phase 0: Ideas (idea card)
   const idea = { kind: "idea", status: "idea", column: 0 };

package/cli/state.test.ts

@@ -145,6 +145,20 @@ test("backlog: open items only, orphanage last, count matches", () => {
   expect(backlog(state)).toBe(3);
 });
 
+test("backlog: an abandoned run's items are neither listed nor counted", () => {
+  const dir = join(state, "runs", "2026-09-03-dropped");
+  mkdirSync(dir, { recursive: true });
+  writeFileSync(join(dir, "state.json"), JSON.stringify({ id: "2026-09-03-dropped", stage: "abandoned" }));
+  writeFileSync(join(dir, "06-backlog.md"), "- open: never happening\n");
+  expect(allBacklog(state).some((g) => g.run === "2026-09-03-dropped")).toBe(false);
+  expect(backlog(state)).toBe(3);
+  rmSync(dir, { recursive: true });
+});
+
+test("abandonRun refuses a done run", () => {
+  expect(abandonRun(state, "2026-09-02-done", "late")).toMatch(/already done/);
+});
+
 test("parseItem: finding line splits into where, severity, text", () => {
   const it = parseItem("`cli/x.ts:12`: low: first thing. Fix: do it.");
   expect([it.where, it.severity, it.text]).toEqual(["x.ts:12", "low", "first thing."]);

package/cli/state.ts

@@ -435,7 +435,7 @@ export function unrechecked(state: string, id: string) {
 export function backlog(state: string) {
   const files = [join(state, "backlog.md")];
   const runs = join(state, "runs");
-  if (existsSync(runs)) for (const id of readdirSync(runs)) files.push(join(runs, id, "06-backlog.md"));
+  if (existsSync(runs)) for (const id of readdirSync(runs)) if (!abandoned(join(runs, id))) files.push(join(runs, id, "06-backlog.md"));
   let n = 0;
   for (const f of files) {
     if (!existsSync(f)) continue;
@@ -581,11 +581,17 @@ export function readBacklog(file: string) {
     .map((l) => l.replace(/^-\s*open:\s*/, ""));
 }
 
+// An abandoned run's backlog is not open work (spec-main.md).
+function abandoned(run: string) {
+  try { return JSON.parse(readText(join(run, "state.json"))).stage === "abandoned"; } catch { return false; }
+}
+
 export function allBacklog(state: string) {
   const out: { run: string; items: string[] }[] = [];
   const runs = join(state, "runs");
   if (existsSync(runs)) {
     for (const id of readdirSync(runs).sort().reverse()) {
+      if (abandoned(join(runs, id))) continue;
       const items = readBacklog(join(runs, id, "06-backlog.md"));
       if (items.length) out.push({ run: id, items });
     }
@@ -880,6 +886,7 @@ export function abandonRun(st: string, id: string, why: string): string | null {
   try {
     const run = JSON.parse(readText(file));
     if (run.stage === "abandoned") return "That run is already abandoned.";
+    if (run.stage === "done") return "That run is already done.";
     run.stage = "abandoned";
     run.awaiting = null;
     run.updated = new Date().toISOString();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kaizen-agent",
-  "version": "1.40.0",
+  "version": "1.40.2",
   "description": "Staged workflow for AI agents — code, writing, research, operations: plan, human approval, build, independent review, bounded fix loop.",
   "type": "module",
   "bin": {
@@ -55,6 +55,7 @@
     "@codemirror/view": "6.43.13",
     "@lezer/highlight": "1.2.5",
     "@xterm/addon-fit": "0.11.0",
+    "@xterm/addon-webgl": "0.19.0",
     "@xterm/xterm": "6.0.0",
     "prettier": "3.9.10"
   }

package/skills/kaizen/SKILL.md

@@ -108,8 +108,9 @@ rather than dispatching them, then stop after the review: report the findings an
 the user choose, rather than entering the fix loop. The loop
 exists because a fix to something intricate can be worse than the defect — it is not
 worth two more stages to confirm a one-line message now reads correctly. A finding at
-`high` or above pulls the run back onto the full path whatever its size, since that
-severity is the loop's whole reason for existing.
+`high` or above still enters the fix loop whatever the run's size, since that
+severity is the loop's whole reason for existing — on the same runner, which is fixed
+at intake.
 
 **Short work still gets a run.** A one-line change is not skipped past the stages: the card on the board moves through plan, build and review like any other. Run it with `runner: lite` and the short path above — a brief plan, the build, a short review — so the cost stays small and the run stays visible.
 

package/skills/kaizen/spec-build.md

@@ -115,7 +115,9 @@ screen; a document gets one of it rendered; a CLI gets one of the terminal runni
 the real output also pasted in a code block. Only a result with no screen at all — a
 library, an API — may give command output alone. A report with no screenshot is incomplete. Save each under
 `runs/<id>/attachments/` (letters, digits, `.`, `-`, `_` only) and embed it as
-`![what it shows](attachments/<name>.png)`; the board shows them in the Preview tab.
+`![what it shows](attachments/<name>.png)`; the board shows them in the Preview tab. A link
+the user should open from the board (a video, a captions file) goes there too, as
+`[label](attachments/<name>)`: the board serves nothing else, and shows any other relative link as its path.
 A headless browser does it: `playwright screenshot <url> runs/<id>/attachments/<name>.png`,
 or `chromium --headless --screenshot=<abs path>.png --window-size=1280,900 <url>` where
 playwright has no browser installed. A file (document, diff, output) opens as `file://<path>`.

package/skills/kaizen/spec.md

@@ -143,7 +143,7 @@ the config said the day someone came back to it.
 If the request remains ambiguous enough that a plan would be guesswork, ask now. One
 round of questions here is cheaper than a rejected plan.
 
-**Short work is not skipped.** Even a one-line change gets `01-plan.md` and `04-review.md`, so the run moves through every stage. Keep them brief and run under `runner: lite`.
+**Short work is not skipped.** Even a one-line change gets `01-plan.md` and `04-review.md`, so the run moves through every stage. Keep them brief and run under `runner: lite`; a `high`+ finding still enters the fix loop, on that same runner.
 
 **Follow-ups are runs too.** Both runners record the same artifacts: `01-plan.md`, `03-impl.md`, `04-review.md`, and a final `state.json` — `lite` and `full` differ only in who executes the stages. A message the user sends later in the same session is judged by what it asks: a question, explanation or chat is answered, no run; a request to change or produce something is building, so it goes through the flow. While the previous run has not reached `done` (including awaiting its final approval), the change is a revise of that run, recorded in `02-approval.md`. After `done` or `abandoned`, open a new run whose `00-request.md` is the user's words verbatim, then a line `Follow-up of: <previous run id>`. Never reopen a finished run. When unsure whether a message asks for a change, ask once.
 

package/web/board/cards.ts

@@ -9,7 +9,7 @@ import { brief } from "./markdown.ts";
 import { open, toggleArchive } from "./panel.ts";
 import { ideaCombo, list, newDir, plogo, short, showList, showable } from "./projects.ts";
 import { askNotify, toast } from "./toast.ts";
-import { resumable, resume } from "./approvals.ts";
+import { answer, decided, resumable, resume } from "./approvals.ts";
 
 let filter = null;
 let query = ""; // the header filter box, matched against a card's title and request
@@ -42,8 +42,11 @@ const nested = (c) =>
 const kidsOpen = new Set();
 
 // What opening a waiting run does, in words: the card's button and its menu say the same.
+// One already answered from this page says so until the agent moves the run on.
 const actionOf = (c) =>
-  c.status !== "waiting"
+  decided.has(answer(c))
+    ? "Sent to agent"
+    : c.status !== "waiting"
     ? "Open"
     : { "approvals.plan": "Review plan", findings: "See findings", "approvals.review": "Review work" }[c.awaiting] || "Open";
 
@@ -114,7 +117,7 @@ function card(c) {
   const act =
     c.kind === "run" && !archive && !pick
       ? c.status === "waiting"
-        ? `<button class="btn sm pri act" data-a="open">${actionOf(c)}</button>`
+        ? `<button class="btn sm${decided.has(answer(c)) ? "" : " pri"} act" data-a="open">${actionOf(c)}</button>`
         : c.status === "stalled" && resumable(c)
           ? `<button class="btn sm act" data-a="resume">Resume</button>`
           : ""

package/web/board/markdown.ts

@@ -9,12 +9,15 @@ function section(impl, name) {
 }
 export const preview = (impl) => section(impl, "Preview");
 // Links to a card's attachments/ (notes, or screenshots the builder put in Preview) via /file.
+// Any other relative link (../video.mp4, out/x.srt) would resolve against the board's URL
+// and open "not found": the board serves attachments only, so it shows as its path instead.
 export const attach = (t, c) =>
-  t.replace(
-    /\]\((attachments\/[^)]+)\)/g,
-    (_, p) =>
-      `](/file?dir=${encodeURIComponent(c.state)}&p=${encodeURIComponent((c.kind === "run" ? `runs/${c.id}/` : "") + p)})`,
-  );
+  t.replace(/(!?)\[([^\]]*)\]\(([^)\s]+)\)/g, (all, img, label, p) => {
+    if (/^attachments\/[^)]+$/.test(p))
+      return `${img}[${label}](/file?dir=${encodeURIComponent(c.state)}&p=${encodeURIComponent((c.kind === "run" ? `runs/${c.id}/` : "") + p)})`;
+    if (/^([a-z][\w+.-]*:|\/|#)/i.test(p)) return all;
+    return `${label} (\`${p}\`)`;
+  });
 export const commitMsg = (impl) => (section(impl, "Commit") || "").replace(/^```\w*\n|\n```$/g, "").trim();
 // A plan's Open questions as a pick list: `Q: question` lines with `- option` lines under
 // each, "(recommended)" marking the default and "multi-select" in the question allowing