opencode-usage-coach 0.3.3 → 0.3.5

package/README.md

@@ -28,6 +28,16 @@ the loop** — and ships a harness agent mode.
   explicitly ask "run this through the harness" or "use the harness for this". The harness
   tools are only available when the harness agent mode is active (see Install).
 
+**Learning from failures (learning loop):**
+- When `grade` returns FAIL, the harness enters a learning cycle: `record_failure` → `investigate` (root-cause analysis) → `verify_diagnosis` → `generalize` (extract a reusable rule).
+- Rules accumulate in `rules.md` → the next `generate` call automatically includes them → the harness avoids repeating the same mistake.
+- Tools: `record_failure`, `investigate`, `verify_diagnosis`, `generalize`.
+
+**Domain knowledge base:**
+- `investigate` and `generate` query a local domain DB before running — known facts are injected into the prompt ("Known facts from domain DB: ...").
+- Unknown domains are investigated (webfetch/docs) and stored as graph nodes/edges → accumulates over time → evidence-based judgments instead of speculation.
+- Storage: `nodes.ndjson` + `edges.ndjson` under the project state dir.
+
 ## Requirements
 - opencode (tested on 1.17.13) with a quota-metered provider configured.
 - `codexbar` CLI with your provider key wired (e.g. `codexbar config set-api-key --provider zai --stdin`).
@@ -203,9 +213,25 @@ Recurring issues and fixes — mostly learned the hard way during development.
 - Harness completion sets `active:false` → hidden from the TUI.
 - Override the state path with `UC_STATE_DIR` (forces global state).
 
+**Key gotcha — opencode TUI `ctx` does NOT carry `session_id`.**
+The slot context passed to `panel(ctx)` contains only `{ theme }`. There is no `session_id`/`sessionID` field. The current session ID lives in **`api.route.current.params.sessionID`** instead — the panel reads it from there. If you ever see harnesses from other sessions leaking in, the cause is almost certainly that `sid` resolved to empty (→ fallback broad scan).
+
+**Debugging session isolation** (if it breaks again):
+1. Check `~/.cache/opencode-usage-coach/projects/<hash>/tui-debug.log` — is `panel` being called? What `routeSid` value?
+2. `api.route.current.params.sessionID` — populated? (Empty → panel falls back to scanning all sessions.)
+3. New TUI code loaded? `tui-loaded.txt` (MARKER) should show `loaded-v2 ...`. If it still says `loaded`, the new dist isn't being picked up.
+4. `appendFileSync` imported in `src/tui.tsx`? If missing, **all TUI debug logging silently fails** (ReferenceError swallowed by try/catch) — this wasted a lot of debugging time once.
+
+**Past issue (fixed v0.3.4):** panel read `ctx.session_id` which was always `undefined` → fallback scanned every session → another session's active harness leaked in. Fixed by reading `api.route.current.params.sessionID`.
+
 ## Status
-- ✅ Quota guardian + TUI panel (per-provider coach view, colors, collapsible Alt+H)
-- ✅ Harness: agent mode (triage) with generate/grade model-specific tools (1 terminal, multi-model)
+- ✅ Quota guardian + TUI panel (per-provider coach view, 5h/1w gauges, collapsible Alt+H)
+- ✅ Harness: agent mode with generate/grade tools (multi-model, 1 terminal)
+- ✅ Deterministic loop via NEXT directives (parallel PATH A / sequential PATH B)
+- ✅ Quota-aware tools (GO/THROTTLE/STOP drive model selection + concurrency)
+- ✅ Learning loop (record_failure → investigate → verify_diagnosis → generalize → rules.md)
+- ✅ Domain knowledge base (graph store, investigate/generate injection)
+- ✅ Session isolation (api.route, per-session harness state)
 - ✅ npm packaging (`opencode plugin install opencode-usage-coach`)
 
 License: MIT.

package/dist/index.js

@@ -1,63 +1,145 @@
 // src/index.ts
-import { mkdirSync, writeFileSync, appendFileSync, readFileSync, existsSync } from "fs";
+import { mkdirSync as mkdirSync2, writeFileSync, appendFileSync as appendFileSync2, readFileSync as readFileSync2, existsSync as existsSync2 } from "fs";
 import { spawn } from "child_process";
 import { createHash } from "crypto";
 import { homedir } from "os";
-import { join, resolve, dirname } from "path";
+import { join as join2, resolve, dirname } from "path";
 import { tool } from "@opencode-ai/plugin";
+
+// src/domain.ts
+import { mkdirSync, appendFileSync, readFileSync, existsSync } from "fs";
+import { join } from "path";
+var BASE_DIR = "";
+function initDomain(stateDir) {
+  BASE_DIR = stateDir;
+}
+var nodesFile = () => join(BASE_DIR, "nodes.ndjson");
+var edgesFile = () => join(BASE_DIR, "edges.ndjson");
+function readNdjson(path) {
+  try {
+    if (!existsSync(path)) return [];
+    return readFileSync(path, "utf8").split("\n").filter(Boolean).map((l) => JSON.parse(l));
+  } catch {
+    return [];
+  }
+}
+function readNodes() {
+  return readNdjson(nodesFile());
+}
+function readEdges() {
+  return readNdjson(edgesFile());
+}
+function uid(prefix) {
+  return `${prefix}_${Date.now().toString(36)}_${Math.random().toString(36).slice(2, 8)}`;
+}
+function addDomainNode(node) {
+  const full = { ...node, id: uid("node"), ts: (/* @__PURE__ */ new Date()).toISOString() };
+  try {
+    mkdirSync(BASE_DIR, { recursive: true });
+    appendFileSync(nodesFile(), JSON.stringify(full) + "\n");
+  } catch {
+  }
+  return full.id;
+}
+function queryDomain(keywords) {
+  const lc = keywords.map((k) => k.toLowerCase());
+  const nodes = readNodes();
+  const matched = nodes.filter((n) => {
+    const hay = (n.name + " " + JSON.stringify(n.props)).toLowerCase();
+    return lc.some((k) => k && hay.includes(k));
+  });
+  const ids = new Set(matched.map((n) => n.id));
+  const edges = readEdges().filter((e) => ids.has(e.from) || ids.has(e.to));
+  return { nodes: matched, edges };
+}
+function saveInvestigationResult(keywords, result, source) {
+  try {
+    return addDomainNode({
+      type: "fact",
+      name: keywords.join(" "),
+      props: { result },
+      source: source || "investigation",
+      confidence: 0.7
+    });
+  } catch {
+    return "";
+  }
+}
+
+// src/index.ts
 var PLUGIN_NAME = "opencode-usage-coach";
 var DEBUG = process.env.UC_DEBUG === "1";
 var TTL_MS = Number(process.env.UC_TTL_MS ?? 6e4);
-var STATE_DIR = join(homedir(), ".cache", "opencode-usage-coach");
-var STATE_FILE = join(STATE_DIR, "state.json");
-var HARNESS_FILE = join(STATE_DIR, "harness.json");
-var LOG_FILE = join(STATE_DIR, "coach.log");
+var STATE_DIR = join2(homedir(), ".cache", "opencode-usage-coach");
+var STATE_FILE = join2(STATE_DIR, "state.json");
+var HARNESS_FILE = join2(STATE_DIR, "harness.json");
+var LOG_FILE = join2(STATE_DIR, "coach.log");
 function projectStateDir(dir) {
   const abs = resolve(dir || ".");
   const h = createHash("sha1").update(abs).digest("hex").slice(0, 12);
-  return join(homedir(), ".cache", "opencode-usage-coach", "projects", h);
+  return join2(homedir(), ".cache", "opencode-usage-coach", "projects", h);
 }
 function setStateDir(dir) {
   STATE_DIR = process.env.UC_STATE_DIR ?? projectStateDir(dir);
-  STATE_FILE = join(STATE_DIR, "state.json");
-  HARNESS_FILE = join(STATE_DIR, "harness.json");
-  LOG_FILE = join(STATE_DIR, "coach.log");
+  STATE_FILE = join2(STATE_DIR, "state.json");
+  HARNESS_FILE = join2(STATE_DIR, "harness.json");
+  LOG_FILE = join2(STATE_DIR, "coach.log");
 }
 var NOOP_HOOKS = {};
 function log(msg) {
   try {
-    appendFileSync(LOG_FILE, `${(/* @__PURE__ */ new Date()).toISOString()} ${msg}
+    appendFileSync2(LOG_FILE, `${(/* @__PURE__ */ new Date()).toISOString()} ${msg}
 `);
   } catch {
   }
 }
 function writeState(c) {
   try {
-    mkdirSync(STATE_DIR, { recursive: true });
+    mkdirSync2(STATE_DIR, { recursive: true });
     writeFileSync(STATE_FILE, JSON.stringify({ ...c, updatedAt: (/* @__PURE__ */ new Date()).toISOString() }));
   } catch {
   }
 }
 function rulesFile() {
-  return join(STATE_DIR, "rules.md");
+  return join2(STATE_DIR, "rules.md");
+}
+function failuresFile() {
+  return join2(STATE_DIR, "failures.ndjson");
 }
 function readRules() {
   try {
     const f = rulesFile();
-    if (!existsSync(f)) return "";
-    return readFileSync(f, "utf8").trim();
+    if (!existsSync2(f)) return "";
+    return readFileSync2(f, "utf8").trim();
   } catch {
     return "";
   }
 }
+function extractKeywords(text) {
+  try {
+    const STOP = /* @__PURE__ */ new Set(["the", "and", "for", "with", "that", "this", "from", "into", "your", "you", "are", "was", "but", "not", "all", "any", "use", "task", "prompt"]);
+    const seen = /* @__PURE__ */ new Set();
+    const out = [];
+    for (const raw of (text ?? "").toLowerCase().split(/[^a-z0-9_]+/)) {
+      const t = raw.trim();
+      if (t.length < 3 || STOP.has(t) || seen.has(t)) continue;
+      seen.add(t);
+      out.push(t);
+      if (out.length >= 16) break;
+    }
+    return out;
+  } catch {
+    return [];
+  }
+}
 function harnessFile(sessionID) {
-  return join(STATE_DIR, sessionID || "_default", "harness.json");
+  return join2(STATE_DIR, sessionID || "_default", "harness.json");
 }
 function readHarness(sessionID) {
   try {
     const f = harnessFile(sessionID);
-    if (!existsSync(f)) return null;
-    return JSON.parse(readFileSync(f, "utf8"));
+    if (!existsSync2(f)) return null;
+    return JSON.parse(readFileSync2(f, "utf8"));
   } catch {
     return null;
   }
@@ -65,7 +147,7 @@ function readHarness(sessionID) {
 function writeHarness(sessionID, h) {
   try {
     const f = harnessFile(sessionID);
-    mkdirSync(dirname(f), { recursive: true });
+    mkdirSync2(dirname(f), { recursive: true });
     h.updatedAt = (/* @__PURE__ */ new Date()).toISOString();
     writeFileSync(f, JSON.stringify(h, null, 2));
   } catch {
@@ -74,14 +156,14 @@ function writeHarness(sessionID, h) {
 function readHarnessCfg(dir) {
   const tryRead = (p) => {
     try {
-      if (existsSync(p)) return JSON.parse(readFileSync(p, "utf8"));
+      if (existsSync2(p)) return JSON.parse(readFileSync2(p, "utf8"));
     } catch {
     }
     return {};
   };
   return {
-    ...tryRead(join(homedir(), ".config", "opencode-usage-coach", "harness.config.json")),
-    ...tryRead(join(dir, "harness.config.json"))
+    ...tryRead(join2(homedir(), ".config", "opencode-usage-coach", "harness.config.json")),
+    ...tryRead(join2(dir, "harness.config.json"))
   };
 }
 async function runModel(client, model, prompt, directory) {
@@ -102,7 +184,12 @@ async function runModel(client, model, prompt, directory) {
     const parts = resp?.data?.parts ?? resp?.parts ?? [];
     const text = parts.filter((p) => p?.type === "text").map((p) => p?.text ?? "").join("");
     try {
-      await client.session.remove?.({ path: { id } });
+      const summary = await client.session.summarize?.({ path: { id } });
+      log(`runModel(${model}): sub-session summary: ${JSON.stringify(summary?.data ?? summary).slice(0, 300)}`);
+    } catch {
+    }
+    try {
+      await client.session.delete?.({ path: { id } });
     } catch {
     }
     log(`runModel(${model}): done ${elapsed}s, ${text.length} chars`);
@@ -242,6 +329,7 @@ var LOADING = { decision: "GO", advice: "quota loading\u2026", weekly: -1, month
 async function UsageCoachPlugin(input) {
   try {
     setStateDir(input.directory);
+    initDomain(STATE_DIR);
     const cfg0 = readHarnessCfg(input.directory);
     const PROVIDER = process.env.UC_PROVIDER ?? cfg0.provider ?? "";
     const LIGHTER = process.env.UC_LIGHTER_MODEL ?? cfg0.lighterModel ?? "a lighter model";
@@ -262,6 +350,10 @@ async function UsageCoachPlugin(input) {
               providers = await fetchProvidersCoach();
             } catch {
             }
+            if (providers.length > 0 && last.weekly < 0) {
+              const p0 = providers[0];
+              last = { ...last, weekly: p0.weekly, fiveHour: p0.fiveHour, monthly: p0.weekly >= 0 ? 0 : -1, advice: p0.advice, decision: p0.weekly >= STOP_WK ? "STOP" : p0.weekly >= THR_WK ? "THROTTLE" : "GO" };
+            }
             writeState({ ...last, providers, updatedAt: (/* @__PURE__ */ new Date()).toISOString() });
             log(`${last.decision} | weekly=${last.weekly}% 5h=${last.fiveHour}% | providers=${providers.length}`);
           } catch (e) {
@@ -292,7 +384,9 @@ async function UsageCoachPlugin(input) {
           log(`event err: ${String(e)}`);
         }
       },
-      // ACT(1) hard gate: only intentional STOP throws. Our own bugs never block.
+      // ACT(1) hard gate — ONLY for harness tools (generate/grade/etc.) that consume quota.
+      // General tools (read/edit/bash/grep/task) are NEVER blocked — they don't consume model quota.
+      // This ensures Agent-Factory-Coordinator and other modes work freely even at STOP.
       "tool.execute.before": async (_input) => {
         let decision = "GO";
         try {
@@ -301,7 +395,10 @@ async function UsageCoachPlugin(input) {
           decision = "GO";
         }
         if (decision === "STOP") {
-          throw new Error(`[${PLUGIN_NAME}] blocked: quota limit exceeded. ${current().advice}`);
+          const harnessTools = ["generate", "generate_batch", "grade", "investigate", "verify_diagnosis", "generalize"];
+          if (harnessTools.includes(_input.tool)) {
+            throw new Error(`[${PLUGIN_NAME}] blocked: quota limit exceeded. ${current().advice}`);
+          }
         }
       },
       // ACT(2) inject coaching into system prompt (double defense). Silent on error.
@@ -358,9 +455,12 @@ Then: harness_done(). Follow the [usage-coach NEXT] directive each tool returns.
             model: tool.schema.string().optional()
           },
           async execute(args, ctx) {
+            const cfg = readHarnessCfg(ctx.directory);
             const h = readHarness(ctx.sessionID) ?? { name: "batch", total: 0, current: 0, tasks: [], usage: {}, active: true };
             h.tasks = h.tasks.filter((x) => x.id !== args.id);
-            h.tasks.push({ id: args.id, title: args.title, status: args.status, model: args.model ?? "", revisions: args.revisions ?? 0, score: args.score ?? null, startedAt: (/* @__PURE__ */ new Date()).toISOString() });
+            const model = args.model || cfg.generator || "";
+            if (!model) return `ERROR: task ${args.id} has no model and no generator configured. Set "generator" in harness.config.json.`;
+            h.tasks.push({ id: args.id, title: args.title, status: args.status, model, revisions: args.revisions ?? 0, score: args.score ?? null, startedAt: (/* @__PURE__ */ new Date()).toISOString() });
             if (args.id > h.current) h.current = args.id;
             writeHarness(ctx.sessionID, h);
             return `task ${args.id} -> ${args.status}${args.score ? ` (${args.score})` : ""}`;
@@ -379,6 +479,136 @@ Then: harness_done(). Follow the [usage-coach NEXT] directive each tool returns.
             return "Harness complete.";
           }
         }),
+        record_failure: tool({
+          description: "Stage 1 (RECORD) of the learning loop. Append a failure record to failures.ndjson for later root-cause analysis.",
+          args: {
+            task: tool.schema.string(),
+            prompt: tool.schema.string(),
+            gradeResult: tool.schema.string(),
+            model: tool.schema.string().optional(),
+            revisions: tool.schema.number().optional()
+          },
+          async execute(args, _ctx) {
+            const rec = { ts: (/* @__PURE__ */ new Date()).toISOString(), task: args.task, prompt: args.prompt, gradeResult: args.gradeResult, model: args.model, revisions: args.revisions };
+            try {
+              mkdirSync2(STATE_DIR, { recursive: true });
+              appendFileSync2(failuresFile(), JSON.stringify(rec) + "\n");
+            } catch (e) {
+              log(`record_failure err: ${String(e)}`);
+            }
+            return `Failure recorded. [usage-coach NEXT] call investigate({failure: ${JSON.stringify(rec)}}) to find the root cause.`;
+          }
+        }),
+        investigate: tool({
+          description: "Stage 2 (INVESTIGATE) of the learning loop. Run the generator to analyze the ROOT CAUSE of a failure (not just the symptom).",
+          args: {
+            task: tool.schema.string(),
+            prompt: tool.schema.string(),
+            gradeResult: tool.schema.string(),
+            model: tool.schema.string().optional()
+          },
+          async execute(args, ctx) {
+            const cfg = readHarnessCfg(ctx.directory);
+            if (!cfg.generator) return 'ERROR: no generator model configured. Set "generator" in harness.config.json (see harness.config.example.json).';
+            let domainPrefix = "";
+            let keywords = [];
+            let domainEmpty = true;
+            try {
+              keywords = extractKeywords(`${args.task} ${args.gradeResult}`);
+              if (keywords.length) {
+                const { nodes, edges } = queryDomain(keywords);
+                if (nodes && nodes.length || edges && edges.length) {
+                  domainEmpty = false;
+                  domainPrefix = `Known facts from domain DB: ${JSON.stringify({ nodes, edges })}. Use these if relevant.
+
+---
+
+`;
+                }
+              }
+            } catch (e) {
+              log(`investigate domain query err: ${String(e)}`);
+            }
+            const rcaPrompt = `A task failed. Analyze the ROOT CAUSE (not just the symptom).
+Task: ${args.task}
+What was expected (from grade): ${args.gradeResult}
+Read relevant files in the directory if needed.
+Output a structured root cause:
+category: (one of: constraint-violation, missing-context, tool-misuse, model-limitation, other)
+explanation: <why it failed>
+evidence: <file/line or specific quote>`;
+            const out = await runModel(input.client, cfg.generator, domainPrefix + rcaPrompt, ctx.directory);
+            if (domainEmpty && keywords.length) {
+              try {
+                saveInvestigationResult(keywords, out, "investigate");
+              } catch (e) {
+                log(`investigate save err: ${String(e)}`);
+              }
+            }
+            return out + "\n[usage-coach NEXT] call verify_diagnosis with this diagnosis.";
+          }
+        }),
+        verify_diagnosis: tool({
+          description: "Stage 3 (VERIFY) of the learning loop. Run the grader to check whether a diagnosis is CORRECT and ACTIONABLE (leads to a useful rule). Returns PASS/FAIL + a [usage-coach NEXT] directive.",
+          args: {
+            diagnosis: tool.schema.string(),
+            task: tool.schema.string(),
+            gradeResult: tool.schema.string()
+          },
+          async execute(args, ctx) {
+            const cfg = readHarnessCfg(ctx.directory);
+            const model = cfg.grader ?? cfg.generator;
+            if (!model) return "FAIL\n(ERROR: no grader/generator model configured.)\n[usage-coach NEXT] configure grader in harness.config.json, then retry verify_diagnosis.";
+            const verifyPrompt = `Verify this root-cause analysis for a failure.
+Task: ${args.task}
+Grade feedback: ${args.gradeResult}
+Diagnosis: ${args.diagnosis}
+Is the diagnosis CORRECT and ACTIONABLE (leads to a useful rule)?
+Output PASS (the diagnosis is right) or FAIL (re-investigate needed), then reason.`;
+            const out = await runModel(input.client, model, verifyPrompt, ctx.directory);
+            let verdict = "FAIL";
+            if (!out.startsWith("ERROR:")) {
+              const f = (out.split("\n").find((l) => l.trim()) ?? "").trim();
+              if (/^pass\b/i.test(f)) verdict = "PASS";
+              else verdict = "FAIL";
+            }
+            const next = verdict === "PASS" ? `
+[usage-coach NEXT] call generalize with this verified diagnosis.` : `
+[usage-coach NEXT] FAIL \u2014 re-investigate the root cause.`;
+            return out + "\n" + next;
+          }
+        }),
+        generalize: tool({
+          description: "Stage 4 (GENERALIZE) of the learning loop. Run the generator to turn a verified root cause into a reusable rule and append it to rules.md, so the next generate call includes it. Returns the rule text and a [usage-coach NEXT] directive.",
+          args: {
+            diagnosis: tool.schema.string(),
+            task: tool.schema.string()
+          },
+          async execute(args, ctx) {
+            const cfg = readHarnessCfg(ctx.directory);
+            if (!cfg.generator) return 'ERROR: no generator model configured. Set "generator" in harness.config.json (see harness.config.example.json).';
+            const genPrompt = `Turn this verified root cause into a GENERAL, REUSABLE rule for future tasks of this kind.
+Diagnosis: ${args.diagnosis}
+Failed task: ${args.task}
+Output a single rule in the form: 'For <task-type> tasks, always <check/do X> because <reason>.'
+Keep it concrete and actionable.`;
+            const out = await runModel(input.client, cfg.generator, genPrompt, ctx.directory);
+            const rule = out;
+            try {
+              const date = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
+              mkdirSync2(STATE_DIR, { recursive: true });
+              appendFileSync2(rulesFile(), `## Rule (${date})
+${rule}
+Origin: ${args.task}
+
+`);
+            } catch (e) {
+              log(`generalize err: ${String(e)}`);
+            }
+            return `${rule}
+[usage-coach NEXT] rule saved to rules.md. The next generate call will include it. Call task_update for the original failed task -> failed, then proceed.`;
+          }
+        }),
         // Per-role model execution (config-driven, quota-aware, same server, no deadlock).
         // P1: quota decision drives model selection + concurrency.
         generate: tool({
@@ -395,13 +625,38 @@ Then: harness_done(). Follow the [usage-coach NEXT] directive each tool returns.
             const throttle = decision === "THROTTLE" && cfg.lighterModel;
             const model = throttle ? cfg.lighterModel : cfg.generator;
             const rules = readRules();
-            const prefix = rules ? `Lessons learned from previous failures (apply where relevant):
+            let prefix = rules ? `Lessons learned from previous failures (apply where relevant):
 ${rules}
 
 ---
 
 ` : "";
+            let keywords = [];
+            let domainEmpty = true;
+            try {
+              keywords = extractKeywords(args.prompt);
+              if (keywords.length) {
+                const { nodes, edges } = queryDomain(keywords);
+                if (nodes && nodes.length || edges && edges.length) {
+                  domainEmpty = false;
+                  prefix = `Known facts from domain DB: ${JSON.stringify({ nodes, edges })}. Use these if relevant.
+
+---
+
+` + prefix;
+                }
+              }
+            } catch (e) {
+              log(`generate domain query err: ${String(e)}`);
+            }
             const out = await runModel(input.client, model, prefix + args.prompt, ctx.directory);
+            if (domainEmpty && keywords.length) {
+              try {
+                saveInvestigationResult(keywords, out, "generate");
+              } catch (e) {
+                log(`generate save err: ${String(e)}`);
+              }
+            }
             return out + (throttle ? `
 [usage-coach] quota THROTTLE \u2014 used lighter model ${cfg.lighterModel}` : "") + `
 [usage-coach NEXT] call task_update(i, title, "grading"), then grade to evaluate this work.`;
@@ -453,7 +708,12 @@ ${rules}
             }
             const next = verdict === "PASS" ? `
 [usage-coach NEXT] PASS -> call task_update(i, title, "completed", "PASS"), then proceed to next task (or harness_done if last).` : `
-[usage-coach NEXT] FAIL -> if revisions < 2: task_update(i, title, "revising", revisions+1) + generate({prompt: "Apply feedback:\\n{grade result}\\nTask: {title}"}); else: task_update(i, title, "failed", "FAIL") -> next task.`;
+[usage-coach NEXT] FAIL -> if revisions < 2: task_update(i, title, "revising", revisions+1) + generate({prompt: "Apply feedback:\\n{grade result}\\nTask: {title}"}); else: run the learning loop before failing \u2014
+  1. record_failure({task, prompt, gradeResult, model, revisions})
+  2. investigate({task, prompt, gradeResult}) -> diagnosis
+  3. verify_diagnosis({diagnosis, task, gradeResult}) -> if PASS: generalize({diagnosis, task}) (saves rule to rules.md)
+  4. task_update(i, title, "failed", "FAIL") -> next task.
+The next generate call will automatically include the new rule.`;
             return out + "\n" + next;
           }
         })

package/dist/tui.js

@@ -5,7 +5,7 @@ import { effect as _$effect } from "@opentui/solid";
 import { createTextNode as _$createTextNode } from "@opentui/solid";
 import { insertNode as _$insertNode } from "@opentui/solid";
 import { createElement as _$createElement } from "@opentui/solid";
-import { readFileSync, existsSync, writeFileSync, mkdirSync, readdirSync, statSync } from "fs";
+import { readFileSync, existsSync, writeFileSync, mkdirSync, readdirSync, statSync, appendFileSync } from "fs";
 import { createHash } from "crypto";
 import { homedir } from "os";
 import { join, resolve } from "path";
@@ -86,26 +86,50 @@ var TLABEL = {
   halted_quota: "quota-halt"
 };
 function barFill(p) {
-  return "\u2588".repeat(Math.max(0, Math.min(10, Math.round(p / 10))));
+  const n = p <= 0 ? 0 : Math.max(1, Math.min(10, Math.round(p / 10)));
+  return "\u2588".repeat(n);
 }
 function barEmpty(p) {
-  return "\u2591".repeat(10 - Math.max(0, Math.min(10, Math.round(p / 10))));
-}
-function short(s, n) {
-  return s.length <= n ? s : s.slice(0, n - 1) + "\u2026";
+  const n = p <= 0 ? 0 : Math.max(1, Math.min(10, Math.round(p / 10)));
+  return "\u2591".repeat(10 - n);
 }
 function initializeTui(api, disposeRoot) {
   STATE_DIR = process.env.UC_STATE_DIR ?? projectStateDir(api.state.path.directory);
   STATE_FILE = join(STATE_DIR, "state.json");
   HARNESS_FILE = join(STATE_DIR, "harness.json");
   MARKER = join(STATE_DIR, "tui-loaded.txt");
+  const TUI_LOG = join(STATE_DIR, "tui-debug.log");
+  const tlog = (msg) => {
+    try {
+      appendFileSync(MARKER, `${(/* @__PURE__ */ new Date()).toISOString()} ${msg}
+`);
+      appendFileSync(TUI_LOG, `${(/* @__PURE__ */ new Date()).toISOString()} ${msg}
+`);
+    } catch (e) {
+      try {
+        appendFileSync(MARKER, `TLOG ERR: ${String(e)}
+`);
+      } catch {
+      }
+    }
+  };
   try {
     mkdirSync(STATE_DIR, {
       recursive: true
     });
-    writeFileSync(MARKER, `loaded ${(/* @__PURE__ */ new Date()).toISOString()} @ ${api.state.path.directory}`);
+    writeFileSync(MARKER, `loaded-v2 ${(/* @__PURE__ */ new Date()).toISOString()} @ ${api.state.path.directory}`);
   } catch {
   }
+  tlog(`init start | dir=${api.state.path.directory} | STATE_DIR=${STATE_DIR}`);
+  try {
+    tlog(`api keys=${Object.keys(api).join(",")}`);
+    tlog(`api.state=${JSON.stringify(api.state).slice(0, 400)}`);
+    tlog(`api.state.path=${JSON.stringify(api.state?.path).slice(0, 300)}`);
+    const r = api.route;
+    tlog(`api.route type=${typeof r} keys=${r && typeof r === "object" ? Object.keys(r).join(",") : "?"} val=${JSON.stringify(r).slice(0, 400)}`);
+  } catch (e) {
+    tlog(`api probe err: ${String(e)}`);
+  }
   const [getState, setState] = createSignal(readState());
   const [getHarness, setHarness] = createSignal(readHarness());
   const timer = setInterval(() => {
@@ -168,7 +192,8 @@ function initializeTui(api, disposeRoot) {
     }
     let h = null;
     try {
-      const sid = ctx.session_id ?? "";
+      const routeSid = api.route?.current?.params?.sessionID ?? "";
+      const sid = routeSid || (ctx.session_id ?? "");
       if (sid) {
         const hf = join(STATE_DIR, sid, "harness.json");
         if (existsSync(hf)) h = JSON.parse(readFileSync(hf, "utf8"));
@@ -213,7 +238,7 @@ function initializeTui(api, disposeRoot) {
             _$insert(_el$12, () => p.fiveHour, _el$14);
             _$insert(_el$12, () => p.fiveHourReset, null);
             _$effect((_p$) => {
-              var _v$ = st("text"), _v$2 = st("textMuted");
+              var _v$ = st("text"), _v$2 = st("text");
               _v$ !== _p$.e && (_p$.e = _$setProp(_el$10, "style", _v$, _p$.e));
               _v$2 !== _p$.t && (_p$.t = _$setProp(_el$11, "style", _v$2, _p$.t));
               return _p$;
@@ -230,7 +255,7 @@ function initializeTui(api, disposeRoot) {
             _$insertNode(_el$15, _el$19);
             _$insertNode(_el$15, _el$20);
             _$setProp(_el$15, "flexDirection", "row");
-            _$insertNode(_el$16, _$createTextNode(` wk `));
+            _$insertNode(_el$16, _$createTextNode(` 1w `));
             _$insert(_el$18, () => barFill(p.weekly));
             _$insert(_el$19, () => barEmpty(p.weekly));
             _$insertNode(_el$20, _el$21);
@@ -238,7 +263,7 @@ function initializeTui(api, disposeRoot) {
             _$insert(_el$20, () => p.weekly, _el$22);
             _$insert(_el$20, () => p.weeklyReset, null);
             _$effect((_p$) => {
-              var _v$3 = st("text"), _v$4 = st("textMuted");
+              var _v$3 = st("text"), _v$4 = st("text");
               _v$3 !== _p$.e && (_p$.e = _$setProp(_el$18, "style", _v$3, _p$.e));
               _v$4 !== _p$.t && (_p$.t = _$setProp(_el$19, "style", _v$4, _p$.t));
               return _p$;
@@ -258,7 +283,7 @@ function initializeTui(api, disposeRoot) {
         }
       } else {
         nodes.push((() => {
-          var _el$27 = _$createElement("box"), _el$28 = _$createElement("text"), _el$30 = _$createElement("text"), _el$31 = _$createElement("text"), _el$32 = _$createElement("text"), _el$33 = _$createTextNode(` `), _el$34 = _$createTextNode(`%`);
+          var _el$27 = _$createElement("box"), _el$28 = _$createElement("text"), _el$30 = _$createElement("text"), _el$31 = _$createElement("text"), _el$32 = _$createElement("text");
           _$insertNode(_el$27, _el$28);
           _$insertNode(_el$27, _el$30);
           _$insertNode(_el$27, _el$31);
@@ -267,11 +292,9 @@ function initializeTui(api, disposeRoot) {
           _$insertNode(_el$28, _$createTextNode(` 5h `));
           _$insert(_el$30, () => barFill(s.fiveHour));
           _$insert(_el$31, () => barEmpty(s.fiveHour));
-          _$insertNode(_el$32, _el$33);
-          _$insertNode(_el$32, _el$34);
-          _$insert(_el$32, () => s.fiveHour, _el$34);
+          _$insertNode(_el$32, _$createTextNode(` 0%`));
           _$effect((_p$) => {
-            var _v$5 = st("text"), _v$6 = st("textMuted");
+            var _v$5 = st("text"), _v$6 = st("text");
             _v$5 !== _p$.e && (_p$.e = _$setProp(_el$30, "style", _v$5, _p$.e));
             _v$6 !== _p$.t && (_p$.t = _$setProp(_el$31, "style", _v$6, _p$.t));
             return _p$;
@@ -282,77 +305,51 @@ function initializeTui(api, disposeRoot) {
           return _el$27;
         })());
         nodes.push((() => {
-          var _el$35 = _$createElement("box"), _el$36 = _$createElement("text"), _el$38 = _$createElement("text"), _el$39 = _$createElement("text"), _el$40 = _$createElement("text"), _el$41 = _$createTextNode(` `), _el$42 = _$createTextNode(`%`);
-          _$insertNode(_el$35, _el$36);
-          _$insertNode(_el$35, _el$38);
-          _$insertNode(_el$35, _el$39);
-          _$insertNode(_el$35, _el$40);
-          _$setProp(_el$35, "flexDirection", "row");
-          _$insertNode(_el$36, _$createTextNode(` wk `));
-          _$insert(_el$38, () => barFill(s.weekly));
-          _$insert(_el$39, () => barEmpty(s.weekly));
-          _$insertNode(_el$40, _el$41);
-          _$insertNode(_el$40, _el$42);
-          _$insert(_el$40, () => s.weekly, _el$42);
-          _$effect((_p$) => {
-            var _v$7 = st("text"), _v$8 = st("textMuted");
-            _v$7 !== _p$.e && (_p$.e = _$setProp(_el$38, "style", _v$7, _p$.e));
-            _v$8 !== _p$.t && (_p$.t = _$setProp(_el$39, "style", _v$8, _p$.t));
-            return _p$;
-          }, {
-            e: void 0,
-            t: void 0
-          });
-          return _el$35;
-        })());
-        nodes.push((() => {
-          var _el$43 = _$createElement("box"), _el$44 = _$createElement("text"), _el$46 = _$createElement("text"), _el$47 = _$createElement("text"), _el$48 = _$createElement("text"), _el$49 = _$createTextNode(` `), _el$50 = _$createTextNode(`%`);
-          _$insertNode(_el$43, _el$44);
-          _$insertNode(_el$43, _el$46);
-          _$insertNode(_el$43, _el$47);
-          _$insertNode(_el$43, _el$48);
-          _$setProp(_el$43, "flexDirection", "row");
-          _$insertNode(_el$44, _$createTextNode(` mo `));
-          _$insert(_el$46, () => barFill(s.monthly));
-          _$insert(_el$47, () => barEmpty(s.monthly));
-          _$insertNode(_el$48, _el$49);
-          _$insertNode(_el$48, _el$50);
-          _$insert(_el$48, () => s.monthly, _el$50);
+          var _el$34 = _$createElement("box"), _el$35 = _$createElement("text"), _el$37 = _$createElement("text"), _el$38 = _$createElement("text"), _el$39 = _$createElement("text");
+          _$insertNode(_el$34, _el$35);
+          _$insertNode(_el$34, _el$37);
+          _$insertNode(_el$34, _el$38);
+          _$insertNode(_el$34, _el$39);
+          _$setProp(_el$34, "flexDirection", "row");
+          _$insertNode(_el$35, _$createTextNode(` 1w `));
+          _$insert(_el$37, () => barFill(s.weekly));
+          _$insert(_el$38, () => barEmpty(s.weekly));
+          _$insertNode(_el$39, _$createTextNode(` 0%`));
           _$effect((_p$) => {
-            var _v$9 = st("text"), _v$0 = st("textMuted");
-            _v$9 !== _p$.e && (_p$.e = _$setProp(_el$46, "style", _v$9, _p$.e));
-            _v$0 !== _p$.t && (_p$.t = _$setProp(_el$47, "style", _v$0, _p$.t));
+            var _v$7 = st("text"), _v$8 = st("text");
+            _v$7 !== _p$.e && (_p$.e = _$setProp(_el$37, "style", _v$7, _p$.e));
+            _v$8 !== _p$.t && (_p$.t = _$setProp(_el$38, "style", _v$8, _p$.t));
             return _p$;
           }, {
             e: void 0,
             t: void 0
           });
-          return _el$43;
+          return _el$34;
         })());
       }
     } else {
       nodes.push((() => {
-        var _el$51 = _$createElement("text");
-        _$insertNode(_el$51, _$createTextNode(`usage-coach: ...`));
-        return _el$51;
+        var _el$41 = _$createElement("text");
+        _$insertNode(_el$41, _$createTextNode(`usage-coach: ...`));
+        return _el$41;
       })());
     }
     if (h && h.active !== false && h.tasks.length > 0) {
       nodes.push((() => {
-        var _el$53 = _$createElement("text");
-        _$insertNode(_el$53, _$createTextNode(` `));
-        return _el$53;
+        var _el$43 = _$createElement("text");
+        _$insertNode(_el$43, _$createTextNode(` `));
+        return _el$43;
       })());
       nodes.push((() => {
-        var _el$55 = _$createElement("text"), _el$56 = _$createTextNode(`harness: `), _el$57 = _$createTextNode(` `), _el$58 = _$createTextNode(`/`);
-        _$insertNode(_el$55, _el$56);
-        _$insertNode(_el$55, _el$57);
-        _$insertNode(_el$55, _el$58);
-        _$insert(_el$55, () => h.name, _el$57);
-        _$insert(_el$55, () => h.current, _el$58);
-        _$insert(_el$55, () => h.total, null);
-        _$effect((_$p) => _$setProp(_el$55, "style", st("textMuted"), _$p));
-        return _el$55;
+        var _el$45 = _$createElement("text"), _el$46 = _$createTextNode(`harness: `), _el$47 = _$createTextNode(` `), _el$48 = _$createTextNode(`/`);
+        _$insertNode(_el$45, _el$46);
+        _$insertNode(_el$45, _el$47);
+        _$insertNode(_el$45, _el$48);
+        _$insert(_el$45, () => h.name, _el$47);
+        _$insert(_el$45, () => h.current, _el$48);
+        _$insert(_el$45, () => h.total, null);
+        _$effect((_$p) => _$setProp(_el$45, "style", st("textMuted"), _$p));
+        return _el$45;
       })());
       for (const t of h.tasks) {
         const sKey = statusKey[t.status] ?? "text";
@@ -362,71 +359,76 @@ function initializeTui(api, disposeRoot) {
         const elapsed = t.startedAt ? Math.max(0, Math.round((Date.now() - new Date(t.startedAt).getTime()) / 1e3)) : 0;
         const elapsedStr = t.status === "completed" || t.status === "failed" ? "" : elapsed > 0 ? ` ${elapsed}s` : "";
         nodes.push((() => {
-          var _el$59 = _$createElement("text"), _el$60 = _$createTextNode(` \u25CF `), _el$61 = _$createTextNode(` `), _el$62 = _$createTextNode(` `);
-          _$insertNode(_el$59, _el$60);
-          _$insertNode(_el$59, _el$61);
-          _$insertNode(_el$59, _el$62);
-          _$insert(_el$59, () => t.id, _el$61);
-          _$insert(_el$59, mdl, _el$61);
-          _$insert(_el$59, lbl, _el$62);
-          _$insert(_el$59, rev, _el$62);
-          _$insert(_el$59, elapsedStr, _el$62);
-          _$insert(_el$59, () => short(t.title, 12), null);
-          _$effect((_$p) => _$setProp(_el$59, "style", st(sKey), _$p));
-          return _el$59;
+          var _el$49 = _$createElement("text"), _el$50 = _$createTextNode(` \u25CF `), _el$51 = _$createTextNode(` `), _el$52 = _$createTextNode(` `);
+          _$insertNode(_el$49, _el$50);
+          _$insertNode(_el$49, _el$51);
+          _$insertNode(_el$49, _el$52);
+          _$insert(_el$49, () => t.id, _el$51);
+          _$insert(_el$49, mdl, _el$51);
+          _$insert(_el$49, lbl, _el$52);
+          _$insert(_el$49, rev, _el$52);
+          _$insert(_el$49, elapsedStr, _el$52);
+          _$insert(_el$49, () => t.title, null);
+          _$effect((_$p) => _$setProp(_el$49, "style", st(sKey), _$p));
+          return _el$49;
         })());
         const pv = t.model ? (t.model.split("/")[0] ?? "").split("-")[0] : "";
-        const provCoach = s?.providers?.find((p) => p.id === pv || pv && p.id.startsWith(pv) || pv && pv.startsWith(p.id));
-        const pct = provCoach ? provCoach.fiveHour : s?.fiveHour ?? 0;
+        const provCoach = pv ? s?.providers?.find((p) => p.id === pv || pv && p.id.startsWith(pv) || pv && pv.startsWith(p.id)) : s?.providers?.[0];
+        const rawPct = provCoach?.fiveHour ?? s?.fiveHour ?? -1;
+        const pct = rawPct < 0 ? 0 : rawPct;
+        const pctLabel = rawPct < 0 ? "n/a" : `${rawPct}%`;
         nodes.push((() => {
-          var _el$63 = _$createElement("box"), _el$64 = _$createElement("text"), _el$66 = _$createElement("text"), _el$67 = _$createElement("text"), _el$68 = _$createElement("text"), _el$69 = _$createTextNode(` `), _el$70 = _$createTextNode(`%`);
-          _$insertNode(_el$63, _el$64);
-          _$insertNode(_el$63, _el$66);
-          _$insertNode(_el$63, _el$67);
-          _$insertNode(_el$63, _el$68);
-          _$setProp(_el$63, "flexDirection", "row");
-          _$insertNode(_el$64, _$createTextNode(` 5h `));
-          _$insert(_el$66, () => barFill(pct));
-          _$insert(_el$67, () => barEmpty(pct));
-          _$insertNode(_el$68, _el$69);
-          _$insertNode(_el$68, _el$70);
-          _$insert(_el$68, pct, _el$70);
+          var _el$53 = _$createElement("box"), _el$54 = _$createElement("text"), _el$56 = _$createElement("text"), _el$57 = _$createElement("text"), _el$58 = _$createElement("text"), _el$59 = _$createTextNode(` `);
+          _$insertNode(_el$53, _el$54);
+          _$insertNode(_el$53, _el$56);
+          _$insertNode(_el$53, _el$57);
+          _$insertNode(_el$53, _el$58);
+          _$setProp(_el$53, "flexDirection", "row");
+          _$insertNode(_el$54, _$createTextNode(` 5h `));
+          _$insert(_el$56, () => barFill(pct));
+          _$insert(_el$57, () => barEmpty(pct));
+          _$insertNode(_el$58, _el$59);
+          _$insert(_el$58, pctLabel, null);
           _$effect((_p$) => {
-            var _v$1 = st("text"), _v$10 = st("textMuted");
-            _v$1 !== _p$.e && (_p$.e = _$setProp(_el$66, "style", _v$1, _p$.e));
-            _v$10 !== _p$.t && (_p$.t = _$setProp(_el$67, "style", _v$10, _p$.t));
+            var _v$9 = st("text"), _v$0 = st("text");
+            _v$9 !== _p$.e && (_p$.e = _$setProp(_el$56, "style", _v$9, _p$.e));
+            _v$0 !== _p$.t && (_p$.t = _$setProp(_el$57, "style", _v$0, _p$.t));
             return _p$;
           }, {
             e: void 0,
             t: void 0
           });
-          return _el$63;
+          return _el$53;
         })());
       }
     }
     return (() => {
-      var _el$71 = _$createElement("box");
-      _$setProp(_el$71, "flexDirection", "column");
-      _$insert(_el$71, nodes);
-      return _el$71;
+      var _el$60 = _$createElement("box");
+      _$setProp(_el$60, "flexDirection", "column");
+      _$insert(_el$60, nodes);
+      return _el$60;
     })();
   };
+  tlog("registering slots");
   api.slots.register({
     order: 80,
     slots: {
       sidebar_footer(ctx) {
+        tlog("sidebar_footer slot called");
         try {
           return panel(ctx);
-        } catch {
+        } catch (e) {
+          tlog(`sidebar_footer err: ${String(e)}`);
           return (() => {
-            var _el$72 = _$createElement("text");
-            _$insertNode(_el$72, _$createTextNode(`usage-coach`));
-            return _el$72;
+            var _el$61 = _$createElement("text");
+            _$insertNode(_el$61, _$createTextNode(`usage-coach`));
+            return _el$61;
           })();
         }
       }
     }
   });
+  tlog("slots registered, init complete");
 }
 var tui = async (api) => {
   createRoot((disposeRoot) => initializeTui(api, disposeRoot));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-usage-coach",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "description": "opencode closed-loop usage coach \u2014 quota SENSE -> coaching DECIDE -> loop ACT + TUI integration",
   "type": "module",
   "main": "./dist/index.js",