Stabilize URL-encoded file paths in console test

Author: samiyac

tests/tools/console.test.js.snapshot

@@ -119,7 +119,40 @@ reqid=<reqid> data={"corsErrorStatus":{"corsError":"PreflightMissingAllowOriginH
 `;
 
 exports[`console > get_console_message > when dialog is open 1`] = `
-{"content":[{"type":"text","text":"# Open dialog\\nalert: test dialog.\\nCall handle_dialog to handle it before continuing.\\nID: 1\\nMessage: error> This is an error\\n### Arguments\\nArg #0: This is an error\\n### Stack trace\\nat  (VM7:1:9)\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(file%3A%2F%2F%2FUsers%2Fsamiyac%2FDesktop%2Fchrome-devtools-mcp%2…:3:26)\\n--- PendingScript ----------------------\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(file%3A%2F%2F%2FUsers%2Fsamiyac%2FDesktop%2Fchrome-devtools-mcp%2…:3:26)\\nNote: line and column numbers use 1-based indexing\\n## Console messages\\nShowing 1-1 of 1 (Page 1 of 1).\\nmsgid=1 [error] This is an error (1 args)"}],"structuredContent":{"dialog":{"type":"alert","message":"test dialog","defaultValue":""},"consoleMessage":{"id":1,"type":"error","text":"This is an error","argsCount":1,"args":["This is an error"],"stackTrace":"at  (VM7:1:9)\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(file%3A%2F%2F%2FUsers%2Fsamiyac%2FDesktop%2Fchrome-devtools-mcp%2…:3:26)\\n--- PendingScript ----------------------\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(file%3A%2F%2F%2FUsers%2Fsamiyac%2FDesktop%2Fchrome-devtools-mcp%2…:3:26)\\nNote: line and column numbers use 1-based indexing"},"pagination":{"currentPage":0,"totalPages":1,"hasNextPage":false,"hasPreviousPage":false,"startIndex":0,"endIndex":1,"invalidPage":false},"consoleMessages":[{"type":"error","text":"This is an error","argsCount":1,"id":1}]}}
+{
+  "dialog": {
+    "type": "alert",
+    "message": "test dialog",
+    "defaultValue": ""
+  },
+  "consoleMessage": {
+    "id": 1,
+    "type": "error",
+    "text": "This is an error",
+    "argsCount": 1,
+    "args": [
+      "This is an error"
+    ],
+    "stackTrace": "at  (VM7:1:9)\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(<file-path>)\\n--- PendingScript ----------------------\\nat  (pptr:;CdpFrame.%3Canonymous%3E%20(<file-path>)\\nNote: line and column numbers use 1-based indexing"
+  },
+  "pagination": {
+    "currentPage": 0,
+    "totalPages": 1,
+    "hasNextPage": false,
+    "hasPreviousPage": false,
+    "startIndex": 0,
+    "endIndex": 1,
+    "invalidPage": false
+  },
+  "consoleMessages": [
+    {
+      "type": "error",
+      "text": "This is an error",
+      "argsCount": 1,
+      "id": 1
+    }
+  ]
+}
 `;
 
 exports[`console > list_console_messages > issues > when dialog is open 1`] = `

tests/tools/console.test.ts

@@ -19,7 +19,11 @@ import {
   listConsoleMessages,
 } from '../../src/tools/console.js';
 import {serverHooks} from '../server.js';
-import {getTextContent, withMcpContext} from '../utils.js';
+import {
+  getTextContent,
+  withMcpContext,
+  stabilizeStructuredContent,
+} from '../utils.js';
 
 describe('console', () => {
   before(async () => {
@@ -524,7 +528,6 @@ describe('console', () => {
         const dialogPromise = new Promise<Dialog>(resolve => {
           page.on('dialog', dialog => resolve(dialog));
         });
-
         page.evaluate(() => {
           alert('test dialog');
         });
@@ -537,9 +540,14 @@ describe('console', () => {
         );
 
         const result = await response.handle('get_console_message', context);
-        t.assert.snapshot?.(JSON.stringify(result));
+        t.assert.snapshot?.(
+          JSON.stringify(
+            stabilizeStructuredContent(result.structuredContent),
+            null,
+            2,
+          ),
+        );
         await dialog.dismiss();
-        context.getSelectedMcpPage().clearDialog();
       });
     });
   });

tests/utils.ts

@@ -287,6 +287,10 @@ export function stabilizeResponseOutput(text: unknown) {
   const acceptLanguageRegEx = /accept-language:.*\n/g;
   output = output.replaceAll(acceptLanguageRegEx, 'accept-language:<lang>\n');
 
+  // Stabilize URL-encoded file paths in stack traces starting with Users/
+  const fileUriRegEx = /file%3A%2F%2F%2FUsers%2F[^)\n]+/g;
+  output = output.replaceAll(fileUriRegEx, '<file-path>');
+
   return output;
 }