fix(tui): Esc+Ctrl-C 同時押下での Application.exit() 二重呼び出しを防止

実 TTY で Esc と Ctrl-C をほぼ同時に押す (または Ctrl-C 連打) と
「Return value already set. Application.exit() failed.」のクラッシュ画面が
出る問題を修正する。

Esc バインドは矢印キーのエスケープシーケンスと区別するため eager=False
(確定待ち) で登録しており、Esc + Ctrl-C が同一入力バッチで届くと
prompt_toolkit は 1 回のキー処理内で「Esc ハンドラ (exit 確定) → 残り
バッファ再処理で questionary 組み込み Ctrl-C ハンドラ」を連続実行し、
exit が二重に呼ばれる。process_keys の is_done ガードは入力キューにしか
効かず、この同一バッファ再処理の経路は防げない。

全プロンプト共通の通過点 _ask_erased に _guard_after_done を追加し、
アプリの key_bindings (questionary 組み込み + 後付けの Esc/←) を
ConditionalKeyBindings(kb, ~is_done) でラップして回答確定後のキー処理を
無効化する。

pty 回帰テストは Esc+Ctrl-C を 1 回の write で送出して再現する (ガードを
外すと実際にクラッシュ出力を検出して失敗することを確認済み)。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: takemi-ohama

lib/devbase/tui/menu.py

@@ -96,16 +96,40 @@ def _cancel(event):
     return _add_escape_binding(question, _cancel)
 
 
+def _guard_after_done(question):
+    """回答確定後 (``Application.exit`` 済み) のキー処理を無効化する。
+
+    prompt_toolkit は 1 回の読み取りで複数キーを同一バッチとして処理するため、
+    確定キーの直後に入力が溜まっていると (例: Ctrl-C 連打 / Enter 直後の Ctrl-C)、
+    1 つ目のキーで exit して戻り値が確定した後も残りのキーが同じバッチ内で
+    処理され、questionary 組み込みの Ctrl-C ハンドラ等が再度 exit を呼んで
+    「Return value already set. Application.exit() failed.」のクラッシュになる
+    (実 TTY でのみ再現)。アプリ単位の key_bindings (questionary 組み込み + 本
+    モジュールが後付けする Esc/← を含む) を ``~is_done`` でガードし、確定後の
+    キーは無視する。
+    """
+    from prompt_toolkit.filters import is_done
+    from prompt_toolkit.key_binding import ConditionalKeyBindings
+
+    kb = question.application.key_bindings
+    if kb is not None:
+        question.application.key_bindings = ConditionalKeyBindings(kb, ~is_done)
+    return question
+
+
 def _ask_erased(question):
     """``erase_when_done`` を立ててから ``ask()`` する共通ヘルパ (全プロンプト用)。
 
     questionary は回答確定時に「質問 + 回答」の collapse 行を画面へ残す。TUI は
     ループでメニューを再描画するため、回答のたびにこの行が蓄積して画面全体が
     下へずれていく (実 TTY でのみ再現する残留・行ずれ不具合)。回答後に描画ごと
     消去することで、メニューを常に同じ位置へ再描画する。
+
+    併せて ``_guard_after_done`` で確定後のキー処理を無効化する (全プロンプトが
+    本ヘルパを通るため、ここが単一の適用点)。
     """
     question.application.erase_when_done = True
-    return question.ask()
+    return _guard_after_done(question).ask()
 
 
 def _ask_with_escape(question):

tests/cli/tui/test_menu.py

@@ -112,6 +112,25 @@ def test_back_handler_sets_erase_when_done():
     assert captured == {"result": menu.MENU_BACK}
 
 
+def test_guard_after_done_wraps_app_key_bindings():
+    """_guard_after_done が app の key_bindings を ~is_done 条件でラップすること。
+
+    回答確定 (Application.exit) 後に同一バッチへ溜まったキー (Ctrl-C 連打 /
+    Enter 直後の Ctrl-C 等) が questionary 組み込みハンドラへ届くと exit が
+    二重に呼ばれ「Return value already set」でクラッシュする (実 TTY のみで
+    再現)。ガード適用で確定後のキーが無視されることの構造検証。
+    """
+    questionary = pytest.importorskip("questionary")
+    from prompt_toolkit.key_binding import ConditionalKeyBindings
+
+    q = questionary.select("t", choices=[questionary.Choice(title="a", value="a")])
+    inner = q.application.key_bindings
+    assert menu._guard_after_done(q) is q
+    wrapped = q.application.key_bindings
+    assert isinstance(wrapped, ConditionalKeyBindings)
+    assert wrapped.key_bindings is inner, "既存バインドを内包したままガードする"
+
+
 # ---------------------------------------------------------------------------
 # select: バインドの仕込みと戻り値
 # ---------------------------------------------------------------------------

tests/cli/tui/test_menu_pty.py

@@ -175,3 +175,44 @@ def test_answered_prompts_are_erased(session):
         if not ln.startswith("@") and _CPR_WARNING not in ln
     ]
     assert residue == [], f"プロンプト行が画面に残留: {residue}"
+
+
+_RAPID_KEYS_DRIVER = """
+from devbase.tui import menu
+
+OPS = [("再起動 (up)", "up"), ("停止 (down)", "down")]
+
+sel = menu.select("SELECT-RAPID を選択:", OPS, back=True)
+print("@SEL=" + ("BACK" if sel is menu.MENU_BACK else repr(sel)), flush=True)
+print("@END", flush=True)
+"""
+
+
+@pytest.fixture
+def rapid_session():
+    s = _PtySession(_RAPID_KEYS_DRIVER)
+    yield s
+    if s.proc.poll() is None:
+        s.proc.kill()
+
+
+def test_buffered_key_after_answer_does_not_crash(rapid_session):
+    """Esc 確定と同時に届いた Ctrl-C で exit が二重に呼ばれないこと。
+
+    Esc バインドは矢印キーのシーケンスと区別するため eager=False (確定待ち)
+    なので、Esc + Ctrl-C を 1 回の write で送ると prompt_toolkit は 1 回の
+    キー処理の中で「Esc ハンドラ (exit 確定) → 残りバッファ再処理で Ctrl-C
+    ハンドラ」を連続実行する。``_guard_after_done`` が無いと questionary
+    組み込みの Ctrl-C ハンドラが確定後に再度 ``Application.exit()`` を呼び、
+    「Return value already set. Application.exit() failed.」のクラッシュ画面が
+    出て入力待ちで固まる (実 TTY でのみ再現。Ctrl-C 連打でも同様)。
+    """
+    rapid_session.wait_for("SELECT-RAPID")
+    rapid_session.send("\x1b\x03")   # Esc 確定 + 直後の Ctrl-C を同一 write で送出
+    rapid_session.wait_for("@SEL=BACK")
+    rapid_session.wait_for("@END")
+
+    rapid_session.finish()
+    raw = bytes(rapid_session._buf).decode("utf-8", errors="replace")
+    assert "Application.exit() failed" not in raw
+    assert "Unhandled exception" not in raw