В компоненте мы описываем функцию, которая возвращает JSX. Почему это может быть не очень хорошим решением?

Обычная функция, возвращающая JSX, допустима; проблема возникает при скрытых зависимостях или создании нового типа компонента на каждом рендере.

Если вложенную функцию рендерят как <Child />, её новая идентичность может приводить к размонтированию и потере state. Небольшой чистый render-helper не равен этому случаю, но хуки внутри произвольного helper вызывать нельзя.