loopengine 0.1.52 → 0.1.53

package/dist/core/run-agent.js

@@ -13,6 +13,7 @@ import { BudgetTracker } from './budget.js';
 import { Compactor } from './compaction.js';
 import { ToolLane } from './toollane.js';
 import { Recovery } from './recovery.js';
+import { collectKnownUrls, correctToolUseInput, recordKnownUrlsFromResult } from './known-urls.js';
 import { loadAgentModule } from './discover-agents.js';
 import { agentAsTool } from './agent-as-tool.js';
 import { loadGatewayToolsFromDir } from './gateway-tools.js';
@@ -458,6 +459,12 @@ async function runLoop(ctx, messages, newMessages, starterMessage) {
         messages.push(message);
         newMessages.push(message);
     }
+    // See known-urls.ts's own doc comment — seeded once from this
+    // session's full prior history (a resumed session's own tool results,
+    // not just this run's), then kept current live as this run's own new
+    // toollane:result events come in below. One map for the whole run, not
+    // re-derived every turn.
+    const knownUrlsByBase = collectKnownUrls(messages);
     const recovery = new Recovery({
         onPromptTooLong: async (currentMessages) => {
             // newMessages (this turn's own content — not durably stored
@@ -566,6 +573,12 @@ async function runLoop(ctx, messages, newMessages, starterMessage) {
         // changing toollane's own types.
         const approvedMetaById = new Map();
         for (const block of toolUseBlocks) {
+            // Before anything else looks at this block's own input at all
+            // (Skill's own args included) — see known-urls.ts's own doc
+            // comment: a URL the model just copied from an earlier tool
+            // result into this call's own arguments gets snapped back to the
+            // known-good one sharing its base, if they differ.
+            correctToolUseInput(block, knownUrlsByBase);
             // Skill invocation injects instructions into context; it isn't a
             // real tool call and doesn't need ActAuth gating or ToolLane
             // scheduling — but it's still a tool_use block the model emitted,
@@ -737,6 +750,11 @@ async function runLoop(ctx, messages, newMessages, starterMessage) {
                 const summary = result.status === 'fulfilled' ? JSON.stringify(result.value) : `ERROR: ${result.error}`;
                 const meta = approvedMetaById.get(result.id);
                 log({ type: 'toollane:result', name: result.name, summary });
+                // A later tool call this same run (or a follow-up message in
+                // this same session) can reference a URL from this result —
+                // see known-urls.ts's own doc comment. Harmless no-op for a
+                // non-JSON/error summary.
+                recordKnownUrlsFromResult(summary, knownUrlsByBase);
                 resultBlocks.push({
                     type: 'tool_result',
                     tool_use_id: result.id,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "loopengine",
-  "version": "0.1.52",
+  "version": "0.1.53",
   "type": "module",
   "description": "Runtime for tool-using AI agents with a transparent ReAct loop.",
   "license": "MIT",