Fix code review issues: prevent infinite recursion and add timeout to interrupt

Co-authored-by: na-trium-144 <100704180+na-trium-144@users.noreply.github.com>

Author: Copilot

app/terminal/javascript/runtime.tsx

@@ -124,12 +124,22 @@ export function JavaScriptProvider({ children }: { children: ReactNode }) {
     // Create a new worker
     initializeWorker();
 
-    // Wait for initialization - use a different approach
-    await new Promise<void>((resolve) => {
+    // Wait for initialization with timeout
+    const maxRetries = 50; // 5 seconds total
+    let retries = 0;
+    
+    await new Promise<void>((resolve, reject) => {
       const checkInterval = setInterval(() => {
+        retries++;
+        if (retries > maxRetries) {
+          clearInterval(checkInterval);
+          reject(new Error("Worker initialization timeout"));
+          return;
+        }
+        
         if (workerRef.current) {
-          // Try to initialize and restore
-          postMessage<InitPayloadFromWorker>({
+          // Try to restore state
+          postMessage<{ success: boolean }>({
             type: "restoreState",
           }).then(() => {
             clearInterval(checkInterval);

public/javascript.worker.js

@@ -46,7 +46,8 @@ async function runJavaScript(id, payload) {
     // Save the successfully executed command for state recovery
     executedCommands.push(code);
   } catch (e) {
-    console.log(e);
+    // Use self.console to avoid recursion with our console proxy
+    self.console.log(e);
     if (e instanceof Error) {
       jsOutput.push({
         type: "error",
@@ -105,7 +106,8 @@ async function restoreState(id) {
       executedCommands.push(command);
     } catch (e) {
       // If restoration fails, we still continue with other commands
-      console.error("Failed to restore command:", command, e);
+      // Use self.console to avoid recursion with our console proxy
+      self.console.error("Failed to restore command:", command, e);
     }
   }