pi-agent-browser-native 0.2.24 → 0.2.25

package/extensions/agent-browser/lib/results/presentation.ts

@@ -10,7 +10,7 @@ import { readFile, stat } from "node:fs/promises";
 import { extname, resolve } from "node:path";
 
 import { isRecord, parsePositiveInteger } from "../parsing.js";
-import { parseCommandInfo, redactSensitiveText, redactSensitiveValue, type CommandInfo } from "../runtime.js";
+import { extractCommandTokens, parseCommandInfo, redactInvocationArgs, redactSensitiveText, redactSensitiveValue, type CommandInfo } from "../runtime.js";
 import {
 	type PersistentSessionArtifactEviction,
 	type PersistentSessionArtifactStore,
@@ -22,6 +22,13 @@ import { buildSnapshotPresentation, formatRawSnapshotText, formatSnapshotSummary
 import {
 	type AgentBrowserBatchResult,
 	type AgentBrowserEnvelope,
+	type AgentBrowserPageChangeSummary,
+	type ArtifactVerificationEntry,
+	type ArtifactVerificationSummary,
+	buildAgentBrowserNextActions,
+	buildAgentBrowserResultCategoryDetails,
+	classifyAgentBrowserFailureCategory,
+	classifyAgentBrowserSuccessCategory,
 	type BatchFailurePresentationDetails,
 	type BatchStepPresentationDetails,
 	type ArtifactStorageScope,
@@ -50,6 +57,32 @@ const IMAGE_EXTENSION_TO_MIME_TYPE: Record<string, string> = {
 const INLINE_IMAGE_MAX_BYTES_ENV = "PI_AGENT_BROWSER_INLINE_IMAGE_MAX_BYTES";
 const DEFAULT_INLINE_IMAGE_MAX_BYTES = 5 * 1_024 * 1_024;
 const NAVIGATION_SUMMARY_COMMANDS = new Set(["back", "click", "dblclick", "forward", "reload"]);
+const PAGE_CHANGE_SUMMARY_COMMANDS = new Set([
+	"back",
+	"check",
+	"click",
+	"dblclick",
+	"dialog",
+	"download",
+	"fill",
+	"forward",
+	"goto",
+	"hover",
+	"navigate",
+	"open",
+	"pdf",
+	"press",
+	"pushstate",
+	"reload",
+	"screenshot",
+	"scroll",
+	"scrollintoview",
+	"select",
+	"swipe",
+	"tap",
+	"type",
+	"uncheck",
+]);
 const NAVIGATION_SUMMARY_FIELD = "navigationSummary";
 const LARGE_OUTPUT_INLINE_MAX_CHARS = 8_000;
 const LARGE_OUTPUT_INLINE_MAX_LINES = 120;
@@ -229,13 +262,89 @@ function formatDiagnosticSummary(commandInfo: CommandInfo, data: Record<string,
 		if (profiles) return `Auth profiles: ${profiles.length}`;
 		const name = getStringField(data, "name") ?? getStringField(data, "profile") ?? commandInfo.subcommand;
 		if (name && commandInfo.subcommand === "show") return `Auth profile: ${name}`;
+		if (name && ["save", "login", "delete"].includes(commandInfo.subcommand ?? "")) return `Auth ${commandInfo.subcommand}: ${name}`;
 	}
 
-	if (commandInfo.command === "network" && commandInfo.subcommand === "requests") {
-		const requests = getArrayField(data, "requests");
-		if (requests) return `Network requests: ${requests.length}`;
+	if (commandInfo.command === "cookies") {
+		const cookies = getArrayField(data, "cookies");
+		if (cookies) return `Cookies: ${cookies.length}`;
+		const name = getStringField(data, "name");
+		if (name) return name;
+		if (data.set === true) return "Cookie set";
+		if (data.cleared === true || data.clear === true) return "Cookies cleared";
 	}
 
+	if (commandInfo.command === "storage") {
+		const entries = getArrayField(data, "entries") ?? getArrayField(data, "items");
+		if (entries) return `Storage entries: ${entries.length}`;
+		const key = getStringField(data, "key");
+		if (key && (commandInfo.subcommand === "set" || data.set === true || Object.hasOwn(data, "value"))) return `Storage set: ${key}`;
+		if (data.cleared === true || data.clear === true) return "Storage cleared";
+	}
+
+	if (commandInfo.command === "dialog") {
+		const open = typeof data.open === "boolean" ? data.open : undefined;
+		if (open !== undefined) return open ? "Dialog open" : "No dialog open";
+		if (data.accepted === true) return "Dialog accepted";
+		if (data.dismissed === true) return "Dialog dismissed";
+	}
+
+	if (commandInfo.command === "frame") {
+		const frame = getStringField(data, "frame") ?? getStringField(data, "name") ?? getStringField(data, "selector") ?? commandInfo.subcommand;
+		if (frame) return `Frame: ${frame}`;
+	}
+
+	if (commandInfo.command === "state") {
+		const states = getArrayField(data, "states") ?? getArrayField(data, "files");
+		if (states) return `States: ${states.length}`;
+		if (commandInfo.subcommand === "load") return undefined;
+		const stateName = getStringField(data, "name") ?? getStringField(data, "file") ?? getStringField(data, "path") ?? commandInfo.subcommand;
+		if (stateName) return `State ${commandInfo.subcommand ?? "result"}: ${stateName}`;
+	}
+
+	if (commandInfo.command === "network") {
+		if (commandInfo.subcommand === "requests") {
+			const requests = getArrayField(data, "requests");
+			if (requests) return `Network requests: ${requests.length}`;
+		}
+		if (commandInfo.subcommand === "route") {
+			const routed = getStringField(data, "routed") ?? getStringField(data, "url") ?? getStringField(data, "pattern");
+			return routed ? `Network route: ${redactModelFacingTextIfSensitive(routed)}` : "Network route configured";
+		}
+		if (commandInfo.subcommand === "unroute") {
+			const unrouted = getStringField(data, "unrouted") ?? getStringField(data, "url") ?? getStringField(data, "pattern");
+			return unrouted ? `Network unroute: ${redactModelFacingTextIfSensitive(unrouted)}` : "Network route removed";
+		}
+		if (commandInfo.subcommand === "har") {
+			const state = getStringField(data, "state") ?? getStringField(data, "status") ?? commandInfo.subcommand;
+			return `Network HAR: ${state}`;
+		}
+	}
+
+	if (commandInfo.command === "diff") {
+		if (commandInfo.subcommand === "snapshot") return "Snapshot diff completed";
+		if (commandInfo.subcommand === "url") return "URL diff completed";
+	}
+
+	if (["trace", "profiler"].includes(commandInfo.command ?? "")) {
+		const state = getStringField(data, "state") ?? getStringField(data, "status") ?? commandInfo.subcommand;
+		if (state) return `${commandInfo.command === "trace" ? "Trace" : "Profiler"}: ${state}`;
+	}
+
+	if (commandInfo.command === "highlight") return "Element highlighted";
+	if (commandInfo.command === "inspect") return "DevTools inspect opened";
+	if (commandInfo.command === "clipboard") return `Clipboard ${commandInfo.subcommand ?? "completed"}`;
+
+	if (commandInfo.command === "stream") {
+		if (commandInfo.subcommand === "enable") {
+			const port = typeof data.port === "number" ? ` on port ${data.port}` : "";
+			return `Stream enabled${port}`;
+		}
+		if (commandInfo.subcommand === "disable") return "Stream disabled";
+	}
+
+	if (commandInfo.command === "chat") return "Chat response";
+
 	if (commandInfo.command === "console") {
 		const messages = getArrayField(data, "messages");
 		if (messages) return `Console messages: ${messages.length}`;
@@ -417,6 +526,7 @@ function formatNativeSkillContent(content: string): string {
 
 function formatSkillsText(commandInfo: CommandInfo, data: unknown): string | undefined {
 	if (commandInfo.command !== "skills") return undefined;
+	if (commandInfo.subcommand === "path") return typeof data === "string" ? redactModelFacingText(data) : undefined;
 	if (commandInfo.subcommand === "list" && Array.isArray(data)) return formatSkillsListText(data);
 	const content = getSkillContent(data);
 	if (content) {
@@ -557,6 +667,15 @@ function formatDashboardText(data: Record<string, unknown>): string | undefined
 	return lines.length > 0 ? lines.join("\n") : undefined;
 }
 
+function formatChatText(data: Record<string, unknown>): string | undefined {
+	const response = getStringField(data, "response") ?? getStringField(data, "message") ?? getStringField(data, "text") ?? getStringField(data, "result");
+	if (response) return redactModelFacingText(response);
+	const model = getStringField(data, "model");
+	const provider = getStringField(data, "provider");
+	const lines = [model ? `Model: ${redactModelFacingText(model)}` : undefined, provider ? `Provider: ${redactModelFacingText(provider)}` : undefined].filter(Boolean);
+	return lines.length > 0 ? lines.join("\n") : undefined;
+}
+
 function formatDoctorText(data: Record<string, unknown>): string | undefined {
 	const lines: string[] = [];
 	const status = getStringField(data, "status") ?? getStringField(data, "result");
@@ -568,6 +687,123 @@ function formatDoctorText(data: Record<string, unknown>): string | undefined {
 	return lines.length > 0 ? lines.join("\n") : undefined;
 }
 
+function formatCookieRecordText(item: Record<string, unknown>, fallbackName: string): string {
+	const name = redactModelFacingText(getStringField(item, "name") ?? fallbackName);
+	const domain = getStringField(item, "domain");
+	const path = getStringField(item, "path");
+	const flags = [item.httpOnly === true ? "httpOnly" : undefined, item.secure === true ? "secure" : undefined].filter(Boolean).join(", ");
+	const location = [domain, path].filter(Boolean).join("");
+	return [name, location ? `(${redactModelFacingText(location)})` : undefined, flags ? `[${flags}]` : undefined].filter(Boolean).join(" ");
+}
+
+function formatCookiesText(data: Record<string, unknown>): string | undefined {
+	const cookies = getArrayField(data, "cookies");
+	if (cookies) {
+		if (cookies.length === 0) return "No cookies.";
+		return cookies
+			.map((item, index) => (isRecord(item) ? formatCookieRecordText(item, `(cookie ${index + 1})`) : `${index + 1}. [REDACTED]`))
+			.join("\n");
+	}
+	if (getStringField(data, "name") || getStringField(data, "domain") || getStringField(data, "path") || Object.hasOwn(data, "value")) {
+		return formatCookieRecordText(data, "cookie");
+	}
+	if (data.set === true) return "Cookie set.";
+	if (data.cleared === true || data.clear === true) return "Cookies cleared.";
+	return undefined;
+}
+
+function formatStorageText(data: Record<string, unknown>): string | undefined {
+	const type = getStringField(data, "type") ?? getStringField(data, "storage") ?? "storage";
+	const entries = getArrayField(data, "entries") ?? getArrayField(data, "items");
+	if (entries) {
+		if (entries.length === 0) return `${type}: no entries.`;
+		return entries
+			.map((item, index) => {
+				if (!isRecord(item)) return `${index + 1}. [REDACTED]`;
+				const rawKey = getStringField(item, "key") ?? getStringField(item, "name") ?? `(entry ${index + 1})`;
+				const key = redactModelFacingText(rawKey);
+				return Object.hasOwn(item, "value") ? `${key}: [REDACTED]` : key;
+			})
+			.join("\n");
+	}
+	const key = getStringField(data, "key");
+	if (key && Object.hasOwn(data, "value")) return `${type} ${redactModelFacingText(key)}: [REDACTED]`;
+	if (key && data.set === true) return `${type} set: ${redactModelFacingText(key)}`;
+	if (data.cleared === true || data.clear === true) return `${type} cleared.`;
+	return undefined;
+}
+
+function formatDialogText(data: Record<string, unknown>): string | undefined {
+	const lines: string[] = [];
+	if (typeof data.open === "boolean") lines.push(data.open ? "Dialog open." : "No dialog open.");
+	const type = getStringField(data, "type");
+	if (type) lines.push(`Type: ${redactModelFacingText(type)}`);
+	const message = getStringField(data, "message");
+	if (message) lines.push(`Message: ${/(?:auth|authorization|bearer|cookie|pass(?:word)?|secret|session|token)/i.test(message) ? "[REDACTED]" : redactModelFacingText(message)}`);
+	if (data.accepted === true) lines.push("Accepted.");
+	if (data.dismissed === true) lines.push("Dismissed.");
+	return lines.length > 0 ? lines.join("\n") : undefined;
+}
+
+function formatFrameText(data: Record<string, unknown>): string | undefined {
+	const frame = getStringField(data, "frame") ?? getStringField(data, "name") ?? getStringField(data, "selector");
+	const url = getStringField(data, "url");
+	const title = getStringField(data, "title");
+	const lines = [frame ? `Frame: ${redactModelFacingText(frame)}` : undefined, title ? `Title: ${redactModelFacingText(title)}` : undefined, url ? `URL: ${redactModelFacingTextIfSensitive(url)}` : undefined].filter(Boolean);
+	return lines.length > 0 ? lines.join("\n") : undefined;
+}
+
+function formatStateText(data: Record<string, unknown>): string | undefined {
+	const states = getArrayField(data, "states") ?? getArrayField(data, "files");
+	if (states) {
+		if (states.length === 0) return "No saved states.";
+		return states
+			.map((item, index) => {
+				if (!isRecord(item)) return `${index + 1}. ${redactModelFacingTextIfSensitive(stringifyModelFacing(item))}`;
+				const name = getStringField(item, "name") ?? getStringField(item, "file") ?? getStringField(item, "path") ?? `(state ${index + 1})`;
+				const url = getStringField(item, "url");
+				return url ? `${index + 1}. ${redactModelFacingText(name)} — ${redactModelFacingTextIfSensitive(url)}` : `${index + 1}. ${redactModelFacingText(name)}`;
+			})
+			.join("\n");
+	}
+	if (data.loaded === true) return `State loaded: ${redactModelFacingText(getStringField(data, "path") ?? getStringField(data, "name") ?? "ok")}`;
+	if (data.cleared === true || data.clear === true) return "State cleared.";
+	return undefined;
+}
+
+function isSensitivePresentationField(key: string): boolean {
+	return /^(?:access(?:_|-)?token|api(?:_|-)?key|auth(?:orization)?|bearer|client(?:_|-)?secret|cookie|id(?:_|-)?token|pass(?:word)?|proxy(?:_|-)?authorization|refresh(?:_|-)?token|secret|session(?:_|-)?id|set(?:_|-)?cookie|sig(?:nature)?|token|x(?:_|-)?api(?:_|-)?key)$/i.test(key);
+}
+
+function redactStructuredPresentationValue(value: unknown): unknown {
+	if (typeof value === "string") return redactModelFacingTextIfSensitive(value);
+	if (Array.isArray(value)) return value.map((item) => redactStructuredPresentationValue(item));
+	if (!isRecord(value)) return value;
+	return Object.fromEntries(
+		Object.entries(value).map(([key, entryValue]) => [
+			key,
+			isSensitivePresentationField(key) ? "[REDACTED]" : redactStructuredPresentationValue(entryValue),
+		]),
+	);
+}
+
+function redactStatefulValues(value: unknown, sensitiveKeys: Set<string>): unknown {
+	if (Array.isArray(value)) return value.map((item) => redactStatefulValues(item, sensitiveKeys));
+	if (!isRecord(value)) return redactStructuredPresentationValue(value);
+	return Object.fromEntries(
+		Object.entries(value).map(([key, entryValue]) => [
+			key,
+			sensitiveKeys.has(key.toLowerCase()) ? "[REDACTED]" : redactStatefulValues(entryValue, sensitiveKeys),
+		]),
+	);
+}
+
+function redactPresentationData(commandInfo: CommandInfo, data: unknown): unknown {
+	if (commandInfo.command === "cookies") return redactStatefulValues(data, new Set(["value"]));
+	if (commandInfo.command === "storage") return redactStatefulValues(data, new Set(["value"]));
+	return redactStructuredPresentationValue(data);
+}
+
 function formatDiagnosticText(commandInfo: CommandInfo, data: Record<string, unknown>): string | undefined {
 	if (commandInfo.command === "session") return formatSessionText(data);
 	if (commandInfo.command === "profiles") {
@@ -579,8 +815,23 @@ function formatDiagnosticText(commandInfo: CommandInfo, data: Record<string, unk
 		if (profiles) return formatProfilesText(profiles, "auth profiles");
 		if (commandInfo.subcommand === "show") return formatAuthShowText(data);
 	}
+	if (commandInfo.command === "cookies") return formatCookiesText(data);
+	if (commandInfo.command === "storage") return formatStorageText(data);
+	if (commandInfo.command === "dialog") return formatDialogText(data);
+	if (commandInfo.command === "frame") return formatFrameText(data);
+	if (commandInfo.command === "state") return formatStateText(data);
 	if (commandInfo.command === "network" && commandInfo.subcommand === "requests") return formatNetworkRequestsText(data);
 	if (commandInfo.command === "network" && commandInfo.subcommand === "request") return formatNetworkRequestText(data);
+	if (commandInfo.command === "diff") return stringifyModelFacing(data);
+	if (commandInfo.command === "clipboard") {
+		const text = getStringField(data, "text") ?? getStringField(data, "value") ?? getStringField(data, "result");
+		if (text) return redactModelFacingText(text);
+	}
+	if (commandInfo.command === "stream") {
+		const streamSummary = getStreamSummary(data);
+		if (streamSummary) return streamSummary;
+	}
+	if (commandInfo.command === "chat") return formatChatText(data);
 	if (commandInfo.command === "console") return formatConsoleText(data);
 	if (commandInfo.command === "errors") return formatErrorsText(data);
 	if (commandInfo.command === "dashboard") return formatDashboardText(data);
@@ -627,7 +878,10 @@ const PATH_FIELD_CANDIDATES = [
 	"filePath",
 	"outputPath",
 	"downloadPath",
+	"diffPath",
 	"harPath",
+	"savedPath",
+	"statePath",
 	"tracePath",
 	"profilePath",
 	"videoPath",
@@ -647,9 +901,11 @@ const ARTIFACT_EXTENSION_TO_MEDIA_TYPE: Record<string, string> = {
 
 function getArtifactKind(commandInfo: CommandInfo): FileArtifactKind | undefined {
 	if (commandInfo.command === "screenshot") return "image";
+	if (commandInfo.command === "diff" && commandInfo.subcommand === "screenshot") return "image";
 	if (commandInfo.command === "pdf") return "pdf";
 	if (commandInfo.command === "download") return "download";
 	if (commandInfo.command === "wait" && commandInfo.subcommand === "--download") return "download";
+	if (commandInfo.command === "state" && commandInfo.subcommand === "save") return "file";
 	if (commandInfo.command === "trace") return "trace";
 	if (commandInfo.command === "profiler") return "profile";
 	if (commandInfo.command === "record") return "video";
@@ -773,16 +1029,124 @@ function isManifestFileArtifact(artifact: FileArtifactMetadata): boolean {
 	return !isRecordingStartArtifact(artifact);
 }
 
+function getArtifactVerificationEntry(artifact: FileArtifactMetadata): ArtifactVerificationEntry {
+	if (isRecordingStartArtifact(artifact)) {
+		return {
+			absolutePath: artifact.absolutePath,
+			exists: artifact.exists,
+			kind: artifact.kind,
+			limitation: "Recording output is pending until record stop completes.",
+			mediaType: artifact.mediaType,
+			path: artifact.path,
+			requestedPath: artifact.requestedPath,
+			retentionState: undefined,
+			sizeBytes: artifact.sizeBytes,
+			state: "pending",
+			status: artifact.status,
+			storageScope: undefined,
+		};
+	}
+	const state = artifact.exists === true
+		? "verified"
+		: artifact.exists === false
+			? "missing"
+			: "unverified";
+	return {
+		absolutePath: artifact.absolutePath,
+		exists: artifact.exists,
+		kind: artifact.kind,
+		limitation: state === "missing"
+			? "The wrapper did not find the reported artifact at absolutePath. Treat the path as unverified until recovered or regenerated."
+			: state === "unverified"
+				? "The wrapper could not prove local filesystem existence for this artifact."
+				: undefined,
+		mediaType: artifact.mediaType,
+		path: artifact.path,
+		requestedPath: artifact.requestedPath,
+		retentionState: artifact.exists === false ? "missing" : "live",
+		sizeBytes: artifact.sizeBytes,
+		state,
+		status: artifact.status,
+		storageScope: "explicit-path",
+	};
+}
+
+function getManifestVerificationEntry(entry: SessionArtifactManifestEntry): ArtifactVerificationEntry | undefined {
+	if (entry.storageScope === "explicit-path") return undefined;
+	const state = entry.retentionState === "live"
+		? "verified"
+		: entry.retentionState === "missing" || entry.retentionState === "evicted"
+			? "missing"
+			: "unverified";
+	return {
+		absolutePath: entry.absolutePath,
+		exists: entry.exists,
+		kind: entry.kind,
+		limitation: entry.retentionState === "ephemeral"
+			? "This spill file is process-temporary and may not survive reload or restart."
+			: entry.retentionState === "evicted"
+				? "This persisted spill file was evicted from the bounded session artifact store."
+				: undefined,
+		mediaType: entry.mediaType,
+		path: entry.path,
+		requestedPath: entry.requestedPath,
+		retentionState: entry.retentionState,
+		sizeBytes: entry.sizeBytes,
+		state,
+		storageScope: entry.storageScope,
+	};
+}
+
+function buildArtifactVerificationSummary(
+	artifacts: FileArtifactMetadata[],
+	manifest?: SessionArtifactManifest,
+	manifestPaths?: ReadonlySet<string>,
+): ArtifactVerificationSummary | undefined {
+	const entries = [
+		...artifacts.map(getArtifactVerificationEntry),
+		...(manifest?.entries.flatMap((entry) => {
+			if (manifestPaths && !manifestPaths.has(entry.path)) return [];
+			const verificationEntry = getManifestVerificationEntry(entry);
+			return verificationEntry ? [verificationEntry] : [];
+		}) ?? []),
+	];
+	if (entries.length === 0) return undefined;
+	const verifiedCount = entries.filter((entry) => entry.state === "verified").length;
+	const missingCount = entries.filter((entry) => entry.state === "missing").length;
+	const pendingCount = entries.filter((entry) => entry.state === "pending").length;
+	const unverifiedCount = entries.filter((entry) => entry.state === "unverified").length;
+	return {
+		artifacts: entries,
+		missingCount,
+		pendingCount,
+		unverifiedCount,
+		verified: entries.length > 0 && verifiedCount === entries.length,
+		verifiedCount,
+	};
+}
+
+function classifyPresentationSuccessCategory(options: {
+	artifactVerification?: ArtifactVerificationSummary;
+	artifacts?: FileArtifactMetadata[];
+	inspection?: boolean;
+	savedFile?: SavedFilePresentationDetails;
+}) {
+	if ((options.artifactVerification?.missingCount ?? 0) > 0 || (options.artifactVerification?.unverifiedCount ?? 0) > 0) {
+		return "artifact-unverified" as const;
+	}
+	return classifyAgentBrowserSuccessCategory(options);
+}
+
 function formatArtifactLabel(artifact: FileArtifactMetadata): string {
 	switch (artifact.kind) {
 		case "download":
 			return artifact.command === "wait" && artifact.subcommand === "--download" ? "Download completed" : "Downloaded file";
 		case "file":
-			return "Saved file";
+			return artifact.command === "state" ? "State file" : "Saved file";
 		case "har":
 			return "Saved HAR";
 		case "image":
-			return "Saved image";
+			return artifact.command === "diff" && artifact.subcommand === "screenshot" ? "Saved diff image" : "Saved image";
 		case "pdf":
 			return "Saved PDF";
 		case "profile":
@@ -951,12 +1315,71 @@ function getNavigationSummary(data: Record<string, unknown>): NavigationSummary
 	return isNavigationSummary(candidate) ? candidate : undefined;
 }
 
+function getTopLevelNavigationSummary(data: Record<string, unknown>): NavigationSummary | undefined {
+	return isNavigationSummary(data)
+		? {
+			title: typeof data.title === "string" ? data.title : undefined,
+			url: typeof data.url === "string" ? data.url : undefined,
+		}
+		: undefined;
+}
+
+function getNormalizedNavigationSummary(summary: NavigationSummary | undefined): { title?: string; url?: string } | undefined {
+	const title = typeof summary?.title === "string" && summary.title.trim().length > 0 ? summary.title.trim() : undefined;
+	const url = typeof summary?.url === "string" && summary.url.trim().length > 0 ? summary.url.trim() : undefined;
+	return title || url ? { title, url } : undefined;
+}
+
 function formatNavigationSummary(summary: NavigationSummary): string | undefined {
-	const title = typeof summary.title === "string" && summary.title.trim().length > 0 ? summary.title.trim() : undefined;
-	const url = typeof summary.url === "string" && summary.url.trim().length > 0 ? summary.url.trim() : undefined;
-	if (!title && !url) return undefined;
-	if (title && url) return `${title}\n${url}`;
-	return title ?? url;
+	const normalized = getNormalizedNavigationSummary(summary);
+	if (!normalized) return undefined;
+	if (normalized.title && normalized.url) return `${normalized.title}\n${normalized.url}`;
+	return normalized.title ?? normalized.url;
+}
+
+function isPageChangeSummaryCommand(command: string | undefined): boolean {
+	return command !== undefined && PAGE_CHANGE_SUMMARY_COMMANDS.has(command);
+}
+
+function buildPageChangeSummary(options: {
+	artifacts?: FileArtifactMetadata[];
+	commandInfo: CommandInfo;
+	data: unknown;
+	nextActions?: Array<{ id: string }>;
+	savedFilePath?: string;
+	summary: string;
+}): AgentBrowserPageChangeSummary | undefined {
+	const { artifacts, commandInfo, data, nextActions, savedFilePath } = options;
+	const artifactCount = artifacts?.length ?? 0;
+	const navigation = isRecord(data)
+		? getNormalizedNavigationSummary(getNavigationSummary(data) ?? (isPageChangeSummaryCommand(commandInfo.command) ? getTopLevelNavigationSummary(data) : undefined))
+		: undefined;
+	const confirmationRequired = detectConfirmationRequired(data) !== undefined;
+	if (!navigation && !confirmationRequired && artifactCount === 0 && !savedFilePath && !isPageChangeSummaryCommand(commandInfo.command)) {
+		return undefined;
+	}
+	const changeType: AgentBrowserPageChangeSummary["changeType"] = savedFilePath || artifactCount > 0
+		? "artifact"
+		: navigation
+			? "navigation"
+			: confirmationRequired
+				? "confirmation"
+				: "mutation";
+	const parts = [commandInfo.command ?? "agent-browser", changeType];
+	if (navigation?.title) parts.push(navigation.title);
+	if (navigation?.url) parts.push(navigation.url);
+	if (savedFilePath) parts.push(savedFilePath);
+	else if (artifactCount > 0) parts.push(`${artifactCount} artifact${artifactCount === 1 ? "" : "s"}`);
+	return {
+		...(artifactCount > 0 ? { artifactCount } : {}),
+		changeType,
+		...(commandInfo.command ? { command: commandInfo.command } : {}),
+		...(nextActions ? { nextActionIds: nextActions.map((action) => action.id) } : {}),
+		...(savedFilePath ? { savedFilePath } : {}),
+		summary: parts.join(" → "),
+		...(navigation?.title ? { title: navigation.title } : {}),
+		...(navigation?.url ? { url: navigation.url } : {}),
+	};
 }
 
 function stripNavigationSummary(data: Record<string, unknown>): Record<string, unknown> {
@@ -1083,6 +1506,36 @@ function getBatchFailureDetails(steps: Array<{ details: BatchStepPresentationDet
 	};
 }
 
+function hasModelFacingArgRedaction(args: string[] | undefined): boolean {
+	return args?.some((arg) => arg === "[REDACTED]" || arg.includes("%5BREDACTED%5D") || arg.includes("[REDACTED]")) === true;
+}
+
+function getStatefulCommandSensitiveValues(command: string[] | undefined): string[] {
+	if (!command) return [];
+	const tokens = extractCommandTokens(command);
+	const values: string[] = [];
+	if (tokens[0] === "cookies" && tokens[1] === "set" && tokens[3]) values.push(tokens[3]);
+	if (tokens[0] === "storage" && ["local", "session"].includes(tokens[1] ?? "") && tokens[2] === "set" && tokens[4]) values.push(tokens[4]);
+	for (let index = 0; index < tokens.length; index += 1) {
+		const token = tokens[index];
+		if (token === "--password" && tokens[index + 1]) values.push(tokens[index + 1]);
+		else if (token?.startsWith("--password=")) values.push(token.slice("--password=".length));
+	}
+	return values.filter((value) => value.length > 0);
+}
+
+function redactExactValues(value: unknown, sensitiveValues: string[]): unknown {
+	if (sensitiveValues.length === 0) return redactSensitiveValue(value);
+	if (typeof value === "string") {
+		let redacted = value;
+		for (const sensitiveValue of sensitiveValues) redacted = redacted.split(sensitiveValue).join("[REDACTED]");
+		return redactSensitiveText(redacted);
+	}
+	if (Array.isArray(value)) return value.map((item) => redactExactValues(item, sensitiveValues));
+	if (!isRecord(value)) return value;
+	return redactSensitiveValue(Object.fromEntries(Object.entries(value).map(([key, entryValue]) => [key, redactExactValues(entryValue, sensitiveValues)])));
+}
+
 async function buildBatchStepPresentation(options: {
 	artifactManifest?: SessionArtifactManifest;
 	artifactRequest?: ArtifactRequestContext;
@@ -1094,21 +1547,43 @@ async function buildBatchStepPresentation(options: {
 }): Promise<{ details: BatchStepPresentationDetails; presentation: ToolPresentation }> {
 	const { artifactManifest, artifactRequest, cwd, index, item, persistentArtifactStore, sessionName } = options;
 	const command = isStringArray(item.command) ? item.command : undefined;
-	const commandText = formatBatchStepCommand(command, index);
+	const redactedCommand = command ? redactInvocationArgs(command) : undefined;
+	const commandText = formatBatchStepCommand(hasModelFacingArgRedaction(redactedCommand) ? redactedCommand : command, index);
 
 	if (item.success === false) {
-		const errorText = formatBatchStepError(item.error);
+		const redactedErrorData = redactExactValues(item.error, getStatefulCommandSensitiveValues(command));
+		const errorText = formatBatchStepError(redactedErrorData);
+		const failureCategory = classifyAgentBrowserFailureCategory({
+			args: command,
+			command: command?.[0],
+			errorText,
+		});
+		const confirmationRequired = detectConfirmationRequired(item.error);
+		const nextActions = buildAgentBrowserNextActions({
+			args: command,
+			command: command?.[0],
+			confirmationId: confirmationRequired?.id,
+			failureCategory,
+			resultCategory: "failure",
+		});
 		const presentation: ToolPresentation = {
 			content: [{ type: "text", text: errorText }],
+			failureCategory,
+			nextActions,
+			resultCategory: "failure",
 			summary: errorText,
 		};
 		return {
 			details: {
+				artifactVerification: presentation.artifactVerification,
 				artifacts: presentation.artifacts,
-				command,
+				command: redactedCommand,
 				commandText,
-				data: item.error,
+				data: redactedErrorData,
+				failureCategory,
 				index,
+				nextActions,
+				resultCategory: "failure",
 				success: false,
 				summary: errorText,
 				text: errorText,
@@ -1122,6 +1597,7 @@ async function buildBatchStepPresentation(options: {
 		artifactRequest,
 		commandInfo: parseCommandInfo(command ?? []),
 		cwd,
+		args: command,
 		envelope: { data: item.result, success: true },
 		persistentArtifactStore,
 		sessionName,
@@ -1135,11 +1611,28 @@ async function buildBatchStepPresentation(options: {
 		secondaryPaths: presentation.imagePaths,
 	});
 	const text = getPresentationText(presentation) || presentation.summary;
+	const nextActions = buildAgentBrowserNextActions({
+		artifacts: presentation.artifacts,
+		args: command,
+		command: command?.[0],
+		resultCategory: "success",
+		savedFilePath: presentation.savedFilePath,
+		successCategory: presentation.successCategory,
+	});
+	const pageChangeSummary = buildPageChangeSummary({
+		artifacts: presentation.artifacts,
+		commandInfo: parseCommandInfo(command ?? []),
+		data: presentation.data,
+		nextActions,
+		savedFilePath: presentation.savedFilePath,
+		summary: presentation.summary,
+	});
 
 	return {
 		details: {
+			artifactVerification: presentation.artifactVerification,
 			artifacts: presentation.artifacts,
-			command,
+			command: redactedCommand,
 			commandText,
 			data: presentation.data,
 			fullOutputPath: fullOutputPaths[0],
@@ -1147,9 +1640,13 @@ async function buildBatchStepPresentation(options: {
 			imagePath: imagePaths[0],
 			imagePaths: imagePaths.length > 0 ? imagePaths : undefined,
 			index,
+			nextActions,
+			pageChangeSummary,
+			resultCategory: "success",
 			savedFile: presentation.savedFile,
 			savedFilePath: presentation.savedFilePath,
 			success: true,
+			successCategory: classifyPresentationSuccessCategory({ artifactVerification: presentation.artifactVerification, artifacts: presentation.artifacts, savedFile: presentation.savedFile }),
 			summary: presentation.summary,
 			text,
 		},
@@ -1195,6 +1692,7 @@ async function buildBatchPresentation(options: {
 	const batchFailure = getBatchFailureDetails(steps);
 	const images = steps.flatMap((step) => getPresentationImages(step.presentation));
 	const artifacts = steps.flatMap((step) => step.presentation.artifacts ?? []);
+	const artifactVerification = buildArtifactVerificationSummary(artifacts);
 	const fullOutputPaths = steps.flatMap((step) => getPresentationPaths({
 		primaryPath: step.presentation.fullOutputPath,
 		secondaryPaths: step.presentation.fullOutputPaths,
@@ -1203,6 +1701,11 @@ async function buildBatchPresentation(options: {
 		primaryPath: step.presentation.imagePath,
 		secondaryPaths: step.presentation.imagePaths,
 	}));
+	const redactedBatchData = steps.map(({ details }) => (
+		details.success
+			? { command: details.command, result: details.data, success: true }
+			: { command: details.command, error: details.text, success: false }
+	));
 	const stepText =
 		steps.length === 0
 			? "(no batch steps)"
@@ -1235,18 +1738,45 @@ async function buildBatchPresentation(options: {
 	const artifactRetentionSummary = currentArtifactManifest ? formatSessionArtifactRetentionSummary(currentArtifactManifest) : undefined;
 	const contentText = artifactRetentionSummary && manifestHasNewNoticeWorthyEntries(options.artifactManifest, currentArtifactManifest) ? `${text}\n\n${artifactRetentionSummary}` : text;
 
+	const nextActions = batchFailure
+		? batchFailure.failedStep.nextActions
+		: buildAgentBrowserNextActions({ artifacts, command: "batch", resultCategory: "success" });
+	const changedSteps = steps.map((step) => step.details).filter((details) => details.pageChangeSummary !== undefined);
+	const pageChangeSummary = artifacts.length > 0
+		? buildPageChangeSummary({
+			artifacts,
+			commandInfo: { command: "batch" },
+			data,
+			nextActions,
+			summary,
+		})
+		: changedSteps.length > 0
+			? {
+				changeType: "mutation" as const,
+				command: "batch",
+				nextActionIds: nextActions?.map((action) => action.id),
+				summary: `batch → mutation → ${changedSteps.length} changed step${changedSteps.length === 1 ? "" : "s"}`,
+			}
+			: undefined;
+
 	return {
 		artifactManifest: currentArtifactManifest,
 		artifactRetentionSummary,
+		artifactVerification,
 		artifacts: artifacts.length > 0 ? artifacts : undefined,
 		batchFailure,
 		batchSteps: steps.map((step) => step.details),
 		content: [{ type: "text", text: contentText }, ...images],
-		data,
+		failureCategory: batchFailure?.failedStep.failureCategory,
+		data: redactedBatchData,
 		fullOutputPath: fullOutputPaths[0],
 		fullOutputPaths: fullOutputPaths.length > 0 ? fullOutputPaths : undefined,
 		imagePath: imagePaths[0],
 		imagePaths: imagePaths.length > 0 ? imagePaths : undefined,
+		nextActions,
+		pageChangeSummary,
+		resultCategory: batchFailure ? "failure" : "success",
+		successCategory: batchFailure ? undefined : classifyPresentationSuccessCategory({ artifactVerification, artifacts }),
 		summary,
 	};
 }
@@ -1270,6 +1800,9 @@ function formatSummary(commandInfo: CommandInfo, data: unknown): string {
 	if (commandInfo.command === "skills" && commandInfo.subcommand === "get") {
 		return "agent-browser skill loaded";
 	}
+	if (commandInfo.command === "skills" && commandInfo.subcommand === "path") {
+		return "agent-browser skill path";
+	}
 	if (isRecord(data)) {
 		const navigationSummary = getNavigationSummary(data);
 		if (navigationSummary && isNavigationObservableCommand(commandInfo.command)) {
@@ -1319,6 +1852,10 @@ function formatContentText(commandInfo: CommandInfo, data: unknown): string {
 		return formatConfirmationRequiredText(confirmationRequired);
 	}
 
+	const skillsText = formatSkillsText(commandInfo, data);
+	if (skillsText) {
+		return skillsText;
+	}
 	if (typeof data === "string") {
 		return redactModelFacingText(data);
 	}
@@ -1328,9 +1865,6 @@ function formatContentText(commandInfo: CommandInfo, data: unknown): string {
 	if (Array.isArray(data) && commandInfo.command === "profiles") {
 		return formatProfilesText(data, "Chrome profiles");
 	}
-	if (Array.isArray(data) && commandInfo.command === "skills") {
-		return formatSkillsText(commandInfo, data) ?? stringifyModelFacing(data);
-	}
 	if (!isRecord(data)) {
 		return stringifyModelFacing(data);
 	}
@@ -1359,10 +1893,6 @@ function formatContentText(commandInfo: CommandInfo, data: unknown): string {
 		const screenshotSummary = getScreenshotSummary(data);
 		if (screenshotSummary) return screenshotSummary;
 	}
-	const skillsText = formatSkillsText(commandInfo, data);
-	if (skillsText) {
-		return skillsText;
-	}
 	const extractionText = formatExtractionText(commandInfo, data);
 	if (extractionText) {
 		return extractionText;
@@ -1595,6 +2125,7 @@ async function compactLargePresentationOutput(options: {
 
 export async function buildToolPresentation(options: {
 	artifactManifest?: SessionArtifactManifest;
+	args?: string[];
 	artifactRequest?: ArtifactRequestContext;
 	batchArtifactRequests?: Array<ArtifactRequestContext | undefined>;
 	commandInfo: CommandInfo;
@@ -1604,17 +2135,22 @@ export async function buildToolPresentation(options: {
 	persistentArtifactStore?: PersistentSessionArtifactStore;
 	sessionName?: string;
 }): Promise<ToolPresentation> {
-	const { artifactManifest, artifactRequest, commandInfo, cwd, envelope, errorText, persistentArtifactStore, sessionName } = options;
+	const { args, artifactManifest, artifactRequest, commandInfo, cwd, envelope, errorText, persistentArtifactStore, sessionName } = options;
 	if (errorText) {
 		const hintedErrorText = appendSelectorRecoveryHint(redactModelFacingText(errorText));
+		const categoryDetails = buildAgentBrowserResultCategoryDetails({ args: [commandInfo.command, commandInfo.subcommand].filter((item): item is string => item !== undefined), command: commandInfo.command, errorText: hintedErrorText, succeeded: false });
 		return {
+			...categoryDetails,
 			content: [{ type: "text", text: hintedErrorText }],
+			nextActions: buildAgentBrowserNextActions({ args, command: commandInfo.command, failureCategory: categoryDetails.failureCategory, resultCategory: "failure" }),
 			summary: hintedErrorText,
 		};
 	}
 
 	const data = enrichStreamStatusData(commandInfo, envelope?.data);
+	const presentationData = redactPresentationData(commandInfo, data);
 	const artifacts = await extractFileArtifacts({ artifactRequest, commandInfo, cwd, data, sessionName });
+	const artifactVerification = buildArtifactVerificationSummary(artifacts);
 	const artifactSummary = formatArtifactSummary(artifacts);
 	const summary = artifactSummary ?? formatSummary(commandInfo, data);
 	const artifactText = artifacts.length > 0 ? formatArtifactMetadataLines(artifacts).join("\n") : undefined;
@@ -1624,14 +2160,16 @@ export async function buildToolPresentation(options: {
 			: commandInfo.command === "snapshot" && isRecord(data)
 				? await buildSnapshotPresentation(data, persistentArtifactStore, artifactManifest)
 				: {
+						artifactVerification,
 						artifacts: artifacts.length > 0 ? artifacts : undefined,
 						content: [{ type: "text" as const, text: artifactText ?? formatContentText(commandInfo, data) }],
-						data,
+						data: presentationData,
 						summary,
 				  };
 	if (artifacts.length > 0 && !presentation.artifacts) {
 		presentation.artifacts = artifacts;
 	}
+	presentation.artifactVerification = presentation.artifactVerification ?? artifactVerification;
 	if (isRecord(data)) {
 		const savedFile = getSavedFileDetails(commandInfo, data);
 		if (savedFile) {
@@ -1645,15 +2183,68 @@ export async function buildToolPresentation(options: {
 	const compactedPresentation = await compactLargePresentationOutput({
 		artifactManifest,
 		commandInfo,
-		data,
+		data: presentationData,
 		persistentArtifactStore,
 		presentation: presentationWithImage,
 	});
-	return sanitizeModelFacingPresentation(
-		applyArtifactManifest(
-			compactedPresentation,
-			compactedPresentation.artifactManifest ?? artifactManifest,
-			buildManifestEntriesForFileArtifacts(artifacts.filter(isManifestFileArtifact)),
-		),
+	const presentationWithManifest = applyArtifactManifest(
+		compactedPresentation,
+		compactedPresentation.artifactManifest ?? artifactManifest,
+		buildManifestEntriesForFileArtifacts(artifacts.filter(isManifestFileArtifact)),
 	);
+	const currentSpillPaths = new Set(getPresentationPaths({
+		primaryPath: presentationWithManifest.fullOutputPath,
+		secondaryPaths: presentationWithManifest.fullOutputPaths,
+	}));
+	presentationWithManifest.artifactVerification = buildArtifactVerificationSummary(
+		artifacts,
+		presentationWithManifest.artifactManifest,
+		currentSpillPaths,
+	) ?? presentationWithManifest.artifactVerification;
+	const confirmationRequired = detectConfirmationRequired(data);
+	if (!presentationWithManifest.resultCategory) {
+		const categoryDetails = buildAgentBrowserResultCategoryDetails({
+			artifacts: presentationWithManifest.artifacts,
+			command: commandInfo.command,
+			confirmationRequired: confirmationRequired !== undefined,
+			errorText: envelope?.success === false ? presentationWithManifest.summary : undefined,
+			savedFile: presentationWithManifest.savedFile,
+			succeeded: envelope?.success !== false,
+		});
+		presentationWithManifest.resultCategory = categoryDetails.resultCategory;
+		presentationWithManifest.successCategory = categoryDetails.resultCategory === "success"
+			? classifyPresentationSuccessCategory({
+				artifactVerification: presentationWithManifest.artifactVerification,
+				artifacts: presentationWithManifest.artifacts,
+				savedFile: presentationWithManifest.savedFile,
+			})
+			: categoryDetails.successCategory;
+		presentationWithManifest.failureCategory = categoryDetails.failureCategory;
+	}
+	if (presentationWithManifest.resultCategory === "success") {
+		presentationWithManifest.successCategory = classifyPresentationSuccessCategory({
+			artifactVerification: presentationWithManifest.artifactVerification,
+			artifacts: presentationWithManifest.artifacts,
+			savedFile: presentationWithManifest.savedFile,
+		});
+	}
+	presentationWithManifest.nextActions = presentationWithManifest.nextActions ?? buildAgentBrowserNextActions({
+		artifacts: presentationWithManifest.artifacts,
+		args,
+		command: commandInfo.command,
+		confirmationId: confirmationRequired?.id,
+		failureCategory: presentationWithManifest.failureCategory,
+		resultCategory: presentationWithManifest.resultCategory ?? "success",
+		savedFilePath: presentationWithManifest.savedFilePath,
+		successCategory: presentationWithManifest.successCategory,
+	});
+	presentationWithManifest.pageChangeSummary = presentationWithManifest.pageChangeSummary ?? buildPageChangeSummary({
+		artifacts: presentationWithManifest.artifacts,
+		commandInfo,
+		data,
+		nextActions: presentationWithManifest.nextActions,
+		savedFilePath: presentationWithManifest.savedFilePath,
+		summary: presentationWithManifest.summary,
+	});
+	return sanitizeModelFacingPresentation(presentationWithManifest);
 }