Проведи дотошное code review кода на [ЯЗЫК/СТЕК: Python, FastAPI, PostgreSQL]. Не выпиливай в первой итерации идеальный код — разберись.
Контекст проекта: [О ПРОЕКТЕ: бэкенд интернет-магазина на FastAPI и PostgreSQL, команда из трёх разработчиков. Релизы раз в неделю, тестов мало]
Что делает этот код: [НАЗНАЧЕНИЕ: обработчик вебхука от платёжного шлюза — принимает уведомление об оплате и переводит заказ в статус «оплачен»]
Кто это написал: [АВТОР: джуниор]
Код:
```
[КОД: @app.post("/webhook/payment") async def payment_webhook(request: Request): data = await request.json() order = db.query(Order).filter(Order.id == data.get("order_id")).first() order.status = "paid" db.commit() return {"ok": True} ]
```
Требуется ревью по 5 уровням приоритета:
🔴 КРИТИЧНО (блокирующее на мерж): баги, утечки безопасности, потеря данных, performance regression более 10x
🟠 ВАЖНО (надо исправить, но не блокер): антипаттерны, грубые нарушения SOLID, дыры в обработке ошибок
🟡 ЖЕЛАТЕЛЬНО: лучшая читаемость, более идиоматичный подход, мелкая оптимизация
🟢 СТИЛЬ: именование, форматирование, комментарии (только если действительно мешает)
💡 ИДЕИ: альтернативные подходы, которые стоит обсудить, без обязательного применения
Для каждого замечания:
— номер строки или блок
— что не так и почему
— конкретный пример исправления
В конце — общая оценка кода по 10-балльной шкале и 3 главных совета автору как разработчику.
Собрать под себя — поля заполнены рабочими значениями, меняйте их и промпт выше обновится сам