@vellumai/assistant 0.8.6 → 0.8.7

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({

package/src/__tests__/overflow-reduce-pipeline.test.ts

@@ -33,10 +33,8 @@ import {
 } from "../daemon/context-overflow-reducer.js";
 import type { InjectionMode } from "../daemon/conversation-runtime-assembly.js";
 import type { TrustContext } from "../daemon/trust-context.js";
-import {
-  defaultOverflowReduceMiddleware,
-  defaultOverflowReducePlugin,
-} from "../plugins/defaults/overflow-reduce.js";
+import defaultOverflowReduceMiddleware from "../plugins/defaults/overflow-reduce/middlewares/overflowReduce.js";
+import { defaultOverflowReducePlugin } from "../plugins/defaults/overflow-reduce/register.js";
 import { runPipeline } from "../plugins/pipeline.js";
 import {
   getMiddlewaresFor,

package/src/__tests__/parallel-tool.benchmark.test.ts

@@ -34,11 +34,13 @@ function createMockProvider(responses: ProviderResponse[]): {
     name: "mock",
     async sendMessage(
       messages: Message[],
-      tools?: ToolDefinition[],
-      systemPrompt?: string,
       options?: SendMessageOptions,
     ): Promise<ProviderResponse> {
-      calls.push({ messages: [...messages], tools, systemPrompt });
+      calls.push({
+        messages: [...messages],
+        tools: options?.tools,
+        systemPrompt: options?.systemPrompt,
+      });
       const response = responses[callIndex] ?? responses[responses.length - 1];
       callIndex++;
 
@@ -126,13 +128,10 @@ describe("Parallel tool execution benchmarks", () => {
       return { content: "ok", isError: false };
     };
 
-    const loop = new AgentLoop(
-      provider,
-      "system",
-      {},
-      dummyTools,
-      toolExecutor,
-    );
+    const loop = new AgentLoop(provider, "system", {
+      tools: dummyTools,
+      toolExecutor: toolExecutor,
+    });
     const start = Date.now();
     await loop.run([userMessage], () => {});
     const elapsed = Date.now() - start;
@@ -170,13 +169,10 @@ describe("Parallel tool execution benchmarks", () => {
       return { content: "ok", isError: false };
     };
 
-    const loop = new AgentLoop(
-      provider,
-      "system",
-      {},
-      dummyTools,
-      toolExecutor,
-    );
+    const loop = new AgentLoop(provider, "system", {
+      tools: dummyTools,
+      toolExecutor: toolExecutor,
+    });
     const start = Date.now();
     await loop.run([userMessage], () => {});
     const elapsed = Date.now() - start;
@@ -226,13 +222,10 @@ describe("Parallel tool execution benchmarks", () => {
       return { content: "ok", isError: false };
     };
 
-    const loop = new AgentLoop(
-      provider,
-      "system",
-      {},
-      dummyTools,
-      toolExecutor,
-    );
+    const loop = new AgentLoop(provider, "system", {
+      tools: dummyTools,
+      toolExecutor: toolExecutor,
+    });
     const events: AgentEvent[] = [];
     const start = Date.now();
     await loop.run([userMessage], collectEvents(events));
@@ -296,19 +289,14 @@ describe("Parallel tool execution benchmarks", () => {
         return { content: "should not return", isError: false };
       };
 
-      const loop = new AgentLoop(
-        provider,
-        "system",
-        {},
-        dummyTools,
-        toolExecutor,
-      );
+      const loop = new AgentLoop(provider, "system", {
+        tools: dummyTools,
+        toolExecutor: toolExecutor,
+      });
       const start = Date.now();
-      const history = await loop.run(
-        [userMessage],
-        () => {},
-        controller.signal,
-      );
+      const { history } = await loop.run([userMessage], () => {}, {
+        signal: controller.signal,
+      });
       const elapsed = Date.now() - start;
 
       // Should exit quickly after the 50ms abort, not wait 500ms

package/src/__tests__/persistence-pipeline.test.ts

@@ -31,10 +31,8 @@ import {
 } from "../memory/conversation-crud.js";
 import { getDb } from "../memory/db-connection.js";
 import { initializeDb } from "../memory/db-init.js";
-import {
-  defaultPersistencePlugin,
-  defaultPersistenceTerminal,
-} from "../plugins/defaults/persistence.js";
+import { defaultPersistencePlugin } from "../plugins/defaults/persistence/register.js";
+import { defaultPersistenceTerminal } from "../plugins/defaults/persistence/terminal.js";
 import { DEFAULT_TIMEOUTS, runPipeline } from "../plugins/pipeline.js";
 import {
   getMiddlewaresFor,
@@ -101,13 +99,10 @@ describe("persistence pipeline", () => {
     const conv = createConversation();
 
     // Baseline: what the direct call produces.
-    const direct = await addMessage(
-      conv.id,
-      "user",
-      "direct-content",
-      { role: "baseline" },
-      { skipIndexing: true },
-    );
+    const direct = await addMessage(conv.id, "user", "direct-content", {
+      metadata: { role: "baseline" },
+      skipIndexing: true,
+    });
     const directRow = getMessageById(direct.id, conv.id);
 
     // Through the pipeline: must produce a row with identical columns
@@ -146,13 +141,10 @@ describe("persistence pipeline", () => {
     registerPlugin(defaultPersistencePlugin);
 
     const conv = createConversation();
-    const msg = await addMessage(
-      conv.id,
-      "user",
-      "to-update",
-      { initial: true },
-      { skipIndexing: true },
-    );
+    const msg = await addMessage(conv.id, "user", "to-update", {
+      metadata: { initial: true },
+      skipIndexing: true,
+    });
 
     // Through the pipeline: result is the `update` envelope (no payload).
     const result = await runPipeline<PersistArgs, PersistResult>(
@@ -170,13 +162,10 @@ describe("persistence pipeline", () => {
     expect(result).toEqual({ op: "update" });
 
     // Direct equivalent call on a second message for parity comparison.
-    const baseline = await addMessage(
-      conv.id,
-      "user",
-      "to-update-direct",
-      { initial: true },
-      { skipIndexing: true },
-    );
+    const baseline = await addMessage(conv.id, "user", "to-update-direct", {
+      metadata: { initial: true },
+      skipIndexing: true,
+    });
     updateMessageMetadata(baseline.id, { extra: "added" });
 
     const pipelineRow = getMessageById(msg.id, conv.id);
@@ -196,7 +185,7 @@ describe("persistence pipeline", () => {
     registerPlugin(defaultPersistencePlugin);
 
     const conv = createConversation();
-    const msg = await addMessage(conv.id, "user", "to-delete", undefined, {
+    const msg = await addMessage(conv.id, "user", "to-delete", {
       skipIndexing: true,
     });
 

package/src/__tests__/persistence-secret-redaction.test.ts

@@ -52,8 +52,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 });
     return { id };

package/src/__tests__/pipeline-runner.test.ts

@@ -235,12 +235,12 @@ describe("runPipeline — timeout", () => {
       value: args.value,
     });
     const result = await runPipeline(
-      "historyRepair",
+      "tokenEstimate",
       [],
       terminal,
       { value: 42 },
       makeCtx(),
-      DEFAULT_TIMEOUTS.historyRepair,
+      DEFAULT_TIMEOUTS.tokenEstimate,
     );
     expect(result).toEqual({ value: 42 });
   });
@@ -550,7 +550,6 @@ describe("DEFAULT_TIMEOUTS", () => {
       llmCall: null,
       toolExecute: null,
       memoryRetrieval: null,
-      historyRepair: null,
       tokenEstimate: null,
       compaction: null,
       overflowReduce: null,