Merge branch 'main' into mcp-dialog-pt1

Author: samiyac

scripts/eval_scenarios/fill_select_and_checkboxes_test.ts

@@ -0,0 +1,67 @@
+/**
+ * @license
+ * Copyright 2026 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import assert from 'node:assert';
+
+import type {TestScenario} from '../eval_gemini.ts';
+
+export const scenario: TestScenario = {
+  prompt:
+    'Go to <TEST_URL>, fill the form with size = 2 CPUs and components = [docker, nginx].',
+  maxTurns: 3,
+  htmlRoute: {
+    path: '/input_test.html',
+    htmlContent: `
+      <form action="/post" method="POST">
+        <div>
+          <label for="size">CPU/Memory size:</label>
+          <select id="size" name="size" required>
+            <option value="small">1 vCPU, 2GB RAM</option>
+            <option value="medium">2 vCPU, 4GB RAM</option>
+            <option value="large">4 vCPU, 8GB RAM</option>
+          </select>
+        </div>
+        <br>
+        <div>
+          <p>Pre-installed components:</p>
+          <input type="checkbox" id="docker" name="components" value="docker">
+          <label for="docker">Docker</label><br>
+          <input type="checkbox" id="nodejs" name="components" value="nodejs">
+          <label for="nodejs">Node.js</label><br>
+          <input type="checkbox" id="python" name="components" value="python">
+          <label for="python">Python</label><br>
+          <input type="checkbox" id="nginx" name="components" value="nginx">
+          <label for="nginx">Nginx</label>
+        </div>
+        <button type="submit">Spawn Server</button>
+      </form>
+    `,
+  },
+  expectations: calls => {
+    assert.strictEqual(calls.length, 3);
+    assert.ok(
+      calls[0].name === 'navigate_page' || calls[0].name === 'new_page',
+    );
+    assert.strictEqual(calls[1].name, 'take_snapshot');
+    assert.strictEqual(calls[2].name, 'fill_form');
+
+    const elements = calls[2].args.elements as Array<{
+      uid: string;
+      value: string;
+    }>;
+    assert.strictEqual(elements.length, 3);
+
+    const uids = new Set(elements.map(e => e.uid));
+    assert.strictEqual(
+      uids.size,
+      3,
+      'fill_form should target three distinct elements',
+    );
+
+    const values = elements.map(e => e.value).sort();
+    assert.deepStrictEqual(values, ['2 vCPU, 4GB RAM', 'true', 'true']);
+  },
+};

scripts/generate-cli.ts

@@ -16,7 +16,7 @@ import {createTools} from '../build/src/tools/tools.js';
 
 const OUTPUT_PATH = path.join(
   import.meta.dirname,
-  '../src/bin/cliDefinitions.ts',
+  '../src/bin/chrome-devtools-cli-options.ts',
 );
 
 async function fetchTools() {

src/McpResponse.ts

@@ -9,6 +9,7 @@ import type {WebMCPTool} from 'puppeteer-core';
 import type {ParsedArguments} from './bin/chrome-devtools-mcp-cli-options.js';
 import {ConsoleFormatter} from './formatters/ConsoleFormatter.js';
 import {HeapSnapshotFormatter} from './formatters/HeapSnapshotFormatter.js';
+import {isNodeLike} from './formatters/HeapSnapshotFormatter.js';
 import {IssueFormatter} from './formatters/IssueFormatter.js';
 import {NetworkFormatter} from './formatters/NetworkFormatter.js';
 import {SnapshotFormatter} from './formatters/SnapshotFormatter.js';
@@ -957,8 +958,12 @@ Call ${handleDialog.name} to handle it before continuing.`);
       }
       const nodes = this.#heapSnapshotOptions.nodes;
       if (nodes) {
+        const sortedItems = nodes.items
+          .filter(isNodeLike)
+          .sort((a, b) => b.retainedSize - a.retainedSize);
+
         const paginationData = this.#dataWithPagination(
-          nodes.items,
+          sortedItems,
           this.#heapSnapshotOptions.pagination,
         );
 

src/bin/chrome-devtools-cli-options.ts

@@ -110,7 +110,7 @@ export const commands: Commands = {
         name: 'geolocation',
         type: 'string',
         description:
-          'Geolocation (`<latitude>x<longitude>`) to emulate. Latitude between -90 and 90. Longitude between -180 and 180. Omit clear the geolocation override.',
+          'Geolocation (`<latitude>x<longitude>`) to emulate. Latitude between -90 and 90. Longitude between -180 and 180. Omit to clear the geolocation override.',
         required: false,
       },
       userAgent: {
@@ -155,11 +155,18 @@ export const commands: Commands = {
         description: 'An optional list of arguments to pass to the function.',
         required: false,
       },
+      dialogAction: {
+        name: 'dialogAction',
+        type: 'string',
+        description:
+          'Handle dialogs while execution. "accept", "dismiss", or string for response of window.prompt. Defaults to accept.',
+        required: false,
+      },
     },
   },
   fill: {
     description:
-      'Type text into a input, text area or select an option from a <select> element.',
+      'Type text into an input, text area or select an option from a <select> element.',
     category: 'Input automation',
     args: {
       uid: {
@@ -184,25 +191,6 @@ export const commands: Commands = {
       },
     },
   },
-  fill_form: {
-    description: 'Fill out multiple form elements at once',
-    category: 'Input automation',
-    args: {
-      elements: {
-        name: 'elements',
-        type: 'array',
-        description: 'Elements from snapshot to fill out.',
-        required: true,
-      },
-      includeSnapshot: {
-        name: 'includeSnapshot',
-        type: 'boolean',
-        description:
-          'Whether to include a snapshot in the response. Default is false.',
-        required: false,
-      },
-    },
-  },
   get_console_message: {
     description:
       'Gets a console message by its ID. You can get all messages by calling list_console_messages.',
@@ -233,14 +221,14 @@ export const commands: Commands = {
         name: 'requestFilePath',
         type: 'string',
         description:
-          'The absolute or relative path to save the request body to. If omitted, the body is returned inline.',
+          'The absolute or relative path to a .network-request file to save the request body to. If omitted, the body is returned inline.',
         required: false,
       },
       responseFilePath: {
         name: 'responseFilePath',
         type: 'string',
         description:
-          'The absolute or relative path to save the response body to. If omitted, the body is returned inline.',
+          'The absolute or relative path to a .network-response file to save the response body to. If omitted, the body is returned inline.',
         required: false,
       },
     },
@@ -324,7 +312,7 @@ export const commands: Commands = {
         name: 'pageSize',
         type: 'integer',
         description:
-          'Maximum number of messages to return. When omitted, returns all requests.',
+          'Maximum number of messages to return. When omitted, returns all messages.',
         required: false,
       },
       pageIdx: {
@@ -388,7 +376,7 @@ export const commands: Commands = {
     },
   },
   list_pages: {
-    description: 'Get a list of pages  open in the browser.',
+    description: 'Get a list of pages open in the browser.',
     category: 'Navigation automation',
     args: {},
   },
@@ -601,8 +589,8 @@ export const commands: Commands = {
   },
   take_memory_snapshot: {
     description:
-      'Capture a memory heapsnapshot of the currently selected page to memory leak debugging',
-    category: 'Performance',
+      'Capture a heap snapshot of the currently selected page. Use to analyze the memory distribution of JavaScript objects and debug memory leaks.',
+    category: 'Memory',
     args: {
       filePath: {
         name: 'filePath',
@@ -637,7 +625,7 @@ export const commands: Commands = {
         name: 'uid',
         type: 'string',
         description:
-          'The uid of an element on the page from the page content snapshot. If omitted takes a pages screenshot.',
+          'The uid of an element on the page from the page content snapshot. If omitted, takes a page screenshot.',
         required: false,
       },
       fullPage: {
@@ -722,24 +710,4 @@ export const commands: Commands = {
       },
     },
   },
-  wait_for: {
-    description: 'Wait for the specified text to appear on the selected page.',
-    category: 'Navigation automation',
-    args: {
-      text: {
-        name: 'text',
-        type: 'array',
-        description:
-          'Non-empty list of texts. Resolves when any value appears on the page.',
-        required: true,
-      },
-      timeout: {
-        name: 'timeout',
-        type: 'integer',
-        description:
-          'Maximum wait time in milliseconds. If set to 0, the default timeout will be used.',
-        required: false,
-      },
-    },
-  },
 } as const;