@vellumai/assistant 0.7.0 → 0.7.1

package/src/__tests__/send-endpoint-busy.test.ts

@@ -10,7 +10,6 @@ import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../config/env.js", () => ({ isHttpAuthDisabled: () => true }));
 
-import { createGuardianBinding } from "../contacts/contacts-write.js";
 import type { Conversation } from "../daemon/conversation.js";
 import type { ServerMessage } from "../daemon/message-protocol.js";
 import {
@@ -21,6 +20,7 @@ import {
   getConversationByKey,
   getOrCreateConversation,
 } from "../memory/conversation-key-store.js";
+import { createGuardianBinding } from "./helpers/create-guardian-binding.js";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -92,6 +92,10 @@ let _conversationFactory: (() => Conversation) | undefined;
 let _approvalGenerator: unknown;
 
 mock.module("../daemon/conversation-store.js", () => ({
+  findConversation: () => {
+    if (!_conversationFactory) return undefined;
+    return _conversationFactory();
+  },
   getOrCreateConversation: async (..._args: unknown[]) => {
     if (!_conversationFactory)
       throw new Error("_conversationFactory not set in test");
@@ -154,11 +158,7 @@ function makeCompletingConversation(): Conversation {
     ensureActorScopedHistory: async () => {},
     usageStats: { inputTokens: 0, outputTokens: 0, estimatedCost: 0 },
     updateClient: () => {},
-    setHostBashProxy: () => {},
     setHostBrowserProxy: () => {},
-    setHostFileProxy: () => {},
-    setHostTransferProxy: () => {},
-    getHostTransferProxy: () => undefined,
     setHostCuProxy: () => {},
     addPreactivatedSkillId: () => {},
     hasAnyPendingConfirmation: () => false,
@@ -214,11 +214,7 @@ function makeHangingConversation(): Conversation {
     ensureActorScopedHistory: async () => {},
     usageStats: { inputTokens: 0, outputTokens: 0, estimatedCost: 0 },
     updateClient: () => {},
-    setHostBashProxy: () => {},
     setHostBrowserProxy: () => {},
-    setHostFileProxy: () => {},
-    setHostTransferProxy: () => {},
-    getHostTransferProxy: () => undefined,
     setHostCuProxy: () => {},
     addPreactivatedSkillId: () => {},
     hasAnyPendingConfirmation: () => false,
@@ -302,11 +298,7 @@ function makePendingApprovalConversation(
     ensureActorScopedHistory: async () => {},
     usageStats: { inputTokens: 0, outputTokens: 0, estimatedCost: 0 },
     updateClient: () => {},
-    setHostBashProxy: () => {},
     setHostBrowserProxy: () => {},
-    setHostFileProxy: () => {},
-    setHostTransferProxy: () => {},
-    getHostTransferProxy: () => undefined,
     setHostCuProxy: () => {},
     addPreactivatedSkillId: () => {},
     hasAnyPendingConfirmation: () => pending.size > 0,
@@ -425,12 +417,12 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     // Subscribe on the module-level singleton that the route handler publishes to
     const { assistantEventHub: routeEventHub } =
       await import("../runtime/assistant-event-hub.js");
-    routeEventHub.subscribe(
-      { assistantId: "self" },
-      (event: AssistantEvent) => {
+    routeEventHub.subscribe({
+      type: "process",
+      callback: (event: AssistantEvent) => {
         publishedEvents.push(event);
       },
-    );
+    });
 
     const res = await fetch(messagesUrl(), {
       method: "POST",
@@ -468,7 +460,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     } = makePendingApprovalConversation(requestId, false);
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -528,7 +519,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     } = makePendingApprovalConversation(requestId, false);
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -596,7 +586,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     } = makePendingApprovalConversation(requestId, true);
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -656,7 +645,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     } = makePendingApprovalConversation(requestId, true, { queueDepth: 2 });
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -716,7 +704,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     } = makePendingApprovalConversation(requestId, false);
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -774,7 +761,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
     );
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });
@@ -969,7 +955,6 @@ describe("POST /v1/messages — queue-if-busy and hub publishing", () => {
       makePendingApprovalConversation(requestId, false);
 
     pendingInteractions.register(requestId, {
-      conversation,
       conversationId,
       kind: "confirmation",
     });

package/src/__tests__/settings-routes.test.ts

@@ -17,12 +17,12 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-import { createGuardianBinding } from "../contacts/contacts-write.js";
 import { getSqlite } from "../memory/db-connection.js";
 import { initializeDb } from "../memory/db-init.js";
 import { BadRequestError, NotFoundError } from "../runtime/routes/errors.js";
 import { ROUTES } from "../runtime/routes/settings-routes.js";
 import type { RouteHandlerArgs } from "../runtime/routes/types.js";
+import { createGuardianBinding } from "./helpers/create-guardian-binding.js";
 
 initializeDb();
 

package/src/__tests__/shell-credential-ref.test.ts

@@ -32,14 +32,6 @@ mock.module("../security/secret-scanner.js", () => ({
   redactSecrets: (s: string) => s,
 }));
 
-// Mock sandbox
-mock.module("../tools/terminal/sandbox.js", () => ({
-  wrapCommand: (cmd: string, _cwd: string) => ({
-    command: "/bin/sh",
-    args: ["-c", cmd],
-  }),
-}));
-
 // Mock safe-env
 mock.module("../tools/terminal/safe-env.js", () => ({
   buildSanitizedEnv: () => ({ PATH: "/usr/bin" }),

package/src/__tests__/shell-tool-proxy-mode.test.ts

@@ -37,8 +37,6 @@ const mockConfig = {
   rateLimit: { maxRequestsPerMinute: 0 },
   secretDetection: {
     enabled: true,
-    action: "warn" as const,
-    entropyThreshold: 4.0,
   },
   auditLog: { retentionDays: 0 },
 };
@@ -65,26 +63,6 @@ mock.module("../security/secret-scanner.js", () => ({
   redactSecrets: (s: string) => s,
 }));
 
-// Track wrapCommand calls to verify networkMode is forwarded
-let wrapCommandCalls: {
-  cmd: string;
-  workingDir: string;
-  config: unknown;
-  options: unknown;
-}[] = [];
-
-mock.module("../tools/terminal/sandbox.js", () => ({
-  wrapCommand: (
-    cmd: string,
-    workingDir: string,
-    config: unknown,
-    options?: unknown,
-  ) => {
-    wrapCommandCalls.push({ cmd, workingDir, config, options });
-    return { command: "/bin/sh", args: ["-c", cmd], sandboxed: false };
-  },
-}));
-
 // --- Proxy session mocks ---
 let mockActiveSession: { id: string; conversationId: string } | undefined;
 let getOrStartSessionCalls: {
@@ -181,7 +159,6 @@ function makeContext(overrides?: Partial<ToolContext>): ToolContext {
 
 afterEach(() => {
   spawnCalls.length = 0;
-  wrapCommandCalls = [];
   getOrStartSessionCalls = [];
   getSessionEnvCalls = [];
   mockActiveSession = undefined;
@@ -303,37 +280,4 @@ describe("shell tool proxy mode", () => {
     expect(props.network_mode).toBeDefined();
     expect(props.credential_ids).toBeDefined();
   });
-
-  test('wrapCommand receives { networkMode: "proxied" } when network_mode=proxied', async () => {
-    const result = await shellTool.execute(
-      { command: "echo proxied-wrap", network_mode: "proxied" },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(wrapCommandCalls).toHaveLength(1);
-    expect(wrapCommandCalls[0].options).toEqual({ networkMode: "proxied" });
-  });
-
-  test('wrapCommand receives { networkMode: "off" } when network_mode is absent', async () => {
-    const result = await shellTool.execute(
-      { command: "echo default-wrap" },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(wrapCommandCalls).toHaveLength(1);
-    expect(wrapCommandCalls[0].options).toEqual({ networkMode: "off" });
-  });
-
-  test('wrapCommand receives { networkMode: "off" } when network_mode=off', async () => {
-    const result = await shellTool.execute(
-      { command: "echo off-wrap", network_mode: "off" },
-      makeContext(),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(wrapCommandCalls).toHaveLength(1);
-    expect(wrapCommandCalls[0].options).toEqual({ networkMode: "off" });
-  });
 });

package/src/__tests__/skill-script-runner-sandbox.test.ts

@@ -20,8 +20,6 @@ const mockConfig = {
   rateLimit: { maxRequestsPerMinute: 0 },
   secretDetection: {
     enabled: true,
-    action: "warn" as const,
-    entropyThreshold: 4.0,
   },
   auditLog: { retentionDays: 0 },
 };
@@ -44,15 +42,6 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-// Pass through without actual sandboxing so tests can run bun directly
-mock.module("../tools/terminal/sandbox.js", () => ({
-  wrapCommand: (command: string, _workingDir: string, _config: unknown) => ({
-    command: "bash",
-    args: ["-c", "--", command],
-    sandboxed: false,
-  }),
-}));
-
 import { runSkillToolScript } from "../tools/skills/skill-script-runner.js";
 import type { ToolContext } from "../tools/types.js";
 

package/src/__tests__/skill-tool-factory.test.ts

@@ -269,6 +269,103 @@ describe("createSkillToolsFromManifest", () => {
   });
 });
 
+// ---------------------------------------------------------------------------
+// createSkillTool — unknown parameter validation
+// ---------------------------------------------------------------------------
+
+describe("createSkillTool — unknown parameter validation", () => {
+  test("rejects input with unknown parameters", async () => {
+    const hash = computeSkillVersionHash(tempDir);
+    const tool = createSkillTool(
+      makeEntry({ executor: "echo.ts" }),
+      "my-skill",
+      tempDir,
+      hash,
+    );
+
+    const result = await tool.execute(
+      { query: "hello", unsubscribe: true },
+      makeContext(),
+    );
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain('Unknown parameter "unsubscribe"');
+    expect(result.content).toContain("Supported parameters");
+    expect(result.content).toContain('"query"');
+  });
+
+  test("rejects multiple unknown parameters", async () => {
+    const hash = computeSkillVersionHash(tempDir);
+    const tool = createSkillTool(
+      makeEntry({ executor: "echo.ts" }),
+      "my-skill",
+      tempDir,
+      hash,
+    );
+
+    const result = await tool.execute(
+      { query: "hello", foo: 1, bar: 2 },
+      makeContext(),
+    );
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain("Unknown parameters");
+    expect(result.content).toContain('"foo"');
+    expect(result.content).toContain('"bar"');
+  });
+
+  test("allows input with only known parameters", async () => {
+    const hash = computeSkillVersionHash(tempDir);
+    const tool = createSkillTool(
+      makeEntry({ executor: "echo.ts" }),
+      "my-skill",
+      tempDir,
+      hash,
+    );
+
+    const result = await tool.execute({ query: "hello" }, makeContext());
+
+    expect(result.isError).toBe(false);
+  });
+
+  test("allows empty input when schema has no required fields", async () => {
+    const hash = computeSkillVersionHash(tempDir);
+    const tool = createSkillTool(
+      makeEntry({
+        executor: "echo.ts",
+        input_schema: {
+          type: "object",
+          properties: { query: { type: "string" } },
+        },
+      }),
+      "my-skill",
+      tempDir,
+      hash,
+    );
+
+    const result = await tool.execute({}, makeContext());
+
+    expect(result.isError).toBe(false);
+  });
+
+  test("skips validation when schema has no properties", async () => {
+    const hash = computeSkillVersionHash(tempDir);
+    const tool = createSkillTool(
+      makeEntry({
+        executor: "echo.ts",
+        input_schema: { type: "object" },
+      }),
+      "my-skill",
+      tempDir,
+      hash,
+    );
+
+    const result = await tool.execute({ anything: "goes" }, makeContext());
+
+    expect(result.isError).toBe(false);
+  });
+});
+
 // ---------------------------------------------------------------------------
 // createSkillTool — expectedSkillVersionHash plumbing
 // ---------------------------------------------------------------------------

package/src/__tests__/skills-file-content-endpoint.test.ts

@@ -278,23 +278,11 @@ mock.module("../daemon/handlers/shared.js", () => ({
 // Imports (after mocks)
 // ---------------------------------------------------------------------------
 
-import type { SkillOperationContext } from "../daemon/handlers/skills.js";
 import {
   _resetFileProvidersForTest,
   getSkillFileContent,
 } from "../daemon/handlers/skills.js";
 
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-const dummyCtx = {
-  debounceTimers: { schedule: () => {} },
-  setSuppressConfigReload: () => {},
-  updateConfigFingerprint: () => {},
-  broadcast: () => {},
-} as unknown as SkillOperationContext;
-
 type FetchFn = typeof globalThis.fetch;
 
 let originalFetch: FetchFn;
@@ -375,7 +363,7 @@ describe("getSkillFileContent — installed skill", () => {
     mockResolvedSkills = [installedSkill("my-skill", skillDir)];
     installFetchForbidden();
 
-    const result = await getSkillFileContent("my-skill", "SKILL.md", dummyCtx);
+    const result = await getSkillFileContent("my-skill", "SKILL.md");
     expect("error" in result).toBe(false);
     if ("error" in result) return;
     expect(result.path).toBe("SKILL.md");
@@ -395,7 +383,7 @@ describe("getSkillFileContent — installed skill", () => {
     mockResolvedSkills = [installedSkill("my-skill", skillDir)];
     installFetchForbidden();
 
-    const result = await getSkillFileContent("my-skill", "img.png", dummyCtx);
+    const result = await getSkillFileContent("my-skill", "img.png");
     expect("error" in result).toBe(false);
     if ("error" in result) return;
     expect(result.isBinary).toBe(true);
@@ -410,7 +398,7 @@ describe("getSkillFileContent — installed skill", () => {
     installFetchForbidden();
 
     for (const bad of ["../secrets", "..", "/etc/passwd", "./../escape"]) {
-      const result = await getSkillFileContent("my-skill", bad, dummyCtx);
+      const result = await getSkillFileContent("my-skill", bad);
       expect("error" in result).toBe(true);
       if (!("error" in result)) continue;
       expect(result.status).toBe(400);
@@ -427,7 +415,6 @@ describe("getSkillFileContent — installed skill", () => {
     const result = await getSkillFileContent(
       "my-skill",
       "SKILL.md\0.png",
-      dummyCtx,
     );
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -440,7 +427,7 @@ describe("getSkillFileContent — installed skill", () => {
     mockResolvedSkills = [installedSkill("my-skill", skillDir)];
     installFetchForbidden();
 
-    const result = await getSkillFileContent("my-skill", "ghost.txt", dummyCtx);
+    const result = await getSkillFileContent("my-skill", "ghost.txt");
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
     expect(result.status).toBe(404);
@@ -474,7 +461,6 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
     const result = await getSkillFileContent(
       "remote-skill",
       "SKILL.md",
-      dummyCtx,
     );
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -495,7 +481,6 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
     const result = await getSkillFileContent(
       "ghost-skill",
       "SKILL.md",
-      dummyCtx,
     );
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -528,7 +513,6 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
     const result = await getSkillFileContent(
       "owner/repo/my-skill",
       "SKILL.md",
-      dummyCtx,
     );
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -559,7 +543,7 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
       toSlimSkill: async () => null,
     };
 
-    const result = await getSkillFileContent("cool-tool", "SKILL.md", dummyCtx);
+    const result = await getSkillFileContent("cool-tool", "SKILL.md");
     expect("error" in result).toBe(false);
     if ("error" in result) return;
     expect(result.content).toBe("# clawhub content yo\n");
@@ -581,7 +565,6 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
     const result = await getSkillFileContent(
       "known-skill",
       "nonexistent.txt",
-      dummyCtx,
     );
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -624,7 +607,6 @@ describe("getSkillFileContent — uninstalled skill (provider chain)", () => {
     const result = await getSkillFileContent(
       "simple-slug",
       "SKILL.md",
-      dummyCtx,
     );
     // Should be "File not found" (vellum handled but returned null)
     expect("error" in result).toBe(true);
@@ -647,7 +629,6 @@ describe("getSkillFileContent — skill not found", () => {
     const result = await getSkillFileContent(
       "ghost-skill",
       "SKILL.md",
-      dummyCtx,
     );
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -688,7 +669,6 @@ describe("getSkillFileContent — installed skill with missing directory", () =>
     const result = await getSkillFileContent(
       "ghost-installed",
       "SKILL.md",
-      dummyCtx,
     );
 
     expect("error" in result).toBe(true);
@@ -713,7 +693,7 @@ describe("getSkillFileContent — hidden / SKIP_DIRS rejection", () => {
     mockResolvedSkills = [installedSkill("leaky-skill", skillDir)];
     installFetchForbidden();
 
-    const result = await getSkillFileContent("leaky-skill", ".env", dummyCtx);
+    const result = await getSkillFileContent("leaky-skill", ".env");
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
     expect(result.status).toBe(400);
@@ -737,7 +717,7 @@ describe("getSkillFileContent — hidden / SKIP_DIRS rejection", () => {
       toSlimSkill: async () => null,
     };
 
-    const result = await getSkillFileContent("catalog-leaky", ".env", dummyCtx);
+    const result = await getSkillFileContent("catalog-leaky", ".env");
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
     expect(result.status).toBe(400);
@@ -753,7 +733,7 @@ describe("getSkillFileContent — hidden / SKIP_DIRS rejection", () => {
     installFetchForbidden();
 
     for (const bad of [".git/config", "docs/.hidden/file.md"]) {
-      const result = await getSkillFileContent("my-skill", bad, dummyCtx);
+      const result = await getSkillFileContent("my-skill", bad);
       expect("error" in result).toBe(true);
       if (!("error" in result)) continue;
       expect(result.status).toBe(400);
@@ -772,7 +752,7 @@ describe("getSkillFileContent — hidden / SKIP_DIRS rejection", () => {
       "__pycache__/cached.pyc",
       "nested/node_modules/mod/index.js",
     ]) {
-      const result = await getSkillFileContent("my-skill", bad, dummyCtx);
+      const result = await getSkillFileContent("my-skill", bad);
       expect("error" in result).toBe(true);
       if (!("error" in result)) continue;
       expect(result.status).toBe(400);
@@ -789,7 +769,6 @@ describe("getSkillFileContent — hidden / SKIP_DIRS rejection", () => {
     const result = await getSkillFileContent(
       "healthy-skill",
       "SKILL.md",
-      dummyCtx,
     );
     expect("error" in result).toBe(false);
     if ("error" in result) return;

package/src/__tests__/skills-files-catalog-fallback.test.ts

@@ -238,12 +238,6 @@ import {
 // Helpers
 // ---------------------------------------------------------------------------
 
-const dummyCtx = {
-  debounceTimers: { schedule: () => {} },
-  setSuppressConfigReload: () => {},
-  updateConfigFingerprint: () => {},
-  broadcast: () => {},
-} as unknown as Parameters<typeof getSkillFiles>[1];
 
 function makeSummary(overrides: Partial<SkillSummary>): SkillSummary {
   return {
@@ -318,7 +312,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("acme-seo", dummyCtx);
+    const result = await getSkillFiles("acme-seo");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -348,7 +342,7 @@ describe("getSkillFiles — provider chain fallback", () => {
   test("returns 404 when skill is neither installed nor handled by any provider", async () => {
     mockResolvedStates = [];
 
-    const result = await getSkillFiles("ghost-skill", dummyCtx);
+    const result = await getSkillFiles("ghost-skill");
 
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -364,7 +358,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       toSlimSkill: async () => null,
     };
 
-    const result = await getSkillFiles("broken-skill", dummyCtx);
+    const result = await getSkillFiles("broken-skill");
 
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -404,7 +398,7 @@ describe("getSkillFiles — provider chain fallback", () => {
         },
       ];
 
-      const result = await getSkillFiles("installed-skill", dummyCtx);
+      const result = await getSkillFiles("installed-skill");
 
       expect("error" in result).toBe(false);
       if ("error" in result) return;
@@ -467,7 +461,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("ghost-installed", dummyCtx);
+    const result = await getSkillFiles("ghost-installed");
 
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -491,7 +485,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("plain-skill", dummyCtx);
+    const result = await getSkillFiles("plain-skill");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -516,7 +510,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("fancy-skill", dummyCtx);
+    const result = await getSkillFiles("fancy-skill");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -559,7 +553,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("owner/repo/my-skill", dummyCtx);
+    const result = await getSkillFiles("owner/repo/my-skill");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -621,7 +615,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("cool-tool", dummyCtx);
+    const result = await getSkillFiles("cool-tool");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;
@@ -635,7 +629,7 @@ describe("getSkillFiles — provider chain fallback", () => {
     mockSkillsshProvider = makeNoopProvider("skillssh");
     mockClawhubProvider = makeNoopProvider("clawhub");
 
-    const result = await getSkillFiles("unknown-origin-skill", dummyCtx);
+    const result = await getSkillFiles("unknown-origin-skill");
 
     expect("error" in result).toBe(true);
     if (!("error" in result)) return;
@@ -715,7 +709,7 @@ describe("getSkillFiles — provider chain fallback", () => {
       }),
     };
 
-    const result = await getSkillFiles("contested-skill", dummyCtx);
+    const result = await getSkillFiles("contested-skill");
 
     expect("error" in result).toBe(false);
     if ("error" in result) return;