Skip to content

Commit d525e03

Browse files
committed
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/.
1 parent 64bb746 commit d525e03

6 files changed

Lines changed: 53 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,11 @@ jobs:
6666
generate-mocks: 'true'
6767

6868
- name: Check formatting
69-
run: dart format --set-exit-if-changed .
69+
run: |
70+
dart format --set-exit-if-changed .
71+
npm ci
72+
npx prettier --check "**/*.{js,ts,mjs}"
73+
mise fmt --check
7074
7175
- name: Analyze code
7276
run: flutter analyze --no-fatal-infos

.prettierignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
build/
2+
android/
3+
ios/

AGENTS.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,10 @@ MISE_ENV=dev ./bin/mise tasks ls # All tasks including build/serve
7272
| Task | Command | Notes |
7373
|------|---------|-------|
7474
| **Pre-commit gate** | `./bin/mise run check` | **Run before every commit** |
75-
| **Format code** | `./bin/mise run --no-deps format` | **Run after every change**`--no-deps` skips unnecessary `pub get` |
75+
| **Format all code** | `./bin/mise run format` | Runs all three formatters below |
76+
| Format Dart | `./bin/mise run --no-deps dart:format` | **Run after every Dart change**`--no-deps` skips unnecessary `pub get` |
77+
| Format JS/TS | `./bin/mise run prettier:format` | After JS/TS changes |
78+
| Format mise.toml | `./bin/mise run mise:format` | After editing mise.toml |
7679
| Generate code (mocks) | `./bin/mise run generate` | After model changes |
7780
| Analyze code | `./bin/mise run analyze` | generate → analyze |
7881
| Run tests | `./bin/mise run test` | generate → test |

mise.toml

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ _.path = ["./bin"]
1717

1818
[tools]
1919
flutter = "3.38.3"
20-
node = "22" # For http_server and Playwright e2e tests
20+
node = "22" # For http_server and Playwright e2e tests
2121

2222
[deps.flutter]
2323
auto = true
@@ -44,10 +44,23 @@ echo "Grep with: grep -n 'FAILED\|ERROR' $TEST_LOG"
4444
exit $EXIT_CODE
4545
'''
4646

47-
[tasks.format]
47+
[tasks."dart:format"]
4848
description = "Format all Dart code in place"
4949
run = 'dart format .'
5050

51+
[tasks."prettier:format"]
52+
description = "Format JS/TS/MJS files with Prettier"
53+
run = 'npm ci && npx prettier --write "**/*.{js,ts,mjs}"'
54+
55+
[tasks."mise:format"]
56+
description = "Format mise.toml"
57+
run = 'mise fmt'
58+
59+
[tasks.format]
60+
description = "Format all code (Dart, JS/TS, mise.toml)"
61+
depends = ['dart:format', 'prettier:format', 'mise:format']
62+
run = 'echo "All formatting complete"'
63+
5164
[tasks.check]
5265
description = "Pre-commit gate: generate → format + analyze + test (run before every commit)"
5366
depends = ['format', 'analyze', 'test']
@@ -91,5 +104,12 @@ run = 'npm ci && node ../scripts/validate-festivals.js'
91104
[tasks."test:worker"]
92105
description = "Run Cloudflare Worker tests (Vitest + workerd)"
93106
dir = "cloudflare-worker"
94-
sources = ['cloudflare-worker/package.json', 'cloudflare-worker/package-lock.json', 'cloudflare-worker/worker.js', 'cloudflare-worker/test/**/*.js', 'cloudflare-worker/vitest.config.js', 'data/festivals.json']
107+
sources = [
108+
'cloudflare-worker/package.json',
109+
'cloudflare-worker/package-lock.json',
110+
'cloudflare-worker/worker.js',
111+
'cloudflare-worker/test/**/*.js',
112+
'cloudflare-worker/vitest.config.js',
113+
'data/festivals.json',
114+
]
95115
run = 'npm ci && npm test'

package-lock.json

Lines changed: 17 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
"devDependencies": {
1313
"@playwright/test": "^1.48.2",
1414
"http-server": "^14.1.1",
15+
"prettier": "^3.0.0",
1516
"tsx": "^4.19.2"
1617
}
1718
}

0 commit comments

Comments
 (0)