ℹ️ No critical issues — two minor labeling/test nits inline.

Reviewed changes — mid-cycle Deploy plan changes now grant the correct usage credits by reusing the existing invoice-paid credit-grant webhook, with netDeployFee extracted as a pure, unit-tested function.

• Immediate prorated charge on upgrades — changeDeployPlan.ts switches proration_behavior to always_invoice for upgrades and none for downgrades, deriving direction from DEPLOY_PLANS.indexOf (the equal-plan case returns earlier, so < is sufficient).
• Self-correcting credit top-up — netDeployFee sums the invoice's Deploy plan-fee lines; the upgrade proration pair (+new fee, -unused old fee) nets to exactly the top-up, a downgrade nets negative (no grant, no clawback), and a renewal grants the full fee.
• Refactor of grantDeployCreditsForInvoice — fee summing, period-end, and plan resolution now flow through netDeployFee; behavior preserved.
• Unit tests — deployCredits.test.ts covers subscribe/renewal, upgrade netting, downgrade netting, unrelated lines, latest-period-end, and missing price details.
• Modal copy — deploy-product-card.tsx footnote and downgrade overage warning now describe the immediate-charge / keep-current-credits semantics.

I verified the load-bearing Stripe assumptions against current docs: always_invoice creates and collects a proration invoice synchronously, proration lines carry pricing.price_details.price and a period.end equal to the subscription period end (so the top-up expires with the current period), error_if_incomplete throws synchronously on payment failure, and invoice.payment_succeeded fires for the proration invoice. All correct. The money paths net correctly and the grant is idempotent per invoice; the only findings are cosmetic.

  ^{｜ Fix all ➔ ｜ Fix 👍s ➔ ｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}

✅ No new issues found.

Reviewed changes — the delta since the prior pullfrog review (3d91734) cleanly addresses both earlier nits via the netDeployFee extraction and added test coverage; the money paths are unchanged and remain correct.

• Plan labeled from the charge line — netDeployFee now derives plan from the largest-amount fee line (chargeLine via reduce), so an upgrade's credit name and deploy_plan metadata reflect the plan the customer ends up on regardless of Stripe's proration line ordering.
• Test coverage for the previously-uncovered paths — split the old test into "ignores metered lines" plus a genuine null-pricing case ("ignores lines without price details"), added "labels the plan from the charge line regardless of proration order", and added a discount_amounts coupon test.
• Behavior-preserving refactor — grantDeployCreditsForInvoice now delegates fee summing, period-end (Math.max), and plan resolution to the pure netDeployFee; the net amount, discount subtraction, and expiry logic are identical to the inlined version.

The chargeLine heuristic is a labeling-only concern — amountCents is still summed independently over all fee lines, so the net top-up math is unaffected by which line wins the reduce. Both prior review threads have been resolved.

  ^{｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}