@fanboynz/network-scanner 1.0.36 → 1.0.37

package/lib/browserhealth.js

@@ -17,7 +17,8 @@ async function checkBrowserHealth(browserInstance, timeout = 5000) {
     pageCount: 0,
     error: null,
     responseTime: 0,
-    recommendations: []
+    recommendations: [],
+    criticalError: false
   };
 
   const startTime = Date.now();
@@ -27,6 +28,7 @@ async function checkBrowserHealth(browserInstance, timeout = 5000) {
     if (!browserInstance || browserInstance.process() === null) {
       healthResult.error = 'Browser process not running';
       healthResult.recommendations.push('Create new browser instance');
+      healthResult.criticalError = true;
       return healthResult;
     }
 
@@ -71,7 +73,12 @@ async function checkBrowserHealth(browserInstance, timeout = 5000) {
         try { await testPage.close(); } catch (e) { /* ignore */ }
       }
       healthResult.error = `Page creation/navigation failed: ${pageTestError.message}`;
-      healthResult.recommendations.push('Browser restart recommended');
+      if (isCriticalProtocolError(pageTestError)) {
+        healthResult.recommendations.push('Browser restart required - critical protocol error');
+        healthResult.criticalError = true;
+      } else {
+        healthResult.recommendations.push('Browser restart recommended');
+      }
       return healthResult;
     }
 
@@ -88,10 +95,14 @@ async function checkBrowserHealth(browserInstance, timeout = 5000) {
     healthResult.responseTime = Date.now() - startTime;
     
     // Categorize error types for better recommendations
-    if (error.message.includes('timeout') || error.message.includes('unresponsive')) {
+    if (error.message.includes('Runtime.callFunctionOn timed out') || 
+        error.message.includes('Protocol error') || 
+        error.message.includes('Target closed')) {
+      healthResult.recommendations.push('Browser restart required - critical protocol error');
+      healthResult.criticalError = true;
+    } else if (error.message.includes('timeout') || error.message.includes('unresponsive')) {
       healthResult.recommendations.push('Browser restart required - unresponsive');
-    } else if (error.message.includes('Protocol error') || error.message.includes('Target closed')) {
-      healthResult.recommendations.push('Browser restart required - protocol error');
+      healthResult.criticalError = true;
     } else {
       healthResult.recommendations.push('Browser restart recommended - unknown error');
     }
@@ -152,6 +163,27 @@ async function checkBrowserMemory(browserInstance) {
   return memoryResult;
 }
 
+ /**
+ * Detects critical protocol errors that require immediate browser restart
+ */
+function isCriticalProtocolError(error) {
+  if (!error || !error.message) return false;
+  
+  const criticalErrors = [
+    'Runtime.callFunctionOn timed out',
+    'Protocol error',
+    'Target closed',
+    'Session closed',
+    'Connection closed',
+    'Browser has been closed',
+    'Runtime.evaluate timed out'
+  ];
+  
+  return criticalErrors.some(criticalError => 
+    error.message.includes(criticalError)
+  );
+}
+
 /**
  * Performs comprehensive browser health assessment
  * @param {import('puppeteer').Browser} browserInstance - Puppeteer browser instance
@@ -196,6 +228,9 @@ async function performHealthAssessment(browserInstance, options = {}) {
   if (!assessment.browser.healthy) {
     assessment.overall = 'unhealthy';
     assessment.needsRestart = true;
+  } else if (assessment.browser.criticalError) {
+    assessment.overall = 'critical';
+    assessment.needsRestart = true;
   } else if (assessment.recommendations.length > 0) {
     assessment.overall = 'degraded';
     assessment.needsRestart = assessment.recommendations.some(rec => 
@@ -252,7 +287,10 @@ async function monitorBrowserHealth(browserInstance, context = {}, options = {})
     result.assessment = assessment;
 
     // Decision logic for restart
-    if (assessment.needsRestart) {
+    if (assessment.browser.criticalError) {
+      result.shouldRestart = true;
+      result.reason = `Critical protocol error detected - immediate restart required`;
+    } else if (assessment.needsRestart) {
       result.shouldRestart = true;
       result.reason = `Browser health: ${assessment.overall} - ${assessment.recommendations[0] || 'restart needed'}`;
     } else if (urlsSinceCleanup >= cleanupInterval) {
@@ -304,5 +342,6 @@ module.exports = {
   checkBrowserMemory,
   performHealthAssessment,
   monitorBrowserHealth,
-  isBrowserHealthy
+  isBrowserHealthy,
+  isCriticalProtocolError
 };

package/nwss.js

@@ -1,4 +1,4 @@
-// === Network scanner script (nwss.js) v1.0.36 ===
+// === Network scanner script (nwss.js) v1.0.37 ===
 
 // puppeteer for browser automation, fs for file system operations, psl for domain parsing.
 // const pLimit = require('p-limit'); // Will be dynamically imported
@@ -33,7 +33,7 @@ const { navigateWithRedirectHandling, handleRedirectTimeout } = require('./lib/r
 const { monitorBrowserHealth, isBrowserHealthy } = require('./lib/browserhealth');
 
 // --- Script Configuration & Constants ---
-const VERSION = '1.0.36'; // Script version
+const VERSION = '1.0.37'; // Script version
 const MAX_CONCURRENT_SITES = 5;
 const RESOURCE_CLEANUP_INTERVAL = 80; // Close browser and restart every N sites to free resources
 
@@ -936,7 +936,7 @@ function setupFrameHandling(page, forceDebug) {
 	    '--disable-image-animation-resync'
         ],
         headless: launchHeadless ? 'shell' : false,
-        protocolTimeout: 500000
+        protocolTimeout: 60000  // 60 seconds
     });
     
     // Store the user data directory on the browser object for cleanup
@@ -2091,7 +2091,21 @@ function setupFrameHandling(page, forceDebug) {
       }
       
     } catch (err) {
-      // Enhanced error handling with rule preservation for partial matches
+    // Enhanced error handling with rule preservation for partial matches
+    if (err.message.includes('Runtime.callFunctionOn timed out') || 
+        err.message.includes('Protocol error') ||
+        err.message.includes('Target closed') ||
+        err.message.includes('Browser has been closed')) {
+      console.error(formatLogMessage('error', `Critical browser protocol error on ${currentUrl}: ${err.message}`));
+      return { 
+        url: currentUrl, 
+        rules: [], 
+        success: false, 
+        needsImmediateRestart: true,
+        error: `Critical protocol error: ${err.message}`
+      };
+    }
+    
       if (err.message.includes('Protocol error') || 
           err.message.includes('Target closed') ||
           err.message.includes('Browser process was killed') ||

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fanboynz/network-scanner",
-  "version": "1.0.36",
+  "version": "1.0.37",
   "description": "A Puppeteer-based network scanner for analyzing web traffic, generating adblock filter rules, and identifying third-party requests. Features include fingerprint spoofing, Cloudflare bypass, content analysis with curl/grep, and multiple output formats.",
   "main": "nwss.js",
   "scripts": {