Skip file-level line-count scoring for HTML templates and test files

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: clanker-lover

codedocent/quality.py

@@ -2,6 +2,8 @@
 
 from __future__ import annotations
 
+import os
+
 from codedocent.parser import CodeNode
 
 # Quality scoring thresholds: (yellow_threshold, red_threshold)
@@ -99,8 +101,27 @@ def _score_radon(node: CodeNode) -> tuple[str, str | None]:
     return "clean", None
 
 
+def _is_exempt_file(node: CodeNode) -> bool:
+    """Check if a file node should skip line-count scoring.
+
+    HTML templates and test files are naturally long, so line count
+    does not indicate poor quality for these file types.
+    """
+    if node.node_type != "file":
+        return False
+    if node.language == "html":
+        return True
+    if node.language is None and node.name.endswith(".html"):
+        return True
+    if os.path.basename(node.name).startswith("test_"):
+        return True
+    return False
+
+
 def _score_line_count(node: CodeNode) -> tuple[str, str | None]:
     """Score based on line-count thresholds (two-tier: yellow/red)."""
+    if _is_exempt_file(node):
+        return "clean", None
     thresholds = LINE_THRESHOLDS.get(node.node_type)
     if thresholds and node.line_count:
         yellow, red = thresholds

tests/test_analyzer.py

@@ -541,3 +541,39 @@ def test_quality_directory_returns_none():
     quality, warnings = _score_quality(node)
     assert quality is None
     assert warnings is None
+
+
+# ---------------------------------------------------------------------------
+# Phase 8: Exempt HTML templates and test files from line-count scoring
+# ---------------------------------------------------------------------------
+
+
+def test_quality_html_file_exempt_from_line_count():
+    from codedocent.quality import _score_quality
+
+    node = _make_file_node(name="base.html", lang="html", source="<div></div>\n")
+    node.line_count = 1500
+    quality, warnings = _score_quality(node)
+    assert quality == "clean"
+    assert warnings is None
+
+
+def test_quality_test_file_exempt_from_line_count():
+    from codedocent.quality import _score_quality
+
+    node = _make_file_node(name="test_foo.py", lang="python", source="x = 1\n")
+    node.line_count = 600
+    quality, warnings = _score_quality(node)
+    assert quality == "clean"
+    assert warnings is None
+
+
+def test_quality_regular_python_file_still_scored():
+    from codedocent.quality import _score_quality
+
+    node = _make_file_node(name="app.py", lang="python", source="x = 1\n")
+    node.line_count = 600
+    quality, warnings = _score_quality(node)
+    assert quality == "complex"
+    assert warnings is not None
+    assert any("600 lines" in w for w in warnings)