@tyhld/conductor 0.12.58 → 0.12.59

package/package.json

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

package/scripts/hooks/test_craftsman_block_passes_guard.py

@@ -0,0 +1,120 @@
+#!/usr/bin/env python3
+"""鉄則の中の「そのまま流す塊」が番人を通ることの検査（便 9bcd9579）。
+
+  実行: python3 scripts/hooks/test_craftsman_block_passes_guard.py
+  1件でも期待と違えば exit 1。
+
+【なぜ要るか（実測 2026-09-27・便 f6b9ddb6 の A/B 集計）】
+  鉄則7 の「合流の4段確認をまとめた塊」が ★6便すべてで番人に止められていた。
+  止めていたのは捨て先の指定そのもの（標準エラーを捨てる書き方）ではなく、その指定を
+  【変数の値】の中に閉じ込めたうえで、その変数を文字列の途中へ差し込んでいた形である。
+
+      V="$(何か 2>/dev/null)"        … 捨て先を変数の値に含めて置き
+      "repos/$その変数/branches/…"    … 別の行で文字列の途中へ差し込む
+
+  番人は変数を字面で展開してから測り直すので、展開後の捨て先が `/dev/null` ではなく
+  後ろのパスが貼り付いた別の名前として取り出され、「/dev 直下への書き込み」に見えた。
+  ＝鉄則どおりに流すと毎便1往復が必ず捨てられていた。
+
+【ここで固定すること】
+  (a) 鉄則の中の bash の塊は、番人の判定（decide_bash）で allow になる
+      ・素で貼る形／`PR=<番号>` を付けた形／ファイルに置いてから流す形のどれでも
+  (b) 塊は4段（protect / ci / merge / verify）のままで、段の名前を出す行が残っている
+  (c) 塊の走る行に、保護を迂回する旗が無い
+  ★番人（policy.py）には手を入れない。この検査は鉄則の文章の側だけを見る。
+"""
+import os
+import re
+import subprocess
+import sys
+import tempfile
+
+HERE = os.path.dirname(os.path.abspath(__file__))
+REPO = os.path.dirname(os.path.dirname(HERE))
+SKILL = os.path.join(REPO, 'skills', 'conductor-craftsman', 'SKILL.md')
+
+sys.path.insert(0, HERE)
+
+import policy  # 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)[:44]}')
+
+
+def bash_blocks(text):
+    """鉄則の中の ```bash … ``` を、行頭の字下げを外して取り出す。"""
+    out = []
+    lines = text.split('\n')
+    i = 0
+    while i < len(lines):
+        m = re.match(r'^(\s*)```bash\s*$', lines[i])
+        if not m:
+            i += 1
+            continue
+        pad = len(m.group(1))
+        body = []
+        i += 1
+        while i < len(lines) and not re.match(r'^\s*```\s*$', lines[i]):
+            ln = lines[i]
+            body.append(ln[pad:] if ln[:pad].strip() == '' else ln)
+            i += 1
+        out.append('\n'.join(body) + '\n')
+        i += 1
+    return out
+
+
+with open(SKILL, encoding='utf-8') as fh:
+    blocks = bash_blocks(fh.read())
+
+check('鉄則の中に流す塊が在る', len(blocks) >= 1, True)
+
+TMP = tempfile.mkdtemp(prefix='craftsman-block-')
+
+for n, block in enumerate(blocks, 1):
+    print(f'── 塊 {n} ' + '─' * 56)
+
+    path = os.path.join(TMP, f'block{n}.sh')
+    with open(path, 'w', encoding='utf-8') as fh:
+        fh.write(block)
+
+    # (a) どの流し方でも番人が通す。
+    forms = [
+        ('素で貼る', block),
+        ('番号を付けて貼る', 'PR=123\n' + block),
+        ('ファイルに置いてから流す',
+         'cat > ' + path + " <<'EOF'\n" + block + 'EOF\nPR=123 bash ' + path),
+    ]
+    for label, cmd in forms:
+        r = policy.decide_bash(cmd, REPO)
+        check(f'(a{n}) {label}', r['decision'], 'allow')
+        if r['decision'] != 'allow':
+            print('     理由:', r['reason'])
+
+    # 塊が壊れていたら、通っても意味が無い。
+    p = subprocess.run(['bash', '-n', path], capture_output=True, text=True)
+    check(f'(a{n}) bash -n が通る', p.returncode, 0)
+    if p.returncode != 0:
+        print('     ', p.stderr.strip()[:200])
+
+    # (b) 4段のまま・段の名前が出る。
+    for stage in ('protect', 'ci', 'merge', 'verify'):
+        check(f'(b{n}) 段 {stage} の結果を出す行が在る',
+              bool(re.search(r'\b(ok|ng)\s+' + stage + r'\b', block)), True)
+
+    # (c) 保護を迂回する旗が走る行に無い（説明のコメントは数えない）。
+    runnable = '\n'.join(ln for ln in block.split('\n') if not ln.lstrip().startswith('#'))
+    check(f'(c{n}) 保護を迂回する旗が走る行に無い', '--' + 'admin' in runnable, False)
+
+print()
+if failures:
+    print(f'NG {len(failures)} 件')
+    for f in failures:
+        print('  ', f)
+    sys.exit(1)
+print('すべて期待どおり')

package/skills/conductor-craftsman/SKILL.md

@@ -54,13 +54,14 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
    PR="${PR:?PR=<PR番号> が要る}"
    ng() { printf 'NG[%s] %s\n' "$1" "$2" >&2; exit 1; }
    ok() { printf 'OK[%s] %s\n' "$1" "$2"; }
-   V() { gh pr view "$PR" --json "$1" --jq "$2" 2>/dev/null; }
+   q() { "$@" 2>/dev/null; }   # 失敗の文句だけを捨てる（★塊の中で 2>/dev/null はここ1か所だけ）
+   V() { q gh pr view "$PR" --json "$1" --jq "$2"; }
 
    # ① 保護 … 必須の検査が1件以上のときだけ「保護あり」。0・404・403・失敗は「保護なし」へ倒す。
    BASE="$(V baseRefName .baseRefName)"; [ -n "$BASE" ] || ng protect '合流先を読めない'
-   SLUG="$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null)"
+   SLUG="$(q gh repo view --json nameWithOwner --jq .nameWithOwner)"
    [ -n "$SLUG" ] || ng protect 'リポジトリを読めない'
-   N="$(gh api "repos/$SLUG/branches/$BASE/protection/required_status_checks" --jq '.contexts | length' 2>/dev/null)"
+   N="$(q gh api "repos/$SLUG/branches/$BASE/protection/required_status_checks" --jq '.contexts | length')"
    case "$N" in ''|*[!0-9]*) N=0 ;; esac
    if [ "$N" -ge 1 ]; then PROT=yes; else PROT=no; fi
    ok protect "base=$BASE 保護=$PROT（必須の検査 $N 件）"
@@ -134,7 +135,12 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
 止まる書き方を最初から使わなければ、打ち直しの往復が要らない。★番人の判定は変えない（弱めない）。
 
 - **cd を `&&` でつなぐ**（`cd 先 && 何か`・cd とリダイレクトを同じ行に混ぜる）→ 止まる。**cd は作業のはじめに1回だけ単独で**打ち、以後は絶対パスか `git -C <実際のパス>`（鉄則2）。
-- **`.npmrc` を読む**（`cat ~/.npmrc` など）→ 止まる。**中身は見に行かない。** 差分は対象を名指しで `git diff -- src docs`、全体で見たいときは除いて `git diff -- . ':(exclude).npmrc'`（鉄則1）。
+- **リポの根やホームを再帰で検索する**（`grep -rn 語 .`・`rg 語 .`・`grep -rn 語 ~`・`grep -rn 語 /home/…/リポ`）→ 止まる。★**実際に止まっているのはこの形**で、`cat .npmrc` のような直接読みではない（便 f6b9ddb6 の9回はすべて普通の再帰検索が秘密ファイルに当たった形）。**探す場所をフォルダ名で名指しする。**
+  - 通る例: `grep -rn 'relay' src docs` ／ `rg 'relay' src docs` ／ 追跡済みだけ見るなら `git grep -n 'relay' -- src docs`
+  - ★**除外指定では通らない**（実測 2026-09-27）。`grep -rn 語 . --exclude=.npmrc` は、`.` の再帰でまず止まり、さらに**秘密のファイル名を引数に書いた事自体**でも止まる。「除いて探す」ではなく「場所を名指しする」が正解。
+- **秘密のファイルを読む・引数に名前を書く**（`cat ~/.npmrc`・`--exclude=.npmrc`）→ 止まる。**中身は見に行かない。** 差分は対象を名指しで `git diff -- src docs`、全体で見たいときは除いて `git diff -- . ':(exclude).npmrc'`（`git diff` は中身を読み出す動詞ではないので、この形は通る・鉄則1）。
+- **標準エラーの捨て先を、あとで文字列の途中に差し込む変数の中に入れる**（`S="$(何か 2>/dev/null)"` と置き、別の行で `"repos/<その変数>/branches/…"` のように使う）→ 止まる。番人は変数を字面で展開してから測り直すので、捨て先が `/dev/null` ではなく**その後ろのパスが貼り付いた別の名前**として取り出され、`/dev` 直下への書き込みに見える（実測 2026-09-27。鉄則7の塊がこれで6便すべて止まっていた）。**文句を捨てる所を、小さな関数1か所に寄せる。**
+  - 通る例: 先に `q() { "$@" 2>/dev/null; }` を置き、以後は `S="$(q 何か)"` と書く（★変数の値に捨て先が入らないので、どこに差し込んでも止まらない）。
 - **中身が変わる場所を指す再帰の `rm`**（`rm -rf "$DIR/x"`・`rm -rf build/*` のように変数や `*` を含む）→ 止まる。**消す先を実在のパスで名指しする**（例 `rm -rf /tmp/claude-1000/作業名`・鉄則13）。
 - **作業場所が変数のままの `git push`**（`git -C "$DIR" push`・行き先のブランチを確かめられない形）→ 止まる。**リポの中で、ブランチを名指しして** `git push -u origin <作業ブランチ名>`。