@rallycry/conveyor-agent 10.3.1 → 10.4.0

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

@@ -2962,7 +2962,8 @@ function defineTool(name, description, schema, handler, options) {
     description,
     schema,
     handler,
-    annotations: options?.annotations
+    annotations: options?.annotations,
+    strict: options?.strict
   };
 }
 
@@ -3759,6 +3760,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 +3815,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;
   }
@@ -6078,7 +6085,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 +6517,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 +6551,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 +6596,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 +6619,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 +6681,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 +6719,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 +6745,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 +6772,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 +6817,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 +6849,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 +6942,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 +6964,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 +6986,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 +7016,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 +7049,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 +7083,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 +7096,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 +7153,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 +7205,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 +7230,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 +7253,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 +7276,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 +7297,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 +7318,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,7 +7356,7 @@ 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.";
 function buildUpdateTaskTool(connection) {
@@ -7355,8 +7364,8 @@ function buildUpdateTaskTool(connection) {
     "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,12 +7386,12 @@ 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)
     },
     async ({ title, description, plan, ordinal, storyPointValue, followParentStatus }) => {
       try {
@@ -7409,13 +7418,13 @@ function buildUpdateSubtaskTool(connection) {
     "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.",
     {
-      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)
     },
     async ({ subtaskId, title, description, plan, storyPointValue, followParentStatus }) => {
       try {
@@ -7439,7 +7448,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 +7486,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 +7506,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 +7526,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 +7564,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 +7608,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 +7631,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 +7655,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) => {
@@ -11026,4 +11046,4 @@ export {
   runStartCommand,
   unshallowRepo
 };
-//# sourceMappingURL=chunk-HAH2S5AD.js.map
+//# sourceMappingURL=chunk-TZYEU4QE.js.map