@librechat/agents 3.2.31 → 3.2.33

package/dist/cjs/tools/local/syntaxCheck.cjs

@@ -1,182 +1,163 @@
-'use strict';
-
-var path = require('path');
-var LocalExecutionEngine = require('./LocalExecutionEngine.cjs');
-
+const require_LocalExecutionEngine = require("./LocalExecutionEngine.cjs");
+let path = require("path");
+//#region src/tools/local/syntaxCheck.ts
 /**
- * Per-file syntax check used by `edit_file` / `write_file` to surface
- * obvious errors immediately after the write — strictly cheaper than
- * full LSP integration and catches the bulk of "you broke the file"
- * regressions a vision-less agent loop would otherwise miss until
- * the next call.
- *
- * Each checker is a tiny shell-out (or in-process function) keyed on
- * file extension. Failures are returned as a single short message;
- * the wiring layer decides whether to append it to the tool result
- * advisorily (`auto`) or to throw and force the model to react
- * (`strict`).
- *
- * We deliberately do NOT cover TypeScript here because per-file `tsc`
- * is slow and per-file syntax (without type info) misses most TS
- * errors anyway. Use the project-level `compile_check` tool for that.
- */
-// Per-backend × per-env cache. Codex P2 #40 — keying by spawn
-// backend alone misses env-driven availability changes (e.g. PATH
-// loses node between Runs that share the same backend). Same fix
-// shape as the ripgrep cache (Codex P1 #34).
-let probeCacheByBackend = new WeakMap();
+* Per-file syntax check used by `edit_file` / `write_file` to surface
+* obvious errors immediately after the write — strictly cheaper than
+* full LSP integration and catches the bulk of "you broke the file"
+* regressions a vision-less agent loop would otherwise miss until
+* the next call.
+*
+* Each checker is a tiny shell-out (or in-process function) keyed on
+* file extension. Failures are returned as a single short message;
+* the wiring layer decides whether to append it to the tool result
+* advisorily (`auto`) or to throw and force the model to react
+* (`strict`).
+*
+* We deliberately do NOT cover TypeScript here because per-file `tsc`
+* is slow and per-file syntax (without type info) misses most TS
+* errors anyway. Use the project-level `compile_check` tool for that.
+*/
+let probeCacheByBackend = /* @__PURE__ */ new WeakMap();
 function envCacheKey(env) {
-    if (env == null)
-        return '';
-    const sorted = {};
-    for (const k of Object.keys(env).sort()) {
-        sorted[k] = env[k];
-    }
-    return JSON.stringify(sorted);
+	if (env == null) return "";
+	const sorted = {};
+	for (const k of Object.keys(env).sort()) sorted[k] = env[k];
+	return JSON.stringify(sorted);
 }
 function cacheFor(config) {
-    const backend = LocalExecutionEngine.getSpawn(config);
-    let envMap = probeCacheByBackend.get(backend);
-    if (envMap == null) {
-        envMap = new Map();
-        probeCacheByBackend.set(backend, envMap);
-    }
-    const envKey = envCacheKey(config.env);
-    let entry = envMap.get(envKey);
-    if (entry == null) {
-        entry = {};
-        envMap.set(envKey, entry);
-    }
-    return entry;
+	const backend = require_LocalExecutionEngine.getSpawn(config);
+	let envMap = probeCacheByBackend.get(backend);
+	if (envMap == null) {
+		envMap = /* @__PURE__ */ new Map();
+		probeCacheByBackend.set(backend, envMap);
+	}
+	const envKey = envCacheKey(config.env);
+	let entry = envMap.get(envKey);
+	if (entry == null) {
+		entry = {};
+		envMap.set(envKey, entry);
+	}
+	return entry;
 }
 async function probe(command, args, cached, config) {
-    const entry = cacheFor(config);
-    let probePromise = entry[cached];
-    if (probePromise == null) {
-        probePromise = LocalExecutionEngine.spawnLocalProcess(command, args, { ...config, timeoutMs: 5000, sandbox: { enabled: false } }, { internal: true })
-            .then((result) => result != null && result.exitCode === 0)
-            .catch(() => false);
-        entry[cached] = probePromise;
-    }
-    return probePromise;
+	const entry = cacheFor(config);
+	let probePromise = entry[cached];
+	if (probePromise == null) {
+		probePromise = require_LocalExecutionEngine.spawnLocalProcess(command, args, {
+			...config,
+			timeoutMs: 5e3,
+			sandbox: { enabled: false }
+		}, { internal: true }).then((result) => result != null && result.exitCode === 0).catch(() => false);
+		entry[cached] = probePromise;
+	}
+	return probePromise;
 }
 /**
- * Test-only reset hook. Clears the per-backend probe cache so tests
- * can swap in mocked spawn backends and reprobe deterministically.
- *
- * @internal Not part of the public SDK surface.
- */
+* Test-only reset hook. Clears the per-backend probe cache so tests
+* can swap in mocked spawn backends and reprobe deterministically.
+*
+* @internal Not part of the public SDK surface.
+*/
 function _resetSyntaxCheckProbeCacheForTests() {
-    probeCacheByBackend = new WeakMap();
+	probeCacheByBackend = /* @__PURE__ */ new WeakMap();
 }
-const jsCheck = async (path, config) => {
-    if (!(await probe('node', ['--version'], 'hasNode', config))) {
-        return { ok: true };
-    }
-    const result = await LocalExecutionEngine.spawnLocalProcess('node', ['--check', path], { ...config, timeoutMs: 5000, sandbox: { enabled: false } }, { internal: true });
-    if (result.exitCode === 0)
-        return { ok: true };
-    return {
-        ok: false,
-        checker: 'node --check',
-        output: result.stderr.trim() || result.stdout.trim() || 'syntax error',
-    };
+const jsCheck = async (path$1, config) => {
+	if (!await probe("node", ["--version"], "hasNode", config)) return { ok: true };
+	const result = await require_LocalExecutionEngine.spawnLocalProcess("node", ["--check", path$1], {
+		...config,
+		timeoutMs: 5e3,
+		sandbox: { enabled: false }
+	}, { internal: true });
+	if (result.exitCode === 0) return { ok: true };
+	return {
+		ok: false,
+		checker: "node --check",
+		output: result.stderr.trim() || result.stdout.trim() || "syntax error"
+	};
 };
-const pythonCheck = async (path, config) => {
-    if (!(await probe('python3', ['--version'], 'hasPython', config))) {
-        return { ok: true };
-    }
-    const program = 'import py_compile, sys\n' +
-        'try:\n' +
-        '  py_compile.compile(sys.argv[1], doraise=True)\n' +
-        'except py_compile.PyCompileError as e:\n' +
-        '  print(e.msg.strip(), file=sys.stderr)\n' +
-        '  sys.exit(1)\n';
-    const result = await LocalExecutionEngine.spawnLocalProcess('python3', ['-c', program, path], { ...config, timeoutMs: 5000, sandbox: { enabled: false } }, { internal: true });
-    if (result.exitCode === 0)
-        return { ok: true };
-    return {
-        ok: false,
-        checker: 'py_compile',
-        output: result.stderr.trim() || result.stdout.trim() || 'syntax error',
-    };
+const pythonCheck = async (path$2, config) => {
+	if (!await probe("python3", ["--version"], "hasPython", config)) return { ok: true };
+	const result = await require_LocalExecutionEngine.spawnLocalProcess("python3", [
+		"-c",
+		"import py_compile, sys\ntry:\n  py_compile.compile(sys.argv[1], doraise=True)\nexcept py_compile.PyCompileError as e:\n  print(e.msg.strip(), file=sys.stderr)\n  sys.exit(1)\n",
+		path$2
+	], {
+		...config,
+		timeoutMs: 5e3,
+		sandbox: { enabled: false }
+	}, { internal: true });
+	if (result.exitCode === 0) return { ok: true };
+	return {
+		ok: false,
+		checker: "py_compile",
+		output: result.stderr.trim() || result.stdout.trim() || "syntax error"
+	};
 };
-const jsonCheck = async (path, config) => {
-    // Route through the configured WorkspaceFS so a Run with a custom
-    // `local.exec.fs` (in-memory or remote engine) validates the same
-    // file the write_file/edit_file path actually wrote — pre-fix this
-    // read went to the host fs and either silently passed (no host
-    // file → catch returns undefined → ok: true) or read a different
-    // file with the same absolute path. Codex P1 #24.
-    const fs = LocalExecutionEngine.getWorkspaceFS(config);
-    const raw = await fs.readFile(path, 'utf8').catch(() => undefined);
-    if (raw == null)
-        return { ok: true };
-    try {
-        JSON.parse(raw);
-        return { ok: true };
-    }
-    catch (err) {
-        return {
-            ok: false,
-            checker: 'JSON.parse',
-            output: err.message,
-        };
-    }
+const jsonCheck = async (path$3, config) => {
+	const raw = await require_LocalExecutionEngine.getWorkspaceFS(config).readFile(path$3, "utf8").catch(() => void 0);
+	if (raw == null) return { ok: true };
+	try {
+		JSON.parse(raw);
+		return { ok: true };
+	} catch (err) {
+		return {
+			ok: false,
+			checker: "JSON.parse",
+			output: err.message
+		};
+	}
 };
-const bashCheck = async (path, config) => {
-    if (!(await probe('bash', ['--version'], 'hasBash', config))) {
-        return { ok: true };
-    }
-    const result = await LocalExecutionEngine.spawnLocalProcess('bash', ['-n', path], { ...config, timeoutMs: 5000, sandbox: { enabled: false } }, { internal: true });
-    if (result.exitCode === 0)
-        return { ok: true };
-    return {
-        ok: false,
-        checker: 'bash -n',
-        output: result.stderr.trim() || result.stdout.trim() || 'syntax error',
-    };
+const bashCheck = async (path$4, config) => {
+	if (!await probe("bash", ["--version"], "hasBash", config)) return { ok: true };
+	const result = await require_LocalExecutionEngine.spawnLocalProcess("bash", ["-n", path$4], {
+		...config,
+		timeoutMs: 5e3,
+		sandbox: { enabled: false }
+	}, { internal: true });
+	if (result.exitCode === 0) return { ok: true };
+	return {
+		ok: false,
+		checker: "bash -n",
+		output: result.stderr.trim() || result.stdout.trim() || "syntax error"
+	};
 };
 const CHECKERS_BY_EXT = {
-    '.js': jsCheck,
-    '.mjs': jsCheck,
-    '.cjs': jsCheck,
-    '.jsx': jsCheck,
-    '.py': pythonCheck,
-    '.pyw': pythonCheck,
-    '.json': jsonCheck,
-    '.sh': bashCheck,
-    '.bash': bashCheck,
+	".js": jsCheck,
+	".mjs": jsCheck,
+	".cjs": jsCheck,
+	".jsx": jsCheck,
+	".py": pythonCheck,
+	".pyw": pythonCheck,
+	".json": jsonCheck,
+	".sh": bashCheck,
+	".bash": bashCheck
 };
 /**
- * Run the post-edit syntax check for `absolutePath`. Returns
- * `null` when no checker matches the extension (most files), or a
- * `SyntaxCheckOutcome`.
- *
- * Truncates `output` to `maxOutputChars` (default 4096) so a
- * 10MB-of-errors transpiler dump can't blow the model context.
- */
+* Run the post-edit syntax check for `absolutePath`. Returns
+* `null` when no checker matches the extension (most files), or a
+* `SyntaxCheckOutcome`.
+*
+* Truncates `output` to `maxOutputChars` (default 4096) so a
+* 10MB-of-errors transpiler dump can't blow the model context.
+*/
 async function runPostEditSyntaxCheck(absolutePath, config) {
-    const ext = path.extname(absolutePath).toLowerCase();
-    const checker = CHECKERS_BY_EXT[ext];
-    if (checker == null)
-        return null;
-    try {
-        const result = await checker(absolutePath, config);
-        if (!result.ok) {
-            return {
-                ok: false,
-                checker: result.checker,
-                output: result.output.slice(0, 4096),
-            };
-        }
-        return result;
-    }
-    catch {
-        return null;
-    }
+	const checker = CHECKERS_BY_EXT[(0, path.extname)(absolutePath).toLowerCase()];
+	if (checker == null) return null;
+	try {
+		const result = await checker(absolutePath, config);
+		if (!result.ok) return {
+			ok: false,
+			checker: result.checker,
+			output: result.output.slice(0, 4096)
+		};
+		return result;
+	} catch {
+		return null;
+	}
 }
-
+//#endregion
 exports._resetSyntaxCheckProbeCacheForTests = _resetSyntaxCheckProbeCacheForTests;
 exports.runPostEditSyntaxCheck = runPostEditSyntaxCheck;
-//# sourceMappingURL=syntaxCheck.cjs.map
+
+//# sourceMappingURL=syntaxCheck.cjs.map

package/dist/cjs/tools/local/syntaxCheck.cjs.map

@@ -1 +1 @@
-{"version":3,"file":"syntaxCheck.cjs","sources":["../../../../src/tools/local/syntaxCheck.ts"],"sourcesContent":["/**\n * Per-file syntax check used by `edit_file` / `write_file` to surface\n * obvious errors immediately after the write — strictly cheaper than\n * full LSP integration and catches the bulk of \"you broke the file\"\n * regressions a vision-less agent loop would otherwise miss until\n * the next call.\n *\n * Each checker is a tiny shell-out (or in-process function) keyed on\n * file extension. Failures are returned as a single short message;\n * the wiring layer decides whether to append it to the tool result\n * advisorily (`auto`) or to throw and force the model to react\n * (`strict`).\n *\n * We deliberately do NOT cover TypeScript here because per-file `tsc`\n * is slow and per-file syntax (without type info) misses most TS\n * errors anyway. Use the project-level `compile_check` tool for that.\n */\n\nimport { extname } from 'path';\nimport type * as t from '@/types';\nimport {\n  getSpawn,\n  getWorkspaceFS,\n  spawnLocalProcess,\n} from './LocalExecutionEngine';\n\nexport type SyntaxCheckOutcome =\n  | { ok: true }\n  | { ok: false; checker: string; output: string };\n\nexport type SyntaxChecker = (\n  path: string,\n  config: t.LocalExecutionConfig\n) => Promise<SyntaxCheckOutcome>;\n\n/**\n * Per-backend availability cache for the post-edit syntax-check probe\n * tools (node, python3, bash). Keyed on the *effective spawn backend*\n * — see `getSpawn(config)` in LocalExecutionEngine — so a Run that\n * probes node over Node's child_process can't poison a subsequent Run\n * whose `local.exec.spawn` routes elsewhere (a remote sandbox might\n * have python but not node, etc.).\n *\n * Mirrors the same fix that landed for the ripgrep cache in\n * `LocalCodingTools.ts` after the first round of Codex review.\n * WeakMap keying lets disposed backends GC their entry; the test\n * reset hook re-creates the map.\n */\ntype ProbeKind = 'hasNode' | 'hasPython' | 'hasBash';\ntype ProbeCache = Partial<Record<ProbeKind, Promise<boolean>>>;\n\n// Per-backend × per-env cache. Codex P2 #40 — keying by spawn\n// backend alone misses env-driven availability changes (e.g. PATH\n// loses node between Runs that share the same backend). Same fix\n// shape as the ripgrep cache (Codex P1 #34).\nlet probeCacheByBackend = new WeakMap<\n  t.LocalSpawn,\n  Map<string, ProbeCache>\n>();\n\nfunction envCacheKey(env: NodeJS.ProcessEnv | undefined): string {\n  if (env == null) return '';\n  const sorted: Record<string, string | undefined> = {};\n  for (const k of Object.keys(env).sort()) {\n    sorted[k] = env[k];\n  }\n  return JSON.stringify(sorted);\n}\n\nfunction cacheFor(\n  config: t.LocalExecutionConfig\n): ProbeCache {\n  const backend = getSpawn(config);\n  let envMap = probeCacheByBackend.get(backend);\n  if (envMap == null) {\n    envMap = new Map();\n    probeCacheByBackend.set(backend, envMap);\n  }\n  const envKey = envCacheKey(config.env);\n  let entry = envMap.get(envKey);\n  if (entry == null) {\n    entry = {};\n    envMap.set(envKey, entry);\n  }\n  return entry;\n}\n\nasync function probe(\n  command: string,\n  args: string[],\n  cached: ProbeKind,\n  config: t.LocalExecutionConfig\n): Promise<boolean> {\n  const entry = cacheFor(config);\n  let probePromise = entry[cached];\n  if (probePromise == null) {\n    probePromise = spawnLocalProcess(\n      command,\n      args,\n      { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n      { internal: true }\n    )\n      .then((result) => result != null && result.exitCode === 0)\n      .catch(() => false);\n    entry[cached] = probePromise;\n  }\n  return probePromise;\n}\n\n/**\n * Test-only reset hook. Clears the per-backend probe cache so tests\n * can swap in mocked spawn backends and reprobe deterministically.\n *\n * @internal Not part of the public SDK surface.\n */\nexport function _resetSyntaxCheckProbeCacheForTests(): void {\n  probeCacheByBackend = new WeakMap();\n}\n\nconst jsCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('node', ['--version'], 'hasNode', config))) {\n    return { ok: true };\n  }\n  const result = await spawnLocalProcess(\n    'node',\n    ['--check', path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'node --check',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst pythonCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('python3', ['--version'], 'hasPython', config))) {\n    return { ok: true };\n  }\n  const program =\n    'import py_compile, sys\\n' +\n    'try:\\n' +\n    '  py_compile.compile(sys.argv[1], doraise=True)\\n' +\n    'except py_compile.PyCompileError as e:\\n' +\n    '  print(e.msg.strip(), file=sys.stderr)\\n' +\n    '  sys.exit(1)\\n';\n  const result = await spawnLocalProcess(\n    'python3',\n    ['-c', program, path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'py_compile',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst jsonCheck: SyntaxChecker = async (path, config) => {\n  // Route through the configured WorkspaceFS so a Run with a custom\n  // `local.exec.fs` (in-memory or remote engine) validates the same\n  // file the write_file/edit_file path actually wrote — pre-fix this\n  // read went to the host fs and either silently passed (no host\n  // file → catch returns undefined → ok: true) or read a different\n  // file with the same absolute path. Codex P1 #24.\n  const fs = getWorkspaceFS(config);\n  const raw = await fs.readFile(path, 'utf8').catch(() => undefined);\n  if (raw == null) return { ok: true };\n  try {\n    JSON.parse(raw);\n    return { ok: true };\n  } catch (err) {\n    return {\n      ok: false,\n      checker: 'JSON.parse',\n      output: (err as Error).message,\n    };\n  }\n};\n\nconst bashCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('bash', ['--version'], 'hasBash', config))) {\n    return { ok: true };\n  }\n  const result = await spawnLocalProcess(\n    'bash',\n    ['-n', path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'bash -n',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst CHECKERS_BY_EXT: Record<string, SyntaxChecker> = {\n  '.js': jsCheck,\n  '.mjs': jsCheck,\n  '.cjs': jsCheck,\n  '.jsx': jsCheck,\n  '.py': pythonCheck,\n  '.pyw': pythonCheck,\n  '.json': jsonCheck,\n  '.sh': bashCheck,\n  '.bash': bashCheck,\n};\n\n/**\n * Run the post-edit syntax check for `absolutePath`. Returns\n * `null` when no checker matches the extension (most files), or a\n * `SyntaxCheckOutcome`.\n *\n * Truncates `output` to `maxOutputChars` (default 4096) so a\n * 10MB-of-errors transpiler dump can't blow the model context.\n */\nexport async function runPostEditSyntaxCheck(\n  absolutePath: string,\n  config: t.LocalExecutionConfig\n): Promise<SyntaxCheckOutcome | null> {\n  const ext = extname(absolutePath).toLowerCase();\n  const checker = (CHECKERS_BY_EXT as Record<string, SyntaxChecker | undefined>)[ext];\n  if (checker == null) return null;\n  try {\n    const result = await checker(absolutePath, config);\n    if (!result.ok) {\n      return {\n        ok: false,\n        checker: result.checker,\n        output: result.output.slice(0, 4096),\n      };\n    }\n    return result;\n  } catch {\n    return null;\n  }\n}\n"],"names":["getSpawn","spawnLocalProcess","getWorkspaceFS","extname"],"mappings":";;;;;AAAA;;;;;;;;;;;;;;;;AAgBG;AAmCH;AACA;AACA;AACA;AACA,IAAI,mBAAmB,GAAG,IAAI,OAAO,EAGlC;AAEH,SAAS,WAAW,CAAC,GAAkC,EAAA;IACrD,IAAI,GAAG,IAAI,IAAI;AAAE,QAAA,OAAO,EAAE;IAC1B,MAAM,MAAM,GAAuC,EAAE;AACrD,IAAA,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QACvC,MAAM,CAAC,CAAC,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IACpB;AACA,IAAA,OAAO,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC;AAC/B;AAEA,SAAS,QAAQ,CACf,MAA8B,EAAA;AAE9B,IAAA,MAAM,OAAO,GAAGA,6BAAQ,CAAC,MAAM,CAAC;IAChC,IAAI,MAAM,GAAG,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC;AAC7C,IAAA,IAAI,MAAM,IAAI,IAAI,EAAE;AAClB,QAAA,MAAM,GAAG,IAAI,GAAG,EAAE;AAClB,QAAA,mBAAmB,CAAC,GAAG,CAAC,OAAO,EAAE,MAAM,CAAC;IAC1C;IACA,MAAM,MAAM,GAAG,WAAW,CAAC,MAAM,CAAC,GAAG,CAAC;IACtC,IAAI,KAAK,GAAG,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;AAC9B,IAAA,IAAI,KAAK,IAAI,IAAI,EAAE;QACjB,KAAK,GAAG,EAAE;AACV,QAAA,MAAM,CAAC,GAAG,CAAC,MAAM,EAAE,KAAK,CAAC;IAC3B;AACA,IAAA,OAAO,KAAK;AACd;AAEA,eAAe,KAAK,CAClB,OAAe,EACf,IAAc,EACd,MAAiB,EACjB,MAA8B,EAAA;AAE9B,IAAA,MAAM,KAAK,GAAG,QAAQ,CAAC,MAAM,CAAC;AAC9B,IAAA,IAAI,YAAY,GAAG,KAAK,CAAC,MAAM,CAAC;AAChC,IAAA,IAAI,YAAY,IAAI,IAAI,EAAE;AACxB,QAAA,YAAY,GAAGC,sCAAiB,CAC9B,OAAO,EACP,IAAI,EACJ,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAC3D,EAAE,QAAQ,EAAE,IAAI,EAAE;AAEjB,aAAA,IAAI,CAAC,CAAC,MAAM,KAAK,MAAM,IAAI,IAAI,IAAI,MAAM,CAAC,QAAQ,KAAK,CAAC;AACxD,aAAA,KAAK,CAAC,MAAM,KAAK,CAAC;AACrB,QAAA,KAAK,CAAC,MAAM,CAAC,GAAG,YAAY;IAC9B;AACA,IAAA,OAAO,YAAY;AACrB;AAEA;;;;;AAKG;SACa,mCAAmC,GAAA;AACjD,IAAA,mBAAmB,GAAG,IAAI,OAAO,EAAE;AACrC;AAEA,MAAM,OAAO,GAAkB,OAAO,IAAI,EAAE,MAAM,KAAI;AACpD,IAAA,IAAI,EAAE,MAAM,KAAK,CAAC,MAAM,EAAE,CAAC,WAAW,CAAC,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,EAAE;AAC5D,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IACrB;AACA,IAAA,MAAM,MAAM,GAAG,MAAMA,sCAAiB,CACpC,MAAM,EACN,CAAC,SAAS,EAAE,IAAI,CAAC,EACjB,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAC3D,EAAE,QAAQ,EAAE,IAAI,EAAE,CACnB;AACD,IAAA,IAAI,MAAM,CAAC,QAAQ,KAAK,CAAC;AAAE,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IAC9C,OAAO;AACL,QAAA,EAAE,EAAE,KAAK;AACT,QAAA,OAAO,EAAE,cAAc;AACvB,QAAA,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,cAAc;KACvE;AACH,CAAC;AAED,MAAM,WAAW,GAAkB,OAAO,IAAI,EAAE,MAAM,KAAI;AACxD,IAAA,IAAI,EAAE,MAAM,KAAK,CAAC,SAAS,EAAE,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,MAAM,CAAC,CAAC,EAAE;AACjE,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IACrB;IACA,MAAM,OAAO,GACX,0BAA0B;QAC1B,QAAQ;QACR,mDAAmD;QACnD,0CAA0C;QAC1C,2CAA2C;AAC3C,QAAA,iBAAiB;AACnB,IAAA,MAAM,MAAM,GAAG,MAAMA,sCAAiB,CACpC,SAAS,EACT,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,CAAC,EACrB,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAC3D,EAAE,QAAQ,EAAE,IAAI,EAAE,CACnB;AACD,IAAA,IAAI,MAAM,CAAC,QAAQ,KAAK,CAAC;AAAE,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IAC9C,OAAO;AACL,QAAA,EAAE,EAAE,KAAK;AACT,QAAA,OAAO,EAAE,YAAY;AACrB,QAAA,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,cAAc;KACvE;AACH,CAAC;AAED,MAAM,SAAS,GAAkB,OAAO,IAAI,EAAE,MAAM,KAAI;;;;;;;AAOtD,IAAA,MAAM,EAAE,GAAGC,mCAAc,CAAC,MAAM,CAAC;AACjC,IAAA,MAAM,GAAG,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,KAAK,CAAC,MAAM,SAAS,CAAC;IAClE,IAAI,GAAG,IAAI,IAAI;AAAE,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;AACpC,IAAA,IAAI;AACF,QAAA,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC;AACf,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IACrB;IAAE,OAAO,GAAG,EAAE;QACZ,OAAO;AACL,YAAA,EAAE,EAAE,KAAK;AACT,YAAA,OAAO,EAAE,YAAY;YACrB,MAAM,EAAG,GAAa,CAAC,OAAO;SAC/B;IACH;AACF,CAAC;AAED,MAAM,SAAS,GAAkB,OAAO,IAAI,EAAE,MAAM,KAAI;AACtD,IAAA,IAAI,EAAE,MAAM,KAAK,CAAC,MAAM,EAAE,CAAC,WAAW,CAAC,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,EAAE;AAC5D,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IACrB;AACA,IAAA,MAAM,MAAM,GAAG,MAAMD,sCAAiB,CACpC,MAAM,EACN,CAAC,IAAI,EAAE,IAAI,CAAC,EACZ,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAC3D,EAAE,QAAQ,EAAE,IAAI,EAAE,CACnB;AACD,IAAA,IAAI,MAAM,CAAC,QAAQ,KAAK,CAAC;AAAE,QAAA,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE;IAC9C,OAAO;AACL,QAAA,EAAE,EAAE,KAAK;AACT,QAAA,OAAO,EAAE,SAAS;AAClB,QAAA,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,cAAc;KACvE;AACH,CAAC;AAED,MAAM,eAAe,GAAkC;AACrD,IAAA,KAAK,EAAE,OAAO;AACd,IAAA,MAAM,EAAE,OAAO;AACf,IAAA,MAAM,EAAE,OAAO;AACf,IAAA,MAAM,EAAE,OAAO;AACf,IAAA,KAAK,EAAE,WAAW;AAClB,IAAA,MAAM,EAAE,WAAW;AACnB,IAAA,OAAO,EAAE,SAAS;AAClB,IAAA,KAAK,EAAE,SAAS;AAChB,IAAA,OAAO,EAAE,SAAS;CACnB;AAED;;;;;;;AAOG;AACI,eAAe,sBAAsB,CAC1C,YAAoB,EACpB,MAA8B,EAAA;IAE9B,MAAM,GAAG,GAAGE,YAAO,CAAC,YAAY,CAAC,CAAC,WAAW,EAAE;AAC/C,IAAA,MAAM,OAAO,GAAI,eAA6D,CAAC,GAAG,CAAC;IACnF,IAAI,OAAO,IAAI,IAAI;AAAE,QAAA,OAAO,IAAI;AAChC,IAAA,IAAI;QACF,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,YAAY,EAAE,MAAM,CAAC;AAClD,QAAA,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE;YACd,OAAO;AACL,gBAAA,EAAE,EAAE,KAAK;gBACT,OAAO,EAAE,MAAM,CAAC,OAAO;gBACvB,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC;aACrC;QACH;AACA,QAAA,OAAO,MAAM;IACf;AAAE,IAAA,MAAM;AACN,QAAA,OAAO,IAAI;IACb;AACF;;;;;"}
+{"version":3,"file":"syntaxCheck.cjs","names":["getSpawn","spawnLocalProcess","path","getWorkspaceFS"],"sources":["../../../../src/tools/local/syntaxCheck.ts"],"sourcesContent":["/**\n * Per-file syntax check used by `edit_file` / `write_file` to surface\n * obvious errors immediately after the write — strictly cheaper than\n * full LSP integration and catches the bulk of \"you broke the file\"\n * regressions a vision-less agent loop would otherwise miss until\n * the next call.\n *\n * Each checker is a tiny shell-out (or in-process function) keyed on\n * file extension. Failures are returned as a single short message;\n * the wiring layer decides whether to append it to the tool result\n * advisorily (`auto`) or to throw and force the model to react\n * (`strict`).\n *\n * We deliberately do NOT cover TypeScript here because per-file `tsc`\n * is slow and per-file syntax (without type info) misses most TS\n * errors anyway. Use the project-level `compile_check` tool for that.\n */\n\nimport { extname } from 'path';\nimport type * as t from '@/types';\nimport {\n  getSpawn,\n  getWorkspaceFS,\n  spawnLocalProcess,\n} from './LocalExecutionEngine';\n\nexport type SyntaxCheckOutcome =\n  | { ok: true }\n  | { ok: false; checker: string; output: string };\n\nexport type SyntaxChecker = (\n  path: string,\n  config: t.LocalExecutionConfig\n) => Promise<SyntaxCheckOutcome>;\n\n/**\n * Per-backend availability cache for the post-edit syntax-check probe\n * tools (node, python3, bash). Keyed on the *effective spawn backend*\n * — see `getSpawn(config)` in LocalExecutionEngine — so a Run that\n * probes node over Node's child_process can't poison a subsequent Run\n * whose `local.exec.spawn` routes elsewhere (a remote sandbox might\n * have python but not node, etc.).\n *\n * Mirrors the same fix that landed for the ripgrep cache in\n * `LocalCodingTools.ts` after the first round of Codex review.\n * WeakMap keying lets disposed backends GC their entry; the test\n * reset hook re-creates the map.\n */\ntype ProbeKind = 'hasNode' | 'hasPython' | 'hasBash';\ntype ProbeCache = Partial<Record<ProbeKind, Promise<boolean>>>;\n\n// Per-backend × per-env cache. Codex P2 #40 — keying by spawn\n// backend alone misses env-driven availability changes (e.g. PATH\n// loses node between Runs that share the same backend). Same fix\n// shape as the ripgrep cache (Codex P1 #34).\nlet probeCacheByBackend = new WeakMap<\n  t.LocalSpawn,\n  Map<string, ProbeCache>\n>();\n\nfunction envCacheKey(env: NodeJS.ProcessEnv | undefined): string {\n  if (env == null) return '';\n  const sorted: Record<string, string | undefined> = {};\n  for (const k of Object.keys(env).sort()) {\n    sorted[k] = env[k];\n  }\n  return JSON.stringify(sorted);\n}\n\nfunction cacheFor(\n  config: t.LocalExecutionConfig\n): ProbeCache {\n  const backend = getSpawn(config);\n  let envMap = probeCacheByBackend.get(backend);\n  if (envMap == null) {\n    envMap = new Map();\n    probeCacheByBackend.set(backend, envMap);\n  }\n  const envKey = envCacheKey(config.env);\n  let entry = envMap.get(envKey);\n  if (entry == null) {\n    entry = {};\n    envMap.set(envKey, entry);\n  }\n  return entry;\n}\n\nasync function probe(\n  command: string,\n  args: string[],\n  cached: ProbeKind,\n  config: t.LocalExecutionConfig\n): Promise<boolean> {\n  const entry = cacheFor(config);\n  let probePromise = entry[cached];\n  if (probePromise == null) {\n    probePromise = spawnLocalProcess(\n      command,\n      args,\n      { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n      { internal: true }\n    )\n      .then((result) => result != null && result.exitCode === 0)\n      .catch(() => false);\n    entry[cached] = probePromise;\n  }\n  return probePromise;\n}\n\n/**\n * Test-only reset hook. Clears the per-backend probe cache so tests\n * can swap in mocked spawn backends and reprobe deterministically.\n *\n * @internal Not part of the public SDK surface.\n */\nexport function _resetSyntaxCheckProbeCacheForTests(): void {\n  probeCacheByBackend = new WeakMap();\n}\n\nconst jsCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('node', ['--version'], 'hasNode', config))) {\n    return { ok: true };\n  }\n  const result = await spawnLocalProcess(\n    'node',\n    ['--check', path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'node --check',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst pythonCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('python3', ['--version'], 'hasPython', config))) {\n    return { ok: true };\n  }\n  const program =\n    'import py_compile, sys\\n' +\n    'try:\\n' +\n    '  py_compile.compile(sys.argv[1], doraise=True)\\n' +\n    'except py_compile.PyCompileError as e:\\n' +\n    '  print(e.msg.strip(), file=sys.stderr)\\n' +\n    '  sys.exit(1)\\n';\n  const result = await spawnLocalProcess(\n    'python3',\n    ['-c', program, path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'py_compile',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst jsonCheck: SyntaxChecker = async (path, config) => {\n  // Route through the configured WorkspaceFS so a Run with a custom\n  // `local.exec.fs` (in-memory or remote engine) validates the same\n  // file the write_file/edit_file path actually wrote — pre-fix this\n  // read went to the host fs and either silently passed (no host\n  // file → catch returns undefined → ok: true) or read a different\n  // file with the same absolute path. Codex P1 #24.\n  const fs = getWorkspaceFS(config);\n  const raw = await fs.readFile(path, 'utf8').catch(() => undefined);\n  if (raw == null) return { ok: true };\n  try {\n    JSON.parse(raw);\n    return { ok: true };\n  } catch (err) {\n    return {\n      ok: false,\n      checker: 'JSON.parse',\n      output: (err as Error).message,\n    };\n  }\n};\n\nconst bashCheck: SyntaxChecker = async (path, config) => {\n  if (!(await probe('bash', ['--version'], 'hasBash', config))) {\n    return { ok: true };\n  }\n  const result = await spawnLocalProcess(\n    'bash',\n    ['-n', path],\n    { ...config, timeoutMs: 5000, sandbox: { enabled: false } },\n    { internal: true }\n  );\n  if (result.exitCode === 0) return { ok: true };\n  return {\n    ok: false,\n    checker: 'bash -n',\n    output: result.stderr.trim() || result.stdout.trim() || 'syntax error',\n  };\n};\n\nconst CHECKERS_BY_EXT: Record<string, SyntaxChecker> = {\n  '.js': jsCheck,\n  '.mjs': jsCheck,\n  '.cjs': jsCheck,\n  '.jsx': jsCheck,\n  '.py': pythonCheck,\n  '.pyw': pythonCheck,\n  '.json': jsonCheck,\n  '.sh': bashCheck,\n  '.bash': bashCheck,\n};\n\n/**\n * Run the post-edit syntax check for `absolutePath`. Returns\n * `null` when no checker matches the extension (most files), or a\n * `SyntaxCheckOutcome`.\n *\n * Truncates `output` to `maxOutputChars` (default 4096) so a\n * 10MB-of-errors transpiler dump can't blow the model context.\n */\nexport async function runPostEditSyntaxCheck(\n  absolutePath: string,\n  config: t.LocalExecutionConfig\n): Promise<SyntaxCheckOutcome | null> {\n  const ext = extname(absolutePath).toLowerCase();\n  const checker = (CHECKERS_BY_EXT as Record<string, SyntaxChecker | undefined>)[ext];\n  if (checker == null) return null;\n  try {\n    const result = await checker(absolutePath, config);\n    if (!result.ok) {\n      return {\n        ok: false,\n        checker: result.checker,\n        output: result.output.slice(0, 4096),\n      };\n    }\n    return result;\n  } catch {\n    return null;\n  }\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;AAuDA,IAAI,sCAAsB,IAAI,QAG5B;AAEF,SAAS,YAAY,KAA4C;CAC/D,IAAI,OAAO,MAAM,OAAO;CACxB,MAAM,SAA6C,CAAC;CACpD,KAAK,MAAM,KAAK,OAAO,KAAK,GAAG,CAAC,CAAC,KAAK,GACpC,OAAO,KAAK,IAAI;CAElB,OAAO,KAAK,UAAU,MAAM;AAC9B;AAEA,SAAS,SACP,QACY;CACZ,MAAM,UAAUA,6BAAAA,SAAS,MAAM;CAC/B,IAAI,SAAS,oBAAoB,IAAI,OAAO;CAC5C,IAAI,UAAU,MAAM;EAClB,yBAAS,IAAI,IAAI;EACjB,oBAAoB,IAAI,SAAS,MAAM;CACzC;CACA,MAAM,SAAS,YAAY,OAAO,GAAG;CACrC,IAAI,QAAQ,OAAO,IAAI,MAAM;CAC7B,IAAI,SAAS,MAAM;EACjB,QAAQ,CAAC;EACT,OAAO,IAAI,QAAQ,KAAK;CAC1B;CACA,OAAO;AACT;AAEA,eAAe,MACb,SACA,MACA,QACA,QACkB;CAClB,MAAM,QAAQ,SAAS,MAAM;CAC7B,IAAI,eAAe,MAAM;CACzB,IAAI,gBAAgB,MAAM;EACxB,eAAeC,6BAAAA,kBACb,SACA,MACA;GAAE,GAAG;GAAQ,WAAW;GAAM,SAAS,EAAE,SAAS,MAAM;EAAE,GAC1D,EAAE,UAAU,KAAK,CACnB,CAAC,CACE,MAAM,WAAW,UAAU,QAAQ,OAAO,aAAa,CAAC,CAAC,CACzD,YAAY,KAAK;EACpB,MAAM,UAAU;CAClB;CACA,OAAO;AACT;;;;;;;AAQA,SAAgB,sCAA4C;CAC1D,sCAAsB,IAAI,QAAQ;AACpC;AAEA,MAAM,UAAyB,OAAO,QAAM,WAAW;CACrD,IAAI,CAAE,MAAM,MAAM,QAAQ,CAAC,WAAW,GAAG,WAAW,MAAM,GACxD,OAAO,EAAE,IAAI,KAAK;CAEpB,MAAM,SAAS,MAAMA,6BAAAA,kBACnB,QACA,CAAC,WAAWC,MAAI,GAChB;EAAE,GAAG;EAAQ,WAAW;EAAM,SAAS,EAAE,SAAS,MAAM;CAAE,GAC1D,EAAE,UAAU,KAAK,CACnB;CACA,IAAI,OAAO,aAAa,GAAG,OAAO,EAAE,IAAI,KAAK;CAC7C,OAAO;EACL,IAAI;EACJ,SAAS;EACT,QAAQ,OAAO,OAAO,KAAK,KAAK,OAAO,OAAO,KAAK,KAAK;CAC1D;AACF;AAEA,MAAM,cAA6B,OAAO,QAAM,WAAW;CACzD,IAAI,CAAE,MAAM,MAAM,WAAW,CAAC,WAAW,GAAG,aAAa,MAAM,GAC7D,OAAO,EAAE,IAAI,KAAK;CASpB,MAAM,SAAS,MAAMD,6BAAAA,kBACnB,WACA;EAAC;EAAM;EAASC;CAAI,GACpB;EAAE,GAAG;EAAQ,WAAW;EAAM,SAAS,EAAE,SAAS,MAAM;CAAE,GAC1D,EAAE,UAAU,KAAK,CACnB;CACA,IAAI,OAAO,aAAa,GAAG,OAAO,EAAE,IAAI,KAAK;CAC7C,OAAO;EACL,IAAI;EACJ,SAAS;EACT,QAAQ,OAAO,OAAO,KAAK,KAAK,OAAO,OAAO,KAAK,KAAK;CAC1D;AACF;AAEA,MAAM,YAA2B,OAAO,QAAM,WAAW;CAQvD,MAAM,MAAM,MADDC,6BAAAA,eAAe,MACP,CAAC,CAAC,SAASD,QAAM,MAAM,CAAC,CAAC,YAAY,KAAA,CAAS;CACjE,IAAI,OAAO,MAAM,OAAO,EAAE,IAAI,KAAK;CACnC,IAAI;EACF,KAAK,MAAM,GAAG;EACd,OAAO,EAAE,IAAI,KAAK;CACpB,SAAS,KAAK;EACZ,OAAO;GACL,IAAI;GACJ,SAAS;GACT,QAAS,IAAc;EACzB;CACF;AACF;AAEA,MAAM,YAA2B,OAAO,QAAM,WAAW;CACvD,IAAI,CAAE,MAAM,MAAM,QAAQ,CAAC,WAAW,GAAG,WAAW,MAAM,GACxD,OAAO,EAAE,IAAI,KAAK;CAEpB,MAAM,SAAS,MAAMD,6BAAAA,kBACnB,QACA,CAAC,MAAMC,MAAI,GACX;EAAE,GAAG;EAAQ,WAAW;EAAM,SAAS,EAAE,SAAS,MAAM;CAAE,GAC1D,EAAE,UAAU,KAAK,CACnB;CACA,IAAI,OAAO,aAAa,GAAG,OAAO,EAAE,IAAI,KAAK;CAC7C,OAAO;EACL,IAAI;EACJ,SAAS;EACT,QAAQ,OAAO,OAAO,KAAK,KAAK,OAAO,OAAO,KAAK,KAAK;CAC1D;AACF;AAEA,MAAM,kBAAiD;CACrD,OAAO;CACP,QAAQ;CACR,QAAQ;CACR,QAAQ;CACR,OAAO;CACP,QAAQ;CACR,SAAS;CACT,OAAO;CACP,SAAS;AACX;;;;;;;;;AAUA,eAAsB,uBACpB,cACA,QACoC;CAEpC,MAAM,UAAW,iBAAA,GAAA,KAAA,QAAA,CADG,YAAY,CAAC,CAAC,YAC+C;CACjF,IAAI,WAAW,MAAM,OAAO;CAC5B,IAAI;EACF,MAAM,SAAS,MAAM,QAAQ,cAAc,MAAM;EACjD,IAAI,CAAC,OAAO,IACV,OAAO;GACL,IAAI;GACJ,SAAS,OAAO;GAChB,QAAQ,OAAO,OAAO,MAAM,GAAG,IAAI;EACrC;EAEF,OAAO;CACT,QAAQ;EACN,OAAO;CACT;AACF"}

package/dist/cjs/tools/local/textEncoding.cjs

@@ -1,30 +1,32 @@
-'use strict';
-
+//#region src/tools/local/textEncoding.ts
 /**
- * BOM and line-ending preservation helpers for the local engine's
- * file-mutating tools. We never *introduce* a BOM or change line
- * endings — only preserve what was already on disk so a Windows-
- * checked-in source file stays CRLF and a UTF-8-with-BOM JSON file
- * keeps its BOM after an edit.
- *
- * Inspired by opencode's `Bom` helper. Trimmed to the cases that
- * actually matter for editing source code (UTF-8 BOM only;
- * UTF-16/UTF-32 are out of scope).
- */
-const UTF8_BOM = '';
+* BOM and line-ending preservation helpers for the local engine's
+* file-mutating tools. We never *introduce* a BOM or change line
+* endings — only preserve what was already on disk so a Windows-
+* checked-in source file stays CRLF and a UTF-8-with-BOM JSON file
+* keeps its BOM after an edit.
+*
+* Inspired by opencode's `Bom` helper. Trimmed to the cases that
+* actually matter for editing source code (UTF-8 BOM only;
+* UTF-16/UTF-32 are out of scope).
+*/
+const UTF8_BOM = "";
 function decodeFile(raw) {
-    const hasBom = raw.startsWith(UTF8_BOM);
-    const stripped = hasBom ? raw.slice(1) : raw;
-    const newline = stripped.includes('\r\n') ? '\r\n' : '\n';
-    // Internally we always work in LF; encode() restores CRLF on write.
-    const lf = newline === '\r\n' ? stripped.replace(/\r\n/g, '\n') : stripped;
-    return { text: lf, hasBom, newline };
+	const hasBom = raw.startsWith(UTF8_BOM);
+	const stripped = hasBom ? raw.slice(1) : raw;
+	const newline = stripped.includes("\r\n") ? "\r\n" : "\n";
+	return {
+		text: newline === "\r\n" ? stripped.replace(/\r\n/g, "\n") : stripped,
+		hasBom,
+		newline
+	};
 }
 function encodeFile(text, encoding) {
-    const out = encoding.newline === '\r\n' ? text.replace(/\n/g, '\r\n') : text;
-    return encoding.hasBom ? `${UTF8_BOM}${out}` : out;
+	const out = encoding.newline === "\r\n" ? text.replace(/\n/g, "\r\n") : text;
+	return encoding.hasBom ? `${UTF8_BOM}${out}` : out;
 }
-
+//#endregion
 exports.decodeFile = decodeFile;
 exports.encodeFile = encodeFile;
-//# sourceMappingURL=textEncoding.cjs.map
+
+//# sourceMappingURL=textEncoding.cjs.map

package/dist/cjs/tools/local/textEncoding.cjs.map

@@ -1 +1 @@
-{"version":3,"file":"textEncoding.cjs","sources":["../../../../src/tools/local/textEncoding.ts"],"sourcesContent":["/**\n * BOM and line-ending preservation helpers for the local engine's\n * file-mutating tools. We never *introduce* a BOM or change line\n * endings — only preserve what was already on disk so a Windows-\n * checked-in source file stays CRLF and a UTF-8-with-BOM JSON file\n * keeps its BOM after an edit.\n *\n * Inspired by opencode's `Bom` helper. Trimmed to the cases that\n * actually matter for editing source code (UTF-8 BOM only;\n * UTF-16/UTF-32 are out of scope).\n */\n\nconst UTF8_BOM = '';\n\nexport interface EncodedFile {\n  /** File contents with BOM stripped. */\n  text: string;\n  /** Whether the on-disk file started with a UTF-8 BOM. */\n  hasBom: boolean;\n  /** Detected newline style. CRLF wins if any CRLF is present. */\n  newline: '\\n' | '\\r\\n';\n}\n\nexport function decodeFile(raw: string): EncodedFile {\n  const hasBom = raw.startsWith(UTF8_BOM);\n  const stripped = hasBom ? raw.slice(1) : raw;\n  const newline = stripped.includes('\\r\\n') ? '\\r\\n' : '\\n';\n  // Internally we always work in LF; encode() restores CRLF on write.\n  const lf = newline === '\\r\\n' ? stripped.replace(/\\r\\n/g, '\\n') : stripped;\n  return { text: lf, hasBom, newline };\n}\n\nexport function encodeFile(text: string, encoding: EncodedFile): string {\n  const out =\n    encoding.newline === '\\r\\n' ? text.replace(/\\n/g, '\\r\\n') : text;\n  return encoding.hasBom ? `${UTF8_BOM}${out}` : out;\n}\n"],"names":[],"mappings":";;AAAA;;;;;;;;;;AAUG;AAEH,MAAM,QAAQ,GAAG,GAAG;AAWd,SAAU,UAAU,CAAC,GAAW,EAAA;IACpC,MAAM,MAAM,GAAG,GAAG,CAAC,UAAU,CAAC,QAAQ,CAAC;AACvC,IAAA,MAAM,QAAQ,GAAG,MAAM,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,GAAG;AAC5C,IAAA,MAAM,OAAO,GAAG,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC,GAAG,MAAM,GAAG,IAAI;;IAEzD,MAAM,EAAE,GAAG,OAAO,KAAK,MAAM,GAAG,QAAQ,CAAC,OAAO,CAAC,OAAO,EAAE,IAAI,CAAC,GAAG,QAAQ;IAC1E,OAAO,EAAE,IAAI,EAAE,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE;AACtC;AAEM,SAAU,UAAU,CAAC,IAAY,EAAE,QAAqB,EAAA;IAC5D,MAAM,GAAG,GACP,QAAQ,CAAC,OAAO,KAAK,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,MAAM,CAAC,GAAG,IAAI;AAClE,IAAA,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAA,EAAG,QAAQ,CAAA,EAAG,GAAG,CAAA,CAAE,GAAG,GAAG;AACpD;;;;;"}
+{"version":3,"file":"textEncoding.cjs","names":[],"sources":["../../../../src/tools/local/textEncoding.ts"],"sourcesContent":["/**\n * BOM and line-ending preservation helpers for the local engine's\n * file-mutating tools. We never *introduce* a BOM or change line\n * endings — only preserve what was already on disk so a Windows-\n * checked-in source file stays CRLF and a UTF-8-with-BOM JSON file\n * keeps its BOM after an edit.\n *\n * Inspired by opencode's `Bom` helper. Trimmed to the cases that\n * actually matter for editing source code (UTF-8 BOM only;\n * UTF-16/UTF-32 are out of scope).\n */\n\nconst UTF8_BOM = '';\n\nexport interface EncodedFile {\n  /** File contents with BOM stripped. */\n  text: string;\n  /** Whether the on-disk file started with a UTF-8 BOM. */\n  hasBom: boolean;\n  /** Detected newline style. CRLF wins if any CRLF is present. */\n  newline: '\\n' | '\\r\\n';\n}\n\nexport function decodeFile(raw: string): EncodedFile {\n  const hasBom = raw.startsWith(UTF8_BOM);\n  const stripped = hasBom ? raw.slice(1) : raw;\n  const newline = stripped.includes('\\r\\n') ? '\\r\\n' : '\\n';\n  // Internally we always work in LF; encode() restores CRLF on write.\n  const lf = newline === '\\r\\n' ? stripped.replace(/\\r\\n/g, '\\n') : stripped;\n  return { text: lf, hasBom, newline };\n}\n\nexport function encodeFile(text: string, encoding: EncodedFile): string {\n  const out =\n    encoding.newline === '\\r\\n' ? text.replace(/\\n/g, '\\r\\n') : text;\n  return encoding.hasBom ? `${UTF8_BOM}${out}` : out;\n}\n"],"mappings":";;;;;;;;;;;;AAYA,MAAM,WAAW;AAWjB,SAAgB,WAAW,KAA0B;CACnD,MAAM,SAAS,IAAI,WAAW,QAAQ;CACtC,MAAM,WAAW,SAAS,IAAI,MAAM,CAAC,IAAI;CACzC,MAAM,UAAU,SAAS,SAAS,MAAM,IAAI,SAAS;CAGrD,OAAO;EAAE,MADE,YAAY,SAAS,SAAS,QAAQ,SAAS,IAAI,IAAI;EAC/C;EAAQ;CAAQ;AACrC;AAEA,SAAgB,WAAW,MAAc,UAA+B;CACtE,MAAM,MACJ,SAAS,YAAY,SAAS,KAAK,QAAQ,OAAO,MAAM,IAAI;CAC9D,OAAO,SAAS,SAAS,GAAG,WAAW,QAAQ;AACjD"}

package/dist/cjs/tools/local/workspaceFS.cjs

@@ -1,51 +1,43 @@
-'use strict';
-
-var promises = require('fs/promises');
-
+let fs_promises = require("fs/promises");
+//#region src/tools/local/workspaceFS.ts
 /**
- * Engine-agnostic filesystem seam for the local-coding tool suite.
- *
- * The current "local" engine maps every operation to Node's
- * `fs/promises` against the host machine. A future engine — e.g. a
- * stateful remote sandbox (e2b / Modal / Daytona / ssh-jail) —
- * supplies its own `WorkspaceFS` implementation and reuses every
- * tool factory unchanged. Same fuzzy-match `edit_file`, same
- * checkpointer, same syntax-check, same image attachments — only
- * the underlying I/O changes.
- *
- * Path semantics belong to the implementation. The local engine
- * interprets paths as host filesystem paths; a remote engine would
- * interpret them as remote-namespace paths. Tool factories don't
- * inspect the strings beyond passing them through.
- *
- * Keep this surface minimal. Add a method only when an existing
- * tool genuinely needs it; resist the temptation to mirror all of
- * `fs/promises`.
- */
+* Engine-agnostic filesystem seam for the local-coding tool suite.
+*
+* The current "local" engine maps every operation to Node's
+* `fs/promises` against the host machine. A future engine — e.g. a
+* stateful remote sandbox (e2b / Modal / Daytona / ssh-jail) —
+* supplies its own `WorkspaceFS` implementation and reuses every
+* tool factory unchanged. Same fuzzy-match `edit_file`, same
+* checkpointer, same syntax-check, same image attachments — only
+* the underlying I/O changes.
+*
+* Path semantics belong to the implementation. The local engine
+* interprets paths as host filesystem paths; a remote engine would
+* interpret them as remote-namespace paths. Tool factories don't
+* inspect the strings beyond passing them through.
+*
+* Keep this surface minimal. Add a method only when an existing
+* tool genuinely needs it; resist the temptation to mirror all of
+* `fs/promises`.
+*/
 /**
- * Default `WorkspaceFS` backed by Node's `fs/promises` module.
- * Returned by `getWorkspaceFS(config)` when the host hasn't supplied
- * an override on `local.exec.fs`.
- */
+* Default `WorkspaceFS` backed by Node's `fs/promises` module.
+* Returned by `getWorkspaceFS(config)` when the host hasn't supplied
+* an override on `local.exec.fs`.
+*/
 const nodeWorkspaceFS = {
-    // The runtime impl ignores the encoding-vs-buffer distinction; the
-    // overload signatures above are what callers see.
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    readFile: ((path, encoding) => encoding != null
-        ? promises.readFile(path, encoding)
-        : promises.readFile(path)),
-    writeFile: (path, content, options) => promises.writeFile(path, content, options ?? 'utf8'),
-    stat: (path) => promises.stat(path),
-    readdir: ((path, options) => options?.withFileTypes === true
-        ? promises.readdir(path, { withFileTypes: true })
-        : promises.readdir(path)),
-    mkdir: async (path, options) => {
-        await promises.mkdir(path, options);
-    },
-    realpath: (path) => promises.realpath(path),
-    unlink: (path) => promises.unlink(path),
-    open: (path, flags) => promises.open(path, flags),
+	readFile: ((path, encoding) => encoding != null ? (0, fs_promises.readFile)(path, encoding) : (0, fs_promises.readFile)(path)),
+	writeFile: (path, content, options) => (0, fs_promises.writeFile)(path, content, options ?? "utf8"),
+	stat: (path) => (0, fs_promises.stat)(path),
+	readdir: ((path, options) => options?.withFileTypes === true ? (0, fs_promises.readdir)(path, { withFileTypes: true }) : (0, fs_promises.readdir)(path)),
+	mkdir: async (path, options) => {
+		await (0, fs_promises.mkdir)(path, options);
+	},
+	realpath: (path) => (0, fs_promises.realpath)(path),
+	unlink: (path) => (0, fs_promises.unlink)(path),
+	open: (path, flags) => (0, fs_promises.open)(path, flags)
 };
-
+//#endregion
 exports.nodeWorkspaceFS = nodeWorkspaceFS;
-//# sourceMappingURL=workspaceFS.cjs.map
+
+//# sourceMappingURL=workspaceFS.cjs.map

package/dist/cjs/tools/local/workspaceFS.cjs.map

@@ -1 +1 @@
-{"version":3,"file":"workspaceFS.cjs","sources":["../../../../src/tools/local/workspaceFS.ts"],"sourcesContent":["/**\n * Engine-agnostic filesystem seam for the local-coding tool suite.\n *\n * The current \"local\" engine maps every operation to Node's\n * `fs/promises` against the host machine. A future engine — e.g. a\n * stateful remote sandbox (e2b / Modal / Daytona / ssh-jail) —\n * supplies its own `WorkspaceFS` implementation and reuses every\n * tool factory unchanged. Same fuzzy-match `edit_file`, same\n * checkpointer, same syntax-check, same image attachments — only\n * the underlying I/O changes.\n *\n * Path semantics belong to the implementation. The local engine\n * interprets paths as host filesystem paths; a remote engine would\n * interpret them as remote-namespace paths. Tool factories don't\n * inspect the strings beyond passing them through.\n *\n * Keep this surface minimal. Add a method only when an existing\n * tool genuinely needs it; resist the temptation to mirror all of\n * `fs/promises`.\n */\n\nimport {\n  mkdir as fsMkdir,\n  open as fsOpen,\n  readdir as fsReaddir,\n  readFile as fsReadFile,\n  realpath as fsRealpath,\n  stat as fsStat,\n  unlink as fsUnlink,\n  writeFile as fsWriteFile,\n} from 'fs/promises';\nimport type { MakeDirectoryOptions, Stats, WriteFileOptions } from 'fs';\nimport type { FileHandle } from 'fs/promises';\n\nexport type ReaddirEntry = {\n  name: string;\n  isFile(): boolean;\n  isDirectory(): boolean;\n  isSymbolicLink(): boolean;\n};\n\nexport interface WorkspaceFS {\n  readFile(path: string, encoding: 'utf8'): Promise<string>;\n  readFile(path: string): Promise<Buffer>;\n  writeFile(\n    path: string,\n    content: string | Buffer,\n    options?: WriteFileOptions\n  ): Promise<void>;\n  stat(path: string): Promise<Stats>;\n  readdir(\n    path: string,\n    options: { withFileTypes: true }\n  ): Promise<ReaddirEntry[]>;\n  readdir(path: string): Promise<string[]>;\n  mkdir(path: string, options?: MakeDirectoryOptions): Promise<void>;\n  realpath(path: string): Promise<string>;\n  unlink(path: string): Promise<void>;\n  /** Open a file for low-level read access (used by binary detection). */\n  open(path: string, flags: 'r'): Promise<FileHandle>;\n}\n\n/**\n * Default `WorkspaceFS` backed by Node's `fs/promises` module.\n * Returned by `getWorkspaceFS(config)` when the host hasn't supplied\n * an override on `local.exec.fs`.\n */\nexport const nodeWorkspaceFS: WorkspaceFS = {\n  // The runtime impl ignores the encoding-vs-buffer distinction; the\n  // overload signatures above are what callers see.\n  // eslint-disable-next-line @typescript-eslint/no-explicit-any\n  readFile: ((path: string, encoding?: 'utf8') =>\n    encoding != null\n      ? fsReadFile(path, encoding)\n      : fsReadFile(path)) as WorkspaceFS['readFile'],\n  writeFile: (path, content, options) =>\n    fsWriteFile(path, content, options ?? 'utf8'),\n  stat: (path) => fsStat(path),\n  readdir: ((path: string, options?: { withFileTypes: true }) =>\n    options?.withFileTypes === true\n      ? fsReaddir(path, { withFileTypes: true })\n      : fsReaddir(path)) as WorkspaceFS['readdir'],\n  mkdir: async (path, options) => {\n    await fsMkdir(path, options);\n  },\n  realpath: (path) => fsRealpath(path),\n  unlink: (path) => fsUnlink(path),\n  open: (path, flags) => fsOpen(path, flags),\n};\n"],"names":["fsReadFile","fsWriteFile","fsStat","fsReaddir","fsMkdir","fsRealpath","fsUnlink","fsOpen"],"mappings":";;;;AAAA;;;;;;;;;;;;;;;;;;;AAmBG;AA2CH;;;;AAIG;AACI,MAAM,eAAe,GAAgB;;;;IAI1C,QAAQ,GAAG,CAAC,IAAY,EAAE,QAAiB,KACzC,QAAQ,IAAI;AACV,UAAEA,iBAAU,CAAC,IAAI,EAAE,QAAQ;AAC3B,UAAEA,iBAAU,CAAC,IAAI,CAAC,CAA4B;AAClD,IAAA,SAAS,EAAE,CAAC,IAAI,EAAE,OAAO,EAAE,OAAO,KAChCC,kBAAW,CAAC,IAAI,EAAE,OAAO,EAAE,OAAO,IAAI,MAAM,CAAC;IAC/C,IAAI,EAAE,CAAC,IAAI,KAAKC,aAAM,CAAC,IAAI,CAAC;AAC5B,IAAA,OAAO,GAAG,CAAC,IAAY,EAAE,OAAiC,KACxD,OAAO,EAAE,aAAa,KAAK;UACvBC,gBAAS,CAAC,IAAI,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE;AACzC,UAAEA,gBAAS,CAAC,IAAI,CAAC,CAA2B;AAChD,IAAA,KAAK,EAAE,OAAO,IAAI,EAAE,OAAO,KAAI;AAC7B,QAAA,MAAMC,cAAO,CAAC,IAAI,EAAE,OAAO,CAAC;IAC9B,CAAC;IACD,QAAQ,EAAE,CAAC,IAAI,KAAKC,iBAAU,CAAC,IAAI,CAAC;IACpC,MAAM,EAAE,CAAC,IAAI,KAAKC,eAAQ,CAAC,IAAI,CAAC;AAChC,IAAA,IAAI,EAAE,CAAC,IAAI,EAAE,KAAK,KAAKC,aAAM,CAAC,IAAI,EAAE,KAAK,CAAC;;;;;"}
+{"version":3,"file":"workspaceFS.cjs","names":[],"sources":["../../../../src/tools/local/workspaceFS.ts"],"sourcesContent":["/**\n * Engine-agnostic filesystem seam for the local-coding tool suite.\n *\n * The current \"local\" engine maps every operation to Node's\n * `fs/promises` against the host machine. A future engine — e.g. a\n * stateful remote sandbox (e2b / Modal / Daytona / ssh-jail) —\n * supplies its own `WorkspaceFS` implementation and reuses every\n * tool factory unchanged. Same fuzzy-match `edit_file`, same\n * checkpointer, same syntax-check, same image attachments — only\n * the underlying I/O changes.\n *\n * Path semantics belong to the implementation. The local engine\n * interprets paths as host filesystem paths; a remote engine would\n * interpret them as remote-namespace paths. Tool factories don't\n * inspect the strings beyond passing them through.\n *\n * Keep this surface minimal. Add a method only when an existing\n * tool genuinely needs it; resist the temptation to mirror all of\n * `fs/promises`.\n */\n\nimport {\n  mkdir as fsMkdir,\n  open as fsOpen,\n  readdir as fsReaddir,\n  readFile as fsReadFile,\n  realpath as fsRealpath,\n  stat as fsStat,\n  unlink as fsUnlink,\n  writeFile as fsWriteFile,\n} from 'fs/promises';\nimport type { MakeDirectoryOptions, Stats, WriteFileOptions } from 'fs';\nimport type { FileHandle } from 'fs/promises';\n\nexport type ReaddirEntry = {\n  name: string;\n  isFile(): boolean;\n  isDirectory(): boolean;\n  isSymbolicLink(): boolean;\n};\n\nexport interface WorkspaceFS {\n  readFile(path: string, encoding: 'utf8'): Promise<string>;\n  readFile(path: string): Promise<Buffer>;\n  writeFile(\n    path: string,\n    content: string | Buffer,\n    options?: WriteFileOptions\n  ): Promise<void>;\n  stat(path: string): Promise<Stats>;\n  readdir(\n    path: string,\n    options: { withFileTypes: true }\n  ): Promise<ReaddirEntry[]>;\n  readdir(path: string): Promise<string[]>;\n  mkdir(path: string, options?: MakeDirectoryOptions): Promise<void>;\n  realpath(path: string): Promise<string>;\n  unlink(path: string): Promise<void>;\n  /** Open a file for low-level read access (used by binary detection). */\n  open(path: string, flags: 'r'): Promise<FileHandle>;\n}\n\n/**\n * Default `WorkspaceFS` backed by Node's `fs/promises` module.\n * Returned by `getWorkspaceFS(config)` when the host hasn't supplied\n * an override on `local.exec.fs`.\n */\nexport const nodeWorkspaceFS: WorkspaceFS = {\n  // The runtime impl ignores the encoding-vs-buffer distinction; the\n  // overload signatures above are what callers see.\n  // eslint-disable-next-line @typescript-eslint/no-explicit-any\n  readFile: ((path: string, encoding?: 'utf8') =>\n    encoding != null\n      ? fsReadFile(path, encoding)\n      : fsReadFile(path)) as WorkspaceFS['readFile'],\n  writeFile: (path, content, options) =>\n    fsWriteFile(path, content, options ?? 'utf8'),\n  stat: (path) => fsStat(path),\n  readdir: ((path: string, options?: { withFileTypes: true }) =>\n    options?.withFileTypes === true\n      ? fsReaddir(path, { withFileTypes: true })\n      : fsReaddir(path)) as WorkspaceFS['readdir'],\n  mkdir: async (path, options) => {\n    await fsMkdir(path, options);\n  },\n  realpath: (path) => fsRealpath(path),\n  unlink: (path) => fsUnlink(path),\n  open: (path, flags) => fsOpen(path, flags),\n};\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;AAmEA,MAAa,kBAA+B;CAI1C,YAAY,MAAc,aACxB,YAAY,QAAA,GAAA,YAAA,SAAA,CACG,MAAM,QAAQ,KAAA,GAAA,YAAA,SAAA,CACd,IAAI;CACrB,YAAY,MAAM,SAAS,aAAA,GAAA,YAAA,UAAA,CACb,MAAM,SAAS,WAAW,MAAM;CAC9C,OAAO,UAAA,GAAA,YAAA,KAAA,CAAgB,IAAI;CAC3B,WAAW,MAAc,YACvB,SAAS,kBAAkB,QAAA,GAAA,YAAA,QAAA,CACb,MAAM,EAAE,eAAe,KAAK,CAAC,KAAA,GAAA,YAAA,QAAA,CAC7B,IAAI;CACpB,OAAO,OAAO,MAAM,YAAY;EAC9B,OAAA,GAAA,YAAA,MAAA,CAAc,MAAM,OAAO;CAC7B;CACA,WAAW,UAAA,GAAA,YAAA,SAAA,CAAoB,IAAI;CACnC,SAAS,UAAA,GAAA,YAAA,OAAA,CAAkB,IAAI;CAC/B,OAAO,MAAM,WAAA,GAAA,YAAA,KAAA,CAAiB,MAAM,KAAK;AAC3C"}