kaizen-agent 1.8.0 → 1.8.1

package/cli/release.sh

@@ -38,8 +38,8 @@ git log --reverse --format='### %s%n%n%b' ${prev:+"$prev.."}"$tag" \
   | grep -vE '^### Release [0-9.]+:' \
   | cat -s > "$notes"
 [ -n "$prev" ] && printf '\n**Full Changelog**: https://github.com/hfadhlullah/kaizen/compare/%s...%s\n' "$prev" "$tag" >> "$notes"
-gh release create "$tag" --title "$title" --notes-file "$notes" >/dev/null 2>&1 \
+err="$(gh release create "$tag" --title "$title" --notes-file "$notes" 2>&1 >/dev/null)" \
   && echo "release: published $tag  https://github.com/hfadhlullah/kaizen/releases/tag/$tag" \
-  || echo "release: tag pushed, release not created"
+  || echo "release: tag pushed, release not created: $err"
 rm -f "$notes"
 exit 0

package/cli/state.test.ts

@@ -8,7 +8,7 @@ import {
   readRuns, readInbox, writeInbox, replaceIdea, abandonRun, allBacklog, backlog,
   statusOf, columnOf, parseItem, startedRuns, boardCards, readArchive, setArchived,
   requestOf, agentFlags, readNotes, writeNotes, manualCommand, parseNotes, formatNote, appendNote, saveAttachment,
-  reviewFindings, pickFindings, fixPrompt, gitStatus, runGit, commitPush, readCommit, findTerminal, pruneRuns, unrechecked, notice, notifier, notifyArgs, LOGO, type Card,
+  reviewFindings, pickFindings, fixPrompt, gitStatus, runGit, cardsGit, commitPush, readCommit, findTerminal, pruneRuns, unrechecked, notice, notifier, notifyArgs, LOGO, type Card,
 } from "./state.ts";
 
 let state: string;
@@ -274,6 +274,8 @@ test("commitPush: commits the run's files only, pushes, and records it; the mess
   writeFileSync(join(st, "runs", "r1", "03-impl.md"), "## Changed\n\n- `.gitignore`: x\n- `a.txt:1`: y\n- c.txt and `c.txt.bak` are not named\n");
   const files = () => (runGit(st, "r1") as any).files as string[];
   expect(files()).toEqual(["?? .gitignore", "?? a.txt"]);
+  // The card's mark: the run with changed files is pending; one with none and no record says nothing.
+  expect(cardsGit(st, ["r1", "nope"])).toEqual({ r1: "pending" });
   // An earlier run that named the same file is not offered it: the later report has it.
   mkdirSync(join(st, "runs", "r0"));
   writeFileSync(join(st, "runs", "r0", "03-impl.md"), "- `a.txt`: z\n");
@@ -303,6 +305,7 @@ test("commitPush: commits the run's files only, pushes, and records it; the mess
   expect(sh(proj, "show", "--name-only", "--format=", "HEAD")).toBe(".gitignore\na.txt");
   expect(Bun.spawnSync(["git", "-C", proj, "status", "--porcelain"]).stdout.toString()).toBe("A  c.txt\n");
   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
 
   // A push that fails leaves a record that says so.
   sh(proj, "remote", "set-url", "origin", join(root, "gone.git"));
@@ -318,6 +321,8 @@ test("commitPush: commits the run's files only, pushes, and records it; the mess
   expect(readCommit(st, "r1")!.pushed).toBe(false);
   expect(sh(proj, "show", "--name-only", "--format=", "HEAD")).toBe("b*.txt");
   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
 });
 
 test.skipIf(process.platform === "win32")("findTerminal: a running herdr wins, then tmux, else neither; the prompt stays one literal argument", () => {

package/cli/state.ts

@@ -918,6 +918,17 @@ function runReport(state: string, id: string) {
 }
 const names = (report: string, path: string) => report.includes("`" + path + "`") || report.includes("`" + path + ":");
 
+type Report = { id: string; text: string; at: number };
+function runReports(state: string): Report[] {
+  let ids: string[] = [];
+  try { ids = readdirSync(join(state, "runs")); } catch {}
+  return ids.map((id) => ({ id, ...runReport(state, id) }));
+}
+const runFiles = (changed: string[], mine: Report, rest: Report[]) => changed.filter((l) => {
+  const p = statusPath(l);
+  return names(mine.text, p) && !rest.some((r) => r.at > mine.at && names(r.text, p));
+});
+
 // The project's status narrowed to one run: a changed file is the run's when its
 // report names the path in backticks and no run that reported later names it too,
 // so a finished run is not offered the work of the ones after it. `others` counts
@@ -929,14 +940,9 @@ const names = (report: string, path: string) => report.includes("`" + path + "`"
 export function runGit(state: string, id: string): GitStatus {
   const dir = dirname(state), s = gitStatus(dir);
   if (!s.repo) return s;
-  const mine = runReport(state, id);
-  let ids: string[] = [];
-  try { ids = readdirSync(join(state, "runs")); } catch {}
-  const rest = ids.filter((o) => o !== id).map((o) => ({ id: o, ...runReport(state, o) }));
-  const files = s.files.filter((l) => {
-    const p = statusPath(l);
-    return names(mine.text, p) && !rest.some((r) => r.at > mine.at && names(r.text, p));
-  });
+  const all = runReports(state), mine = all.find((r) => r.id === id) ?? { id, text: "", at: 0 };
+  const rest = all.filter((o) => o.id !== id);
+  const files = runFiles(s.files, mine, rest);
   const shared = files.flatMap((l) => {
     const p = statusPath(l);
     const since = Number(git(dir, "--literal-pathspecs", "log", "-1", "--format=%ct", "--", p).out) * 1000;
@@ -955,6 +961,24 @@ export function readCommit(state: string, id: string): { sha: string; pushed: bo
   } catch { return null; }
 }
 
+// The card's git mark for each run asked about, by the rules of the panel's button
+// (`gitBtn` in web/board.html): one project status and one read of the reports for
+// the lot. A run with nothing to say either way gets no entry.
+export function cardsGit(state: string, ids: string[]): Record<string, "pending" | "pushed"> {
+  const s = gitStatus(dirname(state)), out: Record<string, "pending" | "pushed"> = {};
+  if (!s.repo) return out;
+  const all = runReports(state);
+  for (const id of ids) {
+    const mine = all.find((r) => r.id === id) ?? { id, text: "", at: 0 }, k = readCommit(state, id);
+    const files = runFiles(s.files, mine, all.filter((o) => o.id !== id)).length;
+    if (files) out[id] = "pending";
+    else if (k?.pushed) out[id] = "pushed";
+    else if (s.ahead > 0) out[id] = "pending";
+    else if (k || mine.text) out[id] = "pushed";
+  }
+  return out;
+}
+
 // A Conventional Commits subject: every commit the board makes says what kind it is.
 const COMMIT_TYPES = ["feat", "fix", "refactor", "perf", "docs", "test", "build", "ci", "chore", "style", "revert"];
 const COMMIT_TYPE = new RegExp(`^(${COMMIT_TYPES.join("|")})(\\([^)\\n]+\\))?!?: \\S`);

package/cli/web.ts

@@ -3,9 +3,9 @@
 import { existsSync, readFileSync, readdirSync, statSync, watch, appendFileSync, mkdirSync, type FSWatcher } from "node:fs";
 import { join, dirname } from "node:path";
 import {
-  home, type Item, boardCards, knownProjects, remember, locate, label, tilde, searchRoots, findProjects,
+  home, type Item, type Card, boardCards, knownProjects, remember, locate, label, tilde, searchRoots, findProjects,
   readInbox, writeInbox, replaceIdea, abandonRun, launchRun, parseItem, short, setArchived, writeNotes,
-  appendNote, saveAttachment, pidOnPort, runGit, readCommit, commitPush, notice, notifier, LOGO,
+  appendNote, saveAttachment, pidOnPort, runGit, cardsGit, readCommit, commitPush, notice, notifier, LOGO,
   reviewFindings, pickFindings, fixPrompt,
 } from "./state.ts";
 import { readSources, addSource, removeSource, moveSource, gatherSource, gatherAll, gatherDirs, due } from "./sources.ts";
@@ -197,11 +197,17 @@ export async function web(repoDir: string, opts: Opts = {}) {
         // Before the project list: it registers folders that turn out to be set up.
         const unset = await fresh();
         const dirs = states(all);
+        const cards = boardCards(dirs, Date.now());
+        // The git mark, for the cards whose panel offers commit and push: one status per project.
+        const marks = new Map<string, Record<string, string>>();
+        const markable = (c: Card) => c.kind === "run" && !c.archived && (c.status === "done" || c.awaiting === "approvals.review") && c.state !== join(home, ".kaizen");
+        for (const d of new Set(cards.filter(markable).map((c) => c.state)))
+          marks.set(d, cardsGit(d, cards.filter((c) => markable(c) && c.state === d).map((c) => c.id!)));
         return json({
           project, all, dir: state,
           projects: states(true).map((d) => ({ dir: d, label: label(d) })),
           fresh: unset.map((d) => ({ dir: join(d, ".kaizen"), label: tilde(d) })),
-          cards: boardCards(dirs, Date.now()).map((c) => ({ ...c, notice: notice(c) })),
+          cards: cards.map((c) => ({ ...c, notice: notice(c), git: markable(c) ? marks.get(c.state)?.[c.id!] : undefined })),
           now: Date.now(),
         });
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kaizen-agent",
-  "version": "1.8.0",
+  "version": "1.8.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

@@ -106,6 +106,7 @@ section h2 .clear{margin-left:auto;min-height:22px;padding:0 8px}
 .chip svg{width:11px;height:11px;stroke-width:1.7}
 .chip.warn{color:var(--warn);background:color-mix(in srgb,var(--warn) 14%,transparent)}.chip.warn i{background:var(--warn)}
 .chip.accent{color:var(--accent);background:color-mix(in srgb,var(--accent) 14%,transparent)}.chip.accent i{background:var(--accent)}
+.chip.ok{color:color-mix(in srgb,var(--ok) 80%,var(--ink));background:color-mix(in srgb,var(--ok) 14%,transparent)}
 .chip.run{color:var(--run);background:color-mix(in srgb,var(--run) 14%,transparent)}.chip.run i{background:var(--run)}
 .st.d-idea{color:var(--dim)}.st.d-run{color:var(--run)}.st.d-stall{color:var(--dim)}.st.d-wait{color:var(--warn)}.st.d-done{color:var(--ok)}.st.d-aband{color:var(--dim)}
 .d-run{color:var(--run)}.d-wait{color:var(--warn)}.d-done{color:var(--ok)}
@@ -224,6 +225,7 @@ aside .foot:empty{display:none}
 <symbol id="c-cursor" viewBox="0 0 16 16"><path d="M4 3l9 5-4 1-1 4z"/></symbol>
 <symbol id="c-full" viewBox="0 0 16 16"><circle cx="5.5" cy="8" r="3"/><circle cx="11" cy="8" r="3"/></symbol>
 <symbol id="c-lite" viewBox="0 0 16 16"><circle cx="8" cy="8" r="3"/></symbol>
+<symbol id="c-git" viewBox="0 0 16 16"><circle cx="4.5" cy="3.5" r="1.5"/><circle cx="4.5" cy="12.5" r="1.5"/><circle cx="11.5" cy="5.5" r="1.5"/><path d="M4.5 5v6M11.5 7c0 2.5-7 1.5-7 4"/></symbol>
 </defs></svg>
 
 <div class="hdr">
@@ -387,6 +389,8 @@ function card(c){
  if(all)chips.push(`<span class="chip">${esc(c.where)}</span>`);
  chips.push(...(c.tags||[]).map(t=>{const mark=MARKS[t]?`<svg><use href="#c-${t}"/></svg>`:"";return `<span class="chip${t==="full"||t==="lite"?" accent":""}">${mark}${esc(NAMES[t]||t)}</span>`}));
  if(c.notes)chips.push(`<span class="chip">notes</span>`);
+ // Information only: the panel's button is where it is committed and pushed.
+ if(c.git){const t=c.git==="pushed"?"Committed & pushed":"Not committed & pushed yet";chips.push(`<span class="chip ${c.git==="pushed"?"ok":"warn"}" role="img" title="${t}" aria-label="${t}"><svg><use href="#c-git"/></svg></span>`)}
  const pick=picking&&!archive&&c.column===4;
  el.innerHTML=(pick?`<input type="checkbox" class="pick" ${picked.has(k)?"checked":""} aria-label="Select">`:"")+`<div class="c-id"><span class="st d-${st}"><svg><use href="#s-${st}"/></svg></span><span class="idt">${esc(idt)}</span><span class="age">${c.kind==="run"?age(c.moved||S.now,S.now):""}</span><button class="ib sm more" data-a="menu" title="Actions" aria-label="Actions">${ico("more")}</button></div>
   <div class="c-title">${esc(c.kind==="idea"?c.text:(c.title||c.id))}</div>
@@ -563,9 +567,9 @@ function gitBtn(c){const g=runDoc.git,k=runDoc.commit;
  if(k&&k.pushed)return `<button class="btn" disabled title="${esc(k.sha)}">Committed &amp; pushed</button>`+(g.files.length?`<button class="btn" data-a="commit">Commit new changes</button>`:g.ahead>0?`<button class="btn" data-a="push">Push</button>`:"");
  if(g.files.length)return `<button class="btn pri" data-a="commit">Commit &amp; push</button>`;
  if(g.ahead>0)return `<button class="btn pri" data-a="push">Push</button>`;
- // Clean and level with the remote: it was committed and pushed somewhere else. With
- // other files still changed and no record, nothing says this run was ever committed.
- return k||!g.others?`<button class="btn" disabled>Committed &amp; pushed</button>`:""}
+ // None of its files changed and level with the remote: it was committed and pushed
+ // somewhere else. With no record and no report, nothing says it was ever committed.
+ return k||runDoc.impl?`<button class="btn" disabled>Committed &amp; pushed</button>`:""}
 function confirmCommit(c,msg){const g=runDoc.git,foot=$("pfoot");
  foot.innerHTML=`<div class="inl" style="flex:1"><div class="q">Commit ${g.files.length} changed file${g.files.length===1?"":"s"} on ${esc(g.branch||"this branch")} and push to ${esc(g.remote)}?</div><div class="q dim">Only the files this run's report names.${g.others?` ${g.others} other changed file${g.others===1?" is":"s are"} left alone.`:""}</div><pre class="files">${esc(g.files.join("\n"))}</pre>${g.shared&&g.shared.length?`<div class="q">These files also hold other runs' changes, and a file is committed whole:</div><pre class="files">${esc(g.shared.join("\n"))}</pre>`:""}<textarea aria-label="Commit message" placeholder="feat(scope): what changed">${esc(msg||commitMsg(runDoc.impl)||c.title||c.text)}</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">Commit &amp; push</button></div></div>`;
  const inp=foot.querySelector("textarea"),go=foot.querySelector("[data-a=s]");inp.focus();