@rallycry/conveyor-agent 11.0.26 → 11.0.28

package/dist/{chunk-4L22UNZD.js → chunk-ND5INHM4.js}

@@ -47,13 +47,14 @@ import {
   Lifecycle,
   MAX_CI_WAIT_TIMEOUT_MINUTES,
   MAX_FILE_SIZE_BYTES,
+  MAX_MANUAL_TESTS_PER_TASK,
+  MAX_MANUAL_TEST_TITLE_LENGTH,
   PM_CHAT_HISTORY_LIMIT,
   PRE_BUILD_TASK_STATUSES,
   PTY_STREAM_PORT_ATTEMPTS,
   PTY_STREAM_PORT_BASE,
   PortDiscovery,
   PtyStreamFrameReader,
-  ReviewGuideContentSchema,
   SEVERITY_ENUM,
   TAG_DESCRIPTION_MAX,
   TAG_OVERVIEW_MAX,
@@ -63,17 +64,20 @@ import {
   awaitGitReady,
   clearForceFreshCooldown,
   createServiceLogger,
+  describeManualTestRejection,
   encodePtyStreamFrame,
   ensureOnTaskBranch,
   flushAllPendingWork,
   flushPendingChanges,
   forceFreshCooldownNotice,
   forceFreshMintBlocked,
+  formatEvaluateQuestionsResult,
   getCurrentBranch,
   git,
   hasTaskPlan,
   hasUncommittedChanges,
   hasUnpushedCommits,
+  isCapabilityServableMime,
   isCodexReasoningEffort,
   isPlaceholderLocator,
   locatorMatchesContent,
@@ -91,7 +95,7 @@ import {
   statWorkspacePath,
   updateRemoteToken,
   verifyGitCredential
-} from "./chunk-BZZ4B37Q.js";
+} from "./chunk-GHNYP67H.js";
 import {
   registerBootMilestoneSocketFallback,
   reportBootMilestone
@@ -5440,6 +5444,20 @@ function formatChatFile(file) {
   }
   return [`[Attached: ${file.fileName} (${file.mimeType}${sizeStr})]`];
 }
+function formatParentTaskFiles(parent) {
+  const lines = [
+    `
+## Files on the parent card ${parent.slug}`,
+    `These are attached to the parent card "${parent.title}", not to this one. Read one with get_attachment(fileId, task_id: "${parent.slug}"); list_task_files(task_id: "${parent.slug}") shows the current set.`
+  ];
+  for (const file of parent.files) {
+    const size = formatFileSize(file.fileSize);
+    lines.push(
+      `- **${file.fileName}** (${file.mimeType}${size ? `, ${size}` : ""}) fileId: ${file.fileId}`
+    );
+  }
+  return lines;
+}
 function formatTaskFile(file) {
   if (file.content && file.contentEncoding === "utf-8") {
     return [`
@@ -5792,7 +5810,7 @@ var PACK_RUNNER_FOOTER = [
   ``,
   `Your turn output appears ONLY in the live agent terminal \u2014 it is NOT posted to the task chat. The team does NOT see your replies unless you post them.`,
   `Use post_to_chat (omit task_id \u2192 this task's chat) to report which child you are on, status as you work the pack, and any blocker or escalation the team needs to act on.`,
-  `Pass task_id only to message a DIFFERENT task's chat (e.g. a child task).`,
+  `Pass task_id only to reach a DIFFERENT card: a child task's chat on post_to_chat and read_task_chat, or a child's files on list_task_files and get_attachment.`,
   `Use read_task_chat only if you need to re-read earlier messages beyond the chat context above.`,
   ``,
   `If a Conveyor tool call fails or reports the MCP server is unavailable/disconnected, this is almost always a transient socket reconnect \u2014 RETRY the same call; it will succeed once the connection re-establishes. A tool error is NOT a reason to go idle or stop the pack. (When a turn may end is the pack path's call \u2014 see its goal and finish line.)`,
@@ -6395,7 +6413,7 @@ ${context.agentInstructions}`);
 Your turn output appears ONLY in the live agent terminal \u2014 it is NOT posted to the task chat. The team does NOT see your replies unless you post them.`,
     `Use post_to_chat (omit task_id \u2192 your own task's chat) to share meaningful status as you work, a summary when you finish, and any blocker or question you need a human to answer.`,
     `A question you need answered goes through AskUserQuestion \u2014 that is what pages a person. post_to_chat with milestone: "blocked" records that you are parked and does not page anyone.`,
-    `Pass task_id only to message a DIFFERENT task's chat (e.g. a parent task via get_task).`,
+    `Pass task_id only to reach a DIFFERENT card: a child task's chat on post_to_chat and read_task_chat, or any card in this project's files on list_task_files and get_attachment (a pack parent's source documents, for example). Omit it for your own card.`,
     `Use read_task_chat only if you need to re-read earlier messages beyond the chat context above.`,
     `
 If a Conveyor tool call fails or reports the MCP server is unavailable/disconnected, this is almost always a transient socket reconnect \u2014 simply RETRY the same call and it will succeed once the connection re-establishes. Do NOT stop work, go idle, or treat a tool error as a reason to end your turn. Keep making progress on the code; only post_to_chat to ask for a human if a tool keeps failing across several retries over a sustained period.`,
@@ -6570,6 +6588,9 @@ ${truncatePlanForPrompt(context.plan)}`);
       parts.push(...formatTaskFile(file));
     }
   }
+  if (context.parentTaskFiles && context.parentTaskFiles.files.length > 0) {
+    parts.push(...formatParentTaskFiles(context.parentTaskFiles));
+  }
   if (context.repoRefs && context.repoRefs.length > 0) {
     parts.push(...formatRepoRefs(context.repoRefs));
   }
@@ -6730,6 +6751,7 @@ function compileNumber(z11, spec) {
 function compileArray(z11, spec) {
   let schema = z11.array(compileField(z11, spec.item));
   if (spec.min !== void 0) schema = schema.min(spec.min);
+  if (spec.max !== void 0) schema = schema.max(spec.max);
   return schema;
 }
 function compileBase(z11, spec) {
@@ -6859,16 +6881,24 @@ var readTaskChatContract = defineToolContract({
     }
   }
 });
+var listTagsDetail = f.optional(
+  f.enum(["summary", "full"], {
+    desc: `"summary" (default) returns one compact row per tag with a contextPathCount. "full" also ships each tag's contextPaths inline (each link with its verification status). Full mode can exceed a client's tool-output budget on a large glossary; use get_tag for one term instead.`
+  })
+);
 var listTagsContract = defineToolContract({
   name: "list_tags",
   agent: {
-    description: "List the project glossary: every tag's id, name, color, description, parent/child tag ids, its contextPaths (the rule/doc/file/folder links it wires into agent context \u2014 `[]` means none), whether it carries a full overview (fetch that with get_tag), and its attachmentCount (files labelled as examples of the term). The context links ship inline, so you only need get_tag for a term's full overview. Use the ids with get_tag / update_tag.",
-    fields: {}
+    description: "List the whole project glossary in one call. The default summary gives every tag's id, name, color, description, parent/child tag ids, whether it carries a full overview (hasOverview), its attachmentCount (files labelled as examples of the term), and its contextPathCount (the rule/doc/file/folder links it wires into agent context \u2014 `0` means none). Pass detail: \"full\" to ship the contextPaths themselves inline. Use get_tag for one term's full overview and full link verification. Use the ids with get_tag / update_tag.",
+    fields: {
+      detail: listTagsDetail
+    }
   },
   mcp: {
-    description: "List all project tags with names, IDs, colors, descriptions, hierarchy (parent/child ids), contextPaths (the rule/doc/file/folder links each tag wires into agent context \u2014 `[]` means none), a hasOverview flag, and an attachmentCount (files labelled as examples of the term \u2014 read the tiles with list_tag_attachments). Context links ship inline; call get_tag only for a term's full overview. Pass projectId to target a specific project; otherwise the configured default project is used.",
+    description: 'List all project tags in one call. The default summary gives names, IDs, colors, descriptions, hierarchy (parent/child ids), a hasOverview flag, an attachmentCount (files labelled as examples of the term \u2014 read the tiles with list_tag_attachments), and a contextPathCount (the rule/doc/file/folder links each tag wires into agent context \u2014 `0` means none). Pass detail: "full" to ship the contextPaths themselves inline. Call get_tag for one term\'s full overview and full link verification. Pass projectId to target a specific project; otherwise the configured default project is used.',
     fields: {
-      projectId: mcpProjectId
+      projectId: mcpProjectId,
+      detail: listTagsDetail
     }
   }
 });
@@ -7004,7 +7034,7 @@ var updateTaskContract = defineToolContract({
     }
   },
   mcp: {
-    description: "Update task fields: title, description, plan, status, risk, story points, assignment, githubBranch, or tags. Set status to claim a card (InProgress), triage it (Open), or cancel it (Cancelled); for review approvals prefer approve_task / request_changes, which guard against stale-state races. Tags are additive/subtractive \u2014 pass addTags/removeTags with tag names (not a replace-set). Pass projectId to target a specific project; otherwise the configured default project is used. Moving a task beyond Planning auto-fills any missing icon, story points, and agent assignment \u2014 don't spend turns on them; pass storyPointValue only to correct the sizing yourself. For subtasks use update_subtask.",
+    description: "Update task fields: title, description, plan, status, risk, story points, assignment, githubBranch, onHold, or tags. Set status to claim a card (InProgress), triage it (Open), or cancel it (Cancelled); for review approvals prefer approve_task / request_changes, which guard against stale-state races. Set onHold to park a card (true) or bring it back to the active board (false) without cancelling it. Tags are additive/subtractive \u2014 pass addTags/removeTags with tag names (not a replace-set). Pass projectId to target a specific project; otherwise the configured default project is used. Moving a task beyond Planning auto-fills any missing icon, story points, and agent assignment \u2014 don't spend turns on them; pass storyPointValue only to correct the sizing yourself. For subtasks use update_subtask.",
     fields: {
       projectId: mcpProjectId,
       taskId: f.string({ desc: "The task ID" }),
@@ -7035,6 +7065,11 @@ var updateTaskContract = defineToolContract({
         )
       ),
       githubBranch: f.optional(f.nullable(f.string({ desc: GITHUB_BRANCH_DESC }))),
+      onHold: f.optional(
+        f.boolean({
+          desc: "Park the card on hold (true) or return it to the active board (false). On hold hides the card from the active columns without cancelling it; a card already at an active work status is demoted to Open when parked."
+        })
+      ),
       addTags: f.optional(
         f.array(f.string(), {
           desc: 'Tag names to add to the card (e.g. ["refactor"]). Additive \u2014 existing tags are kept. Unknown names are rejected; use list_tags to see available tags or manage_tags to create one.'
@@ -7078,9 +7113,20 @@ var testStatuses = f.optional(
   })
 );
 var setManualTestItems = f.array(
-  f.object({ title: f.string({ min: 1, desc: "A concise, actionable test step" }) }),
-  { min: 1, desc: "List of manual test steps to add" }
+  f.object({
+    title: f.string({
+      min: 1,
+      max: MAX_MANUAL_TEST_TITLE_LENGTH,
+      desc: "One sentence a non-engineer can follow: where to go, what to do, what they should see. No commands, no code."
+    })
+  }),
+  {
+    min: 1,
+    max: MAX_MANUAL_TESTS_PER_TASK,
+    desc: `Manual tests to add. A card holds at most ${MAX_MANUAL_TESTS_PER_TASK}; most changes need 1, and a change with no user-visible behavior needs none.`
+  }
 );
+var setManualTestsBar = 'A manual test is a real user path in plain language ("Open a card with a PR: the Pull Request slat sits directly under Description"), not an engineering check. Never record lint, typecheck, unit tests, CI, curl, database, log, or deploy steps; those run automatically. Record at most 5 per card, usually 1, and none when the change has no user-visible behavior. Existing items with the same title are skipped; items that name an automated step or exceed the cap are rejected and reported back.';
 var titleToEdit = f.string({ min: 1, desc: "The current title of the manual test to edit" });
 var newTitle = f.string({ min: 1, desc: "The new title for the manual test" });
 var titleToRemove = f.string({ min: 1, desc: "The title of the manual test to remove" });
@@ -7134,13 +7180,13 @@ var queryManualTestsContract = defineToolContract({
 var setManualTestsContract = defineToolContract({
   name: "set_manual_tests",
   agent: {
-    description: "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.",
+    description: `Record the manual tests a person should run in the app to sign off on this card. ${setManualTestsBar}`,
     fields: {
       items: setManualTestItems
     }
   },
   mcp: {
-    description: "Add manual test steps to a task's checklist. Pass projectId to target a specific project; otherwise the configured default project is used. Existing items with the same title are automatically skipped (deduplication). Use to record specific manual verification steps that reviewers should follow when testing the task's PR.",
+    description: `Record the manual tests a person should run in the app to sign off on a card. Pass projectId to target a specific project; otherwise the configured default project is used. ${setManualTestsBar}`,
     fields: {
       projectId: mcpProjectId,
       taskId: mcpChecklistTaskId,
@@ -7463,11 +7509,18 @@ var subtasksContracts = [
   setTaskParentContract
 ];
 var mcpTaskIdOrSlug = f.string({ desc: "The task ID or slug" });
+var agentTaskIdOrSlug = f.optional(
+  f.string({
+    desc: "Omit for the current card; pass a parent, sibling, or any other card in this project (id or slug) to read its files instead."
+  })
+);
 var listTaskFilesContract = defineToolContract({
   name: "list_task_files",
   agent: {
-    description: "List all files attached to this task with metadata. Use before fetching a specific file to see what is available and how large each is. For file contents use get_attachment.",
-    fields: {}
+    description: "List all files attached to this task with metadata. Use before fetching a specific file to see what is available and how large each is. For file contents use get_attachment. Pass task_id to list another card's files in this project, for example the pack parent's source documents.",
+    fields: {
+      task_id: agentTaskIdOrSlug
+    }
   },
   mcp: {
     description: "List all files attached to a task with metadata (no contents \u2014 fast and small). Pass projectId to target a specific project; otherwise the configured default project is used. Use before fetching a specific file to see what is available and how large each is. For file contents use get_attachment.",
@@ -7480,13 +7533,14 @@ var listTaskFilesContract = defineToolContract({
 var getAttachmentContract = defineToolContract({
   name: "get_attachment",
   agent: {
-    description: "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.",
+    description: "Fetch one task file's content plus metadata by file ID. This is the way to read attachment bytes: a document `downloadUrl` (anything that is not an image or video) answers 404 without a signed-in project member's session, so do not curl or WebFetch it from the pod. Call list_task_files first to discover IDs and check sizes \u2014 large binaries may be truncated by the service's size limit. Pass task_id when the file is attached to another card in this project, such as the pack parent.",
     fields: {
-      fileId: f.string({ desc: "The file ID to retrieve" })
+      fileId: f.string({ desc: "The file ID to retrieve" }),
+      task_id: agentTaskIdOrSlug
     }
   },
   mcp: {
-    description: "Fetch one task file's content plus metadata by file ID (accepts task id or slug). Pass projectId to target a specific project; otherwise the configured default project is used. Images are returned as viewable image blocks. Large text files (logs, JSON) are returned in pages \u2014 use `offset`/`maxBytes` to read more, or fetch `downloadUrl` for the whole file. Call list_task_files first to discover IDs and sizes.",
+    description: "Fetch one task file's content plus metadata by file ID (accepts task id or slug). Pass projectId to target a specific project; otherwise the configured default project is used. Images are returned as viewable image blocks. Large text files (logs, JSON) are returned in pages \u2014 use `offset`/`maxBytes` to read more. This tool is the way to read bytes: a document `downloadUrl` (anything that is not an image or video) answers 404 unless the request carries a signed-in project member's session, so do not curl or fetch it. Call list_task_files first to discover IDs and sizes.",
     fields: {
       projectId: mcpProjectId,
       taskId: mcpTaskIdOrSlug,
@@ -7553,10 +7607,29 @@ var uploadAttachmentContract = defineToolContract({
     }
   }
 });
+var DELETE_ATTACHMENT_WARNING = "Permanent, with no trash and no undo (unlike drive_delete_file): the file and its bytes are gone, and the chat message that announced it keeps its text but loses the attachment. Use it to remove a superseded upload once its replacement is attached. `fileId` comes from list_task_files.";
+var deleteAttachmentContract = defineToolContract({
+  name: "delete_attachment",
+  agent: {
+    description: `Delete one attachment from this task. ${DELETE_ATTACHMENT_WARNING} A pod can only delete files on its own card.`,
+    fields: {
+      fileId: f.string({ desc: "The file ID to delete" })
+    }
+  },
+  mcp: {
+    description: `Delete one attachment from a task (accepts task id or slug). ${DELETE_ATTACHMENT_WARNING} Requires Moderate on the project, the same bar as the web Files delete button. Pass projectId to target a specific project; otherwise the configured default project is used.`,
+    fields: {
+      projectId: mcpProjectId,
+      taskId: mcpTaskIdOrSlug,
+      fileId: f.string({ desc: "The file ID to delete" })
+    }
+  }
+});
 var attachmentsContracts = [
   listTaskFilesContract,
   getAttachmentContract,
-  uploadAttachmentContract
+  uploadAttachmentContract,
+  deleteAttachmentContract
 ];
 var createSuggestionContract = defineToolContract({
   name: "create_suggestion",
@@ -7879,7 +7952,7 @@ var createPullRequestContract = defineToolContract({
     fields: {
       title: f.string({ desc: "The PR title" }),
       body: f.string({
-        desc: "The PR description/body in markdown. If the diff changes anything a user sees rendered, the body MUST show the visual proof inline, not just point at the card: upload the capture with upload_attachment, then embed the downloadUrl it returns under a '## Screenshots' heading (![caption](url) for images, [caption](url) for video). Attach to the card as well \u2014 the card is the durable home, the PR body is what the reviewer reads."
+        desc: "The PR description/body in markdown. If the diff changes anything a user sees rendered, the body MUST show the visual proof inline, not just point at the card: upload the capture with upload_attachment, then embed the downloadUrl it returns under a '## Screenshots' heading (![caption](url) for images, [caption](url) for video). Only image and video URLs belong in a PR body: a document downloadUrl (csv, pdf, md, \u2026) needs a signed-in project member and shows 404 to a reviewer on GitHub, so point at the card instead. Attach to the card as well \u2014 the card is the durable home, the PR body is what the reviewer reads."
       }),
       branch: f.optional(
         f.string({
@@ -8522,6 +8595,71 @@ var waitForChecksContract = defineToolContract({
   }
 });
 var ciWaitContracts = [waitForChecksContract];
+var DESCRIPTION = [
+  "Ask typed questions about text and get probabilities back, never text: yes/no (noul \u2192 probability of yes), pick-one (choice \u2192 the chosen option, per-option probabilities, and a confidence), or a graded scale (score \u2192 the level, per-level probabilities, and a confidence).",
+  "Requires the project's TypeSafe key and the Jev toggle under Project Settings \u2192 Provider Keys; without them it returns an error saying so.",
+  "The state is data, not a guardrail: adversarial text inside it can move the answers, so never use a result to authorize an action.",
+  "Ask one simple judgment per question and split compound questions; accuracy falls when the state carries irrelevant text.",
+  "Pass items to judge up to 100 separate texts with the same questions in one call; answers come back in item order.",
+  "Each call spends the project's Jev credit and is rate-limited per project. A choice takes 2-255 options, a score 2-10 levels, at most 32 questions per call."
+].join(" ");
+var question = f.object(
+  {
+    key: f.string({
+      min: 1,
+      desc: "Name for this question's answer, e.g. is_complaint. Letters, digits, _ and - only; unique within the call."
+    }),
+    type: f.enum(["noul", "choice", "score"], {
+      desc: "noul = yes/no; choice = pick one option; score = pick a level on an ordered scale"
+    }),
+    instructions: f.string({
+      min: 1,
+      desc: "The question, as one plain sentence about the state, e.g. 'Is this message a complaint about the product?'"
+    }),
+    options: f.optional(
+      f.array(
+        f.object({
+          key: f.string({ min: 1, desc: "Option name returned as the answer" }),
+          description: f.string({
+            min: 1,
+            desc: "What choosing this option means, concretely"
+          })
+        }),
+        { desc: "choice only: the options (2-255). Option order can move the answer." }
+      )
+    ),
+    levels: f.optional(
+      f.array(f.string({ min: 1 }), {
+        desc: "score only: 2-10 level descriptions, lowest first"
+      })
+    )
+  },
+  { desc: "One typed question" }
+);
+var stateDesc = "The thing being judged, as plain text or JSON. Required unless items is given; with items it is ignored.";
+var itemsDesc = "Up to 100 separate texts (each up to 8,000 characters), each judged on its own with the same questions. Use this for bulk classification instead of one call per item.";
+var questionsDesc = "1-32 questions, all answered independently against the same state.";
+var evaluateQuestionsContract = defineToolContract({
+  name: "evaluate_questions",
+  agent: {
+    description: DESCRIPTION,
+    fields: {
+      state: f.optional(f.string({ desc: stateDesc })),
+      questions: f.array(question, { min: 1, desc: questionsDesc }),
+      items: f.optional(f.array(f.string(), { desc: itemsDesc }))
+    }
+  },
+  mcp: {
+    description: `${DESCRIPTION} Pass projectId to target a specific project; otherwise the configured default project is used.`,
+    fields: {
+      projectId: mcpProjectId,
+      state: f.optional(f.string({ desc: stateDesc })),
+      questions: f.array(question, { min: 1, desc: questionsDesc }),
+      items: f.optional(f.array(f.string(), { desc: itemsDesc }))
+    }
+  }
+});
+var evaluateContracts = [evaluateQuestionsContract];
 var TOOL_CONTRACTS = Object.fromEntries(
   [
     ...tasksContracts,
@@ -8538,7 +8676,8 @@ var TOOL_CONTRACTS = Object.fromEntries(
     ...driveContracts,
     ...meetingsContracts,
     ...logsContracts,
-    ...ciWaitContracts
+    ...ciWaitContracts,
+    ...evaluateContracts
   ].map((contract) => [contract.name, contract])
 );
 
@@ -8736,17 +8875,20 @@ function buildGetExecutionLogsTool(connection) {
 function buildListTaskFilesTool(connection) {
   return defineContractTool(
     listTaskFilesContract,
-    async () => {
+    async ({ task_id }) => {
       try {
         const files = await connection.call("getTaskFiles", {
-          sessionId: connection.sessionId
+          sessionId: connection.sessionId,
+          ...task_id ? { taskId: task_id } : {}
         });
         const metadata = files.map((file) => {
           const { content: _c, ...rest } = file;
           return rest;
         });
+        const heading = task_id ? `Files on card ${task_id}:
+` : "";
         const content = [
-          { type: "text", text: JSON.stringify(metadata, null, 2) }
+          { type: "text", text: heading + JSON.stringify(metadata, null, 2) }
         ];
         for (const file of files) {
           if (file.content && file.contentEncoding === "base64" && isImageMimeType(file.mimeType)) {
@@ -8764,11 +8906,12 @@ function buildListTaskFilesTool(connection) {
 function buildGetAttachmentTool(connection) {
   return defineContractTool(
     getAttachmentContract,
-    async ({ fileId }) => {
+    async ({ fileId, task_id }) => {
       try {
         const file = await connection.call("getTaskFile", {
           sessionId: connection.sessionId,
-          fileId
+          fileId,
+          ...task_id ? { taskId: task_id } : {}
         });
         const { content: rawContent, ...metadata } = file;
         const content = [
@@ -9355,6 +9498,9 @@ var MIME_BY_EXT = {
   ".yaml": "application/yaml",
   ".yml": "application/yaml",
   ".pdf": "application/pdf",
+  ".docx": "application/vnd.openxmlformats-officedocument.wordprocessingml.document",
+  ".xlsx": "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
+  ".pptx": "application/vnd.openxmlformats-officedocument.presentationml.presentation",
   ".zip": "application/zip",
   ".gz": "application/gzip",
   ".mp4": "video/mp4",
@@ -9377,11 +9523,21 @@ var EMBEDDABLE_IN_MARKDOWN = /* @__PURE__ */ new Set([".png", ".jpg", ".jpeg", "
 function describeEmbed(fileName, downloadUrl) {
   if (!downloadUrl) return "";
   const isImage = EMBEDDABLE_IN_MARKDOWN.has(extname(fileName).toLowerCase());
-  const snippet = isImage ? `![${fileName}](${downloadUrl})` : `[${fileName}](${downloadUrl})`;
-  return `
+  if (isImage) {
+    return `
 
 Paste this into the PR description to show it inline:
-${snippet}`;
+![${fileName}](${downloadUrl})`;
+  }
+  if (isCapabilityServableMime(inferMimeType(fileName))) {
+    return `
+
+Paste this into the PR description to link it:
+[${fileName}](${downloadUrl})`;
+  }
+  return `
+
+downloadUrl (signed-in project members only; agents read it with get_attachment): ${downloadUrl}`;
 }
 function buildUploadAttachmentTool(connection, config) {
   return defineContractTool(uploadAttachmentContract, async ({ path, title, comment, tags }) => {
@@ -9434,6 +9590,23 @@ function buildUploadAttachmentTool(connection, config) {
     }
   });
 }
+function buildDeleteAttachmentTool(connection) {
+  return defineContractTool(deleteAttachmentContract, async ({ fileId }) => {
+    try {
+      const result = await connection.call("deleteTaskFile", {
+        sessionId: connection.sessionId,
+        fileId
+      });
+      return textResult(
+        `Deleted ${result.fileName} (${result.id}) permanently. Any chat message that announced it keeps its text without the attachment.`
+      );
+    } catch (error) {
+      return textResult(
+        `Failed to delete attachment: ${error instanceof Error ? error.message : "Unknown error"}`
+      );
+    }
+  });
+}
 
 // src/tools/checklist-tools.ts
 function buildListManualTestsTool(connection) {
@@ -9509,6 +9682,7 @@ function buildSetManualTestsTool(connection) {
       });
       const parts = [`Created ${result.created} manual test item(s).`];
       if (result.skipped > 0) parts.push(`Skipped ${result.skipped} duplicate(s).`);
+      for (const item of result.rejected ?? []) parts.push(describeManualTestRejection(item));
       return textResult(parts.join(" "));
     } catch (error) {
       const msg = error instanceof Error ? error.message : "Unknown error";
@@ -9630,7 +9804,8 @@ function buildCommonTools(connection, config) {
     buildApproveManualTestTool(connection),
     buildRejectManualTestTool(connection),
     ...buildMutationTools(connection, config),
-    buildUploadAttachmentTool(connection, config)
+    buildUploadAttachmentTool(connection, config),
+    buildDeleteAttachmentTool(connection)
   ];
 }
 
@@ -10069,9 +10244,9 @@ async function rejectBadContextPaths(contextPaths, workspaceDir) {
 function buildListTagsTool(connection, projectId) {
   return defineContractTool(
     listTagsContract,
-    async () => {
+    async ({ detail }) => {
       try {
-        const tags = await connection.call("listProjectTags", { projectId });
+        const tags = await connection.call("listProjectTags", { projectId, detail });
         return textResult(JSON.stringify(tags, null, 2));
       } catch (error) {
         return errText("Failed to list tags", error);
@@ -10936,6 +11111,31 @@ function settleTools(connection, projectId) {
   ];
 }
 
+// src/tools/evaluate-tools.ts
+function buildEvaluateTools(connection, projectId) {
+  return [
+    defineContractTool(
+      evaluateQuestionsContract,
+      async (input) => {
+        try {
+          const res = await connection.call("evaluateQuestions", {
+            projectId,
+            state: input.state,
+            items: input.items,
+            questions: input.questions
+          });
+          return textResult(formatEvaluateQuestionsResult(res));
+        } catch (error) {
+          return textResult(
+            `evaluate_questions failed: ${error instanceof Error ? error.message : "Unknown error"}`
+          );
+        }
+      },
+      { annotations: { readOnlyHint: true } }
+    )
+  ];
+}
+
 // src/tools/connected-tools.ts
 function connectedToolsFor(connection, context) {
   const projectId = context?.projectId;
@@ -10948,6 +11148,9 @@ function connectedToolsFor(connection, context) {
     // Always available: the decision tables exist on every project, and a fork
     // an agent cannot file is a fork it assumes its way past instead.
     ...buildDecisionTools(connection, projectId),
+    // Ungated: whether the project has Jev on is answered by calling it, and
+    // the error names what an admin has to switch on.
+    ...buildEvaluateTools(connection, projectId),
     ...context.googleAnalyticsConfigured ? [buildAnalyticsSummaryTool(connection, projectId)] : [],
     ...context.gcpLogsConfigured ? [buildQueryGcpLogsTool(connection, projectId)] : [],
     ...context.grafanaLogsConfigured ? [buildQueryGrafanaLogsTool(connection, projectId)] : []
@@ -10955,8 +11158,6 @@ function connectedToolsFor(connection, context) {
 }
 
 // src/tools/code-review-tools.ts
-import { execFile } from "child_process";
-import { promisify } from "util";
 import { z as z10 } from "zod";
 async function endReviewSession(connection, reason) {
   await connection.call("endReviewSession", {
@@ -10967,101 +11168,6 @@ async function endReviewSession(connection, reason) {
 var RISK_LEVELS = ["critical", "high", "medium", "low"];
 var reviewedShaSchema = z10.string().regex(/^[0-9a-f]{40}$/i).describe("REQUIRED. The full 40-character commit SHA this verdict reviews.");
 var riskDescription = "REQUIRED. The risk level this change carries, judged by the surface area it touches: critical = touches critical/foundational surface, high = important surface, medium = moderate, low = small/isolated. Set this on every verdict. You have authority to override a risk level already set on the task if you disagree with it.";
-var ReviewGuideToolSchema = z10.strictObject({
-  reviewedSha: z10.string().regex(/^[0-9a-f]{40}$/i).describe(
-    "REQUIRED. The PR's current head as a full 40-char SHA. Run `git rev-parse HEAD` immediately before this call \u2014 never extend an abbreviated hash into 40 characters."
-  ),
-  overview: z10.string().min(1).max(6e4).describe("REQUIRED. Plain-text walkthrough intro, max 3000 characters. Keep it short."),
-  sections: z10.array(
-    z10.strictObject({
-      title: z10.string().min(1).max(160),
-      explanation: z10.string().min(1).max(2e3),
-      classification: z10.enum(["core", "supporting"]).optional(),
-      files: z10.array(
-        z10.strictObject({
-          path: z10.string().min(1).max(500).describe(
-            "A file the PR's diff actually changed. Context files you merely read are rejected."
-          ),
-          startLine: z10.number().int().positive().max(1e6).optional(),
-          endLine: z10.number().int().positive().max(1e6).optional(),
-          hunkHeader: z10.string().min(1).max(300).optional().describe(
-            "Optional anchor, matched byte-exactly against the full hunk header line from `git diff` INCLUDING the context text after the second @@. Copy it verbatim from `git diff <base>..HEAD -- <file> | grep '^@@'`, or omit anchors entirely (path-only entries always validate)."
-          )
-        })
-      ).min(1).max(20)
-    })
-  ).min(1).max(12).optional().describe(
-    "REQUIRED top-level array (never text appended to overview) of ordered conceptual sections."
-  )
-});
-var FLATTENED_SECTIONS_PATTERN = /<\/overview>\s*(?:<parameter name="sections">|<sections>)/;
-var TRAILING_TAGS_PATTERN = /(?:\s*<\/(?:parameter|sections|invoke)>)+\s*$/;
-function sliceOutermostArray(tail) {
-  const start = tail.indexOf("[");
-  const end = tail.lastIndexOf("]");
-  return start >= 0 && end > start ? tail.slice(start, end + 1) : null;
-}
-function recoverFlattenedGuide(overview) {
-  const match = FLATTENED_SECTIONS_PATTERN.exec(overview);
-  if (!match) return null;
-  const head = overview.slice(0, match.index).trim();
-  const tail = overview.slice(match.index + match[0].length).replace(TRAILING_TAGS_PATTERN, "").trim();
-  for (const candidate of [tail, sliceOutermostArray(tail)]) {
-    if (!candidate) continue;
-    try {
-      return { overview: head, sections: JSON.parse(candidate) };
-    } catch {
-    }
-  }
-  return null;
-}
-async function resolveGitHeadSha(cwd) {
-  try {
-    const stdout = workbenchEnabled() ? (await getWorkbenchClient().execFile("git", ["rev-parse", "HEAD"], {
-      cwd,
-      timeout: 1e4
-    })).stdout : (await promisify(execFile)("git", ["rev-parse", "HEAD"], { cwd, timeout: 1e4 })).stdout;
-    const sha = stdout.trim();
-    return /^[0-9a-f]{40}$/i.test(sha) ? sha : null;
-  } catch {
-    return null;
-  }
-}
-function buildPublishReviewGuideTool(connection, options = {}) {
-  const { resolveHeadSha } = options;
-  return defineTool(
-    "publish_review_guide",
-    "Publish or update the PR guide for the CURRENT PR head commit. Call right after create_pull_request succeeds, and again after every push to the branch. The head SHA is resolved from the local repo, so never hand-expand a short hash. Best-effort \u2014 it does not gate opening or updating the PR.",
-    ReviewGuideToolSchema.shape,
-    async ({ reviewedSha, overview, sections }) => {
-      let resolvedOverview = overview;
-      let resolvedSections = sections;
-      if (resolvedSections === void 0) {
-        const recovered = recoverFlattenedGuide(overview);
-        if (!recovered) {
-          throw new Error(
-            "publish_review_guide requires a top-level `sections` array \u2014 an ordered list of conceptual sections, each with title, explanation, and files[]. Your call arrived with only `reviewedSha` and `overview`; the `sections` parameter never reached the server, which usually means the call encoding flattened it into `overview`. Do not append the sections to `overview` as text. Retry with sections as a real array argument, and shorten `overview` if the failure repeats."
-          );
-        }
-        resolvedOverview = recovered.overview;
-        resolvedSections = recovered.sections;
-      }
-      const content = ReviewGuideContentSchema.parse({
-        overview: resolvedOverview,
-        sections: resolvedSections
-      });
-      const result = await connection.call("publishReviewGuide", {
-        sessionId: connection.sessionId,
-        reviewedSha: (resolveHeadSha ? await resolveHeadSha() : null) ?? reviewedSha,
-        ...content
-      });
-      return textResult(
-        `Review guide published for ${result.reviewedSha}${result.replaced ? " (replaced)" : ""}.`
-      );
-    },
-    { strict: true }
-  );
-}
 function buildApproveCodeReviewTool(connection) {
   return defineTool(
     "approve_code_review",
@@ -11171,15 +11277,6 @@ function getModeTools(agentMode, connection, config, context) {
       return config.mode === "pm" ? buildPmTools(connection) : [];
   }
 }
-function buildPrGuideToolsFor(effectiveMode, connection, config, context) {
-  const isLeafBuild = effectiveMode === "building" || effectiveMode === "auto" || effectiveMode === "chat";
-  const isPackRunner = config.mode === "pack";
-  return (isLeafBuild || isPackRunner) && (isPackRunner || !context?.isParentTask) ? [
-    buildPublishReviewGuideTool(connection, {
-      resolveHeadSha: () => resolveGitHeadSha(config.workspaceDir)
-    })
-  ] : [];
-}
 function ciWaitToolsFor(effectiveMode, connection, config) {
   const isPackRunner = config.mode === "pack";
   return effectiveMode === "review" && !isPackRunner ? [] : [buildWaitForChecksTool(connection)];
@@ -11214,8 +11311,6 @@ var ALWAYS_LOADED_TOOLS = /* @__PURE__ */ new Set([
   // The shared-skill card write: one name whose fields match the mcp surface,
   // so a single skill text drives a local session and a pod verbatim.
   "update_task",
-  // Building/auto — the PR guide is published as part of opening the PR
-  "publish_review_guide",
   // Review mode
   "approve_code_review",
   "request_code_changes"
@@ -11269,7 +11364,6 @@ function buildConveyorTools(connection, config, context, agentMode) {
   const modeTools = getModeTools(effectiveMode, connection, config, context);
   const discoveryTools = effectiveMode === "discovery" || effectiveMode === "auto" || effectiveMode === "building" || effectiveMode === "chat" ? buildDiscoveryTools(connection) : [];
   const codeReviewTools = effectiveMode === "review" ? buildCodeReviewTools(connection) : [];
-  const prGuideTools = buildPrGuideToolsFor(effectiveMode, connection, config, context);
   const ciWaitTools = ciWaitToolsFor(effectiveMode, connection, config);
   const handoffTools = config.mode === "pm" && (effectiveMode === "discovery" || effectiveMode === "auto") ? [buildHandoffTool(connection)] : [];
   const emergencyTools = [buildForceUpdateTaskStatusTool(connection)];
@@ -11282,7 +11376,6 @@ function buildConveyorTools(connection, config, context, agentMode) {
       ...modeTools,
       ...discoveryTools,
       ...codeReviewTools,
-      ...prGuideTools,
       ...ciWaitTools,
       ...handoffTools,
       ...glossaryTools,
@@ -13847,9 +13940,9 @@ function buildUnmeasurableEvent(reason, codingAgentKeyId) {
 }
 
 // src/runner/parent-pull-handler.ts
-import { execFile as execFile2 } from "child_process";
-import { promisify as promisify2 } from "util";
-var execFileAsync = promisify2(execFile2);
+import { execFile } from "child_process";
+import { promisify } from "util";
+var execFileAsync = promisify(execFile);
 async function handlePullBranch(workDir, branch) {
   if (!branch) return;
   const current = await getCurrentBranch(workDir);
@@ -15292,6 +15385,7 @@ var SessionRunner = class _SessionRunner {
       claudeSessionId: ctx.claudeSessionId ?? null,
       lastSeenMessageId: ctx.lastSeenMessageId ?? null,
       isParentTask: ctx.isParentTask ?? false,
+      parentTaskFiles: ctx.parentTaskFiles ?? null,
       storyPoints: ctx.storyPoints ?? void 0,
       agents: ctx.agents ?? void 0,
       projectTags: ctx.projectTags ?? void 0,