Update app.py

app.py

@@ -1109,16 +1109,30 @@ async def run_oss_analysis(scenario_name: str):
         )
 
 # ===========================================
-# REAL ENTERPRISE EXECUTION HANDLER
+# REAL ENTERPRISE EXECUTION HANDLER - FIXED VERSION
 # ===========================================
 def execute_enterprise_healing(scenario_name, approval_required, mcp_mode_value):
-    """Execute enterprise healing with real ARF"""
+    """Execute enterprise healing with real ARF - FIXED VERSION"""
+    import gradio as gr
+    
     scenario = get_components()["INCIDENT_SCENARIOS"].get(scenario_name, {})
     
     # Determine mode
     mode = "Approval" if approval_required else "Autonomous"
     if "Advisory" in mcp_mode_value:
-        return gr.HTML.update(value="<div style='padding: 20px; background: #fef2f2; border-radius: 14px;'><p>❌ Cannot execute in Advisory mode. Switch to Approval or Autonomous mode.</p></div>"), {}, []
+        approval_html = """
+        <div style='padding: 20px; background: #fef2f2; border-radius: 14px;'>
+            <p>❌ Cannot execute in Advisory mode. Switch to Approval or Autonomous mode.</p>
+        </div>
+        """
+        enterprise_results = {
+            "status": "❌ Cannot Execute",
+            "error": "Advisory mode only supports analysis, not execution",
+            "requires_mode_change": True,
+            "suggested_mode": "approval"
+        }
+        execution_table_data = get_audit_manager().get_execution_table()
+        return gr.HTML.update(value=approval_html), enterprise_results, execution_table_data
     
     # Calculate savings based on scenario
     impact = scenario.get("business_impact", {})
@@ -1153,13 +1167,13 @@ def execute_enterprise_healing(scenario_name, approval_required, mcp_mode_value)
         """
         
         enterprise_results = {
+            "status": "⏳ Awaiting Approval",
             "execution_mode": mode,
             "scenario": scenario_name,
             "timestamp": datetime.datetime.now().isoformat(),
-            "status": "awaiting_approval",
             "actions_queued": [
                 "Scale resources based on ML recommendations",
-                "Implement circuit breaker pattern",
+                "Implement circuit breaker pattern", 
                 "Deploy enhanced monitoring",
                 "Update RAG memory with outcome"
             ],
@@ -1208,14 +1222,14 @@ def execute_enterprise_healing(scenario_name, approval_required, mcp_mode_value)
                 """
                 
                 enterprise_results = {
+                    "status": "✅ Execution Successful",
                     "execution_mode": mode,
                     "scenario": scenario_name,
                     "timestamp": datetime.datetime.now().isoformat(),
-                    "status": "executed",
                     "actions_executed": [
                         "✅ Scaled resources based on ML recommendations",
                         "✅ Implemented circuit breaker pattern",
-                        "✅ Deployed enhanced monitoring",
+                        "✅ Deployed enhanced monitoring", 
                         "✅ Updated RAG memory with outcome"
                     ],
                     "business_impact": {
@@ -1247,11 +1261,12 @@ def execute_enterprise_healing(scenario_name, approval_required, mcp_mode_value)
                 """
                 
                 enterprise_results = {
+                    "status": "❌ Execution Failed",
                     "execution_mode": mode,
                     "scenario": scenario_name,
                     "timestamp": datetime.datetime.now().isoformat(),
-                    "status": "failed",
-                    "error": execution_result.get("message", "Unknown error")
+                    "error": execution_result.get("message", "Unknown error"),
+                    "suggestion": "Check logs and try again in approval mode"
                 }
                 
         except Exception as e:
@@ -1270,18 +1285,18 @@ def execute_enterprise_healing(scenario_name, approval_required, mcp_mode_value)
             """
             
             enterprise_results = {
+                "status": "❌ Execution Error",
                 "execution_mode": mode,
                 "scenario": scenario_name,
                 "timestamp": datetime.datetime.now().isoformat(),
-                "status": "error",
-                "error": str(e)
+                "error": str(e),
+                "suggestion": "Try running OSS analysis first"
             }
     
     # Update execution table
     execution_table_data = get_audit_manager().get_execution_table()
     
-    return approval_html, enterprise_results, execution_table_data
-
+    return gr.HTML.update(value=approval_html), enterprise_results, execution_table_data
 # ===========================================
 # CREATE DEMO INTERFACE
 # ===========================================