@tyhld/conductor 0.12.57 → 0.12.59

package/dist/headless.js

@@ -16,7 +16,7 @@
  */
 import { spawn } from 'node:child_process';
 import { createInterface } from 'node:readline';
-import { existsSync } from 'node:fs';
+import { createWriteStream, existsSync, mkdirSync } from 'node:fs';
 import path from 'node:path';
 import os from 'node:os';
 import { RELAY_DELIVER_MAX, RELAY_FETCH_TIMEOUT_MS, RELAY_HEARTBEAT_INTERVAL_MS, RELAY_PENDING_TIMEOUT_MS, RELAY_RESTORE_ORPHANS_ENABLED, buildAnchor, buildSendText, canDeliver, completeWithReport, createRelayState, deliveryReport, fetchCommand, fetchCommandStatus, fetchControl, fetchOrphans, isTerminalCommandStatus, kindOf, markDelivered, noteCentralDown, noteCentralUp, noteReadiness, patchStatus, reconnectDelayMs, selectOrphanToRestore, sendHeartbeat, } from './relay.js';
@@ -38,6 +38,10 @@ export const HEADLESS_STOP_CHECK_MS = 15_000;
 export const EVENT_TEXT_MAX = 500;
 /** resume 時に職人へ渡す続行指示。 */
 const RESUME_PROMPT = '前回の作業の続きを行ってください。中断された箇所から再開してください。';
+/** 実行ログの保存先ルート。 */
+export const RUNS_DIR = path.join(os.homedir(), '.local', 'state', 'conductor', 'runs');
+/** セッションありと見なす最低実行時間(ms)。これより短いとセッションファイルが無い可能性が高い。 */
+export const SESSION_MIN_DURATION_MS = 30_000;
 // ─────────────────────────────────────────────────────────────────────────────
 // claude の実行ファイルの探索
 //
@@ -171,6 +175,32 @@ export function isAuthError(job) {
         s.includes('invalid_api_key'));
 }
 // ─────────────────────────────────────────────────────────────────────────────
+// 実行ログ
+// ─────────────────────────────────────────────────────────────────────────────
+/** 実行ログのディレクトリを作り、stdout / stderr の WriteStream を返す。 */
+function openRunLogs(commandId, attempt) {
+    try {
+        const dir = path.join(RUNS_DIR, commandId);
+        mkdirSync(dir, { recursive: true });
+        const stdoutLog = createWriteStream(path.join(dir, `attempt-${attempt}.jsonl`), { flags: 'a' });
+        const stderrLog = createWriteStream(path.join(dir, `attempt-${attempt}.stderr.log`), { flags: 'a' });
+        return { stdoutLog, stderrLog };
+    }
+    catch (err) {
+        log(`WARN 実行ログのディレクトリを作れません: ${err instanceof Error ? err.message : String(err)}`);
+        return null;
+    }
+}
+/** WriteStream を安全に閉じる。 */
+function closeLog(ws) {
+    if (ws) {
+        try {
+            ws.end();
+        }
+        catch { /* ignore */ }
+    }
+}
+// ─────────────────────────────────────────────────────────────────────────────
 // 子プロセスの起動と停止
 // ─────────────────────────────────────────────────────────────────────────────
 /**
@@ -223,6 +253,7 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         log(`ERROR claude の起動に失敗しました (${claudeBin}): ${err instanceof Error ? err.message : String(err)}`);
         return null;
     }
+    const logs = openRunLogs(cmd.id, attempt);
     const job = {
         commandId: cmd.id,
         process: proc,
@@ -237,6 +268,9 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         lastFlushAt: Date.now(),
         lastStopCheckAt: Date.now(),
         stderrBuf: '',
+        stdoutLog: logs?.stdoutLog ?? null,
+        stderrLog: logs?.stderrLog ?? null,
+        durationMs: 0,
     };
     // spawn の 'error' イベント（ENOENT 等は非同期で届く。同期例外と両方捕まえる）
     proc.on('error', (err) => {
@@ -261,10 +295,11 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         }
         proc.stdin.end();
     }
-    // stdout: stream-json を行ごとに読み、CommandEvent に変換してバッファへ
+    // stdout: stream-json を行ごとに読み、CommandEvent に変換してバッファへ。ログファイルにも書く。
     if (proc.stdout) {
         const rl = createInterface({ input: proc.stdout });
         rl.on('line', (line) => {
+            job.stdoutLog?.write(line + '\n');
             try {
                 const parsed = JSON.parse(line);
                 const event = streamEventToCommandEvent(parsed, attempt);
@@ -276,10 +311,12 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
             }
         });
     }
-    // stderr: 蓄積（認証エラー検出用。上限5000字で先頭を切り落とす）
+    // stderr: 蓄積（認証エラー検出用。上限5000字で先頭を切り落とす）。ログファイルにも書く。
     if (proc.stderr) {
         proc.stderr.on('data', (chunk) => {
-            job.stderrBuf += chunk.toString('utf8');
+            const text = chunk.toString('utf8');
+            job.stderrLog?.write(text);
+            job.stderrBuf += text;
             if (job.stderrBuf.length > 5000) {
                 job.stderrBuf = job.stderrBuf.slice(-3000);
             }
@@ -290,6 +327,9 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         job.completed = true;
         job.exitCode = code;
         job.exitSignal = signal?.toString() ?? null;
+        job.durationMs = Date.now() - job.startedAt;
+        closeLog(job.stdoutLog);
+        closeLog(job.stderrLog);
     });
     return job;
 }
@@ -417,9 +457,10 @@ async function checkRunningJob(cfg, state) {
             return;
         }
         // 異常終了 → 次の tick で自動再開を試みる
+        const stderrTail = job.stderrBuf.slice(-300);
         await sendCommandEvents(cfg, job.commandId, [{
                 type: 'error',
-                text: maskSecrets(`異常終了 (exit=${job.exitCode}, signal=${job.exitSignal}): ${job.stderrBuf.slice(0, 300)}`).slice(0, EVENT_TEXT_MAX),
+                text: maskSecrets(`異常終了 (exit=${job.exitCode}, signal=${job.exitSignal}): ${stderrTail}`).slice(0, EVENT_TEXT_MAX),
                 at: new Date().toISOString(),
                 attempt: job.attempt,
             }]);
@@ -429,6 +470,8 @@ async function checkRunningJob(cfg, state) {
             rs.abandoned.add(job.commandId);
         }
         else {
+            // ★ pendingRetry に詰めて、次の tick で実際に spawn する（fetchCommand は sent を返さない）
+            state.pendingRetry = { cmd: { id: job.commandId, body: '' }, attempt: nextAttempt, durationMs: job.durationMs };
             log(`WARN site=${cfg.site} id=${job.commandId} 異常終了(exit=${job.exitCode}) → ` +
                 `次の tick で再開を試みます (attempt ${nextAttempt}/${HEADLESS_MAX_ATTEMPTS})`);
         }
@@ -584,6 +627,39 @@ export async function headlessTick(cfg, state, hooks) {
             log('停止が解除されました。relayを再開します');
         }
         rs.lastHalted = false;
+        // ── 異常終了後の自動再開（pendingRetry）──
+        if (state.pendingRetry !== null) {
+            const retry = state.pendingRetry;
+            state.pendingRetry = null;
+            state.attempts.set(retry.cmd.id, retry.attempt);
+            const sessionId = sessionIdForCommand(retry.cmd.id);
+            // 前回がセッションを作れないほど短命（< 30秒）なら fresh start、そうでなければ resume
+            const isResume = retry.attempt > 1 && retry.durationMs >= SESSION_MIN_DURATION_MS;
+            const job = spawnClaudeProcess(cfg, retry.cmd, sessionId, retry.attempt, isResume);
+            if (job === null) {
+                await sendCommandEvents(cfg, retry.cmd.id, [{
+                        type: 'error',
+                        text: 'claude が見つかりません（自動再開を中止）',
+                        at: new Date().toISOString(),
+                        attempt: retry.attempt,
+                    }]);
+                rs.abandoned.add(retry.cmd.id);
+                log(`ERROR site=${cfg.site} id=${retry.cmd.id} 自動再開: claude が見つからない → 放棄`);
+                return;
+            }
+            state.job = job;
+            rs.lastHeartbeatAt = null;
+            noteReadiness(rs, 'delivered', retry.cmd.id);
+            await sendCommandEvents(cfg, retry.cmd.id, [{
+                    type: 'resumed',
+                    text: `attempt ${retry.attempt}/${HEADLESS_MAX_ATTEMPTS} (自動再開)`,
+                    at: new Date().toISOString(),
+                    attempt: retry.attempt,
+                }]);
+            log(`site=${cfg.site} id=${retry.cmd.id} ` +
+                `自動再開しました (session=${sessionId}, attempt ${retry.attempt}/${HEADLESS_MAX_ATTEMPTS})`);
+            return;
+        }
         // ── 新規指示の取得 ──
         const cmd = await fetchCommand(cfg);
         if (!cmd) {
@@ -654,6 +730,7 @@ export function createHeadlessState() {
         job: null,
         attempts: new Map(),
         restored: false,
+        pendingRetry: null,
     };
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tyhld/conductor",
-  "version": "0.12.57",
+  "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 <作業ブランチ名>`。