fix: ruff lint - datetime.UTC, X|None, E501, B904, F821

Author: Revenue Holdings

.github/dependabot.yml

@@ -0,0 +1,12 @@
+version: 2
+updates:
+  - package-ecosystem: pip
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 5
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 3

.gitignore

@@ -70,3 +70,4 @@ Thumbs.db
 # Project specific
 research/
 fixtures/generated/
+.ruff_cache/

src/deadcode/cli.py

@@ -69,7 +69,8 @@ def _get_fail_threshold(ctx: click.Context) -> int:
 @cli.command()
 @click.option("--json-output", "-j", is_flag=True, help="Output as JSON")
 @click.option("--category", "-c", type=click.Choice(ALL_CATEGORIES), default=None, help="Filter by category")
-@click.option("--fail", "fail_threshold", type=int, default=None, help="Exit code 1 if findings >= threshold (overrides .deadcode.yml)")
+@click.option("--fail", "fail_threshold", type=int, default=None,
+              help="Exit code 1 if findings >= threshold (overrides .deadcode.yml)")
 @click.pass_context
 def scan(ctx: click.Context, json_output: bool, category: str | None, fail_threshold: int | None) -> None:
     """Scan project for dead code."""
@@ -162,7 +163,8 @@ def scan(ctx: click.Context, json_output: bool, category: str | None, fail_thres
 
 @cli.command()
 @click.option("--dry-run", is_flag=True, help="Preview what would be removed without making changes")
-@click.option("--category", "-c", type=click.Choice(ALL_CATEGORIES), default=None, help="Only remove findings in this category")
+@click.option("--category", "-c", type=click.Choice(ALL_CATEGORIES),
+              default=None, help="Only remove findings in this category")
 @click.pass_context
 def remove(ctx: click.Context, dry_run: bool, category: str | None) -> None:
     """Remove dead code (with --dry-run for preview).

tests/test_config_and_fixes.py

@@ -3,14 +3,10 @@
 from __future__ import annotations
 
 import json
-import os
-from pathlib import Path
-
 import pytest
-
-from deadcode.config import DeadCodeConfig
-from deadcode.scanner import DeadCodeScanner, ScanResult, Finding
 from deadcode.cli import cli
+from deadcode.config import DeadCodeConfig
+from deadcode.scanner import DeadCodeScanner
 
 
 @pytest.fixture
@@ -214,7 +210,7 @@ def test_component_imported_not_reported(self, tmp_path):
         result = scanner.scan()
 
         comp_names = {f.name for f in result.unreferenced_components}
-        assert "Button" not in comp_names, f"Button should not be unreferenced — it's imported by App.tsx"
+        assert "Button" not in comp_names, "Button should not be unreferenced — it's imported by App.tsx"
 
     def test_component_not_imported_is_reported(self, tmp_path):
         """Component with zero imports should still be reported."""

tests/test_scanner.py

@@ -3,14 +3,9 @@
 from __future__ import annotations
 
 import json
-import os
-import tempfile
-from pathlib import Path
-
 import pytest
-
-from deadcode.scanner import DeadCodeScanner, ScanResult, Finding
 from deadcode.cli import cli
+from deadcode.scanner import DeadCodeScanner
 
 
 @pytest.fixture
@@ -279,7 +274,7 @@ def test_scan_json_output(self, runner, sample_project):
     def test_scan_category_filter(self, runner, sample_project):
         result = runner.invoke(cli, ["-p", str(sample_project), "scan", "-c", "orphaned_css"])
         assert result.exit_code == 0
-        data = json.loads(result.output) if "--json-output" in [] else None
+        json.loads(result.output) if "--json-output" in [] else None
         # Just check it doesn't crash
         assert "Orphaned CSS" in result.output or result.exit_code == 0