@rallycry/conveyor-agent 10.3.1 → 10.4.4

package/dist/{chunk-HAH2S5AD.js → chunk-VCXKVINO.js}

@@ -719,13 +719,16 @@ var AgentConnection = class _AgentConnection {
     };
   }
   // ── Convenience methods (thin wrappers around call / emit) ─────────
-  async emitStatus(status, reason) {
+  async emitStatus(status, reason, questionText) {
     this.lastEmittedStatus = status;
     await this.flushEvents();
     const payload = {
       sessionId: this.config.sessionId,
       status,
-      ...reason ? { reason } : {}
+      ...reason ? { reason } : {},
+      // Only sent with a pending TUI questionnaire (reason "user_question") so
+      // the server can surface the real question text in the notification.
+      ...questionText ? { questionText } : {}
     };
     const AWAIT_STATUSES = ["idle", "waiting_for_input", "connected"];
     if (AWAIT_STATUSES.includes(status)) {
@@ -2046,7 +2049,14 @@ var ReportAgentStatusRequestSchema = z.object({
   sessionId: z.string(),
   status: z.string(),
   /** Why the agent reports this status (e.g. "user_question" while an AskUserQuestion questionnaire is pending in the TUI). */
-  reason: z.string().optional()
+  reason: z.string().optional(),
+  /**
+   * The pending question text, sent only alongside `reason: "user_question"`
+   * so the server can surface it in the user-question notification body (and
+   * thus the Attention feed) instead of a generic string. Optional: older
+   * agents omit it and the server falls back to the generic wording.
+   */
+  questionText: z.string().optional()
 });
 var NotifyAgentVersionRequestSchema = z.object({
   sessionId: z.string(),
@@ -2069,7 +2079,10 @@ var CreateSubtaskRequestSchema = z.object({
   plan: z.string().optional(),
   storyPointValue: z.number().int().positive().optional(),
   ordinal: z.number().int().nonnegative().optional(),
-  followParentStatus: z.boolean().optional()
+  followParentStatus: z.boolean().optional(),
+  /** Sibling subtask ids or slugs this subtask blocks on (explicit dependency
+   * metadata — preferred over encoding order in plan text / ordinal). */
+  dependsOn: z.array(z.string().min(1)).max(32).optional()
 });
 var UpdateSubtaskRequestSchema = z.object({
   sessionId: z.string(),
@@ -2079,7 +2092,10 @@ var UpdateSubtaskRequestSchema = z.object({
   plan: z.string().optional(),
   status: z.string().optional(),
   storyPointValue: z.number().int().positive().optional(),
-  followParentStatus: z.boolean().optional()
+  followParentStatus: z.boolean().optional(),
+  /** Replace this subtask's dependency edges with these sibling ids/slugs.
+   * Empty array clears all. Omit to leave dependencies unchanged. */
+  dependsOn: z.array(z.string().min(1)).max(32).optional()
 });
 var DeleteSubtaskRequestSchema = z.object({
   sessionId: z.string(),
@@ -2500,6 +2516,9 @@ var CreateProjectSubtaskRequestSchema = z2.object({
   ordinal: z2.number().int().nonnegative().optional(),
   storyPointValue: z2.number().int().positive().optional(),
   followParentStatus: z2.boolean().optional(),
+  /** Sibling subtask ids or slugs this subtask blocks on (explicit dependency
+   * metadata — preferred over encoding order in plan text / ordinal). */
+  dependsOn: z2.array(z2.string().min(1)).max(32).optional(),
   requestingUserId: z2.string().optional()
 });
 var UpdateProjectSubtaskRequestSchema = z2.object({
@@ -2962,7 +2981,8 @@ function defineTool(name, description, schema, handler, options) {
     description,
     schema,
     handler,
-    annotations: options?.annotations
+    annotations: options?.annotations,
+    strict: options?.strict
   };
 }
 
@@ -3759,6 +3779,7 @@ var PtyOutputCoalescer = class {
 
 // src/harness/pty/tool-server.ts
 import { createServer as createServer2 } from "http";
+import { z as z3 } from "zod";
 import { writeFile as writeFile3 } from "fs/promises";
 import { join as join2 } from "path";
 import { randomBytes } from "crypto";
@@ -3813,9 +3834,14 @@ var PtyToolServer = class {
   /** Fresh McpServer with every tool registered — one per MCP session. */
   buildMcpServer() {
     const mcp = new McpServer({ name: this.name, version: "1.0.0" });
-    const register = mcp.tool.bind(mcp);
+    const register = mcp.registerTool.bind(mcp);
     for (const tool2 of this.tools) {
-      register(tool2.name, tool2.description, tool2.schema, (args) => tool2.handler(args));
+      const inputSchema = tool2.strict ? z3.strictObject(tool2.schema) : tool2.schema;
+      register(
+        tool2.name,
+        { description: tool2.description, inputSchema },
+        (args) => tool2.handler(args)
+      );
     }
     return mcp;
   }
@@ -3943,6 +3969,9 @@ async function startToolServers(mcpServers, tempDir) {
 }
 
 // src/harness/pty/spawn-args.ts
+function resolveClaudeBinary() {
+  return process.env.CONVEYOR_CLAUDE_BIN ?? "claude";
+}
 function buildSpawnArgs(input) {
   const args = [];
   if (input.resume) {
@@ -4232,9 +4261,9 @@ function extractSpawn(mod) {
 }
 async function loadPtySpawn() {
   const mod = await import("node-pty");
-  const spawn2 = extractSpawn(mod);
-  if (!spawn2) throw new Error("node-pty: spawn export not found");
-  return spawn2;
+  const spawn3 = extractSpawn(mod);
+  if (!spawn3) throw new Error("node-pty: spawn export not found");
+  return spawn3;
 }
 function inheritedEnv(socketPath) {
   const env = {};
@@ -4733,8 +4762,8 @@ var PtySession = class {
       // server doesn't load).
       ...this.mcpConfigPath ? { mcpConfigPath: this.mcpConfigPath } : {}
     });
-    const spawn2 = await loadPtySpawn();
-    const pty = spawn2(spec.file, spec.args, {
+    const spawn3 = await loadPtySpawn();
+    const pty = spawn3(spec.file, spec.args, {
       name: "xterm-color",
       cols: this.cols,
       rows: this.rows,
@@ -5188,7 +5217,7 @@ function buildPackRunnerSystemPrompt(context, config, setupLog) {
     ``,
     `## Important Rules`,
     `- When dependencies are set on children, use them to determine execution order. Fire all ready tasks in parallel (up to the PACK_CHILD_LIMIT backpressure \u2014 see the loop above).`,
-    `- When NO dependencies are set on any children, fall back to ordinal order (one at a time). This preserves backward compatibility.`,
+    `- Dependencies are explicit card metadata (set at creation via create_subtask's dependsOn, or rewired with update_subtask). Prefer them over reading order out of plan text. When NO dependencies are set on any children, fall back to ordinal order (one at a time) \u2014 this is legacy behavior; set dependsOn when a child truly blocks on another.`,
     `- After firing builds OR when waiting on CI, explicitly state you are going idle. Go idle when waiting \u2014 the system wakes you (or relaunches this environment) when a child changes status.`,
     `- Do NOT attempt to write code yourself. Your role is coordination only.`,
     `- If a child is stuck in "InProgress" for an unusually long time, use get_execution_logs(childTaskId) to check its logs and escalate to the team if it appears stuck.`,
@@ -5809,7 +5838,8 @@ function buildDiscoveryPrompt(context, runnerMode) {
       `### Parent Task Coordination`,
       `You are a parent task with child tasks. Focus on breaking work into child tasks with detailed plans, not planning implementation for yourself.`,
       `- Use \`list_subtasks\` to review existing children. Create or update child tasks using \`create_subtask\` / \`update_subtask\`.`,
-      `- Each child task should be a self-contained unit of work with a clear plan.`
+      `- Each child task should be a self-contained unit of work with a clear plan.`,
+      `- Set ordering as explicit **card metadata**, not prose: pass \`dependsOn\` (sibling ids/slugs) on \`create_subtask\` for any child that blocks on another. Leave independent children with no dependencies so the pack runner fans them out in parallel. Do NOT encode "do X after Y" only in the plan text \u2014 the runner schedules off dependsOn, not narrative.`
     ] : [
       `### Self-Update vs Subtasks`,
       `- If the work fits in a single task (1-3 SP), update YOUR OWN plan and properties \u2014 do not create subtasks`,
@@ -5823,6 +5853,7 @@ function buildDiscoveryPrompt(context, runnerMode) {
     `- Reference existing implementations when relevant (e.g., "follow the pattern in src/services/foo.ts")`,
     `- Include testing requirements and acceptance criteria`,
     `- Set \`storyPointValue\` based on estimated complexity`,
+    `- Express cross-child ordering as \`dependsOn\` (sibling ids/slugs) on \`create_subtask\` \u2014 explicit dependency metadata, not "after task 2" phrasing in the plan. Independent children get no dependencies so they run in parallel.`,
     ``,
     `### Plan Verification Requirements`,
     `Every plan MUST include a **Testing / Verification** section so the build agent has a clear definition of "done". Enumerate:`,
@@ -5869,6 +5900,7 @@ function buildAutoPrompt(context, runnerMode) {
     `- Reference existing implementations when relevant`,
     `- Include testing requirements and acceptance criteria`,
     `- Set \`storyPointValue\` based on estimated complexity`,
+    `- Express cross-child ordering as \`dependsOn\` (sibling ids/slugs) on \`create_subtask\` \u2014 explicit dependency metadata, not "after task 2" phrasing in the plan. Independent children get no dependencies so they run in parallel.`,
     ``,
     ...buildPlanCitationFormat(),
     ``,
@@ -5877,7 +5909,8 @@ function buildAutoPrompt(context, runnerMode) {
       `### Parent Task Guidance`,
       `You are a parent task. Your plan should define child tasks with detailed plans, not direct implementation steps.`,
       `After ExitPlanMode, you'll transition to Review mode to coordinate child task execution.`,
-      `Child task status lifecycle: Open \u2192 InProgress \u2192 ReviewPR \u2192 ReviewDev \u2192 Complete.`
+      `Child task status lifecycle: Open \u2192 InProgress \u2192 ReviewPR \u2192 ReviewDev \u2192 Complete.`,
+      `Set child ordering with \`dependsOn\` (sibling ids/slugs) on \`create_subtask\` \u2014 explicit metadata the pack runner schedules off, not order described in plan text. Independent children get none and run in parallel.`
     ] : [],
     ``,
     `### Autonomous Guidelines:`,
@@ -6078,7 +6111,9 @@ Environment (fully ready \u2014 do NOT verify or set up):`,
     `- All dependencies are installed, database is migrated, and the dev server is running.`,
     `- Git is configured. Commit and push directly to this branch.`,
     `- The shell cwd resets between Bash calls \u2014 always use absolute paths (or \`git -C\`, \`bun run --cwd\`); never spend a tool call on a bare \`cd\`.`,
-    `- When a build/lint/test run fails, capture its output to a file once and grep the file \u2014 never re-run the suite just to re-filter the same output. Don't poll background jobs with sleep/ps/tail loops.`,
+    `- When a build/lint/test run fails, capture its output to a file once and grep the file \u2014 never re-run the suite just to re-filter the same output.`,
+    `- Waiting on long-running commands: if a gate finishes in under ~2 minutes, run it in the foreground with a timeout. Anything longer: launch it with run_in_background and STOP \u2014 a completion notification arrives when it finishes. Never busy-wait with sleep/pgrep/tail loops (foreground shell commands are killed at ~2 minutes), and never re-run the suite to escape a wait that looks stalled.`,
+    `- Deferred tools have no schema loaded until fetched \u2014 before first use, load it via ToolSearch (query "select:<ToolName>", e.g. "select:Monitor"); never guess a deferred tool's parameters.`,
     `- The \`gh\` CLI may not be installed \u2014 use the mcp__conveyor__* tools for PR and task state.`,
     `- Read a file before your first Write/Edit to it, and batch multiple changes to the same file into a single call instead of many sequential edits.`,
     `
@@ -6508,7 +6543,7 @@ async function buildInitialPrompt(mode, context, isAuto, agentMode) {
 }
 
 // src/tools/task-context-tools.ts
-import { z as z3 } from "zod";
+import { z as z4 } from "zod";
 
 // src/tools/helpers.ts
 function textResult(text) {
@@ -6542,8 +6577,8 @@ function buildReadTaskChatTool(connection) {
     "read_task_chat",
     "Read recent human/user chat messages for a task. Omit task_id for the current task; pass a child ID for a child's chat. For agent logs use get_execution_logs.",
     {
-      limit: z3.number().optional().describe("Number of recent messages to fetch (default 20)"),
-      task_id: z3.string().optional().describe("Child task ID to read chat from. Omit to read the current task's chat.")
+      limit: z4.number().optional().describe("Number of recent messages to fetch (default 20)"),
+      task_id: z4.string().optional().describe("Child task ID to read chat from. Omit to read the current task's chat.")
     },
     async ({ limit, task_id }) => {
       try {
@@ -6587,7 +6622,7 @@ function buildGetTaskTool(connection) {
     "get_task",
     "Look up any task by slug or ID. Returns JSON with id, slug, title, description, plan, status, branch, githubPRNumber, githubPRUrl, storyPoints. For children use list_subtasks.",
     {
-      slug_or_id: z3.string().describe("The task slug (e.g. 'my-task') or CUID")
+      slug_or_id: z4.string().describe("The task slug (e.g. 'my-task') or CUID")
     },
     async ({ slug_or_id }) => {
       try {
@@ -6610,9 +6645,9 @@ function buildGetExecutionLogsTool(connection) {
     "get_execution_logs",
     "Read CLI execution logs \u2014 agent reasoning, tool calls, and setup/dev-server output. Filter via source='agent' or 'application'. For human chat use read_task_chat.",
     {
-      task_id: z3.string().optional().describe("Task ID or slug. Omit to read logs from the current task."),
-      source: z3.enum(["agent", "application"]).optional().describe("Filter by log source. Omit for all logs."),
-      limit: z3.number().optional().describe("Max number of log entries to return (default 50, max 500).")
+      task_id: z4.string().optional().describe("Task ID or slug. Omit to read logs from the current task."),
+      source: z4.enum(["agent", "application"]).optional().describe("Filter by log source. Omit for all logs."),
+      limit: z4.number().optional().describe("Max number of log entries to return (default 50, max 500).")
     },
     async ({ task_id, source, limit }) => {
       try {
@@ -6672,7 +6707,7 @@ function buildGetAttachmentTool(connection) {
   return defineTool(
     "get_attachment",
     "Fetch one task file's content plus metadata by file ID. Call list_task_files first to discover IDs and check sizes \u2014 large binaries may be truncated by the service's size limit.",
-    { fileId: z3.string().describe("The file ID to retrieve") },
+    { fileId: z4.string().describe("The file ID to retrieve") },
     async ({ fileId }) => {
       try {
         const file = await connection.call("getTaskFile", {
@@ -6710,7 +6745,7 @@ function buildTaskContextTools(connection) {
 }
 
 // src/tools/dependency-suggestion-tools.ts
-import { z as z4 } from "zod";
+import { z as z5 } from "zod";
 function buildGetDependenciesTool(connection) {
   return defineTool(
     "get_dependencies",
@@ -6736,10 +6771,10 @@ function buildGetSuggestionsTool(connection) {
     "get_suggestions",
     "List project suggestions sorted by vote score. Filter by status or cap with limit (default 20). Suggestions are project-level ideas, not tasks \u2014 use get_task for tasks.",
     {
-      status: z4.string().optional().describe(
+      status: z5.string().optional().describe(
         "Filter by status: Planning, Open, InProgress, ReviewPR, ReviewDev, ReviewLive, Complete, Cancelled"
       ),
-      limit: z4.number().int().min(1).max(100).optional().describe("Max results (default 20)")
+      limit: z5.number().int().min(1).max(100).optional().describe("Max results (default 20)")
     },
     async ({ status, limit }) => {
       try {
@@ -6763,14 +6798,14 @@ function buildGetSuggestionsTool(connection) {
 }
 
 // src/tools/mutation-tools.ts
-import { z as z5 } from "zod";
+import { z as z6 } from "zod";
 function buildPostToChatTool(connection) {
   return defineTool(
     "post_to_chat",
     "Post a message to the task chat for the team to see. Your turn output is NOT shown in chat, so this is the only way the team sees your status, summaries, and questions. Omit task_id to post to the current task's chat; pass a child's ID to message its chat.",
     {
-      message: z5.string().describe("The message to post to the team"),
-      task_id: z5.string().optional().describe("Child task ID to post to. Omit to post to the current task's chat.")
+      message: z6.string().describe("The message to post to the team"),
+      task_id: z6.string().optional().describe("Child task ID to post to. Omit to post to the current task's chat.")
     },
     async ({ message, task_id }) => {
       try {
@@ -6808,8 +6843,8 @@ function buildForceUpdateTaskStatusTool(connection) {
     "force_update_task_status",
     "EMERGENCY ONLY: force-override a task's Kanban status. Use when an automatic transition failed and the task is wedged. Normal flow transitions status automatically.",
     {
-      status: z5.enum(["InProgress", "ReviewPR", "ReviewDev", "Complete"]).describe("The new status for the task"),
-      task_id: z5.string().optional().describe("Child task ID to update. Omit to update the current task.")
+      status: z6.enum(["InProgress", "ReviewPR", "ReviewDev", "Complete"]).describe("The new status for the task"),
+      task_id: z6.string().optional().describe("Child task ID to update. Omit to update the current task.")
     },
     async ({ status, task_id }) => {
       try {
@@ -6840,18 +6875,18 @@ function buildCreatePullRequestTool(connection, config) {
     "create_pull_request",
     "Create a GitHub PR for this task. Auto-stages, commits (commitMessage or title default), pushes to origin, then opens the PR. Always use this instead of gh CLI or raw git.",
     {
-      title: z5.string().describe("The PR title"),
-      body: z5.string().describe("The PR description/body in markdown"),
-      branch: z5.string().optional().describe(
+      title: z6.string().describe("The PR title"),
+      body: z6.string().describe("The PR description/body in markdown"),
+      branch: z6.string().optional().describe(
         "The head branch name for the PR. If the task doesn't have a branch set, this will be used. Defaults to the task's existing branch."
       ),
-      baseBranch: z5.string().optional().describe(
+      baseBranch: z6.string().optional().describe(
         "The base branch to target for the PR (e.g. 'main', 'develop'). Defaults to the project's configured dev branch."
       ),
-      commitMessage: z5.string().optional().describe(
+      commitMessage: z6.string().optional().describe(
         "Commit message for staging uncommitted changes. If not provided, a default message based on the PR title will be used."
       ),
-      skipVerify: z5.boolean().optional().describe(
+      skipVerify: z6.boolean().optional().describe(
         "Controls the local pre-push quality gate (lint/typecheck/test). Defaults to true (--no-verify): the push skips the local gate because you should run gates yourself before opening the PR and CI re-runs them on the resulting PR. Running the full gate synchronously during the push would block the agent's event loop long enough to drop the Conveyor socket connection. Pass false to force the local pre-push hook to run."
       )
     },
@@ -6933,7 +6968,7 @@ function buildAddDependencyTool(connection) {
     "add_dependency",
     "Add a blocking dependency \u2014 this task cannot start until the named task is merged to dev. For post-task follow-ups use create_follow_up_task instead.",
     {
-      depends_on_slug_or_id: z5.string().describe("Slug or ID of the task this task depends on")
+      depends_on_slug_or_id: z6.string().describe("Slug or ID of the task this task depends on")
     },
     async ({ depends_on_slug_or_id }) => {
       try {
@@ -6955,7 +6990,7 @@ function buildRemoveDependencyTool(connection) {
     "remove_dependency",
     "Remove a previously added dependency from this task. When to use: the dependency was added in error or is no longer relevant. Returns: confirmation string.",
     {
-      depends_on_slug_or_id: z5.string().describe("Slug or ID of the task to remove as dependency")
+      depends_on_slug_or_id: z6.string().describe("Slug or ID of the task to remove as dependency")
     },
     async ({ depends_on_slug_or_id }) => {
       try {
@@ -6977,10 +7012,10 @@ function buildCreateFollowUpTaskTool(connection) {
     "create_follow_up_task",
     "Create a follow-up task that depends on the current task. Use for out-of-scope work or cleanup that should land after this task merges. For blockers use add_dependency.",
     {
-      title: z5.string().describe("Follow-up task title"),
-      description: z5.string().optional().describe("Brief description of the follow-up work"),
-      plan: z5.string().optional().describe("Implementation plan if known"),
-      story_point_value: z5.number().optional().describe("Story point estimate (1=Common, 2=Magic, 3=Rare, 5=Unique)")
+      title: z6.string().describe("Follow-up task title"),
+      description: z6.string().optional().describe("Brief description of the follow-up work"),
+      plan: z6.string().optional().describe("Implementation plan if known"),
+      story_point_value: z6.number().optional().describe("Story point estimate (1=Common, 2=Magic, 3=Rare, 5=Unique)")
     },
     async ({ title, description, plan, story_point_value }) => {
       try {
@@ -7007,11 +7042,11 @@ function buildCreateSuggestionTool(connection) {
     "create_suggestion",
     "Suggest a feature, improvement, rule, or idea for the project. Duplicates are deduped and your upvote is recorded. For actionable work on this task open a follow-up task.",
     {
-      title: z5.string().describe("Short title for the suggestion"),
-      description: z5.string().optional().describe(
+      title: z6.string().describe("Short title for the suggestion"),
+      description: z6.string().optional().describe(
         "1-2 sentence description of what should change and why. Keep concise and project-focused."
       ),
-      tag_names: z5.array(z5.string()).optional().describe("Tag names to categorize the suggestion")
+      tag_names: z6.array(z6.string()).optional().describe("Tag names to categorize the suggestion")
     },
     async ({ title, description, tag_names }) => {
       try {
@@ -7040,8 +7075,8 @@ function buildVoteSuggestionTool(connection) {
     "vote_suggestion",
     "Vote +1 or -1 on a project suggestion. Use to express support or disagreement with a specific suggestion returned by get_suggestions.",
     {
-      suggestion_id: z5.string().describe("The suggestion ID to vote on"),
-      value: z5.number().refine((v) => v === 1 || v === -1, { message: "Value must be 1 or -1" }).describe("+1 to upvote, -1 to downvote")
+      suggestion_id: z6.string().describe("The suggestion ID to vote on"),
+      value: z6.number().refine((v) => v === 1 || v === -1, { message: "Value must be 1 or -1" }).describe("+1 to upvote, -1 to downvote")
     },
     async ({ suggestion_id, value }) => {
       try {
@@ -7074,7 +7109,7 @@ function buildMutationTools(connection, config) {
 // src/tools/attachment-tools.ts
 import { readFile as readFile3, stat as stat5 } from "fs/promises";
 import { basename, extname, isAbsolute, join as join5 } from "path";
-import { z as z6 } from "zod";
+import { z as z7 } from "zod";
 var IMAGE_MIME_BY_EXT = {
   ".png": "image/png",
   ".jpg": "image/jpeg",
@@ -7087,8 +7122,8 @@ function buildUploadAttachmentTool(connection, config) {
     "upload_attachment",
     "Upload an image file (e.g. a Playwright screenshot) as a task attachment AND post it to the task chat in one step \u2014 no follow-up post_to_chat call needed. Supports png/jpg/gif/webp.",
     {
-      path: z6.string().describe("Path to the image file \u2014 absolute, or relative to the workspace root"),
-      title: z6.string().optional().describe("Short caption posted with the image (defaults to the file name)")
+      path: z7.string().describe("Path to the image file \u2014 absolute, or relative to the workspace root"),
+      title: z7.string().optional().describe("Short caption posted with the image (defaults to the file name)")
     },
     async ({ path: path4, title }) => {
       try {
@@ -7144,7 +7179,7 @@ function buildUploadAttachmentTool(connection, config) {
 }
 
 // src/tools/checklist-tools.ts
-import { z as z7 } from "zod";
+import { z as z8 } from "zod";
 function buildListManualTestsTool(connection) {
   return defineTool(
     "list_manual_tests",
@@ -7196,8 +7231,8 @@ function buildQueryManualTestsTool(connection) {
     "query_manual_tests",
     "Query manual tests across many tasks in this project, grouped by task. Filter by card status (ReviewDev, ReviewLive, Complete, ...) and/or test status (open | approved | rejected). Use to answer 'show all OPEN manual tests in ReviewDev' or 'show all REJECTED manual tests with the failing reason'. With no filters it defaults to the needs-attention view: open+rejected tests on ReviewDev/ReviewLive cards.",
     {
-      cardStatuses: z7.array(z7.string()).optional().describe('Filter tasks by card status, e.g. ["ReviewDev", "ReviewLive"]'),
-      testStatuses: z7.array(z7.enum(["open", "approved", "rejected"])).optional().describe("Filter tests by status: open | approved | rejected")
+      cardStatuses: z8.array(z8.string()).optional().describe('Filter tasks by card status, e.g. ["ReviewDev", "ReviewLive"]'),
+      testStatuses: z8.array(z8.enum(["open", "approved", "rejected"])).optional().describe("Filter tests by status: open | approved | rejected")
     },
     async ({ cardStatuses, testStatuses }) => {
       try {
@@ -7221,7 +7256,7 @@ function buildSetManualTestsTool(connection) {
     "set_manual_tests",
     "Add manual test steps to the task checklist. Existing items with the same title are automatically skipped (deduplication). Use to record specific manual verification steps that reviewers should follow when testing this PR.",
     {
-      items: z7.array(z7.object({ title: z7.string().min(1).describe("A concise, actionable test step") })).min(1).describe("List of manual test steps to add")
+      items: z8.array(z8.object({ title: z8.string().min(1).describe("A concise, actionable test step") })).min(1).describe("List of manual test steps to add")
     },
     async ({ items }) => {
       try {
@@ -7244,8 +7279,8 @@ function buildEditManualTestTool(connection) {
     "edit_manual_test",
     "Rename an existing manual test step. Identify the test by its current title (case-insensitive); pass the new title to replace it. Use to correct or refine a recorded manual verification step.",
     {
-      title: z7.string().min(1).describe("The current title of the manual test to edit"),
-      newTitle: z7.string().min(1).describe("The new title for the manual test")
+      title: z8.string().min(1).describe("The current title of the manual test to edit"),
+      newTitle: z8.string().min(1).describe("The new title for the manual test")
     },
     async ({ title, newTitle }) => {
       try {
@@ -7267,7 +7302,7 @@ function buildRemoveManualTestTool(connection) {
     "remove_manual_test",
     "Remove an existing manual test step from the task checklist. Identify the test by its title (case-insensitive). Use to delete a stale or incorrect manual verification step.",
     {
-      title: z7.string().min(1).describe("The title of the manual test to remove")
+      title: z8.string().min(1).describe("The title of the manual test to remove")
     },
     async ({ title }) => {
       try {
@@ -7288,7 +7323,7 @@ function buildApproveManualTestTool(connection) {
     "approve_manual_test",
     "Sign off on (approve) a manual test step on behalf of your authenticated user. Identify the test by its title (case-insensitive). Use after you have verified the step passes.",
     {
-      title: z7.string().min(1).describe("The title of the manual test to approve")
+      title: z8.string().min(1).describe("The title of the manual test to approve")
     },
     async ({ title }) => {
       try {
@@ -7309,8 +7344,8 @@ function buildRejectManualTestTool(connection) {
     "reject_manual_test",
     "Flag an issue with (reject) a manual test step on behalf of your authenticated user, recording the reason. Identify the test by its title (case-insensitive). Use when the step fails verification.",
     {
-      title: z7.string().min(1).describe("The title of the manual test to reject"),
-      reason: z7.string().min(1).max(2e3).describe("Why the test failed \u2014 what went wrong, shown to the team")
+      title: z8.string().min(1).describe("The title of the manual test to reject"),
+      reason: z8.string().min(1).max(2e3).describe("Why the test failed \u2014 what went wrong, shown to the team")
     },
     async ({ title, reason }) => {
       try {
@@ -7347,16 +7382,17 @@ function buildCommonTools(connection, config) {
 }
 
 // src/tools/pm-tools.ts
-import { z as z8 } from "zod";
+import { z as z9 } from "zod";
 var SP_DESCRIPTION = "Story point value (1=Common, 2=Magic, 3=Rare, 5=Unique)";
 var FOLLOW_PARENT_STATUS_DESCRIPTION = "Child mirrors the parent task's status automatically \u2014 for subtasks that ship on the parent's branch/PR with no build or PR of their own. Manual status writes on a follower stick only until the parent's next transition.";
+var DEPENDS_ON_DESCRIPTION = "Sibling subtask ids or slugs this subtask blocks on (it won't start until they merge to dev). Set explicit dependency metadata here instead of describing order in the plan text \u2014 the pack runner schedules children off these edges. Omit / leave empty for independent children so they run in parallel.";
 function buildUpdateTaskTool(connection) {
   return defineTool(
     "update_task_plan",
     "Save the finalized plan and/or description to the current task. Use in Plan mode after alignment. For children use update_subtask; for title/tags/PR use update_task_properties.",
     {
-      plan: z8.string().optional().describe("The task plan in markdown"),
-      description: z8.string().optional().describe("Updated task description")
+      plan: z9.string().optional().describe("The task plan in markdown"),
+      description: z9.string().optional().describe("Updated task description")
     },
     async ({ plan, description }) => {
       try {
@@ -7377,14 +7413,23 @@ function buildCreateSubtaskTool(connection) {
     "create_subtask",
     "Create a subtask under the current parent task. Use when breaking a complex parent into smaller pieces during planning. For post-task follow-ups use create_follow_up_task.",
     {
-      title: z8.string().describe("Subtask title"),
-      description: z8.string().optional().describe("Brief description"),
-      plan: z8.string().optional().describe("Implementation plan in markdown"),
-      ordinal: z8.number().optional().describe("Step/order number (0-based)"),
-      storyPointValue: z8.number().optional().describe(SP_DESCRIPTION),
-      followParentStatus: z8.boolean().optional().describe(FOLLOW_PARENT_STATUS_DESCRIPTION)
+      title: z9.string().describe("Subtask title"),
+      description: z9.string().optional().describe("Brief description"),
+      plan: z9.string().optional().describe("Implementation plan in markdown"),
+      ordinal: z9.number().optional().describe("Step/order number (0-based)"),
+      storyPointValue: z9.number().optional().describe(SP_DESCRIPTION),
+      followParentStatus: z9.boolean().optional().describe(FOLLOW_PARENT_STATUS_DESCRIPTION),
+      dependsOn: z9.array(z9.string()).optional().describe(DEPENDS_ON_DESCRIPTION)
     },
-    async ({ title, description, plan, ordinal, storyPointValue, followParentStatus }) => {
+    async ({
+      title,
+      description,
+      plan,
+      ordinal,
+      storyPointValue,
+      followParentStatus,
+      dependsOn
+    }) => {
       try {
         const result = await connection.call("createSubtask", {
           sessionId: connection.sessionId,
@@ -7393,9 +7438,10 @@ function buildCreateSubtaskTool(connection) {
           ...plan !== void 0 && { plan },
           ...storyPointValue !== void 0 && { storyPointValue },
           ...ordinal !== void 0 && { ordinal },
-          ...followParentStatus !== void 0 && { followParentStatus }
+          ...followParentStatus !== void 0 && { followParentStatus },
+          ...dependsOn !== void 0 && { dependsOn }
         });
-        return textResult(`Subtask created with ID: ${result.id}`);
+        return textResult(`Subtask created with ID: ${result.id} (slug: ${result.slug})`);
       } catch (error) {
         return textResult(
           `Failed to create subtask: ${error instanceof Error ? error.message : "Unknown error"}`
@@ -7407,17 +7453,28 @@ function buildCreateSubtaskTool(connection) {
 function buildUpdateSubtaskTool(connection) {
   return defineTool(
     "update_subtask",
-    "Update an existing subtask's fields (title, description, plan, ordinal, storyPointValue). Use when refining a child's plan or reordering. For the current task use update_task_plan.",
+    "Update an existing subtask's fields (title, description, plan, ordinal, storyPointValue, dependsOn). Use when refining a child's plan, reordering, or rewiring dependencies. For the current task use update_task_plan.",
     {
-      subtaskId: z8.string().describe("The subtask ID to update"),
-      title: z8.string().optional(),
-      description: z8.string().optional(),
-      plan: z8.string().optional(),
-      ordinal: z8.number().optional(),
-      storyPointValue: z8.number().optional().describe(SP_DESCRIPTION),
-      followParentStatus: z8.boolean().optional().describe(FOLLOW_PARENT_STATUS_DESCRIPTION)
+      subtaskId: z9.string().describe("The subtask ID to update"),
+      title: z9.string().optional(),
+      description: z9.string().optional(),
+      plan: z9.string().optional(),
+      ordinal: z9.number().optional(),
+      storyPointValue: z9.number().optional().describe(SP_DESCRIPTION),
+      followParentStatus: z9.boolean().optional().describe(FOLLOW_PARENT_STATUS_DESCRIPTION),
+      dependsOn: z9.array(z9.string()).optional().describe(
+        `${DEPENDS_ON_DESCRIPTION} Replaces the full dependency set \u2014 pass [] to clear all, omit to leave unchanged.`
+      )
     },
-    async ({ subtaskId, title, description, plan, storyPointValue, followParentStatus }) => {
+    async ({
+      subtaskId,
+      title,
+      description,
+      plan,
+      storyPointValue,
+      followParentStatus,
+      dependsOn
+    }) => {
       try {
         await connection.call("updateSubtask", {
           sessionId: connection.sessionId,
@@ -7426,7 +7483,8 @@ function buildUpdateSubtaskTool(connection) {
           ...description !== void 0 && { description },
           ...plan !== void 0 && { plan },
           ...storyPointValue !== void 0 && { storyPointValue },
-          ...followParentStatus !== void 0 && { followParentStatus }
+          ...followParentStatus !== void 0 && { followParentStatus },
+          ...dependsOn !== void 0 && { dependsOn }
         });
         return textResult("Subtask updated.");
       } catch (error) {
@@ -7439,7 +7497,7 @@ function buildDeleteSubtaskTool(connection) {
   return defineTool(
     "delete_subtask",
     "Delete a subtask by id. When to use: a subtask was created in error or is no longer needed. Returns: confirmation string.",
-    { subtaskId: z8.string().describe("The subtask ID to delete") },
+    { subtaskId: z9.string().describe("The subtask ID to delete") },
     async ({ subtaskId }) => {
       try {
         await connection.call("deleteSubtask", {
@@ -7477,7 +7535,7 @@ function buildPackTools(connection) {
       "start_child_cloud_build",
       "Start a cloud build (codespace) for a child task. Preconditions: child is in Open status, has a story point value, and has an agent assigned.",
       {
-        childTaskId: z8.string().describe("The child task ID to start a cloud build for")
+        childTaskId: z9.string().describe("The child task ID to start a cloud build for")
       },
       async ({ childTaskId }) => {
         try {
@@ -7497,7 +7555,7 @@ function buildPackTools(connection) {
       "stop_child_build",
       "Send a graceful stop signal to a running child build's agent. Not a force-kill \u2014 the agent may take a moment to wind down.",
       {
-        childTaskId: z8.string().describe("The child task ID whose build should be stopped")
+        childTaskId: z9.string().describe("The child task ID whose build should be stopped")
       },
       async ({ childTaskId }) => {
         try {
@@ -7517,7 +7575,7 @@ function buildPackTools(connection) {
       "approve_and_merge_pr",
       "Approve and merge a child task's PR. Preconditions: child in ReviewPR. Returns { merged }: true = merged (status\u2192ReviewDev); false = automerge queued, wait for ReviewDev.",
       {
-        childTaskId: z8.string().describe("The child task ID whose PR should be approved and merged")
+        childTaskId: z9.string().describe("The child task ID whose PR should be approved and merged")
       },
       async ({ childTaskId }) => {
         try {
@@ -7555,22 +7613,29 @@ function buildPmTools(connection, options) {
 }
 
 // src/tools/discovery-tools.ts
-import { z as z9 } from "zod";
-var SP_DESCRIPTION2 = "Story point value (1=Common, 2=Magic, 3=Rare, 5=Unique)";
+import { z as z10 } from "zod";
+var SP_DESCRIPTION2 = "Story point value (1=Common, 2=Magic, 3=Rare, 5=Unique). The key is 'storyPointValue' \u2014 not 'storyPoints'.";
+var VALID_PROPERTY_KEYS = "title, storyPointValue, tagNames, githubPRUrl, githubBranch";
 function buildDiscoveryTools(connection) {
   return [
     defineTool(
       "update_task_properties",
-      "Set one or more task properties in a single call. All fields are optional \u2014 only include the fields you want to update.",
+      "Set one or more task properties in a single call. Valid keys: title, storyPointValue, tagNames, githubPRUrl, githubBranch. All are optional \u2014 include only the ones you want to update (at least one). Unknown keys are rejected.",
       {
-        title: z9.string().optional().describe("The new task title"),
-        storyPointValue: z9.number().optional().describe(SP_DESCRIPTION2),
-        tagNames: z9.array(z9.string()).optional().describe("Array of tag names to assign"),
-        githubPRUrl: z9.string().url().optional().describe("GitHub pull request URL to link to this task"),
-        githubBranch: z9.string().optional().describe("Set the GitHub branch name for this task (e.g. 'conveyor/my-feature-abc123')")
+        title: z10.string().optional().describe("The new task title"),
+        storyPointValue: z10.number().optional().describe(SP_DESCRIPTION2),
+        tagNames: z10.array(z10.string()).optional().describe("Array of tag names to assign"),
+        githubPRUrl: z10.string().url().optional().describe("GitHub pull request URL to link to this task"),
+        githubBranch: z10.string().optional().describe("Set the GitHub branch name for this task (e.g. 'conveyor/my-feature-abc123')")
       },
       async ({ title, storyPointValue, tagNames, githubPRUrl, githubBranch }) => {
         try {
+          const nothingToUpdate = title === void 0 && storyPointValue === void 0 && tagNames === void 0 && githubPRUrl === void 0 && githubBranch === void 0;
+          if (nothingToUpdate) {
+            return textResult(
+              `No task properties were updated: none of the recognized keys were provided. Valid keys: ${VALID_PROPERTY_KEYS}. (Story points are set via 'storyPointValue', not 'storyPoints'.)`
+            );
+          }
           await connection.call("updateTaskProperties", {
             sessionId: connection.sessionId,
             title,
@@ -7592,13 +7657,17 @@ function buildDiscoveryTools(connection) {
             `Failed to update task properties: ${error instanceof Error ? error.message : "Unknown error"}`
           );
         }
-      }
+      },
+      // Reject unknown keys (e.g. "storyPoints") at validation instead of
+      // stripping them into a silent no-op — every field here is optional, so
+      // a stripped typo would otherwise report success while updating nothing.
+      { strict: true }
     )
   ];
 }
 
 // src/tools/code-review-tools.ts
-import { z as z10 } from "zod";
+import { z as z11 } from "zod";
 async function endReviewSession(connection, reason) {
   await connection.call("endReviewSession", {
     sessionId: connection.sessionId,
@@ -7611,7 +7680,7 @@ function buildCodeReviewTools(connection) {
       "approve_code_review",
       "Approve the code review and exit. Use when the diff passes all review criteria. Takes only a summary \u2014 for changes, use request_code_changes with a structured issues[] list.",
       {
-        summary: z10.string().describe("Brief summary of what was reviewed and why it looks good")
+        summary: z11.string().describe("Brief summary of what was reviewed and why it looks good")
       },
       async ({ summary }) => {
         const content = `**Code Review: Approved** :white_check_mark:
@@ -7635,15 +7704,15 @@ ${summary}`;
       "request_code_changes",
       "Request changes during code review and exit. Use when substantive issues must be fixed before merge. Each issue: { file, line?, severity: critical|major|minor, description }.",
       {
-        issues: z10.array(
-          z10.object({
-            file: z10.string().describe("File path where the issue was found"),
-            line: z10.number().optional().describe("Line number (if applicable)"),
-            severity: z10.enum(["critical", "major", "minor"]).describe("Issue severity"),
-            description: z10.string().describe("What is wrong and how to fix it")
+        issues: z11.array(
+          z11.object({
+            file: z11.string().describe("File path where the issue was found"),
+            line: z11.number().optional().describe("Line number (if applicable)"),
+            severity: z11.enum(["critical", "major", "minor"]).describe("Issue severity"),
+            description: z11.string().describe("What is wrong and how to fix it")
           })
         ).describe("List of issues found during review"),
-        summary: z10.string().describe("Brief overall summary of the review findings")
+        summary: z11.string().describe("Brief overall summary of the review findings")
       },
       async ({ issues, summary }) => {
         const issueLines = issues.map((issue) => {
@@ -8020,9 +8089,17 @@ async function handleResultCase(event, host, context, startTime, isTyping, lastA
 function hasAskUserQuestionBlock(event) {
   return event.message.content.some((b) => b.type === "tool_use" && b.name === "AskUserQuestion");
 }
-async function armQuestionPending(host, state) {
+function questionTextFromEvent(event) {
+  const text = event.questions.map((q) => q.question.trim()).filter((q) => q.length > 0).join("\n");
+  return text.length > 0 ? text : void 0;
+}
+async function armQuestionPending(host, state, event) {
   state.questionPending = true;
-  await host.connection.emitStatus("waiting_for_input", AGENT_STATUS_REASON_USER_QUESTION);
+  await host.connection.emitStatus(
+    "waiting_for_input",
+    AGENT_STATUS_REASON_USER_QUESTION,
+    questionTextFromEvent(event)
+  );
   await host.callbacks.onStatusChange("waiting_for_input");
 }
 async function clearQuestionPending(host, state, options) {
@@ -8036,7 +8113,7 @@ async function clearQuestionPending(host, state, options) {
 async function applyQuestionTransitions(event, host, state) {
   switch (event.type) {
     case "user_question":
-      await armQuestionPending(host, state);
+      await armQuestionPending(host, state, event);
       return;
     case "assistant":
       if (!hasAskUserQuestionBlock(event)) {
@@ -9473,74 +9550,77 @@ function readAgentVersion() {
   return null;
 }
 
-// src/execution/usage-sampler.ts
-var logger4 = createServiceLogger("usage-sampler");
-var OAUTH_USAGE_URL = "https://api.anthropic.com/api/oauth/usage";
-var OAUTH_USAGE_BETA = "oauth-2025-04-20";
-var KNOWN_RATE_LIMIT_TYPES = [
-  "five_hour",
-  "seven_day",
-  "seven_day_opus",
-  "seven_day_sonnet"
-];
-function normalizeUtilization(raw) {
-  if (typeof raw !== "number" || !Number.isFinite(raw)) return null;
-  const fraction = raw > 1 ? raw / 100 : raw;
-  return Math.max(0, Math.min(1, fraction));
-}
-function extractBucket(value) {
-  if (typeof value === "number") {
-    const utilization = normalizeUtilization(value);
-    return utilization === null ? null : { utilization, status: "allowed" };
-  }
-  if (value && typeof value === "object") {
-    const obj = value;
-    const utilization = normalizeUtilization(
-      obj.utilization ?? obj.used ?? obj.percent ?? obj.percentage
-    );
-    if (utilization === null) return null;
-    const status = typeof obj.status === "string" ? obj.status : "allowed";
-    return { utilization, status };
-  }
-  return null;
+// src/usage/parse-usage.ts
+function parseUsageGauges(stdout) {
+  const session = stdout.match(/Current session:\s*(\d+(?:\.\d+)?)%\s*used/i);
+  const weekly = [...stdout.matchAll(/Current week[^:\n]*:\s*(\d+(?:\.\d+)?)%\s*used/gi)];
+  return {
+    sessionUsage: session ? Number(session[1]) / 100 : null,
+    weeklyUsage: weekly.length ? Math.max(...weekly.map((m) => Number(m[1]))) / 100 : null
+  };
 }
-function mapUsageResponse(data) {
-  if (!data || typeof data !== "object") return [];
-  const root = data;
-  const candidates = [root];
-  for (const key of ["usage", "rate_limits", "rateLimits"]) {
-    const nested = root[key];
-    if (nested && typeof nested === "object") {
-      candidates.push(nested);
-    }
-  }
-  for (const container of candidates) {
-    const samples = [];
-    for (const type of KNOWN_RATE_LIMIT_TYPES) {
-      const bucket = extractBucket(container[type]);
-      if (bucket) samples.push({ rateLimitType: type, ...bucket });
+
+// src/usage/run-probe.ts
+import { spawn } from "child_process";
+var PROBE_TIMEOUT_MS = 15e3;
+function runUsageProbe(binary = resolveClaudeBinary(), args = ["-p", "/usage"]) {
+  return new Promise((resolve) => {
+    let stdout = "";
+    let settled = false;
+    const finish = (out) => {
+      if (settled) return;
+      settled = true;
+      resolve(out);
+    };
+    let child;
+    try {
+      child = spawn(binary, args, { stdio: ["ignore", "pipe", "ignore"] });
+    } catch {
+      finish("");
+      return;
     }
-    if (samples.length > 0) return samples;
-  }
-  return [];
+    const timer = setTimeout(() => {
+      try {
+        child.kill("SIGKILL");
+      } catch {
+      }
+      finish("");
+    }, PROBE_TIMEOUT_MS);
+    timer.unref?.();
+    child.stdout?.on("data", (d) => {
+      stdout += d.toString();
+    });
+    child.on("error", () => {
+      clearTimeout(timer);
+      finish("");
+    });
+    child.on("close", (code) => {
+      clearTimeout(timer);
+      finish(code === 0 ? stdout : "");
+    });
+  });
 }
-async function sampleKeyUsage(token) {
+
+// src/execution/usage-sampler.ts
+var logger4 = createServiceLogger("usage-sampler");
+async function sampleKeyUsage(token, probe = runUsageProbe) {
   if (!token) return [];
   try {
-    const res = await fetch(OAUTH_USAGE_URL, {
-      headers: {
-        Authorization: `Bearer ${token}`,
-        "anthropic-beta": OAUTH_USAGE_BETA
-      }
-    });
-    if (!res.ok) {
-      logger4.info("OAuth usage endpoint returned non-200", { status: res.status });
-      return [];
+    const stdout = await probe();
+    const { sessionUsage, weeklyUsage } = parseUsageGauges(stdout);
+    const samples = [];
+    if (sessionUsage !== null) {
+      samples.push({ rateLimitType: "five_hour", utilization: sessionUsage, status: "allowed" });
+    }
+    if (weeklyUsage !== null) {
+      samples.push({ rateLimitType: "seven_day", utilization: weeklyUsage, status: "allowed" });
     }
-    const data = await res.json();
-    return mapUsageResponse(data);
+    if (samples.length === 0) {
+      logger4.info("usage sample produced no gauges", { stdoutLength: stdout.length });
+    }
+    return samples;
   } catch (error) {
-    logger4.info("OAuth usage sample failed", {
+    logger4.info("usage sample failed", {
       error: error instanceof Error ? error.message : String(error)
     });
     return [];
@@ -10060,7 +10140,11 @@ var SessionRunner = class _SessionRunner {
     const staleBatch = [...this.pendingMessages];
     const didExecuteInitialQuery = await this.executeInitialMode();
     if (staleBatch.length > 0 && didExecuteInitialQuery) {
+      const historyContents = new Set(
+        (this.fullContext?.chatHistory ?? []).filter((m) => m.role !== "assistant" && m.content.trim()).map((m) => m.content.trim())
+      );
       for (const stale of staleBatch) {
+        if (stale.content.trim() && !historyContents.has(stale.content.trim())) continue;
         const idx = this.pendingMessages.indexOf(stale);
         if (idx !== -1) this.pendingMessages.splice(idx, 1);
       }
@@ -10417,8 +10501,8 @@ var SessionRunner = class _SessionRunner {
       this.periodicFlushInFlight = false;
     }
   }
-  /** Sample the running Claude subscription key's rate-limit utilization from the
-   *  Anthropic OAuth usage endpoint and report it as `rate_limit_update` events.
+  /** Sample the running Claude subscription key's rate-limit utilization from
+   *  the Claude CLI `/usage` command and report it as `rate_limit_update` events.
    *  The API (`persistRateLimitSnapshot`) attributes them to the key this session
    *  launched under, keeping User Settings + the PtY-tab usage widget fresh and
    *  `selectBestKey` rotation honest. Best-effort — never throws, no-op when the
@@ -10913,10 +10997,10 @@ function loadConveyorConfig() {
 }
 
 // src/setup/commands.ts
-import { spawn, execSync } from "child_process";
+import { spawn as spawn2, execSync } from "child_process";
 function runSetupCommand(cmd, cwd, onOutput) {
   return new Promise((resolve, reject) => {
-    const child = spawn("sh", ["-c", cmd], {
+    const child = spawn2("sh", ["-c", cmd], {
       cwd,
       stdio: ["ignore", "pipe", "pipe"],
       env: { ...process.env }
@@ -10955,7 +11039,7 @@ function runAuthTokenCommand(cmd, userEmail, cwd) {
   }
 }
 function runStartCommand(cmd, cwd, onOutput) {
-  const child = spawn("sh", ["-c", cmd], {
+  const child = spawn2("sh", ["-c", cmd], {
     cwd,
     stdio: ["ignore", "pipe", "pipe"],
     detached: true,
@@ -11026,4 +11110,4 @@ export {
   runStartCommand,
   unshallowRepo
 };
-//# sourceMappingURL=chunk-HAH2S5AD.js.map
+//# sourceMappingURL=chunk-VCXKVINO.js.map