open-agents-ai 0.187.181 → 0.187.183

package/dist/index.js

@@ -3480,13 +3480,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 +3542,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 +244510,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 +244557,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) {
@@ -265356,6 +265379,45 @@ var init_ralphLoop = __esm({
   }
 });
 
+// packages/orchestrator/dist/textSanitize.js
+function cleanScaffolding(text) {
+  if (text == null)
+    return "";
+  let out = String(text);
+  out = out.replace(SIGNPOST_RE, "");
+  out = out.replace(RESTORED_BLOCK_RE, "");
+  out = out.replace(NEW_TASK_RE, "");
+  out = out.replace(SYSTEM_BLOCK_RE, "");
+  return out.trim();
+}
+function stripThinkTags(text) {
+  if (text == null)
+    return "";
+  return String(text).replace(THINK_BLOCK_RE, "").trim();
+}
+function cleanForStorage(text) {
+  if (text == null)
+    return "";
+  let out = String(text);
+  out = cleanScaffolding(out);
+  out = stripThinkTags(out);
+  out = out.replace(ANSI_ESCAPE_RE, "");
+  out = out.replace(/\s+/g, " ").trim();
+  return out;
+}
+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"() {
+    "use strict";
+    SIGNPOST_RE = /^\[Previous sessions exist[^\]]*\]\s*\n*/m;
+    NEW_TASK_RE = /^NEW TASK:\s*/m;
+    RESTORED_BLOCK_RE = /^[\s\S]*?\n---\s*\n\s*NEW TASK:\s*/m;
+    SYSTEM_BLOCK_RE = /^<system>[\s\S]*?<\/system>\s*\n*/m;
+    THINK_BLOCK_RE = /<think>[\s\S]*?<\/think>/g;
+    ANSI_ESCAPE_RE = /\x1b\[[0-9;]*[a-zA-Z]|\x1b\][^\x07]*\x07/g;
+  }
+});
+
 // packages/orchestrator/dist/personality.js
 function compilePersonalityPrompt(profile, presetName) {
   if (presetName === "stark") {
@@ -266224,10 +266286,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() {
@@ -267638,6 +267708,7 @@ var SYSTEM_PROMPT, SYSTEM_PROMPT_MEDIUM, SYSTEM_PROMPT_SMALL, AgenticRunner, Oll
 var init_agenticRunner = __esm({
   "packages/orchestrator/dist/agenticRunner.js"() {
     "use strict";
+    init_textSanitize();
     init_dist();
     init_personality();
     init_promptLoader();
@@ -268279,6 +268350,7 @@ Respond with your assessment, then take action.`;
         if (typeof this.backend.setAbortSignal === "function") {
           this.backend.setAbortSignal(this._abortController.signal);
         }
+        const cleanedTask = cleanForStorage(task) || task.slice(0, 500);
         const start2 = Date.now();
         const taskTimeoutMs = this.options.taskTimeoutMs;
         const selfEvalInterval = taskTimeoutMs;
@@ -268328,7 +268400,7 @@ Respond with your assessment, then take action.`;
         this._pauseResolve = null;
         this.pendingUserMessages.length = 0;
         this._taskState = {
-          goal: task.slice(0, 500),
+          goal: cleanedTask.slice(0, 500),
           completedSteps: [],
           pendingSteps: [],
           currentStep: "",
@@ -268652,7 +268724,7 @@ If you're stuck, try a completely different approach. Do NOT repeat what failed
           }
           if (turn > 0 && turn % 3 === 0 && this._temporalGraph && this._episodeStore) {
             try {
-              const taskGoal = this._taskState.goal || task.slice(0, 200);
+              const taskGoal = this._taskState.goal || cleanedTask.slice(0, 200);
               const pprResult = retrieveByPPR(taskGoal, this._temporalGraph, this._episodeStore, { topK: 3 });
               if (pprResult.episodes.length > 0) {
                 const memoryLines = pprResult.episodes.map(({ episode, pprScore, matchedNodes }) => `- [${episode.toolName ?? episode.modality}] ${episode.content.slice(0, 120)} (via: ${matchedNodes.slice(0, 2).join(", ")})`);
@@ -269907,7 +269979,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
           };
           const consolidation = {
             sessionId: this._sessionId,
-            task: task.slice(0, 500),
+            task: cleanedTask.slice(0, 500),
             outcome: completed ? "success" : this.aborted ? "aborted" : "timeout",
             turns: messages2.filter((m2) => m2.role === "assistant").length,
             toolsUsed: [...new Set(toolCallLog.map((tc) => tc.name))],
@@ -269926,7 +269998,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
           fs4.mkdirSync(provenanceDir, { recursive: true });
           const provenanceGraph = {
             sessionId: this._sessionId,
-            task: task.slice(0, 500),
+            task: cleanedTask.slice(0, 500),
             outcome: consolidation.outcome,
             timestamp: consolidation.timestamp,
             // Full action trace — every tool call with sequence ordering
@@ -269970,7 +270042,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
           fs4.writeFileSync(path5.join(provenanceDir, `${this._sessionId}.json`), JSON.stringify(provenanceGraph, null, 2));
           if (completed && this.tools.has("memory_write")) {
             const memTool = this.tools.get("memory_write");
-            const lessonContent = `Task "${task.slice(0, 100)}" completed successfully. Tools: ${consolidation.toolsUsed.join(", ")}. Files: ${consolidation.filesModified.slice(0, 3).join(", ")}. Duration: ${Math.round(durationMs / 1e3)}s, ${consolidation.turns} turns.`;
+            const lessonContent = `Task "${cleanedTask.slice(0, 100)}" completed successfully. Tools: ${consolidation.toolsUsed.join(", ")}. Files: ${consolidation.filesModified.slice(0, 3).join(", ")}. Duration: ${Math.round(durationMs / 1e3)}s, ${consolidation.turns} turns.`;
             try {
               await memTool.execute({
                 topic: "task_lessons",
@@ -269993,7 +270065,7 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
               sessionId: this._sessionId,
               modality: "gist",
               toolName: "task_complete",
-              content: `Task "${task.slice(0, 200)}" ${completed ? "completed" : "ended"}: ${summary.slice(0, 300)}`,
+              content: `Task "${cleanedTask.slice(0, 200)}" ${completed ? "completed" : "ended"}: ${cleanForStorage(summary).slice(0, 300)}`,
               importance: 9,
               decayClass: "procedural"
             });
@@ -270040,7 +270112,10 @@ Full content available via: repl_exec(code="data = retrieve('${handleId}')")  or
           fs4.mkdirSync(trajDir, { recursive: true });
           const trajectory = {
             id: this._sessionId,
-            task: task.slice(0, 1e3),
+            // CLEAN task — this file is the prerequisite for ALL future RL/RFT
+            // training. Storing the scaffolded version would teach future models
+            // to reproduce signpost text as part of their task understanding.
+            task: cleanedTask.slice(0, 1e3),
             outcome: completed ? "pass" : this.aborted ? "aborted" : "timeout",
             model: this.backend.model ?? "unknown",
             modelTier: this.options.modelTier ?? "large",
@@ -273597,6 +273672,7 @@ var init_sessionMetrics = __esm({
 var init_taskLearning = __esm({
   "packages/orchestrator/dist/taskLearning.js"() {
     "use strict";
+    init_textSanitize();
   }
 });
 
@@ -274200,6 +274276,7 @@ var init_dist8 = __esm({
     init_skill_fork();
     init_streaming_executor();
     init_app_state();
+    init_textSanitize();
   }
 });
 
@@ -303940,9 +304017,11 @@ function loadSessionHistory2(repoRoot) {
     if (!s2.startedAt || !s2.task) continue;
     const status = s2.completed ? "completed" : "incomplete";
     const date = s2.startedAt.split("T")[0];
-    lines.push(`- [${date}] ${s2.task.slice(0, 80)} (${status}, ${s2.turns ?? 0} turns)`);
+    const cleanTask = cleanForStorage(s2.task) || s2.task;
+    lines.push(`- [${date}] ${cleanTask.slice(0, 80)} (${status}, ${s2.turns ?? 0} turns)`);
     if (s2.summary) {
-      lines.push(`  Summary: ${s2.summary.slice(0, 120)}`);
+      const cleanSummary = cleanForStorage(s2.summary) || s2.summary;
+      lines.push(`  Summary: ${cleanSummary.slice(0, 120)}`);
     }
   }
   return lines.join("\n");
@@ -304111,6 +304190,7 @@ These patterns have been repeated 3+ times. Consider using create_tool to automa
 var init_project_context = __esm({
   "packages/cli/src/tui/project-context.ts"() {
     "use strict";
+    init_dist8();
     init_oa_directory();
     init_dist4();
   }
@@ -316618,7 +316698,7 @@ When done, either call task_complete with your answer, or use FINAL_VAR(variable
             ikState.stats.tool_use_count = (ikState.stats.tool_use_count || 0) + result.toolCalls;
           }
           ikState.version = (ikState.version || 1) + 1;
-          ikState.version_history.push({ version: ikState.version, change: "query_served: " + result.summary.slice(0, 60), timestamp: (/* @__PURE__ */ new Date()).toISOString() });
+          ikState.version_history.push({ version: ikState.version, change: "query_served: " + cleanForStorage(result.summary).slice(0, 60), timestamp: (/* @__PURE__ */ new Date()).toISOString() });
           if (ikState.version_history.length > 200) ikState.version_history = ikState.version_history.slice(-200);
           if (ikState.version % 10 === 0) {
             const specList = (ikState.specializations || []).join(", ") || "general-purpose";
@@ -316674,11 +316754,11 @@ When done, either call task_complete with your answer, or use FINAL_VAR(variable
         id: sessionId,
         startedAt: new Date(Date.now() - result.durationMs).toISOString(),
         completedAt: (/* @__PURE__ */ new Date()).toISOString(),
-        task,
+        task: cleanForStorage(task) || task.slice(0, 500),
         turns: result.turns,
         toolCalls: result.toolCalls,
         completed: result.completed,
-        summary: result.summary.slice(0, 500)
+        summary: cleanForStorage(result.summary).slice(0, 500)
       });
     } catch {
     }
@@ -320672,6 +320752,7 @@ var init_interactive = __esm({
     init_dist8();
     init_dist8();
     init_dist8();
+    init_dist8();
     init_dist4();
     init_dist();
     init_listen();

package/package.json

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