@vellumai/assistant 0.4.37 → 0.4.41

package/src/__tests__/handlers-twitter-config.test.ts

@@ -10,18 +10,49 @@ const testDir = mkdtempSync(join(tmpdir(), "handlers-twitter-cfg-test-"));
 let rawConfigStore: Record<string, unknown> = {};
 const saveRawConfigCalls: Record<string, unknown>[] = [];
 
+function getNestedValue(obj: Record<string, unknown>, path: string): unknown {
+  const keys = path.split(".");
+  let current: unknown = obj;
+  for (const key of keys) {
+    if (current == null || typeof current !== "object") {
+      return undefined;
+    }
+    current = (current as Record<string, unknown>)[key];
+  }
+  return current;
+}
+
+function setNestedValue(
+  obj: Record<string, unknown>,
+  path: string,
+  value: unknown,
+): void {
+  const keys = path.split(".");
+  let current = obj;
+  for (let i = 0; i < keys.length - 1; i++) {
+    const key = keys[i]!;
+    if (current[key] == null || typeof current[key] !== "object") {
+      current[key] = {};
+    }
+    current = current[key] as Record<string, unknown>;
+  }
+  current[keys[keys.length - 1]!] = value;
+}
+
 mock.module("../config/loader.js", () => ({
   getConfig: () => ({
     ui: {},
   }),
   loadConfig: () => ({}),
-  loadRawConfig: () => ({ ...rawConfigStore }),
+  loadRawConfig: () => structuredClone(rawConfigStore),
   saveRawConfig: (cfg: Record<string, unknown>) => {
     saveRawConfigCalls.push(cfg);
-    rawConfigStore = { ...cfg };
+    rawConfigStore = structuredClone(cfg);
   },
   saveConfig: () => {},
   invalidateConfigCache: () => {},
+  getNestedValue,
+  setNestedValue,
 }));
 
 mock.module("../util/platform.js", () => ({
@@ -61,20 +92,27 @@ let secureKeyStore: Record<string, string> = {};
 let setSecureKeyOverride: ((account: string, value: string) => boolean) | null =
   null;
 
+function syncSet(account: string, value: string): boolean {
+  if (setSecureKeyOverride) return setSecureKeyOverride(account, value);
+  secureKeyStore[account] = value;
+  return true;
+}
+
+function syncDelete(account: string): "deleted" | "not-found" {
+  if (account in secureKeyStore) {
+    delete secureKeyStore[account];
+    return "deleted";
+  }
+  return "not-found";
+}
+
 mock.module("../security/secure-keys.js", () => ({
   getSecureKey: (account: string) => secureKeyStore[account] ?? undefined,
-  setSecureKey: (account: string, value: string) => {
-    if (setSecureKeyOverride) return setSecureKeyOverride(account, value);
-    secureKeyStore[account] = value;
-    return true;
-  },
-  deleteSecureKey: (account: string) => {
-    if (account in secureKeyStore) {
-      delete secureKeyStore[account];
-      return "deleted";
-    }
-    return "not-found";
-  },
+  setSecureKey: syncSet,
+  deleteSecureKey: syncDelete,
+  setSecureKeyAsync: async (account: string, value: string) =>
+    syncSet(account, value),
+  deleteSecureKeyAsync: async (account: string) => syncDelete(account),
   listSecureKeys: () => Object.keys(secureKeyStore),
   getBackendType: () => "encrypted",
   isDowngradedFromKeychain: () => false,
@@ -134,11 +172,26 @@ mock.module("../tools/credentials/metadata-store.js", () => ({
 
 import { handleMessage, type HandlerContext } from "../daemon/handlers.js";
 import type {
+  ClientMessage,
   ServerMessage,
   TwitterIntegrationConfigRequest,
 } from "../daemon/ipc-contract.js";
 import { DebouncerMap } from "../util/debounce.js";
 
+/**
+ * Wrapper around handleMessage that flushes the microtask queue so async
+ * handlers complete before assertions run. handleMessage() returns void
+ * and swallows the promise, so we need a macrotask tick to settle.
+ */
+async function handleMessageAsync(
+  msg: ClientMessage,
+  socket: net.Socket,
+  ctx: HandlerContext,
+): Promise<void> {
+  handleMessage(msg, socket, ctx);
+  await new Promise<void>((r) => setTimeout(r, 0));
+}
+
 function createTestContext(): { ctx: HandlerContext; sent: ServerMessage[] } {
   const sent: ServerMessage[] = [];
   const ctx: HandlerContext = {
@@ -203,7 +256,7 @@ describe("Twitter integration config handler", () => {
   });
 
   test("get action returns correct status when configured and connected", () => {
-    rawConfigStore = { twitterIntegrationMode: "local_byo" };
+    rawConfigStore = { twitter: { integrationMode: "local_byo" } };
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "test-client-id";
     secureKeyStore["credential:integration:twitter:access_token"] =
@@ -256,10 +309,12 @@ describe("Twitter integration config handler", () => {
     expect(res.mode).toBe("managed");
 
     expect(saveRawConfigCalls).toHaveLength(1);
-    expect(saveRawConfigCalls[0]!.twitterIntegrationMode).toBe("managed");
+    expect(
+      getNestedValue(saveRawConfigCalls[0]!, "twitter.integrationMode"),
+    ).toBe("managed");
   });
 
-  test("set_local_client stores credentials in secure storage", () => {
+  test("set_local_client stores credentials in secure storage", async () => {
     const msg: TwitterIntegrationConfigRequest = {
       type: "twitter_integration_config",
       action: "set_local_client",
@@ -268,7 +323,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -288,14 +343,14 @@ describe("Twitter integration config handler", () => {
     ).toBe("my-client-secret");
   });
 
-  test("set_local_client without clientId returns error", () => {
+  test("set_local_client without clientId returns error", async () => {
     const msg: TwitterIntegrationConfigRequest = {
       type: "twitter_integration_config",
       action: "set_local_client",
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as { type: string; success: boolean; error: string };
@@ -303,7 +358,7 @@ describe("Twitter integration config handler", () => {
     expect(res.error).toContain("clientId is required");
   });
 
-  test("clear_local_client removes credentials", () => {
+  test("clear_local_client removes credentials", async () => {
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
     secureKeyStore["credential:integration:twitter:oauth_client_secret"] =
@@ -315,7 +370,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -336,7 +391,7 @@ describe("Twitter integration config handler", () => {
     ).toBeUndefined();
   });
 
-  test("clear_local_client also disconnects if connected", () => {
+  test("clear_local_client also disconnects if connected", async () => {
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
     secureKeyStore["credential:integration:twitter:access_token"] =
@@ -355,7 +410,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -378,7 +433,7 @@ describe("Twitter integration config handler", () => {
     });
   });
 
-  test("disconnect removes tokens and metadata", () => {
+  test("disconnect removes tokens and metadata", async () => {
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
     secureKeyStore["credential:integration:twitter:access_token"] =
@@ -397,7 +452,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -426,7 +481,7 @@ describe("Twitter integration config handler", () => {
     });
   });
 
-  test("set_local_client returns error when setSecureKey fails for client ID", () => {
+  test("set_local_client returns error when setSecureKey fails for client ID", async () => {
     // Override setSecureKey to return false (storage unavailable, not throwing)
     setSecureKeyOverride = () => false;
 
@@ -438,7 +493,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -453,7 +508,7 @@ describe("Twitter integration config handler", () => {
     expect(res.error).toContain("Failed to store client ID");
   });
 
-  test("set_local_client returns error when setSecureKey fails for client secret", () => {
+  test("set_local_client returns error when setSecureKey fails for client secret", async () => {
     // Override setSecureKey to fail only for the secret
     setSecureKeyOverride = (account: string, value: string) => {
       if (account.includes("client_secret")) return false;
@@ -469,7 +524,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -484,7 +539,7 @@ describe("Twitter integration config handler", () => {
     expect(res.error).toContain("Failed to store client secret");
   });
 
-  test("set_local_client without secret clears stale secret", () => {
+  test("set_local_client without secret clears stale secret", async () => {
     // Pre-populate an old client secret
     secureKeyStore["credential:integration:twitter:oauth_client_id"] = "old-id";
     secureKeyStore["credential:integration:twitter:oauth_client_secret"] =
@@ -497,7 +552,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -576,7 +631,7 @@ describe("Twitter integration config handler", () => {
     expect((sent4[0] as { mode: string }).mode).toBe("local_byo");
   });
 
-  test("set_local_client with only clientId (no secret)", () => {
+  test("set_local_client with only clientId (no secret)", async () => {
     const msg: TwitterIntegrationConfigRequest = {
       type: "twitter_integration_config",
       action: "set_local_client",
@@ -584,7 +639,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -604,7 +659,7 @@ describe("Twitter integration config handler", () => {
     ).toBeUndefined();
   });
 
-  test("set_local_client overwrites existing credentials", () => {
+  test("set_local_client overwrites existing credentials", async () => {
     // Set initial credentials
     secureKeyStore["credential:integration:twitter:oauth_client_id"] = "old-id";
     secureKeyStore["credential:integration:twitter:oauth_client_secret"] =
@@ -618,7 +673,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -638,7 +693,7 @@ describe("Twitter integration config handler", () => {
     ).toBe("new-secret");
   });
 
-  test("clear_local_client when no credentials exist (idempotent)", () => {
+  test("clear_local_client when no credentials exist (idempotent)", async () => {
     // No credentials set at all
     const msg: TwitterIntegrationConfigRequest = {
       type: "twitter_integration_config",
@@ -646,7 +701,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -661,7 +716,7 @@ describe("Twitter integration config handler", () => {
     expect(res.connected).toBe(false);
   });
 
-  test("disconnect when not connected (idempotent) preserves client credentials", () => {
+  test("disconnect when not connected (idempotent) preserves client credentials", async () => {
     // Only client credentials, no access token
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
@@ -674,7 +729,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -696,7 +751,7 @@ describe("Twitter integration config handler", () => {
     ).toBe("my-client-secret");
   });
 
-  test("disconnect preserves client credentials when access token exists", () => {
+  test("disconnect preserves client credentials when access token exists", async () => {
     // Set up both client credentials and tokens
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
@@ -718,7 +773,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -752,7 +807,7 @@ describe("Twitter integration config handler", () => {
     });
   });
 
-  test("clear_local_client cascades to remove tokens and metadata", () => {
+  test("clear_local_client cascades to remove tokens and metadata", async () => {
     // Set up client credentials, tokens, and metadata
     secureKeyStore["credential:integration:twitter:oauth_client_id"] =
       "my-client-id";
@@ -774,7 +829,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as {
@@ -860,7 +915,7 @@ describe("Twitter integration config handler", () => {
     expect(res.connected).toBe(false);
   });
 
-  test("error in secure storage throws and returns error response", () => {
+  test("error in secure storage throws and returns error response", async () => {
     // Override setSecureKey to throw an error, simulating a storage failure
     setSecureKeyOverride = () => {
       throw new Error("Keychain access denied");
@@ -874,7 +929,7 @@ describe("Twitter integration config handler", () => {
     };
 
     const { ctx, sent } = createTestContext();
-    handleMessage(msg, {} as net.Socket, ctx);
+    await handleMessageAsync(msg, {} as net.Socket, ctx);
 
     expect(sent).toHaveLength(1);
     const res = sent[0] as { type: string; success: boolean; error?: string };
@@ -984,7 +1039,9 @@ describe("Twitter integration config handler", () => {
     // Verify persistence via saveRawConfig
     expect(saveRawConfigCalls.length).toBeGreaterThan(0);
     const lastSaved = saveRawConfigCalls[saveRawConfigCalls.length - 1]!;
-    expect(lastSaved.twitterOperationStrategy).toBe("oauth");
+    expect(getNestedValue(lastSaved, "twitter.operationStrategy")).toBe(
+      "oauth",
+    );
   });
 
   test("set_strategy with invalid value returns error", () => {
@@ -1022,7 +1079,7 @@ describe("Twitter integration config handler", () => {
 
   test("get action includes strategy field with strategyConfigured=true when set", () => {
     // Set a specific strategy first
-    rawConfigStore = { twitterOperationStrategy: "browser" };
+    rawConfigStore = { twitter: { operationStrategy: "browser" } };
 
     const msg: TwitterIntegrationConfigRequest = {
       type: "twitter_integration_config",

package/src/__tests__/inbound-invite-redemption.test.ts

@@ -87,7 +87,7 @@ mock.module("../runtime/approval-message-composer.js", () => ({
 }));
 
 import { findContactChannel } from "../contacts/contact-store.js";
-import { upsertMember } from "../contacts/contacts-write.js";
+import { upsertContactChannel } from "../contacts/contacts-write.js";
 import { getDb, initializeDb, resetDb } from "../memory/db.js";
 import { createInvite, revokeInvite } from "../memory/invite-store.js";
 import { handleChannelInbound } from "../runtime/routes/channel-routes.js";
@@ -328,7 +328,7 @@ describe("inbound invite redemption intercept", () => {
 
   test("existing active member sending normal message is unaffected", async () => {
     // Pre-create an active member
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "user-active-member",
       externalChatId: "chat-active",
@@ -377,7 +377,7 @@ describe("inbound invite redemption intercept", () => {
     });
 
     // Pre-create an active member that will click the invite link
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "user-already-active",
       externalChatId: "chat-invite-test",
@@ -403,7 +403,7 @@ describe("inbound invite redemption intercept", () => {
       maxUses: 5,
     });
 
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "user-invite-123",
       externalChatId: "chat-invite-test",

package/src/__tests__/integration-status.test.ts

@@ -1,17 +1,27 @@
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 const secureKeyValues = new Map<string, string>();
+let mockTwilioAccountSid: string | undefined;
 
 mock.module("../security/secure-keys.js", () => ({
   getSecureKey: (account: string) => secureKeyValues.get(account),
 }));
 
+mock.module("../config/loader.js", () => ({
+  loadConfig: () => ({
+    twilio: mockTwilioAccountSid
+      ? { accountSid: mockTwilioAccountSid }
+      : undefined,
+  }),
+}));
+
 const { getIntegrationSummary, formatIntegrationSummary, hasCapability } =
   await import("../schedule/integration-status.js");
 
 describe("integration-status", () => {
   beforeEach(() => {
     secureKeyValues.clear();
+    mockTwilioAccountSid = undefined;
   });
 
   describe("getIntegrationSummary", () => {
@@ -30,7 +40,7 @@ describe("integration-status", () => {
       secureKeyValues.set("credential:integration:gmail:access_token", "tok");
       secureKeyValues.set("credential:integration:twitter:access_token", "tok");
       secureKeyValues.set("credential:integration:slack:access_token", "tok");
-      secureKeyValues.set("credential:twilio:account_sid", "sid");
+      mockTwilioAccountSid = "sid";
       secureKeyValues.set("credential:twilio:auth_token", "auth");
       secureKeyValues.set("credential:telegram:bot_token", "tok");
       secureKeyValues.set("credential:telegram:webhook_secret", "secret");
@@ -42,7 +52,7 @@ describe("integration-status", () => {
     });
 
     test("returns mixed status", () => {
-      secureKeyValues.set("credential:twilio:account_sid", "sid");
+      mockTwilioAccountSid = "sid";
       secureKeyValues.set("credential:twilio:auth_token", "auth");
       secureKeyValues.set("credential:telegram:bot_token", "tok");
       secureKeyValues.set("credential:telegram:webhook_secret", "secret");
@@ -67,7 +77,7 @@ describe("integration-status", () => {
     });
 
     test("SMS disconnected when only account_sid is set (missing auth_token)", () => {
-      secureKeyValues.set("credential:twilio:account_sid", "sid");
+      mockTwilioAccountSid = "sid";
 
       const summary = getIntegrationSummary();
       const sms = summary.find((s: { name: string }) => s.name === "SMS");
@@ -87,7 +97,7 @@ describe("integration-status", () => {
 
   describe("formatIntegrationSummary", () => {
     test("shows checkmarks and crosses", () => {
-      secureKeyValues.set("credential:twilio:account_sid", "sid");
+      mockTwilioAccountSid = "sid";
       secureKeyValues.set("credential:twilio:auth_token", "auth");
       secureKeyValues.set("credential:telegram:bot_token", "tok");
       secureKeyValues.set("credential:telegram:webhook_secret", "secret");
@@ -109,7 +119,7 @@ describe("integration-status", () => {
       secureKeyValues.set("credential:integration:gmail:access_token", "tok");
       secureKeyValues.set("credential:integration:twitter:access_token", "tok");
       secureKeyValues.set("credential:integration:slack:access_token", "tok");
-      secureKeyValues.set("credential:twilio:account_sid", "sid");
+      mockTwilioAccountSid = "sid";
       secureKeyValues.set("credential:twilio:auth_token", "auth");
       secureKeyValues.set("credential:telegram:bot_token", "tok");
       secureKeyValues.set("credential:telegram:webhook_secret", "secret");

package/src/__tests__/integrations-cli.test.ts

@@ -94,7 +94,7 @@ async function runCli(
   };
 }
 
-describe("vellum integrations CLI", () => {
+describe("assistant integrations CLI", () => {
   beforeEach(() => {
     gatewayBase = "http://gateway.test";
     signingKeyInitialized = false;

package/src/__tests__/invite-redemption-service.test.ts

@@ -24,7 +24,8 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-import { upsertMember } from "../contacts/contacts-write.js";
+import { findContactChannel } from "../contacts/contact-store.js";
+import { upsertContactChannel } from "../contacts/contacts-write.js";
 import { getSqlite, initializeDb, resetDb } from "../memory/db.js";
 import {
   createInvite,
@@ -33,7 +34,9 @@ import {
 import {
   type InviteRedemptionOutcome,
   redeemInvite,
+  redeemInviteByCode,
 } from "../runtime/invite-redemption-service.js";
+import { hashVoiceCode } from "../util/voice-code.js";
 
 initializeDb();
 
@@ -76,6 +79,58 @@ describe("invite-redemption-service", () => {
     });
   });
 
+  test("marks channel as verified via invite on redemption", () => {
+    const { rawToken } = createInvite({
+      sourceChannel: "telegram",
+      maxUses: 1,
+    });
+
+    const outcome = redeemInvite({
+      rawToken,
+      sourceChannel: "telegram",
+      externalUserId: "user-1",
+    });
+
+    expect(outcome.ok).toBe(true);
+
+    const result = findContactChannel({
+      channelType: "telegram",
+      externalUserId: "user-1",
+    });
+
+    expect(result).not.toBeNull();
+    expect(result!.channel.verifiedAt).toBeGreaterThan(0);
+    expect(result!.channel.verifiedVia).toBe("invite");
+    expect(result!.channel.status).toBe("active");
+  });
+
+  test("marks channel as verified via invite on 6-digit code redemption", () => {
+    const inviteCode = "123456";
+    createInvite({
+      sourceChannel: "telegram",
+      maxUses: 1,
+      inviteCodeHash: hashVoiceCode(inviteCode),
+    });
+
+    const outcome = redeemInviteByCode({
+      code: inviteCode,
+      sourceChannel: "telegram",
+      externalUserId: "code-user-1",
+    });
+
+    expect(outcome.ok).toBe(true);
+
+    const result = findContactChannel({
+      channelType: "telegram",
+      externalUserId: "code-user-1",
+    });
+
+    expect(result).not.toBeNull();
+    expect(result!.channel.verifiedAt).toBeGreaterThan(0);
+    expect(result!.channel.verifiedVia).toBe("invite");
+    expect(result!.channel.status).toBe("active");
+  });
+
   test("returns invalid_token for a bogus token", () => {
     const outcome = redeemInvite({
       rawToken: "totally-bogus-token",
@@ -179,7 +234,7 @@ describe("invite-redemption-service", () => {
     });
 
     // Pre-create an active member
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "existing-user",
       status: "active",
@@ -209,7 +264,7 @@ describe("invite-redemption-service", () => {
     });
 
     // Pre-create a blocked member — simulates a guardian-initiated block
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "blocked-user",
       status: "blocked",
@@ -231,7 +286,7 @@ describe("invite-redemption-service", () => {
     });
 
     // Pre-create a revoked member
-    const member = upsertMember({
+    const member = upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "revoked-user",
       status: "revoked",
@@ -282,7 +337,7 @@ describe("invite-redemption-service", () => {
 
   test("returns invalid_token for an active member with a bogus token (no membership probing)", () => {
     // Pre-create an active member
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "probed-user",
       status: "active",
@@ -307,7 +362,7 @@ describe("invite-redemption-service", () => {
     });
 
     // Pre-create an active member
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "expired-token-user",
       status: "active",
@@ -331,7 +386,7 @@ describe("invite-redemption-service", () => {
     });
 
     // Pre-create an active member on telegram
-    upsertMember({
+    upsertContactChannel({
       sourceChannel: "telegram",
       externalUserId: "cross-channel-user",
       status: "active",

package/src/__tests__/ipc-snapshot.test.ts

@@ -398,10 +398,6 @@ const clientMessages: Record<ClientMessageType, ClientMessage> = {
     type: "share_app_cloud",
     appId: "app-001",
   },
-  share_to_slack: {
-    type: "share_to_slack",
-    appId: "app-001",
-  },
   slack_webhook_config: {
     type: "slack_webhook_config",
     action: "get",
@@ -1450,10 +1446,6 @@ const serverMessages: Record<ServerMessageType, ServerMessage> = {
     shareToken: "abc123def456",
     shareUrl: "http://localhost:7821/v1/apps/shared/abc123def456",
   },
-  share_to_slack_response: {
-    type: "share_to_slack_response",
-    success: true,
-  },
   slack_webhook_config_response: {
     type: "slack_webhook_config_response",
     webhookUrl: "https://hooks.slack.com/services/T00/B00/xxx",