@zeniai/client-epic-state 5.2.7-betaAK2 → 5.2.7-betaRD2

package/lib/esm/view/expenseAutomationView/selectors/jeNewScheduleViewSelector.js

@@ -0,0 +1,106 @@
+import { getAccountIdsForLabels, getAccountIdsForTypes, } from '../../../entity/account/accountSelector';
+import { getScheduleTransactionById } from '../../../entity/transaction/transactionSelector';
+import { getAccountIdsForLabel, getAccountList, getNestedAccountListHierarchy, } from '../../accountList/accountListSelector';
+import { getClassList, getNestedClassListHierarchy, } from '../../classList/classListSelector';
+import { getAccountsTypesForScheduleDetails } from '../../scheduleView/scheduleDetailView/scheduleDetailLocalDataHelper';
+import { filterNestedAccountHierarchyById } from '../../scheduleView/scheduleDetailView/scheduleDetailSelector';
+import { getAccountLabelForScheduleType, getAccountsTypesForScheduleEpics, } from '../../scheduleView/scheduleListView/scheduleListHelper';
+/**
+ * The Base Transaction search is shared across schedule types (one search
+ * box, not one per prepaid_expenses/fixed_assets) — split out from
+ * getNewScheduleView so a consumer can read it without supplying an
+ * otherwise-irrelevant scheduleType.
+ */
+export const getNewScheduleSearchTransactionsView = (state) => {
+    const { searchTransactions } = state.expenseAutomationJESchedulesViewState.newSchedule;
+    const searchTransactionsResults = searchTransactions.searchResults
+        .map((key) => getScheduleTransactionById(state.transactionState, key))
+        .filter((transaction) => transaction != null);
+    return {
+        searchTransactionsResults,
+        searchTransactionsFetchState: {
+            fetchState: searchTransactions.fetchState,
+            error: searchTransactions.error,
+        },
+        searchTransactionsString: searchTransactions.searchString,
+    };
+};
+const ACCRUED_EXPENSE_ACCOUNT_LABELS = ['accrued_expenses'];
+/**
+ * Account + class options and request status shared by both accrual creation forms — the one-time
+ * accrual and the recurring accrued schedule.
+ *
+ * getAccountsTypesForScheduleDetails('accrued_expenses') deliberately includes
+ * other_current_liabilities, because the accrued liability accounts live there —
+ * so the debit list has to subtract them back out, and the credit list keeps only
+ * them. Mirrors getScheduleAccruedDetailView, which is the form this one is
+ * modelled on.
+ */
+export const getNewAccrualAccountsView = (state) => {
+    const { accountState, accountListState, classState, classListState } = state;
+    const { newSchedule } = state.expenseAutomationJESchedulesViewState;
+    const accountType = getAccountsTypesForScheduleEpics();
+    const accrualAccountIds = getAccountIdsForTypes(accountState, getAccountsTypesForScheduleDetails('accrued_expenses'));
+    const accruedCategoryIds = getAccountIdsForLabels(accountState, ACCRUED_EXPENSE_ACCOUNT_LABELS);
+    const allAccountList = getAccountList(accountState, accountListState, accountType.accountListKey);
+    const nestedAccountHierarchy = getNestedAccountListHierarchy(accountListState, accountType.accountListKey);
+    const debitAccounts = allAccountList.accounts.filter((account) => accrualAccountIds.includes(account.accountId) &&
+        !accruedCategoryIds.includes(account.accountId));
+    const creditAccounts = allAccountList.accounts.filter((account) => accruedCategoryIds.includes(account.accountId));
+    const allClassList = getClassList(classState, classListState);
+    return {
+        accountListNestedAccountHierarchy: filterNestedAccountHierarchyById(nestedAccountHierarchy, debitAccounts.map((account) => account.accountId)),
+        accruedAccountListNestedAccountHierarchy: filterNestedAccountHierarchyById(nestedAccountHierarchy, creditAccounts.map((account) => account.accountId)),
+        allAccountList: debitAccounts,
+        allAccruedExpenseList: creditAccounts,
+        allClassList: allClassList.classes,
+        classListNestedAccountHierarchy: getNestedClassListHierarchy(classListState),
+        createStatus: newSchedule.createOneTimeAccrualStatus,
+        // The recurring accrued schedule saves through the accrued detail epics that the Vendor tab
+        // already uses, so its status is that slice's updateStatus rather than anything new here.
+        accruedScheduleSaveStatus: state.scheduleAccruedDetailState.updateStatus,
+    };
+};
+export const getNewScheduleView = (state, scheduleType) => {
+    const { accountState, accountListState, classState, classListState } = state;
+    const { newSchedule } = state.expenseAutomationJESchedulesViewState;
+    const searchTransactionsView = getNewScheduleSearchTransactionsView(state);
+    const accountType = getAccountsTypesForScheduleEpics();
+    const newScheduleAccountTypes = getAccountsTypesForScheduleDetails(scheduleType);
+    const newScheduleAccountIds = getAccountIdsForTypes(accountState, newScheduleAccountTypes);
+    const allAccountList = getAccountList(accountState, accountListState, accountType.accountListKey);
+    const filteredAccounts = allAccountList.accounts.filter((account) => newScheduleAccountIds.includes(account.accountId));
+    const accountListNestedAccountHierarchy = filterNestedAccountHierarchyById(getNestedAccountListHierarchy(accountListState, accountType.accountListKey), filteredAccounts.map((account) => account.accountId));
+    const allClassList = getClassList(classState, classListState);
+    const classListNestedAccountHierarchy = getNestedClassListHierarchy(classListState);
+    let allDepreciationAccountList;
+    let allDepreciationAccountListNestedAccountHierarchy;
+    if (scheduleType === 'fixed_assets') {
+        const depreciationCategoryIds = getAccountIdsForTypes(accountState, [
+            'fixed_assets',
+            'other_assets',
+        ]);
+        const accountLabel = getAccountLabelForScheduleType('fixed_assets');
+        const fixedAssetLabelAccountIds = accountLabel != null
+            ? getAccountIdsForLabel(accountListState, accountType.accountListKey, accountLabel)
+            : [];
+        allDepreciationAccountList = allAccountList.accounts.filter((account) => depreciationCategoryIds.includes(account.accountId) &&
+            !fixedAssetLabelAccountIds.includes(account.accountId));
+        allDepreciationAccountListNestedAccountHierarchy =
+            filterNestedAccountHierarchyById(getNestedAccountListHierarchy(accountListState, accountType.accountListKey), allDepreciationAccountList.map((account) => account.accountId));
+    }
+    return {
+        scheduleType,
+        accountListNestedAccountHierarchy,
+        allAccountList: filteredAccounts,
+        allClassList: allClassList.classes,
+        allDepreciationAccountList,
+        allDepreciationAccountListNestedAccountHierarchy,
+        classListNestedAccountHierarchy,
+        newScheduleLocalData: newSchedule.newScheduleState[scheduleType],
+        saveStatus: newSchedule.saveStatus,
+        ...searchTransactionsView,
+        fetchState: newSchedule.fetchState,
+        error: newSchedule.error,
+    };
+};

package/lib/esm/view/expenseAutomationView/selectors/jeSchedulesViewSelector.js

@@ -3,7 +3,9 @@ import orderBy from 'lodash/orderBy';
 import { reduceAnyFetchState } from '../../../commonStateTypes/reduceFetchState';
 import { LAST_SCHEDULE_DAY_OF_MONTH, toMonthYearPeriodId, } from '../../../commonStateTypes/timePeriod';
 import { getAccountIdsForTypes } from '../../../entity/account/accountSelector';
-import { getJEScheduledTransactionByJEScheduleTransactionKey, } from '../../../entity/jeSchedules/jeSchedulesSelector';
+import { generateJEOneTimeAccrualKey } from '../../../entity/jeSchedules/jeScheduleHelper';
+import { getJEAccruedScheduleByJEScheduleKey, getJEOneTimeAccrualByKey, getJEScheduledTransactionByJEScheduleTransactionKey, } from '../../../entity/jeSchedules/jeSchedulesSelector';
+import { ALL_SCHEDULES_TYPES, } from '../../../entity/jeSchedules/jeSchedulesTypes';
 import { getCurrentTenant } from '../../../entity/tenant/tenantSelector';
 import { getAccountIdsForLabel, getAccountList, getNestedAccountListHierarchy, getUncategorizedAccounts, } from '../../accountList/accountListSelector';
 import { getClassList, getNestedClassListHierarchy, } from '../../classList/classListSelector';
@@ -11,8 +13,126 @@ import { getAccountsTypesForScheduleDetails } from '../../scheduleView/scheduleD
 import { filterNestedAccountHierarchyById } from '../../scheduleView/scheduleDetailView/scheduleDetailSelector';
 import { getAccountLabelForScheduleType, getAccountsTypesForScheduleList, } from '../../scheduleView/scheduleListView/scheduleListHelper';
 import { getAllSteps } from './getAllStepsHelper';
+import { ALL_JE_PAGE_TABS, getJESchedulePeriodTabKey, } from '../types/jeSchedulesViewState';
 import { getJEAccountSettingsView } from './jeAccountSettingsViewSelector';
+import { getNewScheduleView } from './jeNewScheduleViewSelector';
+// Matches the AI-automation user addresses webc's schedule list row already
+// checks (JEScheduleReadOnlyListRow/JEScheduleDraftListRow "Method" column,
+// strings.ts aiBotEmail/botEmail) to determine whether a schedule was
+// created by the AI accountant vs. a human.
+const AI_BOT_EMAIL = 'ai@zeni.ai';
+const BOT_EMAIL = 'bot@zeni.ai';
+// A one-time accrual has no jeScheduleType of its own — it is not a JE schedule. It is still an
+// accrued expense, so it counts and filters under that type on the page's type tabs.
+const ONE_TIME_ACCRUAL_SCHEDULE_TYPE = 'accrued_expenses';
+// Which of the two list buckets an accrual belongs in. `draft` still needs a human to post it;
+// everything else is already on the books or in a transient claim.
+const ONE_TIME_ACCRUAL_REVIEW_STATUSES = [
+    'draft',
+];
+/**
+ * `posting_blocked` is written by the posting Cron when a JE fails to post. It is an internal
+ * failure state with no human action attached, so it is dropped here rather than hidden at render
+ * time — filtering in the view would leave the group counts and KPI totals counting rows nobody
+ * can see. The expense-automation endpoint applies no status filter on the close-period path, so
+ * these rows do arrive.
+ */
+export function isRenderableOneTimeAccrual(accrual) {
+    return accrual.accrualStatus !== 'posting_blocked';
+}
+export function getJEScheduleRowType(schedule) {
+    return isOneTimeAccrualRow(schedule)
+        ? ONE_TIME_ACCRUAL_SCHEDULE_TYPE
+        : schedule.jeScheduleType;
+}
+export function isPendingReviewRow(schedule) {
+    return isOneTimeAccrualRow(schedule)
+        ? ONE_TIME_ACCRUAL_REVIEW_STATUSES.includes(schedule.accrualStatus)
+        : schedule.status.code === 'draft';
+}
+// Belongs on the Existing Schedules tab. For an accrual the endpoint has already filtered to
+// posted, so this only has to agree with that rather than re-derive it.
+export function isExistingTabRow(schedule) {
+    return isOneTimeAccrualRow(schedule)
+        ? schedule.accrualStatus === 'posted'
+        : schedule.status.code === 'ongoing';
+}
+// Exported (rather than kept as a closure inside the selector) so it can be
+// unit-tested directly — confidenceScore is a separate, more narrowly
+// populated field that can be null on a schedule the bot user created,
+// which previously caused these schedules to display as "AI Accountant" in
+// the list but count under "Manual" in the KPI summary.
+export const isAiAccountantSchedule = (schedule) => {
+    // A one-time accrual records who made it as `origin`, which is authoritative — no email
+    // sniffing needed, and the JE bot is often absent from the user entity entirely.
+    if (isOneTimeAccrualRow(schedule)) {
+        return schedule.origin === 'agent';
+    }
+    return (schedule.createdByUser?.email === AI_BOT_EMAIL ||
+        schedule.createdByUser?.email === BOT_EMAIL);
+};
+// Accrued schedules are structurally distinct — no baseTransaction, carrying recurringAmount and
+// memo in its place — so anything reading baseTransaction has to branch on this first.
+//
+// Discriminates on jeAccruedScheduleKey, which only JEAccruedSchedule carries. jeScheduleType
+// cannot be used: it is a data field present on BOTH row shapes, so an accrued-typed row that
+// arrived through the prepaid/fixed DTO (failed_je_postings, legacy fetches) would be
+// misclassified and then crash on recurringAmount / jeAccruedScheduleKey.
+export function isAccruedScheduleRow(schedule) {
+    return 'jeAccruedScheduleKey' in schedule;
+}
+// Same structural approach as isAccruedScheduleRow, and for the same reason: jeOneTimeAccrualKey
+// exists only on this shape. Note the two guards are NOT interchangeable — a one-time accrual is
+// not an accrued *schedule*, and routing it into the accrued branch would read recurringAmount,
+// render postedCount/lifecycleTotal as a lifecycle and startDate–endDate as a period, none of
+// which it has.
+export function isOneTimeAccrualRow(schedule) {
+    return 'jeOneTimeAccrualKey' in schedule;
+}
+function getScheduleAmount(schedule) {
+    if (isOneTimeAccrualRow(schedule)) {
+        return schedule.estimatedAmount.amount;
+    }
+    if (isAccruedScheduleRow(schedule)) {
+        return schedule.recurringAmount.amount;
+    }
+    return schedule.baseTransaction.amount.amount;
+}
+// A one-time accrual shares almost no sortable field with a schedule, so it gets its own accessor
+// rather than a guard on each of the dozen cases below. Keys with no meaning for a single-month
+// accrual -- amortizationPeriod, remainingMonths, runningBalance, jePostingDate, scheduleCategory --
+// deliberately fall through to undefined, which sorts them together at one end.
+function getOneTimeAccrualSortValue(accrual, sortKey) {
+    switch (sortKey) {
+        case 'vendor':
+            return accrual.vendorName.toLowerCase();
+        case 'category':
+            return accrual.jeDebitAccountName?.toLowerCase();
+        case 'class':
+            return accrual.jeDebitAccountingClassName?.toLowerCase();
+        case 'depCategory':
+            return accrual.jeCreditAccountName?.toLowerCase();
+        case 'type':
+            return ONE_TIME_ACCRUAL_SCHEDULE_TYPE;
+        case 'totalAmount':
+            return accrual.estimatedAmount.amount;
+        case 'transactionDate':
+            return accrual.postingDate?.valueOf();
+        case 'memo':
+            return accrual.memo.toLowerCase();
+        case 'startMonth':
+            return accrual.serviceMonth;
+        default:
+            return undefined;
+    }
+}
 function getJEScheduleSortAccessor(sortKey) {
+    const scheduleAccessor = getScheduleRowSortAccessor(sortKey);
+    return (schedule) => isOneTimeAccrualRow(schedule)
+        ? getOneTimeAccrualSortValue(schedule, sortKey)
+        : scheduleAccessor(schedule);
+}
+function getScheduleRowSortAccessor(sortKey) {
     switch (sortKey) {
         case 'vendor':
             return (s) => s.vendor.name?.toLowerCase();
@@ -23,7 +143,9 @@ function getJEScheduleSortAccessor(sortKey) {
         case 'depCategory':
             return (s) => s.jeCredit.account?.accountName?.toLowerCase();
         case 'scheduleCategory':
-            return (s) => s.baseTransaction.account?.accountName?.toLowerCase();
+            return (s) => isAccruedScheduleRow(s)
+                ? undefined
+                : s.baseTransaction.account?.accountName?.toLowerCase();
         case 'startMonth':
             return (s) => s.startDate?.valueOf();
         case 'type':
@@ -51,11 +173,13 @@ function getJEScheduleSortAccessor(sortKey) {
                 return s.balanceAsOfToday?.amount ?? null;
             };
         case 'totalAmount':
-            return (s) => s.baseTransaction.amount.amount;
+            return (s) => getScheduleAmount(s);
         case 'transactionDate':
-            return (s) => s.baseTransaction.date.valueOf();
+            return (s) => s.createTime?.valueOf();
         case 'memo':
-            return (s) => s.baseTransaction.memo?.toLowerCase();
+            return (s) => isAccruedScheduleRow(s)
+                ? s.memo.toLowerCase()
+                : s.baseTransaction.memo?.toLowerCase();
         default:
             return (s) => s.startDate?.valueOf();
     }
@@ -67,9 +191,15 @@ function sortJEScheduledTransactions(transactions, sortKey, sortOrder) {
 export function getExpenseAutomationJESchedulesView(state) {
     const jeScheduledTransaction = [];
     const failedJeScheduledTransaction = [];
+    const accruedSchedules = [];
+    const oneTimeAccruals = [];
     const { accountState, accountListState, classState, classListState, expenseAutomationJESchedulesViewState, expenseAutomationViewState, monthEndCloseChecksState, monthEndCloseChecksViewState, } = state;
-    const { jeScheduleTransactionKeysByPeriod, failedJeScheduleTransactionKeysByPeriod, jeScheduleLocalDataById, postStatusById, ignoreStatusById, uiState, error, fetchState, refreshStatus, } = expenseAutomationJESchedulesViewState;
+    const { jeScheduleTransactionKeysByPeriodAndTab, jeScheduleAccruedKeysByPeriodAndTab, jeScheduleOneTimeAccrualKeysByPeriodAndTab, failedJeScheduleTransactionKeysByPeriodAndTab, jeScheduleLocalDataById, postStatusById, ignoreStatusById, revertStatusById, oneTimeAccrualActionStatusById, uiState, error, fetchState, refreshStatus, newSchedule, } = expenseAutomationJESchedulesViewState;
     const accountSettingsView = getJEAccountSettingsView(state);
+    const newScheduleView = newSchedule.selectedDetailScheduleType === 'fixed_assets' ||
+        newSchedule.selectedDetailScheduleType === 'prepaid_expenses'
+        ? getNewScheduleView(state, newSchedule.selectedDetailScheduleType)
+        : undefined;
     const currentTenant = getCurrentTenant(state);
     const { selectedPeriodByTenantId } = expenseAutomationViewState;
     const selectedPeriod = selectedPeriodByTenantId[currentTenant.tenantId];
@@ -97,27 +227,48 @@ export function getExpenseAutomationJESchedulesView(state) {
     const filteredDepAccountListNestedAccountHierarchy = filterNestedAccountHierarchyById(depAccountListNestedAccountHierarchy, filteredDepAccounts.map((account) => account.accountId));
     const classListNestedAccountHierarchy = getNestedClassListHierarchy(classListState);
     const monthYearPeriodId = selectedPeriod != null ? toMonthYearPeriodId(selectedPeriod) : null;
-    const jeScheduleTransactionKeysForSelectedPeriod = monthYearPeriodId != null
-        ? jeScheduleTransactionKeysByPeriod[monthYearPeriodId]
-        : null;
-    const failedJeScheduleTransactionKeysForSelectedPeriod = monthYearPeriodId != null
-        ? failedJeScheduleTransactionKeysByPeriod[monthYearPeriodId]
+    const selectedPeriodTabKey = monthYearPeriodId != null
+        ? getJESchedulePeriodTabKey(uiState.selectedPageTab, monthYearPeriodId)
         : null;
-    const jeScheduleTransactionKeys = jeScheduleTransactionKeysForSelectedPeriod ?? [];
-    const failedJeScheduleTransactionKeys = failedJeScheduleTransactionKeysForSelectedPeriod ?? [];
+    const jeScheduleTransactionKeysForSelectedTab = selectedPeriodTabKey != null
+        ? jeScheduleTransactionKeysByPeriodAndTab[selectedPeriodTabKey]
+        : undefined;
+    const failedJeScheduleTransactionKeysForSelectedTab = selectedPeriodTabKey != null
+        ? failedJeScheduleTransactionKeysByPeriodAndTab[selectedPeriodTabKey]
+        : undefined;
+    // Distinguishes "this tab has never been fetched for this period" from "fetched and genuinely
+    // empty". Without it the page cannot tell a first visit from a real empty state, and renders an
+    // empty table where it should render a skeleton.
+    const hasLoadedForSelectedTab = jeScheduleTransactionKeysForSelectedTab != null;
+    const jeScheduleTransactionKeys = jeScheduleTransactionKeysForSelectedTab ?? [];
+    const failedJeScheduleTransactionKeys = failedJeScheduleTransactionKeysForSelectedTab ?? [];
+    const jeScheduleAccruedKeys = (selectedPeriodTabKey != null
+        ? jeScheduleAccruedKeysByPeriodAndTab[selectedPeriodTabKey]
+        : undefined) ?? [];
+    const jeScheduleOneTimeAccrualKeys = (selectedPeriodTabKey != null
+        ? jeScheduleOneTimeAccrualKeysByPeriodAndTab[selectedPeriodTabKey]
+        : undefined) ?? [];
+    jeScheduleOneTimeAccrualKeys.forEach((jeOneTimeAccrualKey) => {
+        const accrual = getJEOneTimeAccrualByKey(jeOneTimeAccrualKey, state.jeSchedulesState);
+        if (accrual != null && isRenderableOneTimeAccrual(accrual)) {
+            oneTimeAccruals.push(accrual);
+        }
+    });
     jeScheduleTransactionKeys.forEach((jeScheduleKey) => {
         const jeScheduleDetails = getJEScheduledTransactionByJEScheduleTransactionKey(jeScheduleKey, state);
         jeScheduledTransaction.push(jeScheduleDetails);
     });
+    jeScheduleAccruedKeys.forEach((jeAccruedScheduleKey) => {
+        const accruedScheduleDetails = getJEAccruedScheduleByJEScheduleKey(jeAccruedScheduleKey, state);
+        if (accruedScheduleDetails != null) {
+            accruedSchedules.push(accruedScheduleDetails);
+        }
+    });
     failedJeScheduleTransactionKeys.forEach((jeScheduleKey) => {
         const jeScheduleDetails = getJEScheduledTransactionByJEScheduleTransactionKey(jeScheduleKey, state);
         failedJeScheduledTransaction.push(jeScheduleDetails);
     });
-    const { sortKey, sortOrder } = uiState;
-    const draftSchedules = sortJEScheduledTransactions(jeScheduledTransaction.filter((schedule) => schedule.status.code === 'draft'), sortKey, sortOrder);
-    const ongoingSchedules = sortJEScheduledTransactions(jeScheduledTransaction.filter((schedule) => schedule.status.code === 'ongoing'), sortKey, sortOrder);
-    const completedSchedules = sortJEScheduledTransactions(jeScheduledTransaction.filter((schedule) => schedule.status.code === 'completed' ||
-        schedule.status.code === 'marked_as_completed'), sortKey, sortOrder);
+    const { sortKey, sortOrder, selectedJETypeFilter } = uiState;
     const unsortedResolveSchedules = [];
     failedJeScheduledTransaction.forEach((schedule) => {
         schedule.scheduledJournalEntry.forEach((journalEntry) => {
@@ -129,10 +280,87 @@ export function getExpenseAutomationJESchedulesView(state) {
     });
     const resolveSchedules = sortJEScheduledTransactions(unsortedResolveSchedules, sortKey, sortOrder);
     const validResolveForPending = unsortedResolveSchedules.filter((r) => jeScheduleLocalDataById[r.scheduledJournalEntry.scheduledJournalEntryID] != null);
+    const matchesTypeFilter = (schedule) => selectedJETypeFilter == null ||
+        getJEScheduleRowType(schedule) === selectedJETypeFilter;
+    const isCreatedStatus = (schedule) => {
+        // An accrual is "created" once it is not review work: posted, reversing, reversed, or in a
+        // transient posting claim.
+        if (isOneTimeAccrualRow(schedule)) {
+            return !isPendingReviewRow(schedule);
+        }
+        return (schedule.status.code === 'ongoing' ||
+            schedule.status.code === 'completed' ||
+            schedule.status.code === 'marked_as_completed');
+    };
+    const allScheduleRows = [
+        ...jeScheduledTransaction,
+        ...accruedSchedules,
+        ...oneTimeAccruals,
+    ];
     const pendingReviewSchedules = sortJEScheduledTransactions([
-        ...jeScheduledTransaction.filter((schedule) => schedule.status.code === 'draft'),
+        ...allScheduleRows.filter((schedule) => isPendingReviewRow(schedule) && matchesTypeFilter(schedule)),
         ...validResolveForPending,
     ], sortKey, sortOrder);
+    const createdSchedules = sortJEScheduledTransactions(allScheduleRows.filter((schedule) => isCreatedStatus(schedule) && matchesTypeFilter(schedule)), sortKey, sortOrder);
+    // KPI summary — derived from unfiltered schedules so counts are always accurate
+    const allSchedules = allScheduleRows;
+    const sumAmount = (schedules) => schedules.reduce((total, schedule) => total + (getScheduleAmount(schedule) ?? 0), 0);
+    const pendingReviewTotal = sumAmount(pendingReviewSchedules);
+    const createdTotal = sumAmount(createdSchedules);
+    const aiAccountantSchedules = allSchedules.filter(isAiAccountantSchedule);
+    const manualSchedules = allSchedules.filter((schedule) => !isAiAccountantSchedule(schedule));
+    const pendingUnfiltered = allSchedules.filter((schedule) => isPendingReviewRow(schedule));
+    const kpiSummary = {
+        aiAccountant: {
+            count: aiAccountantSchedules.length,
+            amount: sumAmount(aiAccountantSchedules),
+        },
+        manual: {
+            count: manualSchedules.length,
+            amount: sumAmount(manualSchedules),
+        },
+        needsReview: {
+            count: pendingUnfiltered.length,
+            amount: sumAmount(pendingUnfiltered),
+        },
+        total: {
+            count: allSchedules.length,
+            amount: sumAmount(allSchedules),
+        },
+    };
+    // Each tab's badge counts from that tab's own bucket. A tab that has not been fetched yet
+    // reports undefined rather than 0, so the UI can hide the badge instead of asserting "none".
+    const scheduleCountsByTab = {};
+    ALL_JE_PAGE_TABS.forEach((pageTab) => {
+        const periodTabKey = monthYearPeriodId != null
+            ? getJESchedulePeriodTabKey(pageTab, monthYearPeriodId)
+            : null;
+        const transactionKeys = periodTabKey != null
+            ? jeScheduleTransactionKeysByPeriodAndTab[periodTabKey]
+            : undefined;
+        if (transactionKeys == null) {
+            return;
+        }
+        const accruedKeys = (periodTabKey != null
+            ? jeScheduleAccruedKeysByPeriodAndTab[periodTabKey]
+            : undefined) ?? [];
+        const oneTimeAccrualKeys = (periodTabKey != null
+            ? jeScheduleOneTimeAccrualKeysByPeriodAndTab[periodTabKey]
+            : undefined) ?? [];
+        const rowsForTab = [
+            ...transactionKeys.map((transactionKey) => getJEScheduledTransactionByJEScheduleTransactionKey(transactionKey, state)),
+            ...accruedKeys
+                .map((accruedKey) => getJEAccruedScheduleByJEScheduleKey(accruedKey, state))
+                .filter((accruedRow) => accruedRow != null),
+            ...oneTimeAccrualKeys
+                .map((accrualKey) => getJEOneTimeAccrualByKey(accrualKey, state.jeSchedulesState))
+                .filter((accrualRow) => accrualRow != null && isRenderableOneTimeAccrual(accrualRow)),
+        ];
+        scheduleCountsByTab[pageTab] =
+            pageTab === 'existing_schedules'
+                ? rowsForTab.filter((row) => isExistingTabRow(row)).length
+                : rowsForTab.filter((row) => isPendingReviewRow(row) || isCreatedStatus(row)).length;
+    });
     const allSteps = monthYearPeriodId != null
         ? getAllSteps(monthEndCloseChecksState, monthYearPeriodId, currentTenant.tenantId)
         : [];
@@ -145,6 +373,20 @@ export function getExpenseAutomationJESchedulesView(state) {
         { fetchState, error },
         monthEndFetchState,
     ]);
+    const countByType = (schedules) => {
+        const counts = ALL_SCHEDULES_TYPES.reduce((acc, type) => ({ ...acc, [type]: 0 }), {});
+        schedules.forEach((schedule) => {
+            const scheduleType = getJEScheduleRowType(schedule);
+            if (scheduleType != null) {
+                counts[scheduleType] += 1;
+            }
+        });
+        return counts;
+    };
+    const jeScheduleTypeCounts = {
+        active: countByType(allSchedules.filter((schedule) => isPendingReviewRow(schedule) || isCreatedStatus(schedule))),
+        ongoing: countByType(allSchedules.filter((schedule) => isExistingTabRow(schedule))),
+    };
     return {
         uncategorizedAccounts,
         allDepreciationAccountList: filteredDepAccounts,
@@ -153,14 +395,21 @@ export function getExpenseAutomationJESchedulesView(state) {
         allAccountList: filteredAccounts,
         accountListNestedAccountHierarchy,
         allClassList: allClasses,
-        completedSchedules,
-        draftSchedules,
-        ongoingSchedules,
+        createdSchedules,
+        createdTotal,
+        hasLoadedForSelectedTab,
+        jeScheduleTypeCounts,
+        scheduleCountsByTab,
+        kpiSummary,
         pendingReviewSchedules,
+        pendingReviewTotal,
         resolveSchedules: resolveSchedules,
         accountSettingsView,
+        newScheduleView,
         postStatusById: postStatusById,
         ignoreStatusById: ignoreStatusById,
+        revertStatusById: revertStatusById,
+        oneTimeAccrualActionStatusById: oneTimeAccrualActionStatusById,
         fetchState: reducedFetchStatus.fetchState,
         jeScheduleLocalDataById: jeScheduleLocalDataById,
         uiState: uiState,
@@ -169,3 +418,26 @@ export function getExpenseAutomationJESchedulesView(state) {
         completionStatus,
     };
 }
+/**
+ * One accrual plus the status of its promote / ignore request, for the detail screen.
+ *
+ * `accrual` is undefined on a cold load — a refresh or a shared link, where the list has never run
+ * — so the screen dispatches `fetchOneTimeAccrual` and waits on `detailFetchState`. The two states
+ * are separate on purpose: `actionFetchState` drives the row spinner and the post-revert navigation,
+ * and a page load must not look like an action that just finished.
+ */
+export function getExpenseAutomationJEOneTimeAccrualDetailView(state, accrualId) {
+    return {
+        accrual: getJEOneTimeAccrualByKey(generateJEOneTimeAccrualKey(accrualId), state.jeSchedulesState),
+        actionFetchState: state.expenseAutomationJESchedulesViewState
+            .oneTimeAccrualActionStatusById[accrualId] ?? {
+            fetchState: 'Not-Started',
+            error: undefined,
+        },
+        detailFetchState: state.expenseAutomationJESchedulesViewState
+            .oneTimeAccrualFetchStatusById[accrualId] ?? {
+            fetchState: 'Not-Started',
+            error: undefined,
+        },
+    };
+}

package/lib/esm/view/expenseAutomationView/selectors/transactionCategorizationSelector.js

@@ -86,7 +86,7 @@ export const toTransactionView = (transaction, transactionLocalData) => {
 export { getLastTransferEntryReplacement } from '../../createTransferEntry/createTransferEntrySelector';
 export function getExpenseAutomationTransactionView(state) {
     const { accountState, classState, classListState, accountListState, createTransferEntryState, expenseAutomationTransactionsViewState, expenseAutomationViewState, projectState, projectListState, transactionState, monthEndCloseChecksState, } = state;
-    const { selectedTransactionCategorizationCompletedSubTab, selectedTransactionCategorizationTab, } = expenseAutomationTransactionsViewState;
+    const { selectedTransactionCategorizationCompletedSubTab, selectedTransactionCategorizationTab, selectedTransactionGroup, } = expenseAutomationTransactionsViewState;
     const { uiState, refreshStatus, saveStatus, markAsNotMiscategorizedStatus, transactionIdsBySelectedPeriod, transactionReviewLocalDataById, selectedCheckBoxTransactionIds, transactionIdsWithUnsavedData, uploadReceiptStatusById, selectedTransactionId, selectedTransactionLineId, filters, } = expenseAutomationTransactionsViewState.transactionCategorizationView[selectedTransactionCategorizationTab];
     const uncategorizedIncomeExpense = getUncategorizedAccounts(accountState, accountListState, 'accountList');
     const accountList = getAccountList(accountState, accountListState, 'accountList');
@@ -240,5 +240,6 @@ export function getExpenseAutomationTransactionView(state) {
         creditCardTransferAccountsList,
         transferAccounts: transferAccountsList,
         selectedTransactionCategorizationCompletedSubTab,
+        selectedTransactionGroup,
     };
 }

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

@@ -16,9 +16,8 @@ const JE_SCHEDULE_SORT_KEYS = [
     'memo',
 ];
 export const toJEScheduleSortKey = (v) => stringToUnion(v, JE_SCHEDULE_SORT_KEYS);
-const JE_SCHEDULE_MAIN_TABS = [
-    'pending_review',
-    'ongoing',
-    'completed',
-];
-export const toJEScheduleMainTab = (v) => stringToUnion(v, JE_SCHEDULE_MAIN_TABS);
+const JE_PAGE_TABS = ['schedules', 'existing_schedules'];
+export const toJEPageTab = (v) => stringToUnion(v, JE_PAGE_TABS);
+export const ALL_JE_PAGE_TABS = JE_PAGE_TABS;
+export const getJESchedulePeriodTabKey = (selectedPageTab, monthYearPeriodId) => `${selectedPageTab}_${monthYearPeriodId}`;
+export const MIN_NEW_SCHEDULE_SEARCH_LENGTH = 2;

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

@@ -0,0 +1,21 @@
+import { stringToUnion } from '../../../commonStateTypes/stringToUnion';
+/**
+ * Statement-level grouping for the Transaction Categorization listing, forwarded
+ * verbatim to the listing API as `transaction_group`. When revenue automation is
+ * enabled for the tenant the backend scopes the whole view by the line
+ * account_type: `'expense'` -> expense accounts, `'income'` -> income accounts,
+ * `'all'` -> both. When off, every value falls back to the legacy expense scope
+ * on the backend, so the group is only sent when the gate is on.
+ *
+ * The wire value for expenses is the singular `'expense'` (matches the backend
+ * `TransactionGroup` enum) even though the picker labels it "Expenses".
+ * Balance-sheet groups (assets, liability) are reserved for later and are not
+ * part of this union yet.
+ */
+export const TRANSACTION_GROUPS = ['all', 'income', 'expense'];
+export const toTransactionGroup = (v) => stringToUnion(v != null ? v.trim().toLowerCase() : '', TRANSACTION_GROUPS);
+/**
+ * Default group for the listing — matches the picker's default selection ("All").
+ * Single source of truth for initial slice state and any fall-back call site.
+ */
+export const DEFAULT_TRANSACTION_GROUP = 'all';

package/lib/esm/view/invoicing/editInvoicingCatalogItemDetailView/editInvoicingCatalogItemDetailViewSelector.js

@@ -1,7 +1,7 @@
 import { createSelector } from '@reduxjs/toolkit';
 import { getInvoicingPlanById, getInvoicingPlansByIds, } from '../../../entity/invoicing/plan/planSelector';
 import { getInvoicingConfigView, invoicingFieldHelpForGroup, } from '../invoicingConfigView/invoicingConfigViewSelector';
-import { INVOICING_INCOME_ACCOUNT_TYPES, getInvoicingQboView, invoicingAccountHierarchyForTypes, isInvoicingAccountingPickerDataLoading, isInvoicingQboConnected, } from '../invoicingQboView/invoicingQboViewSelector';
+import { INVOICING_INCOME_ACCOUNT_TYPES, getInvoicingQboView, invoicingAccountHierarchyForTypes, isInvoicingQboConnected, } from '../invoicingQboView/invoicingQboViewSelector';
 import { INVOICING_CATALOG_APPLICABILITY_VALUES, INVOICING_CATALOG_PRICING_MODEL_VALUES, INVOICING_CATALOG_PRORATION_VALUES, INVOICING_CATALOG_TIER_TYPE_VALUES, entityToInvoicingCatalogItemFormLocalData, } from './invoicingCatalogItemFormConfig';
 export const getEditInvoicingCatalogItemDetailViewState = (state) => ({
     fetchState: state.editInvoicingCatalogItemDetailViewState.fetchState,
@@ -53,7 +53,6 @@ export const getInvoicingCatalogItemFormView = (state, id) => {
     const { billingPeriodUnitOptions, catalogItemTypeOptions, catalogTierTypeOptionsByPricingModel, catalogTrialEndActionOptions, catalogTrialUnitOptions, } = getInvoicingConfigView(state);
     return {
         fieldHelp: invoicingFieldHelpForGroup(state, 'catalog'),
-        isQboPickerDataLoading: isInvoicingAccountingPickerDataLoading(state),
         qboClassHierarchy: isQboConnected
             ? getInvoicingQboView(state).nestedClassHierarchy
             : [],

package/lib/esm/view/invoicing/invoicingConfigView/invoicingConfigViewSelector.js

@@ -1,4 +1,3 @@
-import { isInvoicingQboConnected } from '../invoicingQboView/invoicingQboViewSelector';
 import { deriveInvoicingOnboardingStatus, } from './invoicingConfigViewPayload';
 export const getInvoicingConfigView = (state) => {
     const { acceptTermsFetchState, enableFetchState, ...configState } = state.invoicingConfigViewState;
@@ -15,7 +14,6 @@ export const getInvoicingOnboardingGateView = (state) => {
         acceptTermsState: acceptTermsFetchState,
         enableState: enableFetchState,
         fetchState,
-        isQboConnected: isInvoicingQboConnected(state),
         status: deriveInvoicingOnboardingStatus(state.invoicingConfigViewState.isInvoicingTosAccepted, state.invoicingConfigViewState.isInvoicingEnabled),
     };
 };

package/lib/esm/view/invoicing/invoicingQboView/epics/fetchInvoicingAccountingPickersEpic.js

@@ -9,17 +9,12 @@ import { INVOICING_ACCOUNT_LIST_KEY } from '../invoicingQboViewSelector';
 export const fetchInvoicingAccountingPickersEpic = (actions$, state$) => actions$.pipe(filter(fetchInvoicingAccountingPickers.match), withLatestFrom(state$), mergeMap(([, state]) => {
     const actions = [];
     const accountList = state.accountListState.byReportId[INVOICING_ACCOUNT_LIST_KEY];
-    // `hasValidState` only reports that a fetch finished, not that it
-    // returned anything usable. The shared bucket can be Completed and empty
-    // because a screen that needs fewer accounts filled it first, which would
-    // leave the invoicing pickers empty with no request in flight to fix it.
-    const isAccountListUsable = accountList.hasValidState() && accountList.accountIds.length > 0;
-    if (isAccountListUsable === false &&
+    if (accountList.hasValidState() === false &&
         accountList.fetchState !== 'In-Progress') {
-        // Same arguments the vendor and categorization screens pass, so this
-        // shares their cached list rather than replacing it with a narrower one.
-        // Invoicing narrows to income accounts when it builds its options.
-        actions.push(fetchAccountList(INVOICING_ACCOUNT_LIST_KEY, true));
+        // AP/AR accounts cannot back an invoicing mapping, so they are excluded
+        // here — which is why this uses invoicing's own bucket rather than the
+        // general `accountList` other screens fill without those exclusions.
+        actions.push(fetchAccountList(INVOICING_ACCOUNT_LIST_KEY, true, true, true));
     }
     const classList = state.classListState;
     if (classList.hasValidState() === false &&

package/lib/esm/view/invoicing/invoicingQboView/epics/fetchInvoicingQboAccountsEpic.js

@@ -0,0 +1,25 @@
+import { of } from 'rxjs';
+import { catchError, filter, map, mergeMap } from 'rxjs/operators';
+import { createZeniAPIStatus, isSuccessResponse, } from '../../../../responsePayload';
+import { mapInvoicingQboAccountOptions, } from '../invoicingQboViewPayload';
+import { fetchInvoicingQboAccounts, updateInvoicingQboAccounts, updateInvoicingQboAccountsFailure, } from '../invoicingQboViewReducer';
+export const fetchInvoicingQboAccountsEpic = (actions$, _state$, zeniAPI) => actions$.pipe(filter(fetchInvoicingQboAccounts.match), mergeMap((action) => {
+    const { accountTypes } = action.payload;
+    const apiUrl = `${zeniAPI.apiEndPoints.invoicingMicroServiceBaseUrl}/1.0/qbo/accounts?account_types=${encodeURIComponent(accountTypes)}`;
+    return zeniAPI.getJSON(apiUrl).pipe(map((response) => {
+        if (isSuccessResponse(response)) {
+            return updateInvoicingQboAccounts({
+                accountTypes,
+                accounts: mapInvoicingQboAccountOptions(response.data?.accounts ?? []),
+            });
+        }
+        return updateInvoicingQboAccountsFailure({
+            accountTypes,
+            status: response.status,
+        });
+    }), catchError((error) => of(updateInvoicingQboAccountsFailure({
+        accountTypes,
+        status: createZeniAPIStatus('Unexpected Error', 'Fetch QBO accounts REST API call errored out ' +
+            JSON.stringify(error)),
+    }))));
+}));