maximunited / yomu / 26062157198

84%

push

github

chore: add pre-commit hooks configuration (#40)

* chore: add pre-commit hooks configuration

Adds optional pre-commit framework support for automated code quality checks.

New files:
- .pre-commit-config.yaml: Configuration for pre-commit hooks
- .secrets.baseline: Baseline for secret detection (detect-secrets)

Updated files:
- .prettierignore: Exclude .secrets.baseline from formatting
- CONTRIBUTING.md: Add pre-commit installation and usage instructions

Pre-commit hooks include:
- Code formatting (Prettier)
- Linting (ESLint)
- Commit message validation (Commitlint)
- Secret detection (detect-secrets)
- File cleanup (trailing whitespace, end of file, etc.)
- Type checking and tests on pre-push

Note: Pre-commit is optional. The project already uses Husky + lint-staged
for similar functionality. This provides additional checks like secret detection.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* fix: address Qodo review findings for pre-commit config

Fixes 3 bugs identified by Qodo code review:

1. Missing ESLint dependency (@eslint/js)
   - Added @eslint/js@9.39.4 to ESLint hook's additional_dependencies
   - Prevents runtime failure when eslint.config.mjs imports @eslint/js

2. Pre-push hooks not installed by default
   - Added 'pre-push' to default_install_hook_types
   - Updated CONTRIBUTING.md to include pre-push installation command
   - Ensures typescript-check and jest-related hooks actually run

3. Prettier output inconsistent
   - Removed prettier-plugin-organize-imports from pre-commit config
   - Aligns pre-commit Prettier with standard formatting flow (npm run format)
   - Prevents formatting inconsistencies between pre-commit and lint-staged

All changes ensure pre-commit hooks work correctly and consistently
with the project's existing tooling.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* fix: resolve pre-commit hook failures

Fixes two pre-commit hook issues:

1. check-yaml invalid argument
   - Remo... (continued)

682 of 961 branches covered (70.97%)

Branch coverage included in aggregate %.

9993 of 11370 relevant lines covered (87.89%)

34.18 hits per line