@molecule/api-ai-tools 1.2.0 → 1.2.1

package/dist/backends/sandbox.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sandbox.d.ts","sourceRoot":"","sources":["../../src/backends/sandbox.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAGnD,sFAAsF;AACtF,UAAU,WAAW;IACnB,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAA;IACvC,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAA;IACvD,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAA;IACvC,OAAO,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,GAAG,WAAW,CAAA;KAAE,CAAC,CAAC,CAAA;IACnF,mEAAmE;IAEnE,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CACnB;AAED;;;;;;;GAOG;AACH,wBAAgB,oBAAoB,CAClC,OAAO,EAAE,WAAW,EACpB,WAAW,SAAe,GACzB,gBAAgB,CA2ClB"}
+{"version":3,"file":"sandbox.d.ts","sourceRoot":"","sources":["../../src/backends/sandbox.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAGnD,sFAAsF;AACtF,UAAU,WAAW;IACnB,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAA;IACvC,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAA;IACvD,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAA;IACvC,OAAO,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,GAAG,WAAW,CAAA;KAAE,CAAC,CAAC,CAAA;IACnF,mEAAmE;IAEnE,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CACnB;AAED;;;;;;;GAOG;AACH,wBAAgB,oBAAoB,CAClC,OAAO,EAAE,WAAW,EACpB,WAAW,SAAe,GACzB,gBAAgB,CAmDlB"}

package/dist/backends/sandbox.js

@@ -42,9 +42,17 @@ export function createSandboxBackend(sandbox, projectRoot = '/workspace') {
             // printed still comes back, with exit code 124. Wrapping only the inner
             // command would run it in a child shell that never sees unexported variables
             // the consumer's sourcing set.
+            // Its output goes to files that are printed once the shell exits, never to
+            // the exec's own pipe: `cd app && server > log &` backgrounds a SUBSHELL whose
+            // stdout is still that pipe (the redirect covers the server only), and the
+            // exec waits for every holder of the pipe — past the budget, which kills the
+            // shell but not the detached subshell, until the caller's outer timeout
+            // throws the output away. Measured: three 300 s calls in two runs (X0 x31,
+            // x33). A background child now holds a file, and the call returns when the
+            // command does.
             const budgetSeconds = opts?.budgetMs ? Math.max(1, Math.round(opts.budgetMs / 1000)) : 0;
             const fullCommand = budgetSeconds
-                ? `timeout -k 5 ${budgetSeconds} bash -c ${shellQuote(anchored)}`
+                ? `o=$(mktemp); e=$(mktemp); timeout -k 5 ${budgetSeconds} bash -c ${shellQuote(anchored)} > "$o" 2> "$e" < /dev/null; c=$?; cat "$o"; cat "$e" >&2; rm -f "$o" "$e"; exit $c`
                 : anchored;
             // sandbox.exec is the Sandbox interface method — runs inside Docker, inherently sandboxed
             const result = await sandbox.exec(fullCommand, { timeout: opts?.timeout });

package/dist/tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tools.d.ts","sourceRoot":"","sources":["../src/tools.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAA;AAY9C,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AA+BnE;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CA0tCxF"}
+{"version":3,"file":"tools.d.ts","sourceRoot":"","sources":["../src/tools.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAA;AAY9C,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AA+BnE;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CAmvCxF"}

package/dist/tools.js

@@ -527,6 +527,7 @@ export function buildTools(backend, config) {
                     content = verdict.content;
                 }
                 const oldContent = content;
+                let alreadyApplied = 0;
                 for (const { old_string: oldString, new_string: newString } of replacements) {
                     // Validate each replacement is well-formed BEFORE touching content. A
                     // weak model sometimes emits a batch element missing a field or with a
@@ -539,6 +540,19 @@ export function buildTools(backend, config) {
                         };
                     }
                     const count = content.split(oldString).length - 1;
+                    // The edit is ALREADY in the file: old_string is gone and a distinctive
+                    // new_string is there exactly once. The executor re-sends an edit it
+                    // made a few calls ago (it works from its memory of the file), and the
+                    // "not found" error cost it a read_file plus a retry each time — 6 to 11
+                    // times a build in X0 x30/x34. Report it as applied instead. Short or
+                    // repeated new_strings still error: they could be there by coincidence.
+                    if (count === 0 &&
+                        newString !== oldString &&
+                        newString.trim().length >= 20 &&
+                        content.split(newString).length - 1 === 1) {
+                        alreadyApplied++;
+                        continue;
+                    }
                     if (count === 0) {
                         // A bare "old_string not found" gives the model nothing to correct, so
                         // it retries blindly (the #1 edit_file failure mode). Diagnose WHY the
@@ -648,6 +662,12 @@ export function buildTools(backend, config) {
                     const at = content.indexOf(oldString);
                     content = content.slice(0, at) + newString + content.slice(at + oldString.length);
                 }
+                const alreadyNote = alreadyApplied > 0
+                    ? `${alreadyApplied} of these edits ${alreadyApplied === 1 ? 'was' : 'were'} already in the file (its new_string is there and its old_string is not) — nothing to do for ${alreadyApplied === 1 ? 'it' : 'them'}.`
+                    : null;
+                if (content === oldContent) {
+                    return { path, ok: true, replacementsApplied: 0, alreadyApplied, note: alreadyNote };
+                }
                 if (onFileDiff)
                     onFileDiff({ path, oldContent, newContent: content });
                 await backend.writeFile(path, content);
@@ -658,7 +678,8 @@ export function buildTools(backend, config) {
                 return {
                     path,
                     ok: true,
-                    replacementsApplied: replacements.length,
+                    replacementsApplied: replacements.length - alreadyApplied,
+                    ...(alreadyApplied > 0 ? { alreadyApplied, note: alreadyNote } : {}),
                     ...(await syntaxErrorAfterWrite(path)),
                 };
             }

package/package.json

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