@zeniai/client-epic-state 4.19.84-betaVR6 → 4.19.84-betaVR8

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

@@ -12,9 +12,13 @@ export const bulkUploadReceiptsEpic = (actions$, _state$, zeniAPI) => actions$.p
         .postFormData(`${zeniAPI.apiEndPoints.communicationAgentMicroServiceBaseUrl}/1.0/receipts/bulk-upload`, formData)
         .pipe(mergeMap((response) => {
         if (isSuccessResponse(response) && response.data != null) {
-            return of(bulkUploadReceiptsSuccess({
-                batchId: response.data.batch_id,
-            }));
+            const batchId = String(response.data.batch_id ?? '').trim();
+            if (batchId === '') {
+                return of(bulkUploadReceiptsFailure({
+                    error: createZeniAPIStatus('Unexpected Error', 'Bulk upload succeeded but batch_id was missing or empty.'),
+                }));
+            }
+            return of(bulkUploadReceiptsSuccess({ batchId }));
         }
         return of(bulkUploadReceiptsFailure({ error: response.status }));
     }), catchError((error) => of(bulkUploadReceiptsFailure({

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

@@ -1,5 +1,6 @@
 import { EMPTY, from, merge, of, timer } from 'rxjs';
 import { catchError, debounceTime, filter, mergeMap, switchMap, takeUntil, } from 'rxjs/operators';
+import { openSnackbar } from '../../../../entity/snackbar/snackbarReducer';
 import { updateTransactions } from '../../../../entity/transaction/transactionReducer';
 import { isSuccessResponse } from '../../../../responsePayload';
 import { extractTransactionPayloadsFromBatchFiles, isBatchDetailsApiStatusCompleted, toBatchDetails, } from '../../payload/missingReceiptsPayload';
@@ -18,8 +19,14 @@ export const pusherBatchStatusCompletionEpic = (actions$) => actions$.pipe(filte
     }),
     requestMissingReceiptsTabNavigation({ tab: 'unmatched' }),
 ])));
+/**
+ * Fallback polling when Pusher is slow or unavailable. Network errors on a poll tick show one
+ * error snackbar per upload session (first failure only) so the user is not spammed every
+ * interval; Pusher or a later successful poll still completes the flow.
+ */
 export const pollBulkUploadBatchStatusEpic = (actions$, _state$, zeniAPI) => actions$.pipe(filter(bulkUploadReceiptsSuccess.match), mergeMap((action) => {
     const { batchId } = action.payload;
+    let pollErrorNotified = false;
     return timer(FALLBACK_FIRST_DELAY_MS, FALLBACK_POLL_INTERVAL_MS).pipe(takeUntil(merge(actions$.pipe(filter(clearBulkUpload.match)), actions$.pipe(filter(pusherBatchStatusUpdate.match), filter((a) => a.payload.batchId === batchId)), actions$.pipe(filter(fetchBulkUploadBatchDetailsSuccess.match), filter((a) => a.payload.batchId === batchId)))), switchMap(() => zeniAPI
         .getJSON(`${zeniAPI.apiEndPoints.communicationAgentMicroServiceBaseUrl}/1.0/batches/${batchId}`)
         .pipe(mergeMap((response) => {
@@ -38,5 +45,15 @@ export const pollBulkUploadBatchStatusEpic = (actions$, _state$, zeniAPI) => act
             }
         }
         return EMPTY;
-    }), catchError(() => of()))));
+    }), catchError(() => {
+        if (!pollErrorNotified) {
+            pollErrorNotified = true;
+            return of(openSnackbar({
+                messageSection: 'receipts_upload',
+                messageText: 'failed',
+                type: 'error',
+            }));
+        }
+        return EMPTY;
+    }))));
 }));

package/lib/esm/view/expenseAutomationView/payload/missingReceiptsPayload.js

@@ -1,6 +1,36 @@
 import { toURL } from '../../../commonPayloadTypes/urlPayload';
 import { getTransactionPayloadAmount, } from '../../../entity/transaction/payloadTypes/transactionPayload';
 // -- Converter functions --
+/**
+ * Maps batch-details API `files[].status` strings to canonical {@link BatchFile} statuses.
+ * The service may send aliases (`unmatched`, `possible_match`) or different separators; the UI
+ * only treats `un_matched` / `no_match` as work for the Unmatched tab.
+ *
+ * **Unknown values:** Any string not matched below is returned as-is (typed as
+ * {@link BatchFile} status). Callers should not assume exhaustiveness—treat unknown statuses in
+ * UI as “other” (e.g. show a generic label or fall back to a safe bucket) until the API contract
+ * adds an explicit mapping.
+ */
+export function normalizeBatchFileStatus(raw) {
+    const s = raw.trim().toLowerCase().replace(/-/g, '_');
+    switch (s) {
+        case 'un_matched':
+        case 'unmatched':
+            return 'un_matched';
+        case 'no_match':
+        case 'nomatch':
+            return 'no_match';
+        case 'matched':
+            return 'matched';
+        case 'failed':
+            return 'failed';
+        case 'possible_match':
+        case 'possiblematch':
+            return 'un_matched';
+        default:
+            return raw;
+    }
+}
 export function toBatchFile(payload, filesEndPoint) {
     return {
         attachmentId: payload.attachment_id,
@@ -16,7 +46,7 @@ export function toBatchFile(payload, filesEndPoint) {
         filename: payload.filename,
         matchSource: payload.match_source,
         matchedTransactionId: payload.matched_transaction?.transaction_id,
-        status: payload.status,
+        status: normalizeBatchFileStatus(payload.status),
     };
 }
 export function extractTransactionPayloadsFromBatchFiles(files) {

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

@@ -1,5 +1,6 @@
 import { createSlice } from '@reduxjs/toolkit';
 import { toMonthYearPeriodId } from '../../../commonStateTypes/timePeriod';
+import { MIN_MANUAL_TRANSACTION_SEARCH_LENGTH, } from '../types/missingReceiptsViewState';
 export const getCompletedTransactionsCacheKey = (periodId, sortKey, sortOrder, subTab) => `completed-${subTab}-${sortKey}-${sortOrder === 'ascending' ? 'asc' : 'desc'}-${periodId}`;
 export const initialBulkUploadState = {
     batchDetailsById: {},
@@ -386,13 +387,20 @@ const expenseAutomationMissingReceiptsView = createSlice({
                 const isLoadMore = pageToken != null;
                 const trimmed = query.trim();
                 /**
-                 * Cleared or too-short query: no API (see epic). Reset fully (no In-Progress) so loading
-                 * UI never shows; bump key so inputs remount.
+                 * Cleared or too-short query: no API (see epic). Reset fully (no In-Progress).
+                 * Only bump `manualSearchUiResetKey` when clearing a real search — not when the UI
+                 * dispatches `""` on every keystroke while length is below MIN (web-components always sends
+                 * `onSearch("")` in that case), or ReceiptCard remounts and the input loses focus.
                  */
                 if (!isLoadMore &&
-                    trimmed.length < 2) {
+                    trimmed.length < MIN_MANUAL_TRANSACTION_SEARCH_LENGTH) {
+                    const hadActiveSearch = draft.bulkUpload.manualSearch.searchQuery.trim().length >=
+                        MIN_MANUAL_TRANSACTION_SEARCH_LENGTH ||
+                        draft.bulkUpload.manualSearch.results.length > 0;
                     draft.bulkUpload.manualSearch = initialBulkUploadState.manualSearch;
-                    draft.bulkUpload.manualSearchUiResetKey += 1;
+                    if (hadActiveSearch) {
+                        draft.bulkUpload.manualSearchUiResetKey += 1;
+                    }
                     return;
                 }
                 draft.bulkUpload.manualSearch.searchQuery = query;

package/lib/esm/view/expenseAutomationView/types/missingReceiptsViewState.js

@@ -7,3 +7,9 @@ const MISSING_RECEIPTS_SORT_KEYS = [
     'amount',
 ];
 export const toMissingReceiptsSortKey = (v) => stringToUnion(v, MISSING_RECEIPTS_SORT_KEYS);
+// -- Bulk Receipt Upload Types --
+/**
+ * Minimum trimmed query length before manual-match search calls the expense-automation API.
+ * Kept in sync with `ManualSearchInput` in web-components (fires search only when length ≥ this).
+ */
+export const MIN_MANUAL_TRANSACTION_SEARCH_LENGTH = 2;