[gavin-jeong]

JIRA: https://sendbird.atlassian.net/browse/CPLAT-12785

Summary

Follow-up to #175. Two changes:

1. Async pane lookups and sends. Update no longer runs tmux pane lookups or send-keys directly.
2. One binding table. The keymap now drives footers, menu footers, the action hint box and the ? help.

Changes

Async tmux (tmux_async.go)

• These paths now run FindPane and switch-client in a tea.Cmd: pane jump, live resume, ctrl+g in the live preview, opening the live preview, bulk input, and share-ref.
• Each result message carries a generation. Update drops a superseded result, so a slow lookup never switches to, or opens, an older target.
• The live-preview result also checks that the cursor is still on the same session.
• Two sends used to discard their .Run() errors: the \ continuation (now one sendKeys) and bulk sends (now errors.Join). Both now report the error in the status line.
• Share-ref shows "Sharing…" until the send finishes. On failure it shows "Share failed: " instead of an early "Shared".

Binding table (bindings.go)

• A table covers every Keymap field with a footer label and help text. Keys still come from Keymap, which keeps the existing yaml overrides.
• footer() keeps hints in priority order. When the line is narrow, it drops unpinned hints from the end and shows …. Help and quit stay pinned, and so does esc in the conversation view.
• These surfaces now read from the table: the session, stats, config and plugins footers; the conversation footer and the menu footers; the session actions hint box; and the ? common and session-list rows.
• Dispatch now uses keymap keys where the footer advertises them but the code had them hardcoded: stats q, esc and p, and config/plugins x. Left, Right and resize overrides now reach the split pane through resolveSplitKey.
• scopeKeys now reads from the table.
• Bug fix: the state menu footer said S:states, but the key is s.

Tests

• New tmux_async_test.go. It checks that no tmux call runs on the Update path, that stale results are dropped, and that send errors show up in the status line.
• New bindings_test.go. It covers:
  • table coverage of every keymap field (by reflection)
  • Conflicts()
  • reachability: every bound session action is in ? and not swallowed by a global intercept
  • every footer key appears in ?
  • narrow-footer … with pinned help/quit
  • an override reaching the footer, ? and dispatch together
• One existing test was adapted: it checked the full conversation footer at width 120. The test now renders wide, because its subject is where the key comes from.
• go vet ./... and gofmt -l . are clean. go test -race -count=1 ./... passes except TestDayCursorMoveCost, a frame-budget timing test that also fails under -race on master (about 20 ms per move). It passes without -race, which is how CI runs it.

[jinsekim]

LGTM!