@ait-co/devtools 0.1.127 → 0.1.128

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.127";
+	return "0.1.128";
 }
 /**
 * Derives the next recommended action from a completed diagnostics snapshot.
@@ -3395,7 +3395,7 @@ function parseCaptureLines(raw) {
 //#endregion
 //#region src/test-runner/rpc.ts
 /** Maximum milliseconds to wait for a single evaluate round-trip. */
-const DEFAULT_TIMEOUT_MS = 3e4;
+const DEFAULT_TIMEOUT_MS = 6e4;
 /**
 * Wraps bundle code in a self-executing IIFE that:
 *   1. Evaluates the bundle (registering describe/it/test).
@@ -3454,13 +3454,22 @@ function parseRunTestsResult(rawValue) {
 */
 async function injectAndRunBundle(connection, bundleCode, timeoutMs = DEFAULT_TIMEOUT_MS) {
 	const expression = buildRunTestsExpression(bundleCode);
-	const timeoutPromise = new Promise((_, reject) => setTimeout(() => reject(/* @__PURE__ */ new Error(`rpc: evaluate timed out after ${timeoutMs}ms`)), timeoutMs));
+	const TIMEOUT_SENTINEL = Symbol("timeout");
+	const timeoutPromise = new Promise((resolve) => setTimeout(() => resolve(TIMEOUT_SENTINEL), timeoutMs));
 	const evalPromise = connection.send("Runtime.evaluate", {
 		expression,
 		returnByValue: true,
 		awaitPromise: true
 	});
-	const cdpResult = await Promise.race([evalPromise, timeoutPromise]);
+	const raceResult = await Promise.race([evalPromise.then((v) => ({
+		tag: "eval",
+		v
+	})), timeoutPromise.then(() => ({ tag: "timeout" }))]);
+	if (raceResult.tag === "timeout") return {
+		ok: false,
+		error: `rpc: evaluate timed out after ${timeoutMs}ms`
+	};
+	const cdpResult = raceResult.v;
 	if (cdpResult.exceptionDetails) {
 		const msg = cdpResult.exceptionDetails.exception?.description ?? cdpResult.exceptionDetails.text ?? "Runtime.evaluate threw an exception";
 		throw new Error(`rpc.injectAndRunBundle: ${msg}`);
@@ -3525,12 +3534,16 @@ async function runTestFilesOverRelay(connection, files, opts) {
 				* result is a genuine timeout and the caller should retry.
 				*
 				* We need to distinguish:
-				*   - `rpcResult.ok = false` + timeout error → retry candidate
+				*   - `rpcResult.ok = false` + timeout error → retry candidate (`return null`)
 				*   - `rpcResult.ok = false` + other error   → final error, no retry
-				*   - `injectAndRunBundle` throws             → treated like a final error
-				*     (throws happen on CDP exceptionDetails, not on the Promise.race
-				*     timeout — the timeout produces `rpcResult.ok=false` with the
-				*     EVALUATE_TIMEOUT_MARKER message)
+				*   - `injectAndRunBundle` throws             → CDP exceptionDetails (page
+				*     engine threw); treated as a final (non-retryable) error.
+				*
+				* The Promise.race timeout in rpc.ts RETURNS `{ok:false, error: '…'}` (it
+				* does NOT throw/reject).  Only genuine CDP `exceptionDetails` cause a throw.
+				* This distinction is what makes the EVALUATE_TIMEOUT_MARKER gate below
+				* reachable — the timeout result surfaces as `rpcResult.ok=false` with the
+				* marker string, not as a caught exception.
 				*/
 				const attempt = async () => {
 					let rpcResult;
@@ -6785,7 +6798,7 @@ function createDebugServer(deps) {
 	};
 	const server = new Server({
 		name: "ait-debug",
-		version: "0.1.127"
+		version: "0.1.128"
 	}, { capabilities: { tools: { listChanged: true } } });
 	server.setRequestHandler(ListToolsRequestSchema, () => {
 		const conn = router.active;
@@ -8757,7 +8770,7 @@ function createDevServer(deps = {}) {
 	const aitSource = deps.aitSource ?? new HttpAitSource({ stateEndpoint });
 	const server = new Server({
 		name: "ait-devtools",
-		version: "0.1.127"
+		version: "0.1.128"
 	}, { capabilities: { tools: {} } });
 	server.setRequestHandler(ListToolsRequestSchema, () => ({ tools: DEV_TOOL_DEFINITIONS.map((tool) => ({ ...tool })) }));
 	server.setRequestHandler(CallToolRequestSchema, async (request) => {