refactor(bun-run): load as packaged extension

Author: rcarmo

runtime/extensions/integrations/bun-runner/index.ts

@@ -0,0 +1,3 @@
+import { bunRunner } from "../../../src/extensions/bun-runner.ts";
+
+export default bunRunner;

runtime/src/agent-pool/session.ts

@@ -49,6 +49,7 @@ const EXTENSIONS_DIR = resolve(__dirname, "../../extensions");
 const OPTIONAL_EXTENSIONS: { path: string; envGate?: string }[] = [
   { path: resolve(EXTENSIONS_DIR, "integrations", "azure-openai.ts"), envGate: "AOAI_BASE_URL" },
   { path: resolve(EXTENSIONS_DIR, "integrations", "context-mode.ts") },
+  { path: resolve(EXTENSIONS_DIR, "integrations", "bun-runner", "index.ts") },
   { path: resolve(EXTENSIONS_DIR, "browser", "cdp-browser", "index.ts") },
   { path: resolve(EXTENSIONS_DIR, "platform", "windows", "win-ui", "index.ts") },
   { path: resolve(EXTENSIONS_DIR, "viewers", "office-viewer", "index.ts") },

runtime/src/extensions/index.ts

@@ -16,7 +16,10 @@
  *   - workspaceSearch: search_workspace tool for FTS over workspace files.
  *   - sendAdaptiveCard: send_adaptive_card for agent-owned Adaptive Card posting.
  *   - sendDashboardWidget: send_dashboard_widget for posting the built-in live dashboard widget.
- *   - bunRunner: bun_run for direct Bun script execution without a shell.
+ *
+ * Note: bun_run now lives as a packaged runtime extension under
+ * runtime/extensions/integrations/bun-runner and is loaded via the
+ * additionalExtensionPaths wiring in agent-pool/session.ts.
  *
  * Consumers:
  *   - agent-pool/session.ts passes builtinExtensionFactories to the resource loader.
@@ -34,7 +37,6 @@ import { uiThemeExtension } from "./ui-theme.js";
 import { smartCompaction } from "./smart-compaction.js";
 import { sendAdaptiveCard } from "./send-adaptive-card.js";
 import { sendDashboardWidget } from "./send-dashboard-widget.js";
-import { bunRunner } from "./bun-runner.js";
 import { exitProcess } from "./exit-process.js";
 import { autoresearchSupervisor } from "./autoresearch-supervisor.js";
 
@@ -52,7 +54,6 @@ export const builtinExtensionFactories: ExtensionFactory[] = [
   smartCompaction,
   sendAdaptiveCard,
   sendDashboardWidget,
-  bunRunner,
   exitProcess,
   autoresearchSupervisor,
 ];

runtime/test/agent-pool/agent-pool-tools.test.ts

@@ -3,7 +3,8 @@
  *
  * Verifies that builtinExtensionFactories register the expected tools
  * (attach_file, messages, model control, tool discovery, keychain, SQL introspection, workspace search,
- * adaptive cards, dashboard widget posting, graceful exit, and autoresearch controls)
+ * adaptive cards, dashboard widget posting, graceful exit, and autoresearch controls).
+ * bun_run is now provided by a packaged runtime extension rather than a builtin factory.
  * and slash commands (/tasks, /scheduled, /theme, /tint) on a mock ExtensionAPI.
  */
 

runtime/test/extensions/extensions-bun-runner.test.ts

@@ -5,7 +5,7 @@ import { join } from "path";
 
 import { WORKSPACE_DIR } from "../../src/core/config.js";
 import { getToolOutput } from "../../src/tool-output.js";
-import { bunRunner } from "../../src/extensions/bun-runner.js";
+import bunRunnerExtension from "../../extensions/integrations/bun-runner/index.ts";
 import { createFakeExtensionApi } from "./fake-extension-api.js";
 
 const cleanupPaths: string[] = [];
@@ -28,14 +28,14 @@ afterEach(() => {
 describe("bun-runner extension", () => {
   test("registers bun_run and advertises its hint", async () => {
     const fake = createFakeExtensionApi();
-    bunRunner(fake.api);
+    bunRunnerExtension(fake.api);
 
     expect(fake.tools.has("bun_run")).toBe(true);
   });
 
   test("runs a workspace Bun script directly and discards stdout by default", async () => {
     const fake = createFakeExtensionApi();
-    bunRunner(fake.api);
+    bunRunnerExtension(fake.api);
     const tool = fake.tools.get("bun_run");
     if (!tool) throw new Error("bun_run not registered");
 
@@ -69,7 +69,7 @@ describe("bun-runner extension", () => {
 
   test("captures stdout when requested", async () => {
     const fake = createFakeExtensionApi();
-    bunRunner(fake.api);
+    bunRunnerExtension(fake.api);
     const tool = fake.tools.get("bun_run");
     if (!tool) throw new Error("bun_run not registered");
 
@@ -99,7 +99,7 @@ describe("bun-runner extension", () => {
     db.initDatabase();
 
     const fake = createFakeExtensionApi();
-    bunRunner(fake.api);
+    bunRunnerExtension(fake.api);
     const tool = fake.tools.get("bun_run");
     if (!tool) throw new Error("bun_run not registered");
 
@@ -126,7 +126,7 @@ describe("bun-runner extension", () => {
 
   test("rejects scripts outside the workspace", async () => {
     const fake = createFakeExtensionApi();
-    bunRunner(fake.api);
+    bunRunnerExtension(fake.api);
     const tool = fake.tools.get("bun_run");
     if (!tool) throw new Error("bun_run not registered");
 

runtime/test/extensions/optional-bundled-extensions.test.ts

@@ -63,6 +63,16 @@ function withRouteCapture<T>(state: FakeState, fn: () => Promise<T>): Promise<T>
 }
 
 describe("bundled optional extensions", () => {
+  test("bun-runner registers the bun_run tool", async () => {
+    const { default: registerBunRunner } = await import("../../extensions/integrations/bun-runner/index.ts");
+    const fake = createFakeApi();
+
+    registerBunRunner(fake.api);
+
+    expect(fake.state.tools.has("bun_run")).toBe(true);
+    expect(fake.state.tools.get("bun_run")?.description).toContain("workspace Bun script");
+  });
+
   test("cdp-browser registers the cdp_browser tool and cdp-tabs command", async () => {
     const { default: registerCdpBrowser } = await import("../../extensions/browser/cdp-browser/index.ts");
     const fake = createFakeApi();