Update app.py

app.py

@@ -208,6 +208,7 @@ def check_tool_availability():
 
 # --- Pipeline Functions ---
 def phylogenetic_placement(sequence: str, mafft_cmd: str, iqtree_cmd: str):
+    query_fasta = None
     try:
         if len(sequence.strip()) < 100:
             return False, "Sequence too short (<100 bp).", None, None
@@ -238,12 +239,12 @@ def phylogenetic_placement(sequence: str, mafft_cmd: str, iqtree_cmd: str):
         logger.error(f"Phylogenetic placement failed: {e}", exc_info=True)
         return False, f"Error: {str(e)}", None, None
     finally:
-        if 'query_fasta' in locals() and os.path.exists(query_fasta):
+        # Fixed: Properly handle cleanup with specific exception handling
+        if query_fasta and os.path.exists(query_fasta):
             try:
                 os.unlink(query_fasta)
-            except Exception as e:  # Fixed bare 'except'
-                logger.warning(f"Failed to clean up {query_fasta}: {e}")
-
+            except Exception as cleanup_error:
+                logger.warning(f"Failed to clean up {query_fasta}: {cleanup_error}")
 def analyze_sequence_for_tree(sequence: str, matching_percentage: float):
     try:
         logger.debug("Starting tree analysis...")