@zeniai/client-epic-state 5.0.76 → 5.0.77-betaAD01

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/fetchMultipleBatchDetailsEpic.js

@@ -31,10 +31,19 @@ export function fetchBatchDetailsByIds(batchIds, zeniAPI) {
 }
 export const fetchMultipleBatchDetailsEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(fetchBulkUploadBatchesSuccess.match), mergeMap((action) => {
     const batchList = action.payload.batchList;
-    const { batchDetailsById } = state$.value.expenseAutomationMissingReceiptsViewState.bulkUpload;
+    const bulkUpload = state$.value.expenseAutomationMissingReceiptsViewState.bulkUpload;
+    const { batchDetailsById, batchDetailsRevalidating } = bulkUpload;
     /** Only completed list rows may trigger batch-details API; preserve platform order. */
-    const completedBatchesMissingDetails = filterBatchListItemsForBatchDetailsFetch(batchList).filter((batch) => batchDetailsById[batch.batchId] == null);
-    const completedBatchIds = completedBatchesMissingDetails.map((batch) => batch.batchId);
+    const completedRows = filterBatchListItemsForBatchDetailsFetch(batchList);
+    /**
+     * Stale-while-revalidate: when a manual refresh has set `batchDetailsRevalidating`, refresh
+     * every completed list row (replacing entries via `storeBatchDetails` as they arrive). On a
+     * normal first fetch we still only request rows whose details are missing.
+     */
+    const candidateRows = batchDetailsRevalidating === true
+        ? completedRows
+        : completedRows.filter((batch) => batchDetailsById[batch.batchId] == null);
+    const completedBatchIds = candidateRows.map((batch) => batch.batchId);
     const batchIdsToFetch = completedBatchIds.slice(0, INITIAL_BATCH_PAGE_SIZE);
     if (batchIdsToFetch.length === 0) {
         /**
@@ -42,7 +51,6 @@ export const fetchMultipleBatchDetailsEpic = (actions$, state$, zeniAPI) => acti
          * `currentBatchId` may not appear in the filtered list but still needs `GET /batches/:id`.
          * Fall back to the single-batch-details epic so `phase` and details can resolve.
          */
-        const bulkUpload = state$.value.expenseAutomationMissingReceiptsViewState.bulkUpload;
         const currentId = bulkUpload.currentBatchId;
         if (currentId != null &&
             bulkUpload.phase === 'matching' &&
@@ -50,7 +58,15 @@ export const fetchMultipleBatchDetailsEpic = (actions$, state$, zeniAPI) => acti
             shouldFetchBatchDetailsForBatchId(bulkUpload.batchStatusById, bulkUpload.batchListByPeriod, currentId)) {
             return of(fetchBulkUploadBatchDetails());
         }
-        return EMPTY;
+        /**
+         * Close the pagination loop so the Unmatched-tab loading heuristic can settle.
+         * `fetchMoreBatchDetailsComplete` also clears `batchDetailsRevalidating` and flips
+         * `initialBatchDetailsLoaded` to true. Without this, `batchDetailsPaginationState` stays
+         * at `Not-Started` forever after a list refresh that finds zero completed rows (or all
+         * already cached) — the direct-open-Unmatched-without-upload path used to render an
+         * infinite skeleton.
+         */
+        return of(fetchMoreBatchDetailsComplete());
     }
     return concat(of(setInitialBatchDetailsLoading()), fetchBatchDetailsByIds(batchIdsToFetch, zeniAPI), of(fetchMoreBatchDetailsComplete()));
 }));

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/refreshBatchDetailsForBatchIdEpic.js

@@ -0,0 +1,52 @@
+import { EMPTY, concat, from, of } from 'rxjs';
+import { catchError, filter, mergeMap } from 'rxjs/operators';
+import { updateTransactions } from '../../../../entity/transaction/transactionReducer';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
+import { extractTransactionPayloadsFromBatchFiles, isBatchDetailsApiStatusCompleted, toBatchDetails, } from '../../payload/missingReceiptsPayload';
+import { batchDetailFetchFailed, markBatchDetailRefreshAttempted, refreshBatchDetailsForBatchId, storeBatchDetails, } from '../../reducers/missingReceiptsViewReducer';
+/**
+ * Pusher resends the same `completed` event in some cases (reconnect, server retry). Skip a
+ * targeted refresh if we issued one for the same batch within this window — the cached details
+ * are already fresh, so another `GET /1.0/batches/:id` is wasteful and visually flickers nothing.
+ */
+const TARGETED_REFRESH_DEDUPE_MS = 5000;
+/**
+ * Single-batch stale-while-revalidate refresh. Triggered by `pusherBatchStatusCompletionEpic`
+ * (and any other path that wants to update one batch without wiping the cache). The response is
+ * stored via `storeBatchDetails`, which replaces the entry by `batchId` so the row updates in
+ * place — no skeleton, no list reflow.
+ *
+ * Throttle: `lastBatchDetailRefreshAtById[batchId]` is checked against `TARGETED_REFRESH_DEDUPE_MS`
+ * before issuing the call. The marker is emitted first so concurrent dispatches (e.g. Pusher
+ * resend during an in-flight request) see the dedupe timestamp and collapse to a single call.
+ */
+export const refreshBatchDetailsForBatchIdEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(refreshBatchDetailsForBatchId.match), mergeMap((action) => {
+    const { batchId } = action.payload;
+    const now = Date.now();
+    const lastAttempt = state$.value.expenseAutomationMissingReceiptsViewState.bulkUpload
+        .lastBatchDetailRefreshAtById[batchId];
+    if (lastAttempt != null &&
+        now - lastAttempt < TARGETED_REFRESH_DEDUPE_MS) {
+        return EMPTY;
+    }
+    const apiCall = zeniAPI
+        .getJSON(`${zeniAPI.apiEndPoints.communicationAgentMicroServiceBaseUrl}/1.0/batches/${batchId}`)
+        .pipe(mergeMap((response) => {
+        if (!isSuccessResponse(response) ||
+            response.data == null ||
+            !isBatchDetailsApiStatusCompleted(response.data.status)) {
+            return EMPTY;
+        }
+        const actions = [];
+        const transactionPayloads = extractTransactionPayloadsFromBatchFiles(response.data.files);
+        if (transactionPayloads.length > 0) {
+            actions.push(updateTransactions(transactionPayloads, (value) => value.transaction_id));
+        }
+        actions.push(storeBatchDetails(toBatchDetails(response.data, zeniAPI.apiEndPoints.fileMicroServiceBaseUrl)));
+        return from(actions);
+    }), catchError((error) => {
+        console.error(`Failed to refresh batch details for ${batchId}:`, createZeniAPIStatus('Unexpected Error', 'Refresh batch details errored out: ' + JSON.stringify(error)));
+        return of(batchDetailFetchFailed({ batchId }));
+    }));
+    return concat(of(markBatchDetailRefreshAttempted({ batchId, timestamp: now })), apiCall);
+}));

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/restoreBulkUploadAutomatchingOnMountEpic.js

@@ -1,12 +1,12 @@
 import { EMPTY, concat, from, of } from 'rxjs';
 import { catchError, concatMap, filter, mergeMap, take } from 'rxjs/operators';
+import { toMonthYearPeriodId } from '../../../../commonStateTypes/timePeriod';
+import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
 import { isSuccessResponse } from '../../../../responsePayload';
-import { isBatchDetailsStatusPendingOrProcessing, toBatchListItem, } from '../../payload/missingReceiptsPayload';
-import { fetchBulkUploadBatchesSuccess, restoreBulkUploadAutomatchingOnMount, restoreBulkUploadMatchingState, } from '../../reducers/missingReceiptsViewReducer';
 import { BULK_UPLOAD_BATCH_LIST_DATE_RANGE_DAYS } from '../../helpers/bulkUploadTiming';
 import { rollingCalendarDateRangeInclusive } from '../../helpers/rollingCalendarDateRangeInclusive';
-import { toMonthYearPeriodId } from '../../../../commonStateTypes/timePeriod';
-import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
+import { isBatchDetailsStatusPendingOrProcessing, toBatchListItem, } from '../../payload/missingReceiptsPayload';
+import { fetchBulkUploadBatchesSuccess, restoreBulkUploadAutomatchingOnMount, restoreBulkUploadMatchingState, } from '../../reducers/missingReceiptsViewReducer';
 /**
  * On missing-receipts entry / period switch, hydrate the automatching banner for the logged-in
  * user without relying on UI-driven list fetches (the Missing tab does not fetch the batch list
@@ -49,10 +49,12 @@ export const restoreBulkUploadAutomatchingOnMountEpic = (actions$, state$, zeniA
         bulkUpload.batchListFetchState.fetchState === 'Completed';
     const listSource = alreadyCached
         ? of(null)
-        : zeniAPI.getJSON(listUrl).pipe(catchError(() => of(null)));
+        : zeniAPI
+            .getJSON(listUrl)
+            .pipe(catchError(() => of(null)));
     return listSource.pipe(mergeMap((listResponse) => {
         const batchList = alreadyCached
-            ? bulkUpload.batchListByPeriod[periodId] ?? []
+            ? (bulkUpload.batchListByPeriod[periodId] ?? [])
             : listResponse != null &&
                 isSuccessResponse(listResponse) &&
                 listResponse.data != null

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/searchTransactionsForManualMatchEpic.js

@@ -2,10 +2,11 @@ import { EMPTY, from, merge, of } from 'rxjs';
 import { catchError, debounceTime, filter, mergeMap, switchMap, } from 'rxjs/operators';
 import { updateTransactions } from '../../../../entity/transaction/transactionReducer';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
+import { rollingCalendarDateRangeInclusive } from '../../helpers/rollingCalendarDateRangeInclusive';
 import { supportedTransactionPayloadToManualSearchResult } from '../../payload/missingReceiptsPayload';
 import { searchTransactionsForManualMatch, searchTransactionsForManualMatchFailure, searchTransactionsForManualMatchSuccess, } from '../../reducers/missingReceiptsViewReducer';
+import { DEFAULT_COMPLETED_SUB_TAB } from '../../types/completedSubTab';
 import { toTransactionsSortKey } from '../../types/transactionsViewState';
-import { rollingCalendarDateRangeInclusive } from '../../helpers/rollingCalendarDateRangeInclusive';
 /**
  * `auto_categorized` for manual transaction search (Unmatched).
  * Backend contract TBD — align with `fetchTransactionCategorizationEpic` (`selectedTab === 'autoCategorized'`)
@@ -46,6 +47,11 @@ export const searchTransactionsForManualMatchEpic = (actions$, state$, zeniAPI)
             page_token: pageToken ?? null,
             page_size: manualSearch.pageSize,
             search_text: query,
+            // Manual search ignores the Completed sub-tab axis — always span the
+            // default sentinel so the receipt-matching UX returns results
+            // regardless of where the transaction would otherwise be bucketed on
+            // the Completed tab.
+            sub_tab: DEFAULT_COMPLETED_SUB_TAB,
         };
         return zeniAPI
             .getJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/expense-automation/transactions?query=${encodeURIComponent(JSON.stringify(queryParam))}`)

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/syncTabsAfterAutomatchEpic.js

@@ -0,0 +1,56 @@
+import { EMPTY, from, merge } from 'rxjs';
+import { filter, switchMap, take, takeUntil } from 'rxjs/operators';
+import { convertToPeriod, toAbsoluteDay, } from '../../../../commonStateTypes/timePeriod';
+import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
+import { bulkUploadReceiptsSuccess, clearBulkUpload, fetchBulkUploadBatchDetailsSuccess, fetchCompletedTransactions, fetchMissingReceipts, restoreBulkUploadMatchingState, storeBatchDetails, } from '../../reducers/missingReceiptsViewReducer';
+/**
+ * After automatch completes for the active bulk-upload batch, refresh the two
+ * caches that are not invalidated by the existing batch-details refresh path:
+ *   - Missing Receipts list (`transactionIdsBySelectedPeriod`)
+ *   - Completed transactions cache (`bulkUpload.completedTransactionsByPeriod`)
+ *
+ * The Unmatched tab is already kept fresh via `pusherBatchStatusCompletionEpic`
+ * dispatching `fetchBulkUploadBatches({invalidateBatchDetailsCache: true})`,
+ * which feeds `fetchMultipleBatchDetailsEpic` and re-stores batch details.
+ *
+ * Lifecycle mirrors `bulkUploadMatchResultToastEpic`: trigger on
+ * `bulkUploadReceiptsSuccess` (fresh upload) or `restoreBulkUploadMatchingState`
+ * (restored session), then wait for the first batch-details action for that
+ * batchId — landing as either `fetchBulkUploadBatchDetailsSuccess` or
+ * `storeBatchDetails`. `clearBulkUpload` cancels the in-flight wait.
+ *
+ * Refetches use `cacheOverride: true` to bypass per-tab cache guards, and
+ * `fetchMissingReceipts` uses `refreshViewInBackground: true` to keep the
+ * existing rows visible while fresh data is in flight (the authoritative
+ * server filter `is_attachment_missing_only=true` removes matched rows once
+ * the response lands).
+ */
+export const syncTabsAfterAutomatchEpic = (actions$, state$) => actions$.pipe(filter((action) => bulkUploadReceiptsSuccess.match(action) ||
+    restoreBulkUploadMatchingState.match(action)), switchMap((uploadAction) => {
+    const { batchId } = uploadAction.payload;
+    return merge(actions$.pipe(filter(fetchBulkUploadBatchDetailsSuccess.match)), actions$.pipe(filter(storeBatchDetails.match))).pipe(filter((a) => a.payload.batchId === batchId), take(1), switchMap((action) => {
+        const { summary } = action.payload;
+        if (summary.matched <= 0) {
+            return EMPTY;
+        }
+        const state = state$.value;
+        const currentTenant = getCurrentTenant(state);
+        if (currentTenant == null) {
+            return EMPTY;
+        }
+        const selectedPeriod = state.expenseAutomationViewState.selectedPeriodByTenantId[currentTenant.tenantId];
+        if (selectedPeriod == null) {
+            return from([fetchCompletedTransactions(undefined, true)]);
+        }
+        const period = convertToPeriod(selectedPeriod);
+        const timePeriodStart = toAbsoluteDay(period.start);
+        const timePeriodEnd = toAbsoluteDay(period.end);
+        if (timePeriodStart == null || timePeriodEnd == null) {
+            return from([fetchCompletedTransactions(undefined, true)]);
+        }
+        return from([
+            fetchMissingReceipts({ start: timePeriodStart, end: timePeriodEnd }, { amount: 75.0, currencyCode: 'USD', currencySymbol: '$' }, true, true),
+            fetchCompletedTransactions(undefined, true),
+        ]);
+    }), takeUntil(actions$.pipe(filter(clearBulkUpload.match))));
+}));

package/lib/esm/view/expenseAutomationView/epics/missingReceipts/watchBulkUploadBatchStatusEpic.js

@@ -5,23 +5,27 @@ import { updateTransactions } from '../../../../entity/transaction/transactionRe
 import { isSuccessResponse } from '../../../../responsePayload';
 import { BULK_UPLOAD_AUTOMATCHING_TIMEOUT_MS } from '../../helpers/bulkUploadTiming';
 import { extractTransactionPayloadsFromBatchFiles, isBatchDetailsApiStatusCompleted, toBatchDetails, } from '../../payload/missingReceiptsPayload';
-import { bulkUploadAutomatchingTimedOut, bulkUploadReceiptsSuccess, clearBulkUpload, fetchBulkUploadBatchDetails, fetchBulkUploadBatchDetailsSuccess, fetchBulkUploadBatches, pusherBatchStatusUpdate, restoreBulkUploadMatchingState, } from '../../reducers/missingReceiptsViewReducer';
+import { bulkUploadAutomatchingTimedOut, bulkUploadReceiptsSuccess, clearBulkUpload, fetchBulkUploadBatchDetails, fetchBulkUploadBatchDetailsSuccess, fetchBulkUploadBatches, pusherBatchStatusUpdate, refreshBatchDetailsForBatchId, restoreBulkUploadMatchingState, } from '../../reducers/missingReceiptsViewReducer';
 /** First poll after upload; then interval while batch not resolved via Pusher or API. */
 const FALLBACK_FIRST_DELAY_MS = 30000;
 /** Shorter first poll for restored sessions — batch may already be near completion. */
 const RESTORE_FIRST_DELAY_MS = 5000;
 const FALLBACK_POLL_INTERVAL_MS = 10000;
 /**
- * On Pusher batch completion: refresh batch list (then fetchMultipleBatchDetailsEpic runs).
- * Debounced to reduce duplicate refreshes. The post-match tab decision
- * (Unmatched vs Completed) is made client-side in `ExpenseAutomationPage` once fresh
+ * On Pusher batch completion: refresh the batch list (so summary counts update) and dispatch a
+ * targeted single-batch details refresh for the batch that just completed. Stale-while-revalidate:
+ * we no longer pass `invalidateBatchDetailsCache: true` — existing `batchDetailsById` entries stay
+ * on screen and the just-completed row replaces in place via `storeBatchDetails`. The post-match
+ * tab decision (Unmatched vs Completed) is made client-side in `ExpenseAutomationPage` once fresh
  * batch details land -- do not dispatch `requestMissingReceiptsTabNavigation` here.
+ *
+ * Pusher resend storms (same batch completing twice within the dedupe window) are throttled in
+ * `refreshBatchDetailsForBatchIdEpic` via `lastBatchDetailRefreshAtById`, so this epic stays
+ * stateless and just forwards the event.
  */
-export const pusherBatchStatusCompletionEpic = (actions$) => actions$.pipe(filter(pusherBatchStatusUpdate.match), filter((action) => action.payload.status === 'completed'), debounceTime(300), mergeMap(() => from([
-    fetchBulkUploadBatches({
-        cacheOverride: true,
-        invalidateBatchDetailsCache: true,
-    }),
+export const pusherBatchStatusCompletionEpic = (actions$) => actions$.pipe(filter(pusherBatchStatusUpdate.match), filter((action) => action.payload.status === 'completed'), debounceTime(300), mergeMap((action) => from([
+    fetchBulkUploadBatches({ cacheOverride: true }),
+    refreshBatchDetailsForBatchId({ batchId: action.payload.batchId }),
 ])));
 /**
  * If Pusher and batch-details are delayed beyond {@link BULK_UPLOAD_AUTOMATCHING_TIMEOUT_MS},

package/lib/esm/view/expenseAutomationView/epics/transactionCategorization/fetchTransactionCategorizationEpic.js

@@ -5,7 +5,8 @@ import { getCurrentTenant } from '../../../../entity/tenant/tenantSelector';
 import { updateTransactions } from '../../../../entity/transaction/transactionReducer';
 import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
 import { getLineItemsByTransactionsIdsFromResponse } from '../../helpers/transactionCategorizationLocalDataHelper';
-import { fetchTransactionCategorization, fetchTransactionCategorizationFailure, fetchTransactionCategorizationSuccess, initializeTransactionCategorizationViewLocalData, resetOtherTabsFetchState, updateTotalCountForTransactionCategorization, updateTransactionCategorizationUIState, } from '../../reducers/transactionsViewReducer';
+import { fetchTransactionCategorization, fetchTransactionCategorizationFailure, fetchTransactionCategorizationSuccess, initializeTransactionCategorizationViewLocalData, resetOtherTabsFetchState, updateParentTotalCountForTab, updateTotalCountForTransactionCategorization, updateTransactionCategorizationUIState, } from '../../reducers/transactionsViewReducer';
+import { DEFAULT_COMPLETED_SUB_TAB } from '../../types/completedSubTab';
 import { TRANSACTIONS_TABS, toTransactionsSortKey, } from '../../types/transactionsViewState';
 export const fetchTransactionCategorizationEpic = (actions$, state$, zeniAPI) => actions$.pipe(filter(fetchTransactionCategorization.match), switchMap((action) => {
     const { expenseAutomationViewState: { selectedPeriodByTenantId }, expenseAutomationTransactionsViewState, } = state$.value;
@@ -23,6 +24,9 @@ export const fetchTransactionCategorizationEpic = (actions$, state$, zeniAPI) =>
         page_token: pageToken,
         page_size: 25,
         search_text: uiState.searchString,
+        sub_tab: selectedTab === 'autoCategorized'
+            ? expenseAutomationTransactionsViewState.selectedTransactionCategorizationCompletedSubTab
+            : DEFAULT_COMPLETED_SUB_TAB,
     };
     return zeniAPI
         .getJSON(`${zeniAPI.apiEndPoints.accountMicroServiceBaseUrl}/1.0/expense-automation/transactions?query=${encodeURIComponent(JSON.stringify(queryParam))}`)
@@ -45,6 +49,12 @@ export const fetchTransactionCategorizationEpic = (actions$, state$, zeniAPI) =>
                 selectedTab,
                 totalCount: response.data.total_count,
             }));
+            updateActions.push(updateParentTotalCountForTab({
+                selectedPeriod,
+                selectedTab,
+                parentTotalCount: response.data.parent_tab_total_count ??
+                    response.data.total_count,
+            }));
             updateActions.push(resetOtherTabsFetchState({
                 refreshViewInBackground,
                 tabs: TRANSACTIONS_TABS.filter((tab) => selectedTab !== tab),

package/lib/esm/view/expenseAutomationView/helpers/transactionCategorizationLocalDataHelper.js

@@ -59,6 +59,22 @@ export const filterAutoTabLineItems = (tabView, transactionId, updatedTransactio
     }
 };
 export const MAX_SELECTION_LIMIT = 5;
+// Returns true when the transaction should be auto-added to the
+// selectedCheckBoxTransactionIds list after a categorization-related
+// state mutation: all line items are filled, the selection limit has
+// not been hit, and the id isn't already in the list. Consolidates the
+// triplet that was inlined at four sites (updateSelectedCustomer/Vendor,
+// setAllItemsToCategoryClassInLocalDataForCategorization, and
+// setEntityRecommendationForLineIdInLocalData below).
+export const shouldAutoSelectAndAdd = (transaction, localData, selectedCheckBoxTransactionIds, uncategorizedAccounts, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled) => {
+    if (selectedCheckBoxTransactionIds.includes(transaction.id)) {
+        return false;
+    }
+    if (selectedCheckBoxTransactionIds.length >= MAX_SELECTION_LIMIT) {
+        return false;
+    }
+    return checkIfAllLineItemsAreCategoryClassFilled(transaction, localData, uncategorizedAccounts, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled);
+};
 export const toTransactionDetailLocalData = (transaction, selectedTab, lineItemsByTransactionIds, uncategorizedIncomeExpense, isUncategorizedExpenseCategoryEnabled) => {
     const transactionLocal = {
         ...initialSupportedTransactionCategorization.transactionReviewLocalData,
@@ -544,10 +560,14 @@ export const setEntityRecommendationForLineIdInLocalData = (draft, transaction,
         selectedEntity: transactionDetailLocalData.selectedEntity,
         tabSpecificLineItems: transactionDetailLocalData.tabSpecificLineItems,
     };
+    // Auto-uncheck if not all line items are filled (the row's
+    // category/class became stale after the entity update). Otherwise
+    // route through shouldAutoSelectAndAdd so the MAX_SELECTION_LIMIT
+    // gate is consistently applied across all auto-add sites.
     if (!checkIfAllLineItemsAreCategoryClassFilled(transaction, newLocalData, uncategorizedAccounts, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled)) {
         selectedCheckBoxTransactionIds = selectedCheckBoxTransactionIds.filter((id) => id !== transaction.id);
     }
-    else if (selectedCheckBoxTransactionIds.includes(transaction.id) === false) {
+    else if (shouldAutoSelectAndAdd(transaction, newLocalData, selectedCheckBoxTransactionIds, uncategorizedAccounts, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled)) {
         selectedCheckBoxTransactionIds.push(transaction.id);
     }
     draft.selectedCheckBoxTransactionIds = selectedCheckBoxTransactionIds;

package/lib/esm/view/expenseAutomationView/reducers/missingReceiptsViewReducer.js

@@ -1,5 +1,6 @@
 import { createAction, createSlice } from '@reduxjs/toolkit';
 import { toMonthYearPeriodId } from '../../../commonStateTypes/timePeriod';
+import { DEFAULT_COMPLETED_SUB_TAB, } from '../types/completedSubTab';
 import { MIN_MANUAL_TRANSACTION_SEARCH_LENGTH, } from '../types/missingReceiptsViewState';
 export const getCompletedTransactionsCacheKey = (periodId, sortKey, sortOrder, subTab) => {
     const sortKeyPart = sortKey ?? 'none';
@@ -9,15 +10,18 @@ export const getCompletedTransactionsCacheKey = (periodId, sortKey, sortOrder, s
 export const initialBulkUploadState = {
     batchDetailsById: {},
     batchDetailsPaginationState: { fetchState: 'Not-Started', error: undefined },
+    batchDetailsRevalidating: false,
     batchListByPeriod: {},
     batchListFetchState: { fetchState: 'Not-Started', error: undefined },
     batchStatusById: {},
-    completedSubTab: 'all',
+    completedSubTab: DEFAULT_COMPLETED_SUB_TAB,
     completedTransactionsByPeriod: {},
     confirmMatchStatus: { fetchState: 'Not-Started', error: undefined },
     currentBatchId: undefined,
     currentResultsTab: 'unmatched',
     failedBatchDetailIds: [],
+    initialBatchDetailsLoaded: false,
+    lastBatchDetailRefreshAtById: {},
     manualSearchUiResetKey: 0,
     manualSearch: {
         fetchState: { fetchState: 'Not-Started', error: undefined },
@@ -64,6 +68,13 @@ export const fetchBulkUploadBatchDetails = createAction('expenseAutomationMissin
  * after a hard refresh. The epic owns the list + details API calls end-to-end.
  */
 export const restoreBulkUploadAutomatchingOnMount = createAction('expenseAutomationMissingReceiptsView/restoreBulkUploadAutomatchingOnMount');
+/**
+ * Epic trigger only. Targeted stale-while-revalidate refresh for a single batch's details — used
+ * by `pusherBatchStatusCompletionEpic` so a Pusher 'completed' event re-fetches only that one
+ * batch instead of wiping the entire `batchDetailsById` cache. The epic dedupes via
+ * `lastBatchDetailRefreshAtById` so resend storms don't trigger waves of API calls.
+ */
+export const refreshBatchDetailsForBatchId = createAction('expenseAutomationMissingReceiptsView/refreshBatchDetailsForBatchId');
 const expenseAutomationMissingReceiptsView = createSlice({
     name: 'expenseAutomationMissingReceiptsView',
     initialState,
@@ -250,6 +261,16 @@ const expenseAutomationMissingReceiptsView = createSlice({
             const details = action.payload;
             draft.bulkUpload.batchDetailsById[details.batchId] = details;
             draft.bulkUpload.phase = 'completed';
+            draft.bulkUpload.initialBatchDetailsLoaded = true;
+            /**
+             * A successful detail load supersedes any previous failure for this batch — without this,
+             * `getPrimaryBatchIdFromBatchListOrder` (and pagination) would keep skipping the batch
+             * across the recovery, hiding now-valid Unmatched-tab work.
+             */
+            const failedIndex = draft.bulkUpload.failedBatchDetailIds.indexOf(details.batchId);
+            if (failedIndex >= 0) {
+                draft.bulkUpload.failedBatchDetailIds.splice(failedIndex, 1);
+            }
         },
         fetchBulkUploadBatchDetailsFailure: {
             reducer(draft) {
@@ -271,6 +292,19 @@ const expenseAutomationMissingReceiptsView = createSlice({
                 draft.bulkUpload.currentBatchId === details.batchId) {
                 draft.bulkUpload.phase = 'completed';
             }
+            /** Stale-while-revalidate: any successful detail load means we have rendered data. */
+            draft.bulkUpload.initialBatchDetailsLoaded = true;
+            /**
+             * Stale-while-revalidate recovery: if this batch had previously failed, a successful
+             * targeted refresh (Pusher `completed` → `refreshBatchDetailsForBatchIdEpic`) or
+             * multi-batch revalidation (manual refresh) supersedes the failure. Selectors that gate
+             * on `failedBatchDetailIds` (primary-batch resolution, `hasMoreBatchDetails`) must see the
+             * batch as recovered so it can show fresh Unmatched work.
+             */
+            const failedIndex = draft.bulkUpload.failedBatchDetailIds.indexOf(details.batchId);
+            if (failedIndex >= 0) {
+                draft.bulkUpload.failedBatchDetailIds.splice(failedIndex, 1);
+            }
         },
         batchDetailFetchFailed(draft, action) {
             const { batchId } = action.payload;
@@ -299,6 +333,14 @@ const expenseAutomationMissingReceiptsView = createSlice({
                 fetchState: 'Completed',
                 error: undefined,
             };
+            /**
+             * Stale-while-revalidate: a completed pagination round means the initial page (or a
+             * subsequent revalidation) has fully resolved. Once `initialBatchDetailsLoaded` flips to
+             * `true` it stays `true` for the session — only `clearBulkUpload` /
+             * `clearBulkUploadBatchDetailsForScopeChange` can reset it.
+             */
+            draft.bulkUpload.initialBatchDetailsLoaded = true;
+            draft.bulkUpload.batchDetailsRevalidating = false;
             const currentId = draft.bulkUpload.currentBatchId;
             const currentDetails = currentId != null
                 ? draft.bulkUpload.batchDetailsById[currentId]
@@ -314,12 +356,24 @@ const expenseAutomationMissingReceiptsView = createSlice({
                 fetchState: 'Error',
                 error: action.payload.error,
             };
+            draft.bulkUpload.batchDetailsRevalidating = false;
         },
         fetchBulkUploadBatches: {
             reducer(draft, action) {
+                /**
+                 * Manual refresh button (and any other caller passing `invalidateBatchDetailsCache: true`):
+                 * wipe the details cache, failed-id list, and pagination state so the Unmatched tab drops
+                 * its existing rows and re-renders the initial-load skeleton while the next list +
+                 * details fetches resolve. The post-success epic (`fetchMultipleBatchDetailsEpic`) then
+                 * re-runs the details chain from a clean slate.
+                 */
                 if (action.payload.invalidateBatchDetailsCache === true) {
                     draft.bulkUpload.batchDetailsById = {};
                     draft.bulkUpload.failedBatchDetailIds = [];
+                    draft.bulkUpload.batchDetailsPaginationState = {
+                        fetchState: 'Not-Started',
+                        error: undefined,
+                    };
                 }
                 draft.bulkUpload.batchListFetchState = {
                     fetchState: 'In-Progress',
@@ -336,6 +390,31 @@ const expenseAutomationMissingReceiptsView = createSlice({
                 };
             },
         },
+        /**
+         * Tenant/period scope change — the new scope's data is genuinely unknown so the existing
+         * details cache and `initialBatchDetailsLoaded` flag must be reset (the Unmatched skeleton
+         * is acceptable on scope change). Distinct from a stale-while-revalidate refresh.
+         */
+        clearBulkUploadBatchDetailsForScopeChange(draft) {
+            draft.bulkUpload.batchDetailsById = {};
+            draft.bulkUpload.failedBatchDetailIds = [];
+            draft.bulkUpload.batchDetailsPaginationState = {
+                fetchState: 'Not-Started',
+                error: undefined,
+            };
+            draft.bulkUpload.batchDetailsRevalidating = false;
+            draft.bulkUpload.initialBatchDetailsLoaded = false;
+            draft.bulkUpload.lastBatchDetailRefreshAtById = {};
+        },
+        /**
+         * Records the last time we issued a targeted refresh for `batchId`. Set by
+         * `refreshBatchDetailsForBatchIdEpic` before each API call so subsequent Pusher resends in the
+         * dedupe window are skipped.
+         */
+        markBatchDetailRefreshAttempted(draft, action) {
+            draft.bulkUpload.lastBatchDetailRefreshAtById[action.payload.batchId] =
+                action.payload.timestamp;
+        },
         fetchBulkUploadBatchesSuccess(draft, action) {
             const periodId = toMonthYearPeriodId(action.payload.selectedPeriod);
             draft.bulkUpload.batchListByPeriod[periodId] = action.payload.batchList;
@@ -578,7 +657,7 @@ const expenseAutomationMissingReceiptsView = createSlice({
 });
 export const { fetchMissingReceipts, fetchMissingReceiptsSuccess, fetchMissingReceiptsFailure, uploadMissingReceiptSuccess, updateMissingReceiptUploadState, updateMissingReceiptsUIState, markMissingReceiptAsDone, clearExpenseAutomationMissingReceiptsView, 
 // Bulk Upload
-bulkUploadReceipts, updateBulkUploadProgress, setBulkUploadUploadedFileCount, bulkUploadReceiptsSuccess, bulkUploadReceiptsFailure, restoreBulkUploadMatchingState, bulkUploadAutomatchingTimedOut, pusherBatchStatusUpdate, requestMissingReceiptsTabNavigation, clearMissingReceiptsTabNavigation, fetchBulkUploadBatchDetailsSuccess, fetchBulkUploadBatchDetailsFailure, storeBatchDetails, batchDetailFetchFailed, setInitialBatchDetailsLoading, fetchMoreBatchDetails, fetchMoreBatchDetailsComplete, fetchMoreBatchDetailsFailure, fetchBulkUploadBatches, fetchBulkUploadBatchesSuccess, fetchBulkUploadBatchesFailure, confirmBulkUploadMatch, confirmBulkUploadMatchSuccess, confirmBulkUploadMatchFailure, setBulkUploadResultsTab, setBulkUploadCompletedSubTab, setBulkUploadSortConfig, clearBulkUpload, searchTransactionsForManualMatch, searchTransactionsForManualMatchSuccess, searchTransactionsForManualMatchFailure, clearManualSearchResults, acknowledgeBulkUploadConfirmMatchComplete, 
+bulkUploadReceipts, updateBulkUploadProgress, setBulkUploadUploadedFileCount, bulkUploadReceiptsSuccess, bulkUploadReceiptsFailure, restoreBulkUploadMatchingState, bulkUploadAutomatchingTimedOut, pusherBatchStatusUpdate, requestMissingReceiptsTabNavigation, clearMissingReceiptsTabNavigation, fetchBulkUploadBatchDetailsSuccess, fetchBulkUploadBatchDetailsFailure, storeBatchDetails, batchDetailFetchFailed, setInitialBatchDetailsLoading, fetchMoreBatchDetails, fetchMoreBatchDetailsComplete, fetchMoreBatchDetailsFailure, fetchBulkUploadBatches, fetchBulkUploadBatchesSuccess, fetchBulkUploadBatchesFailure, clearBulkUploadBatchDetailsForScopeChange, markBatchDetailRefreshAttempted, confirmBulkUploadMatch, confirmBulkUploadMatchSuccess, confirmBulkUploadMatchFailure, setBulkUploadResultsTab, setBulkUploadCompletedSubTab, setBulkUploadSortConfig, clearBulkUpload, searchTransactionsForManualMatch, searchTransactionsForManualMatchSuccess, searchTransactionsForManualMatchFailure, clearManualSearchResults, acknowledgeBulkUploadConfirmMatchComplete, 
 // Completed Transactions
 fetchCompletedTransactions, fetchCompletedTransactionsSuccess, fetchCompletedTransactionsFailure, } = expenseAutomationMissingReceiptsView.actions;
 export default expenseAutomationMissingReceiptsView.reducer;