chore: try snapshot

Author: OrKoN

.github/workflows/run-tests.yml

@@ -44,7 +44,7 @@ jobs:
       - name: Build
         run: npm run bundle
         env:
-          NODE_OPTIONS: '--max_old_space_size=4096'
+          NODE_OPTIONS: '--max_old_space_size=8192'
 
       - name: Set up Node.js
         uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0

README.md

@@ -347,7 +347,8 @@ If you run into any issues, checkout our [troubleshooting guide](./docs/troubles
 - **Emulation** (2 tools)
   - [`emulate`](docs/tool-reference.md#emulate)
   - [`resize_page`](docs/tool-reference.md#resize_page)
-- **Performance** (3 tools)
+- **Performance** (4 tools)
+  - [`lighthouse_audit`](docs/tool-reference.md#lighthouse_audit)
   - [`performance_analyze_insight`](docs/tool-reference.md#performance_analyze_insight)
   - [`performance_start_trace`](docs/tool-reference.md#performance_start_trace)
   - [`performance_stop_trace`](docs/tool-reference.md#performance_stop_trace)

docs/tool-reference.md

@@ -21,7 +21,8 @@
 - **[Emulation](#emulation)** (2 tools)
   - [`emulate`](#emulate)
   - [`resize_page`](#resize_page)
-- **[Performance](#performance)** (3 tools)
+- **[Performance](#performance)** (4 tools)
+  - [`lighthouse_audit`](#lighthouse_audit)
   - [`performance_analyze_insight`](#performance_analyze_insight)
   - [`performance_start_trace`](#performance_start_trace)
   - [`performance_stop_trace`](#performance_stop_trace)
@@ -225,6 +226,14 @@
 
 ## Performance
 
+### `lighthouse_audit`
+
+**Description:** Runs a Lighthouse accessibility audit on the currently selected page. This tool analyzes the page for accessibility issues and provides a detailed report with scores and recommendations.
+
+**Parameters:** None
+
+---
+
 ### `performance_analyze_insight`
 
 **Description:** Provides more detailed information on a specific Performance Insight of an insight set that was highlighted in the results of a trace recording.

scripts/prepare.ts

@@ -27,7 +27,9 @@ function removeConflictingGlobalDeclaration(): void {
     projectRoot,
     'node_modules/@paulirish/trace_engine/models/trace/ModelImpl.d.ts',
   );
-  console.log('Removing conflicting global declaration from @paulirish/trace_engine...');
+  console.log(
+    'Removing conflicting global declaration from @paulirish/trace_engine...',
+  );
   const content = readFileSync(filePath, 'utf-8');
   // Remove the declare global block using regex
   // Matches: declare global { ... interface HTMLElementEventMap { ... } ... }

src/third_party/index.ts

@@ -38,5 +38,6 @@ export {
   type ChromeReleaseChannel as BrowsersChromeReleaseChannel,
 } from '@puppeteer/browsers';
 
-export * as Lighthouse from 'lighthouse';
+export {snapshot} from './lighthouse-devtools-mcp-bundle.js';
+
 export * as DevTools from '../../node_modules/chrome-devtools-frontend/mcp/mcp.js';

src/tools/lighthouse.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import {Lighthouse, zod} from '../third_party/index.js';
+import {snapshot} from '../third_party/index.js';
 
 import {ToolCategory} from './categories.js';
 import {defineTool} from './ToolDefinition.js';
@@ -19,20 +19,21 @@ export const lighthouseAudit = defineTool({
   schema: {},
   handler: async (request, response, context) => {
     const page = context.getSelectedPage();
-    const url = page.url();
-
     const flags = {
       onlyCategories: ['accessibility'],
     };
-
-    const result = await Lighthouse.default(url, flags, undefined, page);
+    const result = await snapshot(page, {
+      flags,
+    });
     const lhr = result!.lhr;
     const accessibilityCategory = lhr.categories.accessibility;
 
     const failedAudits = Object.values(lhr.audits).filter(
+      // @ts-ignore
       audit => audit.score !== null && audit.score < 1,
     );
     const passedAudits = Object.values(lhr.audits).filter(
+      // @ts-ignore
       audit => audit.score === 1,
     );
 

tests/tools/lighthouse.test.js.snapshot

@@ -33,8 +33,8 @@ exports[`lighthouse > lighthouse_audit > runs Lighthouse accessibility audit 1`]
                   "top": 0,
                   "bottom": 34,
                   "left": 0,
-                  "right": 412,
-                  "width": 412,
+                  "right": 1200,
+                  "width": 1200,
                   "height": 34
                 },
                 "snippet": "<html lang=\\"en\\">",

tests/tools/lighthouse.test.ts

@@ -4,7 +4,6 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import assert from 'node:assert';
 import {describe, it} from 'node:test';
 
 import {lighthouseAudit} from '../../src/tools/lighthouse.js';
@@ -14,21 +13,18 @@ import {html, withMcpContext} from '../utils.js';
 describe('lighthouse', () => {
   const server = serverHooks();
   describe('lighthouse_audit', () => {
-    it(
-      'runs Lighthouse accessibility audit',
-      async (t) => {
-        server.addHtmlRoute('/test', html`<div>Test</div>`);
+    it('runs Lighthouse accessibility audit', async t => {
+      server.addHtmlRoute('/test', html`<div>Test</div>`);
 
-        await withMcpContext(async (response, context) => {
-          const page = context.getSelectedPage();
-          await page.goto(server.getRoute('/test'));
+      await withMcpContext(async (response, context) => {
+        const page = context.getSelectedPage();
+        await page.goto(server.getRoute('/test'));
 
-          await lighthouseAudit.handler({params: {}}, response, context);
+        await lighthouseAudit.handler({params: {}}, response, context);
 
-          const responseText = response.responseLines.join('\n');
-          t.assert.snapshot?.(responseText);
-        });
-      },
-    );
+        const responseText = response.responseLines.join('\n');
+        t.assert.snapshot?.(responseText);
+      });
+    });
   });
 });