@vellumai/assistant 0.10.2-dev.202606250318.5e7cfb0 → 0.10.2

package/src/__tests__/channel-guardian.test.ts

@@ -83,42 +83,6 @@ mock.module("../ipc/gateway-client.js", () => ({
   }),
 }));
 
-// Guardian-delivery reader mock — the inbound challenge guard reads guardian
-// existence from the gateway. Derive the list from the local binding state so
-// the gateway-backed presence guard mirrors the DB the rest of the test sets up.
-const resolveGuardianList = async (input?: { channelTypes?: string[] }) => {
-  const { findGuardianForChannel } = await import(
-    "../contacts/contact-store.js"
-  );
-  const channels = input?.channelTypes ?? [];
-  return channels
-    .map((channelType) => {
-      const found = findGuardianForChannel(channelType);
-      if (!found) return null;
-      return {
-        channelType,
-        contactId: found.contact.id,
-        principalId: found.contact.principalId ?? null,
-        displayName: found.contact.displayName ?? null,
-        address: found.channel.address,
-        externalChatId: found.channel.externalChatId ?? null,
-        status: "active",
-        verifiedAt: found.channel.verifiedAt ?? null,
-      };
-    })
-    .filter((g) => g !== null);
-};
-
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: resolveGuardianList,
-  getGuardianDeliveryFresh: resolveGuardianList,
-  guardianForChannel: (
-    list: Array<{ channelType: string; status: string }>,
-    channelType: string,
-  ) =>
-    list.find((g) => g.channelType === channelType && g.status === "active"),
-}));
-
 import { handleChannelVerificationSession } from "../daemon/handlers/config-channels.js";
 import type {
   ChannelVerificationSessionRequest,
@@ -355,12 +319,12 @@ describe("guardian service challenge validation", () => {
     }
   });
 
-  test("validateAndConsumeVerification does not create a guardian binding (caller responsibility)", async () => {
+  test("validateAndConsumeVerification does not create a guardian binding (caller responsibility)", () => {
     const { secret } = createInboundVerificationSession("telegram");
 
     validateAndConsumeVerification("telegram", secret, "user-42", "chat-42");
 
-    const binding = await getGuardianBinding("asst-1", "telegram");
+    const binding = getGuardianBinding("asst-1", "telegram");
     expect(binding).toBeNull();
   });
 
@@ -432,7 +396,7 @@ describe("guardian service challenge validation", () => {
     expect(result2.success).toBe(false);
   });
 
-  test("validateAndConsumeVerification succeeds with voice channel", async () => {
+  test("validateAndConsumeVerification succeeds with voice channel", () => {
     const { secret } = createInboundVerificationSession("phone");
 
     const result = validateAndConsumeVerification(
@@ -449,7 +413,7 @@ describe("guardian service challenge validation", () => {
 
     // validateAndConsumeVerification no longer creates bindings — that is
     // now handled by the gateway's verification intercepts.
-    const binding = await getGuardianBinding("asst-1", "phone");
+    const binding = getGuardianBinding("asst-1", "phone");
     expect(binding).toBeNull();
   });
 
@@ -485,7 +449,7 @@ describe("guardian service challenge validation", () => {
     expect(telegramResult.success).toBe(true);
   });
 
-  test("validateAndConsumeVerification succeeds even with existing binding (conflict check is caller responsibility)", async () => {
+  test("validateAndConsumeVerification succeeds even with existing binding (conflict check is caller responsibility)", () => {
     // Create initial guardian binding
     createGuardianBinding({
       channel: "telegram",
@@ -494,7 +458,7 @@ describe("guardian service challenge validation", () => {
       guardianDeliveryChatId: "old-chat",
     });
 
-    const oldBinding = await getGuardianBinding("asst-1", "telegram");
+    const oldBinding = getGuardianBinding("asst-1", "telegram");
     expect(oldBinding).not.toBeNull();
     expect(oldBinding!.guardianExternalUserId).toBe("old-user");
 
@@ -509,7 +473,7 @@ describe("guardian service challenge validation", () => {
     // Challenge validation succeeds — the caller decides how to handle binding conflicts
     expect(result.success).toBe(true);
 
-    const binding = await getGuardianBinding("asst-1", "telegram");
+    const binding = getGuardianBinding("asst-1", "telegram");
     expect(binding).not.toBeNull();
     expect(binding!.guardianExternalUserId).toBe("old-user");
   });
@@ -524,7 +488,7 @@ describe("guardian identity check", () => {
     resetTables();
   });
 
-  test("isGuardian returns true for matching user", async () => {
+  test("isGuardian returns true for matching user", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-42",
@@ -532,10 +496,10 @@ describe("guardian identity check", () => {
       guardianDeliveryChatId: "chat-42",
     });
 
-    expect(await isGuardian("asst-1", "telegram", "user-42")).toBe(true);
+    expect(isGuardian("asst-1", "telegram", "user-42")).toBe(true);
   });
 
-  test("isGuardian returns false for non-matching user", async () => {
+  test("isGuardian returns false for non-matching user", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-42",
@@ -543,14 +507,14 @@ describe("guardian identity check", () => {
       guardianDeliveryChatId: "chat-42",
     });
 
-    expect(await isGuardian("asst-1", "telegram", "user-99")).toBe(false);
+    expect(isGuardian("asst-1", "telegram", "user-99")).toBe(false);
   });
 
-  test("isGuardian returns false when no binding exists", async () => {
-    expect(await isGuardian("asst-1", "telegram", "user-42")).toBe(false);
+  test("isGuardian returns false when no binding exists", () => {
+    expect(isGuardian("asst-1", "telegram", "user-42")).toBe(false);
   });
 
-  test("isGuardian returns false after binding is revoked", async () => {
+  test("isGuardian returns false after binding is revoked", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-42",
@@ -560,10 +524,10 @@ describe("guardian identity check", () => {
 
     serviceRevokeBinding("asst-1", "telegram");
 
-    expect(await isGuardian("asst-1", "telegram", "user-42")).toBe(false);
+    expect(isGuardian("asst-1", "telegram", "user-42")).toBe(false);
   });
 
-  test("getGuardianBinding returns the active binding", async () => {
+  test("getGuardianBinding returns the active binding", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-42",
@@ -571,17 +535,17 @@ describe("guardian identity check", () => {
       guardianDeliveryChatId: "chat-42",
     });
 
-    const binding = await getGuardianBinding("asst-1", "telegram");
+    const binding = getGuardianBinding("asst-1", "telegram");
     expect(binding).not.toBeNull();
     expect(binding!.guardianExternalUserId).toBe("user-42");
   });
 
-  test("getGuardianBinding returns null when no binding exists", async () => {
-    const binding = await getGuardianBinding("asst-1", "telegram");
+  test("getGuardianBinding returns null when no binding exists", () => {
+    const binding = getGuardianBinding("asst-1", "telegram");
     expect(binding).toBeNull();
   });
 
-  test("isGuardian works for voice channel", async () => {
+  test("isGuardian works for voice channel", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "phone-user-1",
@@ -589,13 +553,13 @@ describe("guardian identity check", () => {
       guardianDeliveryChatId: "voice-chat-1",
     });
 
-    expect(await isGuardian("asst-1", "phone", "phone-user-1")).toBe(true);
-    expect(await isGuardian("asst-1", "phone", "phone-user-2")).toBe(false);
+    expect(isGuardian("asst-1", "phone", "phone-user-1")).toBe(true);
+    expect(isGuardian("asst-1", "phone", "phone-user-2")).toBe(false);
     // Telegram guardian should not match voice channel
-    expect(await isGuardian("asst-1", "telegram", "phone-user-1")).toBe(false);
+    expect(isGuardian("asst-1", "telegram", "phone-user-1")).toBe(false);
   });
 
-  test("serviceRevokeBinding revokes the active binding", async () => {
+  test("serviceRevokeBinding revokes the active binding", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-42",
@@ -605,7 +569,7 @@ describe("guardian identity check", () => {
 
     const result = serviceRevokeBinding("asst-1", "telegram");
     expect(result).toBe(true);
-    expect(await getGuardianBinding("asst-1", "telegram")).toBeNull();
+    expect(getGuardianBinding("asst-1", "telegram")).toBeNull();
   });
 });
 
@@ -896,7 +860,7 @@ describe("channel-scoped guardian resolution", () => {
     resetTables();
   });
 
-  test("isGuardian resolves independently per channel", async () => {
+  test("isGuardian resolves independently per channel", () => {
     // Create guardian binding on telegram
     createGuardianBinding({
       channel: "telegram",
@@ -913,15 +877,15 @@ describe("channel-scoped guardian resolution", () => {
     });
 
     // user-alpha is guardian for telegram but not voice
-    expect(await isGuardian("self", "telegram", "user-alpha")).toBe(true);
-    expect(await isGuardian("self", "phone", "user-alpha")).toBe(false);
+    expect(isGuardian("self", "telegram", "user-alpha")).toBe(true);
+    expect(isGuardian("self", "phone", "user-alpha")).toBe(false);
 
     // user-beta is guardian for voice but not telegram
-    expect(await isGuardian("self", "phone", "user-beta")).toBe(true);
-    expect(await isGuardian("self", "telegram", "user-beta")).toBe(false);
+    expect(isGuardian("self", "phone", "user-beta")).toBe(true);
+    expect(isGuardian("self", "telegram", "user-beta")).toBe(false);
   });
 
-  test("getGuardianBinding returns different bindings for different channels", async () => {
+  test("getGuardianBinding returns different bindings for different channels", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-alpha",
@@ -935,8 +899,8 @@ describe("channel-scoped guardian resolution", () => {
       guardianDeliveryChatId: "chat-beta",
     });
 
-    const bindingTelegram = await getGuardianBinding("self", "telegram");
-    const bindingVoice = await getGuardianBinding("self", "phone");
+    const bindingTelegram = getGuardianBinding("self", "telegram");
+    const bindingVoice = getGuardianBinding("self", "phone");
 
     expect(bindingTelegram).not.toBeNull();
     expect(bindingVoice).not.toBeNull();
@@ -944,7 +908,7 @@ describe("channel-scoped guardian resolution", () => {
     expect(bindingVoice!.guardianExternalUserId).toBe("user-beta");
   });
 
-  test("revoking binding for one channel does not affect another", async () => {
+  test("revoking binding for one channel does not affect another", () => {
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "user-alpha",
@@ -960,11 +924,11 @@ describe("channel-scoped guardian resolution", () => {
 
     serviceRevokeBinding("self", "telegram");
 
-    expect(await getGuardianBinding("self", "telegram")).toBeNull();
-    expect(await getGuardianBinding("self", "phone")).not.toBeNull();
+    expect(getGuardianBinding("self", "telegram")).toBeNull();
+    expect(getGuardianBinding("self", "phone")).not.toBeNull();
   });
 
-  test("validateAndConsumeVerification scoped to channel", async () => {
+  test("validateAndConsumeVerification scoped to channel", () => {
     // Create challenge on telegram
     const { secret: secretTelegram } =
       createInboundVerificationSession("telegram");
@@ -997,8 +961,8 @@ describe("channel-scoped guardian resolution", () => {
     );
     expect(resultVoice.success).toBe(true);
 
-    const bindingTelegram = await getGuardianBinding("self", "telegram");
-    const bindingVoice = await getGuardianBinding("self", "phone");
+    const bindingTelegram = getGuardianBinding("self", "telegram");
+    const bindingVoice = getGuardianBinding("self", "phone");
     expect(bindingTelegram).toBeNull();
     expect(bindingVoice).toBeNull();
   });
@@ -1313,7 +1277,7 @@ describe("voice guardian challenge validation", () => {
     }
   });
 
-  test("validateAndConsumeVerification does not create a guardian binding for voice (caller responsibility)", async () => {
+  test("validateAndConsumeVerification does not create a guardian binding for voice (caller responsibility)", () => {
     const { secret } = createInboundVerificationSession("phone");
 
     validateAndConsumeVerification(
@@ -1323,7 +1287,7 @@ describe("voice guardian challenge validation", () => {
       "voice-chat-1",
     );
 
-    const binding = await getGuardianBinding("asst-1", "phone");
+    const binding = getGuardianBinding("asst-1", "phone");
     expect(binding).toBeNull();
   });
 
@@ -1397,7 +1361,7 @@ describe("voice guardian challenge validation", () => {
     expect(result2.success).toBe(false);
   });
 
-  test("validateAndConsumeVerification succeeds even with existing voice binding (conflict check is caller responsibility)", async () => {
+  test("validateAndConsumeVerification succeeds even with existing voice binding (conflict check is caller responsibility)", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "old-voice-user",
@@ -1405,7 +1369,7 @@ describe("voice guardian challenge validation", () => {
       guardianDeliveryChatId: "old-voice-chat",
     });
 
-    const oldBinding = await getGuardianBinding("asst-1", "phone");
+    const oldBinding = getGuardianBinding("asst-1", "phone");
     expect(oldBinding).not.toBeNull();
     expect(oldBinding!.guardianExternalUserId).toBe("old-voice-user");
 
@@ -1421,7 +1385,7 @@ describe("voice guardian challenge validation", () => {
     expect(result.success).toBe(true);
 
     // The original binding is untouched (no side effects)
-    const binding = await getGuardianBinding("asst-1", "phone");
+    const binding = getGuardianBinding("asst-1", "phone");
     expect(binding).not.toBeNull();
     expect(binding!.guardianExternalUserId).toBe("old-voice-user");
   });
@@ -1436,7 +1400,7 @@ describe("voice guardian identity and revocation", () => {
     resetTables();
   });
 
-  test("isGuardian works for voice channel", async () => {
+  test("isGuardian works for voice channel", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "voice-user-1",
@@ -1444,13 +1408,13 @@ describe("voice guardian identity and revocation", () => {
       guardianDeliveryChatId: "voice-chat-1",
     });
 
-    expect(await isGuardian("asst-1", "phone", "voice-user-1")).toBe(true);
-    expect(await isGuardian("asst-1", "phone", "voice-user-2")).toBe(false);
+    expect(isGuardian("asst-1", "phone", "voice-user-1")).toBe(true);
+    expect(isGuardian("asst-1", "phone", "voice-user-2")).toBe(false);
     // Voice guardian should not match telegram channel
-    expect(await isGuardian("asst-1", "telegram", "voice-user-1")).toBe(false);
+    expect(isGuardian("asst-1", "telegram", "voice-user-1")).toBe(false);
   });
 
-  test("getGuardianBinding returns voice binding", async () => {
+  test("getGuardianBinding returns voice binding", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "voice-user-1",
@@ -1458,13 +1422,13 @@ describe("voice guardian identity and revocation", () => {
       guardianDeliveryChatId: "voice-chat-1",
     });
 
-    const binding = await getGuardianBinding("asst-1", "phone");
+    const binding = getGuardianBinding("asst-1", "phone");
     expect(binding).not.toBeNull();
     expect(binding!.channel).toBe("phone");
     expect(binding!.guardianExternalUserId).toBe("voice-user-1");
   });
 
-  test("revokeBinding clears active voice guardian binding", async () => {
+  test("revokeBinding clears active voice guardian binding", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "voice-user-1",
@@ -1474,10 +1438,10 @@ describe("voice guardian identity and revocation", () => {
 
     const result = serviceRevokeBinding("asst-1", "phone");
     expect(result).toBe(true);
-    expect(await getGuardianBinding("asst-1", "phone")).toBeNull();
+    expect(getGuardianBinding("asst-1", "phone")).toBeNull();
   });
 
-  test("revokeBinding for voice does not affect telegram binding", async () => {
+  test("revokeBinding for voice does not affect telegram binding", () => {
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "voice-user-1",
@@ -1493,8 +1457,8 @@ describe("voice guardian identity and revocation", () => {
 
     serviceRevokeBinding("asst-1", "phone");
 
-    expect(await getGuardianBinding("asst-1", "phone")).toBeNull();
-    expect(await getGuardianBinding("asst-1", "telegram")).not.toBeNull();
+    expect(getGuardianBinding("asst-1", "phone")).toBeNull();
+    expect(getGuardianBinding("asst-1", "telegram")).not.toBeNull();
   });
 });
 
@@ -1752,7 +1716,7 @@ describe("HTTP handler voice guardian verification", () => {
     expect(resp!.bound).toBe(false);
 
     // Verify binding is actually revoked
-    expect(await getGuardianBinding("self", "phone")).toBeNull();
+    expect(getGuardianBinding("self", "phone")).toBeNull();
   });
 
   test("revoke for voice does not affect telegram binding", async () => {
@@ -1778,8 +1742,8 @@ describe("HTTP handler voice guardian verification", () => {
 
     await handleChannelVerificationSession(msg);
 
-    expect(await getGuardianBinding("self", "phone")).toBeNull();
-    expect(await getGuardianBinding("self", "telegram")).not.toBeNull();
+    expect(getGuardianBinding("self", "phone")).toBeNull();
+    expect(getGuardianBinding("self", "telegram")).not.toBeNull();
   });
 });
 

package/src/__tests__/channel-reply-delivery.test.ts

@@ -81,8 +81,6 @@ mock.module("../runtime/gateway-client.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   setConversationOriginChannelIfUnset: () => {},
   updateConversationContextWindow: () => {},
   deleteMessageById: () => {},

package/src/__tests__/compaction-events.test.ts

@@ -125,8 +125,6 @@ mock.module("../security/secret-allowlist.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   setConversationOriginChannelIfUnset: () => {},
   updateConversationContextWindow: () => {},
   setConversationHistoryStrippedAt: () => {},

package/src/__tests__/compaction.benchmark.test.ts

@@ -17,8 +17,6 @@ mock.module("../util/logger.js", () => ({
 // The compactor reads the conversation's image attachments from the DB to
 // build its manifest; with no images these return empty.
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
 }));
 mock.module("../memory/attachments-store.js", () => ({

package/src/__tests__/compactor-call-site-logging.test.ts

@@ -30,8 +30,6 @@ mock.module("../util/logger.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
   reserveMessage: mock(async () => ({ id: "msg-reserve" })),
 }));

package/src/__tests__/compactor-low-watermark-cut.test.ts

@@ -32,8 +32,6 @@ mock.module("../util/logger.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
 }));
 

package/src/__tests__/compactor-preserved-tail-count.test.ts

@@ -22,8 +22,6 @@ mock.module("../util/logger.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
   reserveMessage: mock(async () => ({ id: "msg-reserve" })),
 }));

package/src/__tests__/compactor-summary-call-truncation.test.ts

@@ -32,8 +32,6 @@ mock.module("../util/logger.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
 }));
 

package/src/__tests__/compactor-web-search-strip.test.ts

@@ -22,8 +22,6 @@ mock.module("../util/logger.js", () => ({
 }));
 
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getMessages: () => [],
   reserveMessage: mock(async () => ({ id: "msg-reserve" })),
 }));

package/src/__tests__/config-loader-backfill.test.ts

@@ -98,64 +98,6 @@ function writeConfig(obj: unknown): void {
   writeFileSync(CONFIG_PATH, JSON.stringify(obj, null, 2) + "\n");
 }
 
-function latencySeed(): Record<string, unknown> {
-  return {
-    model: "claude-haiku-4-5-20251001",
-    effort: "low",
-    thinking: { enabled: false },
-  };
-}
-
-function fullSeededCallSites(): Record<string, Record<string, unknown>> {
-  return {
-    guardianQuestionCopy: latencySeed(),
-    interactionClassifier: latencySeed(),
-    skillCategoryInference: latencySeed(),
-    inviteInstructionGenerator: latencySeed(),
-    notificationDecision: latencySeed(),
-    preferenceExtraction: latencySeed(),
-    commitMessage: {
-      model: "claude-haiku-4-5-20251001",
-      maxTokens: 120,
-      temperature: 0.2,
-      effort: "low",
-      thinking: { enabled: false },
-    },
-    conversationStarters: latencySeed(),
-    conversationSummarization: {
-      model: "claude-opus-4-7",
-      effort: "low",
-      thinking: { enabled: false },
-    },
-    recall: {
-      profile: "cost-optimized",
-      maxTokens: 4096,
-      effort: "low",
-      thinking: { enabled: false, streamThinking: false },
-      temperature: 0,
-      disableCache: true,
-    },
-    heartbeatAgent: {
-      profile: "cost-optimized",
-      maxTokens: 2048,
-      effort: "low",
-      temperature: 0,
-      thinking: { enabled: false, streamThinking: false },
-      contextWindow: { maxInputTokens: 16000 },
-    },
-    replySuggestion: {
-      model: "claude-haiku-4-5-20251001",
-      effort: "low",
-      thinking: { enabled: false },
-      disableCache: true,
-    },
-    memoryRouter: {
-      profile: "cost-optimized",
-      contextWindow: { maxInputTokens: 1_000_000 },
-    },
-  };
-}
-
 function mergeDefaultConfigAndSeedInferenceProfiles(db?: DrizzleDb): void {
   const defaultConfigMerge = mergeDefaultWorkspaceConfig();
   seedInferenceProfiles({
@@ -430,56 +372,6 @@ describe("loadConfig startup behavior", () => {
     expect(config.memory.v2.bm25_b).toBe(0.4);
   });
 
-  test("default workspace config merge prunes exact seeded call-site defaults", () => {
-    const seededCallSites = fullSeededCallSites();
-    const overlayPath = join(WORKSPACE_DIR, "hatch-overlay.json");
-    writeFileSync(
-      overlayPath,
-      JSON.stringify(
-        {
-          gateway: {
-            unmappedPolicy: "default",
-            defaultAssistantId: "self",
-          },
-          llm: {
-            activeProfile: "balanced",
-            advisorProfile: "frontier",
-            callSites: {
-              ...seededCallSites,
-              recall: {
-                ...seededCallSites.recall,
-                disableCache: false,
-              },
-              customSite: { profile: "frontier" },
-            },
-          },
-        },
-        null,
-        2,
-      ) + "\n",
-    );
-    process.env.VELLUM_DEFAULT_WORKSPACE_CONFIG_PATH = overlayPath;
-
-    const result = mergeDefaultWorkspaceConfig();
-    const raw = JSON.parse(readFileSync(CONFIG_PATH, "utf-8")) as Record<
-      string,
-      Record<string, unknown>
-    >;
-    const llm = raw.llm as Record<string, unknown>;
-    const callSites = llm.callSites as Record<string, Record<string, unknown>>;
-
-    expect(result.hadOverlay).toBe(true);
-    expect(raw.gateway).toEqual({
-      unmappedPolicy: "default",
-      defaultAssistantId: "self",
-    });
-    expect(llm.activeProfile).toBe("balanced");
-    expect(llm.advisorProfile).toBe("frontier");
-    expect(Object.keys(callSites).sort()).toEqual(["customSite", "recall"]);
-    expect(callSites.recall?.disableCache).toBe(false);
-    expect(callSites.customSite).toEqual({ profile: "frontier" });
-  });
-
   test("reloads cached config when config.json is updated externally", () => {
     // Models a CLI subprocess writing twilio.accountSid while the assistant
     // process already has an effective config cached in memory.
@@ -853,11 +745,7 @@ describe("loadConfig startup behavior", () => {
     );
     expect(raw.llm.profiles.frontier.provider).toBe("anthropic");
     expect(raw.llm.profiles.frontier.model).toBe("claude-opus-4-8");
-    // Speed is served by DeepSeek V4 Flash on Fireworks.
-    expect(raw.llm.profiles["cost-optimized"].provider).toBe("fireworks");
-    expect(raw.llm.profiles["cost-optimized"].model).toBe(
-      "accounts/fireworks/models/deepseek-v4-flash",
-    );
+    expect(raw.llm.profiles["cost-optimized"].provider).toBe("anthropic");
   });
 
   test("off-platform managed profiles are overwritten on every boot", () => {
@@ -1703,8 +1591,8 @@ describe("seedInferenceProfiles BYOK-mode managed profile labels", () => {
 // ---------------------------------------------------------------------------
 // Tests: OS Beta flag-gated managed profile. The template is defined but
 // intentionally NOT part of MANAGED_PROFILE_TEMPLATES, so seedInferenceProfiles
-// must never create it. The flag-gated reconcile creates or removes it based on
-// the `os-beta` feature flag.
+// must never create it. A later PR reconciles it in/out based on the `os-beta`
+// feature flag.
 // ---------------------------------------------------------------------------
 
 describe("OS Beta managed profile template", () => {
@@ -1751,21 +1639,20 @@ describe("OS Beta managed profile template", () => {
     expect(MANAGED_PROFILE_NAMES.has("os-beta")).toBe(true);
   });
 
-  test("materializeProfile resolves OS Beta to the Balanced model with low effort", () => {
+  test("materializeProfile honors the explicit OS Beta model", () => {
     const entry = materializeProfile(
       OS_BETA_PROFILE_TEMPLATE,
-      "together",
-      "together-managed",
+      "fireworks",
+      "fireworks-managed",
     );
 
-    expect(entry.model).toBe("MiniMaxAI/MiniMax-M3");
-    expect(entry.provider_connection).toBe("together-managed");
-    expect(entry.provider).toBe("together");
+    expect(entry.model).toBe("accounts/fireworks/models/glm-5p2");
+    expect(entry.provider_connection).toBe("fireworks-managed");
+    expect(entry.provider).toBe("fireworks");
     expect(entry.label).toBe("OS Beta");
     expect(entry.source).toBe("managed");
     expect(entry.maxTokens).toBe(32000);
-    expect(entry.effort).toBe("low");
+    expect(entry.effort).toBe("high");
     expect(entry.thinking?.enabled).toBe(true);
-    expect(entry.topP).toBe(0.95);
   });
 });

package/src/__tests__/config-schema.test.ts

@@ -120,7 +120,6 @@ describe("AssistantConfigSchema", () => {
       shellDefaultTimeoutSec: 120,
       shellMaxTimeoutSec: 600,
       permissionTimeoutSec: 300,
-      questionResponseTimeoutSec: 1800,
       toolExecutionTimeoutSec: 120,
       providerStreamTimeoutSec: 1800,
       backgroundTurnTimeoutSec: 1800,