@tyhld/conductor 0.12.17 → 0.12.19

package/scripts/hooks/test_gate_wait.py

@@ -0,0 +1,218 @@
+#!/usr/bin/env python3
+"""関門は入口の番人が管制に聞いて【答えが返るまで実行させない】（便 ab6e36ca）の実物の試験。
+
+  実行: python3 scripts/hooks/test_gate_wait.py
+  1件でも期待と違えば exit 1。
+
+【なぜ要るか】
+  便 c525bb2b の実機: 職人が auto モードで動くと、関門を ask で返しても Claude Code は
+  確認の番人（管制の答え待ち）を待たずに実行を許した。入口の番人の deny はどのモードでも効くので、
+  入口の番人が自分で聞いて待ち、答えが無ければ deny を返す形にした。
+  ここでは【実物の cc2_guard.py を別の手続きとして起動】し、手元に立てた偽の管制と
+  本物の通信（HTTP）でやり取りさせて確かめる（関数の差し替えではない）。
+
+【何を守っているか】
+  ① 答えが来ない関門は deny（＝どのモードでも実行されない）。カードを閉じる（DELETE）
+  ② 「いいえ」は deny／「はい」は allow（この1回だけ）
+  ③ 管制に届かない関門は deny
+  ④ 管制が応答しないまま固まっても、最後の歯止めで上限の内側に deny を返す
+  ⑤ ふつうの操作は管制に1回も聞かない
+  ⑥ カードは2択・期限（waitMs）は待つ上限と同じ
+  ⑦ 同じ操作の答えを使い回さない（1回の実行ごとに新しい鍵・登録の時点の答えは使わない）
+
+  ★空振り確認: 待つ処理（ask_gate_now）を外して ask を返す形に戻すと ①②④ が落ちる。
+"""
+import json
+import os
+import socket
+import subprocess
+import sys
+import tempfile
+import threading
+import time
+from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
+from urllib.parse import parse_qs, urlparse
+
+HERE = os.path.dirname(os.path.abspath(__file__))
+GUARD = os.path.join(HERE, 'cc2_guard.py')
+
+failures = []
+
+
+def check(label, got, want):
+    ok = got == want
+    if not ok:
+        failures.append(f'{label}: got={got!r} want={want!r}')
+    print(f'{"OK " if ok else "NG "} {label:62} {str(got)[:50]}')
+
+
+class FakeConsole:
+    """偽の管制。mode で答え方を決める: allow / deny / none（答えない）/ hang（固まる）。"""
+
+    def __init__(self, mode):
+        self.mode = mode
+        self.posts, self.gets, self.deletes = [], 0, 0
+        console = self
+
+        class Handler(BaseHTTPRequestHandler):
+            def log_message(self, *a):  # 試験の出力を汚さない
+                pass
+
+            def _send(self, obj):
+                body = json.dumps(obj).encode()
+                self.send_response(200)
+                self.send_header('Content-Type', 'application/json')
+                self.send_header('Content-Length', str(len(body)))
+                self.end_headers()
+                self.wfile.write(body)
+
+            def do_POST(self):
+                n = int(self.headers.get('Content-Length') or 0)
+                console.posts.append(json.loads(self.rfile.read(n) or b'{}'))
+                if console.mode == 'preanswered':
+                    # 管制が同じ鍵の行をまとめ、前に出た答えを返してきた形
+                    self._send({'id': 'req-1', 'status': 'answered', 'decision': 'allow'})
+                    return
+                self._send({'id': 'req-1', 'status': 'waiting'})
+
+            def do_GET(self):
+                console.gets += 1
+                if console.mode == 'hang':
+                    time.sleep(30)
+                    return
+                if console.mode in ('allow', 'deny'):
+                    self._send({'status': 'answered', 'outcome': 'decided', 'decision': console.mode})
+                    return
+                wait_ms = int((parse_qs(urlparse(self.path).query).get('waitMs') or ['0'])[0])
+                time.sleep(wait_ms / 1000)
+                self._send({'status': 'waiting', 'outcome': 'pending'})
+
+            def do_DELETE(self):
+                console.deletes += 1
+                self._send({})
+
+        self.server = ThreadingHTTPServer(('127.0.0.1', 0), Handler)
+        self.server.daemon_threads = True
+        self.url = f'http://127.0.0.1:{self.server.server_address[1]}'
+        threading.Thread(target=self.server.serve_forever, daemon=True).start()
+
+    def close(self):
+        self.server.shutdown()
+
+
+def free_port():
+    s = socket.socket()
+    s.bind(('127.0.0.1', 0))
+    port = s.getsockname()[1]
+    s.close()
+    return port
+
+
+def run_guard(command, url, extra_env=None, home=None):
+    """実物の番人を起動し、(permissionDecision, 理由, かかった秒) を返す。"""
+    env = {
+        'PATH': os.environ.get('PATH', ''),
+        'HOME': home,
+        'CONDUCTOR_URL': url,
+        'CONDUCTOR_TOKEN': 'test-token-not-secret',
+        'CONDUCTOR_GUARD_LOG': os.path.join(home, 'guard.log'),
+        'CONDUCTOR_HOOK_LOG': os.path.join(home, 'hook.log'),
+        'CC2_GATE_WAIT_SEC': '2',
+    }
+    env.update(extra_env or {})
+    event = {'tool_name': 'Bash', 'tool_input': {'command': command, 'description': '見本'},
+             'cwd': '/home/x/projects/demo', 'session_id': 'gate-wait-test'}
+    t0 = time.time()
+    p = subprocess.run([sys.executable, GUARD], input=json.dumps(event), capture_output=True,
+                       text=True, env=env, timeout=60)
+    took = time.time() - t0
+    if not p.stdout.strip():
+        return None, None, took
+    out = json.loads(p.stdout)['hookSpecificOutput']
+    return out.get('permissionDecision'), out.get('permissionDecisionReason'), took
+
+
+# 関門 db の見本（危険語を含まない・実行はしない）。★psql は便 fc22bc34 で deny へ移したので、
+# 今も関門のまま残る「台本に SQL の字面を渡す形」を使う。
+GATE_CMD = 'python3 apply.py --mode grant'
+ROUTINE_CMD = 'npm test'
+
+with tempfile.TemporaryDirectory() as home:
+    print('── ① 答えが来ない関門は deny（どのモードでも実行されない） ' + '─' * 10)
+    c = FakeConsole('none')
+    d, reason, took = run_guard(GATE_CMD, c.url, home=home)
+    check('① 答えが来ない → deny', d, 'deny')
+    check('① 理由は人の言葉（確認が取れなかった）', '確認' in (reason or '') and '実行しません' in reason, True)
+    check('① ★ask を返さない（auto モードで通りうる返事をしない）', d != 'ask', True)
+    check('① 管制へ1回だけ聞いた（カード1枚）', len(c.posts), 1)
+    check('① 答えを待った（問い合わせ1回以上）', c.gets >= 1, True)
+    check('① 答えが来ないのでカードを閉じた', c.deletes, 1)
+    check('① 待つ上限（2秒）で返した', 1.5 <= took < 10, True)
+    c.close()
+
+    print('── ② 「いいえ」は deny／「はい」は allow ' + '─' * 26)
+    c = FakeConsole('deny')
+    d, reason, _ = run_guard(GATE_CMD, c.url, home=home)
+    check('② 「いいえ」→ deny', d, 'deny')
+    check('② 理由に「いいえ」', '「いいえ」' in (reason or ''), True)
+    c.close()
+    c = FakeConsole('allow')
+    d, reason, _ = run_guard(GATE_CMD, c.url, home=home)
+    check('② 「はい」→ allow', d, 'allow')
+    check('② 「はい」のときカードは閉じない（答えがある）', c.deletes, 0)
+    c.close()
+
+    print('── ③ 管制に届かない関門は deny ' + '─' * 36)
+    d, reason, took = run_guard(GATE_CMD, f'http://127.0.0.1:{free_port()}', home=home)
+    check('③ 届かない → deny', d, 'deny')
+    check('③ 理由は人の言葉（つながらない）', 'つながらない' in (reason or ''), True)
+    check('③ すぐ返る', took < 10, True)
+
+    print('── ④ 管制が固まっても、最後の歯止めで上限の内側に deny ' + '─' * 12)
+    c = FakeConsole('hang')
+    d, reason, took = run_guard(GATE_CMD, c.url, home=home,
+                                extra_env={'CC2_GATE_WAIT_SEC': '20', 'CC2_GATE_HARD_LIMIT_SEC': '3'})
+    check('④ 固まった管制 → deny', d, 'deny')
+    check('④ 歯止め（3秒）で返した', 2.5 <= took < 8, True)
+    check('④ 理由は人の言葉（時間内に取れなかった）', '時間内' in (reason or ''), True)
+    c.close()
+
+    print('── ⑤ ふつうの操作は管制に1回も聞かない ' + '─' * 28)
+    c = FakeConsole('none')
+    d, _, took = run_guard(ROUTINE_CMD, c.url, home=home)
+    check('⑤ npm test → allow', d, 'allow')
+    check('⑤ 管制へ0回', (len(c.posts), c.gets), (0, 0))
+    check('⑤ すぐ返る', took < 5, True)
+    c.close()
+
+    print('── ⑥ カードは2択・期限は待つ上限と同じ ' + '─' * 28)
+    c = FakeConsole('allow')
+    run_guard(GATE_CMD, c.url, home=home)
+    p = c.posts[0] if c.posts else {}
+    check('⑥ 2択（今後は聞かないを出さない）', [x.get('value') for x in p.get('choices', [])],
+          ['allow', 'deny'])
+    check('⑥ カードの期限 = 待つ上限（2秒）', p.get('waitMs'), 2000)
+    check('⑥ 題はデータベースの変更', (p.get('summary') or '').startswith('【お客さまの確認：データベース'),
+          True)
+    c.close()
+
+    print('── ⑦ 同じ操作の答えを使い回さない ' + '─' * 34)
+    c = FakeConsole('allow')
+    run_guard(GATE_CMD, c.url, home=home)
+    run_guard(GATE_CMD, c.url, home=home)
+    keys = [x.get('requestKey') for x in c.posts]
+    check('⑦ 同じ操作を2回 → カードも2枚・鍵は毎回ちがう', len(set(k for k in keys if k)), 2)
+    c.close()
+    c = FakeConsole('preanswered')
+    d, _, _ = run_guard(GATE_CMD, c.url, home=home)
+    check('⑦ ★登録の時点で付いてきた「はい」は使わず、答えを待つ（来なければ deny）', d, 'deny')
+    c.close()
+
+print('\n' + '=' * 72)
+if failures:
+    print(f'NG {len(failures)} 件:')
+    for f in failures:
+        print('  -', f)
+    sys.exit(1)
+print('すべて期待どおり')
+sys.exit(0)

package/scripts/hooks/test_policy.py

@@ -40,11 +40,8 @@ def check(label, got, want):
 # (a) 3関門は必ず gate（＝人へ）
 # ─────────────────────────────────────────────────────────────────────────────
 GATE_CASES = [
-    # 関門1 本番ブランチの直接変更（★PRのマージは関門ではない → (a4) で確かめる）
-    ('git push origin main', policy.GATE_MERGE),
-    ('git push -u origin main', policy.GATE_MERGE),
-    ('git push origin HEAD:master', policy.GATE_MERGE),
-    ('git push origin production', policy.GATE_MERGE),
+    # ★旧 関門1（main など保護ブランチへの直接 push）は便 fc22bc34 で deny へ移した
+    #   → test_bypass_deny.py で確かめる。PRのマージは関門ではない → (a4)。
     # 関門2 本番確認
     ('vercel --prod', policy.GATE_RELEASE),
     ('vercel deploy --prod', policy.GATE_RELEASE),
@@ -56,14 +53,10 @@ GATE_CASES = [
     ('supabase functions deploy hello', policy.GATE_RELEASE),
     ('npm run deploy -- --production', policy.GATE_RELEASE),
     # 関門3 DB実行
-    ('psql -c "select 1"', policy.GATE_DB),
-    ('npx prisma migrate deploy', policy.GATE_DB),
-    ('npx prisma db push', policy.GATE_DB),
-    ('supabase db push', policy.GATE_DB),
-    ('supabase migration up', policy.GATE_DB),
-    ('npx drizzle-kit push', policy.GATE_DB),
+    # ★本番 DB を直接変える道具（psql・prisma db push / migrate deploy・supabase db push /
+    #   migration up・drizzle-kit push 等）は便 fc22bc34 で deny へ移した（設定の deny と揃える）
+    #   → test_bypass_deny.py。ここに残るのは、それ以外の DB 実行（見分けられないので人に聞く）。
     ('createdb newdb', policy.GATE_DB),
-    ('echo "CREATE TABLE x (id uuid)" | psql', policy.GATE_DB),
     ('supabase db query "GRANT SELECT ON x TO authenticated"', policy.GATE_DB),
     ('run-sql "ALTER TABLE x ENABLE ROW LEVEL SECURITY"', policy.GATE_DB),
     ('apply "CREATE POLICY p ON x"', policy.GATE_DB),
@@ -81,10 +74,12 @@ for cmd, want_gate in GATE_CASES:
 class GuardDatalink:
     """cc2_guard 用の偽 datalink（到達可否を固定し、ネットワークもディスクも触らない）。"""
 
-    def __init__(self, cfg, reachable):
+    def __init__(self, cfg, reachable, answer='allow'):
         self._cfg = cfg
         self._reachable = reachable
+        self._answer = answer
         self.decisions = []
+        self.registered = 0
 
     def load_config(self):
         return self._cfg
@@ -92,20 +87,40 @@ class GuardDatalink:
     def ping(self, cfg):
         return self._reachable
 
+    # ★便 ab6e36ca: 入口の番人が関門をその場で聞く。届かなければ登録できない。
+    def register(self, cfg, payload):
+        self.registered += 1
+        return {'id': 'r1', 'status': 'waiting'} if self._reachable else None
+
+    def wait_for_decision(self, cfg, request_id, max_wait_sec=None):
+        return self._answer
+
+    def abandon(self, cfg, request_id):
+        pass
+
     def record_decision(self, tool_name, decision, gate=None):
         self.decisions.append((tool_name, decision, gate))
 
 
-GATE_CMDS = ['git push origin main', 'vercel --prod', 'npx prisma migrate deploy',
-             'git push -u origin master', 'npm publish', 'supabase db push',
-             'psql -c "select 1"']
+GATE_CMDS = ['vercel --prod', 'npm publish', 'createdb newdb',
+             'supabase db query "select 1"', 'run-sql "ALTER TABLE x ADD COLUMN y int"']
 
-print('\n=== (a2) 関門は「管制に到達できる」ときは ask（確認を必ず出す） ===')
+print('\n=== (a2) 関門は入口の番人がその場で管制に聞き、答えで allow / deny を返す（便 ab6e36ca） ===')
 gd_ok = GuardDatalink({'url': 'http://x', 'token': 't'}, reachable=True)
 for cmd in GATE_CMDS:
     out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': cmd}}, datalink=gd_ok)
-    check(f'{cmd} → ask（到達可）', out['hookSpecificOutput']['permissionDecision'] if out else None, 'ask')
+    check(f'{cmd} → allow（「はい」）', out['hookSpecificOutput']['permissionDecision'] if out else None,
+          'allow')
+check('関門ごとに管制へ1回聞いた', gd_ok.registered, len(GATE_CMDS))
 check('起動記録が残る（呼ばれた事実）', len(gd_ok.decisions) >= len(GATE_CMDS), True)
+for answer in ('deny', None, 'weird'):
+    gd = GuardDatalink({'url': 'http://x', 'token': 't'}, reachable=True, answer=answer)
+    out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'}},
+                                 datalink=gd)
+    check(f'答え {answer!r} → deny', out['hookSpecificOutput']['permissionDecision'], 'deny')
+for cmd in GATE_CMDS:
+    out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': cmd}}, datalink=gd_ok)
+    check(f'★ask は返さない: {cmd[:40]}', out['hookSpecificOutput']['permissionDecision'] != 'ask', True)
 
 print('\n=== (a3) ★フェイルクローズ: 聞けないとき関門は deny（素通りさせない） ===')
 gd_ng = GuardDatalink({'url': 'http://x', 'token': 't'}, reachable=False)
@@ -115,7 +130,7 @@ for cmd in GATE_CMDS:
     check(f'{cmd} → deny（管制不通）', got, 'deny')
 
 gd_unset = GuardDatalink(None, reachable=False)  # 管制未設定（load_config が None）
-out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'git push origin main'}},
+out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'}},
                              datalink=gd_unset)
 check('管制未設定でも関門は deny', out['hookSpecificOutput']['permissionDecision'], 'deny')
 
@@ -127,11 +142,17 @@ out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'ca
                              datalink=gd_unset)
 check('危険は管制不通でも deny のまま', out['hookSpecificOutput']['permissionDecision'], 'deny')
 
-# 無効化フラグ（人間が「不通の間は止めない」と決めたとき）→ 止めずに ask（確認は出す）。
+# ★便 ab6e36ca: 以前の無効化フラグ（CC2_GATE_FAILCLOSE=0 で止めずに ask）は外した。
+#   ask は auto モードでは人の答えを待たずに実行されうる（便 c525bb2b）＝止めが消える回避路になるため。
 os.environ['CC2_GATE_FAILCLOSE'] = '0'
-out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'git push origin main'}},
+out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'}},
                              datalink=gd_unset)
-check('CC2_GATE_FAILCLOSE=0 なら関門は ask', out['hookSpecificOutput']['permissionDecision'], 'ask')
+check('★CC2_GATE_FAILCLOSE=0 でも関門は deny（回避路を作らない）',
+      out['hookSpecificOutput']['permissionDecision'], 'deny')
+out = cc2_guard.build_output({'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'}},
+                             datalink=gd_ng)
+check('★CC2_GATE_FAILCLOSE=0 でも管制不通の関門は deny',
+      out['hookSpecificOutput']['permissionDecision'], 'deny')
 os.environ.pop('CC2_GATE_FAILCLOSE', None)
 
 # 関門でも、人に見せる日本語1行は用意できている（管制のカードへ送る文言）。
@@ -150,7 +171,8 @@ MERGE_ALLOW = [
     'gh pr merge --auto 12',
     'gh pr merge 3 --merge --delete-branch',
     'gh pr merge --rebase 7',
-    'gh api -X PUT repos/o/r/pulls/1/merge',
+    # ★API での合流（gh api -X PUT …/pulls/N/merge）は便 fc22bc34 で deny へ移した
+    #   （保護の迂回・えふさん確定 2026-09-18）→ test_bypass_deny.py。
     'cd /srv/app && gh pr merge 12 --squash',       # 複合コマンドでも自走
     'bash -c "gh pr merge 12 --squash"',            # 包んでも自走
     'gh pr merge 12 --squash && git pull',
@@ -191,12 +213,12 @@ for _gd in (GuardDatalink({'url': 'http://x', 'token': 't'}, reachable=True),
           out['hookSpecificOutput']['permissionDecision'], 'allow')
 
 # ★マージを外しても、他の関門と危険は一切緩めていない（取り違え防止の回帰テスト）。
-check('main 直 push は関門のまま',
-      policy.decide_bash('git push origin main')['gate'], policy.GATE_MERGE)
+check('main 直 push は deny（便 fc22bc34 で関門から移した）',
+      policy.decide_bash('git push origin main')['decision'], policy.DENY)
 check('force push は deny のまま',
       policy.decide_bash('git push --force origin main')['decision'], policy.DENY)
-check('DB実行は関門のまま',
-      policy.decide_bash('supabase db push')['gate'], policy.GATE_DB)
+check('DB実行（見分けられないもの）は関門のまま',
+      policy.decide_bash('createdb newdb')['gate'], policy.GATE_DB)
 check('SQL DDL は関門のまま',
       policy.decide_bash('run-sql "CREATE TABLE x (id int)"')['gate'], policy.GATE_DB)
 check('本番反映は関門のまま',
@@ -537,12 +559,11 @@ check('枝への push は通常の文言',
       summary_ja.summarize_bash('git push origin feat/main-fix'),
       '作業内容をGitHubへ送ってよいですか')
 # ★見出しは「関門に当たったセグメント」から作る（複合コマンドの先頭語 cd に引きずられない）。
-check('複合コマンドでも関門セグメントから見出し（main直変更）',
-      summary_ja.summarize_bash('cd /srv/app && git push origin main', 'merge'),
-      '【お客さまの確認：本番の元を直接変更】本番の元（main）を直接書き換えてよいですか')
-check('複合コマンドでも関門セグメントから見出し（DB）',
-      summary_ja.summarize_bash('cd /srv && supabase db push', 'db'),
-      '【お客さまの確認：データベースの変更】データベースの構造を変更してよいですか')
+# ★main 直 push と本番 DB を直接変える道具は、便 fc22bc34 で関門から deny へ移した
+#   （カードは出ない）。見出しの作り方は、今も関門のままの操作で確かめる。
+check('複合コマンドでも関門セグメントから見出し（公開）',
+      summary_ja.summarize_bash('cd /srv/app && npm publish', 'release'),
+      '【お客さまの確認：本番へ反映】部品を公開（配布）してよいですか')
 
 
 # ─────────────────────────────────────────────────────────────────────────────
@@ -634,7 +655,7 @@ check('保存先は session（設定を汚さない）', up[0]['destination'], '
 
 # 関門には恒久許可を与えない。
 fd = FakeDatalink('allow_always')
-GATE_EV = {'tool_name': 'Bash', 'tool_input': {'command': 'git push origin main'},
+GATE_EV = {'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'},
            'cwd': '/home/fshim/projects/conductor', 'session_id': 's1'}
 out = permission_request_hook.decide(GATE_EV, cfg={'url': 'http://x', 'token': 't'}, datalink=fd)
 check('関門に恒久許可を与えない',
@@ -664,7 +685,7 @@ check('送る payload に choices が入る（3択）',
 
 # ★フェイルクローズ: 関門は「聞けない」とき deny を返す（委譲＝素通りにしない）。
 print('\n=== (g2) PermissionRequest フックのフェイルクローズ（関門のみ） ===')
-GATE_PR_EV = {'tool_name': 'Bash', 'tool_input': {'command': 'git push origin main'},
+GATE_PR_EV = {'tool_name': 'Bash', 'tool_input': {'command': 'npm publish'},
               'cwd': '/home/fshim/projects/conductor', 'session_id': 's1'}