@oxecli/oxe 1.0.75 → 1.0.77

package/dist/engine.js

@@ -103,7 +103,7 @@ export class InferenceEngine {
                     result = await toolBash(args.command, args.timeout, args.cwd);
                     break;
                 case "glob":
-                    result = toolGlob(args.pattern, args.path, args.limit);
+                    result = toolGlob(args.pattern, args.path, args.limit, args.depth);
                     break;
                 case "grep":
                     result = toolGrep(args.pattern, args.path, args.glob, args.limit);

package/dist/tools.js

@@ -44,7 +44,7 @@ function displayDiff(pathName, oldContent, newContent) {
     if (oldContent.length + newContent.length > max_diff_source_chars) {
         pendingDiffOutput.push(`\x1b[90m${pathName}: ${oldContent.length.toLocaleString()} chars -> ${newContent.length.toLocaleString()} chars ` +
             `(diff hidden: exceeds ${max_diff_source_chars.toLocaleString()} char limit)\x1b[0m`);
-        return;
+        return null;
     }
     const patch = structuredPatch(pathName, pathName, oldContent, newContent, "", "", { context: max_diff_context_lines });
     const contentLines = [];
@@ -74,7 +74,7 @@ function displayDiff(pathName, oldContent, newContent) {
     const added = contentLines.filter((c) => c.kind === "+").length;
     const removed = contentLines.filter((c) => c.kind === "-").length;
     if (!added && !removed)
-        return;
+        return null;
     const shown = [];
     let run = [];
     let lastKind = null;
@@ -121,51 +121,30 @@ function displayDiff(pathName, oldContent, newContent) {
         if (v.newNum)
             maxNum = Math.max(maxNum, v.newNum);
     }
-    const numW = Math.max(String(maxNum).length, 2);
-    let contentW = 0;
-    for (const v of visible) {
-        const len = v.kind === "~" ? plainLen(v.body) : numW + 1 + 2 + plainLen(v.body);
-        contentW = Math.max(contentW, len);
-    }
-    const headerPlain = `${pathName}  +${added}  -${removed}`;
-    const boxW = Math.min(Math.max(contentW + 6, plainLen(headerPlain) + 6), Math.max(termW - 2, 20));
-    const innerW = Math.max(boxW - 4, 1);
-    const maxPath = Math.max(10, boxW - 2 - plainLen(`  +${added}  -${removed}`) - 3);
-    let pathDisp = pathName;
-    if (plainLen(pathDisp) > maxPath)
-        pathDisp = truncateStyled(pathDisp, Math.max(maxPath - 1, 1)) + "…";
-    const header = `\x1b[1m${pathDisp}\x1b[0m \x1b[32m+${added}\x1b[0m \x1b[31m-${removed}\x1b[0m`;
-    const headerStr = `─ ${header} `;
-    const top = `╭${headerStr}${"─".repeat(Math.max(0, boxW - 2 - plainLen(headerStr)))}╮`;
-    pendingDiffOutput.push(`\x1b[90m${top}\x1b[0m`);
-    const bodyMax = Math.max(innerW - numW - 3, 1);
+    const numW = String(maxNum).length;
+    const bodyMax = Math.max(termW - numW - 4, 20);
     const gap = " ".repeat(numW);
     for (const v of visible) {
         if (v.kind === "~") {
-            const body = truncateStyled(v.body, innerW - 2);
-            const text = `\x1b[90m${gap}   ${body}\x1b[0m`;
-            const fill = Math.max(0, innerW - plainLen(text));
-            pendingDiffOutput.push(`\x1b[90m│\x1b[0m ${text}${" ".repeat(fill)} \x1b[90m│\x1b[0m`);
+            const body = truncateStyled(v.body, bodyMax);
+            pendingDiffOutput.push(`\x1b[90m${gap}   ${body}\x1b[0m`);
             continue;
         }
         const num = v.kind === "+" ? v.newNum : v.oldNum;
         const numStr = num ? String(num).padStart(numW, " ") : gap;
         if (v.kind === "+" || v.kind === "-") {
-            const color = v.kind === "+" ? "\x1b[32m" : "\x1b[31m";
+            const bg = v.kind === "+" ? "\x1b[42m" : "\x1b[41m";
+            const sign = v.kind === "+" ? "+" : "-";
             const body = truncateStyled(v.body, bodyMax);
-            const text = `${numStr} ${v.kind === "+" ? "+ " : "- "}${body}`;
-            const fill = Math.max(0, innerW - plainLen(text));
-            // Whole line is painted green/red, filling to the right edge.
-            pendingDiffOutput.push(`\x1b[90m│\x1b[0m ${color}${text}${" ".repeat(fill)}\x1b[0m \x1b[90m│\x1b[0m`);
+            const fill = Math.max(0, termW - numW - 4 - plainLen(body));
+            pendingDiffOutput.push(`${bg}\x1b[97m${numStr} ${sign}  ${body}${" ".repeat(fill)}\x1b[0m`);
         }
         else {
             const body = truncateStyled(v.kind === "\\" ? v.body : highlightCodeLine(v.body), bodyMax);
-            const text = `\x1b[90m${numStr}  ${body}\x1b[0m`;
-            const fill = Math.max(0, innerW - plainLen(text));
-            pendingDiffOutput.push(`\x1b[90m│\x1b[0m ${text}${" ".repeat(fill)} \x1b[90m│\x1b[0m`);
+            pendingDiffOutput.push(`\x1b[90m${numStr}  ${body}\x1b[0m`);
         }
     }
-    pendingDiffOutput.push(`\x1b[90m╰${"─".repeat(boxW - 2)}╯\x1b[0m`);
+    return { added, removed };
 }
 export function truncateToolOutput(output, maxChars = max_output_chars) {
     if (output.length > maxChars) {
@@ -491,7 +470,6 @@ export function toolEditFile(pathName, oldString, newString, replaceAll = false)
     const text = data.toString("utf-8");
     const newline = detectNewline(text);
     let match;
-    let normalized = false;
     if (text.includes(oldString)) {
         match = oldString;
     }
@@ -499,7 +477,6 @@ export function toolEditFile(pathName, oldString, newString, replaceAll = false)
         const norm = withNewlines(oldString, newline);
         if (text.includes(norm)) {
             match = norm;
-            normalized = true;
         }
         else {
             const blocks = findWsBlocks(text, oldString);
@@ -508,7 +485,6 @@ export function toolEditFile(pathName, oldString, newString, replaceAll = false)
                     "(line endings and per-line leading/trailing whitespace are normalized for matching)");
             }
             match = blocks[0];
-            normalized = true;
         }
     }
     const count = text.split(match).length - 1;
@@ -519,15 +495,17 @@ export function toolEditFile(pathName, oldString, newString, replaceAll = false)
     const newContent = replaceAll
         ? text.split(match).join(insert)
         : text.replace(match, insert);
-    displayDiff(pathName, text, newContent);
+    const summary = displayDiff(pathName, text, newContent);
     try {
         fs.writeFileSync(p, newContent, "utf-8");
     }
     catch (err) {
         return `Error: ${err}`;
     }
-    return (`Edited ${pathName} (${count} occurrence${count !== 1 ? "s" : ""} replaced` +
-        `${normalized ? "; line endings and per-line whitespace normalized" : ""})`);
+    if (!summary)
+        return `Edited ${pathName}`;
+    return (`Added ${summary.added} line${summary.added !== 1 ? "s" : ""}, ` +
+        `removed ${summary.removed} line${summary.removed !== 1 ? "s" : ""}`);
 }
 // ---------------------------------------------------------------------------
 // bash
@@ -651,7 +629,7 @@ function globMatch(name, pattern) {
         .replace(/\?/g, ".");
     return new RegExp(`^${regex}$`, "i").test(name);
 }
-function walkFiles(root, current, rel, parts, out) {
+function walkFiles(root, current, rel, parts, out, depthLeft) {
     let entries;
     try {
         entries = fs.readdirSync(current, { withFileTypes: true });
@@ -665,8 +643,10 @@ function walkFiles(root, current, rel, parts, out) {
             if (entry.isDirectory()) {
                 if (ignoredDirs.has(name))
                     continue;
+                if (depthLeft <= 0)
+                    continue;
                 rel.push(name);
-                walkFiles(root, path.join(current, name), rel, parts, out);
+                walkFiles(root, path.join(current, name), rel, parts, out, depthLeft - 1);
                 rel.pop();
             }
             else if (entry.isFile()) {
@@ -683,13 +663,13 @@ function walkFiles(root, current, rel, parts, out) {
         }
     }
 }
-function rglobPruned(base, pattern) {
+function rglobPruned(base, pattern, depth = 12) {
     const norm = pattern.replace(/\\/g, "/");
     const parts = norm.split("/").filter((p) => p !== "" && p !== ".");
     const hasDotDot = parts.includes("..");
     if (hasDotDot) {
         const out = [];
-        const walk = (dir) => {
+        const walk = (dir, depthLeft) => {
             let entries;
             try {
                 entries = fs.readdirSync(dir, { withFileTypes: true });
@@ -700,8 +680,8 @@ function rglobPruned(base, pattern) {
             for (const e of entries) {
                 const full = path.join(dir, e.name);
                 if (e.isDirectory()) {
-                    if (!ignoredDirs.has(e.name))
-                        walk(full);
+                    if (!ignoredDirs.has(e.name) && depthLeft > 0)
+                        walk(full, depthLeft - 1);
                 }
                 else if (e.isFile()) {
                     out.push([full, fs.statSync(full).size]);
@@ -712,7 +692,7 @@ function rglobPruned(base, pattern) {
             out.push([base, fs.statSync(base).size]);
         }
         else {
-            walk(base);
+            walk(base, depth);
         }
         return out;
     }
@@ -724,17 +704,18 @@ function rglobPruned(base, pattern) {
     if (!parts.length)
         return [];
     const out = [];
-    walkFiles(base, base, [], ["**", ...parts], out);
+    walkFiles(base, base, [], ["**", ...parts], out, depth);
     return out;
 }
-export function toolGlob(pattern, pathName = ".", limit = 200) {
+export function toolGlob(pattern, pathName = ".", limit = 200, depth = 12) {
     const base = sanitizePath(pathName) || ".";
     if (!fs.existsSync(base))
         return `Error: path not found: ${pathName}`;
     const lim = typeof limit === "number" ? limit : parseInt(String(limit), 10) || 200;
+    const dep = Math.max(0, typeof depth === "number" ? depth : parseInt(String(depth), 10) || 12);
     let matches;
     try {
-        matches = rglobPruned(base, pattern);
+        matches = rglobPruned(base, pattern, dep);
     }
     catch {
         return `Error: path not found: ${pathName}`;
@@ -913,6 +894,10 @@ export const RESPONSES_TOOLS = [
                     type: "integer",
                     description: "Max number of matches to return (default 200)",
                 },
+                depth: {
+                    type: "integer",
+                    description: "Max directory depth to descend (default 12)",
+                },
             },
             required: ["pattern"],
         },

package/dist/ui.js

@@ -556,7 +556,7 @@ export class Spinner {
     draw() {
         const text = this.render ? this.render() : "";
         const icon = SPINNER_FRAMES[this.frame % SPINNER_FRAMES.length];
-        const rendered = `\x1b[36m${icon}\x1b[0m \x1b[90m${text}\x1b[0m`;
+        const rendered = `\x1b[1;36m${icon}\x1b[0m \x1b[90m${text}\x1b[0m`;
         // Nothing changed on this tick -> do not rewrite the line (avoids flicker).
         if (rendered === this.lastRendered)
             return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oxecli/oxe",
-  "version": "1.0.75",
+  "version": "1.0.77",
   "publishConfig": {
     "access": "public"
   },