Address code review feedback - move imports and clarify lambda closures

Co-authored-by: codingwithnsh <138281862+codingwithnsh@users.noreply.github.com>

Author: Copilot

sandbox_dashboard.py

@@ -473,9 +473,10 @@ def select_template(tid=template_id, frame=btn_frame):
                 update_description()
                 update_limits()
 
-            btn_frame.bind('<Button-1>', lambda e, tid=template_id, frame=btn_frame: select_template(tid, frame))
+            # Bind click events - using default arguments to properly capture loop variables
+            btn_frame.bind('<Button-1>', lambda e, t=template_id, f=btn_frame: select_template(t, f))
             for child in btn_frame.winfo_children():
-                child.bind('<Button-1>', lambda e, tid=template_id, frame=btn_frame: select_template(tid, frame))
+                child.bind('<Button-1>', lambda e, t=template_id, f=btn_frame: select_template(t, f))
 
             template_buttons.append(btn_frame)
 

sandbox_manager.py

@@ -10,6 +10,7 @@
 import threading
 import time
 import uuid
+import traceback
 from pathlib import Path
 from datetime import datetime
 import psutil
@@ -70,7 +71,6 @@ def create(self):
             return True
         except (OSError, PermissionError) as e:
             print(f"Error creating sandbox: {e}")
-            import traceback
             traceback.print_exc()
             return False
 
@@ -323,7 +323,6 @@ def create_sandbox(self, name, sandbox_type="general", config=None):
             return sandbox
         except Exception as e:
             print(f"Error creating sandbox: {e}")
-            import traceback
             traceback.print_exc()
             return None