fix: address Copilot review findings

Update docs to reflect that help toggle is remappable via ActionHelp,
fix N key description in README, improve parse warning log message.

Author: umputun

.claude-plugin/skills/revdiff/references/config.md

@@ -100,4 +100,4 @@ map ctrl+d half_page_down
 
 Available actions: `down`, `up`, `page_down`, `page_up`, `half_page_down`, `half_page_up`, `home`, `end`, `scroll_left`, `scroll_right`, `next_item`, `prev_item`, `next_hunk`, `prev_hunk`, `toggle_pane`, `focus_tree`, `focus_diff`, `search`, `confirm`, `annotate_file`, `delete_annotation`, `annot_list`, `toggle_collapsed`, `toggle_wrap`, `toggle_tree`, `toggle_line_numbers`, `toggle_hunk`, `filter`, `quit`, `discard_quit`, `help`, `dismiss`
 
-Modal keys (annotation input, search input, help overlay, confirm discard) are not remappable.
+Modal keys (annotation input, search input, confirm discard) are not remappable.

CLAUDE.md

@@ -62,7 +62,7 @@ git diff → diff.ParseUnifiedDiff() → []DiffLine
 - Keybindings file: `~/.config/revdiff/keybindings` (`map <key> <action>` / `unmap <key>` format)
 - `--keys` overrides keybindings path, `--dump-keys` prints effective bindings
 - `keymap.Keymap` passed to `Model` via `ModelConfig.Keymap`; handlers switch on `m.keymap.Resolve(msg.String())` instead of raw key strings
-- ~30 `Action` constants in `keymap/keymap.go` (e.g., `ActionDown`, `ActionQuit`); modal keys (annotation input, search input, help overlay) stay hardcoded
+- ~30 `Action` constants in `keymap/keymap.go` (e.g., `ActionDown`, `ActionQuit`); modal text-entry keys (annotation input, search input, confirm discard) stay hardcoded; help toggle is routed through `ActionHelp`
 - Help overlay is dynamically rendered from `m.keymap.HelpSections()`
 
 ## Claude Code Plugin

README.md

@@ -332,7 +332,7 @@ This mode activates when all three conditions are met: single file, markdown ext
 |-----|--------|
 | `/` | Start search in diff pane |
 | `n` | Next search match (overrides next file when search active) |
-| `N` | Previous search match |
+| `N` | Previous file (previous search match when searching) |
 | `Esc` | Cancel search input / clear search results |
 
 **Annotations:**
@@ -383,7 +383,7 @@ mkdir -p ~/.config/revdiff
 revdiff --dump-keys > ~/.config/revdiff/keybindings
 ```
 
-Then edit to taste. Modal keys (annotation input, search input, help overlay, confirm discard) are not remappable.
+Then edit to taste. Modal keys (annotation input, search input, confirm discard) are not remappable.
 
 <details>
 <summary>Available actions (click to expand)</summary>

keymap/keymap.go

@@ -360,7 +360,7 @@ func parse(r io.Reader) (maps []mapEntry, unmaps []string, err error) {
 			key := normalizeKey(fields[1])
 			unmaps = append(unmaps, key)
 		default:
-			log.Printf("[WARN] keybindings:%d: unknown command %q, skipping", lineNum, line)
+			log.Printf("[WARN] keybindings:%d: unknown command %q in line %q, skipping", lineNum, cmd, line)
 		}
 	}
 	if err := scanner.Err(); err != nil {

ui/annotlist.go

@@ -179,7 +179,7 @@ func (m Model) injectBorderTitle(box, title string, popupWidth int) string {
 }
 
 // handleAnnotListKey handles keys when the annotation list popup is visible.
-// j/k/arrows navigate, Enter jumps to annotation, Esc/@/action-key closes, all other keys consumed.
+// j/k/arrows navigate, Enter jumps to annotation, Esc/annot_list key closes, all other keys consumed.
 func (m Model) handleAnnotListKey(msg tea.KeyMsg) (tea.Model, tea.Cmd) {
 	// resolve action to allow closing popup with the same key that opens it (remappable)
 	action := m.keymap.Resolve(msg.String())