quadwork 1.19.2 → 2.0.0

package/server/queue-watcher.js

@@ -1,171 +0,0 @@
-/**
- * Per-agent queue watcher (#393 / quadwork#251).
- *
- * AgentChattr does NOT push chat to agents. When the operator types
- * `@head` in chat, AC writes a job line to `{data_dir}/{name}_queue.jsonl`
- * and walks away. Something on the agent side has to poll that file and
- * inject an `mcp read` prompt into the running CLI's PTY so the agent
- * picks up the chat. Without that injection the agent never responds,
- * even when registration and heartbeats work.
- *
- * Reference: /Users/cho/Projects/agentchattr/wrapper.py lines 438-541
- * (`_queue_watcher`). Polling (not fs.watch) is intentional: matches
- * wrapper.py's behavior and avoids the cross-platform fs.watch
- * footguns.
- *
- * #342 / quadwork#342: the v1 prompt intentionally omitted the
- * identity hints from wrapper.py lines 501-528, which broke
- * Claude Code agent sessions. Claude's default self-concept is
- * `@claude`, so a bare `mcp read #general - you were mentioned`
- * causes chat_read(sender: "claude") and a filter on `@claude`
- * mentions — both wrong when the agent is actually @dev /
- * @reviewerN. Codex doesn't trip the same way because its init
- * path already claims identity. The fix here is to scope the
- * wrapper.py additions to identity only: the injected prompt
- * now explicitly names the agent slug and tells the agent which
- * sender to use on chat_read and which mentions to look for.
- */
-
-const fs = require("fs");
-const path = require("path");
-
-const POLL_INTERVAL_MS = 1000;
-
-/**
- * Pure helper: build the injected prompt text for a given agent
- * slug + trigger shape. Exported so it can be unit-tested without
- * a PTY or a filesystem queue. Priority matches the tick() call
- * site below: customPrompt > jobId > channel.
- *
- * agentName is expected to be the registered agent slug such as
- * `dev`, `head`, `re1`, `re2`. The helper does not
- * validate — upstream already controls who may register.
- */
-function buildInjectionPrompt(agentName, { channel, jobId, customPrompt, attachments } = {}) {
-  if (customPrompt && typeof customPrompt === "string" && customPrompt.trim()) {
-    // Operator-supplied prompts already control the identity
-    // wording; leave them alone.
-    return customPrompt.trim();
-  }
-  let prompt;
-  if (jobId) {
-    prompt =
-      `You are @${agentName} in this AgentChattr instance. ` +
-      `mcp read job_id=${jobId} with sender: "${agentName}" — ` +
-      `you (@${agentName}) were mentioned in a job thread, take appropriate action.`;
-  } else {
-    const ch = channel || "general";
-    prompt =
-      `You are @${agentName} in this AgentChattr instance. ` +
-      `mcp read #${ch} with sender: "${agentName}" — ` +
-      `look for @${agentName} mentions (NOT @claude). ` +
-      `You were mentioned, take appropriate action.`;
-  }
-  // #466: include attachment paths so image-capable agents can read them
-  if (Array.isArray(attachments) && attachments.length > 0) {
-    for (const att of attachments) {
-      if (att && att.path) {
-        prompt += ` Image attached: ${att.path} — use Read tool to view it.`;
-      }
-    }
-  }
-  return prompt;
-}
-
-/**
- * Start polling `{dataDir}/{agentName}_queue.jsonl`. When non-empty,
- * read all lines, truncate the file (atomic-ish claim — same race the
- * Python wrapper accepts), parse each JSON line, build a single
- * injected prompt, and write it into the supplied PTY terminal.
- *
- * Returns an opaque interval handle. Pass it to stopQueueWatcher to
- * cancel; safe to call with null.
- */
-function startQueueWatcher(dataDir, agentName, ptyTerm) {
-  if (!dataDir || !agentName || !ptyTerm) return null;
-  const queueFile = path.join(dataDir, `${agentName}_queue.jsonl`);
-
-  const tick = () => {
-    try {
-      if (!fs.existsSync(queueFile)) return;
-      const stat = fs.statSync(queueFile);
-      if (stat.size === 0) return;
-
-      const content = fs.readFileSync(queueFile, "utf-8");
-      // Atomic claim: truncate immediately so the next AC write lands
-      // in an empty file and we don't double-process the same job on
-      // the next tick. There's a small race if AC writes between the
-      // read and the truncate; wrapper.py accepts the same race.
-      fs.writeFileSync(queueFile, "");
-
-      const lines = content.split("\n").map((l) => l.trim()).filter(Boolean);
-      if (lines.length === 0) return;
-
-      let channel = "general";
-      let customPrompt = "";
-      let jobId = null;
-      let hasTrigger = false;
-      for (const line of lines) {
-        let data;
-        try {
-          data = JSON.parse(line);
-        } catch {
-          continue;
-        }
-        hasTrigger = true;
-        if (data && typeof data === "object") {
-          if (typeof data.channel === "string") channel = data.channel;
-          // AgentChattr serializes job_id as an integer (agents.py
-          // defines `job_id: int | None`), so accept both numbers and
-          // strings here. Without this, job-thread triggers fall back
-          // to the channel prompt and the agent reads the wrong
-          // conversation. Cast to string for the prompt template.
-          if (typeof data.job_id === "number" || typeof data.job_id === "string") {
-            jobId = String(data.job_id);
-          }
-          if (typeof data.prompt === "string" && data.prompt.trim()) {
-            customPrompt = data.prompt.trim();
-          }
-        }
-      }
-      if (!hasTrigger) return;
-
-      const prompt = buildInjectionPrompt(agentName, { channel, jobId, customPrompt });
-
-      // Flatten newlines: multi-line writes trigger paste detection in
-      // Claude Code (shows "[Pasted text +N]") and can break injection
-      // of long prompts. Mirrors wrapper.py:532.
-      const flat = prompt.replace(/\n/g, " ");
-      // Inject text and Enter as SEPARATE writes with a delay between.
-      // Codex's TUI does not submit when text + "\r" arrive in one chunk —
-      // it needs the text to render, then a separate Enter keystroke.
-      // Claude Code accepts either form. Mirrors wrapper_unix.py inject():
-      // tmux send-keys -l <text> ; sleep ; tmux send-keys Enter.
-      // Delay scales with prompt length so longer prompts get more time
-      // to render before submit.
-      ptyTerm.write(flat);
-      const submitDelayMs = Math.max(300, flat.length);
-      setTimeout(() => {
-        try { ptyTerm.write("\r"); } catch { /* swallow */ }
-      }, submitDelayMs);
-    } catch {
-      // Swallow — next tick will retry. Logging here would spam the
-      // server output once per second on a permission error.
-    }
-  };
-
-  return setInterval(tick, POLL_INTERVAL_MS);
-}
-
-/**
- * Stop a watcher started by startQueueWatcher. Safe to call with null.
- */
-function stopQueueWatcher(handle) {
-  if (handle) clearInterval(handle);
-}
-
-module.exports = {
-  startQueueWatcher,
-  stopQueueWatcher,
-  buildInjectionPrompt,
-};

package/server/queue-watcher.test.js

@@ -1,64 +0,0 @@
-// #342 / quadwork#342: unit test for the identity-aware prompt
-// builder. No test runner is wired up in this repo, so this file
-// is a plain node:assert script — run it with `node server/queue-watcher.test.js`
-// and it exits non-zero on any failure.
-
-const assert = require("node:assert/strict");
-const { buildInjectionPrompt } = require("./queue-watcher");
-
-const DEFAULT_AGENT_SLUGS = ["dev", "head", "re1", "re2"];
-
-// 1) Channel prompt — each of the 4 default slugs must:
-//    - name the agent with @<slug>
-//    - pass sender: "<slug>" explicitly
-//    - tell the agent to look for @<slug> mentions (NOT @claude)
-for (const slug of DEFAULT_AGENT_SLUGS) {
-  const p = buildInjectionPrompt(slug, { channel: "general" });
-  assert.match(p, new RegExp(`You are @${slug} `), `channel: names @${slug}`);
-  assert.match(p, new RegExp(`sender: "${slug}"`), `channel: sender string for ${slug}`);
-  assert.match(p, new RegExp(`@${slug} mentions`), `channel: @${slug} mention filter`);
-  assert.match(p, /NOT @claude/, `channel: explicit NOT @claude guard for ${slug}`);
-  assert.match(p, /#general/, `channel: channel name for ${slug}`);
-}
-
-// 2) Channel defaults to "general" when not provided.
-{
-  const p = buildInjectionPrompt("dev", {});
-  assert.match(p, /#general/, "channel defaults to general");
-}
-
-// 3) Non-default channel is passed through.
-{
-  const p = buildInjectionPrompt("dev", { channel: "batch-33" });
-  assert.match(p, /#batch-33/, "custom channel is used");
-}
-
-// 4) Job-thread prompt — each slug must:
-//    - name the agent with @<slug>
-//    - reference the job_id
-//    - pass sender: "<slug>" explicitly
-for (const slug of DEFAULT_AGENT_SLUGS) {
-  const p = buildInjectionPrompt(slug, { jobId: "42" });
-  assert.match(p, new RegExp(`You are @${slug} `), `job: names @${slug}`);
-  assert.match(p, /job_id=42/, `job: job_id for ${slug}`);
-  assert.match(p, new RegExp(`sender: "${slug}"`), `job: sender string for ${slug}`);
-}
-
-// 5) customPrompt wins over channel and jobId and is returned as-is.
-{
-  const p = buildInjectionPrompt("dev", {
-    channel: "general",
-    jobId: "99",
-    customPrompt: "  do the thing  ",
-  });
-  assert.equal(p, "do the thing", "customPrompt overrides + trims");
-}
-
-// 6) Blank customPrompt is ignored (falls through to channel/job path).
-{
-  const p = buildInjectionPrompt("re2", { customPrompt: "   ", channel: "general" });
-  assert.match(p, /You are @re2 /);
-  assert.match(p, /#general/);
-}
-
-console.log(`queue-watcher.test.js: all assertions passed (${DEFAULT_AGENT_SLUGS.length * 2 + 4} cases)`);

package/server/routes.batchProgress.test.js

@@ -1,94 +0,0 @@
-// #350 / quadwork#350: batch-progress no-linked-PR row builder +
-// summarizer tests. Plain node:assert script — run with
-// `node server/routes.batchProgress.test.js`.
-
-const assert = require("node:assert/strict");
-const { buildNoPrRow, summarizeItems } = require("./routes");
-
-// 1) #350 regression fixture: CLOSED issue with no linked PR
-//    must render as 100% complete, not 0% queued.
-{
-  const issue = {
-    number: 336,
-    title: "superseded by #338",
-    state: "CLOSED",
-    url: "https://github.com/realproject7/quadwork/issues/336",
-  };
-  const row = buildNoPrRow(issue);
-  assert.equal(row.status, "closed", "CLOSED with no PR → status=closed");
-  assert.equal(row.progress, 100, "CLOSED with no PR → 100%");
-  assert.match(row.label, /Closed.*✓/, "label has Closed and ✓ marker");
-  assert.equal(row.issue_number, 336);
-  assert.equal(row.url, issue.url);
-}
-
-// 2) OPEN issue with no linked PR still renders as queued.
-{
-  const issue = {
-    number: 400,
-    title: "still open",
-    state: "OPEN",
-    url: "https://github.com/realproject7/quadwork/issues/400",
-  };
-  const row = buildNoPrRow(issue);
-  assert.equal(row.status, "queued", "OPEN with no PR → queued");
-  assert.equal(row.progress, 0);
-  assert.equal(row.label, "Issue · queued");
-}
-
-// 3) summarizeItems with a mix of merged and closed-without-PR:
-//    should count both toward the complete total, label "complete"
-//    when closed > 0.
-{
-  const items = [
-    { status: "merged" },
-    { status: "merged" },
-    { status: "merged" },
-    { status: "merged" },
-    { status: "merged" },
-    { status: "closed" },
-    { status: "closed" },
-  ];
-  const out = summarizeItems(items);
-  assert.equal(out, "7/7 complete", "mixed merged+closed → X/N complete");
-}
-
-// 4) summarizeItems with only merged items keeps the classic
-//    "X/N merged" wording (no behavior change for PR-only batches).
-{
-  const items = [
-    { status: "merged" },
-    { status: "merged" },
-    { status: "merged" },
-  ];
-  assert.equal(summarizeItems(items), "3/3 merged");
-}
-
-// 5) summarizeItems with a queued + closed mix: done count is
-//    closed only, queued surfaces in the detail tail.
-{
-  const items = [
-    { status: "closed" },
-    { status: "queued" },
-    { status: "queued" },
-  ];
-  assert.equal(summarizeItems(items), "1/3 complete · 2 queued");
-}
-
-// 6) summarizeItems with in-flight PR states still tallies them
-//    in the detail tail and keeps the done count at merged-only.
-{
-  const items = [
-    { status: "merged" },
-    { status: "ready" },
-    { status: "approved1" },
-    { status: "in_review" },
-    { status: "queued" },
-  ];
-  assert.equal(
-    summarizeItems(items),
-    "1/5 merged · 1 ready to merge · 1 needs 2nd approval · 1 in review · 1 queued",
-  );
-}
-
-console.log("routes.batchProgress.test.js: all assertions passed (6 cases)");

package/server/routes.chatWsSend.test.js

@@ -1,161 +0,0 @@
-// #236 / quadwork#236: sendViaWebSocket ack/body/error regression.
-// Stands up a minimal fake AgentChattr ws server that mirrors app.py's
-// `type:"message"` handler behavior (accept the frame, assign an id +
-// timestamp, broadcast `{type:"message", data: msg}` back to the
-// sender) and verifies:
-//
-//   1. Successful send resolves with {ok:true, message:{id,…}} — the
-//      echoed broadcast frame, not a fake {ok:true}.
-//   2. Attachments survive the round trip.
-//   3. History replay on connect does NOT satisfy the ack (the echo
-//      must come AFTER the send, not before).
-//   4. A premature close without an echo rejects with an error (the
-//      old fire-and-forget path silently resolved).
-//   5. Close code 4003 rejects with err.code === "EAGENTCHATTR_401"
-//      so the /api/chat handler can surface a proper 401.
-//
-// Run with: node server/routes.chatWsSend.test.js
-
-const assert = require("node:assert/strict");
-const http = require("node:http");
-const { WebSocketServer } = require("ws");
-const { sendViaWebSocket } = require("./routes");
-
-function startFakeAc({ historyBeforeAck = [], rejectWithCode = null, dropBeforeAck = false } = {}) {
-  const server = http.createServer();
-  const wss = new WebSocketServer({ noServer: true });
-  server.on("upgrade", (req, socket, head) => {
-    if (rejectWithCode === 4003) {
-      // Accept the upgrade, then immediately close with 4003 the way
-      // AC's /ws handler does on invalid token.
-      wss.handleUpgrade(req, socket, head, (ws) => {
-        ws.close(4003, "forbidden: invalid session token");
-      });
-      return;
-    }
-    wss.handleUpgrade(req, socket, head, (ws) => {
-      // Simulate history replay on connect — these must NOT satisfy
-      // the ack, even if they match (sender,text,channel).
-      let historyMaxId = 0;
-      for (const msg of historyBeforeAck) {
-        ws.send(JSON.stringify({ type: "message", data: msg }));
-        if (typeof msg.id === "number" && msg.id > historyMaxId) historyMaxId = msg.id;
-      }
-      // Mirror AC: emit one `type:"status"` frame after history so the
-      // client knows the replay is done. See agentchattr/app.py
-      // `broadcast_status()` call in the /ws handler (line ~1082).
-      ws.send(JSON.stringify({ type: "status", data: { ready: true } }));
-      let nextId = Math.max(9000, historyMaxId + 1);
-      ws.on("message", (raw) => {
-        const frame = JSON.parse(raw.toString());
-        if (frame.type !== "message") return;
-        if (dropBeforeAck) {
-          ws.close();
-          return;
-        }
-        // Mirror store.add: assign id + timestamp, rebroadcast.
-        const echoed = {
-          id: nextId++,
-          sender: frame.sender,
-          text: frame.text,
-          channel: frame.channel || "general",
-          attachments: frame.attachments || [],
-          reply_to: frame.reply_to ?? null,
-          timestamp: Date.now() / 1000,
-        };
-        ws.send(JSON.stringify({ type: "message", data: echoed }));
-      });
-    });
-  });
-  return new Promise((resolve) => {
-    server.listen(0, "127.0.0.1", () => {
-      const { port } = server.address();
-      resolve({ url: `http://127.0.0.1:${port}`, close: () => new Promise((r) => server.close(() => r())) });
-    });
-  });
-}
-
-(async () => {
-  // 1 + 2) Success path: echo carries server id + attachments preserved.
-  {
-    const ac = await startFakeAc();
-    const result = await sendViaWebSocket(ac.url, "fake-token", {
-      text: "hello from test",
-      sender: "user",
-      channel: "general",
-      attachments: [{ url: "/uploads/x.png", name: "x.png" }],
-    });
-    assert.equal(result.ok, true);
-    assert.ok(result.message, "expected echoed message object");
-    assert.equal(typeof result.message.id, "number");
-    assert.ok(result.message.id >= 9000);
-    assert.equal(result.message.text, "hello from test");
-    assert.equal(result.message.sender, "user");
-    assert.equal(result.message.attachments.length, 1);
-    assert.equal(result.message.attachments[0].name, "x.png");
-    await ac.close();
-  }
-
-  // 3) Stale history replay must NOT satisfy the ack — even when the
-  //    history contains a message that is IDENTICAL (sender, text,
-  //    channel, reply_to) and has a recent timestamp (e.g. a retry of
-  //    the same message sent <1s ago). Prior heuristic matcher was
-  //    vulnerable to this; the fix uses the (1) status-frame history
-  //    boundary and (2) strictly-greater-id correlation baseline so
-  //    the historical echo is definitionally rejected. Reviewer1
-  //    flagged this race on PR #382 round 1.
-  {
-    const stale = {
-      id: 42, sender: "user", text: "same words",
-      channel: "general", attachments: [], reply_to: null,
-      timestamp: Date.now() / 1000, // RIGHT NOW — old heuristic would accept this
-    };
-    const ac = await startFakeAc({ historyBeforeAck: [stale] });
-    const result = await sendViaWebSocket(ac.url, "fake-token", {
-      text: "same words",
-      sender: "user",
-      channel: "general",
-      attachments: [],
-    });
-    assert.ok(result.message.id > 42, `expected live echo id > 42, got ${result.message.id}`);
-    await ac.close();
-  }
-
-  // 4) Premature close without ack rejects.
-  {
-    const ac = await startFakeAc({ dropBeforeAck: true });
-    let threw = false;
-    try {
-      await sendViaWebSocket(ac.url, "fake-token", {
-        text: "will drop", sender: "user", channel: "general", attachments: [],
-      });
-    } catch (err) {
-      threw = true;
-      assert.match(err.message, /closed before ack|websocket/);
-      assert.notEqual(err.code, "EAGENTCHATTR_401");
-    }
-    assert.equal(threw, true, "expected sendViaWebSocket to reject on premature close");
-    await ac.close();
-  }
-
-  // 5) 4003 (bad token) rejects with EAGENTCHATTR_401.
-  {
-    const ac = await startFakeAc({ rejectWithCode: 4003 });
-    let threw = false;
-    try {
-      await sendViaWebSocket(ac.url, "bad-token", {
-        text: "denied", sender: "user", channel: "general", attachments: [],
-      });
-    } catch (err) {
-      threw = true;
-      assert.equal(err.code, "EAGENTCHATTR_401");
-    }
-    assert.equal(threw, true, "expected sendViaWebSocket to reject with EAGENTCHATTR_401");
-    await ac.close();
-  }
-
-  console.log("routes.chatWsSend.test.js: all assertions passed (5 cases)");
-})().catch((err) => {
-  console.error(err);
-  process.exit(1);
-});

package/server/routes.discordBridge.test.js

@@ -1,80 +0,0 @@
-// #399: Discord bridge integration tests. Plain node:assert script —
-// run with `node server/routes.discordBridge.test.js`.
-
-const assert = require("node:assert/strict");
-const fs = require("node:fs");
-const os = require("node:os");
-const path = require("node:path");
-const {
-  checkDiscordBridgePythonDeps,
-  buildDiscordBridgeToml,
-  patchAgentchattrConfigForDiscordBridge,
-  buildDiscordBridgeSpawnEnv,
-} = require("./routes");
-
-const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "qw-discord-bridge-"));
-
-try {
-  // 1) checkDiscordBridgePythonDeps: broken interpreter returns
-  //    { ok: false, error } without throwing.
-  const broken = checkDiscordBridgePythonDeps(path.join(tmp, "nope", "python3"));
-  assert.equal(broken.ok, false);
-  assert.ok(broken.error && broken.error.length > 0);
-
-  // 2) buildDiscordBridgeToml writes agentchattr_url inside [discord].
-  const toml = buildDiscordBridgeToml({
-    bot_token: "MTIz.abc.def",
-    channel_id: "123456789",
-    agentchattr_url: "http://127.0.0.1:8301",
-  }, "testproject");
-  assert.match(toml, /^\[discord\]/);
-  assert.match(toml, /bot_token = "MTIz\.abc\.def"/);
-  assert.match(toml, /channel_id = "123456789"/);
-  assert.match(toml, /agentchattr_url = "http:\/\/127\.0\.0\.1:8301"/);
-  // Per-project cursor file
-  assert.match(toml, /cursor_file = ".*dc-bridge-cursor-testproject\.json"/);
-  // Must NOT emit a separate [agentchattr] section
-  assert.equal(toml.includes("\n[agentchattr]\n"), false);
-
-  // 3) patchAgentchattrConfigForDiscordBridge is idempotent.
-  const baseConfig =
-    "[agents.head]\nlabel = \"Head\"\n\n[agents.dev]\nlabel = \"Dev\"\n";
-  const first = patchAgentchattrConfigForDiscordBridge(baseConfig);
-  assert.equal(first.changed, true);
-  assert.match(first.text, /^\[agents\.dc\]$/m);
-  assert.match(first.text, /label = "Discord Bridge"/);
-  // Second run is a no-op
-  const second = patchAgentchattrConfigForDiscordBridge(first.text);
-  assert.equal(second.changed, false);
-  assert.equal(second.text, first.text);
-  // #439: old slug [agents.discord-bridge] is migrated to [agents.dc]
-  const handPatched =
-    baseConfig + "\n[agents.discord-bridge]\nlabel = \"Discord Bridge\"\n";
-  const third = patchAgentchattrConfigForDiscordBridge(handPatched);
-  assert.equal(third.changed, true);
-  assert.match(third.text, /^\[agents\.dc\]$/m);
-
-  // 4) buildDiscordBridgeSpawnEnv strips Discord-specific env vars.
-  const scrubbed = buildDiscordBridgeSpawnEnv({
-    PATH: "/usr/bin",
-    HOME: "/home/op",
-    DISCORD_BOT_TOKEN: "wrong-token",
-    DISCORD_CHANNEL_ID: "999",
-    AGENTCHATTR_URL: "http://127.0.0.1:9999",
-  });
-  assert.equal(scrubbed.DISCORD_BOT_TOKEN, undefined);
-  assert.equal(scrubbed.DISCORD_CHANNEL_ID, undefined);
-  assert.equal(scrubbed.AGENTCHATTR_URL, undefined);
-  // Non-discord keys pass through
-  assert.equal(scrubbed.PATH, "/usr/bin");
-  assert.equal(scrubbed.HOME, "/home/op");
-
-  // 5) Missing venv path check
-  const fixtureBridgeDir = fs.mkdtempSync(path.join(tmp, "bridge-no-venv-"));
-  const missingVenvPython = path.join(fixtureBridgeDir, ".venv", "bin", "python3");
-  assert.equal(fs.existsSync(missingVenvPython), false);
-
-  console.log("routes.discordBridge.test.js: all assertions passed (5 cases)");
-} finally {
-  try { fs.rmSync(tmp, { recursive: true, force: true }); } catch {}
-}

package/server/routes.parseActiveBatch.test.js

@@ -1,88 +0,0 @@
-// #341 / quadwork#341: parseActiveBatch regex tests. Plain
-// node:assert script — no test runner is wired up. Run with
-// `node server/routes.parseActiveBatch.test.js`.
-//
-// parseActiveBatch is re-exported from server/routes.js for this
-// test only; it has no production callers outside routes.js.
-
-const assert = require("node:assert/strict");
-const { parseActiveBatch } = require("./routes");
-
-function wrap(body, batchLine = "**Batch:** 33") {
-  return `# Overnight Queue\n\n## Active Batch\n\n${batchLine}\n\n${body}\n\n## Backlog\n\n- #999 something else\n`;
-}
-
-// 1) #341 regression: GFM checkbox items (space between `[` and `#`)
-//    must populate the list.
-{
-  const text = wrap(
-    [
-      "- [ ] #338 — Remove home hero",
-      "- [ ] #337 — Stack SERVER",
-      "- [x] #332 — Commit port drafts",
-      "- [X] #334 — Snapshot stale check",
-    ].join("\n"),
-  );
-  const { batchNumber, issueNumbers } = parseActiveBatch(text);
-  assert.equal(batchNumber, 33, "batch number parsed");
-  assert.deepEqual(issueNumbers, [338, 337, 332, 334], "checkbox items parsed in order");
-}
-
-// 2) Existing shapes keep working.
-{
-  const text = wrap(
-    [
-      "- #295 sub-A heartbeat",
-      "* #296 sub-B",
-      "1. #297 sub-C",
-      "#298 sub-D",
-      "- [#299] sub-E",
-      "[#300] sub-F",
-    ].join("\n"),
-  );
-  const { issueNumbers } = parseActiveBatch(text);
-  assert.deepEqual(issueNumbers, [295, 296, 297, 298, 299, 300], "legacy shapes still parsed");
-}
-
-// 3) Prose references still rejected.
-{
-  const text = wrap(
-    [
-      "- [ ] #400 real item",
-      "Tracking umbrella: #293",
-      "Assigned next after #294 merged.",
-      "See #295 for context.",
-    ].join("\n"),
-  );
-  const { issueNumbers } = parseActiveBatch(text);
-  assert.deepEqual(issueNumbers, [400], "prose references rejected, only real item kept");
-}
-
-// 4) De-dup: same issue number on multiple lines collapses.
-{
-  const text = wrap(
-    [
-      "- [ ] #100 first mention",
-      "- [x] #100 second mention",
-      "- [ ] #101 another",
-    ].join("\n"),
-  );
-  const { issueNumbers } = parseActiveBatch(text);
-  assert.deepEqual(issueNumbers, [100, 101], "de-dup keeps first occurrence");
-}
-
-// 5) Items in Backlog section are NOT picked up.
-{
-  const text = wrap("- [ ] #500 active item");
-  const { issueNumbers } = parseActiveBatch(text);
-  assert.deepEqual(issueNumbers, [500], "Backlog section not scanned");
-}
-
-// 6) Empty / missing Active Batch returns empty.
-{
-  const { batchNumber, issueNumbers } = parseActiveBatch("# no active batch here\n");
-  assert.equal(batchNumber, null);
-  assert.deepEqual(issueNumbers, []);
-}
-
-console.log("routes.parseActiveBatch.test.js: all assertions passed (6 cases)");