@checkstack/anomaly-backend 0.2.1 → 1.0.1

package/src/detector.test.ts

@@ -43,6 +43,14 @@ function createMockCatalogClient() {
   };
 }
 
+function createMockNotificationClient(subscriberIds: string[] = ["user-1"]) {
+  return {
+    notifyForSubscription: mock(async () => ({
+      notifiedCount: subscriberIds.length,
+    })),
+  };
+}
+
 function createMockLogger() {
   return {
     debug: mock(() => {}),
@@ -248,6 +256,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     expect(cache.get).not.toHaveBeenCalled();
   });
@@ -263,6 +272,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     expect(cache.get).not.toHaveBeenCalled();
   });
@@ -280,6 +290,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     // Two fields (statusText, isRunning) means cache.get should be called twice
     expect(cache.get).toHaveBeenCalledTimes(2);
@@ -301,6 +312,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     
     expect(db._insertCalls.length).toBe(1);
@@ -326,6 +338,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     expect(cache.get).toHaveBeenCalledTimes(1);
     expect(db._insertCalls.length).toBe(0);
@@ -345,6 +358,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     expect(db._insertCalls.length).toBe(0);
   });
@@ -363,6 +377,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
     expect(db._insertCalls.length).toBe(1);
     expect(db._insertCalls[0]).toMatchObject({
@@ -380,6 +395,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
     const baseline = createBaseline({ mean: 100, stdDev: 10 });
     const cache = createMockCache(new Map([[cacheKeyPrefix, baseline]]));
     const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient(["user-1"]);
     const db = createMockDb({
       existingAnomaly: {
         id: "anomaly-existing",
@@ -400,18 +416,19 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
     });
 
     expect(db._updateCalls.length).toBe(1);
     expect(db._updateCalls[0]).toMatchObject({ state: "anomaly" });
 
-    expect(catalogClient.notifySystemSubscribers).toHaveBeenCalledTimes(1);
-    const notifArgs = (catalogClient.notifySystemSubscribers as Mock<(...args: unknown[]) => unknown>).mock.calls[0] as unknown[];
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const notifArgs = (notificationClient.notifyForSubscription as Mock<(...args: unknown[]) => unknown>).mock.calls[0] as unknown[];
     const notifPayload = notifArgs[0] as Record<string, unknown>;
     expect(notifPayload).toMatchObject({
-      systemId,
+      specId: "anomaly.system",
+      resourceKeys: [systemId],
       importance: "warning",
-      includeGroupSubscribers: true,
     });
     expect(notifPayload.title).toContain("Anomaly Detected");
   });
@@ -422,6 +439,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
     const baseline = createBaseline({ mean: 100, stdDev: 10 });
     const cache = createMockCache(new Map([[cacheKeyPrefix, baseline]]));
     const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient(["user-1"]);
     const db = createMockDb({
       existingAnomaly: {
         id: "anomaly-confirmed",
@@ -442,16 +460,18 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
     });
 
     expect(db._updateCalls.length).toBe(1);
     expect(db._updateCalls[0]).toMatchObject({ state: "recovered" });
 
-    expect(catalogClient.notifySystemSubscribers).toHaveBeenCalledTimes(1);
-    const notifArgs = (catalogClient.notifySystemSubscribers as Mock<(...args: unknown[]) => unknown>).mock.calls[0] as unknown[];
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const notifArgs = (notificationClient.notifyForSubscription as Mock<(...args: unknown[]) => unknown>).mock.calls[0] as unknown[];
     const notifPayload = notifArgs[0] as Record<string, unknown>;
     expect(notifPayload).toMatchObject({
-      systemId,
+      specId: "anomaly.system",
+      resourceKeys: [systemId],
       importance: "info",
     });
     expect(notifPayload.title).toContain("Recovered");
@@ -482,6 +502,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._deleteCalls.length).toBe(1);
@@ -507,6 +528,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._insertCalls.length).toBe(0);
@@ -535,6 +557,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(cache.get).toHaveBeenCalledTimes(2);
@@ -566,6 +589,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(cache.set).toHaveBeenCalledTimes(1);
@@ -593,6 +617,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._insertCalls.length).toBe(1);
@@ -618,6 +643,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._insertCalls.length).toBe(0);
@@ -639,6 +665,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._insertCalls.length).toBe(1);
@@ -661,6 +688,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     // Baseline lookup happens, but direction resolution short-circuits before insert.
@@ -687,6 +715,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     // No schema direction available, no config direction → silently skipped
@@ -718,6 +747,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     expect(db._updateCalls.length).toBe(1);
@@ -730,6 +760,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
     const baseline = createBaseline({ mean: 100, stdDev: 10 });
     const cache = createMockCache(new Map([[cacheKeyPrefix, baseline]]));
     const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient();
     const db = createMockDb({
       existingAnomaly: {
         id: "anomaly-confirmed",
@@ -750,11 +781,12 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
     });
 
     // Should update observed value but not send another notification
     expect(db._updateCalls.length).toBe(1);
-    expect(catalogClient.notifySystemSubscribers).not.toHaveBeenCalled();
+    expect(notificationClient.notifyForSubscription).not.toHaveBeenCalled();
   });
 
   // ─── Signal emission (F8) ─────────────────────────────────────────────
@@ -774,6 +806,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
       signalService,
     });
 
@@ -808,6 +841,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
       signalService,
     });
 
@@ -842,6 +876,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: createMockLogger() as never,
       catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
       signalService,
     });
 
@@ -883,6 +918,7 @@ describe("Anomaly Detector — processCheckCompleted", () => {
       cache,
       logger: logger as never,
       catalogClient: failingClient as never,
+      notificationClient: createMockNotificationClient() as never,
     });
 
     // State transition still happened despite notification failure

package/src/detector.ts

@@ -11,6 +11,7 @@ import {
 } from "@checkstack/anomaly-common";
 import type { Logger } from "@checkstack/backend-api";
 import type { CatalogApi } from "@checkstack/catalog-common";
+import type { NotificationApi } from "@checkstack/notification-common";
 import type { InferClient } from "@checkstack/common";
 import { AnomalyService } from "./service";
 import type { AnomalySettings, AnomalyDirection } from "@checkstack/anomaly-common";
@@ -38,6 +39,7 @@ export async function processCheckCompleted({
   routerCache,
   logger,
   catalogClient,
+  notificationClient,
   signalService,
   collectorRegistry,
 }: {
@@ -54,6 +56,7 @@ export async function processCheckCompleted({
   };
   logger: Logger;
   catalogClient: InferClient<typeof CatalogApi>;
+  notificationClient: InferClient<typeof NotificationApi>;
   signalService?: SignalService;
   collectorRegistry: CollectorRegistry;
 }) {
@@ -295,6 +298,8 @@ export async function processCheckCompleted({
             observedValue: value,
             baselineMean: baseline.mean,
             catalogClient,
+            notificationClient,
+            db,
             logger,
           });
         } else {
@@ -352,6 +357,8 @@ export async function processCheckCompleted({
             observedValue: value,
             baselineMean: baseline.mean,
             catalogClient,
+            notificationClient,
+            db,
             logger,
           });
         }

package/src/drift-evaluator.test.ts

@@ -21,6 +21,14 @@ function createMockCatalogClient() {
   };
 }
 
+function createMockNotificationClient(subscriberIds: string[] = ["user-1"]) {
+  return {
+    notifyForSubscription: mock(async () => ({
+      notifiedCount: subscriberIds.length,
+    })),
+  };
+}
+
 function createMockLogger() {
   return {
     debug: mock(() => {}),
@@ -132,6 +140,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -147,6 +156,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -161,6 +171,7 @@ describe("evaluateDrift", () => {
         templateConfig: { ...defaultTemplate, driftEnabled: false },
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -175,6 +186,7 @@ describe("evaluateDrift", () => {
         templateConfig: { ...defaultTemplate, enabled: false },
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -189,6 +201,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -206,6 +219,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
         signalService: signalService as never,
       });
@@ -234,6 +248,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._updateCalls.length).toBe(1);
@@ -250,6 +265,7 @@ describe("evaluateDrift", () => {
       };
       const db = createMockDb({ existingAnomaly: existing });
       const catalog = createMockCatalogClient();
+      const notification = createMockNotificationClient();
       const signalService = createMockSignalService();
       await evaluateDrift({
         ...baseProps,
@@ -258,12 +274,13 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: catalog as never,
+        notificationClient: notification as never,
         logger: createMockLogger() as never,
         signalService: signalService as never,
       });
       expect(db._updateCalls.length).toBe(1);
       expect(db._updateCalls[0].state).toBe("anomaly");
-      expect(catalog.notifySystemSubscribers).toHaveBeenCalledTimes(1);
+      expect(notification.notifyForSubscription).toHaveBeenCalledTimes(1);
       // Two signals: state change + trend detected
       expect(signalService.broadcast).toHaveBeenCalledTimes(2);
     });
@@ -277,6 +294,7 @@ describe("evaluateDrift", () => {
       };
       const db = createMockDb({ existingAnomaly: existing });
       const catalog = createMockCatalogClient();
+      const notification = createMockNotificationClient();
       await evaluateDrift({
         ...baseProps,
         baseline: driftingBaseline,
@@ -284,11 +302,12 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: catalog as never,
+        notificationClient: notification as never,
         logger: createMockLogger() as never,
       });
       expect(db._updateCalls.length).toBe(1);
       expect(db._updateCalls[0].state).toBeUndefined();
-      expect(catalog.notifySystemSubscribers).not.toHaveBeenCalled();
+      expect(notification.notifyForSubscription).not.toHaveBeenCalled();
     });
 
     test("deletes suspicious row when drift goes away before confirmation", async () => {
@@ -306,6 +325,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._deleteCalls.length).toBe(1);
@@ -320,6 +340,7 @@ describe("evaluateDrift", () => {
       };
       const db = createMockDb({ existingAnomaly: existing });
       const catalog = createMockCatalogClient();
+      const notification = createMockNotificationClient();
       await evaluateDrift({
         ...baseProps,
         baseline: stableBaseline,
@@ -327,11 +348,12 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: catalog as never,
+        notificationClient: notification as never,
         logger: createMockLogger() as never,
       });
       expect(db._updateCalls.length).toBe(1);
       expect(db._updateCalls[0].state).toBe("recovered");
-      expect(catalog.notifySystemSubscribers).toHaveBeenCalledTimes(1);
+      expect(notification.notifyForSubscription).toHaveBeenCalledTimes(1);
     });
 
     test("does nothing when no row and no drift (steady state)", async () => {
@@ -343,6 +365,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -362,6 +385,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db._insertCalls.length).toBe(0);
@@ -374,6 +398,7 @@ describe("evaluateDrift", () => {
         templateConfig: defaultTemplate,
         db: db2 as never,
         catalogClient: createMockCatalogClient() as never,
+      notificationClient: createMockNotificationClient() as never,
         logger: createMockLogger() as never,
       });
       expect(db2._insertCalls.length).toBe(1);

package/src/drift-evaluator.ts

@@ -1,5 +1,6 @@
 import type { Logger, SafeDatabase } from "@checkstack/backend-api";
 import type { CatalogApi } from "@checkstack/catalog-common";
+import type { NotificationApi } from "@checkstack/notification-common";
 import type { InferClient } from "@checkstack/common";
 import type { SignalService } from "@checkstack/signal-common";
 import {
@@ -24,6 +25,7 @@ export interface EvaluateDriftInput {
   db: SafeDatabase<typeof schema>;
   logger: Logger;
   catalogClient: InferClient<typeof CatalogApi>;
+  notificationClient: InferClient<typeof NotificationApi>;
   signalService?: SignalService;
   systemId: string;
   configurationId: string;
@@ -49,6 +51,7 @@ export async function evaluateDrift({
   db,
   logger,
   catalogClient,
+  notificationClient,
   signalService,
   systemId,
   configurationId,
@@ -160,6 +163,8 @@ export async function evaluateDrift({
           baselineMean: baseline.mean,
           projectedChange: driftResult.projectedChange,
           catalogClient,
+          notificationClient,
+          db,
           logger,
         });
         return;
@@ -225,6 +230,8 @@ export async function evaluateDrift({
       observedValue: baseline.mean,
       baselineMean: baseline.mean,
       catalogClient,
+      notificationClient,
+      db,
       logger,
     });
   }

package/src/jobs/baseline-analyzer.ts

@@ -1,6 +1,7 @@
 import type { CollectorRegistry, Logger, SafeDatabase } from "@checkstack/backend-api";
 import type { CacheProvider } from "@checkstack/cache-api";
 import type { CatalogApi } from "@checkstack/catalog-common";
+import type { NotificationApi } from "@checkstack/notification-common";
 import type { InferClient } from "@checkstack/common";
 import type { HealthCheckApi, HealthCheckRunResult } from "@checkstack/healthcheck-common";
 import { getHealthResultMeta } from "@checkstack/healthcheck-common";
@@ -33,6 +34,7 @@ export async function setupBaselineAnalyzerJob({
   healthCheckClient,
   signalService,
   catalogClient,
+  notificationClient,
   collectorRegistry,
 }: {
   db: SafeDatabase<typeof schema>;
@@ -42,6 +44,7 @@ export async function setupBaselineAnalyzerJob({
   healthCheckClient: InferClient<typeof HealthCheckApi>;
   signalService?: SignalService;
   catalogClient: InferClient<typeof CatalogApi>;
+  notificationClient: InferClient<typeof NotificationApi>;
   collectorRegistry: CollectorRegistry;
 }) {
   const queue = queueManager.getQueue(BASELINE_ANALYZER_QUEUE);
@@ -217,6 +220,7 @@ export async function setupBaselineAnalyzerJob({
             db,
             logger,
             catalogClient,
+            notificationClient,
             signalService,
             systemId: assignment.systemId,
             configurationId: assignment.configurationId,

package/src/notification-mute.test.ts

@@ -0,0 +1,155 @@
+import { describe, test, expect, mock } from "bun:test";
+import { dispatchAnomalyNotification } from "./notification";
+
+/**
+ * The mute filter behaviour is the integration point that the per-field
+ * opt-out feature hangs on. These tests exercise dispatchAnomalyNotification
+ * end-to-end against an in-memory db mock that controls which userIds appear
+ * as muted, since AnomalyService.getMutedUserIds is what the dispatcher
+ * relies on. The other suites (detector / drift-evaluator) cover the
+ * surrounding state-machine logic.
+ */
+
+function createMockCatalogClient() {
+  return {
+    getSystem: mock(async () => ({ name: "Production API" })),
+  };
+}
+
+function createMockNotificationClient(subscriberIds: string[]) {
+  // The new dispatch goes through notifyForSubscription. This mock
+  // simulates the server-side resolution by emitting a synthetic
+  // "userIds passed to notify" surface that tests assert on. Includes
+  // simulated mute filtering since the dispatcher applies excludeUserIds.
+  return {
+    notifyForSubscription: mock(async (input: {
+      excludeUserIds?: string[];
+    }) => {
+      const excluded = new Set(input.excludeUserIds ?? []);
+      const recipients = subscriberIds.filter((id) => !excluded.has(id));
+      return { notifiedCount: recipients.length, recipients };
+    }),
+  };
+}
+
+/**
+ * Minimal Drizzle-shaped mock that only answers the one query
+ * AnomalyService.getMutedUserIds issues. Returns the supplied muted ids.
+ */
+function createDbWithMutes(mutedUserIds: string[]) {
+  const rows = mutedUserIds.map((userId) => ({ userId }));
+  return {
+    select: mock(() => ({
+      from: mock(() => ({
+        where: mock(() => Promise.resolve(rows)),
+      })),
+    })),
+  };
+}
+
+const baseInput = {
+  systemId: "sys-1",
+  fieldPath: "collectors.http.request.responseTimeMs",
+  observedValue: 250,
+  baselineMean: 100,
+  logger: { warn: mock(() => undefined), debug: mock(() => undefined) },
+};
+
+describe("dispatchAnomalyNotification — mute filtering", () => {
+  test("notifies all subscribers when none have muted the field", async () => {
+    const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient([
+      "alice",
+      "bob",
+      "carol",
+    ]);
+    const db = createDbWithMutes([]);
+
+    await dispatchAnomalyNotification({
+      ...baseInput,
+      action: "confirmed",
+      catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
+      db: db as never,
+      logger: baseInput.logger as never,
+    });
+
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const payload = (
+      notificationClient.notifyForSubscription.mock.calls[0] as unknown[]
+    )[0] as { excludeUserIds?: string[]; resourceKeys: string[] };
+    expect(payload.excludeUserIds ?? []).toEqual([]);
+    expect(payload.resourceKeys).toEqual(["sys-1"]);
+  });
+
+  test("excludes users who muted the specific field", async () => {
+    const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient([
+      "alice",
+      "bob",
+      "carol",
+    ]);
+    const db = createDbWithMutes(["bob"]);
+
+    await dispatchAnomalyNotification({
+      ...baseInput,
+      action: "confirmed",
+      catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
+      db: db as never,
+      logger: baseInput.logger as never,
+    });
+
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const payload = (
+      notificationClient.notifyForSubscription.mock.calls[0] as unknown[]
+    )[0] as { excludeUserIds?: string[] };
+    expect(payload.excludeUserIds ?? []).toEqual(["bob"]);
+  });
+
+  test("forwards every muted user as excludeUserIds", async () => {
+    const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient(["alice", "bob"]);
+    const db = createDbWithMutes(["alice", "bob"]);
+
+    await dispatchAnomalyNotification({
+      ...baseInput,
+      action: "confirmed",
+      catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
+      db: db as never,
+      logger: baseInput.logger as never,
+    });
+
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const payload = (
+      notificationClient.notifyForSubscription.mock.calls[0] as unknown[]
+    )[0] as { excludeUserIds?: string[] };
+    expect((payload.excludeUserIds ?? []).sort()).toEqual(["alice", "bob"]);
+  });
+
+  test("delegates dispatch even when there are no mutes — backend resolves subscribers", async () => {
+    const catalogClient = createMockCatalogClient();
+    const notificationClient = createMockNotificationClient([]);
+    const db = createDbWithMutes([]);
+
+    await dispatchAnomalyNotification({
+      ...baseInput,
+      action: "confirmed",
+      catalogClient: catalogClient as never,
+      notificationClient: notificationClient as never,
+      db: db as never,
+      logger: baseInput.logger as never,
+    });
+
+    // Anomaly always delegates to the backend now — subscriber-set
+    // resolution happens server-side. The mock just needs to have been
+    // invoked once with the right specId / resourceKeys.
+    expect(notificationClient.notifyForSubscription).toHaveBeenCalledTimes(1);
+    const payload = (
+      notificationClient.notifyForSubscription.mock.calls[0] as unknown[]
+    )[0] as { specId: string; resourceKeys: string[] };
+    expect(payload.specId).toBe("anomaly.system");
+    expect(payload.resourceKeys).toEqual(["sys-1"]);
+  });
+});