Skip to content

ADR — PR #1249

2026-05-28 16:00 — Break down lengthy rationale into readable bullet points

Section titled “2026-05-28 16:00 — Break down lengthy rationale into readable bullet points”
  • Comment: CodeRabbit nitpick comment on docs/adr/PR-1026.md lines 38-43
  • Priority: nice-to-have
  • Decision: accept
  • Rationale: Split the 500-word single-sentence rationale into concise bullet points for better readability, preserving all technical details about billing-summary.ts rewrite, leftJoin changes, trial state handling, toNumberOrNull helper, and BillingSummary.tsx UI logic.

2026-05-28 16:01 — Reject filename casing change for consistency

Section titled “2026-05-28 16:01 — Reject filename casing change for consistency”
  • Comment: CodeRabbit prompt for AI agents suggesting rename PR-1026.md → pr-1026.md and PR-1054.md → pr-1054.md
  • Priority: skip
  • Decision: reject
  • Rationale: Established ADR convention in docs/adr/ uses uppercase PR- prefix across 16 sibling files. Renaming only these two would create inconsistency; any casing change should apply to all files uniformly.