crackerjack 0.33.0__py3-none-any.whl → 0.33.1__py3-none-any.whl

crackerjack/agents/test_specialist_agent.py

@@ -63,7 +63,6 @@ class TestSpecialistAgent(SubAgent):
         )
 
     def _check_test_patterns(self, message: str) -> float:
-        # Map pattern strings back to SAFE_PATTERNS for safe usage
         pattern_map = {
             SAFE_PATTERNS[
                 "fixture_not_found_pattern"
@@ -191,7 +190,6 @@ class TestSpecialistAgent(SubAgent):
     def _identify_failure_type(self, issue: Issue) -> str:
         message = issue.message
 
-        # Map pattern strings back to SAFE_PATTERNS for safe usage
         pattern_map = {
             SAFE_PATTERNS[
                 "fixture_not_found_pattern"
@@ -218,12 +216,10 @@ class TestSpecialistAgent(SubAgent):
     async def _fix_missing_fixtures(self, issue: Issue) -> list[str]:
         fixes: list[str] = []
 
-        # Use safe pattern to test and extract fixture name
         fixture_pattern = SAFE_PATTERNS["fixture_not_found_pattern"]
         if not fixture_pattern.test(issue.message):
             return fixes
 
-        # Extract fixture name using the safe pattern's search method
         match = fixture_pattern.search(issue.message)
         if not match:
             return fixes

crackerjack/api.py

@@ -131,17 +131,6 @@ class CrackerjackAPI:
         backup: bool = True,
         safe_mode: bool = True,
     ) -> list[CleaningResult] | PackageCleaningResult:
-        """Clean code with comprehensive backup protection.
-
-        Args:
-            target_dir: Directory to clean (defaults to package root)
-            backup: Whether to create backup (deprecated, always True for safety)
-            safe_mode: Use comprehensive backup system (default: True, recommended)
-
-        Returns:
-            PackageCleaningResult with backup metadata if safe_mode=True,
-            otherwise list[CleaningResult] for legacy compatibility
-        """
         target_dir = target_dir or self._get_package_root()
         self.logger.info(f"Cleaning code in {target_dir} (safe_mode={safe_mode})")
 
@@ -153,7 +142,6 @@ class CrackerjackAPI:
             )
             return self._execute_safe_code_cleaning(target_dir)
         else:
-            # Note: Legacy mode still uses backup protection for safety
             self.console.print(
                 "[yellow]⚠️ Legacy mode - backup protection still enabled for safety[/yellow]"
             )
@@ -210,16 +198,13 @@ class CrackerjackAPI:
 
     def _execute_code_cleaning(self, target_dir: Path) -> list[CleaningResult]:
         try:
-            # Use backup protection by default for safety
             results = self.code_cleaner.clean_files(target_dir, use_backup=True)
 
-            # Handle both return types (legacy compatibility)
             if isinstance(results, list):
                 self._report_cleaning_results(results)
             else:
-                # PackageCleaningResult from backup mode
                 self._report_safe_cleaning_results(results)
-                results = results.file_results  # Extract list for compatibility
+                results = results.file_results
 
             return results
         except Exception as e:
@@ -415,7 +400,7 @@ class CrackerjackAPI:
             git_dir = self.project_path / ".git"
             is_git_repo = git_dir.exists()
 
-            python_files = list(self.project_path.rglob("*.py"))
+            python_files = list[t.Any](self.project_path.rglob("*.py"))
 
             return {
                 "project_path": str(self.project_path),
@@ -494,13 +479,13 @@ class CrackerjackAPI:
                     data = tomllib.load(f)
 
                 if "project" in data and "version" in data["project"]:
-                    return data["project"]["version"]
+                    return str(data["project"]["version"])
                 if (
                     "tool" in data
                     and "poetry" in data["tool"]
                     and "version" in data["tool"]["poetry"]
                 ):
-                    return data["tool"]["poetry"]["version"]
+                    return str(data["tool"]["poetry"]["version"])
 
             import importlib.metadata
 
@@ -514,7 +499,6 @@ class CrackerjackAPI:
             return "unknown"
 
     def _check_for_todos(self, target_dir: Path) -> list[tuple[Path, int, str]]:
-        # Use SAFE_PATTERNS for TODO detection
         task_pattern = SAFE_PATTERNS["todo_pattern"]
         python_files = self._get_python_files_for_todo_check(target_dir)
         return self._scan_files_for_todos(python_files, task_pattern)
@@ -574,12 +558,9 @@ class CrackerjackAPI:
         with suppress(UnicodeDecodeError, PermissionError):
             with file_path.open() as f:
                 for line_no, line in enumerate(f, 1):
-                    # For ValidatedPattern, check if applying it changes the line
-                    # If it doesn't change, then it didn't match (identity replacement for match-only patterns)
                     original = line.strip()
                     processed = todo_pattern.apply(original)
-                    # For TODO pattern with identity replacement, a match means no change
-                    # But we need to check if it actually contains TODO
+
                     if "todo" in original.lower() and original == processed:
                         todos.append((file_path, line_no, line))
 
@@ -609,7 +590,7 @@ class CrackerjackAPI:
                 data = tomllib.load(f)
 
             if "project" in data and "name" in data["project"]:
-                return data["project"]["name"]
+                return str(data["project"]["name"])
 
         return None
 

crackerjack/cli/cache_handlers.py

@@ -0,0 +1,204 @@
+import typing as t
+
+from rich.console import Console
+from rich.panel import Panel
+from rich.table import Table
+from rich.text import Text
+
+from crackerjack.services.cache import CrackerjackCache
+
+
+def handle_clear_cache(console: Console) -> None:
+    """Clear all caches and display results."""
+    try:
+        cache = CrackerjackCache()
+
+        # Clear memory caches and get cleanup stats
+        cleanup_results = cache.cleanup_all()
+
+        # Clear disk cache completely
+        if cache.enable_disk_cache and cache.cache_dir:
+            cache.disk_cache.clear()
+
+        # Calculate total items cleared
+        total_cleared = sum(cleanup_results.values())
+
+        # Create results table
+        table = Table(
+            title="Cache Cleared", show_header=True, header_style="bold green"
+        )
+        table.add_column("Cache Type", style="cyan", no_wrap=True)
+        table.add_column("Items Cleared", justify="right", style="yellow")
+
+        for cache_type, count in cleanup_results.items():
+            table.add_row(cache_type.replace("_", " ").title(), str(count))
+
+        table.add_row("", "", end_section=True)
+        table.add_row("Total", str(total_cleared), style="bold green")
+
+        console.print()
+        console.print(table)
+        console.print(f"\n✅ Successfully cleared {total_cleared} cache entries")
+
+    except Exception as e:
+        console.print(f"\n❌ Error clearing cache: {e}", style="bold red")
+
+
+def handle_cache_stats(console: Console) -> None:
+    """Display detailed cache statistics."""
+    try:
+        cache = CrackerjackCache()
+        stats = cache.get_cache_stats()
+
+        main_table = _create_cache_stats_table()
+        totals = _populate_cache_stats_table(main_table, stats)
+        _add_cache_totals_row(main_table, totals)
+
+        console.print()
+        console.print(main_table)
+
+        _display_performance_insights(console, totals)
+        _display_cache_directory_info(console, cache)
+
+    except Exception as e:
+        console.print(f"\n❌ Error retrieving cache stats: {e}", style="bold red")
+
+
+def _create_cache_stats_table() -> Table:
+    """Create and configure the main cache statistics table."""
+    table = Table(title="Cache Statistics", show_header=True, header_style="bold blue")
+    table.add_column("Cache Layer", style="cyan", no_wrap=True)
+    table.add_column("Hit Rate %", justify="right", style="green")
+    table.add_column("Hits", justify="right", style="yellow")
+    table.add_column("Misses", justify="right", style="red")
+    table.add_column("Entries", justify="right", style="magenta")
+    table.add_column("Size (MB)", justify="right", style="blue")
+    return table
+
+
+def _populate_cache_stats_table(
+    table: Table, stats: dict[str, t.Any]
+) -> dict[str, t.Any]:
+    """Populate table with cache statistics and return totals."""
+    totals = {"hits": 0, "misses": 0, "entries": 0, "size": 0.0}
+
+    for cache_name, cache_stats in stats.items():
+        hit_rate = cache_stats.get("hit_rate_percent", 0.0)
+        hits = cache_stats.get("hits", 0)
+        misses = cache_stats.get("misses", 0)
+        entries = cache_stats.get("total_entries", 0)
+        size_mb = cache_stats.get("total_size_mb", 0.0)
+
+        hit_rate_style = _get_hit_rate_style(hit_rate)
+
+        table.add_row(
+            cache_name.replace("_", " ").title(),
+            Text(f"{hit_rate:.1f}", style=hit_rate_style),
+            str(hits),
+            str(misses),
+            str(entries),
+            f"{size_mb:.2f}",
+        )
+
+        totals["hits"] += hits
+        totals["misses"] += misses
+        totals["entries"] += entries
+        totals["size"] += size_mb
+
+    return totals
+
+
+def _get_hit_rate_style(hit_rate: float) -> str:
+    """Get color style for hit rate based on performance."""
+    return "green" if hit_rate > 70 else "yellow" if hit_rate > 40 else "red"
+
+
+def _add_cache_totals_row(table: Table, totals: dict[str, t.Any]) -> None:
+    """Add totals row to cache statistics table."""
+    overall_hit_rate = (
+        (totals["hits"] / (totals["hits"] + totals["misses"]) * 100)
+        if (totals["hits"] + totals["misses"]) > 0
+        else 0
+    )
+    overall_style = _get_hit_rate_style(overall_hit_rate)
+
+    table.add_row("", "", "", "", "", "", end_section=True)
+    table.add_row(
+        "Overall",
+        Text(f"{overall_hit_rate:.1f}", style=f"bold {overall_style}"),
+        str(totals["hits"]),
+        str(totals["misses"]),
+        str(totals["entries"]),
+        f"{totals['size']:.2f}",
+        style="bold",
+    )
+
+
+def _display_performance_insights(console: Console, totals: dict[str, t.Any]) -> None:
+    """Display performance insights panel based on cache statistics."""
+    overall_hit_rate = (
+        (totals["hits"] / (totals["hits"] + totals["misses"]) * 100)
+        if (totals["hits"] + totals["misses"]) > 0
+        else 0
+    )
+
+    insights = _generate_performance_insights(overall_hit_rate, totals["size"])
+
+    if insights:
+        insights_text = "\n".join(insights)
+        insights_panel = Panel(
+            insights_text,
+            title="Performance Insights",
+            border_style="blue",
+            padding=(1, 2),
+        )
+        console.print()
+        console.print(insights_panel)
+
+
+def _generate_performance_insights(hit_rate: float, total_size: float) -> list[str]:
+    """Generate performance insights based on cache metrics."""
+    insights = []
+
+    if hit_rate > 80:
+        insights.append("🚀 Excellent cache performance!")
+    elif hit_rate > 60:
+        insights.append("✅ Good cache performance")
+    elif hit_rate > 30:
+        insights.append("⚠️ Moderate cache performance - consider cache warming")
+    else:
+        insights.append("❌ Poor cache performance - check cache configuration")
+
+    if total_size > 100:
+        insights.append(f"💾 Large cache size ({total_size:.1f}MB) - consider cleanup")
+
+    return insights
+
+
+def _display_cache_directory_info(console: Console, cache: CrackerjackCache) -> None:
+    """Display cache directory information."""
+    if not (cache.enable_disk_cache and cache.cache_dir):
+        return
+
+    cache_dir_info = f"📁 Cache Directory: {cache.cache_dir}"
+    if cache.cache_dir.exists():
+        disk_files = len(list[t.Any](cache.cache_dir.rglob("*.cache")))
+        cache_dir_info += f" ({disk_files} files)"
+
+    console.print()
+    console.print(cache_dir_info)
+
+
+def _handle_cache_commands(
+    clear_cache: bool, cache_stats: bool, console: Console
+) -> bool:
+    """Handle cache management commands. Returns True if a cache command was executed."""
+    if clear_cache:
+        handle_clear_cache(console)
+        return True
+
+    if cache_stats:
+        handle_cache_stats(console)
+        return True
+
+    return False