Files
rea/tests/boundary/mcp/browserSourceCoordinates.test.ts
Paritosh LahreandClaude Opus 5.5 8db538a7ca fix(browser): keep negative script columns Chrome reports (#8684)
* fix(browser): keep negative script columns Chrome reports

Chrome reported a Debugger.scriptParsed event with startColumn -60 and
endColumn -15 for an inline script in WordPress's helper frame. CDP declares
these columns as plain integers, but REA's admission required min(0), so one
script failed the whole observe-web-execution run with unreadable_output.

CDP resource columns now share one representation, cdpResourceColumnSchema,
for script start/end columns, initiator call frames, parser initiators and
event listeners. Positions on a script's first line include its start column,
so keeping the reported values keeps the documented script-relative
conversion consistent. The domain contract and web runtime docs state that a
negative column does not locate a column in the resource.

Fixes #8674

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(browser): apply the CDP column policy to passive capture

inspect_web_page and discover_webmcp_tools still read CDP columns through
cdpSourceCoordinates, whose own >= 0 check turned a negative column into
null and reported invalid_protocol_value. WebMCP discovery also dropped the
tool's whole registration_source.

cdpResourceColumnSchema now lives with the domain's CDP observation schemas.
Runtime admission, the passive coordinate helper and the runtime, console,
initiator and WebMCP output schemas all use it. Lines keep their
non-negative rule.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
2026-11-03 14:50:00 +08:00

196 lines
6.0 KiB
TypeScript

import { Ajv2020 } from "ajv/dist/2020.js";
import { expect, it, onTestFinished } from "vitest";
import { CdpBrowserProvider } from "../../../src/browser/CdpBrowserProvider.js";
import { createApplicationMcpHarness } from "../../fixtures/applicationMcpHarness.js";
import { startFakeCdpBrowser } from "../../fixtures/fakeCdpBrowser.js";
/** Report one call frame for a console event, a request initiator and a WebMCP registration. */
const inspectSourceCoordinates = async (position: {
readonly lineNumber: number;
readonly columnNumber: number;
}) => {
const browser = await startFakeCdpBrowser({
webMcpTools: true,
webMcpStackTrace: (origin) => ({
callFrames: [{ url: `${origin}/app.js`, ...position }],
}),
commandEvents(command, origin) {
const frame = { url: `${origin}/app.js`, ...position };
const session =
command.sessionId === undefined ? {} : { sessionId: command.sessionId };
if (command.method === "Runtime.enable")
return [
{
...session,
method: "Runtime.consoleAPICalled",
params: {
type: "log",
timestamp: 1,
args: [],
stackTrace: { callFrames: [frame] },
},
},
];
if (command.method === "Network.enable")
return [
{
...session,
method: "Network.requestWillBeSent",
params: {
requestId: "coordinate-request",
timestamp: 1,
type: "Script",
request: { url: `${origin}/app.js`, method: "GET" },
initiator: { type: "script", stack: { callFrames: [frame] } },
},
},
];
return undefined;
},
});
onTestFinished(() => browser.close());
const { client } = await createApplicationMcpHarness({
clientIdentity: { name: "source-coordinate-test", version: "1" },
serverOptions: {
environment: { ...process.env },
browserObservation: new CdpBrowserProvider(),
availabilityPolicy: () => ({
processCaptureEnabled: false,
investigationInputRoots: 0,
}),
},
});
const args = {
cdp_endpoint: browser.endpoint,
allowed_origins: [browser.allowedOrigin],
target_id: "allowed-page",
observation_ms: 0,
};
const page = await client.callTool({
name: "inspect_web_page",
arguments: args,
});
const tools = await client.callTool({
name: "discover_webmcp_tools",
arguments: args,
});
return { client, page, tools };
};
it("reports malformed source coordinates as unknown through MCP", async () => {
const { page, tools } = await inspectSourceCoordinates({
lineNumber: -3,
columnNumber: 7.5,
});
expect(page.isError).not.toBe(true);
expect(page.structuredContent).toMatchObject({
normalized_result: {
console: {
events: [expect.objectContaining({ line: null, column: null })],
},
network: {
requests: expect.arrayContaining([
expect.objectContaining({
request_id: "coordinate-request",
initiator: expect.objectContaining({ line: null, column: null }),
}),
]),
},
completeness: {
excluded: expect.arrayContaining([
expect.objectContaining({
section: "console_events",
reason: "invalid_protocol_value",
}),
expect.objectContaining({
section: "network_initiators",
reason: "invalid_protocol_value",
}),
]),
},
},
});
expect(tools.isError).not.toBe(true);
expect(tools.structuredContent).toMatchObject({
normalized_result: {
tools: {
items: [expect.objectContaining({ registration_source: null })],
},
completeness: {
excluded: expect.arrayContaining([
expect.objectContaining({
section: "webmcp_tools",
reason: "invalid_protocol_value",
}),
]),
},
},
});
}, 20_000);
it("keeps a negative column Chrome reports in passive source locations", async () => {
const { client, page, tools } = await inspectSourceCoordinates({
lineNumber: 21,
columnNumber: -40,
});
const advertised = (await client.listTools()).tools;
const ajv = new Ajv2020({ strict: false, validateFormats: false });
for (const [name, response] of [
["inspect_web_page", page],
["discover_webmcp_tools", tools],
] as const) {
expect(response.isError).not.toBe(true);
const outputSchema: Record<string, unknown> | undefined = advertised.find(
(tool) => tool.name === name,
)?.outputSchema;
if (outputSchema === undefined)
throw new Error(`Missing advertised output schema: ${name}`);
expect(
ajv.validate(outputSchema, response.structuredContent),
JSON.stringify(ajv.errors),
).toBe(true);
}
const location = { line: 21, column: -40 };
expect(page.structuredContent).toMatchObject({
normalized_result: {
console: { events: [expect.objectContaining(location)] },
network: {
requests: expect.arrayContaining([
expect.objectContaining({
request_id: "coordinate-request",
initiator: expect.objectContaining(location),
}),
]),
},
},
});
expect(tools.structuredContent).toMatchObject({
normalized_result: {
tools: {
items: [
expect.objectContaining({
registration_source: expect.objectContaining(location),
}),
],
},
},
});
for (const [response, section] of [
[page, "console_events"],
[page, "network_initiators"],
[tools, "webmcp_tools"],
] as const)
expect(response.structuredContent).toMatchObject({
normalized_result: {
completeness: {
excluded: expect.not.arrayContaining([
expect.objectContaining({
section,
reason: "invalid_protocol_value",
}),
]),
},
},
});
}, 20_000);