fix: resolve lint errors and clean up coverage config

- Fix 5 ruff errors in scripts/hooks/ (import sorting, unused import, long lines)
- Migrate .coveragerc into pyproject.toml [tool.coverage] section
- Add .coveragerc to .gitignore

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

Author: acailic

.gitignore

@@ -9,6 +9,7 @@ venv/
 
 .pytest_cache/
 .coverage
+.coveragerc
 coverage.xml
 htmlcov/
 .mypy_cache/

pyproject.toml

@@ -82,9 +82,34 @@ markers = [
     "integration: End-to-end tests requiring API keys (deselected by default)",
 ]
 
-[tool.coverage.report]
+[tool.coverage.run]
+source = ["agent_debugger_sdk", "api", "storage", "collector", "auth", "redaction"]
 omit = [
     "*/tests/*",
+    "*/test_*.py",
+    "*/__pycache__/*",
+    "*/site-packages/*",
+    "*/dist/*",
+    "*/build/*",
+    "*/.venv*",
+    "*/migrations/*",
+]
+branch = true
+
+[tool.coverage.report]
+precision = 2
+show_missing = true
+skip_covered = false
+sort = "Cover"
+exclude_lines = [
+    "pragma: no cover",
+    "def __repr__",
+    "raise AssertionError",
+    "raise NotImplementedError",
+    "if __name__ == .__main__.:",
+    "if TYPE_CHECKING:",
+    "@abstractmethod",
+    "@abc.abstractmethod",
 ]
 
 [tool.ruff]

scripts/hooks/check_layer_imports.py

@@ -9,9 +9,8 @@
   - auth/ files should NOT import from api or collector
 """
 
-import sys
 import json
-import re
+import sys
 from pathlib import Path
 
 
@@ -43,7 +42,10 @@ def check_file_for_violations(file_path: str) -> dict:
             "forbidden_imports": ["from api ", "import api", "from storage ", "import storage",
                                   "from collector ", "import collector", "from auth ", "import auth",
                                   "from redaction ", "import redaction"],
-            "reason": "SDK layer (agent_debugger_sdk/) should not import from server/runtime layers (api, storage, collector, auth, redaction)"
+            "reason": (
+                "SDK layer (agent_debugger_sdk/) should not import from "
+                "server/runtime layers (api, storage, collector, auth, redaction)"
+            )
         },
         # Storage layer should not import from API or collector
         {
@@ -67,7 +69,11 @@ def check_file_for_violations(file_path: str) -> dict:
                 if forbidden in content:
                     return {
                         "decision": "deny",
-                        "reason": f"Architecture violation in {file_path.relative_to(Path.cwd())}: {rule['reason']}. Found forbidden import pattern: '{forbidden}'"
+                        "reason": (
+                            f"Architecture violation in "
+                            f"{file_path.relative_to(Path.cwd())}: {rule['reason']}. "
+                            f"Found forbidden import pattern: '{forbidden}'"
+                        )
                     }
 
     return {"decision": "allow"}

scripts/hooks/post_edit_lint.py

@@ -10,8 +10,8 @@
 This is informational only (exit 0 always) - just prints warnings.
 """
 
-import sys
 import subprocess
+import sys
 from pathlib import Path