move sorting into createListItems

src/App.tsx

@@ -14,7 +14,6 @@ import 'react-toastify/dist/ReactToastify.css';
 import { IS_PRODUCTION } from './tr_constants';
 import { QueryClientProvider } from 'react-query';
 import NavBar from './components/organisms/NavBar';
-import browser from './lib/Browser';
 import { Box, Container, CssBaseline, Stack, Alert } from '@mui/material';
 import { H } from 'highlight.run';
 import getQueryClient from './lib/getQueryClient';

src/components/organisms/TaskList.tsx

@@ -25,12 +25,13 @@ const TaskList = ({ newTask }: TaskListProps): JSX.Element => {
 	const [shouldScroll, setShouldScroll] = useState<boolean>(false);
 
 	useEffect(() => {
-		const sorted = sortTasks(tasks || []);
-		const filtered = filters ? sorted.filter((t) => filters[t.status]) : sorted;
+		const filtered = filters
+			? (tasks ?? []).filter((t) => filters[t.status])
+			: tasks ?? [];
 
 		const { entries: newEntries, newTaskIndex: taskIndexUpdate } =
 			createListItems({
-				sortedTasks: filtered,
+				tasks: filtered,
 				newTask,
 			});
 

src/components/pages/Tasks.spec.tsx

@@ -12,7 +12,6 @@ import { makeTask } from '../../lib/test/makeTask';
 import { withMutedReactQueryLogger } from '../../lib/test/withMutedReactQueryLogger';
 import { getUnloadMessage } from '../../lib/getUnloadMessage';
 import browser from '../../lib/Browser';
-import { __listRef } from '../../../__mocks__/react-list';
 import { editTask } from '../../lib/api/editTask';
 import { vi, Mock, describe, it, expect, beforeEach } from 'vitest';
 import loadControlledPromise from '../../lib/test/loadControlledPromise';

src/lib/createListItems.tsx

@@ -1,4 +1,5 @@
 import browser from './Browser';
+import { sortTasks } from './sortTasks';
 
 function makeTitle(task: TaskType) {
 	return browser.getString(new Date(task.due));
@@ -13,16 +14,17 @@ const isNewTask = (t: TaskType, n: TaskType | undefined): boolean => {
 type Entries = (TaskType | string)[];
 
 type Options = {
-	sortedTasks: TaskType[];
+	tasks: TaskType[];
 	newTask: TaskType | undefined;
 };
 
-export default function createListItems({ sortedTasks, newTask }: Options): {
+export default function createListItems({ tasks, newTask }: Options): {
 	entries: Entries;
 	newTaskIndex: number | undefined;
 } {
 	const now = browser.getNowDate();
 	const lastMidnight = browser.getLastMidnight(now);
+	const sortedTasks = sortTasks(tasks);
 
 	let lastTitle: string;
 	let newTaskIndex: number | undefined = undefined;