orcastrator 0.2.6 → 0.2.7

package/dist/core/config-loader.js

@@ -62,7 +62,7 @@ export async function loadConfig(configPath) {
     const configCandidate = "default" in importedModule ? importedModule.default : importedModule;
     return coerceConfig(configCandidate);
 }
-const TOP_LEVEL_SCALARS = ["runsDir", "sessionLogs", "maxRetries", "anthropicApiKey", "openaiApiKey"];
+const TOP_LEVEL_SCALARS = ["runsDir", "sessionLogs", "maxRetries", "anthropicApiKey", "openaiApiKey", "executor"];
 export function mergeConfigs(...configs) {
     const presentConfigs = configs.filter((config) => config !== undefined);
     if (presentConfigs.length === 0) {

package/dist/core/task-runner.js

@@ -1,12 +1,14 @@
 import path from "node:path";
 import { promises as fs } from "node:fs";
-import { executeTask } from "../agents/claude/session.js";
+import * as claudeAgent from "../agents/claude/session.js";
+import { createCodexSession } from "../agents/codex/session.js";
 import { loadSkills } from "../utils/skill-loader.js";
 import { getRunnable, validateDAG } from "./dependency-graph.js";
 import { shouldRetry } from "./retry-policy.js";
-let executeTaskImpl = executeTask;
+// Non-null only when set by tests — null means "use real executor logic"
+let testExecuteTaskOverride = null;
 export function setExecuteTaskForTests(fn) {
-    executeTaskImpl = fn ?? executeTask;
+    testExecuteTaskOverride = fn;
 }
 function toErrorMessage(error) {
     if (error instanceof Error) {
@@ -94,10 +96,34 @@ async function writeSessionSummary(store, runId, sessionLogsDir) {
 }
 export async function runTaskRunner(options) {
     const emitHook = options.emitHook ?? defaultEmitHook;
-    const executeTaskFn = options.executeTask ?? executeTaskImpl;
     const { runId, store, config } = options;
     const skills = await loadSkills(config);
     const taskSystemContext = skills.length === 0 ? undefined : formatSkillsSection(skills);
+    // Test mocks bypass all executor logic entirely — no real sessions created.
+    const mockFn = options.executeTask ?? testExecuteTaskOverride;
+    // Build real executor (Codex persistent session or Claude stateless fallback).
+    // Only runs in production — skipped completely when a mock is active.
+    let codexSession;
+    let executeTaskFn;
+    if (mockFn) {
+        executeTaskFn = mockFn;
+    }
+    else {
+        const executor = config?.executor ?? "codex";
+        if (executor === "codex") {
+            try {
+                codexSession = await createCodexSession(process.cwd(), config);
+                executeTaskFn = (task, taskRunId, _cfg, systemContext) => codexSession.executeTask(task, taskRunId, systemContext);
+            }
+            catch (sessionError) {
+                console.warn(`[orca] Codex session init failed, falling back to Claude: ${toErrorMessage(sessionError)}`);
+                executeTaskFn = claudeAgent.executeTask;
+            }
+        }
+        else {
+            executeTaskFn = claudeAgent.executeTask;
+        }
+    }
     let run = await store.getRun(runId);
     if (!run) {
         throw new Error(`Run not found: ${runId}`);
@@ -320,4 +346,14 @@ export async function runTaskRunner(options) {
         await writeSessionSummary(store, runId, config?.sessionLogs);
         throw error;
     }
+    finally {
+        if (codexSession) {
+            try {
+                await codexSession.disconnect();
+            }
+            catch {
+                // Best-effort cleanup — don't mask the real error.
+            }
+        }
+    }
 }

package/package.json

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