quadwork 1.4.0 → 1.5.0

package/out/setup.txt

@@ -1,20 +1,20 @@
 1:"$Sreact.fragment"
-2:I[34852,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
-3:I[86081,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
-4:I[12527,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
-5:I[59763,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
-6:I[64618,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js","/_next/static/chunks/035rt-n0oid7d.js"],"default"]
-7:I[11717,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"OutletBoundary"]
+2:I[34852,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
+3:I[86081,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
+4:I[12527,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
+5:I[59763,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default"]
+6:I[64618,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js","/_next/static/chunks/084lff9v4p_vh.js"],"default"]
+7:I[11717,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"OutletBoundary"]
 8:"$Sreact.suspense"
-b:I[11717,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"ViewportBoundary"]
-d:I[11717,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"MetadataBoundary"]
-f:I[92243,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default",1]
-:HL["/_next/static/chunks/0u~7e4fgf-u06.css","style"]
+b:I[11717,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"ViewportBoundary"]
+d:I[11717,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"MetadataBoundary"]
+f:I[92243,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"default",1]
+:HL["/_next/static/chunks/05.po0c1knrbu.css","style"]
 :HL["/_next/static/media/797e433ab948586e-s.p.0.q-h669a_dqa.woff2","font",{"crossOrigin":"","type":"font/woff2"}]
-0:{"P":null,"c":["","setup"],"q":"","i":false,"f":[[["",{"children":["setup",{"children":["__PAGE__",{}]}]},"$undefined","$undefined",16],[["$","$1","c",{"children":[[["$","link","0",{"rel":"stylesheet","href":"/_next/static/chunks/0u~7e4fgf-u06.css","precedence":"next","crossOrigin":"$undefined","nonce":"$undefined"}],["$","script","script-0",{"src":"/_next/static/chunks/05ok82hwk0x-c.js","async":true,"nonce":"$undefined"}],["$","script","script-1",{"src":"/_next/static/chunks/0ox7p_szjhn69.js","async":true,"nonce":"$undefined"}]],["$","html",null,{"lang":"en","className":"geist_mono_8d43a2aa-module__8Li5zG__variable h-full","children":["$","body",null,{"className":"h-full flex flex-col","children":[["$","$L2",null,{}],["$","div",null,{"className":"flex flex-1 min-h-0","children":[["$","$L3",null,{}],["$","main",null,{"className":"flex-1 min-w-0 overflow-auto","children":["$","$L4",null,{"parallelRouterKey":"children","error":"$undefined","errorStyles":"$undefined","errorScripts":"$undefined","template":["$","$L5",null,{}],"templateStyles":"$undefined","templateScripts":"$undefined","notFound":[[["$","title",null,{"children":"404: This page could not be found."}],["$","div",null,{"style":{"fontFamily":"system-ui,\"Segoe UI\",Roboto,Helvetica,Arial,sans-serif,\"Apple Color Emoji\",\"Segoe UI Emoji\"","height":"100vh","textAlign":"center","display":"flex","flexDirection":"column","alignItems":"center","justifyContent":"center"},"children":["$","div",null,{"children":[["$","style",null,{"dangerouslySetInnerHTML":{"__html":"body{color:#000;background:#fff;margin:0}.next-error-h1{border-right:1px solid rgba(0,0,0,.3)}@media (prefers-color-scheme:dark){body{color:#fff;background:#000}.next-error-h1{border-right:1px solid rgba(255,255,255,.3)}}"}}],["$","h1",null,{"className":"next-error-h1","style":{"display":"inline-block","margin":"0 20px 0 0","padding":"0 23px 0 0","fontSize":24,"fontWeight":500,"verticalAlign":"top","lineHeight":"49px"},"children":404}],["$","div",null,{"style":{"display":"inline-block"},"children":["$","h2",null,{"style":{"fontSize":14,"fontWeight":400,"lineHeight":"49px","margin":0},"children":"This page could not be found."}]}]]}]}]],[]],"forbidden":"$undefined","unauthorized":"$undefined"}]}]]}]]}]}]]}],{"children":[["$","$1","c",{"children":[null,["$","$L4",null,{"parallelRouterKey":"children","error":"$undefined","errorStyles":"$undefined","errorScripts":"$undefined","template":["$","$L5",null,{}],"templateStyles":"$undefined","templateScripts":"$undefined","notFound":"$undefined","forbidden":"$undefined","unauthorized":"$undefined"}]]}],{"children":[["$","$1","c",{"children":[["$","$L6",null,{}],[["$","script","script-0",{"src":"/_next/static/chunks/035rt-n0oid7d.js","async":true,"nonce":"$undefined"}]],["$","$L7",null,{"children":["$","$8",null,{"name":"Next.MetadataOutlet","children":"$@9"}]}]]}],{},null,false,null]},null,false,"$@a"]},null,false,null],["$","$1","h",{"children":[null,["$","$Lb",null,{"children":"$Lc"}],["$","div",null,{"hidden":true,"children":["$","$Ld",null,{"children":["$","$8",null,{"name":"Next.Metadata","children":"$Le"}]}]}],["$","meta",null,{"name":"next-size-adjust","content":""}]]}],false]],"m":"$undefined","G":["$f",[["$","link","0",{"rel":"stylesheet","href":"/_next/static/chunks/0u~7e4fgf-u06.css","precedence":"next","crossOrigin":"$undefined","nonce":"$undefined"}]]],"S":true,"h":null,"s":"$undefined","l":"$undefined","p":"$undefined","d":"$undefined","b":"6uvV3nUfwr_t_JKrZJSP8"}
+0:{"P":null,"c":["","setup"],"q":"","i":false,"f":[[["",{"children":["setup",{"children":["__PAGE__",{}]}]},"$undefined","$undefined",16],[["$","$1","c",{"children":[[["$","link","0",{"rel":"stylesheet","href":"/_next/static/chunks/05.po0c1knrbu.css","precedence":"next","crossOrigin":"$undefined","nonce":"$undefined"}],["$","script","script-0",{"src":"/_next/static/chunks/0za4cvk8.n0-y.js","async":true,"nonce":"$undefined"}],["$","script","script-1",{"src":"/_next/static/chunks/0ox7p_szjhn69.js","async":true,"nonce":"$undefined"}]],["$","html",null,{"lang":"en","className":"geist_mono_8d43a2aa-module__8Li5zG__variable h-full","children":["$","body",null,{"className":"h-full flex flex-col","children":[["$","$L2",null,{}],["$","div",null,{"className":"flex flex-1 min-h-0","children":[["$","$L3",null,{}],["$","main",null,{"className":"flex-1 min-w-0 overflow-auto","children":["$","$L4",null,{"parallelRouterKey":"children","error":"$undefined","errorStyles":"$undefined","errorScripts":"$undefined","template":["$","$L5",null,{}],"templateStyles":"$undefined","templateScripts":"$undefined","notFound":[[["$","title",null,{"children":"404: This page could not be found."}],["$","div",null,{"style":{"fontFamily":"system-ui,\"Segoe UI\",Roboto,Helvetica,Arial,sans-serif,\"Apple Color Emoji\",\"Segoe UI Emoji\"","height":"100vh","textAlign":"center","display":"flex","flexDirection":"column","alignItems":"center","justifyContent":"center"},"children":["$","div",null,{"children":[["$","style",null,{"dangerouslySetInnerHTML":{"__html":"body{color:#000;background:#fff;margin:0}.next-error-h1{border-right:1px solid rgba(0,0,0,.3)}@media (prefers-color-scheme:dark){body{color:#fff;background:#000}.next-error-h1{border-right:1px solid rgba(255,255,255,.3)}}"}}],["$","h1",null,{"className":"next-error-h1","style":{"display":"inline-block","margin":"0 20px 0 0","padding":"0 23px 0 0","fontSize":24,"fontWeight":500,"verticalAlign":"top","lineHeight":"49px"},"children":404}],["$","div",null,{"style":{"display":"inline-block"},"children":["$","h2",null,{"style":{"fontSize":14,"fontWeight":400,"lineHeight":"49px","margin":0},"children":"This page could not be found."}]}]]}]}]],[]],"forbidden":"$undefined","unauthorized":"$undefined"}]}]]}]]}]}]]}],{"children":[["$","$1","c",{"children":[null,["$","$L4",null,{"parallelRouterKey":"children","error":"$undefined","errorStyles":"$undefined","errorScripts":"$undefined","template":["$","$L5",null,{}],"templateStyles":"$undefined","templateScripts":"$undefined","notFound":"$undefined","forbidden":"$undefined","unauthorized":"$undefined"}]]}],{"children":[["$","$1","c",{"children":[["$","$L6",null,{}],[["$","script","script-0",{"src":"/_next/static/chunks/084lff9v4p_vh.js","async":true,"nonce":"$undefined"}]],["$","$L7",null,{"children":["$","$8",null,{"name":"Next.MetadataOutlet","children":"$@9"}]}]]}],{},null,false,null]},null,false,"$@a"]},null,false,null],["$","$1","h",{"children":[null,["$","$Lb",null,{"children":"$Lc"}],["$","div",null,{"hidden":true,"children":["$","$Ld",null,{"children":["$","$8",null,{"name":"Next.Metadata","children":"$Le"}]}]}],["$","meta",null,{"name":"next-size-adjust","content":""}]]}],false]],"m":"$undefined","G":["$f",[["$","link","0",{"rel":"stylesheet","href":"/_next/static/chunks/05.po0c1knrbu.css","precedence":"next","crossOrigin":"$undefined","nonce":"$undefined"}]]],"S":true,"h":null,"s":"$undefined","l":"$undefined","p":"$undefined","d":"$undefined","b":"OzDK1Fplm2eUu23bzILlU"}
 10:[]
 a:"$W10"
 c:[["$","meta","0",{"charSet":"utf-8"}],["$","meta","1",{"name":"viewport","content":"width=device-width, initial-scale=1"}]]
-11:I[80070,["/_next/static/chunks/05ok82hwk0x-c.js","/_next/static/chunks/0ox7p_szjhn69.js"],"IconMark"]
+11:I[80070,["/_next/static/chunks/0za4cvk8.n0-y.js","/_next/static/chunks/0ox7p_szjhn69.js"],"IconMark"]
 9:null
 e:[["$","title","0",{"children":"QuadWork"}],["$","meta","1",{"name":"description","content":"Unified dashboard for multi-agent coding teams"}],["$","link","2",{"rel":"icon","href":"/favicon.ico?favicon.0x3dzn~oxb6tn.ico","sizes":"256x256","type":"image/x-icon"}],["$","$L11","3",{}]]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "quadwork",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "Unified dashboard for multi-agent coding teams — 4 AI agents, one terminal",
   "bin": {
     "quadwork": "./bin/quadwork.js"

package/server/index.js

@@ -312,6 +312,32 @@ async function buildAgentArgs(projectId, agentId) {
     if (flags) args.push(...flags);
   }
 
+  // #343: per-agent model + reasoning effort overrides. Persist in
+  // project.agents[agentId].{model,reasoning_effort} via the
+  // dashboard Agent Models widget. When unset, fall back to the
+  // CLI's own default so existing projects without overrides keep
+  // their current behavior.
+  //
+  // Codex: -c model="<slug>" / -c model_reasoning_effort="<level>"
+  //   reasoning levels: minimal | low | medium | high (xhigh is
+  //   deliberately NOT offered — it's the capacity-failure hot
+  //   spot #343 was filed for).
+  // Claude: --model <slug>
+  //   reasoning_effort is not wired for Claude — Anthropic's CLI
+  //   doesn't expose an equivalent flag.
+  if (cliBase === "codex") {
+    if (agentCfg.model && typeof agentCfg.model === "string") {
+      args.push("-c", `model="${agentCfg.model}"`);
+    }
+    if (agentCfg.reasoning_effort && typeof agentCfg.reasoning_effort === "string") {
+      args.push("-c", `model_reasoning_effort="${agentCfg.reasoning_effort}"`);
+    }
+  } else if (cliBase === "claude") {
+    if (agentCfg.model && typeof agentCfg.model === "string") {
+      args.push("--model", agentCfg.model);
+    }
+  }
+
   // MCP config injection
   const mcpHttpPort = project.mcp_http_port;
   const token = project.agentchattr_token;

package/server/queue-watcher.js

@@ -11,8 +11,19 @@
  * 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. The role/rules/identity-hint additions from wrapper.py
- * lines 501-528 are intentionally out of scope for v1 per the issue.
+ * 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");
@@ -20,6 +31,38 @@ 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`, `reviewer1`, `reviewer2`. The helper does not
+ * validate — upstream already controls who may register.
+ */
+function buildInjectionPrompt(agentName, { channel, jobId, customPrompt } = {}) {
+  if (customPrompt && typeof customPrompt === "string" && customPrompt.trim()) {
+    // Operator-supplied prompts already control the identity
+    // wording; leave them alone.
+    return customPrompt.trim();
+  }
+  if (jobId) {
+    return (
+      `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.`
+    );
+  }
+  const ch = channel || "general";
+  return (
+    `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.`
+  );
+}
+
 /**
  * Start polling `{dataDir}/{agentName}_queue.jsonl`. When non-empty,
  * read all lines, truncate the file (atomic-ish claim — same race the
@@ -78,14 +121,7 @@ function startQueueWatcher(dataDir, agentName, ptyTerm) {
       }
       if (!hasTrigger) return;
 
-      let prompt;
-      if (customPrompt) {
-        prompt = customPrompt;
-      } else if (jobId) {
-        prompt = `mcp read job_id=${jobId} - you were mentioned in a job thread, take appropriate action`;
-      } else {
-        prompt = `mcp read #${channel} - you were mentioned, take appropriate action`;
-      }
+      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
@@ -122,4 +158,5 @@ function stopQueueWatcher(handle) {
 module.exports = {
   startQueueWatcher,
   stopQueueWatcher,
+  buildInjectionPrompt,
 };

package/server/queue-watcher.test.js

@@ -0,0 +1,64 @@
+// #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", "reviewer1", "reviewer2"];
+
+// 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("reviewer2", { customPrompt: "   ", channel: "general" });
+  assert.match(p, /You are @reviewer2 /);
+  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

@@ -0,0 +1,94 @@
+// #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)");