@ait-co/devtools 0.1.131 → 0.1.132

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.131";
+	return "0.1.132";
 }
 /**
 * Derives the next recommended action from a completed diagnostics snapshot.
@@ -3007,6 +3007,23 @@ async function injectDebugIndicator(conn, opts) {
 * safe to import from the MCP daemon graph.
 */
 /**
+* Filename suffix that opts a test file into manual-variant scheduling
+* (devtools#741). A file named `<name>.manual.ait.test.ts` is EXCLUDED from
+* `discoverTestFiles`'s default output (so existing unattended runs are
+* unaffected — the zero-diff-when-off constraint) and is only surfaced when
+* the caller explicitly asks for manual files via {@link partitionManualTests}
+* (wired from the CLI's `--manual-blocking` flag).
+*
+* This is the entire tagging contract for v1 — no separate manifest/config,
+* just a filename convention. Documented here + in the CLI `--help` text
+* (cli.ts USAGE) and the test-runner README.
+*/
+const MANUAL_TEST_SUFFIX = ".manual.ait.test.ts";
+/** True when `file`'s basename ends with {@link MANUAL_TEST_SUFFIX}. */
+function isManualTestFile(file) {
+	return basename(file).endsWith(MANUAL_TEST_SUFFIX);
+}
+/**
 * Expands `patterns` (globs or plain paths) into a sorted, de-duplicated list of
 * ABSOLUTE test file paths, resolved relative to `cwd`.
 *
@@ -3015,14 +3032,26 @@ async function injectDebugIndicator(conn, opts) {
 * resolved against `cwd`. `bundleTestFile` requires an absolute path, so the
 * absolute output feeds it directly.
 *
+* By default, files matching {@link MANUAL_TEST_SUFFIX} are EXCLUDED from the
+* result (devtools#741) — blocking-UI tests opt in via that filename
+* convention and must never appear in an unattended run unless the caller
+* explicitly asks for them via `includeManual: true`. This keeps the
+* default (flag-off) discovery path byte-for-byte identical to before this
+* option existed.
+*
 * @param patterns Glob patterns or file paths (e.g. `['src/**\/*.ait.test.ts']`).
 * @param cwd      Base directory for relative patterns/results.
+* @param opts     `{ includeManual }` — when true, manual-tagged files are kept
+*                 in the (still-sorted) output instead of being filtered out.
+*                 Use {@link partitionManualTests} to separate + reorder them.
 * @returns Sorted, de-duplicated absolute file paths. Empty when nothing matches.
 */
-async function discoverTestFiles(patterns, cwd) {
+async function discoverTestFiles(patterns, cwd, opts) {
 	const out = /* @__PURE__ */ new Set();
 	for await (const match of glob(patterns, { cwd })) out.add(isAbsolute(match) ? match : resolve(cwd, match));
-	return [...out].sort();
+	const sorted = [...out].sort();
+	if (opts?.includeManual) return sorted;
+	return sorted.filter((f) => !isManualTestFile(f));
 }
 //#endregion
 //#region src/shared/relay-auth-close.ts
@@ -4069,6 +4098,13 @@ async function injectAndRunBundle(connection, bundleCode, timeoutMs = DEFAULT_TI
 //#endregion
 //#region src/test-runner/relay-worker.ts
 /**
+* Per-file evaluate timeout (ms) for manual-variant files (devtools#741) — a
+* human is expected to be tapping through a native sheet, so the timeout must
+* be far longer than the unattended default (60s, rpc.ts DEFAULT_TIMEOUT_MS).
+* Fixed constant (not configurable in v1) — documented here + CLI `--help`.
+*/
+const MANUAL_FILE_TIMEOUT_MS = 5 * 6e4;
+/**
 * Sentinel string embedded in the error message by `injectAndRunBundle` when
 * the per-file evaluate race hits the timeout. Used by the retry guard so only
 * genuine timeouts get a second attempt — not bundle errors or parse failures.
@@ -4139,12 +4175,21 @@ async function runTestFilesOverRelay(connection, files, opts) {
 			process.stderr.write(`relay-worker: reconnect failed (${e instanceof Error ? e.message : String(e)}) — continuing, remaining files may fail\n`);
 		}
 	};
+	const manualFiles = opts?.manualFiles;
+	const manualTotal = manualFiles?.size ?? 0;
+	let manualIndex = 0;
 	try {
 		for (const file of files) {
 			if (pendingReconnectCheck) {
 				await attemptReconnect("next-file");
 				pendingReconnectCheck = false;
 			}
+			const isManual = manualFiles?.has(file) === true;
+			if (isManual) {
+				manualIndex += 1;
+				opts?.onManualFile?.(file, manualIndex, manualTotal);
+			}
+			const fileTimeoutMs = isManual ? MANUAL_FILE_TIMEOUT_MS : opts?.timeoutMs;
 			let fileEntry;
 			try {
 				const { code } = await bundleTestFile(file, opts?.bundleOptions);
@@ -4170,7 +4215,7 @@ async function runTestFilesOverRelay(connection, files, opts) {
 				const attempt = async () => {
 					let rpcResult;
 					try {
-						rpcResult = await injectAndRunBundle(connection, code, opts?.timeoutMs);
+						rpcResult = await injectAndRunBundle(connection, code, fileTimeoutMs);
 					} catch (e) {
 						return {
 							file,
@@ -4206,6 +4251,7 @@ async function runTestFilesOverRelay(connection, files, opts) {
 				};
 			}
 			if ("error" in fileEntry.result && isRelayDisconnectMessage(fileEntry.result.error)) pendingReconnectCheck = true;
+			if (isManual) fileEntry.mode = "manual";
 			fileResults.push(fileEntry);
 		}
 	} finally {
@@ -4293,6 +4339,24 @@ OPTIONS
   --headless              Disable browser auto-open (text QR only)
   --project-root <dir>    Project root for .ait_relay secret lookup
                           (default: current working directory)
+  --manual-blocking       Run manual-tagged test files (*.manual.ait.test.ts)
+                          LAST, after all regular files, with a human present.
+                          Before each manual file, the QR dashboard is pushed
+                          a step-by-step Korean prompt naming the file + its
+                          progress (k/n), and the same line is printed to
+                          stdout. Manual files get a 5-minute per-file evaluate
+                          timeout (vs. --timeout for everything else) since a
+                          human is expected to tap through a native sheet
+                          (photo picker, permission dialog, fullscreen ad).
+                          Without this flag (default off), *.manual.ait.test.ts
+                          files are EXCLUDED from the glob expansion entirely —
+                          existing unattended runs are byte-for-byte unaffected.
+                          With --report-dir, a run that included manual files
+                          ALSO writes <sdkLine>.<platform>.manual.json
+                          alongside (never replacing) the standard report, and
+                          each manual file's report entry is stamped
+                          mode: 'manual' — never diff a manual run against an
+                          unattended baseline as if they were equivalent.
   --help, -h              Show this help message
 
 DESCRIPTION
@@ -6638,7 +6702,8 @@ async function startQrHttpServer(getDashboardState, options) {
 			pages: state.pages,
 			attachUrl: state.attachUrl,
 			inspectorUrl: state.inspectorUrl ?? null,
-			phase: state.phase ?? "active"
+			phase: state.phase ?? "active",
+			manualPrompt: state.manualPrompt ?? null
 		});
 		res.write(`data: ${payload}\n\n`);
 	}
@@ -6947,7 +7012,7 @@ function createDebugServer(deps) {
 	};
 	const server = new Server({
 		name: "ait-debug",
-		version: "0.1.131"
+		version: "0.1.132"
 	}, { capabilities: { tools: { listChanged: true } } });
 	server.setRequestHandler(ListToolsRequestSchema, () => {
 		const conn = router.active;
@@ -8931,7 +8996,7 @@ function createDevServer(deps = {}) {
 	const aitSource = deps.aitSource ?? new HttpAitSource({ stateEndpoint });
 	const server = new Server({
 		name: "ait-devtools",
-		version: "0.1.131"
+		version: "0.1.132"
 	}, { capabilities: { tools: {} } });
 	server.setRequestHandler(ListToolsRequestSchema, () => ({ tools: DEV_TOOL_DEFINITIONS.map((tool) => ({ ...tool })) }));
 	server.setRequestHandler(CallToolRequestSchema, async (request) => {