@zeniai/client-epic-state 5.1.42 → 5.1.43

package/lib/esm/view/taskManager/taskDetailView/taskDetailSelector.js

@@ -1,8 +1,8 @@
 import isEqual from 'lodash/isEqual';
-import { reduceFetchState } from '../../../commonStateTypes/reduceFetchState';
+import { reduceAnyFetchState } from '../../../commonStateTypes/reduceFetchState';
 import { getClassesByIds } from '../../../entity/class/classSelector';
 import { getFilesByFileIds } from '../../../entity/file/fileSelector';
-import { getTaskById } from '../../../entity/task/taskSelector';
+import { getTaskById, getTasksByIds } from '../../../entity/task/taskSelector';
 import { getUserAndUserRole, } from '../../companyView/types/userAndRole';
 import { getFileDeleteStatusById, getFileUpdateNameStatusById, } from '../../fileView/fileViewSelector';
 import { getUserList, } from '../../userListView/userListViewSelector';
@@ -27,8 +27,13 @@ export const getTaskDetail = (state, taskId) => {
     let recurringSourceTaskId = undefined;
     let snoozedUntil = undefined;
     let taskGroupId = undefined;
+    let subtasks = [];
     if (taskId != null && sourceTaskDetail != null) {
         const taskEntity = getTaskById(taskState, taskId);
+        subtasks =
+            taskEntity != null
+                ? getTasksByIds(taskState, taskEntity.subTasksIds)
+                : [];
         const fileIdsInEntity = taskEntity?.fileIds ?? [];
         if (taskEntity != null) {
             createdByUser = getUserAndUserRole(userState, userRoleState, addressState, taskEntity.createdBy);
@@ -38,26 +43,37 @@ export const getTaskDetail = (state, taskId) => {
             snoozedUntil = taskEntity.snoozedUntil;
             taskGroupId = taskEntity.taskGroupIds[0];
         }
+        // Treat "no entry" as Completed so it doesn't block the combined
+        // fetch state. The Not-Started default would pin the detail page
+        // below Completed forever before the first fetchSubTasks dispatches.
+        const completedSubTaskStatus = {
+            fetchState: 'Completed',
+            error: undefined,
+        };
+        const subTaskStatus = taskDetailState.subTaskListFetchStatusByParentId[taskId] ??
+            completedSubTaskStatus;
         if (fileIdsInEntity.length > 0) {
-            fetchStatus = reduceFetchState([
+            fetchStatus = reduceAnyFetchState([
                 fileViewState.fetchFilesStatus,
                 sourceTaskDetail.fetchTaskStatus,
                 userList,
                 classListState,
+                subTaskStatus,
             ]);
         }
         else {
-            fetchStatus = reduceFetchState([
+            fetchStatus = reduceAnyFetchState([
                 sourceTaskDetail.fetchTaskStatus,
                 userList,
                 classListState,
+                subTaskStatus,
             ]);
         }
         showTaskDetailFormFooter = showFormFooter(sourceTaskDetail.taskDetailLocalData, taskEntity);
         taskHistory = taskDetailState.taskHistoryById[taskId]?.historicEvents ?? [];
     }
     else if (taskId == null) {
-        fetchStatus = reduceFetchState([userList, classListState]);
+        fetchStatus = reduceAnyFetchState([userList, classListState]);
     }
     const { classIds } = classListState;
     const allAccountingClasses = getClassesByIds(classState, {
@@ -90,6 +106,7 @@ export const getTaskDetail = (state, taskId) => {
         showTaskDetailFormFooter,
         snoozedUntil,
         taskGroupId,
+        subtasks,
     };
 };
 export const allTaskStatus = [
@@ -137,13 +154,16 @@ export const allTaskPriority = [
     },
 ];
 const showFormFooter = (taskDetailLocalData, taskDetailInStore) => {
-    if (taskDetailLocalData.name !== taskDetailInStore?.name ||
+    if (taskDetailInStore == null) {
+        return false;
+    }
+    if (taskDetailLocalData.name !== taskDetailInStore.name ||
         taskDetailLocalData.description !== taskDetailInStore.description ||
         taskDetailLocalData.priority !== taskDetailInStore.priority.code ||
         taskDetailLocalData.status !== taskDetailInStore.status.code ||
         !isEqual(taskDetailInStore.tagIds, taskDetailLocalData.tagIds) ||
-        !isEqual(taskDetailLocalData.assignee, taskDetailInStore?.assignees ?? []) ||
-        !isEqual(taskDetailLocalData.groupAssignees, taskDetailInStore?.groupAssignees ?? []) ||
+        !isEqual(taskDetailLocalData.assignee, taskDetailInStore.assignees ?? []) ||
+        !isEqual(taskDetailLocalData.groupAssignees, taskDetailInStore.groupAssignees ?? []) ||
         !isEqual(taskDetailLocalData.fileIds, taskDetailInStore.fileIds ?? []) ||
         !isEqual(taskDetailLocalData.dueDate, taskDetailInStore.dueDate) ||
         !isEqual(taskDetailLocalData.type, taskDetailInStore.type) ||
@@ -151,7 +171,10 @@ const showFormFooter = (taskDetailLocalData, taskDetailInStore) => {
         !isEqual(taskDetailLocalData.recurringFrequency, taskDetailInStore.recurringFrequency) ||
         !isEqual(taskDetailLocalData.recurringDaysOfWeek, taskDetailInStore.recurringDaysOfWeek) ||
         !isEqual(taskDetailLocalData.recurringStartDate, taskDetailInStore.recurringStartDate) ||
-        !isEqual(taskDetailLocalData.timeSpent, taskDetailInStore.timeSpent)) {
+        !isEqual(taskDetailLocalData.timeSpent, taskDetailInStore.timeSpent) ||
+        !isEqual(taskDetailLocalData.taskGroupId, taskDetailInStore.taskGroupIds.length > 0
+            ? taskDetailInStore.taskGroupIds[0]
+            : undefined)) {
         return true;
     }
     return false;

package/lib/esm/view/taskManager/taskListView/epics/fetchTaskListEpic.js

@@ -9,15 +9,17 @@ export const fetchTaskListEpic = (actions$, _state$, zeniAPI) => actions$.pipe(f
         .getJSON(`${zeniAPI.apiEndPoints.taskMicroServiceBaseUrl}/1.0/task-manager/tasks?query=${encodeURIComponent(`{"task_type": "all"}`)}`)
         .pipe(mergeMap((response) => {
         if (isSuccessResponse(response) && response.data != null) {
-            const { tasks, deleted, archived, snoozed } = response.data;
+            const { tasks, deleted, archived, snoozed, completed } = response.data;
             const allTasks = [
                 ...tasks,
+                ...(completed ?? []),
                 ...(deleted ?? []),
                 ...(archived ?? []),
                 ...(snoozed ?? []),
             ];
             return of(updateTasks(allTasks), updateTaskList({
                 data: tasks,
+                completed: completed ?? [],
                 deleted: deleted ?? [],
                 archived: archived ?? [],
                 snoozed: snoozed ?? [],

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

@@ -1,6 +1,7 @@
 import { stringToUnion, stringToUnionStrict, } from '../../../commonStateTypes/stringToUnion';
 export const ALL_TASK_LIST_TABS = [
     'live',
+    'completed',
     'archived',
     'deleted',
     'snoozed',

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

@@ -1,4 +1,5 @@
 import { createSlice } from '@reduxjs/toolkit';
+import { updateTasks } from '../../../entity/task/taskReducer';
 import { toPriorityCodeType, toTaskStatusCodeType, } from '../../../entity/task/taskState';
 import { date, dateNow } from '../../../zeniDayJS';
 import { toDueDateGroupKeyType, } from './taskList';
@@ -42,6 +43,7 @@ export const initialState = {
     currentTab: 'live',
     byTab: {
         live: { ...initialTabData },
+        completed: { ...initialTabData },
         archived: { ...initialTabData },
         deleted: { ...initialTabData },
         snoozed: { ...initialTabData },
@@ -63,14 +65,12 @@ export const initialState = {
     },
     filters: {
         categoryCombinationOperator: 'AND',
-        categories: [
-            {
-                field: 'status',
-                matchingOperator: 'not_equal',
-                values: ['resolved'],
-                valuesCombinationOperator: 'ANY',
-            },
-        ],
+        // Default to no filters. The Completed tab now owns the "hide resolved
+        // tasks from the live list" responsibility via the byTab split in
+        // updateTaskList / updateTaskFromListView / the updateTasks
+        // extraReducer. Keeping the historical `status != resolved` filter
+        // here would empty out the Completed tab.
+        categories: [],
     },
     hasValidState() {
         return this.fetchState == 'Completed';
@@ -102,13 +102,16 @@ const taskList = createSlice({
             draft.error = undefined;
         },
         updateTaskList(draft, action) {
-            const { data, archived, deleted, snoozed } = action.payload;
+            const { data, archived, completed, deleted, snoozed } = action.payload;
             draft.fetchState = 'Completed';
             draft.error = undefined;
             // Group live tasks
             const liveGrouped = getGroupedTaskIds(data);
             draft.byTab.live = toTabData(liveGrouped);
             // Group other tabs if provided
+            if (completed != null) {
+                draft.byTab.completed = toTabData(getGroupedTaskIds(completed));
+            }
             if (archived != null) {
                 draft.byTab.archived = toTabData(getGroupedTaskIds(archived));
             }
@@ -354,9 +357,26 @@ const taskList = createSlice({
                 draft.taskIdsByAssignees[assigneeKey] = draft.taskIdsByAssignees[assigneeKey].filter((taskId) => taskId !== currentTaskState.id);
                 draft.taskIdsByTags[tagsKey] = draft.taskIdsByTags[tagsKey].filter((taskId) => taskId !== currentTaskState.id);
                 draft.taskIdsByDueDate[dueDateKey] = draft.taskIdsByDueDate[dueDateKey].filter((taskId) => taskId !== currentTaskState.id);
-                draft.taskIdsByGroupsIds[taskGroupId] = draft.taskIdsByGroupsIds[taskGroupId].filter((taskId) => taskId !== currentTaskState.id);
+                // Remove from EVERY previous group bucket the task lived in, not
+                // just the destination bucket. Otherwise a detail-page save that
+                // moves a task from group A to group B leaves the row listed
+                // under both A and B until refetch.
+                const previousGroupIds = currentTaskState.taskGroupIds ?? [];
+                const allGroupIdsToClean = new Set([
+                    ...previousGroupIds,
+                    taskGroupId,
+                ]);
+                allGroupIdsToClean.forEach((gid) => {
+                    if (draft.taskIdsByGroupsIds[gid] != null) {
+                        draft.taskIdsByGroupsIds[gid] = draft.taskIdsByGroupsIds[gid].filter((id) => id !== currentTaskState.id);
+                    }
+                });
             }
             doUpdateTaskList(draft, task, taskGroupId);
+            // Mirror top-level grouping into the active tab's byTab snapshot,
+            // otherwise the next updateTaskListTab call copies stale byTab data
+            // back over this optimistic update.
+            syncTopLevelToActiveTab(draft);
         },
         updateTaskListOnCreateTaskFromTemplateSuccess(draft, action) {
             const { tasks } = action.payload;
@@ -380,6 +400,7 @@ const taskList = createSlice({
             const taskId = task.id;
             draft.updateTasksFetchState.fetchState = 'In-Progress';
             draft.updateTasksFetchState.error = undefined;
+            let didMigrateTab = false;
             if (updates.priority != null) {
                 const prevPriorityCode = toPriorityCodeType(task.priority.code);
                 draft.taskIdsByPriority[prevPriorityCode] = draft.taskIdsByPriority[prevPriorityCode].filter((id) => id != taskId);
@@ -391,6 +412,25 @@ const taskList = createSlice({
                 draft.taskIdsByStatus[prevStatusCode] = draft.taskIdsByStatus[prevStatusCode].filter((id) => id != taskId);
                 const newStatusCode = toTaskStatusCodeType(updates.status.code);
                 draft.taskIdsByStatus[newStatusCode].push(taskId);
+                // Optimistically re-bucket between the live and completed tabs
+                // when status transitions in/out of "resolved", so the task
+                // moves to the Completed tab without waiting for the next
+                // fetch. Mirrors backend split in task_handler.py: only
+                // `resolved` counts as completed; `done` stays Active.
+                //
+                // Subtasks travel with their parent — they appear nested under
+                // the parent row, not as standalone entries in byTab.taskIds.
+                // So we only rebucket parent tasks (tasks with no parentTaskId).
+                const isTopLevelTask = task.parentTaskId == null;
+                const wasCompleted = prevStatusCode === 'resolved';
+                const isCompleted = newStatusCode === 'resolved';
+                if (isTopLevelTask && wasCompleted !== isCompleted) {
+                    const from = isCompleted ? 'live' : 'completed';
+                    const to = isCompleted ? 'completed' : 'live';
+                    const updatedTask = { ...task, ...updates };
+                    migrateTaskBetweenTabs(draft, taskId, updatedTask, from, to);
+                    didMigrateTab = true;
+                }
             }
             if (updates.assignees != null) {
                 const prevGroupId = getAssigneesGroupKey(task.assignees);
@@ -430,7 +470,18 @@ const taskList = createSlice({
                     };
                 }
             }
-            syncTopLevelToActiveTab(draft);
+            if (didMigrateTab) {
+                // The migration moved the task out of the source byTab and into
+                // the destination byTab. The active tab's grouping arrays are
+                // now authoritative — mirror them back to the top-level so the
+                // UI sees the task gone from Live (or freshly arrived in
+                // Completed). Inverse of syncTopLevelToActiveTab, used only on
+                // this path.
+                copyTabDataToTopLevel(draft, draft.byTab[draft.currentTab]);
+            }
+            else {
+                syncTopLevelToActiveTab(draft);
+            }
         },
         updateTasksListOnUpdateTaskSuccess(draft) {
             draft.updateTasksFetchState.fetchState = 'Completed';
@@ -443,12 +494,80 @@ const taskList = createSlice({
         removeTaskFromList(draft, action) {
             removeTaskIds(draft, [action.payload]);
         },
+        /**
+         * Remove a task id from a single group bucket without touching tab
+         * membership or other groupings. Used when the user clears a task's
+         * group on the Detail page so the row stops appearing under the old
+         * group while saveTaskDetailEpic skips the standard list re-insert.
+         */
+        removeTaskFromGroupBucket(draft, action) {
+            const { taskId, taskGroupId } = action.payload;
+            if (draft.taskIdsByGroupsIds[taskGroupId] != null) {
+                draft.taskIdsByGroupsIds[taskGroupId] = draft.taskIdsByGroupsIds[taskGroupId].filter((id) => id !== taskId);
+            }
+            // Mirror to the active tab's grouping so the UI re-renders.
+            const activeTab = draft.byTab[draft.currentTab];
+            if (activeTab.taskIdsByGroupsIds[taskGroupId] != null) {
+                activeTab.taskIdsByGroupsIds[taskGroupId] =
+                    activeTab.taskIdsByGroupsIds[taskGroupId].filter((id) => id !== taskId);
+            }
+        },
         clearTaskList(draft) {
             Object.assign(draft, initialState);
         },
     },
+    extraReducers: (builder) => {
+        // When the entity store receives updated tasks (e.g. from task detail
+        // save, snooze/archive/unarchive, etc.), re-bucket parent tasks between
+        // the live and completed tabs based on the new status. Mirrors the
+        // backend split in task_handler.py: only `resolved` parents count as
+        // completed. Subtasks are nested under their parent row, so they
+        // never move between tabs on their own.
+        builder.addCase(updateTasks, (draft, action) => {
+            action.payload.forEach((taskPayload) => {
+                // Only re-bucket parents; subtasks (parent_task_id is a non-empty
+                // string) are nested under their parent row and don't move tabs.
+                if (taskPayload.parent_task_id != null &&
+                    taskPayload.parent_task_id !== '') {
+                    return;
+                }
+                const taskId = taskPayload.task_id;
+                // `TaskPayload.status` is required per the type — every producer
+                // in this repo sends it, and `mapTaskPayloadToTask` dereferences
+                // it unguarded. The optional chain + null guard are defensive
+                // only, in case a downstream consumer (web-app-ui / cockpit-ui)
+                // ever dispatches `updateTasks` with a partial payload; in that
+                // case we treat missing status as "no status change" and skip
+                // re-bucketing so we don't migrate a Completed task back into
+                // the Live tab.
+                const statusCode = taskPayload.status?.code;
+                if (statusCode == null) {
+                    return;
+                }
+                const isCompleted = statusCode === 'resolved';
+                const targetTab = isCompleted ? 'completed' : 'live';
+                const otherTab = isCompleted ? 'live' : 'completed';
+                // Skip if the task isn't currently in either of these buckets
+                // (archived/snoozed/deleted are managed by their own actions).
+                const inLive = draft.byTab.live.taskIds.includes(taskId);
+                const inCompleted = draft.byTab.completed.taskIds.includes(taskId);
+                if (!inLive && !inCompleted) {
+                    return;
+                }
+                if (draft.byTab[otherTab].taskIds.includes(taskId)) {
+                    migrateTaskPayloadBetweenTabs(draft, taskId, taskPayload, otherTab, targetTab);
+                }
+                else if (!draft.byTab[targetTab].taskIds.includes(taskId)) {
+                    draft.byTab[targetTab].taskIds.push(taskId);
+                }
+            });
+            // Re-sync the top-level taskIds / grouping arrays for whichever
+            // tab the user is currently looking at.
+            copyTabDataToTopLevel(draft, draft.byTab[draft.currentTab]);
+        });
+    },
 });
-export const { fetchTaskListPage, fetchTaskList, updateTaskList, updateTaskListTab, updateTaskListFetchStatus, updateTaskListSearchText, updateTaskListUIState, updateTaskFilters, updateTaskListOnNewGroupCreationSuccess, bulkUpdateTaskList, updateTasksListOnBulkUpdateTasksSuccess, updateTasksListOnBulkUpdateTasksFailure, initiateTaskListLocalData, updateTaskListLocalData, updateTaskListOnTaskGroupDeletion, dragNDropTasks, updateTaskListOnNewTaskCreationSuccess, updateTaskListOnTaskGroupNameUpdate, updateTaskFromListView, updateTasksListOnUpdateTaskSuccess, updateTasksListOnUpdateTaskFailure, updateTaskListOnCreateTaskFromTemplateSuccess, removeTaskFromList, clearTaskList, } = taskList.actions;
+export const { fetchTaskListPage, fetchTaskList, updateTaskList, updateTaskListTab, updateTaskListFetchStatus, updateTaskListSearchText, updateTaskListUIState, updateTaskFilters, updateTaskListOnNewGroupCreationSuccess, bulkUpdateTaskList, updateTasksListOnBulkUpdateTasksSuccess, updateTasksListOnBulkUpdateTasksFailure, initiateTaskListLocalData, updateTaskListLocalData, updateTaskListOnTaskGroupDeletion, dragNDropTasks, updateTaskListOnNewTaskCreationSuccess, updateTaskListOnTaskGroupNameUpdate, updateTaskFromListView, updateTasksListOnUpdateTaskSuccess, updateTasksListOnUpdateTaskFailure, updateTaskListOnCreateTaskFromTemplateSuccess, removeTaskFromList, removeTaskFromGroupBucket, clearTaskList, } = taskList.actions;
 export default taskList.reducer;
 /**
  *  helpers
@@ -555,6 +674,115 @@ const copyTabDataToTopLevel = (draft, tabData) => {
     draft.taskIdsByDueDate = tabData.taskIdsByDueDate;
     draft.taskIdsByTags = tabData.taskIdsByTags;
 };
+/**
+ * Migrate a parent task between two `byTab` buckets (live ↔ completed)
+ * when its status flips into / out of `resolved`. The destination tab
+ * picks up the task in EVERY grouping dimension (status, priority,
+ * group id, assignees, dueDate, tags) using the UPDATED task as the
+ * source-of-truth for the new bucket keys. Otherwise the Completed tab
+ * shows the task in the right tab-level list but the grouped views
+ * (Group by Status, Group by Priority, etc.) miss it until the next
+ * full fetch.
+ */
+const migrateTaskBetweenTabs = (draft, taskId, updatedTask, from, to) => {
+    const fromTab = draft.byTab[from];
+    const toTab = draft.byTab[to];
+    // Source tab: drop the task from every grouping dimension.
+    removeIdsFromTabData(fromTab, [taskId]);
+    // Destination tab: add the task to taskIds (deduped) plus the
+    // appropriate bucket inside each grouping dimension based on the
+    // updated task.
+    if (!toTab.taskIds.includes(taskId)) {
+        toTab.taskIds.push(taskId);
+    }
+    const priorityKey = toPriorityCodeType(updatedTask.priority.code);
+    if (!toTab.taskIdsByPriority[priorityKey].includes(taskId)) {
+        toTab.taskIdsByPriority[priorityKey].push(taskId);
+    }
+    const statusKey = toTaskStatusCodeType(updatedTask.status.code);
+    if (!toTab.taskIdsByStatus[statusKey].includes(taskId)) {
+        toTab.taskIdsByStatus[statusKey].push(taskId);
+    }
+    const dueDateKey = updatedTask.dueDate != null
+        ? getDueDateGroupKey(updatedTask.dueDate.toString())
+        : 'upcoming';
+    if (!toTab.taskIdsByDueDate[dueDateKey].includes(taskId)) {
+        toTab.taskIdsByDueDate[dueDateKey].push(taskId);
+    }
+    const assigneeKey = getAssigneesGroupKey(updatedTask.assignees);
+    if (toTab.taskIdsByAssignees[assigneeKey] == null) {
+        toTab.taskIdsByAssignees[assigneeKey] = [taskId];
+    }
+    else if (!toTab.taskIdsByAssignees[assigneeKey].includes(taskId)) {
+        toTab.taskIdsByAssignees[assigneeKey].push(taskId);
+    }
+    const tagIds = [...updatedTask.tagIds].sort();
+    const tagsKey = tagIds.join(',');
+    if (toTab.taskIdsByTags[tagsKey] == null) {
+        toTab.taskIdsByTags[tagsKey] = [taskId];
+    }
+    else if (!toTab.taskIdsByTags[tagsKey].includes(taskId)) {
+        toTab.taskIdsByTags[tagsKey].push(taskId);
+    }
+    (updatedTask.taskGroupIds ?? []).forEach((groupId) => {
+        if (toTab.taskIdsByGroupsIds[groupId] == null) {
+            toTab.taskIdsByGroupsIds[groupId] = [taskId];
+        }
+        else if (!toTab.taskIdsByGroupsIds[groupId].includes(taskId)) {
+            toTab.taskIdsByGroupsIds[groupId].push(taskId);
+        }
+    });
+};
+/**
+ * `updateTasks` extraReducer variant — same migration semantics but
+ * keyed off the raw `TaskPayload` instead of the rich `Task` entity.
+ */
+const migrateTaskPayloadBetweenTabs = (draft, taskId, payload, from, to) => {
+    const fromTab = draft.byTab[from];
+    const toTab = draft.byTab[to];
+    removeIdsFromTabData(fromTab, [taskId]);
+    if (!toTab.taskIds.includes(taskId)) {
+        toTab.taskIds.push(taskId);
+    }
+    if (payload.priority?.code != null) {
+        const priorityKey = toPriorityCodeType(payload.priority.code);
+        if (!toTab.taskIdsByPriority[priorityKey].includes(taskId)) {
+            toTab.taskIdsByPriority[priorityKey].push(taskId);
+        }
+    }
+    if (payload.status?.code != null) {
+        const statusKey = toTaskStatusCodeType(payload.status.code);
+        if (!toTab.taskIdsByStatus[statusKey].includes(taskId)) {
+            toTab.taskIdsByStatus[statusKey].push(taskId);
+        }
+    }
+    const dueDateKey = getDueDateGroupKey(payload.due_date);
+    if (!toTab.taskIdsByDueDate[dueDateKey].includes(taskId)) {
+        toTab.taskIdsByDueDate[dueDateKey].push(taskId);
+    }
+    const assigneeKey = getAssigneesGroupKey(payload.assignees ?? []);
+    if (toTab.taskIdsByAssignees[assigneeKey] == null) {
+        toTab.taskIdsByAssignees[assigneeKey] = [taskId];
+    }
+    else if (!toTab.taskIdsByAssignees[assigneeKey].includes(taskId)) {
+        toTab.taskIdsByAssignees[assigneeKey].push(taskId);
+    }
+    const tagsKey = getTagsGroupKey(payload.tags ?? []);
+    if (toTab.taskIdsByTags[tagsKey] == null) {
+        toTab.taskIdsByTags[tagsKey] = [taskId];
+    }
+    else if (!toTab.taskIdsByTags[tagsKey].includes(taskId)) {
+        toTab.taskIdsByTags[tagsKey].push(taskId);
+    }
+    (payload.task_group_ids ?? []).forEach((groupId) => {
+        if (toTab.taskIdsByGroupsIds[groupId] == null) {
+            toTab.taskIdsByGroupsIds[groupId] = [taskId];
+        }
+        else if (!toTab.taskIdsByGroupsIds[groupId].includes(taskId)) {
+            toTab.taskIdsByGroupsIds[groupId].push(taskId);
+        }
+    });
+};
 const removeIdsFromTabData = (tabData, taskIds) => {
     const removedSet = new Set(taskIds);
     const isRemoved = (id) => removedSet.has(id);

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

@@ -334,6 +334,7 @@ export const getAllTasks = createSelector((state) => state.taskListState, (state
     const taskGroupTemplates = getTaskGroupTemplateByIds(taskGroupTemplateState, taskGroupTemplateViewState.taskGroupTemplateIds);
     const tabCounts = {
         live: taskListState.byTab.live.taskIds.length,
+        completed: taskListState.byTab.completed.taskIds.length,
         archived: taskListState.byTab.archived.taskIds.length,
         deleted: taskListState.byTab.deleted.taskIds.length,
         snoozed: taskListState.byTab.snoozed.taskIds.length,

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

@@ -48,3 +48,61 @@ export const getTaskUpdates = (groupByKey, groupId) => {
         taskGroupIds: [groupId],
     };
 };
+/**
+ * Sort a subtask list by one of the sortable columns. Pure — no state
+ * dependency. `statusOrder` is derived from `allTaskStatus` so any new
+ * status added upstream slots in by its enum position instead of
+ * falling into a catch-all bucket.
+ */
+export const sortSubtasks = (subtasks, sortKey, sortOrder, userList, tagList) => {
+    if (sortKey == null) {
+        return subtasks;
+    }
+    const statusOrder = allTaskStatus.reduce((acc, status, index) => {
+        acc[status.code] = index;
+        return acc;
+    }, {});
+    const unknownStatusRank = allTaskStatus.length;
+    const cmpStr = (a, b) => a.localeCompare(b, undefined, { sensitivity: 'base' });
+    const keyFn = (task) => {
+        if (sortKey === 'name') {
+            return task.name ?? '';
+        }
+        if (sortKey === 'status') {
+            return statusOrder[task.status.code] ?? unknownStatusRank;
+        }
+        if (sortKey === 'assignee') {
+            const firstId = task.assignees?.[0];
+            if (firstId == null) {
+                return '';
+            }
+            const user = userList.find((candidate) => candidate.userId === firstId);
+            return user
+                ? `${user.firstName ?? ''} ${user.lastName ?? ''}`.trim()
+                : '';
+        }
+        if (sortKey === 'tag') {
+            const firstTagId = task.tagIds?.[0];
+            if (firstTagId == null) {
+                return '';
+            }
+            const tag = tagList.find((candidate) => candidate.tagId === firstTagId);
+            return tag?.name ?? '';
+        }
+        return '';
+    };
+    const copy = [...subtasks];
+    copy.sort((a, b) => {
+        const av = keyFn(a);
+        const bv = keyFn(b);
+        let cmp;
+        if (typeof av === 'number' && typeof bv === 'number') {
+            cmp = av - bv;
+        }
+        else {
+            cmp = cmpStr(String(av), String(bv));
+        }
+        return sortOrder === 'asc' ? cmp : -cmp;
+    });
+    return copy;
+};

package/lib/index.d.ts

@@ -256,7 +256,7 @@ import { CompanyPassportLocalStoreDataView, CompanyPassportView, getCompanyPassp
 import { CompanyPortfolioView, getCompanyPortfolioView, isInfiniteRunway } from './view/companyView/selector/companyPortfolioViewSelector';
 import { AllCockpitTabsView, getAllCockpitTabsFilterView } from './view/companyView/selector/getAllCockpitTabsFilterView';
 import { ALL_COCKPIT_TABS_FILE_TYPES, ALL_COCKPIT_TABS_IDS, CockpitDownloadTabsID, CockpitTabsFileType, CockpitTabsID, toCockpitTabsFileTypeStrict, toCockpitTabsIDStrict } from './view/companyView/types/cockpitTabsID';
-import { CockpitFilterCategory, CockpitFilters, DownloadCockpitTabOptions, FilterCombinationsByTabs, HealthMonitorViewFilterCategoryField, HealthViewSortKey, ManagementViewFilterCategoryField, ManagementViewSortKey, OnboardingViewSortKey, PortfolioViewFilterCategoryField, PortfolioViewSortKey, SubscriptionViewFilterCategoryField, TaskManagerViewFilterCategoryField, TaskManagerViewSortKey, toHealthSortKeyType, toManagementSortKeyType, toOnboardingSortKeyType, toPortfolioSortKeyType, toTaskManagerSortKeyType } from './view/companyView/types/cockpitTypes';
+import { CockpitFilterCategory, CockpitFilters, DownloadCockpitTabOptions, FilterCombinationsByTabs, HealthMonitorViewFilterCategoryField, HealthViewSortKey, ManagementViewFilterCategoryField, ManagementViewSortKey, OnboardingViewSortKey, PortfolioViewFilterCategoryField, PortfolioViewSortKey, SubscriptionViewFilterCategoryField, TaskGroupScope, TaskManagerViewFilterCategoryField, TaskManagerViewSortKey, toHealthSortKeyType, toManagementSortKeyType, toOnboardingSortKeyType, toPortfolioSortKeyType, toTaskManagerSortKeyType } from './view/companyView/types/cockpitTypes';
 import { ManagementUIState } from './view/companyView/types/companyManagementViewState';
 import { CompanyDetailsLocalData, CompanyInfoLocalData, IncInfoLocalData, TaxDetailsLocalData } from './view/companyView/types/companyPassport/companyDetailsLocalData';
 import { CompanyPassportLocalData, toCompanyPassportLocalData } from './view/companyView/types/companyPassport/companyPassportLocalData';
@@ -563,15 +563,15 @@ import { TagViewState } from './view/tagView/tagViewState';
 import { CannedResponse, CannedResponsesViewState } from './view/taskManager/cannedResponsesView/cannedResponses';
 import { deleteCannedResponse, fetchCannedResponses, saveCannedResponse } from './view/taskManager/cannedResponsesView/cannedResponsesReducer';
 import { getCannedResponsesView } from './view/taskManager/cannedResponsesView/cannedResponsesSelector';
-import { EditTaskLocalData, initialTaskDetail, initialTaskDetailLocalData } from './view/taskManager/taskDetailView/taskDetail';
-import { archiveTask, deleteTask, discardTaskUpdatesInLocalStore, fetchTaskDetailPage, saveTaskDetail, saveTaskUpdatesToLocalStore, snoozeTask, unsnoozeTask } from './view/taskManager/taskDetailView/taskDetailReducer';
+import { CreateSubTaskPayload, EditTaskLocalData, initialTaskDetail, initialTaskDetailLocalData } from './view/taskManager/taskDetailView/taskDetail';
+import { archiveTask, createSubTask, deleteTask, discardTaskUpdatesInLocalStore, fetchSubTasks, fetchTaskDetailPage, resetSubTaskCreateStatus, saveTaskDetail, saveTaskUpdatesToLocalStore, snoozeTask, unsnoozeTask } from './view/taskManager/taskDetailView/taskDetailReducer';
 import { TaskDetailSelectorView, allTaskPriority, allTaskStatus, getTaskDetail } from './view/taskManager/taskDetailView/taskDetailSelector';
 import { createTaskFromTaskGroupTemplate } from './view/taskManager/taskGroupTemplateView/taskGroupTemplateViewReducer';
 import { createNewTaskGroup, deleteTaskGroup, fetchAllTaskGroups, updateTaskGroupName } from './view/taskManager/taskGroupView/taskGroupViewReducer';
 import { ALL_TASK_LIST_TABS, DueDateGroupKey, TASK_LIST_FILTER_CATEGORIES, TASK_LIST_GROUP_BY_CATEGORIES, TaskFilterCategory, TaskGroupKey, TaskListFilterCategoryField, TaskListFilters, TaskListLocalData, TaskListTab, TaskListUIState, TaskListViewSortKey, toDueDateGroupKeyType, toTaskListGroupByKeyType, toTaskListGroupByKeyTypeStrict } from './view/taskManager/taskListView/taskList';
 import { bulkUpdateTaskList, dragNDropTasks, fetchTaskListPage, initiateTaskListLocalData, removeTaskFromList, updateTaskFilters, updateTaskFromListView, updateTaskListLocalData, updateTaskListSearchText, updateTaskListTab, updateTaskListUIState } from './view/taskManager/taskListView/taskListReducer';
 import { TaskGroupWithTasksList, TaskListSelectorView, TaskWithUserDetails, getAllTasks } from './view/taskManager/taskListView/taskListSelector';
-import { getDueDateValueFromDueDateGroupId, getTaskUpdates } from './view/taskManager/taskListView/taskListViewHelpers';
+import { SubtaskSortKey, SubtaskSortOrder, getDueDateValueFromDueDateGroupId, getTaskUpdates, sortSubtasks } from './view/taskManager/taskListView/taskListViewHelpers';
 import { fetchTasksCard } from './view/tasksCard/tasksCardReducer';
 import { TasksCardReport } from './view/tasksCard/tasksCardSelector';
 import { fetchTopEx, updateTopExDownloadState, updateTopExTimeFrame } from './view/topEx/topExReducer';
@@ -784,7 +784,7 @@ export { AllCockpitTabsView, getAllCockpitTabsFilterView };
 export { AuditSummaryData, fetchImproveUsingZeniGPT, fetchCompanyMonthEndReportView, sendCompanyMonthEndReport, saveCompanyMonthEndReport, resetCompanyMonthEndReportFetchStatus, resetSendCompanyMonthEndReportFetchStatus, getCompanyMonthEndReportSelectorView, CompanyMonthEndReportSelectorView, fetchCompanyMonthEndReportHistoricDates, fetchCompanyMonthEndReportHistoricData, fetchCompanyMonthEndReportTemplates, getCompanyMonthEndReportHistoricData, CompanyMonthEndReportHistoricDataSelectorView, CompanyMonthEndReportData, TemplatesList, CompanyMonthReportTemplates, };
 export { Runway, AccountingSummary };
 export { newAddressInLocalStore, getAddress, getNewAddress, getAllNewAddresses, getFormattedAddress, fetchAddress, clearAddressView, AddressType, NewAddressData, AddressUpdatableInfo, createAddress, removeNewAddressDataInLocalStore, };
-export { CockpitFilterCategory, CockpitFilters, DownloadCockpitTabOptions, FilterCombinationsByTabs, HealthViewSortKey, ManagementViewFilterCategoryField, ManagementViewSortKey, OnboardingViewSortKey, PortfolioViewFilterCategoryField, HealthMonitorViewFilterCategoryField, TaskManagerViewFilterCategoryField, PortfolioViewSortKey, SubscriptionViewFilterCategoryField, toManagementSortKeyType, toOnboardingSortKeyType, toPortfolioSortKeyType, toHealthSortKeyType, TaskManagerViewSortKey, toTaskManagerSortKeyType, updateCompanyTaskManagerViewFilters, };
+export { CockpitFilterCategory, CockpitFilters, DownloadCockpitTabOptions, FilterCombinationsByTabs, HealthViewSortKey, ManagementViewFilterCategoryField, ManagementViewSortKey, OnboardingViewSortKey, PortfolioViewFilterCategoryField, HealthMonitorViewFilterCategoryField, TaskManagerViewFilterCategoryField, PortfolioViewSortKey, SubscriptionViewFilterCategoryField, toManagementSortKeyType, toOnboardingSortKeyType, toPortfolioSortKeyType, toHealthSortKeyType, TaskGroupScope, TaskManagerViewSortKey, toTaskManagerSortKeyType, updateCompanyTaskManagerViewFilters, };
 export { CockpitDownloadTabsID, CockpitTabsFileType, CockpitTabsID, ALL_COCKPIT_TABS_FILE_TYPES, ALL_COCKPIT_TABS_IDS, toCockpitTabsIDStrict, toCockpitTabsFileTypeStrict, };
 export { AllowedValueWithCode, AllowedValue, AllowedValueWithID, isAllowedValueWithCode, isAllowedValueWithID, };
 export { ManagementUIState, PortfolioUIState };
@@ -893,7 +893,7 @@ export { fetchGlobalMerchantRecommendation, createGlobalMerchant, updateCreateGl
 export { approveVendorGlobalReview, rejectVendorGlobalReview, fetchVendorGlobalReviewView, updateSelectedGlobalMerchant, updateVendorGlobalReviewViewUIState, getVendorGlobalReviewView, getTenantMerchantByMerchantId, toVendorGlobalReviewColumnSortKeyType, VendorGlobalReview, VendorGlobalReviewViewUIState, VendorGlobalReviewViewColumnSortKey, VendorGlobalReviewViewSelectorView, VendorGlobalReviewViewState, VendorGlobalReviewViewLocalData, updateVendorGlobalReviewViewLocalData, GlobalReviewPageCurrentSelectionByColumn, GlobalRecommendationFetchSource, };
 export { fetchArAging, updateArAgingUIState, fetchArAgingDetail, updateArAgingDetailUIState, ArAgingReport, ArAgingDetail, NestedAgingBalancesByCustomer, getArAgingReport, getArAgingDetailForCustomer, updateArAgingNodeCollapseState, };
 export { toRecurringBillFrequency, toRecurringBillFrequencyStrict, RecurringBillFrequency, RecurringBillDetail, RecurringBillConfig, RecurringBillStatus, RecurringBillPaymentStatus, };
-export { Task, TaskPayload, TaskListSelectorView, TaskWithUserDetails, TaskListViewSortKey, TaskListUIState, TaskDetailSelectorView, EditTaskLocalData, TaskStatusCodeType, PriorityCodeType, getTaskGroupById, TaskStatus, TaskPriority, TaskCodeType, getAllTasks, fetchTaskListPage, getTaskDetail, fetchTaskDetailPage, saveTaskUpdatesToLocalStore, saveTaskDetail, archiveTask, CannedResponse, CannedResponsesViewState, fetchCannedResponses, saveCannedResponse, deleteCannedResponse, getCannedResponsesView, updateTaskListSearchText, updateTaskListUIState, allTaskStatus, allTaskPriority, TaskListFilters, updateTaskFilters, TASK_LIST_FILTER_CATEGORIES, TASK_LIST_GROUP_BY_CATEGORIES, TaskListFilterCategoryField, deleteTask, discardTaskUpdatesInLocalStore, TaskFilterCategory, TaskType, TaskGroupWithTasksList, TaskListLocalData, bulkUpdateTaskList, createNewTaskGroup, initiateTaskListLocalData, deleteTaskGroup, updateTaskListLocalData, dragNDropTasks, updateTaskGroupName, TaskGroupState, TaskGroupKey, toTaskListGroupByKeyType, toTaskListGroupByKeyTypeStrict, getTaskUpdates, fetchAllTaskGroups, initialTaskDetail, getDueDateValueFromDueDateGroupId, toPriorityCodeType, toTaskStatusCodeType, DueDateGroupKey, toDueDateGroupKeyType, updateTaskFromListView, updateTaskListTab, removeTaskFromList, snoozeTask, unsnoozeTask, convertHHMMStrToMinutes, initialTaskDetailLocalData, TaskListTab, ALL_TASK_LIST_TABS, };
+export { Task, TaskPayload, TaskListSelectorView, TaskWithUserDetails, TaskListViewSortKey, TaskListUIState, TaskDetailSelectorView, EditTaskLocalData, TaskStatusCodeType, PriorityCodeType, getTaskGroupById, TaskStatus, TaskPriority, TaskCodeType, getAllTasks, fetchTaskListPage, getTaskDetail, fetchTaskDetailPage, saveTaskUpdatesToLocalStore, saveTaskDetail, createSubTask, resetSubTaskCreateStatus, fetchSubTasks, CreateSubTaskPayload, archiveTask, CannedResponse, CannedResponsesViewState, fetchCannedResponses, saveCannedResponse, deleteCannedResponse, getCannedResponsesView, updateTaskListSearchText, updateTaskListUIState, allTaskStatus, allTaskPriority, TaskListFilters, updateTaskFilters, TASK_LIST_FILTER_CATEGORIES, TASK_LIST_GROUP_BY_CATEGORIES, TaskListFilterCategoryField, deleteTask, discardTaskUpdatesInLocalStore, TaskFilterCategory, TaskType, TaskGroupWithTasksList, TaskListLocalData, bulkUpdateTaskList, createNewTaskGroup, initiateTaskListLocalData, deleteTaskGroup, updateTaskListLocalData, dragNDropTasks, updateTaskGroupName, TaskGroupState, TaskGroupKey, toTaskListGroupByKeyType, toTaskListGroupByKeyTypeStrict, SubtaskSortKey, SubtaskSortOrder, getTaskUpdates, sortSubtasks, fetchAllTaskGroups, initialTaskDetail, getDueDateValueFromDueDateGroupId, toPriorityCodeType, toTaskStatusCodeType, DueDateGroupKey, toDueDateGroupKeyType, updateTaskFromListView, updateTaskListTab, removeTaskFromList, snoozeTask, unsnoozeTask, convertHHMMStrToMinutes, initialTaskDetailLocalData, TaskListTab, ALL_TASK_LIST_TABS, };
 export { Tag, TagState, TagViewState, getAllTags, TagViewSelectorView, fetchTagList, createTag, deleteTag, };
 export { CardPayment, CardPaymentViewState, SetupIntentData, CardPaymentSelectorView, getAllCardsAndBankPaymentMethods, createCardSetupIntent, confirmCardSetupIntent, addCardPaymentSource, fetchPaymentSources, resetCardPaymentErrorStatuses, clearCardPaymentView, AddCardPaymentSourceType, };
 export { getAuditReportGroupViewSelectorView, getAuditRuleGroupViewSelectorView, getUserFromAllUsers, AuditReportGroupSelectorView, AuditReportGroupViewSelectorView, AuditRuleGroupViewSelectorView, fetchAuditRuleGroupView, fetchAuditReportGroupView, saveReasonForAuditRule, AuditRuleGroup, AuditRuleGroupReport, AuditReportRule, AuditRuleEntityType, AuditRuleEntityIdType, AuditRuleBypassReason, clearAuditReportGroupViewByCompanyId, };