Fix comments

Author: Natallia Harshunova

src/McpResponse.ts

@@ -23,6 +23,7 @@ import {
 } from './formatters/networkFormatter.js';
 import {formatSnapshotNode} from './formatters/snapshotFormatter.js';
 import {getIssueDescription} from './issue-descriptions.js';
+import {logger} from './logger.js';
 import type {McpContext} from './McpContext.js';
 import type {
   ConsoleMessage,
@@ -283,57 +284,58 @@ export class McpResponse implements Response {
         });
       }
 
-      consoleListData = await Promise.all(
-        messages.map(async (item): Promise<ConsoleMessageData> => {
-          const consoleMessageStableId =
-            context.getConsoleMessageStableId(item);
-          if ('args' in item) {
-            const consoleMessage = item as ConsoleMessage;
-            return {
-              consoleMessageStableId,
-              type: consoleMessage.type(),
-              message: consoleMessage.text(),
-              args: await Promise.all(
-                consoleMessage.args().map(async arg => {
-                  const stringArg = await arg.jsonValue().catch(() => {
-                    // Ignore errors.
-                  });
-                  return typeof stringArg === 'object'
-                    ? JSON.stringify(stringArg)
-                    : String(stringArg);
-                }),
-              ),
-            };
-          }
-          if (item instanceof AggregatedIssue) {
-            const count = item.getAggregatedIssuesCount();
-            const filename = item.getDescription()?.file;
-            const rawMarkdown = filename ? getIssueDescription(filename) : null;
-            if (!rawMarkdown) {
+      consoleListData = (
+        await Promise.all(
+          messages.map(async (item): Promise<ConsoleMessageData | null> => {
+            const consoleMessageStableId =
+              context.getConsoleMessageStableId(item);
+            if ('args' in item) {
+              const consoleMessage = item as ConsoleMessage;
+              return {
+                consoleMessageStableId,
+                type: consoleMessage.type(),
+                message: consoleMessage.text(),
+                args: await Promise.all(
+                  consoleMessage.args().map(async arg => {
+                    const stringArg = await arg.jsonValue().catch(() => {
+                      // Ignore errors.
+                    });
+                    return typeof stringArg === 'object'
+                      ? JSON.stringify(stringArg)
+                      : String(stringArg);
+                  }),
+                ),
+              };
+            }
+            if (item instanceof AggregatedIssue) {
+              const count = item.getAggregatedIssuesCount();
+              const filename = item.getDescription()?.file;
+              const rawMarkdown = filename
+                ? getIssueDescription(filename)
+                : null;
+              if (!rawMarkdown) {
+                logger(`no markdown ${filename} found for issue:` + item.code);
+                return null;
+              }
+              const markdownAst = Marked.Marked.lexer(rawMarkdown);
+              const title = findTitleFromMarkdownAst(markdownAst);
               return {
                 consoleMessageStableId,
                 type: 'issue',
-                message: `${item.code()} (count: ${count})`,
+                message: `${title}`,
+                count,
                 args: [],
               };
             }
-            const markdownAst = Marked.Marked.lexer(rawMarkdown);
-            const title = findTitleFromMarkdownAst(markdownAst);
             return {
               consoleMessageStableId,
-              type: 'issue',
-              message: `${title} (count: ${count})`,
+              type: 'error',
+              message: (item as Error).message,
               args: [],
             };
-          }
-          return {
-            consoleMessageStableId,
-            type: 'error',
-            message: (item as Error).message,
-            args: [],
-          };
-        }),
-      );
+          }),
+        )
+      ).filter(item => item !== null);
     }
 
     return this.format(toolName, context, {

src/PageCollector.ts

@@ -13,6 +13,7 @@ import {
 } from '../node_modules/chrome-devtools-frontend/mcp/mcp.js';
 
 import {FakeIssuesManager} from './DevtoolsUtils.js';
+import {logger} from './logger.js';
 import type {ConsoleMessage} from './third_party/index.js';
 import {
   type Browser,
@@ -242,28 +243,36 @@ export class ConsoleCollector extends PageCollector<
       },
     );
 
-    const session = await page.createCDPSession();
-    session.on('Audits.issueAdded', data => {
-      // @ts-expect-error Types of protocol from Puppeteer and CDP are incopatible for Issues but it's the same type
-      const issue = createIssuesFromProtocolIssue(null, data.issue)[0];
-      if (!issue) {
-        return;
-      }
-      const seenKeys = this.#seenIssueKeys.get(page)!;
-      const primaryKey = issue.primaryKey();
-      if (seenKeys.has(primaryKey)) return;
-      seenKeys.add(primaryKey);
-
-      const mockManager = this.#mockIssuesManagers.get(page);
-      if (mockManager) {
-        mockManager.dispatchEventToListeners(IssuesManagerEvents.ISSUE_ADDED, {
-          issue,
-          // @ts-expect-error We don't care that issues model is null
-          issuesModel: null,
-        });
-      }
-    });
-    await session.send('Audits.enable');
+    try {
+      const session = await page.createCDPSession();
+      session.on('Audits.issueAdded', data => {
+        // @ts-expect-error Types of protocol from Puppeteer and CDP are incopatible for Issues but it's the same type
+        const issue = createIssuesFromProtocolIssue(null, data.issue)[0];
+        if (!issue) {
+          return;
+        }
+        const seenKeys = this.#seenIssueKeys.get(page)!;
+        const primaryKey = issue.primaryKey();
+        if (seenKeys.has(primaryKey)) return;
+        seenKeys.add(primaryKey);
+
+        const mockManager = this.#mockIssuesManagers.get(page);
+        if (mockManager) {
+          mockManager.dispatchEventToListeners(
+            IssuesManagerEvents.ISSUE_ADDED,
+            {
+              issue,
+              // @ts-expect-error We don't care that issues model is null
+              issuesModel: null,
+            },
+          );
+        }
+      });
+      await session.send('Audits.enable');
+    } catch (e) {
+      const errorText = e instanceof Error ? e.message : JSON.stringify(e);
+      logger(`Error subscribing to issues: ${errorText}`);
+    }
   }
 
   override cleanupPageDestroyed(page: Page) {

src/formatters/consoleFormatter.ts

@@ -8,11 +8,15 @@ export interface ConsoleMessageData {
   consoleMessageStableId: number;
   type?: string;
   message?: string;
+  count?: number;
   args?: string[];
 }
 
 // The short format for a console message, based on a previous format.
 export function formatConsoleEventShort(msg: ConsoleMessageData): string {
+  if (msg.type === 'issue') {
+    return `msgid=${msg.consoleMessageStableId} [${msg.type}] ${msg.message} (count: ${msg.count}) (${msg.args?.length ?? 0} args)`;
+  }
   return `msgid=${msg.consoleMessageStableId} [${msg.type}] ${msg.message} (${msg.args?.length ?? 0} args)`;
 }