fix: Correct pre-commit template path in PrecommitHooksFixer (ambient-code#269)

* fix precommit hook template path reference in PrecommitHooksFixer

Signed-off-by: Rahul Shetty <rashetty@redhat.com>

* fix format issue

Signed-off-by: Rahul Shetty <rashetty@redhat.com>

---------

Signed-off-by: Rahul Shetty <rashetty@redhat.com>

Author: rh-rahulshetty

src/agentready/assessors/documentation.py

@@ -453,8 +453,7 @@ def _create_remediation(self) -> Remediation:
             ],
             tools=[],
             commands=[],
-            examples=[
-                """# Project Name
+            examples=["""# Project Name
 
 ## Overview
 What this project does and why it exists.
@@ -477,8 +476,7 @@ def _create_remediation(self) -> Remediation:
 # Format code
 black .
 ```
-"""
-            ],
+"""],
             citations=[
                 Citation(
                     source="GitHub",

src/agentready/assessors/testing.py

@@ -286,8 +286,7 @@ def _create_remediation(self) -> Remediation:
                 "pre-commit install",
                 "pre-commit run --all-files",
             ],
-            examples=[
-                """# .pre-commit-config.yaml
+            examples=["""# .pre-commit-config.yaml
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.4.0
@@ -306,8 +305,7 @@ def _create_remediation(self) -> Remediation:
     rev: 5.12.0
     hooks:
       - id: isort
-"""
-            ],
+"""],
             citations=[
                 Citation(
                     source="pre-commit.com",

src/agentready/fixers/testing.py

@@ -46,10 +46,11 @@ def generate_fix(self, repository: Repository, finding: Finding) -> Optional[Fix
         # Try to load language-specific template, fallback to python
         try:
             template = self.env_bootstrap.get_template(
-                f"precommit-{primary_lang}.yaml.j2"
+                f"{primary_lang}/precommit.yaml.j2"
             )
         except Exception:
-            template = self.env_bootstrap.get_template("precommit-python.yaml.j2")
+            # Template doesn't exist, fallback to python
+            template = self.env_bootstrap.get_template("python/precommit.yaml.j2")
 
         content = template.render()
 

src/agentready/services/assessment_cache.py

@@ -33,8 +33,7 @@ def _initialize_db(self) -> None:
         """Initialize database schema."""
         try:
             with sqlite3.connect(self.db_path) as conn:
-                conn.execute(
-                    """
+                conn.execute("""
                     CREATE TABLE IF NOT EXISTS assessments (
                         id INTEGER PRIMARY KEY AUTOINCREMENT,
                         repository_url TEXT NOT NULL,
@@ -45,23 +44,18 @@ def _initialize_db(self) -> None:
                         expires_at TIMESTAMP,
                         UNIQUE(repository_url, commit_hash)
                     )
-                    """
-                )
+                    """)
 
                 # Create index for faster queries
-                conn.execute(
-                    """
+                conn.execute("""
                     CREATE INDEX IF NOT EXISTS idx_repo_commit
                     ON assessments(repository_url, commit_hash)
-                    """
-                )
+                    """)
 
-                conn.execute(
-                    """
+                conn.execute("""
                     CREATE INDEX IF NOT EXISTS idx_expires_at
                     ON assessments(expires_at)
-                    """
-                )
+                    """)
 
                 conn.commit()
         except sqlite3.Error as e:

tests/e2e/test_critical_paths.py

@@ -276,14 +276,12 @@ def test_assess_with_valid_config(self):
         with tempfile.TemporaryDirectory() as tmp_dir:
             # Create valid config file
             config_file = Path(tmp_dir) / "config.yaml"
-            config_file.write_text(
-                """
+            config_file.write_text("""
 weights:
   claude_md: 2.0
 excluded_attributes:
   - repomix_config
-"""
-            )
+""")
 
             output_dir = Path(tmp_dir) / "output"
 

tests/e2e/test_critical_paths_simplified.py

@@ -219,14 +219,12 @@ def test_valid_config_application(self, temp_output_dir):
         with tempfile.TemporaryDirectory() as tmp_dir:
             # Create valid config
             config_file = Path(tmp_dir) / "config.yaml"
-            config_file.write_text(
-                """
+            config_file.write_text("""
 weights:
   claude_md: 2.0
 excluded_attributes:
   - repomix_config
-"""
-            )
+""")
 
             # Run assessment with config
             result = helper.run_assessment(

tests/unit/cli/test_main.py

@@ -355,14 +355,12 @@ class TestConfigLoading:
     def test_load_config_valid_yaml(self, tmp_path):
         """Test loading valid config file."""
         config_file = tmp_path / "config.yaml"
-        config_file.write_text(
-            """
+        config_file.write_text("""
 weights:
   claude_md_file: 2.0
 excluded_attributes:
   - test_attribute
-"""
-        )
+""")
 
         config = load_config(config_file)
 

tests/unit/test_assessors_code_quality.py

@@ -69,14 +69,12 @@ def test_python_pylint_configured(self, tmp_path):
 
         # Create .pylintrc
         pylintrc = tmp_path / ".pylintrc"
-        pylintrc.write_text(
-            """[MASTER]
+        pylintrc.write_text("""[MASTER]
 max-line-length=100
 
 [MESSAGES CONTROL]
 disable=C0111
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -105,11 +103,9 @@ def test_python_ruff_configured(self, tmp_path):
 
         # Create ruff.toml
         ruff_toml = tmp_path / "ruff.toml"
-        ruff_toml.write_text(
-            """line-length = 100
+        ruff_toml.write_text("""line-length = 100
 select = ["E", "F", "W"]
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -135,14 +131,12 @@ def test_python_pyproject_toml(self, tmp_path):
 
         # Create pyproject.toml with both tools
         pyproject = tmp_path / "pyproject.toml"
-        pyproject.write_text(
-            """[tool.pylint]
+        pyproject.write_text("""[tool.pylint]
 max-line-length = 100
 
 [tool.ruff]
 line-length = 100
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -171,15 +165,13 @@ def test_javascript_eslint_configured(self, tmp_path):
 
         # Create .eslintrc.json
         eslintrc = tmp_path / ".eslintrc.json"
-        eslintrc.write_text(
-            """{
+        eslintrc.write_text("""{
   "extends": "eslint:recommended",
   "rules": {
     "no-console": "warn"
   }
 }
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -234,14 +226,12 @@ def test_ruby_rubocop_configured(self, tmp_path):
 
         # Create .rubocop.yml
         rubocop = tmp_path / ".rubocop.yml"
-        rubocop.write_text(
-            """AllCops:
+        rubocop.write_text("""AllCops:
   TargetRubyVersion: 3.0
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -269,14 +259,12 @@ def test_go_golangci_lint_configured(self, tmp_path):
 
         # Create .golangci.yml
         golangci = tmp_path / ".golangci.yml"
-        golangci.write_text(
-            """linters:
+        golangci.write_text("""linters:
   enable:
     - gofmt
     - golint
     - govet
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -311,14 +299,12 @@ def test_actionlint_in_precommit(self, tmp_path):
 
         # Create .pre-commit-config.yaml with actionlint
         precommit = tmp_path / ".pre-commit-config.yaml"
-        precommit.write_text(
-            """repos:
+        precommit.write_text("""repos:
   - repo: https://github.com/rhysd/actionlint
     rev: v1.6.0
     hooks:
       - id: actionlint
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -344,13 +330,11 @@ def test_markdownlint_configured(self, tmp_path):
 
         # Create .markdownlint.json
         markdownlint = tmp_path / ".markdownlint.json"
-        markdownlint.write_text(
-            """{
+        markdownlint.write_text("""{
   "default": true,
   "MD013": false
 }
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,

tests/unit/test_assessors_containers.py

@@ -98,8 +98,7 @@ def test_multi_stage_build(self, tmp_path):
 
         # Create multi-stage Dockerfile
         dockerfile = tmp_path / "Dockerfile"
-        dockerfile.write_text(
-            """FROM node:18 AS builder
+        dockerfile.write_text("""FROM node:18 AS builder
 WORKDIR /app
 COPY . .
 RUN npm ci && npm run build
@@ -108,8 +107,7 @@ def test_multi_stage_build(self, tmp_path):
 WORKDIR /app
 COPY --from=builder /app/dist ./dist
 CMD ["node", "dist/index.js"]
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -138,15 +136,13 @@ def test_docker_compose(self, tmp_path):
 
         # Create docker-compose.yml
         compose = tmp_path / "docker-compose.yml"
-        compose.write_text(
-            """version: '3.8'
+        compose.write_text("""version: '3.8'
 services:
   app:
     build: .
     ports:
       - "8000:8000"
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -177,15 +173,13 @@ def test_dockerignore_file(self, tmp_path):
 
         # Create .dockerignore
         dockerignore = tmp_path / ".dockerignore"
-        dockerignore.write_text(
-            """.git
+        dockerignore.write_text(""".git
 .venv
 __pycache__
 *.pyc
 .env
 node_modules
-"""
-        )
+""")
 
         repo = Repository(
             path=tmp_path,
@@ -239,14 +233,12 @@ def test_comprehensive_container_setup(self, tmp_path):
         subprocess.run(["git", "init"], cwd=tmp_path, capture_output=True, check=True)
 
         # Multi-stage Dockerfile
-        (tmp_path / "Dockerfile").write_text(
-            """FROM python:3.12 AS builder
+        (tmp_path / "Dockerfile").write_text("""FROM python:3.12 AS builder
 RUN pip install build
 
 FROM python:3.12-slim
 COPY --from=builder /app /app
-"""
-        )
+""")
 
         # docker-compose.yml
         (tmp_path / "docker-compose.yml").write_text(