[codex] Report preview action failures

[juliusmarminge]

Summary

• report previously swallowed preview bridge and local API failures with operation, thread, tab, artifact, and annotation context
• preserve the exact upstream cause so original stack chains remain available
• replace raw preview URLs in logged context with hostname, protocol, and input length
• cover credential, path-token, query-token, and fragment redaction with a focused regression test

Stack dependency

• Based on [codex] Share safe URL diagnostics #3403 for @t3tools/shared/urlDiagnostics.

Validation

• vp test run apps/web/src/components/preview/reportPreviewActionFailure.test.ts (1 test)
• vp check (passes with 20 pre-existing warnings)
• vp run typecheck

---

Note

Low Risk
Observability-only change: user-visible behavior and toasts are largely unchanged except more console noise on failures. URL redaction reduces accidental credential leakage in logs.

Overview
Preview bridge and related failures are no longer swallowed with empty .catch() handlers. A shared reportPreviewActionFailure helper logs [preview] action failed to console.error with structured context (operation, threadKey, tabId, artifact/annotation ids where relevant) and passes through the original cause.

PreviewView and PreviewMoreMenu now route navigation, zoom, history, refresh, DevTools, cache/cookies, recordings, screenshots, clipboard, element picker, and open-external errors through that helper. URL-related context uses previewUrlFailureContext, backed by new getUrlDiagnostics in @t3tools/shared/urlDiagnostics, so logs keep hostname, protocol, and length only—not full URLs, credentials, or query tokens. openPreviewSession atom failures are reported when not interrupted; picker failures stay silent in the UI but are logged.

Shared also adds redactDpopRequestTarget (scheme/host/path only) with tests; it is not wired into callers in this PR.

^{Reviewed by Cursor Bugbot for commit 8fe6259. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Report preview action failures via console.error with sanitized context

• Adds reportPreviewActionFailure in reportPreviewActionFailure.ts that logs a structured console.error with operation name and context, replacing silent error swallowing across preview components.
• Adds previewUrlFailureContext to derive sanitized URL diagnostics (hostname, protocol, input length) without leaking raw URLs, credentials, or query params.
• Wires failure reporting into PreviewView.tsx for URL submission phases, navigation, session open, zoom, refresh, recording, screenshots, and element picking flows.
• Wires failure reporting into PreviewMoreMenu.tsx for hard reload, devtools, zoom, cookie/cache clearing operations.
• Behavioral Change: previously silent failures in preview actions now emit console.error entries with structured metadata.

^{Macroscope summarized 8fe6259.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 6279b65c-8ed0-4cf7-9c48-51e3362e7b1b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/web-preview-action-errors

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Open session failures not reported
  • Added an explicit check for result._tag === "Failure" after awaiting openPreviewSession, calling reportPreviewActionFailure with the squashed cause so open-session failures get structured [preview] diagnostics instead of silently resolving.

Or push these changes by commenting:

@cursor push 2d48ae75fc

Preview (2d48ae75fc)

diff --git a/apps/web/src/components/preview/PreviewView.tsx b/apps/web/src/components/preview/PreviewView.tsx
--- a/apps/web/src/components/preview/PreviewView.tsx
+++ b/apps/web/src/components/preview/PreviewView.tsx
@@ -4,6 +4,7 @@
 import { type ScopedThreadRef } from "@t3tools/contracts";
 import { useCallback, useEffect, useRef, useState } from "react";
 
+import { squashAtomCommandFailure } from "@t3tools/client-runtime/state/runtime";
 import { useComposerDraftStore } from "~/composerDraftStore";
 import { previewAnnotationScreenshotFile } from "~/lib/previewAnnotation";
 import { ensureLocalApi } from "~/localApi";
@@ -109,11 +110,17 @@
           rememberPreviewUrl(threadRef, resolvedUrl);
         } else {
           operation = "open-session";
-          await openPreviewSession({
+          const result = await openPreviewSession({
             openPreview: open,
             threadRef,
             url: resolvedUrl,
           });
+          if (result._tag === "Failure") {
+            reportPreviewActionFailure(
+              { operation, threadKey, url: targetUrl },
+              squashAtomCommandFailure(result),
+            );
+          }
         }
       } catch (cause) {
         reportPreviewActionFailure(

_{You can send follow-ups to the cloud agent here.}

^{Reviewed by Cursor Bugbot for commit df0a48f. Configure here.}

[macroscopeapp]

Approvability

Verdict: Approved

This PR replaces silent error swallowing with console.error logging for preview action failures. No runtime behavior changes to actual functionality - purely a debugging/observability improvement with tests and careful handling to avoid logging sensitive URL data.

^{You can customize Macroscope's approvability policy. Learn more.}

Dismissing prior approval to re-evaluate 3280e2c

Dismissing prior approval to re-evaluate 260cf6d

Dismissing prior approval to re-evaluate 8fe6259