Skip to content

Commit 7f57b36

Browse files
authored
refactor(webui): split the sidebar's observability pages out of Workspace (#845)
Workspace was doing two jobs: things you CONFIGURE (Servers, Tools, Secrets, Agent Tokens) sat in the same list as things you OBSERVE (Sessions, Activity Log, Security Scanners), so nine items read as one undifferentiated column and Sessions in particular looked misfiled next to Secrets. The three observation pages move to their own OBSERVABILITY section, Activity Log first with Sessions beneath it — they are two lenses on the same data, and the Sessions page's only action is to open the Activity Log filtered to one session. No routes, no components, no behaviour: sidebar grouping only.
1 parent b93a952 commit 7f57b36

1 file changed

Lines changed: 26 additions & 11 deletions

File tree

frontend/src/components/SidebarNav.vue

Lines changed: 26 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -263,27 +263,42 @@
263263
<span v-show="!collapsed">Agent Tokens</span>
264264
</router-link>
265265
</li>
266+
</ul>
267+
268+
<!-- Section: Observability
269+
Workspace is what you CONFIGURE (servers, tools, secrets);
270+
these are what you OBSERVE. Keeping them apart stops the two
271+
kinds of page reading as one undifferentiated list. -->
272+
<div
273+
v-if="!collapsed"
274+
class="mt-5 mb-1 px-3 text-[10px] font-semibold uppercase tracking-[0.12em] text-base-content/40"
275+
>
276+
Observability
277+
</div>
278+
<div v-else class="mt-3 mb-1 mx-auto w-6 h-px bg-base-300"></div>
279+
280+
<ul class="menu menu-sm w-full gap-0.5 p-0">
266281
<li>
267282
<router-link
268-
to="/sessions"
269-
:class="{ 'active': isActiveRoute('/sessions') }"
283+
to="/activity"
284+
:class="{ 'active': isActiveRoute('/activity') }"
270285
class="rounded-lg font-medium"
271-
:title="collapsed ? 'Sessions' : ''"
272-
data-test="sidebar-sessions"
286+
:title="collapsed ? 'Activity Log' : ''"
273287
>
274-
<IconSessions class="w-5 h-5 shrink-0" />
275-
<span v-show="!collapsed">Sessions</span>
288+
<IconActivity class="w-5 h-5 shrink-0" />
289+
<span v-show="!collapsed">Activity Log</span>
276290
</router-link>
277291
</li>
278292
<li>
279293
<router-link
280-
to="/activity"
281-
:class="{ 'active': isActiveRoute('/activity') }"
294+
to="/sessions"
295+
:class="{ 'active': isActiveRoute('/sessions') }"
282296
class="rounded-lg font-medium"
283-
:title="collapsed ? 'Activity Log' : ''"
297+
:title="collapsed ? 'Sessions' : ''"
298+
data-test="sidebar-sessions"
284299
>
285-
<IconActivity class="w-5 h-5 shrink-0" />
286-
<span v-show="!collapsed">Activity Log</span>
300+
<IconSessions class="w-5 h-5 shrink-0" />
301+
<span v-show="!collapsed">Sessions</span>
287302
</router-link>
288303
</li>
289304
<li>

0 commit comments

Comments
 (0)