Code review

[BigDeepBlue]

Здравствуйте ! Хорошо поработали, аккуратный код, постарались с архитектурой. ! Есть совсем небольшие замечания:

1. Тут немного не понял зачем обернули в dataclass ?
2. Тут я бы рекомендовал придержаться PEP8 https://peps.python.org/pep-0008/#method-names-and-instance-variables при именовании полей.
3. Тут использовали драйвер psycopg (вижу что 3й) , тут https://docs.sqlalchemy.org/en/20/orm/extensions/asyncio.html описан подход с asyncpg и на практике я использовал именно его. Если можно отпишите в коментах к тикету почему не взяли asyncpg и если можно ссылки где можно почитать что эта связка удачно работает. Есть небольшие сомнения https://github.innominds.com/psycopg/psycopg/issues/527 и насколько я знаю как реализована асинхронность в алхимии есть шанс что связка может быть проблемной. (но не утверждаю, сам пользуюсь проверенным asyncpg)
4. Тут можно тоже использовать HTTPStatus как в остальных случаях.

[mspiridonov2706]

@BigDeepBlue
Здравствуйте! Спасибо за ревью.

1. Что бы не писать

def __init__(self, session: Session) -> None:
    self._session = session

Так просто компактнее и более симпатично получается. Особенно если нужно много параметров передать

2. Поправили

3. Честно говоря не знал про возможные проблемы с psycopg. Мы его использовали, чтобы иметь возможность работать с базой в асинхронном режиме (в fastapi приложении) и в синхронном (в worker’е) без добавления дополнительного драйвера в зависимости проекта

4. Поправили