@vellumai/assistant 0.8.6 → 0.8.7

package/src/__tests__/migration-import-preflight-http.test.ts

@@ -710,10 +710,10 @@ describe("DefaultPathResolver", () => {
 
 describe("route policy registration", () => {
   test("migrations/import-preflight policy requires settings.write scope", async () => {
-    const { getPolicy } = await import("../runtime/auth/route-policy.js");
-    const policy = getPolicy("migrations/import-preflight");
+    const { ROUTES } = await import("../runtime/routes/index.js");
+    const policy = (ROUTES.find((r) => r.endpoint === "migrations/import-preflight")?.policy ?? null);
 
-    expect(policy).toBeDefined();
+    expect(policy).not.toBeNull();
     expect(policy?.requiredScopes).toContain("settings.write");
     expect(policy?.allowedPrincipalTypes).toContain("actor");
     expect(policy?.allowedPrincipalTypes).toContain("svc_gateway");
@@ -721,10 +721,10 @@ describe("route policy registration", () => {
   });
 
   test("import-preflight policy matches export but differs from validate on scopes", async () => {
-    const { getPolicy } = await import("../runtime/auth/route-policy.js");
-    const preflightPolicy = getPolicy("migrations/import-preflight");
-    const validatePolicy = getPolicy("migrations/validate");
-    const exportPolicy = getPolicy("migrations/export");
+    const { ROUTES } = await import("../runtime/routes/index.js");
+    const preflightPolicy = (ROUTES.find((r) => r.endpoint === "migrations/import-preflight")?.policy ?? null);
+    const validatePolicy = (ROUTES.find((r) => r.endpoint === "migrations/validate")?.policy ?? null);
+    const exportPolicy = (ROUTES.find((r) => r.endpoint === "migrations/export")?.policy ?? null);
 
     // preflight and export both require settings.write
     expect(preflightPolicy!.requiredScopes).toEqual(

package/src/__tests__/migration-validate-http.test.ts

@@ -630,10 +630,10 @@ describe("handleMigrationValidate", () => {
 describe("route policy registration", () => {
   test("migrations/validate policy requires settings.read scope", async () => {
     // Import route-policy to verify the registration exists
-    const { getPolicy } = await import("../runtime/auth/route-policy.js");
-    const policy = getPolicy("migrations/validate");
+    const { ROUTES } = await import("../runtime/routes/index.js");
+    const policy = (ROUTES.find((r) => r.endpoint === "migrations/validate")?.policy ?? null);
 
-    expect(policy).toBeDefined();
+    expect(policy).not.toBeNull();
     expect(policy?.requiredScopes).toContain("settings.read");
     expect(policy?.allowedPrincipalTypes).toContain("actor");
     expect(policy?.allowedPrincipalTypes).toContain("svc_gateway");

package/src/__tests__/native-web-search.test.ts

@@ -324,7 +324,7 @@ describe("Native Web Search — Tool Filtering", () => {
       useNativeWebSearch: true,
     });
 
-    await provider.sendMessage([userMsg("Hi")], sampleTools);
+    await provider.sendMessage([userMsg("Hi")], { tools: sampleTools });
 
     const tools = lastStreamParams!.tools as Array<Record<string, unknown>>;
 
@@ -351,7 +351,7 @@ describe("Native Web Search — Tool Filtering", () => {
       useNativeWebSearch: false,
     });
 
-    await provider.sendMessage([userMsg("Hi")], sampleTools);
+    await provider.sendMessage([userMsg("Hi")], { tools: sampleTools });
 
     const tools = lastStreamParams!.tools as Array<Record<string, unknown>>;
 
@@ -385,7 +385,9 @@ describe("Native Web Search — Tool Filtering", () => {
       },
     ];
 
-    await provider.sendMessage([userMsg("Hi")], toolsWithoutWebSearch);
+    await provider.sendMessage([userMsg("Hi")], {
+      tools: toolsWithoutWebSearch,
+    });
 
     const tools = lastStreamParams!.tools as Array<Record<string, unknown>>;
     expect(tools).toHaveLength(1);
@@ -398,7 +400,7 @@ describe("Native Web Search — Tool Filtering", () => {
       useNativeWebSearch: true,
     });
 
-    await provider.sendMessage([userMsg("Hi")], sampleTools);
+    await provider.sendMessage([userMsg("Hi")], { tools: sampleTools });
 
     const tools = lastStreamParams!.tools as Array<{
       name: string;
@@ -445,14 +447,10 @@ describe("Native Web Search — Streaming Events", () => {
     ];
 
     const events: ProviderEvent[] = [];
-    await provider.sendMessage(
-      [userMsg("Search something")],
-      sampleTools,
-      undefined,
-      {
-        onEvent: (event) => events.push(event),
-      },
-    );
+    await provider.sendMessage([userMsg("Search something")], {
+      tools: sampleTools,
+      onEvent: (event) => events.push(event),
+    });
 
     const serverToolEvents = events.filter(
       (e) => e.type === "server_tool_start",
@@ -484,14 +482,10 @@ describe("Native Web Search — Streaming Events", () => {
     ];
 
     const events: ProviderEvent[] = [];
-    await provider.sendMessage(
-      [userMsg("Read a file")],
-      sampleTools,
-      undefined,
-      {
-        onEvent: (event) => events.push(event),
-      },
-    );
+    await provider.sendMessage([userMsg("Read a file")], {
+      tools: sampleTools,
+      onEvent: (event) => events.push(event),
+    });
 
     const serverToolEvents = events.filter(
       (e) => e.type === "server_tool_start",

package/src/__tests__/notification-decision-identity.test.ts

@@ -107,12 +107,9 @@ describe("identity context in notification decision engine", () => {
     mockIdentityContext = "I am Jarvis, a helpful assistant";
 
     configuredProvider = {
-      sendMessage: async (
-        _messages: unknown,
-        _tools: unknown,
-        systemPrompt: unknown,
-      ) => {
-        capturedSystemPrompt = systemPrompt as string;
+      sendMessage: async (...args: unknown[]) => {
+        const options = args[1] as { systemPrompt?: string } | undefined;
+        capturedSystemPrompt = options?.systemPrompt;
         return { content: [] };
       },
     };
@@ -146,12 +143,9 @@ describe("identity context in notification decision engine", () => {
     mockIdentityContext = null;
 
     configuredProvider = {
-      sendMessage: async (
-        _messages: unknown,
-        _tools: unknown,
-        systemPrompt: unknown,
-      ) => {
-        capturedSystemPrompt = systemPrompt as string;
+      sendMessage: async (...args: unknown[]) => {
+        const options = args[1] as { systemPrompt?: string } | undefined;
+        capturedSystemPrompt = options?.systemPrompt;
         return { content: [] };
       },
     };
@@ -185,12 +179,9 @@ describe("identity context in notification decision engine", () => {
     mockIdentityContext = "A".repeat(3000);
 
     configuredProvider = {
-      sendMessage: async (
-        _messages: unknown,
-        _tools: unknown,
-        systemPrompt: unknown,
-      ) => {
-        capturedSystemPrompt = systemPrompt as string;
+      sendMessage: async (...args: unknown[]) => {
+        const options = args[1] as { systemPrompt?: string } | undefined;
+        capturedSystemPrompt = options?.systemPrompt;
         return { content: [] };
       },
     };

package/src/__tests__/notification-decision-recipient-context.test.ts

@@ -104,12 +104,9 @@ function makeSignal(
 
 function setupLLMProvider() {
   configuredProvider = {
-    sendMessage: async (
-      _messages: unknown,
-      _tools: unknown,
-      systemPrompt: unknown,
-    ) => {
-      capturedSystemPrompt = systemPrompt as string;
+    sendMessage: async (...args: unknown[]) => {
+      const options = args[1] as { systemPrompt?: string } | undefined;
+      capturedSystemPrompt = options?.systemPrompt;
       return { content: [] };
     },
   };

package/src/__tests__/oauth-commands-routes.test.ts

@@ -248,7 +248,7 @@ describe("oauth-commands-routes route registry", () => {
 
   test("every route enforces policy", () => {
     for (const route of ROUTES) {
-      expect(route.requirePolicyEnforcement).toBe(true);
+      expect(route.policy).not.toBeNull();
     }
   });
 });

package/src/__tests__/onboarding-template-contract.test.ts

@@ -69,6 +69,16 @@ describe("onboarding template contracts", () => {
       expect(bootstrap).toContain("One-shot");
     });
 
+    test("offers assistant migration during low-signal first openings", () => {
+      expect(bootstrap).toContain("## Assistant migration");
+      expect(bootstrap).toContain(
+        "If the first real user turn is only a greeting",
+      );
+      expect(bootstrap).toContain("that first response is the natural opening");
+      expect(bootstrap).toContain("assistant-migration");
+      expect(bootstrap).toContain("do not load or activate");
+    });
+
     test("does not contain personality quiz references", () => {
       expect(bootstrap).not.toMatch(/show.*personality quiz/i);
       expect(bootstrap).not.toMatch(/present.*personality quiz/i);

package/src/__tests__/openai-provider.test.ts

@@ -379,8 +379,6 @@ describe("OpenAIProvider", () => {
     const events: ProviderEvent[] = [];
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { onEvent: (e) => events.push(e) },
     );
 
@@ -422,7 +420,7 @@ describe("OpenAIProvider", () => {
     ];
 
     const events: ProviderEvent[] = [];
-    await provider.sendMessage([userMsg("Hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("Hi")], {
       onEvent: (e) => events.push(e),
     });
 
@@ -470,14 +468,9 @@ describe("OpenAIProvider", () => {
     ];
 
     const events: ProviderEvent[] = [];
-    const result = await provider.sendMessage(
-      [userMsg("Hi")],
-      undefined,
-      undefined,
-      {
-        onEvent: (e) => events.push(e),
-      },
-    );
+    const result = await provider.sendMessage([userMsg("Hi")], {
+      onEvent: (e) => events.push(e),
+    });
 
     expect(result.content).toHaveLength(2);
     expect(result.content[0]).toEqual({
@@ -504,14 +497,9 @@ describe("OpenAIProvider", () => {
     ];
 
     const events: ProviderEvent[] = [];
-    const result = await provider.sendMessage(
-      [userMsg("Hi")],
-      undefined,
-      undefined,
-      {
-        onEvent: (e) => events.push(e),
-      },
-    );
+    const result = await provider.sendMessage([userMsg("Hi")], {
+      onEvent: (e) => events.push(e),
+    });
 
     expect(result.content).toHaveLength(2);
     expect(result.content[0]).toEqual({
@@ -544,8 +532,7 @@ describe("OpenAIProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      "You are a helpful assistant.",
+      { systemPrompt: "You are a helpful assistant." },
     );
 
     const messages = lastCreateParams!.messages as Array<
@@ -578,7 +565,7 @@ describe("OpenAIProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Read /tmp/test" }] }],
-      tools,
+      { tools: tools },
     );
 
     const sentTools = lastCreateParams!.tools as Array<Record<string, unknown>>;
@@ -903,8 +890,6 @@ describe("OpenAIProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { max_tokens: 64000 } },
     );
 
@@ -947,8 +932,6 @@ describe("OpenAIProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { signal: controller.signal },
     );
 
@@ -967,8 +950,6 @@ describe("OpenAIProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { signal: controller.signal },
     );
 
@@ -1024,8 +1005,6 @@ describe("OpenAIProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false); // should not reach
@@ -1044,8 +1023,6 @@ describe("OpenAIProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false);
@@ -1063,8 +1040,6 @@ describe("OpenAIProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false);
@@ -1412,7 +1387,7 @@ describe("effort config passthrough", () => {
   ): Provider {
     return {
       name,
-      async sendMessage(_messages, _tools, _systemPrompt, options) {
+      async sendMessage(_messages: Message[], options?: SendMessageOptions) {
         onCall(options);
         return makeResponse();
       },
@@ -1426,7 +1401,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: { effort: "high" },
     });
 
@@ -1441,7 +1416,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: { effort: "medium" },
     });
 
@@ -1456,7 +1431,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: { effort: "low" },
     });
 
@@ -1471,7 +1446,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: { effort: "high" },
     });
 
@@ -1486,7 +1461,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: { effort: "high" },
     });
 
@@ -1501,7 +1476,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: {
         thinking: { enabled: true, budgetTokens: 10000 },
         effort: "high",
@@ -1520,7 +1495,7 @@ describe("effort config passthrough", () => {
     });
     const retry = new RetryProvider(inner);
 
-    await retry.sendMessage(DUMMY_MESSAGES, undefined, undefined, {
+    await retry.sendMessage(DUMMY_MESSAGES, {
       config: {
         thinking: { type: "adaptive" },
       },
@@ -1546,7 +1521,7 @@ describe("OpenRouterProvider reasoning", () => {
 
   test("sends reasoning.enabled=true with default detailed summary when thinking config is present", async () => {
     const provider = new OpenRouterProvider("or-key", "x-ai/grok-4");
-    await provider.sendMessage([userMsg("hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("hi")], {
       config: { thinking: { type: "adaptive" } },
     });
 
@@ -1559,7 +1534,7 @@ describe("OpenRouterProvider reasoning", () => {
 
   test("omits reasoning when thinking is explicitly disabled", async () => {
     const provider = new OpenRouterProvider("or-key", "x-ai/grok-4");
-    await provider.sendMessage([userMsg("hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("hi")], {
       config: { thinking: { type: "disabled" } },
     });
 
@@ -1569,7 +1544,7 @@ describe("OpenRouterProvider reasoning", () => {
 
   test("omits reasoning when thinking config is absent", async () => {
     const provider = new OpenRouterProvider("or-key", "x-ai/grok-4");
-    await provider.sendMessage([userMsg("hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("hi")], {
       config: {},
     });
 
@@ -1587,7 +1562,7 @@ describe("OpenRouterProvider reasoning", () => {
 
   test("sends OpenRouter app-attribution headers on OpenAI-compatible requests", async () => {
     const provider = new OpenRouterProvider("or-key", "x-ai/grok-4");
-    await provider.sendMessage([userMsg("hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("hi")], {
       config: {
         usageAttributionHeaders: {
           "Vellum-Organization-Id": "org-123",
@@ -1614,7 +1589,7 @@ describe("OpenRouterProvider reasoning", () => {
     const retry = new RetryProvider(provider);
 
     // thinking enabled at loop-level → config.thinking set
-    await retry.sendMessage([userMsg("hi")], undefined, undefined, {
+    await retry.sendMessage([userMsg("hi")], {
       config: { thinking: { type: "adaptive" } },
     });
     expect(lastCreateParams!.reasoning).toEqual({
@@ -1627,7 +1602,7 @@ describe("OpenRouterProvider reasoning", () => {
     const provider = new OpenRouterProvider("or-key", "x-ai/grok-4");
     const retry = new RetryProvider(provider);
 
-    await retry.sendMessage([userMsg("hi")], undefined, undefined, {
+    await retry.sendMessage([userMsg("hi")], {
       config: { thinking: { type: "disabled" } },
     });
     expect(lastCreateParams!.reasoning).toBeUndefined();
@@ -1635,7 +1610,7 @@ describe("OpenRouterProvider reasoning", () => {
 
   test("nests effort under reasoning and omits top-level reasoning_effort", async () => {
     const provider = new OpenRouterProvider("or-key", "moonshotai/kimi-k2.6");
-    await provider.sendMessage([userMsg("hi")], undefined, undefined, {
+    await provider.sendMessage([userMsg("hi")], {
       config: { thinking: { enabled: true }, effort: "max" },
     });
 
@@ -1709,7 +1684,8 @@ describe("OpenAIProvider reasoning_effort", () => {
   });
 
   test('effort: "low" maps to reasoning_effort: "low"', async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "low" },
     });
     expect(lastCreateParams).toBeTruthy();
@@ -1717,35 +1693,40 @@ describe("OpenAIProvider reasoning_effort", () => {
   });
 
   test('effort: "medium" maps to reasoning_effort: "medium"', async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "medium" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("medium");
   });
 
   test('effort: "high" maps to reasoning_effort: "high"', async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "high" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
   });
 
   test('effort: "max" maps to reasoning_effort: "xhigh"', async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("xhigh");
   });
 
   test('effort: "xhigh" maps to reasoning_effort: "xhigh"', async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "xhigh" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("xhigh");
   });
 
   test("no effort config means no reasoning_effort in params", async () => {
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: {},
     });
     expect(lastCreateParams).toBeTruthy();
@@ -1756,7 +1737,8 @@ describe("OpenAIProvider reasoning_effort", () => {
     // OpenAI defaults `reasoning_effort` to "medium" when the field is
     // omitted, so the user's opt-out is only honored when we send the
     // explicit "none" value on the wire.
-    await provider.sendMessage([userMsg("hi")], undefined, "system", {
+    await provider.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "none" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("none");
@@ -1766,7 +1748,8 @@ describe("OpenAIProvider reasoning_effort", () => {
     const providerWithExtra = new OpenAIProvider("test-key", "gpt-5", {
       extraCreateParams: { reasoning_effort: "medium" },
     });
-    await providerWithExtra.sendMessage([userMsg("hi")], undefined, "system", {
+    await providerWithExtra.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: {},
     });
     expect(lastCreateParams!.reasoning_effort).toBe("medium");
@@ -1776,15 +1759,18 @@ describe("OpenAIProvider reasoning_effort", () => {
     const capped = new OpenAIProvider("test-key", "gpt-5", {
       maxReasoningEffort: "high",
     });
-    await capped.sendMessage([userMsg("hi")], undefined, "system", {
+    await capped.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "xhigh" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
-    await capped.sendMessage([userMsg("hi")], undefined, "system", {
+    await capped.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
-    await capped.sendMessage([userMsg("hi")], undefined, "system", {
+    await capped.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "medium" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("medium");
@@ -1794,11 +1780,13 @@ describe("OpenAIProvider reasoning_effort", () => {
     const uncapped = new OpenAIProvider("test-key", "gpt-5", {
       maxReasoningEffort: "max",
     });
-    await uncapped.sendMessage([userMsg("hi")], undefined, "system", {
+    await uncapped.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("max");
-    await uncapped.sendMessage([userMsg("hi")], undefined, "system", {
+    await uncapped.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "xhigh" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("xhigh");
@@ -1816,7 +1804,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/deepseek-v4-pro",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("max");
@@ -1827,7 +1816,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/deepseek-v4-pro",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "xhigh" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("xhigh");
@@ -1838,11 +1828,13 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/kimi-k2p6",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "xhigh" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
@@ -1853,7 +1845,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/some-future-model",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "max" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("high");
@@ -1864,7 +1857,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/deepseek-v4-pro",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "medium" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("medium");
@@ -1875,7 +1869,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/kimi-k2p6",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: { effort: "none" },
     });
     expect(lastCreateParams!.reasoning_effort).toBe("none");
@@ -1886,7 +1881,8 @@ describe("FireworksProvider reasoning_effort ceiling", () => {
       "fw-key",
       "accounts/fireworks/models/kimi-k2p6",
     );
-    await fw.sendMessage([userMsg("hi")], undefined, "system", {
+    await fw.sendMessage([userMsg("hi")], {
+      systemPrompt: "system",
       config: {
         effort: "max",
         model: "accounts/fireworks/models/deepseek-v4-pro",