@molecule/api-ai-tools 1.10.0 → 1.10.1

package/dist/schemas.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"schemas.d.ts","sourceRoot":"","sources":["../src/schemas.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAA;AAElD;;GAEG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAA;IACZ,WAAW,EAAE,MAAM,CAAA;IACnB,UAAU,EAAE,UAAU,CAAA;CACvB;AAED;;GAEG;AACH,eAAO,MAAM,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAyU1C,CAAA"}
+{"version":3,"file":"schemas.d.ts","sourceRoot":"","sources":["../src/schemas.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAA;AAElD;;GAEG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAA;IACZ,WAAW,EAAE,MAAM,CAAA;IACnB,UAAU,EAAE,UAAU,CAAA;CACvB;AAED;;GAEG;AACH,eAAO,MAAM,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CA2U1C,CAAA"}

package/dist/schemas.js

@@ -132,7 +132,8 @@ export const TOOL_SCHEMAS = {
                 },
                 include: {
                     type: 'string',
-                    description: "File glob pattern to filter files (e.g. '*.ts', '*.tsx'). Optional.",
+                    description: "File glob pattern to filter files (e.g. '*.ts', '*.tsx'). Several extensions go in one " +
+                        "brace group of comma-separated alternatives: '*.{ts,tsx,yml,json,md}' (up to 16). Optional.",
                 },
                 contextLines: {
                     type: 'number',

package/dist/tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tools.d.ts","sourceRoot":"","sources":["../src/tools.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAA;AA4K9C;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAahG;AA6ZD,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AAqFnE;;;GAGG;AACH,eAAO,MAAM,iBAAiB,QAEA,CAAA;AAE9B;;;;;;;;;;;;;GAaG;AACH,eAAO,MAAM,wBAAwB,QAI6C,CAAA;AAmBlF;;;;;;;;;;;GAWG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAUhE;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,WAAW,qBAAqB;IACpC,iFAAiF;IACjF,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAA;IACzB,8FAA8F;IAC9F,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAA;CAC5B;AAED;;;;;;;GAOG;AACH,MAAM,MAAM,qBAAqB,GAAG,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,qBAAqB,CAAC,CAAA;AAoBpF;;;;;;;GAOG;AACH,wBAAgB,0BAA0B,CACxC,GAAG,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC;IAAE,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,CAAC,GACxE,qBAAqB,CAkBvB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AACH,wBAAsB,4BAA4B,CAChD,IAAI,EAAE,MAAM,EACZ,KAAK,CAAC,EAAE,qBAAqB,GAC5B,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAiCxB;AA2GD;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CAguFxF"}
+{"version":3,"file":"tools.d.ts","sourceRoot":"","sources":["../src/tools.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAA;AA4K9C;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAahG;AA6ZD,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AAqFnE;;;GAGG;AACH,eAAO,MAAM,iBAAiB,QAEA,CAAA;AAE9B;;;;;;;;;;;;;GAaG;AACH,eAAO,MAAM,wBAAwB,QAI6C,CAAA;AA2ElF;;;;;;;;;;;GAWG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAUhE;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,WAAW,qBAAqB;IACpC,iFAAiF;IACjF,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAA;IACzB,8FAA8F;IAC9F,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAA;CAC5B;AAED;;;;;;;GAOG;AACH,MAAM,MAAM,qBAAqB,GAAG,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,qBAAqB,CAAC,CAAA;AAoBpF;;;;;;;GAOG;AACH,wBAAgB,0BAA0B,CACxC,GAAG,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC;IAAE,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,CAAC,GACxE,qBAAqB,CAkBvB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AACH,wBAAsB,4BAA4B,CAChD,IAAI,EAAE,MAAM,EACZ,KAAK,CAAC,EAAE,qBAAqB,GAC5B,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAiCxB;AA2GD;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CAyuFxF"}

package/dist/tools.js

@@ -596,6 +596,62 @@ export const SUITE_OVER_BUDGET_ADVICE = 'A test suite is not a bigger unit to sp
  * sentence, which names an option the executor cannot set.
  */
 const SEARCH_TIMEOUT_MS = 10_000;
+/**
+ * The most alternatives one brace group in a `search_files` include may expand
+ * to. The point is to accept the ordinary multi-extension form a model writes
+ * from habit (`*.{ts,tsx,yml,json,md}`), not to grow a glob engine: a group is
+ * bounded so a pattern cannot turn one grep call into hundreds.
+ */
+const MAX_INCLUDE_BRACE_ALTERNATIVES = 16;
+/**
+ * Expand ONE level of brace alternatives in a `search_files` include pattern,
+ * validating every expanded pattern.
+ *
+ * `grep --include` has no brace expansion of its own (the shell that would do it
+ * is never involved — the pattern is `shellQuote`d first), so the group has to
+ * be expanded here and each alternative passed as its own `--include`. The
+ * m189 run wrote the everyday `*.{ts,tsx,yml,json,md}` and the search was
+ * refused outright, costing a round trip and pushing the agent to a narrower
+ * search.
+ *
+ * Bounded on purpose: one level of braces, alternatives split on `,`, at most
+ * {@link MAX_INCLUDE_BRACE_ALTERNATIVES}, none empty, and each EXPANDED
+ * pattern must still pass {@link isValidGlob} — so a shell metacharacter in any
+ * alternative (`*.{ts,$(rm)}`) is refused exactly as a bare one is.
+ *
+ * @param pattern - The user-supplied `include` value.
+ * @returns Every pattern to pass to grep, or null when the value is not
+ *   accepted (the caller reports the include-glob error).
+ */
+function expandIncludeGlob(pattern) {
+    const open = pattern.indexOf('{');
+    if (open === -1)
+        return isValidGlob(pattern) ? [pattern] : null;
+    const close = pattern.indexOf('}', open + 1);
+    if (close === -1)
+        return null;
+    // One level only: a second `{` before the close or a second `}` after it is a
+    // nested/malformed group, not an alternative.
+    const nextOpen = pattern.indexOf('{', open + 1);
+    if (nextOpen !== -1 && nextOpen < close)
+        return null;
+    if (pattern.indexOf('}', close + 1) !== -1)
+        return null;
+    const alternatives = pattern.slice(open + 1, close).split(',');
+    if (alternatives.length > MAX_INCLUDE_BRACE_ALTERNATIVES)
+        return null;
+    if (alternatives.some((alt) => alt.length === 0))
+        return null;
+    const expanded = alternatives.map((alt) => `${pattern.slice(0, open)}${alt}${pattern.slice(close + 1)}`);
+    return expanded.every((p) => isValidGlob(p)) ? expanded : null;
+}
+/**
+ * The include-glob error. It keeps the original character-rule sentence and
+ * names the accepted multi-extension form, so a refusal teaches the syntax
+ * rather than only reporting that the value was wrong.
+ */
+const INCLUDE_GLOB_ERROR = 'Invalid include glob pattern. Allowed: alphanumeric and @ * ? . _ - / [ ] ( ), ' +
+    'or one brace group of up to 16 comma-separated alternatives such as *.{ts,tsx}.';
 /**
  * The share of its budget a command must use before the "you asked for more than
  * this tool's ceiling" note is worth attaching. Below it the clamp changed nothing
@@ -1654,7 +1710,14 @@ export function buildTools(backend, config) {
                 await backend.writeFile(path, content);
                 if (onAfterWrite)
                     await onAfterWrite(path).catch(() => { });
-                const finalContent = await backend.readFile(path).catch(() => content);
+                // The text on disk is the text just written unless an after-write hook (a formatter)
+                // may have rewritten it. Without one, the re-read is a whole sandbox round trip per
+                // write that can only return what is already in hand — measured in the write path's
+                // own timings (x0 fast-file-writes: four sandbox calls per write, one of them this
+                // re-read, and the excerpt it feeds is identical either way).
+                const finalContent = onAfterWrite
+                    ? await backend.readFile(path).catch(() => content)
+                    : content;
                 const diff = computeDiff(oldContent, finalContent);
                 if (onFileChange)
                     onFileChange({ type: oldContent === null ? 'created' : 'modified', path });
@@ -2179,15 +2242,18 @@ export function buildTools(backend, config) {
             const contextLines = Number.isFinite(rawContext)
                 ? Math.min(Math.max(Math.trunc(rawContext), 0), MAX_SEARCH_CONTEXT_LINES)
                 : 0;
-            if (include && !isValidGlob(include))
-                return {
-                    error: 'Invalid include glob pattern. Allowed: alphanumeric and @ * ? . _ - / [ ] ( ).',
-                };
+            // `--include` has no brace expansion inside the quoted grep argument, so
+            // a brace group is expanded here and grep gets one --include per pattern.
+            const includes = include ? expandIncludeGlob(include) : [];
+            if (includes === null)
+                return { error: INCLUDE_GLOB_ERROR };
             const symlinkErr = await checkSymlink(path);
             if (symlinkErr)
                 return { error: symlinkErr };
             try {
-                const globArg = include ? `--include=${shellQuote(include)}` : '';
+                const globArg = includes.length
+                    ? includes.map((p) => `--include=${shellQuote(p)}`).join(' ')
+                    : '';
                 // With context, grep emits `file-line-content` for context lines and
                 // `file:line:content` for matches, plus `--` group separators. Both
                 // shapes are parsed below; without context the output is unchanged.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@molecule/api-ai-tools",
-  "version": "1.10.0",
+  "version": "1.10.1",
   "description": "Shared AI agent tools with backend abstraction for sandbox and local execution",
   "type": "module",
   "main": "dist/index.js",