@azure-id/orc 2.0.2 → 2.1.1

package/bin/webui/api.js

@@ -284,6 +284,9 @@ const READS = {
     // applies these to a copy of the saved layout before it compiles.
     if (q.theme) argv.push("--theme", String(q.theme));
     if (q.glyphs) argv.push("--glyphs", String(q.glyphs));
+    // v2.0.4 — an animated preview: N frames, M ms apart. Render-only.
+    if (q.frames) argv.push("--frames", String(q.frames));
+    if (q.frame_ms) argv.push("--frame-ms", String(q.frame_ms));
     return argv;
   },
   "/api/statusline/explain": (q) => ["statusline", "explain", String(q.at || "1:1"), ...slBoard(q)],
@@ -527,17 +530,41 @@ const WRITES = {
       ["width", "--width"], ["precision", "--precision"],
       ["min_width", "--min-width"], ["min_cols", "--min-cols"],
       ["max_cols", "--max-cols"], ["priority", "--priority"],
+      // v2.0.4 — the design fields. Same rule: absent sends nothing, "" clears.
+      ["label_pos", "--label-pos"], ["value_pos", "--value-pos"],
+      ["caption", "--caption"], ["caption_pos", "--caption-pos"],
+      ["caption_align", "--caption-align"], ["caption_color", "--caption-color"],
+      ["caption_case", "--caption-case"], ["fill_color", "--fill-color"],
+      ["empty_color", "--empty-color"], ["ramp_colors", "--ramp-colors"],
+      ["ramp_stops", "--ramp-stops"], ["fill_char", "--fill-char"],
+      ["empty_char", "--empty-char"], ["brackets", "--brackets"],
+      ["threshold", "--threshold"], ["pad_left", "--pad-left"],
+      ["pad_right", "--pad-right"], ["align", "--align"], ["sign", "--sign"],
+      ["max_len", "--max-len"], ["unknown", "--unknown"],
+      ["color_by_state", "--state-color"], ["glyph_by_state", "--state-glyph"],
     ]) {
-      if (b[k] !== undefined && b[k] !== null && b[k] !== "") argv.push(f, String(b[k]));
+      // An absent key sends nothing; a PRESENT "" or null sends `<flag> ""`,
+      // which the CLI reads as "delete the key" (inherit). Dropping it made a
+      // clear "succeed" while it changed nothing.
+      if (b[k] === undefined) continue;
+      argv.push(f, b[k] === null ? "" : String(b[k]));
     }
     if (b.draw_empty) argv.push("--draw-empty");
+    // `params` is an OBJECT, one `--param k=v` per key. The CLI scans every
+    // `--param`, and `k=` (a null or "" value) deletes that key.
+    if (b.params && typeof b.params === "object")
+      for (const [k, v] of Object.entries(b.params)) argv.push("--param", String(k) + "=" + (v === null || v === undefined ? "" : String(v)));
     return argv.concat(slBoard(b));
   },
   "/api/statusline/move": (b) => ["statusline", "move", String(b.from), String(b.to), ...slBoard(b)],
   "/api/statusline/remove": (b) => ["statusline", "remove", String(b.at), ...slBoard(b)],
   "/api/statusline/line": (b) => {
     const argv = ["statusline", "line", String(b.line)];
-    if (b.separator !== undefined) argv.push("--separator", String(b.separator));
+    // A PRESENT key is forwarded even when it is "" — that is the clear.
+    for (const [k, f] of [["separator", "--separator"], ["prefix", "--prefix"], ["align", "--align"], ["sep_color", "--sep-color"]]) {
+      if (b[k] === undefined) continue;
+      argv.push(f, b[k] === null ? "" : String(b[k]));
+    }
     if (b.theme) argv.push("--theme", String(b.theme));
     if (b.max_width !== undefined) argv.push("--max-width", String(b.max_width));
     return argv.concat(slBoard(b));
@@ -564,6 +591,8 @@ const WRITES = {
   "/api/statusline/clone": (b) => ["statusline", "clone", String(b.at), ...slBoard(b)],
   "/api/statusline/reset": (b) => ["statusline", "reset", ...slBoard(b)],
   "/api/statusline/compile": (b) => ["statusline", "compile", ...slBoard(b)],
+  // v2.0.4 — the redraw timer: a settings write, not a layout op.
+  "/api/statusline/refresh": (b) => ["statusline", "refresh", String(b.seconds), ...slBoard(b)],
   "/api/wait/unblock": (b) => (b.slug ? ["wait", "unblock", String(b.slug)] : ["wait", "unblock"]),
   "/api/wait/cancel": (b) => (b.slug ? ["wait", "cancel", String(b.slug)] : ["wait", "cancel"]),
   "/api/wiki/sync": () => ["wiki", "sync"],
@@ -836,6 +865,20 @@ const MAINTENANCE = {
     label: "Recompile AGENTS.md from the wiki, patterns, PACT.md and boundary cards",
     preview: ["export", "--check"],
   },
+  // v2.1.0 W7 — clear old logs. The preview is the CLI's own print mode, which
+  // names EVERY file it would delete and every file it keeps, with the reason
+  // (a count is not consent). The day value comes from config
+  // (`log_retention_days`): the panel sends only this id. `status` is the
+  // no-scan summary the row shows before any preview (days, last sweep).
+  "clear-logs": {
+    apply: ["clear", "logs", "--apply"],
+    restarts_ui: false,
+    label: "Delete finished traces and run folders older than log_retention_days (never user data, never a waiting run)",
+    preview: ["clear", "logs"],
+    status: ["clear", "logs", "--summary"],
+    names_files: true,
+    danger: true,
+  },
   // ADVANCED (v0.44.0) — the one action on this panel that does not target the
   // project. Every other route pins `--dir <projectRoot>`; `--global` outranks
   // `--dir` in `resolveClaudeDir`, so this pair reaches ~/.claude and its
@@ -1146,7 +1189,11 @@ async function handleApi(req, res, url, ctx) {
       try {
         body = await readBody(req);
       } catch (_) {}
-      const canned = fixtures.post(route, body);
+      // The staged preview has nothing to replay here, so it answers with the
+    // canned picture — the shape the panel reads, not "nothing ran".
+    if (route === "/api/statusline/stage-preview")
+      return json(res, 200, { ok: true, preview: fixtures.get("/api/statusline/preview", body), show: fixtures.get("/api/statusline/show", body) });
+    const canned = fixtures.post(route, body);
       if (canned)
         return json(res, 200, {
           ok: true,
@@ -1209,6 +1256,9 @@ async function handleApi(req, res, url, ctx) {
         names_files: !!m.names_files,
         advanced: !!m.advanced,
         restarts_ui: !!m.restarts_ui,
+        danger: !!m.danger,
+        // The CLI's own no-scan summary (v2.1.0 W7) — never a number the panel computes.
+        status: m.status ? readCli(m.status, ctx).data || null : null,
       }));
       return json(res, 200, { ok: true, exit_code: 0, data: { actions } });
     }
@@ -1226,6 +1276,7 @@ async function handleApi(req, res, url, ctx) {
           network: !!m.network,
           names_files: !!m.names_files,
           advanced: !!m.advanced,
+          danger: !!m.danger,
           // Said in the confirmation, not discovered afterwards. A panel that
           // reloads itself without warning reads as a crash.
           restarts_ui: !!m.restarts_ui,
@@ -1408,6 +1459,10 @@ async function handleApi(req, res, url, ctx) {
     return json(res, 200, { ok: true, ...started });
   }
 
+  // v2.0.4 — the staged preview. NOT a mutation of this project: it runs in a
+  // throwaway copy, so it neither waits on nor blocks a running job.
+  if (route === "/api/statusline/stage-preview") return json(res, 200, stagePreview(body, ctx));
+
   const build = WRITES[route];
   if (!build) return json(res, 404, { error: "unknown endpoint " + route });
   if (job && job.running) return json(res, 409, { error: "busy", job: jobView() });
@@ -1417,8 +1472,7 @@ async function handleApi(req, res, url, ctx) {
   } catch (_) {
     return json(res, 400, { error: "bad request body" });
   }
-  if (argv.some((a) => a === "undefined" || a === "null" || a === ""))
-    return json(res, 400, { error: "missing argument" });
+  if (!argvComplete(argv)) return json(res, 400, { error: "missing argument" });
   const out = runCli(argv, ctx);
   clearCache();
   // A write's exit code is a REAL failure signal (validators exit 1), unlike a
@@ -1432,6 +1486,71 @@ async function handleApi(req, res, url, ctx) {
   });
 }
 
+// A write's argv is complete when no element is a lost value. `""` is a real
+// value right after a `--flag` — it is how a v2.0.3 clear is spelled — and is
+// refused only as a positional, together with "undefined" and "null".
+function argvComplete(argv) {
+  return !argv.some((a, i) => {
+    if (a === "undefined" || a === "null") return true;
+    if (a !== "") return false;
+    const prev = argv[i - 1];
+    return !(typeof prev === "string" && prev.startsWith("--"));
+  });
+}
+
+// The files a board's layout lives in, under `.claude/orc/`. The CLI's own
+// names; only the AUTHORED file is copied — the rest is compiled from it.
+const SL_LAYOUT_FILE = { status: "statusline-layout.json", subagent: "subagent-layout.json" };
+
+// v2.0.4 — THE STAGED PREVIEW. The staged writes are replayed through the SAME
+// `WRITES` builders against a throwaway copy of this project's layout, then
+// the copy is previewed by the hook's own engine. Nothing touches the real
+// project, and the copy is always deleted.
+function stagePreview(body, ctx) {
+  const b = body || {};
+  const board = b.board === "subagent" ? "subagent" : "status";
+  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "orc-stage-"));
+  const sub = { ...ctx, projectRoot: tmp };
+  try {
+    const orcDir = path.join(tmp, ".claude", "orc");
+    fs.mkdirSync(orcDir, { recursive: true });
+    const src = path.join(ctx.projectRoot || "", ".claude", "orc", SL_LAYOUT_FILE[board]);
+    if (ctx.projectRoot && fs.existsSync(src)) fs.copyFileSync(src, path.join(orcDir, SL_LAYOUT_FILE[board]));
+    const actions = Array.isArray(b.actions) ? b.actions : [];
+    for (let i = 0; i < actions.length; i++) {
+      const a = actions[i] || {};
+      const route = String(a.route || "");
+      const build = route.startsWith("/api/statusline/") ? WRITES[route] : null;
+      let argv = null;
+      try {
+        argv = build ? build(Object.assign({}, a.body || {}, { board: board })) : null;
+      } catch (_) {
+        argv = null;
+      }
+      if (!argv || !argvComplete(argv)) return { ok: false, failed: { index: i, output: "refused: " + route } };
+      const out = runCli(argv, sub);
+      if (out.exit_code !== 0)
+        return { ok: false, failed: { index: i, output: (out.stdout + (out.stderr ? chr10 + out.stderr : "")).trim() } };
+    }
+    const q = { board };
+    if (b.width) q.width = b.width;
+    if (b.state) q.state = b.state;
+    if (b.frames) q.frames = b.frames;
+    if (b.frame_ms) q.frame_ms = b.frame_ms;
+    const preview = runCli(READS["/api/statusline/preview"](q), sub, { json: true });
+    const show = runCli(READS["/api/statusline/show"](q), sub, { json: true });
+    return { ok: true, preview: preview.data, show: show.data };
+  } finally {
+    rmTree(tmp);
+  }
+}
+
+function rmTree(dir) {
+  try {
+    fs.rmSync(dir, { recursive: true, force: true });
+  } catch (_) {}
+}
+
 // `orc upgrade` replaces the package while your working tree may hold changes.
 // Worth a warning before, not a surprise after.
 function isDirtyTree(ctx) {
@@ -1448,4 +1567,4 @@ function isDirtyTree(ctx) {
   }
 }
 
-module.exports = { handleApi, clearCache, encodeBody, READS, WRITES, MAINTENANCE };
+module.exports = { handleApi, clearCache, encodeBody, READS, WRITES, MAINTENANCE, argvComplete, stagePreview };

package/bin/webui/css/panels/hookui.css

@@ -772,3 +772,82 @@ button.hk-pick-row:hover {
 .hk-boards {
   margin-bottom: 4px;
 }
+
+/* v2.0.4 — the line head's options and the editor's new controls. All flex,
+   so each one wraps on a narrow screen by itself. */
+.hk-line-opts-wrap {
+  display: flex;
+  align-items: center;
+  flex-wrap: wrap;
+  gap: 8px 14px;
+}
+.hk-input.hk-short {
+  width: 7em;
+}
+.hk-input.hk-char {
+  width: 3.5em;
+  text-align: center;
+}
+.hk-colour,
+.hk-hex,
+.hk-pair,
+.hk-state-row {
+  display: flex;
+  align-items: center;
+  flex-wrap: wrap;
+  gap: 6px;
+}
+.hk-colour {
+  flex-direction: column;
+  align-items: flex-start;
+}
+.hk-hex-pick {
+  width: 32px;
+  height: 26px;
+  padding: 0;
+  border: 1px solid var(--line);
+  border-radius: 6px;
+  background: none;
+  cursor: pointer;
+}
+.hk-state-row {
+  margin: 4px 0;
+}
+.hk-state-name {
+  min-width: 8em;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
+  font-size: 12px;
+}
+/* A case button's sample, under its name. */
+.hk-pick-tip {
+  display: block;
+  font-size: 11px;
+  color: var(--text-dim);
+}
+/* The CLI's own refusal text for a staged change. */
+.hk-stage-out {
+  margin: 0;
+  padding: 8px 10px;
+  white-space: pre-wrap;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
+  font-size: 12px;
+  border: 1px solid var(--line);
+  border-radius: 6px;
+  background: var(--surface-2);
+}
+.hk-staged-tag {
+  color: var(--ok);
+}
+
+/* v2.0.4 — the redraw timer, under the advanced card. */
+.hk-anim {
+  display: flex;
+  flex-direction: column;
+  gap: 6px;
+  margin-top: 10px;
+  padding-top: 10px;
+  border-top: 1px solid var(--line);
+}
+.hk-anim-head {
+  font-weight: 600;
+}

package/bin/webui/fixtures/behaviour.js

@@ -451,6 +451,29 @@ const gotchaQuality = {
   ok: true,
   window_days: 90,
   reviews: 12,
+  reviews_traced: 12,
+  reviews_observed: 9,
+  by_reviewer: { orc: 12, "/code-review": 2 },
+  project_reviews: 2,
+  // v2.1.0 W6 — `orc fix record`: the fixes made after a review, and the misses.
+  fixes: {
+    total: 5,
+    by_introduced_by: { orc: 2, ai: 1, human: 1, unknown: 1 },
+    by_source: { sonar: 2, defect: 2, ci: 1 },
+    causes: [
+      { id: "sonar", n: 2, share: 0.4 },
+      { id: "orc", n: 1, share: 0.2 },
+      { id: "ai", n: 1, share: 0.2 },
+      { id: "other", n: 1, share: 0.2 },
+    ],
+  },
+  misses: {
+    total: 1,
+    by_category: [{ category: "functional.logic", n: 1 }],
+    list: [
+      { obs: "9f2c41ab", at: new Date(NOW - 3 * DAY).toISOString().slice(0, 10), path: "src/orders/total.ts", lines: [40, 58], source: "sonar", rule: "sonar:typescript:S3776", category: "functional.logic", introduced_by: "orc", missed_by: "run-mini-total-290926-101500", fix_run: "run-quick-total-fix-021026-091200" },
+    ],
+  },
   floor: 5,
   below_floor: false,
   findings: 118,
@@ -473,6 +496,13 @@ const gotchaQualityLow = {
   ok: true,
   window_days: 30,
   reviews: 3,
+  // v2.1.0 — one review counted from its trace only (a clean review).
+  reviews_traced: 3,
+  reviews_observed: 2,
+  by_reviewer: { orc: 3 },
+  project_reviews: 0,
+  fixes: { total: 0, by_introduced_by: { orc: 0, ai: 0, human: 0, unknown: 0 }, by_source: {}, causes: [{ id: "sonar", n: 0, share: 0 }, { id: "orc", n: 0, share: 0 }, { id: "ai", n: 0, share: 0 }, { id: "other", n: 0, share: 0 }] },
+  misses: { total: 0, by_category: [], list: [] },
   floor: 5,
   below_floor: true,
   findings: 19,