Flutter/Dart Code Review Best Practices

Comprehensive, library-agnostic checklist for reviewing Flutter/Dart applications.

1. General Project Health

• Project follows consistent folder structure (feature-first or layer-first)
• Proper separation of concerns: UI, business logic, data layers
• No business logic in widgets; widgets are purely presentational
• analysis_options.yaml includes a strict lint set with strict analyzer settings enabled

2. Dart Language Pitfalls

• Missing type annotations leading to dynamic -- enable strict-casts, strict-inference, strict-raw-types
• Null safety misuse: Excessive ! (bang operator) instead of proper null checks
• Catching too broadly: catch (e) without on clause
• late overuse where nullable or constructor initialization would be safer
• Prefer final for locals and const for compile-time constants
• Use package: imports for consistency