:sapling: Add more verbose logs to solution server

Signed-off-by: Fabian von Feilitzsch <[email protected]>

Author: fabianvf

kai_mcp_solution_server/src/kai_mcp_solution_server/constants.py

@@ -1,10 +1,32 @@
-# TODO: More robust logging
+import logging
 import sys
+from datetime import datetime
 from typing import Any
 
+# Configure logging with more detail
+logging.basicConfig(
+    level=logging.INFO,
+    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
+    handlers=[
+        logging.FileHandler("kai_mcp_server.log"),
+        logging.StreamHandler(sys.stderr),
+    ],
+)
+
+logger = logging.getLogger("kai_mcp_solution_server")
+
+# Keep the original log function for compatibility but enhance it
 log_file = open("stderr.log", "a+")
 log_file.close()
 
 
 def log(*args: Any, **kwargs: Any) -> None:
-    print(*args, file=log_file if not log_file.closed else sys.stderr, **kwargs)
+    """Legacy log function - enhanced with timestamp and logging level"""
+    timestamp = datetime.now().isoformat()
+    message = " ".join(str(arg) for arg in args)
+    logger.info(message)
+    print(
+        f"[{timestamp}] {message}",
+        file=log_file if not log_file.closed else sys.stderr,
+        **kwargs,
+    )

kai_mcp_solution_server/src/kai_mcp_solution_server/db/dao.py

@@ -1,9 +1,12 @@
 from __future__ import annotations
 
+import logging
 import sys
 from datetime import datetime
 from typing import Any
 
+logger = logging.getLogger("kai_mcp_solution_server.db")
+
 from sqlalchemy import (
     ARRAY,
     JSON,
@@ -308,12 +311,21 @@ class DBSolution(Base):
     )
 
     def update_solution_status(self) -> None:
+        previous_status = self.solution_status
         for file in self.after:
             if file.status != SolutionStatus.ACCEPTED:
                 self.solution_status = file.status
+                if previous_status != self.solution_status:
+                    logger.info(
+                        f"[DB_STATE_CHANGE] Solution {self.id} status auto-updated: {previous_status} -> {self.solution_status} (due to file {file.uri} with status {file.status})"
+                    )
                 return
 
         self.solution_status = SolutionStatus.ACCEPTED
+        if previous_status != self.solution_status:
+            logger.info(
+                f"[DB_STATE_CHANGE] Solution {self.id} status auto-updated: {previous_status} -> ACCEPTED (all files accepted)"
+            )
 
     def __hash__(self) -> int:
         return hash(self.id)