open-agents-ai 0.96.0 → 0.97.0

package/dist/index.js

@@ -1467,6 +1467,30 @@ ${stderr}` : ""),
 // packages/execution/dist/tools/file-read.js
 import { readFile } from "node:fs/promises";
 import { resolve } from "node:path";
+function extractPath(args) {
+  if (typeof args["path"] === "string" && args["path"].trim()) {
+    return args["path"].trim();
+  }
+  for (const key of ["file", "file_path", "filename", "filepath", "filePath"]) {
+    if (typeof args[key] === "string" && args[key].trim()) {
+      return args[key].trim();
+    }
+  }
+  if (typeof args["_raw"] === "string") {
+    const raw = args["_raw"].trim();
+    const match = raw.match(/["']?(?:path|file|filename)["']?\s*[:=]\s*["']([^"'\s}]+)["']/i);
+    if (match?.[1])
+      return match[1];
+    if (raw && !raw.includes("{") && !raw.includes(":") && raw.length < 500) {
+      return raw;
+    }
+  }
+  const stringVals = Object.values(args).filter((v) => typeof v === "string" && v.trim().length > 0 && v.length < 500);
+  if (stringVals.length === 1 && /[./]/.test(stringVals[0])) {
+    return stringVals[0].trim();
+  }
+  return null;
+}
 var FileReadTool;
 var init_file_read = __esm({
   "packages/execution/dist/tools/file-read.js"() {
@@ -1493,15 +1517,15 @@ var init_file_read = __esm({
         this._contextWindowSize = size;
       }
       async execute(args) {
-        const filePath = args["path"];
+        const filePath = extractPath(args);
         const offset = args["offset"];
         const limit = args["limit"];
         const start = performance.now();
-        if (!filePath || typeof filePath !== "string" || !filePath.trim()) {
+        if (!filePath) {
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: path. Provide the file path to read.",
+            error: `file_read requires "path" parameter. Call: file_read({"path": "src/example.ts"})`,
             durationMs: performance.now() - start
           };
         }
@@ -1534,10 +1558,21 @@ var init_file_read = __esm({
             durationMs: performance.now() - start
           };
         } catch (error) {
+          const errMsg = error instanceof Error ? error.message : String(error);
+          if (/ENOENT|no such file/i.test(errMsg)) {
+            const hasSlash = filePath.includes("/");
+            const hint = hasSlash ? `File not found: "${filePath}". Use list_directory on the parent directory to discover valid paths.` : `File not found: "${filePath}". This name is not a valid path. If you saw "${filePath}" inside a directory listing, use the FULL path (e.g., "parent_dir/${filePath}"). If it is a directory, use list_directory("${filePath}") instead of file_read.`;
+            return {
+              success: false,
+              output: "",
+              error: hint,
+              durationMs: performance.now() - start
+            };
+          }
           return {
             success: false,
             output: "",
-            error: error instanceof Error ? error.message : String(error),
+            error: errMsg,
             durationMs: performance.now() - start
           };
         }
@@ -1549,6 +1584,14 @@ var init_file_read = __esm({
 // packages/execution/dist/tools/file-write.js
 import { writeFile, mkdir } from "node:fs/promises";
 import { resolve as resolve2, dirname } from "node:path";
+function extractWritePath(args) {
+  for (const key of ["path", "file", "file_path", "filename", "filepath", "filePath"]) {
+    if (typeof args[key] === "string" && args[key].trim()) {
+      return args[key].trim();
+    }
+  }
+  return null;
+}
 var FileWriteTool;
 var init_file_write = __esm({
   "packages/execution/dist/tools/file-write.js"() {
@@ -1569,14 +1612,14 @@ var init_file_write = __esm({
         this.workingDir = workingDir;
       }
       async execute(args) {
-        const filePath = args["path"];
-        const content = args["content"];
+        const filePath = extractWritePath(args);
+        const content = args["content"] ?? args["text"] ?? args["data"];
         const start = performance.now();
-        if (!filePath || typeof filePath !== "string" || !filePath.trim()) {
+        if (!filePath) {
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: path. Provide the file path to write.",
+            error: `file_write requires "path" parameter. Call: file_write({"path": "src/file.ts", "content": "..."})`,
             durationMs: performance.now() - start
           };
         }
@@ -1584,7 +1627,7 @@ var init_file_write = __esm({
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: content. Provide the content to write.",
+            error: `file_write requires "content" parameter. Call: file_write({"path": "${filePath}", "content": "..."})`,
             durationMs: performance.now() - start
           };
         }
@@ -2425,6 +2468,14 @@ Meta: ${metaKeys.map((k) => `${k}="${meta[k]?.slice(0, 80)}"`).join(", ")}`);
 // packages/execution/dist/tools/file-edit.js
 import { readFile as readFile2, writeFile as writeFile2 } from "node:fs/promises";
 import { resolve as resolve5 } from "node:path";
+function extractEditPath(args) {
+  for (const key of ["path", "file", "file_path", "filename", "filepath", "filePath"]) {
+    if (typeof args[key] === "string" && args[key].trim()) {
+      return args[key].trim();
+    }
+  }
+  return null;
+}
 function countOccurrences(haystack, needle) {
   let count = 0;
   let pos = 0;
@@ -2478,16 +2529,16 @@ var init_file_edit = __esm({
         this.workingDir = workingDir;
       }
       async execute(args) {
-        const filePath = args["path"];
-        const oldString = args["old_string"];
-        const newString = args["new_string"];
-        const replaceAll = args["replace_all"] === true;
+        const filePath = extractEditPath(args);
+        const oldString = args["old_string"] ?? args["oldString"] ?? args["search"] ?? args["find"];
+        const newString = args["new_string"] ?? args["newString"] ?? args["replace"] ?? args["replacement"];
+        const replaceAll = args["replace_all"] === true || args["replaceAll"] === true;
         const start = performance.now();
-        if (!filePath || typeof filePath !== "string" || !filePath.trim()) {
+        if (!filePath) {
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: path. Provide the file path to edit.",
+            error: `file_edit requires "path" parameter. Call: file_edit({"path": "src/file.ts", "old_string": "...", "new_string": "..."})`,
             durationMs: performance.now() - start
           };
         }
@@ -2495,7 +2546,7 @@ var init_file_edit = __esm({
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: old_string. Provide the text to find and replace.",
+            error: `file_edit requires "old_string" parameter. Call: file_edit({"path": "${filePath}", "old_string": "text to find", "new_string": "replacement"})`,
             durationMs: performance.now() - start
           };
         }
@@ -2503,7 +2554,7 @@ var init_file_edit = __esm({
           return {
             success: false,
             output: "",
-            error: "Missing required parameter: new_string. Provide the replacement text.",
+            error: `file_edit requires "new_string" parameter. Call: file_edit({"path": "${filePath}", "old_string": "...", "new_string": "replacement text"})`,
             durationMs: performance.now() - start
           };
         }
@@ -3153,7 +3204,7 @@ var init_list_directory = __esm({
     MAX_ENTRIES = 100;
     ListDirectoryTool = class {
       name = "list_directory";
-      description = "List files and directories at a given path. Shows file sizes and types.";
+      description = "List files and directories at a given path. Shows file sizes and types. Output includes full relative paths you can use directly in subsequent tool calls.";
       parameters = {
         type: "object",
         properties: {
@@ -3177,18 +3228,40 @@ var init_list_directory = __esm({
           const entries = readdirSync(fullPath, { withFileTypes: true });
           const visible = entries.filter((e) => !EXCLUDED.has(e.name));
           const limited = visible.slice(0, MAX_ENTRIES);
+          const dirs = [];
+          const files = [];
           const lines = limited.map((entry) => {
-            const prefix = entry.isDirectory() ? "d" : "f";
+            const relPath = dirPath === "." ? entry.name : join7(dirPath, entry.name);
             if (entry.isDirectory()) {
-              return `${prefix}  ${entry.name}`;
+              dirs.push(relPath);
+              return `d  ${relPath}/`;
             }
             let size = 0;
             try {
               size = statSync(join7(fullPath, entry.name)).size;
             } catch {
             }
-            return `${prefix}  ${entry.name}  ${size}`;
+            files.push(relPath);
+            return `f  ${relPath}  ${size}`;
           });
+          if (dirs.length > 0 || files.length > 0) {
+            lines.push("");
+            lines.push("Next steps \u2014 use these exact paths:");
+            if (dirs.length > 0) {
+              for (const d of dirs.slice(0, 8)) {
+                lines.push(`  list_directory("${d}")`);
+              }
+              if (dirs.length > 8)
+                lines.push(`  ... and ${dirs.length - 8} more directories`);
+            }
+            if (files.length > 0) {
+              for (const f of files.slice(0, 5)) {
+                lines.push(`  file_read("${f}")`);
+              }
+              if (files.length > 5)
+                lines.push(`  ... and ${files.length - 5} more files`);
+            }
+          }
           return {
             success: true,
             output: lines.join("\n"),
@@ -17104,14 +17177,15 @@ ${result.output.length > maxLen ? this.foldOutput(result.output, maxLen) : resul
                 });
               }
               const enoentTools = /* @__PURE__ */ new Set(["file_read", "list_directory", "find_files"]);
-              if (!result.success && enoentTools.has(tc.name) && /ENOENT|no such file|does not exist/i.test(result.error ?? result.output)) {
+              if (!result.success && enoentTools.has(tc.name) && /ENOENT|no such file|does not exist|not found/i.test(result.error ?? result.output)) {
                 this._consecutiveEnoent++;
                 if (this._consecutiveEnoent >= 2) {
-                  this.pendingUserMessages.push(`[SYSTEM] You have hit ${this._consecutiveEnoent} consecutive ENOENT errors. STOP guessing paths. Instead:
-1. Use list_directory on the PROJECT ROOT to discover what actually exists
-2. If the missing path came from memory, use memory_write to remove the stale reference
-3. Navigate from the root to find the correct location
-Do NOT continue walking up parent directories.`);
+                  const failedPath = String(tc.arguments?.["path"] ?? tc.arguments?.["file"] ?? "unknown");
+                  this.pendingUserMessages.push(`[SYSTEM] ${this._consecutiveEnoent} consecutive file-not-found errors (last: "${failedPath}"). You are repeating failed paths. CHANGE YOUR APPROACH:
+1. Call list_directory(".") to see what exists at the project root
+2. Entries in a directory listing are RELATIVE to that directory. If you listed "parent/" and saw "child", the full path is "parent/child" \u2014 NOT ".child" or just "child"
+3. If an entry is a directory (marked "d"), use list_directory on it, NOT file_read
+4. Use the FULL relative paths shown in list_directory output \u2014 they are copy-paste ready`);
                 }
               } else {
                 this._consecutiveEnoent = 0;
@@ -17359,14 +17433,15 @@ Integrate this guidance into your current approach. Continue working on the task
 ${result.output}`;
                 this.emit({ type: "tool_result", toolName: tc.name, content: output.slice(0, 200), success: result.success, turn, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
                 const enoentTools2 = /* @__PURE__ */ new Set(["file_read", "list_directory", "find_files"]);
-                if (!result.success && enoentTools2.has(tc.name) && /ENOENT|no such file|does not exist/i.test(result.error ?? result.output)) {
+                if (!result.success && enoentTools2.has(tc.name) && /ENOENT|no such file|does not exist|not found/i.test(result.error ?? result.output)) {
                   this._consecutiveEnoent++;
                   if (this._consecutiveEnoent >= 2) {
-                    this.pendingUserMessages.push(`[SYSTEM] You have hit ${this._consecutiveEnoent} consecutive ENOENT errors. STOP guessing paths. Instead:
-1. Use list_directory on the PROJECT ROOT to discover what actually exists
-2. If the missing path came from memory, use memory_write to remove the stale reference
-3. Navigate from the root to find the correct location
-Do NOT continue walking up parent directories.`);
+                    const failedPath2 = String(tc.arguments?.["path"] ?? tc.arguments?.["file"] ?? "unknown");
+                    this.pendingUserMessages.push(`[SYSTEM] ${this._consecutiveEnoent} consecutive file-not-found errors (last: "${failedPath2}"). You are repeating failed paths. CHANGE YOUR APPROACH:
+1. Call list_directory(".") to see what exists at the project root
+2. Entries in a directory listing are RELATIVE to that directory. If you listed "parent/" and saw "child", the full path is "parent/child" \u2014 NOT ".child" or just "child"
+3. If an entry is a directory (marked "d"), use list_directory on it, NOT file_read
+4. Use the FULL relative paths shown in list_directory output \u2014 they are copy-paste ready`);
                   }
                 } else {
                   this._consecutiveEnoent = 0;

package/package.json

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

package/prompts/agentic/system-large.md

@@ -385,3 +385,11 @@ When a file_read, list_directory, or find_files call returns ENOENT (file/direct
 - If the missing path came from memory, update memory to remove the stale reference
 - After discovering the real structure, navigate to the correct path
 - Never make more than 2 consecutive attempts at paths that don't exist
+
+## Directory Listing Path Rules
+
+Entries in a directory listing are RELATIVE to the directory you listed.
+- If you call list_directory(".oa") and see "context", the full path is ".oa/context" — NOT ".context" or "context"
+- If an entry is marked "d" (directory), use list_directory on it — NOT file_read
+- list_directory output includes full relative paths you can copy directly into your next tool call
+- Prefer list_directory over shell ls — it shows full relative paths you can copy directly into your next tool call

package/prompts/agentic/system-medium.md

@@ -42,3 +42,6 @@ NEVER say "I can't do that". ALWAYS attempt the task using your tools. If a tool
 - You MUST call task_complete when done
 - Do NOT output long explanations. Focus on tool calls.
 - If file_read/list_directory returns ENOENT, use list_directory on the project root — do NOT guess parent paths
+- Directory listing entries are RELATIVE to the listed directory. If you list "parent/" and see "child", the full path is "parent/child" — NOT ".child" or just "child"
+- If an entry is a directory (d), use list_directory on it — NOT file_read
+- Prefer list_directory over shell ls — it shows full paths ready for your next tool call

package/prompts/agentic/system-small.md

@@ -15,3 +15,5 @@ Rules:
 - Run tests after every change.
 - Call task_complete when done.
 - If ENOENT, list_directory on project root. Don't guess paths.
+- Directory entries are RELATIVE. If you list "parent/" and see "child", the path is "parent/child" — NOT ".child".
+- Use list_directory for directories, NOT file_read. Prefer list_directory over shell ls.