роут у тебя по сути и есть конроллер

Сейчас бы в контроллере бизнес логику сувать

при чем тут бизнес логика и проверка, что прилетевшее поле логина не пустое

Не пустое поле, и длина от 10 символов это и есть бизнес логика

от 50/50 всегда
и хз как правильно

перехват ошибок кривой немного, класс странный, вроде он как обертка для JWT,  а вроде и логинит и валидирует и создает нового юзера, а может и на дудке будет играть в следующем комите, сомнительно все сделано

это требование к входящим данным и не более того...

Валидация данных и есть задача бизнес логики. И не важно, это валидация длины или наличия названия фирмы в списке контрагентов

я за то, чтобы проверку подтверждения паролей тоже перенести в сервис. и на этом можно остановиться.

но если очень хочется, то можно в контроллер добавить валидацию req.body, т.е. убедиться, что body содержит только нужные тебе поля и эти поля в нужном формате (строка/число/и т.п.). а сервис может дополнительно проверять, что email -- это почта, что пароли совпадают и т.п.

кто-то может сказать, что это двойная проверка, но разница есть.

аха
хм насчёт обертки для жвт по делу
надо подумать
та я хз, просто пишу пока как умею

контроллер подготавливает данные для бизнес-логики, а она их проверяет

плюс выше звучало желание абстрагировать уровни абстракции, но прокидывание body в "контроллер" этому явно ен способствует

а еще очень режет глаз, когда пихают await в свойства объекта или в if, но это наверное больше имхо

та просто переменную не было смысла создавать
лишняя строка

А потом ты захочешь через cli создавать пользователя, или через email. Сделаешь ещё один контроллер и скопипастишь проверку полей туда?

для того, чтобы контроллер подготовил данные для бизнеслогики, он должен знать их структуру... т.е. что-то не очень вяжется с абтрагированием

не, это норм

лишняя строка = больше нагрузки на мозг при попытке понять что за нахер тут происходит, не надо экономить строки, через полгода, когда будешь рефакторить скажешь себе спасибо за то что понятно написано, что и откуда и в каком формате приходит

контроллер знает какие данные ему пришли. и так же контроллер знает в каком формате бизнес-логика ожидает данные.

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