Skip to content

refactor(app): simplify layout hierarchy#33588

Merged
Brendonovich merged 4 commits into
anomalyco:devfrom
Brendonovich:simplify-target-layouts
Jun 24, 2026
Merged

refactor(app): simplify layout hierarchy#33588
Brendonovich merged 4 commits into
anomalyco:devfrom
Brendonovich:simplify-target-layouts

Conversation

@Brendonovich

Copy link
Copy Markdown
Member

Remove TargetServerLayout and TargetDirectoryLayout, which bundled
providers for home, draft, and session routes together unnecessarily.

  • NewLayout moves to the router root so it stays mounted across home,
    draft, and session route changes.
  • TargetSessionRoute resolves its own server connection, session
    directory, SDK, and data providers locally.
  • DraftRoute resolves its own server, SDK, and data providers from
    the draft tab state.
  • Home and legacy URL redirects use a minimal selected-server shell
    without directory SDK or data providers.
  • Titlebar fetches session metadata via LayoutRoute.server through
    Global.createServerCtx instead of the surrounding ServerSDK
    provider, preserving cross-server tab behavior.

The SDKProvider and DirectoryDataProvider now only mount when
a directory-backed route (session or draft) is active.

Remove TargetServerLayout and TargetDirectoryLayout, moving directory
resolution and SDK providers into route-specific components. NewLayout
now renders from the router root so it stays mounted across home,
draft, and session routes. Titlebar resolves session metadata from
LayoutRoute.server instead of the surrounding server provider.
@Brendonovich Brendonovich requested a review from Hona as a code owner June 24, 2026 04:05
@Brendonovich Brendonovich enabled auto-merge (squash) June 24, 2026 04:06
@Brendonovich Brendonovich merged commit 800d41d into anomalyco:dev Jun 24, 2026
8 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