@molecule/api-ai-tools 1.5.6 → 1.5.7

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;AA2B9C,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AAuCnE;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CA+8CxF"}
+{"version":3,"file":"tools.d.ts","sourceRoot":"","sources":["../src/tools.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAA;AA2B9C,OAAO,KAAK,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAA;AAuCnE;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,gBAAgB,EAAE,MAAM,CAAC,EAAE,eAAe,GAAG,MAAM,EAAE,CAs+CxF"}

package/dist/tools.js

@@ -103,9 +103,15 @@ export function buildTools(backend, config) {
             .run(command, { cwd: path.slice(0, path.lastIndexOf('/')) || '/', timeout: 15_000 })
             .catch(() => null);
         const problem = result ? parseCheckProblem(result) : null;
+        // JSX in a `.ts` file parses as a type assertion and fails on the first
+        // attribute — `Expected ">" but found "className"` — which does not say the
+        // file just needs to be `.tsx` (X1 flash-m28: three writes and 2.5 minutes).
+        const jsxInTs = problem !== null && /\.ts$/.test(path) && /Expected ">" but found|JSX/i.test(problem)
+            ? '\nThis looks like JSX in a `.ts` file: JSX only parses in a `.tsx` file — rename it (and its imports keep the `.js` extension).'
+            : '';
         return problem
             ? {
-                syntaxError: `The file no longer parses after this change — fix it before anything else, every build and test fails until it does:\n${problem}`,
+                syntaxError: `The file no longer parses after this change — fix it before anything else, every build and test fails until it does:\n${problem}${jsxInTs}`,
             }
             : {};
     }
@@ -801,10 +807,23 @@ export function buildTools(backend, config) {
                     }
                 }
                 const excerpt = renderEditExcerpt(finalContent, spans);
+                // A replacement whose old text ended in a newline and whose new text does not
+                // joins the line after it onto its last line. Applied as written, but said:
+                // flash-m28 saw two lines merged only one round trip later.
+                const joined = replacements.filter((r) => typeof r.old_string === 'string' &&
+                    typeof r.new_string === 'string' &&
+                    r.old_string.endsWith('\n') &&
+                    r.new_string !== '' &&
+                    !r.new_string.endsWith('\n')).length;
                 return {
                     path,
                     ok: true,
                     replacementsApplied: replacements.length - alreadyApplied,
+                    ...(joined > 0
+                        ? {
+                            newlineNote: `${joined === 1 ? 'A replacement' : `${joined} replacements`} removed the line break after the replaced text (old_string ended with a newline, new_string does not), so the next line now continues on the same line. Check the excerpt; add the newline back if that was not intended.`,
+                        }
+                        : {}),
                     ...(alreadyApplied > 0 ? { alreadyApplied, note: alreadyNote } : {}),
                     totalLines: countLines(finalContent),
                     ...(excerpt ? { excerpt: sanitizeFileContent(excerpt, path) } : {}),

package/package.json

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