fix: improve Herdr usability with mouse capture disabled#18
Merged
Conversation
…l arrows Both settings are consequences of the mouse_capture=false mitigation. With the host mouse capture disabled, the Herdr header click-toggle for agent panel sort is unreachable, so pin agent_panel_sort="spaces" explicitly. Likewise WezTerm would synthesize Up/Down arrows for wheel scroll in the alt-screen Herdr client (a pane REPL then reads them as history navigation), so set alternate_buffer_wheel_scroll_speed=0.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Intent
Ship two approved, additive-only Herdr/WezTerm config changes to this PUBLIC dotfiles repo for parity with the already-shipped mouse_capture=false mitigation (PR #17). Both changes are direct consequences of that mitigation, so anyone following this setup hits the same issues.
home/.config/herdr/config.toml: inside the SAME existing [ui] table that holds mouse_capture=false, add agent_panel_sort = "spaces" with an explanatory comment. Rationale: with mouse_capture=false the Herdr header click-toggle for the agents-panel sort is unreachable, so the sort must be pinned explicitly to its default ("spaces", grouped by space; the alternative is "priority").
home/.config/wezterm/wezterm.lua: before the final 'return config', add config.alternate_buffer_wheel_scroll_speed = 0 with an explanatory comment, matching the file's existing style. Rationale: with mouse_capture=false the Herdr client is a full-screen alt-screen app enabling no mouse reporting, so WezTerm would otherwise synthesize Up/Down arrow keys for wheel scroll (default speed 3) and a pane REPL reads them as history navigation; 0 disables that synthesis. Scrollback still works via PageUp/PageDown or copy-mode (prefix+y).
Deliberate constraints (do NOT flag as mistakes): mouse_capture stays false and is intentionally unchanged; no [keys] binding or any other line is touched; both edits are purely additive (net +11 lines, comments included) and the comments are reader-facing on purpose since this is a public repo. No AGENTS.md/CLAUDE.md agent files are added. Footprint is exactly these two additions.
What Changed
spacesbecause its header toggle is unavailable when mouse capture is disabled.Risk Assessment
✅ Low: The change is narrowly scoped, purely additive, matches the authoritative intent exactly, and introduces no substantiated correctness or compatibility risks.
Testing
Verified the exact two-file, additive-only +11-line footprint, launched Herdr with the changed configuration and observed
spaceson the real agents-panel surface, loaded the WezTerm configuration successfully through WezTerm itself, captured a reviewer-readable evidence transcript, and confirmed the worktree remained clean.Evidence: End-to-end configuration evidence
Herdr rendered the agents-panel header as "spaces"; WezTerm loaded the changed configuration successfully. The transcript also records the exact two-file, +11-line additive footprint.Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
git diff --stat,git diff --name-only,git diff --numstat, and the full scoped diff from207ffe8d7f45baa4e8c20c16f77244d8eeb74b08to7bffee9ec00e7b6f1246ecd07d35420a67d38753HOME=.test-home HERDR_CONFIG_PATH="$PWD/home/.config/herdr/config.toml" herdr --no-sessionin an interactive PTY, confirming the real agents-panel header renderedspaceswezterm --config-file "$PWD/home/.config/wezterm/wezterm.lua" show-keys, confirming WezTerm accepted and evaluated the complete configurationgit status --shortand transient.test-homecheck after cleanup✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.