fix(tui): pass through Ctrl/Cmd+L so the dashboard redraw byte never inserts a stray 'l'

[jazzzzmaybe]

Summary

Fixes #87069 — the dashboard Chat tab inserted a stray l into the composer on every session switch.

Root cause: PtySession.attach() injects a raw Ctrl-L byte (0x0c) to force a TUI repaint on keep-alive PTY reattach (#86332 / #68071). The Ink keypress parser maps that byte to a ctrl+l key event which reaches both listeners — the global hotkey handler (which redraws but doesn't stop propagation) and the composer's TextInput, which falls through to the generic insert branch and types a literal l.

Change

One line: add isAction(key, input, 'l') to shouldPassThroughToGlobalHandler so the composer ignores the redraw keypress, mirroring the global handler's own isAction(key, ch, 'l') check in useInputHandlers.ts. Unmodified l still types normally; the existing ctrl+c/x/o pass-through entries are untouched.

Test plan

• cd ui-tui && npx tsc --noEmit
• cd ui-tui && npx vitest run src/__tests__/textInputPassThrough.test.ts (7 passed)
• cd ui-tui && npx vitest run src/__tests__/textInput (107 passed)
• Manual: with the composer focused, Ctrl+L no longer inserts l; typing l normally is unaffected.

Related

• Closes [Dashboard Chat] Stray "l" inserted into the composer on every session switch (Ctrl-L force-redraw byte leaks into TextInput) #87069

[Enough1122]

AI code review — automated review for reference, author can ignore or act on any point.

fix(tui): pass through Ctrl/Cmd+L so the dashboard redraw byte never inserts a stray 'l'

1. Blocking concern — docker.yml retargets the official image pipeline to a personal namespace: the workflow change (.github/workflows/docker.yml) flips IMAGE_NAME to ghcr.io/jazzzzmaybe/hermes-agent and gates build/publish/merge on github.repository == 'jazzzzmaybe/hermes-agent'. Merged into NousResearch/hermes-agent, this would (a) silently stop all main-branch container publishing (the repository gate is false upstream), and (b) if the gate ever matched, publish under a personal GHCR namespace using the upstream token, plus a "Make package public" PATCH to a user-level package. If this is a fork-local CI change, it should not land in the upstream tree — please scope it out (or confirm the intent to migrate the project's official images, which would need maintainer sign-off and a separate namespace/secret story).
2. scrollback: 0 on the outer xterm (ChatPage.tsx line 521): with the inner TUI now owning all transcript scroll, a TUI crash/exit leaves the dashboard terminal with zero scrollback — the previous inline mode at least preserved history in xterm. Acceptable given the alt-screen design, but worth confirming a fallback (e.g. transient scrollback during TUI absence) isn't needed.
3. Wheel→Shift+Up/Down flood control: wheelScrollSequences caps at 6 sequences per tick (web/src/lib/pty-wheel-scroll.ts line 22) — good; note a fast trackpad flick on a long transcript now translates to at most 6 lines per event, so scrolling long histories requires repeated flicks (acceptable trade-off; the TUI may also have its own PageUp handling).
4. Sanitizer change is clean: stripErase: false for alt-screen (erase codes are legitimate in-place redraws there), burst-collapse kept always-on; the removed PTY_RESUME_SANITIZE_WINDOW_MS timer simplifies the resume path. HERMES_TUI_INLINE removal is documented in both web_server.py and ui-tui/src/config/env.ts. Tests cover the Ctrl+L pass-through, wheel mapping, and sanitizer flag.

[jazzzzmaybe]

Superseded by a clean single-commit PR from the fix/ctrl-l-redraw branch. This PR's branch mixed in personal fork CI changes; sorry for the noise.