@vellumai/assistant 0.10.4 → 0.10.5-dev.202607022329.d89bf2f

package/src/notifications/approval-card-builder.ts

@@ -70,21 +70,38 @@ export type ApprovalCardBlock = z.infer<typeof ApprovalCardBlockSchema>;
 
 // ── Public types ────────────────────────────────────────────────────────────
 
+/** A card action option resolved from the request's domain (id + label + style). */
+export interface ApprovalCardActionOption {
+  /** Canonical action id — becomes the `apr:<requestId>:<id>` callback. */
+  id: string;
+  label: string;
+  style?: SurfaceAction["style"];
+}
+
 export interface ApprovalCardParams {
   /** Prefix for `surfaceId` — combined with `requestId` to form e.g. "access-request-abc123". */
   surfaceIdPrefix: string;
   /** Top-level card title shown in the surface header (e.g. "Access Request", "Tool approval"). */
   cardTitle: string;
-  /** Primary line inside the card (typically the requester's display name). */
-  requesterName: string;
-  /** Secondary line below the requester name (e.g. "Requesting access to the assistant"). */
+  /**
+   * Primary line inside the card — the subject of the decision: the
+   * requester's display name for access requests, the tool name for tool
+   * approvals.
+   */
+  primaryLine: string;
+  /** Secondary line below the primary line (e.g. "Requesting access to the assistant"). */
   subtitle: string;
   /** Markdown body — blockquotes, warnings, links, etc. */
   body: string;
   /** Key-value metadata grid rendered below the body. */
   metadata: Array<{ label: string; value: string }>;
-  /** When present, Approve/Reject buttons are wired with `apr:<requestId>:*` action IDs. */
+  /** When present, action buttons are wired with `apr:<requestId>:*` action IDs. */
   requestId?: string;
+  /**
+   * Domain-specific action set (e.g. the signal-driven introduction-card
+   * actions). When absent, the generic Approve/Reject pair is rendered.
+   */
+  actions?: ApprovalCardActionOption[];
   /** Plain-text content for the fallback `text` block. */
   fallbackText: string;
 }
@@ -105,7 +122,7 @@ export function buildApprovalCardBlocks(
   const {
     surfaceIdPrefix,
     cardTitle,
-    requesterName,
+    primaryLine,
     subtitle,
     body,
     metadata,
@@ -113,23 +130,21 @@ export function buildApprovalCardBlocks(
     fallbackText,
   } = params;
 
+  const actionOptions: ApprovalCardActionOption[] = params.actions ?? [
+    { id: "approve_once", label: "Approve", style: "primary" },
+    { id: "reject", label: "Reject", style: "destructive" },
+  ];
+
   const actions: SurfaceAction[] | undefined = requestId
-    ? [
-        {
-          id: `apr:${requestId}:approve_once`,
-          label: "Approve",
-          style: "primary",
-        },
-        {
-          id: `apr:${requestId}:reject`,
-          label: "Reject",
-          style: "destructive",
-        },
-      ]
+    ? actionOptions.map((option) => ({
+        id: `apr:${requestId}:${option.id}`,
+        label: option.label,
+        ...(option.style ? { style: option.style } : {}),
+      }))
     : undefined;
 
   const data: CardSurfaceData = {
-    title: requesterName,
+    title: primaryLine,
     subtitle,
     body,
     metadata,

package/src/notifications/approval-card-data.ts

@@ -16,9 +16,11 @@
 import {
   buildAccessRequestCardView,
   buildAccessRequestContractText,
+  buildIntroductionActionsForPayload,
   parseAccessRequestPayload,
 } from "./access-request-copy.js";
 import {
+  type ApprovalCardActionOption,
   type ApprovalCardBlock,
   type ApprovalCardParams,
   buildApprovalCardBlocks,
@@ -97,7 +99,18 @@ export type ApprovalCardData = AccessRequestCardData | ToolApprovalCardData;
 function resolveAccessRequestCard(
   payload: Record<string, unknown>,
 ): ApprovalCardParams {
-  const view = buildAccessRequestCardView(parseAccessRequestPayload(payload));
+  const parsed = parseAccessRequestPayload(payload);
+  const view = buildAccessRequestCardView(parsed);
+
+  // Signal-driven introduction actions; the emphasis (primary lead,
+  // destructive Block) is resolved by introduction-policy and maps 1:1 onto
+  // the Surface style vocabulary.
+  const actions: ApprovalCardActionOption[] =
+    buildIntroductionActionsForPayload(parsed).map((action) => ({
+      id: action.id,
+      label: action.label,
+      style: action.emphasis,
+    }));
 
   const metadata: Array<{ label: string; value: string }> = [];
 
@@ -139,11 +152,12 @@ function resolveAccessRequestCard(
   return {
     surfaceIdPrefix: ACCESS_REQUEST_SURFACE_PREFIX,
     cardTitle: "Access Request",
-    requesterName: view.displayName,
+    primaryLine: view.displayName,
     subtitle: "Requesting access to the assistant",
     body,
     metadata,
     requestId: view.requestId,
+    actions,
     fallbackText: buildAccessRequestContractText(payload),
   };
 }
@@ -170,14 +184,22 @@ function isLenientToolApproval(payload: LenientToolApprovalPayload): boolean {
     payload.requestKind,
     payload.toolName,
   );
-  if (!modeResolution) return false;
+  if (!modeResolution) {
+    return false;
+  }
   return (
     modeResolution.mode === "approval" &&
     payload.requestKind !== "access_request"
   );
 }
 
-/** Shape a tool-approval/grant payload (strict or lenient) into card params. */
+/**
+ * Shape a tool-approval/grant payload (strict or lenient) into card params.
+ *
+ * The card is about the tool: the primary line names the tool awaiting
+ * approval. The requester appears only as metadata context, never as the
+ * subject of the decision.
+ */
 function extractToolApprovalCard(
   p: GuardianQuestionPayload | LenientToolApprovalPayload,
 ): ApprovalCardParams {
@@ -186,12 +208,16 @@ function extractToolApprovalCard(
   const rawRequester = nonEmpty(p.requesterIdentifier);
   const requester = rawRequester
     ? sanitizeIdentityField(rawRequester)
-    : "Someone";
+    : undefined;
 
   const isGrant = p.requestKind === "tool_grant_request";
 
+  // Who is asking is a fact the guardian weighs, so the row always renders:
+  // an unresolvable requester surfaces as "Unknown" rather than a silently
+  // missing row. In practice the producers always carry at least the raw
+  // channel user ID — this placeholder covers defensive/lenient parses.
   const metadata: Array<{ label: string; value: string }> = [];
-  metadata.push({ label: "Tool", value: toolName });
+  metadata.push({ label: "Requested by", value: requester ?? "Unknown" });
   const sourceChannel = nonEmpty(p.sourceChannel);
   if (sourceChannel) {
     metadata.push({ label: "Source", value: sourceChannel });
@@ -203,7 +229,8 @@ function extractToolApprovalCard(
 
   // Fallback text with request-code instructions for older clients.
   const baseFallback =
-    p.questionText ?? `${requester} is requesting approval to use ${toolName}`;
+    p.questionText ??
+    `Approve tool: ${toolName} (requested by ${requester ?? "Unknown"})`;
   let fallbackText = baseFallback;
   const requestCode = nonEmpty(p.requestCode);
   if (requestCode) {
@@ -222,10 +249,8 @@ function extractToolApprovalCard(
   return {
     surfaceIdPrefix: TOOL_APPROVAL_SURFACE_PREFIX,
     cardTitle: isGrant ? "Tool Grant Request" : "Tool Approval",
-    requesterName: requester,
-    subtitle: isGrant
-      ? "Requesting permission to use this tool"
-      : "Requesting approval to run this tool",
+    primaryLine: toolName,
+    subtitle: "Requires your approval to run",
     body,
     metadata,
     requestId: nonEmpty(p.requestId),
@@ -246,13 +271,19 @@ function resolveToolApprovalCard(
 ): ApprovalCardParams | null {
   const strict = parseGuardianQuestionPayload(payload);
   if (strict) {
-    if (!isToolApprovalPayload(strict)) return null;
+    if (!isToolApprovalPayload(strict)) {
+      return null;
+    }
     return extractToolApprovalCard(strict);
   }
 
   const lenient = LenientToolApprovalPayloadSchema.safeParse(payload);
-  if (!lenient.success) return null;
-  if (!isLenientToolApproval(lenient.data)) return null;
+  if (!lenient.success) {
+    return null;
+  }
+  if (!isLenientToolApproval(lenient.data)) {
+    return null;
+  }
   return extractToolApprovalCard(lenient.data);
 }
 
@@ -268,7 +299,9 @@ export function resolveApprovalCardData(
   sourceEventName: string,
   contextPayload: Record<string, unknown> | undefined,
 ): ApprovalCardData | null {
-  if (!contextPayload) return null;
+  if (!contextPayload) {
+    return null;
+  }
 
   if (sourceEventName === "ingress.access_request") {
     return {

package/src/notifications/broadcaster.ts

@@ -17,6 +17,7 @@ import type { ApprovalUIMetadata } from "../runtime/channel-approval-types.js";
 import { getLogger } from "../util/logger.js";
 import {
   buildAccessRequestContractText,
+  buildIntroductionActionsForPayload,
   parseAccessRequestPayload,
 } from "./access-request-copy.js";
 import { isGuardianSensitiveEvent } from "./adapters/macos.js";
@@ -28,10 +29,7 @@ import {
   updateDeliveryStatus,
 } from "./deliveries-store.js";
 import { resolveDestinations } from "./destination-resolver.js";
-import {
-  parseGuardianQuestionPayload,
-  resolveGuardianInstructionModeFromPayload,
-} from "./guardian-question-mode.js";
+import { parseInteractiveApprovalPayload } from "./guardian-question-mode.js";
 import { nonEmpty } from "./notification-utils.js";
 import type { NotificationSignal } from "./signal.js";
 import type {
@@ -59,27 +57,32 @@ function resolveApprovalContext(
   signal: NotificationSignal,
 ): ApprovalUIMetadata | undefined {
   const payload = signal.contextPayload;
-  if (!payload) return undefined;
+  if (!payload) {
+    return undefined;
+  }
 
   if (signal.sourceEventName === "ingress.access_request") {
     const requestId = nonEmpty(
       typeof payload.requestId === "string" ? payload.requestId : undefined,
     );
-    if (!requestId) return undefined;
+    if (!requestId) {
+      return undefined;
+    }
     return {
       requestId,
-      actions: APPROVAL_ACTIONS,
+      actions: buildIntroductionActionsForPayload(
+        parseAccessRequestPayload(payload),
+      ),
       plainTextFallback: buildAccessRequestContractText(payload),
     };
   }
 
   if (signal.sourceEventName === "guardian.question") {
-    const parsed = parseGuardianQuestionPayload(payload);
-    if (!parsed) return undefined;
-    const { mode } = resolveGuardianInstructionModeFromPayload(parsed);
-    if (mode !== "approval") return undefined;
-    const requestId = nonEmpty(parsed.requestId);
-    if (!requestId) return undefined;
+    const parsed = parseInteractiveApprovalPayload(payload);
+    if (!parsed) {
+      return undefined;
+    }
+    const requestId = parsed.requestId;
 
     // Extract tool context so channel adapters can render structured
     // approval cards without re-parsing contextPayload.
@@ -184,8 +187,12 @@ export class NotificationBroadcaster {
     // event fires immediately, before slower channel sends (e.g. Telegram 30s
     // timeout) can delay it past the macOS deep-link retry window.
     const orderedChannels = [...decision.selectedChannels].sort((a, b) => {
-      if (a === "vellum") return -1;
-      if (b === "vellum") return 1;
+      if (a === "vellum") {
+        return -1;
+      }
+      if (b === "vellum") {
+        return 1;
+      }
       return 0;
     });
 
@@ -552,7 +559,9 @@ export class NotificationBroadcaster {
 function resolveSourceConversationId(
   sourceContextId: string | undefined,
 ): string | undefined {
-  if (!sourceContextId) return undefined;
+  if (!sourceContextId) {
+    return undefined;
+  }
   try {
     return getConversation(sourceContextId) ? sourceContextId : undefined;
   } catch {

package/src/notifications/decision-engine.ts

@@ -32,6 +32,8 @@ import {
   buildAccessRequestInviteDirective,
   hasAccessRequestInstructions,
   hasInviteFlowDirective,
+  isHandshakeOfferedForPayload,
+  parseAccessRequestPayload,
 } from "./access-request-copy.js";
 import {
   buildAccessRequestSeedContentBlocks,
@@ -47,8 +49,10 @@ import { createDecision } from "./decisions-store.js";
 import {
   buildGuardianRequestCodeInstruction,
   hasGuardianRequestCodeInstruction,
+  parseInteractiveApprovalPayload,
   resolveGuardianQuestionInstructionMode,
   stripConflictingGuardianRequestInstructions,
+  stripGuardianRequestCodeInstructions,
 } from "./guardian-question-mode.js";
 import { nonEmpty, readPayloadString } from "./notification-utils.js";
 import { getPreferenceSummary } from "./preference-summary.js";
@@ -143,6 +147,7 @@ function buildSystemPrompt(
     `  - Avoid meta-send phrasing (e.g. "I'd like to send a notification", "May I go ahead with that?"). Write the recipient-facing message directly.`,
     `  - Avoid intermediary-instruction phrasing like "consider telling the guardian", "ask the recipient to", or "the assistant should remind them". Rewrite it as final copy the recipient can act on directly.`,
     `  - For telegram: 1-2 concise sentences.`,
+    `  - For slack approval requests: the message renders inside an interactive card with Approve/Reject buttons. Describe only what needs approval — never include approval/reference codes, code-reply instructions, or directions to respond in another app.`,
     `- \`conversationSeedMessage\` is the opening message in the internal notification conversation and also the expanded detail shown in the home feed. It should be richer and more contextual than the popup body.`,
     `  - For vellum (desktop): use structured markdown for readability. Break content into bullet points, numbered lists, or short sections with **bold** labels. Avoid long unbroken paragraphs — scan-friendly formatting is preferred.`,
     `  - Never dump raw JSON. Include only human-readable context.`,
@@ -382,11 +387,19 @@ function validateDecisionOutput(
   availableChannels: NotificationChannel[],
   candidateSet?: ConversationCandidateSet,
 ): NotificationDecision | null {
-  if (typeof input.shouldNotify !== "boolean") return null;
-  if (typeof input.reasoningSummary !== "string") return null;
-  if (typeof input.dedupeKey !== "string") return null;
+  if (typeof input.shouldNotify !== "boolean") {
+    return null;
+  }
+  if (typeof input.reasoningSummary !== "string") {
+    return null;
+  }
+  if (typeof input.dedupeKey !== "string") {
+    return null;
+  }
 
-  if (!Array.isArray(input.selectedChannels)) return null;
+  if (!Array.isArray(input.selectedChannels)) {
+    return null;
+  }
   const validatedChannels = (input.selectedChannels as unknown[]).filter(
     (ch): ch is NotificationChannel =>
       typeof ch === "string" &&
@@ -482,7 +495,9 @@ export function validateConversationActions(
 ): Partial<Record<NotificationChannel, ConversationAction>> {
   const result: Partial<Record<NotificationChannel, ConversationAction>> = {};
 
-  if (!raw || typeof raw !== "object") return result;
+  if (!raw || typeof raw !== "object") {
+    return result;
+  }
 
   const actionsObj = raw as Record<string, unknown>;
   const channelSet = new Set(validChannels);
@@ -502,8 +517,12 @@ export function validateConversationActions(
   }
 
   for (const [ch, actionRaw] of Object.entries(actionsObj)) {
-    if (!channelSet.has(ch as NotificationChannel)) continue;
-    if (!actionRaw || typeof actionRaw !== "object") continue;
+    if (!channelSet.has(ch as NotificationChannel)) {
+      continue;
+    }
+    if (!actionRaw || typeof actionRaw !== "object") {
+      continue;
+    }
 
     const channel = ch as NotificationChannel;
     const action = actionRaw as Record<string, unknown>;
@@ -558,8 +577,9 @@ function ensureGuardianRequestCodeInCopy(
       requestCode,
       mode,
     );
-    if (hasGuardianRequestCodeInstruction(sanitized, requestCode, mode))
+    if (hasGuardianRequestCodeInstruction(sanitized, requestCode, mode)) {
       return sanitized;
+    }
     return sanitized.length > 0
       ? `${sanitized}\n\n${instruction}`
       : instruction;
@@ -577,36 +597,79 @@ function ensureGuardianRequestCodeInCopy(
   };
 }
 
+/**
+ * Remove request-code reply instructions from copy for a channel whose
+ * approval UI makes them redundant. Instruction-only fields keep their
+ * original text rather than becoming empty — downstream treats an empty
+ * body as missing copy.
+ */
+function stripGuardianRequestCodeInCopy(
+  copy: RenderedChannelCopy,
+  requestCode: string,
+): RenderedChannelCopy {
+  const strip = (text: string): string => {
+    const stripped = stripGuardianRequestCodeInstructions(text, requestCode);
+    return stripped.length > 0 ? stripped : text;
+  };
+
+  return {
+    ...copy,
+    body: strip(copy.body),
+    deliveryText: copy.deliveryText
+      ? strip(copy.deliveryText)
+      : copy.deliveryText,
+    conversationSeedMessage: copy.conversationSeedMessage
+      ? strip(copy.conversationSeedMessage)
+      : copy.conversationSeedMessage,
+  };
+}
+
 /**
  * Guardian questions that share a conversation require explicit request-code
  * targeting. Enforce request-code instructions in rendered copy so guardians
  * can always disambiguate replies even when model copy omits them.
+ *
+ * Slack is the exception for approval-mode questions: the Slack adapter
+ * renders those as an interactive card with Approve/Reject buttons (see
+ * `resolveApprovalContext` in broadcaster.ts, gated on the same
+ * `parseInteractiveApprovalPayload`), so code-reply instructions are
+ * redundant noise there and are stripped instead of enforced in.
  */
 function enforceGuardianRequestCode(
   decision: NotificationDecision,
   signal: NotificationSignal,
 ): NotificationDecision {
-  if (signal.sourceEventName !== "guardian.question") return decision;
+  if (signal.sourceEventName !== "guardian.question") {
+    return decision;
+  }
   const rawCode = signal.contextPayload.requestCode;
-  if (typeof rawCode !== "string" || rawCode.trim().length === 0)
+  if (typeof rawCode !== "string" || rawCode.trim().length === 0) {
     return decision;
+  }
 
   const requestCode = rawCode.trim().toUpperCase();
   const modeResolution = resolveGuardianQuestionInstructionMode(
     signal.contextPayload,
   );
+  const slackRendersApprovalButtons =
+    parseInteractiveApprovalPayload(signal.contextPayload) != null;
   const nextCopy: Partial<Record<NotificationChannel, RenderedChannelCopy>> = {
     ...decision.renderedCopy,
   };
 
   for (const channel of Object.keys(nextCopy) as NotificationChannel[]) {
     const copy = nextCopy[channel];
-    if (!copy) continue;
-    nextCopy[channel] = ensureGuardianRequestCodeInCopy(
-      copy,
-      requestCode,
-      modeResolution.mode,
-    );
+    if (!copy) {
+      continue;
+    }
+    nextCopy[channel] =
+      channel === "slack" && slackRendersApprovalButtons
+        ? stripGuardianRequestCodeInCopy(copy, requestCode)
+        : ensureGuardianRequestCodeInCopy(
+            copy,
+            requestCode,
+            modeResolution.mode,
+          );
   }
 
   return {
@@ -624,14 +687,18 @@ function ensureSeedContentBlocks(
   decision: NotificationDecision,
   blocks: unknown[] | null | undefined,
 ): NotificationDecision {
-  if (!blocks) return decision;
+  if (!blocks) {
+    return decision;
+  }
 
   const nextCopy: Partial<Record<NotificationChannel, RenderedChannelCopy>> = {
     ...decision.renderedCopy,
   };
   for (const channel of Object.keys(nextCopy) as NotificationChannel[]) {
     const copy = nextCopy[channel];
-    if (!copy) continue;
+    if (!copy) {
+      continue;
+    }
     if (!copy.seedContentBlocks) {
       nextCopy[channel] = { ...copy, seedContentBlocks: blocks };
     }
@@ -651,7 +718,9 @@ function enforceToolApprovalSeedBlocks(
   decision: NotificationDecision,
   signal: NotificationSignal,
 ): NotificationDecision {
-  if (signal.sourceEventName !== "guardian.question") return decision;
+  if (signal.sourceEventName !== "guardian.question") {
+    return decision;
+  }
   return ensureSeedContentBlocks(
     decision,
     buildToolApprovalSeedContentBlocks(signal.contextPayload),
@@ -675,7 +744,9 @@ function enforceAccessRequestInstructions(
   decision: NotificationDecision,
   signal: NotificationSignal,
 ): NotificationDecision {
-  if (signal.sourceEventName !== "ingress.access_request") return decision;
+  if (signal.sourceEventName !== "ingress.access_request") {
+    return decision;
+  }
 
   const rawCode = signal.contextPayload.requestCode;
   const hasRequestCode =
@@ -688,14 +759,20 @@ function enforceAccessRequestInstructions(
   if (hasRequestCode) {
     const requestCode = rawCode.trim().toUpperCase();
     const contractText = buildAccessRequestContractText(signal.contextPayload);
+    const handshakeOffered = isHandshakeOfferedForPayload(
+      parseAccessRequestPayload(signal.contextPayload),
+    );
 
     for (const channel of Object.keys(nextCopy) as NotificationChannel[]) {
       const copy = nextCopy[channel];
-      if (!copy) continue;
+      if (!copy) {
+        continue;
+      }
       nextCopy[channel] = ensureAccessRequestInstructionsInCopy(
         copy,
         requestCode,
         contractText,
+        handshakeOffered,
       );
     }
   } else {
@@ -704,7 +781,9 @@ function enforceAccessRequestInstructions(
 
     for (const channel of Object.keys(nextCopy) as NotificationChannel[]) {
       const copy = nextCopy[channel];
-      if (!copy) continue;
+      if (!copy) {
+        continue;
+      }
       nextCopy[channel] = ensureInviteFlowDirectiveInCopy(
         copy,
         inviteDirective,
@@ -730,10 +809,13 @@ function ensureAccessRequestInstructionsInCopy(
   copy: RenderedChannelCopy,
   requestCode: string,
   contractText: string,
+  handshakeOffered: boolean,
 ): RenderedChannelCopy {
   const ensureText = (text: string | undefined): string => {
     const base = typeof text === "string" ? text.trim() : "";
-    if (hasAccessRequestInstructions(base, requestCode)) return base;
+    if (hasAccessRequestInstructions(base, requestCode, { handshakeOffered })) {
+      return base;
+    }
     return base.length > 0 ? `${base}\n\n${contractText}` : contractText;
   };
 
@@ -755,7 +837,9 @@ function ensureInviteFlowDirectiveInCopy(
 ): RenderedChannelCopy {
   const ensureText = (text: string | undefined): string => {
     const base = typeof text === "string" ? text.trim() : "";
-    if (hasInviteFlowDirective(base)) return base;
+    if (hasInviteFlowDirective(base)) {
+      return base;
+    }
     return base.length > 0 ? `${base}\n\n${inviteDirective}` : inviteDirective;
   };
 
@@ -1138,15 +1222,21 @@ export function enforceRoutingIntent(
       // Preserve the decision's selected channels first, then add connected
       // channels until we reach two channels total.
       for (const ch of selectedConnected) {
-        if (seen.has(ch)) continue;
+        if (seen.has(ch)) {
+          continue;
+        }
         expanded.push(ch);
         seen.add(ch);
       }
       for (const ch of connectedChannels) {
-        if (seen.has(ch)) continue;
+        if (seen.has(ch)) {
+          continue;
+        }
         expanded.push(ch);
         seen.add(ch);
-        if (expanded.length >= 2) break;
+        if (expanded.length >= 2) {
+          break;
+        }
       }
 
       const enforced = { ...decision };
@@ -1184,15 +1274,20 @@ export function enforceGuardianCallConversationAffinity(
   decision: NotificationDecision,
   signal: NotificationSignal,
 ): NotificationDecision {
-  if (signal.sourceEventName !== "guardian.question") return decision;
+  if (signal.sourceEventName !== "guardian.question") {
+    return decision;
+  }
 
   const callSessionId = signal.contextPayload?.callSessionId;
-  if (typeof callSessionId !== "string" || callSessionId.trim().length === 0)
+  if (typeof callSessionId !== "string" || callSessionId.trim().length === 0) {
     return decision;
+  }
 
   // If an affinity hint already exists for vellum, the second+ dispatch
   // will be handled by enforceConversationAffinity — nothing to do here.
-  if (signal.conversationAffinityHint?.vellum) return decision;
+  if (signal.conversationAffinityHint?.vellum) {
+    return decision;
+  }
 
   const enforced = { ...decision };
   const conversationActions: Partial<
@@ -1226,13 +1321,17 @@ function enforceConversationAffinity(
   decision: NotificationDecision,
   affinityHint: Partial<Record<string, string>> | undefined,
 ): NotificationDecision {
-  if (!affinityHint) return decision;
+  if (!affinityHint) {
+    return decision;
+  }
 
   const entries = Object.entries(affinityHint).filter(
     ([, conversationId]) =>
       typeof conversationId === "string" && conversationId.length > 0,
   );
-  if (entries.length === 0) return decision;
+  if (entries.length === 0) {
+    return decision;
+  }
 
   const enforced = { ...decision };
   const conversationActions: Partial<