@rallycry/conveyor-agent 7.1.0 → 7.1.3

package/dist/{chunk-QLY35BX7.js → chunk-AZ6CRVYN.js}

@@ -1,10 +1,8 @@
 import {
   createHarness,
+  createServiceLogger,
   defineTool
-} from "./chunk-KNBG2634.js";
-import {
-  createServiceLogger
-} from "./chunk-CYZPFJGN.js";
+} from "./chunk-U6AYNVS7.js";
 
 // src/connection/agent-connection.ts
 import { io } from "socket.io-client";
@@ -503,7 +501,7 @@ var ModeController = class {
   }
   get isReadOnly() {
     const m = this.effectiveMode;
-    if (["discovery", "code-review", "help"].includes(m)) return true;
+    if (["discovery", "help"].includes(m)) return true;
     return m === "auto" && !this._hasExitedPlanMode;
   }
   get isAutoPlanning() {
@@ -511,13 +509,13 @@ var ModeController = class {
   }
   get isBuildCapable() {
     const m = this.effectiveMode;
-    return m === "building" || m === "auto" && this._hasExitedPlanMode;
+    return m === "building" || m === "review" || m === "auto" && this._hasExitedPlanMode;
   }
   // ── Mode resolution ────────────────────────────────────────────────
   /** Resolve the initial mode based on task context */
   resolveInitialMode(context) {
     if (this._runnerMode === "code-review") {
-      this._mode = "code-review";
+      this._mode = "review";
       return this._mode;
     }
     if (this._mode === "auto" && this.canBypassPlanning(context)) {
@@ -536,8 +534,12 @@ var ModeController = class {
   // ── Mode transitions ───────────────────────────────────────────────
   /** Handle mode change from server */
   handleModeChange(newMode) {
-    if (this._runnerMode !== "pm") return { type: "noop" };
     if (newMode === this._mode) return { type: "noop" };
+    if (this._runnerMode === "task" && newMode === "review") {
+      this._mode = newMode;
+      return { type: "restart_query", newMode: "review" };
+    }
+    if (this._runnerMode !== "pm") return { type: "noop" };
     this._mode = newMode;
     this.updateExitedPlanModeFlag(newMode);
     if (this.isBuildCapable) {
@@ -811,6 +813,28 @@ function updateRemoteToken(cwd, token) {
   } catch {
   }
 }
+async function flushPendingChanges(cwd, opts) {
+  let committed = false;
+  let pushed = false;
+  let hadWork = false;
+  try {
+    if (hasUncommittedChanges(cwd)) {
+      hadWork = true;
+      const message = opts?.wipMessage ?? "WIP: auto-commit on conveyor-agent shutdown";
+      const hash = stageAndCommit(cwd, message);
+      committed = hash !== null;
+    }
+  } catch {
+  }
+  try {
+    if (hasUnpushedCommits(cwd)) {
+      hadWork = true;
+      pushed = await pushToOrigin(cwd, opts?.refreshToken);
+    }
+  } catch {
+  }
+  return { committed, pushed, hadWork };
+}
 async function pushToOrigin(cwd, refreshToken) {
   try {
     const currentBranch = getCurrentBranch(cwd);
@@ -922,6 +946,8 @@ var PlanSync = class {
 
 // ../shared/dist/index.js
 import { z } from "zod";
+import { z as z2 } from "zod";
+import { z as z3 } from "zod";
 var MAX_FILE_SIZE_BYTES = 25 * 1024 * 1024;
 var EMBED_THRESHOLD_IMAGES = 5 * 1024 * 1024;
 var EMBED_THRESHOLD_TEXT = 2 * 1024 * 1024;
@@ -1144,157 +1170,6 @@ var UpdateChildStatusRequestSchema = z.object({
   childTaskId: z.string(),
   status: z.string()
 });
-var RegisterProjectAgentRequestSchema = z.object({
-  projectId: z.string(),
-  capabilities: z.array(z.string())
-});
-var ProjectRunnerHeartbeatRequestSchema = z.object({
-  projectId: z.string()
-});
-var ReportProjectAgentStatusRequestSchema = z.object({
-  projectId: z.string(),
-  status: z.enum(["busy", "idle"]),
-  activeChatId: z.string().nullish(),
-  activeTaskId: z.string().nullish(),
-  activeBranch: z.string().nullish()
-});
-var DisconnectProjectRunnerRequestSchema = z.object({
-  projectId: z.string()
-});
-var GetProjectAgentContextRequestSchema = z.object({
-  projectId: z.string()
-});
-var GetProjectChatHistoryRequestSchema = z.object({
-  projectId: z.string(),
-  limit: z.number().int().positive().optional().default(50),
-  chatId: z.string().optional()
-});
-var PostProjectAgentMessageRequestSchema = z.object({
-  projectId: z.string(),
-  content: z.string().min(1),
-  chatId: z.string().optional()
-});
-var ListProjectTasksRequestSchema = z.object({
-  projectId: z.string(),
-  status: z.string().optional(),
-  assigneeId: z.string().optional(),
-  limit: z.number().int().positive().optional().default(50)
-});
-var GetProjectTaskRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string()
-});
-var SearchProjectTasksRequestSchema = z.object({
-  projectId: z.string(),
-  tagNames: z.array(z.string()).optional(),
-  searchQuery: z.string().optional(),
-  statusFilters: z.array(z.string()).optional(),
-  limit: z.number().int().positive().optional().default(20)
-});
-var ListProjectTagsRequestSchema = z.object({
-  projectId: z.string()
-});
-var GetProjectSummaryRequestSchema = z.object({
-  projectId: z.string()
-});
-var CreateProjectTaskRequestSchema = z.object({
-  projectId: z.string(),
-  title: z.string().min(1),
-  description: z.string().optional(),
-  plan: z.string().optional(),
-  status: z.string().optional(),
-  isBug: z.boolean().optional()
-});
-var UpdateProjectTaskRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string(),
-  title: z.string().optional(),
-  description: z.string().optional(),
-  plan: z.string().optional(),
-  status: z.string().optional(),
-  assignedUserId: z.string().nullish()
-});
-var ReportProjectAgentEventRequestSchema = z.object({
-  projectId: z.string(),
-  event: z.record(z.string(), z.unknown())
-});
-var ReportTagAuditProgressRequestSchema = z.object({
-  projectId: z.string(),
-  requestId: z.string(),
-  activity: z.object({
-    tool: z.string(),
-    input: z.string().optional(),
-    timestamp: z.string()
-  })
-});
-var ReportTagAuditResultRequestSchema = z.object({
-  projectId: z.string(),
-  requestId: z.string(),
-  recommendations: z.array(z.record(z.string(), z.unknown())),
-  summary: z.string(),
-  complete: z.boolean()
-});
-var ReportNewCommitsDetectedRequestSchema = z.object({
-  projectId: z.string(),
-  commits: z.array(
-    z.object({
-      sha: z.string(),
-      message: z.string(),
-      author: z.string()
-    })
-  ),
-  branch: z.string()
-});
-var ReportEnvironmentReadyRequestSchema = z.object({
-  projectId: z.string(),
-  branch: z.string(),
-  setupComplete: z.boolean(),
-  startCommandRunning: z.boolean()
-});
-var ReportEnvSwitchProgressRequestSchema = z.object({
-  projectId: z.string(),
-  step: z.string(),
-  progress: z.number(),
-  message: z.string().optional()
-});
-var ForwardProjectChatMessageRequestSchema = z.object({
-  projectId: z.string(),
-  chatId: z.string(),
-  content: z.string().min(1),
-  targetUserId: z.string().optional()
-});
-var CancelQueuedProjectMessageRequestSchema = z.object({
-  projectId: z.string(),
-  index: z.number().int().nonnegative()
-});
-var GetProjectCliHistoryRequestSchema = z.object({
-  projectId: z.string(),
-  limit: z.number().int().positive().optional().default(50),
-  source: z.string().optional()
-});
-var PostToProjectTaskChatRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string(),
-  content: z.string()
-});
-var GetProjectTaskCliRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string(),
-  limit: z.number().int().positive().optional().default(50),
-  source: z.string().optional()
-});
-var StartProjectBuildRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string()
-});
-var StopProjectBuildRequestSchema = z.object({
-  projectId: z.string(),
-  taskId: z.string()
-});
-var ApproveProjectMergePRRequestSchema = z.object({
-  projectId: z.string(),
-  childTaskId: z.string()
-});
 var GetAgentStatusRequestSchema = z.object({
   taskId: z.string()
 });
@@ -1390,7 +1265,200 @@ var RegisterProjectAgentResponseSchema = z.object({
   agentSettings: z.record(z.string(), z.unknown()).nullable(),
   branchSwitchCommand: z.string().nullable()
 });
-var CRITICAL_AUTOMATED_SOURCES = /* @__PURE__ */ new Set(["ci_failure"]);
+var RegisterProjectAgentRequestSchema = z2.object({
+  projectId: z2.string(),
+  capabilities: z2.array(z2.string())
+});
+var ProjectRunnerHeartbeatRequestSchema = z2.object({
+  projectId: z2.string()
+});
+var ReportProjectAgentStatusRequestSchema = z2.object({
+  projectId: z2.string(),
+  status: z2.enum(["busy", "idle"]),
+  activeChatId: z2.string().nullish(),
+  activeTaskId: z2.string().nullish(),
+  activeBranch: z2.string().nullish()
+});
+var DisconnectProjectRunnerRequestSchema = z2.object({
+  projectId: z2.string()
+});
+var GetProjectAgentContextRequestSchema = z2.object({
+  projectId: z2.string()
+});
+var GetProjectChatHistoryRequestSchema = z2.object({
+  projectId: z2.string(),
+  limit: z2.number().int().positive().optional().default(50),
+  chatId: z2.string().optional()
+});
+var PostProjectAgentMessageRequestSchema = z2.object({
+  projectId: z2.string(),
+  content: z2.string().min(1),
+  chatId: z2.string().optional()
+});
+var ListProjectTasksRequestSchema = z2.object({
+  projectId: z2.string(),
+  status: z2.string().optional(),
+  assigneeId: z2.string().optional(),
+  limit: z2.number().int().positive().optional().default(50)
+});
+var GetProjectTaskRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string()
+});
+var SearchProjectTasksRequestSchema = z2.object({
+  projectId: z2.string(),
+  tagNames: z2.array(z2.string()).optional(),
+  searchQuery: z2.string().optional(),
+  statusFilters: z2.array(z2.string()).optional(),
+  limit: z2.number().int().positive().optional().default(20)
+});
+var ListProjectTagsRequestSchema = z2.object({
+  projectId: z2.string()
+});
+var GetProjectSummaryRequestSchema = z2.object({
+  projectId: z2.string()
+});
+var CreateProjectTaskRequestSchema = z2.object({
+  projectId: z2.string(),
+  title: z2.string().min(1),
+  description: z2.string().optional(),
+  plan: z2.string().optional(),
+  status: z2.string().optional(),
+  isBug: z2.boolean().optional()
+});
+var UpdateProjectTaskRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string(),
+  title: z2.string().optional(),
+  description: z2.string().optional(),
+  plan: z2.string().optional(),
+  status: z2.string().optional(),
+  assignedUserId: z2.string().nullish()
+});
+var ReportProjectAgentEventRequestSchema = z2.object({
+  projectId: z2.string(),
+  event: z2.record(z2.string(), z2.unknown())
+});
+var ReportTagAuditProgressRequestSchema = z2.object({
+  projectId: z2.string(),
+  requestId: z2.string(),
+  activity: z2.object({
+    tool: z2.string(),
+    input: z2.string().optional(),
+    timestamp: z2.string()
+  })
+});
+var ReportTagAuditResultRequestSchema = z2.object({
+  projectId: z2.string(),
+  requestId: z2.string(),
+  recommendations: z2.array(z2.record(z2.string(), z2.unknown())),
+  summary: z2.string(),
+  complete: z2.boolean()
+});
+var ReportNewCommitsDetectedRequestSchema = z2.object({
+  projectId: z2.string(),
+  commits: z2.array(
+    z2.object({
+      sha: z2.string(),
+      message: z2.string(),
+      author: z2.string()
+    })
+  ),
+  branch: z2.string()
+});
+var ReportEnvironmentReadyRequestSchema = z2.object({
+  projectId: z2.string(),
+  branch: z2.string(),
+  setupComplete: z2.boolean(),
+  startCommandRunning: z2.boolean()
+});
+var ReportEnvSwitchProgressRequestSchema = z2.object({
+  projectId: z2.string(),
+  step: z2.string(),
+  progress: z2.number(),
+  message: z2.string().optional()
+});
+var ForwardProjectChatMessageRequestSchema = z2.object({
+  projectId: z2.string(),
+  chatId: z2.string(),
+  content: z2.string().min(1),
+  targetUserId: z2.string().optional()
+});
+var CancelQueuedProjectMessageRequestSchema = z2.object({
+  projectId: z2.string(),
+  index: z2.number().int().nonnegative()
+});
+var GetProjectCliHistoryRequestSchema = z2.object({
+  projectId: z2.string(),
+  limit: z2.number().int().positive().optional().default(50),
+  source: z2.string().optional()
+});
+var PostToProjectTaskChatRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string(),
+  content: z2.string()
+});
+var GetProjectTaskCliRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string(),
+  limit: z2.number().int().positive().optional().default(50),
+  source: z2.string().optional()
+});
+var StartProjectBuildRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string()
+});
+var StopProjectBuildRequestSchema = z2.object({
+  projectId: z2.string(),
+  taskId: z2.string()
+});
+var ApproveProjectMergePRRequestSchema = z2.object({
+  projectId: z2.string(),
+  childTaskId: z2.string()
+});
+var CRITICAL_AUTOMATED_SOURCES = /* @__PURE__ */ new Set(["ci_failure", "review_trigger"]);
+var StartTaskAuditRequestSchema = z3.object({
+  projectId: z3.string(),
+  taskIds: z3.array(z3.string()).min(1)
+});
+var ReportTaskAuditProgressRequestSchema = z3.object({
+  projectId: z3.string(),
+  requestId: z3.string(),
+  taskId: z3.string(),
+  activity: z3.string()
+});
+var ReportTaskAuditResultRequestSchema = z3.object({
+  projectId: z3.string(),
+  requestId: z3.string(),
+  taskId: z3.string(),
+  summary: z3.string(),
+  turnGrades: z3.array(
+    z3.object({
+      turnIndex: z3.number(),
+      phase: z3.enum(["planning", "building", "human"]),
+      grade: z3.enum(["correct", "neutral", "blunder"]),
+      reasoning: z3.string(),
+      eventType: z3.string(),
+      eventSummary: z3.string()
+    })
+  ),
+  planningAccuracy: z3.number().nullable(),
+  buildingAccuracy: z3.number().nullable(),
+  humanAccuracy: z3.number().nullable(),
+  planningCorrect: z3.number(),
+  planningNeutral: z3.number(),
+  planningBlunder: z3.number(),
+  buildingCorrect: z3.number(),
+  buildingNeutral: z3.number(),
+  buildingBlunder: z3.number(),
+  humanCorrect: z3.number(),
+  humanNeutral: z3.number(),
+  humanBlunder: z3.number(),
+  suggestionIds: z3.array(z3.string()),
+  auditCostUsd: z3.number().nullable(),
+  model: z3.string().nullable(),
+  error: z3.string().optional()
+});
 
 // src/execution/pack-runner-prompt.ts
 function findLastAgentMessageIndex(history) {
@@ -1882,8 +1950,6 @@ function buildModePrompt(agentMode, context) {
       return buildReviewPrompt(context);
     case "auto":
       return buildAutoPrompt(context);
-    case "code-review":
-      return buildCodeReviewPrompt();
     default:
       return null;
   }
@@ -1892,8 +1958,9 @@ function buildReviewPrompt(context) {
   const parts = [
     `
 ## Mode: Review`,
-    `You are in Review mode \u2014 reviewing and coordinating.`,
-    `- You have read-only access plus light edit capability (can suggest fixes, run tests, check linting)`,
+    `You are in Review mode \u2014 performing code review with fix capability.`,
+    `- You have full write access \u2014 you can audit code, make fixes, push changes, and run tests`,
+    `- Safety rules: no destructive operations, use --force-with-lease instead of --force`,
     ``
   ];
   if (context?.isParentTask) {
@@ -1915,80 +1982,62 @@ function buildReviewPrompt(context) {
     );
   } else {
     parts.push(
-      `### Leaf Task Review`,
-      `You are reviewing your own work before completion.`,
-      `- Run tests and check linting to verify the PR is ready.`,
-      `- If the PR is already open, review the diff for correctness.`,
-      `- You can get hands dirty \u2014 if a fix is small, make it directly.`,
-      `- If follow-up work is needed, use \`create_follow_up_task\`.`
+      `### Code Review Process`,
+      `1. Run \`git diff ${context?.baseBranch ?? "dev"}..HEAD\` to see all changes in this PR`,
+      `2. Read the task plan to understand the intended changes`,
+      `3. Explore the surrounding codebase to verify pattern consistency`,
+      `4. Review against the criteria below`,
+      ``,
+      `### Review Criteria`,
+      `- **Correctness**: Does the code do what the plan says? Logic errors, off-by-one, race conditions?`,
+      `- **Pattern Consistency**: Does the code follow existing patterns in the codebase? Check nearby files.`,
+      `- **Security**: No hardcoded secrets, no injection vulnerabilities, proper input validation at boundaries.`,
+      `- **Performance**: No unnecessary loops, no N+1 queries, no blocking in async contexts.`,
+      `- **Error Handling**: Appropriate error handling at system boundaries. No swallowed errors.`,
+      `- **Test Coverage**: Are new code paths tested? Edge cases covered?`,
+      `- **TypeScript Best Practices**: Proper typing (no unnecessary \`any\`), correct React patterns, proper async/await.`,
+      `- **Naming & Readability**: Clear names, no misleading comments, self-documenting code.`,
+      ``,
+      `### Fix Capability`,
+      `You have full write access. If you find issues:`,
+      `- **Small fixes**: Make the fix directly, commit, and push. Then re-review.`,
+      `- **Larger issues**: Use \`request_code_changes\` to flag them for the team.`,
+      `- After pushing fixes, wait for CI to pass before approving.`,
+      ``,
+      `### Output \u2014 You MUST do exactly ONE of:`,
+      ``,
+      `#### If code passes review (or after you've fixed all issues):`,
+      `Use the \`approve_code_review\` tool with a brief summary of what looks good.`,
+      ``,
+      `#### If changes are needed that you cannot fix:`,
+      `Use the \`request_code_changes\` tool with specific issues:`,
+      `- Reference specific files and line numbers`,
+      `- Explain what's wrong and suggest fixes`,
+      `- Focus on substantive issues, not style nitpicks (linting handles that)`,
+      ``,
+      `### Previous Review Feedback`,
+      `If previous review feedback is present in the chat history, verify those specific issues were addressed before raising new concerns.`,
+      ``,
+      `### Rules`,
+      `- Do NOT re-review things CI already validates (formatting, lint rules).`,
+      `- Be concise \u2014 actionable specifics over general observations.`,
+      `- Max 5-7 issues per review. Prioritize the most important ones.`
     );
   }
-  parts.push(
-    ``,
-    `### General Review Guidelines`,
-    `- For larger issues, create a follow-up task rather than fixing directly.`,
-    `- Focus on correctness, pattern consistency, and test coverage.`,
-    `- Be concise in feedback \u2014 actionable specifics over general observations.`,
-    `- Goal: ensure quality, provide feedback, coordinate progression.`
-  );
   if (process.env.CLAUDESPACE_NAME) {
     parts.push(
       ``,
       `### Resource Management`,
-      `Your pod starts with minimal resources (0.25 CPU / 1 Gi). You MUST call \`scale_up_resources\``,
-      `BEFORE running any of these operations \u2014 they WILL fail or OOM at baseline resources:`,
-      `- **light** (1 CPU / 4 Gi) \u2014 bun/npm/yarn install, pip install, basic dev servers, light builds`,
-      `- **standard** (2 CPU / 8 Gi) \u2014 full dev servers, test suites, typecheck, lint`,
-      `- **heavy** (4 CPU / 16 Gi) \u2014 E2E/browser automation, large parallel builds`,
+      `Your pod starts with minimal resources. You MUST call \`scale_up_resources\``,
+      `BEFORE running any resource-intensive operations \u2014 they WILL fail or OOM at baseline resources:`,
+      `- **setup** \u2014 package installs, basic dev servers, light builds`,
+      `- **build** \u2014 full dev servers, test suites, typecheck, lint, E2E tests`,
       `Scaling is one-way (up only) and capped by project limits.`,
-      `CRITICAL: Always scale to at least "light" before running any package install command.`
+      `CRITICAL: Always scale to at least "setup" before running any package install command.`
     );
   }
   return parts.join("\n");
 }
-function buildCodeReviewPrompt() {
-  return [
-    `
-## Mode: Code Review`,
-    `You are an automated code reviewer. A PR has passed all CI checks and you are performing a final code quality review before merge.`,
-    ``,
-    `## Review Process`,
-    `1. Run \`git diff <devBranch>..HEAD\` to see all changes in this PR`,
-    `2. Read the task plan to understand the intended changes`,
-    `3. Explore the surrounding codebase to verify pattern consistency`,
-    `4. Review against the criteria below`,
-    ``,
-    `### Review Criteria`,
-    `- **Correctness**: Does the code do what the plan says? Logic errors, off-by-one, race conditions?`,
-    `- **Pattern Consistency**: Does the code follow existing patterns in the codebase? Check nearby files.`,
-    `- **Security**: No hardcoded secrets, no injection vulnerabilities, proper input validation at boundaries.`,
-    `- **Performance**: No unnecessary loops, no N+1 queries, no blocking in async contexts.`,
-    `- **Error Handling**: Appropriate error handling at system boundaries. No swallowed errors.`,
-    `- **Test Coverage**: Are new code paths tested? Edge cases covered?`,
-    `- **TypeScript Best Practices**: Proper typing (no unnecessary \`any\`), correct React patterns, proper async/await.`,
-    `- **Naming & Readability**: Clear names, no misleading comments, self-documenting code.`,
-    ``,
-    `## Output \u2014 You MUST do exactly ONE of:`,
-    ``,
-    `### If code passes review:`,
-    `Use the \`approve_code_review\` tool with a brief summary of what looks good.`,
-    ``,
-    `### If changes are needed:`,
-    `Use the \`request_code_changes\` tool with specific issues:`,
-    `- Reference specific files and line numbers`,
-    `- Explain what's wrong and suggest fixes`,
-    `- Focus on substantive issues, not style nitpicks (linting handles that)`,
-    ``,
-    `## Previous Review Feedback`,
-    `If previous review feedback is present in the chat history, verify those specific issues were addressed before raising new concerns. Focus on whether the requested changes were implemented correctly.`,
-    ``,
-    `## Rules`,
-    `- You are READ-ONLY. Do NOT modify any files.`,
-    `- Do NOT re-review things CI already validates (formatting, lint rules).`,
-    `- Be concise \u2014 the task agent needs actionable feedback, not essays.`,
-    `- Max 5-7 issues per review. Prioritize the most important ones.`
-  ].join("\n");
-}
 
 // src/execution/system-prompt.ts
 function formatProjectAgentLine(pa) {
@@ -2415,18 +2464,6 @@ ${context.plan}`);
   }
   return parts;
 }
-function buildCodeReviewInstructions(context) {
-  const parts = [
-    `You are performing an automated code review for this task.`,
-    `The PR branch is "${context.githubBranch}"${context.baseBranch ? ` based on "${context.baseBranch}"` : ""}.`,
-    `Begin your code review by running \`git diff ${context.baseBranch ?? "dev"}..HEAD\` to see all changes.`,
-    ``,
-    `CRITICAL: You are in Code Review mode. You are READ-ONLY \u2014 do NOT modify any files.`,
-    `After reviewing, you MUST call exactly one of: \`approve_code_review\` or \`request_code_changes\`.`,
-    `Do NOT go idle, ask for confirmation, or wait for instructions \u2014 complete the review and exit.`
-  ];
-  return parts;
-}
 function buildFreshInstructions(isPm, isAutoMode, context, agentMode) {
   if (isPm && agentMode === "building") {
     return [
@@ -2463,16 +2500,16 @@ CRITICAL: You are in Auto mode. Do NOT report status, ask for confirmation, or g
     return [
       `You are the project manager for this task and its subtasks.`,
       `Review existing subtasks via \`list_subtasks\` and the chat history before taking action.`,
-      `Read the task description and chat history carefully \u2014 the team may have already provided context and requirements. Respond to what's been discussed rather than starting from scratch.`,
-      `The task details are provided above. Wait for the team to provide instructions before taking action.`,
+      `Read the task description and chat history carefully \u2014 the team has provided the initial context below. Acknowledge what they've asked for and respond in chat before taking silent tool actions.`,
+      `Start planning now \u2014 explore the codebase, ask clarifying questions if needed, and propose a subtask breakdown. Do not wait for additional team input before engaging.`,
       `When you finish planning, save the plan with update_task and end your turn. Your reply will be visible to the team in chat.`
     ];
   }
   if (isPm) {
     return [
       `You are the project manager for this task.`,
-      `Read the task description and chat history carefully \u2014 the team may have already provided context and requirements. Respond to what's been discussed rather than starting from scratch.`,
-      `The task details are provided above. Wait for the team to ask questions or provide additional requirements before starting to plan.`,
+      `Read the task description and chat history carefully \u2014 the team has provided the initial context below. Acknowledge what they've asked for and respond in chat before taking silent tool actions.`,
+      `Start planning now \u2014 explore the codebase, ask clarifying questions if needed, and draft a plan. Do not wait for additional team input before engaging; the initial message IS the team engaging.`,
       `When you finish planning, save the plan with update_task and end your turn. Your reply summarizing the plan will be visible in chat. A separate task agent will execute the plan after review.`
     ];
   }
@@ -2577,10 +2614,6 @@ function buildIdleRelaunchInstructions(context, isPm, agentMode, isAuto) {
 function buildInstructions(mode, context, scenario, agentMode, isAuto) {
   const parts = [`
 ## Instructions`];
-  if (agentMode === "code-review") {
-    parts.push(...buildCodeReviewInstructions(context));
-    return parts;
-  }
   const isPm = mode === "pm";
   if (scenario === "fresh") {
     parts.push(...buildFreshInstructions(isPm, agentMode === "auto", context, agentMode));
@@ -2595,7 +2628,7 @@ function buildInstructions(mode, context, scenario, agentMode, isAuto) {
 }
 async function buildInitialPrompt(mode, context, isAuto, agentMode) {
   const isPackRunner = mode === "pm" && !!isAuto && !!context.isParentTask;
-  if (!isPackRunner && agentMode !== "code-review") {
+  if (!isPackRunner) {
     const sessionRelaunch = buildRelaunchWithSession(mode, context, agentMode, isAuto);
     if (sessionRelaunch) return sessionRelaunch;
   }
@@ -2610,7 +2643,7 @@ async function buildInitialPrompt(mode, context, isAuto, agentMode) {
 }
 
 // src/tools/common-tools.ts
-import { z as z2 } from "zod";
+import { z as z4 } from "zod";
 
 // src/tools/helpers.ts
 function textResult(text) {
@@ -2644,8 +2677,8 @@ function buildReadTaskChatTool(connection) {
     "read_task_chat",
     "Read recent messages from a task chat. Omit task_id to read the current task's chat, or provide a child task ID to read a child's chat.",
     {
-      limit: z2.number().optional().describe("Number of recent messages to fetch (default 20)"),
-      task_id: z2.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 {
@@ -2689,7 +2722,7 @@ function buildGetTaskTool(connection) {
     "get_task",
     "Look up a task by slug or ID to get its title, description, plan, and status",
     {
-      slug_or_id: z2.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 {
@@ -2712,9 +2745,9 @@ function buildGetTaskCliTool(connection) {
     "get_task_cli",
     "Read CLI execution logs from a task. Returns agent reasoning, tool calls, setup output, and other execution events. Use 'source' to filter: 'agent' for agent reasoning/tool calls only, 'application' for setup/dev-server output only.",
     {
-      task_id: z2.string().optional().describe("Task ID or slug. Omit to read logs from the current task."),
-      source: z2.enum(["agent", "application"]).optional().describe("Filter by log source. Omit for all logs."),
-      limit: z2.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 {
@@ -2774,7 +2807,7 @@ function buildGetTaskFileTool(connection) {
   return defineTool(
     "get_task_file",
     "Get a specific task file's content and download URL by file ID",
-    { fileId: z2.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", {
@@ -2805,8 +2838,8 @@ function buildSearchIncidentsTool(connection) {
     "search_incidents",
     "Search incidents in the current project. Optionally filter by status (Open, Acknowledged, Investigating, Resolved, Closed) or source.",
     {
-      status: z2.string().optional().describe("Filter by incident status"),
-      source: z2.string().optional().describe("Filter by source (e.g., 'conveyor', 'agent', 'build')")
+      status: z4.string().optional().describe("Filter by incident status"),
+      source: z4.string().optional().describe("Filter by source (e.g., 'conveyor', 'agent', 'build')")
     },
     async ({ status, source }) => {
       try {
@@ -2830,7 +2863,7 @@ function buildGetTaskIncidentsTool(connection) {
     "get_task_incidents",
     "Get all incidents linked to the current task (or a specified task). Returns full incident details including title, description, severity, status, and source.",
     {
-      task_id: z2.string().optional().describe("Task ID (defaults to current task)")
+      task_id: z4.string().optional().describe("Task ID (defaults to current task)")
     },
     async ({ task_id }) => {
       try {
@@ -2853,12 +2886,12 @@ function buildQueryGcpLogsTool(connection) {
     "query_gcp_logs",
     "Query GCP Cloud Logging for the current project. Returns log entries matching the given filters. Use severity to filter by minimum log level. The project's GCP credentials are used automatically.",
     {
-      filter: z2.string().optional().describe(
+      filter: z4.string().optional().describe(
         `Cloud Logging filter expression (e.g., 'resource.type="gce_instance"'). Max 1000 chars.`
       ),
-      start_time: z2.string().optional().describe("Start time in ISO 8601 format (e.g., '2024-01-01T00:00:00Z')"),
-      end_time: z2.string().optional().describe("End time in ISO 8601 format (e.g., '2024-01-02T00:00:00Z')"),
-      severity: z2.enum([
+      start_time: z4.string().optional().describe("Start time in ISO 8601 format (e.g., '2024-01-01T00:00:00Z')"),
+      end_time: z4.string().optional().describe("End time in ISO 8601 format (e.g., '2024-01-02T00:00:00Z')"),
+      severity: z4.enum([
         "DEFAULT",
         "DEBUG",
         "INFO",
@@ -2869,7 +2902,7 @@ function buildQueryGcpLogsTool(connection) {
         "ALERT",
         "EMERGENCY"
       ]).optional().describe("Minimum severity level to filter by (default: all levels)"),
-      page_size: z2.number().optional().describe("Number of log entries to return (default 100, max 500)")
+      page_size: z4.number().optional().describe("Number of log entries to return (default 100, max 500)")
     },
     async ({ filter, start_time, end_time, severity, page_size }) => {
       try {
@@ -2923,8 +2956,8 @@ function buildGetSuggestionsTool(connection) {
     "get_suggestions",
     "List project suggestions sorted by vote score. Use this to see what the team thinks is important.",
     {
-      status: z2.string().optional().describe("Filter by status: Open, Accepted, Rejected, Implemented"),
-      limit: z2.number().optional().describe("Max results (default 20)")
+      status: z4.string().optional().describe("Filter by status: Open, Accepted, Rejected, Implemented"),
+      limit: z4.number().optional().describe("Max results (default 20)")
     },
     async ({ status: _status, limit: _limit }) => {
       try {
@@ -2949,8 +2982,8 @@ function buildPostToChatTool(connection) {
     "post_to_chat",
     "Post a message to a task chat. Your normal replies already appear in chat \u2014 only use this for explicit out-of-band updates or posting to a child task's chat.",
     {
-      message: z2.string().describe("The message to post to the team"),
-      task_id: z2.string().optional().describe("Child task ID to post to. Omit to post to the current task's chat.")
+      message: z4.string().describe("The message to post to the team"),
+      task_id: z4.string().optional().describe("Child task ID to post to. Omit to post to the current task's chat.")
     },
     async ({ message, task_id }) => {
       try {
@@ -2977,8 +3010,8 @@ function buildForceUpdateTaskStatusTool(connection) {
     "force_update_task_status",
     "EMERGENCY ONLY: Force-override a task's Kanban status. Status transitions happen automatically (building sets InProgress, PR creation sets ReviewPR, merge sets ReviewDev). Only use this if an automatic transition failed or a task is stuck in the wrong status. Omit task_id to update the current task, or provide a child task ID.",
     {
-      status: z2.enum(["InProgress", "ReviewPR", "ReviewDev", "Complete"]).describe("The new status for the task"),
-      task_id: z2.string().optional().describe("Child task ID to update. Omit to update the current task.")
+      status: z4.enum(["InProgress", "ReviewPR", "ReviewDev", "Complete"]).describe("The new status for the task"),
+      task_id: z4.string().optional().describe("Child task ID to update. Omit to update the current task.")
     },
     async ({ status, task_id }) => {
       try {
@@ -3009,15 +3042,15 @@ function buildCreatePullRequestTool(connection, config) {
     "create_pull_request",
     "Create a GitHub pull request for this task. Automatically stages uncommitted changes, commits them, and pushes before creating the PR. Use this instead of gh CLI or git commands to create PRs.",
     {
-      title: z2.string().describe("The PR title"),
-      body: z2.string().describe("The PR description/body in markdown"),
-      branch: z2.string().optional().describe(
+      title: z4.string().describe("The PR title"),
+      body: z4.string().describe("The PR description/body in markdown"),
+      branch: z4.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: z2.string().optional().describe(
+      baseBranch: z4.string().optional().describe(
         "The base branch to target for the PR (e.g. 'main', 'develop'). Defaults to the project's configured dev branch."
       ),
-      commitMessage: z2.string().optional().describe(
+      commitMessage: z4.string().optional().describe(
         "Commit message for staging uncommitted changes. If not provided, a default message based on the PR title will be used."
       )
     },
@@ -3095,7 +3128,7 @@ function buildAddDependencyTool(connection) {
     "add_dependency",
     "Add a dependency \u2014 this task cannot start until the specified task is merged to dev",
     {
-      depends_on_slug_or_id: z2.string().describe("Slug or ID of the task this task depends on")
+      depends_on_slug_or_id: z4.string().describe("Slug or ID of the task this task depends on")
     },
     async ({ depends_on_slug_or_id }) => {
       try {
@@ -3117,7 +3150,7 @@ function buildRemoveDependencyTool(connection) {
     "remove_dependency",
     "Remove a dependency from this task",
     {
-      depends_on_slug_or_id: z2.string().describe("Slug or ID of the task to remove as dependency")
+      depends_on_slug_or_id: z4.string().describe("Slug or ID of the task to remove as dependency")
     },
     async ({ depends_on_slug_or_id }) => {
       try {
@@ -3139,10 +3172,10 @@ function buildCreateFollowUpTaskTool(connection) {
     "create_follow_up_task",
     "Create a follow-up task in this project that depends on the current task. Use for out-of-scope work, v1.1 features, or cleanup that should happen after this task merges.",
     {
-      title: z2.string().describe("Follow-up task title"),
-      description: z2.string().optional().describe("Brief description of the follow-up work"),
-      plan: z2.string().optional().describe("Implementation plan if known"),
-      story_point_value: z2.number().optional().describe("Story point estimate (1=Common, 2=Magic, 3=Rare, 5=Unique)")
+      title: z4.string().describe("Follow-up task title"),
+      description: z4.string().optional().describe("Brief description of the follow-up work"),
+      plan: z4.string().optional().describe("Implementation plan if known"),
+      story_point_value: z4.number().optional().describe("Story point estimate (1=Common, 2=Magic, 3=Rare, 5=Unique)")
     },
     async ({ title, description, plan, story_point_value }) => {
       try {
@@ -3169,9 +3202,9 @@ function buildCreateSuggestionTool(connection) {
     "create_suggestion",
     "Suggest a feature, improvement, or idea for the project. If you want to recommend something \u2014 a document, a rule, a feature, a task, an optimization \u2014 use this tool. If a similar suggestion already exists, your vote will be added to it instead of creating a duplicate.",
     {
-      title: z2.string().describe("Short title for the suggestion"),
-      description: z2.string().optional().describe("Details about the suggestion"),
-      tag_names: z2.array(z2.string()).optional().describe("Tag names to categorize the suggestion")
+      title: z4.string().describe("Short title for the suggestion"),
+      description: z4.string().optional().describe("Details about the suggestion"),
+      tag_names: z4.array(z4.string()).optional().describe("Tag names to categorize the suggestion")
     },
     async ({ title, description, tag_names }) => {
       try {
@@ -3200,8 +3233,8 @@ function buildVoteSuggestionTool(connection) {
     "vote_suggestion",
     "Vote on a project suggestion. Use +1 to upvote or -1 to downvote.",
     {
-      suggestion_id: z2.string().describe("The suggestion ID to vote on"),
-      value: z2.number().refine((v) => v === 1 || v === -1, { message: "Value must be 1 or -1" }).describe("+1 to upvote, -1 to downvote")
+      suggestion_id: z4.string().describe("The suggestion ID to vote on"),
+      value: z4.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 {
@@ -3224,8 +3257,8 @@ function buildScaleUpResourcesTool(connection) {
     "scale_up_resources",
     "Scale up the pod's CPU and memory resources to the 'build' tier. Use before running heavy operations like full test suites, integration tests, typechecks, or production builds. Pods start at the 'setup' tier by default \u2014 only call this when you actually need more. Scaling is one-way (up only).",
     {
-      tier: z2.literal("build").describe("The resource phase to scale up to"),
-      reason: z2.string().optional().describe("Brief reason for scaling (e.g., 'running test suite')")
+      tier: z4.literal("build").describe("The resource phase to scale up to"),
+      reason: z4.string().optional().describe("Brief reason for scaling (e.g., 'running test suite')")
     },
     async ({ tier, reason }) => {
       try {
@@ -3278,15 +3311,15 @@ function buildCommonTools(connection, config) {
 }
 
 // src/tools/pm-tools.ts
-import { z as z3 } from "zod";
+import { z as z5 } from "zod";
 var SP_DESCRIPTION = "Story point value (1=Common, 2=Magic, 3=Rare, 5=Unique)";
 function buildUpdateTaskTool(connection) {
   return defineTool(
     "update_task",
     "Save the finalized task plan and/or description",
     {
-      plan: z3.string().optional().describe("The task plan in markdown"),
-      description: z3.string().optional().describe("Updated task description")
+      plan: z5.string().optional().describe("The task plan in markdown"),
+      description: z5.string().optional().describe("Updated task description")
     },
     async ({ plan, description }) => {
       try {
@@ -3307,11 +3340,11 @@ function buildCreateSubtaskTool(connection) {
     "create_subtask",
     "Create a subtask under the current parent task. Use for breaking complex tasks into smaller pieces.",
     {
-      title: z3.string().describe("Subtask title"),
-      description: z3.string().optional().describe("Brief description"),
-      plan: z3.string().optional().describe("Implementation plan in markdown"),
-      ordinal: z3.number().optional().describe("Step/order number (0-based)"),
-      storyPointValue: z3.number().optional().describe(SP_DESCRIPTION)
+      title: z5.string().describe("Subtask title"),
+      description: z5.string().optional().describe("Brief description"),
+      plan: z5.string().optional().describe("Implementation plan in markdown"),
+      ordinal: z5.number().optional().describe("Step/order number (0-based)"),
+      storyPointValue: z5.number().optional().describe(SP_DESCRIPTION)
     },
     async ({ title, description, plan, ordinal, storyPointValue }) => {
       try {
@@ -3337,12 +3370,12 @@ function buildUpdateSubtaskTool(connection) {
     "update_subtask",
     "Update an existing subtask's fields",
     {
-      subtaskId: z3.string().describe("The subtask ID to update"),
-      title: z3.string().optional(),
-      description: z3.string().optional(),
-      plan: z3.string().optional(),
-      ordinal: z3.number().optional(),
-      storyPointValue: z3.number().optional().describe(SP_DESCRIPTION)
+      subtaskId: z5.string().describe("The subtask ID to update"),
+      title: z5.string().optional(),
+      description: z5.string().optional(),
+      plan: z5.string().optional(),
+      ordinal: z5.number().optional(),
+      storyPointValue: z5.number().optional().describe(SP_DESCRIPTION)
     },
     async ({ subtaskId, title, description, plan, storyPointValue }) => {
       try {
@@ -3365,7 +3398,7 @@ function buildDeleteSubtaskTool(connection) {
   return defineTool(
     "delete_subtask",
     "Delete a subtask",
-    { subtaskId: z3.string().describe("The subtask ID to delete") },
+    { subtaskId: z5.string().describe("The subtask ID to delete") },
     async ({ subtaskId }) => {
       try {
         await connection.call("deleteSubtask", {
@@ -3403,7 +3436,7 @@ function buildPackTools(connection) {
       "start_child_cloud_build",
       "Start a cloud build for a child task. The child must be in Open status with story points and an agent assigned.",
       {
-        childTaskId: z3.string().describe("The child task ID to start a cloud build for")
+        childTaskId: z5.string().describe("The child task ID to start a cloud build for")
       },
       async ({ childTaskId }) => {
         try {
@@ -3423,7 +3456,7 @@ function buildPackTools(connection) {
       "stop_child_build",
       "Stop a running cloud build for a child task. Sends a stop signal to the child agent.",
       {
-        childTaskId: z3.string().describe("The child task ID whose build should be stopped")
+        childTaskId: z5.string().describe("The child task ID whose build should be stopped")
       },
       async ({ childTaskId }) => {
         try {
@@ -3443,7 +3476,7 @@ function buildPackTools(connection) {
       "approve_and_merge_pr",
       "Approve and merge a child task's pull request. Only succeeds if all CI/CD checks are passing. Returns an error if checks are pending (retry after waiting) or failed (investigate). The child task must be in ReviewPR status.",
       {
-        childTaskId: z3.string().describe("The child task ID whose PR should be approved and merged")
+        childTaskId: z5.string().describe("The child task ID whose PR should be approved and merged")
       },
       async ({ childTaskId }) => {
         try {
@@ -3481,7 +3514,7 @@ function buildPmTools(connection, options) {
 }
 
 // src/tools/discovery-tools.ts
-import { z as z4 } from "zod";
+import { z as z6 } from "zod";
 var SP_DESCRIPTION2 = "Story point value (1=Common, 2=Magic, 3=Rare, 5=Unique)";
 function buildDiscoveryTools(connection) {
   return [
@@ -3489,10 +3522,10 @@ function buildDiscoveryTools(connection) {
       "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.",
       {
-        title: z4.string().optional().describe("The new task title"),
-        storyPointValue: z4.number().optional().describe(SP_DESCRIPTION2),
-        tagIds: z4.array(z4.string()).optional().describe("Array of tag IDs to assign"),
-        githubPRUrl: z4.string().url().optional().describe("GitHub pull request URL to link to this task")
+        title: z6.string().optional().describe("The new task title"),
+        storyPointValue: z6.number().optional().describe(SP_DESCRIPTION2),
+        tagIds: z6.array(z6.string()).optional().describe("Array of tag IDs to assign"),
+        githubPRUrl: z6.string().url().optional().describe("GitHub pull request URL to link to this task")
       },
       async ({ title, storyPointValue, tagIds, githubPRUrl }) => {
         try {
@@ -3521,14 +3554,14 @@ function buildDiscoveryTools(connection) {
 }
 
 // src/tools/code-review-tools.ts
-import { z as z5 } from "zod";
+import { z as z7 } from "zod";
 function buildCodeReviewTools(connection) {
   return [
     defineTool(
       "approve_code_review",
       "Approve the code review. Use this when the code passes all review criteria and is ready to merge.",
       {
-        summary: z5.string().describe("Brief summary of what was reviewed and why it looks good")
+        summary: z7.string().describe("Brief summary of what was reviewed and why it looks good")
       },
       async ({ summary }) => {
         const content = `**Code Review: Approved** :white_check_mark:
@@ -3551,15 +3584,15 @@ ${summary}`;
       "request_code_changes",
       "Request changes during code review. Use this when substantive issues are found that need to be fixed before merge.",
       {
-        issues: z5.array(
-          z5.object({
-            file: z5.string().describe("File path where the issue was found"),
-            line: z5.number().optional().describe("Line number (if applicable)"),
-            severity: z5.enum(["critical", "major", "minor"]).describe("Issue severity"),
-            description: z5.string().describe("What is wrong and how to fix it")
+        issues: z7.array(
+          z7.object({
+            file: z7.string().describe("File path where the issue was found"),
+            line: z7.number().optional().describe("Line number (if applicable)"),
+            severity: z7.enum(["critical", "major", "minor"]).describe("Issue severity"),
+            description: z7.string().describe("What is wrong and how to fix it")
           })
         ).describe("List of issues found during review"),
-        summary: z5.string().describe("Brief overall summary of the review findings")
+        summary: z7.string().describe("Brief overall summary of the review findings")
       },
       async ({ issues, summary }) => {
         const issueLines = issues.map((issue) => {
@@ -3589,10 +3622,10 @@ ${issueLines}`;
 }
 
 // src/tools/debug-tools.ts
-import { z as z8 } from "zod";
+import { z as z10 } from "zod";
 
 // src/tools/telemetry-tools.ts
-import { z as z6 } from "zod";
+import { z as z8 } from "zod";
 
 // src/debug/telemetry-injector.ts
 var BUFFER_SIZE = 200;
@@ -3987,12 +4020,12 @@ function buildGetTelemetryTool(manager) {
     "debug_get_telemetry",
     "Query structured telemetry events (HTTP requests, database queries, Socket.IO events, errors) captured from the running dev server. Returns filtered, structured data instead of raw logs.",
     {
-      type: z6.enum(["http", "db", "socket", "error"]).optional().describe("Filter by event type"),
-      urlPattern: z6.string().optional().describe("Regex pattern to filter HTTP events by URL"),
-      minDuration: z6.number().optional().describe("Minimum duration in ms \u2014 only return events slower than this"),
-      errorOnly: z6.boolean().optional().describe("Only return error events and HTTP 4xx/5xx responses"),
-      since: z6.number().optional().describe("Only return events after this timestamp (ms since epoch)"),
-      limit: z6.number().optional().describe("Max events to return (default: 20, from most recent)")
+      type: z8.enum(["http", "db", "socket", "error"]).optional().describe("Filter by event type"),
+      urlPattern: z8.string().optional().describe("Regex pattern to filter HTTP events by URL"),
+      minDuration: z8.number().optional().describe("Minimum duration in ms \u2014 only return events slower than this"),
+      errorOnly: z8.boolean().optional().describe("Only return error events and HTTP 4xx/5xx responses"),
+      since: z8.number().optional().describe("Only return events after this timestamp (ms since epoch)"),
+      limit: z8.number().optional().describe("Max events to return (default: 20, from most recent)")
     },
     async ({ type, urlPattern, minDuration, errorOnly, since, limit }) => {
       const clientOrErr = requireDebugClient(manager);
@@ -4062,7 +4095,7 @@ function buildTelemetryTools(manager) {
 }
 
 // src/tools/client-debug-tools.ts
-import { z as z7 } from "zod";
+import { z as z9 } from "zod";
 function requirePlaywrightClient(manager) {
   if (!manager.isClientDebugMode()) {
     return "Client debug mode is not active. Use debug_enter_mode with clientSide: true first.";
@@ -4082,11 +4115,11 @@ function buildClientBreakpointTools(manager) {
       "debug_set_client_breakpoint",
       "Set a breakpoint in client-side code running in the headless Chromium browser. V8 resolves source maps automatically, so original .tsx/.ts file paths work. Use this for React components, client utilities, and browser-side code.",
       {
-        file: z7.string().describe(
+        file: z9.string().describe(
           "Original source file path (e.g., src/components/App.tsx) \u2014 source maps resolve automatically"
         ),
-        line: z7.number().describe("Line number (1-based) in the original source file"),
-        condition: z7.string().optional().describe("JavaScript condition expression \u2014 breakpoint only triggers when truthy")
+        line: z9.number().describe("Line number (1-based) in the original source file"),
+        condition: z9.string().optional().describe("JavaScript condition expression \u2014 breakpoint only triggers when truthy")
       },
       async ({ file, line, condition }) => {
         const clientOrErr = requirePlaywrightClient(manager);
@@ -4108,7 +4141,7 @@ Breakpoint ID: ${breakpointId}${sourceMapNote}`
       "debug_remove_client_breakpoint",
       "Remove a previously set client-side breakpoint by its ID.",
       {
-        breakpointId: z7.string().describe("The breakpoint ID returned by debug_set_client_breakpoint")
+        breakpointId: z9.string().describe("The breakpoint ID returned by debug_set_client_breakpoint")
       },
       async ({ breakpointId }) => {
         const clientOrErr = requirePlaywrightClient(manager);
@@ -4188,8 +4221,8 @@ ${JSON.stringify(queuedHits, null, 2)}`
       "debug_evaluate_client",
       "Evaluate a JavaScript expression in the client-side browser context. When paused at a client breakpoint, evaluates in the paused scope. Can access DOM, window, React internals, etc.",
       {
-        expression: z7.string().describe("JavaScript expression to evaluate in the browser context"),
-        frameIndex: z7.number().optional().describe("Call stack frame index (0 = top frame). Defaults to the top frame.")
+        expression: z9.string().describe("JavaScript expression to evaluate in the browser context"),
+        frameIndex: z9.number().optional().describe("Call stack frame index (0 = top frame). Defaults to the top frame.")
       },
       async ({ expression, frameIndex }) => {
         const clientOrErr = requirePlaywrightClient(manager);
@@ -4262,7 +4295,7 @@ function buildClientInteractionTools(manager) {
       "debug_navigate_client",
       "Navigate the headless browser to a specific URL. Use this to reproduce specific flows or visit different pages.",
       {
-        url: z7.string().describe("URL to navigate to (e.g., http://localhost:3000/dashboard)")
+        url: z9.string().describe("URL to navigate to (e.g., http://localhost:3000/dashboard)")
       },
       async ({ url }) => {
         const clientOrErr = requirePlaywrightClient(manager);
@@ -4279,7 +4312,7 @@ function buildClientInteractionTools(manager) {
       "debug_click_client",
       "Click an element on the page in the headless browser. Use CSS selectors to target elements. Useful for reproducing bugs by interacting with the UI programmatically.",
       {
-        selector: z7.string().describe(
+        selector: z9.string().describe(
           "CSS selector of the element to click (e.g., 'button.submit', '#login-form input[type=submit]')"
         )
       },
@@ -4301,8 +4334,8 @@ function buildClientConsoleTool(manager) {
     "debug_get_client_console",
     "Get console messages captured from the headless browser. Includes console.log, warn, error, etc.",
     {
-      level: z7.string().optional().describe("Filter by console level: log, warn, error, info, debug"),
-      limit: z7.number().optional().describe("Maximum number of recent messages to return (default: all)")
+      level: z9.string().optional().describe("Filter by console level: log, warn, error, info, debug"),
+      limit: z9.number().optional().describe("Maximum number of recent messages to return (default: all)")
     },
     // oxlint-disable-next-line require-await
     async ({ level, limit }) => {
@@ -4329,8 +4362,8 @@ function buildClientNetworkTool(manager) {
     "debug_get_client_network",
     "Get network requests captured from the headless browser. Shows URLs, methods, status codes, and timing.",
     {
-      filter: z7.string().optional().describe("Regex pattern to filter requests by URL"),
-      limit: z7.number().optional().describe("Maximum number of recent requests to return (default: all)")
+      filter: z9.string().optional().describe("Regex pattern to filter requests by URL"),
+      limit: z9.number().optional().describe("Maximum number of recent requests to return (default: all)")
     },
     // oxlint-disable-next-line require-await
     async ({ filter, limit }) => {
@@ -4358,7 +4391,7 @@ function buildClientErrorsTool(manager) {
     "debug_get_client_errors",
     "Get uncaught errors captured from the headless browser. Includes error messages and source-mapped stack traces.",
     {
-      limit: z7.number().optional().describe("Maximum number of recent errors to return (default: all)")
+      limit: z9.number().optional().describe("Maximum number of recent errors to return (default: all)")
     },
     // oxlint-disable-next-line require-await
     async ({ limit }) => {
@@ -4452,12 +4485,12 @@ function buildDebugLifecycleTools(manager) {
       "debug_enter_mode",
       "Activate debug mode: restarts the dev server with Node.js --inspect flag and connects the CDP debugger. Optionally launch a headless Chromium browser for client-side debugging. Use serverSide for backend breakpoints, clientSide for frontend breakpoints, or both for full-stack.",
       {
-        hypothesis: z8.string().optional().describe("Your hypothesis about the bug \u2014 helps track debugging intent"),
-        serverSide: z8.boolean().optional().describe(
+        hypothesis: z10.string().optional().describe("Your hypothesis about the bug \u2014 helps track debugging intent"),
+        serverSide: z10.boolean().optional().describe(
           "Enable server-side Node.js debugging (default: true if clientSide is not set)"
         ),
-        clientSide: z8.boolean().optional().describe("Enable client-side browser debugging via headless Chromium + Playwright"),
-        previewUrl: z8.string().optional().describe(
+        clientSide: z10.boolean().optional().describe("Enable client-side browser debugging via headless Chromium + Playwright"),
+        previewUrl: z10.string().optional().describe(
           "Preview URL for client-side debugging (e.g., http://localhost:3000). Required when clientSide is true."
         )
       },
@@ -4493,9 +4526,9 @@ function buildBreakpointTools(manager) {
       "debug_set_breakpoint",
       "Set a breakpoint at the specified file and line number. Optionally provide a condition expression that must evaluate to true for the breakpoint to pause execution.",
       {
-        file: z8.string().describe("Absolute or relative file path to set the breakpoint in"),
-        line: z8.number().describe("Line number (1-based) to set the breakpoint on"),
-        condition: z8.string().optional().describe("JavaScript condition expression \u2014 breakpoint only triggers when truthy")
+        file: z10.string().describe("Absolute or relative file path to set the breakpoint in"),
+        line: z10.number().describe("Line number (1-based) to set the breakpoint on"),
+        condition: z10.string().optional().describe("JavaScript condition expression \u2014 breakpoint only triggers when truthy")
       },
       async ({ file, line, condition }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4517,7 +4550,7 @@ Breakpoint ID: ${breakpointId}`
       "debug_remove_breakpoint",
       "Remove a previously set breakpoint by its ID.",
       {
-        breakpointId: z8.string().describe("The breakpoint ID returned by debug_set_breakpoint")
+        breakpointId: z10.string().describe("The breakpoint ID returned by debug_set_breakpoint")
       },
       async ({ breakpointId }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4598,8 +4631,8 @@ ${JSON.stringify(queuedHits, null, 2)}`
       "debug_evaluate",
       "Evaluate a JavaScript expression in the current paused scope (or globally if not paused). When paused, use frameIndex to evaluate in a specific call frame.",
       {
-        expression: z8.string().describe("The JavaScript expression to evaluate"),
-        frameIndex: z8.number().optional().describe("Call stack frame index (0 = top frame). Defaults to the top frame.")
+        expression: z10.string().describe("The JavaScript expression to evaluate"),
+        frameIndex: z10.number().optional().describe("Call stack frame index (0 = top frame). Defaults to the top frame.")
       },
       async ({ expression, frameIndex }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4627,12 +4660,12 @@ function buildProbeManagementTools(manager) {
       "debug_add_probe",
       "Add a debug probe at a specific code location. Captures expression values each time the line executes \u2014 without pausing or modifying source files. Like console.log but better: structured, no diff pollution, auto-cleaned on debug exit.",
       {
-        file: z8.string().describe("File path to probe"),
-        line: z8.number().describe("Line number (1-based) to probe"),
-        expressions: z8.array(z8.string()).describe(
+        file: z10.string().describe("File path to probe"),
+        line: z10.number().describe("Line number (1-based) to probe"),
+        expressions: z10.array(z10.string()).describe(
           'JavaScript expressions to capture when the line executes (e.g., ["req.params.id", "user.role"])'
         ),
-        label: z8.string().optional().describe("Optional label for this probe (defaults to file:line)")
+        label: z10.string().optional().describe("Optional label for this probe (defaults to file:line)")
       },
       async ({ file, line, expressions, label }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4657,7 +4690,7 @@ Trigger the code path, then use debug_get_probe_results to see captured values.`
       "debug_remove_probe",
       "Remove a previously set debug probe by its ID.",
       {
-        probeId: z8.string().describe("The probe ID returned by debug_add_probe")
+        probeId: z10.string().describe("The probe ID returned by debug_add_probe")
       },
       async ({ probeId }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4697,9 +4730,9 @@ function buildProbeResultTools(manager) {
       "debug_get_probe_results",
       "Fetch captured probe hit data. Returns expression values from each time a probed line executed.",
       {
-        probeId: z8.string().optional().describe("Filter results by probe ID (resolves to its label)"),
-        label: z8.string().optional().describe("Filter results by probe label"),
-        limit: z8.number().optional().describe("Maximum number of recent hits to return (default: all)")
+        probeId: z10.string().optional().describe("Filter results by probe ID (resolves to its label)"),
+        label: z10.string().optional().describe("Filter results by probe label"),
+        limit: z10.number().optional().describe("Maximum number of recent hits to return (default: all)")
       },
       async ({ probeId, label, limit }) => {
         const clientOrErr = requireDebugClient2(manager);
@@ -4862,23 +4895,20 @@ function getModeTools(agentMode, connection, config, context) {
 }
 function buildConveyorTools(connection, config, context, agentMode, debugManager) {
   const effectiveMode = agentMode ?? context?.agentMode ?? void 0;
-  if (effectiveMode === "code-review") {
-    return [
-      buildReadTaskChatTool(connection),
-      buildGetTaskPlanTool(connection),
-      buildGetTaskTool(connection),
-      buildGetTaskCliTool(connection),
-      buildListTaskFilesTool(connection),
-      buildGetTaskFileTool(connection),
-      ...buildCodeReviewTools(connection)
-    ];
-  }
   const commonTools = buildCommonTools(connection, config);
   const modeTools = getModeTools(effectiveMode, connection, config, context);
   const discoveryTools = effectiveMode === "discovery" || effectiveMode === "auto" ? buildDiscoveryTools(connection) : [];
+  const codeReviewTools = effectiveMode === "review" ? buildCodeReviewTools(connection) : [];
   const emergencyTools = [buildForceUpdateTaskStatusTool(connection)];
   const debugTools = debugManager && effectiveMode === "building" ? buildDebugTools(debugManager) : [];
-  return [...commonTools, ...modeTools, ...discoveryTools, ...emergencyTools, ...debugTools];
+  return [
+    ...commonTools,
+    ...modeTools,
+    ...discoveryTools,
+    ...codeReviewTools,
+    ...emergencyTools,
+    ...debugTools
+  ];
 }
 function createConveyorMcpServer(harness, connection, config, context, agentMode, debugManager) {
   return harness.createMcpServer({
@@ -5297,7 +5327,6 @@ function collectMissingProps(taskProps) {
 // src/execution/tool-access.ts
 var PM_PLAN_FILE_TOOLS = /* @__PURE__ */ new Set(["Write", "Edit", "MultiEdit"]);
 var DESTRUCTIVE_CMD_PATTERN = /git\s+push\s+--force(?!\s*-with-lease)|git\s+reset\s+--hard|rm\s+-rf\s+\//;
-var CODE_REVIEW_WRITE_CMD_PATTERN = /git\s+push|git\s+commit|git\s+add|rm\s+|mv\s+|cp\s+|mkdir\s+|touch\s+|chmod\s+|chown\s+/;
 var RESOURCE_HEAVY_PATTERNS = [
   /\bbun\s+(install|i|add)\b/,
   /\bnpm\s+(install|ci|i)\b/,
@@ -5334,48 +5363,12 @@ function handleBuildingToolAccess(toolName, input) {
   }
   return { behavior: "allow", updatedInput: input };
 }
-function handleReviewToolAccess(toolName, input, isParentTask) {
-  if (isParentTask) {
-    return handleBuildingToolAccess(toolName, input);
-  }
-  if (PM_PLAN_FILE_TOOLS.has(toolName)) {
-    if (isPlanFile(input)) {
-      return { behavior: "allow", updatedInput: input };
-    }
-    return {
-      behavior: "deny",
-      message: "Review mode restricts file writes. Use bash to run tests and linting instead."
-    };
-  }
-  if (toolName === "Bash") {
-    const cmd = String(input.command ?? "");
-    if (DESTRUCTIVE_CMD_PATTERN.test(cmd)) {
-      return { behavior: "deny", message: "Destructive operation blocked in review mode." };
-    }
-  }
-  return { behavior: "allow", updatedInput: input };
-}
-function handleCodeReviewToolAccess(toolName, input) {
-  if (PM_PLAN_FILE_TOOLS.has(toolName)) {
-    return {
-      behavior: "deny",
-      message: "Code review mode is fully read-only. File writes are not permitted."
-    };
-  }
-  if (toolName === "Bash") {
-    const cmd = String(input.command ?? "");
-    if (DESTRUCTIVE_CMD_PATTERN.test(cmd) || CODE_REVIEW_WRITE_CMD_PATTERN.test(cmd)) {
-      return {
-        behavior: "deny",
-        message: "Code review mode is read-only. Write operations and destructive commands are blocked."
-      };
-    }
-  }
-  return { behavior: "allow", updatedInput: input };
+function handleReviewToolAccess(toolName, input) {
+  return handleBuildingToolAccess(toolName, input);
 }
 function handleAutoToolAccess(toolName, input, hasExitedPlanMode, isParentTask) {
   if (hasExitedPlanMode) {
-    return isParentTask ? handleReviewToolAccess(toolName, input, true) : handleBuildingToolAccess(toolName, input);
+    return isParentTask ? handleReviewToolAccess(toolName, input) : handleBuildingToolAccess(toolName, input);
   }
   return { behavior: "allow", updatedInput: input };
 }
@@ -5423,6 +5416,7 @@ async function handleExitPlanMode(host, input) {
     }
     if (host.agentMode === "discovery") {
       host.hasExitedPlanMode = true;
+      host.pendingModeRestart = true;
       void host.connection.triggerIdentification();
       host.connection.postChatMessage(
         "Plan complete. Running identification \u2014 icon and agent assignment will be set automatically."
@@ -5488,11 +5482,9 @@ function resolveToolAccess(host, toolName, input) {
     case "building":
       return handleBuildingToolAccess(toolName, input);
     case "review":
-      return handleReviewToolAccess(toolName, input, host.isParentTask);
+      return handleReviewToolAccess(toolName, input);
     case "auto":
       return handleAutoToolAccess(toolName, input, host.hasExitedPlanMode, host.isParentTask);
-    case "code-review":
-      return handleCodeReviewToolAccess(toolName, input);
     default:
       return { behavior: "allow", updatedInput: input };
   }
@@ -5571,13 +5563,10 @@ function buildHooks(host) {
   };
 }
 function isReadOnlyMode(mode, hasExitedPlanMode) {
-  return mode === "discovery" || mode === "help" || mode === "code-review" || mode === "auto" && !hasExitedPlanMode;
+  return mode === "discovery" || mode === "help" || mode === "auto" && !hasExitedPlanMode;
 }
 function buildDisallowedTools(settings, mode, hasExitedPlanMode) {
   const modeDisallowed = isReadOnlyMode(mode, hasExitedPlanMode) ? ["TodoWrite", "TodoRead", "NotebookEdit"] : [];
-  if (mode === "code-review") {
-    modeDisallowed.push("ExitPlanMode", "EnterPlanMode");
-  }
   const configured = settings.disallowedTools ?? [];
   const combined = [...configured, ...modeDisallowed];
   return combined.length > 0 ? combined : void 0;
@@ -5613,11 +5602,11 @@ function buildQueryOptions(host, context) {
     },
     sandbox: context.useSandbox ? { enabled: true } : { enabled: false },
     hooks: buildHooks(host),
-    maxTurns: mode === "code-review" ? Math.min(settings.maxTurns ?? 15, 15) : settings.maxTurns,
+    maxTurns: settings.maxTurns,
     effort: settings.effort,
     thinking: settings.thinking,
     betas: settings.betas,
-    maxBudgetUsd: mode === "code-review" ? Math.min(settings.maxBudgetUsd ?? 10, 10) : settings.maxBudgetUsd ?? 50,
+    maxBudgetUsd: settings.maxBudgetUsd ?? 50,
     abortController: host.abortController ?? void 0,
     disallowedTools: buildDisallowedTools(settings, mode, host.hasExitedPlanMode),
     enableFileCheckpointing: settings.enableFileCheckpointing,
@@ -6337,14 +6326,7 @@ var SessionRunner = class _SessionRunner {
   async executeInitialMode() {
     if (!this.taskContext || !this.fullContext) return false;
     const effectiveMode = this.mode.effectiveMode;
-    if (effectiveMode === "code-review") {
-      await this.setState("running");
-      await this.callbacks.onEvent({ type: "execute_mode", mode: effectiveMode });
-      await this.executeQuery();
-      this.stopped = true;
-      return true;
-    }
-    const shouldRun = effectiveMode === "building" || effectiveMode === "auto" || effectiveMode === "review" && this.mode.isPackRunner(this.taskContext);
+    const shouldRun = effectiveMode === "building" || effectiveMode === "auto" || effectiveMode === "review";
     if (shouldRun) {
       await this.setState("running");
       await this.callbacks.onEvent({ type: "execute_mode", mode: effectiveMode });
@@ -6393,6 +6375,36 @@ var SessionRunner = class _SessionRunner {
     }
   }
   // ── Stop / soft-stop ───────────────────────────────────────────────
+  /** Best-effort WIP commit + push on shutdown so in-flight work isn't lost
+   *  when a claudespace pod is killed. Must be called BEFORE stop() so the
+   *  connection is still alive for token refresh. Never throws. */
+  async flushGitOnShutdown() {
+    try {
+      const result = await flushPendingChanges(this.config.workspaceDir, {
+        wipMessage: "WIP: auto-commit on conveyor-agent shutdown",
+        refreshToken: async () => {
+          try {
+            const res = await this.connection.call("refreshGithubToken", {
+              sessionId: this.connection.sessionId
+            });
+            return res.token;
+          } catch {
+            return void 0;
+          }
+        }
+      });
+      if (result.hadWork) {
+        process.stderr.write(
+          `[conveyor-agent] Shutdown git flush: committed=${result.committed} pushed=${result.pushed}
+`
+        );
+      }
+    } catch (err) {
+      const msg = err instanceof Error ? err.message : String(err);
+      process.stderr.write(`[conveyor-agent] Shutdown git flush failed: ${msg}
+`);
+    }
+  }
   stop() {
     this.stopped = true;
     this.queryBridge?.stop();
@@ -6469,6 +6481,7 @@ var SessionRunner = class _SessionRunner {
     }
   }
   // ── Context & bridge construction ────────────────────────────────
+  // oxlint-disable-next-line complexity
   buildFullContext(ctx) {
     const chatHistory = mapChatHistory(ctx.chatHistory);
     return {
@@ -6495,7 +6508,8 @@ var SessionRunner = class _SessionRunner {
       projectTags: ctx.projectTags ?? void 0,
       taskTagIds: ctx.taskTagIds ?? void 0,
       projectObjectives: ctx.projectObjectives ?? void 0,
-      incidents: ctx.incidents ?? void 0
+      incidents: ctx.incidents ?? void 0,
+      agentSettings: ctx.agentSettings ?? null
     };
   }
   createQueryBridge() {
@@ -6545,6 +6559,12 @@ var SessionRunner = class _SessionRunner {
       if (action.type === "start_auto") {
         this.connection.emitModeChanged(this.mode.effectiveMode);
         this.softStop();
+      } else if (action.type === "restart_query") {
+        if (this.fullContext && action.newMode === "review") {
+          this.fullContext.claudeSessionId = null;
+        }
+        this.connection.emitModeChanged(action.newMode);
+        this.softStop();
       }
     });
     this.connection.onWake(() => this.lifecycle.wake());
@@ -7000,7 +7020,7 @@ import * as path from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 
 // src/tools/project-tools.ts
-import { z as z9 } from "zod";
+import { z as z11 } from "zod";
 function buildTaskListTools(connection) {
   const projectId = connection.projectId;
   return [
@@ -7008,9 +7028,9 @@ function buildTaskListTools(connection) {
       "list_tasks",
       "List tasks in the project. Optionally filter by status or assignee.",
       {
-        status: z9.string().optional().describe("Filter by task status"),
-        assigneeId: z9.string().optional().describe("Filter by assigned user ID"),
-        limit: z9.number().optional().describe("Max number of tasks to return (default 50)")
+        status: z11.string().optional().describe("Filter by task status"),
+        assigneeId: z11.string().optional().describe("Filter by assigned user ID"),
+        limit: z11.number().optional().describe("Max number of tasks to return (default 50)")
       },
       async (params) => {
         try {
@@ -7027,7 +7047,7 @@ function buildTaskListTools(connection) {
     defineTool(
       "get_task",
       "Get detailed information about a task including chat messages, child tasks, and session.",
-      { task_id: z9.string().describe("The task ID to look up") },
+      { task_id: z11.string().describe("The task ID to look up") },
       async ({ task_id }) => {
         try {
           const task = await connection.call("getProjectTask", { projectId, taskId: task_id });
@@ -7044,10 +7064,10 @@ function buildTaskListTools(connection) {
       "search_tasks",
       "Search tasks by tags, text query, or status filters.",
       {
-        tagNames: z9.array(z9.string()).optional().describe("Filter by tag names"),
-        searchQuery: z9.string().optional().describe("Text search in title/description"),
-        statusFilters: z9.array(z9.string()).optional().describe("Filter by statuses"),
-        limit: z9.number().optional().describe("Max results (default 20)")
+        tagNames: z11.array(z11.string()).optional().describe("Filter by tag names"),
+        searchQuery: z11.string().optional().describe("Text search in title/description"),
+        statusFilters: z11.array(z11.string()).optional().describe("Filter by statuses"),
+        limit: z11.number().optional().describe("Max results (default 20)")
       },
       async (params) => {
         try {
@@ -7107,11 +7127,11 @@ function buildMutationTools(connection) {
       "create_task",
       "Create a new task in the project.",
       {
-        title: z9.string().describe("Task title"),
-        description: z9.string().optional().describe("Task description"),
-        plan: z9.string().optional().describe("Implementation plan in markdown"),
-        status: z9.string().optional().describe("Initial status (default: Planning)"),
-        isBug: z9.boolean().optional().describe("Whether this is a bug report")
+        title: z11.string().describe("Task title"),
+        description: z11.string().optional().describe("Task description"),
+        plan: z11.string().optional().describe("Implementation plan in markdown"),
+        status: z11.string().optional().describe("Initial status (default: Planning)"),
+        isBug: z11.boolean().optional().describe("Whether this is a bug report")
       },
       async (params) => {
         try {
@@ -7128,12 +7148,12 @@ function buildMutationTools(connection) {
       "update_task",
       "Update an existing task's title, description, plan, status, or assignee.",
       {
-        task_id: z9.string().describe("The task ID to update"),
-        title: z9.string().optional().describe("New title"),
-        description: z9.string().optional().describe("New description"),
-        plan: z9.string().optional().describe("New plan in markdown"),
-        status: z9.string().optional().describe("New status"),
-        assignedUserId: z9.string().nullable().optional().describe("Assign to user ID, or null to unassign")
+        task_id: z11.string().describe("The task ID to update"),
+        title: z11.string().optional().describe("New title"),
+        description: z11.string().optional().describe("New description"),
+        plan: z11.string().optional().describe("New plan in markdown"),
+        status: z11.string().optional().describe("New status"),
+        assignedUserId: z11.string().nullable().optional().describe("Assign to user ID, or null to unassign")
       },
       async ({ task_id, ...fields }) => {
         try {
@@ -7502,6 +7522,32 @@ var ProjectRunner = class {
       this.resolveLifecycle = resolve2;
     });
   }
+  /** Best-effort WIP commit + push of every tracked working tree on shutdown.
+   *  Iterates the main project dir plus any active agent worktrees so nothing
+   *  pending is lost when the pod is killed. Never throws. */
+  async flushGitOnShutdown() {
+    const dirs = /* @__PURE__ */ new Set([this.projectDir]);
+    for (const agent of this.activeAgents.values()) {
+      if (agent.worktreePath) dirs.add(agent.worktreePath);
+    }
+    for (const dir of dirs) {
+      try {
+        const result = await flushPendingChanges(dir, {
+          wipMessage: "WIP: auto-commit on conveyor-agent shutdown"
+        });
+        if (result.hadWork) {
+          logger7.info("Shutdown git flush", {
+            dir,
+            committed: result.committed,
+            pushed: result.pushed
+          });
+        }
+      } catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        logger7.warn("Shutdown git flush failed", { dir, error: msg });
+      }
+    }
+  }
   async stop() {
     if (this.stopping) return;
     this.stopping = true;
@@ -7529,6 +7575,7 @@ var ProjectRunner = class {
         setTimeout(resolve2, STOP_TIMEOUT_MS + 5e3);
       })
     ]);
+    await this.flushGitOnShutdown();
     try {
       await this.connection.call("disconnectProjectRunner", {
         projectId: this.connection.projectId
@@ -7582,7 +7629,7 @@ var ProjectRunner = class {
   async handleAuditTags(request) {
     this.connection.emitStatus("busy");
     try {
-      const { handleTagAudit } = await import("./tag-audit-handler-L7YPDXTA.js");
+      const { handleTagAudit } = await import("./tag-audit-handler-BM6MMS5T.js");
       await handleTagAudit(request, this.connection, this.projectDir);
     } catch (error) {
       const msg = error instanceof Error ? error.message : String(error);
@@ -7987,6 +8034,7 @@ export {
   hasUnpushedCommits,
   stageAndCommit,
   updateRemoteToken,
+  flushPendingChanges,
   pushToOrigin,
   injectTelemetry,
   PlanSync,
@@ -8003,4 +8051,4 @@ export {
   loadForwardPorts,
   loadConveyorConfig
 };
-//# sourceMappingURL=chunk-QLY35BX7.js.map
+//# sourceMappingURL=chunk-AZ6CRVYN.js.map