feat(sidebar): add manual session status labels (todo/in-progress/done) (#3199)

[rodboev]

Thinking Path

• The WebUI auto-derives session "needs attention" from agent activity, but users triaging multiple concurrent sessions have no way to track their own workflow state per session.
• A server-side status field would require a SessionDB schema migration and coordination with the gateway SSE watcher path; localStorage keyed to session_id delivers the same result with zero backend changes.
• The existing pattern for per-session localStorage maps (e.g., hermes-session-viewed-counts at sessions.js:158, hermes-session-completion-unread at sessions.js:159) is already established; this PR adds a fourth map hermes-session-manual-status.
• The session row builder at sessions.js:4433-4483 already appends pin, worktree, branch, and project-dot indicators inline in titleRow. A session-manual-status badge fits the same slot.
• The three-dot action menu at sessions.js:2395-2566 provides the primary interaction surface; badge click-to-cycle adds a secondary shortcut.
• Three status values (todo, in-progress, done) match the established kanban_status_todo vocabulary in i18n.js:666 and keep the option set minimal.

What Changed

• static/sessions.js: add SESSION_MANUAL_STATUS_KEY constant; add _getSessionManualStatuses(), getSessionManualStatus(), setSessionManualStatus(), and _cycleSessionManualStatus() helpers; inject status badge into session row title row when a status is set; add three status-picker entries to the three-dot action menu before _mountSessionActionMenu.
• static/style.css: add .session-manual-status base rule and three status-specific color variants after .session-project-dot.
• static/i18n.js: add session_status_todo, session_status_in_progress, session_status_done, and session_status_click_to_change keys.

Why It Matters

Users with multiple concurrent sessions can now mark each one Todo, In Progress, or Done directly from the sidebar, without renaming or relying on the auto-derived attention state. The badge persists across page reloads via localStorage and requires zero backend changes.

Verification

# Full suite
pytest tests/ -v --timeout=60

# Manual: open WebUI, right-click a session, set a status, verify badge appears
# Reload page, confirm badge still present
# Click badge, confirm cycle: Todo → In Progress → Done → cleared

Risks / Follow-ups

• localStorage is per-browser, not synced across devices. A follow-up API endpoint could persist status server-side.
• Stale localStorage entries for deleted sessions accumulate silently. A prune pass keyed to _allSessions during render would fix this; deferred to follow-up.
• No competing PR on this axis as of 2026-06-04.

Model Used

Claude Opus 4.8 via Claude Code CLI

Closes #3199

[greptile-apps]

Greptile Summary

This PR adds manual session status labels (Todo / In Progress / Done) to the sidebar, stored in localStorage under a new key hermes-session-manual-status. Statuses are surfaced as a colored badge in the session title row (click-to-cycle) and as a picker group in the three-dot action menu.

• sessions.js: new SESSION_MANUAL_STATUS_KEY constant, four helper functions (_getSessionManualStatuses, getSessionManualStatus, setSessionManualStatus, _cycleSessionManualStatus), badge injection into renderSessionListFromCache, and status picker entries in _openSessionActionMenu.
• i18n.js / style.css: four new i18n keys propagated to all locale blocks (English literals used for non-English locales as a known gap) and three color-variant CSS rules for the badge.
• tests: only a function-body size sentinel bump; no new unit tests for the status helpers.

Confidence Score: 5/5

Safe to merge; the change is fully additive and isolated to localStorage with no backend or schema impact.

All changes are purely additive: a new localStorage key, badge DOM injection, CSS rules, and i18n keys. The logic is straightforward and the worst failure mode is a silent no-op when localStorage is unavailable. The in-memory cache gap means redundant JSON-parse calls during list renders but causes no incorrect behavior.

static/sessions.js — the _getSessionManualStatuses function diverges from the established lazy-init cache pattern used by the three sibling readers.

Important Files Changed

Filename	Overview
static/sessions.js	Adds manual status localStorage helpers and badge/menu injection. The new _getSessionManualStatuses() does not follow the module-level in-memory cache pattern used by the three sibling readers, causing O(n) JSON-parse calls per render instead of O(1).
static/i18n.js	Four new keys added to all locale blocks. Non-English locales carry English string literals (acknowledged in PR description as a known gap).
static/style.css	Adds .session-manual-status base rule and three color-variant modifiers; straightforward and consistent with adjacent .session-project-dot styling.
tests/test_1466_sidebar_cancel_clarify.py	Only updates the _openSessionActionMenu function-body size sentinel from 7200→8000; no new tests cover the status getter/setter/cycle logic or badge rendering.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User opens three-dot menu] --> B[_openSessionActionMenu]
    B --> C[getSessionManualStatus]
    C --> D[_getSessionManualStatuses reads localStorage every call]
    B --> E[Render Todo / In Progress / Done picker items]
    E --> F{User clicks item}
    F -->|same as current| G[setSessionManualStatus sid null - clears]
    F -->|different| H[setSessionManualStatus sid status - sets]
    G & H --> I[Write to localStorage]
    I --> J[renderSessionListFromCache]
    J --> K[getSessionManualStatus per session row]
    K --> L[Append colored badge to titleRow]
    L --> M[Badge onclick cycles todo to in-progress to done to cleared]

Loading

_{Reviews (3): Last reviewed commit: "fix(sidebar): bump action-menu test wind..." | Re-trigger Greptile}