claude-code-conductor 2.71.0__tar.gz → 2.71.1__tar.gz

{claude_code_conductor-2.71.0 → claude_code_conductor-2.71.1}/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## [2.71.1] - 2026-08-14
+
+### 修正
+
+- **`c3 ask` の Windows 非対話判定が機能せず無限ハングする欠陥を修正した**（v2.71.0
+  「既知の問題」の解消）。`sys.stdin.isatty()` が Windows で stdin を NUL / `/dev/null` へ
+  リダイレクトした場合にも True を誤返却し、`--response` 未指定で exit 1 にならず
+  `msvcrt.getwch()`（実コンソール直読み）で無限ハングしていた。判定を
+  `stdin_is_interactive_console()`（Windows は GetConsoleMode で実コンソール接続を判定・
+  POSIX は従来どおり `isatty()`）へ置換し、入口ゲートと対話パス直前の二重ゲートを
+  同一判定へ統一した
+- 判定ヘルパーを共有モジュール `src/c3/_terminal.py` へ移設し、関数全体を一箇所集約で
+  fail-closed 化した（想定外例外でも非対話扱い＝exit 1 契約を維持・KeyboardInterrupt /
+  SystemExit は伝播。`question.py` 側は re-export でシーム維持）
+- `_read_key` の POSIX 分岐に空読み EOF ガードを追加した（EOF 時のビジーループを解消し
+  既存の exit 1 経路へ接続）
+- `docs/cli-reference.md` の v2.71 既知問題注記を修正後の実挙動へ更新した
+
+### 内部
+
+- `tests/test_cli_ask.py` に回帰・凍結テスト計 17 件を追加した（修正前状態での赤化実証・
+  内部依存注入による fail-closed 凍結・Windows 限定 subprocess+timeout 統合テストを含む。
+  フルスイート 3322 緑・CI 全緑 run 31755508514）
+
 ## [2.71.0] - 2026-08-14
 
 ### 追加

{claude_code_conductor-2.71.0 → claude_code_conductor-2.71.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: claude-code-conductor
-Version: 2.71.0
+Version: 2.71.1
 Summary: Multi-agent orchestration framework for Claude Code with Codex/Cursor adapters (C3)
 Project-URL: Homepage, https://github.com/satoh-y-0323/claude-code-conductor
 Project-URL: Repository, https://github.com/satoh-y-0323/claude-code-conductor

{claude_code_conductor-2.71.0 → claude_code_conductor-2.71.1}/src/c3/__init__.py

@@ -1,3 +1,3 @@
 """Claude Code Conductor (C3) - multi-agent orchestration framework for Claude Code."""
 
-__version__ = "2.71.0"
+__version__ = "2.71.1"

claude_code_conductor-2.71.1/src/c3/_terminal.py

@@ -0,0 +1,118 @@
+"""Shared terminal helpers for c3 CLI subcommands.
+
+Used by cli_doctor.py, cli_tier.py, cli_ask.py, question.py, and future
+cli_*.py to keep terminal-facing logic in one place:
+
+- ``supports_color`` / ``strip_ansi`` / ``sanitize_terminal_text``: ANSI
+  color support detection, escape-sequence stripping, and sanitization of
+  untrusted text before printing.
+- ``stdin_is_interactive_console``: console-attachment detection for stdin
+  (whether an interactive console is really attached, as opposed to a NUL /
+  pipe / file redirection).
+"""
+
+from __future__ import annotations
+
+import os
+import re
+import sys
+
+
+# CSI sequences with final byte 'm' (color / SGR). Other escape sequences
+# (e.g. \033c reset, OSC \033]0;...\007 title) are not handled by _strip_ansi
+# and would skew column-width calculations if they appeared in cell text.
+_CSI_M_RE = re.compile(r"\033\[[0-9;]*m")
+
+# Control characters disallowed when printing untrusted text to the terminal.
+# Allow newline (\n), tab (\t), carriage return (\r) but strip any other
+# C0 control or escape character (\x1b) so that ANSI/title-injection cannot
+# happen via DB-stored values such as ``current_step`` / ``error_message``.
+# Also strip the Unicode line/paragraph separators U+2028 / U+2029 so they
+# match the init-session SR L-2 sanitization spec (removes U+2028 / U+2029).
+# NEL (U+0085) is intentionally NOT included per that spec.
+_DISALLOWED_CONTROL_RE = re.compile("[\x00-\x08\x0b\x0c\x0e-\x1f\x7f" + chr(0x2028) + chr(0x2029) + "]")
+
+
+def supports_color() -> bool:
+    """Return True if stdout supports ANSI color sequences.
+
+    Honors the NO_COLOR environment variable (https://no-color.org/) and
+    requires stdout to be a TTY.
+    """
+    if os.environ.get("NO_COLOR"):
+        return False
+    if not sys.stdout.isatty():
+        return False
+    return True
+
+
+def strip_ansi(s: str) -> str:
+    """Remove ANSI CSI 'm' (SGR / color) sequences for visible-width calc.
+
+    Note: handles only ``\\033[...m`` style sequences. Other escape sequences
+    such as cursor movement (``\\033[H``), screen clear (``\\033[2J``), or
+    OSC title-set are intentionally not stripped here because the cells we
+    measure should never contain them.
+    """
+    return _CSI_M_RE.sub("", s)
+
+
+def sanitize_terminal_text(s: str) -> str:
+    """Strip control / escape characters from untrusted text before printing.
+
+    Used for DB-sourced strings (``current_step`` / ``error_message``) so
+    they cannot inject ANSI escape sequences (title, cursor, screen clear)
+    into the terminal. Newlines / tabs / carriage returns are preserved.
+    The Unicode line/paragraph separators U+2028 / U+2029 are also stripped
+    (init-session SR L-2 sanitization spec); NEL (U+0085) is out of scope.
+    """
+    if not s:
+        return s
+    return _DISALLOWED_CONTROL_RE.sub("", s)
+
+
+def stdin_is_interactive_console() -> bool:
+    """Return True only when stdin is attached to a real interactive console.
+
+    Windows の CRT ``_isatty()``（``sys.stdin.isatty()`` の内部実装）は対象ハンドルが
+    ``FILE_TYPE_CHAR`` かどうかしか見ておらず、NUL デバイス（``subprocess.DEVNULL`` /
+    ``NUL`` リダイレクト）もコンソールと同じ ``FILE_TYPE_CHAR`` に分類されるため
+    誤って True を返す（実測: debug-analysis-20260814-011032.md）。
+    Windows では ``GetConsoleMode``（実コンソールハンドルに対してのみ成功する
+    Win32 API）で実際のコンソール接続を確認し、NUL・パイプ・ファイルいずれの
+    リダイレクトも False として扱う。POSIX は ``isatty()`` が ``/dev/null`` に
+    対して標準どおり正しく False を返すためそのまま使う。
+
+    関数全体を ``except Exception`` で包み、内部でどの分岐がどの例外型を投げても
+    送出せず False を返す (fail-closed)。POSIX 分岐の ``sys.stdin.isatty()`` は
+    ``sys.stdin`` が None のケース (pythonw.exe / PyInstaller --windowed など) で
+    ``AttributeError`` を投げうるが、これを呼び出し元へ伝播させると
+    ``c3 ask`` の「exit 1」契約が破れる (SR-R-001 / CR-E-001)。呼び出し位置ごとに
+    try/except を足すのではなく、ここ一箇所へ集約して将来の呼び出し箇所追加にも
+    自動的に効かせる。``KeyboardInterrupt`` / ``SystemExit`` は ``BaseException``
+    直属で ``Exception`` を継承しないため握り潰さない (Ctrl-C は従来どおり伝播)。
+    """
+    try:
+        if os.name == "nt":
+            return _windows_console_attached()
+        return sys.stdin.isatty()
+    except Exception:
+        return False
+
+
+def _windows_console_attached() -> bool:
+    try:
+        import ctypes
+        import msvcrt
+        from ctypes import wintypes
+
+        handle = msvcrt.get_osfhandle(0)
+        kernel32 = ctypes.windll.kernel32
+        kernel32.GetConsoleMode.argtypes = [wintypes.HANDLE, ctypes.POINTER(wintypes.DWORD)]
+        kernel32.GetConsoleMode.restype = wintypes.BOOL
+        mode = wintypes.DWORD()
+        return bool(kernel32.GetConsoleMode(wintypes.HANDLE(handle), ctypes.byref(mode)))
+    except Exception:
+        # 判定不能ならすべて非対話扱いに倒す (fail-closed)。answer_questions() 経由の
+        # 呼び出し位置も含め、想定外の例外型で対話ゲートを突破させない。
+        return False

{claude_code_conductor-2.71.0 → claude_code_conductor-2.71.1}/src/c3/cli_ask.py

@@ -7,6 +7,7 @@ import json
 import sys
 from pathlib import Path
 
+from c3._terminal import stdin_is_interactive_console
 from c3.question import answer_questions, load_questions
 
 
@@ -47,9 +48,16 @@ def register(subparsers: argparse._SubParsersAction) -> None:
 
 
 def handle(args: argparse.Namespace) -> int:
-    if args.response is None and not sys.stdin.isatty():
-        print("c3 ask: --response is required in non-interactive mode", file=sys.stderr)
-        return 1
+    if args.response is None:
+        # 判定関数が想定外の例外を出しても traceback で落とさず「非対話」へ倒す
+        # (fail-closed)。--response 未指定時の exit 1 契約を例外型に依存させない。
+        try:
+            interactive = stdin_is_interactive_console()
+        except Exception:
+            interactive = False
+        if not interactive:
+            print("c3 ask: --response is required in non-interactive mode", file=sys.stderr)
+            return 1
 
     try:
         if args.file is not None:

{claude_code_conductor-2.71.0 → claude_code_conductor-2.71.1}/src/c3/question.py

@@ -13,6 +13,12 @@ from dataclasses import dataclass
 from pathlib import Path
 from typing import Any
 
+# stdin_is_interactive_console() は共有ターミナルヘルパー _terminal.py が正本
+# (CR-R-003)。ここでの import は question.py 内部の呼び出し元 (answer_questions /
+# _raw_keyboard_supported) が使うと同時に、``question.stdin_is_interactive_console``
+# という従来の参照名を後方互換で維持する re-export を兼ねる。
+from c3._terminal import stdin_is_interactive_console
+
 
 @dataclass(frozen=True)
 class Option:
@@ -63,7 +69,7 @@ def answer_questions(
         if current_response is not None:
             selected = _select_from_response(question, current_response)
         else:
-            use_interactive = sys.stdin.isatty() if interactive is None else interactive
+            use_interactive = stdin_is_interactive_console() if interactive is None else interactive
             selected = _select_interactively(question) if use_interactive else _select_default(question)
         answers.append(_answer_payload(question, selected))
     return {"answers": answers}
@@ -383,7 +389,16 @@ def _clear_screen() -> None:
 
 
 def _raw_keyboard_supported() -> bool:
-    return os.name in ("nt", "posix")
+    # OS が raw keyboard API を持つかではなく、今この stdin に実コンソールが
+    # 接続されているかを見る。旧実装（os.name in ("nt", "posix")）は事実上
+    # 常に True を返し、NUL/パイプ/ファイルへのリダイレクト時でも
+    # _select_with_line_input() の安全網（input() → EOFError）を恒久的に
+    # 迂回させ msvcrt.getwch() の無限ブロックを招いていた
+    # （debug-analysis-20260814-011032.md 欠陥B）。
+    # answer_questions() の外側ゲートと同じ判定をここでも呼ぶのは、interactive=True を
+    # 明示指定して外側ゲートを迂回する呼び出し元にも安全網を効かせるための意図的な
+    # 再検証であり、単純な重複ではない（削除・簡略化しないこと）。
+    return stdin_is_interactive_console()
 
 
 def _read_key() -> str:
@@ -410,6 +425,11 @@ def _read_key() -> str:
     try:
         tty.setraw(fd)
         ch = sys.stdin.read(1)
+        if ch == "":
+            # EOF（read が空文字列）。空文字列を返すと _select_interactively() の
+            # ループが再描画しながら回り続け CPU ビジーループになるため、
+            # EOFError を送出して呼び出し元（cli_ask.handle の except EOFError）へ委ねる。
+            raise EOFError("stdin reached EOF while reading a key")
         if ch == "\x1b":
             nxt = sys.stdin.read(2)
             return {"[A": "up", "[B": "down", "[C": "right", "[D": "left"}.get(nxt, "escape")