@vellumai/assistant 0.4.30 → 0.4.31

package/src/__tests__/memory-lifecycle-e2e.test.ts

@@ -135,7 +135,6 @@ const TEST_CONFIG = {
       enabled: true,
       gateMode: "soft" as const,
       relevanceThreshold: 0.2,
-      reaskCooldownTurns: 3,
       resolverLlmTimeoutMs: 250,
     },
     profile: {
@@ -397,7 +396,7 @@ describe("Memory lifecycle E2E regression", () => {
           confidence: 0.88,
           importance: 0.8,
           fingerprint: "fp-item-ui-existing",
-          verificationState: "assistant_inferred",
+          verificationState: "user_reported",
           scopeId: "default",
           firstSeenAt: now + 30,
           lastSeenAt: now + 30,
@@ -413,7 +412,7 @@ describe("Memory lifecycle E2E regression", () => {
           confidence: 0.84,
           importance: 0.8,
           fingerprint: "fp-item-ui-candidate",
-          verificationState: "assistant_inferred",
+          verificationState: "user_reported",
           scopeId: "default",
           firstSeenAt: now + 31,
           lastSeenAt: now + 31,
@@ -432,23 +431,25 @@ describe("Memory lifecycle E2E regression", () => {
     });
 
     const conflictGate = new ConflictGate();
-    const firstGateDecision = await conflictGate.evaluate(
+
+    // First evaluation: conflict remains pending; evaluate returns void (no
+    // user-facing output — conflict handling is fully internal)
+    const firstResult = await conflictGate.evaluate(
       "Need react roadmap update today",
       TEST_CONFIG.memory.conflicts,
     );
-    expect(firstGateDecision).not.toBeNull();
-    expect(firstGateDecision?.relevant).toBe(true);
-    expect(firstGateDecision?.question).toContain("React");
+    expect(firstResult).toBeUndefined();
 
     const pendingAfterFirstGate = getConflictById(gatedConflict.id);
     expect(pendingAfterFirstGate?.status).toBe("pending_clarification");
-    expect(pendingAfterFirstGate?.lastAskedAt).not.toBeNull();
 
-    const secondGateDecision = await conflictGate.evaluate(
+    // Second evaluation: clarification-like reply resolves the conflict
+    // internally; still no user-facing prompt is produced
+    const secondResult = await conflictGate.evaluate(
       "Use the new renderer going forward.",
       TEST_CONFIG.memory.conflicts,
     );
-    expect(secondGateDecision).toBeNull();
+    expect(secondResult).toBeUndefined();
 
     const resolvedAfterSecondGate = getConflictById(gatedConflict.id);
     expect(resolvedAfterSecondGate?.status).toBe("resolved_keep_candidate");

package/src/__tests__/registry.test.ts

@@ -255,16 +255,6 @@ describe("baseline characterization: hardcoded tool loading", () => {
     }
   });
 
-  test("weather tool is NOT in global registry after initializeTools()", async () => {
-    await initializeTools();
-    const tool = getTool("get_weather");
-    expect(tool).toBeUndefined();
-  });
-
-  test("weather tool name is NOT in eagerModuleToolNames manifest", () => {
-    expect(eagerModuleToolNames).not.toContain("get_weather");
-  });
-
   test("claude_code is NOT in global registry after initializeTools()", async () => {
     await initializeTools();
     const tool = getTool("claude_code");

package/src/__tests__/script-proxy-session-runtime.test.ts

@@ -1,7 +1,12 @@
 import { createServer, request as httpRequest, type Server } from "node:http";
 import { afterEach, describe, expect, test } from "bun:test";
 
-import { createSession, startSession, stopAllSessions, stopSession } from '../tools/network/script-proxy/session-manager.js';
+import {
+  createSession,
+  startSession,
+  stopAllSessions,
+  stopSession,
+} from "../tools/network/script-proxy/session-manager.js";
 
 let upstreamServer: Server | null = null;
 

package/src/__tests__/session-agent-loop.test.ts

@@ -189,9 +189,7 @@ mock.module("../daemon/session-memory.js", () => ({
       latencyMs: 0,
     },
     dynamicProfile: { text: "" },
-    softConflictInstruction: null,
     recallInjectionStrategy: "prepend_user_block" as const,
-    conflictClarification: null,
   }),
 }));