Fix formatting and linting issues

Author: sinaatalay

.claude/settings.json

@@ -0,0 +1,17 @@
+{
+    "hooks": {
+        "PostToolUse": [
+            {
+                "matcher": "Edit|Write",
+                "hooks": [
+                    {
+                        "type": "command",
+                        "command": "cd \"$CLAUDE_PROJECT_DIR\" && just check",
+                        "timeout": 120,
+                        "statusMessage": "Running just check...",
+                    }
+                ],
+            }
+        ]
+    }
+}

scripts/ats_proof/analyze_pdfs.py

@@ -11,7 +11,6 @@
 from pathlib import Path
 
 import fitz  # PyMuPDF
-
 from common import (
     RENDERED_DIR,
     RESULTS_DIR,
@@ -23,7 +22,7 @@
     normalize_quotes,
     write_json,
 )
-
+from ruamel.yaml import YAML
 
 # ---------------------------------------------------------------------------
 # Extractors
@@ -102,8 +101,6 @@ def analyze_pdf(pdf_path: Path) -> dict:
     # Get CV name for reading order check
     name = ""
     if yaml_path:
-        from ruamel.yaml import YAML
-
         yaml = YAML()
         with yaml_path.open(encoding="utf-8") as f:
             data = yaml.load(f)
@@ -227,18 +224,18 @@ def main() -> None:
     )
 
     # Print summary
-    print(
+    print(  # noqa: T201
         f"\nStructural: {structural_pass}/{len(pdfs)} passed ({structural_summary['pass_rate']})"
-    )  # noqa: T201
+    )
     for name, s in extraction_summary["extractors"].items():
-        print(
+        print(  # noqa: T201
             f"Extraction ({name}): {s['average_accuracy']} avg accuracy, {s['garbled_count']} garbled"
-        )  # noqa: T201
+        )
 
     if structural_pass < len(pdfs):
-        print(
+        print(  # noqa: T201
             f"\nWARNING: {len(pdfs) - structural_pass} PDFs failed structural analysis."
-        )  # noqa: T201
+        )
         sys.exit(1)
 
 

scripts/ats_proof/evaluate.py

@@ -18,7 +18,6 @@
     write_json,
 )
 
-
 # ---------------------------------------------------------------------------
 # Matching functions
 # ---------------------------------------------------------------------------
@@ -28,8 +27,7 @@ def normalize(text: str) -> str:
     """Normalize text for comparison."""
     text = text.lower().strip()
     text = re.sub(r"\s+", " ", text)
-    text = re.sub(r"[^\w\s@.+\-/]", "", text)
-    return text
+    return re.sub(r"[^\w\s@.+\-/]", "", text)
 
 
 def exact_match(expected: str, extracted: str) -> float:

scripts/ats_proof/generate_report.py

@@ -7,7 +7,6 @@
 from pathlib import Path
 
 import jinja2
-
 from common import (
     ANALYSIS_DIR,
     RENDERED_DIR,

scripts/ats_proof/render_pdfs.py

@@ -8,11 +8,11 @@
 import tempfile
 from pathlib import Path
 
+from common import RENDERED_DIR, THEMES, find_corpus_yamls
+
 from rendercv.cli.render_command.progress_panel import ProgressPanel
 from rendercv.cli.render_command.run_rendercv import run_rendercv
 
-from common import RENDERED_DIR, THEMES, find_corpus_yamls
-
 
 def render_with_theme(yaml_path: Path, theme: str, output_dir: Path) -> Path | None:
     """Render a corpus YAML with a given theme using the RenderCV API."""

scripts/ats_proof/submit_commercial.py

@@ -12,7 +12,6 @@
 from pathlib import Path
 
 import httpx
-
 from common import RENDERED_DIR, RESULTS_DIR, find_rendered_pdfs, write_json
 
 RESULTS_SUBDIR: str = "commercial/edenai"

tests/renderer/templater/test_entry_templates_from_input.py

@@ -691,9 +691,17 @@ class TestRemoveConnectorsOfMissingPlaceholders:
             # No connector word to remove (just space) — stays as-is
             ("AREA DEGREE", {"DEGREE"}, "AREA DEGREE"),
             # Formatting preserved, only connector word removed
-            ("**INSTITUTION**, DEGREE in AREA -- LOCATION", {"DEGREE"}, "**INSTITUTION**, DEGREE  AREA -- LOCATION"),
+            (
+                "**INSTITUTION**, DEGREE in AREA -- LOCATION",
+                {"DEGREE"},
+                "**INSTITUTION**, DEGREE  AREA -- LOCATION",
+            ),
             # Connector "at" removed between JOB_TITLE and COMPANY
-            ("**JOB_TITLE** at COMPANY_NAME", {"COMPANY_NAME"}, "**JOB_TITLE**  COMPANY_NAME"),
+            (
+                "**JOB_TITLE** at COMPANY_NAME",
+                {"COMPANY_NAME"},
+                "**JOB_TITLE**  COMPANY_NAME",
+            ),
             # No connector between placeholders separated by punctuation only
             ("NAME, LOCATION", {"LOCATION"}, "NAME, LOCATION"),
             # Hindi connector removed
@@ -721,7 +729,9 @@ def test_no_connector_word_when_degree_is_none(self):
             entry,
             templates=Templates(
                 education_entry=EducationEntryOptions(
-                    main_column="**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS",
+                    main_column=(
+                        "**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS"
+                    ),
                     degree_column=None,
                 )
             ),
@@ -745,7 +755,9 @@ def test_no_connector_word_in_french_when_degree_is_none(self):
             entry,
             templates=Templates(
                 education_entry=EducationEntryOptions(
-                    main_column="**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS",
+                    main_column=(
+                        "**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS"
+                    ),
                     degree_column=None,
                 )
             ),
@@ -770,7 +782,9 @@ def test_no_connector_word_when_area_is_empty(self):
             entry,
             templates=Templates(
                 education_entry=EducationEntryOptions(
-                    main_column="**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS",
+                    main_column=(
+                        "**INSTITUTION**, DEGREE_WITH_AREA\nSUMMARY\nHIGHLIGHTS"
+                    ),
                     degree_column=None,
                 )
             ),