@ai-setting/roy-task-plugin 0.7.4 → 0.7.6

package/dist/claude-plugin/agents/roy.md

@@ -46,13 +46,20 @@ Report progress as you go, not only at the end:
 If the scope changes, `task_update` the description/goals instead of silently
 doing something else.
 
-### 4. Record milestones — `task_operation_create`
+### 4. Record milestones — `task_operation_create` — **as you work**
+Call this tool the moment a stage is reached — **not after the fact**,
+not in a batch at the end. Finished a chunk of code? A test just went
+green? Hit a blocker? Made a decision? Record it **immediately**, then
+keep working. The record is the running log of the work, written while
+it happens.
+
 This tool has **two orthogonal dimensions** and you must keep them distinct:
 - **state / milestone dimension** — `milestone_type` + `milestone_title` +
   `milestone_description`: *what* was reached (e.g. "GREEN: all tests pass").
 - **process dimension** — `process_description`: *how* you got there —
-  strategy + tool steps, strictly **100–150 characters**.
-  Too short = rejected (no value); too long = rejected (use the session log).
+  the operations just performed and why, written in the moment,
+  **100–500 characters**. Too short = rejected (no value);
+  too long = rejected (that is a log, not a summary).
 
 Rules: do **not** use the process field to restate the milestone, and do not
 use milestones as a substitute for progress. Milestones mark *state

package/dist/claude-plugin/commands/task-milestone.md

@@ -32,7 +32,7 @@ If you do have the MCP server, call `task_operation_create` with:
   `solution`, `decision`, `review`, `completed`
 - `milestone_title` — one line
 - `milestone_description` — what was reached
-- `process_description` — **how** you got there, strictly 100–150 characters
+- `process_description` — **how** you got there, 100–500 characters
   (this is enforced; too short or too long is rejected)
 
 Keep the two dimensions distinct: the milestone says *what* was reached, the

package/dist/cli.js

@@ -444,126 +444,6 @@ class NdjsonAssembler {
   }
 }
 
-// ../../../packages/cli/src/daemon/runners/codex-stream.ts
-var CODEX_ITEM_TYPES = Object.freeze({
-  AGENT_MESSAGE: "agent_message",
-  COMMAND_EXECUTION: "command_execution",
-  MCP_TOOL_CALL: "mcp_tool_call",
-  ERROR: "error"
-});
-
-class CodexStreamDecoder {
-  lines = new LineSplitter;
-  state = { tools: [], errors: [], turnCompleted: false };
-  mcpSeen = new Set;
-  pushChunk(chunk) {
-    for (const line of this.lines.pushChunk(chunk))
-      this.consume(line);
-  }
-  finish() {
-    for (const line of this.lines.finish())
-      this.consume(line);
-    return this.toResult();
-  }
-  snapshot() {
-    return { ...this.state, tools: [...this.state.tools], errors: [...this.state.errors] };
-  }
-  consume(rawLine) {
-    const line = rawLine.trim();
-    if (!line.startsWith("{"))
-      return;
-    let event;
-    try {
-      event = JSON.parse(line);
-    } catch {
-      return;
-    }
-    if (typeof event !== "object" || event === null)
-      return;
-    const ev = event;
-    if (ev.type === "thread.started") {
-      const id = ev.thread_id;
-      if (typeof id === "string" && id)
-        this.state.sessionId = id;
-      return;
-    }
-    if (ev.type === "turn.completed") {
-      this.state.turnCompleted = true;
-      return;
-    }
-    if (ev.type === "item.started" || ev.type === "item.completed") {
-      this.consumeItem(ev.item, ev.type === "item.completed");
-    }
-  }
-  consumeItem(raw, completed) {
-    if (typeof raw !== "object" || raw === null)
-      return;
-    const item = raw;
-    if (item.type === CODEX_ITEM_TYPES.AGENT_MESSAGE) {
-      if (typeof item.text === "string" && item.text.length > 0) {
-        this.state.text = item.text;
-      }
-      return;
-    }
-    if (item.type === CODEX_ITEM_TYPES.ERROR) {
-      if (typeof item.message === "string" && item.message.length > 0) {
-        this.state.errors.push(item.message);
-      }
-      return;
-    }
-    if (item.type === CODEX_ITEM_TYPES.COMMAND_EXECUTION) {
-      const id = item.id;
-      if (typeof id === "string" && id && this.state.tools.includes(id)) {} else {
-        this.state.tools.push(typeof id === "string" ? id : `cmd-${this.state.tools.length}`);
-      }
-      if (completed && this.state.createdTask === undefined) {
-        this.state.createdTask = extractCreatedTask(item.aggregated_output);
-      }
-      return;
-    }
-    if (item.type === CODEX_ITEM_TYPES.MCP_TOOL_CALL) {
-      const id = item.id;
-      if (typeof id === "string" && id) {
-        if (this.mcpSeen.has(id))
-          return;
-        this.mcpSeen.add(id);
-      }
-      const name = typeof item.tool === "string" && item.tool ? item.tool : undefined;
-      this.state.tools.push(name ?? (typeof id === "string" && id ? id : `mcp-${this.state.tools.length}`));
-      return;
-    }
-  }
-  toResult() {
-    const result = {
-      text: this.state.text ?? ""
-    };
-    if (this.state.sessionId)
-      result.sessionId = this.state.sessionId;
-    if (this.state.tools.length > 0) {
-      result.toolCalls = this.state.tools.length;
-      result.tools = [...this.state.tools];
-    }
-    if (this.state.createdTask)
-      result.createdTask = this.state.createdTask;
-    return result;
-  }
-}
-
-// ../../../packages/cli/src/daemon/runners/codex-sessions.ts
-function createCodexSessionRegistry() {
-  const sessions = new Map;
-  return {
-    get: (key) => sessions.get(key),
-    set: (key, id) => {
-      sessions.set(key, id);
-    },
-    delete: (key) => {
-      sessions.delete(key);
-    },
-    size: () => sessions.size
-  };
-}
-
 // ../../../packages/cli/src/daemon/runners/claude-agents.ts
 var CLAUDE_ROY_AGENT_NAME = "roy";
 var ROY_AGENT_DESCRIPTION = "Roy — task-as-first-class-citizen agent. Uses the roy-agent task lifecycle " + "(task_tree → task_create → task_update → task_operation_create → task_complete) " + "so every piece of work is tracked in the local task store.";
@@ -631,13 +511,20 @@ Report progress as you go, not only at the end:
 If the scope changes, \`task_update\` the description/goals instead of silently
 doing something else.
 
-### 4. Record milestones — \`task_operation_create\`
+### 4. Record milestones — \`task_operation_create\` — **as you work**
+Call this tool the moment a stage is reached — **not after the fact**,
+not in a batch at the end. Finished a chunk of code? A test just went
+green? Hit a blocker? Made a decision? Record it **immediately**, then
+keep working. The record is the running log of the work, written while
+it happens.
+
 This tool has **two orthogonal dimensions** and you must keep them distinct:
 - **state / milestone dimension** — \`milestone_type\` + \`milestone_title\` +
   \`milestone_description\`: *what* was reached (e.g. "GREEN: all tests pass").
 - **process dimension** — \`process_description\`: *how* you got there —
-  strategy + tool steps, strictly **100–150 characters**.
-  Too short = rejected (no value); too long = rejected (use the session log).
+  the operations just performed and why, written in the moment,
+  **100–500 characters**. Too short = rejected (no value);
+  too long = rejected (that is a log, not a summary).
 
 Rules: do **not** use the process field to restate the milestone, and do not
 use milestones as a substitute for progress. Milestones mark *state
@@ -714,6 +601,159 @@ function buildClaudeAgentsJson(opts = {}) {
   return JSON.stringify({ [opts.name ?? CLAUDE_ROY_AGENT_NAME]: def });
 }
 
+// ../../../packages/cli/src/daemon/runners/codex-stream.ts
+var CODEX_ITEM_TYPES = Object.freeze({
+  AGENT_MESSAGE: "agent_message",
+  COMMAND_EXECUTION: "command_execution",
+  MCP_TOOL_CALL: "mcp_tool_call",
+  ERROR: "error"
+});
+function isCreationTool(tool) {
+  if (isTaskCreateTool(tool))
+    return true;
+  const bare = tool.includes("/") ? tool.slice(tool.lastIndexOf("/") + 1) : tool;
+  return isTaskCreateTool(bare);
+}
+function isSuccessfulMcpStatus(status) {
+  return status === undefined || status === "completed";
+}
+function isRoyTaskServer(server) {
+  return server === undefined || server === ROY_TASK_MCP_SERVER;
+}
+function mcpResultText(result) {
+  if (typeof result !== "object" || result === null)
+    return;
+  const content = result.content;
+  if (!Array.isArray(content))
+    return;
+  for (const block of content) {
+    if (typeof block === "object" && block !== null && block.type === "text" && typeof block.text === "string") {
+      return block.text;
+    }
+  }
+  return;
+}
+
+class CodexStreamDecoder {
+  lines = new LineSplitter;
+  state = { tools: [], errors: [], turnCompleted: false };
+  mcpSeen = new Set;
+  pushChunk(chunk) {
+    for (const line of this.lines.pushChunk(chunk))
+      this.consume(line);
+  }
+  finish() {
+    for (const line of this.lines.finish())
+      this.consume(line);
+    return this.toResult();
+  }
+  snapshot() {
+    return { ...this.state, tools: [...this.state.tools], errors: [...this.state.errors] };
+  }
+  consume(rawLine) {
+    const line = rawLine.trim();
+    if (!line.startsWith("{"))
+      return;
+    let event;
+    try {
+      event = JSON.parse(line);
+    } catch {
+      return;
+    }
+    if (typeof event !== "object" || event === null)
+      return;
+    const ev = event;
+    if (ev.type === "thread.started") {
+      const id = ev.thread_id;
+      if (typeof id === "string" && id)
+        this.state.sessionId = id;
+      return;
+    }
+    if (ev.type === "turn.completed") {
+      this.state.turnCompleted = true;
+      return;
+    }
+    if (ev.type === "item.started" || ev.type === "item.completed") {
+      this.consumeItem(ev.item, ev.type === "item.completed");
+    }
+  }
+  consumeItem(raw, completed) {
+    if (typeof raw !== "object" || raw === null)
+      return;
+    const item = raw;
+    if (item.type === CODEX_ITEM_TYPES.AGENT_MESSAGE) {
+      if (typeof item.text === "string" && item.text.length > 0) {
+        this.state.text = item.text;
+      }
+      return;
+    }
+    if (item.type === CODEX_ITEM_TYPES.ERROR) {
+      if (typeof item.message === "string" && item.message.length > 0) {
+        this.state.errors.push(item.message);
+      }
+      return;
+    }
+    if (item.type === CODEX_ITEM_TYPES.COMMAND_EXECUTION) {
+      const id = item.id;
+      if (typeof id === "string" && id && this.state.tools.includes(id)) {} else {
+        this.state.tools.push(typeof id === "string" ? id : `cmd-${this.state.tools.length}`);
+      }
+      if (completed && this.state.createdTask === undefined) {
+        this.state.createdTask = extractCreatedTask(item.aggregated_output);
+      }
+      return;
+    }
+    if (item.type === CODEX_ITEM_TYPES.MCP_TOOL_CALL) {
+      const id = item.id;
+      const seen = typeof id === "string" && id && this.mcpSeen.has(id);
+      if (typeof id === "string" && id)
+        this.mcpSeen.add(id);
+      if (!seen) {
+        const name = typeof item.tool === "string" && item.tool ? item.tool : undefined;
+        this.state.tools.push(name ?? (typeof id === "string" && id ? id : `mcp-${this.state.tools.length}`));
+      }
+      if (completed && this.state.createdTask === undefined && isSuccessfulMcpStatus(item.status) && typeof item.tool === "string" && isCreationTool(item.tool) && isRoyTaskServer(item.server)) {
+        const text = mcpResultText(item.result);
+        if (text !== undefined) {
+          const args = item.arguments;
+          const hint = typeof args?.title === "string" ? args.title : undefined;
+          this.state.createdTask = extractCreatedTask(text, hint);
+        }
+      }
+      return;
+    }
+  }
+  toResult() {
+    const result = {
+      text: this.state.text ?? ""
+    };
+    if (this.state.sessionId)
+      result.sessionId = this.state.sessionId;
+    if (this.state.tools.length > 0) {
+      result.toolCalls = this.state.tools.length;
+      result.tools = [...this.state.tools];
+    }
+    if (this.state.createdTask)
+      result.createdTask = this.state.createdTask;
+    return result;
+  }
+}
+
+// ../../../packages/cli/src/daemon/runners/codex-sessions.ts
+function createCodexSessionRegistry() {
+  const sessions = new Map;
+  return {
+    get: (key) => sessions.get(key),
+    set: (key, id) => {
+      sessions.set(key, id);
+    },
+    delete: (key) => {
+      sessions.delete(key);
+    },
+    size: () => sessions.size
+  };
+}
+
 // ../../../packages/cli/src/daemon/runners/claude.ts
 import { createHash } from "node:crypto";
 import { appendFileSync, existsSync } from "node:fs";
@@ -3230,7 +3270,7 @@ function formatStatus(report) {
 }
 
 // ../../../packages/cli/src/main.ts
-var VERSION = "0.7.4";
+var VERSION = "0.7.6";
 async function main(argv) {
   const parsed = parseArgs(argv);
   switch (parsed.command) {