@proagentstore/cli 0.4.144 → 0.4.145

package/dist/browser-runner/coding/inspect.js

@@ -1,3 +1,4 @@
+import { globMatches, isGlob } from "./path-glob.js";
 import { execFileSync } from "node:child_process";
 import { existsSync, readdirSync, readFileSync, realpathSync, statSync } from "node:fs";
 import { relative, resolve, sep } from "node:path";
@@ -331,9 +332,12 @@ export function repoSearch(workDir, opts) {
     const lines = out.split("\n").filter((l) => l !== "");
     if (mode === "path") {
         // The pattern never reaches git here — the filter is a plain case-insensitive substring
-        // over the path, which is what "find the file called X" actually means.
-        const all = lines.filter((p) => p.toLowerCase().includes(pattern.toLowerCase())).map((p) => ({ path: p }));
-        return { mode, pattern, matches: all.slice(0, limit), shown: Math.min(all.length, limit), total: all.length, truncated: all.length > limit };
+        // over the path, which is what "find the file called X" actually means. A `glob` (#1070) is
+        // matched instead, over the WHOLE list, so `total` counts real matches, not the literal's.
+        const glob = (opts.glob ?? "").trim();
+        const useGlob = glob !== "" && isGlob(glob);
+        const all = lines.filter((p) => (useGlob ? globMatches(glob, p) : p.toLowerCase().includes(pattern.toLowerCase()))).map((p) => ({ path: p }));
+        return { mode, pattern, matches: all.slice(0, limit), shown: Math.min(all.length, limit), total: all.length, truncated: all.length > limit, ...(useGlob ? { glob: true } : {}) };
     }
     const all = lines.map(parseGrepLine).filter((m) => m !== null);
     const perFile = new Map();

package/dist/browser-runner/coding/path-glob.js

@@ -0,0 +1,82 @@
+/**
+ * Glob patterns over repo-relative paths, for `repo_find` (#1070).
+ *
+ * `repo_find` matched its pattern as a plain substring, so `*.test.js` returned NOTHING — no path
+ * contains an asterisk — and an agent concluded a repository with ~90 test files had none. Silent false
+ * negatives are the failure; this is the matcher that ends them.
+ *
+ * Semantics (gitignore/ripgrep-like, case-insensitive like the substring search it extends):
+ *   `*`    any run of characters within ONE path segment (never a `/`)
+ *   `**`   any run across segments; `**` followed by `/` also matches zero folders
+ *   `?`    one character, not `/`
+ *   `[ab]` `[a-z]` `[!ab]`  a character class (`!` negates), never `/`
+ * A pattern WITHOUT a `/` matches any single segment — a file name or a folder name at any depth, a
+ * folder matching taking everything beneath it — so `*.test.js` and `*e2e*` mean what they mean in a
+ * shell or `rg -g`. A pattern WITH a `/` is anchored at the repository root (`src/*.ts`,
+ * `**\/handlers/*.js`); a leading `/` or `./` is accepted and means the same.
+ *
+ * DUPLICATED in `workers/api/src/lib/path-glob.ts`: the cloud filters with it when the runner is too old
+ * to (an old runner ignores `glob`). `workers/api/src/lib/path-glob.parity.test.ts` holds the two equal.
+ */
+/** Does this pattern use glob syntax? A pattern without any is the plain substring search, unchanged. */
+export function isGlob(pattern) {
+    return /[*?[]/.test(pattern);
+}
+function toRegExp(glob) {
+    let re = "";
+    for (let i = 0; i < glob.length; i++) {
+        const c = glob[i];
+        if (c === "*") {
+            if (glob[i + 1] === "*") {
+                i++;
+                if (glob[i + 1] === "/") {
+                    i++;
+                    re += "(?:.*/)?";
+                }
+                else
+                    re += ".*";
+            }
+            else
+                re += "[^/]*";
+        }
+        else if (c === "?")
+            re += "[^/]";
+        else if (c === "[") {
+            const end = glob.indexOf("]", i + 2);
+            if (end === -1) {
+                re += "\\[";
+                continue;
+            }
+            let body = glob.slice(i + 1, end).replace(/[\\^\]]/g, "\\$&");
+            const negate = body.startsWith("!");
+            if (negate)
+                body = body.slice(1);
+            re += `(?!/)[${negate ? "^" : ""}${body}]`;
+            i = end;
+        }
+        else
+            re += c.replace(/[.+^${}()|\\\]/]/g, "\\$&");
+    }
+    return new RegExp(`^${re}$`, "i");
+}
+/** Does `path` (repo-relative, `/`-separated) match `glob`? */
+export function globMatches(glob, path) {
+    const g = glob.trim().replace(/^\.?\/+/, "").replace(/\/+$/, "");
+    if (!g)
+        return false;
+    const re = toRegExp(g);
+    if (g.includes("/"))
+        return re.test(path);
+    return path.split("/").some((segment) => re.test(segment));
+}
+/**
+ * The longest run of literal text a match MUST contain — what an old runner's substring search is given,
+ * so the glob filter then runs over a superset that cannot have dropped a match. `""` when there is none
+ * (`*`, `**`, `???`): nothing literal to search by.
+ */
+export function globLiteral(glob) {
+    // Normalised exactly as `globMatches` normalises, or `./src/*.ts` would ask for "./src", which no path contains.
+    const pieces = glob.trim().replace(/^\.?\/+/, "").split(/\[[^\]]*\]|[*?]+/);
+    const longest = pieces.reduce((a, b) => (b.length > a.length ? b : a), "");
+    return longest.replace(/^\/+|\/+$/g, "");
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proagentstore/cli",
-  "version": "0.4.144",
+  "version": "0.4.145",
   "description": "CLI for creating, publishing, and running ProAgentStore agents",
   "license": "MIT",
   "type": "module",