flutter-dart-code-review · git:20260610.5639e7f · 2026-06-10 · sha256 89b2a80ba0db071e
flutter-dart-code-review git:20260610.5639e7fA
Immutable. This exact content never changes and is served at /api/v1/blob/89b2a80ba0db071e.
--- name: flutter-dart-code-review description: Library-agnostic Flutter/Dart code review checklist covering widget best practices, state management patterns (BLoC, Riverpod, Provider, GetX, MobX, Signals), Dart idioms, performance, accessibility, security, and clean architecture. mcp: code-review-graph --- # /supergraph:flutter-dart-code-review Library-agnostic Flutter/Dart code review. Graph-first, then checklist. Announce: "🐦 /supergraph:flutter-dart-code-review — reviewing Flutter/Dart code..." ## Steps ### 1. Graph analysis (surface hotspots before manual review) ``` mcp__code-review-graph__find_large_functions_tool() # build() > 80-100 lines mcp__code-review-graph__get_hub_nodes_tool() # widgets with 10+ dependents mcp__code-review-graph__query_graph_tool(query_type="cycles") # circular DI deps mcp__code-review-graph__get_surprising_connections_tool() # unexpected layer coupling mcp__code-review-graph__get_knowledge_gaps_tool() # untested files ``` **Serena (optional — if available):** ``` mcp__serena__get_diagnostics_for_file(<file>) # type errors, null safety violations mcp__serena__find_referencing_symbols(<symbol>) # callers of changed public APIs mcp__serena__find_implementations(<interface>) # subclasses after interface change ``` Skip if Serena unavailable. ### 2. Manual checklist review Work through [CHECKLIST.md](./CHECKLIST.md) — 15 sections covering: General health · Dart pitfalls · Widgets · State management · Performance · Testing · Accessibility · Platform · Security · Dependencies · Navigation · Error handling · l10n · DI · Static analysis Focus on areas flagged by step 1 first. ### 3. Report ``` ## Flutter/Dart Review - Graph hotspots: [list/none] - Serena diagnostics: [issues/clean/skipped] - Checklist: PASS | ISSUES (list critical items) - Next: /supergraph:fix → /supergraph:verify ``` ## Rules - Run graph analysis before opening checklist — prioritize hotspots - Serena diagnostics catch null safety / type errors manual review misses - State management rules apply to ALL solutions (BLoC, Riverpod, GetX, MobX, Signals) - Full checklist in [CHECKLIST.md](./CHECKLIST.md)