open-agents-ai 0.96.1 → 0.97.0

package/dist/index.js

@@ -1558,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
           };
         }
@@ -3193,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: {
@@ -3217,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"),
@@ -17144,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;
@@ -17399,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.1",
+  "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.