@tyhld/conductor 0.12.66 → 0.12.68

package/dist/headless.js

@@ -16,7 +16,7 @@
  */
 import { execFileSync, spawn } from 'node:child_process';
 import { createInterface } from 'node:readline';
-import { createWriteStream, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from 'node:fs';
+import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, unlinkSync, writeFileSync } 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';
@@ -129,35 +129,51 @@ export function loadAndClearLegacyActiveJob() {
     }
 }
 /**
- * 手元の記録が自分（この中継）のものか照合する。
- * site, machine, workDir, gitOrigin を比べる。
- * 1点でも不一致なら理由を返す。すべて一致なら null。
+ * 手元の記録が自分（この中継）のものか照合する（Fail Closed）。
+ * site, machine, workDir, gitOrigin の4項目を比べる。
+ *
+ * ★Fail Closed: 確認できない項目（記録に無い・今の値が取れない）は
+ *   不一致と同じ扱いにする。確認できた項目がすべて一致した場合だけ null を返す。
  *
  * ★commandId は「この便が中央でまだ sent か」の確認（ネットワーク照合）で別に行う。
  *   ここではローカルの所有者照合のみ。
  */
 export function validateActiveJobOwnership(rec, cfg) {
     const mismatches = [];
-    if (rec.site !== undefined && rec.site !== cfg.site) {
+    // site — 記録に無ければ確認不能
+    if (rec.site === undefined) {
+        mismatches.push('site: 記録に無い（確認不能）');
+    }
+    else if (rec.site !== cfg.site) {
         mismatches.push(`site: 記録=${rec.site} 自分=${cfg.site}`);
     }
-    if (rec.machine !== undefined && rec.machine !== cfg.machine) {
+    // machine — 記録に無ければ確認不能
+    if (rec.machine === undefined) {
+        mismatches.push('machine: 記録に無い（確認不能）');
+    }
+    else if (rec.machine !== cfg.machine) {
         mismatches.push(`machine: 記録=${rec.machine} 自分=${cfg.machine}`);
     }
-    if (rec.workDir !== undefined && rec.workDir !== cfg.workDir) {
+    // workDir — 記録に無ければ確認不能
+    if (rec.workDir === undefined) {
+        mismatches.push('workDir: 記録に無い（確認不能）');
+    }
+    else if (rec.workDir !== cfg.workDir) {
         mismatches.push(`workDir: 記録=${rec.workDir} 自分=${cfg.workDir}`);
     }
-    // ★git リポの同一性: パスが同じでも別のリポなら止める
-    if (rec.gitOrigin !== undefined) {
+    // gitOrigin — 記録に無い or 今の値が取れない → 確認不能
+    if (rec.gitOrigin === undefined) {
+        mismatches.push('gitOrigin: 記録に無い（確認不能）');
+    }
+    else {
         const currentOrigin = resolveGitOrigin(cfg.workDir);
-        if (currentOrigin !== null && rec.gitOrigin !== currentOrigin) {
+        if (currentOrigin === null) {
+            mismatches.push('gitOrigin: 今のリポの origin を取得できない（確認不能）');
+        }
+        else if (rec.gitOrigin !== currentOrigin) {
             mismatches.push(`gitOrigin: 記録=${rec.gitOrigin} 自分=${currentOrigin}`);
         }
     }
-    // 照合項目がそろわない古い記録は安全に拾えない。
-    if (rec.site === undefined || rec.machine === undefined || rec.workDir === undefined) {
-        mismatches.push('照合項目が不足（古い記録）');
-    }
     return mismatches.length > 0 ? mismatches.join(', ') : null;
 }
 // ─────────────────────────────────────────────────────────────────────────────
@@ -174,7 +190,6 @@ function claudeBinCandidates() {
     // nvm の各 node 版の bin（新しい版から）
     const nvmDir = path.join(h, '.nvm', 'versions', 'node');
     try {
-        const { readdirSync } = require('node:fs');
         const versions = readdirSync(nvmDir)
             .filter((v) => v.startsWith('v'))
             .sort((a, b) => {
@@ -191,8 +206,11 @@ function claudeBinCandidates() {
             candidates.push(path.join(nvmDir, v, 'bin', 'claude'));
         }
     }
-    catch {
-        // nvm が無い場合
+    catch (e) {
+        // nvm が無い（ENOENT）は正常。それ以外は探索の失敗として記録する。
+        if (e?.code !== 'ENOENT') {
+            console.error('[conductor] nvm claude exploration failed:', e);
+        }
     }
     // その他の既知の場所
     candidates.push(path.join(h, '.local', 'bin', 'claude'), path.join(h, '.claude', 'local', 'claude'), '/usr/local/bin/claude', '/usr/bin/claude');
@@ -755,25 +773,38 @@ export async function restoreHeadlessOrphans(cfg, state) {
         log(`site=${cfg.site} 古い共有の active-job.json を消しました（現場ごとの記録を優先）`);
     }
     if (saved !== null) {
-        // ★4点照合: machine, site, commandId, workDir がすべて一致しなければ再開しない
+        // ★所有者照合（Fail Closed）: site, machine, workDir, gitOrigin がすべて確認でき一致しなければ再開しない
         const mismatch = validateActiveJobOwnership(saved, cfg);
         if (mismatch !== null) {
-            log(`WARN site=${cfg.site} id=${saved.commandId} 手元の記録は別の中継のもの → 再開しない（不一致: ${mismatch}）`);
+            log(`WARN site=${cfg.site} id=${saved.commandId} 手元の記録は自分のものと確認できない → 再開しない（${mismatch}）`);
             await sendCommandEvents(cfg, saved.commandId, [{
                     type: 'error',
-                    text: `手元の記録の4点照合で不一致 → 再開しない（${mismatch}）`.slice(0, EVENT_TEXT_MAX),
+                    text: `所有者照合で再開しない（${mismatch}）`.slice(0, EVENT_TEXT_MAX),
                     at: new Date().toISOString(),
                     attempt: saved.attempt,
                 }]);
             return;
         }
-        // ★便IDの照合: 中央でまだ sent のまま残っている便か確かめる
+        // ★便IDの照合（Fail Closed）: 中央でまだ sent のまま残っている便か確かめる。
+        // 確かめられなかった（null）ときも再開しない。ただし記録を戻して次の tick で再試行する。
         const centralStatus = await fetchCommandStatus(cfg, saved.commandId);
-        if (centralStatus !== null && centralStatus !== 'sent') {
-            log(`WARN site=${cfg.site} id=${saved.commandId} 手元の記録の便は中央で ${centralStatus} → 再開しない`);
+        if (centralStatus === null) {
+            log(`WARN site=${cfg.site} id=${saved.commandId} 中央の状態を確かめられない → 再開しない（次の tick で再試行）`);
+            // ★記録を戻す: 次の tick でもう一度確かめられるようにする（便を黙って捨てない）
+            saveActiveJob(cfg.site, saved);
+            await sendCommandEvents(cfg, saved.commandId, [{
+                    type: 'error',
+                    text: '中央の便の状態を確かめられない（ネットワーク障害等）→ 再開しない（次の tick で再試行）'.slice(0, EVENT_TEXT_MAX),
+                    at: new Date().toISOString(),
+                    attempt: saved.attempt,
+                }]);
+            return;
+        }
+        if (centralStatus !== 'sent') {
+            log(`WARN site=${cfg.site} id=${saved.commandId} 中央で ${centralStatus} → 再開しない`);
             await sendCommandEvents(cfg, saved.commandId, [{
                     type: 'error',
-                    text: `便IDの照合: 中央の状態が sent ではない（${centralStatus}）→ 再開しない`.slice(0, EVENT_TEXT_MAX),
+                    text: `中央の便の状態が sent ではない（${centralStatus}）→ 再開しない`.slice(0, EVENT_TEXT_MAX),
                     at: new Date().toISOString(),
                     attempt: saved.attempt,
                 }]);
@@ -794,6 +825,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 が見つかりません（手元の孤児復元を中止）');
             state.relay.abandoned.add(pick.id);
             log(`ERROR site=${cfg.site} id=${pick.id} 手元の孤児復元: claude が見つからない → 放棄`);
             return;
@@ -813,10 +845,31 @@ export async function restoreHeadlessOrphans(cfg, state) {
         return;
     }
     // ── 経路2: 中央から孤児を拾う（PC 再起動後） ──
+    // ★Fail Closed: 中央の孤児には machine/workDir/gitOrigin が無い。
+    //   手元の記録（active-job ファイル）が無い場合、所有者照合で確認不能 → 再開しない。
+    //   手元の記録があれば経路1で拾われているので、経路2に来る時点で手元の記録は無い。
     const orphans = await fetchOrphans(cfg);
     const pick = selectOrphanToRestore(orphans, null);
     if (pick === null)
         return;
+    // ★所有者照合（Fail Closed）: 中央の孤児は所有者情報を持たないため確認不能。
+    //   手元の記録があれば経路1で処理済み。ここに来る＝手元の記録が無い＝確認不能。
+    const orphanAsRecord = {
+        commandId: pick.id, body: pick.body, sessionId: '', attempt: 0,
+        startedAt: 0, site: pick.site,
+        // machine, workDir, gitOrigin は中央に無い → undefined → 確認不能
+    };
+    const orphanMismatch = validateActiveJobOwnership(orphanAsRecord, cfg);
+    if (orphanMismatch !== null) {
+        log(`WARN site=${cfg.site} id=${pick.id} 中央の孤児は所有者を確認できない → 再開しない（${orphanMismatch}）`);
+        await sendCommandEvents(cfg, pick.id, [{
+                type: 'error',
+                text: `中央の孤児の所有者照合で再開しない（${orphanMismatch}）`.slice(0, EVENT_TEXT_MAX),
+                at: new Date().toISOString(),
+                attempt: 0,
+            }]);
+        return;
+    }
     const attempt = (state.attempts.get(pick.id) ?? 0) + 1;
     state.attempts.set(pick.id, attempt);
     if (attempt > HEADLESS_MAX_ATTEMPTS) {
@@ -830,6 +883,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 sendCommandEvents(cfg, pick.id, [{
                 type: 'error',
                 text: 'claude が見つかりません（孤児復元を中止）',
@@ -923,6 +977,7 @@ 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 sendCommandEvents(cfg, retry.cmd.id, [{
                         type: 'error',
                         text: 'claude が見つかりません（自動再開を中止）',
@@ -979,6 +1034,7 @@ 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 sendCommandEvents(cfg, cmd.id, [{
                     type: 'error',
                     text: 'claude が見つかりません（起動できませんでした）',

package/package.json

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