fix(mobile): show an error instead of an endless preview spinner

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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

[t3dotgg]

On mobile, an image, PDF, or other asset preview kept spinning on "Preparing ... preview" when the environment was disconnected or the asset URL request failed. useAssetUrlState reported a pending query and a dead connection as the same Loading value, so previews had no terminal state and no way to retry.

The hook now folds the environment connection phase into its result. A query that is still pending while the environment is offline, retrying, or in error resolves to Failure with a disconnected reason. A failed query resolves to Failure with a failed reason. The derivation moved into apps/mobile/src/state/asset-url-state.ts so it can be tested on its own.

On the file screen, image, SVG, browser, and video previews now render an error instead of a spinner when the asset URL will not arrive. A disconnected environment gets the existing EnvironmentConnectionNotice, the same offline surface the terminal and review sheets use, with its "Retry now" button wired to a connection retry plus a fresh URL mint. A failed URL gets a short "Preview unavailable" line with "Try again". The video preview lost its own unavailable branch because the screen now owns that state for every preview type.

The other consumers, the full screen file preview modal, both video preview modals, and the chat feed image and video, already branched on Failure, so they pick up the disconnected case without changes. The web app has its own copy of the hook in apps/web/src/assets/assetUrls.ts and is left alone to keep this to one concern.

Verified with vp test run apps/mobile/src/state/asset-url-state.test.ts, tsc --noEmit in apps/mobile, and lint on the touched files. Not verified on a device or simulator.

Closes#7630

Built by Claude Opus 5 in the Claude Code harness.

---

Note

Low Risk
Scoped to mobile asset URL presentation and thread file previews; logic is covered by unit tests and does not change auth or shared web hooks.

Overview
Mobile workspace file previews (image, browser/PDF, video) no longer spin forever when a signed asset URL cannot be obtained. useAssetUrlState now combines the shared URL query result with the environment connection phase via deriveAssetUrlState, yielding Failure with disconnected vs failed so UI can distinguish outage from a bad/missing asset.

On ThreadFileScreen, FileContent shows the new WorkspaceFilePreviewError for previews that depend on an asset URL: EnvironmentConnectionNotice (with connection retry + URL refresh) when disconnected, or a “Preview unavailable” empty state with Try again otherwise. Retry refreshes the asset URL and bumps previewRevision to remount the preview. WorkspaceFileVideoPreview drops its local unavailable UI because the parent owns that terminal state.

Unit tests cover deriveAssetUrlState; other mobile useAssetUrlState consumers that already treat Failure as an error surface should pick up disconnected cases without further changes.

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

Note

Fix endless mobile preview spinner by showing reason-specific asset URL errors

• Adds WorkspaceFilePreviewError in WorkspaceFilePreviewError.tsx to render an environment connection notice for disconnected failures and a generic "Preview unavailable" retry state for other asset URL failures.
• Adds deriveAssetUrlState in asset-url-state.ts to distinguish disconnected environment failures from connected query failures, and suppresses resolved URLs while the environment is offline, reconnecting, or in error.
• Updates useAssetUrlState and useConnectionPhase in assets.ts to fold the shared asset result with the current connection phase, and treats a waiting post-outage query as Loading so stale failures are not shown.
• Wires the typed failure reason and a retry handler (which remints the asset URL and remounts the preview) through FileContent and ThreadFileScreen in ThreadFilesRouteScreen.tsx, removing the old boolean previewUnavailable prop.
• Risk: WorkspaceFileVideoPreview no longer renders its own "Video unavailable" state; video asset failures must now be handled by the parent before the media player renders. Existing consumers relying on the unavailable prop or the boolean previewUnavailable prop need updating.

^{Macroscope summarized 9cd4887.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.8 KiB	13.6 KiB	−189 B (−1.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−14 B (−0.2%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.5 KiB	−175 B (−2.5%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	57.1 KiB	−1.4 KiB (−2.4%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−28 B (−0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−7 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−21 B (−0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	8	−1 (−11.1%)	21	✅

Baseline: 18573d6 · PR result: 9cd4887 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at 9cd4887

Macroscope's review found this PR approvable — This is a focused mobile bug fix that replaces terminal preview spinners with actionable connection or asset-unavailable states, including retry support. The shared hook’s broader impact is limited to mobile asset previews and is covered by explicit state-derivation tests, with no schema, deployment, security, billing, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 13efe5b

[cursor]

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

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

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