fix(web): keep agent browser preview visible by maria-rcks · Pull Request #9484 · pingdotgg/t3code

No issues identified (1 code object reviewed).

• Reviewed files modified since 6fd3251; other PR files not modified since then were skipped.
• Merge Base: 373be93
• Head: 533421d

✅	File Path	U3 Bytes	Comments Posted	Reason
✅	`apps/web/src/components/preview/PreviewAutomationHosts.tsx`	2557	0

Billed Total: 10.00KB of diff | $0.50 (This review was charged at our per-review byte minimum of 10.00KB. Learn more here)

apps/web/src/components/preview/PreviewAutomationHosts.tsx

line 475: The explicit-hide suppression is not recorded until line 475, after open has awaited tab creation/default viewport work. Requests are consumed concurrently, so a snapshot/click/other non-open request for the same tab can enter requireReadyTab during that window, see no suppression entry, and auto-open the mini player. Thus an open request with show: false can still visibly surface the tab when a follow-up automation request overlaps it. [ Already posted ]
line 480: Entries added to presentationSuppressedRuntimeTabsRef are never removed when a tab closes, disappears from a reconciled server list, or its server epoch changes. An automation client that repeatedly opens tabs with show: false (especially with reuseExistingTab: false) leaves an unreachable runtime-tab id in this long-lived map for every tab, so the host's memory usage grows for its lifetime. [ Out of scope (post-validation triage) ]