@dynamicu/chromedebug-mcp 2.7.1 → 2.7.2

package/chrome-extension/background.js

@@ -1375,12 +1375,26 @@ chrome.runtime.onInstalled.addListener(async (details) => {
 chrome.tabs.onUpdated.addListener((tabId, changeInfo, tab) => {
   // Inject console logging on navigation during recording
   if (changeInfo.status === 'loading') {
-    // Use session manager validation if available
+    // Skip restricted URLs
+    if (!tab.url || tab.url.startsWith('chrome://') || tab.url.startsWith('chrome-extension://')) {
+      return;
+    }
+
+    // Check for WORKFLOW recording first (takes priority)
+    if (workflowRecordingTabs.has(tabId) && workflowIncludeLogs.get(tabId)) {
+      // console.log('[Navigation] Re-injecting workflow console logging for tab:', tabId);
+      self.ConsoleInterceptionLibrary.startConsoleInterception(tabId, WORKFLOW_RECORDING_CONSOLE_CONFIG).catch(err => {
+        console.error('Failed to re-inject workflow console logging:', err);
+      });
+      return; // Don't also inject screen recording logs
+    }
+
+    // Check for SCREEN recording
     if (sessionManager) {
       isCurrentlyRecordingAsync().then(async (recording) => {
         const currentTabId = await getCurrentTabIdAsync();
-        if (recording && tabId === currentTabId && tab.url && !tab.url.startsWith('chrome://') && !tab.url.startsWith('chrome-extension://')) {
-          // console.log('Re-injecting console logging for navigation during recording');
+        if (recording && tabId === currentTabId) {
+          // console.log('[Navigation] Re-injecting screen recording console logging for tab:', tabId);
           startCapturingLogs(tabId).catch(err => {
             console.error('Failed to re-inject console logging:', err);
           });
@@ -1389,9 +1403,9 @@ chrome.tabs.onUpdated.addListener((tabId, changeInfo, tab) => {
         console.error('Error checking recording state for navigation:', error);
       });
     } else {
-      // Legacy fallback
-      if (tabId === recordingTabId && isCurrentlyRecording && tab.url && !tab.url.startsWith('chrome://') && !tab.url.startsWith('chrome-extension://')) {
-        // console.log('Re-injecting console logging for navigation during recording');
+      // Legacy fallback for screen recording
+      if (tabId === recordingTabId && isCurrentlyRecording) {
+        // console.log('[Navigation] Re-injecting screen recording console logging (legacy) for tab:', tabId);
         startCapturingLogs(tabId).catch(err => {
           console.error('Failed to re-inject console logging:', err);
         });
@@ -1522,7 +1536,7 @@ let frameCounter = new Map(); // DEPRECATED: Session manager handles frame count
 // =============================================================================
 // INACTIVITY AUTO-STOP - Stop recording if user is inactive
 // =============================================================================
-const DEFAULT_INACTIVITY_TIMEOUT_MS = 10 * 1000; // 10 seconds default
+const DEFAULT_INACTIVITY_TIMEOUT_MS = 60 * 1000; // 60 seconds default (1 minute)
 let lastUserActivityTime = Date.now();
 let inactivityCheckInterval = null;
 let currentInactivityTimeout = DEFAULT_INACTIVITY_TIMEOUT_MS;
@@ -1546,7 +1560,7 @@ async function startInactivityMonitoring(tabId) {
 
   // Load user's configured timeout from storage
   const settings = await chrome.storage.sync.get(['inactivityTimeout']);
-  currentInactivityTimeout = (settings.inactivityTimeout || 10) * 1000; // Convert seconds to ms (default 10s)
+  currentInactivityTimeout = (settings.inactivityTimeout || 60) * 1000; // Convert seconds to ms (default 60s)
 
   lastUserActivityTime = Date.now();
 

package/chrome-extension/manifest.free.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "ChromeDebug MCP Assistant FREE",
-  "version": "2.7.1",
+  "version": "2.7.2",
   "description": "AI-powered browser debugging with visual element selection and screen recording for developers",
   "permissions": [
     "activeTab",

package/chrome-extension/options.html

@@ -156,10 +156,10 @@
     <div class="setting-group">
       <h3>Recording Safety</h3>
       <label for="inactivity-timeout">Auto-Stop After Inactivity (seconds)</label>
-      <input type="number" id="inactivity-timeout" min="5" max="3600" value="10">
+      <input type="number" id="inactivity-timeout" min="5" max="3600" value="60">
       <div class="help-text">
         Screen recording will automatically stop if no mouse or keyboard activity is detected.
-        Default: 10 seconds. Increase for long demo recordings or presentations.
+        Default: 60 seconds. Adjust for long demo recordings or presentations.
       </div>
     </div>
     

package/chrome-extension/options.js

@@ -9,8 +9,8 @@ chrome.storage.sync.get(['serverPort', 'chromePilotMode', 'chromePilotAllowedSit
     document.getElementById('server-port').value = data.serverPort;
   }
 
-  // Load inactivity timeout (default 10 seconds)
-  document.getElementById('inactivity-timeout').value = data.inactivityTimeout || 10;
+  // Load inactivity timeout (default 60 seconds)
+  document.getElementById('inactivity-timeout').value = data.inactivityTimeout || 60;
   
   // Load site restriction settings
   const mode = data.chromePilotMode || 'whitelist';
@@ -38,7 +38,7 @@ document.getElementById('save').addEventListener('click', async () => {
 
   // Get inactivity timeout
   const inactivityTimeoutInput = document.getElementById('inactivity-timeout');
-  const inactivityTimeout = parseInt(inactivityTimeoutInput.value) || 10;
+  const inactivityTimeout = parseInt(inactivityTimeoutInput.value) || 60;
   if (inactivityTimeout < 5 || inactivityTimeout > 3600) {
     showStatus('error', 'Inactivity timeout must be between 5 and 3600 seconds');
     return;
@@ -68,7 +68,7 @@ document.getElementById('save').addEventListener('click', async () => {
 // Reset to default
 document.getElementById('reset').addEventListener('click', async () => {
   document.getElementById('server-port').value = '';
-  document.getElementById('inactivity-timeout').value = '10';
+  document.getElementById('inactivity-timeout').value = '60';
   document.querySelector('input[value="whitelist"]').checked = true;
   document.getElementById('allowed-sites').value = 'localhost:*\n127.0.0.1:*\n*.local\n*.test\n*.dev';
   document.getElementById('blocked-sites').value = 'youtube.com\n*.youtube.com\ngoogle.com\n*.google.com\nfacebook.com\ntwitter.com\nx.com';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dynamicu/chromedebug-mcp",
-  "version": "2.7.1",
+  "version": "2.7.2",
   "description": "ChromeDebug MCP - MCP server that provides full control over a Chrome browser instance for debugging and automation with AI assistants like Claude Code",
   "main": "src/index.js",
   "type": "module",

package/src/database.js

@@ -132,9 +132,20 @@ class ChromeDebugDatabase {
     // Migrate from old location if needed
     this.migrateIfNeeded();
 
+    // Check if database exists and is corrupted
+    if (fs.existsSync(this.dbPath)) {
+      const integrityStatus = this.checkDatabaseIntegrity();
+      if (!integrityStatus.ok) {
+        logger.warn(`[Database] Database corruption detected: ${integrityStatus.error}`);
+        this.handleCorruptedDatabase();
+      }
+    }
+
     // Open database connection
     this.db = new Database(this.dbPath);
     this.db.pragma('journal_mode = WAL'); // Enable WAL mode for better concurrent access
+    this.db.pragma('synchronous = NORMAL'); // Balance between safety and performance
+    this.db.pragma('foreign_keys = ON'); // Enforce foreign key constraints
 
     // Create tables
     this.createTables();
@@ -142,6 +153,64 @@ class ChromeDebugDatabase {
     logger.debug(`ChromeDebug MCP database initialized at: ${this.dbPath}`);
   }
 
+  /**
+   * Check database integrity before opening
+   * @returns {{ ok: boolean, error?: string }}
+   */
+  checkDatabaseIntegrity() {
+    try {
+      const tempDb = new Database(this.dbPath, { readonly: true });
+      const result = tempDb.pragma('integrity_check');
+      tempDb.close();
+
+      if (result && result[0] && result[0].integrity_check === 'ok') {
+        return { ok: true };
+      } else {
+        const errorMsg = result?.[0]?.integrity_check || 'Unknown integrity error';
+        return { ok: false, error: errorMsg };
+      }
+    } catch (error) {
+      return { ok: false, error: error.message };
+    }
+  }
+
+  /**
+   * Handle corrupted database by backing it up and creating a fresh one
+   */
+  handleCorruptedDatabase() {
+    const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
+    const backupPath = `${this.dbPath}.corrupted_${timestamp}`;
+
+    logger.warn(`[Database] Backing up corrupted database to: ${backupPath}`);
+
+    try {
+      // Move corrupted database and associated files
+      if (fs.existsSync(this.dbPath)) {
+        fs.renameSync(this.dbPath, backupPath);
+      }
+      if (fs.existsSync(`${this.dbPath}-shm`)) {
+        fs.renameSync(`${this.dbPath}-shm`, `${backupPath}-shm`);
+      }
+      if (fs.existsSync(`${this.dbPath}-wal`)) {
+        fs.renameSync(`${this.dbPath}-wal`, `${backupPath}-wal`);
+      }
+
+      logger.info(`[Database] Corrupted database backed up. A fresh database will be created.`);
+    } catch (error) {
+      logger.error(`[Database] Failed to backup corrupted database: ${error.message}`);
+      // Try to delete instead of rename
+      try {
+        fs.unlinkSync(this.dbPath);
+        if (fs.existsSync(`${this.dbPath}-shm`)) fs.unlinkSync(`${this.dbPath}-shm`);
+        if (fs.existsSync(`${this.dbPath}-wal`)) fs.unlinkSync(`${this.dbPath}-wal`);
+        logger.info(`[Database] Corrupted database removed. A fresh database will be created.`);
+      } catch (deleteError) {
+        logger.error(`[Database] Failed to remove corrupted database: ${deleteError.message}`);
+        throw new Error(`Cannot recover from database corruption: ${deleteError.message}`);
+      }
+    }
+  }
+
   createTables() {
     // Recordings table for frame capture sessions
     this.db.exec(`

package/src/validation/schemas.js

@@ -1,5 +1,10 @@
 import Joi from 'joi';
 
+// Valid console log levels - must match what Chrome extension captures
+// Chrome extension captures: log, error, warn, info, debug, trace, table, dir, group, groupEnd, time, timeEnd, count
+// See: chrome-extension/console-interception-library.js line 228
+const VALID_LOG_LEVELS = ['log', 'info', 'warn', 'error', 'debug', 'trace', 'table', 'dir', 'group', 'groupEnd', 'time', 'timeEnd', 'count'];
+
 // Common validation patterns
 const patterns = {
   sessionId: Joi.string().pattern(/^[a-zA-Z0-9_-]+$/).min(1).max(100),
@@ -53,7 +58,7 @@ export const workflowRecordingSchema = Joi.object({
   ).required(),
   logs: Joi.array().items(
     Joi.object({
-      level: Joi.string().valid('log', 'info', 'warn', 'error', 'debug').required(),
+      level: Joi.string().valid(...VALID_LOG_LEVELS).required(),
       message: Joi.string().required(),
       timestamp: Joi.number().required(),
       args: Joi.array().optional()
@@ -93,7 +98,7 @@ export const frameBatchSchema = Joi.object({
       index: Joi.number().integer().min(0).optional(), // Chrome extension sends this
       logs: Joi.array().items(
         Joi.object({
-          level: Joi.string().valid('log', 'info', 'warn', 'error', 'debug').required(),
+          level: Joi.string().valid(...VALID_LOG_LEVELS).required(),
           message: Joi.string().required(),
           timestamp: Joi.number().required(),
           args: Joi.array().optional()
@@ -121,7 +126,7 @@ export const associateLogsSchema = Joi.object({
   sessionId: patterns.sessionId.required(),
   logs: Joi.array().items(
     Joi.object({
-      level: Joi.string().valid('log', 'info', 'warn', 'error', 'debug').required(),
+      level: Joi.string().valid(...VALID_LOG_LEVELS).required(),
       message: Joi.string().required(),
       timestamp: Joi.number().required(),
       args: Joi.array().optional()
@@ -134,7 +139,7 @@ export const streamLogsSchema = Joi.object({
   sessionId: patterns.sessionId.required(),
   logs: Joi.array().items(
     Joi.object({
-      level: Joi.string().valid('log', 'info', 'warn', 'error', 'debug', 'trace', 'table', 'dir', 'group', 'groupEnd', 'time', 'timeEnd', 'count').required(),
+      level: Joi.string().valid(...VALID_LOG_LEVELS).required(),
       message: Joi.string().required(),
       timestamp: Joi.number().required(),
       sequence: Joi.number().integer().min(0).required(),