-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Description
Objective
Run the pre-commit hooks, identify all linting errors, and fix every single one of them. Do NOT make any changes outside of what is needed to fix the linting errors -- no refactoring, no feature changes, no config changes.
Step 1: Setup
pip install pre-commit
npm ci
cd frontend && npm ci && cd ..
npm ci
cd frontend && npm ci && cd ..
Step 2: Identify Issues
pre-commit run --all-files
Read the output carefully. Note every failing hook, its errors, affected files, and line numbers.
Step 3: Fix All Errors
Fix every error reported. For semgrep security findings where the code is safe (e.g., hardcoded template paths, internally guarded endpoints), suppress with an inline # nosemgrep: comment after verifying the code is genuinely safe. For all other linting errors, fix the underlying code issue.
Step 4: Verify
Re-run until clean:
pre-commit run --all-files
Every hook must show Passed or Skipped. Zero hooks should show Failed.
Constraints
- Do NOT modify any config files (
.pre-commit-config.yaml,pyproject.toml,.semgrepignore,.flake8,eslint.config.js,package.json, etc.) - Only modify source files to fix the reported errors
- Do not break any existing functionality