@tyhld/conductor 0.12.5 → 0.12.7

package/package.json

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

package/scripts/hooks/skill_sync_notify.py

@@ -0,0 +1,131 @@
+#!/usr/bin/env python3
+"""skill_sync_notify.py — 職人の鉄則を配り損ねたことを管制へ知らせる（★便 29612ff1）。
+
+【なぜ要るか（えふさん確定 2026-09-16）】
+  「職人ルールを1時間ごとに自動更新 承認。★更新失敗時に古いルールのまま黙って動かないこと。
+    適用version/時刻を確認可能にする」
+
+  1時間ごとの自動同期が鉄則を配れなかったとき、いちばん困るのは
+  【古い鉄則のまま職人が動き続け、誰もそれに気づかない】こと。実際に便 4a5bbb9f で、
+  鉄則7を直したのに現場の職人は古い文のまま動いていた（人が APPLY=1 を打つまで）。
+  だから配り損ねは、機体の中のログだけでなく【管制からも見える】形にする。
+
+【なぜ「確認カード」を選んだか（生存報告・開発履歴と比べて）】
+  ・生存報告（POST /api/conductor/report）… 送っているのは現場ごとの relay で、項目は
+    machine / site / branch / status / lastAction / artisanReady に決まっている。
+    「鉄則が古い」を載せる場所が無く、足すと中央側の変更が要る。★中央は触らない方針。
+  ・開発履歴（devlog）… 書き込みはログインの JWT が要る（MCP 経路）。1時間ごとの
+    無人の同期は JWT を持たない。★持たせると、常駐に新しい秘密を増やすことになる。
+  ・確認カード（POST /api/conductor/hook-requests）… ★これを選んだ。
+      - 「この機体で人が1つ操作しないと直らない」を伝えるために作られた口である（ADR-016）。
+      - 送るのに要るのは、番人がもう持っている接続情報だけ（hook_datalink が読む）。
+        新しい鍵も新しい API も増えない。
+      - 答えの選択肢を付けなければ、管制では【お知らせ】として出て「閉じる」だけになる
+        （便 927da60e で確かめた描き方）。押せないボタンは出ない。
+      - 閉じられるまで出たままなので、★古い鉄則のままであることが外から見え続ける。
+
+【何があっても同期を止めない】
+  接続情報が無い機体・管制へ届かない機体でも、ここは黙って終わる（exit 0）。
+  その場合でも、配り損ねた事実は $HOME/.tyhld/craftsman-skill.last に残っている。
+
+【使い方】
+  python3 skill_sync_notify.py <直近の試みの控えのパス>
+  ★控えの1行目が status=failed のときだけカードを出す。ok のときは何もしない。
+"""
+import os
+import sys
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+import hook_datalink  # noqa: E402
+
+#: カードを出したままにする時間（ミリ秒）＝60分。管制側の上限（HOOK_MAX_WAIT_MS）と同じ。
+#:   ★このスクリプトは待たない。ここで宣言した時間だけ管制にカードが出たままになり、
+#:     配り直しが成功すれば次の回にはもう出さないので、自然に消える。
+#:   ★1時間ごとの同期と同じ幅にしてあるので、直らない限りカードは途切れない。
+NOTIFY_WAIT_MS = 60 * 60 * 1000
+
+#: 同じカードを積み上げないための鍵の頭。★機体ごとに1枚だけにする。
+#:   管制は同じ鍵の再登録を【寿命の延長】として扱う（upsertHookRequest）。だから
+#:   毎時の失敗でカードが増えず、直ればその回で登録が止まって数分後に自然に閉じる。
+REQUEST_KEY_PREFIX = 'craftsman-skill-stale'
+
+
+def parse_last(path):
+    """直近の試みの控えを {鍵: 値} に読む。読めなければ空。
+
+    形: `<ISO時刻> status=... reason=... version=... sha=... src=... dst=...`
+    """
+    try:
+        with open(path, encoding='utf-8') as f:
+            line = f.readline().strip()
+    except Exception:  # noqa: BLE001
+        return {}
+    if not line:
+        return {}
+    parts = line.split(' ')
+    out = {'at': parts[0]}
+    for p in parts[1:]:
+        if '=' in p:
+            k, v = p.split('=', 1)
+            out[k] = v
+    return out
+
+
+def build_payload(rec, env=None):
+    """カードの中身。★何が起きていて、何をすれば直るかを日本語で書く。"""
+    env = os.environ if env is None else env
+    machine = env.get('CONDUCTOR_MACHINE') or os.uname().nodename
+    reason = rec.get('reason', '不明')
+    body = '\n'.join([
+        f'いま止まっているもの: この PC（{machine}）の職人の決まり（鉄則）が、'
+        '新しい版に入れ替わっていません。職人は古い決まりのまま動きます。',
+        '再開のしかた: この PC で次の1行を打ってください。'
+        '　cd ~/projects/conductor　そのあと　APPLY=1 bash scripts/update-guard.sh',
+        '★このカードは管制からは押せません（この PC での操作が要ります）。',
+        f'PC: {machine}／うまくいかなかった理由: {reason}',
+        f'最後に試した時刻: {rec.get("at", "不明")}',
+        f'配ろうとした版: {rec.get("version", "不明")}',
+    ])
+    return {
+        'site': env.get('CONDUCTOR_SITE') or machine,
+        'machine': machine,
+        # ★sessionId は絶対に空で送らない（空だと管制が 400 で弾く）。
+        'sessionId': hook_datalink.stable_session_id('', machine, 'guard-sync'),
+        'kind': 'message',
+        'question': body,
+        'summary': f'職人の決まりが古いままです（{machine}）',
+        'waitMs': NOTIFY_WAIT_MS,
+        # ★機体ごとに固定の鍵。毎時の失敗が同じ1枚に集約される（増殖しない）。
+        'requestKey': f'{REQUEST_KEY_PREFIX}:{machine}',
+    }
+
+
+def run(last_path, cfg=None, datalink=hook_datalink):
+    """返り値: 出したカードのID / None（出さなかった）。"""
+    rec = parse_last(last_path)
+    if rec.get('status') != 'failed':
+        return None
+    cfg = datalink.load_config() if cfg is None else cfg
+    if cfg is None:
+        # 接続情報が無い機体。★控えには残っているので、黙って消えるわけではない。
+        return None
+    reg = datalink.register(cfg, build_payload(rec))
+    if not reg or not reg.get('id'):
+        return None
+    # ★待たない。押されるまで管制に出たままにする＝古い鉄則のままであることが見え続ける。
+    return reg['id']
+
+
+def main() -> int:
+    if len(sys.argv) < 2:
+        return 0
+    try:
+        run(sys.argv[1])
+    except Exception:  # noqa: BLE001  知らせに失敗しても同期は止めない
+        pass
+    return 0
+
+
+if __name__ == '__main__':
+    sys.exit(main())

package/scripts/hooks/test_policy.py

@@ -959,8 +959,15 @@ _conductor = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))  # …/
 _conductor = os.path.dirname(_conductor)                                   # …/conductor
 
 check('同期モードがある', 'SYNC_ONLY' in _ug, True)
-check('同期モードは掃除・鉄則スキル・pnpm 点検を飛ばす',
-      '①-b〜①-d2 は省略（同期モード）' in _ug, True)
+check('同期モードは掃除・タイマー・pnpm 点検を飛ばす',
+      '①-b・①-c・①-d2 は省略（同期モード）' in _ug, True)
+# ★職人の鉄則（①-d）は同期モードでも【配る】（えふさん確定 2026-09-16・便 29612ff1）。
+#   番人のコードは1時間で届くのに鉄則だけ人待ちだと、直した鉄則が現場に届かないまま
+#   毎時再生産される（便 4a5bbb9f で実測）。中身の検査は test/craftsman-skill-sync.test.ts。
+check('職人の鉄則は同期モードでも配る',
+      _ug.index('①-d 職人の鉄則スキル') < _ug.index('①-b・①-c・①-d2 は省略（同期モード）'), True)
+check('配り損ねたら成功と偽らず管制へ知らせる',
+      'craftsman_notify_failure' in _ug and 'skill_sync_notify.py' in _ug, True)
 check('同期モードは権限ルールの整理より前で終わる',
       _ug.index('同期のみ完了しました') < _ug.index('④ 各現場の権限ルール'), True)
 # ★同期モードは「現場の二重登録を外す」（＝番人の配線）までは行う（ADR-013）。

package/scripts/hooks/test_skill_sync_notify.py

@@ -0,0 +1,178 @@
+#!/usr/bin/env python3
+"""配り損ねの知らせ（skill_sync_notify.py）の単体テスト。
+
+  実行: python3 scripts/hooks/test_skill_sync_notify.py
+  1件でも期待と違えば exit 1。
+
+【何を守っているか（便 29612ff1・えふさん確定 2026-09-16）】
+  「★更新失敗時に古いルールのまま黙って動かないこと。適用version/時刻を確認可能にする」
+  (a) 成功の控えでは【1件も知らせない】（毎時のカードで管制を埋めない）
+  (b) 失敗の控えでは【必ず知らせる】＝黙って進まない
+  (c) カードの中身に「いま止まっているもの」「再開のしかた」が入っている
+      ＝管制が知らせとして描ける形（ai-kana の hook-notice-text.ts が探す見出し語）
+  (d) カードにボタンが1つも出ない形で送る（kind=message・choices なし）
+      ＝押せないボタンをお客さまの画面に置かない（ai-kana の hook-answerable.ts）
+  (e) 機体ごとに鍵が固定＝毎時の失敗が同じ1枚に集約される（カードが増殖しない）
+  (f) 知らせに失敗しても例外を外へ出さない（★同期そのものは止めない）
+  (g) 控えが読めない・空・壊れていても落ちない
+"""
+import os
+import sys
+import tempfile
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+import skill_sync_notify as sn  # 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:64} {got}')
+
+
+def check_true(label, got):
+    check(label, bool(got), True)
+
+
+CFG = {'url': 'https://example.invalid', 'token': 'dummy-token-not-real'}
+OK_LINE = ('2026-09-16T10:00:00+0900 status=ok reason=applied version=0.12.7 '
+           'sha=abc123def456 src=/r/skills/x/SKILL.md dst=/h/.claude/skills/x/SKILL.md')
+NG_LINE = ('2026-09-16T11:00:00+0900 status=failed reason=write-failed version=0.12.7 '
+           'sha=abc123def456 src=/r/skills/x/SKILL.md dst=/h/.claude/skills/x/SKILL.md')
+
+
+class FakeLink:
+    """連絡線の代役。★通信は1度も行わない。"""
+
+    #: 「既定のまま」と「わざと None を返させる」を区別するための印。
+    DEFAULT = object()
+
+    def __init__(self, cfg=CFG, reg=DEFAULT):
+        self.cfg = cfg
+        self.reg = {'id': 'req-1'} if reg is FakeLink.DEFAULT else reg
+        self.sent = []
+
+    def load_config(self, env=None):
+        return self.cfg
+
+    def register(self, cfg, payload):
+        self.sent.append(payload)
+        return self.reg
+
+    # ★本物と同じ実装を借りる（sessionId が空にならないことを一緒に確かめるため）。
+    stable_session_id = staticmethod(sn.hook_datalink.stable_session_id)
+
+
+def write(tmp, name, text):
+    p = os.path.join(tmp, name)
+    with open(p, 'w', encoding='utf-8') as f:
+        f.write(text)
+    return p
+
+
+# ── (a)(b) 出す・出さない ────────────────────────────────────────────────────
+print('\n【(a)(b) 成功では出さない・失敗では必ず出す】')
+with tempfile.TemporaryDirectory() as tmp:
+    link = FakeLink()
+    check('成功の控え: 知らせない', sn.run(write(tmp, 'ok.last', OK_LINE + '\n'), datalink=link), None)
+    check('成功の控え: 1件も送っていない', len(link.sent), 0)
+
+    link = FakeLink()
+    got = sn.run(write(tmp, 'ng.last', NG_LINE + '\n'), datalink=link)
+    check('失敗の控え: 知らせた（カードIDが返る）', got, 'req-1')
+    check('失敗の控え: 1件だけ送った', len(link.sent), 1)
+
+    # ★控えが無い・空・壊れている＝出さない（落ちない）＝(g)
+    link = FakeLink()
+    check('控えが無い: 落ちずに何もしない', sn.run(os.path.join(tmp, 'none.last'), datalink=link), None)
+    check('控えが空: 落ちずに何もしない', sn.run(write(tmp, 'e.last', ''), datalink=link), None)
+    check('控えが壊れている: 落ちずに何もしない', sn.run(write(tmp, 'b.last', 'ゴミ\n'), datalink=link), None)
+    check('壊れた控えでは1件も送らない', len(link.sent), 0)
+
+# ── (c)(d)(e) カードの形 ────────────────────────────────────────────────────
+print('\n【(c)(d)(e) カードの形】')
+with tempfile.TemporaryDirectory() as tmp:
+    link = FakeLink()
+    sn.run(write(tmp, 'ng.last', NG_LINE + '\n'), datalink=link)
+    p = link.sent[0]
+    q = p['question']
+    # (c) 管制が本文として取り出す見出し語（ai-kana: NOTICE_BODY_LABELS）。
+    check_true('本文に「いま止まっているもの」がある', 'いま止まっているもの:' in q)
+    check_true('本文に「再開のしかた」がある', '再開のしかた:' in q)
+    check_true('再開の打ち方が書いてある', 'update-guard.sh' in q)
+    # ★「古い決まりのまま動く」ことが本文から分かる＝黙って動いていない。
+    check_true('古いまま動くと書いてある', '古い' in q)
+    # 失敗の理由・時刻・版が読める＝(2) 適用version/時刻を確認可能にする。
+    check_true('うまくいかなかった理由が載る', 'write-failed' in q)
+    check_true('最後に試した時刻が載る', '2026-09-16T11:00:00+0900' in q)
+    check_true('配ろうとした版が載る', '0.12.7' in q)
+    # (d) ボタンを1つも出さない形（kind=message かつ choices なし）。
+    check('種別は message', p['kind'], 'message')
+    check('選択肢を付けない（押せないボタンを置かない）', 'choices' in p, False)
+    check_true('sessionId が空でない（管制が 400 で弾かない）', bool(p['sessionId'].strip()))
+    check_true('見出しが1行ある', bool(p['summary'].strip()))
+    check('待つと申告する時間は管制の上限と同じ60分', p['waitMs'], 3_600_000)
+    # (e) 鍵が機体ごとに固定。
+    check_true('鍵が付いている', p['requestKey'].startswith(sn.REQUEST_KEY_PREFIX + ':'))
+    link2 = FakeLink()
+    sn.run(write(tmp, 'ng2.last', NG_LINE + '\n'), datalink=link2)
+    check('2回目も同じ鍵（カードが増えない）', link2.sent[0]['requestKey'], p['requestKey'])
+    # ★控えの中身（パス）以外の余計な物を載せない＝秘密は1文字も入らない。
+    check('本文にトークンは入らない', CFG['token'] in q, False)
+
+# ── (f) 知らせに失敗しても同期は止めない ────────────────────────────────────
+print('\n【(f) 知らせに失敗しても同期を止めない】')
+with tempfile.TemporaryDirectory() as tmp:
+    last = write(tmp, 'ng.last', NG_LINE + '\n')
+
+    # 接続情報が無い機体。
+    check('接続情報が無い: 何もせず終わる', sn.run(last, datalink=FakeLink(cfg=None)), None)
+    # 管制が受け取らなかった。
+    check('登録が空返事: 何もせず終わる', sn.run(last, datalink=FakeLink(reg=None)), None)
+    check('登録にIDが無い: 何もせず終わる', sn.run(last, datalink=FakeLink(reg={})), None)
+
+    # ★main() は何があっても 0 を返す（update-guard.sh が止まらない）。
+    argv = sys.argv
+    sys.argv = ['skill_sync_notify.py', last]
+    orig_load, orig_reg = sn.hook_datalink.load_config, sn.hook_datalink.register
+    try:
+        sn.hook_datalink.load_config = lambda env=None: CFG
+
+        def boom(cfg, payload):
+            raise RuntimeError('通信断')
+
+        sn.hook_datalink.register = boom
+        check('通信が落ちても exit 0', sn.main(), 0)
+    finally:
+        sn.hook_datalink.load_config, sn.hook_datalink.register = orig_load, orig_reg
+        sys.argv = argv
+    check('引数が無くても exit 0', (lambda: (sys.argv.__setitem__(slice(None), ['x']), sn.main())[1])(), 0)
+    sys.argv = argv
+
+# ── ★空振り確認 ─────────────────────────────────────────────────────────────
+print('\n【★空振り確認】')
+with tempfile.TemporaryDirectory() as tmp:
+    # 「status を見ずに必ず出す」ようにしたら (a) が落ちる＝(a) は本当に効いている。
+    orig_parse = sn.parse_last
+    sn.parse_last = lambda p: dict(orig_parse(p), status='failed')
+    try:
+        link = FakeLink()
+        sn.run(write(tmp, 'ok.last', OK_LINE + '\n'), datalink=link)
+        check('★空振り: 判定を壊すと成功の控えでも出てしまう', len(link.sent), 1)
+    finally:
+        sn.parse_last = orig_parse
+    link = FakeLink()
+    check('★戻したあと: 成功では出ない', sn.run(write(tmp, 'ok2.last', OK_LINE + '\n'), datalink=link), None)
+
+print('\n' + '=' * 72)
+if failures:
+    print(f'NG {len(failures)} 件:')
+    for f in failures:
+        print('  -', f)
+    sys.exit(1)
+print('すべて期待どおり')
+sys.exit(0)

package/skills/conductor-craftsman/SKILL.md

@@ -13,6 +13,13 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
 ## 鉄則13項
 
 1. **着手時の確認と枝分かれ** — `git status && git log --oneline -3 && git stash list` を確認してから main を最新化し、新ブランチを切る。
+   - ★**リポ全体の差分が「読めないファイル」で落ちたとき**（`unsupported file type` / `cannot hash <名前>` が出る／`.npmrc` のような設定が身に覚えなく `M` で出る）。**これは壊れていない。**
+     秘密に分類されたファイルは職人から中身を隠してあるので、git がそのファイルを数えられないだけ。**中身を見に行く方へ回らない。**
+     - ① そのファイルを**触らない・開かない・`git diff` の対象にしない**。「読めるようにする」方向へ回避しない（鉄則4）。
+     - ② 差分は**対象を名指しして**見る。例 `git diff -- src docs`。全体で見たいときは除いて見る＝`git diff -- . ':(exclude).npmrc'`。
+     - ③ `M` と出ていても、**自分が触っていなければ触っていない**。★職人には本当に変わったかを確かめられない。`git status --porcelain -- <そのファイル>` の**表示だけ**を報告に書き、**中身は1文字も書かない**。
+     - ④ **コミットに含めない**（`git add` は対象を名指しする＝`git add -A` を使わない）。
+     - ⑤ それで進めないときだけ、回避せず**人の手番として報告して止まる**。
 2. **cd は1回だけ** — cd は作業開始時に1回だけ実行する。以後 cd を `&&` で他コマンドと連結しない・リダイレクトと混ぜない。
 3. **持ち場の外に出ない** — 指示された作業ディレクトリの外を変更しない。
 4. **壁に当たったら止まる** — サンドボックス制約・保護されたパスに当たったら回避せず中断して報告する（★通信で塞がれたときの読み方は鉄則12）。