Fix NameError, improve logging and UUID handling

app/api/routes/analysis.py

@@ -10,7 +10,10 @@ from app.schemas.all_schemas import AnalysisRequest, AnalysisResponse
 from uuid import UUID
 import uuid
 from typing import List, Optional
+import json
 import os
+import re
+import traceback
 import glob
 from app.services.mock_intelligence import MockIntelligenceService
 from app.rag.pdf_parser import PDFParser
@@ -130,6 +133,7 @@ def _normalize_k2_result_for_frontend(result_dict: dict) -> dict:
     from app.core.logging import logger
     
     logger.info(f"NORMALIZE: Input result keys: {list(result_dict.keys())}")
+    logger.info(f"NORMALIZE: Status is: {result_dict.get('status')}")
     
     # Ensure comparative_analysis exists and has required structure
     if 'comparative_analysis' not in result_dict or not result_dict['comparative_analysis']:
@@ -211,13 +215,21 @@ async def get_chat_history(
     db: Session = Depends(get_db)
 ):
     """Récupère l'historique de chat pour une analyse"""
-    if analysis_id.startswith("demo_"):
+    if str(analysis_id).startswith("demo_") or not any(c in str(analysis_id) for c in "0123456789abcdef"):
         return []
     
     try:
         from app.db.repositories.chat_repo import ChatRepository
         chat_repo = ChatRepository(db)
-        messages = chat_repo.get_history(UUID(analysis_id))
+        
+        # Validate UUID format
+        try:
+            target_id = UUID(analysis_id)
+        except ValueError:
+            logger.warning(f"Invalid UUID for chat history: {analysis_id}")
+            return []
+            
+        messages = chat_repo.get_history(target_id)
         
         return [
             {

app/services/analysis_service.py

@@ -175,17 +175,24 @@ class AnalysisService:
 
         except Exception as e:
             db.rollback() # CLEAN TRANSACTION
+            logger.error(f"FATAL ERROR in process_analysis for {analysis_id}: {str(e)}")
             import traceback
-            logger.error(f"Background analysis error for {analysis_id}: {str(e)}")
             logger.error(traceback.format_exc())
+            
             try:
                 error_data = {
                     "status": "FAILED",
-                    "reasoning_summary": f"Erreur technique lors de l'analyse : {str(e)}",
+                    "reasoning_summary": f"Erreur technique lors de l'analyse : {str(e)}\n\nTrace: {traceback.format_exc()[:500]}...",
                     "confidence_overall": 0
                 }
-                self.complete_analysis(UUID(analysis_id), status="FAILED", result=error_data)
-                db.commit()
+                # Use a safe UUID conversion
+                try:
+                    target_uuid = UUID(analysis_id) if isinstance(analysis_id, str) else analysis_id
+                    self.complete_analysis(target_uuid, status="FAILED", result=error_data)
+                    db.commit()
+                    logger.info(f"Marked analysis {analysis_id} as FAILED in DB")
+                except Exception as uuid_err:
+                    logger.error(f"Could not convert {analysis_id} to UUID or save failure: {uuid_err}")
             except Exception as final_err:
                 logger.error(f"Failed to even mark analysis as FAILED: {final_err}")
                 db.rollback()

app/services/k2_think_engine.py

@@ -13,6 +13,7 @@ from app.reasoning.k2_client import K2ThinkClient
 from app.core.settings import settings
 from app.core.logging import logger
 import json
+import os
 
 
 class K2ThinkEngine: