reasonix 0.27.0 → 0.27.2

package/dist/index.js

@@ -4163,6 +4163,7 @@ function applyMemoryStack(basePrompt, rootDir) {
 // src/tools/filesystem.ts
 import { promises as fs } from "fs";
 import * as pathMod from "path";
+import picomatch2 from "picomatch";
 
 // src/index/config.ts
 import picomatch from "picomatch";
@@ -4257,6 +4258,20 @@ var AUTO_PREVIEW_HEAD_LINES = 80;
 var AUTO_PREVIEW_TAIL_LINES = 40;
 var SKIP_DIR_NAMES = new Set(DEFAULT_INDEX_EXCLUDES.dirs);
 var BINARY_EXTENSIONS = new Set(DEFAULT_INDEX_EXCLUDES.exts);
+function displayRel(rootDir, full) {
+  return pathMod.relative(rootDir, full).replaceAll("\\", "/");
+}
+var GLOB_METACHARS = /[*?{[]/;
+function compileNameFilter(filter) {
+  if (!filter) return null;
+  if (!GLOB_METACHARS.test(filter)) {
+    const needle = filter.toLowerCase();
+    return (name) => name.toLowerCase().includes(needle);
+  }
+  const matchPath = filter.includes("/");
+  const isMatch = picomatch2(filter, { dot: true, nocase: true });
+  return matchPath ? (_n, rel) => isMatch(rel) : (name) => isMatch(name);
+}
 function isLikelyBinaryByName(name) {
   const dot = name.lastIndexOf(".");
   if (dot < 0) return false;
@@ -4461,7 +4476,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
   });
   registry.register({
     name: "search_files",
-    description: "Find files whose NAME matches a substring or regex. Case-insensitive. Walks the directory recursively under the sandbox root. Returns one path per line.",
+    description: "Find files whose NAME matches a substring or regex. Case-insensitive. Walks the directory recursively under the sandbox root. Returns one path per line. Skips dependency / VCS / build directories (node_modules, .git, dist, build, .next, target, .venv) by default.",
     readOnly: true,
     parameters: {
       type: "object",
@@ -4470,6 +4485,10 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
         pattern: {
           type: "string",
           description: "Substring (or regex) to match against filenames."
+        },
+        include_deps: {
+          type: "boolean",
+          description: "When true, also walk node_modules / .git / dist / build / etc. Off by default \u2014 most filename searches are about the user's own code."
         }
       },
       required: ["pattern"]
@@ -4477,6 +4496,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
     fn: async (args) => {
       const startAbs = safePath(args.path ?? ".");
       const needle = args.pattern.toLowerCase();
+      const includeDeps = args.include_deps === true;
       let re = null;
       try {
         re = new RegExp(args.pattern, "i");
@@ -4497,7 +4517,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
           const lower = e.name.toLowerCase();
           const hit = re ? re.test(e.name) : lower.includes(needle);
           if (hit) {
-            const rel = pathMod.relative(rootDir, full);
+            const rel = displayRel(rootDir, full);
             if (totalBytes + rel.length + 1 > maxListBytes) {
               matches.push("[\u2026 search truncated \u2014 refine pattern \u2026]");
               return;
@@ -4505,7 +4525,10 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
             matches.push(rel);
             totalBytes += rel.length + 1;
           }
-          if (e.isDirectory()) await walk2(full);
+          if (e.isDirectory()) {
+            if (!includeDeps && SKIP_DIR_NAMES.has(e.name)) continue;
+            await walk2(full);
+          }
         }
       };
       await walk2(startAbs);
@@ -4529,7 +4552,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
         },
         glob: {
           type: "string",
-          description: "Optional file-name suffix or substring filter. Examples: '.ts' (only TypeScript), 'test' (any file with 'test' in the name). Reduces noise when you know the file shape."
+          description: "Optional filename filter. Real glob when the value contains `*`, `?`, `{`, or `[` \u2014 e.g. '*.ts', '**/*.tsx', 'src/**/*.{ts,tsx}'. Plain substring otherwise \u2014 e.g. '.ts' (suffix), 'test' (anywhere in the name). Patterns containing `/` match against the path relative to the search root; otherwise just the basename."
         },
         case_sensitive: {
           type: "boolean",
@@ -4546,7 +4569,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
       const startAbs = safePath(args.path ?? ".");
       const caseSensitive = args.case_sensitive === true;
       const includeDeps = args.include_deps === true;
-      const nameFilter = typeof args.glob === "string" ? args.glob.toLowerCase() : null;
+      const nameMatch = compileNameFilter(typeof args.glob === "string" ? args.glob : null);
       let re = null;
       try {
         re = new RegExp(args.pattern, caseSensitive ? "" : "i");
@@ -4574,9 +4597,9 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
             continue;
           }
           if (!e.isFile()) continue;
-          if (nameFilter && !e.name.toLowerCase().includes(nameFilter)) continue;
-          if (isLikelyBinaryByName(e.name)) continue;
           const full = pathMod.join(dir, e.name);
+          if (nameMatch && !nameMatch(e.name, displayRel(rootDir, full))) continue;
+          if (isLikelyBinaryByName(e.name)) continue;
           let stat2;
           try {
             stat2 = await fs.stat(full);
@@ -4593,7 +4616,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
           const firstNul = raw.indexOf(0);
           if (firstNul !== -1 && firstNul < 8 * 1024) continue;
           const text = raw.toString("utf8");
-          const rel = pathMod.relative(rootDir, full);
+          const rel = displayRel(rootDir, full);
           const lines = text.split(/\r?\n/);
           for (let li = 0; li < lines.length; li++) {
             const line = lines[li];
@@ -4658,7 +4681,7 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
       const abs = safePath(args.path);
       await fs.mkdir(pathMod.dirname(abs), { recursive: true });
       await fs.writeFile(abs, args.content, "utf8");
-      return `wrote ${args.content.length} chars to ${pathMod.relative(rootDir, abs)}`;
+      return `wrote ${args.content.length} chars to ${displayRel(rootDir, abs)}`;
     }
   });
   registry.register({
@@ -4684,17 +4707,17 @@ Prefer \`list_directory\` for a single-level view, \`search_files\` to find spec
       const adaptedReplace = args.replace.replace(/\r?\n/g, le);
       const firstIdx = before.indexOf(adaptedSearch);
       if (firstIdx < 0) {
-        throw new Error(`edit_file: search text not found in ${pathMod.relative(rootDir, abs)}`);
+        throw new Error(`edit_file: search text not found in ${displayRel(rootDir, abs)}`);
       }
       const nextIdx = before.indexOf(adaptedSearch, firstIdx + 1);
       if (nextIdx >= 0) {
         throw new Error(
-          `edit_file: search text appears multiple times in ${pathMod.relative(rootDir, abs)} \u2014 include more context to disambiguate`
+          `edit_file: search text appears multiple times in ${displayRel(rootDir, abs)} \u2014 include more context to disambiguate`
         );
       }
       const after = before.slice(0, firstIdx) + adaptedReplace + before.slice(firstIdx + adaptedSearch.length);
       await fs.writeFile(abs, after, "utf8");
-      const rel = pathMod.relative(rootDir, abs);
+      const rel = displayRel(rootDir, abs);
       const header = `edited ${rel} (${adaptedSearch.length}\u2192${adaptedReplace.length} chars)`;
       const startLine = before.slice(0, firstIdx).split(/\r?\n/).length;
       const diff = renderEditDiff(adaptedSearch, adaptedReplace, startLine);
@@ -4713,7 +4736,7 @@ ${diff}`;
     fn: async (args) => {
       const abs = safePath(args.path);
       await fs.mkdir(abs, { recursive: true });
-      return `created ${pathMod.relative(rootDir, abs)}/`;
+      return `created ${displayRel(rootDir, abs)}/`;
     }
   });
   registry.register({
@@ -4732,7 +4755,7 @@ ${diff}`;
       const dst = safePath(args.destination);
       await fs.mkdir(pathMod.dirname(dst), { recursive: true });
       await fs.rename(src, dst);
-      return `moved ${pathMod.relative(rootDir, src)} \u2192 ${pathMod.relative(rootDir, dst)}`;
+      return `moved ${displayRel(rootDir, src)} \u2192 ${displayRel(rootDir, dst)}`;
     }
   });
   return registry;
@@ -5866,7 +5889,7 @@ function splitOnChainOps(cmd) {
     const ch = cmd[i];
     if (quote) {
       if (ch === quote) quote = null;
-      else if (ch === "\\" && quote === '"' && i + 1 < cmd.length) i++;
+      else if (quote === '"' && isDqEscape(ch, cmd[i + 1])) i++;
       i++;
       atTokenStart = false;
       continue;
@@ -5938,7 +5961,7 @@ function parseSegment(segStr) {
     if (quote) {
       if (ch === quote) {
         quote = null;
-      } else if (ch === "\\" && quote === '"' && i + 1 < segStr.length) {
+      } else if (quote === '"' && isDqEscape(ch, segStr[i + 1])) {
         cur += segStr[++i] ?? "";
         curHasContent = true;
       } else {
@@ -6044,6 +6067,14 @@ function parseCommandChain(cmd) {
     }
     segments.push(parseSegment(trimmed));
   }
+  for (const seg of segments) {
+    const cmdName = seg.argv[0] ?? "";
+    if (cmdName.toLowerCase() === "cd") {
+      throw new UnsupportedSyntaxError(
+        "cd in parsed command chains does not change cwd for later segments. Use a command-native cwd flag instead, such as `npm --prefix <dir> run <script>`, `git -C <dir> ...`, or `cargo -C <dir> ...`."
+      );
+    }
+  }
   if (ops.length === 0 && segments[0].redirects.length === 0) return null;
   return { segments, ops };
 }
@@ -6341,6 +6372,9 @@ var BUILTIN_ALLOWLIST = [
   "ruff",
   "mypy"
 ];
+function isDqEscape(prev, next) {
+  return prev === "\\" && (next === '"' || next === "\\");
+}
 function tokenizeCommand(cmd) {
   const out = [];
   let cur = "";
@@ -6350,7 +6384,7 @@ function tokenizeCommand(cmd) {
     if (quote) {
       if (ch === quote) {
         quote = null;
-      } else if (ch === "\\" && quote === '"' && i + 1 < cmd.length) {
+      } else if (quote === '"' && isDqEscape(ch, cmd[i + 1])) {
         cur += cmd[++i];
       } else {
         cur += ch;
@@ -6392,7 +6426,7 @@ function detectShellOperator(cmd) {
     if (quote) {
       if (ch === quote) {
         quote = null;
-      } else if (ch === "\\" && quote === '"' && i + 1 < cmd.length) {
+      } else if (quote === '"' && isDqEscape(ch, cmd[i + 1])) {
         cur += cmd[++i];
         curQuoted = true;
       } else {
@@ -6708,7 +6742,7 @@ function registerShellTools(registry, opts) {
   const isAllowAll = typeof opts.allowAll === "function" ? opts.allowAll : () => opts.allowAll === true;
   registry.register({
     name: "run_command",
-    description: "Run a shell command in the project root and return its combined stdout+stderr.\n\nConstraints (read these before the first call):\n\u2022 Chain operators `|`, `||`, `&&`, `;` ARE supported \u2014 parsed natively, no shell invoked, so semantics are identical on Windows / macOS / Linux. Each chain segment is allowlist-checked individually: `git status | grep main` runs if both halves are allowed.\n\u2022 File redirects ARE supported: `>` truncate, `>>` append, `<` stdin from file, `2>` / `2>>` stderr to file, `2>&1` merge stderr\u2192stdout, `&>` both to file. Targets resolve relative to the project root. At most one redirect per fd per segment.\n\u2022 Background `&`, heredoc `<<`, command substitution `$(\u2026)`, subshells `(\u2026)`, and process substitution `<(\u2026)` are NOT supported. Wrap a literal `&` arg in quotes; for input use a `<` file or the binary's own --input flag.\n\u2022 Env-var expansion `$VAR` is NOT performed \u2014 `$VAR` is passed as a literal string. Use the binary's own --env flag or substitute the value yourself.\n\u2022 `cd` DOES NOT PERSIST between calls \u2014 each call spawns a fresh process rooted at the project. If a tool needs a subdirectory, pass it via the tool's own flag (`npm --prefix`, `cargo -C`, `git -C`, `pytest tests/\u2026`), NOT via a preceding `cd`.\n\u2022 Glob patterns (`*.ts`) are passed through as literal arguments \u2014 no shell expansion. Use `grep -r`, `rg`, `find -name`, etc.\n\u2022 Avoid commands with unbounded output (`netstat -ano`, `find /`, etc.) \u2014 they waste tokens. Filter at source: `netstat -ano -p TCP`, `find src -name '*.ts'`, `grep -c`, `wc -l`.\n\nCommon read-only inspection and test/lint/typecheck commands run immediately; anything that could mutate state, install dependencies, or touch the network is refused until the user confirms it in the TUI. Prefer this over asking the user to run a command manually \u2014 after edits, run the project's tests to verify.",
+    description: "Run a shell command in the project root and return its combined stdout+stderr.\n\nConstraints (read these before the first call):\n\u2022 Chain operators `|`, `||`, `&&`, `;` ARE supported \u2014 parsed natively, no shell invoked, so semantics are identical on Windows / macOS / Linux. Each chain segment is allowlist-checked individually: `git status | grep main` runs if both halves are allowed.\n\u2022 File redirects ARE supported: `>` truncate, `>>` append, `<` stdin from file, `2>` / `2>>` stderr to file, `2>&1` merge stderr\u2192stdout, `&>` both to file. Targets resolve relative to the project root. At most one redirect per fd per segment.\n\u2022 Background `&`, heredoc `<<`, command substitution `$(\u2026)`, subshells `(\u2026)`, and process substitution `<(\u2026)` are NOT supported. Wrap a literal `&` arg in quotes; for input use a `<` file or the binary's own --input flag.\n\u2022 Env-var expansion `$VAR` is NOT performed \u2014 `$VAR` is passed as a literal string. Use the binary's own --env flag or substitute the value yourself.\n\u2022 `cd` DOES NOT PERSIST between calls \u2014 each call spawns a fresh process rooted at the project. `cd` also does not persist within parsed chains like `cd dir && command`. Use a command-native cwd flag instead: `npm --prefix <dir> run <script>`, `npm --prefix <dir> exec -- <bin>`, `git -C <dir> ...`, `cargo -C <dir> ...`, `pytest <dir>/tests`.\n\u2022 Glob patterns (`*.ts`) are passed through as literal arguments \u2014 no shell expansion. Use `grep -r`, `rg`, `find -name`, etc.\n\u2022 Avoid commands with unbounded output (`netstat -ano`, `find /`, etc.) \u2014 they waste tokens. Filter at source: `netstat -ano -p TCP`, `find src -name '*.ts'`, `grep -c`, `wc -l`.\n\nCommon read-only inspection and test/lint/typecheck commands run immediately; anything that could mutate state, install dependencies, or touch the network is refused until the user confirms it in the TUI. Prefer this over asking the user to run a command manually \u2014 after edits, run the project's tests to verify.",
     // Plan-mode gate: allow allowlisted commands through (git status,
     // cargo check, ls, grep …) so the model can actually investigate
     // during planning. Anything that would otherwise trigger a