runbooks 1.1.1__py3-none-any.whl → 1.1.3__py3-none-any.whl

runbooks/finops/workspaces_analyzer.py

@@ -75,7 +75,7 @@ class WorkSpacesCostAnalyzer:
     """
     WorkSpaces cost optimization analyzer following enterprise patterns.
     
-    Implements FinOps-24 requirements with proven profile management and Rich CLI standards.
+    Implements WorkSpaces optimization requirements with proven profile management and Rich CLI standards.
     """
     
     def __init__(self, profile: Optional[str] = None):
@@ -84,7 +84,7 @@ class WorkSpacesCostAnalyzer:
         self.profile = get_profile_for_operation("operational", profile)
         self.session = boto3.Session(profile_name=self.profile)
         
-        # FinOps-24 business targets
+        # WorkSpaces optimization business targets
         self.target_annual_savings = 12518.0
         self.unused_threshold_days = 90
         self.analysis_period_days = 30
@@ -286,7 +286,7 @@ class WorkSpacesCostAnalyzer:
         print_header("WorkSpaces Cost Analysis Summary")
         
         summary_table = create_table(
-            title="FinOps-24: WorkSpaces Optimization Summary",
+            title="WorkSpaces Optimization Summary",
             columns=[
                 {"header": "Metric", "style": "cyan"},
                 {"header": "Count", "style": "green bold"},
@@ -532,7 +532,7 @@ class WorkSpacesCostAnalyzer:
         }
 
 
-def analyze_workspaces_finops_24(
+def analyze_workspaces(
     profile: Optional[str] = None,
     unused_days: int = 90,
     analysis_days: int = 30,
@@ -541,7 +541,7 @@ def analyze_workspaces_finops_24(
     dry_run: bool = True
 ) -> Dict[str, Any]:
     """
-    FinOps-24 WorkSpaces analysis wrapper for CLI and notebook integration.
+    WorkSpaces analysis wrapper for CLI and notebook integration.
     
     Args:
         profile: AWS profile to use
@@ -555,6 +555,11 @@ def analyze_workspaces_finops_24(
         Analysis results with cost optimization recommendations
     """
     try:
+        # Initialize variables to prevent scope errors
+        results = []
+        summary = None
+        export_file = None
+
         analyzer = WorkSpacesCostAnalyzer(profile=profile)
         results, summary = analyzer.analyze_workspaces(
             unused_days=unused_days,
@@ -573,17 +578,36 @@ def analyze_workspaces_finops_24(
             )
         
         # Return comprehensive results
-        return {
-            "summary": summary.to_dict(),
-            "workspaces": [result.to_dict() for result in results],
-            "export_file": export_file,
-            "achievement_rate": summary.target_achievement_rate,
-            "status": "success"
-        }
-        
+        if summary is not None:
+            return {
+                "summary": summary.to_dict(),
+                "workspaces": [result.to_dict() for result in results],
+                "export_file": export_file,
+                "achievement_rate": summary.target_achievement_rate,
+                "status": "success"
+            }
+        else:
+            return {
+                "summary": {"error": "Analysis failed before completion"},
+                "workspaces": [],
+                "export_file": None,
+                "achievement_rate": 0,
+                "status": "partial_failure"
+            }
+
     except Exception as e:
-        print_error(f"FinOps-24 analysis failed: {e}")
+        print_error(f"WorkSpaces analysis failed: {e}")
         return {
             "error": str(e),
-            "status": "failed"
-        }
+            "status": "failed",
+            "summary": {"error": str(e)},
+            "workspaces": [],
+            "export_file": None,
+            "achievement_rate": 0
+        }
+
+
+# Legacy alias for backward compatibility
+def analyze_workspaces_finops_24(*args, **kwargs):
+    """Legacy alias for analyze_workspaces - deprecated, use analyze_workspaces instead."""
+    return analyze_workspaces(*args, **kwargs)