@ait-co/devtools 0.1.100 → 0.1.101

package/dist/mcp/cli.js

@@ -4858,7 +4858,7 @@ async function readMcpSdkVersion() {
 * some test environments that skip the build step).
 */
 function readDevtoolsVersion() {
-	return "0.1.100";
+	return "0.1.101";
 }
 /**
 * Derives the next recommended action from a completed diagnostics snapshot.
@@ -5313,6 +5313,52 @@ function makeTunnelStatus(up, wssUrl, droppedAt = null, reissueAttempts = 0) {
 * Node-only.
 */
 /**
+* Maximum age (ms) of a page's `lastSeenAt` before it is treated as a ghost
+* and excluded from the `wait_for_attach` short-circuit in `build_attach_url`
+* (issue #610).
+*
+* Rationale: the env-2 relay is owned by the dev server (unplugin), so every
+* `dev:phone:cdp` restart produces a new quick-tunnel. The old relay goes
+* offline immediately, but the daemon's warm `ChiiCdpConnection` still lists
+* the last-seen target — its `lastSeenAt` freezes at the moment the old relay
+* died. A 5-minute threshold is large enough to be invisible in normal usage
+* (active CDP sessions see a message every few seconds) while being small
+* enough to catch a relay that went down before the daemon was re-entered.
+*
+* Injectable for tests via {@link DebugServerDeps.stalePageThresholdMs}.
+*/
+const RELAY_SANDBOX_STALE_PAGE_MS = 300 * 1e3;
+/**
+* Predicate used by `build_attach_url`'s `wait_for_attach` loop to decide
+* whether the relay-sandbox connection has a genuinely fresh page attached.
+*
+* Stale-ghost gating (issue #610): when the dev server restarts with a new
+* quick-tunnel, the warm `ChiiCdpConnection` still lists the last-seen target
+* but its `lastSeenAt` is frozen. A page whose `lastSeenAt` exceeds
+* `stalePageThresholdMs` is a ghost from the dead relay — it must NOT
+* short-circuit `wait_for_attach`.
+*
+* Rules:
+* - `pages.length === 0` → false (nothing attached).
+* - Connection has no `getLastSeenAt` (test fakes, local-browser) → falls back
+*   to `pages.length > 0` (regression-safe).
+* - `seenMs === null` → treat as fresh (no CDP message received yet, first
+*   message pending — the connection is alive).
+* - Otherwise: at least one page must satisfy `nowMs - seenMs <=
+*   stalePageThresholdMs`.
+*
+* Exported for unit testing.
+*/
+function isSandboxPageFresh(pages, getLastSeenAt, nowMs, stalePageThresholdMs) {
+	if (pages.length === 0) return false;
+	if (getLastSeenAt === null) return true;
+	return pages.some((p) => {
+		const seenMs = getLastSeenAt(p.id);
+		if (seenMs === null) return true;
+		return nowMs - seenMs <= stalePageThresholdMs;
+	});
+}
+/**
 * Parses `_deploymentId` from the query string of a scheme URL.
 *
 * Returns `null` when the param is absent or empty — callers treat that as
@@ -5395,7 +5441,7 @@ function waitForAttachWithEvents(connection, filterFn, timeoutMs, pollIntervalMs
 * naturally via `enableDomains`). The tier only controls visibility.
 */
 function createDebugServer(deps) {
-	const { connection, router: routerDep, aitSource, getTunnelStatus, waitForAttachTimeoutMs = 6e4, qrHttpServer, getEnvironment: getEnvDep, getEnvironmentReason: getEnvReasonDep, diagnosticsCollector: collectorDep, totpSecret, onAttachUrlBuilt, getTunnelChildPid, readLock: readLockDep } = deps;
+	const { connection, router: routerDep, aitSource, getTunnelStatus, waitForAttachTimeoutMs = 6e4, qrHttpServer, getEnvironment: getEnvDep, getEnvironmentReason: getEnvReasonDep, diagnosticsCollector: collectorDep, totpSecret, onAttachUrlBuilt, getTunnelChildPid, readLock: readLockDep, stalePageThresholdMs = RELAY_SANDBOX_STALE_PAGE_MS, nowMs = () => Date.now() } = deps;
 	const getTotpSecret = deps.getTotpSecret ?? (() => totpSecret);
 	const readLockFn = readLockDep ?? readServerLock;
 	const router = routerDep ?? makeSingleConnectionRouter(connection);
@@ -5404,7 +5450,7 @@ function createDebugServer(deps) {
 	const collector = collectorDep ?? new InMemoryDiagnosticsCollector();
 	const server = new Server({
 		name: "ait-debug",
-		version: "0.1.100"
+		version: "0.1.101"
 	}, { capabilities: { tools: { listChanged: true } } });
 	server.setRequestHandler(ListToolsRequestSchema, () => {
 		const conn = router.active;
@@ -5533,7 +5579,10 @@ function createDebugServer(deps) {
 				});
 				const relayUrl = tunnelStatus.wssUrl;
 				const totp = totpMeta;
-				const isMatchingPage = (pages) => pages.length > 0;
+				const connAsAny = conn;
+				const getLastSeenAt = typeof connAsAny.getTargetLastSeenAt === "function" ? (id) => connAsAny.getTargetLastSeenAt(id) : null;
+				const callNow = nowMs();
+				const isMatchingPage = (pages) => isSandboxPageFresh(pages, getLastSeenAt, callNow, stalePageThresholdMs);
 				const buildTimeoutError = (baseText, timeoutSec, observed) => {
 					const observedUrls = observed.slice(0, 3).map((p) => p.url.slice(0, 80)).join(", ");
 					return `${baseText}\n\nNo page attached within ${timeoutSec}s${observed.length > 0 ? ` — previously attached pages: [${observedUrls}]` : ""} — launcher QR을 폰 카메라로 스캔한 뒤 call list_pages를 다시 호출하세요.`;
@@ -6459,10 +6508,39 @@ var DualConnectionRouter = class {
 	* `projectRoot` is forwarded to `bootLazyFor` so `relay-sandbox` boot can
 	* fall back to `.ait_urls` file discovery (#424) when `AIT_RELAY_BASE_URL` is
 	* not set in the environment.
+	*
+	* **Relay-sandbox stale-URL rebuild (issue #610):** when the `relay-sandbox`
+	* family is already warm, reads the current relay URL via
+	* `deps.readSandboxRelayUrl` and compares it against the cached
+	* `relayHttpUrl`. If they differ (dev server was restarted → new tunnel),
+	* the stale family is torn down, evicted from the map, and a fresh one is
+	* booted. If they match, or if the URL cannot be read, the warm family is
+	* reused (fail-open — no unnecessary teardown on transient read errors).
+	*
+	* SECRET-HANDLING: fresh and cached relay URLs carry the tunnel host. The
+	* comparison result (same/different) is the only thing surfaced — URLs are
+	* never logged.
 	*/
 	async familyFor(key, projectRoot) {
 		const warm = this.lazyFamilies.get(key);
-		if (warm) return warm;
+		if (warm) {
+			if (key === "relay-sandbox" && this.deps.readSandboxRelayUrl !== void 0) {
+				let freshUrl = null;
+				try {
+					freshUrl = await this.deps.readSandboxRelayUrl(projectRoot);
+				} catch {
+					freshUrl = null;
+				}
+				if (freshUrl !== null && freshUrl !== warm.relayHttpUrl) {
+					warm.stop();
+					this.lazyFamilies.delete(key);
+					const booted = await this.deps.bootLazyFor(key, projectRoot);
+					this.lazyFamilies.set(key, booted);
+					return booted;
+				}
+			}
+			return warm;
+		}
 		const booted = await this.deps.bootLazyFor(key, projectRoot);
 		this.lazyFamilies.set(key, booted);
 		return booted;
@@ -6522,7 +6600,8 @@ async function runDebugServer(options = {}) {
 		diagnosticsCollector,
 		devtoolsOpener,
 		onPageAttach: () => qrServer?.notifyStateChange(),
-		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null
+		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null,
+		readSandboxRelayUrl: (pr) => readMobileRelayBaseUrl(process.env, pr).catch(() => null)
 	});
 	const aitSource = new RoutingAitSource(() => {
 		return router.active;
@@ -6734,7 +6813,8 @@ async function runLocalDebugServer(options = {}) {
 		diagnosticsCollector,
 		devtoolsOpener,
 		onPageAttach: () => qrServer?.notifyStateChange(),
-		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null
+		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null,
+		readSandboxRelayUrl: (pr) => readMobileRelayBaseUrl(process.env, pr).catch(() => null)
 	});
 	const aitSource = new RoutingAitSource(() => {
 		return router.active;
@@ -6929,7 +7009,8 @@ async function runMobileDebugServer(options = {}) {
 		diagnosticsCollector,
 		devtoolsOpener,
 		onPageAttach: () => qrServer?.notifyStateChange(),
-		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null
+		getInspectorStableUrl: () => qrServer?.inspectorStableUrl ?? null,
+		readSandboxRelayUrl: (pr) => readMobileRelayBaseUrl(process.env, pr ?? options.projectRoot ?? process.cwd()).catch(() => null)
 	});
 	const aitSource = new RoutingAitSource(() => {
 		return router.active;
@@ -7503,7 +7584,7 @@ function createDevServer(deps = {}) {
 	const aitSource = deps.aitSource ?? new HttpAitSource({ stateEndpoint });
 	const server = new Server({
 		name: "ait-devtools",
-		version: "0.1.100"
+		version: "0.1.101"
 	}, { capabilities: { tools: {} } });
 	server.setRequestHandler(ListToolsRequestSchema, () => ({ tools: DEV_TOOL_DEFINITIONS.map((tool) => ({ ...tool })) }));
 	server.setRequestHandler(CallToolRequestSchema, async (request) => {