Dana OkaforBackend engineer · Compito: correggere un doppio addebito · Claude Code · 2 h 40 min
Verificato da Lemonsignal 89%Forte corrispondenza
- Ha trovato la causa prima di lasciare che l’AI scrivesse una correzione.
- Ha testato la correzione dell’AI invece di fidarsi.
- Ha corretto il secondo servizio con ciò che aveva costruito per il primo.
- Capire Forte Le ha fatto riprodurre il doppio addebito e ha seguito un pagamento nel codice prima di qualsiasi correzione. Turno 2
- Dare il brief Solido Le ha dato i log e i file che contano, e ha detto che cosa non doveva cambiare. Turno 4
- Verificare Forte Ha eseguito i test sulla prima correzione dell’AI e l’ha respinta. Ha dimostrato la propria correzione con un test. Turni 12, 19
- Correggere Forte Le ha spiegato perché la prima correzione era sbagliata e dove cercare invece. Turno 13
- Approvare Solido Una piccola modifica con il suo test e una nota su ciò che non ha testato.
- Riutilizzare Forte Ha salvato i passaggi come skill e l’ha usata per correggere il servizio dei rimborsi.
Competenze per il ruolo
- Trovare la causa principaleSopra la soglia
- Dimostrare una correzione con un testSopra la soglia
- Codice che altri possono mantenereAlla soglia
Che cosa ha consegnato, così come l’ha consegnato. Ogni segno è una competenza del suo risultato.
Nota di modifica Un solo addebito per ordine, anche se il gateway riprova
- Causa: dopo un timeout del gateway, il nuovo tentativo creava un secondo addebito.
- Correzione: ogni addebito porta una chiave del suo ordine, così un nuovo tentativo restituisce il primo addebito invece di crearne uno nuovo.
- Test: un nuovo tentativo dopo un timeout ora addebita una sola volta. Tutti i 48 test passano.
- Non testato: le catture parziali, che non usiamo ancora.
payments/charge.pyLa correzione
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.pyFallisce prima della correzione, passa dopo
+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.pyLa stessa correzione nel servizio dei rimborsi
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)