fix: Address review feedback on pre-commit hooks

franciscojavierarceo · claude · franciscojavierarceo · commit 42019da4f320 · 2026-02-10T12:07:38.000-05:00
- Format hook now runs both ruff check --fix AND ruff format
- Lint hook now runs both ruff check AND ruff format --check
- Template hook file filter now includes docs/roadmap.md

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,28 +4,30 @@ repos:
   - repo: local
     hooks:
       # File-aware format hook - only runs on changed Python files
+      # Runs both ruff check --fix and ruff format
       - id: format-files
         name: Format Changed Files
         stages: [commit]
         language: system
         types: [python]
-        entry: uv run ruff check --fix
+        entry: bash -c 'uv run ruff check --fix "$@" && uv run ruff format "$@"' --
         pass_filenames: true
 
       # File-aware lint hook - only runs on changed Python files
+      # Runs both ruff check and ruff format --check
       - id: lint-files
         name: Lint Changed Files
         stages: [commit]
         language: system
         types: [python]
-        entry: uv run ruff check
+        entry: bash -c 'uv run ruff check "$@" && uv run ruff format --check "$@"' --
         pass_filenames: true
 
-      # Conditional template hook - only runs when template files change
+      # Conditional template hook - only runs when template files or roadmap change
       - id: template
         name: Build Templates
         stages: [commit]
         language: system
-        files: ^infra/templates/|\.jinja2$
+        files: ^infra/templates/|\.jinja2$|^docs/roadmap\.md$
         entry: make build-templates
         pass_filenames: false
