fix(tui): make Ctrl+O inspector complete and draft-safe

[Hmbown]

Summary

• route Ctrl+O to the whole-turn inspector even when the composer contains a draft
• move external-editor access to Ctrl+Shift+O and align help/localized keybinding copy
• preserve the complete final assistant result in the pager and copied pager body
• keep the exported handoff intentionally compact
• preserve Sh1Zuku / @SparkofSpike report credit in commit metadata and contributor records

Fixes #4482.

Verification

• cargo test -p codewhale-tui key_shortcuts::tests --locked
• cargo test -p codewhale-tui turn_inspector_pager_retains_complete_long_final_result --locked
• cargo test -p codewhale-tui shipped_complete_packs --locked
• cargo test -p codewhale-tui ctrl_o_help_copy_matches_turn_inspector_behavior --locked
• python3 scripts/test_check_coauthor_trailers.py
• cargo fmt --all
• git diff --check

Release note

This PR is intentionally scoped to the reproducible Ctrl+O failures. The separate intermittent Windows repaint report in #4479 remains open until its terminal/version evidence supports a narrow fix rather than a speculative repaint loop.

[Hmbown]

Exact-head local gate update for 2883f95: cargo check -p codewhale-tui --bin codewhale-tui --locked passed. The branch is clean and tracking origin/agent/091-tui-regressions.

[Hmbown]

Follow-up review fixes are public at exact head f62e0d2: Caps Lock plus Ctrl+O now stays on the inspector, external-editor routing requires explicit Shift and has an F4 fallback for legacy terminals, c/y uses a source-faithful Turn Inspector payload (indentation, tabs, repeated/blank lines retained), and blank display rows are no longer doubled. Focused shortcut, 36 pager tests, long-result regression, locked binary check, fmt, and diff check pass. CI restarted on this head.

[Hmbown]

Rebased onto public main 7885f8db8e6ebdd4fe577dcdc2799808a22dadbd after #4491. Contributor credit for @luismateusvargas, @redjade75723, and @SparkofSpike/Sh1Zuku is preserved. New exact head: 227d0e5c297ec5ed7d37cdd5a732473310669b28. Post-rebase gates: pager 36/36, shortcut routing 2/2, complete long Turn Inspector result 1/1, fmt and diff checks pass. CI is rerunning on this exact head.

[Hmbown]

Pushed follow-up 0ee5719f4 for the Windows CI failure. Foreground and background hooks now receive a closed pipe when there is no payload, which is a reliable EOF signal through cmd/PowerShell (unlike a NUL handle, which may reopen console input). Exact local proof: cargo fmt --all -- --check; cargo test -p codewhale-tui --bin codewhale-tui observer_hook_receives_eof_instead_of_inheriting_terminal_stdin --locked; git diff --check. Waiting for the public Windows exact-head rerun before merging.

[Hmbown]

Public handoff update: pushed exact head 10834f5fc9eeef62757f18f726dd8509d74b69e9 (verify with the PR API). The earlier Windows rerun proved the closed-stdin hook regression passes. Its remaining Windows failure was a separate acceptance fixture using Unix-only printf; this head uses portable echo and a newline-tolerant assertion. Focused exact test and cargo fmt --check pass locally. Fresh exact-head CI is now authoritative. The macOS approval_modal_keeps_wheel_for_review_and_denies_without_side_effect timeout remains a separate PTY-test stabilization item; the failure frame already contained DENIAL-HONORED and no side effect was reported, so do not conflate it with the hook production fix.

[Hmbown]

Correction to the preceding comment: the exact remote head is 10834f5fc8b4a55f57e36419bcec5de1f5856c8c (the earlier SHA text contained a transcription typo). The branch and CI are running on this corrected exact SHA.

[Hmbown]

Rebased onto public main de7f45586fb8ce21f8093315080f5058cacb38c3 after #4505 and #4506. The only conflict was the contributor ledger; both @w1w218 and @SparkofSpike entries are preserved. New exact head: 44cddb80fe18c7bc5f8d5cdd41acb7f67b755a2e. Local exact-head gates: hook EOF regression passed; portable Operate shell acceptance passed; Ctrl+O shortcut tests 2/2; pager tests 36/36; cargo fmt --all -- --check; git diff --check; co-author/author validation passed for all four commits. Fresh hosted CI on this head is authoritative.

[Hmbown]

Correction: the exact remote head is 44cddb80fb46ae437de39999951cdd26b7af8428 (the preceding comment had a transcription error). The branch and fresh CI are running on this exact SHA.

[cursor]

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[claude]

Claude encountered an error after 0s —— View job

---

I'll analyze this and get back to you.