kaizen-agent 1.20.0 → 1.21.1

package/cli/board.test.ts

@@ -109,12 +109,16 @@ test("card menu items: appropriate actions and distinct icons across phases", ()
 
   const ico = (n: string) => `<svg><use href="#i-${n}"/></svg>`;
   const menuSnippet = (html.slice(html.indexOf("if(k===menuKey){"), html.indexOf("el.innerHTML+=`<div class=\"menu-pop\"", html.indexOf("if(k===menuKey){"))) + "}").replace("const items=[];", "items=[];");
-  const getItems = new Function("c", "archive", `const ico = ${ico.toString()};\nlet k = 1, menuKey = 1, items = [];\n${menuSnippet}\nreturn items;`);
+  const getItems = new Function("c", "archive", "S", `const ico = ${ico.toString()};\nlet k = 1, menuKey = 1, items = [];\n${menuSnippet}\nreturn items;`);
 
   // Phase 0: Ideas (idea card)
   const idea = { kind: "idea", status: "idea", column: 0 };
-  const ideaItems = getItems(idea, false);
+  const ideaItems = getItems(idea, false, { projects: [] });
   expect(ideaItems.map((i: any) => i ? i[1] : null)).toEqual(["Run", "Edit", "Archive", null, "Reject", "Delete"]);
+  // Move only when there is another project to move to.
+  const moveItems = getItems({ ...idea, state: "/a" }, false, { projects: [{ dir: "/a" }, { dir: "/b" }] });
+  expect(moveItems.map((i: any) => i ? i[1] : null)).toEqual(["Run", "Edit", "Move to project", "Archive", null, "Reject", "Delete"]);
+  expect(moveItems[2][2]).toBe(ico("ext"));
   expect(ideaItems[0][2]).toBe(ico("play"));
   expect(ideaItems[1][2]).toBe(ico("pen"));
   expect(ideaItems[2][2]).toBe(ico("archive"));

package/cli/state.test.ts

@@ -7,7 +7,7 @@ import { tmpdir } from "node:os";
 import { issueUrl, systemInfo, scrubHome, REPO_URL, home as realHome, trustFolder } from "./state.ts";
 import * as stateModule from "./state.ts";
 import {
-  readRuns, readInbox, writeInbox, replaceIdea, abandonRun, allBacklog, backlog,
+  readRuns, readInbox, writeInbox, replaceIdea, moveIdea, abandonRun, allBacklog, backlog,
   statusOf, columnOf, parseItem, startedRuns, boardCards, readArchive, setArchived,
   requestOf, agentFlags, plainModel, readNotes, writeNotes, manualCommand, parseNotes, formatNote, appendNote, saveAttachment,
   reviewFindings, pickFindings, fixPrompt, approvalPrompt, sessionLaunch, closeSessions, sweepSessions, readSessions, ownerOf, CLOSE_GRACE_MS, gitStatus, runGit, cardsGit, commitPush, readCommit, findTerminal, nativePath, pruneRuns, unrechecked, notice, notifier, notifyArgs, LOGO, type Card,
@@ -827,3 +827,20 @@ test("trustFolder: marks the repo root trusted in a temp config, keeps the rest,
   expect(readFileSync(cfg, "utf8")).toBe("{bad");
   rmSync(t, { recursive: true, force: true });
 });
+
+test("moveIdea carries notes and linked attachments, leaves the old inbox without it", () => {
+  const root = mkdtempSync(join(tmpdir(), "kaizen-mv-"));
+  const a = join(root, "a", ".kaizen"), b = join(root, "b", ".kaizen");
+  mkdirSync(join(a, "attachments"), { recursive: true });
+  writeFileSync(join(a, "attachments", "x-shot.png"), "png");
+  writeInbox(a, [{ status: "open", text: "keep" }, { status: "open", text: "go", notes: "see [shot](attachments/x-shot.png)" }]);
+  expect(moveIdea(a, a, "go")).toBe("already in that project");
+  expect(moveIdea(a, b, "nope")).toBe("no such idea");
+  expect(moveIdea(a, b, "go")).toBeNull();
+  expect(readInbox(a).map((l) => l.text)).toEqual(["keep"]);
+  expect(readInbox(b)).toEqual([{ status: "open", text: "go", notes: "see [shot](attachments/x-shot.png)" }]);
+  expect(readFileSync(join(b, "attachments", "x-shot.png"), "utf8")).toBe("png");
+  writeInbox(a, [{ status: "open", text: "go" }]);
+  expect(moveIdea(a, b, "go")).toBe("that project already has this idea");
+  rmSync(root, { recursive: true, force: true });
+});

package/cli/state.ts

@@ -1,7 +1,7 @@
 // Everything kaizen knows about a project's state, read from and written to the
 // `.kaizen/` directory. No terminal, no HTTP: the TUI (`dashboard.ts`) and the web
 // board (`web.ts`) both import this and draw it their own way.
-import { existsSync, readFileSync, readdirSync, appendFileSync, writeFileSync, mkdirSync, statSync, realpathSync, rmSync, renameSync } from "node:fs";
+import { existsSync, readFileSync, readdirSync, appendFileSync, writeFileSync, mkdirSync, statSync, realpathSync, rmSync, renameSync, copyFileSync } from "node:fs";
 import { join, dirname, win32 } from "node:path";
 import { homedir, arch, cpus, release, totalmem, version as osVersion } from "node:os";
 
@@ -817,6 +817,29 @@ export function replaceIdea(state: string, text: string, next: InboxLine | null)
   writeInbox(state, lines);
 }
 
+// An open idea, notes and all, from one project's inbox to another's. Files its notes
+// link (attachments/...) are copied along, so the links resolve from the new state dir.
+// Written to the new project first and taken back out if the old one cannot be
+// written, like moveSource: a crash between the two leaves it listed twice, never lost.
+export function moveIdea(from: string, to: string, text: string): string | null {
+  if (from === to) return "already in that project";
+  const src = readInbox(from);
+  const at = src.findIndex((l) => l.status === "open" && l.text === text);
+  if (at < 0) return "no such idea";
+  const dest = readInbox(to);
+  if (dest.some((l) => l.status === "open" && l.text === text)) return "that project already has this idea";
+  const idea = src[at]!;
+  for (const m of (idea.notes ?? "").matchAll(/\]\((attachments\/[^)\/]+)\)/g)) {
+    if (!existsSync(join(from, m[1]!)) || !statSync(join(from, m[1]!)).isFile()) continue;
+    mkdirSync(join(to, "attachments"), { recursive: true });
+    copyFileSync(join(from, m[1]!), join(to, m[1]!));
+  }
+  writeInbox(to, [...dest, idea]);
+  try { writeInbox(from, src.filter((_, i) => i !== at)); }
+  catch (e) { writeInbox(to, dest); throw e; }
+  return null;
+}
+
 // Archived runs stay exactly where they are on disk -- run directories are never
 // moved or renamed, that is the resume contract -- and are listed here instead.
 // One id per line; the board hides them unless asked for the archive.

package/cli/web.ts

@@ -4,7 +4,7 @@ import { existsSync, readdirSync, statSync, watch, appendFileSync, mkdirSync, ty
 import { join, dirname, resolve, sep } from "node:path";
 import {
   home, type Item, type Card, boardCards, knownProjects, remember, locate, label, tilde, searchRoots, findProjects,
-  readInbox, writeInbox, replaceIdea, abandonRun, launchRun, parseItem, agentChoices, projectOf, KNOWN_AGENTS, plainModel, short, setArchived, writeNotes,
+  readInbox, writeInbox, replaceIdea, moveIdea, abandonRun, launchRun, parseItem, agentChoices, projectOf, KNOWN_AGENTS, plainModel, short, setArchived, writeNotes,
   appendNote, saveAttachment, pidOnPort, readText, cmdExe, sysExe, powershellExe, runGit, cardsGit, readCommit, commitPush, notice, notifier, LOGO,
   reviewFindings, pickFindings, fixPrompt, approvalPrompt, canApproveAwaiting, nativePath, closeSessions, sweepSessions, issueUrl,
 } from "./state.ts";
@@ -18,10 +18,17 @@ const NOTES_JS = join(dirname(import.meta.dir), "web", "notebook.js");
 const DEFAULT_PORT = 7420;
 // Read once at start: a board keeps the code it started with, so this is what it
 // runs, not what is installed now. `/` sends it; `current()` compares the two.
-const VERSION = (() => {
+const installed = () => {
   try { return JSON.parse(readText(join(dirname(import.meta.dir), "package.json"))).version as string; }
   catch { return "unknown"; }
-})();
+};
+const VERSION = installed();
+// An upgrade replaces the page files under a running board but not its code, and a
+// page newer than its server sends ops the server does not know ("unknown op").
+// So the pages are kept as they were at start, and read live (dev edits) only while
+// the installed version is still the one this board runs.
+const AT_START = new Map([PAGE, NOTES_PAGE, NOTES_JS].map((f) => [f, existsSync(f) ? readText(f) : null]));
+const served = (f: string) => installed() === VERSION && existsSync(f) ? readText(f) : AT_START.get(f) ?? null;
 
 // A board answers at url and runs the installed version. A board from before the
 // version header sends none, so it counts as stale too.
@@ -211,17 +218,19 @@ export async function web(repoDir: string, opts: Opts = {}) {
       const path = url.pathname;
 
       if (req.method === "GET" && path === "/") {
-        if (!existsSync(PAGE)) return new Response("web/board.html missing", { status: 500 });
+        const page = served(PAGE);
+        if (page === null) return new Response("web/board.html missing", { status: 500 });
         // The page gets the notify rule from state.ts rather than keeping its own copy.
-        return new Response(readText(PAGE).replace("/*notifier*/", () => notifier.toString()), {
+        return new Response(page.replace("/*notifier*/", () => notifier.toString()), {
           headers: { "content-type": "text/html; charset=utf-8", "cache-control": "no-store", "x-kaizen-version": VERSION },
         });
       }
 
       if (req.method === "GET" && (path === "/notebook" || path === "/notebook.js")) {
         const [f, type] = path === "/notebook" ? [NOTES_PAGE, "text/html; charset=utf-8"] : [NOTES_JS, "application/javascript; charset=utf-8"];
-        if (!existsSync(f)) return new Response(`${path} missing; run bun run build:web`, { status: 500 });
-        return new Response(Bun.file(f), { headers: { "content-type": type, "cache-control": "no-store" } });
+        const body = served(f);
+        if (body === null) return new Response(`${path} missing; run bun run build:web`, { status: 500 });
+        return new Response(body, { headers: { "content-type": type, "cache-control": "no-store" } });
       }
 
       // Every note of one project, text included: search, tags and backlinks are the page's.
@@ -444,6 +453,10 @@ export async function web(repoDir: string, opts: Opts = {}) {
             replaceIdea(dir, text, { status: "rejected", text: `${text} | ${why}` });
           } else if (body.op === "delete") {
             replaceIdea(dir, text, null);
+          } else if (body.op === "move") {
+            if (typeof body.to !== "string" || !states(true).includes(body.to)) return bad("unknown state dir", 404);
+            const why = moveIdea(dir, body.to, text);
+            if (why) return bad(why, why === "no such idea" ? 404 : 409);
           } else return bad("unknown op");
           changed();
           return json({ ok: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kaizen-agent",
-  "version": "1.20.0",
+  "version": "1.21.1",
   "description": "Staged workflow for AI agents — code, writing, research, operations: plan, human approval, build, independent review, bounded fix loop.",
   "type": "module",
   "bin": {

package/web/board.html

@@ -518,7 +518,7 @@ function card(c){
    items.push(["open","Open",ico("open")]);
    if(c.kind==="idea")items.push(null,["delete","Delete",ico("trash"),"danger"]);
   }else if(c.kind==="idea"&&c.status==="idea"){
-   items.push(["run","Run",ico("play")],["edit","Edit",ico("pen")],["archive","Archive",ico("archive")],null,["reject","Reject",ico("x")],["delete","Delete",ico("trash"),"danger"]);
+   items.push(["run","Run",ico("play")],["edit","Edit",ico("pen")],...(S.projects.some(p=>p.dir!==c.state)?[["move","Move to project",ico("ext")]]:[]),["archive","Archive",ico("archive")],null,["reject","Reject",ico("x")],["delete","Delete",ico("trash"),"danger"]);
   }else if(c.kind==="idea"){
    items.push(["open","Open",ico("open")],["run","Run again",ico("play")],["archive","Archive",ico("archive")],null,["delete","Remove",ico("trash"),"danger"]);
   }else{
@@ -542,6 +542,7 @@ function inlineForm(c){
  if(mode==="edit"){li.innerHTML=`<input value="${esc(c.text)}"><textarea placeholder="Notes: links, paths, background">${esc(c.notes||"")}</textarea><div class="q dim" id="clong" hidden>The first line is over 72 characters; logs will cut it short.</div><div class="row"><button class="btn ghost" data-a="c">Cancel</button><button class="btn pri" data-a="s">Save</button></div>`;li.querySelector('[data-a=s]').onclick=async()=>{const next=li.querySelector("input").value.trim(),notes=li.querySelector("textarea").value.trim();mode=null;if(next&&(next!==c.text||notes!==(c.notes||"")))await post("/inbox",{op:"edit",dir,text:c.text,next,notes});load()}}
  if(mode==="reject"){const idea=c.kind==="idea";li.innerHTML=`<div class="q">${idea?"Reject this idea?":"Abandon this run?"} The reason is recorded.</div><input placeholder="Reason"><div class="row"><button class="btn ghost" data-a="c">Keep it</button><button class="btn warn" data-a="s">${idea?"Reject":"Abandon"}</button></div>`;li.querySelector('[data-a=s]').onclick=async()=>{const why=li.querySelector("input").value.trim();if(!why){toast("A reason is required");return}mode=null;const r=idea?await post("/inbox",{op:"reject",dir,text:c.text,reason:why}):await post("/abort",{dir,id:c.id,why});if(r.ok)toast(idea?"Rejected":"Abandoned, reason recorded");load()}}
  if(mode==="delete"){li.innerHTML=`<div class="q">Delete "${esc(c.text)}"?</div><div class="q dim">Not recorded anywhere.</div><div class="row"><button class="btn ghost" data-a="c">Keep it</button><button class="btn danger" data-a="s">Delete</button></div>`;li.querySelector('[data-a=s]').onclick=async()=>{mode=null;const r=await post("/inbox",{op:"delete",dir,text:c.text});if(r.ok)toast("Deleted");load()}}
+ if(mode==="move"){const ps=S.projects.filter(p=>p.dir!==dir);li.innerHTML=`<div class="q">Move "${esc(c.text)}" to</div><select class="sel" aria-label="Project">${ps.map(p=>`<option value="${esc(p.dir)}">${esc(p.label)}</option>`).join("")}</select><div class="row"><button class="btn ghost" data-a="c">Cancel</button><button class="btn pri" data-a="s">Move</button></div>`;li.querySelector('[data-a=s]').onclick=async()=>{const sel=li.querySelector("select"),to=sel.value,name=sel.selectedOptions[0]?.textContent;mode=null;const r=await post("/inbox",{op:"move",dir,to,text:c.text});if(r.ok)toast(`Moved to ${name}`);load()}}
  if(mode==="run"){li.innerHTML=`<div class="q">Start a run for "${esc(c.text)}"</div><div class="q dim">Opens your agent in a new terminal, or a herdr tab or tmux window when one is running, with the request typed in.</div><div class="q dim">Full: planned, built, then checked by a separate reviewer. Lite: one session does it all, faster and cheaper.</div><div id="run-pick" style="display:grid;gap:6px"><div id="run-agent" class="agents" role="radiogroup" aria-label="Agent"></div></div><div class="row"><label class="yolo" id="run-yolo" title="Yolo: the agent runs every command and edit without asking" hidden><button type="button" class="chk" role="switch" aria-checked="false"></button>Yolo</label><button class="btn ghost" data-a="c">Cancel</button><button class="btn" data-a="l">Lite</button><button class="btn pri" data-a="s">Full run</button></div>`;li.querySelector('[data-a=s]').onclick=()=>startRun(c,"full");li.querySelector('[data-a=l]').onclick=()=>startRun(c,"lite");runForm(li,c)}
  li.querySelector('[data-a=c]').onclick=()=>{mode=null;render()};
  // Edit and reject take dictation; the run prompt has no field.