@rallycry/conveyor-mcp 4.3.13 → 4.3.15

package/dist/cli.js

@@ -140,7 +140,7 @@ function registerUpdateProjectSettings(server2, conn2) {
       projectId: z3.string().optional().describe("Target Conveyor project ID"),
       name: z3.string().optional().describe("New project name"),
       description: z3.string().optional().describe("New project description"),
-      settings: z3.record(z3.unknown()).optional().describe(
+      settings: z3.record(z3.string(), z3.unknown()).optional().describe(
         "Deep-merged patch of the project settings JSON (JSON Merge Patch semantics: null deletes a key; advanced)"
       ),
       defaultPmAgentId: z3.string().nullable().optional().describe("Default PM agent ID"),
@@ -447,13 +447,13 @@ var childTaskIdForMerge = f.string({
 var approveAndMergePrContract = defineToolContract({
   name: "approve_and_merge_pr",
   agent: {
-    description: "Approve and merge a child task's PR. Preconditions: child in ReviewPR. Returns { merged }: true = merged (status\u2192ReviewDev); false = automerge queued, wait for ReviewDev.",
+    description: "Approve and merge a child task's PR. Preconditions: child in ReviewPR. Returns { merged }: true = merged (status\u2192ReviewDev); false = automerge queued, wait for ReviewDev. Requires project Admin, or a sub-project merge grant covering every changed file; release PRs require Admin.",
     fields: {
       childTaskId: childTaskIdForMerge
     }
   },
   mcp: {
-    description: "Approve and merge a child task's pull request. Pass projectId to target a specific project; otherwise the configured default project is used. Only succeeds if all CI/CD checks are passing. The child task must be in ReviewPR status with a PR.",
+    description: "Approve and merge a child task's pull request. Pass projectId to target a specific project; otherwise the configured default project is used. Only succeeds if all CI/CD checks are passing. The child task must be in ReviewPR status with a PR. Requires project Admin, or a sub-project merge grant covering every changed file; release PRs require Admin.",
     fields: {
       projectId: mcpProjectId,
       childTaskId: childTaskIdForMerge
@@ -870,13 +870,13 @@ var getAttachmentContract = defineToolContract({
 var uploadAttachmentContract = defineToolContract({
   name: "upload_attachment",
   agent: {
-    description: "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.",
+    description: "Upload a file (doc, notes, data, diagram, screenshot \u2014 any file type, up to 25MB) as a task attachment AND post it to the task chat in one step \u2014 no follow-up post_to_chat call needed. This is how you deliver a file the user should keep: it attaches to the card. Never publish deliverables as an external Claude artifact.",
     fields: {
       path: f.string({
-        desc: "Path to the image file \u2014 absolute, or relative to the workspace root"
+        desc: "Path to the file \u2014 absolute, or relative to the workspace root"
       }),
       title: f.optional(
-        f.string({ desc: "Short caption posted with the image (defaults to the file name)" })
+        f.string({ desc: "Short caption posted with the file (defaults to the file name)" })
       )
     }
   },
@@ -988,7 +988,17 @@ import { z as z4 } from "zod";
 function mcpShape(surface) {
   return compileShape(z4, surface.fields);
 }
-function registerContractTool(server2, contract, handler) {
+function registerContractTool(server2, contract, handler, options) {
+  if (options?.alwaysLoad) {
+    registerHotTool(
+      server2,
+      contract.name,
+      contract.mcp.description,
+      mcpShape(contract.mcp),
+      handler
+    );
+    return;
+  }
   server2.tool(
     contract.name,
     contract.mcp.description,
@@ -996,6 +1006,17 @@ function registerContractTool(server2, contract, handler) {
     handler
   );
 }
+function registerHotTool(server2, name, description, schema, handler) {
+  server2.registerTool(
+    name,
+    {
+      description,
+      inputSchema: schema,
+      _meta: { "anthropic/alwaysLoad": true }
+    },
+    handler
+  );
+}
 
 // src/tools/tasks-format.ts
 var CLI_EVENT_FORMATTERS = {
@@ -1009,9 +1030,23 @@ var CLI_EVENT_FORMATTERS = {
   start_command_output: (data) => `[${data.stream ?? "stdout"}] ${String(data.data ?? "")}`,
   turn_end: (data) => `Turn complete (${Array.isArray(data.toolCalls) ? data.toolCalls.length : 0} tool calls)`
 };
+var EVENT_SUMMARY_MAX_CHARS = 1200;
 function formatCliEventSummary(type, data) {
   const formatter = CLI_EVENT_FORMATTERS[type];
-  return formatter ? formatter(data) : JSON.stringify(data);
+  const summary = formatter ? formatter(data) : JSON.stringify(data);
+  if (summary.length <= EVENT_SUMMARY_MAX_CHARS) return summary;
+  return `${summary.slice(0, EVENT_SUMMARY_MAX_CHARS)}\u2026[+${summary.length - EVENT_SUMMARY_MAX_CHARS}c]`;
+}
+var CHAT_MESSAGE_PREVIEW_CHARS = 2e3;
+function compactChatMessage(message) {
+  if (typeof message !== "object" || message === null) return message;
+  const obj = { ...message };
+  const content = obj.content;
+  if (typeof content === "string" && content.length > CHAT_MESSAGE_PREVIEW_CHARS) {
+    const overflow = content.length - CHAT_MESSAGE_PREVIEW_CHARS;
+    obj.content = `${content.slice(0, CHAT_MESSAGE_PREVIEW_CHARS)}\u2026[+${overflow}c truncated]`;
+  }
+  return obj;
 }
 function formatLegacyComputeSession(s) {
   return `  - session ${s.id} [${s.provider}] pod=${s.instanceName ?? "-"} status=${s.status} runner=${s.agentRunnerStatus ?? "-"} lastHeartbeat=${s.lastHeartbeatAt ?? "-"} created=${s.createdAt} stopped=${s.stoppedAt ?? "-"}` + (s.deletionRequestedAt ? ` deletionRequested=${s.deletionRequestedAt} (attempts ${s.deletionAttempts})` : "") + (s.lastAgentEvent ? ` lastAgentEvent=${s.lastAgentEvent}` : "");
@@ -1030,19 +1065,32 @@ function formatWorkspace(w) {
   lines.push(...w.sessions.map(formatWorkspaceSession));
   return lines;
 }
-function formatTaskSessionState(t) {
+function latestByCreatedAt(items, limit) {
+  if (items.length <= limit) return { kept: items, omitted: 0 };
+  const sorted = [...items].sort((a, b) => b.createdAt.localeCompare(a.createdAt));
+  return { kept: sorted.slice(0, limit), omitted: items.length - limit };
+}
+function formatTaskSessionState(t, limit = 20) {
   const review = t.codeReviewStatus ? ` codeReview=${t.codeReviewStatus} (attempts ${t.codeReviewAttempts})` : "";
   const kind = t.reviewTargetTaskId ? `review card for ${t.reviewTargetTaskId}` : t.type;
   const lines = [`# ${t.slug} \u2014 ${t.title} [${kind}, status ${t.status}]${review}`];
   if (t.sessions.length === 0) {
     lines.push("  (no compute sessions)");
   } else {
-    lines.push(...t.sessions.map(formatLegacyComputeSession));
+    const { kept, omitted } = latestByCreatedAt(t.sessions, limit);
+    lines.push(...kept.map(formatLegacyComputeSession));
+    if (omitted > 0) {
+      lines.push(`  (\u2026 +${omitted} older compute sessions \u2014 pass a higher limit to see them)`);
+    }
   }
   if (t.workspaces.length === 0) {
     lines.push("  (no v3 workspaces)");
   } else {
-    lines.push(...t.workspaces.flatMap(formatWorkspace));
+    const { kept, omitted } = latestByCreatedAt(t.workspaces, limit);
+    lines.push(...kept.flatMap(formatWorkspace));
+    if (omitted > 0) {
+      lines.push(`  (\u2026 +${omitted} older workspaces \u2014 pass a higher limit to see them)`);
+    }
   }
   return lines;
 }
@@ -1076,7 +1124,8 @@ var BOARD_ASSIGN = z5.string().nullable().optional().describe(
   "Assign the card to a sub-project board. Omit to use the connection's default board (CONVEYOR_SUBPROJECT_ID) when set, else the parent project; pass null to force the parent project. Use list_accessible_subprojects to find board IDs."
 );
 function registerListTasks(server2, conn2) {
-  server2.tool(
+  registerHotTool(
+    server2,
     "list_tasks",
     "List project cards, optionally filtered by card type, status, or assignment (a specific assignee, or unassigned tasks). Defaults to type=task \u2014 pass typeFilters to list incidents/suggestions. Results are relevance-ordered: highest priority first then newest; suggestions-only queries rank by upvote score. Pass projectId to target a specific project; otherwise the configured default project is used. Returns summaries \u2014 plan omitted, description truncated; use get_task for full details.",
     {
@@ -1099,10 +1148,15 @@ function registerListTasks(server2, conn2) {
   );
 }
 function registerGetTask(server2, conn2) {
-  registerContractTool(server2, getTaskContract, async (params) => {
-    const task = await conn2.getTask(params.taskId, params.projectId);
-    return { content: [{ type: "text", text: JSON.stringify(task, null, 2) }] };
-  });
+  registerContractTool(
+    server2,
+    getTaskContract,
+    async (params) => {
+      const task = await conn2.getTask(params.taskId, params.projectId);
+      return { content: [{ type: "text", text: JSON.stringify(task, null, 2) }] };
+    },
+    { alwaysLoad: true }
+  );
 }
 function registerGetCardBySlug(server2, conn2) {
   server2.tool(
@@ -1209,27 +1263,34 @@ function registerMoveCard(server2, conn2) {
 }
 function registerChatTools(server2, conn2) {
   registerContractTool(server2, readTaskChatContract, async (params) => {
-    const messages = await conn2.getTaskChat(params.taskId, params.limit, params.projectId);
-    return { content: [{ type: "text", text: JSON.stringify(messages, null, 2) }] };
-  });
-  registerContractTool(server2, postToChatContract, async (params) => {
-    const text = params.content ?? params.message;
-    if (text === void 0) {
-      return {
-        content: [
-          {
-            type: "text",
-            text: "Nothing to post \u2014 provide `content` (or its alias `message`)."
-          }
-        ]
-      };
-    }
-    await conn2.postToTaskChat(params.taskId, text, params.projectId);
-    return { content: [{ type: "text", text: "Message posted" }] };
+    const messages = await conn2.getTaskChat(params.taskId, params.limit ?? 50, params.projectId);
+    const compacted = messages.map(compactChatMessage);
+    return { content: [{ type: "text", text: JSON.stringify(compacted, null, 2) }] };
   });
+  registerContractTool(
+    server2,
+    postToChatContract,
+    async (params) => {
+      const text = params.content ?? params.message;
+      if (text === void 0) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: "Nothing to post \u2014 provide `content` (or its alias `message`)."
+            }
+          ]
+        };
+      }
+      await conn2.postToTaskChat(params.taskId, text, params.projectId);
+      return { content: [{ type: "text", text: "Message posted" }] };
+    },
+    { alwaysLoad: true }
+  );
 }
 function registerGetTaskCli(server2, conn2) {
-  server2.tool(
+  registerHotTool(
+    server2,
     "get_task_logs",
     "Read CLI execution logs from a task. Pass projectId to target a specific project; otherwise the configured default project is used. Returns agent reasoning, tool calls, setup output, and other execution events. For human chat use read_task_chat.",
     {
@@ -1258,11 +1319,12 @@ function registerGetTaskSessions(server2, conn2) {
     "Read compute-session state for a task: legacy CodespaceSession rows plus v3 workspaces, including purpose=review review workspaces. Shows pod identity, liveness, lifecycle, and code-review claim state. Use to diagnose stalled/dead agents or code-review runs. Pass projectId to target a specific project; otherwise the configured default project is used.",
     {
       projectId: z5.string().optional().describe("Target Conveyor project ID"),
-      taskId: z5.string().describe("The task ID or slug")
+      taskId: z5.string().describe("The task ID or slug"),
+      limit: z5.number().int().min(1).max(200).optional().describe("Max sessions/workspaces listed per task, newest first (default 20)")
     },
-    async ({ taskId, projectId: projectId2 }) => {
+    async ({ taskId, projectId: projectId2, limit }) => {
       const tasks = await conn2.getTaskSessions(taskId, projectId2);
-      const lines = tasks.flatMap(formatTaskSessionState);
+      const lines = tasks.flatMap((t) => formatTaskSessionState(t, limit ?? 20));
       return {
         content: [
           { type: "text", text: lines.join("\n") || "No sessions found for this task." }
@@ -1272,7 +1334,8 @@ function registerGetTaskSessions(server2, conn2) {
   );
 }
 function registerSearchTasks(server2, conn2) {
-  server2.tool(
+  registerHotTool(
+    server2,
     "search_tasks",
     "Search cards by tag name, text query, status, type, and/or assignment. Defaults to type=task \u2014 pass typeFilters to include incidents/suggestions. Results are relevance-ordered: highest priority first then newest; suggestions-only queries rank by upvote score. Pass projectId to target a specific project; otherwise the configured default project is used. Use tag names like 'agent-runner', not IDs. Returns summaries \u2014 plan omitted, description truncated; use get_task for full details.",
     {
@@ -1644,12 +1707,17 @@ function registerAttachmentTools(server2, conn2) {
 
 // src/tools/pull-request.ts
 function registerPullRequestTools(server2, conn2) {
-  registerContractTool(server2, createPullRequestContract, async (params) => {
-    const result = await conn2.createPullRequest(params);
-    return {
-      content: [{ type: "text", text: `PR #${result.prNumber} opened: ${result.prUrl}` }]
-    };
-  });
+  registerContractTool(
+    server2,
+    createPullRequestContract,
+    async (params) => {
+      const result = await conn2.createPullRequest(params);
+      return {
+        content: [{ type: "text", text: `PR #${result.prNumber} opened: ${result.prUrl}` }]
+      };
+    },
+    { alwaysLoad: true }
+  );
 }
 
 // src/tools/subtasks.ts
@@ -2183,7 +2251,8 @@ async function runQueryGrafanaLogs(conn2, params, now = Date.now) {
     `entries=${result.entries.length}`
   ].join(" ");
   const lines = result.entries.map(formatLogEntryLine);
-  const footer = result.hasMore ? ["-- hit the limit: narrow the window (startTime/endTime) for older lines"] : [];
+  const oldest = result.entries.map((e) => e.timestamp).sort()[0];
+  const footer = result.hasMore ? [`-- hit the limit: older lines exist \u2014 pass endTime="${oldest}" to page further back`] : [];
   if (lines.length === 0) {
     return [
       header,