feat: Enhance network log preservation features with improved messaging and descriptions for AI Agents

Author: nicetrykiddo

src/McpResponse.ts

@@ -223,7 +223,6 @@ Call ${handleDialog.name} to handle it before continuing.`);
     if (this.#networkRequestsOptions?.include) {
       let requests = context.getNetworkRequests();
 
-      // Apply resource type filtering if specified
       if (this.#networkRequestsOptions.resourceTypes?.length) {
         const normalizedTypes = new Set(
           this.#networkRequestsOptions.resourceTypes,
@@ -234,7 +233,15 @@ Call ${handleDialog.name} to handle it before continuing.`);
         });
       }
 
+      const preservationEnabled = context.isNetworkLogPreservationEnabled();
+      
       response.push('## Network requests');
+      if (preservationEnabled) {
+        response.push('🔒 **Preservation Mode: ACTIVE** (requests persist across navigations)');
+      } else {
+        response.push('📋 Normal mode (logs cleared on navigation). Enable preservation to keep history.');
+      }
+      
       if (requests.length) {
         const data = this.#dataWithPagination(
           requests,

src/tools/network.ts

@@ -34,7 +34,7 @@ const FILTERABLE_RESOURCE_TYPES: readonly [ResourceType, ...ResourceType[]] = [
 
 export const listNetworkRequests = defineTool({
   name: 'list_network_requests',
-  description: `List all requests for the currently selected page`,
+  description: `List all requests for the currently selected page. By default, only shows requests since the last navigation. To preserve requests across navigations, first call enable_network_log_preservation before navigating.`,
   annotations: {
     category: ToolCategories.NETWORK,
     readOnlyHint: true,
@@ -89,7 +89,7 @@ export const getNetworkRequest = defineTool({
 
 export const enableNetworkLogPreservation = defineTool({
   name: 'enable_network_log_preservation',
-  description: `Enable network log preservation mode. When enabled, all network requests are preserved across page navigations and response bodies are automatically captured. By default, logs are cleaned on navigation for performance.`,
+  description: `Enable network log preservation mode to keep ALL network requests across navigations. IMPORTANT: Call this BEFORE navigating or interacting with the page if you want to analyze request patterns across multiple actions. When enabled, all request/response bodies are automatically captured and cached for later analysis. Use this when you need to compare requests before/after certain actions or track API calls across page transitions.`,
   annotations: {
     category: ToolCategories.NETWORK,
     readOnlyHint: false,
@@ -115,31 +115,44 @@ export const enableNetworkLogPreservation = defineTool({
       ),
   },
   handler: async (request, response, context) => {
+    const wasAlreadyEnabled = context.isNetworkLogPreservationEnabled();
+    
     context.enableNetworkLogPreservation({
       includeRequestBodies: request.params.includeRequestBodies,
       includeResponseBodies: request.params.includeResponseBodies,
       maxRequests: request.params.maxRequests,
     });
-    response.appendResponseLine(
-      'Network log preservation enabled. All network requests will be preserved across navigations.',
-    );
+    
+    if (wasAlreadyEnabled) {
+      response.appendResponseLine(
+        '⚠️  Network log preservation was already enabled. Settings updated.',
+      );
+    } else {
+      response.appendResponseLine(
+        '✅ Network log preservation enabled. All network requests will be preserved across navigations.',
+      );
+    }
+    
     if (request.params.includeRequestBodies) {
-      response.appendResponseLine('Request bodies will be captured.');
+      response.appendResponseLine('📤 Request bodies will be captured.');
     }
     if (request.params.includeResponseBodies) {
-      response.appendResponseLine('Response bodies will be captured.');
+      response.appendResponseLine('📥 Response bodies will be captured.');
     }
     if (request.params.maxRequests) {
       response.appendResponseLine(
-        `Maximum ${request.params.maxRequests} requests will be preserved.`,
+        `🔢 Maximum ${request.params.maxRequests} requests will be preserved.`,
       );
     }
+    response.appendResponseLine(
+      '\n💡 TIP: Preservation is now active. Navigate, click buttons, or interact with the page - all network activity will be recorded.',
+    );
   },
 });
 
 export const disableNetworkLogPreservation = defineTool({
   name: 'disable_network_log_preservation',
-  description: `Disable network log preservation mode and optionally clear existing preserved logs. After disabling, network logs will be cleaned on navigation (default behavior).`,
+  description: `Disable network log preservation mode and optionally clear existing preserved logs. After disabling, network logs will be cleaned on navigation (default behavior). Call this when you're done analyzing preserved requests to restore normal performance.`,
   annotations: {
     category: ToolCategories.NETWORK,
     readOnlyHint: false,
@@ -154,33 +167,58 @@ export const disableNetworkLogPreservation = defineTool({
       ),
   },
   handler: async (request, response, context) => {
+    const wasEnabled = context.isNetworkLogPreservationEnabled();
+    
+    if (!wasEnabled) {
+      response.appendResponseLine(
+        '⚠️  Network log preservation was not enabled. No action taken.',
+      );
+      return;
+    }
+    
     context.disableNetworkLogPreservation();
     if (request.params.clearExisting) {
       context.clearPreservedNetworkLogs();
       response.appendResponseLine(
-        'Network log preservation disabled and existing logs cleared.',
+        '✅ Network log preservation disabled and existing logs cleared.',
       );
     } else {
       response.appendResponseLine(
-        'Network log preservation disabled. Existing logs retained.',
+        '✅ Network log preservation disabled. Existing logs retained.',
       );
     }
+    response.appendResponseLine(
+      '💡 Normal behavior restored: network logs will be cleared on navigation.',
+    );
   },
 });
 
 export const clearPreservedNetworkLogs = defineTool({
   name: 'clear_preserved_network_logs',
-  description: `Clear all preserved network logs for the currently selected page. This does not disable preservation mode.`,
+  description: `Clear all preserved network logs for the currently selected page without disabling preservation mode. Use this to reset the preserved request history while keeping preservation active for future requests.`,
   annotations: {
     category: ToolCategories.NETWORK,
     readOnlyHint: false,
   },
   schema: {},
   handler: async (_request, response, context) => {
+    if (!context.isNetworkLogPreservationEnabled()) {
+      response.appendResponseLine(
+        '⚠️  Network log preservation is not enabled. No preserved logs to clear.',
+      );
+      response.appendResponseLine(
+        '💡 TIP: Call enable_network_log_preservation first to start preserving logs.',
+      );
+      return;
+    }
+    
     const preservedCount = context.getPreservedNetworkRequests().length;
     context.clearPreservedNetworkLogs();
     response.appendResponseLine(
-      `Cleared ${preservedCount} preserved network request(s).`,
+      `✅ Cleared ${preservedCount} preserved network request(s).`,
+    );
+    response.appendResponseLine(
+      '💡 Preservation mode is still active. New requests will continue to be preserved.',
     );
   },
 });