feat(kebab-tui): p9-fb-12 follow-up — heuristic 제거, mode-authoritative dispatch #85
Reference in New Issue
Block a user
Delete Branch "feat/p9-fb-12-mode-dispatch"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
요약
p9-fb-12 partial (PR #84) 의 deferred 부분 finalize. spec status
in_progress→completed.변경
search::is_typing_mod함수 삭제. search Char dispatch 가state.mode기반:j/k→ 선택 이동search::handle_key_search의i(chunk inspect) /g(editor jump) pre-pass 가 Normal 모드에서만 fireask::handle_key_askinput-empty heuristic 삭제. e/j/k 가 mode 분기 — Normal toggle/scroll, Insert typingapp.mode = Mode::auto_for(focus)mirror — run loop 동작 일치app.mode = Mode::Normal테스트
cargo test --workspace --no-fail-fast -j 1exit 0cargo clippy --workspace --all-targets -- -D warningsclean문서
kebab tui: mode-authoritative dispatch 명시in_progress→completedp9-fb-12 partial 의 HOTFIXES entry 가 이미 deferral 사유 + 해결 조건 명시했으므로 새 entry 불필요.
회차 1 — heuristic 제거 + mode-authoritative 전환 깔끔. test fixture 가 run-loop 의 auto_for mirror 하는 패턴 정확. 신규 4 회귀 테스트도 Insert-types vs Normal-commands 양 방향 cover.
actionable nit 2 건 (cosmetic) — (a) j/k 두 개 arm → body-branching 으로 flatten, (b) 자리 코멘트 노이즈.
@@ -248,0 +249,4 @@// `is_typing_mod` heuristic (SHIFT-aware char filter) is gone —// mode now decides whether a Char goes to the input buffer or// becomes a navigation command. `Tab` (mode cycle), `Enter`// (refresh), `Backspace`, arrow keys, Esc work in both modesj/k 가 각각 두 개 arm 으로 분기 (Insert 가드 + Normal no-guard) — 동작은 정확하지만 reader 가 "왜 두 개" 한 박자 멈춤. body-branching 으로 flatten 하면 의도가 더 명확:
4 arm → 2 arm. "j/k 가 mode 따라 다르다" 가 한 자리에서 보임. ask.rs 의 패턴과도 정렬.
@@ -317,11 +341,9 @@ fn cycle_mode(m: SearchMode) -> SearchMode {}}is_typing_mod함수 자리에 "removed" 안내 코멘트 3 줄이 영구 남아있는데, 같은 정보가 매치 블록 안 "p9-fb-12 follow-up" 코멘트 + commit message 에 이미 충분히 documented 입니다. 향후 reader 한테 "여기 뭔가 있었나" 만 묻게 만드는 노이즈.그냥 삭제하는 게 깔끔. (HOTFIXES / commit history 가 reference).
회차 2 — nit 2 건 깔끔히 반영.
추가 지적 없음. 머지 OK.