Add formatting

Author: Natallia Harshunova

scripts/post-build.ts

@@ -173,16 +173,11 @@ export const hostConfig = {};
 }
 
 function copyDevToolsDescriptionFiles() {
-  const devtoolsIssuesDescriptionPath = 'node_modules/chrome-devtools-frontend/front_end/models/issues_manager/descriptions';
-  const sourceDir = path.join(
-   process.cwd(),
-   devtoolsIssuesDescriptionPath,
-    );
-   const destDir = path.join(
-    BUILD_DIR,
-    devtoolsIssuesDescriptionPath,
-    );
-    fs.cpSync(sourceDir, destDir, {recursive: true});
+  const devtoolsIssuesDescriptionPath =
+    'node_modules/chrome-devtools-frontend/front_end/models/issues_manager/descriptions';
+  const sourceDir = path.join(process.cwd(), devtoolsIssuesDescriptionPath);
+  const destDir = path.join(BUILD_DIR, devtoolsIssuesDescriptionPath);
+  fs.cpSync(sourceDir, destDir, {recursive: true});
 }
 
 main();

src/DevtoolsUtils.ts

@@ -7,7 +7,7 @@
 import {
   type Issue,
   type IssuesManagerEventTypes,
-  Common
+  Common,
 } from '../node_modules/chrome-devtools-frontend/mcp/mcp.js';
 
 export function extractUrlLikeFromDevToolsTitle(
@@ -61,7 +61,8 @@ function normalizeUrl(url: string): string {
  * A mock implementation of an issues manager that only implements the methods
  * that are actually used by the IssuesAggregator
  */
-export class FakeIssuesManager extends Common.ObjectWrapper.ObjectWrapper<IssuesManagerEventTypes> {
+export class FakeIssuesManager extends Common.ObjectWrapper
+  .ObjectWrapper<IssuesManagerEventTypes> {
   issues(): Issue[] {
     return [];
   }

src/McpContext.ts

@@ -215,7 +215,9 @@ export class McpContext implements Context {
     return this.#consoleCollector.getData(page, includePreservedMessages);
   }
 
-  getConsoleMessageStableId(message: ConsoleMessage | Error | AggregatedIssue): number {
+  getConsoleMessageStableId(
+    message: ConsoleMessage | Error | AggregatedIssue,
+  ): number {
     return this.#consoleCollector.getIdForResource(message);
   }
 

src/McpResponse.ts

@@ -4,7 +4,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import {
-  AggregatedIssue, Marked, findTitleFromMarkdownAst
+  AggregatedIssue,
+  Marked,
+  findTitleFromMarkdownAst,
 } from '../node_modules/chrome-devtools-frontend/mcp/mcp.js';
 
 import type {ConsoleMessageData} from './formatters/consoleFormatter.js';
@@ -306,16 +308,14 @@ export class McpResponse implements Response {
           if (item instanceof AggregatedIssue) {
             const count = item.getAggregatedIssuesCount();
             const filename = item.getDescription()?.file;
-            const rawMarkdown = filename
-              ? getIssueDescription(filename)
-              : null;
+            const rawMarkdown = filename ? getIssueDescription(filename) : null;
             if (!rawMarkdown) {
-            return {
-              consoleMessageStableId,
-              type: 'issue',
-              message: `${item.code()} (count: ${count})`,
-              args: [],
-            };
+              return {
+                consoleMessageStableId,
+                type: 'issue',
+                message: `${item.code()} (count: ${count})`,
+                args: [],
+              };
             }
             const markdownAst = Marked.Marked.lexer(rawMarkdown);
             const title = findTitleFromMarkdownAst(markdownAst);

src/PageCollector.ts

@@ -101,7 +101,7 @@ export class PageCollector<T> {
     this.#initializePage(page);
   }
 
-   #initializePage(page: Page) {
+  #initializePage(page: Page) {
     const idGenerator = createIdGenerator();
     const storedLists: Array<Array<WithSymbolId<T>>> = [[]];
     this.storage.set(page, storedLists);
@@ -206,7 +206,9 @@ export class PageCollector<T> {
   }
 }
 
-export class ConsoleCollector extends PageCollector<ConsoleMessage | Error | AggregatedIssue> {
+export class ConsoleCollector extends PageCollector<
+  ConsoleMessage | Error | AggregatedIssue
+> {
   #seenIssueKeys = new WeakMap<Page, Set<string>>();
   #issuesAggregators = new WeakMap<Page, IssueAggregator>();
   #mockIssuesManagers = new WeakMap<Page, FakeIssuesManager>();
@@ -243,7 +245,7 @@ export class ConsoleCollector extends PageCollector<ConsoleMessage | Error | Agg
     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];
+      const issue = createIssuesFromProtocolIssue(null, data.issue)[0];
       if (!issue) {
         return;
       }
@@ -254,14 +256,17 @@ export class ConsoleCollector extends PageCollector<ConsoleMessage | Error | Agg
 
       const mockManager = this.#mockIssuesManagers.get(page);
       if (mockManager) {
-        // @ts-expect-error We don't care that issues model is null
-        mockManager.dispatchEventToListeners(IssuesManagerEvents.ISSUE_ADDED, {issue, issuesModel: null});
+        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');
   }
 
-  override  cleanupPageDestroyed(page: Page) {
+  override cleanupPageDestroyed(page: Page) {
     super.cleanupPageDestroyed(page);
     this.#seenIssueKeys.delete(page);
     this.#issuesAggregators.delete(page);

src/tools/console.ts

@@ -34,7 +34,7 @@ const FILTERABLE_MESSAGE_TYPES: readonly [
   'count',
   'timeEnd',
   'verbose',
-  'issue'
+  'issue',
 ];
 
 export const listConsoleMessages = defineTool({

tests/PageCollector.test.ts

@@ -6,7 +6,14 @@
 import assert from 'node:assert';
 import {describe, it} from 'node:test';
 
-import type {Browser, Frame, HTTPRequest, Page, Target, CDPSession} from 'puppeteer-core';
+import type {
+  Browser,
+  Frame,
+  HTTPRequest,
+  Page,
+  Target,
+  CDPSession,
+} from 'puppeteer-core';
 import sinon from 'sinon';
 
 import {AggregatedIssue} from '../node_modules/chrome-devtools-frontend/mcp/mcp.js';
@@ -302,7 +309,7 @@ describe('NetworkCollector', () => {
     assert.equal(collector.getData(page).length, 2);
   });
 
-  it('works with previous navigatedations', async () => {
+  it('works with previous navigations', async () => {
     const browser = getMockBrowser();
     const page = (await browser.pages())[0];
     const mainFrame = page.mainFrame();
@@ -357,8 +364,8 @@ describe('ConsoleCollector', () => {
       code: 'MixedContentIssue' as const,
       details: {
         mixedContentIssueDetails: {
-          insecureURL: "test.url"
-        }
+          insecureURL: 'test.url',
+        },
       },
     };
 
@@ -388,22 +395,22 @@ describe('ConsoleCollector', () => {
       code: 'MixedContentIssue' as const,
       details: {
         mixedContentIssueDetails: {
-          insecureURL: "test.url"
+          insecureURL: 'test.url',
         },
       },
     };
     const issue2 = {
       code: 'PropertyRuleIssue' as const,
       details: {
         propertyRuleIssueDetails: {
-          test: "test"
+          test: 'test',
         },
       },
     };
 
     // @ts-expect-error Types of protocol from Puppeteer and CDP are incopatible for Issues but it's the same type
     cdpSession.emit('Audits.issueAdded', {issue});
-       // @ts-expect-error Types of protocol from Puppeteer and CDP are incopatible for Issues but it's the same type
+    // @ts-expect-error Types of protocol from Puppeteer and CDP are incopatible for Issues but it's the same type
     cdpSession.emit('Audits.issueAdded', {issue: issue2});
     const data = collector.getData(page);
     assert.equal(data.length, 2);
@@ -427,7 +434,7 @@ describe('ConsoleCollector', () => {
       code: 'MixedContentIssue' as const,
       details: {
         mixedContentIssueDetails: {
-          insecureURL: "test.url"
+          insecureURL: 'test.url',
         },
       },
     };