umputun / revdiff / 24821351290

90%

master: 90%

Pull #143

github

fix(keymap): address Copilot findings on chord parser

Two edge cases Copilot flagged on PR #143:

1. Standalone ">" key was unbindable — the chord gate was
   strings.Contains(rawKey, ">"), which treats a bare ">" as
   malformed chord syntax. Narrow the gate to also require
   rawKey != ">" so "map > quit" and "unmap >" work.

2. "map ctrl+w>esc foo" parsed and stored a binding that could
   never fire, because handleChordSecond treats esc as unconditional
   cancel before ResolveChord. Reject esc (and the "escape" alias)
   as a chord second-stage key at parse time with a warning, so the
   silent failure becomes loud and actionable. Matches the locked
   design decision that esc is reserved for cancel.

133 of 133 new or added lines in 6 files covered. (100.0%)

8036 of 8974 relevant lines covered (89.55%)

465.68 hits per line