修正はプルリクエストです。 Polylaneが書き、マージするのはチームです。
ウェイトリストに登録 github.com/coreplane/payments-api/pull/491
Cap retries on the checkout webhook worker #491
polylane Open polylane wants to merge 1 commit into
main from polylane/autofix/chat/k3x9f2-4e7d21a Conversation 1 Commits 1 Checks 1 Files changed 2
polylane bot commented 6 minutes ago ···
Retries on the checkout webhook worker were unbounded: a failing delivery re-queued itself forever and amplified load on payments-api. This caps delivery at 5 attempts with exponential backoff and dead-letters the payload after the last one.
What changed
worker/deliver.ts gains MAX_DELIVERY_ATTEMPTS = 5 and backoff between attempts; exhausted payloads land in checkout-webhooks-dlq instead of re-queueing.
Validation
npm test — 214 passed. A forced failing delivery stopped after 5 attempts and appeared in the dead-letter queue.
Root cause · Why it's safe · Out of scope
polylane added commit 4e7d21a Verified
Review required At least 1 approving review is required
ci / test Successful in 3m 12s Details
Review required Waiting on your review: Polylane never merges on its own
Merging is blocked
自分で書かなくてよかった修正。 そして、それを信頼できる理由。
-
チームメイトのように書く
サンドボックスの中で、ブランチ上で、コミット規約とリポジトリの指示に従って書きます。
-
開く前に検査する
安全性レビューが、正しい差分でもデプロイを壊し得る経路を探します。ロックを取るマイグレーションや、バージョン混在による破損などです。
-
レビューで止まる
マージを決めるのはレビューとCIです。それ以外にはありません。
-
「いいえ」から学ぶ
プルリクエストをマージせずに閉じると、Polylaneはその理由を突き止め、同じことを繰り返しません。