Skip to content

Commit 5e29a5d

Browse files
committed
docs: add mandatory pre-completion checks to contribution guidelines
- Included formatting, linting, and compilation checks for JavaScript and Rust. - Specified tools and commands for verifying code quality before marking tasks complete.
1 parent c4b2615 commit 5e29a5d

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

CLAUDE.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -471,6 +471,12 @@ Key updates from recent commits (cd9ebcd to current):
471471

472472
## Key Patterns
473473

474+
- **MANDATORY: Pre-completion checks**: Before considering ANY task complete, ALWAYS run these checks and fix all errors:
475+
1. `npx prettier --check .` (formatting)
476+
2. `npx eslint .` (lint)
477+
3. `npx tsc --noEmit` (TypeScript)
478+
4. `cargo fmt --manifest-path src-tauri/Cargo.toml` (Rust formatting, if Rust files were changed)
479+
5. `cargo check --manifest-path src-tauri/Cargo.toml` (Rust compilation, if Rust files were changed)
474480
- **Code Quality**: ESLint and Prettier enforce code standards with Husky hooks. Use type-only imports (`import type`) and consolidate imports from same modules.
475481
- **No dynamic imports**: All imports must be static `import` statements at the top of the file. Do not use `await import()` or `import().then()` inside functions or code blocks. Use try/catch around Tauri API calls for non-Tauri environments instead.
476482
- **No localStorage**: Avoid `localStorage` and `sessionStorage`; use Redux (and persist) for app state. Remove any direct usage when working on affected code.

0 commit comments

Comments
 (0)