@tyhld/conductor 0.11.7 → 0.11.8

package/package.json

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

package/scripts/hooks/hook_datalink.py

@@ -507,30 +507,49 @@ def record_autorun_local(tool_name, gate=None, sent=True, reason='', env=None):
 def record_autorun(cfg, payload):
     """「聞かずに通した（自走）」1件を管制へ【履歴として】送る。返り: True/False。
 
-    【なぜ2手（登録→即クローズ）なのか＝いまの管制で“カードを出さずに”残せる唯一の形】
-      管制の受け口は今のところ「人に聞くカード」しか受け取らない:
-        ・kind は 'tool' か 'message' の2つだけ（未知の種別は 400 で弾かれる）
-        ・登録した行は必ず status='waiting' から始まり、寿命は必ず +2分の余白が付く
-          （nextLeaseExpiry。waitMs=0 でも 2 分は「人が押せるカード」として画面に出る）
-      そこで、登録したその場で DELETE を送り status='abandoned' にする。管制の一覧は
-      status='waiting' の行だけを出すので【カードとしては出ない】が、行は消えないので
-      「いつ・どの現場で・何を聞かずに通したか」は後から必ず読める＝要求どおりの履歴になる。
-
-    【★次の便への申し送り（管制側・DDL不要）】
-      HookRequest.kind は enum ではなく text 列（「種別が増えても DDL 不要」と明記されている）。
-      管制側で kind='auto' を1つ受け付け、その行は一覧に出さない扱いにすれば、この DELETE は
-      要らなくなり、一覧にも「自走の履歴」として素直に出せる。★DBの作り替えは要らない。
+    【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 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
+        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

@@ -49,6 +49,10 @@ import summary_ja  # noqa: E402
 #   localSettings に恒久 allow が書かれ、関門が静かに消える経路になっていた。塞ぐため固定する。
 ALLOW_ALWAYS_DESTINATION = 'session'
 
+# 自走の記録として管制へ送るときの種別（管制 PR#276 が 'recorded' で受ける＝カードにしない）。
+#   ★古い管制はこれを知らず 400 で弾く。そのときは hook_datalink が昔の形で送り直す（互換）。
+AUTORUN_KIND = 'auto'
+
 # 履歴の1行説明の頭に付ける印（人が読んで「判断を求められていない」と分かるように）。
 AUTORUN_SUMMARY_PREFIX = '【自走で通しました】'
 # 聞くときの見出し「【お客さまの確認：本番へ反映】」を、履歴の見出しへ置き換えるための形。
@@ -186,18 +190,18 @@ def autorun_enabled(env=None, datalink=hook_datalink):
 
 
 def build_autorun_payload(data, gate=None):
-    """管制へ【履歴として】送る値。人に聞くカードではないことが一目で分かる形にする。
+    """管制へ【履歴として】送る値。★種別は 'auto'（人に聞くカードではない・管制 PR#276）。
 
-    ★中身は聞くときと同じ（現場・PC・ツール名・実行される内容・1行説明）＝あとで
-      「何が通ったか」を人が読める。違うのは次の3つだけ:
-        ・1行説明の頭に【自走で通しました】と入れる（読んだ人が判断を求められない）
-        ・待つ気が無いので waitMs=0
-        ・選択肢は「押す必要はありません」の1つだけ（万一カードが残っても人を迷わせない）
+    ★主役は summary（人がそのまま読む1行）。管制はこれを「自走で通した」の一覧に出す。
+      例) 【自走で通しました：本番へ反映】部品を公開（配布）しています
+    ★toolName / toolInput は「実行された中身」として今までどおり添える（あとで追える）。
+    ★聞くための持ち物（待ち時間・選択肢）は付けない。誰も待っていないし、押させないため。
     """
     payload = build_payload(data, gate)
+    payload['kind'] = AUTORUN_KIND
     payload['summary'] = autorun_summary(payload.get('summary'))
-    payload['waitMs'] = 0
-    payload['choices'] = [{'value': 'allow', 'label': '（自走で通しました・押す必要はありません）'}]
+    payload.pop('waitMs', None)
+    payload.pop('choices', None)
     return payload
 
 

package/scripts/hooks/test_autorun.py

@@ -6,7 +6,8 @@
 
 【何を守っているか】
   (a) 許可の質問が【1つも人に届かない】＝その場で allow が返る（端末の確認画面にも落ちない）
-  (b) 管制には「聞かずに通した」が【履歴として】残る＝登録したその場で閉じる（カードは出さない）
+  (b) 管制には「聞かずに通した」が【履歴として】残る＝kind='auto' で1手だけ送る（カードは出さない）
+      ★古い管制（auto を知らない版）には昔の形で送り直す（互換）
   (c) 管制へ送れなくても職人は止まらない（allow は変わらない）＋この機の記録に1行残る
   (d) ★deny は1ミリも変わらない＝自走でも通さない
   (e) 各現場の .claude/settings.json を汚さない（updatedPermissions を付けない）
@@ -138,31 +139,76 @@ check('見出しの種別は残す（何を通したかが読める）',
 check('見出しが無い文にも印を付ける',
       prh.autorun_summary('ファイルを削除してよいですか'), '【自走で通しました】ファイルを削除しています')
 check('想定外の形でも印だけは必ず付く', prh.autorun_summary('謎の文'), '【自走で通しました】謎の文')
-check('待つ気は無い（waitMs=0）', sent.get('waitMs'), 0)
-check('押させる選択肢は出さない（1つだけ）', len(sent.get('choices') or []), 1)
-check_true('その選択肢は「押す必要はありません」', '押す必要はありません' in sent['choices'][0]['label'])
+check('★種別は auto（人に聞くカードではない・管制 PR#276）', sent.get('kind'), 'auto')
+check('待つための持ち物は付けない（waitMs）', 'waitMs' in sent, False)
+check('押させる選択肢は付けない（choices）', 'choices' in sent, False)
 
-print('\n── (b2) 送り方は「登録したその場で閉じる」＝カードにしない ' + '─' * 14)
+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))
+    calls.append((method, path.split('?')[0], timeout, dict(body or {})))
     return {'id': 'req-9'} if method == 'POST' else {'ok': True}
 
 
-orig_request = dl._request
-try:
-    dl._request = fake_request
-    check('送れた', dl.record_autorun(CFG, {'site': 'demo'}), True)
-finally:
-    dl._request = orig_request
-check('2手だけ（登録→閉じる）', len(calls), 2)
-check('1手目は登録', calls[0][0], 'POST')
-check('2手目は閉じる（カードを出したままにしない）', calls[1][0], 'DELETE')
-check('閉じる先は登録した行', calls[1][1], '/api/conductor/hook-requests/req-9')
+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)),
@@ -214,6 +260,8 @@ check('Codex: updatedInput を添えない',
 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 でも deny は変わらない',
       behavior(prh.decide(event('cat .env'), datalink=FakeDatalink(), codex=True)), 'deny')
 out_claude = prh.decide(event(), datalink=FakeDatalink())