chromedevtools--chrome-devtools-mcp
Stanislav Publika
caf601a328
## Summary
Adds optional `pageId` routing to page-scoped tools, gated behind
`--experimental-page-id-routing`. When enabled, multi-agent callers can
target a specific page without relying on global selection state. Fully
backward-compatible: without the flag, behavior is unchanged.
### Key changes
- **`pageScoped` annotation**: tools declare `pageScoped: true`; the
server merges `pageId` into their schema at registration time (when the
flag is on)
- **`McpPage` wrapper**: consolidates per-page state (numeric id,
isolated context name, focus tracking) into a single class
- **Request-scoped page routing**: `resolvePageById()` resolves the
target page, `setRequestPage()` threads it through the handler so tools
like `getSelectedPage()` see the right page
- **`assertPageIsFocused`**: keyboard/input tools validate that the
target page holds browser focus, returning an actionable error ("call
select_page first") instead of silently dispatching to the wrong page
- **`--experimental-page-id-routing` CLI flag** (hidden): gates schema
injection and request-scoped routing so the feature can be tested before
graduating
- **Eval scenarios**: `page_id_routing_test` and
`page_focus_keyboard_test` with `serverArgs` support in the eval harness
Addresses #1019
41 行
1.5 KiB
TypeScript
41 行
1.5 KiB
TypeScript
/**
|
|
* @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 = {
|
|
serverArgs: ['--experimental-page-id-routing'],
|
|
prompt: `Open two new pages in isolated contexts:
|
|
- Page A (isolatedContext "contextA") at data:text/html,<button>Click A</button>
|
|
- Page B (isolatedContext "contextB") at data:text/html,<button>Click B</button>
|
|
Then take a snapshot of Page A, take a snapshot of Page B, and then click the button on Page A.`,
|
|
maxTurns: 12,
|
|
expectations: calls => {
|
|
// Should have 2 new_page calls with isolatedContext.
|
|
const newPages = calls.filter(c => c.name === 'new_page');
|
|
assert.strictEqual(newPages.length, 2, 'Should open 2 pages');
|
|
for (const np of newPages) {
|
|
assert.strictEqual(
|
|
typeof np.args.isolatedContext,
|
|
'string',
|
|
'new_page should use isolatedContext',
|
|
);
|
|
}
|
|
|
|
// Should have at least 2 take_snapshot calls (one per page).
|
|
// The model may use pageId directly or select_page before each snapshot.
|
|
const snapshots = calls.filter(c => c.name === 'take_snapshot');
|
|
assert.ok(snapshots.length >= 2, 'Should take at least 2 snapshots');
|
|
|
|
// Should have a click call (resolving uid from Page A's snapshot
|
|
// even though Page B was snapshotted after).
|
|
const clicks = calls.filter(c => c.name === 'click');
|
|
assert.ok(clicks.length >= 1, 'Should click the button on Page A');
|
|
},
|
|
};
|