refactor(goal): code analysis engine

changes:
  - file: disk_analyzer.py
    area: analyzer
    modified: [__init__, suggest_cleanup_actions, _identify_cache_type, DiskAnalyzer]
  - file: cleanup_planner.py
    area: core
    modified: [__init__, CleanupType, CleanupPlanner, _get_category_for_action, _dict_to_action]

stats:
  lines: "+64/-734 (net -670)"
  files: 10
  complexity: "Large structural change (normalized)"

Author: tom-sapletta-com

CHANGELOG.md

@@ -1,3 +1,26 @@
+## [2.1.18] - 2026-02-26
+
+### Summary
+
+refactor(goal): code analysis engine
+
+### Test
+
+- update test_click.py
+- update test_click2.py
+
+### Other
+
+- update fixos/diagnostics/disk_analyzer.py
+- update fixos/interactive/cleanup_planner.py
+- update patch_cli.py
+- update patch_cli2.py
+- update patch_cli3.py
+- update patch_cli4.py
+- update patch_docs.py
+- update patch_parser.py
+
+
 ## [2.1.17] - 2026-02-26
 
 ### Summary

VERSION

@@ -1 +1 @@
-2.1.17
+2.1.18

fixos/__init__.py

@@ -1,2 +1,2 @@
 """fixos – AI-powered Linux/Windows diagnostics and repair."""
-__version__ = "2.1.17"
+__version__ = "2.1.18"

fixos/diagnostics/disk_analyzer.py

@@ -19,7 +19,8 @@ def __init__(self, base_path: str = "/"):
         self.base_path = Path(base_path)
         self.cache_patterns = [
             ".cache", "__pycache__", "node_modules", ".npm", 
-            ".pip", "cache", "Cache", ".gradle", ".maven", ".cargo"
+            ".pip", "cache", "Cache", ".gradle", ".maven", ".cargo",
+            "apt", "dnf", "yum", "pacman", "pkg", "docker/overlay2", "docker/image", "Containers"
         ]
         self.log_patterns = [
             ".log", "logs", "Logs", "*.log", "*.out", "*.err"
@@ -232,11 +233,35 @@ def suggest_cleanup_actions(self, path: Path) -> List[Dict]:
                         "path": log_dir["path"],
                         "size_gb": log_dir["size_gb"],
                         "description": f"Clean old log files",
-                        "command": f"find {log_dir['path']} -name '*.log' -mtime +30 -delete",
+                        "command": f"find {log_dir['path']} -name '*.log' -mtime +30 -delete || rm -rf {log_dir['path']}/*.log",
                         "safe": True,
                         "impact": "medium"
                     })
 
+            # Docker and Package Manager specific suggestions
+            # These are generated regardless if we found them in cache dirs to guarantee they are surfaced
+            suggestions.append({
+                "type": "docker_cleanup",
+                "priority": "high",
+                "path": "/var/lib/docker",
+                "size_gb": 0.0, # Will be recalculated by planner
+                "description": "Clean unused Docker images, containers, and volumes",
+                "command": "docker system prune -af --volumes",
+                "safe": False,
+                "impact": "high"
+            })
+            
+            suggestions.append({
+                "type": "package_cleanup",
+                "priority": "medium",
+                "path": "/var/cache",
+                "size_gb": 0.0, # Will be recalculated
+                "description": "Clean system package manager cache (apt/dnf/pacman)",
+                "command": "apt-get clean || dnf clean all || pacman -Scc --noconfirm",
+                "safe": True,
+                "impact": "medium"
+            })
+            
             # Temp directory cleanup
             for temp_dir in temp_dirs[:3]:
                 if temp_dir["size_mb"] > 20:
@@ -329,6 +354,10 @@ def _identify_cache_type(self, dir_path: Path) -> str:
             return "maven"
         elif "cargo" in name:
             return "cargo"
+        elif "docker" in path_str or "containers" in path_str:
+            return "docker"
+        elif "apt" in path_str or "dnf" in path_str or "yum" in path_str or "pacman" in path_str:
+            return "package_manager"
         elif "browser" in path_str or "chrome" in path_str or "firefox" in path_str:
             return "browser"
         else:

fixos/interactive/cleanup_planner.py

@@ -24,6 +24,8 @@ class CleanupType(Enum):
     LARGE_FILE = "large_file"
     SYSTEM = "system_cleanup"
     USER = "user_cleanup"
+    DOCKER = "docker_cleanup"
+    PACKAGE_MGR = "package_cleanup"
 
 
 @dataclass
@@ -86,6 +88,18 @@ def __init__(self):
                 "description": "User-specific cleanup actions",
                 "color": "green",
                 "icon": "👤"
+            },
+            "docker": {
+                "name": "Docker & Containers",
+                "description": "Unused images, containers, and volumes",
+                "color": "cyan",
+                "icon": "🐳"
+            },
+            "package_manager": {
+                "name": "Package Cache",
+                "description": "Cached system packages (apt/dnf/pacman)",
+                "color": "magenta",
+                "icon": "📦"
             }
         }
 
@@ -217,9 +231,22 @@ def interactive_selection(self, plan: Dict[str, Any]) -> Dict[str, Any]:
     def _dict_to_action(self, suggestion: Dict) -> CleanupAction:
         """Convert dictionary suggestion to CleanupAction"""
         try:
+            # Need to handle strings if suggestion payload comes in as string not Enum
+            ctype_val = suggestion.get("type", "large_file")
+            try:
+                ctype = CleanupType(ctype_val)
+            except ValueError:
+                ctype = CleanupType.LARGE_FILE
+                
+            prio_val = suggestion.get("priority", "low")
+            try:
+                prio = Priority(prio_val)
+            except ValueError:
+                prio = Priority.LOW
+
             return CleanupAction(
-                type=CleanupType(suggestion.get("type", "large_file")),
-                priority=Priority(suggestion.get("priority", "low")),
+                type=ctype,
+                priority=prio,
                 path=suggestion.get("path", ""),
                 size_gb=float(suggestion.get("size_gb", 0)),
                 description=suggestion.get("description", ""),
@@ -269,6 +296,10 @@ def _get_category_for_action(self, action: CleanupAction) -> str:
             return "temp"
         elif action.type == CleanupType.LARGE_FILE:
             return "large_files"
+        elif action.type.value == "docker_cleanup" or "docker" in action.path.lower():
+            return "docker"
+        elif action.type.value == "package_cleanup" or action.path == "/var/cache":
+            return "package_manager"
         elif "system" in action.path.lower() or action.path.startswith("/"):
             return "system"
         else: