fix(preview): return to pip when closing the right panel

[maria-rcks]

closing the right panel with a browser selected now returns that browser to pip, including after restoring it from pip. the preview toggle shares the panel close handler, covering the panel button, keyboard toggle, and sheet dismissal while preserving browser-tab close behavior.

validation: 191 existing tests passed across chat logic, preview view, mini player state, and right panel state; web typecheck, scoped lint, and formatting passed (existing warnings). github ci passed on c4b7c8c; two independent gpt-6-astra reviews found no code issues. no unresolved github review threads.

recording: recreated browser content, as requested, rendered with the actual preview panel and mini player. the existing panel close handler returns the preview to pip twice, including after restoring it to the panel. electron guest rendering is not exercised by this recreation.

written by gpt-6 using codex.

Summary by CodeRabbit

• New Features

  • Closing the preview panel now opens the preview mini player for the active resource.

• Bug Fixes

  • Preview panel state is now cleared consistently when the panel is closed or toggled.

[macroscopeapp]

Approvability

Verdict: Approved at c4b7c8c

Macroscope's review found this PR approvable — This is a narrowly scoped one-file UI bug fix that preserves an active preview in the existing mini-player when the right panel closes. Other panel types and existing browser-tab cleanup behavior remain unchanged.

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 01f377e8-0b1c-4957-99f1-e96ec7f41e34

📥 Commits

Reviewing files that changed from the base of the PR and between 3836890 and c4b7c8c.

📒 Files selected for processing (1)

• apps/web/src/components/ChatView.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The preview panel close flow now opens the active preview resource in the mini player before clearing the maximized thread key and closing the right panel.

Changes

Preview panel handoff

Layer / File(s)	Summary
Close preview and open mini player apps/web/src/components/ChatView.tsx	closePreviewPanel opens the active preview resource in the mini player, clears the maximized thread key, and closes the panel. togglePreviewPanel uses this callback for its close branch and dependency list.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: t3dotgg, juliusmarminge, bil0000

Merge Risk: ⚪ Minimal · up to c4b7c

Closing a preview panel now returns the selected resource to the mini player while preserving ordinary panel-close behavior. The change is ready to merge.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the main change: returning the preview to PiP when the right panel closes.
Description check	✅ Passed	The description explains the behavior change, affected interactions, validation results, and UI recording. It does not use the template headings or include the checklist, but it contains the required …

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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