orcastrator 0.2.8 → 0.2.9

package/dist/agents/codex/session.js

@@ -26,9 +26,11 @@ function buildTaskExecutionPrompt(task, runId, cwd, systemContext) {
         "Acceptance Criteria:",
         ...task.acceptance_criteria.map((criterion, index) => `${index + 1}. ${criterion}`),
         "Execute this task. You have full shell access — run commands, read/write files, and do whatever is needed.",
-        "When done, output ONLY JSON on the last line:",
-        '{"outcome":"done"|"failed","error"?:"string"}',
-        "If you cannot complete the task, return outcome=failed with a short error reason.",
+        "IMPORTANT: When done, you MUST output the following JSON on its own line as the very last line of your response (no trailing text after it):",
+        '{"outcome":"done"}',
+        "Or if the task failed:",
+        '{"outcome":"failed","error":"short reason"}',
+        "Do not wrap it in markdown fences. Do not add any text after the JSON line. The JSON line is required.",
     ].join("\n\n");
 }
 function extractAgentText(result) {
@@ -83,6 +85,21 @@ const POSITIVE_COMPLETION_PATTERNS = [
     /\bwritten\b/i,
     /\bcreated\b/i,
     /\bfinished\b/i,
+    // Additional patterns to catch natural-language Codex narration
+    /\bapplied\b/i,
+    /\bimplemented\b/i,
+    /\badded\b/i,
+    /\bupdated\b/i,
+    /\bmodified\b/i,
+    /\binstalled\b/i,
+    /\bfixed\b/i,
+    /\brefactored\b/i,
+    /\bchanges?\s+(?:have\s+been\s+)?made\b/i,
+    /\bthe\s+task\s+(?:has\s+been|is)\b/i,
+    /\bi\s+have\b/i,
+    /\ball\s+(?:tasks?|steps?|criteria)\b/i,
+    /\btest(?:s|ing)?\s+pass/i,
+    /\bno\s+(?:errors?|issues?|failures?)\b/i,
 ];
 const FAILURE_PATTERNS = [
     /\berror\b/i,
@@ -176,6 +193,22 @@ export async function createCodexSession(cwd, config) {
                 ],
             });
             const rawResponse = extractAgentText(result);
+            // Primary signal: use the SDK's structured turn status.
+            const status = result.turn.status;
+            if (status === "completed") {
+                return { outcome: "done", rawResponse };
+            }
+            if (status === "failed") {
+                return {
+                    outcome: "failed",
+                    error: result.turn.error?.message ?? "Turn failed",
+                    rawResponse,
+                };
+            }
+            if (status === "interrupted") {
+                return { outcome: "failed", error: "Turn was interrupted", rawResponse };
+            }
+            // Fallback: status is unexpected/missing — parse text as before.
             return parseTaskExecution(rawResponse);
         },
         async consultTaskGraph(tasks) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "orcastrator",
-  "version": "0.2.8",
+  "version": "0.2.9",
   "type": "module",
   "bin": {
     "orca": "dist/cli/index.js"