Skip to content

feat(viewer): homeView host flag — don't auto-select a session when the host owns Home#174

Merged
benvinegar merged 1 commit into
mainfrom
feat/host-home-view
Jun 29, 2026
Merged

feat(viewer): homeView host flag — don't auto-select a session when the host owns Home#174
benvinegar merged 1 commit into
mainfrom
feat/host-home-view

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Adds an opt-in homeView flag to the embed SideshowHost contract.

When an embedder owns its own session-less landing (e.g. sideshow cloud's new Home feed), the engine auto-selecting a session on boot leaves that session highlighted in the sidebar behind the host's landing, and navigating back to the session-less route never clears it (applyRoute deliberately ignores a null route).

With homeView: true, the engine:

  • honors a deep-linked route.sessionId but otherwise stays session-less on boot (no selection, nothing highlighted);
  • clears its selection when the route later becomes session-less.

Self-hosted leaves the flag unset and is unchanged — it still auto-selects the latest session on boot and deselects explicitly via the wordmark goHome(). Covered by e2e/embed-home-view.spec.ts (flag on → nothing selected; flag off → auto-selects). Changeset (minor).

🤖 Generated with Claude Code

…he host owns Home

When a host owns its own session-less landing (e.g. sideshow cloud's Home feed),
auto-selecting a session on boot leaves it highlighted in the sidebar behind the
host's landing, and navigating back to the session-less route never clears it
(applyRoute ignores a null route). Add an opt-in `homeView` flag to SideshowHost:
when set, the engine honors a deep-linked route session but otherwise stays
session-less on boot (no selection, no highlight), and clears its selection when
the route becomes session-less. Self-hosted leaves the flag unset and is
unchanged (auto-selects the latest on boot; deselects via the wordmark goHome).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
benvinegar added a commit that referenced this pull request Jun 29, 2026
@benvinegar benvinegar marked this pull request as ready for review June 29, 2026 21:36
@benvinegar benvinegar merged commit 1b7a28c into main Jun 29, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant