Ana extracted a shared retry middleware and added a redaction unit test that the security team signed off on.
task
ticketacme/worker-svc#412submitted by@anacreated12d ago
plansigned
Refactor retry logic into a shared @acme/retry middleware so other workers can adopt it.
Add structured logs via @acme/log with redacted PII.
Backfill tests for the exponential backoff jitter.
findings2
simplicityaddressedf_1
Extracts retry primitive cleanly without changing call sites
The middleware shape preserves existing call sites and only adds an opt-in wrapper. Low blast radius.
compliancedefendedf_2
Confirm redaction list covers payment metadata
Payment metadata can contain partial card BIN. Verify the redactor's allowlist excludes that field before merging.
Audited with security: redactor is keyed on a denylist that includes all payment.* fields. Added a unit test asserting redaction.