Tooling & CI
Priority: P1 (HIGH)
Implementation Guidelines
- Linter: Use
analysis_options.yaml. Enforce always_use_package_imports and require_trailing_commas.
- Formatting: Use
dart format . --line-length 80. Run on every commit.
- DCM: Use
dart_code_metrics for complexity checks (Max cyclomatic complexity: 15).
- Build Runner: Always use
--delete-conflicting-outputs with code generation.
- CI Pipeline: All PRs MUST pass
analyze, format, and test steps.
- Imports: Group imports:
dart:, package:, then relative.
- Documentation: Use
/// for public APIs. Link symbols using [Class].
- Linting Commands:
flutter analyze --fatal-infos --fatal-warnings
dart run dart_code_metrics:metrics analyze lib
- Pre-commit: Keep
lefthook.yml in sync with analyze/format/metrics commands.
Code
# analysis_options.yaml
analyzer:
errors:
todo: ignore
missing_required_param: error
linter:
rules:
- prefer_single_quotes
- unawaited_futures
Anti-Patterns
- No build_runner without --delete-conflicting-outputs: Causes stale generated file conflicts that break compilation.
- No flutter build before flutter analyze: Analyze fast; always fail fast before building.
- No ignore comment without explanation: Always annotate why lint ignore justified.
- No skipping dart format in pre-commit: Unformatted code breaks CI; enforce via
lefthook.yml.
References