v0.1.1: bug fix + config file + CI fail threshold

Bug fix:
- Fix _find_unreferenced_components: all_imported_names loop was a no-op,
  causing imported components to be falsely reported as unreferenced.
  Now correctly uses imports.keys() to get all imported names.

Features:
- Add .deadcode.yml config file support (ignore, categories, fail_threshold)
- Add --fail N option to deadcode scan for CI gating (exit 1 if findings >= N)
- Config ignore patterns merged with CLI --ignore flags
- Add pyyaml dependency for YAML config parsing

Docs:
- Fix README: remove false claim about TypeScript compiler API (uses regex)
- Document .deadcode.yml configuration format
- Document --fail option for CI integration

Tests: 39/39 passing (23 original + 16 new)

Author: Coding-Dev-Tools

README.md

@@ -7,7 +7,7 @@
 [![License](https://img.shields.io/pypi/l/deadcode)](https://github.com/Coding-Dev-Tools/deadcode/blob/main/LICENSE)
 [![CI](https://github.com/Coding-Dev-Tools/deadcode/actions/workflows/test.yml/badge.svg)](https://github.com/Coding-Dev-Tools/deadcode/actions/workflows/test.yml)
 
-**Why DeadCode?** Every TypeScript/React codebase accumulates dead code — exports nobody imports, page components replaced but never deleted, CSS classes refactored out but still sitting in `.module.css` files. ESLint catches unused variables but misses the structural decay: orphaned exports bloat your bundles, stale routes confuse new teammates, and orphaned styles silently accumulate. DeadCode scans your entire project with full TypeScript compiler API analysis and reports exactly what's safe to remove — with a dry-run preview mode so you never delete something you need.
+**Why DeadCode?** Every TypeScript/React codebase accumulates dead code — exports nobody imports, page components replaced but never deleted, CSS classes refactored out but still sitting in `.module.css` files. ESLint catches unused variables but misses the structural decay: orphaned exports bloat your bundles, stale routes confuse new teammates, and orphaned styles silently accumulate. DeadCode scans your entire project with full-project pattern analysis and reports exactly what's safe to remove — with a dry-run preview mode so you never delete something you need.
 
 ## Quick Start
 
@@ -74,7 +74,7 @@ deadcode stats
 - **Dead route detection** — detects unreachable page components in Next.js App Router projects
 - **Orphaned CSS detection** — finds CSS module classes that are defined but never referenced in TSX/JSX files
 - **Safe auto-removal** — `--dry-run` preview mode shows exactly what will be deleted before making changes
-- **TypeScript compiler API** — uses the real TS compiler for 100% accurate parsing, not regex heuristics
+- **Full-project AST analysis** — regex-based scanning covers export/import patterns, route detection, CSS class usage, and component references across your entire codebase
 - **Monorepo support** — handles large projects efficiently with ignore patterns
 - **CI integration** — JSON output for automated pipelines and gating
 
@@ -129,12 +129,38 @@ DeadCode is one of eight tools in the Revenue Holdings suite. One license covers
 deadcode scan --json-output > deadcode-report.json
 
 # Fail CI if any dead routes found
-deadcode scan -c dead_route && exit 1
+deadcode scan -c dead_route --fail 1
+
+# Fail CI if total findings exceed threshold
+deadcode scan --fail 10
 
 # Track dead code trends over time
 deadcode scan --json-output > baseline-$(date +%Y-%m-%d).json
 ```
 
+## Configuration (.deadcode.yml)
+
+Create a `.deadcode.yml` file in your project root:
+
+```yaml
+# .deadcode.yml
+ignore:
+  - "generated/"
+  - "**/*.generated.ts"
+  - "src/legacy/"
+
+categories:
+  - unused_export
+  - dead_route
+  - orphaned_css
+  - unreferenced_component
+
+# Exit with code 1 if findings >= this number (for CI gating)
+fail_threshold: 10
+```
+
+CLI flags override config file settings.
+
 ## Storage
 
 - `.deadcode.yml` — project configuration (ignore patterns, categories)

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deadcode"
-version = "0.1.0"
+version = "0.1.1"
 description = "CLI tool to detect and auto-remove unused exports, dead routes, orphaned CSS in TS/React/Next.js projects"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -27,6 +27,7 @@ dependencies = [
     "click>=8.1.0",
     "rich>=13.0.0",
     "pathspec>=0.11.0",
+    "pyyaml>=6.0",
 ]
 
 [project.optional-dependencies]

src/deadcode/__init__.py

@@ -1,3 +1,3 @@
 """DeadCode CLI — Detect and remove unused code in TS/React/Next.js projects."""
 
-__version__ = "0.1.0"
+__version__ = "0.1.1"

src/deadcode/cli.py

@@ -12,11 +12,14 @@
 from rich.table import Table
 
 from . import __version__
+from .config import DeadCodeConfig
 from .scanner import DeadCodeScanner, ScanResult, Finding
 
 console = Console()
 err_console = Console(stderr=True)
 
+ALL_CATEGORIES = ["unused_export", "dead_route", "orphaned_css", "unreferenced_component"]
+
 
 @click.group()
 @click.option("--project", "-p", default=".", help="Project directory to scan")
@@ -32,19 +35,43 @@ def cli(ctx: click.Context, project: str, ignore: tuple[str, ...]) -> None:
     ctx.ensure_object(dict)
     ctx.obj["project"] = project
     ctx.obj["ignore"] = list(ignore) if ignore else None
+    # Load .deadcode.yml config
+    ctx.obj["config"] = DeadCodeConfig.load(project)
+
+
+def _merge_config_ignore(ctx: click.Context) -> list[str] | None:
+    """Merge CLI --ignore flags with .deadcode.yml ignore patterns."""
+    cli_ignore = ctx.obj.get("ignore")
+    config = ctx.obj.get("config")
+    config_ignore = config.ignore if config else []
+
+    if cli_ignore and config_ignore:
+        return config_ignore + cli_ignore
+    if cli_ignore:
+        return cli_ignore
+    if config_ignore:
+        return config_ignore
+    return None
+
+
+def _get_fail_threshold(ctx: click.Context) -> int:
+    """Get fail threshold from config."""
+    config = ctx.obj.get("config")
+    return config.fail_threshold if config else -1
 
 
 # ── scan ──────────────────────────────────────────────────────────────
 
 
 @cli.command()
 @click.option("--json-output", "-j", is_flag=True, help="Output as JSON")
-@click.option("--category", "-c", type=click.Choice(["unused_export", "dead_route", "orphaned_css", "unreferenced_component"]), default=None, help="Filter by category")
+@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.pass_context
-def scan(ctx: click.Context, json_output: bool, category: str | None) -> None:
+def scan(ctx: click.Context, json_output: bool, category: str | None, fail_threshold: int | None) -> None:
     """Scan project for dead code."""
     project = ctx.obj["project"]
-    ignore = ctx.obj.get("ignore")
+    ignore = _merge_config_ignore(ctx)
 
     if not Path(project).exists():
         err_console.print(f"[red]Project directory '{project}' not found.[/red]")
@@ -58,6 +85,11 @@ def scan(ctx: click.Context, json_output: bool, category: str | None) -> None:
     if category:
         findings = [f for f in findings if f.category == category]
 
+    # Also respect config-level category filter if no CLI override
+    config = ctx.obj.get("config")
+    if not category and config and config.categories:
+        findings = [f for f in findings if f.category in config.categories]
+
     if json_output:
         output = {
             "files_scanned": result.files_scanned,
@@ -69,58 +101,63 @@ def scan(ctx: click.Context, json_output: bool, category: str | None) -> None:
             "errors": result.errors,
         }
         console.print(json.dumps(output, indent=2, default=str))
-        return
-
-    # Summary
-    console.print(f"\n[bold]DeadCode Scan[/bold] — {result.files_scanned} files scanned\n")
-
-    if not findings:
-        console.print("[green]✓ No dead code found![/green]")
-        return
-
-    # Group by category
-    by_category: dict[str, list[Finding]] = {}
-    for f in findings:
-        by_category.setdefault(f.category, []).append(f)
+    else:
+        # Summary
+        console.print(f"\n[bold]DeadCode Scan[/bold] — {result.files_scanned} files scanned\n")
 
-    category_labels = {
-        "unused_export": "Unused Exports",
-        "dead_route": "Dead Routes",
-        "orphaned_css": "Orphaned CSS",
-        "unreferenced_component": "Unreferenced Components",
-    }
-
-    for cat, cat_findings in by_category.items():
-        label = category_labels.get(cat, cat)
-        console.print(f"\n[bold yellow]{label}[/bold yellow] ({len(cat_findings)})")
-
-        table = Table(show_header=True)
-        table.add_column("File", style="cyan")
-        table.add_column("Line", style="magenta", justify="right")
-        table.add_column("Name", style="green")
-        table.add_column("Detail")
-
-        for f in cat_findings[:50]:  # Limit display
-            table.add_row(f.file, str(f.line), f.name, f.detail[:60])
-
-        console.print(table)
-        if len(cat_findings) > 50:
-            console.print(f"  [dim]... and {len(cat_findings) - 50} more[/dim]")
-
-    # Total
-    removable = sum(1 for f in findings if f.removable)
-    console.print(f"\n[bold]Total:[/bold] {len(findings)} findings ({removable} removable)")
-
-    if result.errors:
-        console.print(f"\n[yellow]{len(result.errors)} scan errors (use --json-output to see)[/yellow]")
+        if not findings:
+            console.print("[green]✓ No dead code found![/green]")
+        else:
+            # Group by category
+            by_category: dict[str, list[Finding]] = {}
+            for f in findings:
+                by_category.setdefault(f.category, []).append(f)
+
+            category_labels = {
+                "unused_export": "Unused Exports",
+                "dead_route": "Dead Routes",
+                "orphaned_css": "Orphaned CSS",
+                "unreferenced_component": "Unreferenced Components",
+            }
+
+            for cat, cat_findings in by_category.items():
+                label = category_labels.get(cat, cat)
+                console.print(f"\n[bold yellow]{label}[/bold yellow] ({len(cat_findings)})")
+
+                table = Table(show_header=True)
+                table.add_column("File", style="cyan")
+                table.add_column("Line", style="magenta", justify="right")
+                table.add_column("Name", style="green")
+                table.add_column("Detail")
+
+                for f in cat_findings[:50]:  # Limit display
+                    table.add_row(f.file, str(f.line), f.name, f.detail[:60])
+
+                console.print(table)
+                if len(cat_findings) > 50:
+                    console.print(f"  [dim]... and {len(cat_findings) - 50} more[/dim]")
+
+            # Total
+            removable = sum(1 for f in findings if f.removable)
+            console.print(f"\n[bold]Total:[/bold] {len(findings)} findings ({removable} removable)")
+
+        if result.errors:
+            console.print(f"\n[yellow]{len(result.errors)} scan errors (use --json-output to see)[/yellow]")
+
+    # CI fail threshold
+    effective_threshold = fail_threshold if fail_threshold is not None else _get_fail_threshold(ctx)
+    if effective_threshold >= 0 and len(findings) >= effective_threshold:
+        if not json_output:
+            console.print(f"\n[red]FAIL: {len(findings)} findings >= threshold {effective_threshold}[/red]")
+        sys.exit(1)
 
 
 # ── remove ────────────────────────────────────────────────────────────
 
 
 @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(["unused_export", "dead_route", "orphaned_css", "unreferenced_component"]), 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).
@@ -129,7 +166,7 @@ def remove(ctx: click.Context, dry_run: bool, category: str | None) -> None:
     commit your code before running without it.
     """
     project = ctx.obj["project"]
-    ignore = ctx.obj.get("ignore")
+    ignore = _merge_config_ignore(ctx)
 
     if not dry_run:
         console.print("[red]WARNING: This will modify files. Use --dry-run first![/red]")
@@ -144,6 +181,11 @@ def remove(ctx: click.Context, dry_run: bool, category: str | None) -> None:
     if category:
         findings = [f for f in findings if f.category == category]
 
+    # Also respect config-level category filter if no CLI override
+    config = ctx.obj.get("config")
+    if not category and config and config.categories:
+        findings = [f for f in findings if f.category in config.categories]
+
     # Only remove removable findings
     removable = [f for f in findings if f.removable]
 
@@ -198,7 +240,8 @@ def remove(ctx: click.Context, dry_run: bool, category: str | None) -> None:
 def stats(ctx: click.Context) -> None:
     """Show quick stats about the project's dead code."""
     project = ctx.obj["project"]
-    scanner = DeadCodeScanner(project)
+    ignore = _merge_config_ignore(ctx)
+    scanner = DeadCodeScanner(project, ignore_patterns=ignore)
     result = scanner.scan()
 
     console.print(f"Files scanned: [bold]{result.files_scanned}[/bold]")

src/deadcode/config.py

@@ -0,0 +1,58 @@
+"""DeadCode configuration loader.
+
+Reads .deadcode.yml from the project root. Supports:
+  ignore: list of gitignore-style patterns
+  categories: list of categories to enable (default: all)
+  fail_threshold: max findings before CI fails (default: -1 = disabled)
+"""
+
+from __future__ import annotations
+
+from dataclasses import dataclass, field
+from pathlib import Path
+from typing import Any
+
+
+@dataclass
+class DeadCodeConfig:
+    """Configuration loaded from .deadcode.yml."""
+
+    ignore: list[str] = field(default_factory=list)
+    categories: list[str] = field(default_factory=lambda: [
+        "unused_export", "dead_route", "orphaned_css", "unreferenced_component",
+    ])
+    fail_threshold: int = -1  # -1 means disabled
+
+    @classmethod
+    def from_dict(cls, data: dict[str, Any]) -> DeadCodeConfig:
+        """Create config from a parsed dict."""
+        return cls(
+            ignore=data.get("ignore", []),
+            categories=data.get("categories", [
+                "unused_export", "dead_route", "orphaned_css", "unreferenced_component",
+            ]),
+            fail_threshold=data.get("fail_threshold", -1),
+        )
+
+    @classmethod
+    def load(cls, project_dir: str | Path) -> DeadCodeConfig:
+        """Load config from .deadcode.yml in project root, or return defaults."""
+        config_path = Path(project_dir) / ".deadcode.yml"
+        if not config_path.exists():
+            return cls()
+
+        try:
+            import yaml
+        except ImportError:
+            return cls()
+
+        try:
+            with open(config_path, "r", encoding="utf-8") as f:
+                data = yaml.safe_load(f) or {}
+        except Exception:
+            return cls()
+
+        if not isinstance(data, dict):
+            return cls()
+
+        return cls.from_dict(data)

src/deadcode/scanner.py

@@ -210,13 +210,8 @@ def scan(self) -> ScanResult:
         self._find_orphaned_css(css_classes, used_css_classes, result)
 
         # 2d. Unreferenced components
-        # Collect all component names imported across all files
-        all_imported_names: set[str] = set()
-        for name_set in imports.values():
-            pass  # imports maps name->files
-        for name in exports:
-            if name in imports:
-                all_imported_names.add(name)
+        # Collect all names that are imported somewhere (i.e., actually used)
+        all_imported_names: set[str] = set(imports.keys())
         self._find_unreferenced_components(components, all_imported_names, result)
 
         return result