fix(sandbox): add 30-second timeout to proxy wait loops

Author: spencer426

packages/cli/src/utils/sandbox.ts

@@ -184,8 +184,9 @@ export async function start_sandbox(
           );
         });
         debugLogger.log('waiting for proxy to start ...');
-        // Simple wait loop using spawnAsync
-        while (true) {
+        // Simple wait loop using spawnAsync with 30s timeout
+        const start = Date.now();
+        while (Date.now() - start < 30000) {
           try {
             await spawnAsync('curl', ['-s', 'http://localhost:8877'], {
               timeout: 250,
@@ -194,6 +195,11 @@ export async function start_sandbox(
           } catch {
             await new Promise((r) => setTimeout(r, 250));
           }
+          if (Date.now() - start >= 30000) {
+            throw new FatalSandboxError(
+              'Timed out waiting for proxy to start after 30 seconds',
+            );
+          }
         }
       }
       // spawn child and let it inherit stdio
@@ -718,8 +724,9 @@ export async function start_sandbox(
         );
       });
       debugLogger.log('waiting for proxy to start ...');
-      // Simple wait loop using spawnAsync
-      while (true) {
+      // Simple wait loop using spawnAsync with 30s timeout
+      const start = Date.now();
+      while (Date.now() - start < 30000) {
         try {
           await spawnAsync('curl', ['-s', 'http://localhost:8877'], {
             timeout: 250,
@@ -728,6 +735,11 @@ export async function start_sandbox(
         } catch {
           await new Promise((r) => setTimeout(r, 250));
         }
+        if (Date.now() - start >= 30000) {
+          throw new FatalSandboxError(
+            'Timed out waiting for proxy to start after 30 seconds',
+          );
+        }
       }
       // connect proxy container to sandbox network
       // (workaround for older versions of docker that don't support multiple --network args)