@zeniai/client-epic-state 5.0.78 → 5.0.79

package/lib/esm/view/spendManagement/chargeCards/chargeCardDetail/fetchChargeCardDetailEpic.js

@@ -13,13 +13,24 @@ export const fetchChargeCardDetailEpic = (actions$, state$, zeniAPI) => actions$
     const chargeCardAction = [];
     const state = state$.value;
     const chargeCardDetail = recordGet(state.chargeCardDetailState.detailById, chargeCardId, undefined);
+    // Skip if a user-triggered fetch is already in-flight (not applicable
+    // to cacheOverride/Pusher refreshes, which always proceed).
     if (cacheOverride === true ||
         chargeCardDetail == null ||
         chargeCardDetail.fetchState !== 'In-Progress') {
+        // Silent refresh: preserve fetchState to avoid skeleton flash.
+        // Discriminate via fetchState (not null-check) because the reducer
+        // always creates an entry before the epic runs.
+        const isSilentRefresh = cacheOverride === true &&
+            chargeCardDetail != null &&
+            chargeCardDetail.fetchState !== 'Not-Started';
         chargeCardAction.push(updateChargeCardDetailFetchState({
             chargeCardId,
-            isRefreshingViewInBackground: cacheOverride,
-            fetchState: 'In-Progress',
+            isRefreshingViewInBackground: isSilentRefresh,
+            fetchState: isSilentRefresh
+                ? chargeCardDetail.fetchState
+                : 'In-Progress',
+            error: isSilentRefresh ? chargeCardDetail.error : undefined,
         }));
         const query = {
             is_include_card_activation_events: true,
@@ -37,6 +48,8 @@ export const fetchChargeCardDetailEpic = (actions$, state$, zeniAPI) => actions$
                 }));
             }
             else {
+                // Surface errors even on silent refresh so the user
+                // knows data is stale (follow-up: consider toast instead).
                 return of(updateChargeCardDetailFetchState({
                     chargeCardId,
                     fetchState: 'Error',
@@ -45,6 +58,7 @@ export const fetchChargeCardDetailEpic = (actions$, state$, zeniAPI) => actions$
                 }));
             }
         }), catchError((error) => {
+            // Surface network failures even on silent refresh.
             return of(updateChargeCardDetailFetchState({
                 chargeCardId,
                 fetchState: 'Error',

package/lib/esm/view/spendManagement/chargeCards/chargeCardDetail/fetchChargeCardDetailPageEpic.js

@@ -15,15 +15,16 @@ export const fetchChargeCardDetailPageEpic = (actions$, state$) => actions$.pipe
         ? exsitingChargeCardDetail.transactionList.uiState.pageTokenType
         : null;
     if (cacheOverride === true) {
+        // Forward cacheOverride to all sub-fetches to prevent skeleton flash.
         chargeCardTransactionListAction.push(fetchChargeCardDetail(chargeCardId, cacheOverride));
         //latest transactions need to be fetched hence removing pageToken and pageTokenType
-        chargeCardTransactionListAction.push(fetchChargeCardTransactionList(chargeCardId, null, null));
-        chargeCardTransactionListAction.push(fetchChargeCardConfig());
+        chargeCardTransactionListAction.push(fetchChargeCardTransactionList(chargeCardId, null, null, true));
+        chargeCardTransactionListAction.push(fetchChargeCardConfig(true));
         if (doFetchCreditAccounts && isCreditCardEnabled) {
-            chargeCardTransactionListAction.push(fetchCreditAccount());
+            chargeCardTransactionListAction.push(fetchCreditAccount(true));
         }
         if (doFetchCreditAccounts && isCreditCardEnabled) {
-            chargeCardTransactionListAction.push(fetchCreditAccountRepayment());
+            chargeCardTransactionListAction.push(fetchCreditAccountRepayment(true));
         }
     }
     else {

package/lib/esm/view/spendManagement/chargeCards/chargeCardList/chargeCardListReducer.js

@@ -1,5 +1,6 @@
 import { createSlice } from '@reduxjs/toolkit';
 import { toAmount } from '../../../../commonStateTypes/amount';
+import { createCacheOverrideFetchReducer } from '../../../../commonStateTypes/cacheOverrideFetchReducer';
 import { date } from '../../../../zeniDayJS';
 import { initialCreditAcc, initialCreditAccountRepayment, initialDebitCardSummaries, initialResendRevokeCardInvite, toCreditAccountStatusType, } from './chargeCardList';
 export const initialState = {
@@ -43,6 +44,10 @@ export const initialState = {
     revokeCardInvite: initialResendRevokeCardInvite,
     creditAccountRepayment: initialCreditAccountRepayment,
 };
+// Silent-Pusher-refresh contract: when cacheOverride=true, fetch reducers
+// leave fetchState and error unchanged so the list page's reduceFetchState
+// selector stays 'Completed' and doesn't flash ChargeCardListLoading.
+// See createCacheOverrideFetchReducer for details.
 const chargeCardList = createSlice({
     name: 'chargeCardList',
     initialState,
@@ -62,10 +67,10 @@ const chargeCardList = createSlice({
                 };
             },
         },
-        fetchCreditAccountRepayment(draft) {
+        fetchCreditAccountRepayment: createCacheOverrideFetchReducer((draft) => {
             draft.creditAccountRepayment.fetchState = 'In-Progress';
             draft.creditAccountRepayment.error = undefined;
-        },
+        }),
         updateCreditAccountRepaymentAmountAndDate(draft, action) {
             draft.creditAccountRepayment.initiatedRepayments = action.payload.amount;
             draft.creditAccountRepayment.initiatedRepaymentDate = action.payload.date;
@@ -83,10 +88,10 @@ const chargeCardList = createSlice({
         updateRowActionCardId(draft, action) {
             draft.rowActionCardId = action.payload;
         },
-        fetchCreditAccount(draft) {
+        fetchCreditAccount: createCacheOverrideFetchReducer((draft) => {
             draft.creditAccount.fetchState = 'In-Progress';
             draft.creditAccount.error = undefined;
-        },
+        }),
         updateCreditAccount(draft, action) {
             if (action.payload.length > 0) {
                 draft.creditAccount = toCreditAccount(action.payload[0]);
@@ -99,10 +104,10 @@ const chargeCardList = createSlice({
                 error: action.payload,
             };
         },
-        fetchDebitCardSummary(draft) {
+        fetchDebitCardSummary: createCacheOverrideFetchReducer((draft) => {
             draft.debitCardSummaries.fetchState = 'In-Progress';
             draft.debitCardSummaries.error = undefined;
-        },
+        }),
         updateDebitCardSummary(draft, action) {
             draft.debitCardSummaries = toDebitCardSummaries(action.payload);
         },
@@ -113,10 +118,10 @@ const chargeCardList = createSlice({
                 error: action.payload,
             };
         },
-        fetchChargeCardList(draft) {
+        fetchChargeCardList: createCacheOverrideFetchReducer((draft) => {
             draft.fetchState = 'In-Progress';
             draft.error = undefined;
-        },
+        }),
         updateChargeCardList(draft, action) {
             const { cards, unassigned_credit } = action.payload.chargeCardListPayload;
             const newCard = cards.map((card) => card.card_id);
@@ -152,6 +157,18 @@ const chargeCardList = createSlice({
                 draft.uiState.scrollYOffset = action.payload.uiState.scrollYOffset;
             }
         },
+        updateCreditAccountBalanceFromPusher(draft, action) {
+            const { currencyCode, currencySymbol } = action.payload;
+            const amt = (v) => v !== undefined ? toAmount(v, currencyCode, currencySymbol) : undefined;
+            draft.creditAccount.available =
+                amt(action.payload.available) ?? draft.creditAccount.available;
+            draft.creditAccount.balance =
+                amt(action.payload.balance) ?? draft.creditAccount.balance;
+            draft.creditAccount.hold =
+                amt(action.payload.hold) ?? draft.creditAccount.hold;
+            draft.creditAccount.cashback =
+                amt(action.payload.cashback) ?? draft.creditAccount.cashback;
+        },
         clearChargeCardList(draft) {
             Object.assign(draft, initialState);
         },
@@ -307,7 +324,7 @@ const chargeCardList = createSlice({
         },
     },
 });
-export const { fetchChargeCardListPage, fetchCreditAccountRepayment, updateCreditAccountRepayment, updateCreditAccountRepaymentFailure, updateCreditAccountRepaymentAmountAndDate, updateRowActionCardId, fetchCreditAccount, updateCreditAccount, updateCreditAccountFailure, fetchDebitCardSummary, updateDebitCardSummary, updateDebitCardSummaryFailure, fetchChargeCardList, updateChargeCardList, updateChargeCardListFailure, updateSearchText, updateBulkActionCardIds, updateChargeCardListUIState, clearChargeCardList, resendCardInvite, resendCardInviteSuccess, resendCardInviteFailure, revokeCardInvite, revokeCardInviteSuccess, revokeCardInviteFailure, updateChargeCardsSpendLimit, updateChargeCardsSpendLimitSuccessOrFailure, lockChargeCards, lockChargeCardsSuccessOrFailure, unlockChargeCards, unlockChargeCardsSuccessOrFailure, closeChargeCards, closeChargeCardsSuccessOrFailure, revokeChargeCardsInvite, revokeChargeCardsInviteSuccessOrFailure, } = chargeCardList.actions;
+export const { fetchChargeCardListPage, fetchCreditAccountRepayment, updateCreditAccountRepayment, updateCreditAccountRepaymentFailure, updateCreditAccountRepaymentAmountAndDate, updateRowActionCardId, fetchCreditAccount, updateCreditAccount, updateCreditAccountFailure, fetchDebitCardSummary, updateDebitCardSummary, updateDebitCardSummaryFailure, fetchChargeCardList, updateChargeCardList, updateChargeCardListFailure, updateSearchText, updateBulkActionCardIds, updateChargeCardListUIState, updateCreditAccountBalanceFromPusher, clearChargeCardList, resendCardInvite, resendCardInviteSuccess, resendCardInviteFailure, revokeCardInvite, revokeCardInviteSuccess, revokeCardInviteFailure, updateChargeCardsSpendLimit, updateChargeCardsSpendLimitSuccessOrFailure, lockChargeCards, lockChargeCardsSuccessOrFailure, unlockChargeCards, unlockChargeCardsSuccessOrFailure, closeChargeCards, closeChargeCardsSuccessOrFailure, revokeChargeCardsInvite, revokeChargeCardsInviteSuccessOrFailure, } = chargeCardList.actions;
 export default chargeCardList.reducer;
 // helper
 const toCreditAccount = (payload) => ({

package/lib/esm/view/spendManagement/chargeCards/chargeCardList/fetchChargeCardListPageEpic.js

@@ -14,18 +14,21 @@ export const fetchChargeCardListPageEpic = (actions$, state$) => actions$.pipe(f
     const state = state$.value;
     const cashbackDetailState = state.cashbackDetailState;
     if (cacheOverride === true) {
+        // Forward cacheOverride to all sub-fetches to prevent skeleton flash.
         if (doFetchCreditAccountsDebitSummaryAndCashback === true) {
             if (isCreditCardEnabled) {
-                chargeCardListPageAction.push(fetchCreditAccount());
-                chargeCardListPageAction.push(fetchCreditAccountRepayment());
+                chargeCardListPageAction.push(fetchCreditAccount(true));
+                chargeCardListPageAction.push(fetchCreditAccountRepayment(true));
             }
             if (isDebitCardEnabled) {
-                chargeCardListPageAction.push(fetchDebitCardSummary());
+                chargeCardListPageAction.push(fetchDebitCardSummary(true));
             }
-            chargeCardListPageAction.push(fetchCashbackDetail());
+            chargeCardListPageAction.push(fetchCashbackDetail(true));
         }
-        chargeCardListPageAction.push(fetchChargeCardList());
-        chargeCardListPageAction.push(fetchChargeCardConfig());
+        chargeCardListPageAction.push(fetchChargeCardList(true));
+        // Config participates in the list page's reduceFetchState — needs cacheOverride.
+        chargeCardListPageAction.push(fetchChargeCardConfig(true));
+        // These don't participate in the list page's reduceFetchState.
         chargeCardListPageAction.push(fetchUserListByType('cardHolderCandidate', true));
         chargeCardListPageAction.push(fetchUserListByType('cardAdmin', true));
         chargeCardListPageAction.push(fetchClassList());

package/lib/esm/view/spendManagement/chargeCards/chargeCardPaymentHistory/chargeCardPaymentHistoryReducer.js

@@ -1,4 +1,5 @@
 import { createSlice } from '@reduxjs/toolkit';
+import { createCacheOverrideFetchReducer } from '../../../../commonStateTypes/cacheOverrideFetchReducer';
 import { initialChargeCardPaymentHistoryState, } from './chargeCardPaymentHistory';
 export const initialState = initialChargeCardPaymentHistoryState;
 const chargeCardPaymentHistory = createSlice({
@@ -17,10 +18,10 @@ const chargeCardPaymentHistory = createSlice({
                 };
             },
         },
-        fetchChargeCardPaymentHistory(draft) {
+        fetchChargeCardPaymentHistory: createCacheOverrideFetchReducer((draft) => {
             draft.fetchState = 'In-Progress';
             draft.error = undefined;
-        },
+        }),
         updateChargeCardPaymentHistory(draft, action) {
             const payload = action.payload;
             draft.creditAccountId = payload.credit_account_id;

package/lib/esm/view/spendManagement/chargeCards/chargeCardPaymentHistory/fetchChargeCardPaymentPageEpic.js

@@ -8,7 +8,7 @@ export const fetchChargeCardPaymentPageEpic = (actions$, state$) => actions$.pip
     const state = state$.value;
     const chargeCardPaymentHistoryState = state.chargeCardPaymentHistoryState;
     if (cacheOverride === true) {
-        chargeCardPaymentPageActions.push(fetchChargeCardPaymentHistory());
+        chargeCardPaymentPageActions.push(fetchChargeCardPaymentHistory(true));
     }
     else {
         if (chargeCardPaymentHistoryState.fetchState !== 'Completed' &&

package/lib/esm/view/spendManagement/chargeCards/chargeCardRepaymentDetail/chargeCardRepaymentDetailReducer.js

@@ -1,4 +1,5 @@
 import { createSlice } from '@reduxjs/toolkit';
+import { createCacheOverrideFetchReducer } from '../../../../commonStateTypes/cacheOverrideFetchReducer';
 export const initialState = {
     localData: {
         amount: {
@@ -17,9 +18,12 @@ const chargeCardRepaymentDetail = createSlice({
     name: 'chargeCardRepaymentDetail',
     initialState,
     reducers: {
-        fetchChargeCardRepaymentDetail() {
-            //do nothing
-        },
+        // No-op reducer — fetchState is managed by the Pay flow reducers.
+        // Uses createCacheOverrideFetchReducer for API consistency with the
+        // silent-refresh family (cacheOverride flag at dispatch site).
+        fetchChargeCardRepaymentDetail: createCacheOverrideFetchReducer(() => {
+            // do nothing
+        }),
         updateChargeCardRepaymentLocalStore(draft, action) {
             draft.localData = action.payload;
         },

package/lib/esm/view/spendManagement/commonHistoryView/commonHistoryPayload.js

@@ -1,9 +1,14 @@
-import { dateLocal } from '../../../zeniDayJS';
+import { dateInUTC, dateLocal } from '../../../zeniDayJS';
 import { toHistoricEventUpdateDisplayType, toHistoricEventUpdateType, } from './commonHistory';
-export const toHistoricEvent = (payload) => ({
+export const toHistoricEvent = (payload, keepTimeInUTC = false) => ({
     eventText: payload.event_text,
     eventSubText: payload.event_sub_text ?? [],
-    timestamp: payload.timestamp != null ? dateLocal(payload.timestamp) : undefined,
+    // Bills: parse as UTC (dateInUTC). Other entities: convert to local (dateLocal).
+    timestamp: payload.timestamp != null
+        ? keepTimeInUTC
+            ? dateInUTC(payload.timestamp)
+            : dateLocal(payload.timestamp)
+        : undefined,
     updates: payload.updates.map((eventUpdatePayload) => toHistoricEventUpdate(eventUpdatePayload)),
     paymentTraceId: payload.metadata?.payment_trace_id ?? null,
 });

package/lib/esm/view/spendManagement/commonHistoryView/commonHistoryReducer.js

@@ -29,9 +29,10 @@ const commonHistory = createSlice({
             },
         },
         updateCommonHistory(draft, action) {
-            const { entityId, entityType, historicEventsPayload } = action.payload;
+            const { entityId, entityType, historicEventsPayload, keepTimeInUTC } = action.payload;
             const key = getEntityHistoryDetailKey(entityId, entityType);
-            const historicEvents = historicEventsPayload.map((historicEventPayload) => toHistoricEvent(historicEventPayload));
+            // Bills: keepTimeInUTC true → dateInUTC. Other entities: undefined → dateLocal.
+            const historicEvents = historicEventsPayload.map((historicEventPayload) => toHistoricEvent(historicEventPayload, keepTimeInUTC));
             draft.historyById[key] = {
                 ...initialCommonHistory,
                 fetchState: 'Completed',

package/lib/esm/view/spendManagement/commonHistoryView/fetchEntityHistoryEpic.js

@@ -27,6 +27,8 @@ export const fetchEntityHistoryEpic = (actions$, state$, zeniAPI) => actions$.pi
                     entityId,
                     entityType,
                     historicEventsPayload: response.data.activities,
+                    // Bills: parse timestamps as UTC. Other entities: convert to local.
+                    keepTimeInUTC: entityType === 'bill_pay',
                 }));
             }
             else {

package/lib/esm/view/spendManagement/spendManagementFilterHelpers.js

@@ -8,6 +8,8 @@ import { getActualPaymentDate } from './helpers';
 import { getApproversOrRejectors as getApproversOrRejectorsForReimbursement, } from './reimbursement/remiListView/remiListSelector';
 import { ALL_REMI_TABS, getRemiListKey, } from './reimbursement/remiListView/remiListState';
 //filters items against filter values representing a range. ex: date (from-to), amount(1k<-->5k)
+// Spend Management uses only equal/not_equal — TC amount operators
+// (inBetween, lessThan, greaterThan) live in the TC filter pipeline.
 const filterItemOnRangeCategoryType = (valueForItem, currentFilter) => {
     let doesItemFallInValueRange = false;
     const filterField = currentFilter.field;
@@ -41,7 +43,7 @@ const filterItemOnCategoriesWithSingleValue = (categoryValue, currentFilter, ran
         return filterItemOnRangeCategoryType(categoryValue, currentFilter);
     }
     // if filter category is not rangeType
-    if (currentFilter.matchingOperator === 'not-equal') {
+    if (currentFilter.matchingOperator === 'not_equal') {
         return currentFilter.values.indexOf(categoryValue.toString()) === -1;
     }
     return currentFilter.values.indexOf(categoryValue.toString()) > -1;
@@ -115,7 +117,7 @@ export const applyAdvancedFiltersOnList = (entity, allItems, filters, filteredIt
                     ? getCategoryValueForTaskList(key, item)
                     : getCategoryValueForPaymentHistory(key, item);
         if (categoryValue == null) {
-            if (currentFilter.matchingOperator === 'not-equal') {
+            if (currentFilter.matchingOperator === 'not_equal') {
                 return true;
             }
             return false;

package/lib/esm/view/subscriptionView/subscriptionViewSelector.js

@@ -208,7 +208,7 @@ const applyAdvancedFiltersOnCompaniesList = (allCompanies, filters, filteredComp
         const key = currentFilter.field;
         const categoryValue = getCategoryValueForCompany(key, company);
         if (categoryValue == null) {
-            if (currentFilter.matchingOperator === 'not-equal') {
+            if (currentFilter.matchingOperator === 'not_equal') {
                 return true;
             }
             return false;

package/lib/esm/view/taskManager/cannedResponsesView/epics/fetchCannedResponsesEpic.js

@@ -1,5 +1,5 @@
 import { of } from 'rxjs';
-import { catchError, filter, switchMap, mergeMap } from 'rxjs/operators';
+import { catchError, filter, mergeMap, switchMap } from 'rxjs/operators';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
 import { mapCannedResponsePayloadToCannedResponse, } from '../cannedResponsesPayload';
 import { fetchCannedResponses, updateCannedResponses, updateCannedResponsesFetchStatus, } from '../cannedResponsesReducer';

package/lib/esm/view/taskManager/cannedResponsesView/epics/saveCannedResponseEpic.js

@@ -1,5 +1,5 @@
 import { of } from 'rxjs';
-import { catchError, filter, switchMap, mergeMap } from 'rxjs/operators';
+import { catchError, filter, mergeMap, switchMap } from 'rxjs/operators';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
 import { fetchCannedResponses, saveCannedResponse, saveCannedResponseFailure, saveCannedResponseSuccess, } from '../cannedResponsesReducer';
 export const saveCannedResponseEpic = (actions$, _state$, zeniAPI) => actions$.pipe(filter(saveCannedResponse.match), switchMap((action) => {

package/lib/esm/view/taskManager/taskDetailView/epics/unsnoozeTaskEpic.js

@@ -4,7 +4,7 @@ import { openSnackbar } from '../../../../entity/snackbar/snackbarReducer';
 import { updateTasks } from '../../../../entity/task/taskReducer';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
 import { fetchTaskList, removeTaskFromList, } from '../../taskListView/taskListReducer';
-import { snoozeTaskSuccessOrFailure, unsnoozeTask, } from '../taskDetailReducer';
+import { snoozeTaskSuccessOrFailure, unsnoozeTask } from '../taskDetailReducer';
 export const unsnoozeTaskEpic = (actions$, _state$, zeniAPI) => actions$.pipe(filter(unsnoozeTask.match), mergeMap((action) => handleUnsnoozeTask(action.payload.taskId, zeniAPI)));
 const handleUnsnoozeTask = (taskId, zeniAPI) => {
     const unsnoozeTaskApi$ = zeniAPI.deleteAndGetJSON(`${zeniAPI.apiEndPoints.taskMicroServiceBaseUrl}/1.0/task-manager/tasks/${taskId}/snooze`);

package/lib/esm/view/taskManager/taskListView/taskListReducer.js

@@ -66,7 +66,7 @@ export const initialState = {
         categories: [
             {
                 field: 'status',
-                matchingOperator: 'not-equal',
+                matchingOperator: 'not_equal',
                 values: ['resolved'],
                 valuesCombinationOperator: 'ANY',
             },

package/lib/esm/view/transactionDetail/transactionDetailReducer.js

@@ -238,12 +238,34 @@ const transactionDetail = createSlice({
         saveTransactionDetailLocalData(draft, action) {
             const key = getTransactionDetailKey(action.payload.transactionId);
             const detail = recordGet(draft.transactionDetailById, key, undefined);
-            if (detail != null) {
-                draft.transactionDetailById[key] = {
-                    ...detail,
-                    transactionDetailLocalData: action.payload.transactionLocalData,
-                };
+            if (detail == null) {
+                return;
             }
+            const incoming = action.payload.transactionLocalData;
+            const prevById = detail.transactionDetailLocalData.lineItemById;
+            // Per-line `categoryClassRecommendationsFetchState` is owned by the
+            // recommendation epic/reducers (markInProgress / markCompleted /
+            // markFailure / setEntityRecommendationForLineIdsForTransactionDetail).
+            // Callers that rebuild `transactionLocalData` from a UI snapshot will
+            // typically have an out-of-date copy of this field (e.g. payee-change
+            // captures the snapshot before the epic's start arm runs and would
+            // otherwise clobber In-Progress back to undefined). Treat the field as
+            // sticky: only let the caller override it when they explicitly set it.
+            const mergedLineItemById = {};
+            Object.entries(incoming.lineItemById ?? {}).forEach(([lineId, line]) => {
+                mergedLineItemById[lineId] = {
+                    ...line,
+                    categoryClassRecommendationsFetchState: line.categoryClassRecommendationsFetchState ??
+                        prevById?.[lineId]?.categoryClassRecommendationsFetchState,
+                };
+            });
+            draft.transactionDetailById[key] = {
+                ...detail,
+                transactionDetailLocalData: {
+                    ...incoming,
+                    lineItemById: mergedLineItemById,
+                },
+            };
         },
         saveTransactionDetail(draft, action) {
             const key = getTransactionDetailKey(action.payload.transactionId);
@@ -365,6 +387,21 @@ const transactionDetail = createSlice({
                 const detail = recordGet(draft.transactionDetailById, key, undefined);
                 if (detail != null) {
                     const newLocalData = setEntityRecommendationForLineIdInLocalData(transaction, detail.transactionDetailLocalData, action.payload.entity, lineIds, uncategorizedIncomeExpense, recommendationWithCOTByLineId);
+                    // Release the per-line fetch flag for every line this fetch
+                    // covered so the inline skeleton resolves and the page-level
+                    // Save button can re-enable.
+                    lineIds.forEach((lineId) => {
+                        const lineItem = newLocalData.lineItemById[lineId];
+                        if (lineItem != null) {
+                            newLocalData.lineItemById[lineId] = {
+                                ...lineItem,
+                                categoryClassRecommendationsFetchState: {
+                                    fetchState: 'Completed',
+                                    error: undefined,
+                                },
+                            };
+                        }
+                    });
                     draft.transactionDetailById[key] = {
                         ...detail,
                         transactionDetailLocalData: newLocalData,
@@ -385,6 +422,84 @@ const transactionDetail = createSlice({
                 };
             },
         },
+        // Start arm of the line-update recommendations fetch for the detail
+        // page. Flips covered lines to In-Progress; detail page has no
+        // checkbox, so no auto-uncheck side effect.
+        markCategoryClassRecommendationsInProgressForTransactionDetail: {
+            reducer(draft, action) {
+                const { transactionId, lineIds } = action.payload;
+                const key = getTransactionDetailKey(transactionId);
+                const detail = recordGet(draft.transactionDetailById, key, undefined);
+                if (detail != null) {
+                    lineIds.forEach((lineId) => {
+                        const lineItem = detail.transactionDetailLocalData.lineItemById[lineId];
+                        if (lineItem != null) {
+                            detail.transactionDetailLocalData.lineItemById[lineId] = {
+                                ...lineItem,
+                                categoryClassRecommendationsFetchState: {
+                                    fetchState: 'In-Progress',
+                                    error: undefined,
+                                },
+                            };
+                        }
+                    });
+                }
+            },
+            prepare(transactionId, lineIds) {
+                return { payload: { transactionId, lineIds } };
+            },
+        },
+        // Completed arm. Used when the HTTP call succeeded but the server
+        // returned no per-line recommendations — still need to release the
+        // In-Progress flag.
+        markCategoryClassRecommendationsCompletedForTransactionDetail: {
+            reducer(draft, action) {
+                const { transactionId, lineIds } = action.payload;
+                const key = getTransactionDetailKey(transactionId);
+                const detail = recordGet(draft.transactionDetailById, key, undefined);
+                if (detail != null) {
+                    lineIds.forEach((lineId) => {
+                        const lineItem = detail.transactionDetailLocalData.lineItemById[lineId];
+                        if (lineItem != null) {
+                            detail.transactionDetailLocalData.lineItemById[lineId] = {
+                                ...lineItem,
+                                categoryClassRecommendationsFetchState: {
+                                    fetchState: 'Completed',
+                                    error: undefined,
+                                },
+                            };
+                        }
+                    });
+                }
+            },
+            prepare(transactionId, lineIds) {
+                return { payload: { transactionId, lineIds } };
+            },
+        },
+        markCategoryClassRecommendationsFailureForTransactionDetail: {
+            reducer(draft, action) {
+                const { transactionId, lineIds, error } = action.payload;
+                const key = getTransactionDetailKey(transactionId);
+                const detail = recordGet(draft.transactionDetailById, key, undefined);
+                if (detail != null) {
+                    lineIds.forEach((lineId) => {
+                        const lineItem = detail.transactionDetailLocalData.lineItemById[lineId];
+                        if (lineItem != null) {
+                            detail.transactionDetailLocalData.lineItemById[lineId] = {
+                                ...lineItem,
+                                categoryClassRecommendationsFetchState: {
+                                    fetchState: 'Error',
+                                    error,
+                                },
+                            };
+                        }
+                    });
+                }
+            },
+            prepare(transactionId, lineIds, error) {
+                return { payload: { transactionId, lineIds, error } };
+            },
+        },
         removeEntityRecommendationForLineIdForTransactionDetail: {
             reducer(draft, action) {
                 const { transaction, lineId } = action.payload;
@@ -656,7 +771,7 @@ const transactionDetail = createSlice({
         },
     },
 });
-export const { downloadAccountingProviderAttachment, downloadAccountingProviderAttachmentSuccess, downloadAccountingProviderAttachmentFailure, fetchTransactionDetail, updateTransactionDetailFetchState, removeTransactionDetail, clearTransactionDetail, uploadMissingAttachmentSuccess, updateTransactionDetail, updateTransactionDetailSaveStatus, clearTransactionDetailSaveStatus, markTransactionAsNotMiscategorized, deleteTransactionAttachment, deleteTransactionAttachmentFailure, deleteTransactionAttachmentSuccess, updateStatusForTransactionNotMiscategorizedUpdate, updateStatusConsolidatedSaveTransactionDetail, updateTransactionStateIfUpdateActionNotAllowed, saveTransactionDetail, initializeTransactionDetailLocalData, saveTransactionDetailLocalData, clearTransactionDetailLocalData, resetLineItemsCategoryClassInLocalData, setAllLineItemsToCategoryClassInLocalData, setAllLineItemsLocalDataIsUpdateAllChecked, setEntityRecommendationForLineIdsForTransactionDetail, removeEntityRecommendationForLineIdForTransactionDetail, updateSelectedVendor, updateSelectedCustomer, resetVendor, resetSelectedCustomer, updateLocalDataWithTransactionsUpdateWithVendorCheckbox, updateVendorToAllLineItems, resetAllLineItemsToVendor, updateLatestSelectedEntity, updateVendorBulkRecommendationsFetchStatus, setIsVendorUpdateAllChecked, } = transactionDetail.actions;
+export const { downloadAccountingProviderAttachment, downloadAccountingProviderAttachmentSuccess, downloadAccountingProviderAttachmentFailure, fetchTransactionDetail, updateTransactionDetailFetchState, removeTransactionDetail, clearTransactionDetail, uploadMissingAttachmentSuccess, updateTransactionDetail, updateTransactionDetailSaveStatus, clearTransactionDetailSaveStatus, markTransactionAsNotMiscategorized, deleteTransactionAttachment, deleteTransactionAttachmentFailure, deleteTransactionAttachmentSuccess, updateStatusForTransactionNotMiscategorizedUpdate, updateStatusConsolidatedSaveTransactionDetail, updateTransactionStateIfUpdateActionNotAllowed, saveTransactionDetail, initializeTransactionDetailLocalData, saveTransactionDetailLocalData, clearTransactionDetailLocalData, resetLineItemsCategoryClassInLocalData, setAllLineItemsToCategoryClassInLocalData, setAllLineItemsLocalDataIsUpdateAllChecked, setEntityRecommendationForLineIdsForTransactionDetail, markCategoryClassRecommendationsInProgressForTransactionDetail, markCategoryClassRecommendationsCompletedForTransactionDetail, markCategoryClassRecommendationsFailureForTransactionDetail, removeEntityRecommendationForLineIdForTransactionDetail, updateSelectedVendor, updateSelectedCustomer, resetVendor, resetSelectedCustomer, updateLocalDataWithTransactionsUpdateWithVendorCheckbox, updateVendorToAllLineItems, resetAllLineItemsToVendor, updateLatestSelectedEntity, updateVendorBulkRecommendationsFetchStatus, setIsVendorUpdateAllChecked, } = transactionDetail.actions;
 export default transactionDetail.reducer;
 function getUpdatedEntityLocalData(detail, updates, entity, resetRecommendations, type) {
     const key = type === 'customer' ? 'customerUpdates' : 'vendorUpdates';

package/lib/esm/view/transactionDetail/transactionDetailSelector.js

@@ -10,6 +10,26 @@ import { getAccountList, getNestedAccountListHierarchy, } from '../accountList/a
 import { getClassList, getNestedClassListHierarchy, } from '../classList/classListSelector';
 import { getProjectList, } from '../projectList/projectListSelector';
 import { getAllLinkedTransactions, getTransactionDetailKey, initialSupportedTransactionDetail, } from './transactionDetailState';
+// "Any line on this transaction is fetching category/class recommendations?"
+// Inlined as a plain function because the upstream selector
+// `getTransactionDetail` is also un-memoized and the iteration is O(lines)
+// — typically <10. Exported standalone so consumers that only need the
+// flag (e.g. footer save-disable) can avoid materializing the whole
+// detail report.
+export const getDetailInFlightRecommendations = (state, transactionId) => {
+    const detailKey = getTransactionDetailKey(transactionId);
+    const detail = recordGet(state.transactionDetailState.transactionDetailById, detailKey, undefined);
+    if (detail == null) {
+        return false;
+    }
+    for (const lineItem of Object.values(detail.transactionDetailLocalData.lineItemById)) {
+        if (lineItem.categoryClassRecommendationsFetchState?.fetchState ===
+            'In-Progress') {
+            return true;
+        }
+    }
+    return false;
+};
 export const getTransactionDetail = (state, transactionId, fetchLinkedTransactions = true) => {
     const { transactionDetailState, transactionState, vendorState, vendorListState, accountState, classState, classListState, accountListState, projectState, projectListState, } = state;
     const detailKey = getTransactionDetailKey(transactionId);
@@ -56,7 +76,13 @@ export const getTransactionDetail = (state, transactionId, fetchLinkedTransactio
     const isAccountingClassesEnabled = getIsAccountingClassesEnabled(state);
     const classList = isAccountingClassesEnabled
         ? getClassList(classState, classListState)
-        : { classes: [], nestedClassHierarchy: [], fetchState: 'Completed', error: undefined, version: 0 };
+        : {
+            classes: [],
+            nestedClassHierarchy: [],
+            fetchState: 'Completed',
+            error: undefined,
+            version: 0,
+        };
     const isAccountingProjectsEnabled = getIsAccountingProjectsEnabled(state);
     const projectList = isAccountingProjectsEnabled
         ? getProjectList(projectState, projectListState)
@@ -72,6 +98,7 @@ export const getTransactionDetail = (state, transactionId, fetchLinkedTransactio
         : [];
     const transactionDetailAnyUpdateStatus = reduceUpdateFetchStates(transactionDetail);
     const vendors = getVendorsByVendorIds(vendorState, vendorListState.vendorIds);
+    const hasInFlightCategoryClassRecommendations = getDetailInFlightRecommendations(state, transactionId);
     return {
         reportId: 'transaction_detail',
         reportTitle: 'Transaction Detail',
@@ -87,6 +114,7 @@ export const getTransactionDetail = (state, transactionId, fetchLinkedTransactio
         classHierarchyList,
         accountsHierarchyList,
         classList: classList,
+        hasInFlightCategoryClassRecommendations,
         isAccountingClassesEnabled,
         isAccountingProjectsEnabled,
         projectList,