From dc5c70ccda0fea8cbba72ac2148095383830e467 Mon Sep 17 00:00:00 2001
From: Jiayuan Zhang <forrestchang7@gmail.com>
Date: Fri, 10 Apr 2026 01:56:01 +0800
Subject: [PATCH] refactor(sidebar): move Settings back to Workspace nav,
 footer shows only user info

Settings is a navigable page that belongs with other nav items.
Footer now cleanly separates identity (user profile) from navigation.
---
 .../app/(dashboard)/_components/app-sidebar.tsx   | 15 ++-------------
 1 file changed, 2 insertions(+), 13 deletions(-)

diff --git a/apps/web/app/(dashboard)/_components/app-sidebar.tsx b/apps/web/app/(dashboard)/_components/app-sidebar.tsx
index 727de25187..84826f5919 100644
--- a/apps/web/app/(dashboard)/_components/app-sidebar.tsx
+++ b/apps/web/app/(dashboard)/_components/app-sidebar.tsx
@@ -66,6 +66,7 @@ const workspaceNav = [
   { href: "/agents", label: "Agents", icon: Bot },
   { href: "/runtimes", label: "Runtimes", icon: Monitor },
   { href: "/skills", label: "Skills", icon: BookOpenText },
+  { href: "/settings", label: "Settings", icon: Settings },
 ];
 
 function DraftDot() {
@@ -257,19 +258,7 @@ export function AppSidebar() {
         </SidebarContent>
 
         <SidebarFooter className="p-2">
-          <SidebarMenu>
-            <SidebarMenuItem>
-              <SidebarMenuButton
-                isActive={pathname === "/settings"}
-                render={<Link href="/settings" />}
-                className="text-muted-foreground hover:not-data-active:bg-sidebar-accent/70 data-active:bg-sidebar-accent data-active:text-sidebar-accent-foreground"
-              >
-                <Settings />
-                <span>Settings</span>
-              </SidebarMenuButton>
-            </SidebarMenuItem>
-          </SidebarMenu>
-          <div className="border-t pt-2 mt-1">
+          <div className="border-t pt-2">
             <div className="flex items-center gap-2.5 rounded-md px-2 py-1.5">
               <ActorAvatar
                 name={user?.name ?? ""}