@azure-id/orc 1.9.1 → 2.0.2

package/bin/verify-package.js

@@ -46,6 +46,7 @@ const required = [
   "bin/webui/css/panels/mockrun.css",
   "bin/webui/css/panels/maintenance.css",
   "bin/webui/css/panels/pact.css",
+  "bin/webui/css/panels/behaviour.css",
   "bin/webui/css/panels/boundary.css",
   "bin/webui/css/panels/wait.css",
   "bin/webui/css/panels/hookui.css",
@@ -78,6 +79,7 @@ const required = [
   "bin/webui/js/panels/experiment.js",
   "bin/webui/js/panels/maintenance.js",
   "bin/webui/js/panels/pact.js",
+  "bin/webui/js/panels/behaviour.js",
   "bin/webui/js/panels/boundary.js",
   "bin/webui/js/panels/wait.js",
   "bin/webui/js/panels/hookui.js",
@@ -101,6 +103,7 @@ const required = [
   "bin/webui/fixtures/runs.js",
   "bin/webui/fixtures/stats.js",
   "bin/webui/fixtures/pact.js",
+  "bin/webui/fixtures/behaviour.js",
   "bin/webui/fixtures/boundary.js",
   "bin/webui/fixtures/wait.js",
   "bin/webui/fixtures/hookui.js",
@@ -136,6 +139,7 @@ const required = [
   "bin/webui/i18n/en/mockrun.json",
   "bin/webui/i18n/en/maintenance.json",
   "bin/webui/i18n/en/pact.json",
+  "bin/webui/i18n/en/behaviour.json",
   "bin/webui/i18n/en/boundary.json",
   "bin/webui/i18n/en/wait.json",
   "bin/webui/i18n/en/hookui.json",
@@ -162,6 +166,7 @@ const required = [
   "bin/webui/i18n/id/mockrun.json",
   "bin/webui/i18n/id/maintenance.json",
   "bin/webui/i18n/id/pact.json",
+  "bin/webui/i18n/id/behaviour.json",
   "bin/webui/i18n/id/boundary.json",
   "bin/webui/i18n/id/wait.json",
   "bin/webui/i18n/id/hookui.json",
@@ -202,6 +207,22 @@ const required = [
   // the only answer that says WHY a THIN map is thin. Without it the status
   // line still says THIN and names a command that is not there.
   "bin/graph-audit.js",
+  // v2.0.0 W2 — the habit engine. `orc habit` requires it, and so do
+  // `orc lane config` and `orc config list` when `habits` is not off — a publish
+  // that drops it breaks the resolver for every lane that turned habits on.
+  "bin/habit.js",
+  // v2.0.0 W4 / W6a — the gotchas engine and its importers. `orc gotcha` and
+  // `orc lane config` require the first; `import`, `sync`, `orc pr threads`
+  // and `orc ci failed` require the second. A publish that drops the importer
+  // makes every review step's sync fail — silently, because a sync never blocks.
+  "bin/gotcha.js",
+  "bin/gotcha-import.js",
+  // v2.0.0 T21 — the trace pen. `orc trace write` requires it; a publish that
+  // drops it sends every lane to the Haiku fallback on every phase close.
+  "bin/trace-write.js",
+  // v2.0.0 W6b — `orc run snapshot` + `orc undo`. A publish that drops it
+  // leaves every end-of-run card printing an undo command that crashes.
+  "bin/run-undo.js",
 
   // The two DATED data files (v0.50.0). Both ship inside the package and both
   // are load-bearing on absence rather than on content: without the catalog
@@ -368,6 +389,18 @@ const required = [
   // trimmed lane reading a full-lane procedure.
   "templates/skills/_shared/phases/README.md",
   "templates/skills/_shared/phases/trace.md",
+  // v2.0.0 T1 — the CLOSED verb table, split out of trace.md and read on
+  // demand. A publish missing it leaves every lane's fallback for a verb not
+  // in `trace_grammar` pointing at nothing.
+  "templates/skills/_shared/phases/trace-verbs.md",
+  // v2.0.0 T3 — the common text of the coding-lane spine blocks. A publish
+  // missing it leaves every exit ≠ 0 fallback pointing at nothing.
+  "templates/skills/_shared/lane-contract.md",
+  // v2.0.0 W3 — the lane side of habits, read on demand. A publish missing it
+  // leaves every `(H <qid>)` mark pointing at nothing once habits is on.
+  "templates/skills/_shared/habits.md",
+  // v2.0.0 W5a — the one review slice, read on demand at a review dispatch.
+  "templates/skills/_shared/review-slice.md",
   "templates/skills/_shared/phases/preflight.md",
   "templates/skills/_shared/phases/stop-resume.md",
   // v1.0.0 W12 — the seven files that were already SHARED while living in one
@@ -431,8 +464,10 @@ const required = [
   // v1.8.0 — the code graph's notes writer (Layer 2).
   "templates/agents/orc-graph-noter-sonnet-4-6-med.md",
   // Core non-generated agents — named explicitly so a dropped file is REPORTED
-  // by name, not merely absorbed by the count floor. (The 8 executor agents are
-  // checked separately by `build-agents.js --check`.)
+  // by name, not merely absorbed by the count floor. (The 10 executor agents are
+  // checked separately by `build-agents.js --check`. v2.0.0 T15: the 16 model
+  // twins below are GENERATED too — from agents-src/twins/ — and `--check`
+  // guards their content; this list still guards that each one ships.)
   "templates/agents/orc-system-analyst-opus-5-high.md",
   "templates/agents/orc-planner-opus-5-med.md",
   "templates/agents/orc-reviewer-opus-5-med.md",
@@ -469,7 +504,7 @@ const required = [
   // v0.47.0 — the three /orc-challenge agents. All THREE must ship: a cycle that
   // cannot dispatch the reader silently loses D4, and one that cannot dispatch
   // the advisor turns a FAIL into a list with no order. All three are already
-  // Opus 5, so `opus5_only` needs no twin for any of them.
+  // Opus 5.5, so `opus5_only` needs no twin for any of them.
   "templates/agents/orc-challenge-judge-opus-5-high.md",
   "templates/agents/orc-challenge-advisor-opus-5-med.md",
   "templates/agents/orc-challenge-reader-opus-5-low.md",
@@ -479,7 +514,7 @@ const required = [
   // first-principles thinker disputes the goal itself, and the expansionist is
   // the only lens not looking for a defect. All five must ship: a roster lens
   // that cannot be dispatched is a NOT-RUN row a user paid to select, and rule
-  // 15 makes that loud rather than silent. All five are claude-opus-5, so
+  // 15 makes that loud rather than silent. All five are claude-opus-5-5, so
   // `opus5_only` adds NO pair and the floor moves by exactly five.
   "templates/agents/orc-challenge-contrarian-opus-5-high.md",
   "templates/agents/orc-challenge-outsider-opus-5-low.md",
@@ -489,7 +524,7 @@ const required = [
   // v0.48.0 — the two /orc-doc agents. BOTH must ship: a write wave that cannot
   // dispatch the writer has nothing to assemble, and a check wave that cannot
   // dispatch the checker silently turns the only judgment step into the free
-  // lint. Both are already Opus 5, so `opus5_only` adds no twin for either.
+  // lint. Both are already Opus 5.5, so `opus5_only` adds no twin for either.
   "templates/agents/orc-doc-writer-opus-5-med.md",
   "templates/agents/orc-doc-checker-opus-5-low.md",
   // v1.5.0 - /orc-test's two agents, and they are two INSTRUMENTS rather than
@@ -498,7 +533,7 @@ const required = [
   // interpreter is `low` because a harder-thinking interpreter reasons its way
   // to why a leaked stack trace is probably fine in staging, which is exactly
   // the gap it exists to find. Nothing may ever upgrade the interpreter. Both
-  // are claude-opus-5, so `opus5_only` adds NO pair and the floor moves by two.
+  // are claude-opus-5-5, so `opus5_only` adds NO pair and the floor moves by two.
   "templates/agents/orc-test-designer-opus-5-high.md",
   "templates/agents/orc-test-interpreter-opus-5-low.md",
   // v1.2.1 — the status line explained for the person reading it, in
@@ -522,6 +557,9 @@ const required = [
   // the ORCHESTRATOR. Ships wired but OFF; `read_gate` arms it.
   "templates/hooks/orc-read-gate.js",
   "templates/hooks/orc-trace.js",
+  // v2.0.0 W6c — the session hook: the bell at a run's turn end (Q8, `notify`,
+  // off by default) and the run pointer after a compaction (Q9).
+  "templates/hooks/orc-session-hook.js",
 ];
 
 const missing = [];
@@ -569,7 +607,7 @@ const agentCount = walkCount(path.join(ROOT, "templates/agents"), ".md");
 // tier of the same one. The reader is `low` on purpose (a harder-thinking reader
 // papers over exactly the gaps D4 measures), the judge is `high` because D2 is
 // the only dimension no computer can reach, and the advisor is `medium` because
-// grouping findings is pattern work. All three are claude-opus-5, so `opus5_only`
+// grouping findings is pattern work. All three are claude-opus-5-5, so `opus5_only`
 // adds NO pair and the floor moves by exactly three.
 // v0.48.0: +1 skill (orc-doc) and +2 agents. The split follows v0.47.0's rule —
 // an agent exists when it is a DIFFERENT INSTRUMENT, not a tier of the same one.
@@ -577,13 +615,13 @@ const agentCount = walkCount(path.join(ROOT, "templates/agents"), ".md");
 // LINE RANGE and has `Read` only. The checker is `low` on purpose (a
 // harder-thinking checker reasons its way past a gap a real reader would trip
 // on — the challenge cold reader's reasoning), so nothing may upgrade it. Both
-// are claude-opus-5, so `opus5_only` adds NO pair and the floor moves by two.
+// are claude-opus-5-5, so `opus5_only` adds NO pair and the floor moves by two.
 // v0.49.1: +0 skills and +5 agents — THE COUNCIL. Same rule as v0.47.0, applied
 // five times: each of these is a DIFFERENT INSTRUMENT, not a tier of one. The
 // outsider is `low` and the contrarian is `high` for the same class of reason
 // the cold reader is `low` — effort here is a MEASUREMENT choice, not a cost
 // one, and a key that let either be tuned would be a key that let the
-// instrument be broken. All five are claude-opus-5, so `opus5_only` adds NO
+// instrument be broken. All five are claude-opus-5-5, so `opus5_only` adds NO
 // pair and the floor moves by exactly five.
 // v1.0.0 W3: +0 skills and -5 agents — the Fable 5 role override is REMOVED.
 // It is the first time this floor has ever gone DOWN, and the reason it may is
@@ -624,13 +662,22 @@ function scanEncoding(dir, hits) {
   if (!fs.existsSync(dir)) return;
   const st = fs.statSync(dir);
   if (st.isFile()) {
-    let text;
+    let buf;
     try {
-      text = fs.readFileSync(dir, "utf8");
+      buf = fs.readFileSync(dir);
     } catch (_) {
       return; // unreadable → not our concern here
     }
     const rel = path.relative(ROOT, dir).replace(/\\/g, "/");
+    // v2.0.0 Q10 — a raw 0x00 byte. Grep and ripgrep then treat the WHOLE file as
+    // binary and stop reporting matches after it, so every agent that greps the
+    // file gets a partial answer. Write the "\u0000" escape instead.
+    const nul = buf.indexOf(0);
+    if (nul >= 0) {
+      hits.push(`${rel} (raw NUL byte at offset ${nul} — write the "\\u0000" escape)`);
+      return;
+    }
+    const text = buf.toString("utf8");
     // Reference U+FFFD via escape, never as a literal, so this scanner does not
     // flag its own source.
     if (text.includes(REPL)) hits.push(`${rel} (U+FFFD replacement char — corrupted bytes)`);

package/bin/webui/api.js

@@ -453,6 +453,22 @@ const READS = {
   "/api/test/ui": () => ["test", "ui", "tools"],
   "/api/patterns": () => ["pattern", "status"],
   "/api/gotchas": () => ["gotcha", "list"],
+  // v2.0.0 W7 — the Behaviour panel. Every one is a CLI read, and every exit
+  // code is DATA: `habit show` 0 · 1 no ASK lines yet · 3 `habits: off`;
+  // `habit log` 0 · 1 empty; `gotcha card` 0 · 1 no match (an ANSWER: no card);
+  // `gotcha quality` 0 · 1 below the 5-review floor. The window is DROPPED
+  // unless it is one of the three the CLI knows — a query string is user input.
+  "/api/habits": (q) => (["30d", "90d", "all"].includes(String(q.window || "")) ? ["habit", "show", "--window", String(q.window)] : ["habit", "show"]),
+  "/api/habits/log": () => ["habit", "log", "--limit", "40"],
+  "/api/habits/states": () => ["habit", "log", "--states"],
+  "/api/habits/points": () => ["habit", "points"],
+  "/api/gotcha/card": (q) => ["gotcha", "card", "--files", String(q.files || ""), "--full"],
+  // The panel window (30d · 90d · all) in the days `gotcha quality --window` takes.
+  "/api/gotcha/quality": (q) => {
+    const days = { "30d": "30", "90d": "90", all: "3650" }[String(q.window || "")];
+    return days ? ["gotcha", "quality", "--window", days] : ["gotcha", "quality"];
+  },
+  "/api/gotcha/candidates": () => ["gotcha", "list", "--candidates"],
   "/api/stats": (q) => (q.since ? ["stats", "--since", String(q.since)] : ["stats"]),
   "/api/diy": () => ["diy", "show"],
   "/api/crosslink": () => ["crosslink", "list"],
@@ -555,6 +571,16 @@ const WRITES = {
   "/api/graph/update": () => ["graph", "update"],
   "/api/wiki/usage/rebuild": () => ["wiki", "usage", "--rebuild"],
   "/api/gotcha/prune": () => ["gotcha", "prune"],
+  // v2.0.0 W7 — the Behaviour panel writes. FREE, local and reversible: each
+  // one changes `.claude/orc/habits-state.json` only (or promotes ONE gotcha
+  // candidate), and each is a person's click on a confirmation that names the
+  // command. `--by panel` is how the history says who made the change. The
+  // learning switch itself is `/api/config/set` — no second route for it.
+  "/api/habit/accept": (b) => ["habit", "accept", String(b.id), "--by", "panel"],
+  "/api/habit/decline": (b) => (b.never ? ["habit", "decline", String(b.id), "--never", "--by", "panel"] : ["habit", "decline", String(b.id), "--by", "panel"]),
+  "/api/habit/forget": (b) => ["habit", "forget", String(b.id), "--by", "panel"],
+  "/api/habit/reset": (b) => ["habit", "reset", String(b.id), "--by", "panel"],
+  "/api/gotcha/accept": (b) => ["gotcha", "accept", String(b.id)],
   "/api/pact/check": (b) => (b.id ? ["pact", "check", String(b.id)] : ["pact", "check"]),
   "/api/pact/sync": () => ["pact", "sync"],
   // The surface id, the key and the value all come from the browser — and all