Fix LoggerManager and webapp URL imports

- Fix LoggerManager usage: use LoggerManager().get_logger() instead of LoggerManager.get_logger()
- Fix webapp URL import: use get_absolute_userpod_api_url() instead of non-existent get_webapp_url()
- All imports and functionality tests now pass

Author: hc2p

deepnote_toolkit/execution_timeout.py

@@ -11,7 +11,7 @@
 import requests
 from IPython.core.interactiveshell import ExecutionInfo, ExecutionResult
 
-from .get_webapp_url import get_webapp_url
+from .get_webapp_url import get_absolute_userpod_api_url
 from .logging import LoggerManager
 
 
@@ -34,7 +34,7 @@ def __init__(
             timeout_seconds: Seconds after which to consider execution stuck (default: 300s = 5min)
             enable_auto_interrupt: Whether to automatically interrupt stuck executions (default: False)
         """
-        self.logger = LoggerManager.get_logger("execution_timeout")
+        self.logger = LoggerManager().get_logger()
         self.warning_threshold = warning_threshold_seconds
         self.timeout_threshold = timeout_seconds
         self.enable_auto_interrupt = enable_auto_interrupt
@@ -152,19 +152,8 @@ def _report_to_webapp(
             warning: Whether this is a warning (True) or timeout (False)
         """
         try:
-            webapp_url = get_webapp_url()
-            project_id = os.getenv("DEEPNOTE_PROJECT_ID")
-
-            if not webapp_url or not project_id:
-                self.logger.debug(
-                    "Webapp URL or project ID not available, skipping report"
-                )
-                return
-
-            endpoint = (
-                "warning" if warning else "timeout"
-            )
-            url = f"{webapp_url}/userpod-api/{project_id}/execution/{endpoint}"
+            endpoint = "warning" if warning else "timeout"
+            url = get_absolute_userpod_api_url(f"execution/{endpoint}")
 
             payload = {
                 "duration": duration,
@@ -209,7 +198,7 @@ def setup_execution_timeout_monitor(
 
         ip = get_ipython()
         if ip is None:
-            LoggerManager.get_logger("execution_timeout").warning(
+            LoggerManager().get_logger().warning(
                 "IPython instance not available, skipping timeout monitor setup"
             )
             return
@@ -224,14 +213,14 @@ def setup_execution_timeout_monitor(
         ip.events.register("pre_execute", _timeout_monitor.on_pre_execute)
         ip.events.register("post_execute", _timeout_monitor.on_post_execute)
 
-        LoggerManager.get_logger("execution_timeout").info(
+        LoggerManager().get_logger().info(
             "Execution timeout monitor initialized: warning=%ds, timeout=%ds, auto_interrupt=%s",
             warning_threshold_seconds,
             timeout_seconds,
             enable_auto_interrupt,
         )
 
     except Exception as e:  # pylint: disable=broad-exception-caught
-        LoggerManager.get_logger("execution_timeout").error(
+        LoggerManager().get_logger().error(
             "Failed to set up timeout monitor: %s", e
         )

deepnote_toolkit/execution_tracking.py

@@ -17,7 +17,7 @@ class ExecutionTracker:
     """Tracks execution state of notebook cells."""
 
     def __init__(self):
-        self.logger = LoggerManager.get_logger("execution_tracker")
+        self.logger = LoggerManager().get_logger()
         self.current_execution: Optional[Dict[str, Any]] = None
         self.execution_count = 0
 
@@ -109,7 +109,7 @@ def setup_execution_tracking() -> None:
 
         ip = get_ipython()
         if ip is None:
-            LoggerManager.get_logger("execution_tracking").warning(
+            LoggerManager().get_logger().warning(
                 "IPython instance not available, skipping execution tracking setup"
             )
             return
@@ -122,11 +122,11 @@ def setup_execution_tracking() -> None:
         ip.events.register("pre_run_cell", _execution_tracker.on_pre_run_cell)
         ip.events.register("post_run_cell", _execution_tracker.on_post_run_cell)
 
-        LoggerManager.get_logger("execution_tracking").info(
+        LoggerManager().get_logger().info(
             "Execution tracking initialized successfully"
         )
 
     except Exception as e:  # pylint: disable=broad-exception-caught
-        LoggerManager.get_logger("execution_tracking").error(
+        LoggerManager().get_logger().error(
             "Failed to set up execution tracking: %s", e
         )