flutter-dart-code-review

Use a structured checklist covering widget decomposition, const usage, state management discipline, performance, and security. This Skill provides 15 review categories with concrete checkboxes and Dart code examples for common pitfalls.

Check that business logic lives outside widgets, state objects are immutable with proper equality, async states model loading/success/error explicitly, and subscriptions are disposed. The checklist adapts rules to BLoC, Riverpod, Provider, GetX, MobX, and Signals.

Yes, the checklist is library-agnostic and includes a quick-reference table mapping universal principles to BLoC, Riverpod, Provider, GetX, MobX, Signals, and built-in state. Solution-specific conventions like ref.watch chains and BlocObserver are addressed.

Look for missing const constructors, expensive operations in build(), missing ListView.builder for large lists, unlocalized setState calls, and Opacity in animations. The performance section lists concrete checks for rebuilds, images, and lazy loading.

Verify tokens use platform-secure storage, API keys are not hardcoded in Dart source, user input is validated, HTTPS is enforced, and no sensitive data is logged. The security section covers storage, key handling, validation, and network concerns.