Update app.py

app.py

@@ -221,7 +221,8 @@ def _convert_gr_history_to_api_messages(system_prompt, gr_history, current_user_
     return messages
 
 def _generate_ui_outputs_from_cache(owner, space_name):
-    global parsed_code_blocks_state_cache
+    # This function only reads the global variable, does not need 'global' declaration
+    global parsed_code_blocks_state_cache # Added global here just in case, although technically not needed for reads only.
     preview_md_val = "*No files in cache to display.*"
     formatted_md_val = f"# Space: {owner}/{space_name}\n## File Structure\n{bbb}\n📁 Root\n{bbb}\n\n*No files in cache.*" if owner or space_name else "*Load or define a Space to see its Markdown structure.*"
     download_file = None
@@ -263,6 +264,8 @@ def handle_chat_submit(user_message, chat_history, api_key_input, provider_selec
         try:
             parsed_from_md = build_logic_parse_markdown(_current_formatted_markdown)
             new_cache_state = []
+            # This loop accesses parsed_code_blocks_state_cache, but the global declaration
+            # is correctly placed *before* this use in this function.
             existing_structure_block = next((b for b in parsed_code_blocks_state_cache if b.get("is_structure_block")), None)
             if existing_structure_block:
                  new_cache_state.append(existing_structure_block.copy())
@@ -291,6 +294,9 @@ def handle_chat_submit(user_message, chat_history, api_key_input, provider_selec
                                  "is_structure_block": False
                             })
             new_cache_state.sort(key=lambda b: (0, b["filename"]) if b.get("is_structure_block") else (1, b["filename"]))
+            # This is an assignment to parsed_code_blocks_state_cache.
+            # The 'global' declaration needs to be before the first assignment *or* use.
+            # It is correctly placed at the start of the function.
             parsed_code_blocks_state_cache = new_cache_state
 
         except Exception as e:
@@ -306,6 +312,7 @@ def handle_chat_submit(user_message, chat_history, api_key_input, provider_selec
     current_sys_prompt = system_prompt.strip() or DEFAULT_SYSTEM_PROMPT
 
     current_files_context = ""
+    # This block accesses parsed_code_blocks_state_cache, after the global declaration.
     if parsed_code_blocks_state_cache:
         current_files_context = "\n\n## Current Files in Space\n"
         for block in parsed_code_blocks_state_cache:
@@ -348,13 +355,15 @@ def handle_chat_submit(user_message, chat_history, api_key_input, provider_selec
         elif full_bot_response_content and not full_bot_response_content.startswith("Error: "):
             _status = f"Streaming complete. Processing files from {model_select} response..."
 
+            # This block accesses and assigns to parsed_code_blocks_state_cache,
+            # after the global declaration at the top of the function.
             parsing_res = _parse_chat_stream_logic(full_bot_response_content, existing_files_state=parsed_code_blocks_state_cache)
 
             if parsing_res["error_message"]:
                 _status = f"Parsing Error: {parsing_res['error_message']}"
                 _detected_files_update = gr.Markdown(f"## Parsing Error\n`{escape_html_for_markdown(parsing_res['error_message'])}`")
             else:
-                global parsed_code_blocks_state_cache
+                global parsed_code_blocks_state_cache # Redundant global, but harmless. The one at the top is effective.
                 parsed_code_blocks_state_cache = parsing_res["parsed_code_blocks"]
 
                 _formatted_output_update, _detected_files_update, _download_btn_update = _generate_ui_outputs_from_cache(hf_owner_name, hf_repo_name)
@@ -502,7 +511,7 @@ def handle_build_space_button(hf_api_key_ui, ui_space_name_part, ui_owner_name_p
 
     try:
         parsed_from_md_for_build = build_logic_parse_markdown(formatted_markdown_content)
-        parsed_code_blocks_state_cache = []
+        parsed_code_blocks_state_cache = [] # Assignment to global variable
         if parsed_from_md_for_build.get("owner_md"):
              ui_owner_name_part = parsed_from_md_for_build["owner_md"]
         if parsed_from_md_for_build.get("repo_name_md"):
@@ -510,7 +519,7 @@ def handle_build_space_button(hf_api_key_ui, ui_space_name_part, ui_owner_name_p
 
         structure_block_md = next((f for f in parsed_from_md_for_build.get("files", []) if f.get("path") == "File Structure (original)"), None)
         if structure_block_md:
-             parsed_code_blocks_state_cache.append({
+             parsed_code_blocks_state_cache.append({ # Modifies global list
                  "filename": structure_block_md["path"],
                  "code": structure_block_md["content"],
                  "language": "plaintext",
@@ -521,14 +530,14 @@ def handle_build_space_button(hf_api_key_ui, ui_space_name_part, ui_owner_name_p
         for f_info in parsed_from_md_for_build.get("files", []):
              if f_info.get("path") and f_info["path"] != "File Structure (original)":
                   is_binary_repr = isinstance(f_info.get("content"), str) and (f_info["content"].startswith("[Binary file") or f_info["content"].startswith("[Error loading content:") or f_info["content"].startswith("[Binary or Skipped file]"))
-                  parsed_code_blocks_state_cache.append({
+                  parsed_code_blocks_state_cache.append({ # Modifies global list
                        "filename": f_info["path"],
                        "code": f_info.get("content", ""),
                        "language": "binary" if is_binary_repr else _infer_lang_from_filename(f_info["path"]),
                        "is_binary": is_binary_repr,
                        "is_structure_block": False
                   })
-        parsed_code_blocks_state_cache.sort(key=lambda b: (0, b["filename"]) if b.get("is_structure_block") else (1, b["filename"]))
+        parsed_code_blocks_state_cache.sort(key=lambda b: (0, b["filename"]) if b.get("is_structure_block") else (1, b["filename"])) # Modifies global list
 
     except Exception as e:
          _build_status = f"Build Error: Failed to parse Markdown structure before building: {e}";
@@ -554,7 +563,7 @@ def handle_build_space_button(hf_api_key_ui, ui_space_name_part, ui_owner_name_p
     yield _build_status, _iframe_html, _file_browser_update, owner_name_output, space_name_output
 
 def handle_load_file_for_editing(hf_api_key_ui, ui_space_name_part, ui_owner_name_part, selected_file_path):
-    global parsed_code_blocks_state_cache
+    global parsed_code_blocks_state_cache # This function only reads, does not strictly need global.
     _file_content_val, _edit_status_val, _commit_msg_val, _lang_update = "", "Error: No file selected.", gr.update(value=""), gr.update(language="python")
     if not selected_file_path or selected_file_path in ["No files found", "Error loading files", "Error refreshing files"]:
          yield _file_content_val, "Select a file from the dropdown.", _commit_msg_val, _lang_update
@@ -625,6 +634,8 @@ def handle_commit_file_changes(hf_api_key_ui, ui_space_name_part, ui_owner_name_
 
     if "Successfully updated" in status_msg:
         found_in_cache = False
+        # Modifies item within the global list. No need for global declaration here if the list itself isn't reassigned.
+        # But if the file was new, it might be added later, which requires global.
         for block in parsed_code_blocks_state_cache:
             if block["filename"] == file_to_edit_path:
                 block["code"] = edited_content
@@ -634,6 +645,7 @@ def handle_commit_file_changes(hf_api_key_ui, ui_space_name_part, ui_owner_name_
                 found_in_cache = True
                 break
         if not found_in_cache:
+             # Assignment to global variable
              parsed_code_blocks_state_cache = [b for b in parsed_code_blocks_state_cache if b["filename"] != file_to_edit_path]
              parsed_code_blocks_state_cache.append({
                  "filename": file_to_edit_path,
@@ -642,7 +654,7 @@ def handle_commit_file_changes(hf_api_key_ui, ui_space_name_part, ui_owner_name_
                  "is_binary": False,
                  "is_structure_block": False
              })
-             parsed_code_blocks_state_cache.sort(key=lambda b: (0, b["filename"]) if b.get("is_structure_block") else (1, b["filename"]))
+             parsed_code_blocks_state_cache.sort(key=lambda b: (0, b["filename"]) if b.get("is_structure_block") else (1, b["filename"))
 
         _formatted_md_out, _detected_preview_out, _download_btn_out = _generate_ui_outputs_from_cache(owner_to_use, ui_space_name_part)
 
@@ -692,6 +704,7 @@ def handle_delete_file(hf_api_key_ui, ui_space_name_part, ui_owner_name_part, fi
     new_file_list, err_list = list_space_files_for_browsing(hf_api_key_ui, ui_space_name_part, owner_to_use)
 
     if "Successfully deleted" in deletion_status_msg:
+        # Assignment to global variable
         parsed_code_blocks_state_cache = [b for b in parsed_code_blocks_state_cache if b["filename"] != file_to_delete_path]
         _formatted_md_out, _detected_preview_out, _download_btn_out = _generate_ui_outputs_from_cache(owner_to_use, ui_space_name_part)
 
@@ -887,7 +900,7 @@ with gr.Blocks(theme=custom_theme, css=custom_css) as demo:
     delete_file_outputs = [edit_status_display, file_browser_dropdown, file_browser_dropdown, file_content_editor, commit_message_input, file_content_editor, formatted_space_output_display, detected_files_preview, download_button]
     delete_file_button.click(fn=handle_delete_file, inputs=[hf_api_key_input, space_name_input, owner_name_input, file_browser_dropdown], outputs=delete_file_outputs)
 
-    refresh_status_button.click(fn=handle_refresh_space_status, inputs=[hf_api_key_input, owner_name_input, space_name_input], outputs=[space_runtime_status_display])
+    refresh_space_status_button.click(fn=handle_refresh_space_status, inputs=[hf_api_key_input, owner_name_input, space_name_input], outputs=[space_runtime_status_display])
 
 if __name__ == "__main__":
     demo.launch(debug=True, share=False)