fix: restore default terminal mouse behavior#19
Merged
Conversation
…ll override Revert to parity with the private repo after the captain corrected live behavior. Config-only: - herdr: remove ui.mouse_capture=false so Herdr's default host mouse capture applies again; keep agent_panel_sort="spaces" (now an explicit ordering choice, comment updated accordingly). - wezterm: remove alternate_buffer_wheel_scroll_speed=0 and its comment. The Escape-swallow durable fix remains an upstream Herdr follow-up; this change does not resolve it. Neovim mouse setting and keybindings untouched.
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
Revert two config lines in this PUBLIC dotfiles repo to restore parity with the private repo after the captain corrected the live behavior. This is a pure CONFIG revert - NOT a code fix and deliberately NOT a Herdr source patch or vendored fork.
Two files changed:
Deliberate scope decisions (should NOT be flagged as mistakes/omissions): The Neovim mouse setting (home/.config/nvim/lua/vim_config.lua o.mouse='') is intentionally left EXACTLY as-is and untouched. No keybindings touched. No local patched Herdr fork was added/built/vendored - that upstream Herdr fix remains a FUTURE follow-up only, by design. Nothing restarted or rebuilt. The Escape-swallow durable fix is explicitly NOT solved by this change and remains an upstream Herdr follow-up.
Validation already done locally: Herdr TOML parses (single [ui] table, mouse_capture gone, agent_panel_sort still "spaces"); WezTerm Lua parses cleanly via wezterm and alternate_buffer_wheel_scroll_speed no longer appears.
What Changed
spacesagent panel ordering.Risk Assessment
✅ Low: The narrowly scoped configuration revert exactly matches the authoritative intent, preserves the required settings and untouched files, and introduces no material correctness or compatibility risks.
Testing
Inspected the target diff, loaded the explicit WezTerm config with the installed binary, parsed Herdr TOML, verified Herdr’s shipped defaults and exact untouched scope, and captured focused evidence; all executable checks passed, while a live Herdr launch was intentionally not disturbed.
Evidence: Config revert evidence
Herdr’s shipped default has mouse capture enabled; the worktree config omits that override while retainingagent_panel_sort = "spaces". WezTerm loaded the worktree configuration successfully. Exact-scope assertions also passed.Evidence: Parsed Herdr configuration
Nix-parsed Herdr configuration showing the complete unchanged keys table and UI containing onlyagent_panel_sort = "spaces".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.
Inspectedgit diff --unified=80 b9ea27acd31a936ac11cc4f27e391f7096eb478b..7f14462b595475c420699585a31b4c0aae655877for both changed files.Ranwezterm --config-file "$PWD/home/.config/wezterm/wezterm.lua" show-keysto load the worktree configuration through the installed application.Ranherdr --default-configand compared its relevant[ui]defaults with the worktree configuration.Parsedhome/.config/herdr/config.tomlusingbuiltins.fromTOMLthroughnix eval.Ran repository-widegit grepassertions for removedmouse_capture = falseandalternate_buffer_wheel_scroll_speedoverrides.Asserted only the two intended files changed, Herdr’s[keys]section was unchanged, Neovim’svim_config.luawas unchanged, and WezTerm retained one blank line beforereturn config.AttemptedHERDR_CONFIG_PATH="$PWD/home/.config/herdr/config.toml" herdr --no-session; the existing live Herdr instance correctly prevented a second launch, so it was not restarted or reloaded.Checkedgit status --shortand confirmed testing left no worktree changes.✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.