@tyhld/conductor 0.12.80 → 0.12.81

package/dist/cli.js

@@ -195,6 +195,9 @@ async function sendReport(cfg) {
             body.undeliveredSec = d.undeliveredSec;
         if (d.queuedCommandId !== null)
             body.queuedCommandId = d.queuedCommandId;
+        // ★配った便の完了待ちの長さ（'waiting-done' のときだけ）。管制が長すぎる待ちを見つけられる。
+        if (d.waitingSec !== null)
+            body.waitingSec = d.waitingSec;
     }
     // ★未処理の報告（まだ管制へ渡せていない控え）の指示IDを載せる（便 57c04e6d）。
     //   1件も無ければ載せない＝今までと同じ本文。多すぎるときは上限までにして総数を添える。
@@ -348,6 +351,10 @@ async function start(args) {
         }
         nudgeWatcher = watchNudges(cfg.site, () => {
             console.log(`[relay] site=${cfg.site} 耳から合図 → 指示の取得を前倒しします`);
+            // ★合図が空振りしたときに理由を1行出せるよう、受けた時刻を relay へ渡す（案B-3）。
+            //   relayTick が「前の周が走行中」「追跡中」で return するとき、しぼり付きで理由を記録する。
+            if (relayStateForReport)
+                relayStateForReport.nudgedAt = Date.now();
             void relayRunner?.run();
         });
     }

package/dist/relay.js

@@ -239,6 +239,8 @@ export function createRelayState() {
         abandoned: new Set(),
         promptPending: false,
         busyRaisedId: null,
+        nudgedAt: null,
+        lastNudgeLogAt: null,
         central: { failures: 0, detail: '', downSince: null },
         readiness: createReadiness(),
     };
@@ -277,12 +279,58 @@ export function noteReadiness(state, next, commandId, now = Date.now()) {
 export function deliveryReport(state, now = Date.now()) {
     const r = state.readiness;
     const blocked = BLOCKED_STATES.has(r.state) && r.commandId !== null;
+    const waiting = r.state === 'waiting-done' && r.commandId !== null;
     return {
         deliveryState: r.state,
         queuedCommandId: blocked ? r.commandId : null,
         undeliveredSec: blocked ? Math.max(0, Math.round((now - r.since) / 1000)) : null,
+        waitingSec: waiting ? Math.max(0, Math.round((now - r.since) / 1000)) : null,
     };
 }
+/**
+ * 配った便の完了を待っている1周を記録する（★無言の周をなくす・案B-2）。
+ *
+ * 【なぜ要るか】追跡中(pending)の周は、理由を1行も書かずに return する経路が複数あった。
+ *   `noteBusyEdge` は【変わり目だけ】出す設計なので、同じ状態が続く限り何も出ない。
+ *   「作業中の可能性ありスキップ」のログは pending が無い周にしか到達しない。
+ *   ＝実機 2026-10-02 は、空の入力待ち画面を「作業中」と誤読したまま10分以上、
+ *   現場のログに1行も出なかった。**この1行があれば即座に原因が分かった。**
+ *
+ * 【★画面をどう読んだかを必ず書く】これが要。「経過」だけでは誤読に気づけない。
+ * 【★記録を埋めない】生存通知と同じしぼり（RELAY_HEARTBEAT_INTERVAL_MS=150秒）に相乗りする。
+ *
+ * @returns 実際に1行出したら true（しぼりで出さなかったら false）。
+ */
+export function noteWaitingDone(state, site, screen, askedCentral, logLine, now = Date.now()) {
+    const p = state.pending;
+    if (p === null)
+        return false;
+    const last = p.lastWaitLogAt ?? null;
+    if (last !== null && now - last < RELAY_HEARTBEAT_INTERVAL_MS)
+        return false;
+    p.lastWaitLogAt = now;
+    logLine(`site=${site} id=${p.id} 完了待ち（経過${Math.max(0, Math.round((now - p.sentAt) / 1000))}s` +
+        `・画面=${screen}・中央へ問い合わせ=${askedCentral ? 'した' : 'しない'}）`);
+    return true;
+}
+/**
+ * 合図（耳からの「今すぐ1周」）を受けたのに取得へ進まなかった理由を1行出す（★案B-3）。
+ *   ★しぼり（RELAY_HEARTBEAT_INTERVAL_MS）付き＝記録を埋めない。出したら合図は消費する。
+ *
+ * @returns 実際に1行出したら true。
+ */
+export function noteNudgeSkipped(state, site, reason, logLine, now = Date.now()) {
+    if (state.nudgedAt === null)
+        return false;
+    state.nudgedAt = null; // 合図は1回ぶん。消費して次の合図を待つ。
+    if (state.lastNudgeLogAt !== null &&
+        now - state.lastNudgeLogAt < RELAY_HEARTBEAT_INTERVAL_MS) {
+        return false;
+    }
+    state.lastNudgeLogAt = now;
+    logLine(`site=${site} 合図を受けましたが${reason}のため、この周では指示を取得しません`);
+    return true;
+}
 /**
  * 管制へつながらないときの、次の再試行までの待ち時間(ms)。純粋関数（I/Oなし＝単体テスト可能）。
  *
@@ -425,6 +473,16 @@ export const CODEX_MARKERS = [
 export const CODEX_BUSY_MARKERS = ['esc to interrupt', 'working ('];
 /** Codex の選択メニューのカーソル（実測: U+203A）。★Claude の `❯`(U+276F) とは別物。 */
 export const CODEX_MENU_CURSOR = '›';
+/**
+ * Codex の「待機」に要求する、入力欄の行の位置（末尾から何行以内か）。
+ *
+ * 【根拠（実測）】0.160.0 の起動画面は末尾の空行を落とすと
+ *   `› Ask Codex to do anything` / 空行 / `GPT-6.1-Sol default · ~/codex-probe` / `? for shortcuts`
+ *   ＝入力欄は末尾から4行目。0.152.1 の待機画面は末尾から2〜3行目（test/relay-agent-kind.test.ts）。
+ *   余裕を見て8行に取る。★画面を覆う表示（警告・全画面のダイアログ）では入力欄が
+ *   1行も見えないので、何行に取っても弾ける＝この値は「見えている待機」を取りこぼさないための幅。
+ */
+export const CODEX_IDLE_INPUT_WITHIN_LINES = 8;
 /**
  * 種類ごとの目印を返す。★関数にしてあるのは、定数の宣言順に縛られないようにするため
  * （CLAUDE_MARKERS などはこの下で宣言される）。
@@ -434,29 +492,52 @@ export function agentProfile(kind) {
         return {
             markers: CODEX_MARKERS,
             busyMarkers: CODEX_BUSY_MARKERS,
+            // ★Codex は回転印を見ない（起動画面の点字ロゴを作業中と誤読する＝調査 67f8c51f）。
+            busyGlyphs: [],
             cursor: CODEX_MENU_CURSOR,
             idleRule: 'not-busy',
+            idleInputLineWithin: CODEX_IDLE_INPUT_WITHIN_LINES,
         };
     }
     return {
         markers: CLAUDE_MARKERS,
         busyMarkers: BUSY_MARKERS,
+        // ★claude は従来どおり BUSY_GLYPHS そのまま（判定を1文字も変えない）。
+        busyGlyphs: BUSY_GLYPHS,
         cursor: '❯',
         idleRule: 'prompt',
+        idleInputLineWithin: null,
     };
 }
 /** 種類を見て「作業中か」を判定する。claude は従来の hasBusySignal と同一。 */
 export function hasBusySignalFor(text, kind) {
     if (kind === 'claude')
         return hasBusySignal(text);
+    const profile = agentProfile(kind);
     const regionText = busyScanRegion(text);
     const lower = regionText.toLowerCase();
-    if (agentProfile(kind).busyMarkers.some((m) => lower.includes(m)))
+    if (profile.busyMarkers.some((m) => lower.includes(m)))
         return true;
-    if (BUSY_GLYPHS.some((g) => regionText.includes(g)))
+    // ★回転印は種類ごとの表から引く（Codex は空＝見ない）。従来は BUSY_GLYPHS を共有していた。
+    if (profile.busyGlyphs.some((g) => regionText.includes(g)))
         return true;
     return false;
 }
+/**
+ * 「入力欄の行（cursor で始まる最後の行）が末尾から within 行以内に在るか」を返す純粋関数。
+ *   ★枠の装飾（│ など）は剥がしてから見る。末尾の空行は数えない（tmux は下を空行で埋める）。
+ */
+export function hasInputLineNearBottom(text, cursor, within) {
+    const lines = text.replace(/\r/g, '').split('\n');
+    while (lines.length > 0 && lines[lines.length - 1].trim() === '')
+        lines.pop();
+    const from = Math.max(0, lines.length - within);
+    for (let i = lines.length - 1; i >= from; i--) {
+        if (lines[i].replace(/^[│|╎┃\s]+/, '').startsWith(cursor))
+            return true;
+    }
+    return false;
+}
 /**
  * 画面テキストから「アイドルか」を判定する純粋関数（判定(a)）。
  *   方針:「確実な処理中シグナル（esc to interrupt / compacting / スピナーグリフ）が無く、
@@ -698,12 +779,21 @@ export function looksLikeAgent(text, kind) {
 export function looksIdleFor(text, kind) {
     if (kind === 'claude')
         return looksIdle(text);
+    const profile = agentProfile(kind);
     if (hasBusySignalFor(text, kind))
         return false;
     if (hasChoiceMenuFor(text, kind))
         return false;
     // ★その窓が Codex だと分かるときだけ「待機」と言い切る。分からない窓は待機にしない。
-    return looksLikeAgent(text, kind);
+    if (!looksLikeAgent(text, kind))
+        return false;
+    // ★入力欄が末尾付近に見えることを必須にする（案C-2・AgentProfile.idleInputLineWithin 参照）。
+    //   画面を覆う表示（0.160.0 の警告の表示など）は入力欄を隠すので、ここで弾ける。
+    if (profile.idleInputLineWithin !== null &&
+        !hasInputLineNearBottom(text, profile.cursor, profile.idleInputLineWithin)) {
+        return false;
+    }
+    return true;
 }
 /**
  * 画面テキストに「完了マーカー DONE:{site} が単独で出力されているか」を判定する純粋関数。
@@ -2894,6 +2984,7 @@ export function startKeepalive(state, pending, now) {
         anchor: pending.anchor,
         startedAt: now,
         lastHeartbeatAt: null,
+        lastCentralProbeAt: null,
         screenDoneUsable: true,
     };
     return true;
@@ -2916,8 +3007,12 @@ export function decideKeepalive(input) {
         return 'wait';
     if (input.doneSeen)
         return 'done';
-    if (input.alive)
+    if (input.alive) {
+        // ★alive が続いていても、しぼりが空いたら中央へ答え合わせする（画面の誤読から必ず抜ける道）。
+        if (input.centralProbeOk === true)
+            return 'alive-probe';
         return input.throttleOk ? 'heartbeat' : 'wait';
+    }
     if (input.idle)
         return 'probe';
     return 'wait';
@@ -2970,9 +3065,34 @@ export async function keepaliveTick(cfg, state, deps = {}) {
         alive: cap !== null && (kaBusySignal || kaMenu),
         idle: cap !== null && looksIdleFor(cap, kindOf(cfg)),
         throttleOk: ka.lastHeartbeatAt === null || now - ka.lastHeartbeatAt >= RELAY_HEARTBEAT_INTERVAL_MS,
+        // ★中央への答え合わせのしぼり。生存通知と同じ150秒（RELAY_HEARTBEAT_INTERVAL_MS）。
+        centralProbeOk: ka.lastCentralProbeAt === null || now - ka.lastCentralProbeAt >= RELAY_HEARTBEAT_INTERVAL_MS,
     });
     if (action === 'wait')
         return;
+    if (action === 'alive-probe') {
+        // ★画面は「作業中」に見えるが、中央にも必ず聞く（画面の誤読から抜ける唯一の道・案A-2）。
+        ka.lastCentralProbeAt = now;
+        const alive = await getStatus(cfg, ka.id);
+        if (isCentralFinished(alive)) {
+            logLine(`site=${cfg.site} id=${ka.id} 見張り終了: 中央で終端(status=${alive})を確認` +
+                '（画面は作業中に見えたが中央の真実で畳む）');
+            noteBusyEdge(state, cfg.site, ka.id, false, logLine);
+            state.keepalive = null;
+            return;
+        }
+        // 終端でない＝従来の 'heartbeat' と同じ扱い（手は挙げたまま生存通知）。しぼりは尊重する。
+        const raiseAlive = shouldRaiseBusy(kaBusySignal, kaMenu);
+        noteBusyEdge(state, cfg.site, ka.id, raiseAlive, logLine);
+        if (ka.lastHeartbeatAt === null || now - ka.lastHeartbeatAt >= RELAY_HEARTBEAT_INTERVAL_MS) {
+            ka.lastHeartbeatAt = now;
+            logLine(`site=${cfg.site} id=${ka.id} 見張り継続: 職人は生きています→生存通知` +
+                `（中央へ問い合わせ=した・status=${alive ?? '不明'}` +
+                `・経過${Math.round((now - ka.startedAt) / 1000)}s）`);
+            await heartbeat(cfg, ka.id, raiseAlive);
+        }
+        return;
+    }
     if (action === 'heartbeat') {
         ka.lastHeartbeatAt = now;
         const raise = shouldRaiseBusy(kaBusySignal, kaMenu);
@@ -3102,8 +3222,11 @@ export async function restoreOrphans(cfg, state) {
     log(`site=${cfg.site} id=${pick.id} を sent孤児から pending へ復元（★配達はしない・DONE待ちと生存通知だけ）`);
 }
 export async function relayTick(cfg, state, hooks) {
-    if (state.running)
+    if (state.running) {
+        // ★合図を受けていたなら、空振りした理由を1行残す（しぼり付き・案B-3）。
+        noteNudgeSkipped(state, cfg.site, '前の周がまだ走っている', log);
         return; // 前回の tick が走行中ならスキップ
+    }
     state.running = true;
     // 承認待ちの観測値は毎tick「出ていない」から始める（下でプロンプトを検知したときだけ true にする）。
     // ここで false に戻しておくと、プロンプトが消えた／pending が無い／早期 return のどの経路でも、
@@ -3135,9 +3258,18 @@ export async function relayTick(cfg, state, hooks) {
         await keepaliveTick(cfg, state);
         // ── 完了マーカー検出（pending があれば最優先・停止中でも継続） ──
         if (state.pending) {
-            const cap = tmuxCapture(cfg.site);
-            if (cap === null)
+            // ★この周の結末を「配った便の完了待ち」として記録する（案B-1）。ここで1回呼べば、
+            //   以下のどの return 経路を通っても管制の札が 'delivered' のまま固まらない。
+            //   同じ状態・同じ指示の間 since は動かないので、待っている長さがそのまま測れる。
+            noteReadiness(state, 'waiting-done', state.pending.id);
+            // ★合図が来ていたなら、取得へ進まない理由を1行残す（しぼり付き・案B-3）。
+            noteNudgeSkipped(state, cfg.site, `配った便(id=${state.pending.id})の完了を待っている`, log);
+            const cap = (hooks?.capture ?? tmuxCapture)(cfg.site);
+            if (cap === null) {
+                // ★無言で返さない（案B-2）。画面が読めない周も、しぼり付きで1行残す。
+                noteWaitingDone(state, cfg.site, '読めない', false, log);
                 return; // tmux未取得: 次回再確認
+            }
             // ── 完了検出の走査範囲を広げてよいかの観測（★状態更新には使わない・ADR-023） ──
             // アンカーを一度でも画面で見たら、以後アンカーが流れて消えても全画面から DONE を拾ってよい
             // （ADR-020 の根拠）。★ここで観測するのは「どこまで見てよいか」だけで、配達の成否や
@@ -3211,9 +3343,25 @@ export async function relayTick(cfg, state, hooks) {
             // 【副シグナル温存＝無損失縮退】中央 status が読めない(null＝エンドポイント未実装/不達)ときだけ、
             //   従来の detectExternalDone（別idが配れる＝終端の確証）へフォールバックする。中央側 status API が
             //   入るまではこの副シグナルが従来どおり効き、入った後は主シグナル(status)が確実に固着を断つ。
-            // HTTP コストを抑えるため「印が付いている＋アイドル」のときだけ probe する（busyな通常作業中は叩かない）。
+            // 【★画面の判定から切り離す（根治・案A-2・調査 67f8c51f）】
+            //   従来の条件は「印が付いている＋**画面がアイドルに見える**」だった。ところが画面の目印は
+            //   職人側の版上げで必ず壊れる。実機 2026-10-02 は Codex 0.160.0 の起動画面（点字のロゴ）を
+            //   「作業中」と誤読し、idle が永久に false になった＝**中央の真実を知る唯一の道が閉じた**。
+            //   中央でとうに done の便を15分抱え、そのあと見張りへ渡って最長3時間ふさがった。
+            //   ＝画面が外れたら逃げ道も消える、という構造そのものが穴。
+            //   そこで条件を「印が付いている」だけにし、代わりに時計でしぼる:
+            //     ・アイドルに見える周 … 従来どおり毎tick叩く（★いちばん速い道は1バイトも遅くしない）
+            //     ・それ以外の周      … 150秒（RELAY_HEARTBEAT_INTERVAL_MS・生存通知と同じ時計）に1回
+            //   ＝HTTP の増分は1指示あたり最大で150秒に1回。画面の読み取りが外れても、中央で終わった便は
+            //     必ず150秒以内の周で追跡解除され、次の便が配られる。
             const idle = looksIdleFor(cap, kindOf(cfg));
-            if (state.pending.sentMarked && idle) {
+            const probeBase = state.pending.lastStatusProbeAt ?? state.pending.sentAt;
+            const probeDue = Date.now() - probeBase >= RELAY_HEARTBEAT_INTERVAL_MS;
+            const askedCentral = state.pending.sentMarked && (idle || probeDue);
+            // ★画面をどう読んだかを必ず残す（案B-2）。これが1行あれば今回の誤読は即座に分かった。
+            noteWaitingDone(state, cfg.site, idle ? '待機' : '作業中', askedCentral, log);
+            if (askedCentral) {
+                state.pending.lastStatusProbeAt = Date.now();
                 const centralStatus = await fetchCommandStatus(cfg, state.pending.id);
                 // ★本文未着(instruction_missing)も追跡をやめる（便 7d810bbb）。管制は占有を外して人の判断待ちに
                 //   出している＝ここで待ち続けると次の便が拾えない。★配り直さない（人が決める）。
@@ -3226,7 +3374,9 @@ export async function relayTick(cfg, state, hooks) {
                 }
                 // 中央 status が未知(null)のときのみ、従来の副シグナル（別idが配信可能＝終端の確証）で救う。
                 //   中央が sent/queued 等の「生存中」を明示的に返した場合は解除しない（中央の真実を尊重）。
-                if (centralStatus === null) {
+                // ★副シグナルは「アイドル」を根拠の一部にしている（detectExternalDone）。だから画面が
+                //   アイドルに見える周だけ叩く＝従来と同じ条件・同じ結論。無駄な取得も増やさない。
+                if (centralStatus === null && idle) {
                     const next = await fetchCommand(cfg);
                     const nextId = next?.id ?? null;
                     if (detectExternalDone({
@@ -3498,7 +3648,15 @@ export async function relayTick(cfg, state, hooks) {
         // 新しい指示の生存ハートビートは先頭から数え直す（前指示のスロットルを引き継がない）。
         state.lastHeartbeatAt = null;
         noteReadiness(state, 'delivered', cmd.id);
-        log(`site=${cfg.site} id=${cmd.id} ${landed ? '配達しました' : '配達を試みましたが貼り付けが入らず、何も送信していません'}`);
+        // ★記録の文面を事実に合わせる（案C-4・調査 67f8c51f）。
+        //   「貼り付けが入力欄に入ったか」を確かめているのは、受領確認を通る職人（claude）だけ。
+        //   それ以外（Codex）は tmux へ送れたかしか見ていないので、画面を覆う表示（起動画面・警告の
+        //   表示）へ打ち込んで本文が捨てられても「配達しました」と書いていた＝人が記録を信じて誤診した
+        //   （実機 2026-10-02）。確認が入る（要判断3）までの間、書く言葉を実際に確かめた範囲に合わせる。
+        //   ★claude の文面は1文字も変えない。
+        log(useAck
+            ? `site=${cfg.site} id=${cmd.id} ${landed ? '配達しました' : '配達を試みましたが貼り付けが入らず、何も送信していません'}`
+            : `site=${cfg.site} id=${cmd.id} 送信しました（入力欄に入ったかは未確認）`);
         // ★配った時点で管制へ印を書く（便 7d810bbb）。Claude の職人は「配達中(受領確認待ち)」、
         //   それ以外は今までどおり「配達済み(sent)」。書けなければ次tick以降で書けるまでやり直す
         //   （markSentAtCentral）。★書けなくても再配達はしない。

package/package.json

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