protoagent 0.1.15 → 0.1.17

package/dist/tools/write-file.js

@@ -4,7 +4,6 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 import { validatePath } from '../utils/path-validation.js';
-import { findSimilarPaths } from '../utils/path-suggestions.js';
 import { requestApproval } from '../utils/approval.js';
 import { recordRead } from '../utils/file-time.js';
 export const writeFileTool = {
@@ -23,22 +22,8 @@ export const writeFileTool = {
     },
 };
 export async function writeFile(filePath, content, sessionId) {
-    let validated;
-    try {
-        validated = await validatePath(filePath);
-    }
-    catch (err) {
-        // If file not found, try to suggest similar paths
-        if (err.message?.includes('does not exist') || err.code === 'ENOENT') {
-            const suggestions = await findSimilarPaths(filePath);
-            let msg = `File not found: '${filePath}'`;
-            if (suggestions.length > 0) {
-                msg += '\nDid you mean one of these?\n' + suggestions.map(s => `  ${s}`).join('\n');
-            }
-            return msg;
-        }
-        throw err;
-    }
+    // validatePath handles non-existent files by validating the parent directory
+    const validated = await validatePath(filePath);
     // Request approval
     const preview = content.length > 500
         ? `${content.slice(0, 250)}\n... (${content.length} chars total) ...\n${content.slice(-250)}`

package/dist/utils/path-validation.js

@@ -7,11 +7,14 @@
  */
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import isPathInside from 'is-path-inside';
 const workingDirectory = process.cwd();
 let allowedRoots = [];
+function isWithinRoot(targetPath, rootPath) {
+    const relative = path.relative(rootPath, targetPath);
+    return relative === '' || (!relative.startsWith('..') && !path.isAbsolute(relative));
+}
 function isAllowedPath(targetPath) {
-    return isPathInside(targetPath, workingDirectory) || allowedRoots.some((root) => isPathInside(targetPath, root));
+    return isWithinRoot(targetPath, workingDirectory) || allowedRoots.some((root) => isWithinRoot(targetPath, root));
 }
 export async function setAllowedPathRoots(roots) {
     const normalizedRoots = await Promise.all(roots.map(async (root) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "protoagent",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "type": "module",
   "files": [
     "dist",
@@ -52,7 +52,6 @@
     "he": "^1.2.0",
     "html-to-text": "^9.0.5",
     "ink": "^6.8.0",
-    "is-path-inside": "^4.0.0",
     "jsonc-parser": "^3.3.1",
     "leven": "^4.1.0",
     "openai": "^5.23.1",