@tyhld/conductor 0.3.0 → 0.6.0

package/scripts/hooks/notification_hook.py

@@ -0,0 +1,167 @@
+#!/usr/bin/env python3
+"""notification_hook.py — Notification フック（★端末でしか押せない問いを管制へ出す・ADR-016）。
+
+【なぜ要るか（実機 2026-09-02 14:12）】
+  devlog-tracker の職人が端末でこう聞かれて止まっていた。★管制には1件も出ていなかった。
+
+      Network request outside of sandbox
+      Host: results-receiver.actions.githubusercontent.com
+      Do you want to allow this connection?
+      1. Yes  2. Yes, and don't ask again for …  3. No, …
+
+  直前に打っていたのは `gh run view … --log`（GitHub Actions の記録を読む操作）。
+  同じ型は 2026-09-01 にも Playwright（cdn.playwright.dev）で起きている。
+
+【調べて分かったこと（本体の実物で確認）】
+  ・この問いは PermissionRequest を鳴らさない（番人より外側の OS ゲート）。
+  ・しかし Claude Code 2.1.246 以降は【Notification フックを鳴らす】
+    （本体同梱の変更履歴に「サンドボックスの許可待ち中に Notification が鳴らない不具合を修正」）。
+    実機は 2.1.258 なので鳴る。
+  ・ところが Notification は【ホームにも現場にも1つも登録されていなかった】。
+    ＝鳴っていても誰も聞いていなかった。これが「管制に出ない」の根本原因。
+
+【この入口の役目】
+  1. 受けた通知を必ず1行記録する（見張り。送った／送らなかったを後から数えられる）
+  2. 「人が端末で操作しないと進まない」種類だけを、管制へカードとして出す
+  3. カードには★答えを付けない。この問いは端末でしか押せないので、
+     「端末で押してください」と分かる形で出し、押されるまで出したままにする
+
+【★二重に出さない】
+  PermissionRequest / Stop は自分でカードを出して待つ。その最中の通知は送らない
+  （hook_datalink の印で判る）。同じ止まりで2枚出す病気は ADR-013 で潰した。
+
+【入出力】
+  入力(stdin JSON): {"hook_event_name":"Notification","message":…,"title":…,
+                     "notification_type":…,"session_id":…,"cwd":…}
+  出力: 無し（Notification は判断を返せない入口。何があっても exit 0 で会話を止めない）
+"""
+import json
+import os
+import sys
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+import hook_datalink  # noqa: E402
+
+# 人が【端末で】操作しないと先へ進まない通知の種別。ここだけを管制へ出す。
+#   permission_prompt        … 許可待ち（サンドボックスの通信許可もここに乗る）
+#   worker_permission_prompt … 手伝いの職人が許可待ち
+#   idle_prompt              … 入力待ちのまま止まっている
+HUMAN_BLOCKING_TYPES = frozenset({
+    'permission_prompt',
+    'worker_permission_prompt',
+    'idle_prompt',
+})
+
+# 進行の知らせで、人が何かする必要のないもの（＝出さない）。
+#   ★「知らない種別は出す」側に倒す。新しい種別が増えたとき黙って見えなくなるより、
+#     1枚多く出るほうが害が小さい（見えない停止こそが今回の事故）。
+INFORMATIONAL_TYPES = frozenset({
+    'agent_completed',
+    'auth_success',
+    'computer_use_exit',
+    'elicitation_complete',
+    'elicitation_response',
+    'push_notification',
+})
+
+# カードを出したままにする上限（秒）。押されれば人は端末で先へ進む。
+#   ★フックの打ち切り(600秒)より先に切り上げ、後始末（カードを閉じる）を必ず走らせる。
+NOTIFY_WAIT_SEC = int(os.environ.get('CC2_NOTIFY_WAIT_SEC') or 540)
+
+
+def should_forward(notification_type):
+    """管制へ出すか。返り: (出すか, 出さない理由)"""
+    kind = (notification_type or '').strip()
+    if not kind:
+        return False, 'no-type'
+    if kind in INFORMATIONAL_TYPES:
+        return False, 'informational'
+    if kind in HUMAN_BLOCKING_TYPES:
+        return True, ''
+    # 知らない種別＝出す（見えない停止を作らないため安全側に倒す）。
+    return True, ''
+
+
+def build_payload(data):
+    """管制へ出すカードの中身。★答えの選択肢は付けない（端末でしか押せないため）。"""
+    cwd = data.get('cwd') or ''
+    site = os.environ.get('CONDUCTOR_SITE') or (cwd.rstrip('/').rsplit('/', 1)[-1] if cwd else '')
+    machine = os.environ.get('CONDUCTOR_MACHINE') or os.uname().nodename
+    message = (data.get('message') or '').strip()
+    title = (data.get('title') or '').strip()
+    kind = (data.get('notification_type') or '').strip() or 'unknown'
+    detail = ' / '.join(x for x in (title, message) if x)[:400]
+    lines = [
+        '★この確認は管制からは押せません。職人の端末（黒い画面）で選んでください。',
+        f'現場: {site or "不明"}／種別: {kind}',
+    ]
+    if detail:
+        lines.append(detail)
+    body = '\n'.join(lines)
+    return {
+        'site': site or 'unknown',
+        'machine': machine,
+        # ★sessionId は絶対に空で送らない（空だと管制が 400 で弾く）。
+        'sessionId': hook_datalink.stable_session_id(data.get('session_id'), machine, site),
+        'kind': 'message',
+        'question': body,
+        'summary': f'端末で操作が要ります（{site or "現場不明"}）',
+        'waitMs': NOTIFY_WAIT_SEC * 1000,
+    }
+
+
+def run(data, cfg=None, datalink=hook_datalink):
+    """返り値: 出したカードのID / None（出さなかった）。"""
+    kind = data.get('notification_type')
+    session_id = data.get('session_id') or ''
+
+    forward, reason = should_forward(kind)
+    if forward and datalink.handoff_active(session_id):
+        # 別のフック（PermissionRequest / Stop）が既に同じ止まりを管制へ出して待っている。
+        forward, reason = False, 'already-on-console'
+
+    cfg = datalink.load_config() if (forward and cfg is None) else cfg
+    if forward and cfg is None:
+        forward, reason = False, 'no-console'
+
+    datalink.record_notification(kind, forward, reason)   # ★出しても出さなくても必ず残す
+    if not forward:
+        return None
+
+    reg = datalink.register(cfg, build_payload(data))
+    if not reg or not reg.get('id'):
+        return None
+    request_id = reg['id']
+
+    # ★押されるまでカードを出したままにする＝「止まっていること」が管制で見え続ける。
+    #   管制側の答えは使わない（端末でしか押せない）。閉じる合図としてだけ受け取る。
+    datalink.mark_handoff(session_id)
+    answered = None
+    try:
+        answered = datalink.wait_for_decision(cfg, request_id, max_wait_sec=NOTIFY_WAIT_SEC)
+    finally:
+        datalink.abandon(cfg, request_id)   # 去るときは必ず閉じる（残骸カードを作らない）
+        datalink.clear_handoff(session_id)
+        # ★時間切れ＝「まだ端末で押されていない可能性が高い」。見えなくして終わりにせず、
+        #   後から数えられるように記録に残す（ADR-016 の見張り）。
+        if answered is None:
+            datalink.record_notification(kind, False, 'timeout-still-stuck')
+    return request_id
+
+
+def main() -> int:
+    try:
+        raw = sys.stdin.read()
+        data = json.loads(raw) if raw.strip() else {}
+    except Exception:  # noqa: BLE001
+        return 0
+    try:
+        run(data)
+    except Exception:  # noqa: BLE001
+        pass
+    return 0  # ★Notification は会話を止めない（何があっても exit 0・出力なし）
+
+
+if __name__ == '__main__':
+    sys.exit(main())

package/scripts/hooks/permission_request_hook.py

@@ -0,0 +1,207 @@
+#!/usr/bin/env python3
+"""permission_request_hook.py — PermissionRequest フック（段2・管制へデータで確認を渡す）。
+
+【いつ走るか】
+  Claude Code が「人に聞く必要がある」と判定して確認ダイアログを出す瞬間。
+  番人（cc2_guard.py）が関門を "ask" で返したときが主な入口。
+
+【何をするか】
+  1. 管制へ確認を1件登録する（POST /api/conductor/hook-requests）
+  2. 人の答えが出るまで待つ（GET を短く繰り返して最大600秒）
+  3. 答えを permissionDecision へ写して返す
+  4. 途中で去るときは DELETE（管制のカードを即座に閉じる）
+
+【★答えの扱い】
+  allow        → そのまま許可
+  allow_always → 許可＋「今後は聞かない」を updatedPermissions で登録する。
+                 ★配列形式で返す（文字列形式は 2.1.218 で無効＝検証済 27c5a9d1）。
+                 ★destination は既定 'session'（設定ファイルを汚さない）。
+                 ★ただし【関門】には恒久許可を与えない（allow へ落とす）。関門は毎回人が判断する。
+  deny         → 却下
+  答えが取れない（時間切れ・通信不能・管制未設定）→ 何も出力しない＝従来の確認ダイアログのまま。
+  ★「答えないこと」を許可に読み替えない。
+
+【接続先が未設定の環境】何もせず委譲する（このフックを入れても壊れない）。
+"""
+import json
+import os
+import sys
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+import hook_datalink  # noqa: E402
+import policy  # noqa: E402
+import summary_ja  # noqa: E402
+
+# 「今後は聞かない」の保存先。★'session' 固定（設定ファイルを絶対に書き換えない）。
+#   以前は CC2_ALLOW_ALWAYS_DESTINATION で上書きできたが、環境変数の設定ミス1つで
+#   localSettings に恒久 allow が書かれ、関門が静かに消える経路になっていた。塞ぐため固定する。
+ALLOW_ALWAYS_DESTINATION = 'session'
+
+
+def build_choices(gate=None):
+    """管制のカードに出す選択肢。端末の確認と同じ数にする（確定仕様）。
+
+    ・関門(gate)      … 2択［はい／いいえ］。★関門に「今後は聞かない」は出さない
+                        （恒久許可を与えない方針。decide でも gate は allow_always を無視する）。
+    ・関門でない確認  … 3択［はい／今後は聞かない／いいえ］。
+    取得できない状況は無い（gate の有無だけで決まる）が、将来の互換のため
+    空を返す場合は build_payload 側で送らない（管制側にフォールバックがある）。
+    """
+    yes = {'value': 'allow', 'label': 'はい'}
+    no = {'value': 'deny', 'label': 'いいえ'}
+    if gate:
+        return [yes, no]
+    return [yes, {'value': 'allow_always', 'label': '今後は聞かない'}, no]
+
+
+def build_payload(data, gate=None):
+    """管制へ送る値を組み立てる。★人が読む文言は日本語1行（summary）。"""
+    tool = data.get('tool_name')
+    tin = data.get('tool_input') if isinstance(data.get('tool_input'), dict) else {}
+    cwd = data.get('cwd') or ''
+    site = os.environ.get('CONDUCTOR_SITE') or (cwd.rstrip('/').rsplit('/', 1)[-1] if cwd else '')
+    machine = os.environ.get('CONDUCTOR_MACHINE') or os.uname().nodename
+    payload = {
+        'site': site or 'unknown',
+        'machine': machine,
+        # ★sessionId は絶対に空で送らない（空だと管制が 400 で弾く）。取れないときは
+        #   機械名・現場・親PID から復元可能な値を作る（hook_datalink.stable_session_id）。
+        'sessionId': hook_datalink.stable_session_id(data.get('session_id'), machine, site),
+        'kind': 'tool',
+        'toolName': tool or 'unknown',
+        'toolInput': tin,
+        'summary': summary_ja.summarize_tool(tool, tin, gate),
+        'waitMs': hook_datalink.MAX_WAIT_SEC * 1000,
+    }
+    # 選択肢を添える（取得できたときだけ。空なら送らない＝管制のフォールバックに任せる）。
+    choices = build_choices(gate)
+    if choices:
+        payload['choices'] = choices
+    return payload
+
+
+def build_updated_permissions(data):
+    """「今後は聞かない」を表す更新（★配列形式・destination は既定 session）。"""
+    tool = data.get('tool_name')
+    tin = data.get('tool_input') if isinstance(data.get('tool_input'), dict) else {}
+    rule = tin.get('command') if tool == 'Bash' else None
+    return [{
+        'type': 'addRules',
+        'rules': [{'toolName': tool, **({'ruleContent': rule} if rule else {})}],
+        'behavior': 'allow',
+        'destination': ALLOW_ALWAYS_DESTINATION,
+    }]
+
+
+# 「聞けない」ときに関門を止める理由（お客さまがカードで読む日本語）。
+#   ★不通（到達不能・認証・タイムアウト）用。相手側/連絡線の問題として「回復を待つ」を促す。
+_GATE_FAILCLOSE_MESSAGE = (
+    'お客さまへの確認（管制）に今つながらないため、安全のためこの操作を止めました。'
+    'マージ・本番反映・データベースの変更は、確認できないときは実行しません。'
+    '連絡線が回復してから、もう一度お試しください。'
+)
+
+# ★送信内容の不備（4xx）用。管制には届いているのに登録が弾かれた＝システム内部の不具合。
+#   「つながらない（待てば直る）」と取り違えないよう、別の文言で不具合として明示する。
+_GATE_DEFECT_MESSAGE = (
+    'お客さまへの確認（管制）の登録に失敗しました（システム内部の不具合の可能性）。'
+    '安全のためこの操作を止めました。時間をおいても直らない場合は担当者へご連絡ください。'
+)
+
+
+def _register_failure_kind(datalink):
+    """register が失敗した直後の種別（FAIL_*）を読む。読めなければ None（従来どおり不通扱い）。
+    ★テスト用の偽 datalink には last_failure_kind が無いことがあるため getattr で守る。"""
+    fn = getattr(datalink, 'last_failure_kind', None)
+    if not callable(fn):
+        return None
+    try:
+        return fn()
+    except Exception:  # noqa: BLE001
+        return None
+
+
+def _gate_deny_for_failure(kind):
+    """登録できなかったときに関門へ出す deny（種別で文言を分ける・止めるのは変えない）。"""
+    if kind == hook_datalink.FAIL_CLIENT:
+        return _deny(_GATE_DEFECT_MESSAGE)
+    return _deny(_GATE_FAILCLOSE_MESSAGE)
+
+
+def _deny(message):
+    return {'hookSpecificOutput': {
+        'hookEventName': 'PermissionRequest',
+        'decision': {'behavior': 'deny', 'message': message},
+    }}
+
+
+def decide(data, cfg=None, datalink=hook_datalink):
+    """1件分の判定。返り値 None は「何も出力しない（従来の確認ダイアログへ委譲）」。
+
+    ★関門(gate)は「聞けない」ときに委譲(None)ではなく deny で返す（フェイルクローズ）。
+      委譲すると端末の確認ダイアログに落ち、非対話の現場では黙って素通りするため。
+      関門でない確認は従来どおり委譲する（全部は止めない）。
+    """
+    # 何の関門かを先に知る（フェイルクローズの判断に要る）。
+    verdict = policy.decide_event(data)
+    gate = verdict.get('gate')
+
+    cfg = cfg if cfg is not None else datalink.load_config()
+    if cfg is None:
+        # 管制につながっていない環境。関門は素通りさせず止める。関門以外は従来どおり委譲。
+        return _deny(_GATE_FAILCLOSE_MESSAGE) if gate else None
+
+    reg = datalink.register(cfg, build_payload(data, gate))
+    if not reg or not reg.get('id'):
+        # 登録できない。関門は止める（フェイルクローズ）、関門以外は委譲。
+        # ★止めること自体は変えないが、原因が「送信内容の不備(4xx)」か「不通」かで文言を分ける。
+        #   400 を「つながらない」と読み替えない（報告 CC1・本ADR の根治）。
+        return _gate_deny_for_failure(_register_failure_kind(datalink)) if gate else None
+    request_id = reg['id']
+
+    # 既に答えが出ている（同じ確認の再送）ならそれを使う。
+    answer = reg.get('decision') if reg.get('status') == 'answered' else None
+    if answer is None:
+        # ★待っている間だけ「この止まりは管制が受け持っている」印を置く（ADR-016）。
+        #   Claude Code の Notification（「許可待ちです」）で同じ止まりが2枚出るのを防ぐ。
+        datalink.mark_handoff(data.get('session_id') or '')
+        try:
+            answer = datalink.wait_for_decision(cfg, request_id)
+        finally:
+            datalink.clear_handoff(data.get('session_id') or '')
+    if answer is None:
+        # 時間切れ・通信不能 → カードを閉じる。関門は止める、関門以外は従来ダイアログへ委譲。
+        datalink.abandon(cfg, request_id)
+        return _deny(_GATE_FAILCLOSE_MESSAGE) if gate else None
+
+    if answer == 'deny':
+        return _deny('お客さまが「いいえ」を選びました。')
+
+    decision = {'behavior': 'allow'}
+    tin = data.get('tool_input')
+    if isinstance(tin, dict) and tin:
+        decision['updatedInput'] = tin
+    if answer == 'allow_always' and not gate:
+        # ★関門には恒久許可を与えない（gate のときは付けない）。
+        decision['updatedPermissions'] = build_updated_permissions(data)
+    return {'hookSpecificOutput': {'hookEventName': 'PermissionRequest', 'decision': decision}}
+
+
+def main() -> int:
+    try:
+        raw = sys.stdin.read()
+        data = json.loads(raw) if raw.strip() else {}
+    except Exception:  # noqa: BLE001
+        return 0
+    try:
+        out = decide(data)
+        if out is not None:
+            sys.stdout.write(json.dumps(out, ensure_ascii=False))
+        return 0
+    except Exception:  # noqa: BLE001
+        return 0  # 何が起きても許可を勝手に出さない＝委譲
+
+
+if __name__ == '__main__':
+    sys.exit(main())