From 8476cef3ce494d4fab87465390d62de891d82380 Mon Sep 17 00:00:00 2001 From: "clagentic-builder[bot]" <290147524+clagentic-builder[bot]@users.noreply.github.com> Date: Mon, 22 Jun 2026 19:08:09 -0400 Subject: [PATCH] chore: clean up lr-teams-icon-strip TODO aliases in sidebar-users.js --- lib/public/app.js | 2 +- lib/public/modules/app-messages.js | 4 +--- lib/public/modules/sidebar-users.js | 14 ++------------ 3 files changed, 4 insertions(+), 16 deletions(-) diff --git a/lib/public/app.js b/lib/public/app.js index 0b6a7d1..7b57941 100644 --- a/lib/public/app.js +++ b/lib/public/app.js @@ -11,7 +11,7 @@ import { renderIconStrip, updateProjectBadge } from './modules/sidebar-projects.js'; import { - renderUserStrip, closeDmUserPicker, setCurrentDmUser, + closeDmUserPicker, setCurrentDmUser, updateDmBadge, renderSidebarPresence, setMentionActive, clearAllMentionActive, initSidebarUsers } from './modules/sidebar-users.js'; diff --git a/lib/public/modules/app-messages.js b/lib/public/modules/app-messages.js index c688d90..64e7806 100644 --- a/lib/public/modules/app-messages.js +++ b/lib/public/modules/app-messages.js @@ -11,7 +11,7 @@ import { refreshIcons, iconHtml } from './icons.js'; import { renderMarkdown, highlightCodeBlocks, highlightCodeBlocksBatched, renderMermaidBlocks } from './markdown.js'; import { updatePageTitle } from './sidebar.js'; import { renderSessionList, updateSessionPresence, populateCliSessionList, handleSearchResults, updateSessionBadge } from './sidebar-sessions.js'; -import { updateDmBadge, renderSidebarPresence, setMentionActive, renderUserStrip, handleAgentsList as handleAgentsListTeam } from './sidebar-users.js'; +import { updateDmBadge, renderSidebarPresence, setMentionActive } from './sidebar-users.js'; import { refreshMobileChatSheet } from './sidebar-mobile.js'; import { handlePaletteSessionSwitch, setPaletteVersion } from './command-palette.js'; import { handleAgentsList, handleAgentFavoriteToggled } from './agent-picker.js'; @@ -459,7 +459,6 @@ export function processMessage(msg) { case "agents_list": handleAgentsList(msg); - handleAgentsListTeam(msg); break; case "agent_favorite_toggled": @@ -1222,7 +1221,6 @@ export function processMessage(msg) { case "projects_updated": updateProjectList(msg); - renderUserStrip(); break; case "project_owner_changed": diff --git a/lib/public/modules/sidebar-users.js b/lib/public/modules/sidebar-users.js index ecb9fd6..3f1ec8d 100644 --- a/lib/public/modules/sidebar-users.js +++ b/lib/public/modules/sidebar-users.js @@ -61,10 +61,6 @@ export function initSidebarUsers() { }); } -// handleAgentsList: no-op stub — agent-picker.js is the authoritative consumer. -// This export is kept because app-messages.js imports it as handleAgentsListTeam. -// TODO(lr-teams-icon-strip): remove this alias once app-messages.js import is cleaned up. -export function handleAgentsList(_msg) { /* no-op: agent-picker.js handles agents_list */ } export function showIconTooltip(el, text) { hideIconTooltip(); @@ -143,11 +139,6 @@ export function renderSidebarPresence(onlineUsers) { } } -// renderUserStrip: retained as a no-op export — callers in app-messages.js -// invoke this on projects_updated. The icon strip no longer shows per-user -// avatars; all that lives in the team panel (opened via #icon-strip-team-btn). -// TODO(lr-teams-icon-strip): remove call sites in app-messages.js once callers are updated. -export function renderUserStrip() { /* no-op: team panel replaced icon-strip avatars */ } export function closeDmUserPicker() { @@ -167,9 +158,8 @@ export function setCurrentDmUser(userId) { // for use by the team panel and DM picker. } -// updateDmBadge: retained as a no-op export — DM unread counts are now shown -// only inside the team panel, not in the icon strip. -// TODO(lr-teams-icon-strip): wire to team panel badge once panel shows DM counts. +// updateDmBadge: no-op — DM unread counts are now shown only inside the team panel. +// Future: wire DM unread count to team panel badge. No-op until team panel supports per-user DM count display. export function updateDmBadge(userId, count) { /* no-op: icon-strip user badges removed */ } export function getCurrentDmUserId() {