fapony 0.1.0

package/src/digest/html.ts

@@ -0,0 +1,208 @@
+// src/digest/html.ts — render digest as a single HTML page
+//
+// คอลัมน์เดียว max-width: 68rem · ไม่มี tab ไม่มีปุ่มพับ ไม่มี JS
+// ใช้ DARK_THEME_CSS + TABLE_CSS จาก src/web/html.ts
+
+import { DARK_THEME_CSS, esc, TABLE_CSS } from "../web/html.js";
+import type {
+  CostRow,
+  DigestData,
+  MemRow,
+  PlanRow,
+  RegimeModelRow,
+  Tally,
+} from "./collect.js";
+
+function fmtDate(iso: string): string {
+  return iso.slice(0, 10);
+}
+
+function fmtTokens(n: number): string {
+  if (n >= 1_000_000) return `${(n / 1_000_000).toFixed(1)}M`;
+  if (n >= 1_000) return `${(n / 1_000).toFixed(0)}K`;
+  return String(Math.round(n));
+}
+
+function memRow(r: MemRow): string {
+  const date = r.ts.slice(5, 10);
+  return `<tr>
+    <td class="muted">${esc(date)}</td>
+    <td>${esc(r.agent)}</td>
+    <td>${esc(r.text.replace(/\n/g, " ").slice(0, 120))}</td>
+  </tr>`;
+}
+
+function planRow(p: PlanRow): string {
+  const name = esc(p.file.replace(/\.md$/, ""));
+  const status = esc(p.status ?? "active");
+  const progress = p.total > 0 ? `${p.done}/${p.total}` : "\u2014";
+  return `<tr>
+    <td>${name}</td>
+    <td class="muted">${status}</td>
+    <td class="muted">${progress}</td>
+  </tr>`;
+}
+
+function costRow(c: CostRow): string {
+  const name = c.provider ? `${esc(c.provider)}/${esc(c.model)}` : esc(c.model);
+  const costStr =
+    c.imputed > 0
+      ? `~$${c.imputed.toFixed(4)} imputed`
+      : c.cost > 0
+        ? `$${c.cost.toFixed(4)} recorded`
+        : "unpriced";
+  return `<tr>
+    <td>${name}</td>
+    <td class="muted">${c.sessions}</td>
+    <td>${fmtTokens(c.input)}</td>
+    <td>${fmtTokens(c.output)}</td>
+    <td>${costStr}</td>
+  </tr>`;
+}
+
+function tallyRow(t: Tally): string {
+  return `<tr><td>${esc(t.key)}</td><td>${t.count}</td></tr>`;
+}
+
+function regimeModelRow(r: RegimeModelRow): string {
+  return `<tr>
+    <td class="muted">${esc(r.regime)}</td>
+    <td>${esc(r.model)}</td>
+    <td>${r.gates}</td>
+    <td>${r.fails}</td>
+  </tr>`;
+}
+
+function section(title: string, content: string, cls = ""): string {
+  return `<h2 class="${cls}">${esc(title)}</h2>\n${content}`;
+}
+
+export function renderDigestHtml(d: DigestData): string {
+  const units = d.verdicts.units_graded;
+  const costBits: string[] = [];
+  if (d.cost.imputed_usd > 0)
+    costBits.push(`~$${d.cost.imputed_usd.toFixed(4)} (imputed)`);
+  if (d.cost.total_usd > 0)
+    costBits.push(`$${d.cost.total_usd.toFixed(4)} (recorded)`);
+  const costLabel = costBits.length > 0 ? costBits.join(" · ") : "$0";
+
+  // decisions table
+  const decisionsHtml =
+    d.decisions.length > 0
+      ? `<table><thead><tr><th>Date</th><th>Agent</th><th>Decision</th></tr></thead><tbody>${d.decisions.map(memRow).join("")}</tbody></table>`
+      : '<p class="muted">(none this period)</p>';
+
+  // open bugs
+  const bugsOpenHtml =
+    d.bugs.open.length > 0
+      ? `<table><thead><tr><th>Date</th><th>Agent</th><th>Bug</th></tr></thead><tbody>${d.bugs.open.map(memRow).join("")}</tbody></table>`
+      : '<p class="muted">(none)</p>';
+  const closedNote =
+    d.bugs.closed.length > 0
+      ? `<p class="muted">${d.bugs.closed.length} closed this period</p>`
+      : "";
+
+  // plans
+  const inFlight = d.plans.pending.filter(
+    (p) => p.status === "active" || !p.status,
+  );
+  const plansHtml =
+    inFlight.length > 0
+      ? `<table><thead><tr><th>Plan</th><th>Status</th><th>Progress</th></tr></thead><tbody>${inFlight.map(planRow).join("")}</tbody></table>`
+      : '<p class="muted">(none)</p>';
+  const shippedNote =
+    d.plans.shipped.length > 0
+      ? `<p class="muted">${d.plans.shipped.length} shipped this period</p>`
+      : "";
+
+  // cost
+  const costHtml =
+    d.cost.by_model.length > 0
+      ? `<div style="overflow-x:auto"><table><thead><tr><th>Model</th><th>Sessions</th><th>In</th><th>Out</th><th>Cost</th></tr></thead><tbody>${d.cost.by_model.map(costRow).join("")}</tbody></table></div>` +
+        (d.cost.unpriced_sessions > 0
+          ? `<p class="muted">${d.cost.unpriced_sessions} sessions unpriced (no rate found — not counted as free)</p>`
+          : "")
+      : '<p class="muted">(no usage data)</p>';
+
+  // verdicts — by_grade
+  const gradeHtml =
+    d.verdicts.by_grade.length > 0
+      ? `<table><thead><tr><th>Grade</th><th>Count</th></tr></thead><tbody>${d.verdicts.by_grade.map(tallyRow).join("")}</tbody></table>`
+      : "";
+
+  // verdicts — by_reason_code
+  const reasonHtml =
+    d.verdicts.by_reason_code.length > 0
+      ? `<table><thead><tr><th>Reason</th><th>Count</th></tr></thead><tbody>${d.verdicts.by_reason_code.map(tallyRow).join("")}</tbody></table>`
+      : '<p class="muted">no graded fails in this period</p>';
+
+  // verdicts — by_regime_model
+  const regimeHtml =
+    d.verdicts.by_regime_model.length > 0
+      ? `<div style="overflow-x:auto"><table><thead><tr><th>Regime</th><th>Model</th><th>Gates</th><th>Fails</th></tr></thead><tbody>${d.verdicts.by_regime_model.map(regimeModelRow).join("")}</tbody></table></div>`
+      : "";
+
+  // sources
+  const sourcesHtml = `<table><thead><tr><th>Source</th><th>Status</th><th>Detail</th></tr></thead><tbody>${d.sources
+    .map(
+      (s) =>
+        `<tr><td>${esc(s.name)}</td><td class="${s.ok ? "pass" : "fail"}">${s.ok ? "ok" : "!!"}</td><td class="muted">${esc(s.detail)}</td></tr>`,
+    )
+    .join("")}</tbody></table>`;
+
+  const malformedNote =
+    d.skipped_malformed > 0
+      ? `<p class="muted">${d.skipped_malformed} malformed log lines skipped</p>`
+      : "";
+
+  return `<!DOCTYPE html>
+<html lang="en">
+<head>
+<meta charset="utf-8">
+<meta name="viewport" content="width=device-width, initial-scale=1">
+<title>fapony digest — ${fmtDate(d.since)} → ${fmtDate(d.generated_at)}</title>
+<style>
+  ${DARK_THEME_CSS}
+  ${TABLE_CSS}
+  body { max-width: 68rem; }
+  .headline { font-size: 1.1rem; margin: 1rem 0 1.5rem; }
+  .scope-note { color: var(--muted); font-size: 0.85rem; margin-bottom: 1.5rem; }
+  table { margin-bottom: 1.5rem; }
+  td:first-child { font-weight: 600; }
+</style>
+</head>
+<body>
+
+<h1>fapony digest — ${esc(fmtDate(d.since))} → ${esc(fmtDate(d.generated_at))}</h1>
+<div class="scope-note">${esc(d.worktree)}<br>${esc(d.scope_note)}</div>
+
+<div class="headline">
+  <strong>${units}</strong> unit${units === 1 ? "" : "s"} graded · <strong>${d.verdicts.round1_pct}%</strong> passed round 1 · <strong>${esc(costLabel)}</strong>
+</div>
+
+${section("Decisions", decisionsHtml)}
+
+${section(
+  `Notes (${d.notes.length})`,
+  d.notes.length > 0
+    ? `<table><thead><tr><th>Date</th><th>Agent</th><th>Note</th></tr></thead><tbody>${d.notes.map(memRow).join("")}</tbody></table>`
+    : '<p class="muted">(none this period)</p>',
+)}
+
+${section(`Open Bugs (${d.bugs.open.length})`, bugsOpenHtml + closedNote)}
+
+${section(`In Flight (${inFlight.length})`, plansHtml + shippedNote)}
+
+${section("Cost", costHtml)}
+
+${section("How Work Failed", reasonHtml)}
+${gradeHtml ? `\n${section("By Grade", gradeHtml)}` : ""}
+${regimeHtml ? `\n${section("By Regime × Model", regimeHtml)}` : ""}
+
+${section("Sources", sourcesHtml)}
+
+${malformedNote}
+
+</body>
+</html>`;
+}

package/src/digest/text.ts

@@ -0,0 +1,191 @@
+// src/digest/text.ts — render digest as terminal text
+//
+// ลำดับ section ตาม SPEC §3 (เล่าเรื่อง ไม่ใช่ dashboard)
+
+import type { CostRow, DigestData, MemRow, PlanRow } from "./collect.js";
+
+function fmtDate(iso: string): string {
+  return iso.slice(0, 10);
+}
+
+function fmtTime(iso: string): string {
+  return iso.slice(5, 10); // MM-DD
+}
+
+function section(name: string, lines: string[]): string {
+  return `${name}\n${lines.map((l) => `  ${l}`).join("\n")}`;
+}
+
+function memLine(r: MemRow): string {
+  const date = fmtTime(r.ts);
+  return `${date}  ${r.agent}  ${r.text.replace(/\n/g, " ").slice(0, 80)}`;
+}
+
+function planLine(p: PlanRow): string {
+  const name = p.file.replace(/\.md$/, "");
+  const status = p.status ?? "active";
+  const progress = p.total > 0 ? `${p.done}/${p.total}` : "—";
+  return `${name.padEnd(30)} ${status.padEnd(12)} ${progress}`;
+}
+
+function fmtTokens(n: number): string {
+  if (n >= 1_000_000) return `${(n / 1_000_000).toFixed(1)}M`;
+  if (n >= 1_000) return `${(n / 1_000).toFixed(0)}K`;
+  return String(Math.round(n));
+}
+
+function costLine(c: CostRow): string {
+  const name = c.provider ? `${c.provider}/${c.model}` : c.model;
+  const costStr =
+    c.imputed > 0
+      ? `$${c.imputed.toFixed(4)} imputed`
+      : c.cost > 0
+        ? `$${c.cost.toFixed(4)} recorded`
+        : "unpriced";
+  return `${name.padEnd(45)} ${String(c.sessions).padStart(5)} sessions  ${fmtTokens(c.input).padStart(7)} in  ${fmtTokens(c.output).padStart(7)} out  ${costStr}`;
+}
+
+export function renderDigestText(d: DigestData): string {
+  const lines: string[] = [];
+
+  // 1. Header
+  lines.push(
+    `fapony digest — ${fmtDate(d.since)} → ${fmtDate(d.generated_at)}`,
+  );
+  lines.push(`  ${d.worktree}`);
+  lines.push(`  scope: ${d.scope_note}`);
+  lines.push("");
+
+  // 2. Headline — run-based counts from collect (gate tallies are not units)
+  const units = d.verdicts.units_graded;
+  const costBits: string[] = [];
+  if (d.cost.imputed_usd > 0)
+    costBits.push(`$${d.cost.imputed_usd.toFixed(4)} (imputed)`);
+  if (d.cost.total_usd > 0)
+    costBits.push(`$${d.cost.total_usd.toFixed(4)} (recorded)`);
+  const costLabel = costBits.length > 0 ? costBits.join(" · ") : "$0";
+
+  lines.push(
+    `  ${units} unit${units === 1 ? "" : "s"} graded · ${d.verdicts.round1_pct}% passed round 1 · ${costLabel}`,
+  );
+  lines.push("");
+
+  // 3. Decisions
+  lines.push(
+    section(`DECISIONS (${d.decisions.length})`, [
+      ...d.decisions.map(memLine),
+      ...(d.decisions.length === 0 ? ["(none this period)"] : []),
+    ]),
+  );
+  lines.push("");
+
+  // 3b. Notes — mem rows like decisions; the sync payload must stay visible
+  lines.push(
+    section(`NOTES (${d.notes.length})`, [
+      ...d.notes.map(memLine),
+      ...(d.notes.length === 0 ? ["(none this period)"] : []),
+    ]),
+  );
+  lines.push("");
+
+  // 4. Open bugs
+  const openBugLines = d.bugs.open.map(memLine);
+  const closedCount = d.bugs.closed.length;
+  lines.push(
+    section(`OPEN BUGS (${d.bugs.open.length})`, [
+      ...openBugLines,
+      ...(d.bugs.open.length === 0 ? ["(none)"] : []),
+      ...(closedCount > 0 ? [`(${closedCount} closed this period)`] : []),
+    ]),
+  );
+  lines.push("");
+
+  // 5. In flight
+  const inFlight = d.plans.pending.filter(
+    (p) => p.status === "active" || !p.status,
+  );
+  const shipped = d.plans.shipped;
+  lines.push(
+    section(`IN FLIGHT (${inFlight.length})`, [
+      ...inFlight.map(planLine),
+      ...(inFlight.length === 0 ? ["(none)"] : []),
+      ...(shipped.length > 0 ? [`${shipped.length} shipped this period`] : []),
+    ]),
+  );
+  lines.push("");
+
+  // 6. Cost
+  if (d.cost.by_model.length > 0) {
+    lines.push(
+      section("COST", [
+        "model".padEnd(45) +
+          "sessions".padStart(5) +
+          "     in".padStart(9) +
+          "    out".padStart(9) +
+          "  cost",
+        ...d.cost.by_model.map(costLine),
+        ...(d.cost.unpriced_sessions > 0
+          ? [
+              `${d.cost.unpriced_sessions} sessions unpriced (no rate found — not counted as free)`,
+            ]
+          : []),
+      ]),
+    );
+  } else {
+    lines.push(section("COST", ["(no usage data)"]));
+  }
+  lines.push("");
+
+  // 7. How work failed — by_grade first (every payload field needs a place)
+  if (d.verdicts.by_grade.length > 0) {
+    lines.push(
+      section("BY GRADE", [
+        ...d.verdicts.by_grade.map((t) => `${t.key.padEnd(20)} ${t.count}`),
+      ]),
+    );
+    lines.push("");
+  }
+
+  if (d.verdicts.by_reason_code.length > 0) {
+    lines.push(
+      section("HOW WORK FAILED", [
+        ...d.verdicts.by_reason_code.map(
+          (t) => `${t.key.padEnd(20)} ${t.count}`,
+        ),
+      ]),
+    );
+  } else {
+    lines.push(section("HOW WORK FAILED", ["no graded fails in this period"]));
+  }
+
+  if (d.verdicts.by_regime_model.length > 0) {
+    lines.push("");
+    lines.push(
+      section("BY REGIME × MODEL", [
+        `${"regime".padEnd(10)}${"model".padEnd(40)}gates  fails`,
+        ...d.verdicts.by_regime_model.map(
+          (r) =>
+            `${r.regime.padEnd(10)}${r.model.padEnd(40)}${String(r.gates).padStart(5)}  ${String(r.fails).padStart(5)}`,
+        ),
+      ]),
+    );
+  }
+  lines.push("");
+
+  // 8. Sources
+  lines.push(
+    section(
+      "SOURCES",
+      d.sources.map(
+        (s) => `${s.name.padEnd(10)} ${s.ok ? "ok" : "!!"}    ${s.detail}`,
+      ),
+    ),
+  );
+
+  if (d.skipped_malformed > 0) {
+    lines.push("");
+    lines.push(`  (${d.skipped_malformed} malformed log lines skipped)`);
+  }
+
+  return lines.join("\n");
+}

package/src/gate.ts

@@ -0,0 +1,153 @@
+// gateOnce is the core review-verdict logic — still live, called by the MCP
+// verdict_submit tool (src/mcp/tools/verdict.ts). The CLI wrapper (cmdGate)
+// was removed with the rest of the execute→review→fix loop (Wave 2).
+import {
+  addEvent,
+  getRun,
+  incrementRound,
+  loadConfig,
+  openDb,
+  setStatus,
+} from "./db/index.js";
+import { closeMemory, kickoffMemory } from "./memory.js";
+import { isPassFamily, type VerdictGrade } from "./parse.js";
+
+export interface GateResult {
+  runId: number;
+  status: "passed" | "fixing" | "stopped" | "stalled";
+  round?: number;
+  error?: string;
+}
+
+/**
+ * Core gate logic — does NOT call process.exit.
+ * CLI wrapper (cmdGate) is responsible for exit codes.
+ *
+ * Verdict routing (3 groups):
+ *   pass-family (pass-excellent|pass-good|pass-adequate|pass) → passed
+ *   fail → fixing (+ round cap check → stalled)
+ *   uncertain → stopped (same shape as round-cap path)
+ */
+export function gateOnce(
+  runId: number,
+  verdict: VerdictGrade,
+  note: string,
+  files?: string[],
+): GateResult {
+  const db = openDb();
+  const run = getRun(db, runId);
+  if (!run) {
+    return { runId, status: "stopped", error: `run ${runId} not found` };
+  }
+  if (
+    run.status === "passed" ||
+    run.status === "stopped" ||
+    run.status === "stalled"
+  ) {
+    return {
+      runId,
+      status: run.status,
+      round: run.round,
+      error: `run ${runId} is already ${run.status}`,
+    };
+  }
+
+  const config = loadConfig();
+  // runs.worktree เก็บ absolute path (SERVER_INSTRUCTIONS บังคับ) — ไม่ใช่ key ของ config.worktrees
+  // การ lookup จึงพลาดเสมอ และ fallback เดิม "." ทำให้คำสั่ง memory ของโปรเจกต์ไปรันใน cwd ของ
+  // MCP server แทนที่จะเป็นโปรเจกต์นั้น · lookup ยังอยู่เผื่อ row เก่าที่เคยเก็บเป็น key
+  const worktree = config.worktrees[run.worktree] ?? run.worktree;
+
+  // --- pass family (4 grades) ---
+  if (isPassFamily(verdict)) {
+    setStatus(db, runId, "passed");
+    // round = run's current round at review time (pre-increment for fail).
+    addEvent(db, runId, "gate", {
+      verdict,
+      note,
+      round: run.round,
+      ...(files && files.length > 0 ? { files } : {}),
+    });
+
+    if (run.mem_id && config.memory) {
+      closeMemory(config, worktree, run.mem_id, note || `run ${runId} passed`);
+      addEvent(db, runId, "memory_claim_closed", { mem_id: run.mem_id });
+    }
+
+    const kickoff = kickoffMemory(config, worktree);
+    if (kickoff) console.log(kickoff);
+
+    return { runId, status: "passed", round: run.round };
+  }
+
+  // --- uncertain → stop (plan problem, same as round-cap) ---
+  if (verdict === "uncertain") {
+    setStatus(db, runId, "stopped");
+    addEvent(db, runId, "gate", {
+      verdict,
+      note,
+      round: run.round,
+      ...(files && files.length > 0 ? { files } : {}),
+    });
+    addEvent(db, runId, "stop", { reason: "verdict_uncertain" });
+
+    if (run.mem_id) {
+      closeMemory(
+        config,
+        worktree,
+        run.mem_id,
+        `run ${runId} stopped — uncertain verdict`,
+      );
+      addEvent(db, runId, "memory_claim_closed", { mem_id: run.mem_id });
+    }
+    return { runId, status: "stopped", round: run.round };
+  }
+
+  // --- fail → back to executor, one more round ---
+  incrementRound(db, runId);
+  setStatus(db, runId, "fixing");
+  addEvent(db, runId, "gate", {
+    verdict,
+    note,
+    round: run.round,
+    ...(files && files.length > 0 ? { files } : {}),
+  });
+
+  const updated = getRun(db, runId);
+  if (!updated) {
+    setStatus(db, runId, "stopped");
+    return {
+      runId,
+      status: "stopped",
+      error: `run ${runId} disappeared after update`,
+    };
+  }
+
+  if (updated.round > config.review.maxRounds) {
+    // Round cap reached — the plan is the problem, stall for real. Persist it
+    // (else the run sits in 'fixing' forever) and release the memory claim.
+    // stalled (not stopped): tells the caller to take it back to the human
+    // instead of sending the agent for another fix round.
+    setStatus(db, runId, "stalled");
+    addEvent(db, runId, "stop", {
+      reason: `round ${updated.round} > maxRounds ${config.review.maxRounds}`,
+    });
+    if (run.mem_id) {
+      closeMemory(
+        config,
+        worktree,
+        run.mem_id,
+        `run ${runId} stalled at round cap`,
+      );
+      addEvent(db, runId, "memory_claim_closed", { mem_id: run.mem_id });
+    }
+    return {
+      runId,
+      status: "stalled",
+      round: updated.round,
+      error: `round ${updated.round} > maxRounds ${config.review.maxRounds} — stop fixing and take it back to the human, the plan likely has a problem`,
+    };
+  }
+
+  return { runId, status: "fixing", round: updated.round };
+}