Hold
feat(orders): add idempotency keys to order creation
Do not merge yet
15 files changed (+480 −102). 65 of 582 lines carry the risk; read them in about 20 minutes, and skim or skip the other 517. Written by Claude Code; no person has committed on top of it.
- 3 serious findings: tls verification turned off, destructive schema change, sql built from interpolated values.
- Adds 1 outbound destination.
- Beyond its description: Schema migrations, Authentication, Infrastructure.
- Imports express-idempotency-guard, which the registry has never heard of.
- 2 source files changed with no test change, led by idempotency.ts.
- Tests weakened: 1 test skipped.
- formatCents() is written 2 times with near-identical bodies: extract one shared helper.
+480−10215 filesWritten by Claude CodeIntent: partial2 live resources
Why hold: 2 source files changed without a matching test change. Depends on express-idempotency-guard, which the registry does not know — a hallucinated or squatted name. Risk 100/100. Policy: default.
You have 20 minutes
A careful read of everything: 2.1 hours65 lines to read190 to skim327 to skip