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

@@ -372,7 +372,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)
@@ -228,6 +229,20 @@
 
 ## Performance
 
+### `lighthouse_audit`
+
+**Description:** Runs a Lighthouse audit on the currently selected page.
+
+**Parameters:**
+
+- **categories** (array) _(optional)_: The categories to audit. Defaults to all available categories.
+- **device** (enum: "desktop", "mobile") _(optional)_: The device to [`emulate`](#emulate). "desktop" is the default. "mobile" emulates a mobile device.
+- **formats** (array) _(optional)_: Report formats to produce.
+- **mode** (enum: "navigation", "snapshot") _(optional)_: The mode to run Lighthouse in. "navigation" is the default and will reload the current page. "snapshot" analyzes the page in its current state.
+- **outputDirPath** (string) _(optional)_: The directory to output the reports to. If not provided, temporary files will be created.
+
+---
+
 ### `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.

eslint.config.mjs

@@ -14,7 +14,11 @@ import tseslint from 'typescript-eslint';
 import localPlugin from './scripts/eslint_rules/local-plugin.js';
 
 export default defineConfig([
-  globalIgnores(['**/node_modules', '**/build/']),
+  globalIgnores([
+    '**/node_modules',
+    '**/build/',
+    'src/third_party/lighthouse-devtools-mcp-bundle.js',
+  ]),
   importPlugin.flatConfigs.typescript,
   {
     languageOptions: {

package-lock.json

@@ -61,7 +61,7 @@
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-import": "^2.32.0",
     "globals": "^17.0.0",
-    "lighthouse": "^13.0.1",
+    "lighthouse": "13.0.2",
     "prettier": "^3.6.2",
     "puppeteer": "24.37.2",
     "rollup": "4.57.1",

package.json

@@ -107,7 +107,8 @@ function listBundledDeps() {
           Object.entries(devDependencies).filter(
             ([name]) =>
               aggregatedStats.bundledPackages.has(name) ||
-              name === 'chrome-devtools-frontend',
+              name === 'chrome-devtools-frontend' ||
+              name === 'lighthouse',
           ),
         );
 
@@ -260,11 +261,11 @@ export default [
         return true;
       }
 
-      const existingExternals = ['./bidi.js', '../bidi/bidi.js'];
+      const existingExternals = ['./bidi.js', '../bidi/bidi.js', './lighthouse-devtools-mcp-bundle.js'];
+      
       if (existingExternals.includes(source)) {
         return true;
       }
-
       return false;
     },
   ),

rollup.config.mjs

@@ -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 { ... } ... }

scripts/prepare.ts

@@ -93,18 +93,6 @@ function getNetworkMultiplierFromString(condition: string | null): number {
   return 1;
 }
 
-function getExtensionFromMimeType(mimeType: string) {
-  switch (mimeType) {
-    case 'image/png':
-      return 'png';
-    case 'image/jpeg':
-      return 'jpeg';
-    case 'image/webp':
-      return 'webp';
-  }
-  throw new Error(`No mapping for Mime type ${mimeType}.`);
-}
-
 export class McpContext implements Context {
   browser: Browser;
   logger: Debugger;
@@ -677,22 +665,19 @@ export class McpContext implements Context {
 
   async saveTemporaryFile(
     data: Uint8Array<ArrayBufferLike>,
-    mimeType: 'image/png' | 'image/jpeg' | 'image/webp',
-  ): Promise<{filename: string}> {
+    filename: string,
+  ): Promise<{filepath: string}> {
     try {
       const dir = await fs.mkdtemp(
         path.join(os.tmpdir(), 'chrome-devtools-mcp-'),
       );
 
-      const filename = path.join(
-        dir,
-        `screenshot.${getExtensionFromMimeType(mimeType)}`,
-      );
-      await fs.writeFile(filename, data);
-      return {filename};
+      const filepath = path.join(dir, filename);
+      await fs.writeFile(filepath, data);
+      return {filepath};
     } catch (err) {
       this.logger(err);
-      throw new Error('Could not save a screenshot to a file', {cause: err});
+      throw new Error('Could not save a file', {cause: err});
     }
   }
   async saveFile(
@@ -701,11 +686,12 @@ export class McpContext implements Context {
   ): Promise<{filename: string}> {
     try {
       const filePath = path.resolve(filename);
+      await fs.mkdir(path.dirname(filePath), {recursive: true});
       await fs.writeFile(filePath, data);
-      return {filename};
+      return {filename: filePath};
     } catch (err) {
       this.logger(err);
-      throw new Error('Could not save a screenshot to a file', {cause: err});
+      throw new Error('Could not save a file', {cause: err});
     }
   }