@tyhld/conductor 0.12.50 → 0.12.51

package/package.json

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

package/skills/conductor-craftsman/SKILL.md

@@ -44,6 +44,66 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
    - ★**合流したかの確かめ方（題名では判定しない・えふさん確定 2026-09-18）。** 合流完了の判定はコミット題名では行わない。GitHub上で対象PRが MERGED であること、merge先が意図したbase branchであること、merge後のbase branchに対象PRの変更内容が含まれることを確認する。（squash merge / merge commit / rebase merge の方式に依存しない確認にする）
      - ★これは共通正本 `claude-code-operations.md §11.3` と、この鉄則7 の2か所を正本にする（各リポの CLAUDE.md へは複製しない）。
      - 確かめ方の例（値は実際の番号・ブランチ名に置き換える）：`gh pr view 番号 --json state,baseRefName,mergeCommit` で、`state` が `MERGED`、`baseRefName` が意図した base branch（ふつうは `main`）であることを見る。`git fetch origin` のあと `git merge-base --is-ancestor mergeCommitのoid origin/base` が成功する（＝合流でできたコミットが base に含まれる）ことを見る。`gh pr diff 番号` の変更が、`git diff` や `git show origin/base:ファイル` で base branch に入っていることを、変更したファイルごとに見比べる。
+   - ★**上の4つ（保護の確認・CI・合流・合流後の確かめ）を1回で流す形**（便 03bef270）。★**確かめは1つも省いていない**：段の名前（`protect` / `ci` / `merge` / `verify`）付きで結果を出し、**落ちた段で止まる**。`--admin` は使わない。確かめを減らしたいときにこの塊を削らない。
+     そのままコピーして、1回の実行として流す（`PR=<PR番号>`）。手で1つずつ打つ従来の手順も、そのまま有効。
+   ```bash
+   #!/usr/bin/env bash
+   # 鉄則7 の4つの確かめ（保護 → CI → 合流 → 合流後）を1回で流す。確かめは1つも省かない。
+   # 使い方: PR=<PR番号> bash <この塊>   落ちた段は NG[段名] で分かる。★--admin は使わない。
+   set -uo pipefail
+   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; }
+
+   # ① 保護 … 必須の検査が1件以上のときだけ「保護あり」。0・404・403・失敗は「保護なし」へ倒す。
+   BASE="$(V baseRefName .baseRefName)"; [ -n "$BASE" ] || ng protect '合流先を読めない'
+   SLUG="$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null)"
+   [ -n "$SLUG" ] || ng protect 'リポジトリを読めない'
+   N="$(gh api "repos/$SLUG/branches/$BASE/protection/required_status_checks" --jq '.contexts | length' 2>/dev/null)"
+   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 件）"
+
+   # ② CI … リンクが /actions/runs/ の行だけを CI と数える（Vercel は数えない）。
+   ROLL="$(V statusCheckRollup '[.statusCheckRollup[]? | {u:(.detailsUrl//.targetUrl//""), s:(.status//""), c:((.conclusion//.state//"")|ascii_upcase)} | select(.u|test("/actions/runs/"))]')"
+   [ -n "$ROLL" ] || ng ci 'CI の一覧を読めない'
+   read -r TOTAL RUNNING RED NOTPASS <<<"$(printf '%s' "$ROLL" | jq -r '[length,
+     ([.[]|select(.s!="" and .s!="COMPLETED")]|length),
+     ([.[]|select(.s=="COMPLETED" or .s=="")|select(.c|test("^(SUCCESS|NEUTRAL|SKIPPED)$")|not)]|length),
+     ([.[]|select(.c|test("^(SUCCESS|NEUTRAL|SKIPPED)$")|not)]|length)] | @tsv')"
+   [ "${TOTAL:-0}" -ge 1 ] || ng ci 'このリポに CI が無い（鉄則7-④: 手元で検査を流し、人の判断で合流する）'
+   [ "$RED" -eq 0 ] || ng ci "CI が赤（$RED 件）"
+   if [ "$PROT" = no ]; then
+     [ "$RUNNING" -eq 0 ] || ng ci "CI が実行中（$RUNNING 件・終わってから確かめ直す）"
+     [ "$NOTPASS" -eq 0 ] || ng ci "pass でない CI がある（$NOTPASS 件）"
+   fi
+   ok ci "CI $TOTAL 件・赤 0 件（保護=$PROT）"
+
+   # ③ 合流 … 保護あり＝予約（緑になり次第 GitHub が合流）／保護なし＝緑を確かめたうえで合流。
+   if [ "$PROT" = yes ]; then
+     gh pr merge "$PR" --auto --merge --delete-branch || ng merge '合流の予約に失敗（鉄則7-② で確かめ直す。旗を足さない）'
+     ok merge '保護あり＝合流を予約した'
+   else
+     gh pr merge "$PR" --merge --delete-branch || ng merge '合流に失敗'
+     ok merge '保護なし＝CI の緑を確かめてから合流した'
+   fi
+
+   # ④ 合流後 … MERGED・base が意図どおり・変更が base に入っている（題名では判定しない）。
+   END=$(( $(date +%s) + ${WAIT:-600} ))
+   while :; do
+     ST="$(V state .state)"
+     [ "$ST" = MERGED ] && break
+     [ "$(date +%s)" -lt "$END" ] || ng verify "まだ MERGED でない（state=$ST）"
+     sleep 15
+   done
+   GOT="$(V baseRefName .baseRefName)"
+   [ "$GOT" = "$BASE" ] || ng verify "合流先が違う（$GOT ≠ $BASE）"
+   OID="$(V mergeCommit .mergeCommit.oid)"; [ -n "$OID" ] || ng verify '合流コミットを読めない'
+   git fetch origin --quiet || ng verify 'fetch に失敗'
+   git merge-base --is-ancestor "$OID" "origin/$BASE" || ng verify "変更が origin/$BASE に入っていない"
+   ok verify "MERGED / base=$BASE / $OID が origin/$BASE に入っている"
+   ```
 8. **ブラウザ検品（Playwright）の作法** — ★取りに行かない。PCに1回だけ入れた共有の道具を指すだけ（ADR-010）。
    - ① **起動は必ず `chromium_sandbox=False`**（`--no-sandbox`）。付けないと起動できない。
    - ② **道具は共有の場所を使う。自分でダウンロードしない。** python は `$HOME/.tyhld/pw-venv/bin/python`、`PLAYWRIGHT_BROWSERS_PATH` は `$HOME/.tyhld/pw-browsers`。**無ければそこで止めて「人の手番＝`APPLY=1 bash scripts/pw-browser-install.sh`」と報告する。** `playwright install` も `pip install playwright` も職人は実行しない（毎便まるごとダウンロードが走り、便が遅くなる。通信自体は鉄則12のとおり許可済みだが、取りに行かないのが本筋＝ADR-010）。
@@ -69,6 +129,15 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
     - **雛形（`.env.example` など）を直すときの正しい形＝「案を作る→人が1回で反映」。** 職人は変更後の中身を**リポジトリの外（`/tmp/claude-1000/` の下）**に作り、報告に次の2つを載せる。① 変更の**差分**（何を足す／消すか）。② 人が**素のターミナルで1回貼るだけで反映できる1つのコマンド**（例：`cp /tmp/claude-1000/<下書き> <リポ>/.env.example`）。★このコマンドは**人が自分のターミナルで**打つ（職人の番人は人の端末には掛からない）。職人が `.env.example` を直接書き換えようとしても番人が止めるので、**回避せず、この形で案を出す**。
     - **本物の `.env`・`.env.local` は、これまでどおり中身も書き込みも扱わない**（秘密の値は1文字も読まない・書かない＝鉄則12の金庫）。
 
+## 番人に止められる常連（こう書くと止まる → 代わりにこう書く）
+
+止まる書き方を最初から使わなければ、打ち直しの往復が要らない。★番人の判定は変えない（弱めない）。
+
+- **cd を `&&` でつなぐ**（`cd 先 && 何か`・cd とリダイレクトを同じ行に混ぜる）→ 止まる。**cd は作業のはじめに1回だけ単独で**打ち、以後は絶対パスか `git -C <実際のパス>`（鉄則2）。
+- **`.npmrc` を読む**（`cat ~/.npmrc` など）→ 止まる。**中身は見に行かない。** 差分は対象を名指しで `git diff -- src docs`、全体で見たいときは除いて `git diff -- . ':(exclude).npmrc'`（鉄則1）。
+- **中身が変わる場所を指す再帰の `rm`**（`rm -rf "$DIR/x"`・`rm -rf build/*` のように変数や `*` を含む）→ 止まる。**消す先を実在のパスで名指しする**（例 `rm -rf /tmp/claude-1000/作業名`・鉄則13）。
+- **作業場所が変数のままの `git push`**（`git -C "$DIR" push`・行き先のブランチを確かめられない形）→ 止まる。**リポの中で、ブランチを名指しして** `git push -u origin <作業ブランチ名>`。
+
 ## 便の終え方
 
 便の最後には、配達役が付けた1行「【便の終え方】…報告の置き場: ～／最終行: DONE:<現場> とだけ出力／完了の知らせ: conductor_complete を commandId="<指示ID>" で1回だけ」と、管理用ジョブ印 `CONDUCTOR_JOB:<指示ID>` がある。作業がすべて終わったら、この節のとおりに次の順で終える（便 a5306df2：以前は便ごとに付いていた全文を、ここへ寄せた）。