Dana OkaforBackend engineer · Задание: исправить двойное списание · Claude Code · 2 ч 40 мин
Проверено Lemonsignal 89%Сильное соответствие
- Нашла причину, прежде чем дать ИИ писать исправление.
- Проверила исправление от ИИ тестом, а не поверила на слово.
- Исправила второй сервис тем, что создала для первого.
- Понимание Сильно Попросила воспроизвести двойное списание и прошла за одним платежом по коду, прежде чем что-либо исправлять. Реплика 2
- Постановка задачи Уверенно Дала логи и нужные файлы и сказала, что менять нельзя. Реплика 4
- Проверка Сильно Прогнала тесты на первом исправлении от ИИ и отклонила его. Своё исправление подтвердила тестом. Реплики 12, 19
- Исправление Сильно Объяснила, почему первое исправление неверно и где искать вместо этого. Реплика 13
- Приёмка Уверенно Небольшое изменение с тестом и заметка о том, что она не проверяла.
- Повторяемость Сильно Сохранила шаги как скилл и исправила с его помощью сервис возвратов.
Навыки для работы
- Поиск первопричиныВыше планки
- Подтверждение исправления тестомВыше планки
- Код, который смогут поддерживать другиеНа уровне планки
Что она сдала, в том виде, в каком сдала. Каждая пометка это навык из её результата.
Описание изменения Одно списание на заказ, даже когда шлюз повторяет запрос
- Причина: после тайм-аута шлюза повторный запрос создавал второе списание.
- Исправление: каждое списание несёт ключ своего заказа, поэтому повторный запрос возвращает первое списание, а не создаёт новое.
- Тест: повторный запрос после тайм-аута теперь даёт одно списание. Все тесты проходят (48).
- Не проверено: частичные списания, мы их пока не используем.
payments/charge.pyИсправление
def create_charge(order, card):
- return gateway.charge(order.total, card)
+ key = f"order-{order.id}"
+ return gateway.charge(order.total, card,
+ idempotency_key=key)
tests/test_charge.pyПадает до исправления, проходит после
+def test_retry_charges_once(gw, order, card):
+ gw.fail_next_with(Timeout)
+ create_charge(order, card)
+ create_charge(order, card) # the retry
+ assert gw.charges_for(order) == 1
refunds/refund.pyТо же исправление в сервисе возвратов
def create_refund(charge, amount):
- return gateway.refund(charge.id, amount)
+ key = f"refund-{charge.id}"
+ return gateway.refund(charge.id, amount,
+ idempotency_key=key)