hooks mutations - switch to useMemo

TodoProvider.jsx

// main change - change to use useMemo which means existingTodos is the todos, not a function.
const existingTodos = useMemo(() => {
  const todoMap = new Map();
  apiResponse?.todos.forEach((todo) => {
    todoMap.set(todo.id, todo);
  });

  return todoMap;
}, [apiResponse]);

useEffect(() => {
  // removed `existingTodos` from being a function
  if (!draftTodos && existingTodos.size) {
    setTodoList(existingTodos);
  }
}, [existingTodos]);