chore: roll puppeteer (#25)

This should solve flakiness related to multiple browser instances.

Author: OrKoN

package-lock.json

@@ -35,7 +35,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "1.18.0",
     "debug": "4.4.3",
-    "puppeteer-core": "24.20.0",
+    "puppeteer-core": "24.21.0",
     "yargs": "18.0.0"
   },
   "devDependencies": {
@@ -51,7 +51,7 @@
     "eslint": "^9.35.0",
     "globals": "^16.4.0",
     "prettier": "^3.6.2",
-    "puppeteer": "^24.20.0",
+    "puppeteer": "24.21.0",
     "sinon": "^21.0.0",
     "typescript": "^5.9.2",
     "typescript-eslint": "^8.43.0"

package.json

@@ -113,9 +113,7 @@ export async function launch(options: McpLaunchOptions): Promise<Browser> {
     // your profile directory` instead.
     if (
       userDataDir &&
-      (error as Error).message.includes(
-        '(Target.setDiscoverTargets): Target closed',
-      )
+      (error as Error).message.includes('The browser is already running')
     ) {
       throw new Error(
         `The browser is already running for ${userDataDir}. Use --isolated to run multiple browser instances.`,

src/browser.ts

@@ -147,8 +147,6 @@ describe('input', () => {
           return Number(button?.textContent);
         });
 
-        console.log(handlerResolveTime, buttonChangeTime);
-
         assert(handlerResolveTime > buttonChangeTime, 'Waited for navigation');
       });
     });