@browserbridge/bbx 1.8.0 → 1.10.0

package/packages/native-host/src/daemon.js

@@ -29,6 +29,9 @@ import {
   getSupportedProtocolVersions,
   MAX_DAEMON_PENDING_TIMEOUT_MS,
   parseJsonLines,
+  RecoveryTelemetryCollector,
+  normalizeRecoveryTelemetrySummary,
+  sanitizeIncidentalValue,
   setProtocolPackageVersion,
   validateBridgeRequest,
 } from '../../protocol/src/index.js';
@@ -37,10 +40,13 @@ import {
   formatBridgeTransport,
   getBridgeListenTarget,
   getBridgeTransport,
+  getArtifactStorePath,
   getSocketPath,
 } from './config.js';
+import { ArtifactStore } from './artifact-store.js';
 import { bridgeAuthTokensEqual, ensureBridgeAuthToken, readBridgeAuthToken } from './auth-token.js';
 import { normalizeDaemonLogger } from './daemon-logger.js';
+import { DomBaselineOwnerRegistry, isValidBaselineId } from './dom-baseline-owners.js';
 import { writeJsonLine } from './framing.js';
 
 const DAEMON_VERSION = loadDaemonVersion();
@@ -54,7 +60,7 @@ setProtocolPackageVersion(DAEMON_VERSION);
 /** @typedef {import('./daemon-logger.js').DaemonLoggerLike} DaemonLoggerLike */
 /** @typedef {'agent' | 'extension'} SocketRole */
 /** @typedef {import('node:net').Socket & { readonly __role?: SocketRole, __clientId?: string, __extensionId?: string, __browserName?: string, __profileLabel?: string, __browserExtensionId?: string, __accessEnabled?: boolean, __lastActiveAt?: number }} ClientSocket */
-/** @typedef {{ socket: ClientSocket, timeoutId: NodeJS.Timeout, source?: string, method?: string, protocolVersion?: string, targets: Set<ClientSocket>, lastErrorResponse?: import('../../protocol/src/types.js').BridgeResponse }} PendingEntry */
+/** @typedef {{ socket: ClientSocket, timeoutId: NodeJS.Timeout, source?: string, method?: string, protocolVersion?: string, baselineId?: string | null, automaticMcpRetry?: boolean, targets: Set<ClientSocket>, lastErrorResponse?: import('../../protocol/src/types.js').BridgeResponse }} PendingEntry */
 /**
  * @typedef {{
  *   installAgentFiles: typeof import('../../agent-client/src/install.js').installAgentFiles,
@@ -110,6 +116,13 @@ const HEALTH_DEBUGGER_REASONS = new Set([
   'target_closed',
 ]);
 const HEALTH_CAPTURE_STATES = new Set(['stopped', 'armed', 'active', 'stop_failed']);
+const DIAGNOSTIC_METHODS = new Set([
+  'health.ping',
+  'daemon.metrics',
+  'log.tail',
+  'setup.get_status',
+  'setup.install',
+]);
 
 /**
  * @param {unknown} value
@@ -146,6 +159,12 @@ function normalizeExtensionHealthResult(value) {
   /** @type {Record<string, unknown>} */
   const normalized = {};
   if (result.extension === 'ok') normalized.extension = 'ok';
+  if (
+    typeof result.extensionVersion === 'string' &&
+    /^\d+\.\d+\.\d+$/u.test(result.extensionVersion)
+  ) {
+    normalized.extensionVersion = result.extensionVersion;
+  }
 
   const access = asHealthRecord(result.access);
   if (access) {
@@ -216,6 +235,49 @@ function normalizeExtensionHealthResult(value) {
     normalized.capture = normalizedCapture;
   }
 
+  const domBaselines = asHealthRecord(result.domBaselines);
+  if (domBaselines) {
+    /** @type {Record<string, unknown>} */
+    const normalizedBaselines = {};
+    for (const key of ['baselineCount', 'bytes', 'tabCount']) {
+      const count = normalizeHealthCount(domBaselines[key]);
+      if (count !== undefined) normalizedBaselines[key] = count;
+    }
+    const operationSource = asHealthRecord(domBaselines.operations);
+    if (operationSource) {
+      /** @type {Record<string, unknown>} */
+      const operations = {};
+      for (const name of [
+        'create',
+        'get',
+        'compare',
+        'release',
+        'invalidate',
+        'clearTab',
+        'clearWindow',
+        'clearAll',
+        'invalidateNavigation',
+        'pruneExpired',
+      ]) {
+        const source = asHealthRecord(operationSource[name]);
+        if (!source) continue;
+        const calls = normalizeHealthCount(source.calls);
+        const totalLatencyMs = normalizeHealthCount(source.totalLatencyMs);
+        const maxLatencyMs = normalizeHealthCount(source.maxLatencyMs);
+        operations[name] = {
+          ...(calls === undefined ? {} : { calls }),
+          ...(totalLatencyMs === undefined ? {} : { totalLatencyMs }),
+          ...(maxLatencyMs === undefined ? {} : { maxLatencyMs }),
+        };
+      }
+      normalizedBaselines.operations = operations;
+    }
+    normalized.domBaselines = normalizedBaselines;
+  }
+
+  const recovery = normalizeRecoveryTelemetrySummary(result.recovery, 'routedExtension');
+  if (recovery) normalized.recovery = recovery;
+
   const extensionSupportedVersions = Array.isArray(result.supported_versions)
     ? result.supported_versions
         .filter(
@@ -230,6 +292,19 @@ function normalizeExtensionHealthResult(value) {
   return normalized;
 }
 
+/**
+ * @param {import('../../protocol/src/types.js').RecoveryTelemetrySummary} daemon
+ * @param {unknown} extensionResult
+ */
+function createHealthRecoveryPayload(daemon, extensionResult) {
+  const source = asHealthRecord(extensionResult);
+  const routedExtension = normalizeRecoveryTelemetrySummary(source?.recovery, 'routedExtension');
+  return {
+    daemon,
+    ...(routedExtension ? { routedExtension } : {}),
+  };
+}
+
 /**
  * @param {string | undefined} requestedVersion
  * @returns {{ supported_versions: readonly string[], deprecated_since?: string, migration_hint?: string }}
@@ -289,9 +364,15 @@ export function isWindowsNamedPipePath(socketPath) {
  *   accessEnabled?: boolean,
  *   authToken?: string,
  *   at?: number
+ *   artifactId?: string,
+ *   artifact?: Record<string, unknown>,
+ *   chunkIndex?: number,
+ *   data?: string
  * }} DaemonMessage
  */
 
+/** @typedef {import('../../protocol/src/types.js').ArtifactKind} ArtifactKind */
+
 export class BridgeDaemon {
   /**
    * @param {{
@@ -301,7 +382,8 @@ export class BridgeDaemon {
    *   setupStatusLoader?: () => Promise<SetupStatus>,
    *   setupInstaller?: (params: Record<string, unknown>) => Promise<SetupInstallResult>,
    *   logger?: DaemonLoggerLike | Pick<Console, 'log' | 'error'>,
-   *   authToken?: string | null
+   *   authToken?: string | null,
+   *   artifactStore?: ArtifactStore
    * }} [options={}]
    */
   constructor({
@@ -312,6 +394,7 @@ export class BridgeDaemon {
     setupInstaller = installSetupTarget,
     logger = undefined,
     authToken = undefined,
+    artifactStore = new ArtifactStore(getArtifactStorePath()),
   } = {}) {
     this.transport = socketPath ? createSocketBridgeTransport(socketPath) : transport;
     this.socketPath =
@@ -354,6 +437,13 @@ export class BridgeDaemon {
     this.requestStartTimes = new Map();
     /** @type {string | null | undefined} */
     this.authToken = authToken;
+    this.artifactStore = artifactStore;
+    this.artifactStoreInitialized = false;
+    this.domBaselines = new DomBaselineOwnerRegistry({
+      isCurrentExtensionSocket: (socket) =>
+        this.extensionSockets.get(socket.__extensionId ?? '') === socket,
+    });
+    this.recoveryTelemetry = new RecoveryTelemetryCollector();
   }
 
   /**
@@ -421,11 +511,17 @@ export class BridgeDaemon {
    * @param {PendingEntry | undefined} [pending]
    * @returns {PendingEntry | undefined}
    */
-  clearPendingRequest(requestId, pending = this.pendingRequests.get(requestId)) {
+  clearPendingRequest(
+    requestId,
+    pending = this.pendingRequests.get(requestId),
+    preserveArtifacts = false
+  ) {
     if (!pending) {
       return undefined;
     }
     clearTimeout(pending.timeoutId);
+    if (preserveArtifacts) this.artifactStore.abortRequest(requestId);
+    else this.artifactStore.deleteRequest(requestId);
     this.pendingRequests.delete(requestId);
     this.removePendingRequestIndex(this.pendingRequestsByOwnerSocket, pending.socket, requestId);
     for (const targetSocket of pending.targets) {
@@ -618,6 +714,9 @@ export class BridgeDaemon {
       await fs.promises.chmod(this.socketPath, 0o600);
     }
 
+    this.artifactStore.reset();
+    this.artifactStoreInitialized = true;
+
     this.logger.info('Daemon listening', {
       transport: formatBridgeTransport(this.transport),
       socketPath: this.socketPath ?? null,
@@ -665,6 +764,11 @@ export class BridgeDaemon {
     }
     this.extensionSockets.clear();
     this.invalidateConnectedExtensionsCache();
+    this.domBaselines.clear();
+    if (this.artifactStoreInitialized) {
+      this.artifactStore.reset();
+      this.artifactStoreInitialized = false;
+    }
 
     if (this.server) {
       const server = this.server;
@@ -737,6 +841,13 @@ export class BridgeDaemon {
       return this.handleExtensionActivity(socket, message);
     }
 
+    if (message?.type?.startsWith('extension.artifact.')) {
+      if (socket.__role !== 'extension') {
+        return this.rejectMessageForRole(socket, message);
+      }
+      return this.handleExtensionArtifact(socket, message);
+    }
+
     if (message?.type === 'extension.setup_status.request') {
       if (socket.__role !== 'extension') {
         return this.rejectMessageForRole(socket, message);
@@ -745,11 +856,12 @@ export class BridgeDaemon {
     }
 
     if (message?.type === 'agent.request') {
-      const isExtensionSetupRequest =
+      const isExtensionLocalRequest =
         socket.__role === 'extension' &&
-        (message.request?.method === 'setup.get_status' ||
+        (message.request?.method === 'health.ping' ||
+          message.request?.method === 'setup.get_status' ||
           message.request?.method === 'setup.install');
-      if (socket.__role !== 'agent' && !isExtensionSetupRequest) {
+      if (socket.__role !== 'agent' && !isExtensionLocalRequest) {
         return this.rejectMessageForRole(socket, message);
       }
       return this.handleAgentRequest(socket, message);
@@ -848,6 +960,7 @@ export class BridgeDaemon {
         `Request id ${JSON.stringify(request.id)} is already in flight.`
       );
       await writeJsonLine(socket, { type: 'agent.response', response });
+      this.recordDaemonRequestOutcome(request, false);
       return;
     }
 
@@ -861,6 +974,9 @@ export class BridgeDaemon {
           transport: formatBridgeTransport(this.transport),
           proxy: this.getProxyStatusPayload(),
           connectedExtensions: [],
+          recovery: {
+            daemon: this.recoveryTelemetry.snapshot('daemon'),
+          },
           daemon_supported_versions: getSupportedProtocolVersions(),
           ...getVersionNegotiationPayload(request.meta?.protocol_version),
         });
@@ -880,6 +996,7 @@ export class BridgeDaemon {
     }
 
     if (request.method === 'daemon.metrics') {
+      this.domBaselines.prune();
       const now = Date.now();
       const uptimeMs = this.startedAt > 0 ? now - this.startedAt : 0;
       const avgResponseTimeMs =
@@ -894,11 +1011,62 @@ export class BridgeDaemon {
         requestsProcessed: this.requestsProcessed,
         requestsFailed: this.requestsFailed,
         avgResponseTimeMs,
+        domBaselineOwners: this.domBaselines.size,
+        recovery: this.recoveryTelemetry.snapshot('daemon'),
       });
       await writeJsonLine(socket, { type: 'agent.response', response });
       return;
     }
 
+    if (request.method === 'artifact.read' || request.method === 'artifact.delete') {
+      const ownerId = socket.__clientId;
+      if (!ownerId) {
+        const response = createFailure(
+          request.id,
+          ERROR_CODES.ACCESS_DENIED,
+          'Artifact access requires a registered agent client.',
+          null,
+          { method: request.method }
+        );
+        await writeJsonLine(socket, { type: 'agent.response', response });
+        this.recordDaemonRequestOutcome(request, false);
+        return;
+      }
+      try {
+        const result =
+          request.method === 'artifact.read'
+            ? this.artifactStore.read(
+                String(request.params.artifactId),
+                ownerId,
+                Number(request.params.offset),
+                Number(request.params.maxBytes)
+              )
+            : this.artifactStore.delete(String(request.params.artifactId), ownerId);
+        await writeJsonLine(socket, {
+          type: 'agent.response',
+          response: createSuccess(request.id, result, { method: request.method }),
+        });
+        this.recordDaemonRequestOutcome(request, true);
+      } catch (error) {
+        const code =
+          error && typeof error === 'object' && 'code' in error
+            ? /** @type {import('../../protocol/src/types.js').ErrorCode} */ (error.code)
+            : ERROR_CODES.INTERNAL_ERROR;
+        await writeJsonLine(socket, {
+          type: 'agent.response',
+          response: createFailure(
+            request.id,
+            code,
+            error instanceof Error ? error.message : String(error),
+            null,
+            { method: request.method }
+          ),
+        });
+        this.recordDaemonRequestOutcome(request, false);
+      }
+      return;
+    }
+
     if (request.method === 'setup.get_status') {
       const response = createSuccess(request.id, await this.setupStatusLoader(), {
         method: request.method,
@@ -947,17 +1115,48 @@ export class BridgeDaemon {
       typeof request.meta?.target_profile === 'string' ? request.meta.target_profile : null;
     const hasExplicitTarget = Boolean(targetBrowser || targetProfile);
 
-    const target = this.selectExtensionTarget(targetBrowser, targetProfile);
+    const baselineId =
+      typeof request.params.baselineId === 'string' ? request.params.baselineId : null;
+    const usesBaselineOwner =
+      request.method === 'dom.baseline.compare' ||
+      request.method === 'dom.baseline.describe' ||
+      request.method === 'dom.baseline.release';
+    const baselineOwner = usesBaselineOwner ? this.domBaselines.get(baselineId ?? '') : null;
+    const target = usesBaselineOwner
+      ? baselineOwner &&
+        (!targetBrowser || baselineOwner.__browserName === targetBrowser) &&
+        (!targetProfile || baselineOwner.__profileLabel === targetProfile)
+        ? baselineOwner
+        : null
+      : this.selectExtensionTarget(targetBrowser, targetProfile);
 
     if (!target) {
+      if (usesBaselineOwner) {
+        const response =
+          request.method === 'dom.baseline.release'
+            ? createSuccess(request.id, { baselineId, released: false }, { method: request.method })
+            : createFailure(
+                request.id,
+                ERROR_CODES.DOM_BASELINE_NOT_FOUND,
+                'DOM baseline is missing, expired, released, or its browser destination disconnected.',
+                { baselineId },
+                { method: request.method }
+              );
+        await writeJsonLine(socket, { type: 'agent.response', response });
+        this.recordDaemonRequestOutcome(request, response.ok);
+        return;
+      }
       const response = createFailure(
         request.id,
         ERROR_CODES.EXTENSION_DISCONNECTED,
         hasExplicitTarget
           ? `No connected extension matches target_browser="${targetBrowser ?? '*'}" target_profile="${targetProfile ?? '*'}".`
-          : 'The Chrome extension is not connected to the local bridge daemon.'
+          : 'The Chrome extension is not connected to the local bridge daemon.',
+        null,
+        { method: request.method }
       );
       await writeJsonLine(socket, { type: 'agent.response', response });
+      this.recordDaemonRequestOutcome(request, false);
       return;
     }
 
@@ -966,16 +1165,21 @@ export class BridgeDaemon {
       method: request.method,
       protocolVersion: request.meta?.protocol_version,
       source: typeof request.meta?.source === 'string' ? request.meta.source : '',
+      baselineId,
+      automaticMcpRetry: isAutomaticMcpRetry(request.meta),
       targets: new Set([target]),
       timeoutId: setTimeout(() => {
         const pending = this.pendingRequests.get(request.id);
         if (!pending) return;
+        this.domBaselines.markAbandonedCreate(request.id, pending);
         this.clearPendingRequest(request.id, pending);
-        this.recordRequestCompletion(request.id, false);
+        this.recordRequestCompletion(request.id, false, pending);
         const response = createFailure(
           request.id,
           ERROR_CODES.TIMEOUT,
-          'Extension did not respond in time.'
+          'Extension did not respond in time.',
+          null,
+          { method: pending.method }
         );
         void writeJsonLine(pending.socket, {
           type: 'agent.response',
@@ -1066,6 +1270,34 @@ export class BridgeDaemon {
     return candidates[0]?.[1] ?? null;
   }
 
+  /**
+   * @param {ClientSocket} socket
+   * @param {import('../../protocol/src/types.js').BridgeResponse} response
+   */
+  async releaseOrphanDomBaseline(socket, response) {
+    const result =
+      response.result && typeof response.result === 'object'
+        ? /** @type {Record<string, unknown>} */ (response.result)
+        : {};
+    const baselineId = typeof result.baselineId === 'string' ? result.baselineId : '';
+    if (!isValidBaselineId(baselineId)) return;
+    const scope =
+      result.scope && typeof result.scope === 'object'
+        ? /** @type {Record<string, unknown>} */ (result.scope)
+        : {};
+    const tabId = Number.isSafeInteger(scope.tabId) ? Number(scope.tabId) : null;
+    await writeJsonLine(socket, {
+      type: 'extension.request',
+      request: {
+        id: `orphan_release_${randomUUID().replaceAll('-', '')}`,
+        method: 'dom.baseline.release',
+        tab_id: tabId,
+        params: { baselineId },
+        meta: { protocol_version: getProtocolVersion(), token_budget: null },
+      },
+    });
+  }
+
   /**
    * @param {BridgeRequest} request
    * @returns {number}
@@ -1140,6 +1372,10 @@ export class BridgeDaemon {
    */
   handleExtensionAccessUpdate(socket, message) {
     const accessEnabled = Boolean(message.accessEnabled);
+    if (!accessEnabled && socket.__extensionId) {
+      this.artifactStore.deleteByExtension(socket.__extensionId);
+      this.domBaselines.clearForSocket(socket);
+    }
     if (socket.__accessEnabled !== accessEnabled) {
       socket.__accessEnabled = accessEnabled;
       this.invalidateConnectedExtensionsCache();
@@ -1148,6 +1384,58 @@ export class BridgeDaemon {
     socket.__accessEnabled = accessEnabled;
   }
 
+  /**
+   * @param {ClientSocket} socket
+   * @param {DaemonMessage} message
+   * @returns {void}
+   */
+  handleExtensionArtifact(socket, message) {
+    const artifact =
+      message.artifact && typeof message.artifact === 'object' ? message.artifact : {};
+    const requestId = typeof artifact.requestId === 'string' ? artifact.requestId : '';
+    const pending = this.pendingRequests.get(requestId);
+    if (!pending || !pending.targets.has(socket) || !socket.__extensionId) return;
+    const ownerId = pending.socket.__clientId;
+    if (!ownerId) return;
+    try {
+      if (message.type === 'extension.artifact.begin') {
+        this.artifactStore.begin({
+          artifactId: String(artifact.artifactId ?? ''),
+          requestId,
+          ownerId,
+          extensionId: socket.__extensionId,
+          kind: /** @type {ArtifactKind} */ (artifact.kind),
+          mimeType: String(artifact.mimeType ?? ''),
+          totalBytes: Number(artifact.byteLength),
+          sha256: String(artifact.sha256 ?? ''),
+          chunkCount: Number(artifact.chunkCount),
+          createdAt: String(artifact.createdAt ?? ''),
+          expiresAt: String(artifact.expiresAt ?? ''),
+        });
+      } else if (message.type === 'extension.artifact.chunk') {
+        this.artifactStore.writeChunk(
+          String(message.artifactId ?? ''),
+          Number(message.chunkIndex),
+          String(message.data ?? '')
+        );
+      } else if (message.type === 'extension.artifact.commit') {
+        this.artifactStore.commit(String(message.artifactId ?? ''));
+      }
+    } catch (error) {
+      const code =
+        error && typeof error === 'object' && 'code' in error
+          ? /** @type {import('../../protocol/src/types.js').ErrorCode} */ (error.code)
+          : ERROR_CODES.ARTIFACT_TRANSFER_INVALID;
+      pending.lastErrorResponse = createFailure(
+        requestId,
+        code,
+        error instanceof Error ? error.message : String(error),
+        null,
+        { method: pending.method }
+      );
+    }
+  }
+
   /**
    * Structured remote-exposure status for health payloads. The proxy counts
    * as enabled only when the daemon listens on a TCP address other than
@@ -1188,22 +1476,83 @@ export class BridgeDaemon {
     }
 
     const pending = this.pendingRequests.get(responseMessage.id);
-    if (!pending || !pending.targets.has(socket)) {
+    if (!pending) {
+      if (this.domBaselines.takeAbandonedCreate(responseMessage.id, socket)) {
+        if (responseMessage.ok) {
+          await this.releaseOrphanDomBaseline(socket, responseMessage).catch((error) => {
+            this.logger.error('orphan DOM baseline release failed', {
+              requestId: responseMessage.id,
+              message: error instanceof Error ? error.message : String(error),
+            });
+          });
+        }
+      }
+      return;
+    }
+    if (!pending.targets.has(socket)) {
       return;
     }
 
     this.removePendingTarget(responseMessage.id, pending, socket);
 
     if (responseMessage.ok) {
-      this.clearPendingRequest(responseMessage.id, pending);
-      this.recordRequestCompletion(responseMessage.id, true);
-      this.pushLog({
-        at: new Date().toISOString(),
-        method: responseMessage.meta?.method ?? null,
-        ok: true,
-        id: responseMessage.id,
-        source: pending.source || null,
-      });
+      const result =
+        responseMessage.result && typeof responseMessage.result === 'object'
+          ? /** @type {Record<string, unknown>} */ (responseMessage.result)
+          : {};
+      const artifact =
+        result.artifact && typeof result.artifact === 'object'
+          ? /** @type {Record<string, unknown>} */ (result.artifact)
+          : null;
+      const expectedArtifactKind = getArtifactKindForMethod(pending.method);
+      if (
+        result.delivery === 'artifact' &&
+        (!artifact ||
+          artifact.kind !== expectedArtifactKind ||
+          !pending.socket.__clientId ||
+          !this.artifactStore.matchesCommitted(
+            String(artifact.artifactId ?? ''),
+            pending.socket.__clientId,
+            responseMessage.id,
+            artifact,
+            expectedArtifactKind ?? undefined
+          ))
+      ) {
+        pending.lastErrorResponse ??= createFailure(
+          responseMessage.id,
+          ERROR_CODES.ARTIFACT_TRANSFER_INVALID,
+          'Artifact metadata does not match the request or was not committed before the response.',
+          null,
+          { method: pending.method }
+        );
+      }
+      if (pending.lastErrorResponse) {
+        await this.finishPendingRequestIfExhausted(responseMessage.id, pending);
+        return;
+      }
+      if (pending.method === 'dom.baseline.create') {
+        const baselineId = typeof result.baselineId === 'string' ? result.baselineId : '';
+        const evicted = Array.isArray(result.evicted) ? result.evicted : [];
+        for (const item of evicted) {
+          if (item && typeof item === 'object' && 'baselineId' in item) {
+            this.domBaselines.deleteIfOwnedBy(String(item.baselineId), socket);
+          }
+        }
+        if (!this.domBaselines.register(baselineId, socket, result.expiresAt)) {
+          await this.releaseOrphanDomBaseline(socket, responseMessage);
+          pending.lastErrorResponse = createFailure(
+            responseMessage.id,
+            ERROR_CODES.DOM_BASELINE_INVALIDATED,
+            'DOM baseline owner metadata is invalid.',
+            null,
+            { method: pending.method }
+          );
+          await this.finishPendingRequestIfExhausted(responseMessage.id, pending);
+          return;
+        }
+      } else if (pending.method === 'dom.baseline.release' && pending.baselineId) {
+        this.domBaselines.delete(pending.baselineId);
+      }
       const response =
         pending.method === 'health.ping'
           ? createSuccess(
@@ -1219,6 +1568,10 @@ export class BridgeDaemon {
                 daemonVersion: DAEMON_VERSION,
                 daemon_supported_versions: getSupportedProtocolVersions(),
                 proxy: this.getProxyStatusPayload(),
+                recovery: createHealthRecoveryPayload(
+                  this.recoveryTelemetry.snapshot('daemon'),
+                  responseMessage.result
+                ),
               },
               {
                 method: pending.method,
@@ -1226,14 +1579,38 @@ export class BridgeDaemon {
             )
           : responseMessage;
 
-      await writeJsonLine(pending.socket, {
-        type: 'agent.response',
-        response,
+      try {
+        await writeJsonLine(pending.socket, {
+          type: 'agent.response',
+          response,
+        });
+      } catch (error) {
+        if (pending.method === 'dom.baseline.create') {
+          const baselineId =
+            result && typeof result.baselineId === 'string' ? result.baselineId : null;
+          if (baselineId) this.domBaselines.delete(baselineId);
+          await this.releaseOrphanDomBaseline(socket, responseMessage).catch(() => {});
+        }
+        this.clearPendingRequest(responseMessage.id, pending);
+        this.recordRequestCompletion(responseMessage.id, false, pending);
+        throw error;
+      }
+      this.clearPendingRequest(responseMessage.id, pending, true);
+      this.recordRequestCompletion(responseMessage.id, true, pending);
+      this.pushLog({
+        at: new Date().toISOString(),
+        method: responseMessage.meta?.method ?? null,
+        ok: true,
+        id: responseMessage.id,
+        source: pending.source || null,
       });
       return;
     }
 
     // A routed request has one target, so its error is final.
+    if (pending.baselineId && responseMessage.error?.code === ERROR_CODES.DOM_BASELINE_NOT_FOUND) {
+      this.domBaselines.delete(pending.baselineId);
+    }
     pending.lastErrorResponse = responseMessage;
 
     await this.finishPendingRequestIfExhausted(responseMessage.id, pending);
@@ -1245,6 +1622,8 @@ export class BridgeDaemon {
    */
   handleSocketClose(socket) {
     if (socket.__extensionId) {
+      this.artifactStore.deleteByExtension(socket.__extensionId);
+      this.domBaselines.clearForSocket(socket);
       this.logger.info('extension disconnected', { extensionId: socket.__extensionId });
       if (this.extensionSockets.get(socket.__extensionId) === socket) {
         this.extensionSockets.delete(socket.__extensionId);
@@ -1266,8 +1645,9 @@ export class BridgeDaemon {
         if (!pending) {
           continue;
         }
+        this.domBaselines.markAbandonedCreate(id, pending);
         this.clearPendingRequest(id, pending);
-        this.recordRequestCompletion(id, false);
+        this.recordRequestCompletion(id, false, pending);
       }
     }
 
@@ -1304,7 +1684,7 @@ export class BridgeDaemon {
     }
 
     this.clearPendingRequest(requestId, pending);
-    this.recordRequestCompletion(requestId, false);
+    this.recordRequestCompletion(requestId, false, pending);
 
     const response =
       pending.lastErrorResponse ??
@@ -1333,9 +1713,10 @@ export class BridgeDaemon {
   /**
    * @param {string} requestId
    * @param {boolean} ok
+   * @param {PendingEntry | undefined} [pending]
    * @returns {void}
    */
-  recordRequestCompletion(requestId, ok) {
+  recordRequestCompletion(requestId, ok, pending) {
     const startedAt = this.requestStartTimes.get(requestId);
     this.requestStartTimes.delete(requestId);
     this.requestsProcessed += 1;
@@ -1345,6 +1726,34 @@ export class BridgeDaemon {
     if (typeof startedAt === 'number') {
       this.totalResponseTimeMs += Date.now() - startedAt;
     }
+    if (pending?.method && !DIAGNOSTIC_METHODS.has(pending.method)) {
+      this.recordDaemonRequestOutcome(
+        {
+          method: /** @type {BridgeRequest['method']} */ (pending.method),
+          meta: {
+            protocol_version: pending.protocolVersion ?? getProtocolVersion(),
+            token_budget: null,
+            ...(pending.automaticMcpRetry
+              ? { source: 'mcp', automatic_retry: { attempt: 2, reason: 'retryable_error' } }
+              : {}),
+          },
+        },
+        ok
+      );
+    }
+  }
+
+  /**
+   * @param {Pick<BridgeRequest, 'method' | 'meta'>} request
+   * @param {boolean} ok
+   */
+  recordDaemonRequestOutcome(request, ok) {
+    if (DIAGNOSTIC_METHODS.has(request.method)) return;
+    const outcome = ok ? 'success' : 'failure';
+    this.recoveryTelemetry.record('request_outcome', outcome, request.method);
+    if (isAutomaticMcpRetry(request.meta)) {
+      this.recoveryTelemetry.record('automatic_mcp_retry', outcome, request.method);
+    }
   }
 
   /**
@@ -1352,13 +1761,34 @@ export class BridgeDaemon {
    * @returns {void}
    */
   pushLog(entry) {
-    this.recentLog.push(entry);
+    this.recentLog.push(/** @type {Record<string, unknown>} */ (sanitizeIncidentalValue(entry)));
     if (this.recentLog.length > DAEMON_RECENT_LOG_LIMIT) {
       this.recentLog.shift();
     }
   }
 }
 
+/** @param {Record<string, unknown>} meta */
+function isAutomaticMcpRetry(meta) {
+  const marker = asHealthRecord(meta.automatic_retry);
+  if (meta.source !== 'mcp' || !marker) return false;
+  const keys = Object.keys(marker).sort();
+  return (
+    keys.length === 2 &&
+    keys[0] === 'attempt' &&
+    keys[1] === 'reason' &&
+    marker.attempt === 2 &&
+    marker.reason === 'retryable_error'
+  );
+}
+
+/** @param {string | undefined} method @returns {ArtifactKind | null} */
+function getArtifactKindForMethod(method) {
+  if (method?.startsWith('screenshot.')) return 'screenshot';
+  if (method === 'network.export_har') return 'har';
+  return null;
+}
+
 /**
  * Check whether a daemon is already listening on the given transport.
  * Connects, sends a health.ping, and waits up to 500 ms for a response.