@tyhld/conductor 0.12.41 → 0.12.42

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tyhld/conductor",
-  "version": "0.12.41",
+  "version": "0.12.42",
   "description": "采配くん管制の見守りアプリ（conductor-agent）。各PCで常駐し、中央(devlog-tracker)へ定期的に生存報告(heartbeat)を送る常駐CLI。",
   "type": "module",
   "bin": {

package/scripts/hooks/permission_request_hook.py

@@ -155,6 +155,15 @@ _GATE_NO_ANSWER_MESSAGE = (
 )
 _GATE_REFUSED_MESSAGE = 'お客さまが「いいえ」を選んだので、実行しませんでした。'
 
+# 職人が人に答えを求める道具（Claude Code 組込み）。
+ASK_TOOL_NAME = 'AskUserQuestion'
+# ★自走モードでは答えを返す道がまだ無い（便 8c506176 で作る）。答えなしで進ませない。
+_ASK_NO_ANSWER_MESSAGE = (
+    '人の答えが無いので進めません（自走モードでは、この質問に答える人がいません）。'
+    '勝手に決めず、質問の中身（質問文と選択肢）を報告に書いて blocked として報告し'
+    '（conductor_complete も outcome=blocked で呼ぶ）、そこで止まって人の答えを待ってください。'
+)
+
 
 def ask_gate(data, gate, cfg, datalink=hook_datalink, wait_sec=None):
     """関門を管制の2択で聞き、答えが出るまで待つ（便 ab6e36ca・入口の番人から呼ぶ）。
@@ -232,6 +241,13 @@ def decide_autorun(data, verdict, gate, cfg=None, datalink=hook_datalink, codex=
     """
     if verdict.get('decision') == policy.DENY:
         return _deny(f'安全のため実行しません（{verdict.get("reason", "危険な操作")}）。')
+    if data.get('tool_name') == ASK_TOOL_NAME:
+        # ★職人の質問（AskUserQuestion）は、答えが無いまま通さない（便 7aefb5e1・えふさん確定 2026-09-21）。
+        #   以前は下の allow に落ち、updatedInput に元の入力（答え無し）が入るため、0.3秒で
+        #   「The user did not answer the questions.」として職人へ返り、人の判断を飛ばして進みえた
+        #   （2026-09-21 01:46 conductor「PR #194 を合流してよいか」3択・便 8c506176 の重大発見①）。
+        #   ★管制へ上げて答えを返す本体は便 8c506176 の範囲（ここでは作らない）。
+        return _deny(_ASK_NO_ANSWER_MESSAGE)
     if gate:
         return decide_by_asking(data, verdict, gate, cfg, datalink, codex)
 

package/scripts/hooks/test_ask_no_answer.py

@@ -0,0 +1,141 @@
+#!/usr/bin/env python3
+"""自走モードで職人の質問（AskUserQuestion）を「答えなし」で通さない検査（便 7aefb5e1）。
+
+  実行: python3 scripts/hooks/test_ask_no_answer.py
+  1件でも期待と違えば exit 1。
+
+【何が起きていたか（便 8c506176 の重大発見①・実物 2026-09-21 01:46 conductor）】
+  職人が AskUserQuestion で「PR #194 を今すぐ合流してよいですか？」と3択で聞いた。自走モードの
+  PermissionRequest が関門でないのでその場で allow を返し、updatedInput に元の入力（答え無し）を入れた。
+  0.3秒後に職人へ返ったのは「The user did not answer the questions.」（answers={}）＝人の判断を飛ばして進みえた。
+
+【何を守っているか】
+  (1) 自走モードの AskUserQuestion は allow にならない（deny）。答えの入っていない入力を返さない
+  (2) 職人へ返す文に「進めない」「blocked で報告」「止まって待つ」が入る
+  (3) 待たない・管制に頼らない：管制が無くても、登録が壊れても、同じ deny（時間切れで続行する道が無い）
+  (4) ほかの道具の扱いは変わらない（ふつうの命令は allow・関門は2択で聞く・危険は deny のまま）
+  (5) 従来どおり聞く動き（CC2_AUTORUN=0）は変えていない
+  ★空振り確認：直す前の版（origin/main）で流すと (1)(2) が落ちること（報告に実測を書く）。
+"""
+import os
+import sys
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+import permission_request_hook as prh  # noqa: E402
+
+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)[:40]}')
+
+
+CFG = {'url': 'https://example.invalid', 'token': 'dummy'}
+
+# 01:46 に実際に届いた形（中身は短くした）。
+ASK_INPUT = {'questions': [{
+    'question': 'PR #194 を今すぐ合流してよいですか？',
+    'header': '合流の可否',
+    'multiSelect': False,
+    'options': [{'label': '合流しない', 'description': '待つ'},
+                {'label': '今すぐ合流する', 'description': '配布も走る'}],
+}]}
+
+
+def event(tool, tin):
+    return {'hook_event_name': 'PermissionRequest', 'tool_name': tool, 'tool_input': tin,
+            'session_id': 's1', 'cwd': '/home/x/projects/demo'}
+
+
+class FakeDatalink:
+    def __init__(self, cfg=CFG, autorun=True, answer='allow', boom=False):
+        self.cfg, self.autorun, self.answer, self.boom = cfg, autorun, answer, boom
+        self.registered, self.waited, self.local = 0, 0, []
+
+    def autorun_enabled(self, env=None):
+        return self.autorun
+
+    def load_config(self):
+        return self.cfg
+
+    def record_autorun_local(self, tool, gate=None):
+        self.local.append((tool, gate))
+
+    def register(self, cfg, payload):
+        if self.boom:
+            raise RuntimeError('管制が壊れている')
+        self.registered += 1
+        return {'id': 'req-1', 'status': 'waiting'}
+
+    def wait_for_decision(self, cfg, request_id, max_wait_sec=None):
+        self.waited += 1
+        return self.answer
+
+    def mark_handoff(self, sid):
+        pass
+
+    def clear_handoff(self, sid):
+        pass
+
+    def abandon(self, cfg, request_id):
+        pass
+
+
+def decision(out):
+    return (out or {}).get('hookSpecificOutput', {}).get('decision', {})
+
+
+print('── (1)(2) 自走モードの AskUserQuestion は「答えなし」で通らない ' + '─' * 8)
+fake = FakeDatalink()
+d = decision(prh.decide(event('AskUserQuestion', ASK_INPUT), datalink=fake))
+check('(1) 判定は deny', d.get('behavior'), 'deny')
+check('(1) 答えの無い入力（updatedInput）を返さない', 'updatedInput' in d, False)
+msg = d.get('message', '')
+check('(2) 文に「進めません」', '進めません' in msg, True)
+check('(2) 文に「blocked として報告」', 'blocked として報告' in msg, True)
+check('(2) 文に「止まって」「待って」', '止まって' in msg and '待って' in msg, True)
+check('(2) 文に「勝手に決めず」', '勝手に決めず' in msg, True)
+check('(2) 文に「質問の中身」を報告に書く', '質問の中身' in msg, True)
+check('(1) 「聞かずに通した」と記録しない（allow の1行を書かない）', fake.local, [])
+
+print('── (3) 待たない・管制に頼らない（時間切れで続行する道が無い） ' + '─' * 8)
+check('(3) 管制へ登録しない', fake.registered, 0)
+check('(3) 答えを待たない', fake.waited, 0)
+for label, dl in [('管制が未設定', FakeDatalink(cfg=None)), ('管制が壊れている', FakeDatalink(boom=True))]:
+    check(f'(3) {label}でも deny', decision(prh.decide(event('AskUserQuestion', ASK_INPUT), datalink=dl)).get('behavior'), 'deny')
+check('(3) Codex から呼ばれても deny',
+      decision(prh.decide(event('AskUserQuestion', ASK_INPUT), datalink=FakeDatalink(), codex=True)).get('behavior'), 'deny')
+
+print('── (4) ほかの道具の扱いは変わらない ' + '─' * 30)
+fake = FakeDatalink()
+d = decision(prh.decide(event('Bash', {'command': 'npx tsc --noEmit'}), datalink=fake))
+check('(4) ふつうの命令は allow', d.get('behavior'), 'allow')
+check('(4) ふつうの命令は入力をそのまま返す', d.get('updatedInput'), {'command': 'npx tsc --noEmit'})
+check('(4) ふつうの命令は聞かずに通した記録を1行', fake.local, [('Bash', None)])
+d = decision(prh.decide(event('Read', {'file_path': '/home/x/projects/demo/README.md'}), datalink=FakeDatalink()))
+check('(4) 読むだけの道具は allow', d.get('behavior'), 'allow')
+fake = FakeDatalink(answer='allow')
+d = decision(prh.decide(event('Bash', {'command': 'npm publish'}), datalink=fake))
+check('(4) 関門は管制の2択で聞く（登録した）', fake.registered, 1)
+check('(4) 関門は人の「はい」で allow', d.get('behavior'), 'allow')
+d = decision(prh.decide(event('Bash', {'command': 'cat .env'}), datalink=FakeDatalink()))
+check('(4) 危険（秘密の読み出し）は deny のまま', d.get('behavior'), 'deny')
+check('(4) 危険の文は今までどおり', d.get('message', '').startswith('安全のため実行しません'), True)
+
+print('── (5) 従来どおり聞く動き（CC2_AUTORUN=0）は変えていない ' + '─' * 11)
+fake = FakeDatalink(autorun=False, answer='deny')
+d = decision(prh.decide(event('AskUserQuestion', ASK_INPUT), datalink=fake))
+check('(5) 聞く動きでは管制へ登録する（今までどおり）', fake.registered, 1)
+check('(5) 人の「いいえ」で deny（今までどおり）', d.get('behavior'), 'deny')
+
+print()
+if failures:
+    print(f'NG {len(failures)} 件')
+    for f in failures:
+        print('  ', f)
+    sys.exit(1)
+print('すべて期待どおり')