chore(web): drop legacy <Tabs> from /agents layout #1025

New issue

Closed

opened 2026-05-09 18:18:26 +00:00 by claude-desktop · 1 comment

claude-desktop commented

2026-05-09 18:18:26 +00:00

Collaborator

As a maintainer,
I want the legacy internal <Tabs> removed from /agents,
so that the route stops doing nav-inside-nav now that the sidebar covers the same hierarchy.

Acceptance criteria

Cleanup

apps/web/src/routes/agents.tsx (or agents.index.tsx) no longer imports <Tabs> / <TabList> / <TabPanel> for the Fleet / Globals / JSON top-level navigation.
Layout route renders a header (page title + repo selector) + <Outlet />. No tab UI.
Any leftover tab-specific helpers (SECTION_TABS, SectionPane wrappers that exist only to host the tab structure) are deleted or inlined into their child route.

Other tabs

If a sub-route legitimately uses <Tabs> for an inner concern (e.g. agent-detail page sub-views), that stays.

Tests

Old <Tabs> test cases for /agents deleted; per-sub-route tests cover the equivalent surface (#nav-v2-1).

Out of scope

Re-styling individual sub-route pages.
Settings tabs (separate concern, stays).

References

Spec: docs/specs/nav-consolidation.md § Migration plan step 9.
Current internal tabs: apps/web/src/routes/agents.index.tsx:73-77.
Depends on #nav-v2-1 (sub-routes already exist).

**As a** maintainer, **I want** the legacy internal `<Tabs>` removed from `/agents`, **so that** the route stops doing nav-inside-nav now that the sidebar covers the same hierarchy. ## Acceptance criteria ### Cleanup - [ ] `apps/web/src/routes/agents.tsx` (or `agents.index.tsx`) no longer imports `<Tabs>` / `<TabList>` / `<TabPanel>` for the Fleet / Globals / JSON top-level navigation. - [ ] Layout route renders a header (page title + repo selector) + `<Outlet />`. No tab UI. - [ ] Any leftover tab-specific helpers (`SECTION_TABS`, `SectionPane` wrappers that exist only to host the tab structure) are deleted or inlined into their child route. ### Other tabs - [ ] If a sub-route legitimately uses `<Tabs>` for an inner concern (e.g. agent-detail page sub-views), that stays. ### Tests - [ ] Old `<Tabs>` test cases for `/agents` deleted; per-sub-route tests cover the equivalent surface (#nav-v2-1). ## Out of scope - Re-styling individual sub-route pages. - Settings tabs (separate concern, stays). ## References - Spec: `docs/specs/nav-consolidation.md` § Migration plan step 9. - Current internal tabs: `apps/web/src/routes/agents.index.tsx:73-77`. - Depends on #nav-v2-1 (sub-routes already exist).

claude-desktop added this to the nav-v2 milestone

2026-05-09 18:18:26 +00:00

claude-desktop added the

area:dashboard

type:user-story

labels

2026-05-09 18:18:26 +00:00