Update app.py

app.py

@@ -4,6 +4,7 @@ import json
 import os
 import time
 import tempfile
+import re
 from configparser import ConfigParser
 from typing import List, Dict, Any
 
@@ -216,7 +217,7 @@ def build_master_log(system_log_list, coder_log_list, strategist_log_list, audit
     return html_output
 
 def execute_remote_workflow(blueprint, intent, coder_model_label, strategist_model_label, enable_audit):
-    """Execute workflow with final_code tracking"""
+    """Execute workflow with final_code and master_log tracking"""
     mission_start_time = time.time()
     final_code = None
     
@@ -234,7 +235,7 @@ def execute_remote_workflow(blueprint, intent, coder_model_label, strategist_mod
         
         with requests.post(target_url, json=payload, stream=True, timeout=300) as response:
             if response.status_code == 404:
-                yield f"ERROR 404: Streaming endpoint not found at {target_url}. Check server version.", "", "", "", None
+                yield f"ERROR 404: Streaming endpoint not found at {target_url}. Check server version.", "", "", "", None, None
                 return
             
             response.raise_for_status()
@@ -273,7 +274,8 @@ def execute_remote_workflow(blueprint, intent, coder_model_label, strategist_mod
                                                strategist_log_list, audit_log_list, 
                                                mission_start_time, final_code)
                 
-                yield sys_log, cod_log, strat_log, master_log, final_code
+                # DIRECTIVE #3: Update both master_log_output AND master_log_state
+                yield sys_log, cod_log, strat_log, master_log, final_code, master_log
         
     except requests.exceptions.RequestException as e:
         error_detail = str(e)
@@ -281,10 +283,10 @@ def execute_remote_workflow(blueprint, intent, coder_model_label, strategist_mod
             error_detail += f"\nServer Response: {e.response.text}"
             
         error_msg = f"CONNECTION ERROR: Could not connect to MCP Server.\nDetails: {error_detail}"
-        yield error_msg, "", "", "", None
+        yield error_msg, "", "", "", None, None
     except Exception as e:
         error_msg = f"CLIENT ERROR: {str(e)}"
-        yield error_msg, "", "", "", None
+        yield error_msg, "", "", "", None, None
 
 def create_interface():
     DEMO_BLUEPRINT = """# Gradio Application Blueprint
@@ -330,19 +332,20 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
     def start_execution_sequence():
         return gr.update(visible=False), gr.update(visible=True), gr.update(visible=False)
 
-    # ISSUE #1 FIX: Button visibility control in end sequence
-    def end_execution_sequence(sys_log, cod_log, strat_log, master_log, final_code):
-        # Button becomes visible only when final_code exists
+    # DIRECTIVE #3: Button visibility control in end sequence
+    def end_execution_sequence(sys_log, cod_log, strat_log, master_log, final_code, master_log_html):
+        # Buttons become visible only when final_code exists
         button_visibility = final_code is not None and final_code.strip() != ""
         return (
             sys_log, 
             cod_log, 
             strat_log, 
             master_log, 
-            final_code, 
+            final_code,
+            master_log_html,
             gr.update(visible=False),  # working_video
             gr.update(visible=True),   # celebrate_video
-            gr.update(visible=button_visibility)  # download_btn
+            gr.update(visible=button_visibility)  # download_buttons_row
         )
 
     def reset_to_default():
@@ -361,7 +364,7 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
             return gr.update(choices=[], value=None)
         return gr.update(choices=models, value=models[0])
     
-    # ISSUE #2 FIX: Download code function with proper file handling
+    # DIRECTIVE #1: NO LOGIC CHANGE - Existing download_code function preserved
     def download_code(final_code_content):
         """Generate downloadable Python file from final code"""
         if not final_code_content or final_code_content.strip() == "":
@@ -380,6 +383,35 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
         
         return filepath
     
+    # DIRECTIVE #2: NEW FUNCTION - Download master log
+    def download_master_log(master_log_html):
+        """Generate downloadable text file from master log HTML"""
+        if not master_log_html or master_log_html.strip() == "":
+            return None
+        
+        # Strip all HTML tags using regex
+        clean_text = re.sub(r'<[^>]+>', '', master_log_html)
+        
+        # Decode HTML entities
+        clean_text = clean_text.replace('&lt;', '<').replace('&gt;', '>').replace('&amp;', '&')
+        
+        # Clean up extra whitespace
+        clean_text = re.sub(r'\n\s*\n', '\n\n', clean_text)
+        clean_text = clean_text.strip()
+        
+        # Create temporary file with timestamp
+        timestamp = time.strftime("%Y%m%d_%H%M%S")
+        filename = f"master_log_{timestamp}.txt"
+        
+        # Write to temporary file and return filepath
+        temp_dir = tempfile.gettempdir()
+        filepath = os.path.join(temp_dir, filename)
+        
+        with open(filepath, 'w', encoding='utf-8') as f:
+            f.write(clean_text)
+        
+        return filepath
+    
     # RESET MISSION function
     def reset_mission():
         """Total system reset: clear all fields and reset to defaults"""
@@ -402,15 +434,17 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
             gr.update(choices=strategist_models_for_reset, value=strategist_reset_value),  # strategist_model
             gr.update(value=False),  # audit_checkbox
             None,  # final_code_state
-            gr.update(visible=False)  # ISSUE #1 FIX: Hide download button on reset
+            None,  # master_log_state
+            gr.update(visible=False)  # DIRECTIVE #3: Hide download buttons row on reset
         )
 
     with gr.Blocks(title="Project: Conductor V4.0 Client") as interface:
         gr.Markdown("# Project: Conductor V4.0 (Client Mode)")
         gr.Markdown(f"### The Command Center: Connected to MCP Server at `{MCP_SERVER_URL}`")
         
-        # Hidden state for final code
+        # DIRECTIVE #3: Hidden states for final code and master log
         final_code_state = gr.State(value=None)
+        master_log_state = gr.State(value=None)
         
         # PRIMARY INTERFACE ROW - Three Columns
         with gr.Row():
@@ -503,13 +537,18 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
                 label="MASTER LOG WINDOW"
             )
         
-        # ISSUE #1 FIX: Download button initialized as invisible
-        with gr.Row():
-            download_btn = gr.DownloadButton(
-                label="DOWNLOAD CODE (.py)", 
-                variant="primary", 
-                visible=False
-            )
+        # DIRECTIVE #3: Two-button layout with clean, constrained width
+        with gr.Row(visible=False) as download_buttons_row:
+            with gr.Column():
+                download_code_btn = gr.DownloadButton(
+                    label="DOWNLOAD GENERATED CODE", 
+                    variant="primary"
+                )
+            with gr.Column():
+                download_log_btn = gr.DownloadButton(
+                    label="DOWNLOAD MASTER LOG",
+                    variant="secondary"
+                )
         
         # EVENT HANDLERS
         
@@ -541,18 +580,24 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
         ).then(
             fn=execute_remote_workflow,
             inputs=[blueprint_input, intent_input, coder_model, strategist_model, audit_checkbox],
-            outputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state]
+            outputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state, master_log_state]
         ).then(
             fn=end_execution_sequence,
-            inputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state],
-            outputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state, working_video, celebrate_video, download_btn]
+            inputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state, master_log_state],
+            outputs=[system_terminal, coder_log, strategist_log, master_log_output, final_code_state, master_log_state, working_video, celebrate_video, download_buttons_row]
         )
         
-        # ISSUE #2 FIX: Download button handler with proper file return
-        download_btn.click(
+        # DIRECTIVE #1 & #2: Download button handlers
+        download_code_btn.click(
             fn=download_code,
             inputs=[final_code_state],
-            outputs=[download_btn]
+            outputs=[download_code_btn]
+        )
+        
+        download_log_btn.click(
+            fn=download_master_log,
+            inputs=[master_log_state],
+            outputs=[download_log_btn]
         )
         
         # Celebrate video auto-reset
@@ -562,7 +607,7 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
             outputs=[default_image, working_video, celebrate_video]
         )
         
-        # RESET MISSION button with download button visibility control
+        # RESET MISSION button with download buttons row visibility control
         reset_btn.click(
             fn=reset_mission,
             inputs=[],
@@ -579,7 +624,8 @@ Produce a pro level styled and tuned them GUI that has 1 button that when presse
                 strategist_model,
                 audit_checkbox,
                 final_code_state,
-                download_btn
+                master_log_state,
+                download_buttons_row
             ]
         )