@@ -3,27 +3,29 @@ alwaysApply: true
33---
44
55Proceed in small iterative steps and test each change.
6- Do not block for approvals; follow `docs/SPEC.md` and keep momentum.
6+ Refactor the code to make it more elegant, concise and simple. It should be clear and easy to maintain.
7+ Improve the tests as you go, make sure the tests are clear and focused. Don't create lots of trivial tests.
8+ Improve the docs and diagramsas you go.
9+ Remove dead code.
10+ Use whitespace to make the code more readable.
11+
12+ Do not block for approvals; follow `docs/SPEC.md` and `docs/TODO.md` and keep momentum.
713Prefer WebGPU via `wgpu` v24.0 and avoid WebGL2 fallbacks.
814Prefer Node for servers/tooling; avoid Python for this project.
9- Default to pure web flow first; keep native in parity where simple.
10-
11- Pre-commit and push procedure:
12-
13- - Update `README.md` and `docs/TODO.md`
14- - Run `npm run check` to format, lint, build, and execute headless tests.
15- - Ensure the check is green (no warnings promoted to errors, builds succeed, tests pass).
16- - Commit and push only after the above succeed.
1715
1816Testing guidelines:
1917
20- - Rust unit tests live alongside code in `crates/app-core` (and other crates as added) . Use `cargo test`.
18+ - Rust unit tests live alongside code. Use `cargo test`.
2119- Headless web test is executed via `npm run ci:web` which builds the WASM, serves it, and runs Puppeteer (`web-test.js`).
2220- The `npm run check` command runs:
2321 - `cargo fmt --check` and `cargo clippy -D warnings` for formatting/lint
2422 - `cargo test` (workspace, excluding `app-web`)
25- - `cargo build -p app-native`
2623 - Web CI build + headless test
27- - Add tests for:
28- - Core logic invariants (e.g., frequency math, event scheduling, mute/solo behavior)
29- - Regressions for reported bugs
24+
25+ Pre-commit and push procedure:
26+
27+ - Update `README.md`, `docs/TODO.md` and `docs/SPEC.md`
28+ - Run `npm run check` to format, lint, build, and execute headless tests.
29+ - Ensure the check is green (no warnings promoted to errors, builds succeed, tests pass).
30+ - Also try and fix warnings.
31+ - Commit and push only after the above succeed.
0 commit comments