@andromarces/agent-loops 0.2.2 → 0.3.0

package/src/runtime.mjs

@@ -1,4 +1,5 @@
 import { defaultAgents, runAgent } from "./agents/index.mjs";
+import { DEFAULT_MAX_STEPS } from "./lib/args.mjs";
 import { logError, logInfo, logWarn } from "./lib/log.mjs";
 import { withMutationCheck } from "./lib/snapshot.mjs";
 import { decide } from "./orchestrator.mjs";
@@ -59,8 +60,8 @@ export async function runChild(options) {
     ? workerPrompt(prompt, role.sessionId === null)
     : reviewerPrompt(prompt);
 
-  const runFn = async () => {
-    return invoke(
+  const runFn = () =>
+    invoke(
       agents,
       role,
       roleName,
@@ -69,7 +70,6 @@ export async function runChild(options) {
       onEvent,
       stepsUsed,
     );
-  };
 
   try {
     const response = readOnly ? await withMutationCheck(cwd, roleName, runFn) : await runFn();
@@ -102,7 +102,7 @@ export async function runLoop(options) {
   const {
     task,
     cwd,
-    maxSteps = 20,
+    maxSteps = DEFAULT_MAX_STEPS,
     timeout,
     signal,
     roles,
@@ -119,6 +119,8 @@ export async function runLoop(options) {
     return { exitCode, ...detail };
   }
 
+  let stepsUsed = 0;
+
   const orchAdapter = {
     async run(state, p, opts) {
       return withMutationCheck(cwd, "orchestrator", () =>
@@ -127,7 +129,6 @@ export async function runLoop(options) {
     },
   };
 
-  let stepsUsed = 0;
   let prompt = initialPrompt({ task, maxSteps });
 
   while (true) {
@@ -140,11 +141,10 @@ export async function runLoop(options) {
         options: { cwd, timeout, signal },
       });
     } catch (err) {
-      if (err?.name === "MutationError") {
-        // Already logged at the detection site in withMutationCheck.
-        throw err;
+      if (err?.name !== "MutationError") {
+        // A MutationError is already logged at the detection site in withMutationCheck.
+        logError(`orchestrator turn failed: ${String(err?.message ?? err).split("\n")[0]}`);
       }
-      logError(`orchestrator turn failed: ${String(err?.message ?? err).split("\n")[0]}`);
       throw err;
     }