@ait-co/devtools 0.1.133 → 0.1.134

package/dist/mcp/cli.js

@@ -2024,7 +2024,7 @@ async function readMcpSdkVersion() {
 * some test environments that skip the build step).
 */
 function readDevtoolsVersion() {
-	return "0.1.133";
+	return "0.1.134";
 }
 /**
 * Derives the next recommended action from a completed diagnostics snapshot.
@@ -3513,9 +3513,13 @@ var ChiiCdpConnection = class {
 	/**
 	* Issue a CDP command and resolve with its result (Phase 2). Rejects on a CDP
 	* error frame or when no websocket is open (no page attached yet).
+	*
+	* @param opts.timeoutMs - Per-call override for this connection's command
+	*   watchdog (devtools#747) — see the `CdpConnection.send` docblock for the
+	*   contract callers racing their own longer timeout must follow.
 	*/
-	send(method, params) {
-		return this.sendCommand(method, params ?? {});
+	send(method, params, opts) {
+		return this.sendCommand(method, params ?? {}, opts);
 	}
 	/**
 	* Issue an arbitrary request→response command over the relay and resolve with
@@ -3526,16 +3530,22 @@ var ChiiCdpConnection = class {
 	* auto-reconnect). Caller should re-run `list_pages` or `enableDomains` to
 	* reattach.
 	*
-	* Times out after `commandTimeoutMs` (default 30s, env
-	* `AIT_CDP_COMMAND_TIMEOUT_MS`). On timeout the pending entry is cleaned up
-	* and the promise rejects with a descriptive Korean error.
+	* Times out after `opts.timeoutMs` when given, else `commandTimeoutMs`
+	* (default 30s, env `AIT_CDP_COMMAND_TIMEOUT_MS`) — see devtools#747: the
+	* default 30s watchdog used to undercut the test-runner's own longer
+	* file-evaluate race no matter what `--timeout` the caller asked for. On
+	* timeout the pending entry is cleaned up and the promise rejects with a
+	* descriptive Korean error. `Number.isFinite` guards against a non-finite
+	* override (e.g. `Infinity`, mirroring `waitForFirstTarget`'s convention)
+	* so an intentional "no watchdog" override doesn't get clamped by
+	* `setTimeout`.
 	*/
-	sendCommand(method, params = {}) {
+	sendCommand(method, params = {}, opts) {
 		if (this.connectionState === "disconnected") return Promise.reject(/* @__PURE__ */ new Error(`relay에 연결되어 있지 않습니다 (${method}). list_pages로 attach 상태를 확인하고 enableDomains()로 재연결하세요.`));
 		if (!this.ws || this.ws.readyState !== WebSocket.OPEN) return Promise.reject(/* @__PURE__ */ new Error("No mini-app page attached to the Chii relay yet. Call enableDomains() first."));
 		const id = this.nextCommandId++;
 		const ws = this.ws;
-		const timeoutMs = this.commandTimeoutMs;
+		const timeoutMs = opts?.timeoutMs !== void 0 && Number.isFinite(opts.timeoutMs) && opts.timeoutMs > 0 ? opts.timeoutMs : this.commandTimeoutMs;
 		return new Promise((resolve, reject) => {
 			const handle = setTimeout(() => {
 				this.pending.delete(id);
@@ -4120,6 +4130,14 @@ function parseCaptureLines(raw) {
 /** Maximum milliseconds to wait for a single evaluate round-trip. */
 const DEFAULT_TIMEOUT_MS = 6e4;
 /**
+* Extra headroom (ms) added on top of the rpc-level `timeoutMs` when telling
+* the underlying `CdpConnection` its per-command watchdog budget (devtools#747).
+* The rpc-level race above is the authoritative file timeout — this margin
+* only ensures the connection's own (shorter-by-default) watchdog never fires
+* first and masks the rpc-level timeout error with a connection-level one.
+*/
+const CONNECTION_TIMEOUT_MARGIN_MS = 5e3;
+/**
 * Wraps bundle code in a self-executing IIFE that:
 *   1. Evaluates the bundle (registering describe/it/test).
 *   2. Calls `__testBundle.runTestModule(...)` — the entry the runtime exports.
@@ -4166,12 +4184,15 @@ function parseRunTestsResult(rawValue) {
 * Injects `bundleCode` into the attached page and awaits test execution.
 *
 * Uses `Runtime.evaluate` with `awaitPromise: true` to wait for the
-* async IIFE to settle.  The 30-second CDP command timeout covers even
-* long-running test suites; split into smaller files if you hit it.
+* async IIFE to settle. `timeoutMs` (default 60s) is the authoritative file
+* timeout — this function races it against the evaluate call AND (devtools#747)
+* threads it down to the underlying `CdpConnection`'s own per-command watchdog
+* (plus a small margin) so that watchdog can never undercut this race. Split
+* into smaller files if you hit the timeout.
 *
 * @param connection   - Active CDP connection (relay or local).
 * @param bundleCode   - IIFE bundle string from `bundleTestFile`.
-* @param timeoutMs    - Override the default 30 s timeout.
+* @param timeoutMs    - Override the default 60 s timeout.
 *
 * SECRET-HANDLING: `bundleCode` and the raw CDP result value are never logged.
 */
@@ -4183,7 +4204,7 @@ async function injectAndRunBundle(connection, bundleCode, timeoutMs = DEFAULT_TI
 		expression,
 		returnByValue: true,
 		awaitPromise: true
-	});
+	}, { timeoutMs: timeoutMs + CONNECTION_TIMEOUT_MARGIN_MS });
 	const raceResult = await Promise.race([evalPromise.then((v) => ({
 		tag: "eval",
 		v
@@ -5244,8 +5265,13 @@ var LocalCdpConnection = class {
 	/**
 	* Issue a CDP command and resolve with its typed result. Rejects on a CDP
 	* error frame or when no websocket is open.
+	*
+	* `opts.timeoutMs` is accepted for `CdpConnection` interface parity but
+	* ignored — this connection has no per-command watchdog of its own
+	* (devtools#747 only affects `ChiiCdpConnection`, which layers a
+	* relay-command watchdog on top of the caller's own race).
 	*/
-	send(method, params) {
+	send(method, params, _opts) {
 		return this.sendCommand(method, params ?? {});
 	}
 	/**
@@ -7119,7 +7145,7 @@ function createDebugServer(deps) {
 	};
 	const server = new Server({
 		name: "ait-debug",
-		version: "0.1.133"
+		version: "0.1.134"
 	}, { capabilities: { tools: { listChanged: true } } });
 	server.setRequestHandler(ListToolsRequestSchema, () => {
 		const conn = router.active;
@@ -9103,7 +9129,7 @@ function createDevServer(deps = {}) {
 	const aitSource = deps.aitSource ?? new HttpAitSource({ stateEndpoint });
 	const server = new Server({
 		name: "ait-devtools",
-		version: "0.1.133"
+		version: "0.1.134"
 	}, { capabilities: { tools: {} } });
 	server.setRequestHandler(ListToolsRequestSchema, () => ({ tools: DEV_TOOL_DEFINITIONS.map((tool) => ({ ...tool })) }));
 	server.setRequestHandler(CallToolRequestSchema, async (request) => {