chore: clean up lr-teams-icon-strip TODO aliases in sidebar-users.js

[clagentic-builder]

Clean up three TODO(lr-teams-icon-strip) comments in sidebar-users.js and their corresponding call sites.

handleAgentsList (sidebar-users.js): The no-op stub was imported by app-messages.js as handleAgentsListTeam and called redundantly alongside the real handleAgentsList from agent-picker.js. Removed the import alias and the duplicate call in the agents_list message case. Removed the stub export from sidebar-users.js entirely. The TODO is gone.

renderUserStrip (sidebar-users.js): The no-op stub was imported and called in two places: app-messages.js (projects_updated case) and app.js (import-only, no active call site). Removed the call in app-messages.js, the import in app-messages.js, the import in app.js, and the export from sidebar-users.js. The TODO is gone.

updateDmBadge (sidebar-users.js): Genuine future feature. Export and no-op body retained. TODO replaced with a plain forward-looking comment. Still imported and called by app-messages.js on dm_message.

Task: lr-9b0f

Tests: npm test 309/309 pass. ESM named-import resolution check confirms no broken imports after the removals.

[clagentic-reviewer]

PEACHES — clean (0 nits)

Three no-op stubs (renderUserStrip, handleAgentsList alias, updateDmBadge comment) cleanly removed; imports and call sites synchronized. TODO aliases resolved. All 309 tests pass.

No findings.

[clagentic-merger]

clagentic gate-note — merged by naomi

• Platform: github
• PR: chore: clean up lr-teams-icon-strip TODO aliases in sidebar-users.js #258
• Merged SHA: 8476cef3ce494d4fab87465390d62de891d82380
• CI SHA at HEAD: (not recorded)
• PEACHES review: reviewed
• Authorize rationale: PEACHES clean 0 findings; 309/309 tests pass; chore removes stale TODO aliases and dead import/call sites; no blocking holds