kaizen-agent 1.22.0 → 1.22.1

package/README.md

@@ -49,6 +49,20 @@ Then, in any project:
 
 That's it. No init and no config.
 
+kaizen needs an AI coding agent to run it, such as [Claude Code](https://docs.claude.com/en/docs/claude-code); install one first. Git is needed for the board's commit and push.
+
+## Use it
+
+There are three ways in, all on the same runs:
+
+| | How | What you do there |
+|---|---|---|
+| **Your agent** | `/kaizen <request>` in Claude Code, Codex, Cursor, … | Start a run and answer it in the chat. All commands are under [Commands](#commands). |
+| **The board** | `kaizen web` opens it in the browser; `kaizen` in a terminal. `kaizen web --shortcut` once adds a launcher: Spotlight on macOS, the app menu on Linux, a Desktop shortcut on Windows | Every project's ideas and runs. Start a run, approve or revise a plan, answer its questions, pick findings to fix, then commit and push. |
+| **Kaizen Bot** | The [desktop app](#download-the-app), or `bun start` in `bot/` | Chat with the board: ask what's waiting on you, hand teammates their asks. The app starts the board itself; from `bot/`, run `kaizen web` first. See [Kaizen Bot](#kaizen-bot). |
+
+Settings for a project or for all of them: `kaizen settings`. New version: `kaizen upgrade`.
+
 ## See it
 
 ```

package/cli/install.test.ts

@@ -90,7 +90,7 @@ const fromPackage = (make: (k: string) => void, custom = false) => {
   const root = mkdtempSync(join(tmpdir(), "kz-replace-"));
   const home = join(root, "home"), pkg = join(root, "pkg");
   const k = custom ? join(root, "kaizen") : join(home, "kaizen");
-  mkdirSync(home);
+  mkdirSync(join(home, ".claude"), { recursive: true });   // an agent: the installer stops on a box with none
   const src = dirname(dirname(import.meta.path));
   for (const d of ["cli", "skills", "agents", "commands", "web", "package.json"]) {
     cpSync(join(src, d), join(pkg, d), { recursive: true, filter: (p) => !p.includes("node_modules") });
@@ -129,7 +129,7 @@ it.skipIf(!Bun.which("git"))("an npm package never replaces a development checko
 it("upgrade with npm unreachable says not upgraded, never up to date", () => {
   const root = mkdtempSync(join(tmpdir(), "kz-offline-"));
   const home = join(root, "home");
-  mkdirSync(home);
+  mkdirSync(join(home, ".claude"), { recursive: true });   // an agent: the installer stops on a box with none
   kaizenFiles(join(home, "kaizen"));
   const env = { PATH: process.env.PATH ?? "", HOME: home, TMPDIR: root, HTTPS_PROXY: "http://127.0.0.1:9", https_proxy: "http://127.0.0.1:9" };
   const p = Bun.spawnSync([process.execPath, script, "upgrade"], { cwd: home, env });

package/cli/notes.test.ts

@@ -71,11 +71,11 @@ test("list: recursive, newest first, dot folders skipped", () => {
 test("list: only names the notebook can save back; a symlinked folder once, never a loop", () => {
   const n = join(state, "notes");
   saveNote(state, "ok", "x", 0);
-  writeFileSync(join(n, "What?.md"), "x");
-  writeFileSync(join(n, "a: b.md"), "x");
+  // Windows refuses these names outright, so there is nothing to list.
+  if (process.platform !== "win32") { writeFileSync(join(n, "What?.md"), "x"); writeFileSync(join(n, "a: b.md"), "x"); }
   mkdirSync(join(n, "1/2/3/4/5/6"), { recursive: true }); writeFileSync(join(n, "1/2/3/4/5/6/deep.md"), "x");
   mkdirSync(join(n, "trail ")); writeFileSync(join(n, "trail ", "x.md"), "x");
-  symlinkSync("..", join(n, "up"));
+  symlinkSync(process.platform === "win32" ? dirname(n) : "..", join(n, "up"), "junction");   // a junction needs no admin
   expect(listNotes(state).map((x) => x.name)).toEqual(["ok"]);
 });
 

package/cli/sources.test.ts

@@ -498,11 +498,12 @@ test("gather everything: each source's ideas land in its own project and no othe
 // The real server, in a scratch home on its own port. Nothing is fetched: only add, list and move.
 test("the board refuses to add a source without a project, and moves one between two", async () => {
   const homeDir = join(root, "home" + n), a = join(homeDir, "a", ".kaizen"), b = join(homeDir, "b", ".kaizen");
-  for (const d of [a, b, join(homeDir, ".kaizen")]) mkdirSync(d, { recursive: true });
+  // ~/.claude: kaizen will not start on a machine with no agent, and a CI box has none.
+  for (const d of [a, b, join(homeDir, ".kaizen"), join(homeDir, ".claude")]) mkdirSync(d, { recursive: true });
   writeFileSync(join(homeDir, ".kaizen", "projects"), [join(homeDir, "a"), join(homeDir, "b")].join("\n") + "\n");
   const port = 20000 + Math.floor(Math.random() * 20000);
   const repo = join(import.meta.dir, "..");
-  const proc = Bun.spawn([process.execPath, join(repo, "cli", "install.ts"), "web", "--port", String(port), "--no-open"], {
+  const proc = Bun.spawn([process.execPath, join(repo, "cli", "install.ts"), "web", "--port", String(port), "--no-open", "--foreground"], {
     cwd: join(homeDir, "a"), env: { ...process.env, HOME: homeDir, USERPROFILE: homeDir, KAIZEN_HOME: repo }, stdout: "ignore", stderr: "ignore",
   });
   try {
@@ -530,4 +531,4 @@ test("the board refuses to add a source without a project, and moves one between
     expect(await post({ op: "add", dir: b, url: SHEET })).toMatchObject({ status: 200 });
     expect(await post({ op: "move", dir: b, to: a, url: SHEET })).toMatchObject({ status: 409, error: "that project already has this link" });
   } finally { proc.kill(); await proc.exited; }
-});
+}, 30_000);   // a cold board start is slow on Windows

package/cli/state.test.ts

@@ -399,6 +399,8 @@ test("commitPush: commits the run's files only, pushes, and records it; the mess
   expect(readCommit(st, "r1")).toEqual({ sha: r.sha, pushed: true });
   expect(cardsGit(st, ["r1", "r0", "nope"])).toEqual({ r1: "pushed", r0: "pushed" });   // r0: no record, but its file is in; c.txt still changed is not its business
 
+  // Windows has no file named b*.txt, so the rest is for everywhere else.
+  if (process.platform === "win32") return;
   // A push that fails leaves a record that says so.
   sh(proj, "remote", "set-url", "origin", join(root, "gone.git"));
   // A fix round's report names files too, and a name with a star in it is not a pattern.
@@ -415,7 +417,7 @@ test("commitPush: commits the run's files only, pushes, and records it; the mess
   expect(runGit(st, "r1")).toMatchObject({ files: [], ahead: 1, others: 2 });
   expect(cardsGit(st, ["r1"])).toEqual({ r1: "pending" });   // committed, not pushed
   expect(cardsGit(join(root, ".kaizen"), ["r1"])).toEqual({});   // not a repo
-});
+}, 60_000);   // dozens of git calls, ~250ms each on Windows
 
 test.skipIf(process.platform === "win32")("findTerminal: a running herdr wins, then tmux, else neither; the prompt stays one literal argument", () => {
   const bin = mkdtempSync(join(tmpdir(), "kz-bin-"));
@@ -427,11 +429,11 @@ test.skipIf(process.platform === "win32")("findTerminal: a running herdr wins, t
     for (const k of keys.slice(1)) delete process.env[k];
     const prompt = `/kaizen it's "q" $HOME \`id\`\n\nnote`;
     const full = ["/usr/bin/claude", "--model", "opus", prompt];
-    expect(findTerminal("/p", full)?.cmd[0]).not.toMatch(/^(sh|tmux)$/);   // neither installed
+    expect(findTerminal("/p", full)?.cmd[0] ?? "none").not.toMatch(/^(sh|tmux)$/);   // neither installed (none: no terminal, as on CI)
     stub("herdr", 1); stub("tmux", 1);
-    expect(findTerminal("/p", full)?.cmd[0]).not.toMatch(/^(sh|tmux)$/);   // installed, no server
+    expect(findTerminal("/p", full)?.cmd[0] ?? "none").not.toMatch(/^(sh|tmux)$/);   // installed, no server
     process.env.TMUX = "/tmp/x,1,0"; process.env.HERDR_ENV = "1";
-    expect(findTerminal("/p", full)?.cmd[0]).not.toMatch(/^(sh|tmux)$/);   // stale env from a server since stopped
+    expect(findTerminal("/p", full)?.cmd[0] ?? "none").not.toMatch(/^(sh|tmux)$/);   // stale env from a server since stopped
     delete process.env.TMUX; delete process.env.HERDR_ENV; stub("tmux", 0);
     const t = findTerminal("/p", full)!;                                   // outside it, server up
     expect(t.cmd.slice(0, 4)).toEqual(["tmux", "new-window", "-c", "/p"]);
@@ -456,8 +458,9 @@ test("nativePath on Windows reads the spellings Git Bash writes into the project
     for (const p of ["/c/Users/me/proj", "C:/Users/me/proj", "c:\\Users\\me\\proj\\", "C:\\Users\\me\\proj"])
       expect(nativePath(p)).toBe("C:\\Users\\me\\proj");
     expect(nativePath("/d")).toBe("D:\\");
+    Object.defineProperty(process, "platform", { value: "linux" });
+    expect(nativePath("/c/Users/me")).toBe("/c/Users/me");   // elsewhere a path is left alone
   } finally { Object.defineProperty(process, "platform", platform); }
-  expect(nativePath("/c/Users/me")).toBe("/c/Users/me");   // elsewhere a path is left alone
 });
 
 test("findTerminal on Windows names cmd and PowerShell by full path, so a PATH without System32 still launches", () => {
@@ -474,7 +477,9 @@ test("findTerminal on Windows names cmd and PowerShell by full path, so a PATH w
     expect(t.cmd[6]).toBe("D:\\Win\\System32\\WindowsPowerShell\\v1.0\\powershell.exe");
     process.env.ComSpec = "E:\\cmd.exe";
     expect(findTerminal("C:\\p", ["claude"])!.cmd[0]).toBe("E:\\cmd.exe");
-    // A running herdr: a tab, and the run typed into it as one encoded line.
+    // A running herdr: a tab, and the run typed into it as one encoded line. The fake
+    // is a shell script, which real Windows cannot run with ComSpec pointed elsewhere.
+    if (platform.value === "win32") return;
     writeFileSync(join(process.env.PATH, "herdr"), "#!/bin/sh\nexit 0\n", { mode: 0o755 });
     const h = findTerminal("C:\\p", ["claude", "/kaizen it's\nnote"])!;
     expect([h.cmd[0], h.cmd[1], h.cmd[2], h.detached]).toEqual(["D:\\Win\\System32\\WindowsPowerShell\\v1.0\\powershell.exe", "-NoProfile", "-EncodedCommand", false]);
@@ -815,13 +820,14 @@ test("trustFolder: marks the repo root trusted in a temp config, keeps the rest,
   mkdirSync(sub, { recursive: true }); mkdirSync(plain);
   Bun.spawnSync(["git", "init", "-q", repo]);
   const cfg = join(t, ".claude.json");
-  writeFileSync(cfg, JSON.stringify({ keep: 1, projects: { [plain]: { x: 2 } } }));
+  const key = (p: string) => process.platform === "win32" ? p.replace(/\\/g, "/") : p;   // Claude Code's spelling
+  writeFileSync(cfg, JSON.stringify({ keep: 1, projects: { [key(plain)]: { x: 2 } } }));
   trustFolder(sub, cfg);
   trustFolder(plain, cfg);
   const d = JSON.parse(readFileSync(cfg, "utf8"));
   expect(d.keep).toBe(1);
-  expect(d.projects[realpathSync(repo)]).toEqual({ hasTrustDialogAccepted: true });
-  expect(d.projects[plain]).toEqual({ x: 2, hasTrustDialogAccepted: true });
+  expect(d.projects[key(realpathSync.native(repo))]).toEqual({ hasTrustDialogAccepted: true });
+  expect(d.projects[key(plain)]).toEqual({ x: 2, hasTrustDialogAccepted: true });
   writeFileSync(cfg, "{bad");
   trustFolder(plain, cfg);
   expect(readFileSync(cfg, "utf8")).toBe("{bad");

package/cli/state.ts

@@ -2,7 +2,7 @@
 // `.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, copyFileSync } from "node:fs";
-import { join, dirname, win32 } from "node:path";
+import { join, dirname, win32, posix } from "node:path";
 import { homedir, arch, cpus, release, totalmem, version as osVersion } from "node:os";
 
 // The real path: process.cwd() is always physical, so a raw $HOME that crosses a
@@ -730,11 +730,12 @@ export function appendNote(state: string, target: { id: string } | { text: strin
 // ponytail: no dedupe, no size accounting beyond the request cap in web.ts.
 export function saveAttachment(state: string, id: string | null, name: string, bytes: Uint8Array) {
   const safe = name.replace(/[^\w.-]+/g, "_").replace(/^\.+/, "") || "file";
-  const rel = id ? join("runs", id, "attachments") : "attachments";
+  // posix: the note links it as attachments/..., and a backslash on Windows breaks that.
+  const rel = id ? posix.join("runs", id, "attachments") : "attachments";
   mkdirSync(join(state, rel), { recursive: true });
   const file = `${Date.now().toString(36)}-${safe}`;
   writeFileSync(join(state, rel, file), bytes);
-  return join(rel, file);
+  return posix.join(rel, file);
 }
 
 // What a card is worth interrupting someone for, in the words both UIs send: a run
@@ -1133,7 +1134,9 @@ export function sessionLaunch(projectDir: string, cmd: string, base: string[], p
 export function trustFolder(dir: string, cfg = join(process.env.CLAUDE_CONFIG_DIR ?? homedir(), ".claude.json")) {
   try {
     const top = Bun.spawnSync(["git", "-C", dir, "rev-parse", "--show-toplevel"], { stderr: "ignore" });
-    const root = top.success ? top.stdout.toString().trim() || dir : dir;
+    // Claude Code keys its projects with forward slashes on Windows, as git prints them.
+    const found = top.success ? top.stdout.toString().trim() || dir : dir;
+    const root = process.platform === "win32" ? found.replace(/\\/g, "/") : found;
     const d = JSON.parse(readFileSync(cfg, "utf8"));
     if (d.projects?.[root]?.hasTrustDialogAccepted) return;
     d.projects ??= {};

package/package.json

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