@vellumai/assistant 0.4.32 → 0.4.33

package/src/__tests__/trusted-contact-approval-notifier.test.ts

@@ -26,7 +26,6 @@ mock.module("../util/platform.js", () => ({
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
-  readHttpToken: () => "test-token",
   ensureDataDir: () => {},
   migrateToDataLayout: () => {},
   migrateToWorkspaceLayout: () => {},
@@ -80,10 +79,15 @@ mock.module("../runtime/gateway-client.js", () => ({
 }));
 
 // ── Guardian binding mock ──
-let mockGuardianBinding: Record<string, unknown> | null = null;
+// mockGuardianContact controls what findGuardianForChannel returns.
+// When non-null, it should look like { contact: { displayName: "..." }, channel: { ... } }.
+let mockGuardianContact: {
+  contact: { displayName: string };
+  channel: Record<string, unknown>;
+} | null = null;
 
 mock.module("../runtime/channel-guardian-service.js", () => ({
-  getGuardianBinding: () => mockGuardianBinding,
+  getGuardianBinding: () => null,
   // Re-export stubs for other functions to prevent import errors
   bindSessionIdentity: () => {},
   createOutboundSession: () => ({}),
@@ -100,6 +104,11 @@ mock.module("../runtime/channel-guardian-service.js", () => ({
   }),
 }));
 
+// ── Contact store mock ──
+mock.module("../contacts/contact-store.js", () => ({
+  findGuardianForChannel: () => mockGuardianContact,
+}));
+
 // ── Pending interactions mock ──
 let mockPendingApprovals: Array<{
   requestId: string;
@@ -123,11 +132,22 @@ mock.module("../config/env.js", () => ({
 // ── User reference mock ──
 mock.module("../config/user-reference.js", () => ({
   resolveUserReference: () => "my human",
+  DEFAULT_USER_REFERENCE: "my human",
+  resolveGuardianName: (guardianDisplayName?: string | null): string => {
+    // Mirror the real implementation: USER.md name > guardianDisplayName > default
+    const userRef = "my human"; // In tests, resolveUserReference() returns this
+    if (userRef !== "my human") return userRef;
+    if (guardianDisplayName && guardianDisplayName.trim().length > 0) {
+      return guardianDisplayName.trim();
+    }
+    return "my human";
+  },
 }));
 
 // Import module under test AFTER mocks are set up
 import type { ChannelId } from "../channels/types.js";
-import { resolveUserReference } from "../config/user-reference.js";
+import { resolveGuardianName } from "../config/user-reference.js";
+import { findGuardianForChannel } from "../contacts/contact-store.js";
 import type { TrustContext } from "../daemon/session-runtime-assembly.js";
 
 // We need to test the private functions by importing the module.
@@ -178,8 +198,6 @@ async function simulateNotifierPoll(params: {
   const { getApprovalInfoByConversation } =
     await import("../runtime/channel-approvals.js");
   const { deliverChannelReply } = await import("../runtime/gateway-client.js");
-  const { getGuardianBinding } =
-    await import("../runtime/channel-guardian-service.js");
 
   const pending = getApprovalInfoByConversation(params.conversationId);
   const info = pending[0];
@@ -198,37 +216,14 @@ async function simulateNotifierPoll(params: {
 
   notifiedRequestIds.set(info.requestId, conversationId);
 
-  // Resolve guardian name
-  let guardianName: string | undefined;
-  const binding = getGuardianBinding(
-    params.assistantId ?? "self",
+  // Resolve guardian name via the contacts-based approach
+  const guardian = findGuardianForChannel(
     params.sourceChannel,
+    params.assistantId ?? "self",
   );
-  if (binding?.metadataJson) {
-    try {
-      const parsed = JSON.parse(binding.metadataJson as string) as Record<
-        string,
-        unknown
-      >;
-      if (
-        typeof parsed.displayName === "string" &&
-        parsed.displayName.trim().length > 0
-      ) {
-        guardianName = parsed.displayName.trim();
-      } else if (
-        typeof parsed.username === "string" &&
-        parsed.username.trim().length > 0
-      ) {
-        guardianName = `@${parsed.username.trim()}`;
-      }
-    } catch {
-      // ignore
-    }
-  }
+  const guardianName = resolveGuardianName(guardian?.contact.displayName);
 
-  const waitingText = `Waiting for ${
-    guardianName ?? resolveUserReference()
-  }'s approval...`;
+  const waitingText = `Waiting for ${guardianName}'s approval...`;
 
   try {
     await deliverChannelReply(
@@ -256,7 +251,7 @@ describe("trusted-contact pending-approval notifier", () => {
     deliveredReplies.length = 0;
     deliverShouldFail = false;
     mockPendingApprovals = [];
-    mockGuardianBinding = null;
+    mockGuardianContact = null;
   });
 
   afterAll(() => {
@@ -277,9 +272,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ displayName: "Mom" }),
+    mockGuardianContact = {
+      contact: { displayName: "Mom" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -304,7 +299,7 @@ describe("trusted-contact pending-approval notifier", () => {
     expect(notified.has("req-1")).toBe(true);
   });
 
-  test("uses username with @ prefix when display name is not available", async () => {
+  test("uses contact displayName from contact store", async () => {
     mockPendingApprovals = [
       {
         requestId: "req-2",
@@ -314,9 +309,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ username: "guardian_user" }),
+    mockGuardianContact = {
+      contact: { displayName: "Guardian User" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -332,11 +327,11 @@ describe("trusted-contact pending-approval notifier", () => {
 
     expect(deliveredReplies).toHaveLength(1);
     expect(deliveredReplies[0].payload.text).toBe(
-      "Waiting for @guardian_user's approval...",
+      "Waiting for Guardian User's approval...",
     );
   });
 
-  test("falls back to user reference when no guardian name is available", async () => {
+  test("falls back to user reference when guardian has empty display name", async () => {
     mockPendingApprovals = [
       {
         requestId: "req-3",
@@ -346,10 +341,10 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    // No binding metadata
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: null,
+    // Guardian contact exists but has an empty displayName
+    mockGuardianContact = {
+      contact: { displayName: "" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -369,7 +364,7 @@ describe("trusted-contact pending-approval notifier", () => {
     );
   });
 
-  test("falls back to user reference when no guardian binding exists", async () => {
+  test("falls back to user reference when no guardian contact exists", async () => {
     mockPendingApprovals = [
       {
         requestId: "req-4",
@@ -379,7 +374,7 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = null;
+    mockGuardianContact = null;
 
     const notified = new Map<string, string>();
     await simulateNotifierPoll({
@@ -408,9 +403,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ displayName: "Guardian" }),
+    mockGuardianContact = {
+      contact: { displayName: "Guardian" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -436,9 +431,9 @@ describe("trusted-contact pending-approval notifier", () => {
   });
 
   test("sends separate messages for different requestIds", async () => {
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ displayName: "Guardian" }),
+    mockGuardianContact = {
+      contact: { displayName: "Guardian" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -478,9 +473,9 @@ describe("trusted-contact pending-approval notifier", () => {
   });
 
   test("concurrent pollers for different conversations do not evict each other", async () => {
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ displayName: "Guardian" }),
+    mockGuardianContact = {
+      contact: { displayName: "Guardian" },
+      channel: {},
     };
 
     // Shared dedupe map simulating the module-level global
@@ -630,9 +625,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({ displayName: "Guardian" }),
+    mockGuardianContact = {
+      contact: { displayName: "Guardian" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -678,7 +673,7 @@ describe("trusted-contact pending-approval notifier", () => {
     expect(deliveredReplies).toHaveLength(0);
   });
 
-  test("prefers displayName over username when both are present", async () => {
+  test("uses contact displayName from guardian contact record", async () => {
     mockPendingApprovals = [
       {
         requestId: "req-10",
@@ -688,12 +683,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: JSON.stringify({
-        displayName: "Sarah",
-        username: "sarah_bot",
-      }),
+    mockGuardianContact = {
+      contact: { displayName: "Sarah" },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -713,7 +705,7 @@ describe("trusted-contact pending-approval notifier", () => {
     );
   });
 
-  test("handles malformed metadataJson gracefully", async () => {
+  test("falls back to default when guardian contact has whitespace-only displayName", async () => {
     mockPendingApprovals = [
       {
         requestId: "req-11",
@@ -723,9 +715,9 @@ describe("trusted-contact pending-approval notifier", () => {
       },
     ];
 
-    mockGuardianBinding = {
-      id: "binding-1",
-      metadataJson: "not-valid-json{{{",
+    mockGuardianContact = {
+      contact: { displayName: "   " },
+      channel: {},
     };
 
     const notified = new Map<string, string>();
@@ -740,7 +732,7 @@ describe("trusted-contact pending-approval notifier", () => {
     });
 
     expect(deliveredReplies).toHaveLength(1);
-    // Falls back to generic phrasing
+    // Falls back to default user reference
     expect(deliveredReplies[0].payload.text).toBe(
       "Waiting for my human's approval...",
     );

package/src/__tests__/trusted-contact-inline-approval-integration.test.ts

@@ -38,7 +38,6 @@ mock.module("../util/platform.js", () => ({
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
-  readHttpToken: () => "test-token",
   ensureDataDir: () => {},
   migrateToDataLayout: () => {},
   migrateToWorkspaceLayout: () => {},

package/src/__tests__/trusted-contact-lifecycle-notifications.test.ts

@@ -33,7 +33,6 @@ mock.module("../util/platform.js", () => ({
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
-  readHttpToken: () => "test-bearer-token",
 }));
 
 mock.module("../util/logger.js", () => ({

package/src/__tests__/trusted-contact-multichannel.test.ts

@@ -28,7 +28,6 @@ mock.module("../util/platform.js", () => ({
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
-  readHttpToken: () => "test-bearer-token",
 }));
 
 mock.module("../util/logger.js", () => ({

package/src/__tests__/trusted-contact-verification.test.ts

@@ -31,7 +31,6 @@ mock.module("../util/platform.js", () => ({
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
-  readHttpToken: () => "test-bearer-token",
 }));
 
 mock.module("../util/logger.js", () => ({

package/src/__tests__/twilio-routes.test.ts

@@ -43,7 +43,6 @@ mock.module("../util/platform.js", () => ({
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
-  readHttpToken: () => null,
 }));
 
 mock.module("../util/logger.js", () => ({

package/src/__tests__/update-bulletin.test.ts

@@ -57,7 +57,6 @@ mock.module("../util/platform.js", () => ({
   getHooksDir: () => "",
   getSocketPath: () => "",
   getSessionTokenPath: () => "",
-  getHttpTokenPath: () => "",
   getPlatformTokenPath: () => "",
   getPidPath: () => "",
   getWorkspaceConfigPath: () => "",
@@ -72,7 +71,6 @@ mock.module("../util/platform.js", () => ({
   writeLockfile: () => {},
   readPlatformToken: () => null,
   readSessionToken: () => null,
-  readHttpToken: () => null,
   removeSocketFile: () => {},
   getTCPPort: () => 8765,
   isTCPEnabled: () => false,

package/src/__tests__/user-reference.test.ts

@@ -26,7 +26,8 @@ mock.module("node:fs", () => ({
 }));
 
 // Import after mocks are in place
-const { resolveUserReference } = await import("../config/user-reference.js");
+const { resolveUserReference, resolveGuardianName, DEFAULT_USER_REFERENCE } =
+  await import("../config/user-reference.js");
 
 describe("resolveUserReference", () => {
   beforeEach(() => {
@@ -69,3 +70,48 @@ describe("resolveUserReference", () => {
     expect(resolveUserReference()).toBe("Alice");
   });
 });
+
+describe("resolveGuardianName", () => {
+  beforeEach(() => {
+    mockFileExists = false;
+    mockFileContent = "";
+  });
+
+  test("returns USER.md name when present, ignoring guardianDisplayName", () => {
+    mockFileExists = true;
+    mockFileContent = [
+      "## Onboarding Snapshot",
+      "",
+      "- Preferred name/reference: John",
+    ].join("\n");
+    expect(resolveGuardianName("Jane")).toBe("John");
+  });
+
+  test('returns "my human" when USER.md explicitly sets name to default value', () => {
+    mockFileExists = true;
+    mockFileContent = [
+      "## Onboarding Snapshot",
+      "",
+      "- Preferred name/reference: my human",
+    ].join("\n");
+    // The user's explicit choice must be respected even though it matches the default sentinel
+    expect(resolveGuardianName("Jane")).toBe("my human");
+  });
+
+  test("falls back to guardianDisplayName when USER.md is empty", () => {
+    mockFileExists = false;
+    expect(resolveGuardianName("Jane")).toBe("Jane");
+  });
+
+  test("falls back to DEFAULT_USER_REFERENCE when both are empty", () => {
+    mockFileExists = false;
+    expect(resolveGuardianName()).toBe(DEFAULT_USER_REFERENCE);
+    expect(resolveGuardianName(null)).toBe(DEFAULT_USER_REFERENCE);
+    expect(resolveGuardianName("")).toBe(DEFAULT_USER_REFERENCE);
+  });
+
+  test("trims whitespace on guardianDisplayName fallback", () => {
+    mockFileExists = false;
+    expect(resolveGuardianName("  Jane  ")).toBe("Jane");
+  });
+});

package/src/__tests__/voice-scoped-grant-consumer.test.ts

@@ -38,7 +38,6 @@ mock.module("../util/platform.js", () => ({
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),
   ensureDataDir: () => {},
-  readHttpToken: () => null,
 }));
 
 mock.module("../util/logger.js", () => ({

package/src/__tests__/workspace-git-service.test.ts

@@ -655,7 +655,7 @@ describe("WorkspaceGitService", () => {
         cwd: testDir,
       });
       const oldGitignore =
-        "# Runtime state - excluded from git tracking\ndata/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\nhttp-token\n";
+        "# Runtime state - excluded from git tracking\ndata/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\n";
       writeFileSync(join(testDir, ".gitignore"), oldGitignore);
       writeFileSync(join(testDir, "file.txt"), "content");
       execFileSync("git", ["add", "-A"], { cwd: testDir });
@@ -725,7 +725,7 @@ describe("WorkspaceGitService", () => {
         cwd: testDir,
       });
       const gitignoreContent =
-        "# Runtime state - excluded from git tracking\ndata/db/\ndata/qdrant/\ndata/ipc-blobs/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\nhttp-token\n";
+        "# Runtime state - excluded from git tracking\ndata/db/\ndata/qdrant/\ndata/ipc-blobs/\nlogs/\n*.log\n*.sock\n*.pid\n*.sqlite\n*.sqlite-journal\n*.sqlite-wal\n*.sqlite-shm\n*.db\n*.db-journal\n*.db-wal\n*.db-shm\nvellum.sock\nvellum.pid\nsession-token\n";
       writeFileSync(join(testDir, ".gitignore"), gitignoreContent);
       writeFileSync(join(testDir, "file.txt"), "content");
       execFileSync("git", ["add", "-A"], { cwd: testDir });

package/src/calls/relay-server.ts

@@ -10,7 +10,7 @@ import { randomInt } from "node:crypto";
 
 import type { ServerWebSocket } from "bun";
 
-import { resolveUserReference } from "../config/user-reference.js";
+import { resolveGuardianName } from "../config/user-reference.js";
 import {
   findGuardianForChannel,
   listGuardianChannels,
@@ -31,7 +31,6 @@ import {
   toTrustContext,
 } from "../runtime/actor-trust-resolver.js";
 import { DAEMON_INTERNAL_ASSISTANT_ID } from "../runtime/assistant-scope.js";
-import { getGuardianBinding } from "../runtime/channel-guardian-service.js";
 import {
   composeVerificationVoice,
   GUARDIAN_VERIFY_TEMPLATE_KEYS,
@@ -1597,70 +1596,21 @@ export class RelayConnection {
 
   /**
    * Resolve a human-readable guardian label for voice wait copy.
-   * Prefers displayName from the guardian binding metadata, falls back
-   * to @username, then the user's preferred name from USER.md.
+   * Delegates to the shared resolveGuardianName() which checks USER.md
+   * first, then falls back to Contact.displayName, then DEFAULT_USER_REFERENCE.
    */
   private resolveGuardianLabel(): string {
     const assistantId =
       this.accessRequestAssistantId ?? DAEMON_INTERNAL_ASSISTANT_ID;
 
-    // Try the voice-channel binding first, then fall back to any active
-    // binding for the assistant (mirrors the cross-channel fallback pattern
-    // in access-request-helper.ts).
-    let metadataJson: string | null = null;
-    // Contacts-first: prefer the voice-bound guardian, then fall back to
-    // any guardian channel (mirrors the voice-first pattern in the legacy path).
+    // Look up the guardian contact for a displayName fallback
     const voiceGuardian = findGuardianForChannel("voice", assistantId);
     const guardianChannels = voiceGuardian
       ? null
       : listGuardianChannels(assistantId);
     const guardianContact = voiceGuardian?.contact ?? guardianChannels?.contact;
-    if (guardianContact) {
-      const meta: Record<string, string> = {};
-      if (guardianContact.displayName) {
-        meta.displayName = guardianContact.displayName;
-      }
-      // Preserve the username fallback: use the voice channel's externalUserId
-      // so downstream parsing can fall back to @username when displayName is a
-      // raw external ID (e.g., phone number from contact-sync).
-      const voiceChannel =
-        voiceGuardian?.channel ??
-        guardianChannels?.channels.find((ch) => ch.type === "voice");
-      if (voiceChannel?.externalUserId) {
-        meta.username = voiceChannel.externalUserId;
-      }
-      if (Object.keys(meta).length > 0) {
-        metadataJson = JSON.stringify(meta);
-      }
-    }
-    if (!metadataJson) {
-      const voiceBinding = getGuardianBinding(assistantId, "voice");
-      if (voiceBinding?.metadataJson) {
-        metadataJson = voiceBinding.metadataJson;
-      }
-    }
-
-    if (metadataJson) {
-      try {
-        const parsed = JSON.parse(metadataJson) as Record<string, unknown>;
-        if (
-          typeof parsed.displayName === "string" &&
-          parsed.displayName.trim().length > 0
-        ) {
-          return parsed.displayName.trim();
-        }
-        if (
-          typeof parsed.username === "string" &&
-          parsed.username.trim().length > 0
-        ) {
-          return `@${parsed.username.trim()}`;
-        }
-      } catch {
-        // ignore malformed metadata
-      }
-    }
 
-    return resolveUserReference();
+    return resolveGuardianName(guardianContact?.displayName);
   }
 
   /**

package/src/channels/config.ts

@@ -1,5 +1,5 @@
 /**
- * Canonical per-channel notification policy registry.
+ * Canonical per-channel policy registry.
  *
  * Every ChannelId must have an entry here. The `satisfies` constraint
  * ensures that adding a new ChannelId to channels/types.ts will fail
@@ -13,11 +13,17 @@ export type ConversationStrategy =
   | "continue_existing_conversation"
   | "not_deliverable";
 
+export interface ChannelInvitePolicy {
+  /** Whether inbound invite code redemption is supported on this channel. */
+  codeRedemptionEnabled: boolean;
+}
+
 export interface ChannelNotificationPolicy {
   notification: {
     deliveryEnabled: boolean;
     conversationStrategy: ConversationStrategy;
   };
+  invite: ChannelInvitePolicy;
 }
 
 const CHANNEL_POLICIES = {
@@ -26,48 +32,69 @@ const CHANNEL_POLICIES = {
       deliveryEnabled: true,
       conversationStrategy: "start_new_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: false,
+    },
   },
   telegram: {
     notification: {
       deliveryEnabled: true,
       conversationStrategy: "continue_existing_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: true,
+    },
   },
   sms: {
     notification: {
       deliveryEnabled: true,
       conversationStrategy: "continue_existing_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: true,
+    },
   },
   whatsapp: {
     notification: {
       deliveryEnabled: false,
       conversationStrategy: "continue_existing_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: false,
+    },
   },
   slack: {
     notification: {
       deliveryEnabled: true,
       conversationStrategy: "continue_existing_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: true,
+    },
   },
   email: {
     notification: {
       deliveryEnabled: false,
       conversationStrategy: "continue_existing_conversation",
     },
+    invite: {
+      codeRedemptionEnabled: true,
+    },
   },
   voice: {
     notification: {
       deliveryEnabled: false,
       conversationStrategy: "not_deliverable",
     },
+    invite: {
+      codeRedemptionEnabled: false,
+    },
   },
 } as const satisfies Record<ChannelId, ChannelNotificationPolicy>;
 
 export type ChannelPolicies = typeof CHANNEL_POLICIES;
 
-/** Returns the full notification policy for a channel. */
+/** Returns the full policy for a channel. */
 export function getChannelPolicy(
   channelId: ChannelId,
 ): ChannelNotificationPolicy {
@@ -97,3 +124,15 @@ export function getConversationStrategy(
 ): ConversationStrategy {
   return CHANNEL_POLICIES[channelId].notification.conversationStrategy;
 }
+
+/** Returns the invite policy for the given channel. */
+export function getChannelInvitePolicy(
+  channelId: ChannelId,
+): ChannelInvitePolicy {
+  return CHANNEL_POLICIES[channelId].invite;
+}
+
+/** Whether invite code redemption is enabled for the given channel. */
+export function isInviteCodeRedemptionEnabled(channelId: ChannelId): boolean {
+  return CHANNEL_POLICIES[channelId].invite.codeRedemptionEnabled;
+}

package/src/config/bundled-skills/slack/SKILL.md

@@ -69,6 +69,8 @@ When you need to **send** content to Slack proactively (e.g. a scheduled digest,
 - `send_notification` is appropriate for short alerts and status updates where you want the router to pick the best channel. `messaging_send` is appropriate when you have specific content to deliver to a specific Slack destination.
 - For scheduled tasks (cron/RRULE), always end with a `messaging_send` call so the results actually reach the user. Without it, the output only lives in the conversation log.
 
+For setting up recurring digests, load the `slack-digest-setup` skill which covers the full configuration, scheduling, and delivery protocol.
+
 ## Watcher Integration
 
 For real-time monitoring (not just on-demand scanning), the user can set up a Slack watcher using the watcher skill with the same channel IDs. Mention this if the user wants ongoing monitoring.