@tyhld/conductor 0.11.9 → 0.11.11

package/package.json

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

package/scripts/hooks/hook_datalink.py

@@ -483,73 +483,15 @@ def autorun_enabled(env=None):
     return not _AUTORUN_OFF_RE.match(str(raw).strip())
 
 
-# 自走の履歴を管制へ送るときの上限時間（秒）。★短くする。
-#   これは「あとで人が読む記録」であって、職人を待たせてよい理由が1つも無い。
-#   管制が遅い・落ちているときに職人の手が止まらないよう、通常の 35 秒ではなくここで切る。
-AUTORUN_SEND_TIMEOUT_SEC = 4
-
-
-def record_autorun_local(tool_name, gate=None, sent=True, reason='', env=None):
+def record_autorun_local(tool_name, gate=None, env=None):
     """「聞かずに通した（自走）」1回を、この機の記録に1行残す。
 
-    形: `<ISO時刻> Autorun <tool> -> allow[ gate=<種別>][ 管制へ送れず(<理由>)]`
-    ★管制へ送れなくても【必ず】ここには残す（＝黙って通したことにならない）。
-    ★秘密もコマンド全文も残さない: ツール名・関門種別・送れたか、だけ。
+    形: `<ISO時刻> Autorun <tool> -> allow[ gate=<種別>]`
+    ★管制へは何も送らない（えふさん決定 2026-09-07・この記録は不要）。残すのはこの1行だけで、
+      「黙って通したことにならない」状態は保つ（後から何を通したか数えられる）。
+    ★秘密もコマンド全文も残さない: ツール名と関門種別だけ。
     """
     safe_tool = str(tool_name or 'unknown').split()[0][:40]
     suffix = f' gate={str(gate)[:16]}' if gate else ''
-    if not sent:
-        suffix += f' 管制へ送れず({str(reason)[:40]})'
     line = f'{time.strftime("%Y-%m-%dT%H:%M:%S%z")} Autorun {safe_tool} -> allow{suffix}\n'
     return _append_line(decision_log_path(env), line)
-
-
-def record_autorun(cfg, payload):
-    """「聞かずに通した（自走）」1件を管制へ【履歴として】送る。返り: True/False。
-
-    【1手で済むようになった（管制 PR#276・2026-09-06）】
-      管制が kind='auto' を受け付け、その行は status='recorded' で生まれる＝
-      「確認待ち」を1度も通らないので、カードとして人の前に出ない。
-      ★以前は「登録してすぐ閉じる（POST→DELETE）」でカードを消していた。管制が
-        『人に聞くカード』しか受け取らなかった頃の逃げ方で、登録から DELETE までの
-        一瞬だけカードが出得た。もう要らないので【丸ごと消した】。
-
-    【古い管制への互換（★止まらないための逃げ道）】
-      kind='auto' を知らない版は 400（こちらの送信内容の不備）で弾く。そのときだけ、
-      昔の形（kind='message' ＋ 登録してすぐ閉じる）で送り直す＝記録は必ず残る。
-      ★400 以外（不通・401・5xx）では送り直さない。相手が受け取れない理由が違うので、
-        同じものを2回投げても増えるのは負荷だけ。
-
-    ★何が起きても職人は止めない（例外は握る）。送れなかった事実は _request が1行残す。
-    """
-    try:
-        reg = _request(cfg, 'POST', '/api/conductor/hook-requests', payload,
-                       timeout=AUTORUN_SEND_TIMEOUT_SEC)
-        if reg and reg.get('id'):
-            return True
-        if last_failure_kind() != FAIL_CLIENT:
-            return False   # 不通・認証・管制側異常 … 送り直しても同じ。次の機会に任せる。
-        return _record_autorun_legacy(cfg, payload)
-    except Exception:  # noqa: BLE001  記録のために職人を止めない
-        return False
-
-
-def _record_autorun_legacy(cfg, payload):
-    """★古い管制（kind='auto' を知らない版）への送り直し。返り: True/False。
-
-    昔の形＝「人に聞くカード」として登録し、その場で閉じる（status='abandoned'）。
-    一覧は『確認待ち』の行しか出さないのでカードにはならず、行は残るので後から読める。
-    ★この道は互換のためだけに残している。管制が新しければ1手目で終わる（ここへ来ない）。
-    """
-    legacy = dict(payload)
-    legacy['kind'] = 'message'
-    # 古い管制の kind='message' は question が必須。人が読む1行をそのまま入れる。
-    legacy['question'] = payload.get('summary') or '（自走で通しました）'
-    legacy['waitMs'] = 0
-    reg = _request(cfg, 'POST', '/api/conductor/hook-requests', legacy,
-                   timeout=AUTORUN_SEND_TIMEOUT_SEC)
-    if not reg or not reg.get('id'):
-        return False
-    _request(cfg, 'DELETE', f"/api/conductor/hook-requests/{reg['id']}",
-             timeout=AUTORUN_SEND_TIMEOUT_SEC)
-    return True

package/scripts/hooks/permission_request_hook.py

@@ -24,8 +24,9 @@
 【★自走モード（既定・えふさん決定 2026-09-06「すべてを自走できるようにして」）】
   上の1〜4は【聞く】ための手順で、自走モードでは通らない。自走モードでは:
     ・人には一切聞かない。その場で allow を返す（★Claude Code 自身の確認画面にも落とさない）。
-    ・代わりに「聞かずに通した」を管制へ【履歴として】送る（★3ボタンのカードは出さない）。
-      送れなくても職人は止めない（送れなかった事実はこの機の記録に1行残る）。
+    ・★管制へは何も送らない（えふさん決定 2026-09-07）。2026-09-06 の自走化では「聞かずに
+      通した」を管制へ1件ずつ送っていたが、要らないと決まったので送る仕組みごと消した。
+      残すのはこの機の記録1行だけ（$HOME/.conductor-guard.log の `Autorun …`）。
     ・★deny（持ち出し・秘密・番人の設定への書き込み等）は1ミリも変えない。自走でも通さない。
   切り替えは【1か所】: 環境変数 CC2_AUTORUN、または $HOME/.conductor.env の CC2_AUTORUN。
     未設定＝自走。`CC2_AUTORUN=0`（0/false/no/off）で【従来どおり3ボタンで聞く】動きに戻る。
@@ -35,7 +36,6 @@
 """
 import json
 import os
-import re
 import sys
 
 sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
@@ -49,33 +49,6 @@ import summary_ja  # noqa: E402
 #   localSettings に恒久 allow が書かれ、関門が静かに消える経路になっていた。塞ぐため固定する。
 ALLOW_ALWAYS_DESTINATION = 'session'
 
-# 自走の記録として管制へ送るときの種別（管制 PR#276 が 'recorded' で受ける＝カードにしない）。
-#   ★古い管制はこれを知らず 400 で弾く。そのときは hook_datalink が昔の形で送り直す（互換）。
-AUTORUN_KIND = 'auto'
-
-# 履歴の1行説明の頭に付ける印（人が読んで「判断を求められていない」と分かるように）。
-AUTORUN_SUMMARY_PREFIX = '【自走で通しました】'
-# 聞くときの見出し「【お客さまの確認：本番へ反映】」を、履歴の見出しへ置き換えるための形。
-_ASK_HEAD_RE = re.compile(r'^【お客さまの確認：([^】]*)】')
-# 「…してよいですか」を「…しています」に直す（履歴なのに問いかけのままだと人が迷う）。
-_ASK_TAIL_RE = re.compile(r'([てで])よいですか\s*$')
-
-
-def autorun_summary(summary):
-    """聞くための1行説明を、読むための1行説明（履歴）に直す。
-
-    例) 【お客さまの確認：本番へ反映】…を実行してよいですか
-        → 【自走で通しました：本番へ反映】…を実行しています
-    ★形が想定と違っても必ず印は付ける（読んだ人が「判断を求められていない」と分かることが第一）。
-    """
-    text = (summary or '').strip()
-    head = _ASK_HEAD_RE.match(text)
-    if head:
-        text = f'【自走で通しました：{head.group(1)}】' + _ASK_HEAD_RE.sub('', text)
-    else:
-        text = AUTORUN_SUMMARY_PREFIX + text
-    return _ASK_TAIL_RE.sub(r'\1います', text)
-
 
 def build_choices(gate=None):
     """管制のカードに出す選択肢。端末の確認と同じ数にする（確定仕様）。
@@ -189,22 +162,6 @@ def autorun_enabled(env=None, datalink=hook_datalink):
         return False
 
 
-def build_autorun_payload(data, gate=None):
-    """管制へ【履歴として】送る値。★種別は 'auto'（人に聞くカードではない・管制 PR#276）。
-
-    ★主役は summary（人がそのまま読む1行）。管制はこれを「自走で通した」の一覧に出す。
-      例) 【自走で通しました：本番へ反映】部品を公開（配布）しています
-    ★toolName / toolInput は「実行された中身」として今までどおり添える（あとで追える）。
-    ★聞くための持ち物（待ち時間・選択肢）は付けない。誰も待っていないし、押させないため。
-    """
-    payload = build_payload(data, gate)
-    payload['kind'] = AUTORUN_KIND
-    payload['summary'] = autorun_summary(payload.get('summary'))
-    payload.pop('waitMs', None)
-    payload.pop('choices', None)
-    return payload
-
-
 def decide_autorun(data, verdict, gate, cfg=None, datalink=hook_datalink, codex=False):
     """自走モードの判定。★人には一切聞かず、その場で allow を返す。
 
@@ -215,18 +172,14 @@ def decide_autorun(data, verdict, gate, cfg=None, datalink=hook_datalink, codex=
     if verdict.get('decision') == policy.DENY:
         return _deny(f'安全のため実行しません（{verdict.get("reason", "危険な操作")}）。')
 
-    # 管制へ履歴を送る。★送れても送れなくても、この下の allow は変わらない（職人は止まらない）。
-    sent, why = False, 'not-configured'
+    # ★管制へ履歴は送らない（えふさん決定 2026-09-07・この記録は不要）。
+    #   2026-09-06 の自走化(便 7abe05e2)で「聞かずに通した」を管制へ1件ずつ送っていたが、
+    #   要らないと決まったので【送る仕組みごと消した】。設定で切り替える形にはしない
+    #   （切り替えを残すと「いま送っているのか」を毎回確かめないと分からなくなる）。
+    #   ★この機の記録には今までどおり1行だけ残す（何を通したかを後から数えられる状態は保つ）。
     try:
-        cfg = cfg if cfg is not None else datalink.load_config()
-        if cfg is not None:
-            sent = bool(datalink.record_autorun(cfg, build_autorun_payload(data, gate)))
-            why = '' if sent else 'send-failed'
+        datalink.record_autorun_local(data.get('tool_name'), gate)
     except Exception:  # noqa: BLE001  記録のために職人を止めない
-        sent, why = False, 'error'
-    try:
-        datalink.record_autorun_local(data.get('tool_name'), gate, sent, why)
-    except Exception:  # noqa: BLE001
         pass
 
     decision = {'behavior': 'allow'}

package/scripts/hooks/test_autorun.py

@@ -61,8 +61,9 @@ def behavior(out):
 class FakeDatalink:
     """管制の代わり。呼ばれた道筋を全部覚える。"""
 
-    def __init__(self, cfg=CFG, send_ok=True, autorun=True):
-        self.cfg, self.send_ok, self.autorun = cfg, send_ok, autorun
+    def __init__(self, cfg=CFG, autorun=True):
+        self.cfg, self.autorun = cfg, autorun
+        # sent は「管制へ送ってしまった件数」。★0 のままであることを固定するために残す。
         self.sent, self.local, self.asked, self.marks = [], [], 0, 0
 
     # ── 自走で使うもの ──
@@ -72,12 +73,8 @@ class FakeDatalink:
     def load_config(self):
         return self.cfg
 
-    def record_autorun(self, cfg, payload):
-        self.sent.append(payload)
-        return self.send_ok
-
-    def record_autorun_local(self, tool, gate=None, sent=True, reason=''):
-        self.local.append((tool, gate, sent, reason))
+    def record_autorun_local(self, tool, gate=None):
+        self.local.append((tool, gate))
 
     # ── 従来（聞く）経路で使うもの。★自走では1回も呼ばれてはいけない ──
     def register(self, cfg, payload):
@@ -119,119 +116,43 @@ for label, cmd in [
     check(f'{label}: 人には聞かない（管制へ質問0回）', fake.asked, 0)
     check(f'{label}: 端末の確認画面に落とさない（Noneでない）', out is not None, True)
 
-print('\n── (b) 管制には「聞かずに通した」が履歴として残る ' + '─' * 22)
+print('\n── (b) ★管制へは何も送らない（えふさん決定 2026-09-07・この記録は不要） ' + '─' * 4)
+# 2026-09-06 の自走化では「聞かずに通した」を管制へ1件ずつ送っていたが、要らないと決まった。
+# ★設定での切り替えにせず【送る仕組みごと消した】ので、ここでは「送る道が無いこと」を固定する。
 fake = FakeDatalink()
 prh.decide(event(), datalink=fake)
-check('履歴を1件だけ送る', len(fake.sent), 1)
-sent = fake.sent[0]
-check_true('現場が入っている', sent.get('site') == 'demo')
-check_true('PCが入っている', bool(sent.get('machine')))
-check_true('ツール名が入っている', sent.get('toolName') == 'Bash')
-check_true('実行される中身が入っている', 'vercel' in str(sent.get('toolInput')))
-check_true('1行説明が「聞く文」ではなく「履歴の文」になっている',
-           sent['summary'].startswith('【自走で通しました'))
-check_true('1行説明に「お客さまの確認」が残っていない（人に判断を求めない）',
-           'お客さまの確認' not in sent['summary'])
-check_true('1行説明が問いかけで終わらない', not sent['summary'].endswith('よいですか'))
-check('見出しの種別は残す（何を通したかが読める）',
-      prh.autorun_summary('【お客さまの確認：本番へ反映】部品を公開（配布）してよいですか'),
-      '【自走で通しました：本番へ反映】部品を公開（配布）しています')
-check('見出しが無い文にも印を付ける',
-      prh.autorun_summary('ファイルを削除してよいですか'), '【自走で通しました】ファイルを削除しています')
-check('想定外の形でも印だけは必ず付く', prh.autorun_summary('謎の文'), '【自走で通しました】謎の文')
-check('★種別は auto（人に聞くカードではない・管制 PR#276）', sent.get('kind'), 'auto')
-check('待つための持ち物は付けない（waitMs）', 'waitMs' in sent, False)
-check('押させる選択肢は付けない（choices）', 'choices' in sent, False)
-
-print('\n── (b2) ★送り方は1手だけ（登録して終わり・即閉じは消した） ' + '─' * 12)
-# 管制が kind='auto' を status='recorded' で受けるので、カードにならない＝閉じる必要が無い。
-# 以前の「登録→すぐ閉じる」は、その一瞬だけカードが出得た逃げ方だった。丸ごと消してある。
-calls = []
-
-
-def fake_request(cfg, method, path, body=None, timeout=None):
-    calls.append((method, path.split('?')[0], timeout, dict(body or {})))
-    return {'id': 'req-9'} if method == 'POST' else {'ok': True}
-
-
-def with_fake_request(handler, run):
-    """_request を差し替えて run() を回す（元へ必ず戻す）。"""
-    orig = dl._request
-    dl._request = handler
-    try:
-        return run()
-    finally:
-        dl._request = orig
-
-
-calls.clear()
-check('送れた', with_fake_request(fake_request,
-      lambda: dl.record_autorun(CFG, {'site': 'demo', 'kind': 'auto', 'summary': 'x'})), True)
-check('★1手だけ（登録して終わり）', len(calls), 1)
-check('その1手は登録', calls[0][0], 'POST')
-check('★閉じる手はもう無い', [c[0] for c in calls].count('DELETE'), 0)
-check_true('職人を待たせない短い上限', all(c[2] == dl.AUTORUN_SEND_TIMEOUT_SEC for c in calls))
-
-print('\n── (b3) ★古い管制（auto を知らない版）には昔の形で送り直す ' + '─' * 10)
-
-
-def refuse_auto(cfg, method, path, body=None, timeout=None):
-    """kind='auto' を 400 で弾く古い管制のふり。message なら受ける。"""
-    calls.append((method, path.split('?')[0], timeout, dict(body or {})))
-    if method == 'POST' and (body or {}).get('kind') == 'auto':
-        dl._last_failure = dl.FAIL_CLIENT      # 400＝こちらの送信内容の不備
-        return None
-    dl._last_failure = None
-    return {'id': 'old-1'} if method == 'POST' else {'ok': True}
-
-
-calls.clear()
-check('古い管制でも記録できた', with_fake_request(refuse_auto,
-      lambda: dl.record_autorun(CFG, {'site': 'demo', 'kind': 'auto', 'summary': '【自走で通しました】x'})),
-      True)
-kinds = [c[3].get('kind') for c in calls if c[0] == 'POST']
-check('1手目は新しい形(auto)', kinds[0], 'auto')
-check('★弾かれたら昔の形(message)で送り直す', kinds[1], 'message')
-check('昔の形には question を入れる（無いと古い管制も400）',
-      calls[1][3].get('question'), '【自走で通しました】x')
-check('★昔の形のときだけ閉じる（カードにしない）', [c[0] for c in calls].count('DELETE'), 1)
-
-
-def unreachable(cfg, method, path, body=None, timeout=None):
-    """管制へ届かない（不通・401・5xx）。★送り直さない＝同じものを2回投げない。"""
-    calls.append((method, path.split('?')[0], timeout, dict(body or {})))
-    dl._last_failure = dl.FAIL_UNREACHABLE
-    return None
-
-
-calls.clear()
-check('届かないときは False', with_fake_request(unreachable,
-      lambda: dl.record_autorun(CFG, {'site': 'demo', 'kind': 'auto', 'summary': 'x'})), False)
-check('★届かないときは送り直さない（1回だけ）', len(calls), 1)
-
-print('\n── (c) 管制へ送れなくても職人は止まらない ' + '─' * 28)
-for label, fake in [
-    ('送信に失敗', FakeDatalink(send_ok=False)),
-    ('管制が未設定', FakeDatalink(cfg=None)),
-]:
-    out = prh.decide(event(), datalink=fake)
-    check(f'{label}: それでも allow', behavior(out), 'allow')
-    check(f'{label}: この機の記録に1行残る', len(fake.local), 1)
-    check(f'{label}: 送れなかったと分かる', fake.local[0][2], False)
+check('★管制へ1件も送らない', len(fake.sent), 0)
+check('★聞きにも行かない（従来の3ボタンの道も通らない）', fake.asked, 0)
+check('この機の記録には1行残る（黙って通したことにならない）', len(fake.local), 1)
+check('記録はツール名と関門種別だけ', fake.local[0][:2], ('Bash', 'release'))
 
-fake = FakeDatalink()
-prh.decide(event(), datalink=fake)
-check('送れたときも記録に1行残る', len(fake.local), 1)
-check('送れたと分かる', fake.local[0][2], True)
+# ★送るための部品が【1つも残っていない】こと。残っていると、いつか誰かが呼び戻せてしまう。
+for gone in ('record_autorun', '_record_autorun_legacy', 'AUTORUN_SEND_TIMEOUT_SEC'):
+    check(f'連絡線に {gone} が残っていない', hasattr(dl, gone), False)
+for gone in ('build_autorun_payload', 'autorun_summary', 'AUTORUN_KIND', 'AUTORUN_SUMMARY_PREFIX'):
+    check(f'番人に {gone} が残っていない', hasattr(prh, gone), False)
+
+# ★自走の判定そのものに、管制へ送る呼び出しが1つも書かれていないこと（実物の行を見る）。
+import inspect  # noqa: E402
+_src = inspect.getsource(prh.decide_autorun)
+for word in ('record_autorun(', 'register(', 'hook-requests'):
+    check(f'自走の判定に {word} が無い', word in _src, False)
+
+print('\n── (c) 記録できなくても職人は止まらない ' + '─' * 28)
 
 
 class BoomDatalink(FakeDatalink):
-    def record_autorun(self, cfg, payload):
-        raise RuntimeError('管制が爆発した')
+    def record_autorun_local(self, tool, gate=None):
+        raise RuntimeError('記録が爆発した')
 
 
 out = prh.decide(event(), datalink=BoomDatalink())
-check('記録が例外でも allow は返る', behavior(out), 'allow')
+check('記録が例外でも allow は返る（職人は止まらない）', behavior(out), 'allow')
+
+fake = FakeDatalink(cfg=None)
+out = prh.decide(event(), datalink=fake)
+check('管制が未設定でも allow', behavior(out), 'allow')
+check('管制が未設定でも管制を叩かない', len(fake.sent) + fake.asked, 0)
 
 print('\n── (d) ★deny は1ミリも変わらない（自走でも通さない） ' + '─' * 18)
 for label, ev in [
@@ -259,9 +180,8 @@ check('Codex: updatedInput を添えない',
       'updatedInput' in out_codex['hookSpecificOutput']['decision'], False)
 fake_cx = FakeDatalink()
 prh.decide(event(), datalink=fake_cx, codex=True)
-check('Codex でも自走の履歴は送る', len(fake_cx.sent), 1)
-check('★Codex でも同じ1か所・同じ形（kind=auto）', fake_cx.sent[0].get('kind'), 'auto')
-check_true('★Codex でも人が読む1行が入る', fake_cx.sent[0]['summary'].startswith('【自走で通しました'))
+check('★Codex でも管制へは何も送らない', len(fake_cx.sent), 0)
+check('★Codex でもこの機の記録には1行残る', len(fake_cx.local), 1)
 check('Codex でも deny は変わらない',
       behavior(prh.decide(event('cat .env'), datalink=FakeDatalink(), codex=True)), 'deny')
 out_claude = prh.decide(event(), datalink=FakeDatalink())