@a5c-ai/agent-core 5.0.1-staging.3a341c33c → 5.0.1-staging.40a93c240e7b

package/README.md

@@ -5,16 +5,16 @@ Built-in programmatic session wrapper and agentic tool surface for Babysitter ru
 <!-- docs-status:start -->
 > Status: Public advanced/runtime package.
 > Canonical docs home: [Package and Plugin Docs Map](../../docs/package-and-plugin-map.md).
-> This README defines the package contract for runtime consumers and published dependents such as `@a5c-ai/babysitter-agent`.
+> This README defines the package contract for runtime consumers and published dependents such as `@a5c-ai/agent-platform`.
 <!-- docs-status:end -->
 
 ## Package role
 
-`@a5c-ai/agent-core` sits between `@a5c-ai/babysitter-agent`, `@a5c-ai/agent-mux`, and `@a5c-ai/babysitter-sdk`:
+`@a5c-ai/agent-core` sits between `@a5c-ai/agent-platform`, `@a5c-ai/agent-mux`, and `@a5c-ai/babysitter-sdk`:
 
 - `createAgentCoreSession()` wraps an `@a5c-ai/agent-mux` client for in-process prompt execution.
 - `createAgentCoreToolDefinitions()` assembles the built-in Babysitter-flavored tool surface that host runtimes can inject into planning, resume, or delegated-task flows.
-- `@a5c-ai/babysitter-agent` re-exports these APIs from `src/harness/index.ts`, uses `createAgentCoreSession()` for direct `agent-core` harness invocation in `src/harness/invoker.ts`, and injects tool definitions into plan-process and resume-run flows.
+- `@a5c-ai/agent-platform` re-exports these APIs from `src/harness/index.ts`, uses `createAgentCoreSession()` for direct `agent-core` harness invocation in `src/harness/invoker.ts`, and injects tool definitions into plan-process and resume-run flows.
 - `@a5c-ai/babysitter-sdk` still owns run directories, journals, task/effect lifecycle, and config defaults. `agent-core` does not replace the SDK orchestration layer.
 
 This package is published as a runtime dependency surface for higher-level Babysitter runtimes. It is still an advanced/operator-facing building block rather than the primary entrypoint for new users.
@@ -66,6 +66,8 @@ Core handle methods:
 - `prompt(text, timeout?)`: starts a run, streams events to subscribers, and returns `{ output, duration, success, exitCode }`.
 - `steer(text)`: sends immediate steering text while a prompt is active, or queues it for the next prompt.
 - `followUp(text)`: queues a post-response follow-up when streaming, or appends it to the next prompt when idle.
+- `getHistory()`: returns a defensive copy of persisted user/assistant turns for the session handle.
+- `clearHistory()`: clears persisted user/assistant turns without changing listeners, options, or the current session id.
 - `subscribe(listener)`: receives normalized `AgentCoreSessionEvent` payloads, including `session_start`.
 - `abort()`: aborts the active run if one is in progress.
 - `dispose()`: aborts the active run, clears listeners, and drops queued follow-ups.
@@ -75,6 +77,8 @@ Core handle methods:
 Session behavior that matters to host integrations:
 
 - Agent-core reuses the `sessionId` learned from prior runs, so later prompts continue the same agent-mux session when the backend supports it.
+- Agent-core also keeps successful user/assistant turns on the session handle and includes bounded prior history in later direct provider prompts. System prompts are rebuilt from options for each request and are not stored in mutable history.
+- Failed prompts, timed-out requests, aborted requests, and malformed streams do not append partial assistant output to history.
 - Concurrent `prompt()` calls on the same handle are rejected.
 - Event payloads are normalized before subscribers see them. Non-object payloads become `{ type: "unknown", value }`.
 - Approval mode is `prompt` only when `uiContext` is present; otherwise agent-core uses `yolo`.
@@ -86,6 +90,8 @@ Session behavior that matters to host integrations:
 | `workspace` | Forwarded to agent-mux as `cwd`. |
 | `model` | Forwarded to agent-mux as `model`. |
 | `timeout` | Forwarded to agent-mux as `timeout`. |
+| `maxHistoryTurns` | Maximum persisted user/assistant history entries retained on the session handle. Defaults to 20. |
+| `maxHistoryTokens` | Maximum estimated tokens from prior history sent with a prompt. Uses the package's rough token estimator and drops oldest entries first. |
 | `thinkingLevel` | Translated to agent-mux `thinkingEffort` (`minimal`/`low` -> `low`, `medium` -> `medium`, `high` -> `high`, `xhigh` -> `max`). |
 | `systemPrompt` | Used as the base `systemPrompt`. |
 | `appendSystemPrompt` | Appended to the final `systemPrompt` before dispatch. |
@@ -98,7 +104,7 @@ These fields remain on `AgentCoreSessionOptions` for compatibility, but the curr
 
 | Option | Status | Migration note |
 | --- | --- | --- |
-| `toolsMode` | Deprecated, ignored by agent-core | Use backend-native configuration, or the PI wrapper in `@a5c-ai/babysitter-agent`, if you still need tool-surface control. |
+| `toolsMode` | Deprecated, ignored by agent-core | Use backend-native configuration, or the PI wrapper in `@a5c-ai/agent-platform`, if you still need tool-surface control. |
 | `customTools` | Deprecated, ignored by agent-core | Register host-side tools with `createAgentCoreToolDefinitions()` or use the PI wrapper for runtime custom-tool injection. |
 | `isolated` | Deprecated, ignored by agent-core | Use the PI wrapper if you still need extension/skills isolation controls. |
 | `ephemeral` | Deprecated, ignored by agent-core | Session persistence is determined by the selected agent-mux backend. |
@@ -106,7 +112,7 @@ These fields remain on `AgentCoreSessionOptions` for compatibility, but the curr
 | `enableCompaction` | Deprecated, ignored by agent-core | Compaction behavior belongs to the selected backend/runtime. |
 | `agentDir` | Deprecated, ignored by agent-core | Configure agent directories through the target backend instead. |
 
-If you still need the PI-era controls above, use the PI wrapper exposed from `@a5c-ai/babysitter-agent` rather than `@a5c-ai/agent-core`.
+If you still need the PI-era controls above, use the PI wrapper exposed from `@a5c-ai/agent-platform` rather than `@a5c-ai/agent-core`.
 
 ## Tool-definition API
 
@@ -192,7 +198,7 @@ Source disambiguation uses `(source, sourceQualifier, name)`, so duplicate tool
 
 Current downstream integration boundaries in this repo:
 
-- `@a5c-ai/babysitter-agent`
+- `@a5c-ai/agent-platform`
   - re-exports the session/tool APIs from `src/harness/index.ts`
   - uses `createAgentCoreSession()` for the direct `agent-core` harness path in `src/harness/invoker.ts`
   - injects `createAgentCoreToolDefinitions()` into plan-process and delegated-task flows in `src/harness/internal/createRun/planProcess/*`
@@ -202,7 +208,7 @@ Current downstream integration boundaries in this repo:
 - `@a5c-ai/babysitter-sdk`
   - provides config defaults/env wiring used by the `config` tool and remains the owner of orchestration/run-state semantics outside this package
 
-In practice, use `agent-core` when you need an in-process runtime wrapper or bundled tool surface. Use `babysitter-agent` when you need the higher-level harness CLI/runtime entrypoints.
+In practice, use `agent-core` when you need an in-process runtime wrapper or bundled tool surface. Use `agent-platform` when you need the higher-level harness CLI/runtime entrypoints.
 
 ## Build, test, and CI
 

package/dist/agenticTools/background/state.d.ts

@@ -1,5 +1,7 @@
-import { BackgroundProcessRegistry } from "../../backgroundProcessRegistry";
-import type { AgenticToolOptions } from "../types";
-export declare function getBackgroundRegistry(options: AgenticToolOptions): BackgroundProcessRegistry;
-export declare function disposeBackgroundRegistry(options: AgenticToolOptions): void;
+/**
+ * Backwards-compatibility shim -- the canonical implementation now lives in
+ * `@a5c-ai/agent-runtime`.  This re-export keeps internal agent-core consumers
+ * working without changes.
+ */
+export { getBackgroundRegistry, disposeBackgroundRegistry, } from "@a5c-ai/agent-runtime";
 //# sourceMappingURL=state.d.ts.map

package/dist/agenticTools/background/state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/background/state.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,yBAAyB,EAAE,MAAM,iCAAiC,CAAC;AAC5E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,UAAU,CAAC;AAInD,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,kBAAkB,GAAG,yBAAyB,CAW5F;AAED,wBAAgB,yBAAyB,CAAC,OAAO,EAAE,kBAAkB,GAAG,IAAI,CAO3E"}
+{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/background/state.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,OAAO,EACL,qBAAqB,EACrB,yBAAyB,GAC1B,MAAM,uBAAuB,CAAC"}

package/dist/agenticTools/background/state.js

@@ -1,25 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getBackgroundRegistry = getBackgroundRegistry;
-exports.disposeBackgroundRegistry = disposeBackgroundRegistry;
-const backgroundProcessRegistry_1 = require("../../backgroundProcessRegistry");
-const scopedRegistries = new WeakMap();
-function getBackgroundRegistry(options) {
-    if (options.backgroundRegistry) {
-        return options.backgroundRegistry;
-    }
-    let registry = scopedRegistries.get(options);
-    if (!registry) {
-        registry = new backgroundProcessRegistry_1.BackgroundProcessRegistry({ maxConcurrent: options.maxBackgroundProcesses });
-        scopedRegistries.set(options, registry);
-    }
-    return registry;
-}
-function disposeBackgroundRegistry(options) {
-    const registry = scopedRegistries.get(options);
-    if (!registry) {
-        return;
-    }
-    registry.dispose();
-    scopedRegistries.delete(options);
-}
+exports.disposeBackgroundRegistry = exports.getBackgroundRegistry = void 0;
+/**
+ * Backwards-compatibility shim -- the canonical implementation now lives in
+ * `@a5c-ai/agent-runtime`.  This re-export keeps internal agent-core consumers
+ * working without changes.
+ */
+var agent_runtime_1 = require("@a5c-ai/agent-runtime");
+Object.defineProperty(exports, "getBackgroundRegistry", { enumerable: true, get: function () { return agent_runtime_1.getBackgroundRegistry; } });
+Object.defineProperty(exports, "disposeBackgroundRegistry", { enumerable: true, get: function () { return agent_runtime_1.disposeBackgroundRegistry; } });

package/dist/agenticTools/browser/tool.js

@@ -25,8 +25,8 @@ function createBrowserTool() {
                 // eslint-disable-next-line @typescript-eslint/no-implied-eval, @typescript-eslint/no-unsafe-assignment
                 puppeteer = await new Function("id", "return import(id)")("puppeteer");
             }
-            catch {
-                return (0, results_1.errorResult)("puppeteer is not installed. Install it with: npm install puppeteer");
+            catch (error) {
+                return (0, results_1.errorResult)(`puppeteer import failed: ${error instanceof Error ? error.message : String(error)}. Install with: npm install puppeteer`);
             }
             if (!browserInstance) {
                 browserInstance = await puppeteer.launch({

package/dist/agenticTools/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/agenticTools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,MAAM,SAAS,CAAC;AAmBxE,wBAAgB,8BAA8B,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAsBlG;AAED,wBAAgB,+BAA+B,CAAC,WAAW,EAAE,oBAAoB,EAAE,GAAG,IAAI,CAOzF;AAED,eAAO,MAAM,4BAA4B,uCAAiC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/agenticTools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,MAAM,SAAS,CAAC;AAuBxE,wBAAgB,8BAA8B,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAsBlG;AAED,wBAAgB,+BAA+B,CAAC,WAAW,EAAE,oBAAoB,EAAE,GAAG,IAAI,CAOzF;AAED,eAAO,MAAM,4BAA4B,uCAAiC,CAAC"}

package/dist/agenticTools/index.js

@@ -15,6 +15,10 @@ const delegation_1 = require("./tools/delegation");
 const execution_1 = require("./tools/execution");
 const fileSystem_1 = require("./tools/fileSystem");
 const programmaticToolCalling_1 = require("./tools/programmaticToolCalling");
+// HERE BE DRAGONS: WeakMap keyed by exact array reference. Dispose MUST be called
+// with the same array object returned by createAgentCoreToolDefinitions().
+// Passing a copy, spread, or recreated array will silently fail to clean up
+// background processes, causing resource leaks.
 const toolDefinitionScopes = new WeakMap();
 function createAgentCoreToolDefinitions(options) {
     const baseTools = [

package/dist/agenticTools/shared/paths.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"paths.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/paths.ts"],"names":[],"mappings":"AAGA,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI,CAU7E;AAED,wBAAgB,WAAW,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAIvE;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CA2BnD;AAED,wBAAgB,OAAO,CACrB,SAAS,EAAE,MAAM,EACjB,aAAa,EAAE,OAAO,EACtB,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,MAAM,EAAE,GAChB,IAAI,CA2BN"}
+{"version":3,"file":"paths.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/paths.ts"],"names":[],"mappings":"AAGA,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI,CAU7E;AAED,wBAAgB,WAAW,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAIvE;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CA2BnD;AAED,wBAAgB,OAAO,CACrB,SAAS,EAAE,MAAM,EACjB,aAAa,EAAE,OAAO,EACtB,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,MAAM,EAAE,GAChB,IAAI,CA4BN"}

package/dist/agenticTools/shared/paths.js

@@ -93,7 +93,8 @@ function walkDir(directory, includeHidden, limit, results) {
     try {
         entries = fs.readdirSync(directory, { withFileTypes: true });
     }
-    catch {
+    catch (e) {
+        process.stderr.write(`[agent-core] walkDir: cannot read ${directory}: ${e instanceof Error ? e.message : String(e)}\n`);
         return;
     }
     for (const entry of entries) {

package/dist/agenticTools/shared/process.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"process.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/process.ts"],"names":[],"mappings":"AAGA,eAAO,MAAM,oBAAoB,SAAU,CAAC;AAC5C,eAAO,MAAM,sBAAsB,QAAS,CAAC;AAC7C,eAAO,MAAM,cAAc,QAAS,CAAC;AAKrC,wBAAgB,SAAS,IAAI,MAAM,CAgBlC;AAED,wBAAgB,UAAU,CACxB,OAAO,EAAE,MAAM,EACf,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE;IACP,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB,GACA,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAgE/D"}
+{"version":3,"file":"process.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/process.ts"],"names":[],"mappings":"AAGA,eAAO,MAAM,oBAAoB,SAAU,CAAC;AAC5C,eAAO,MAAM,sBAAsB,QAAS,CAAC;AAC7C,eAAO,MAAM,cAAc,QAAS,CAAC;AAKrC,wBAAgB,SAAS,IAAI,MAAM,CAgBlC;AAED,wBAAgB,UAAU,CACxB,OAAO,EAAE,MAAM,EACf,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE;IACP,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB,GACA,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAkE/D"}

package/dist/agenticTools/shared/process.js

@@ -83,6 +83,7 @@ function spawnAsync(command, args, options) {
             stdoutBytes += chunk.length;
             if (stdoutBytes > MAX_SPAWN_OUTPUT_BYTES) {
                 stdoutTruncated = true;
+                stdoutChunks.push(Buffer.from(`\n[babysitter] WARNING: stdout truncated at ${MAX_SPAWN_OUTPUT_BYTES} bytes. Subsequent output discarded.\n`));
                 return;
             }
             stdoutChunks.push(chunk);
@@ -94,6 +95,7 @@ function spawnAsync(command, args, options) {
             stderrBytes += chunk.length;
             if (stderrBytes > MAX_SPAWN_OUTPUT_BYTES) {
                 stderrTruncated = true;
+                stderrChunks.push(Buffer.from(`\n[babysitter] WARNING: stderr truncated at ${MAX_SPAWN_OUTPUT_BYTES} bytes. Subsequent output discarded.\n`));
                 return;
             }
             stderrChunks.push(chunk);
@@ -114,7 +116,7 @@ function spawnAsync(command, args, options) {
         processHandle.on("error", (error) => {
             resolve({
                 stdout: concatSafe(stdoutChunks, stdoutTruncated, stdoutBytes),
-                stderr: error.message,
+                stderr: `${error.message}${error.code ? ` (${error.code})` : ''}`,
                 exitCode: 1,
             });
         });

package/dist/agenticTools/shared/results.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"results.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/results.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAEjE,eAAO,MAAM,sBAAsB,kCAAkC,CAAC;AAEtE,wBAAgB,EAAE,CAAC,IAAI,EAAE,MAAM,GAAG,UAAU,CAE3C;AAED,wBAAgB,UAAU,CAAC,IAAI,EAAE,OAAO,GAAG,UAAU,CAEpD;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,UAAU,CAEvD;AAED,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAUhE;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,UAAU,CAM1E;AAED,wBAAgB,kBAAkB,CAChC,UAAU,EAAE,oBAAoB,EAChC,SAAS,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,KAAK,IAAI,GACtD,oBAAoB,CAkBtB"}
+{"version":3,"file":"results.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/shared/results.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAEjE,eAAO,MAAM,sBAAsB,kCAAkC,CAAC;AAEtE,wBAAgB,EAAE,CAAC,IAAI,EAAE,MAAM,GAAG,UAAU,CAE3C;AAED,wBAAgB,UAAU,CAAC,IAAI,EAAE,OAAO,GAAG,UAAU,CAEpD;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,UAAU,CAEvD;AAED,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAUhE;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,UAAU,CAM1E;AAED,wBAAgB,kBAAkB,CAChC,UAAU,EAAE,oBAAoB,EAChC,SAAS,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,KAAK,IAAI,GACtD,oBAAoB,CAmBtB"}

package/dist/agenticTools/shared/results.js

@@ -43,6 +43,7 @@ function wrapToolDefinition(definition, onToolUse) {
             return await Promise.resolve(originalExecute(toolCallId, params, onUpdate, toolContext));
         }
         catch (error) {
+            process.stderr.write(`[agent-core] tool ${definition.name} error: ${error instanceof Error ? error.stack ?? error.message : String(error)}\n`);
             return errorResultFor(error);
         }
     };

package/dist/agenticTools/tools/code.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"code.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/code.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAwErF,wBAAgB,eAAe,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAuOnF"}
+{"version":3,"file":"code.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/code.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAwErF,wBAAgB,eAAe,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAwOnF"}

package/dist/agenticTools/tools/code.js

@@ -122,6 +122,7 @@ function createCodeTools(options) {
                         continue;
                     }
                     if (astResult.exitCode !== 0) {
+                        process.stderr.write(`[agent-core] ast-grep failed (exit ${astResult.exitCode}), falling back to ripgrep\n`);
                         const grepArgs = ["--no-heading", "--line-number", "--color", "never"];
                         if (params.glob) {
                             grepArgs.push("--glob", String(params.glob));
@@ -218,21 +219,21 @@ function createCodeTools(options) {
                         return (0, results_1.ok)(fs.readFileSync(outputPath, "utf8"));
                     }
                 }
-                catch {
-                    // fall through to source return
+                catch (error) {
+                    process.stderr.write(`[agent-core] mmdc rendering failed: ${error instanceof Error ? error.message : String(error)}, returning source\n`);
                 }
                 finally {
                     try {
                         fs.unlinkSync(inputPath);
                     }
-                    catch {
-                        // ignore
+                    catch (unlinkErr) {
+                        process.stderr.write(`[agent-core] failed to clean up ${inputPath}: ${unlinkErr instanceof Error ? unlinkErr.message : String(unlinkErr)}\n`);
                     }
                     try {
                         fs.unlinkSync(outputPath);
                     }
-                    catch {
-                        // ignore
+                    catch (unlinkErr) {
+                        process.stderr.write(`[agent-core] failed to clean up ${outputPath}: ${unlinkErr instanceof Error ? unlinkErr.message : String(unlinkErr)}\n`);
                     }
                 }
                 return (0, results_1.ok)(`(mmdc not available — returning source)\n\n\`\`\`mermaid\n${source}\n\`\`\``);

package/dist/agenticTools/tools/execution.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"execution.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/execution.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;AAMzE,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CA4IxF"}
+{"version":3,"file":"execution.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/execution.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;AAMzE,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAiJxF"}

package/dist/agenticTools/tools/execution.js

@@ -47,6 +47,11 @@ function createExecutionTools(options) {
                         description: record.description,
                     });
                 }
+                // HERE BE DRAGONS: Shell invocation is duplicated in 5 files. All must use the same flags.
+                // See also: agent-core/session.ts, agent-platform/tools/execution.ts,
+                // agent-platform/backgroundProcessRegistry.ts, agent-runtime/backgroundProcessRegistry.ts
+                // HERE BE DRAGONS: Assumes /bin/bash exists on non-Windows. macOS Catalina+
+                // defaults to zsh, and some containers may not have bash at this path.
                 const shell = process.platform === "win32" ? "cmd.exe" : "/bin/bash";
                 const shellArgs = process.platform === "win32"
                     ? ["/c", String(params.command)]

package/dist/agenticTools/tools/fileSystem.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fileSystem.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/fileSystem.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAKrF,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CA+KzF"}
+{"version":3,"file":"fileSystem.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/fileSystem.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAsBrF,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,kBAAkB,GAAG,oBAAoB,EAAE,CAuLzF"}

package/dist/agenticTools/tools/fileSystem.js

@@ -40,6 +40,21 @@ const typebox_1 = require("@sinclair/typebox");
 const process_1 = require("../shared/process");
 const results_1 = require("../shared/results");
 const paths_1 = require("../shared/paths");
+function escapePatchPath(value) {
+    return value.replace(/\\/g, "/");
+}
+function buildUnifiedPatch(filePath, before, after) {
+    const label = escapePatchPath(filePath);
+    const beforeLines = before.split("\n");
+    const afterLines = after.split("\n");
+    return [
+        `--- a/${label}`,
+        `+++ b/${label}`,
+        `@@ -1,${beforeLines.length} +1,${afterLines.length} @@`,
+        ...beforeLines.map((line) => `-${line}`),
+        ...afterLines.map((line) => `+${line}`),
+    ].join("\n");
+}
 function createFileSystemTools(options) {
     const { workspace } = options;
     return [
@@ -56,8 +71,13 @@ function createFileSystemTools(options) {
                 const filePath = (0, paths_1.resolveSafe)(workspace, String(params.path));
                 const lines = fs.readFileSync(filePath, "utf8").split("\n");
                 const start = Math.max(0, (params.offset ?? 1) - 1);
-                const count = Math.min(params.limit ?? process_1.MAX_READ_LINES, process_1.MAX_READ_LINES);
-                return (0, results_1.ok)(lines.slice(start, start + count).map((line, index) => `${start + index + 1}\t${line}`).join("\n"));
+                const requestedLimit = params.limit ?? process_1.MAX_READ_LINES;
+                const count = Math.min(requestedLimit, process_1.MAX_READ_LINES);
+                const content = lines.slice(start, start + count).map((line, index) => `${start + index + 1}\t${line}`).join("\n");
+                if (requestedLimit > process_1.MAX_READ_LINES) {
+                    return (0, results_1.ok)(`${content}\n(capped at ${process_1.MAX_READ_LINES} lines — requested ${requestedLimit})`);
+                }
+                return (0, results_1.ok)(content);
             },
         },
         {
@@ -97,7 +117,8 @@ function createFileSystemTools(options) {
                     ? content.split(oldString).join(newString)
                     : content.replace(oldString, newString);
                 fs.writeFileSync(filePath, updated, "utf8");
-                return (0, results_1.ok)(`File edited: ${filePath}`);
+                const relativePath = path.relative(workspace, filePath) || String(params.path);
+                return (0, results_1.ok)(`File edited: ${filePath}\nUnified patch:\n${buildUnifiedPatch(relativePath, content, updated)}`);
             },
         },
         {
@@ -189,7 +210,9 @@ function createFileSystemTools(options) {
                 limit: typebox_1.Type.Optional(typebox_1.Type.Number({ description: "Max results (default: 500)" })),
             }),
             execute: (_toolCallId, params) => {
-                const maxResults = Math.min(params.limit ?? 500, 5000);
+                const DEFAULT_GLOB_LIMIT = 500;
+                const MAX_GLOB_LIMIT = 5000;
+                const maxResults = Math.min(params.limit ?? DEFAULT_GLOB_LIMIT, MAX_GLOB_LIMIT);
                 const allFiles = [];
                 (0, paths_1.walkDir)(workspace, Boolean(params.hidden), maxResults * 5, allFiles);
                 const regex = (0, paths_1.globToRegex)(String(params.pattern));

package/dist/agenticTools/tools/programmaticToolCalling.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"programmaticToolCalling.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/programmaticToolCalling.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAgBrF,wBAAgB,mCAAmC,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAExF;AAED,wBAAgB,iCAAiC,CAC/C,OAAO,EAAE,kBAAkB,EAC3B,cAAc,EAAE,oBAAoB,EAAE,GACrC,oBAAoB,CAwEtB"}
+{"version":3,"file":"programmaticToolCalling.d.ts","sourceRoot":"","sources":["../../../src/agenticTools/tools/programmaticToolCalling.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,kBAAkB,EAAE,oBAAoB,EAAc,MAAM,UAAU,CAAC;AAkBrF,wBAAgB,mCAAmC,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAExF;AAED,wBAAgB,iCAAiC,CAC/C,OAAO,EAAE,kBAAkB,EAC3B,cAAc,EAAE,oBAAoB,EAAE,GACrC,oBAAoB,CAwEtB"}

package/dist/agenticTools/tools/programmaticToolCalling.js

@@ -9,6 +9,8 @@ const node_vm_1 = __importDefault(require("node:vm"));
 const typebox_1 = require("@sinclair/typebox");
 const results_1 = require("../shared/results");
 const DEFAULT_TIMEOUT_MS = 120_000;
+// Prevents infinite tool-calling loops. 25 is high enough for complex multi-step work
+// but low enough to catch runaway recursion.
 const DEFAULT_MAX_TOOL_CALLS = 25;
 function shouldEnableProgrammaticToolCalling(options) {
     return Boolean(options.programmaticToolCalling);

package/dist/backgroundProcessRegistry.d.ts

@@ -1,63 +1,7 @@
 /**
- * Background process lifecycle management for the bash agentic tool (GAP-TOOLS-036).
- *
- * Tracks child processes spawned with `run_in_background: true`, collects
- * stdout/stderr, fires completion callbacks, and enforces a concurrency limit.
+ * Backwards-compatibility shim -- the canonical implementation now lives in
+ * `@a5c-ai/agent-runtime`.  This re-export keeps internal agent-core consumers
+ * (and any external code that deep-imported this path) working without changes.
  */
-/** Snapshot of a tracked background process. */
-export interface BackgroundTaskRecord {
-    backgroundTaskId: string;
-    pid: number;
-    command: string;
-    description?: string;
-    startedAt: string;
-    status: "running" | "completed" | "exited" | "cancelled" | "killed";
-    exitCode: number | null;
-    stdout: string;
-    stderr: string;
-    durationMs: number | null;
-}
-/** Payload delivered to `onComplete` callbacks. */
-export interface BackgroundCompletionEvent {
-    backgroundTaskId: string;
-    pid: number;
-    command: string;
-    description?: string;
-    status: "completed" | "exited";
-    exitCode: number;
-    stdout: string;
-    stderr: string;
-    durationMs: number;
-}
-/** Options for spawning a background process. */
-export interface SpawnOptions {
-    command: string;
-    cwd: string;
-    env?: Record<string, string>;
-    description?: string;
-    onComplete?: (event: BackgroundCompletionEvent) => void;
-}
-export declare class BackgroundProcessRegistry {
-    private readonly processes;
-    private readonly maxConcurrent;
-    constructor(options?: {
-        maxConcurrent?: number;
-    });
-    /**
-     * Spawn a command in the background and begin tracking it.
-     * Returns a snapshot of the initial record.
-     */
-    spawn(opts: SpawnOptions): BackgroundTaskRecord;
-    /** Get a snapshot of a tracked process by ID, or undefined. */
-    get(backgroundTaskId: string): BackgroundTaskRecord | undefined;
-    /** List snapshots of all tracked processes. */
-    list(): BackgroundTaskRecord[];
-    /** Cancel (SIGTERM) a running process. Returns true if found and killed. */
-    cancel(backgroundTaskId: string): boolean;
-    /** Kill all running processes. */
-    killAll(): void;
-    /** Kill all processes and clear tracking state. */
-    dispose(): void;
-    private snapshot;
-}
+export { BackgroundProcessRegistry, type BackgroundTaskRecord, type BackgroundCompletionEvent, type SpawnOptions, } from "@a5c-ai/agent-runtime";
 //# sourceMappingURL=backgroundProcessRegistry.d.ts.map

package/dist/backgroundProcessRegistry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"backgroundProcessRegistry.d.ts","sourceRoot":"","sources":["../src/backgroundProcessRegistry.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AASH,gDAAgD;AAChD,MAAM,WAAW,oBAAoB;IACnC,gBAAgB,EAAE,MAAM,CAAC;IACzB,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,SAAS,GAAG,WAAW,GAAG,QAAQ,GAAG,WAAW,GAAG,QAAQ,CAAC;IACpE,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAED,mDAAmD;AACnD,MAAM,WAAW,yBAAyB;IACxC,gBAAgB,EAAE,MAAM,CAAC;IACzB,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,MAAM,EAAE,WAAW,GAAG,QAAQ,CAAC;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,iDAAiD;AACjD,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,CAAC,KAAK,EAAE,yBAAyB,KAAK,IAAI,CAAC;CACzD;AA4BD,qBAAa,yBAAyB;IACpC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAqC;IAC/D,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAS;gBAE3B,OAAO,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,MAAM,CAAA;KAAE;IAIhD;;;OAGG;IACH,KAAK,CAAC,IAAI,EAAE,YAAY,GAAG,oBAAoB;IAuF/C,+DAA+D;IAC/D,GAAG,CAAC,gBAAgB,EAAE,MAAM,GAAG,oBAAoB,GAAG,SAAS;IAM/D,+CAA+C;IAC/C,IAAI,IAAI,oBAAoB,EAAE;IAI9B,4EAA4E;IAC5E,MAAM,CAAC,gBAAgB,EAAE,MAAM,GAAG,OAAO;IAczC,kCAAkC;IAClC,OAAO,IAAI,IAAI;IAcf,mDAAmD;IACnD,OAAO,IAAI,IAAI;IASf,OAAO,CAAC,QAAQ;CAcjB"}
+{"version":3,"file":"backgroundProcessRegistry.d.ts","sourceRoot":"","sources":["../src/backgroundProcessRegistry.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,OAAO,EACL,yBAAyB,EACzB,KAAK,oBAAoB,EACzB,KAAK,yBAAyB,EAC9B,KAAK,YAAY,GAClB,MAAM,uBAAuB,CAAC"}