pi-agent-browser-native 0.3.0 → 0.6.5

package/dist/extensions/agent-browser/lib/orchestration/browser-run/process-output.js

@@ -1,22 +1,24 @@
 import { rm } from "node:fs/promises";
-import { getAgentBrowserSessionIdentityKey } from "../../argv-grammar.js";
-import { isCloseCommand, isNavigationObservableCommandName, isOpenNavigationCommand } from "../../command-taxonomy.js";
+import { getAgentBrowserSessionIdentityKey, isAgentBrowserSessionIdentityKeyInNamespace } from "../../argv-grammar.js";
+import { batchHasSuccessfulCloseAll, getSuccessfulBatchCloseLifecycle } from "../../batch-lifecycle.js";
+import { isCloseAllCommand, isCloseCommand, isOpenNavigationCommand, isUnverifiedPageTransitionCommand } from "../../command-taxonomy.js";
 import { OPEN_RESULT_TAB_CORRECTION_FLAGS } from "../../launch-scoped-flags.js";
 import { cleanupElectronLaunchResources, inspectElectronLaunchStatus } from "../../electron/cleanup.js";
-import { getAllowedDomainsViolation, parseAllowedDomainsPolicyFromArgs } from "../../navigation-policy.js";
-import { getManagedSessionResultingPageState, getObservedBrowserPageValidationError, managedSessionCommandRequiresLivePageVerification } from "../../managed-session-state-policy.js";
+import { getResultingPageTargetState, commandRequiresLivePageVerification } from "../../page-target-validation.js";
 import { analyzeNetworkSourceLookupResults, analyzeSourceLookupResults, redactNetworkSourceLookupAnalysis } from "../../input-modes/lookups.js";
-import { analyzeQaPresetResults, analyzeQaPresetTimeout, buildQaCompactPassText, extractQaPageContext } from "../../input-modes/job.js";
+import { analyzeQaPresetResults, analyzeQaPresetTimeout, buildQaCompactFailureText, buildQaCompactPassText, extractQaPageContext } from "../../input-modes/job.js";
 import { applyNetworkRouteRecords, buildNetworkRouteDiagnostics } from "../../results/network-routes.js";
 import { buildToolPresentation } from "../../results/presentation.js";
 import { getAgentBrowserErrorText, parseAgentBrowserEnvelope } from "../../results/envelope.js";
+import { omitUpstreamLifecycle } from "../../results/presentation/common.js";
 import { getClipboardWritePayloadCandidates, redactClipboardPermissionEcho, redactClipboardPermissionErrorValue } from "../../results/presentation/errors.js";
 import { shouldCaptureSemanticActionNavigationSummary } from "../../results/presentation/semantic-action.js";
-import { commandExplicitlyTargetsAboutBlank, deriveSessionTabTarget, extractLatestRefSnapshotStateFromBatchResults, extractRefSnapshotFromData, extractSessionTabTargetFromBatchResults, extractSessionTabTargetFromCommandData, isAboutBlankSessionTabTarget, normalizeSessionTabTarget, } from "../../session-page-state.js";
+import { buildPageTransitionRefSnapshotInvalidation, commandExplicitlyTargetsAboutBlank, getCommandRefSnapshotInvalidation, deriveSessionTabTarget, extractLatestRefSnapshotStateFromBatchResults, extractRefSnapshotFromData, extractSessionTabTargetFromBatchResults, extractSessionTabTargetFromCommandData, isAboutBlankSessionTabTarget, normalizeSessionTabTarget, } from "../../session-page-state.js";
 import { isRecord } from "../../parsing.js";
 import { pruneOwnedManagedSessionRestoreSnapshots } from "../../managed-session-restore.js";
 import { isManagedSessionRestoreKey } from "../../managed-session-storage.js";
-import { createFreshSessionName, extractCommandTokens, resolveManagedSessionState } from "../../runtime.js";
+import { createFreshSessionName, extractUpstreamCommandTokens, resolveManagedSessionState } from "../../runtime.js";
+import { getUpstreamEffectiveBatchSteps } from "../batch-stdin.js";
 import { closeManagedSession, inspectManagedSessionDaemon } from "./managed-session-daemon-policy.js";
 import { applyOpenResultTabCorrection, buildAboutBlankRecoveryHint, buildAboutBlankWarning, buildElectronPostCommandHealthDiagnostic, buildElectronRefFreshnessDiagnostic, buildElectronSessionMismatch, buildManagedSessionOutcome, collectOpenResultTabCorrection, collectSessionTabSelection, extractNavigationSummaryFromData, extractStringResultField, findElectronLaunchRecordForSession, formatElectronPostCommandHealthText, formatElectronSessionMismatchText, getSessionContextKey, getStaleRefArgs, mergeNavigationSummaryIntoData, shouldCaptureNavigationSummary, shouldCorrectSessionTabAfterCommand, shouldInspectElectronPostCommandHealth, unwrapPinnedSessionBatchEnvelope, updateTraceOwnerState, } from "./session-state.js";
 import { collectClickDispatchDiagnostic } from "./click-dispatch.js";
@@ -61,6 +63,35 @@ function isStreamEnableAlreadyEnabledNoop(options) {
     const message = (getEnvelopeErrorString(options.envelope) ?? "").trim().replace(/[.!]+$/, "").toLowerCase();
     return message === "streaming is already enabled for this session" || message === "streaming is already enabled" || message === "stream already enabled";
 }
+function isPendingWebMcpMutation(command, subcommand, data) {
+    return command === "webmcp"
+        && ["invoke", "result"].includes(subcommand ?? "")
+        && isRecord(data)
+        && data.status === "pending";
+}
+function isWebMcpSettlementCommand(command, subcommand) {
+    return command === "webmcp" && ["result", "cancel"].includes(subcommand ?? "");
+}
+function batchHasPendingWebMcpMutation(data) {
+    if (!Array.isArray(data))
+        return false;
+    return data.some((row) => {
+        if (!isRecord(row) || row.success === false || !Array.isArray(row.command) || !row.command.every((token) => typeof token === "string"))
+            return false;
+        const [command, subcommand] = extractUpstreamCommandTokens(row.command);
+        return isPendingWebMcpMutation(command, subcommand, row.result);
+    });
+}
+function batchHasFailedWebMcpSettlement(data) {
+    if (!Array.isArray(data))
+        return false;
+    return data.some((row) => {
+        if (!isRecord(row) || row.success !== false || !Array.isArray(row.command) || !row.command.every((token) => typeof token === "string"))
+            return false;
+        const [command, subcommand] = extractUpstreamCommandTokens(row.command);
+        return isWebMcpSettlementCommand(command, subcommand);
+    });
+}
 function batchStartedManagedBrowser(data) {
     if (!Array.isArray(data))
         return false;
@@ -71,6 +102,15 @@ function batchStartedManagedBrowser(data) {
         return command === "connect" || command === "goto" || command === "navigate" || isOpenNavigationCommand(command);
     });
 }
+function withoutNamespaceEntries(entries, namespace) {
+    return new Map([...entries].filter(([key]) => !isAgentBrowserSessionIdentityKeyInNamespace(key, namespace)));
+}
+function deleteNamespaceEntries(entries, namespace) {
+    for (const key of entries.keys()) {
+        if (isAgentBrowserSessionIdentityKeyInNamespace(key, namespace))
+            entries.delete(key);
+    }
+}
 function setNetworkRouteState(options) {
     if (!options.sessionName)
         return options.routesBySession;
@@ -95,7 +135,12 @@ function applyBatchNetworkRouteState(options) {
     for (const item of options.data) {
         if (!isRecord(item) || !Array.isArray(item.command) || !item.command.every((token) => typeof token === "string"))
             continue;
-        routes = applyNetworkRouteRecords(routes, extractCommandTokens(item.command), item.success !== false);
+        const commandTokens = extractUpstreamCommandTokens(item.command);
+        const stepSucceeded = item.success !== false;
+        if (stepSucceeded && isCloseCommand(commandTokens[0]))
+            routes = undefined;
+        else
+            routes = applyNetworkRouteRecords(routes, commandTokens, stepSucceeded);
     }
     return setNetworkRouteState({ routes, routesBySession: options.routesBySession, sessionName: options.sessionName });
 }
@@ -103,7 +148,6 @@ export async function processBrowserOutput(input) {
     const { ctx, cwd, electronPostCommandStatusSettleMs, implicitSessionCloseTimeoutMs, sessionPageStateUpdate, signal, state } = input;
     const { prepared, processResult } = input;
     const { electronChildProcesses, electronLaunchRecords, sessionPageState, traceOwners } = state;
-    let allowedDomainsBySession = state.allowedDomainsBySession;
     let artifactManifest = state.artifactManifest;
     let freshSessionOrdinal = state.freshSessionOrdinal;
     let managedSessionActive = state.managedSessionActive;
@@ -120,6 +164,7 @@ export async function processBrowserOutput(input) {
         let parseError = parsed.parseError;
         let presentationEnvelope = parsed.envelope;
         let navigationSummary = undefined;
+        let failedTransitionReverification = false;
         if (prepared.pinnedBatchUnwrapMode) {
             const pinnedBatchResult = unwrapPinnedSessionBatchEnvelope({ envelope: parsed.envelope, includeNavigationSummary: prepared.includePinnedNavigationSummary, mode: prepared.pinnedBatchUnwrapMode });
             parseError = pinnedBatchResult.parseError ?? parseError;
@@ -132,11 +177,22 @@ export async function processBrowserOutput(input) {
         presentationEnvelope = repairedBatchScreenshots.envelope;
         const screenshotArtifactRequest = repairedScreenshot.request;
         const batchScreenshotArtifactRequests = repairedBatchScreenshots.requests;
+        const batchCommandSteps = prepared.executionPlan.commandInfo.command === "batch"
+            ? getUpstreamEffectiveBatchSteps(prepared.commandTokens, prepared.runtimeToolStdin)
+            : [];
+        const nestedBatchClose = prepared.executionPlan.commandInfo.command === "batch"
+            ? getSuccessfulBatchCloseLifecycle(presentationEnvelope?.data, batchCommandSteps)
+            : undefined;
+        const nestedBatchClosed = nestedBatchClose?.endsClosed === true;
+        const nestedBatchRemainsActive = nestedBatchClose?.endsClosed === false;
+        const nestedBatchClosesAll = prepared.executionPlan.commandInfo.command === "batch"
+            && batchHasSuccessfulCloseAll(presentationEnvelope?.data, batchCommandSteps);
+        const directCloseAllRequested = isCloseAllCommand(extractUpstreamCommandTokens(prepared.commandTokens));
         const rawCloseStatePath = isCloseCommand(prepared.executionPlan.commandInfo.command)
             && isRecord(presentationEnvelope?.data)
             && typeof presentationEnvelope.data.statePath === "string"
             ? presentationEnvelope.data.statePath
-            : undefined;
+            : nestedBatchClose?.statePath;
         if (presentationEnvelope && prepared.exactSensitiveValues.length > 0)
             presentationEnvelope = redactExactSensitiveValue(presentationEnvelope, prepared.exactSensitiveValues);
         const parseFailureOutput = parseError && processResult.stdoutSpillPath
@@ -152,7 +208,41 @@ export async function processBrowserOutput(input) {
         let succeeded = processSucceeded && parseSucceeded && envelopeSuccess;
         const inspectionText = plainTextInspection ? processResult.stdout.trim() : undefined;
         const sessionStateKey = getSessionContextKey(prepared.executionPlan.sessionName, prepared.executionPlan.namespace);
-        updateTraceOwnerState({ command: prepared.executionPlan.commandInfo.command, sessionName: sessionStateKey, subcommand: prepared.executionPlan.commandInfo.subcommand, succeeded, traceOwners });
+        const closeAllApplied = nestedBatchClosesAll || (directCloseAllRequested && succeeded);
+        if (closeAllApplied) {
+            networkRoutesBySession = withoutNamespaceEntries(networkRoutesBySession, prepared.executionPlan.namespace);
+            deleteNamespaceEntries(state.attachedSessionKeys, prepared.executionPlan.namespace);
+            deleteNamespaceEntries(traceOwners, prepared.executionPlan.namespace);
+            sessionPageState.clearNamespace(prepared.executionPlan.namespace);
+            const retainedSessionKey = nestedBatchRemainsActive ? sessionStateKey : undefined;
+            for (const [key, owner] of state.ownedManagedSessions) {
+                if (!isAgentBrowserSessionIdentityKeyInNamespace(key, prepared.executionPlan.namespace) || key === retainedSessionKey)
+                    continue;
+                state.closedManagedSessionNames.add(key);
+                state.managedSessionRestoreState.clear(owner.sessionName, owner.namespace);
+            }
+        }
+        else if (nestedBatchClose && sessionStateKey) {
+            networkRoutesBySession = new Map(networkRoutesBySession);
+            networkRoutesBySession.delete(sessionStateKey);
+            sessionPageState.clearSession(sessionStateKey);
+        }
+        if (prepared.executionPlan.commandInfo.command === "batch" && Array.isArray(presentationEnvelope?.data)) {
+            for (const [index, row] of presentationEnvelope.data.entries()) {
+                if (!isRecord(row))
+                    continue;
+                const rowCommand = Array.isArray(row.command) && row.command.every((token) => typeof token === "string")
+                    ? row.command
+                    : batchCommandSteps[index];
+                if (!rowCommand)
+                    continue;
+                const [command, subcommand] = extractUpstreamCommandTokens(rowCommand);
+                updateTraceOwnerState({ command, sessionName: sessionStateKey, subcommand, succeeded: row.success === true, traceOwners });
+            }
+        }
+        else {
+            updateTraceOwnerState({ command: prepared.executionPlan.commandInfo.command, sessionName: sessionStateKey, subcommand: prepared.executionPlan.commandInfo.subcommand, succeeded, traceOwners });
+        }
         let clickDispatchDiagnostic;
         if (succeeded && prepared.clickDispatchProbe) {
             clickDispatchDiagnostic = await collectClickDispatchDiagnostic({ cwd, namespace: prepared.executionPlan.namespace, probe: prepared.clickDispatchProbe, sessionName: prepared.executionPlan.sessionName, signal });
@@ -161,29 +251,38 @@ export async function processBrowserOutput(input) {
                 presentationEnvelope = { ...(presentationEnvelope ?? {}), error: clickDispatchDiagnostic.summary, success: false };
             }
         }
-        const resultingPageState = getManagedSessionResultingPageState({
+        const tabTransition = prepared.executionPlan.commandInfo.command === "tab" && prepared.executionPlan.commandInfo.subcommand !== undefined && !["list", "new"].includes(prepared.executionPlan.commandInfo.subcommand);
+        const resultingPageState = getResultingPageTargetState({
             args: prepared.executionPlan.effectiveArgs,
             currentPageUrl: prepared.priorSessionTabTarget?.url,
             pageUrlUnknown: prepared.priorSessionTabTargetUnknown === true,
             stdin: prepared.runtimeToolStdin,
             trustedFirstBatchTabSelection: prepared.pinnedBatchUnwrapMode !== undefined,
         });
-        const presentationDataRecord = isRecord(presentationEnvelope?.data) ? presentationEnvelope.data : undefined;
-        const dataClicked = typeof presentationDataRecord?.clicked === "string" ? presentationDataRecord.clicked : undefined;
-        const cssClickWithoutHref = prepared.executionPlan.commandInfo.command === "click" && dataClicked !== undefined && !dataClicked.startsWith("@") && !dataClicked.startsWith("ref=") && typeof presentationDataRecord?.href !== "string";
-        const parsedAllowedDomainsPolicy = parseAllowedDomainsPolicyFromArgs(prepared.runtimeToolArgs);
-        const sessionAllowedDomainsPolicy = sessionStateKey
-            ? parsedAllowedDomainsPolicy ?? allowedDomainsBySession.get(sessionStateKey)
-            : parsedAllowedDomainsPolicy;
-        const shouldCaptureAllowedDomainNavigationSummary = sessionAllowedDomainsPolicy !== undefined && !cssClickWithoutHref && (prepared.executionPlan.commandInfo.command === "batch" || isNavigationObservableCommandName(prepared.executionPlan.commandInfo.command));
         if (succeeded &&
             !navigationSummary &&
-            (shouldCaptureNavigationSummary(prepared.executionPlan.commandInfo.command, presentationEnvelope?.data) ||
+            (shouldCaptureNavigationSummary(prepared.executionPlan.commandInfo.command, presentationEnvelope?.data, prepared.executionPlan.commandInfo.subcommand) ||
                 shouldCaptureSemanticActionNavigationSummary(prepared.compiledSemanticAction, presentationEnvelope?.data) ||
-                shouldCaptureAllowedDomainNavigationSummary ||
-                managedSessionCommandRequiresLivePageVerification(prepared.executionPlan.effectiveArgs, prepared.runtimeToolStdin) ||
-                (prepared.executionPlan.commandInfo.command === "tab" && prepared.executionPlan.commandInfo.subcommand === "close"))) {
-            navigationSummary = await collectNavigationSummary({ cwd, namespace: prepared.executionPlan.namespace, sessionName: prepared.executionPlan.sessionName, signal });
+                commandRequiresLivePageVerification(prepared.executionPlan.effectiveArgs, prepared.runtimeToolStdin) ||
+                tabTransition)) {
+            navigationSummary = await collectNavigationSummary({ cwd, namespace: prepared.executionPlan.namespace, priorTarget: prepared.priorSessionTabTarget, reusePriorTitle: !tabTransition, sessionName: prepared.executionPlan.sessionName, signal });
+        }
+        // A failed eval/back/forward/reload/state-load/tab would otherwise drop the page to unverified and force the
+        // agent through a manual get url round trip. Probe the live URL ourselves instead: an observed page
+        // stays verified, and a failed
+        // probe preserves the existing unknown-target behavior.
+        if (succeeded === false &&
+            !navigationSummary &&
+            processResult.agentBrowserStarted &&
+            !processResult.aborted &&
+            !processResult.timedOut &&
+            prepared.executionPlan.commandInfo.command !== "batch" &&
+            isUnverifiedPageTransitionCommand(prepared.executionPlan.commandInfo.command, prepared.executionPlan.commandInfo.subcommand)) {
+            navigationSummary = await collectNavigationSummary({ cwd, namespace: prepared.executionPlan.namespace, priorTarget: prepared.priorSessionTabTarget, sessionName: prepared.executionPlan.sessionName, signal });
+            // A failed transition command can still have mutated or replaced the document before throwing, so
+            // keeping the verified URL must not keep the prior snapshot refs (markTabTargetUnknown dropped
+            // them before this probe existed); invalidate so the next ref use requires a fresh snapshot.
+            failedTransitionReverification = navigationSummary !== undefined;
         }
         if (navigationSummary && presentationEnvelope && prepared.executionPlan.commandInfo.command !== "eval" && !Array.isArray(presentationEnvelope.data))
             presentationEnvelope = { ...presentationEnvelope, data: mergeNavigationSummaryIntoData(presentationEnvelope.data, navigationSummary) };
@@ -198,24 +297,28 @@ export async function processBrowserOutput(input) {
         }
         const verifiesCurrentUrl = prepared.executionPlan.commandInfo.command === "get" && prepared.executionPlan.commandInfo.subcommand === "url";
         const trustsReportedPageTarget = !resultingPageState.pageUrlUnknown || verifiesCurrentUrl;
-        const observedSessionTabTarget = normalizeSessionTabTarget(navigationSummary)
-            ?? (trustsReportedPageTarget ? extractSessionTabTargetFromBatchResults(presentationEnvelope?.data) : undefined)
-            ?? (succeeded && trustsReportedPageTarget ? extractSessionTabTargetFromCommandData(prepared.commandTokens, presentationEnvelope?.data) : undefined);
-        const observedPageValidationError = succeeded ? getObservedBrowserPageValidationError(prepared.executionPlan.effectiveArgs, observedSessionTabTarget?.url, cwd) : undefined;
-        if (observedPageValidationError) {
-            succeeded = false;
-            presentationEnvelope = { error: observedPageValidationError, success: false };
+        const pendingWebMcpMutation = isPendingWebMcpMutation(prepared.executionPlan.commandInfo.command, prepared.executionPlan.commandInfo.subcommand, presentationEnvelope?.data) || (prepared.executionPlan.commandInfo.command === "batch" && batchHasPendingWebMcpMutation(presentationEnvelope?.data));
+        const failedWebMcpSettlement = prepared.priorSessionTabTargetUnknown === true && ((!succeeded && isWebMcpSettlementCommand(prepared.executionPlan.commandInfo.command, prepared.executionPlan.commandInfo.subcommand))
+            || (prepared.executionPlan.commandInfo.command === "batch" && batchHasFailedWebMcpSettlement(presentationEnvelope?.data)));
+        const unsettledWebMcpMutation = pendingWebMcpMutation || failedWebMcpSettlement;
+        const observedSessionTabTarget = unsettledWebMcpMutation
+            ? undefined
+            : normalizeSessionTabTarget(navigationSummary)
+                ?? (trustsReportedPageTarget ? extractSessionTabTargetFromBatchResults(presentationEnvelope?.data) : undefined)
+                ?? (succeeded && trustsReportedPageTarget ? extractSessionTabTargetFromCommandData(prepared.commandTokens, presentationEnvelope?.data) : undefined);
+        const safeObservedSessionTabTarget = observedSessionTabTarget;
+        let currentSessionTabTarget = safeObservedSessionTabTarget;
+        if (!currentSessionTabTarget && nestedBatchClose === undefined) {
+            currentSessionTabTarget = resultingPageState.pageTargetMayHaveChanged
+                ? succeeded ? normalizeSessionTabTarget({ url: resultingPageState.currentPageUrl }) : undefined
+                : deriveSessionTabTarget({ command: prepared.executionPlan.commandInfo.command, data: presentationEnvelope?.data, navigationSummary, previousTarget: prepared.priorSessionTabTarget, subcommand: prepared.executionPlan.commandInfo.subcommand });
         }
-        const safeObservedSessionTabTarget = observedPageValidationError ? normalizeSessionTabTarget({ url: observedSessionTabTarget?.url }) : observedSessionTabTarget;
-        let currentSessionTabTarget = safeObservedSessionTabTarget ?? (resultingPageState.pageTargetMayHaveChanged
-            ? succeeded ? normalizeSessionTabTarget({ url: resultingPageState.currentPageUrl }) : undefined
-            : deriveSessionTabTarget({ command: prepared.executionPlan.commandInfo.command, data: presentationEnvelope?.data, navigationSummary, previousTarget: prepared.priorSessionTabTarget, subcommand: prepared.executionPlan.commandInfo.subcommand }));
         let aboutBlankSessionMismatch;
         let electronPostCommandHealth;
         let electronRefFreshnessDiagnostic;
         let electronSessionMismatch;
         let electronStatusAfterCommand;
-        const shouldTreatAboutBlankAsMismatch = succeeded && prepared.priorSessionTabTarget !== undefined && !isAboutBlankSessionTabTarget(prepared.priorSessionTabTarget) && isAboutBlankSessionTabTarget(observedSessionTabTarget ?? currentSessionTabTarget) && !commandExplicitlyTargetsAboutBlank(prepared.commandTokens);
+        const shouldTreatAboutBlankAsMismatch = succeeded && !tabTransition && nestedBatchClose === undefined && prepared.priorSessionTabTarget !== undefined && !isAboutBlankSessionTabTarget(prepared.priorSessionTabTarget) && isAboutBlankSessionTabTarget(observedSessionTabTarget ?? currentSessionTabTarget) && !commandExplicitlyTargetsAboutBlank(prepared.commandTokens);
         let sessionTabCorrection = prepared.sessionTabCorrection;
         if (shouldTreatAboutBlankAsMismatch && prepared.priorSessionTabTarget) {
             const aboutBlankObservedTarget = observedSessionTabTarget ?? currentSessionTabTarget;
@@ -228,7 +331,7 @@ export async function processBrowserOutput(input) {
             else
                 currentSessionTabTarget = aboutBlankObservedTarget ?? normalizeSessionTabTarget({ url: "about:blank" });
             aboutBlankSessionMismatch = { activeUrl: "about:blank", recoveryApplied: appliedAboutBlankRecovery !== undefined, recoveryHint: buildAboutBlankRecoveryHint(), targetTitle: prepared.priorSessionTabTarget.title, targetUrl: prepared.priorSessionTabTarget.url };
-            const electronRecord = findElectronLaunchRecordForSession(prepared.executionPlan.sessionName, electronLaunchRecords);
+            const electronRecord = findElectronLaunchRecordForSession(prepared.executionPlan.sessionName, electronLaunchRecords, prepared.executionPlan.namespace);
             if (electronRecord && prepared.executionPlan.sessionName) {
                 electronStatusAfterCommand = await inspectElectronLaunchStatus(electronRecord);
                 electronSessionMismatch = buildElectronSessionMismatch({ managedSession: { sessionName: prepared.executionPlan.sessionName, title: aboutBlankObservedTarget?.title, url: aboutBlankObservedTarget?.url ?? "about:blank" }, record: electronRecord, statusTargets: electronStatusAfterCommand.targets });
@@ -242,19 +345,7 @@ export async function processBrowserOutput(input) {
                     sessionTabCorrection = appliedPostCommandCorrection;
             }
         }
-        if (succeeded && sessionStateKey && parsedAllowedDomainsPolicy) {
-            allowedDomainsBySession = new Map(allowedDomainsBySession);
-            allowedDomainsBySession.set(sessionStateKey, parsedAllowedDomainsPolicy);
-        }
-        const allowedDomainsViolation = succeeded ? getAllowedDomainsViolation({
-            policy: sessionAllowedDomainsPolicy,
-            url: currentSessionTabTarget?.url ?? observedSessionTabTarget?.url ?? navigationSummary?.url,
-        }) : undefined;
-        if (allowedDomainsViolation) {
-            succeeded = false;
-            presentationEnvelope = { ...(presentationEnvelope ?? {}), error: allowedDomainsViolation.summary, success: false };
-        }
-        const electronRecordForCommand = findElectronLaunchRecordForSession(prepared.executionPlan.sessionName, electronLaunchRecords);
+        const electronRecordForCommand = findElectronLaunchRecordForSession(prepared.executionPlan.sessionName, electronLaunchRecords, prepared.executionPlan.namespace);
         if (succeeded && electronRecordForCommand && shouldInspectElectronPostCommandHealth(prepared.executionPlan.commandInfo.command)) {
             electronStatusAfterCommand ??= await inspectElectronLaunchStatus(electronRecordForCommand);
             electronPostCommandHealth = buildElectronPostCommandHealthDiagnostic({ command: prepared.executionPlan.commandInfo.command, record: electronRecordForCommand, status: electronStatusAfterCommand, target: observedSessionTabTarget ?? currentSessionTabTarget });
@@ -288,7 +379,7 @@ export async function processBrowserOutput(input) {
         if (succeeded) {
             selectorTextVisibilityDiagnostics = await collectSelectorTextVisibilityDiagnostics({ commandInfo: prepared.executionPlan.commandInfo, commandTokens: prepared.commandTokens, cwd, data: presentationEnvelope?.data, namespace: prepared.executionPlan.namespace, sessionName: prepared.executionPlan.sessionName, signal });
             if (electronRecordForCommand)
-                electronBroadGetTextScopeDiagnostics = collectElectronBroadGetTextScopeDiagnostics({ commandInfo: prepared.executionPlan.commandInfo, commandTokens: prepared.commandTokens, currentTarget: currentSessionTabTarget, data: presentationEnvelope?.data, electronLaunchRecords, priorTarget: prepared.priorSessionTabTarget, sessionName: prepared.executionPlan.sessionName });
+                electronBroadGetTextScopeDiagnostics = collectElectronBroadGetTextScopeDiagnostics({ commandInfo: prepared.executionPlan.commandInfo, commandTokens: prepared.commandTokens, currentTarget: currentSessionTabTarget, data: presentationEnvelope?.data, electronLaunchRecords, namespace: prepared.executionPlan.namespace, priorTarget: prepared.priorSessionTabTarget, sessionName: prepared.executionPlan.sessionName });
         }
         const activeNetworkRoutes = sessionStateKey ? networkRoutesBySession.get(sessionStateKey) : undefined;
         const networkRouteDiagnostics = succeeded && prepared.executionPlan.commandInfo.command === "network" && prepared.executionPlan.commandInfo.subcommand === "requests" && prepared.executionPlan.sessionName
@@ -298,36 +389,65 @@ export async function processBrowserOutput(input) {
         const comboboxFocusDiagnostic = succeeded ? await collectComboboxFocusDiagnostic({ command: prepared.executionPlan.commandInfo.command, commandTokens: prepared.commandTokens, cwd, namespace: prepared.executionPlan.namespace, semanticAction: prepared.compiledSemanticAction, sessionName: prepared.executionPlan.sessionName, signal }) : undefined;
         const recordingDependencyWarning = await collectRecordingDependencyWarning({ command: prepared.executionPlan.commandInfo.command, commandTokens: prepared.commandTokens, succeeded });
         const scrollNoopDiagnostic = succeeded && prepared.shouldProbeScrollNoop ? buildScrollNoopDiagnostic(prepared.scrollPositionBefore, await collectScrollPositionSnapshot({ cwd, namespace: prepared.executionPlan.namespace, sessionName: prepared.executionPlan.sessionName, signal })) : undefined;
+        const batchRefSnapshotState = prepared.executionPlan.commandInfo.command === "batch" ? extractLatestRefSnapshotStateFromBatchResults(presentationEnvelope?.data) : undefined;
         let currentRefSnapshot;
         let currentRefSnapshotInvalidation;
-        const batchRefSnapshotState = prepared.executionPlan.commandInfo.command === "batch" ? extractLatestRefSnapshotStateFromBatchResults(presentationEnvelope?.data) : undefined;
         if (sessionStateKey) {
-            if (isCloseCommand(prepared.executionPlan.commandInfo.command) && succeeded) {
-                allowedDomainsBySession = new Map(allowedDomainsBySession);
-                allowedDomainsBySession.delete(sessionStateKey);
+            const sessionClosed = (isCloseCommand(prepared.executionPlan.commandInfo.command) && succeeded) || nestedBatchClosed;
+            if (sessionClosed) {
+                state.attachedSessionKeys.delete(sessionStateKey);
                 networkRoutesBySession = new Map(networkRoutesBySession);
                 networkRoutesBySession.delete(sessionStateKey);
                 sessionPageState.clearSession(sessionStateKey);
                 state.closedManagedSessionNames.add(sessionStateKey);
             }
-            else if (currentSessionTabTarget) {
-                const tabUpdate = sessionPageState.applyTabTarget({ sessionName: sessionStateKey, target: currentSessionTabTarget, update: sessionPageStateUpdate });
-                if (!tabUpdate.applied && succeeded)
-                    sessionPageState.markPinning(sessionStateKey, "drift");
-            }
-            else if (processResult.agentBrowserStarted && (resultingPageState.pageUrlUnknown || resultingPageState.pageTargetMayHaveChanged)) {
-                sessionPageState.markTabTargetUnknown({ sessionName: sessionStateKey, update: sessionPageStateUpdate });
-            }
-            const refSnapshot = prepared.executionPlan.commandInfo.command === "batch" ? batchRefSnapshotState?.snapshot : succeeded ? prepared.executionPlan.commandInfo.command === "snapshot" ? extractRefSnapshotFromData(presentationEnvelope?.data) : prepared.resolvedSemanticActionRefSnapshot ?? overlayBlockerDiagnostic?.snapshot : undefined;
-            if (refSnapshot) {
-                const refUpdate = sessionPageState.applyRefSnapshot({ fallbackTarget: currentSessionTabTarget, sessionName: sessionStateKey, snapshot: refSnapshot, update: sessionPageStateUpdate });
-                currentRefSnapshot = refUpdate.refSnapshot;
-                currentRefSnapshotInvalidation = refUpdate.refSnapshotInvalidation;
-            }
             else {
-                const stateView = sessionPageState.get(sessionStateKey);
-                currentRefSnapshot = stateView.refSnapshot;
-                currentRefSnapshotInvalidation = stateView.refSnapshotInvalidation;
+                // A batch that times out or returns unparseable output yields no result rows, but the daemon
+                // may already have executed a recording swap or page-provided WebMCP tool; fall back to the
+                // planned steps then. This can over-invalidate by one snapshot, but never under-invalidates.
+                const directTransitionInvalidation = getCommandRefSnapshotInvalidation(prepared.commandTokens);
+                const plannedBatchTransitionInvalidation = !Array.isArray(presentationEnvelope?.data)
+                    ? batchCommandSteps.map(getCommandRefSnapshotInvalidation).find((invalidation) => invalidation !== undefined)
+                    : undefined;
+                const pageTransitionInvalidation = unsettledWebMcpMutation
+                    ? buildPageTransitionRefSnapshotInvalidation("A detached WebMCP invocation is still pending or failed to settle and can mutate, rerender, or navigate the page, so prior snapshot refs remain invalid after URL verification. Run webmcp result or cancel, then take a fresh snapshot before using page-scoped refs.")
+                    : processResult.agentBrowserStarted && (directTransitionInvalidation || plannedBatchTransitionInvalidation)
+                        ? directTransitionInvalidation ?? plannedBatchTransitionInvalidation
+                        : failedTransitionReverification
+                            ? buildPageTransitionRefSnapshotInvalidation("A failed eval/back/forward/reload/connect/state-load/tab command may still have changed the page, so the prior snapshot refs were invalidated. Run snapshot -i before using page-scoped refs.")
+                            : batchRefSnapshotState?.invalidation?.reason === "page-transition"
+                                ? batchRefSnapshotState.invalidation
+                                : undefined;
+                if (currentSessionTabTarget) {
+                    const tabUpdate = sessionPageState.applyTabTarget({ sessionName: sessionStateKey, target: currentSessionTabTarget, update: sessionPageStateUpdate });
+                    if (!tabUpdate.applied && succeeded)
+                        sessionPageState.markPinning(sessionStateKey, "drift");
+                }
+                else if (processResult.agentBrowserStarted && (resultingPageState.pageUrlUnknown || resultingPageState.pageTargetMayHaveChanged)) {
+                    sessionPageState.markTabTargetUnknown({ sessionName: sessionStateKey, update: sessionPageStateUpdate });
+                }
+                const refSnapshot = unsettledWebMcpMutation
+                    ? undefined
+                    : prepared.executionPlan.commandInfo.command === "batch"
+                        ? batchRefSnapshotState?.snapshot
+                        : succeeded
+                            ? prepared.executionPlan.commandInfo.command === "snapshot" ? extractRefSnapshotFromData(presentationEnvelope?.data) : prepared.resolvedSemanticActionRefSnapshot ?? overlayBlockerDiagnostic?.snapshot
+                            : undefined;
+                if (refSnapshot) {
+                    const refUpdate = sessionPageState.applyRefSnapshot({ fallbackTarget: currentSessionTabTarget, sessionName: sessionStateKey, snapshot: refSnapshot, update: sessionPageStateUpdate });
+                    currentRefSnapshot = refUpdate.refSnapshot;
+                    currentRefSnapshotInvalidation = refUpdate.refSnapshotInvalidation;
+                }
+                else if (pageTransitionInvalidation) {
+                    const refUpdate = sessionPageState.applyRefSnapshotInvalidation({ invalidation: pageTransitionInvalidation, sessionName: sessionStateKey, update: sessionPageStateUpdate });
+                    currentRefSnapshot = refUpdate.refSnapshot;
+                    currentRefSnapshotInvalidation = refUpdate.refSnapshotInvalidation;
+                }
+                else {
+                    const stateView = sessionPageState.get(sessionStateKey);
+                    currentRefSnapshot = stateView.refSnapshot;
+                    currentRefSnapshotInvalidation = stateView.refSnapshotInvalidation;
+                }
             }
         }
         const priorManagedSessionActive = managedSessionActive;
@@ -335,12 +455,22 @@ export async function processBrowserOutput(input) {
         const priorManagedSessionHeadedAutosaveInterval = managedSessionHeadedAutosaveInterval;
         const priorManagedSessionName = managedSessionName;
         const priorManagedSessionNamespace = managedSessionNamespace;
-        const commandClosesSession = isCloseCommand(prepared.executionPlan.commandInfo.command);
+        const priorManagedSessionKey = getSessionContextKey(priorManagedSessionName, priorManagedSessionNamespace) ?? priorManagedSessionName;
+        const closeAllTargetsPriorManagedSession = closeAllApplied
+            && priorManagedSessionActive
+            && isAgentBrowserSessionIdentityKeyInNamespace(priorManagedSessionKey, prepared.executionPlan.namespace);
+        const closeAllRetainsPriorManagedSession = closeAllTargetsPriorManagedSession
+            && nestedBatchRemainsActive
+            && sessionStateKey === priorManagedSessionKey;
+        const closeAllClosesPriorManagedSession = closeAllTargetsPriorManagedSession && !closeAllRetainsPriorManagedSession;
+        const commandClosesSession = isCloseCommand(prepared.executionPlan.commandInfo.command) || nestedBatchClosed || closeAllClosesPriorManagedSession;
+        const closeCommandSucceeded = (isCloseCommand(prepared.executionPlan.commandInfo.command) && succeeded) || nestedBatchClosed || closeAllClosesPriorManagedSession;
         const closeTargetsPriorManagedNamespace = prepared.executionPlan.namespace === priorManagedSessionNamespace;
-        const managedCloseSessionName = commandClosesSession && succeeded && prepared.executionPlan.sessionName === priorManagedSessionName && closeTargetsPriorManagedNamespace
-            ? prepared.executionPlan.sessionName
-            : prepared.executionPlan.managedSessionName;
-        const policyBlockedFreshManagedSession = allowedDomainsViolation !== undefined && prepared.sessionMode === "fresh" && prepared.executionPlan.managedSessionName === prepared.executionPlan.sessionName;
+        const managedCloseSessionName = closeAllClosesPriorManagedSession
+            ? priorManagedSessionName
+            : closeCommandSucceeded && prepared.executionPlan.sessionName === priorManagedSessionName && closeTargetsPriorManagedNamespace
+                ? prepared.executionPlan.sessionName
+                : prepared.executionPlan.managedSessionName;
         const postLaunchBatchFailure = !succeeded && processSucceeded && parseSucceeded && prepared.sessionMode === "fresh" && prepared.executionPlan.commandInfo.command === "batch" && batchStartedManagedBrowser(presentationEnvelope?.data);
         const postLaunchTimeoutWithPage = !succeeded && processResult.timedOut && prepared.sessionMode === "fresh" && prepared.executionPlan.commandInfo.command === "batch" && timeoutPartialProgress?.liveUrlRecovered === true;
         const failedFreshSessionMayHaveStarted = !succeeded
@@ -361,8 +491,8 @@ export async function processBrowserOutput(input) {
         }
         // Only a confirmed inactive daemon proves that a started fresh command did not establish browser ownership.
         const postLaunchFreshFailure = failedFreshDaemon !== undefined && failedFreshDaemon.status !== "inactive";
-        const managedTransitionSucceeded = succeeded || policyBlockedFreshManagedSession || postLaunchBatchFailure || postLaunchTimeoutWithPage || postLaunchFreshFailure;
-        const managedSessionState = resolveManagedSessionState({ command: prepared.executionPlan.commandInfo.command, managedSessionName: managedCloseSessionName, managedSessionNamespace: prepared.executionPlan.namespace, priorActive: priorManagedSessionActive, priorNamespace: priorManagedSessionNamespace, priorSessionName: priorManagedSessionName, succeeded: managedTransitionSucceeded });
+        const managedTransitionSucceeded = succeeded || nestedBatchClosed || nestedBatchRemainsActive || postLaunchBatchFailure || postLaunchTimeoutWithPage || postLaunchFreshFailure;
+        const managedSessionState = resolveManagedSessionState({ command: commandClosesSession ? "close" : prepared.executionPlan.commandInfo.command, managedSessionName: managedCloseSessionName, managedSessionNamespace: prepared.executionPlan.namespace, priorActive: priorManagedSessionActive, priorNamespace: priorManagedSessionNamespace, priorSessionName: priorManagedSessionName, succeeded: managedTransitionSucceeded });
         if (!managedTransitionSucceeded && prepared.sessionMode === "fresh" && prepared.executionPlan.managedSessionName) {
             state.managedSessionRestoreState.clear(prepared.executionPlan.managedSessionName, prepared.executionPlan.namespace);
         }
@@ -383,7 +513,7 @@ export async function processBrowserOutput(input) {
             managedSessionHeadedAutosaveDisabled = prepared.ownedManagedSessionContext?.headedManagedAutosaveDisabled === true;
             managedSessionHeadedAutosaveInterval = prepared.ownedManagedSessionContext?.headedManagedAutosaveInterval;
         }
-        if (commandClosesSession && succeeded && managedCloseSessionName === priorManagedSessionName && !managedSessionActive) {
+        if (closeCommandSucceeded && managedCloseSessionName === priorManagedSessionName && !managedSessionActive) {
             const daemonRestoreKey = state.managedSessionRestoreState.getDaemonRestoreKey(managedCloseSessionName, priorManagedSessionNamespace);
             const ownedRestoreKey = !state.managedSessionRestoreState.isDisabled(managedCloseSessionName, priorManagedSessionNamespace)
                 && isManagedSessionRestoreKey(daemonRestoreKey) ? daemonRestoreKey : null;
@@ -398,7 +528,7 @@ export async function processBrowserOutput(input) {
             managedSessionName = createFreshSessionName(state.managedSessionBaseName, state.ephemeralSessionSeed, freshSessionOrdinal);
             managedSessionNamespace = undefined;
         }
-        let managedSessionOutcome = buildManagedSessionOutcome({ activeAfter: managedSessionActive, activeBefore: priorManagedSessionActive, attemptedSessionName: managedCloseSessionName, command: prepared.executionPlan.commandInfo.command, currentSessionName: managedSessionName, currentSessionNamespace: managedSessionNamespace, previousSessionName: priorManagedSessionName, replacedSessionName: replacedManagedSessionName, replacedSessionNamespace: priorManagedSessionNamespace, sessionMode: prepared.sessionMode, succeeded: managedTransitionSucceeded });
+        let managedSessionOutcome = buildManagedSessionOutcome({ activeAfter: managedSessionActive, activeBefore: priorManagedSessionActive, attemptedSessionName: managedCloseSessionName, command: commandClosesSession ? "close" : prepared.executionPlan.commandInfo.command, currentSessionName: managedSessionName, currentSessionNamespace: managedSessionNamespace, previousSessionName: priorManagedSessionName, replacedSessionName: replacedManagedSessionName, replacedSessionNamespace: priorManagedSessionNamespace, sessionMode: prepared.sessionMode, succeeded: managedTransitionSucceeded });
         if (prepared.executionPlan.managedSessionName && managedTransitionSucceeded && managedSessionActive) {
             managedSessionCwd = cwd;
             managedSessionNamespace = prepared.executionPlan.namespace;
@@ -410,9 +540,7 @@ export async function processBrowserOutput(input) {
                 sessionPageState.clearRestorePinning(sessionStateKey);
         }
         if (replacedManagedSessionName) {
-            allowedDomainsBySession = new Map(allowedDomainsBySession);
             const replacedSessionStateKey = getSessionContextKey(replacedManagedSessionName, priorManagedSessionNamespace);
-            allowedDomainsBySession.delete(replacedSessionStateKey ?? replacedManagedSessionName);
             networkRoutesBySession = new Map(networkRoutesBySession);
             networkRoutesBySession.delete(replacedSessionStateKey ?? replacedManagedSessionName);
             sessionPageState.clearSession(replacedSessionStateKey ?? replacedManagedSessionName);
@@ -439,7 +567,7 @@ export async function processBrowserOutput(input) {
             if (succeeded && prepared.executionPlan.sessionName) {
                 const electronSessionName = prepared.executionPlan.sessionName;
                 const electronSessionStateKey = sessionStateKey ?? electronSessionName;
-                electronLaunchRecord = { ...prepared.electronLaunch.record, sessionName: electronSessionName };
+                electronLaunchRecord = { ...prepared.electronLaunch.record, namespace: prepared.executionPlan.namespace, sessionName: electronSessionName };
                 const electronHandoffMode = prepared.compiledElectron?.action === "launch" ? prepared.compiledElectron.handoff : "connect";
                 try {
                     electronHandoff = await collectElectronHandoff({ cwd, handoff: electronHandoffMode, namespace: prepared.executionPlan.namespace, sessionName: electronSessionName, signal });
@@ -467,8 +595,6 @@ export async function processBrowserOutput(input) {
                     }
                     if (!closeError) {
                         state.closedManagedSessionNames.add(electronSessionStateKey);
-                        allowedDomainsBySession = new Map(allowedDomainsBySession);
-                        allowedDomainsBySession.delete(electronSessionStateKey);
                         networkRoutesBySession = new Map(networkRoutesBySession);
                         networkRoutesBySession.delete(electronSessionStateKey);
                         sessionPageState.clearSession(electronSessionStateKey);
@@ -507,9 +633,7 @@ export async function processBrowserOutput(input) {
             if (presentationEnvelope?.error !== undefined)
                 presentationEnvelope = { ...presentationEnvelope, error: redactClipboardPermissionErrorValue(prepared.executionPlan.commandInfo, presentationEnvelope.error, clipboardWritePayloadCandidates) };
         }
-        const presentation = plainTextInspection ? { artifacts: undefined, batchFailure: undefined, batchSteps: undefined, content: [{ type: "text", text: inspectionText ?? "" }], data: undefined, fullOutputPath: undefined, fullOutputPaths: undefined, imagePath: undefined, imagePaths: undefined, savedFile: undefined, savedFilePath: undefined, summary: `${prepared.redactedArgs.join(" ")} completed` } : await buildToolPresentation({ args: prepared.redactedProcessArgs, artifactManifest, artifactRequest: screenshotArtifactRequest, batchArtifactRequests: batchScreenshotArtifactRequests, commandInfo: prepared.executionPlan.commandInfo, compiledSemanticAction: prepared.compiledSemanticAction, cwd, envelope: presentationEnvelope, errorText, namespace: prepared.executionPlan.namespace, networkRouteDiagnostics, networkRoutes: activeNetworkRoutes, persistentArtifactStore, sessionName: prepared.executionPlan.sessionName });
-        if (observedPageValidationError)
-            presentation.failureCategory = "validation-error";
+        const presentation = plainTextInspection ? { artifacts: undefined, batchFailure: undefined, batchSteps: undefined, content: [{ type: "text", text: inspectionText ?? "" }], data: undefined, fullOutputPath: undefined, fullOutputPaths: undefined, imagePath: undefined, imagePaths: undefined, savedFile: undefined, savedFilePath: undefined, summary: `${prepared.redactedArgs.join(" ")} completed` } : await buildToolPresentation({ args: prepared.redactedProcessArgs, artifactManifest, artifactMaxUpdatedAtMs: Date.now(), artifactMinUpdatedAtMs: input.artifactRunStartedAtMs, artifactRequest: screenshotArtifactRequest, batchArtifactRequests: batchScreenshotArtifactRequests, commandInfo: prepared.executionPlan.commandInfo, compiledSemanticAction: prepared.compiledSemanticAction, cwd, envelope: presentationEnvelope, errorText, namespace: prepared.executionPlan.namespace, networkRouteDiagnostics, networkRoutes: activeNetworkRoutes, persistentArtifactStore, sessionName: prepared.executionPlan.sessionName });
         if (electronHandoff?.error && electronHandoff.failureCategory)
             presentation.failureCategory = electronHandoff.failureCategory;
         networkRoutesBySession = applyBatchNetworkRouteState({ data: presentationEnvelope?.data, routesBySession: networkRoutesBySession, sessionName: sessionStateKey, succeeded });
@@ -518,13 +642,20 @@ export async function processBrowserOutput(input) {
             presentationEnvelope = { ...(presentationEnvelope ?? {}), error: presentation.summary, success: false };
         }
         if (scrollNoopDiagnostic) {
+            succeeded = false;
+            presentation.resultCategory = "failure";
+            presentation.failureCategory = "upstream-error";
+            presentationEnvelope = { ...(presentationEnvelope ?? {}), error: "Scroll completed with no observed movement.", success: false };
             presentation.summary = "Scroll completed with no observed movement.";
             if (isRecord(presentation.data))
                 presentation.data = { ...presentation.data, noMovement: true, scrolled: false };
-            if (presentation.content[0]?.type === "text")
-                presentation.content[0] = { ...presentation.content[0], text: `Scroll completed with no observed movement.\n\n${presentation.content[0].text}` };
-            else
+            if (presentation.content[0]?.type === "text") {
+                const details = isRecord(presentation.data) ? JSON.stringify(omitUpstreamLifecycle(presentation.data), null, 2) : presentation.content[0].text;
+                presentation.content[0] = { ...presentation.content[0], text: `Scroll completed with no observed movement.\n\n${details}` };
+            }
+            else {
                 presentation.content.unshift({ type: "text", text: "Scroll completed with no observed movement." });
+            }
         }
         if (parseFailureOutput.artifactManifest) {
             presentation.artifactManifest = parseFailureOutput.artifactManifest;
@@ -544,7 +675,7 @@ export async function processBrowserOutput(input) {
         let qaAttachedTarget = prepared.compiledQaPreset?.checks.attached
             ? await collectQaAttachedTarget({ currentTarget: currentSessionTabTarget ?? prepared.priorSessionTabTarget, cwd, namespace: prepared.executionPlan.namespace, sessionName: prepared.executionPlan.sessionName, signal })
             : undefined;
-        const sourceLookupElectronContext = prepared.compiledSourceLookup ? getSourceLookupElectronContext({ currentTarget: currentSessionTabTarget, electronLaunchRecords, priorTarget: prepared.priorSessionTabTarget, sessionName: prepared.executionPlan.sessionName }) : undefined;
+        const sourceLookupElectronContext = prepared.compiledSourceLookup ? getSourceLookupElectronContext({ currentTarget: currentSessionTabTarget, electronLaunchRecords, namespace: prepared.executionPlan.namespace, priorTarget: prepared.priorSessionTabTarget, sessionName: prepared.executionPlan.sessionName }) : undefined;
         const sourceLookup = prepared.compiledSourceLookup ? await analyzeSourceLookupResults(presentationEnvelope?.data, prepared.compiledSourceLookup, cwd, { electronContext: sourceLookupElectronContext, workspaceRoot: cwd }) : undefined;
         const networkSourceLookup = prepared.compiledNetworkSourceLookup ? redactNetworkSourceLookupAnalysis(await analyzeNetworkSourceLookupResults(presentationEnvelope?.data, prepared.compiledNetworkSourceLookup, cwd)) : undefined;
         if (networkSourceLookup && presentation.content[0]?.type === "text")
@@ -555,14 +686,22 @@ export async function processBrowserOutput(input) {
             presentation.content[0] = { ...presentation.content[0], text: `${sourceLookup.summary}\n\n${presentation.content[0].text}` };
         else if (sourceLookup)
             presentation.content.unshift({ type: "text", text: sourceLookup.summary });
-        if (qaPreset && !qaPreset.passed && presentation.failureCategory !== "artifact-missing") {
+        if (qaPreset && !qaPreset.passed && prepared.compiledQaPreset && presentation.failureCategory !== "artifact-missing") {
             succeeded = false;
             presentation.failureCategory = "qa-failure";
             presentation.summary = qaPreset.summary;
-            if (presentation.content[0]?.type === "text")
-                presentation.content[0] = { ...presentation.content[0], text: `${qaPreset.summary}\n\n${presentation.content[0].text}` };
-            else
-                presentation.content.unshift({ type: "text", text: qaPreset.summary });
+            const compactText = buildQaCompactFailureText({
+                batchStepCount: presentation.batchSteps?.length ?? prepared.compiledQaPreset.steps.length,
+                checks: prepared.compiledQaPreset.checks,
+                page: extractQaPageContext({
+                    attachedTarget: qaAttachedTarget,
+                    batchData: presentationEnvelope?.data,
+                    compiled: prepared.compiledQaPreset,
+                }),
+                qaPreset,
+            });
+            const nonTextContent = presentation.content.filter((item) => item.type !== "text");
+            presentation.content = [{ type: "text", text: compactText }, ...nonTextContent];
         }
         else if (qaPreset?.passed && prepared.compiledQaPreset && succeeded) {
             const compactText = buildQaCompactPassText({
@@ -616,9 +755,12 @@ export async function processBrowserOutput(input) {
         const resultHeadedManagedAutosaveInterval = resultRetainsPreparedManagedSession && !(commandClosesSession && succeeded)
             ? prepared.ownedManagedSessionContext?.headedManagedAutosaveInterval
             : undefined;
-        const result = buildFinalAgentBrowserToolResult({ aboutBlankSessionMismatch, artifactCleanup, categoryDetails: finalRecoveryState.categoryDetails, clickDispatchDiagnostic, commandTokens: prepared.commandTokens, comboboxFocusDiagnostic, compiledNetworkSourceLookup: prepared.compiledNetworkSourceLookup, compiledSemanticAction: prepared.compiledSemanticAction, compatibilityWorkaround: prepared.compatibilityWorkaround, currentRefSnapshot, currentRefSnapshotInvalidation, currentSessionTabTarget, currentSessionTabTargetUnknown, electronBroadGetTextScopeDiagnostics, electronFailedConnectCleanup, electronHandoff, electronLaunch: prepared.electronLaunch, electronLaunchRecord, electronLaunchRecords, electronPostCommandHealth, electronProfileIsolationDetails: input.electronProfileIsolationDetails, electronRefFreshnessDiagnostic, electronSessionMismatch, errorText, evalResultWarning, evalStdinHint, exactSensitiveValues: prepared.exactSensitiveValues, executionPlan: prepared.executionPlan, fillVerificationDiagnostic, inspectionText, managedSessionHeadedAutosaveDisabled: resultHeadedManagedAutosaveDisabled || undefined, managedSessionHeadedAutosaveInterval: resultHeadedManagedAutosaveInterval, managedSessionOutcome, managedSessionRestoreDisabled: state.managedSessionRestoreState.isDisabled(prepared.executionPlan.sessionName, prepared.executionPlan.namespace), navigationSummary, networkSourceLookup, noActivePageSnapshotFailure: finalRecoveryState.noActivePageSnapshotFailure, openResultTabCorrection, overlayBlockerDiagnostic, parseError, parseFailureOutput, parseSucceeded, plainTextInspection, presentation, presentationEnvelope, priorSessionTabTarget: prepared.priorSessionTabTarget, processResult, qaAttachedTarget, qaPreset, recordingDependencyWarning, redactedArgs: prepared.redactedArgs, redactedCompiledElectron: prepared.redactedCompiledElectron, redactedCompiledJob: prepared.redactedCompiledJob, redactedCompiledNetworkSourceLookup: prepared.redactedCompiledNetworkSourceLookup, redactedCompiledQaPreset: prepared.redactedCompiledQaPreset, redactedCompiledSemanticAction: prepared.redactedCompiledSemanticAction, redactedCompiledSourceLookup: prepared.redactedCompiledSourceLookup, redactedContent, redactedProcessArgs: prepared.redactedProcessArgs, redactedRecoveryHint: prepared.redactedRecoveryHint, resultArtifactManifest, richInputRecoveryDiagnostic: finalRecoveryState.richInputRecoveryDiagnostic, scrollNoopDiagnostic, selectorTextVisibilityDiagnostics, sessionMode: prepared.sessionMode, sessionTabCorrection, sourceLookup, succeeded, timeoutPartialProgress, userRequestedJson: prepared.userRequestedJson, visibleRefFallbackDiagnostic: finalRecoveryState.visibleRefFallbackDiagnostic, visibleRefFallbackSessionName: finalRecoveryState.visibleRefFallbackSessionName });
-        const statePatch = { allowedDomainsBySession, artifactManifest, freshSessionOrdinal, managedSessionActive, managedSessionCompatibilityWorkaround, managedSessionHeadedAutosaveDisabled, managedSessionHeadedAutosaveInterval, managedSessionCwd, managedSessionName, managedSessionNamespace, networkRoutesBySession };
-        return { result, statePatch };
+        const result = buildFinalAgentBrowserToolResult({ aboutBlankSessionMismatch, artifactCleanup, categoryDetails: finalRecoveryState.categoryDetails, clickDispatchDiagnostic, commandTokens: prepared.commandTokens, comboboxFocusDiagnostic, compiledNetworkSourceLookup: prepared.compiledNetworkSourceLookup, compiledSemanticAction: prepared.compiledSemanticAction, compatibilityWorkaround: prepared.compatibilityWorkaround, currentRefSnapshot, currentRefSnapshotInvalidation, currentSessionTabTarget, currentSessionTabTargetUnknown, electronBroadGetTextScopeDiagnostics, electronFailedConnectCleanup, electronHandoff, electronLaunch: prepared.electronLaunch, electronLaunchRecord, electronLaunchRecords, electronPostCommandHealth, electronProfileIsolationDetails: input.electronProfileIsolationDetails, electronRefFreshnessDiagnostic, electronSessionMismatch, errorText, evalResultWarning, evalStdinHint, exactSensitiveValues: prepared.exactSensitiveValues, executionPlan: prepared.executionPlan, fillVerificationDiagnostic, headedLaunch: prepared.headedLaunch, inspectionText, preserveAttachedBrowserSession: input.preserveAttachedBrowserSession === true, providerLaunch: prepared.providerLaunch, managedSessionHeadedAutosaveDisabled: resultHeadedManagedAutosaveDisabled || undefined, managedSessionHeadedAutosaveInterval: resultHeadedManagedAutosaveInterval, managedSessionOutcome, managedSessionRestoreDisabled: state.managedSessionRestoreState.isDisabled(prepared.executionPlan.sessionName, prepared.executionPlan.namespace), navigationSummary, networkSourceLookup, noActivePageSnapshotFailure: finalRecoveryState.noActivePageSnapshotFailure, openResultTabCorrection, overlayBlockerDiagnostic, parseError, parseFailureOutput, parseSucceeded, plainTextInspection, presentation, presentationEnvelope, priorSessionTabTarget: prepared.priorSessionTabTarget, processResult, qaAttachedTarget, qaPreset, recordingDependencyWarning, redactedArgs: prepared.redactedArgs, redactedCompiledElectron: prepared.redactedCompiledElectron, redactedCompiledJob: prepared.redactedCompiledJob, redactedCompiledNetworkSourceLookup: prepared.redactedCompiledNetworkSourceLookup, redactedCompiledQaPreset: prepared.redactedCompiledQaPreset, redactedCompiledSemanticAction: prepared.redactedCompiledSemanticAction, redactedCompiledSourceLookup: prepared.redactedCompiledSourceLookup, redactedContent, redactedProcessArgs: prepared.redactedProcessArgs, redactedRecoveryHint: prepared.redactedRecoveryHint, resultArtifactManifest, richInputRecoveryDiagnostic: finalRecoveryState.richInputRecoveryDiagnostic, scrollNoopDiagnostic, selectorTextVisibilityDiagnostics, sessionMode: prepared.sessionMode, sessionTabCorrection, sourceLookup, succeeded, timeoutPartialProgress, unsettledWebMcpMutation, userRequestedJson: prepared.userRequestedJson, visibleRefFallbackDiagnostic: finalRecoveryState.visibleRefFallbackDiagnostic, visibleRefFallbackSessionName: finalRecoveryState.visibleRefFallbackSessionName });
+        const resultWithCloseAll = closeAllApplied
+            ? { ...result, details: { ...(isRecord(result.details) ? result.details : {}), closeAllApplied: true } }
+            : result;
+        const statePatch = { artifactManifest, freshSessionOrdinal, managedSessionActive, managedSessionCompatibilityWorkaround, managedSessionHeadedAutosaveDisabled, managedSessionHeadedAutosaveInterval, managedSessionCwd, managedSessionName, managedSessionNamespace, networkRoutesBySession };
+        return { result: resultWithCloseAll, statePatch };
     }
     finally {
         if (processResult.stdoutSpillPath)