open-agents-ai 0.187.182 → 0.187.184

package/dist/index.js

@@ -2652,14 +2652,16 @@ var init_glob_find = __esm({
 });
 
 // packages/execution/dist/tools/web-fetch.js
-var DEFAULT_MAX_LENGTH, SHORT_CONTENT_THRESHOLD, WebFetchTool;
+var DEFAULT_MAX_LENGTH, SHORT_CONTENT_THRESHOLD, CACHE_TTL_MS, WebFetchTool;
 var init_web_fetch = __esm({
   "packages/execution/dist/tools/web-fetch.js"() {
     "use strict";
     DEFAULT_MAX_LENGTH = 5e3;
     SHORT_CONTENT_THRESHOLD = 200;
+    CACHE_TTL_MS = 6e4;
     WebFetchTool = class {
       name = "web_fetch";
+      _fetchCache = /* @__PURE__ */ new Map();
       description = "Fetch a single web page and return its text content (HTML stripped to plain text). FASTEST web tool \u2014 use this for reading any single URL: documentation, articles, README files, API references, Stack Overflow answers. Limitations: no JavaScript rendering (SPAs/React apps return empty), no link following, no cookies/auth, no structured data extraction. If the page is blank or incomplete, switch to web_crawl with strategy='playwright'. For scraping/extracting structured data (prices, listings, tables), use web_crawl instead. For search engine queries, use web_search instead. For interactive browser sessions (login, form filling, clicking), use browser_action instead.";
       parameters = {
         type: "object",
@@ -2679,6 +2681,19 @@ var init_web_fetch = __esm({
         const url = args["url"];
         const maxLength = args["max_length"] ?? DEFAULT_MAX_LENGTH;
         const start2 = performance.now();
+        const cached = this._fetchCache.get(url);
+        if (cached && Date.now() - cached.fetchedAt < CACHE_TTL_MS) {
+          const slicedCache = cached.text.length > maxLength ? `${cached.text.slice(0, maxLength)}
+
+[Content truncated to ${maxLength} characters]` : cached.text;
+          return {
+            success: true,
+            output: `[CACHED \u2014 already fetched this URL in this session; the content below is the full response. Do NOT re-fetch the same URL. If the content seems short, the page may just be short or require JavaScript \u2014 use browser_action with a headless browser instead.]
+
+` + slicedCache,
+            durationMs: performance.now() - start2
+          };
+        }
         try {
           const response = await fetch(url, {
             headers: {
@@ -2697,6 +2712,7 @@ var init_web_fetch = __esm({
           }
           const html = await response.text();
           const text = this.#stripHtml(html);
+          this._fetchCache.set(url, { text, fetchedAt: Date.now() });
           if (text.length < SHORT_CONTENT_THRESHOLD) {
             return {
               success: true,
@@ -2711,7 +2727,7 @@ var init_web_fetch = __esm({
             success: true,
             output: truncated ? `${text.slice(0, maxLength)}
 
-[Content truncated to ${maxLength} characters]` : text,
+[Content truncated to ${maxLength} characters. The full response is cached for this session \u2014 subsequent calls to the same URL return the cached content.]` : text,
             durationMs: performance.now() - start2
           };
         } catch (error) {
@@ -3480,13 +3496,32 @@ ${JSON.stringify(entries, null, 2)}`,
 });
 
 // packages/execution/dist/tools/memory-write.js
-import { readFile as readFile4, writeFile as writeFile3, mkdir as mkdir2 } from "node:fs/promises";
+import { readFile as readFile4, writeFile as writeFile3, mkdir as mkdir2, rename } from "node:fs/promises";
 import { resolve as resolve8, join as join7 } from "node:path";
 import { randomBytes as randomBytes3 } from "node:crypto";
-var MemoryWriteTool;
+async function withTopicLock(topicFile, fn) {
+  const prev = _writeLocks.get(topicFile) ?? Promise.resolve();
+  let release2 = () => {
+  };
+  const current = new Promise((resolve39) => {
+    release2 = resolve39;
+  });
+  _writeLocks.set(topicFile, prev.then(() => current));
+  try {
+    await prev;
+    return await fn();
+  } finally {
+    release2();
+    if (_writeLocks.get(topicFile) === prev.then(() => current)) {
+      _writeLocks.delete(topicFile);
+    }
+  }
+}
+var _writeLocks, MemoryWriteTool;
 var init_memory_write = __esm({
   "packages/execution/dist/tools/memory-write.js"() {
     "use strict";
+    _writeLocks = /* @__PURE__ */ new Map();
     MemoryWriteTool = class {
       name = "memory_write";
       description = "Store a fact or pattern in the agent's persistent memory. Use this to remember solutions, code patterns, or debugging insights for future tasks.";
@@ -3523,19 +3558,23 @@ var init_memory_write = __esm({
           for (const memoryDir of [oaMemDir, legacyMemDir]) {
             await mkdir2(memoryDir, { recursive: true });
             const topicFile = join7(memoryDir, `${topic}.json`);
-            let entries = {};
-            try {
-              const raw = await readFile4(topicFile, "utf-8");
-              entries = JSON.parse(raw);
-            } catch {
-            }
-            const artifactId = `mem-${randomBytes3(6).toString("hex")}`;
-            entries[key] = {
-              value: value2,
-              timestamp: (/* @__PURE__ */ new Date()).toISOString(),
-              artifactId
-            };
-            await writeFile3(topicFile, JSON.stringify(entries, null, 2), "utf-8");
+            await withTopicLock(topicFile, async () => {
+              let entries = {};
+              try {
+                const raw = await readFile4(topicFile, "utf-8");
+                entries = JSON.parse(raw);
+              } catch {
+              }
+              const artifactId = `mem-${randomBytes3(6).toString("hex")}`;
+              entries[key] = {
+                value: value2,
+                timestamp: (/* @__PURE__ */ new Date()).toISOString(),
+                artifactId
+              };
+              const tempFile = `${topicFile}.tmp-${randomBytes3(4).toString("hex")}`;
+              await writeFile3(tempFile, JSON.stringify(entries, null, 2), "utf-8");
+              await rename(tempFile, topicFile);
+            });
           }
           const provenanceDir = resolve8(this.workingDir, ".oa", "provenance");
           try {
@@ -244487,7 +244526,7 @@ var init_src107 = __esm({
 });
 
 // ../node_modules/steno/lib/index.js
-import { rename, writeFile as writeFile9 } from "node:fs/promises";
+import { rename as rename2, writeFile as writeFile9 } from "node:fs/promises";
 import { basename as basename5, dirname as dirname6, join as join22 } from "node:path";
 import { fileURLToPath as fileURLToPath2 } from "node:url";
 function getTempFilename(file) {
@@ -244534,7 +244573,7 @@ var init_lib3 = __esm({
         try {
           await writeFile9(this.#tempFilename, data, "utf-8");
           await retryAsyncOperation(async () => {
-            await rename(this.#tempFilename, this.#filename);
+            await rename2(this.#tempFilename, this.#filename);
           }, 10, 100);
           this.#prev?.[0]();
         } catch (err) {
@@ -265382,6 +265421,35 @@ function cleanForStorage(text) {
   out = out.replace(/\s+/g, " ").trim();
   return out;
 }
+function extractTaskCompleteSummary(args) {
+  if (args == null)
+    return "";
+  if (typeof args === "string")
+    return stripThinkTags(args);
+  if (typeof args === "object") {
+    const obj = args;
+    if (typeof obj.summary === "string" && obj.summary.trim().length > 0) {
+      return stripThinkTags(obj.summary);
+    }
+    for (const alt of ["text", "message", "result", "output", "content", "response", "answer"]) {
+      if (typeof obj[alt] === "string" && obj[alt].trim().length > 0) {
+        return stripThinkTags(obj[alt]);
+      }
+    }
+    if (typeof obj[""] === "string" && obj[""].trim().length > 0) {
+      let value2 = obj[""];
+      value2 = value2.replace(/^[^>]*>\s*\n?/, "");
+      value2 = value2.replace(/\s*<\/[^>]+>\s*$/, "");
+      return stripThinkTags(value2.trim());
+    }
+    const stringValues = Object.values(obj).filter((v) => typeof v === "string" && v.trim().length > 0);
+    if (stringValues.length > 0) {
+      const longest = stringValues.reduce((a2, b) => a2.length >= b.length ? a2 : b);
+      return stripThinkTags(longest);
+    }
+  }
+  return "";
+}
 var SIGNPOST_RE, NEW_TASK_RE, RESTORED_BLOCK_RE, SYSTEM_BLOCK_RE, THINK_BLOCK_RE, ANSI_ESCAPE_RE;
 var init_textSanitize = __esm({
   "packages/orchestrator/dist/textSanitize.js"() {
@@ -266080,6 +266148,15 @@ import { join as join63 } from "node:path";
 import { mkdirSync as mkdirSync24, existsSync as existsSync47 } from "node:fs";
 import { randomUUID as randomUUID5 } from "node:crypto";
 import { createHash as createHash3 } from "node:crypto";
+function sanitizeImportance(raw, fallback = 5) {
+  if (typeof raw !== "number" || !Number.isFinite(raw))
+    return fallback;
+  if (raw < 0)
+    return 0;
+  if (raw > 10)
+    return 10;
+  return raw;
+}
 function autoImportance(toolName, modality, content) {
   if (toolName === "task_complete")
     return 9;
@@ -266180,7 +266257,8 @@ var init_episodeStore = __esm({
         const now = Date.now();
         const contentHash = createHash3("sha256").update(ep.content).digest("hex").slice(0, 16);
         const modality = ep.modality ?? "text";
-        const importance = ep.importance ?? autoImportance(ep.toolName ?? null, modality, ep.content);
+        const rawImportance = ep.importance ?? autoImportance(ep.toolName ?? null, modality, ep.content);
+        const importance = sanitizeImportance(rawImportance);
         const decayClass = ep.decayClass ?? autoDecayClass(ep.toolName ?? null, modality, ep.content);
         const existing = this.db.prepare("SELECT id FROM episodes WHERE content_hash = ? AND session_id = ? LIMIT 1").get(contentHash, ep.sessionId ?? null);
         if (existing)
@@ -266263,10 +266341,18 @@ var init_episodeStore = __esm({
         const params = sessionId ? [sessionId, limit] : [limit];
         return this.db.prepare(sql).all(...params).map((r2) => this.rowToEpisode(r2));
       }
-      /** Count episodes. */
+      /** Count episodes.
+       *
+       *  BUG FIX (2026-04-07): previously passed `sessionId` to `.get()` even
+       *  when the SQL had no placeholder, causing `RangeError: Too many
+       *  parameter values were provided` on every `count()` call without args.
+       *  Now we only bind when the placeholder is actually present.
+       */
       count(sessionId) {
-        const sql = sessionId ? "SELECT COUNT(*) as n FROM episodes WHERE session_id = ?" : "SELECT COUNT(*) as n FROM episodes";
-        return this.db.prepare(sql).get(sessionId).n;
+        if (sessionId) {
+          return this.db.prepare("SELECT COUNT(*) as n FROM episodes WHERE session_id = ?").get(sessionId).n;
+        }
+        return this.db.prepare("SELECT COUNT(*) as n FROM episodes").get().n;
       }
       /** Prune expired session-class episodes (call at session end). */
       pruneExpired() {
@@ -266304,9 +266390,12 @@ var init_episodeStore = __esm({
           contentHash: row.content_hash,
           metadata: row.metadata ? JSON.parse(row.metadata) : null,
           embedding: row.embedding ? new Float32Array(new Uint8Array(row.embedding).buffer) : null,
-          importance: row.importance,
+          // Belt-and-braces: rows from pre-fix DBs may have null importance
+          // (from legacy inserts with NaN that SQLite coerced to NULL).
+          // Re-sanitize on read so downstream code always sees a valid number.
+          importance: sanitizeImportance(row.importance),
           decayClass: row.decay_class,
-          strength: row.strength,
+          strength: typeof row.strength === "number" && Number.isFinite(row.strength) ? row.strength : 1,
           lastRetrieved: row.last_retrieved,
           gist: row.gist,
           sourceEpisodeId: row.source_episode_id
@@ -269410,7 +269499,7 @@ ${sr.result.output}`;
                   messages2.push(this.buildToolMessage(output, matchTc.id));
                   if (matchTc.name === "task_complete") {
                     completed = true;
-                    summary = matchTc.arguments.summary || "";
+                    summary = extractTaskCompleteSummary(matchTc.arguments);
                     if (summary && !this._assistantTextEmitted) {
                       this.emit({ type: "assistant_text", content: summary, turn, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
                       this._assistantTextEmitted = true;
@@ -269429,7 +269518,7 @@ ${sr.result.output}`;
                     messages2.push(this.buildToolMessage(r2.output, r2.tc.id));
                     if (r2.tc.name === "task_complete") {
                       completed = true;
-                      summary = r2.tc.arguments.summary || "";
+                      summary = extractTaskCompleteSummary(r2.tc.arguments);
                       if (summary && !this._assistantTextEmitted) {
                         this.emit({ type: "assistant_text", content: summary, turn, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
                         this._assistantTextEmitted = true;
@@ -269458,7 +269547,7 @@ ${sr.result.output}`;
                     messages2.push(this.buildToolMessage(r2.output, r2.tc.id));
                     if (r2.tc.name === "task_complete") {
                       completed = true;
-                      summary = r2.tc.arguments.summary || "";
+                      summary = extractTaskCompleteSummary(r2.tc.arguments);
                       if (summary && !this._assistantTextEmitted) {
                         this.emit({ type: "assistant_text", content: summary, turn, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
                         this._assistantTextEmitted = true;
@@ -269858,7 +269947,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
                 messages2.push(toolMsg);
                 if (tc.name === "task_complete") {
                   completed = true;
-                  summary = tc.arguments.summary || "";
+                  summary = extractTaskCompleteSummary(tc.arguments);
                   if (summary && !this._assistantTextEmitted) {
                     this.emit({ type: "assistant_text", content: summary, turn, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
                     this._assistantTextEmitted = true;
@@ -274014,6 +274103,7 @@ var init_coordinator = __esm({
     "use strict";
     init_agent_types();
     init_agent_task();
+    init_textSanitize();
   }
 });
 
@@ -307861,6 +307951,7 @@ var init_dmn_engine = __esm({
   "packages/cli/src/tui/dmn-engine.ts"() {
     "use strict";
     init_dist8();
+    init_dist8();
     init_dist4();
     init_project_context();
     init_render();
@@ -307920,7 +308011,9 @@ var init_dmn_engine = __esm({
       }
       /** Record a task failure for Reflexion-style learning */
       recordTaskFailure(summary, errorContext, category) {
-        const reflection = errorContext ? `FAILED: ${summary.slice(0, 200)} \u2014 Error: ${errorContext.slice(0, 300)}` : `FAILED: ${summary.slice(0, 500)}`;
+        const cleanSummary = cleanForStorage(summary);
+        const cleanError = errorContext ? cleanForStorage(errorContext) : "";
+        const reflection = cleanError ? `FAILED: ${cleanSummary.slice(0, 200)} \u2014 Error: ${cleanError.slice(0, 300)}` : `FAILED: ${cleanSummary.slice(0, 500)}`;
         this.state.reflectionBuffer.push(reflection);
         if (this.state.reflectionBuffer.length > 5) {
           this.state.reflectionBuffer.shift();
@@ -315264,7 +315357,8 @@ function createSubAgentTool(config, repoRoot, ctxWindowSize) {
         return { success: false, output: "", error: "task is required" };
       }
       const agentId = `sub-agent-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
-      const agentLabel = `sub_agent: ${task.slice(0, 40)}${task.length > 40 ? "..." : ""}`;
+      const cleanedLabelTask = cleanForStorage(task) || task;
+      const agentLabel = `sub_agent: ${cleanedLabelTask.slice(0, 40)}${cleanedLabelTask.length > 40 ? "..." : ""}`;
       if (onViewRegister) {
         onViewRegister(agentId, agentLabel);
       }
@@ -315324,7 +315418,7 @@ function createSubAgentTool(config, repoRoot, ctxWindowSize) {
           if (onComplete) onComplete(agentId, task, result2.completed ? 0 : 1, output2);
           return output2;
         });
-        const taskId = taskManager.trackPromise(`sub_agent: ${task.slice(0, 80)}`, promise);
+        const taskId = taskManager.trackPromise(`sub_agent: ${cleanedLabelTask.slice(0, 80)}`, promise);
         return {
           success: true,
           output: `Sub-agent started in background: ${taskId}
@@ -320951,7 +321045,8 @@ function statusCommand(jobId, repoPath) {
   const runtime = job.completedAt ? `${((new Date(job.completedAt).getTime() - new Date(job.startedAt).getTime()) / 1e3).toFixed(0)}s` : `${((Date.now() - new Date(job.startedAt).getTime()) / 1e3).toFixed(0)}s`;
   const icon = job.status === "completed" ? "\u2713" : job.status === "failed" ? "\u2717" : "\u25CF";
   console.log(`${icon} ${job.id} [${job.status}] ${runtime}`);
-  console.log(`  Task: ${job.task.slice(0, 80)}`);
+  const cleanTask = cleanForStorage(job.task) || job.task;
+  console.log(`  Task: ${cleanTask.slice(0, 80)}`);
   console.log(`  PID: ${job.pid}`);
   if (job.summary) console.log(`  Summary: ${job.summary}`);
   if (job.error) console.log(`  Error: ${job.error}`);
@@ -320970,7 +321065,8 @@ function jobsCommand(repoPath) {
       const job = JSON.parse(readFileSync61(join95(dir, file), "utf-8"));
       const icon = job.status === "completed" ? "\u2713" : job.status === "failed" ? "\u2717" : "\u25CF";
       const runtime = job.completedAt ? `${((new Date(job.completedAt).getTime() - new Date(job.startedAt).getTime()) / 1e3).toFixed(0)}s` : `${((Date.now() - new Date(job.startedAt).getTime()) / 1e3).toFixed(0)}s`;
-      console.log(`  ${icon} ${job.id} [${job.status}] ${runtime} \u2014 ${job.task.slice(0, 60)}`);
+      const cleanListTask = cleanForStorage(job.task) || job.task;
+      console.log(`  ${icon} ${job.id} [${job.status}] ${runtime} \u2014 ${cleanListTask.slice(0, 60)}`);
     } catch {
     }
   }
@@ -320979,6 +321075,7 @@ var init_run = __esm({
   "packages/cli/src/commands/run.ts"() {
     "use strict";
     init_interactive();
+    init_dist8();
   }
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.182",
+  "version": "0.187.184",
   "description": "AI coding agent powered by open-source models (Ollama/vLLM) — interactive TUI with agentic tool-calling loop",
   "type": "module",
   "main": "./dist/index.js",