@zeniai/client-epic-state 5.2.34-beta0ND → 5.2.35

package/lib/entity/notificationRegistry/notificationRegistryReducer.d.ts

@@ -1,10 +1,9 @@
 import { NotificationRegistryPayload } from './notificationRegistryPayload';
-import { NotificationDomain, NotificationRegistryState } from './notificationRegistryState';
+import { NotificationRegistryState } from './notificationRegistryState';
 export declare const initialState: NotificationRegistryState;
 export interface UpdateNotificationRegistryPayload {
     registry: NotificationRegistryPayload | undefined;
     registryEpochAtDispatch: number;
-    registriesByDomain?: Partial<Record<NotificationDomain, NotificationRegistryPayload | undefined>>;
 }
 export declare const updateNotificationRegistry: import("@reduxjs/toolkit").ActionCreatorWithPayload<UpdateNotificationRegistryPayload, "notificationRegistry/updateNotificationRegistry">;
 declare const _default: import("redux").Reducer<NotificationRegistryState>;

package/lib/entity/notificationRegistry/notificationRegistryReducer.js

@@ -4,7 +4,6 @@ exports.updateNotificationRegistry = exports.initialState = void 0;
 const toolkit_1 = require("@reduxjs/toolkit");
 const rootActions_1 = require("../../rootActions");
 const notificationRegistryPayload_1 = require("./notificationRegistryPayload");
-const notificationRegistryState_1 = require("./notificationRegistryState");
 exports.initialState = { registryEpoch: 0 };
 const notificationRegistry = (0, toolkit_1.createSlice)({
     name: 'notificationRegistry',
@@ -23,24 +22,16 @@ const notificationRegistry = (0, toolkit_1.createSlice)({
             // `task_checklist_registry` CLEARS so tenant B doesn't inherit A's
             // Task Checklist section. Guarded by the epoch check above so a stale
             // absent-response can't wipe a fresh registry.
-            const { registriesByDomain } = action.payload;
-            // Absent `registriesByDomain` means a pre-domain caller: treat `registry`
-            // as the task_checklist snapshot so their behaviour is unchanged.
-            const snapshots = registriesByDomain ?? { task_checklist: action.payload.registry };
-            draft.registryByDomain = {};
-            for (const domain of notificationRegistryState_1.NOTIFICATION_DOMAINS) {
-                const payload = snapshots[domain];
-                // Full-replace per domain, and an absent snapshot CLEARS that domain so
-                // tenant B never inherits A's section. The gates are independent
-                // server-side, so one domain clearing must not touch the other.
-                if (payload != null) {
-                    draft.registryByDomain[domain] =
-                        (0, notificationRegistryPayload_1.mapNotificationRegistryPayloadToRegistry)(payload);
-                }
+            if (action.payload.registry == null) {
+                draft.registry = undefined;
+                return;
             }
-            // Mirrored for consumers that predate the domain-keyed map.
-            draft.registry = draft.registryByDomain.task_checklist;
-            return;
+            // Full-replace contract: the registry payload is atomic — every fetch
+            // returns the complete group/event tree. If the backend ever ships a
+            // partial registry response, this write MUST switch to a merge (mirror
+            // the preferences slice's `mergePreferences`). Do not assume the
+            // current payload is partial without also updating this reducer.
+            draft.registry = (0, notificationRegistryPayload_1.mapNotificationRegistryPayloadToRegistry)(action.payload.registry);
         },
     },
     extraReducers: (builder) => {
@@ -53,8 +44,6 @@ const notificationRegistry = (0, toolkit_1.createSlice)({
             // — Cursor Bugbot 67d6918e. Returning `initialState` verbatim would
             // rewind the counter and let a stale in-flight GET restore A's
             // registry over B's.
-            // Drops `registry` and `registryByDomain` together: a tenant switch must
-            // not leave one domain's section behind.
             return { registryEpoch: draft.registryEpoch + 1 };
         });
     },

package/lib/entity/notificationRegistry/notificationRegistrySelector.d.ts

@@ -1,3 +1,2 @@
-import { NotificationDomain, NotificationRegistry, NotificationRegistryState } from './notificationRegistryState';
+import { NotificationRegistry, NotificationRegistryState } from './notificationRegistryState';
 export declare const getNotificationRegistry: (slice: NotificationRegistryState) => NotificationRegistry | undefined;
-export declare const getNotificationRegistryForDomain: (slice: NotificationRegistryState, domain: NotificationDomain) => NotificationRegistry | undefined;

package/lib/entity/notificationRegistry/notificationRegistrySelector.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getNotificationRegistryForDomain = exports.getNotificationRegistry = void 0;
+exports.getNotificationRegistry = void 0;
 // Current behavior: a null / undefined registry means the whole notification
 // section is hidden by consumers. This conflates two distinct states —
 // "user has no saved preferences yet" vs "the Task Checklist feature gate is
@@ -8,9 +8,3 @@ exports.getNotificationRegistryForDomain = exports.getNotificationRegistry = voi
 // them (e.g. an explicit `gate_off` signal from the server) is a follow-up.
 const getNotificationRegistry = (slice) => slice.registry;
 exports.getNotificationRegistry = getNotificationRegistry;
-// `undefined` conflates gate-off with no-preferences-yet; consumers hide the
-// section either way. Falls back to `registry` for task_checklist so state built
-// in the pre-domain shape (consumer mocks, older callers) still resolves.
-const getNotificationRegistryForDomain = (slice, domain) => slice.registryByDomain?.[domain] ??
-    (domain === 'task_checklist' ? slice.registry : undefined);
-exports.getNotificationRegistryForDomain = getNotificationRegistryForDomain;

package/lib/entity/notificationRegistry/notificationRegistryState.d.ts

@@ -1,7 +1,3 @@
-export declare const NOTIFICATION_DOMAINS: readonly ["task_checklist", "commenting"];
-export type NotificationDomain = (typeof NOTIFICATION_DOMAINS)[number];
-export declare const DEFAULT_NOTIFICATION_DOMAIN: NotificationDomain;
-export declare const toNotificationDomain: (raw: string) => NotificationDomain;
 export declare const NOTIFICATION_CHANNELS: readonly ["email", "slack", "in_app"];
 export type NotificationChannel = (typeof NOTIFICATION_CHANNELS)[number];
 export declare const toNotificationChannel: (raw: string) => NotificationChannel;
@@ -34,5 +30,4 @@ export interface NotificationRegistry {
 export interface NotificationRegistryState {
     registryEpoch: number;
     registry?: NotificationRegistry;
-    registryByDomain?: Partial<Record<NotificationDomain, NotificationRegistry>>;
 }

package/lib/entity/notificationRegistry/notificationRegistryState.js

@@ -1,13 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.toNotificationFrequency = exports.NOTIFICATION_FREQUENCIES = exports.toNotificationChannel = exports.NOTIFICATION_CHANNELS = exports.toNotificationDomain = exports.DEFAULT_NOTIFICATION_DOMAIN = exports.NOTIFICATION_DOMAINS = void 0;
+exports.toNotificationFrequency = exports.NOTIFICATION_FREQUENCIES = exports.toNotificationChannel = exports.NOTIFICATION_CHANNELS = void 0;
 const stringToUnion_1 = require("../../commonStateTypes/stringToUnion");
-// Wire vocabulary owned by the notifications service, not by this slice.
-exports.NOTIFICATION_DOMAINS = ['task_checklist', 'commenting'];
-// Callers that predate the domain split mean Task Checklist.
-exports.DEFAULT_NOTIFICATION_DOMAIN = 'task_checklist';
-const toNotificationDomain = (raw) => (0, stringToUnion_1.stringToUnion)(raw, exports.NOTIFICATION_DOMAINS);
-exports.toNotificationDomain = toNotificationDomain;
 exports.NOTIFICATION_CHANNELS = ['email', 'slack', 'in_app'];
 const toNotificationChannel = (raw) => (0, stringToUnion_1.stringToUnion)(raw, exports.NOTIFICATION_CHANNELS);
 exports.toNotificationChannel = toNotificationChannel;

package/lib/entity/sectionProjectView/sectionProjectViewReducer.js

@@ -35,9 +35,7 @@ const sectionProjectView = (0, toolkit_1.createSlice)({
                     Object.keys(sectionPayload.projects).forEach((projectId) => {
                         const projectPayload = sectionPayload.projects[projectId];
                         // A calculated section is grouped by project with no accounts under it.
-                        const projectAccounts = 'accounts' in projectPayload
-                            ? projectPayload.accounts
-                            : undefined;
+                        const projectAccounts = 'accounts' in projectPayload ? projectPayload.accounts : undefined;
                         const nestedAccountIds = projectAccounts != null
                             ? (0, getNestedAccountIDHierarchyForSection_1.getNestedAccountIDHierarchyForAccounts)(Object.keys(projectAccounts), accountsMetadata)
                             : [];

package/lib/epic.js

@@ -53,21 +53,21 @@ const acceptMasterTOSEpic_1 = require("./view/aiCfoView/epics/acceptMasterTOSEpi
 const createSessionAndSubmitEpic_1 = require("./view/aiCfoView/epics/createSessionAndSubmitEpic");
 const createSessionEpic_1 = require("./view/aiCfoView/epics/createSessionEpic");
 const deleteChatSessionEpic_1 = require("./view/aiCfoView/epics/deleteChatSessionEpic");
-const deleteRoutineEpic_1 = require("./view/aiCfoView/epics/deleteRoutineEpic");
 const deleteSkillEpic_1 = require("./view/aiCfoView/epics/deleteSkillEpic");
 const fetchChatHistoryEpic_1 = require("./view/aiCfoView/epics/fetchChatHistoryEpic");
 const fetchChatSessionsForUserEpic_1 = require("./view/aiCfoView/epics/fetchChatSessionsForUserEpic");
-const fetchRoutineRunsEpic_1 = require("./view/aiCfoView/epics/fetchRoutineRunsEpic");
-const fetchRoutinesEpic_1 = require("./view/aiCfoView/epics/fetchRoutinesEpic");
 const fetchSkillDetailEpic_1 = require("./view/aiCfoView/epics/fetchSkillDetailEpic");
 const fetchSkillVersionBodyEpic_1 = require("./view/aiCfoView/epics/fetchSkillVersionBodyEpic");
 const fetchSkillVersionsEpic_1 = require("./view/aiCfoView/epics/fetchSkillVersionsEpic");
 const fetchSkillsEpic_1 = require("./view/aiCfoView/epics/fetchSkillsEpic");
 const fetchSuggestedQuestionsEpic_1 = require("./view/aiCfoView/epics/fetchSuggestedQuestionsEpic");
+const deleteRoutineEpic_1 = require("./view/aiCfoView/epics/deleteRoutineEpic");
+const fetchRoutineRunsEpic_1 = require("./view/aiCfoView/epics/fetchRoutineRunsEpic");
+const fetchRoutinesEpic_1 = require("./view/aiCfoView/epics/fetchRoutinesEpic");
 const runRoutineEpic_1 = require("./view/aiCfoView/epics/runRoutineEpic");
 const saveRoutineEpic_1 = require("./view/aiCfoView/epics/saveRoutineEpic");
-const saveSkillEpic_1 = require("./view/aiCfoView/epics/saveSkillEpic");
 const setRoutineEnabledEpic_1 = require("./view/aiCfoView/epics/setRoutineEnabledEpic");
+const saveSkillEpic_1 = require("./view/aiCfoView/epics/saveSkillEpic");
 const setSkillVisibilityEpic_1 = require("./view/aiCfoView/epics/setSkillVisibilityEpic");
 const stopSubmitEpic_1 = require("./view/aiCfoView/epics/stopSubmitEpic");
 const stopSubmitQuestionEpic_1 = require("./view/aiCfoView/epics/stopSubmitQuestionEpic");

package/lib/esm/entity/notificationRegistry/notificationRegistryReducer.js

@@ -1,7 +1,6 @@
 import { createSlice } from '@reduxjs/toolkit';
 import { clearAll } from '../../rootActions';
 import { mapNotificationRegistryPayloadToRegistry, } from './notificationRegistryPayload';
-import { NOTIFICATION_DOMAINS, } from './notificationRegistryState';
 export const initialState = { registryEpoch: 0 };
 const notificationRegistry = createSlice({
     name: 'notificationRegistry',
@@ -20,24 +19,16 @@ const notificationRegistry = createSlice({
             // `task_checklist_registry` CLEARS so tenant B doesn't inherit A's
             // Task Checklist section. Guarded by the epoch check above so a stale
             // absent-response can't wipe a fresh registry.
-            const { registriesByDomain } = action.payload;
-            // Absent `registriesByDomain` means a pre-domain caller: treat `registry`
-            // as the task_checklist snapshot so their behaviour is unchanged.
-            const snapshots = registriesByDomain ?? { task_checklist: action.payload.registry };
-            draft.registryByDomain = {};
-            for (const domain of NOTIFICATION_DOMAINS) {
-                const payload = snapshots[domain];
-                // Full-replace per domain, and an absent snapshot CLEARS that domain so
-                // tenant B never inherits A's section. The gates are independent
-                // server-side, so one domain clearing must not touch the other.
-                if (payload != null) {
-                    draft.registryByDomain[domain] =
-                        mapNotificationRegistryPayloadToRegistry(payload);
-                }
+            if (action.payload.registry == null) {
+                draft.registry = undefined;
+                return;
             }
-            // Mirrored for consumers that predate the domain-keyed map.
-            draft.registry = draft.registryByDomain.task_checklist;
-            return;
+            // Full-replace contract: the registry payload is atomic — every fetch
+            // returns the complete group/event tree. If the backend ever ships a
+            // partial registry response, this write MUST switch to a merge (mirror
+            // the preferences slice's `mergePreferences`). Do not assume the
+            // current payload is partial without also updating this reducer.
+            draft.registry = mapNotificationRegistryPayloadToRegistry(action.payload.registry);
         },
     },
     extraReducers: (builder) => {
@@ -50,8 +41,6 @@ const notificationRegistry = createSlice({
             // — Cursor Bugbot 67d6918e. Returning `initialState` verbatim would
             // rewind the counter and let a stale in-flight GET restore A's
             // registry over B's.
-            // Drops `registry` and `registryByDomain` together: a tenant switch must
-            // not leave one domain's section behind.
             return { registryEpoch: draft.registryEpoch + 1 };
         });
     },

package/lib/esm/entity/notificationRegistry/notificationRegistrySelector.js

@@ -4,8 +4,3 @@
 // off for this tenant". Both currently surface as `undefined`. Distinguishing
 // them (e.g. an explicit `gate_off` signal from the server) is a follow-up.
 export const getNotificationRegistry = (slice) => slice.registry;
-// `undefined` conflates gate-off with no-preferences-yet; consumers hide the
-// section either way. Falls back to `registry` for task_checklist so state built
-// in the pre-domain shape (consumer mocks, older callers) still resolves.
-export const getNotificationRegistryForDomain = (slice, domain) => slice.registryByDomain?.[domain] ??
-    (domain === 'task_checklist' ? slice.registry : undefined);

package/lib/esm/entity/notificationRegistry/notificationRegistryState.js

@@ -1,9 +1,4 @@
 import { stringToUnion } from '../../commonStateTypes/stringToUnion';
-// Wire vocabulary owned by the notifications service, not by this slice.
-export const NOTIFICATION_DOMAINS = ['task_checklist', 'commenting'];
-// Callers that predate the domain split mean Task Checklist.
-export const DEFAULT_NOTIFICATION_DOMAIN = 'task_checklist';
-export const toNotificationDomain = (raw) => stringToUnion(raw, NOTIFICATION_DOMAINS);
 export const NOTIFICATION_CHANNELS = ['email', 'slack', 'in_app'];
 export const toNotificationChannel = (raw) => stringToUnion(raw, NOTIFICATION_CHANNELS);
 export const NOTIFICATION_FREQUENCIES = ['instant', 'daily'];

package/lib/esm/entity/sectionProjectView/sectionProjectViewReducer.js

@@ -32,9 +32,7 @@ const sectionProjectView = createSlice({
                     Object.keys(sectionPayload.projects).forEach((projectId) => {
                         const projectPayload = sectionPayload.projects[projectId];
                         // A calculated section is grouped by project with no accounts under it.
-                        const projectAccounts = 'accounts' in projectPayload
-                            ? projectPayload.accounts
-                            : undefined;
+                        const projectAccounts = 'accounts' in projectPayload ? projectPayload.accounts : undefined;
                         const nestedAccountIds = projectAccounts != null
                             ? getNestedAccountIDHierarchyForAccounts(Object.keys(projectAccounts), accountsMetadata)
                             : [];

package/lib/esm/epic.js

@@ -50,21 +50,21 @@ import { acceptMasterTOSEpic, } from './view/aiCfoView/epics/acceptMasterTOSEpic
 import { createSessionAndSubmitEpic, } from './view/aiCfoView/epics/createSessionAndSubmitEpic';
 import { createSessionEpic, } from './view/aiCfoView/epics/createSessionEpic';
 import { deleteChatSessionEpic, } from './view/aiCfoView/epics/deleteChatSessionEpic';
-import { deleteRoutineEpic } from './view/aiCfoView/epics/deleteRoutineEpic';
 import { deleteSkillEpic } from './view/aiCfoView/epics/deleteSkillEpic';
 import { fetchChatHistoryEpic, } from './view/aiCfoView/epics/fetchChatHistoryEpic';
 import { fetchChatSessionsForUserEpic, } from './view/aiCfoView/epics/fetchChatSessionsForUserEpic';
-import { fetchRoutineRunsEpic } from './view/aiCfoView/epics/fetchRoutineRunsEpic';
-import { fetchRoutinesEpic } from './view/aiCfoView/epics/fetchRoutinesEpic';
 import { fetchSkillDetailEpic } from './view/aiCfoView/epics/fetchSkillDetailEpic';
 import { fetchSkillVersionBodyEpic } from './view/aiCfoView/epics/fetchSkillVersionBodyEpic';
 import { fetchSkillVersionsEpic } from './view/aiCfoView/epics/fetchSkillVersionsEpic';
 import { fetchSkillsEpic, } from './view/aiCfoView/epics/fetchSkillsEpic';
 import { fetchSuggestedQuestionsEpic, } from './view/aiCfoView/epics/fetchSuggestedQuestionsEpic';
+import { deleteRoutineEpic } from './view/aiCfoView/epics/deleteRoutineEpic';
+import { fetchRoutineRunsEpic } from './view/aiCfoView/epics/fetchRoutineRunsEpic';
+import { fetchRoutinesEpic } from './view/aiCfoView/epics/fetchRoutinesEpic';
 import { runRoutineEpic } from './view/aiCfoView/epics/runRoutineEpic';
 import { saveRoutineEpic } from './view/aiCfoView/epics/saveRoutineEpic';
-import { saveSkillEpic } from './view/aiCfoView/epics/saveSkillEpic';
 import { setRoutineEnabledEpic } from './view/aiCfoView/epics/setRoutineEnabledEpic';
+import { saveSkillEpic } from './view/aiCfoView/epics/saveSkillEpic';
 import { setSkillVisibilityEpic } from './view/aiCfoView/epics/setSkillVisibilityEpic';
 import { stopSubmitEpic, } from './view/aiCfoView/epics/stopSubmitEpic';
 import { stopSubmitQuestionEpic, } from './view/aiCfoView/epics/stopSubmitQuestionEpic';

package/lib/esm/index.js

@@ -64,8 +64,8 @@ import { ALL_JE_SCHEDULE_IGNORE_REASONS, ALL_JE_SCHEDULE_REVERT_REASONS, ALL_ONE
 import { ALL_MONTH_END_CLOSE_CHECKS_FREQUENCY, } from './entity/monthEndCloseChecks/monthEndCloseChecksState';
 import { updateCommentsNotifications, updateCommentsNotificationsStatuses, } from './entity/notification/notificationReducer';
 import { toNotificationModeStrict, } from './entity/notification/types/notificationTypes';
-import { getNotificationRegistry, getNotificationRegistryForDomain, } from './entity/notificationRegistry/notificationRegistrySelector';
-import { DEFAULT_NOTIFICATION_DOMAIN, NOTIFICATION_DOMAINS, toNotificationChannel, toNotificationDomain, toNotificationFrequency, } from './entity/notificationRegistry/notificationRegistryState';
+import { getNotificationRegistry } from './entity/notificationRegistry/notificationRegistrySelector';
+import { toNotificationChannel, toNotificationFrequency, } from './entity/notificationRegistry/notificationRegistryState';
 import { getPlaidPaymentAccounts } from './entity/paymentAccount/paymentAccountSelector';
 import { VERIFIED_PAYMENT_ACCOUNT_PROVIDER_VERIFICATION_STATUS, } from './entity/paymentAccount/paymentAccountState';
 import { closeSnackbar, openSnackbar } from './entity/snackbar/snackbarReducer';
@@ -234,7 +234,7 @@ import { toAverageMonthsCount } from './view/netBurnOrIncomeStoryCard/epic/updat
 import { fetchNetBurnOrIncomeStoryCard, updateNetBurnOrIncomeStoryCardSettings, } from './view/netBurnOrIncomeStoryCard/netBurnOrIncomeStoryCardReducer';
 import { getNetBurnOrIncomeStoryCardReport } from './view/netBurnOrIncomeStoryCard/netBurnOrIncomeStoryCardSelector';
 import { clearAllNotificationPreferencesView, clearNotificationPreferencesLocalOverrides, saveNotificationPreferences, setGroupChannelMaster, setGroupFrequency, toggleEventChannel, } from './view/notificationPreferencesView/notificationPreferencesViewReducer';
-import { getEffectiveNotificationPreferences, getEffectiveNotificationPreferencesForDomain, getNotificationLocalOverrides, getNotificationLocalOverridesForDomain, getNotificationPreferencesSaveState, hasUnsavedNotificationPreferences, hasUnsavedNotificationPreferencesForDomain, } from './view/notificationPreferencesView/notificationPreferencesViewSelector';
+import { getEffectiveNotificationPreferences, getNotificationLocalOverrides, getNotificationPreferencesSaveState, hasUnsavedNotificationPreferences, } from './view/notificationPreferencesView/notificationPreferencesViewSelector';
 import { fetchNotificationUnreadCount, fetchNotificationUnreadCountSuccess, fetchNotificationView, updateNotificationViewAllNotificationsStatus, updateNotificationViewCurrentTabAndSubTab, updateNotificationViewNotificationStatus, updateNotificationViewSubTab, updateNotificationViewTabState, updateNotificationViewUIState, } from './view/notificationView/notificationViewReducer';
 import { getExternalNotificationsForSelectedSubTab, getNotificationView, getNotificationsForSelectedSubTab, } from './view/notificationView/notificationViewSelector';
 import { toNotificationSubTabTypeStrict, toNotificationTabTypeStrict, } from './view/notificationView/notificationViewState';
@@ -677,7 +677,7 @@ export { toNotificationModeStrict, updateCommentsNotifications, updateCommentsNo
 export { toNotificationSubTabTypeStrict, toNotificationTabTypeStrict, fetchNotificationView, fetchNotificationUnreadCount, fetchNotificationUnreadCountSuccess, updateNotificationViewAllNotificationsStatus, updateNotificationViewNotificationStatus, updateNotificationViewTabState, updateNotificationViewCurrentTabAndSubTab, updateNotificationViewSubTab, updateNotificationViewUIState, getNotificationView, getExternalNotificationsForSelectedSubTab, getNotificationsForSelectedSubTab, };
 export { fetchRegisteredInterests, notifyMeForFeature, getFeatureNotificationView, getRegisteredInterests, getRegisteredInterestsByFeature, isFeatureInterestRegistered, };
 export { pushToastNotification, getLastNotificationTime, getNotifications, };
-export { getEffectiveNotificationPreferences, getEffectiveNotificationPreferencesForDomain, getNotificationLocalOverrides, getNotificationLocalOverridesForDomain, getNotificationPreferencesSaveState, getNotificationRegistry, getNotificationRegistryForDomain, hasUnsavedNotificationPreferences, hasUnsavedNotificationPreferencesForDomain, NOTIFICATION_DOMAINS, DEFAULT_NOTIFICATION_DOMAIN, clearAllNotificationPreferencesView, clearNotificationPreferencesLocalOverrides, saveNotificationPreferences, setGroupChannelMaster, setGroupFrequency, toggleEventChannel, toNotificationChannel, toNotificationDomain, toNotificationFrequency, };
+export { getEffectiveNotificationPreferences, getNotificationLocalOverrides, getNotificationPreferencesSaveState, getNotificationRegistry, hasUnsavedNotificationPreferences, clearAllNotificationPreferencesView, clearNotificationPreferencesLocalOverrides, saveNotificationPreferences, setGroupChannelMaster, setGroupFrequency, toggleEventChannel, toNotificationChannel, toNotificationFrequency, };
 export { getReferralListView, getInviteFormView, toReferralListViewSortKeyType, StatusTypes, AmountStatusTypes, DEFAULT_REFERRER_AMOUNT, fetchReferrals, sendReferralInvite, clearReferrals, saveReferralFormDataInLocalStore, updateReferralListSortUiState, resendReferralInvite, fetchRewardsPlan, getRewardsPlanCard, updateReferViewed, };
 export { ALL_WEEK_DAYS, SEMI_WEEKLY_REQUIRED_DAYS_COUNT, getMinAllowedEndDate, getRecurringEndDateFromCount, toDayOfWeek, toRecurringFrequency, };
 export { fetchCockpitContext, fetchCompanyTaskManagerView, fetchTaskManagerMetrics, getCompanyTaskManagerView, createTaskFromTaskGroupTemplate, };

package/lib/esm/view/aiCfoView/aiCfoViewReducer.js

@@ -835,8 +835,7 @@ const aiCfoView = createSlice({
             // The server has it. Nothing it sends from here on will contain this row,
             // so the tombstone has done its job and holding it any longer is what
             // makes a routine recreated under the same id invisible forever.
-            draft.routines.deletedScheduleIds =
-                draft.routines.deletedScheduleIds.filter((id) => id !== action.payload.scheduleId);
+            draft.routines.deletedScheduleIds = draft.routines.deletedScheduleIds.filter((id) => id !== action.payload.scheduleId);
             draft.routineMutation = {
                 fetchState: 'Completed',
                 error: undefined,
@@ -848,8 +847,7 @@ const aiCfoView = createSlice({
             // is what lets the refetch put the row back — leaving it hidden would
             // tell the user they had stopped something they had not, which is the
             // worst outcome this list can produce.
-            draft.routines.deletedScheduleIds =
-                draft.routines.deletedScheduleIds.filter((id) => id !== action.payload.scheduleId);
+            draft.routines.deletedScheduleIds = draft.routines.deletedScheduleIds.filter((id) => id !== action.payload.scheduleId);
             // Its own slice: the refetch that puts the row back would otherwise wipe
             // this error, leaving the row silently reappearing with no explanation.
             draft.routineMutation = {

package/lib/esm/view/aiCfoView/epics/fetchRoutineRunsEpic.js

@@ -27,9 +27,7 @@ export const fetchRoutineRunsEpic = (actions$, _state$, zeniAPI) => actions$.pip
             // Filtered, not trusted: one malformed row must not take the
             // whole history down, and a run without an id is not addressable
             // by anything the UI does with it.
-            runs: (data.runs ?? [])
-                .filter(isRoutineRunPayload)
-                .map(toRoutineRun),
+            runs: (data.runs ?? []).filter(isRoutineRunPayload).map(toRoutineRun),
             scheduleId,
             sessionId: data.session_id ?? null,
         });

package/lib/esm/view/notificationPreferencesView/epics/saveNotificationPreferencesEpic.js

@@ -1,12 +1,11 @@
 import { EMPTY, of } from 'rxjs';
 import { catchError, filter, finalize, mergeMap, switchMap, takeUntil, withLatestFrom, } from 'rxjs/operators';
-import { NOTIFICATION_DOMAINS } from '../../../entity/notificationRegistry/notificationRegistryState';
 import { openSnackbar } from '../../../entity/snackbar/snackbarReducer';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../responsePayload';
 import { notificationPreferencesUrl } from '../notificationPreferencesEndpoint';
 import { mapPreferencesToPayload, } from '../notificationPreferencesViewPayload';
 import { clearAllNotificationPreferencesView, clearNotificationPreferencesLocalOverrides, saveNotificationPreferences, saveNotificationPreferencesFailure, saveNotificationPreferencesSuccess, } from '../notificationPreferencesViewReducer';
-import { getNotificationLocalOverridesForDomain } from '../notificationPreferencesViewSelector';
+import { getNotificationLocalOverrides } from '../notificationPreferencesViewSelector';
 export const saveNotificationPreferencesEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter((action) => saveNotificationPreferences.match(action)), withLatestFrom(state$), 
 // `switchMap` unsubscribes from any in-flight PUT observable when a newer
 // Save fires. The prior HTTP request has already left the
@@ -14,19 +13,9 @@ export const saveNotificationPreferencesEpic = (actions$, state$, zeniAPI) => ac
 // ignored, so stale success/failure actions cannot clobber the newer
 // batch's state. Server-side ordering is arrival-order LWW.
 switchMap(([, state]) => {
-    // A clean domain is omitted entirely: an empty patch is still a write against it.
-    const overridesByDomain = {};
-    const body = {};
-    NOTIFICATION_DOMAINS.forEach((domain) => {
-        const overrides = getNotificationLocalOverridesForDomain(state, domain);
-        const preferencesPatch = mapPreferencesToPayload(overrides);
-        if (Object.keys(preferencesPatch).length === 0) {
-            return;
-        }
-        overridesByDomain[domain] = overrides;
-        body[`${domain}_preferences`] = preferencesPatch;
-    });
-    if (Object.keys(body).length === 0) {
+    const overrides = getNotificationLocalOverrides(state);
+    const preferencesPatch = mapPreferencesToPayload(overrides);
+    if (Object.keys(preferencesPatch).length === 0) {
         // No diff → do not dispatch a synthetic success (that would wipe
         // stored preferences via the reducer). Simply no-op.
         return EMPTY;
@@ -39,6 +28,7 @@ switchMap(([, state]) => {
     // in-flight save's response (Success/Failure never dispatched, so
     // the counter never drained and future saves got their merge dropped).
     const cancelEpochAtDispatch = state.notificationPreferencesViewState.cancelEpoch;
+    const body = { task_checklist_preferences: preferencesPatch };
     // Suppress the error snackbar when the response lands AFTER the user
     // pressed Cancel — the Failure reducer already swallows the state
     // update, so the toast would be a stale error for a save the user
@@ -67,11 +57,8 @@ switchMap(([, state]) => {
         if (isSuccessResponse(response)) {
             return of(saveNotificationPreferencesSuccess({
                 cancelEpochAtDispatch,
-                savedOverridesByDomain: overridesByDomain,
-                serverEchoByDomain: {
-                    task_checklist: response.data?.task_checklist_preferences,
-                    commenting: response.data?.commenting_preferences,
-                },
+                savedOverrides: overrides,
+                serverEcho: response.data?.task_checklist_preferences,
             }));
         }
         const actions = [

package/lib/esm/view/notificationPreferencesView/notificationPreferencesViewReducer.js

@@ -1,25 +1,14 @@
 import { createSlice } from '@reduxjs/toolkit';
-import { DEFAULT_NOTIFICATION_DOMAIN, } from '../../entity/notificationRegistry/notificationRegistryState';
 import { clearAll } from '../../rootActions';
 import { mapPayloadToPreferences, } from './notificationPreferencesViewPayload';
 import { emptyNotificationPreferences, } from './notificationPreferencesViewState';
 export const initialState = {
     cancelEpoch: 0,
-    localOverridesByDomain: {},
-    preferencesByDomain: {},
+    localOverrides: emptyNotificationPreferences(),
+    preferences: emptyNotificationPreferences(),
     saveEpoch: 0,
     savePreferencesState: { fetchState: 'Not-Started' },
 };
-// Lazy: a gate-off tenant never receives that domain's key at all.
-const overridesFor = (byDomain, domain) => {
-    const existing = byDomain[domain];
-    if (existing != null) {
-        return existing;
-    }
-    const created = emptyNotificationPreferences();
-    byDomain[domain] = created;
-    return created;
-};
 const mergePreferences = (base, patch) => {
     const eventEnabledByChannel = {
         ...base.eventEnabledByChannel,
@@ -157,8 +146,7 @@ const notificationPreferencesView = createSlice({
             // Success/Failure never dispatches, subsequent legitimate saves
             // still work because their captured epoch matches the current one.
             draft.cancelEpoch += 1;
-            // One shared Save/Cancel footer, so Cancel discards every domain.
-            draft.localOverridesByDomain = {};
+            draft.localOverrides = emptyNotificationPreferences();
             // Also drop `savePreferencesState` back to Not-Started so any
             // "Saving…" affordance bound to `getNotificationPreferencesSaveState`
             // clears on Cancel. Without this, a Cancel-during-save whose stale
@@ -175,6 +163,9 @@ const notificationPreferencesView = createSlice({
             // them; the save-success path is the only clearer.
             // A `null`/`undefined` payload isn't a snapshot at all (e.g. the
             // envelope key was omitted); no-op instead of wiping preferences.
+            if (action.payload.preferences == null) {
+                return;
+            }
             // Stale-GET guard (Cursor Bugbot 3632510237): if the fetch epic
             // captured a `saveEpoch` lower than the current one, a save has
             // succeeded since this GET dispatched — the response is carrying
@@ -184,13 +175,7 @@ const notificationPreferencesView = createSlice({
             if (action.payload.saveEpochAtDispatch < draft.saveEpoch) {
                 return;
             }
-            Object.entries(action.payload.preferencesByDomain).forEach(([domain, payload]) => {
-                if (payload == null) {
-                    return;
-                }
-                draft.preferencesByDomain[domain] =
-                    mapPayloadToPreferences(payload);
-            });
+            draft.preferences = mapPayloadToPreferences(action.payload.preferences);
         },
         saveNotificationPreferences(draft) {
             draft.savePreferencesState = { fetchState: 'In-Progress' };
@@ -249,44 +234,33 @@ const notificationPreferencesView = createSlice({
             // skip the `preferences` replace. Cursor Bugbot 3632510237.
             draft.saveEpoch += 1;
             draft.savePreferencesState = { fetchState: 'Completed' };
-            Object.entries(action.payload.savedOverridesByDomain).forEach(([rawDomain, savedOverrides]) => {
-                if (savedOverrides == null) {
-                    return;
-                }
-                const domain = rawDomain;
-                const serverEcho = action.payload.serverEchoByDomain[domain];
-                const patch = serverEcho == null
-                    ? savedOverrides
-                    : mapPayloadToPreferences(serverEcho);
-                draft.preferencesByDomain[domain] = mergePreferences(draft.preferencesByDomain[domain] ?? emptyNotificationPreferences(), patch);
-                draft.localOverridesByDomain[domain] = subtractSavedOverrides(draft.localOverridesByDomain[domain] ??
-                    emptyNotificationPreferences(), savedOverrides);
-            });
+            const patch = action.payload.serverEcho == null
+                ? action.payload.savedOverrides
+                : mapPayloadToPreferences(action.payload.serverEcho);
+            draft.preferences = mergePreferences(draft.preferences, patch);
+            draft.localOverrides = subtractSavedOverrides(draft.localOverrides, action.payload.savedOverrides);
         },
         setGroupChannelMaster(draft, action) {
             // Mirror of `toggleEventChannel` but scoped to the group-channel master
             // held under `groupChannelEnabledByGroupId[groupId][channel]`.
             const { channel, enabled, groupId } = action.payload;
-            const overrides = overridesFor(draft.localOverridesByDomain, action.payload.domain ?? DEFAULT_NOTIFICATION_DOMAIN);
             const groupMap = {
-                ...(overrides.groupChannelEnabledByGroupId[groupId] ?? {}),
+                ...(draft.localOverrides.groupChannelEnabledByGroupId[groupId] ?? {}),
             };
             groupMap[channel] = enabled;
-            overrides.groupChannelEnabledByGroupId[groupId] = groupMap;
+            draft.localOverrides.groupChannelEnabledByGroupId[groupId] = groupMap;
         },
         setGroupFrequency(draft, action) {
-            const overrides = overridesFor(draft.localOverridesByDomain, action.payload.domain ?? DEFAULT_NOTIFICATION_DOMAIN);
-            overrides.groupFrequencyByGroupId[action.payload.groupId] =
+            draft.localOverrides.groupFrequencyByGroupId[action.payload.groupId] =
                 action.payload.frequency;
         },
         toggleEventChannel(draft, action) {
             const { channel, enabled, eventId } = action.payload;
-            const overrides = overridesFor(draft.localOverridesByDomain, action.payload.domain ?? DEFAULT_NOTIFICATION_DOMAIN);
             const eventMap = {
-                ...(overrides.eventEnabledByChannel[eventId] ?? {}),
+                ...(draft.localOverrides.eventEnabledByChannel[eventId] ?? {}),
             };
             eventMap[channel] = enabled;
-            overrides.eventEnabledByChannel[eventId] = eventMap;
+            draft.localOverrides.eventEnabledByChannel[eventId] = eventMap;
         },
     },
     extraReducers: (builder) => {

package/lib/esm/view/notificationPreferencesView/notificationPreferencesViewSelector.js

@@ -1,18 +1,11 @@
 import { createSelector } from '@reduxjs/toolkit';
-import { DEFAULT_NOTIFICATION_DOMAIN, } from '../../entity/notificationRegistry/notificationRegistryState';
-import { emptyNotificationPreferences, } from './notificationPreferencesViewState';
 const getView = (state) => state.notificationPreferencesViewState;
 export const getNotificationPreferencesSaveState = (state) => getView(state).savePreferencesState;
 // Memoised on `preferences` + `localOverrides` so the merged shape keeps a
 // stable reference while neither input changes — a plain function would
 // allocate a fresh object on every call and re-render every `useSelector`
 // consumer on any unrelated dispatch.
-const EMPTY_PREFERENCES = emptyNotificationPreferences();
-// One per domain: a shared createSelector has a cache of one and would thrash across two.
-const effectivePreferencesSelectorByDomain = new Map();
-const createEffectivePreferencesSelector = (domain) => createSelector((state) => getView(state).preferencesByDomain[domain], (state) => getView(state).localOverridesByDomain[domain], (storedPreferences, storedOverrides) => {
-    const preferences = storedPreferences ?? EMPTY_PREFERENCES;
-    const localOverrides = storedOverrides ?? EMPTY_PREFERENCES;
+export const getEffectiveNotificationPreferences = createSelector((state) => getView(state).preferences, (state) => getView(state).localOverrides, (preferences, localOverrides) => {
     const mergedEventEnabledByChannel = { ...preferences.eventEnabledByChannel };
     Object.entries(localOverrides.eventEnabledByChannel).forEach(([eventId, channels]) => {
         mergedEventEnabledByChannel[eventId] = {
@@ -36,20 +29,7 @@ const createEffectivePreferencesSelector = (domain) => createSelector((state) =>
         },
     };
 });
-export const getEffectiveNotificationPreferencesForDomain = (state, domain) => {
-    let selector = effectivePreferencesSelectorByDomain.get(domain);
-    if (selector == null) {
-        selector = createEffectivePreferencesSelector(domain);
-        effectivePreferencesSelectorByDomain.set(domain, selector);
-    }
-    return selector(state);
-};
-export const getEffectiveNotificationPreferences = (state) => getEffectiveNotificationPreferencesForDomain(state, DEFAULT_NOTIFICATION_DOMAIN);
-export const getNotificationLocalOverridesForDomain = (state, domain) => getView(state).localOverridesByDomain[domain] ?? EMPTY_PREFERENCES;
-export const getNotificationLocalOverrides = (state) => getNotificationLocalOverridesForDomain(state, DEFAULT_NOTIFICATION_DOMAIN);
-const isDirty = (localOverrides) => Object.keys(localOverrides.eventEnabledByChannel).length > 0 ||
+export const getNotificationLocalOverrides = (state) => getView(state).localOverrides;
+export const hasUnsavedNotificationPreferences = createSelector((state) => getView(state).localOverrides, (localOverrides) => Object.keys(localOverrides.eventEnabledByChannel).length > 0 ||
     Object.keys(localOverrides.groupChannelEnabledByGroupId).length > 0 ||
-    Object.keys(localOverrides.groupFrequencyByGroupId).length > 0;
-export const hasUnsavedNotificationPreferencesForDomain = (state, domain) => isDirty(getNotificationLocalOverridesForDomain(state, domain));
-// Drives the shared page-level Save/Cancel footer.
-export const hasUnsavedNotificationPreferences = (state) => Object.values(getView(state).localOverridesByDomain).some((localOverrides) => localOverrides != null && isDirty(localOverrides));
+    Object.keys(localOverrides.groupFrequencyByGroupId).length > 0);

package/lib/esm/view/settingsView/epic/fetchNotificationSettingsEpic.js

@@ -38,21 +38,11 @@ export const fetchNotificationSettingsEpic = (actions$, state$, zeniAPI) => acti
             // 67d6918e.
             updateActions.push(updateNotificationRegistry({
                 registry: response.data.task_checklist_registry ?? undefined,
-                // An absent `<domain>_registry` means that domain's gate is off.
-                registriesByDomain: {
-                    task_checklist: response.data.task_checklist_registry ?? undefined,
-                    commenting: response.data.commenting_registry ?? undefined,
-                },
                 registryEpochAtDispatch,
             }));
-            // One action so the domains cannot land on different store epochs.
-            const preferencesByDomain = {
-                task_checklist: response.data.task_checklist_preferences ?? undefined,
-                commenting: response.data.commenting_preferences ?? undefined,
-            };
-            if (Object.values(preferencesByDomain).some((preferences) => preferences != null)) {
+            if (response.data.task_checklist_preferences != null) {
                 updateActions.push(fetchNotificationPreferencesSuccess({
-                    preferencesByDomain,
+                    preferences: response.data.task_checklist_preferences,
                     saveEpochAtDispatch,
                 }));
             }