runbooks 0.9.2__py3-none-any.whl → 0.9.4__py3-none-any.whl

runbooks/finops/multi_dashboard.py

@@ -68,12 +68,12 @@ from .cost_processor import (
     get_cost_data,
     process_service_costs,
 )
-from .dashboard_runner import (
-    _create_cost_session,
-    _create_management_session,
-    _create_operational_session,
-    _initialize_profiles,
+from runbooks.common.profile_utils import (
+    create_cost_session,
+    create_management_session,
+    create_operational_session,
 )
+from .dashboard_runner import _initialize_profiles
 from .enhanced_progress import track_multi_account_analysis
 from .helpers import export_cost_dashboard_to_pdf
 from .service_mapping import get_service_display_name
@@ -564,8 +564,8 @@ class MultiAccountDashboard:
                 display_profile = profile
 
             # Initialize sessions using base profile
-            cost_session = _create_cost_session(base_profile)
-            mgmt_session = _create_management_session(base_profile)
+            cost_session = create_cost_session(base_profile)
+            mgmt_session = create_management_session(base_profile)
 
             # SRE FIX: Get account ID - use target account for Organizations API or session account
             if target_account_id:
@@ -1244,15 +1244,24 @@ class MultiAccountDashboard:
 
         top_org_services = sorted(all_services.items(), key=lambda x: x[1], reverse=True)[:5]
 
-        # Create summary panel
-        overall_trend = ((total_spend - total_last_month) / total_last_month * 100) if total_last_month > 0 else 0
-        trend_icon = "⬆" if overall_trend > 0 else "⬇" if overall_trend < 0 else "➡"
+        # Create summary panel with enhanced trend analysis
+        from .cost_processor import calculate_trend_with_context
+        
+        # For multi-account analysis, we generally have full month data, but check for consistency
+        overall_trend_display = calculate_trend_with_context(total_spend, total_last_month)
+        
+        # Extract trend direction for icon (maintaining existing functionality)
+        if total_last_month > 0:
+            overall_trend_pct = ((total_spend - total_last_month) / total_last_month * 100)
+            trend_icon = "⬆" if overall_trend_pct > 0 else "⬇" if overall_trend_pct < 0 else "➡"
+        else:
+            trend_icon = "➡"
 
         summary_text = f"""
 [highlight]Organization Summary[/]
 • Total Accounts: {len(accounts)}
 • Total Monthly Spend: {format_cost(total_spend)}
-• Overall Trend: {trend_icon} {abs(overall_trend):.1f}%
+• Overall Trend: {overall_trend_display}
 • Budget Alerts: {over_budget_count} over budget, {warning_count} warnings
 
 [highlight]Top Organization Services[/]
@@ -1426,9 +1435,15 @@ class MultiAccountDashboard:
                     f"| {account_display_escaped} | ${last:.0f} | ${current:.0f} | {services_text_escaped} | {budget_clean_escaped} | {optimization_escaped} |"
                 )
 
-            # Add summary section
-            overall_trend = ((total_current - total_last) / total_last * 100) if total_last > 0 else 0
-            trend_direction = "↗️" if overall_trend > 0 else "↘️" if overall_trend < 0 else "➡️"
+            # Add summary section with enhanced trend analysis
+            overall_trend_display = calculate_trend_with_context(total_current, total_last)
+            
+            # Extract trend direction for emoji (maintaining existing markdown export format)
+            if total_last > 0:
+                overall_trend_pct = ((total_current - total_last) / total_last * 100)
+                trend_direction = "↗️" if overall_trend_pct > 0 else "↘️" if overall_trend_pct < 0 else "➡️"
+            else:
+                trend_direction = "➡️"
 
             lines.append("")
             lines.append("## Organization Summary")
@@ -1436,7 +1451,7 @@ class MultiAccountDashboard:
             lines.append(f"- **Total Accounts Analyzed**: {len(accounts)}")
             lines.append(f"- **Total Current Month**: ${total_current:,.2f}")
             lines.append(f"- **Total Last Month**: ${total_last:,.2f}")
-            lines.append(f"- **Overall Trend**: {trend_direction} {abs(overall_trend):.1f}%")
+            lines.append(f"- **Overall Trend**: {overall_trend_display}")
             lines.append(f"- **Analysis Performance**: {execution_time:.1f}s execution")
             lines.append("")