@tyhld/conductor 0.12.78 → 0.12.80

package/README.md

@@ -358,6 +358,12 @@ shellcheck -x --severity=warning scripts/*.sh sales-template/*.sh sales-template
 | `version` が公開 npm の `latest` より**新しい** | build → テスト → `npm publish`（公開 npm へ） |
 | 同じ・古い | **何もしない**（版を上げ忘れても事故りません） |
 
+**版を上げる必要があるのは、配布物を変えたときだけです**（便 fb248f6e）。
+配布物＝`package.json` の `files` に載る物（`README.md` も含む）と、`dist` の元（`src`・`tsconfig.json`）、`package.json`・`package-lock.json`。
+`docs/`・`test/`・`.github/` だけの変更は版を上げずに緑で合流でき、合流しても公開は「出しません」で何も起きません。
+配布物を変えたのに版がそのままだと、試験（`test/release-public-npm.test.ts` の「★空振り確認」）が赤で止めます。
+くらべる相手は公開 npm の `latest` を出したコミット（`gitHead`）で、それが確かめられないときは安全側（版上げを求める）に倒れます。
+
 ★**人が `npm publish` を打ってはいけません。**
 
 <details><summary>なぜ手で打たないのか（実測 2026-09-04）</summary>

package/dist/headless.js

@@ -273,6 +273,53 @@ export function validateActiveJobOwnership(rec, cfg) {
 // PATH に居ないため spawn('claude') が ENOENT で失敗する（実測 2026-10-01）。
 // conductor-run が node を探すのと同じ考え方で、既知の場所を順に探す。
 // ─────────────────────────────────────────────────────────────────────────────
+// ── Claude Code の版管理 ─────────────────────────────────────────────────
+//
+// 1台に複数の Claude Code が入っていると、古い方を使って通信の許可 "*" が
+// 効かない等の不具合が起きる（実測 2026-10-02 メインPC）。
+// 候補のうち実在する物の版を確かめ、いちばん新しい版を使う。
+// ─────────────────────────────────────────────────────────────────────────
+/**
+ * 必要な最低の版。これより古い Claude Code では職人を起動しない。
+ *
+ * 根拠: Claude Code 2.1.x 系で画面なし方式（-p, --output-format stream-json）と
+ * 通信の許可 "*"（allowedTools: "*"）が動くことが確かめられたのは太田さんの PC の
+ * 2.1.286（便 bcce7be9）。2.1.86 では "*" が効かない疑い（便 a9754912、メインPC）。
+ * 安全側に倒して 2.1.200 を最低とする（200 未満は "*" 対応が不明確な版）。
+ * ★確かめられた版は 2.1.286 だけなので、200 は推定の境界。
+ */
+export const CLAUDE_MIN_VERSION = '2.1.200';
+/** claude --version の出力から版番号を取り出す。取れなければ null。 */
+export function parseClaudeVersion(output) {
+    // "2.1.286 (Claude Code)" → "2.1.286"
+    const m = output.match(/(\d+\.\d+\.\d+)/);
+    return m ? m[1] : null;
+}
+/** 版番号を比べる。a > b なら正、a < b なら負、同じなら 0。 */
+export function compareVersions(a, b) {
+    const pa = a.split('.').map(Number);
+    const pb = b.split('.').map(Number);
+    for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+        const diff = (pa[i] ?? 0) - (pb[i] ?? 0);
+        if (diff !== 0)
+            return diff;
+    }
+    return 0;
+}
+/** claude の実行ファイルから版番号を取得する。取れなければ null。 */
+export function getClaudeVersion(binPath) {
+    try {
+        const out = execFileSync(binPath, ['--version'], {
+            timeout: 5_000,
+            encoding: 'utf8',
+            stdio: ['ignore', 'pipe', 'ignore'],
+        });
+        return parseClaudeVersion(out);
+    }
+    catch {
+        return null;
+    }
+}
 /** claude の実行ファイルの既知の候補を返す（新しい node の版から順）。 */
 function claudeBinCandidates() {
     const h = os.homedir();
@@ -307,27 +354,42 @@ function claudeBinCandidates() {
     return candidates;
 }
 /**
- * claude の実行ファイルのフルパスを返す。見つからなければ null。
+ * claude の実行ファイルのフルパスと版を返す。見つからなければ null。
  *
  * 探す順:
- *   1. 環境変数 CONDUCTOR_CLAUDE_BIN（テスト用の差し替え口）
- *   2. nvm の各 node 版の bin/claude（新しい版から）
- *   3. ~/.local/bin/claude
- *   4. ~/.claude/local/claude
- *   5. /usr/local/bin/claude, /usr/bin/claude
+ *   1. 環境変数 CONDUCTOR_CLAUDE_BIN（テスト用の差し替え口）→ 指定があればそれだけ
+ *   2. 候補のうち実在する物の版を確かめ、いちばん新しい版を使う
+ *   3. 版を確かめられない物は後回し（版が分かる物が1つもないときだけ使う）
  */
 export function resolveClaudeBin() {
     const override = process.env.CONDUCTOR_CLAUDE_BIN;
     if (override) {
-        if (existsSync(override))
-            return override;
+        if (!existsSync(override))
+            return null;
+        return { bin: override, version: getClaudeVersion(override) };
+    }
+    const existing = claudeBinCandidates().filter(c => existsSync(c));
+    if (existing.length === 0)
         return null;
+    // 版が分かる候補と分からない候補を分ける
+    const withVersion = [];
+    const withoutVersion = [];
+    for (const bin of existing) {
+        const ver = getClaudeVersion(bin);
+        if (ver) {
+            withVersion.push({ bin, version: ver });
+        }
+        else {
+            withoutVersion.push(bin);
+        }
     }
-    for (const candidate of claudeBinCandidates()) {
-        if (existsSync(candidate))
-            return candidate;
+    // 版が分かる候補の中でいちばん新しい物を選ぶ
+    if (withVersion.length > 0) {
+        withVersion.sort((a, b) => compareVersions(b.version, a.version));
+        return withVersion[0];
     }
-    return null;
+    // 版を確かめられない物だけ → 最初の候補を返す（後方互換）
+    return { bin: withoutVersion[0], version: null };
 }
 // ─────────────────────────────────────────────────────────────────────────────
 // 秘密のマスク
@@ -724,15 +786,26 @@ function closeLog(ws) {
  * 【stdout】stream-json 形式。行ごとに JSON を出す。readline で1行ずつ読む。
  * 【stderr】蓄積して認証エラーの検出に使う。
  *
- * 【起動失敗】claude が見つからない（ENOENT）・実行権限がない（EACCES）などで
- *   spawn が失敗した場合、null を返す。呼び出し側はエラーイベントを管制へ送る。
+ * 【起動失敗】claude が見つからない（ENOENT）・実行権限がない（EACCES）・
+ *   版が古すぎるなどで起動しない場合、null を返す。呼び出し側はエラーイベントを管制へ送る。
+ *   失敗の理由は lastSpawnError に入る。
  */
+export let lastSpawnError = '';
 export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
-    const claudeBin = resolveClaudeBin();
-    if (claudeBin === null) {
-        log(`ERROR claude が見つかりません。探した場所: nvm の各版の bin, ~/.local/bin, ~/.claude/local, /usr/local/bin, /usr/bin`);
+    const resolved = resolveClaudeBin();
+    if (resolved === null) {
+        lastSpawnError = 'claude が見つかりません。探した場所: nvm の各版の bin, ~/.local/bin, ~/.claude/local, /usr/local/bin, /usr/bin';
+        log(`ERROR ${lastSpawnError}`);
+        return null;
+    }
+    const { bin: claudeBin, version: claudeVersion } = resolved;
+    // ── 最低版チェック ──
+    if (claudeVersion && compareVersions(claudeVersion, CLAUDE_MIN_VERSION) < 0) {
+        lastSpawnError = `Claude Code が古すぎます（${claudeBin} = ${claudeVersion}、最低 ${CLAUDE_MIN_VERSION}）。npm install -g @anthropic-ai/claude-code で更新してください`;
+        log(`ERROR ${lastSpawnError}`);
         return null;
     }
+    log(`claude bin=${claudeBin} version=${claudeVersion ?? 'unknown'}`);
     const env = { ...process.env };
     delete env.CLAUDECODE;
     delete env.ANTHROPIC_API_KEY;
@@ -794,6 +867,8 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         sessionId,
         startedAt: Date.now(),
         attempt,
+        claudeBin,
+        claudeVersion,
         completed: false,
         exitCode: null,
         exitSignal: null,
@@ -1142,6 +1217,8 @@ async function checkRunningJob(cfg, state) {
             startedAt: new Date(job.startedAt).toISOString(),
             lastActivityAt: new Date(job.lastActivityAt).toISOString(),
             ...(job.process.pid !== undefined ? { pid: job.process.pid } : {}),
+            claudeBin: job.claudeBin,
+            ...(job.claudeVersion ? { claudeVersion: job.claudeVersion } : {}),
         });
     }
     // イベントバッファのフラッシュ
@@ -1238,7 +1315,7 @@ export async function restoreHeadlessOrphans(cfg, state) {
         const cmd = { id: pick.id, body: pick.body };
         const job = spawnClaudeProcess(cfg, cmd, sessionId, attempt, true);
         if (job === null) {
-            await patchStatus(cfg, pick.id, 'failed', 'claude が見つかりません（手元の孤児復元を中止）');
+            await patchStatus(cfg, pick.id, 'failed', `${lastSpawnError || 'claude が見つかりません'}（手元の孤児復元を中止）`);
             state.relay.abandoned.add(pick.id);
             log(`ERROR site=${cfg.site} id=${pick.id} 手元の孤児復元: claude が見つからない → 放棄`);
             return;
@@ -1296,10 +1373,10 @@ export async function restoreHeadlessOrphans(cfg, state) {
     const cmd = { id: pick.id, body: pick.body };
     const job = spawnClaudeProcess(cfg, cmd, sessionId, attempt, true);
     if (job === null) {
-        await patchStatus(cfg, pick.id, 'failed', 'claude が見つかりません（孤児復元を中止）');
+        await patchStatus(cfg, pick.id, 'failed', `${lastSpawnError || 'claude が見つかりません'}（孤児復元を中止）`);
         await sendCommandEvents(cfg, pick.id, [{
                 type: 'error',
-                text: 'claude が見つかりません（孤児復元を中止）',
+                text: `${lastSpawnError || 'claude が見つかりません'}（孤児復元を中止）`,
                 at: new Date().toISOString(),
                 attempt,
             }]);
@@ -1426,10 +1503,10 @@ export async function headlessTick(cfg, state, hooks) {
                 : sessionIdForCommand(retry.cmd.id, retry.attempt);
             const job = spawnClaudeProcess(cfg, retry.cmd, sessionId, retry.attempt, isResume);
             if (job === null) {
-                await patchStatus(cfg, retry.cmd.id, 'failed', 'claude が見つかりません（自動再開を中止）');
+                await patchStatus(cfg, retry.cmd.id, 'failed', `${lastSpawnError || 'claude が見つかりません'}（自動再開を中止）`);
                 await sendCommandEvents(cfg, retry.cmd.id, [{
                         type: 'error',
-                        text: 'claude が見つかりません（自動再開を中止）',
+                        text: `${lastSpawnError || 'claude が見つかりません'}（自動再開を中止）`,
                         at: new Date().toISOString(),
                         attempt: retry.attempt,
                     }]);
@@ -1483,10 +1560,10 @@ export async function headlessTick(cfg, state, hooks) {
         const job = spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume);
         if (job === null) {
             // ★起動失敗を管制へ知らせる（「起動しました」とは言わない）
-            await patchStatus(cfg, cmd.id, 'failed', 'claude が見つかりません（起動できませんでした）');
+            await patchStatus(cfg, cmd.id, 'failed', `${lastSpawnError || 'claude が見つかりません'}（起動できませんでした）`);
             await sendCommandEvents(cfg, cmd.id, [{
                     type: 'error',
-                    text: 'claude が見つかりません（起動できませんでした）',
+                    text: `${lastSpawnError || 'claude が見つかりません'}（起動できませんでした）`,
                     at: new Date().toISOString(),
                     attempt,
                 }]);

package/package.json

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

package/skills/conductor-craftsman/SKILL.md

@@ -152,7 +152,9 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
 - **秘密のファイルを読む・引数に名前を書く**（`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）。
+- **`-f` 付き、または中身が変わる場所を指す再帰の `rm`**（`rm -rf`・`rm -fr`・`rm -r -f`・`rm -R`・`rm --recursive`／`rm -r "$DIR/x"`・`rm -r build/*` のように変数や `*` を含む）→ 止まる。`-f` 付きは配布している禁止の一覧（現場の `.claude/settings.json` の deny）で**必ず**止まる（番人より先に効く＝消す先が作業ファイルでも通らない・便 fb248f6e）。変数・`*` は番人が止める。**`-f` を付けず、消す先を実在のパスで名指しする**（`/tmp/claude-1000/` の下・鉄則13）。★止められたら**別の消し方を探さず**（`find -delete`・`python -c` 等に回らない）、消せなかった場所を報告に書く（鉄則4）。
+  - 通る例: `rm -r /tmp/claude-1000/作業名` ／ `rm -r node_modules/.tmp-作業名`（★`-f` を付けない・自分で作った物だけ）
+  - 止まる例: `rm -rf /tmp/claude-1000/作業名`（禁止の一覧）／ `rm -r "$DIR/x"`・`rm -r build/*`（番人）／ `rm -r .git`・`rm -r .claude/hooks`・`rm -r .tyhld`（番人＝相対パスでも消せない）
 - **作業場所が変数のままの `git push`**（`git -C "$DIR" push`・行き先のブランチを確かめられない形）→ 止まる。**リポの中で、ブランチを名指しして** `git push -u origin <作業ブランチ名>`。
 
 ## 便の終え方