@mcp-b/chrome-devtools-mcp 2.2.0 → 2.3.0

package/README.md

@@ -689,10 +689,7 @@ Add the `--browser-url` option to your MCP client configuration. The value of th
   "mcpServers": {
     "chrome-devtools": {
       "command": "npx",
-      "args": [
-        "chrome-devtools-mcp@latest",
-        "--browser-url=http://127.0.0.1:9222"
-      ]
+      "args": ["chrome-devtools-mcp@latest", "--browser-url=http://127.0.0.1:9222"]
     }
   }
 }

package/build/src/DevToolsConnectionAdapter.js

@@ -34,8 +34,8 @@ export class PuppeteerDevToolsConnection {
         /* eslint-disable @typescript-eslint/no-explicit-any */
         return session
             .send(method, params)
-            .then(result => ({ result }))
-            .catch(error => ({ error }));
+            .then((result) => ({ result }))
+            .catch((error) => ({ error }));
         /* eslint-enable @typescript-eslint/no-explicit-any */
     }
     observe(observer) {
@@ -55,11 +55,12 @@ export class PuppeteerDevToolsConnection {
             session.off('*', handler);
         }
     }
-    #handleEvent(sessionId, type, event) {
+    #handleEvent(sessionId, type, event // eslint-disable-line @typescript-eslint/no-explicit-any
+    ) {
         if (typeof type === 'string' &&
             type !== CDPSessionEvent.SessionAttached &&
             type !== CDPSessionEvent.SessionDetached) {
-            this.#observers.forEach(observer => observer.onEvent({
+            this.#observers.forEach((observer) => observer.onEvent({
                 method: type,
                 sessionId,
                 params: event,

package/build/src/DevtoolsUtils.js

@@ -23,14 +23,13 @@ DevTools.I18n.DevToolsLocale.DevToolsLocale.instance({
     data: {
         navigatorLanguage: 'en-US',
         settingLanguage: 'en-US',
-        lookupClosestDevToolsLocale: l => l,
+        lookupClosestDevToolsLocale: (l) => l,
     },
 });
 DevTools.I18n.i18n.registerLocaleDataForTest('en-US', {});
 DevTools.Formatter.FormatterWorkerPool.FormatterWorkerPool.instance({
     forceNew: true,
-    entrypointURL: import.meta
-        .resolve('./third_party/devtools-formatter-worker.js'),
+    entrypointURL: import.meta.resolve('./third_party/devtools-formatter-worker.js'),
 });
 export class UniverseManager {
     #browser;
@@ -47,7 +46,7 @@ export class UniverseManager {
             await this.#mutex.acquire();
             const promises = [];
             for (const page of pages) {
-                promises.push(this.#createUniverseFor(page).then(targetUniverse => this.#universes.set(page, targetUniverse)));
+                promises.push(this.#createUniverseFor(page).then((targetUniverse) => this.#universes.set(page, targetUniverse)));
             }
             this.#browser.on('targetcreated', this.#onTargetCreated);
             this.#browser.on('targetdestroyed', this.#onTargetDestroyed);
@@ -196,8 +195,7 @@ export class SymbolizedError {
         // we have to manually parse out the error text from the exception description.
         // In the case of Runtime.getExceptionDetails, `details.text` has the Error.message.
         if (details.text === 'Uncaught' && details.exception) {
-            return ('Uncaught ' +
-                SymbolizedError.#getMessageFromException(details.exception));
+            return 'Uncaught ' + SymbolizedError.#getMessageFromException(details.exception);
         }
         return details.text;
     }
@@ -230,7 +228,7 @@ export class SymbolizedError {
         if (properties.getError()) {
             return null;
         }
-        return properties.result.find(prop => prop.name === 'cause')?.value ?? null;
+        return properties.result.find((prop) => prop.name === 'cause')?.value ?? null;
     }
     static createForTesting(message, stackTrace, cause) {
         return new SymbolizedError(message, stackTrace, cause);
@@ -246,9 +244,7 @@ export async function createStackTraceForConsoleMessage(devTools, consoleMessage
 }
 export async function createStackTrace(devTools, rawStackTrace, targetId) {
     const targetManager = devTools.universe.context.get(DevTools.TargetManager);
-    const target = targetId
-        ? targetManager.targetById(targetId) || devTools.target
-        : devTools.target;
+    const target = targetId ? targetManager.targetById(targetId) || devTools.target : devTools.target;
     const model = target.model(DevTools.DebuggerModel);
     // DevTools doesn't wait for source maps to attach before building a stack trace, rather it'll send
     // an update event once a source map was attached and the stack trace retranslated. This doesn't
@@ -265,8 +261,8 @@ export async function createStackTrace(devTools, rawStackTrace, targetId) {
         }
     }
     const signal = AbortSignal.timeout(1_000);
-    await Promise.all([...scriptIds].map(id => waitForScript(model, id, signal)
-        .then(script => model.sourceMapManager().sourceMapForClientPromise(script))
+    await Promise.all([...scriptIds].map((id) => waitForScript(model, id, signal)
+        .then((script) => model.sourceMapManager().sourceMapForClientPromise(script))
         .catch()));
     const binding = devTools.universe.context.get(DevTools.DebuggerWorkspaceBinding);
     // DevTools uses branded types for ScriptId and others. Casting the puppeteer protocol type to the DevTools protocol type is safe.

package/build/src/McpContext.js

@@ -7,12 +7,12 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import { UniverseManager } from './DevtoolsUtils.js';
 import { McpPage } from './McpPage.js';
-import { NetworkCollector, ConsoleCollector } from './PageCollector.js';
+import { NetworkCollector, ConsoleCollector, } from './PageCollector.js';
 import { Locator } from './third_party/index.js';
 import { PredefinedNetworkConditions } from './third_party/index.js';
 import { listPages } from './tools/pages.js';
 import { CLOSE_PAGE_ERROR } from './tools/ToolDefinition.js';
-import { ExtensionRegistry, } from './utils/ExtensionRegistry.js';
+import { ExtensionRegistry } from './utils/ExtensionRegistry.js';
 import { saveTemporaryFile } from './utils/files.js';
 import { WaitForHelper } from './WaitForHelper.js';
 const DEFAULT_TIMEOUT = 5_000;
@@ -62,15 +62,15 @@ export class McpContext {
         this.#locatorClass = locatorClass;
         this.#options = options;
         this.#networkCollector = new NetworkCollector(this.browser);
-        this.#consoleCollector = new ConsoleCollector(this.browser, collect => {
+        this.#consoleCollector = new ConsoleCollector(this.browser, (collect) => {
             return {
-                console: event => {
+                console: (event) => {
                     collect(event);
                 },
-                uncaughtError: event => {
+                uncaughtError: (event) => {
                     collect(event);
                 },
-                issue: event => {
+                issue: (event) => {
                     collect(event);
                 },
             };
@@ -109,7 +109,7 @@ export class McpContext {
             this.logger('no network request');
             return;
         }
-        const request = this.#networkCollector.find(page.pptrPage, request => {
+        const request = this.#networkCollector.find(page.pptrPage, (request) => {
             // @ts-expect-error id is internal.
             return request.id === cdpRequestId;
         });
@@ -241,9 +241,7 @@ export class McpContext {
             newSettings.userAgent = options.userAgent;
         }
         if (!options.colorScheme || options.colorScheme === 'auto') {
-            await page.emulateMediaFeatures([
-                { name: 'prefers-color-scheme', value: '' },
-            ]);
+            await page.emulateMediaFeatures([{ name: 'prefers-color-scheme', value: '' }]);
             delete newSettings.colorScheme;
         }
         else {
@@ -267,9 +265,7 @@ export class McpContext {
             await page.setViewport(viewport);
             newSettings.viewport = viewport;
         }
-        mcpPage.emulationSettings = Object.keys(newSettings).length
-            ? newSettings
-            : {};
+        mcpPage.emulationSettings = Object.keys(newSettings).length ? newSettings : {};
         this.#updateSelectedPageTimeouts();
     }
     setIsRunningPerformanceTrace(x) {
@@ -302,7 +298,7 @@ export class McpContext {
         return this.#getMcpPage(page);
     }
     getPageById(pageId) {
-        const page = this.#mcpPages.values().find(mcpPage => mcpPage.id === pageId);
+        const page = this.#mcpPages.values().find((mcpPage) => mcpPage.id === pageId);
         if (!page) {
             throw new Error('No page found');
         }
@@ -357,16 +353,15 @@ export class McpContext {
      */
     async createExtensionServiceWorkersSnapshot() {
         const allTargets = await this.browser.targets();
-        const serviceWorkers = allTargets.filter(target => {
-            return (target.type() === 'service_worker' &&
-                target.url().includes('chrome-extension://'));
+        const serviceWorkers = allTargets.filter((target) => {
+            return target.type() === 'service_worker' && target.url().includes('chrome-extension://');
         });
         for (const serviceWorker of serviceWorkers) {
             if (!this.#extensionServiceWorkerMap.has(serviceWorker)) {
                 this.#extensionServiceWorkerMap.set(serviceWorker, 'sw-' + this.#nextExtensionServiceWorkerId++);
             }
         }
-        this.#extensionServiceWorkers = serviceWorkers.map(serviceWorker => {
+        this.#extensionServiceWorkers = serviceWorkers.map((serviceWorker) => {
             return {
                 target: serviceWorker,
                 id: this.#extensionServiceWorkerMap.get(serviceWorker),
@@ -383,7 +378,7 @@ export class McpContext {
                 mcpPage = new McpPage(page, this.#nextPageId++);
                 this.#mcpPages.set(page, mcpPage);
                 // We emulate a focused page for all pages to support multi-agent workflows.
-                void page.emulateFocusedPage(true).catch(error => {
+                void page.emulateFocusedPage(true).catch((error) => {
                     this.logger('Error turning on focused page emulation', error);
                 });
             }
@@ -397,12 +392,10 @@ export class McpContext {
                 this.#mcpPages.delete(page);
             }
         }
-        this.#pages = allPages.filter(page => {
-            return (this.#options.experimentalDevToolsDebugging ||
-                !page.url().startsWith('devtools://'));
+        this.#pages = allPages.filter((page) => {
+            return this.#options.experimentalDevToolsDebugging || !page.url().startsWith('devtools://');
         });
-        if ((!this.#selectedPage ||
-            this.#pages.indexOf(this.#selectedPage.pptrPage) === -1) &&
+        if ((!this.#selectedPage || this.#pages.indexOf(this.#selectedPage.pptrPage) === -1) &&
             this.#pages[0]) {
             this.selectPage(this.#getMcpPage(this.#pages[0]));
         }
@@ -413,9 +406,8 @@ export class McpContext {
         const defaultCtx = this.browser.defaultBrowserContext();
         const allPages = await this.browser.pages(this.#options.experimentalIncludeAllPages);
         const allTargets = this.browser.targets();
-        const extensionTargets = allTargets.filter(target => {
-            return (target.url().startsWith('chrome-extension://') &&
-                target.type() === 'page');
+        const extensionTargets = allTargets.filter((target) => {
+            return target.url().startsWith('chrome-extension://') && target.type() === 'page';
         });
         for (const target of extensionTargets) {
             // Right now target.page() returns null for popup and side panel pages.
@@ -472,10 +464,18 @@ export class McpContext {
             if (!mcpPage) {
                 return;
             }
-            if (await page.hasDevTools()) {
-                mcpPage.devToolsPage = await page.openDevTools();
+            // Prior to Chrome 144.0.7559.59, the command fails,
+            // Some Electron apps still use older version
+            // Fall back to not exposing DevTools at all.
+            try {
+                if (await page.hasDevTools()) {
+                    mcpPage.devToolsPage = await page.openDevTools();
+                }
+                else {
+                    mcpPage.devToolsPage = undefined;
+                }
             }
-            else {
+            catch {
                 mcpPage.devToolsPage = undefined;
             }
         }));
@@ -557,9 +557,7 @@ export class McpContext {
             const nodeWithId = {
                 ...node,
                 id,
-                children: node.children
-                    ? node.children.map(child => assignIds(child))
-                    : [],
+                children: node.children ? node.children.map((child) => assignIds(child)) : [],
             };
             // The AXNode for an option doesn't contain its `value`.
             // Therefore, set text content of the option as value.
@@ -632,10 +630,7 @@ export class McpContext {
     waitForTextOnPage(text, timeout, targetPage) {
         const page = targetPage ?? this.getSelectedPptrPage();
         const frames = page.frames();
-        let locator = this.#locatorClass.race(frames.flatMap(frame => text.flatMap(value => [
-            frame.locator(`aria/${value}`),
-            frame.locator(`text/${value}`),
-        ])));
+        let locator = this.#locatorClass.race(frames.flatMap((frame) => text.flatMap((value) => [frame.locator(`aria/${value}`), frame.locator(`text/${value}`)])));
         if (timeout) {
             locator = locator.setTimeout(timeout);
         }
@@ -645,9 +640,9 @@ export class McpContext {
      * We need to ignore favicon request as they make our test flaky
      */
     async setUpNetworkCollectorForTesting() {
-        this.#networkCollector = new NetworkCollector(this.browser, collect => {
+        this.#networkCollector = new NetworkCollector(this.browser, (collect) => {
             return {
-                request: req => {
+                request: (req) => {
                     if (req.url().includes('favicon.ico')) {
                         return;
                     }

package/build/src/McpResponse.js

@@ -24,9 +24,7 @@ export class McpResponse {
     #attachedTraceInsight;
     #attachedLighthouseResult;
     #textResponseLines = [];
-    #mcpContent = [];
     #images = [];
-    #isError = false;
     #networkRequestsOptions;
     #consoleDataOptions;
     #listExtensions;
@@ -116,12 +114,6 @@ export class McpResponse {
     attachLighthouseResult(result) {
         this.#attachedLighthouseResult = result;
     }
-    appendMcpContent(value) {
-        this.#mcpContent.push(value);
-    }
-    setToolResultError(value) {
-        this.#isError = value;
-    }
     get includePages() {
         return this.#includePages;
     }
@@ -164,9 +156,6 @@ export class McpResponse {
     get images() {
         return this.#images;
     }
-    get isError() {
-        return this.#isError;
-    }
     get snapshotParams() {
         return this.#snapshotParams;
     }
@@ -203,7 +192,7 @@ export class McpResponse {
             const request = context.getNetworkRequestById(this.#page, this.#attachedNetworkRequestId);
             const formatter = await NetworkFormatter.from(request, {
                 requestId: this.#attachedNetworkRequestId,
-                requestIdResolver: req => context.getNetworkRequestStableId(req),
+                requestIdResolver: (req) => context.getNetworkRequestStableId(req),
                 fetchData: true,
                 requestFilePath: this.#attachedNetworkRequestOptions?.requestFilePath,
                 responseFilePath: this.#attachedNetworkRequestOptions?.responseFilePath,
@@ -252,7 +241,7 @@ export class McpResponse {
             let messages = context.getConsoleData(this.#page, this.#consoleDataOptions.includePreservedMessages);
             if (this.#consoleDataOptions.types?.length) {
                 const normalizedTypes = new Set(this.#consoleDataOptions.types);
-                messages = messages.filter(message => {
+                messages = messages.filter((message) => {
                     if ('type' in message) {
                         return normalizedTypes.has(message.type());
                     }
@@ -283,7 +272,7 @@ export class McpResponse {
                     return formatter;
                 }
                 return null;
-            }))).filter(item => item !== null);
+            }))).filter((item) => item !== null);
         }
         let networkRequests;
         if (this.#networkRequestsOptions?.include) {
@@ -294,13 +283,13 @@ export class McpResponse {
             // Apply resource type filtering if specified
             if (this.#networkRequestsOptions.resourceTypes?.length) {
                 const normalizedTypes = new Set(this.#networkRequestsOptions.resourceTypes);
-                requests = requests.filter(request => {
+                requests = requests.filter((request) => {
                     const type = request.resourceType();
                     return normalizedTypes.has(type);
                 });
             }
             if (requests.length) {
-                networkRequests = await Promise.all(requests.map(request => NetworkFormatter.from(request, {
+                networkRequests = await Promise.all(requests.map((request) => NetworkFormatter.from(request, {
                     requestId: context.getNetworkRequestStableId(request),
                     selectedInDevToolsUI: context.getNetworkRequestStableId(request) ===
                         this.#networkRequestsOptions?.networkRequestIdInDevToolsUI,
@@ -358,9 +347,7 @@ export class McpResponse {
         }
         const dialog = this.#page?.getDialog();
         if (dialog) {
-            const defaultValueIfNeeded = dialog.type() === 'prompt'
-                ? ` (default value: "${dialog.defaultValue()}")`
-                : '';
+            const defaultValueIfNeeded = dialog.type() === 'prompt' ? ` (default value: "${dialog.defaultValue()}")` : '';
             response.push(`# Open dialog
 ${dialog.type()}: ${dialog.message()}${defaultValueIfNeeded}.
 Call ${handleDialog.name} to handle it before continuing.`);
@@ -386,9 +373,7 @@ Call ${handleDialog.name} to handle it before continuing.`);
                 const structuredPages = [];
                 for (const page of regularPages) {
                     const isolatedContextName = context.getIsolatedContextName(page);
-                    const contextLabel = isolatedContextName
-                        ? ` isolatedContext=${isolatedContextName}`
-                        : '';
+                    const contextLabel = isolatedContextName ? ` isolatedContext=${isolatedContextName}` : '';
                     parts.push(`${context.getPageId(page)}: ${page.url()}${context.isPageSelected(page) ? ' [selected]' : ''}${contextLabel}`);
                     structuredPages.push(createStructuredPage(page, context));
                 }
@@ -420,7 +405,7 @@ Call ${handleDialog.name} to handle it before continuing.`);
             }
             structuredContent.extensionServiceWorkers = context
                 .getExtensionServiceWorkers()
-                .map(extensionServiceWorker => {
+                .map((extensionServiceWorker) => {
                 return {
                     id: extensionServiceWorker.id,
                     url: extensionServiceWorker.url,
@@ -486,13 +471,11 @@ Call ${handleDialog.name} to handle it before continuing.`);
         }
         if (data.detailedNetworkRequest) {
             response.push(data.detailedNetworkRequest.toStringDetailed());
-            structuredContent.networkRequest =
-                data.detailedNetworkRequest.toJSONDetailed();
+            structuredContent.networkRequest = data.detailedNetworkRequest.toJSONDetailed();
         }
         if (data.detailedConsoleMessage) {
             response.push(data.detailedConsoleMessage.toStringDetailed());
-            structuredContent.consoleMessage =
-                data.detailedConsoleMessage.toJSONDetailed();
+            structuredContent.consoleMessage = data.detailedConsoleMessage.toJSONDetailed();
         }
         if (data.extensions) {
             structuredContent.extensions = data.extensions;
@@ -502,7 +485,7 @@ Call ${handleDialog.name} to handle it before continuing.`);
             }
             else {
                 const extensionsMessage = data.extensions
-                    .map(extension => {
+                    .map((extension) => {
                     return `id=${extension.id} "${extension.name}" v${extension.version} ${extension.isEnabled ? 'Enabled' : 'Disabled'}`;
                 })
                     .join('\n');
@@ -535,34 +518,25 @@ Call ${handleDialog.name} to handle it before continuing.`);
                 const paginationData = this.#dataWithPagination(messages, this.#consoleDataOptions.pagination);
                 structuredContent.pagination = paginationData.pagination;
                 response.push(...paginationData.info);
-                response.push(...paginationData.items.map(message => message.toString()));
-                structuredContent.consoleMessages = paginationData.items.map(message => message.toJSON());
+                response.push(...paginationData.items.map((message) => message.toString()));
+                structuredContent.consoleMessages = paginationData.items.map((message) => message.toJSON());
             }
             else {
                 response.push('<no console messages found>');
             }
         }
-        const textBlocks = response.length || !this.#mcpContent.length
-            ? [
-                {
-                    type: 'text',
-                    text: response.join('\n'),
-                },
-            ]
-            : [];
-        const images = this.#images.map(imageData => {
+        const text = {
+            type: 'text',
+            text: response.join('\n'),
+        };
+        const images = this.#images.map((imageData) => {
             return {
                 type: 'image',
                 ...imageData,
             };
         });
-        const content = [
-            ...textBlocks,
-            ...this.#mcpContent,
-            ...images,
-        ];
         return {
-            content,
+            content: [text, ...images],
             structuredContent,
         };
     }

package/build/src/PageCollector.js

@@ -28,7 +28,7 @@ export class PageCollector {
     #browser;
     #listenersInitializer;
     #listeners = new WeakMap();
-    #maxNavigationSaved = 3;
+    maxNavigationSaved = 3;
     /**
      * This maps a Page to a list of navigations with a sub-list
      * of all collected resources.
@@ -84,7 +84,7 @@ export class PageCollector {
         const idGenerator = createIdGenerator();
         const storedLists = [[]];
         this.storage.set(page, storedLists);
-        const listeners = this.#listenersInitializer(value => {
+        const listeners = this.#listenersInitializer((value) => {
             const withId = value;
             withId[stableIdSymbol] = idGenerator();
             const navigations = this.storage.get(page) ?? [[]];
@@ -109,7 +109,7 @@ export class PageCollector {
         }
         // Add the latest navigation first
         navigations.unshift([]);
-        navigations.splice(this.#maxNavigationSaved);
+        navigations.splice(this.maxNavigationSaved);
     }
     cleanupPageDestroyed(page) {
         const listeners = this.#listeners.get(page);
@@ -129,7 +129,7 @@ export class PageCollector {
             return navigations[0];
         }
         const data = [];
-        for (let index = this.#maxNavigationSaved; index >= 0; index--) {
+        for (let index = this.maxNavigationSaved; index >= 0; index--) {
             if (navigations[index]) {
                 data.push(...navigations[index]);
             }
@@ -144,7 +144,7 @@ export class PageCollector {
         if (!navigations) {
             throw new Error('No requests found for selected page');
         }
-        const item = this.find(page, item => item[stableIdSymbol] === stableId);
+        const item = this.find(page, (item) => item[stableIdSymbol] === stableId);
         if (item) {
             return item;
         }
@@ -275,9 +275,9 @@ class PageEventSubscriber {
     };
 }
 export class NetworkCollector extends PageCollector {
-    constructor(browser, listeners = collect => {
+    constructor(browser, listeners = (collect) => {
         return {
-            request: req => {
+            request: (req) => {
                 collect(req);
             },
         };
@@ -290,10 +290,8 @@ export class NetworkCollector extends PageCollector {
             return;
         }
         const requests = navigations[0];
-        const lastRequestIdx = requests.findLastIndex(request => {
-            return request.frame() === page.mainFrame()
-                ? request.isNavigationRequest()
-                : false;
+        const lastRequestIdx = requests.findLastIndex((request) => {
+            return request.frame() === page.mainFrame() ? request.isNavigationRequest() : false;
         });
         // Keep all requests since the last navigation request including that
         // navigation request itself.
@@ -305,5 +303,6 @@ export class NetworkCollector extends PageCollector {
         else {
             navigations.unshift([]);
         }
+        navigations.splice(this.maxNavigationSaved);
     }
 }

package/build/src/WaitForHelper.js

@@ -24,7 +24,7 @@ export class WaitForHelper {
      * for the DOM to be stable before returning.
      */
     async waitForStableDom() {
-        const stableDomObserver = await this.#page.evaluateHandle(timeout => {
+        const stableDomObserver = await this.#page.evaluateHandle((timeout) => {
             let timeoutId;
             function callback() {
                 clearTimeout(timeoutId);
@@ -49,7 +49,7 @@ export class WaitForHelper {
         }, this.#stableDomFor);
         this.#abortController.signal.addEventListener('abort', async () => {
             try {
-                await stableDomObserver.evaluate(observer => {
+                await stableDomObserver.evaluate((observer) => {
                     observer.observer.disconnect();
                     observer.resolver.resolve();
                 });
@@ -71,13 +71,9 @@ export class WaitForHelper {
     async waitForNavigationStarted() {
         // Currently Puppeteer does not have API
         // For when a navigation is about to start
-        const navigationStartedPromise = new Promise(resolve => {
+        const navigationStartedPromise = new Promise((resolve) => {
             const listener = (event) => {
-                if ([
-                    'historySameDocument',
-                    'historyDifferentDocument',
-                    'sameDocument',
-                ].includes(event.navigationType)) {
+                if (['historySameDocument', 'historyDifferentDocument', 'sameDocument'].includes(event.navigationType)) {
                     resolve(false);
                     return;
                 }
@@ -95,7 +91,7 @@ export class WaitForHelper {
         ]);
     }
     timeout(time) {
-        return new Promise(res => {
+        return new Promise((res) => {
             const id = setTimeout(res, time);
             this.#abortController.signal.addEventListener('abort', () => {
                 res();
@@ -105,7 +101,7 @@ export class WaitForHelper {
     }
     async waitForEventsAfterAction(action, options) {
         const navigationFinished = this.waitForNavigationStarted()
-            .then(navigationStated => {
+            .then((navigationStated) => {
             if (navigationStated) {
                 return this.#page.waitForNavigation({
                     timeout: options?.timeout ?? this.#navigationTimeout,
@@ -114,7 +110,7 @@ export class WaitForHelper {
             }
             return;
         })
-            .catch(error => logger(error));
+            .catch((error) => logger(error));
         try {
             await action();
         }