涉及支付的 PR 审查清单
面向具体领域的审查清单——支付、鉴权、基础设施——让审查者不漏掉显而易见的问题
查看输入和输出预览
输入
- Language
- TypeScript
- Domain
- payments
- Risk
- high
输出(节选)
## Code review checklist -- payments PR (high risk) - [ ] All money handled as integer cents (no float arithmetic anywhere in the diff). - [ ] Currency code travels alongside every amount (no implicit USD). - [ ] Idempotency key required on every write to the gateway; reuse checked against a 24h store. - [ ] PII (card last4, billing address) never logged, including in error stack traces. - [ ] DB writes wrapped in a transaction with the gateway call after commit. - [ ] Tests cover: success, 3DS challenge, declined, gateway timeout, partial refund, double-submit replay. - [ ] Feature flag in place; rollout plan documented in PR description.