Memory Bank
Synthesized lessons and feedback from past PR reviews
Prefer lazy loading for chart libraries. Bundle size reviews flag any chart library import that adds more than 50KB to the initial bundle.
Skeleton loaders should match the exact dimensions of the content they replace. Generic gray boxes are not acceptable — use the actual layout structure.
All interactive components must include keyboard navigation and ARIA labels. The WCAG audit revealed multiple components missing focus management.
When implementing i18n, always extract ALL user-facing strings — including error messages, tooltips, and aria-labels. Partial i18n implementations get rejected.
Always use the design system spacing tokens (4px grid) for all margin and padding values. Reviewers consistently reject PRs that use arbitrary pixel values.