fix(hooks): add dart fix step and worktree-safe wrappers

[runyaga]

Summary

• Add dart fix --apply as pre-commit hook step before dart format
• Add wrapper scripts that unset GIT_DIR for worktree compatibility

Changes

• .pre-commit-config.yaml: New dart-fix hook runs first; all hooks use scripts/ wrappers with language: script
• scripts/dart-fix.sh: Unsets GIT_DIR, runs dart fix --apply
• scripts/dart-format.sh: Unsets GIT_DIR, runs dart format
• scripts/flutter-analyze.sh: Unsets GIT_DIR, runs flutter analyze

Problem

dart format wraps long argument lists to multiple lines but does not add trailing commas. flutter analyze --fatal-infos then rejects the result due to require_trailing_commas. This creates a cycle where format produces code that analyze rejects.

Additionally, git worktrees set GIT_DIR which breaks Flutter's SDK version detection (reports 0.0.0-unknown), causing flutter analyze to fail with version resolution errors.

Solution

Hook order: dart fix → dart format → flutter analyze

dart fix --apply inserts trailing commas (and other auto-fixable lints) before dart format touches whitespace. Wrapper scripts unset GIT_DIR for worktree compatibility.

Test plan

• Pre-commit hooks pass on commit
• Hooks work from git worktrees (unset GIT_DIR)
• Verify hooks on fresh clone (manual)

Generated with Claude Code

[WilliamKarolDiCioccio]

@runyaga the pre-commit hooks fail on Windows as they're calling .sh scripts. We need either to set the language to system and use platform agnostic commands or set it to Python and change the scripts. I'd go for the latter.

[WilliamKarolDiCioccio]

@runyaga We will need to run pre-commit --all-files before merging this. I suggest waiting for the weekend.