Потому что useAction() возвращает не ту же самую функцию, а функцию забинденную на dispatch и выглядит это примерно вот так:

function bindActionCreator(actionCreator, dispatch) {
  return function () {
    return dispatch(actionCreator.apply(this, arguments));
  };
}

то есть по ключу fetchTodos лежит уже вызванная функция bindActionCreator(fetchTodos, dispatch) , и вызывая fetchTodos вы по сути вызываете вот этот участок:

function () {
    return dispatch(actionCreator.apply(this, arguments));
  };

надеюсь, понятно объяснил. Но лучше вывести самому в консоль лог и изучить.

Props это объект - у него нет функции map. Вытащи из пропса нужный массив и к нему примени map.

По вопросам:
1 Особой разницы нет, только для вашей IDE - она правильнее подсказки будет делать в файле jsx
2 Первое предупреждение о том, что используете key={todo.id} немного не там - нужно в родительский div это переместить

<div className="todos" вот сюда>
       <div key={todo.id} className="item-todo" >

второе предупреждение связано с тем, что хук в своем массиве зависимостей не видит вызываемую в нем функцию

useEffect(() => {
        getTodos()
    }, [вот тут])

пока можете поставить такую строку, чтобы не видеть ошибку

useEffect(() => {
        getTodos()
// eslint-disable-next-line
    }, [])

Если хотите разобраться с правильным использованием useEffect

Скорее всего вам нужен метод для раздачи статичных файлов:

app.use(express.static(path.join(__dirname, 'public')))

все файлы, лежащие в папке public (можно задать свою) могут быть получены клиентом. Нужно только сформировать список существующих файлов и выдавать их url на фронт.