@rowan-agent/agent 0.35.0 → 0.35.2

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) {
@@ -5325,7 +5345,7 @@ var InMemoryStore = class _InMemoryStore {
   interruptOwner(ownerEpoch, message = "The previous Runtime owner expired or closed.") {
     this.ownerEpoch = Math.max(this.ownerEpoch, ownerEpoch);
     for (const run of this.runs.values()) {
-      if (run.state !== "running" || run.execution?.ownerEpoch !== ownerEpoch) continue;
+      if (run.state !== "running" || !run.execution || run.execution.ownerEpoch > ownerEpoch) continue;
       const activeToolCalls = [...this.toolCalls.values()].filter((toolCall) => toolCall.runId === run.id && toolCall.executionId === run.execution?.executionId && (toolCall.status === "pending" || toolCall.status === "in_progress" || toolCall.state === "pending" || toolCall.state === "in_progress"));
       const indeterminateToolCallIds = [];
       for (const toolCall of activeToolCalls) {
@@ -10668,12 +10688,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 +10811,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 +11187,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 +11630,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") {
@@ -11896,10 +11926,6 @@ var SqliteStore = class {
       }
       const state = this.readState();
       const memory = InMemoryStore.fromState(state);
-      if (row.owner_id && row.owner_token && row.epoch > 0) {
-        memory.attachOwner(ownerLease(row));
-        memory.interruptOwner(row.epoch);
-      }
       const epoch = row.epoch + 1;
       const token = `${state.incarnation}:${epoch}:${input.ownerId}`;
       const next = {
@@ -11921,6 +11947,7 @@ var SqliteStore = class {
       const tracker = new ChangeTracker();
       memory.setChangeRecorder(tracker);
       try {
+        if (row.epoch > 0) memory.interruptOwner(row.epoch);
         memory.dropSettledClaimReceipts();
         memory.compactTerminalReceipts();
       } finally {

package/package.json

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