mcpbrowser 0.2.8 → 0.2.10

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.8",
+  "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.8",
+  "version": "0.2.10",
   "mcpName": "io.github.cherchyk/browser",
   "type": "module",
   "description": "MCP server that loads authenticated web pages using Chrome DevTools Protocol",

package/src/mcp-browser.js

@@ -200,12 +200,25 @@ async function fetchPage({
   // Try to reuse existing pages first (when Chrome opened with profile)
   if (!page) {
     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];
+    // Filter out pages that might not be controllable
+    const controllablePages = [];
+    for (const p of pages) {
+      try {
+        const url = p.url();
+        // Skip chrome:// pages and other internal pages
+        if (!url.startsWith('chrome://') && !url.startsWith('chrome-extension://')) {
+          controllablePages.push(p);
+        }
+      } catch {
+        // Skip pages we can't access
+      }
+    }
+    
+    // Use first controllable page or create new one
+    if (controllablePages.length > 0) {
+      page = controllablePages[0];
     } else {
-      // Create new tab if no existing pages
+      // Create new tab if no controllable pages
       page = await browser.newPage();
     }
   }