@mycadet/assistant 0.12.9 → 0.12.11

package/src/runtime/routes/__tests__/deprecated-route-context.test.ts

@@ -1,77 +1,49 @@
 /**
  * The deprecated `/x/*` route context is a compatibility shim: each field
- * delegates to a `@mycadet/plugin-api` call and records a one-per-route
- * deprecation-usage telemetry signal. These tests mock the three collaborators
- * (`publishEvent`, `runConversationTurn`, `recordWatchdogEvent`) and assert the
- * delegation, the telemetry payload, and the fire-once dedup.
+ * delegates to a `@mycadet/plugin-api` call. These tests mock the two
+ * collaborators (`publishEvent`, `runConversationTurn`) and assert the
+ * delegation.
  */
 
-import { beforeEach, describe, expect, mock, test } from "bun:test";
+import { describe, expect, mock, test } from "bun:test";
 
 const publishCalls: Array<{ event: unknown; options: unknown }> = [];
 const turnCalls: Array<{ conversationId?: string; content: unknown }> = [];
-const watchdogCalls: Array<{
-  checkName: string;
-  detail?: Record<string, unknown> | null;
-}> = [];
 
 mock.module("../../../plugin-api/publish-event.js", () => ({
-  publishEvent: async (event: unknown, options: unknown) => {
+  publishEvent: (event: unknown, options?: unknown) => {
     publishCalls.push({ event, options });
   },
 }));
 mock.module("../../../plugin-api/conversation-turn.js", () => ({
-  runConversationTurn: async (opts: {
+  runConversationTurn: async (input: {
     conversationId?: string;
     content: unknown;
   }) => {
-    turnCalls.push(opts);
-    return {
-      content: [],
-      userMessageId: "user-msg-1",
-      conversationId: opts.conversationId ?? "generated",
-    };
-  },
-}));
-mock.module("../../../telemetry/watchdog-events-store.js", () => ({
-  recordWatchdogEvent: (record: {
-    checkName: string;
-    detail?: Record<string, unknown> | null;
-  }) => {
-    watchdogCalls.push(record);
+    turnCalls.push({
+      conversationId: input.conversationId,
+      content: input.content,
+    });
+    return { userMessageId: "user-msg-1" };
   },
 }));
 
 const { buildDeprecatedRouteContext } =
   await import("../deprecated-route-context.js");
 
-beforeEach(() => {
-  publishCalls.length = 0;
-  turnCalls.length = 0;
-  watchdogCalls.length = 0;
-});
-
 describe("deprecated route context shim", () => {
-  test("assistantEventHub.publish forwards to publishEvent and records a signal", async () => {
-    const ctx = buildDeprecatedRouteContext("route-publish");
+  test("assistantEventHub.publish forwards to publishEvent", async () => {
+    const ctx = buildDeprecatedRouteContext();
     const event = { id: "e1" } as never;
     const options = { excludeClientId: "tab-1" } as never;
 
     await ctx.assistantEventHub.publish(event, options);
 
     expect(publishCalls).toEqual([{ event, options }]);
-    expect(watchdogCalls).toHaveLength(1);
-    expect(watchdogCalls[0].checkName).toBe("deprecated_api_use");
-    expect(watchdogCalls[0].detail).toEqual({
-      surface: "custom_route",
-      api: "context.assistantEventHub.publish",
-      replacement: "plugin-api:publishEvent",
-      route_path: "route-publish",
-    });
   });
 
   test("conversations.postMessage runs a turn and returns its user message id", async () => {
-    const ctx = buildDeprecatedRouteContext("route-post");
+    const ctx = buildDeprecatedRouteContext();
 
     const result = await ctx.conversations.postMessage("conv-1", "hello there");
 
@@ -82,29 +54,10 @@ describe("deprecated route context shim", () => {
         content: [{ type: "text", text: "hello there" }],
       },
     ]);
-    expect(watchdogCalls).toHaveLength(1);
-    expect(watchdogCalls[0].detail).toEqual({
-      surface: "custom_route",
-      api: "context.conversations.postMessage",
-      replacement: "plugin-api:runConversationTurn",
-      route_path: "route-post",
-    });
-  });
-
-  test("records the deprecation signal once per route + api (dedup)", async () => {
-    const ctx = buildDeprecatedRouteContext("route-dedup");
-
-    await ctx.assistantEventHub.publish({} as never);
-    await ctx.assistantEventHub.publish({} as never);
-    await ctx.assistantEventHub.publish({} as never);
-
-    // Three calls, but only the first records a telemetry signal.
-    expect(publishCalls).toHaveLength(3);
-    expect(watchdogCalls).toHaveLength(1);
   });
 
   test("the context and its members are frozen", () => {
-    const ctx = buildDeprecatedRouteContext("route-frozen");
+    const ctx = buildDeprecatedRouteContext();
     expect(Object.isFrozen(ctx)).toBe(true);
     expect(Object.isFrozen(ctx.assistantEventHub)).toBe(true);
     expect(Object.isFrozen(ctx.conversations)).toBe(true);

package/src/runtime/routes/__tests__/inference-profiles-routes.test.ts

@@ -38,7 +38,7 @@ let managedProxyEnabled = true;
 mock.module("../../../providers/cloud-proxy/context.js", () => ({
   resolveCloudProxyContext: async () => ({
     enabled: managedProxyEnabled,
-    cloudBaseUrl: "https://platform.example",
+    cloudBaseUrl: "https://test-cloud.mycadet.ai",
     assistantApiKey: managedProxyEnabled ? "key" : "",
   }),
 }));
@@ -556,9 +556,8 @@ describe("PATCH inference/profiles/:name (update) validation", () => {
     setConfig("llm", {
       profiles: {
         balanced: {
-          source: "managed",
-          provider: "fireworks",
-          model: "accounts/fireworks/models/glm-5p2",
+          provider: "cadet",
+          model: "claude-sonnet-4-6",
         },
       },
     });
@@ -583,13 +582,6 @@ describe("PATCH inference/profiles/:name (update) validation", () => {
 // ── delete protection ───────────────────────────────────────────────────────
 
 describe("DELETE inference/profiles/:name protection", () => {
-  test("rejects deleting a cloud default profile", async () => {
-    setConfig("llm", { profiles: { balanced: { source: "managed" } } });
-    await expect(
-      call("inference_profiles_delete", { pathParams: { name: "balanced" } }),
-    ).rejects.toThrow(/cloud profile/);
-  });
-
   test("404s an unknown profile", async () => {
     setConfig("llm", { profiles: {} });
     await expect(
@@ -757,7 +749,7 @@ describe("GET inference/profiles honors the llm.default blob", () => {
     setConfig("llm", {
       profiles: {
         "zed-custom": { source: "user", provider: "anthropic" },
-        auto: { source: "managed" },
+        auto: { source: "cloud" },
         "alpha-custom": { source: "user", provider: "anthropic" },
       },
       profileOrder: ["auto", "balanced", "zed-custom"],
@@ -793,6 +785,107 @@ describe("GET inference/profiles honors the llm.default blob", () => {
   });
 });
 
+// ── resolved vision capability on the list ────────────────────────────────────
+
+describe("GET inference/profiles resolves supportsVision per entry", () => {
+  type Listed = { profiles: Array<{ name: string; supportsVision?: boolean }> };
+
+  test("a vision-capable (provider, model) carries supportsVision: true", async () => {
+    setConfig("llm", {
+      profiles: {
+        vision: {
+          source: "user",
+          provider: "anthropic",
+          model: "claude-fable-5-1",
+        },
+      },
+    });
+
+    const listed = (await call("inference_profiles_list", {})) as Listed;
+    const entry = listed.profiles.find((p) => p.name === "vision");
+    expect(entry?.supportsVision).toBe(true);
+  });
+
+  test("the entry's inputModalities.image override beats the catalog", async () => {
+    setConfig("llm", {
+      profiles: {
+        gated: {
+          source: "user",
+          provider: "anthropic",
+          model: "claude-fable-5-1",
+          inputModalities: { image: { enabled: false } },
+        },
+      },
+    });
+
+    const listed = (await call("inference_profiles_list", {})) as Listed;
+    const entry = listed.profiles.find((p) => p.name === "gated");
+    expect(entry?.supportsVision).toBe(false);
+  });
+
+  test("an override can enable image on a catalog-non-vision model", async () => {
+    setConfig("llm", {
+      profiles: {
+        forced: {
+          source: "user",
+          provider: "ollama",
+          model: "llama3.2",
+          inputModalities: { image: { enabled: true, supported: true } },
+        },
+      },
+    });
+
+    const listed = (await call("inference_profiles_list", {})) as Listed;
+    const entry = listed.profiles.find((p) => p.name === "forced");
+    expect(entry?.supportsVision).toBe(true);
+  });
+
+  test("profiles without a provider or model stay unflagged", async () => {
+    setConfig("llm", {
+      profiles: {
+        sparse: { source: "cloud" },
+      },
+    });
+
+    const listed = (await call("inference_profiles_list", {})) as Listed;
+    const entry = listed.profiles.find((p) => p.name === "sparse");
+    expect(entry).toBeDefined();
+    expect(entry).not.toHaveProperty("supportsVision");
+  });
+
+  test("mix profiles carry their arms so pickers can judge dispatchability", async () => {
+    setConfig("llm", {
+      profiles: {
+        "arm-a": {
+          source: "user",
+          provider: "anthropic",
+          model: "claude-fable-5-1",
+        },
+        "arm-b": { source: "user", provider: "ollama", model: "llama3.2" },
+        blend: {
+          source: "user",
+          mix: [
+            { profile: "arm-a", weight: 1 },
+            { profile: "arm-b", weight: 1 },
+          ],
+        },
+      },
+    });
+
+    const listed = (await call("inference_profiles_list", {})) as {
+      profiles: Array<{
+        name: string;
+        mix?: Array<{ profile: string; weight: number }>;
+      }>;
+    };
+    const entry = listed.profiles.find((p) => p.name === "blend");
+    expect(entry?.mix).toEqual([
+      { profile: "arm-a", weight: 1 },
+      { profile: "arm-b", weight: 1 },
+    ]);
+  });
+});
+
 // ── active-profile setter validation (Finding 3) ──────────────────────────────
 
 describe("POST inference/profiles/:name/validate billing guards", () => {

package/src/runtime/routes/__tests__/ingress-status-routes.test.ts

@@ -86,7 +86,7 @@ describe("integrations_ingress_status route", () => {
   test("reports unconfigured for a cloud-hosted assistant", async () => {
     ingressConfig = {
       ...ingressConfig,
-      publicBaseUrl: "https://platform.example.com/gateway/callbacks/a-1",
+      publicBaseUrl: "https://test-cloud.mycadet.ai/gateway/callbacks/a-1",
       managedCallbacks: true,
     };
     lastTunnel = { provider: "tailscale", publicBaseUrl: TUNNEL_URL };

package/src/runtime/routes/__tests__/monitoring-routes.test.ts

@@ -14,7 +14,7 @@
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 import type { ResourceSample } from "../../../monitoring/resource-sample-types.js";
-import { getMonitoringPidPath } from "../../../util/platform.js";
+import { getMonitoringPidPath } from "../../../util/layout.js";
 
 class FakeSpawnError extends Error {}
 

package/src/runtime/routes/__tests__/plugins-routes.test.ts

@@ -83,16 +83,11 @@ import {
   sanitizePluginName,
 } from "../../../cli/lib/install-from-github.js";
 import type { InstalledPluginInfo } from "../../../cli/lib/list-installed-plugins.js";
-import { DEFAULT_PIN_HISTORY_LIMIT } from "../../../cli/lib/plugin-constants.js";
 import {
   type PluginDetails,
   PluginDetailsNotFoundError,
   type PluginDetailsOptions,
 } from "../../../cli/lib/plugin-details.js";
-import {
-  type PluginPinHistoryEntry,
-  PluginPinHistoryError,
-} from "../../../cli/lib/plugin-pin-history.js";
 import type {
   PluginCatalog,
   PluginSearchMatch,
@@ -284,27 +279,6 @@ mock.module("../../../cli/lib/diff-plugin.js", () => ({
   diffPlugin: diffSpy,
 }));
 
-// Mock the pin-history lib: the GitHub commit-history walk is covered by
-// plugin-pin-history.test.ts; the routes only forward the name, validate a pin
-// against the resolved history, and map errors.
-const listPinHistorySpy = mock(
-  async (..._args: unknown[]): Promise<PluginPinHistoryEntry[]> => {
-    throw new Error("listPinHistorySpy default impl not configured");
-  },
-);
-const resolvePinSpy = mock(
-  async (..._args: unknown[]): Promise<PluginPinHistoryEntry | null> => {
-    throw new Error("resolvePinSpy default impl not configured");
-  },
-);
-
-mock.module("../../../cli/lib/plugin-pin-history.js", () => ({
-  DEFAULT_PIN_HISTORY_LIMIT,
-  PluginPinHistoryError,
-  listPinHistory: listPinHistorySpy,
-  resolvePinToMarketplaceCommit: resolvePinSpy,
-}));
-
 // Mock the toggle-plugin lib. `enablePlugin` / `disablePlugin` flip a
 // `.disabled` sentinel on disk in production; here we spy on them so the
 // route's broadcast + error mapping is the wiring under test. The error
@@ -381,7 +355,7 @@ mock.module("../../../skills/categories-cache.js", () => ({
   getLocalCategorySlugs: () => SKILLS_CATEGORY_SLUGS,
 }));
 
-import { getWorkspacePluginsDir } from "../../../util/platform.js";
+import { getWorkspacePluginsDir } from "../../../util/layout.js";
 import {
   BadRequestError,
   ConflictError,
@@ -416,7 +390,6 @@ const uninstallHandler = findHandler("plugins_uninstall");
 const getHandler = findHandler("plugins_get");
 const installHandler = findHandler("plugins_install");
 const inspectHandler = findHandler("plugins_inspect");
-const versionsHandler = findHandler("plugins_versions");
 const upgradeHandler = findHandler("plugins_upgrade");
 const diffHandler = findHandler("plugins_diff");
 const enableHandler = findHandler("plugins_enable");
@@ -1084,12 +1057,12 @@ describe("GET /v1/plugins/search", () => {
     expect("icon" in byName.get("plain-plugin")!).toBe(false);
   });
 
-  test("preserves a bundled MCP package source and integration metadata", async () => {
+  test("preserves MCP integration metadata from the catalog entry", async () => {
     getCatalogSpy.mockImplementation(async (ref) =>
       catalog(ref, [
         {
           name: "fathom",
-          path: "local:plugins/mcp-catalog/fathom@1.0.0",
+          path: `github:acme/fathom@${CAVEMAN_PIN}`,
           description: "Search meeting transcripts.",
           category: null,
           integration: {
@@ -1105,9 +1078,9 @@ describe("GET /v1/plugins/search", () => {
             logo: "fathom.png",
           },
           source: {
-            kind: "local",
-            path: "plugins/mcp-catalog/fathom",
-            version: "1.0.0",
+            kind: "github",
+            repo: "acme/fathom",
+            ref: CAVEMAN_PIN,
           },
         },
       ]),
@@ -1118,7 +1091,7 @@ describe("GET /v1/plugins/search", () => {
     expect(result.matches).toEqual([
       {
         name: "fathom",
-        path: "local:plugins/mcp-catalog/fathom@1.0.0",
+        path: `github:acme/fathom@${CAVEMAN_PIN}`,
         description: "Search meeting transcripts.",
         category: null,
         integration: {
@@ -1134,9 +1107,9 @@ describe("GET /v1/plugins/search", () => {
           logo: "fathom.png",
         },
         source: {
-          kind: "local",
-          path: "plugins/mcp-catalog/fathom",
-          version: "1.0.0",
+          kind: "github",
+          repo: "acme/fathom",
+          ref: CAVEMAN_PIN,
         },
       },
     ]);
@@ -1613,7 +1586,6 @@ function trustedInstallResult(opts: InstallPluginOptions): InstallPluginResult {
 describe("POST /v1/plugins/install", () => {
   beforeEach(() => {
     installSpy.mockReset();
-    resolvePinSpy.mockReset();
     broadcastMessageSpy.mockReset();
     getCatalogSpy.mockReset();
     // Default: the gated catalog resolves `caveman` to its bundled full-SHA pin.
@@ -1623,91 +1595,36 @@ describe("POST /v1/plugins/install", () => {
   });
 
   test("no-pin install resolves from the gated catalog and installs via trustedSource", async () => {
-    // The default (no-pin) path reads the same gated catalog `search`
-    // advertises and installs from a trusted pre-resolved source — no direct
+    // The default (no-pin) path reads the same catalog `search` advertises
+    // and installs from a trusted pre-resolved source — no direct
     // `plugins/marketplace.json` fetch (the pin/marketplace path is untouched).
-    const prev = process.env.CADET_DISABLE_CLOUD;
-    process.env.CADET_DISABLE_CLOUD = "true";
-    try {
-      installSpy.mockImplementation(async (opts) => trustedInstallResult(opts));
-
-      const result = await invokeInstall({
-        body: { name: "caveman", force: true },
-      });
-
-      // Resolved through the gated catalog, not the pin/marketplace path.
-      expect(getCatalogSpy).toHaveBeenCalledTimes(1);
-      expect(resolvePinSpy).not.toHaveBeenCalled();
-      // installPlugin receives the pre-resolved trusted coordinates (owner/repo
-      // split from the catalog `repo`, `rootPath` from its path, the full-SHA
-      // pin as `ref`) — no caller-supplied `ref`.
-      expect(installSpy.mock.calls[0]?.[0]).toEqual({
-        name: "caveman",
-        force: true,
-        trustedSource: {
-          owner: "JuliusBrussee",
-          repo: "caveman",
-          rootPath: "",
-          ref: CAVEMAN_PIN,
-        },
-      });
-      expect(result).toEqual({
-        ok: true,
-        name: "caveman",
-        target: "/workspace/.cadet/plugins/caveman",
-        fileCount: 7,
-        ref: CAVEMAN_PIN,
-      });
-    } finally {
-      if (prev === undefined) {
-        delete process.env.CADET_DISABLE_CLOUD;
-      } else {
-        process.env.CADET_DISABLE_CLOUD = prev;
-      }
-    }
-  });
-
-  test("installs a bundled package from the exact catalog source", async () => {
-    getCatalogSpy.mockImplementation(async (ref) =>
-      catalog(ref, [
-        {
-          name: "fathom",
-          path: "local:plugins/mcp-catalog/fathom@1.0.0",
-          category: null,
-          source: {
-            kind: "local",
-            path: "plugins/mcp-catalog/fathom",
-            version: "1.0.0",
-          },
-        },
-      ]),
-    );
-    installSpy.mockImplementation(async (opts) => ({
-      name: opts.name,
-      target: `/workspace/.cadet/plugins/${opts.name}`,
-      fileCount: 2,
-      ref: "1.0.0",
-      commit: null,
-      committedAt: null,
-    }));
+    installSpy.mockImplementation(async (opts) => trustedInstallResult(opts));
 
-    const result = await invokeInstall({ body: { name: "fathom" } });
+    const result = await invokeInstall({
+      body: { name: "caveman", force: true },
+    });
 
+    // Resolved through the catalog, not the pin/marketplace path.
+    expect(getCatalogSpy).toHaveBeenCalledTimes(1);
+    // installPlugin receives the pre-resolved trusted coordinates (owner/repo
+    // split from the catalog `repo`, `rootPath` from its path, the full-SHA
+    // pin as `ref`) — no caller-supplied `ref`.
     expect(installSpy.mock.calls[0]?.[0]).toEqual({
-      name: "fathom",
-      force: undefined,
+      name: "caveman",
+      force: true,
       trustedSource: {
-        kind: "local",
-        path: "plugins/mcp-catalog/fathom",
-        version: "1.0.0",
+        owner: "JuliusBrussee",
+        repo: "caveman",
+        rootPath: "",
+        ref: CAVEMAN_PIN,
       },
     });
     expect(result).toEqual({
       ok: true,
-      name: "fathom",
-      target: "/workspace/.cadet/plugins/fathom",
-      fileCount: 2,
-      ref: "1.0.0",
+      name: "caveman",
+      target: "/workspace/.cadet/plugins/caveman",
+      fileCount: 7,
+      ref: CAVEMAN_PIN,
     });
   });
 
@@ -1873,121 +1790,40 @@ describe("POST /v1/plugins/install", () => {
     expect((caught as Error).message).toContain("ECONNRESET");
   });
 
-  test("a reviewed pin installs from the marketplace commit that introduced it", async () => {
-    // GIVEN a pin that resolves to a marketplace commit in the reviewed history
-    resolvePinSpy.mockImplementation(async () => ({
-      pin: "a".repeat(40),
-      marketplaceCommit: "f".repeat(40),
-      promotedAt: "2026-06-01T00:00:00.000Z",
-      current: false,
-    }));
-    installSpy.mockImplementation(async (opts) => ({
-      name: opts.name,
-      target: `/workspace/.cadet/plugins/${opts.name}`,
-      fileCount: 7,
-      ref: opts.ref ?? "main",
-      commit: "a".repeat(40),
-      committedAt: null,
-    }));
+  test("a pin matching the catalog's current ref installs via trustedSource", async () => {
+    // The only reviewed revision is the one the live catalog pins now; a
+    // matching pin installs the same trusted coordinates as the no-pin path.
+    installSpy.mockImplementation(async (opts) => trustedInstallResult(opts));
 
     const result = await invokeInstall({
-      body: { name: "caveman", pin: "a".repeat(40) },
+      body: { name: "caveman", pin: CAVEMAN_PIN },
     });
 
-    // THEN the install reads the manifest at the resolving marketplace commit,
-    // not the default branch, and the gated catalog resolver is bypassed — the
-    // pin path stays on the reviewed pin-history/GitHub route.
-    expect(getCatalogSpy).not.toHaveBeenCalled();
     expect(installSpy.mock.calls[0]?.[0]).toEqual({
       name: "caveman",
-      ref: "f".repeat(40),
       force: undefined,
+      trustedSource: {
+        owner: "JuliusBrussee",
+        repo: "caveman",
+        rootPath: "",
+        ref: CAVEMAN_PIN,
+      },
     });
-    expect(result.ref).toBe("f".repeat(40));
+    expect(result.ref).toBe(CAVEMAN_PIN);
   });
 
-  test("an unreviewed pin is refused as a BadRequest before installing", async () => {
-    resolvePinSpy.mockImplementation(async () => null);
-
+  test("a pin that is not the catalog's current ref is refused as BadRequest", async () => {
     await expect(
       invokeInstall({ body: { name: "caveman", pin: "b".repeat(40) } }),
     ).rejects.toBeInstanceOf(BadRequestError);
     expect(installSpy).not.toHaveBeenCalled();
   });
 
-  test("a pin-history read failure surfaces as ServiceUnavailable (503)", async () => {
-    resolvePinSpy.mockImplementation(async () => {
-      throw new PluginPinHistoryError("HTTP 403");
-    });
-
+  test("a pin for an unknown plugin is refused as BadRequest", async () => {
     await expect(
-      invokeInstall({ body: { name: "caveman", pin: "c".repeat(40) } }),
-    ).rejects.toBeInstanceOf(ServiceUnavailableError);
-    expect(installSpy).not.toHaveBeenCalled();
-  });
-});
-
-describe("GET /v1/plugins/:name/versions", () => {
-  beforeEach(() => {
-    listPinHistorySpy.mockReset();
-  });
-
-  test("returns the resolved pin history", async () => {
-    const history: PluginPinHistoryEntry[] = [
-      {
-        pin: "c".repeat(40),
-        marketplaceCommit: "3".repeat(40),
-        promotedAt: "2026-06-10T00:00:00.000Z",
-        current: true,
-      },
-      {
-        pin: "a".repeat(40),
-        marketplaceCommit: "1".repeat(40),
-        promotedAt: "2026-06-01T00:00:00.000Z",
-        current: false,
-      },
-    ];
-    listPinHistorySpy.mockImplementation(async () => history);
-
-    const result = await versionsHandler({ pathParams: { name: "level-up" } });
-    expect(result).toEqual(history);
-  });
-
-  test("forwards a positive limit to the lib", async () => {
-    listPinHistorySpy.mockImplementation(async () => []);
-    await versionsHandler({
-      pathParams: { name: "level-up" },
-      queryParams: { limit: "3" },
-    });
-    expect(listPinHistorySpy.mock.calls[0]?.[2]).toEqual({ limit: 3 });
-  });
-
-  test("rejects a non-positive limit as BadRequest before calling the lib", async () => {
-    await expect(
-      versionsHandler({
-        pathParams: { name: "level-up" },
-        queryParams: { limit: "0" },
-      }),
+      invokeInstall({ body: { name: "ghost", pin: "b".repeat(40) } }),
     ).rejects.toBeInstanceOf(BadRequestError);
-    expect(listPinHistorySpy).not.toHaveBeenCalled();
-  });
-
-  test("InvalidPluginNameError → BadRequestError (400)", async () => {
-    listPinHistorySpy.mockImplementation(async () => {
-      throw new InvalidPluginNameError("../escape");
-    });
-    await expect(
-      versionsHandler({ pathParams: { name: "../escape" } }),
-    ).rejects.toBeInstanceOf(BadRequestError);
-  });
-
-  test("PluginPinHistoryError → ServiceUnavailableError (503)", async () => {
-    listPinHistorySpy.mockImplementation(async () => {
-      throw new PluginPinHistoryError("HTTP 429");
-    });
-    await expect(
-      versionsHandler({ pathParams: { name: "level-up" } }),
-    ).rejects.toBeInstanceOf(ServiceUnavailableError);
+    expect(installSpy).not.toHaveBeenCalled();
   });
 });
 

package/src/runtime/routes/__tests__/route-host-worker-routes.test.ts

@@ -10,7 +10,7 @@
 
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
-import { getProcPidPath } from "../../../util/platform.js";
+import { getProcPidPath } from "../../../util/layout.js";
 
 class FakeSpawnError extends Error {}