@nimbus21.ai/chrome-devtools-mcp 0.17.6 → 0.17.7

package/build/src/McpContext.js

@@ -72,9 +72,18 @@ export class McpContext {
     // Idle page reaper: tracks last activity time per page.
     static PAGE_IDLE_TIMEOUT_MS = 5 * 60 * 1000; // 5 minutes
     static IDLE_CHECK_INTERVAL_MS = 60 * 1000; // 60 seconds
+    static #INTERNAL_URL_PREFIXES = [
+        'about:',
+        'chrome://',
+        'chrome-extension://',
+        'chrome-untrusted://',
+        'devtools://',
+    ];
     #pageLastActivity = new Map();
     #idleReaperTimer;
     #onIdleBrowserEmpty;
+    // Tracks when we first noticed only internal pages remaining.
+    #onlyInternalPagesSince;
     #uniqueBackendNodeIdToMcpId = new Map();
     constructor(browser, logger, options, locatorClass) {
         this.browser = browser;
@@ -114,6 +123,8 @@ export class McpContext {
      */
     touchPage(page) {
         this.#pageLastActivity.set(page, Date.now());
+        // A user is interacting with a page — reset the internal-only timer.
+        this.#onlyInternalPagesSince = undefined;
     }
     /**
      * Record activity on all currently tracked pages.
@@ -149,6 +160,10 @@ export class McpContext {
             this.#idleReaperTimer = undefined;
         }
     }
+    static #isInternalPage(page) {
+        const url = page.url();
+        return _a.#INTERNAL_URL_PREFIXES.some(prefix => url.startsWith(prefix));
+    }
     async #reapIdlePages() {
         const now = Date.now();
         const idleTimeout = _a.PAGE_IDLE_TIMEOUT_MS;
@@ -160,8 +175,21 @@ export class McpContext {
         catch {
             return; // Browser may be disconnected.
         }
+        // Also get ALL pages from the browser (including those filtered by
+        // createPagesSnapshot) so we can detect internal-only state.
+        let allBrowserPages;
+        try {
+            allBrowserPages = await this.browser.pages(true);
+        }
+        catch {
+            return;
+        }
         const pagesToClose = [];
         for (const page of pages) {
+            // Skip internal pages — they can't be closed and will respawn.
+            if (_a.#isInternalPage(page)) {
+                continue;
+            }
             const lastActivity = this.#pageLastActivity.get(page);
             // Pages without a recorded timestamp get one now (first seen).
             if (lastActivity === undefined) {
@@ -172,10 +200,7 @@ export class McpContext {
                 pagesToClose.push(page);
             }
         }
-        if (pagesToClose.length === 0) {
-            return;
-        }
-        // Close idle pages. If ALL pages are idle, close them all.
+        // Close idle user pages.
         for (const page of pagesToClose) {
             try {
                 this.logger(`Closing idle page ${this.#pageIdMap.get(page)}: ${page.url()} (idle ${Math.round((now - (this.#pageLastActivity.get(page) ?? now)) / 1000)}s)`);
@@ -186,16 +211,39 @@ export class McpContext {
                 // Page may already be closed.
             }
         }
-        // Refresh after closing.
+        // Re-check all browser pages to determine if only internal pages remain.
         try {
-            pages = await this.createPagesSnapshot();
+            allBrowserPages = await this.browser.pages(true);
         }
         catch {
-            pages = [];
+            allBrowserPages = [];
+        }
+        const hasUserPages = allBrowserPages.some(p => !p.isClosed() && !_a.#isInternalPage(p));
+        if (hasUserPages) {
+            // User pages still exist — reset the internal-only timer.
+            this.#onlyInternalPagesSince = undefined;
+            return;
+        }
+        // Only internal pages (or no pages) remain.
+        if (!this.#onIdleBrowserEmpty) {
+            return;
         }
-        if (pages.length === 0 && this.#onIdleBrowserEmpty) {
+        if (allBrowserPages.length === 0) {
+            // Truly empty — kill immediately.
             this.logger('All pages closed by idle reaper, triggering browser cleanup');
             this.#onIdleBrowserEmpty();
+            return;
+        }
+        // Internal pages only — start or check the grace timer.
+        if (this.#onlyInternalPagesSince === undefined) {
+            this.#onlyInternalPagesSince = now;
+            this.logger(`Only internal pages remain (${allBrowserPages.length}), starting ${idleTimeout / 1000}s grace period`);
+            return;
+        }
+        if (now - this.#onlyInternalPagesSince > idleTimeout) {
+            this.logger(`Only internal pages for ${Math.round((now - this.#onlyInternalPagesSince) / 1000)}s, triggering browser cleanup`);
+            this.#onlyInternalPagesSince = undefined;
+            this.#onIdleBrowserEmpty();
         }
     }
     static async from(browser, logger, opts, 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nimbus21.ai/chrome-devtools-mcp",
-  "version": "0.17.6",
+  "version": "0.17.7",
   "description": "MCP server for Chrome DevTools with stealth mode support",
   "type": "module",
   "bin": "./build/src/index.js",