chore: add dart format to pre-commit check and CI (#296)

* chore: add dart format to pre-commit check and CI

Adds a `format` mise task (`dart format .`) and includes it in the
`check` pre-commit gate. CI gains a `dart format --set-exit-if-changed .`
step before `flutter analyze` so unformatted code is a hard failure on
PRs rather than a silent auto-fix.

The local task reformats in place; CI uses `--set-exit-if-changed` to
fail the build without modifying files.

* style: apply dart format to entire codebase

Bulk-format all Dart files with `dart format .`. No logic changes —
line wrapping, trailing whitespace, and indentation only.

* docs: document dart format --no-deps usage in AGENTS.md

* chore: add dart:format, prettier:format and mise:format tasks

Splits the single `format` task into three named sub-tasks:
- `dart:format` — dart format . (use --no-deps for speed after Dart changes)
- `prettier:format` — Prettier for JS/TS/MJS files
- `mise:format` — mise fmt for mise.toml

The top-level `format` task now depends on all three. CI check
formatting step expanded to cover all three. Prettier added to root
package.json; .prettierignore excludes build/, android/, ios/.

* style: apply prettier to JS/TS/MJS files

* fix: correct CI format check and dart:format task ordering

- Add --output=none to dart format CI check so it doesn't modify
  files (--set-exit-if-changed alone still writes)
- Remove mise fmt --check from CI: mise is not installed in the CI
  environment (setup-flutter-app uses subosito/flutter-action directly)
- Make dart:format depend on generate so mock files are formatted
  after codegen, not before

* style: format files introduced by PR #295 bug fixes

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: richardthe3rd

.github/workflows/ci.yml

@@ -65,6 +65,12 @@ jobs:
           google-services-json: ${{ secrets.GOOGLE_SERVICES_JSON }}
           generate-mocks: 'true'
 
+      - name: Check formatting
+        run: |
+          dart format --output=none --set-exit-if-changed .
+          npm ci
+          npx prettier --check "**/*.{js,ts,mjs}"
+
       - name: Analyze code
         run: flutter analyze --no-fatal-infos
 

.prettierignore

@@ -0,0 +1,3 @@
+build/
+android/
+ios/

AGENTS.md

@@ -72,6 +72,10 @@ MISE_ENV=dev ./bin/mise tasks ls      # All tasks including build/serve
 | Task | Command | Notes |
 |------|---------|-------|
 | **Pre-commit gate** | `./bin/mise run check` | **Run before every commit** |
+| **Format all code** | `./bin/mise run format` | Runs all three formatters below |
+| Format Dart | `./bin/mise run --no-deps dart:format` | **Run after every Dart change** — `--no-deps` skips unnecessary `pub get` |
+| Format JS/TS | `./bin/mise run prettier:format` | After JS/TS changes |
+| Format mise.toml | `./bin/mise run mise:format` | After editing mise.toml |
 | Generate code (mocks) | `./bin/mise run generate` | After model changes |
 | Analyze code | `./bin/mise run analyze` | generate → analyze |
 | Run tests | `./bin/mise run test` | generate → test |