@rallycry/conveyor-agent 7.0.3 → 7.0.4

package/dist/{chunk-L7ZH423N.js → chunk-K2TJHPMA.js}

@@ -263,6 +263,7 @@ var AgentConnection = class {
     };
     const heartbeatStatus = statusMap[this.lastEmittedStatus ?? "idle"] ?? "active";
     void this.call("heartbeat", {
+      sessionId: this.config.sessionId,
       timestamp: (/* @__PURE__ */ new Date()).toISOString(),
       status: heartbeatStatus
     }).catch(() => {
@@ -1172,6 +1173,14 @@ function buildDiscoveryPrompt(context) {
     `- If the work fits in a single task (1-3 SP), update YOUR OWN plan and properties \u2014 do not create subtasks`,
     `- Only create subtasks when the work genuinely requires multiple independent pieces (e.g., Pack-tier work, 8+ SP)`,
     ``,
+    `### Subtask Plan Requirements`,
+    `When creating subtasks, each MUST include a detailed \`plan\` field:`,
+    `- Plans should be multi-step implementation guides, not vague descriptions`,
+    `- Include specific file paths, function names, and code patterns to modify`,
+    `- Reference existing implementations when relevant (e.g., "follow the pattern in src/services/foo.ts")`,
+    `- Include testing requirements and acceptance criteria`,
+    `- Set \`storyPointValue\` based on estimated complexity`,
+    ``,
     `### Finishing Planning`,
     `Once your plan is complete and all required properties are set, call the **ExitPlanMode** tool.`,
     `- Required before ExitPlanMode will succeed: **plan** (via update_task), **story points** (via update_task_properties), **title** (via update_task_properties)`,
@@ -1203,6 +1212,14 @@ function buildAutoPrompt(context) {
     `- Once ExitPlanMode succeeds, the system will automatically restart your session in Building mode with the appropriate model`,
     `- You do NOT need to do anything after calling ExitPlanMode \u2014 the transition is handled for you`,
     ``,
+    `### Subtask Plan Requirements`,
+    `When creating subtasks, each MUST include a detailed \`plan\` field:`,
+    `- Plans should be multi-step implementation guides, not vague descriptions`,
+    `- Include specific file paths, function names, and code patterns to modify`,
+    `- Reference existing implementations when relevant`,
+    `- Include testing requirements and acceptance criteria`,
+    `- Set \`storyPointValue\` based on estimated complexity`,
+    ``,
     `### Autonomous Guidelines:`,
     `- Make decisions independently \u2014 do not ask the team for approval at each step`,
     `- Only escalate when genuinely blocked (ambiguous requirements, missing access, conflicting instructions)`,
@@ -1229,13 +1246,12 @@ function buildModePrompt(agentMode, context) {
         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`,
-          `Scaling is one-way (up only) and capped by project limits.`,
-          `CRITICAL: Always scale to at least "light" before running any package install command.`
+          `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`,
+          `Actual CPU/memory values are configured per-project. Scaling is one-way (up only).`,
+          `CRITICAL: Always scale to at least "setup" before running any package install command.`
         );
       }
       return parts.join("\n");
@@ -2348,7 +2364,15 @@ Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>`;
         return textResult(`Pull request #${result.prNumber} created: ${result.prUrl}`);
       } catch (error) {
         const msg = error instanceof Error ? error.message : "Unknown error";
-        return textResult(`Failed to create pull request: ${msg}`);
+        return textResult(
+          `Failed to create pull request: ${msg}
+
+Troubleshooting:
+- Ensure all changes are committed and pushed to the remote branch
+- Check that the branch exists on the remote (run: git push -u origin HEAD)
+- Verify there isn't already an open PR for this branch
+- If git auth fails, the token may have expired \u2014 retry the operation`
+        );
       }
     }
   );
@@ -2485,9 +2509,9 @@ function buildVoteSuggestionTool(connection) {
 function buildScaleUpResourcesTool(connection) {
   return defineTool(
     "scale_up_resources",
-    "Scale up the pod's CPU and memory resources. Use before running dev servers, tests, builds, or other resource-intensive operations. Tiers: 'light' (1 CPU / 4 Gi \u2014 installs, basic dev servers), 'standard' (2 CPU / 8 Gi \u2014 full dev servers, test suites, typecheck), 'heavy' (4 CPU / 16 Gi \u2014 E2E tests, large parallel builds). Scaling is one-way (up only) and capped by project limits.",
+    "Scale up the pod's CPU and memory resources. Use before running dev servers, tests, builds, or other resource-intensive operations. Phases: 'setup' (installs, basic dev servers), 'build' (full dev servers, test suites, typecheck, builds). Actual CPU/memory values are configured per-project. Scaling is one-way (up only).",
     {
-      tier: z.enum(["initial", "light", "standard", "heavy"]).describe("The resource tier to scale up to"),
+      tier: z.enum(["initial", "setup", "build"]).describe("The resource phase to scale up to"),
       reason: z.string().optional().describe("Brief reason for scaling (e.g., 'running test suite')")
     },
     async ({ tier, reason }) => {
@@ -2498,13 +2522,8 @@ function buildScaleUpResourcesTool(connection) {
           reason
         });
         if (result.success) {
-          if (result.currentTier === result.previousTier) {
-            return textResult(
-              `Already at ${result.currentTier} tier (${result.cpu} CPU / ${result.memory} Gi). No scaling needed.`
-            );
-          }
           return textResult(
-            `Scaled to ${result.cpu} CPU / ${result.memory} Gi (${result.currentTier} tier, was ${result.previousTier}).`
+            `Scaled to ${result.currentTier} phase (${result.cpu}). Was ${result.previousTier}.`
           );
         }
         return textResult(
@@ -2581,14 +2600,15 @@ function buildCreateSubtaskTool(connection) {
       ordinal: z2.number().optional().describe("Step/order number (0-based)"),
       storyPointValue: z2.number().optional().describe(SP_DESCRIPTION)
     },
-    async ({ title, description, plan: _plan, ordinal, storyPointValue }) => {
+    async ({ title, description, plan, ordinal, storyPointValue }) => {
       try {
         const result = await connection.call("createSubtask", {
           sessionId: connection.sessionId,
           title,
-          description,
-          storyPoints: storyPointValue,
-          ordinal
+          ...description !== void 0 && { description },
+          ...plan !== void 0 && { plan },
+          ...storyPointValue !== void 0 && { storyPointValue },
+          ...ordinal !== void 0 && { ordinal }
         });
         return textResult(`Subtask created with ID: ${result.id}`);
       } catch (error) {
@@ -2611,20 +2631,15 @@ function buildUpdateSubtaskTool(connection) {
       ordinal: z2.number().optional(),
       storyPointValue: z2.number().optional().describe(SP_DESCRIPTION)
     },
-    async ({
-      subtaskId,
-      title,
-      description,
-      plan: _plan,
-      ordinal: _ordinal,
-      storyPointValue: _sp
-    }) => {
+    async ({ subtaskId, title, description, plan, storyPointValue }) => {
       try {
         await connection.call("updateSubtask", {
           sessionId: connection.sessionId,
           subtaskId,
-          title,
-          description
+          ...title !== void 0 && { title },
+          ...description !== void 0 && { description },
+          ...plan !== void 0 && { plan },
+          ...storyPointValue !== void 0 && { storyPointValue }
         });
         return textResult("Subtask updated.");
       } catch (error) {
@@ -4686,6 +4701,14 @@ function handleAutoToolAccess(toolName, input, hasExitedPlanMode, isParentTask)
   }
   return { behavior: "allow", updatedInput: input };
 }
+function collectMissingProps(taskProps) {
+  const missing = [];
+  if (!taskProps.plan?.trim()) missing.push("plan (save via update_task)");
+  if (!taskProps.storyPointId) missing.push("story points (use update_task_properties)");
+  if (!taskProps.title || taskProps.title === "Untitled")
+    missing.push("title (use update_task_properties)");
+  return missing;
+}
 async function handleExitPlanMode(host, input) {
   if (host.hasExitedPlanMode) {
     return { behavior: "allow", updatedInput: input };
@@ -4693,11 +4716,24 @@ async function handleExitPlanMode(host, input) {
   try {
     host.syncPlanFile();
     const taskProps = await host.connection.getTaskProperties();
-    const missingProps = [];
-    if (!taskProps.plan?.trim()) missingProps.push("plan (save via update_task)");
-    if (!taskProps.storyPointId) missingProps.push("story points (use update_task_properties)");
-    if (!taskProps.title || taskProps.title === "Untitled")
-      missingProps.push("title (use update_task_properties)");
+    const missingProps = collectMissingProps(taskProps);
+    if (host.isParentTask) {
+      try {
+        const subtasks = await host.connection.call("listSubtasks", {
+          sessionId: host.connection.sessionId
+        });
+        const subtasksWithoutPlans = subtasks.filter(
+          (s) => !s.plan?.trim()
+        );
+        if (subtasksWithoutPlans.length > 0) {
+          const names = subtasksWithoutPlans.map((s) => s.title).join(", ");
+          missingProps.push(
+            `subtask plans \u2014 these subtasks are missing plans: ${names} (use update_subtask with plan field)`
+          );
+        }
+      } catch {
+      }
+    }
     if (missingProps.length > 0) {
       return {
         behavior: "deny",
@@ -7087,4 +7123,4 @@ export {
   removeWorktree,
   ProjectRunner
 };
-//# sourceMappingURL=chunk-L7ZH423N.js.map
+//# sourceMappingURL=chunk-K2TJHPMA.js.map