@vellumai/assistant 0.8.6 → 0.8.7-dev.202606052118.34cd356

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

@@ -291,8 +291,6 @@ describe("OpenAIResponsesProvider", () => {
     const events: ProviderEvent[] = [];
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { onEvent: (e) => events.push(e) },
     );
 
@@ -309,8 +307,7 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      "You are a helpful assistant.",
+      { systemPrompt: "You are a helpful assistant." },
     );
 
     expect(lastStreamParams!.instructions).toBe("You are a helpful assistant.");
@@ -342,7 +339,7 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Read /tmp/test" }] }],
-      tools,
+      { tools: tools },
     );
 
     const sentTools = lastStreamParams!.tools as Array<Record<string, unknown>>;
@@ -547,8 +544,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { max_tokens: 64000 } },
     );
 
@@ -563,8 +558,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { effort: "high" } },
     );
 
@@ -576,8 +569,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { effort: "max" } },
     );
 
@@ -589,8 +580,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { effort: "xhigh" } },
     );
 
@@ -602,8 +591,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: {} },
     );
 
@@ -618,8 +605,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { effort: "none" } },
     );
 
@@ -634,8 +619,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "low" } },
     );
 
@@ -647,8 +630,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "high" } },
     );
 
@@ -660,8 +641,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: {} },
     );
 
@@ -672,10 +651,7 @@ describe("OpenAIResponsesProvider", () => {
     fakeStreamEvents = [textDeltaEvent("OK"), completedEvent(10, 2)];
 
     await provider.sendMessage(
-      [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
-      // Cast through unknown — the test deliberately exercises the runtime
+      [{ role: "user", content: [{ type: "text", text: "Hi" }] }], // Cast through unknown — the test deliberately exercises the runtime
       // guard against malformed values that would bypass the type system
       // (e.g. arriving via the index signature on `SendMessageConfig`).
       { config: { verbosity: "bogus" as unknown as "low" } },
@@ -692,8 +668,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await oSeriesProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "high" } },
     );
 
@@ -706,8 +680,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await oSeriesProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "high", model: "gpt-5.2" } },
     );
 
@@ -723,8 +695,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await ftProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "low" } },
     );
 
@@ -742,8 +712,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { model: "gpt-5.4" } },
     );
 
@@ -986,8 +954,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { signal: controller.signal },
     );
 
@@ -1003,8 +969,6 @@ describe("OpenAIResponsesProvider", () => {
 
     await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { signal: controller.signal },
     );
 
@@ -1076,8 +1040,6 @@ describe("OpenAIResponsesProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false);
@@ -1099,8 +1061,6 @@ describe("OpenAIResponsesProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false);
@@ -1118,8 +1078,6 @@ describe("OpenAIResponsesProvider", () => {
     try {
       await provider.sendMessage(
         [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-        undefined,
-        undefined,
         { signal: controller.signal },
       );
       expect(true).toBe(false);
@@ -1151,8 +1109,7 @@ describe("OpenAIResponsesProvider", () => {
 
     const result = await provider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      "System prompt",
+      { systemPrompt: "System prompt" },
     );
 
     // rawRequest should contain the params sent to responses.create()
@@ -1363,7 +1320,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for cats" }] }],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     const sentTools = lastStreamParams!.tools as Array<Record<string, unknown>>;
@@ -1377,7 +1334,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await nonNativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for cats" }] }],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     const sentTools = lastStreamParams!.tools as Array<Record<string, unknown>>;
@@ -1402,7 +1359,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search and read" }] }],
-      [fileReadTool, webSearchTool],
+      { tools: [fileReadTool, webSearchTool] },
     );
 
     const sentTools = lastStreamParams!.tools as Array<Record<string, unknown>>;
@@ -1431,7 +1388,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Read file" }] }],
-      [fileReadTool],
+      { tools: [fileReadTool] },
     );
 
     const sentTools = lastStreamParams!.tools as Array<Record<string, unknown>>;
@@ -1466,9 +1423,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
     const events: ProviderEvent[] = [];
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for cats" }] }],
-      [webSearchTool],
-      undefined,
-      { onEvent: (e) => events.push(e) },
+      { tools: [webSearchTool], onEvent: (e) => events.push(e) },
     );
 
     const startEvents = events.filter((e) => e.type === "server_tool_start");
@@ -1494,9 +1449,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
     const events: ProviderEvent[] = [];
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for dogs" }] }],
-      [webSearchTool],
-      undefined,
-      { onEvent: (e) => events.push(e) },
+      { tools: [webSearchTool], onEvent: (e) => events.push(e) },
     );
 
     const completeEvents = events.filter(
@@ -1524,9 +1477,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
     const events: ProviderEvent[] = [];
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search multiple" }] }],
-      [webSearchTool],
-      undefined,
-      { onEvent: (e) => events.push(e) },
+      { tools: [webSearchTool], onEvent: (e) => events.push(e) },
     );
 
     const startEvents = events.filter((e) => e.type === "server_tool_start");
@@ -1574,9 +1525,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
     const events: ProviderEvent[] = [];
     await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Read file" }] }],
-      [fileReadTool],
-      undefined,
-      { onEvent: (e) => events.push(e) },
+      { tools: [fileReadTool], onEvent: (e) => events.push(e) },
     );
 
     const serverToolEvents = events.filter(
@@ -1602,7 +1551,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     const result = await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for cats" }] }],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     // server_tool_use + web_search_tool_result pair should appear before text
@@ -1642,7 +1591,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
           content: [{ type: "text", text: "Search two things" }],
         },
       ],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     expect(result.content).toHaveLength(5);
@@ -1685,7 +1634,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     const result = await nativeProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hello" }] }],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     expect(result.content).toHaveLength(1);
@@ -1706,8 +1655,6 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { max_tokens: 64000 } },
     );
 
@@ -1722,8 +1669,6 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { effort: "high" } },
     );
 
@@ -1738,8 +1683,6 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      undefined,
       { config: { verbosity: "low" } },
     );
 
@@ -1769,7 +1712,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
     };
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      [sampleTool],
+      { tools: [sampleTool] },
     );
 
     expect(lastStreamParams!.tools).toEqual([
@@ -1792,7 +1735,7 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Search for cats" }] }],
-      [webSearchTool],
+      { tools: [webSearchTool] },
     );
 
     expect(lastStreamParams!.tools).toEqual([
@@ -1808,9 +1751,10 @@ describe("OpenAIResponsesProvider — Native Web Search", () => {
 
     await codexProvider.sendMessage(
       [{ role: "user", content: [{ type: "text", text: "Hi" }] }],
-      undefined,
-      "You are helpful.",
-      { config: { effort: "max", verbosity: "high", max_tokens: 64000 } },
+      {
+        systemPrompt: "You are helpful.",
+        config: { effort: "max", verbosity: "high", max_tokens: 64000 },
+      },
     );
 
     expect(lastStreamParams!.model).toBe("gpt-5.4");

package/src/__tests__/outbound-slack-persistence.test.ts

@@ -76,8 +76,9 @@ mock.module("../memory/conversation-crud.js", () => ({
     conversationId: string,
     role: string,
     content: string,
-    metadata?: Record<string, unknown>,
+    options?: { metadata?: Record<string, unknown> },
   ) => {
+    const metadata = options?.metadata;
     const id = `mock-msg-${addMessageCalls.length + 1}`;
     addMessageCalls.push({ id, conversationId, role, content, metadata });
     persistedRows.push({