fix: add 5s timeout to target.page() and reduce protocolTimeout to 30s

Author: 20syldev

src/McpContext.ts

@@ -578,7 +578,20 @@ export class McpContext implements Context {
     const pageResults = await Promise.all(
       pageTargets.map(async target => {
         try {
-          return await target.page();
+          const page = await Promise.race([
+            target.page(),
+            new Promise<null>(resolve =>
+              setTimeout(() => resolve(null), DEFAULT_TIMEOUT),
+            ),
+          ]);
+          if (!page) {
+            this.logger(
+              'Timed out attaching to target at',
+              target.url(),
+              '— likely frozen or discarded',
+            );
+          }
+          return page;
         } catch (err) {
           this.logger(
             'Skipping frozen/discarded target at',
@@ -599,7 +612,12 @@ export class McpContext implements Context {
 
     for (const target of extensionTargets) {
       // Right now target.page() returns null for popup and side panel pages.
-      let page = await target.page();
+      let page = await Promise.race([
+        target.page(),
+        new Promise<null>(resolve =>
+          setTimeout(() => resolve(null), DEFAULT_TIMEOUT),
+        ),
+      ]);
       if (!page) {
         // We need to cache pages instances for targets because target.asPage()
         // returns a new page instance every time.

src/browser.ts

@@ -61,6 +61,7 @@ export async function ensureBrowserConnected(options: {
     targetFilter: makeTargetFilter(enableExtensions),
     defaultViewport: null,
     handleDevToolsAsPage: true,
+    protocolTimeout: 30_000,
   };
 
   let autoConnect = false;
@@ -229,6 +230,7 @@ export async function launch(options: McpLaunchOptions): Promise<Browser> {
       acceptInsecureCerts: options.acceptInsecureCerts,
       handleDevToolsAsPage: true,
       enableExtensions: options.enableExtensions,
+      protocolTimeout: 30_000,
     });
     if (options.logFile) {
       // FIXME: we are probably subscribing too late to catch startup logs. We