@rowan-agent/agent 0.35.1 → 0.35.3

package/dist/runtime.js

@@ -1767,6 +1767,17 @@ function resolveCoreToolPath(context, path = ".") {
 function toolResult(input) {
   return input.ok ? { ok: true, content: input.content } : { ok: false, content: input.content, error: input.error ?? "Tool execution failed" };
 }
+function isSchemaParseError(error) {
+  return !(error instanceof Error) && Array.isArray(error?.errors);
+}
+function toolErrorMessage(error) {
+  if (error instanceof Error) return error.message;
+  if (isSchemaParseError(error) && error.errors.length > 0) {
+    const { errors } = error;
+    return `Invalid arguments: ${errors.map((item) => `${item.instancePath || "/"} ${item.message ?? "is invalid"}`).join("; ")}`;
+  }
+  return String(error);
+}
 async function executeRuntimeToolCall(input) {
   const tool = input.tools.find((candidate) => candidate.name === input.toolCall.name);
   if (!tool) {
@@ -1780,13 +1791,19 @@ async function executeRuntimeToolCall(input) {
   }
   let args;
   try {
-    const rawArgs = input.toolCall.arguments;
+    let rawArgs = input.toolCall.arguments;
+    if (typeof rawArgs === "string") {
+      try {
+        rawArgs = JSON.parse(rawArgs);
+      } catch {
+      }
+    }
     args = validatorFor(tool.parameters).Parse(rawArgs);
   } catch (error) {
     const result = toolResult({
       ok: false,
       content: null,
-      error: error instanceof Error ? error.message : String(error)
+      error: toolErrorMessage(error)
     });
     await input.observe?.({ type: "tool_end", toolName: tool.name, result });
     return result;
@@ -1963,7 +1980,7 @@ function createReadTool(context) {
         return toolResult({
           ok: false,
           content: null,
-          error: error instanceof Error ? error.message : String(error)
+          error: toolErrorMessage(error)
         });
       }
       if (!fileStat.isFile()) {
@@ -1980,7 +1997,7 @@ function createReadTool(context) {
         return toolResult({
           ok: false,
           content: null,
-          error: error instanceof Error ? error.message : String(error)
+          error: toolErrorMessage(error)
         });
       }
       const source = new TextDecoder().decode(bytes);
@@ -2095,7 +2112,7 @@ function createEditTool(context) {
         return toolResult({
           ok: false,
           content: null,
-          error: error instanceof Error ? error.message : String(error)
+          error: toolErrorMessage(error)
         });
       }
       await writeFile(resolved.absolutePath, next, "utf8");
@@ -2632,9 +2649,6 @@ function createRouteToolResultContent(toolCall, error, executionContext) {
   ];
 }
 async function executePhaseWithModel(ctx) {
-  const executableToolNames = new Set(
-    ctx.context.tools.filter((tool) => tool.name !== PhaseRouteTool).map((tool) => tool.name)
-  );
   let output = {
     message: "",
     phase: ctx.phase.name,
@@ -2652,7 +2666,10 @@ async function executePhaseWithModel(ctx) {
     if (checkpoint && typeof checkpoint === "object" && checkpoint.kind === "tool_call_suspension") {
       const toolSuspension = checkpoint;
       executableToolCalls = toolSuspension.toolCalls.filter(
-        (toolCall) => executableToolNames.has(toolCall.name)
+        (toolCall) => (
+          // An unknown name still runs, so the model gets "not available" back instead of a silent end of turn.
+          toolCall.name !== PhaseRouteTool
+        )
       );
       output = {
         message: "",
@@ -2686,7 +2703,10 @@ async function executePhaseWithModel(ctx) {
         await ctx.messageManager.end(messageId);
       }
       executableToolCalls = collected.toolCalls.filter(
-        (toolCall) => executableToolNames.has(toolCall.name)
+        (toolCall) => (
+          // An unknown name still runs, so the model gets "not available" back instead of a silent end of turn.
+          toolCall.name !== PhaseRouteTool
+        )
       );
       roundContentBlocks = collected.contentBlocks;
       if (executableToolCalls.length === 0) {
@@ -2706,6 +2726,7 @@ async function executePhaseWithModel(ctx) {
       });
       await ctx.messageManager.end(messageId);
     }
+    ctx.progressed = true;
   }
 }
 async function runTurn(fn) {
@@ -2729,6 +2750,7 @@ async function runPhaseLoop(config, state, registry) {
   if (resumingSuspendedRun) {
     state.status = "running";
     delete state.entryPhases;
+    state.attempt = 0;
   }
   let previousPayload = resumingSuspendedRun ? state.continuation && Object.hasOwn(state.continuation, "previousPayload") ? state.continuation.previousPayload : state.initialPhasePayload : state.initialPhasePayload;
   let previousPhaseMsgId = resumingSuspendedRun ? state.continuation?.previousPhaseMessageId : void 0;
@@ -2973,6 +2995,7 @@ async function runPhaseLoop(config, state, registry) {
     }
     const runtime = { phase, config, state, execution, messageManager, registry, context: phaseContext, completedPhases };
     let output = await executePhase(runtime);
+    if (runtime.progressed) state.attempt = 0;
     delete state.initialPhasePayload;
     delete state.entryPhases;
     let modelRouteDecision = resolveModelRouteDecision(output.toolCalls, registry, state);
@@ -10668,12 +10691,12 @@ var AgentRuntime = class _AgentRuntime {
               expectedRevision: revision,
               toolCallId,
               state: "failed",
-              result: { ok: false, content: null, error: `Tool ${toolCall.name} is not available.` },
+              result: { ok: false, content: null, error: `Tool ${toolCall.name} is not available. Available tools: ${input.toolConfig.tools.map(({ name }) => name).join(", ")}.` },
               title: toolCall.name,
               kind: "other"
             });
             setRevision(failed.run.revision);
-            results2.push(protocolFailure(`Tool ${toolCall.name} is not available.`));
+            results2.push(protocolFailure(`Tool ${toolCall.name} is not available. Available tools: ${input.toolConfig.tools.map(({ name }) => name).join(", ")}.`));
             continue;
           }
           const decision = preDecisions[i] ?? { allow: true };
@@ -10791,12 +10814,12 @@ var AgentRuntime = class _AgentRuntime {
           expectedRevision: revision,
           toolCallId,
           state: "failed",
-          result: { ok: false, content: null, error: `Tool ${toolCall.name} is not available.` },
+          result: { ok: false, content: null, error: `Tool ${toolCall.name} is not available. Available tools: ${input.toolConfig.tools.map(({ name }) => name).join(", ")}.` },
           title: toolCall.name,
           kind: "other"
         });
         setRevision(failed.run.revision);
-        results.push(protocolFailure(`Tool ${toolCall.name} is not available.`));
+        results.push(protocolFailure(`Tool ${toolCall.name} is not available. Available tools: ${input.toolConfig.tools.map(({ name }) => name).join(", ")}.`));
         continue;
       }
       const pendingPres = await presentToolPending(durableTool, toolCallArgs(toolCall));
@@ -11167,7 +11190,11 @@ var AgentRuntime = class _AgentRuntime {
         throw error;
       }
       let result = await withAbort(
-        input.durableTool.execute(toolCallArgs(input.toolCall), context, effectiveSignal),
+        // Invalid arguments are the model's to fix: a result it can read, not an indeterminate outcome.
+        Promise.resolve(input.durableTool.execute(toolCallArgs(input.toolCall), context, effectiveSignal)).catch((error) => {
+          if (isSchemaParseError(error)) return { ok: false, content: null, error: toolErrorMessage(error) };
+          throw error;
+        }),
         input.options?.signal
       );
       if (!("ok" in result)) {
@@ -11606,6 +11633,12 @@ function toJsonValue(value) {
   return value;
 }
 function toolCallArgs(toolCall) {
+  if (typeof toolCall.arguments === "string") {
+    try {
+      return toJsonValue(JSON.parse(toolCall.arguments));
+    } catch {
+    }
+  }
   return toJsonValue(toolCall.arguments);
 }
 function createMergedToolCall(toolCallId, status, current, fallbackTitle, fallbackKind = "other") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rowan-agent/agent",
-  "version": "0.35.1",
+  "version": "0.35.3",
   "license": "MIT",
   "repository": {
     "type": "git",