From dc4e512fcaa73f0be31b7ff949252b0a29ba4a78 Mon Sep 17 00:00:00 2001
From: "reason-machines[bot]"
 <293529679+reason-machines[bot]@users.noreply.github.com>
Date: Fri, 6 Nov 2026 04:23:34 +0000
Subject: [PATCH] remove the Chat tab from the sidebar

Co-authored-by: arnestrickmann <115920878+arnestrickmann@users.noreply.github.com>
---
 packages/web/src/components/nav.test.tsx | 34 ++++------------------
 packages/web/src/components/nav.tsx      | 36 ++----------------------
 2 files changed, 7 insertions(+), 63 deletions(-)

diff --git a/packages/web/src/components/nav.test.tsx b/packages/web/src/components/nav.test.tsx
index 2cb2b616..a3ed70f3 100644
--- a/packages/web/src/components/nav.test.tsx
+++ b/packages/web/src/components/nav.test.tsx
@@ -7,7 +7,6 @@ import {SidebarProvider} from '@proxy/ui/components/sidebar';
 import {ThemeProvider} from '@proxy/ui/components/theme-provider';
 import {TooltipProvider} from '@proxy/ui/components/tooltip';
 import {AppSidebar} from '@/components/nav';
-import {type Unread, UnreadContext} from '@/hooks/use-unread';
 import type {Human} from '@/lib/orgs';
 import {testHuman, testOrg} from '@/test-org';
 
@@ -15,16 +14,7 @@ afterEach(() => {
 	cleanup();
 });
 
-// What is unread, as the provider around the human side would have it.
-function unreadOf(ids: string[]): Unread {
-	return {
-		unreadIds: new Set(ids),
-		answers: null,
-		setViewing: () => undefined,
-	};
-}
-
-function renderSidebar(human: Human, entry = '/acme', unread = unreadOf([])) {
+function renderSidebar(human: Human, entry = '/acme') {
 	const router = createMemoryRouter(
 		[
 			{
@@ -41,9 +31,7 @@ function renderSidebar(human: Human, entry = '/acme', unread = unreadOf([])) {
 		<ThemeProvider>
 			<TooltipProvider>
 				<SidebarProvider collapsed>
-					<UnreadContext.Provider value={unread}>
-						<RouterProvider router={router} />
-					</UnreadContext.Provider>
+					<RouterProvider router={router} />
 				</SidebarProvider>
 			</TooltipProvider>
 		</ThemeProvider>,
@@ -78,28 +66,16 @@ describe('AppSidebar', () => {
 		expect(settings.getAttribute('href')).toBe('/acme/settings');
 	});
 
-	test('shows Chat beside Home, and Alerts under Activity, with the harness experiment', async () => {
+	test('shows Alerts under Activity with the harness experiment, but no Chat tab', async () => {
 		renderSidebar(testHuman({...testOrg, experiments: ['harness']}));
 		await screen.findByRole('link', {name: 'Home'});
 
 		expect(groupLabels()).toEqual([
-			['Home', 'Chat'],
+			['Home'],
 			['Connections', 'Agent permissions'],
 			['Activity', 'Alerts'],
 		]);
-	});
-
-	test('counts the conversations with something unread on Chat', async () => {
-		renderSidebar(
-			testHuman({...testOrg, experiments: ['harness']}),
-			'/acme',
-			unreadOf(['general', 'finance']),
-		);
-		await screen.findByRole('link', {name: 'Chat'});
-
-		const badge = screen.getByRole('status', {name: '2 unread'});
-		expect(badge.textContent).toBe('2');
-		expect(badge.closest('li')?.querySelector('a')?.textContent).toBe('Chat');
+		expect(screen.queryByRole('link', {name: 'Chat'})).toBeNull();
 	});
 
 	test("lights the switcher on the organization's members and settings", async () => {
diff --git a/packages/web/src/components/nav.tsx b/packages/web/src/components/nav.tsx
index 00aa4fed..5f5988a1 100644
--- a/packages/web/src/components/nav.tsx
+++ b/packages/web/src/components/nav.tsx
@@ -1,12 +1,4 @@
-import {
-	BellIcon,
-	BotIcon,
-	HomeIcon,
-	MessageSquareIcon,
-	PlugIcon,
-	ScrollTextIcon,
-	type LucideIcon,
-} from 'lucide-react';
+import {BellIcon, BotIcon, HomeIcon, PlugIcon, ScrollTextIcon, type LucideIcon} from 'lucide-react';
 import type {Experiment} from '@proxy/utils';
 import {Link, useLocation} from 'react-router';
 import {OrgSwitcher} from '@/components/org-switcher';
@@ -24,34 +16,14 @@ import {
 	useSidebar,
 } from '@proxy/ui/components/sidebar';
 import {useHuman} from '@/hooks/use-human';
-import {useUnread} from '@/hooks/use-unread';
 
 // `experiment` is the experiment a destination is behind, shown only with it on.
 type Destination = {to: string; label: string; icon: LucideIcon; experiment?: Experiment};
 
-const CHAT = '/chat';
-
-// How many conversations have something unread, on Chat: over the icon's corner on the rail, at
-// the row's end where the labels show.
-function UnreadBadge({count}: {count: number}) {
-	return (
-		<span
-			role="status"
-			aria-label={`${count} unread`}
-			className="pointer-events-none absolute top-1/2 right-2 flex h-4 min-w-4 -translate-y-1/2 items-center justify-center rounded-full bg-primary px-1 text-[10px] leading-none font-medium text-primary-foreground group-data-[collapsible=icon]:top-0 group-data-[collapsible=icon]:right-0 group-data-[collapsible=icon]:translate-y-0"
-		>
-			{count}
-		</span>
-	);
-}
-
 // The organization's, under the switcher, each under its slug, in groups set apart from each
 // other. Its members and settings are in the switcher.
 const GROUPS: Destination[][] = [
-	[
-		{to: '', label: 'Home', icon: HomeIcon},
-		{to: '/chat', label: 'Chat', icon: MessageSquareIcon, experiment: 'harness'},
-	],
+	[{to: '', label: 'Home', icon: HomeIcon}],
 	[
 		{to: '/connections', label: 'Connections', icon: PlugIcon},
 		{to: '/agents', label: 'Agent permissions', icon: BotIcon},
@@ -76,7 +48,6 @@ export function AppSidebar() {
 	const {pathname} = useLocation();
 	const human = useHuman();
 	const experiments = useExperiments();
-	const unread = useUnread();
 	const {setOpenMobile} = useSidebar();
 	const home = `/${human?.org.slug ?? ''}`;
 	const groups = GROUPS.map((group) =>
@@ -111,9 +82,6 @@ export function AppSidebar() {
 										<destination.icon />
 										<span>{destination.label}</span>
 									</SidebarMenuButton>
-									{destination.to === CHAT && unread.unreadIds.size > 0 && (
-										<UnreadBadge count={unread.unreadIds.size} />
-									)}
 								</SidebarMenuItem>
 							))}
 						</SidebarMenu>