mcpbrowser 0.2.7 → 0.2.9

package/extension/package.json

@@ -2,7 +2,7 @@
   "name": "mcpbrowser",
   "displayName": "MCP Browser",
   "description": "Extends Copilot's web access to protected pages - handles login, SSO, and anti-crawler restrictions",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "publisher": "cherchyk",
   "icon": "icon.png",
   "engines": {

package/extension/src/extension.js

@@ -39,8 +39,21 @@ async function installMcpBrowser() {
     try {
         vscode.window.showInformationMessage('Installing MCPBrowser npm package...');
         
-        // Install globally so npx can find it reliably
-        await execPromise('npm install -g mcpbrowser@latest');
+        // Try with sudo if in Linux/Mac environment (like dev containers)
+        let installCmd = 'npm install -g mcpbrowser@latest';
+        
+        // Check if we need sudo (Linux/Mac and not running as root)
+        if (process.platform !== 'win32' && process.getuid && process.getuid() !== 0) {
+            // Check if sudo is available
+            try {
+                await execPromise('which sudo');
+                installCmd = 'sudo ' + installCmd;
+            } catch {
+                // sudo not available, try without it
+            }
+        }
+        
+        await execPromise(installCmd);
         
         vscode.window.showInformationMessage('MCPBrowser package installed successfully!');
         return true;

package/package.json

@@ -1,5 +1,5 @@
 {  "name": "mcpbrowser",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "mcpName": "io.github.cherchyk/browser",
   "type": "module",
   "description": "MCP server that loads authenticated web pages using Chrome DevTools Protocol",

package/src/mcp-browser.js

@@ -60,6 +60,7 @@ const defaultChromePaths = getDefaultChromePaths();
 
 let cachedBrowser = null;
 let lastKeptPage = null; // reuse the same tab when requested
+let chromeLaunchPromise = null; // prevent multiple simultaneous launches
 
 async function devtoolsAvailable() {
   try {
@@ -80,24 +81,47 @@ function findChromePath() {
 
 async function launchChromeIfNeeded() {
   if (explicitWSEndpoint) return; // user provided explicit endpoint; assume managed externally
+  
+  // If Chrome is already available, don't launch
   if (await devtoolsAvailable()) return;
-
-  const chromePath = findChromePath();
-  if (!chromePath) {
-    throw new Error("Chrome/Edge not found. Set CHROME_PATH to your browser executable.");
+  
+  // If a launch is already in progress, wait for it
+  if (chromeLaunchPromise) {
+    return await chromeLaunchPromise;
   }
+  
+  // Create a new launch promise to prevent multiple simultaneous launches
+  chromeLaunchPromise = (async () => {
+    try {
+      // Double-check after acquiring the launch lock
+      if (await devtoolsAvailable()) return;
+
+      const chromePath = findChromePath();
+      if (!chromePath) {
+        throw new Error("Chrome/Edge not found. Set CHROME_PATH to your browser executable.");
+      }
 
-  const args = [`--remote-debugging-port=${chromePort}`, `--user-data-dir=${userDataDir}`];
-  const child = spawn(chromePath, args, { detached: true, stdio: "ignore" });
-  child.unref();
+      const args = [
+        `--remote-debugging-port=${chromePort}`, 
+        `--user-data-dir=${userDataDir}`,
+        'about:blank' // Open with a blank page
+      ];
+      const child = spawn(chromePath, args, { detached: true, stdio: "ignore" });
+      child.unref();
 
-  // Wait for DevTools to come up
-  const deadline = Date.now() + 20000;
-  while (Date.now() < deadline) {
-    if (await devtoolsAvailable()) return;
-    await new Promise((r) => setTimeout(r, 500));
-  }
-  throw new Error("Chrome did not become available on DevTools port; check CHROME_PATH/port/profile.");
+      // Wait for DevTools to come up
+      const deadline = Date.now() + 20000;
+      while (Date.now() < deadline) {
+        if (await devtoolsAvailable()) return;
+        await new Promise((r) => setTimeout(r, 500));
+      }
+      throw new Error("Chrome did not become available on DevTools port; check CHROME_PATH/port/profile.");
+    } finally {
+      chromeLaunchPromise = null;
+    }
+  })();
+  
+  return await chromeLaunchPromise;
 }
 
 async function resolveWSEndpoint() {
@@ -173,9 +197,17 @@ async function fetchPage({
     }
   }
   
-  // Create new tab if no reuse
+  // Try to reuse existing pages first (when Chrome opened with profile)
   if (!page) {
-    page = await browser.newPage();
+    const pages = await browser.pages();
+    // Find an existing page that's not being used
+    if (pages.length > 0) {
+      // Use the first available page (usually the blank tab Chrome opens with)
+      page = pages[0];
+    } else {
+      // Create new tab if no existing pages
+      page = await browser.newPage();
+    }
   }
 
   let shouldKeepOpen = keepPageOpen || page === lastKeptPage;