@tyhld/conductor 0.12.88 → 0.12.90

package/dist/headless.js

@@ -19,7 +19,7 @@ import { createInterface } from 'node:readline';
 import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, unlinkSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
 import os from 'node:os';
-import { CENTRAL_BUSY_HOLD_MAX_MS, RELAY_DELIVER_MAX, RELAY_FETCH_TIMEOUT_MS, RELAY_HEARTBEAT_INTERVAL_MS, RELAY_RESTORE_ORPHANS_ENABLED, buildAnchor, buildSendText, canDeliver, completeWithReport, createRelayState, deliverJobEndFacts, deliveryReport, fetchCommand, fetchCommandStatus, fetchControl, fetchOrphans, flushPendingJobEndFacts, isTerminalCommandStatus, kindOf, markDelivered, noteCentralDown, noteCentralUp, noteDrain, noteReadiness, patchStatus, recoverStuckProcessedReports, reconnectDelayMs, reportPhase, selectOrphanToRestore, sendHeartbeat, } from './relay.js';
+import { CENTRAL_BUSY_HOLD_MAX_MS, RELAY_DELIVER_MAX, RELAY_FETCH_TIMEOUT_MS, RELAY_HEARTBEAT_INTERVAL_MS, RELAY_RESTORE_ORPHANS_ENABLED, buildAnchor, buildSendText, canDeliver, completeWithReport, createRelayState, deliverJobEndFacts, deliveryReport, fetchCommand, fetchCommandStatus, fetchControl, fetchOrphans, flushPendingJobEndFacts, isTerminalCommandStatus, kindOf, markDelivered, noteCentralDown, noteCentralUp, noteDrain, noteReadiness, patchStatus, recoverStuckProcessedReports, reconnectDelayMs, FINISH_GATE_KINDS, reportDelivery, selectOrphanToRestore, sendHeartbeat, } from './relay.js';
 import { claudePinPath, readClaudePinLine, } from './autoupdate-flags.js';
 const log = (msg) => {
     console.log(`[headless] ${msg}`);
@@ -557,6 +557,73 @@ export function jobEndKindOf(j) {
 export function shouldAbortForBodyHandoff(j) {
     return j.bodyHandoff === 'failed' && !j.stopped && !j.timedOut;
 }
+/**
+ * 便の終わりの関門が残した「途中で終わった」理由の控え（便 7aa63a97・便 461be339 で欄を足した）。
+ *
+ * ★番人側（`scripts/hooks/finish_gate.py`）が、同じ理由で止める回数の上限に達したときだけ書く。
+ *   中継はそれを【読んで消すだけ】で、判定はしない（判定は関門が持つ）。
+ *   これが要るのは「管制には『途中で終わった』と分かる形で届く」ため。新しい状態は作らず、
+ *   既にある「終了の事実」に相乗りする（管制の口を変えない）。
+ *   ・人向けの1行 … `jobEndDetail`（今までのまま）
+ *   ・機械で読む形 … `finishGate`（便 461be339 で足した。管制が知らないうちは黙って捨てられる）
+ *   控えが無い便＝関門が止めずに済んだ便なので、今までどおり null。
+ * ★古い版の番人が書いた控え（JSON ではない1行）も読める＝入れ替えの途中でも落とさない。
+ */
+export function takeFinishGateResidue(commandId, env = process.env) {
+    const xdg = (env.XDG_STATE_HOME ?? '').trim();
+    const root = xdg || path.join(env.HOME ?? os.homedir(), '.local', 'state');
+    const safe = String(commandId ?? '').replace(/[^A-Za-z0-9_-]/g, '').slice(0, 128);
+    if (!safe)
+        return null;
+    const file = path.join(root, 'conductor', 'finish-gate', `${safe}.residue`);
+    let raw;
+    try {
+        raw = readFileSync(file, 'utf8').trim();
+    }
+    catch {
+        return null;
+    }
+    // ★読んだら消す（次の周で二重に届けない）。消せなくても届けることを優先する。
+    try {
+        unlinkSync(file);
+    }
+    catch { /* 消せなくても害は無い（同じ便の控えは1つだけ） */ }
+    if (raw.length === 0)
+        return null;
+    return parseFinishGateResidue(raw);
+}
+/** 控えの字面を読む（★純粋関数。JSON でなければ人向けの1行として扱う）。 */
+export function parseFinishGateResidue(raw) {
+    const text = String(raw ?? '').trim();
+    if (text.length === 0)
+        return null;
+    let got = null;
+    try {
+        got = JSON.parse(text);
+    }
+    catch {
+        return { text: text.slice(0, 500), facts: null }; // 古い形（1行だけ）
+    }
+    if (got === null || typeof got !== 'object' || Array.isArray(got)) {
+        return { text: text.slice(0, 500), facts: null };
+    }
+    const rec = got;
+    const line = typeof rec.text === 'string' ? rec.text.trim() : '';
+    if (line.length === 0)
+        return null;
+    const reasons = Array.isArray(rec.reasons)
+        ? FINISH_GATE_KINDS.filter((k) => rec.reasons.includes(k))
+        : [];
+    const countsRaw = (rec.counts ?? null);
+    const counts = {};
+    for (const k of FINISH_GATE_KINDS) {
+        const n = Number(countsRaw?.[k] ?? 0);
+        counts[k] = Number.isFinite(n) && n > 0 ? Math.floor(n) : 0;
+    }
+    // ★「何が残っていたか」が読めない控えは、機械で読む形を作らない（作り話の事実を送らない）。
+    const facts = reasons.length > 0 ? { exhausted: true, reasons: reasons, counts } : null;
+    return { text: line.slice(0, 500), facts };
+}
 /**
  * いま画面なし方式が追っている便の指示ID（★回収の道から除く物・便 6ebba0d0 の b）。
  *
@@ -1251,6 +1318,11 @@ async function checkRunningJob(cfg, state) {
                 authError: isAuthError(job),
                 exitCode: job.exitCode,
             });
+            // ★「報告を出せたか」はファイルの有無【と形】で見る（便 461be339）。
+            //   形が整っていない＝書きかけの報告は「届いた」と数えず、理由を添えて管制へ届ける。
+            //   ★本文の言葉は1文字も読まない（2026-09-23 の確定のまま。下書きの印は関門の側だけ）。
+            const delivery = reportDelivery(job.commandId);
+            const residue = takeFinishGateResidue(job.commandId);
             await deliverJobEndFacts(cfg, {
                 commandId: job.commandId,
                 site: cfg.site,
@@ -1258,12 +1330,20 @@ async function checkRunningJob(cfg, state) {
                 endKind,
                 exitCode: job.exitCode,
                 exitSignal: job.exitSignal,
-                reportDelivered: reportPhase(job.commandId) !== 'none',
+                reportDelivered: delivery.delivered,
                 // ★画面なし方式は中継が自分で立ち上げた処理なので、終わりを直接つかんでいる。
                 //   つかめていないときは false を送る（1-3 の「つかめていない」と同じ形）。
                 processEndObserved: job.finishedAt !== null,
                 startedAt: new Date(job.startedAt).toISOString(),
-                detail: job.bodyHandoffError ?? `attempt ${job.attempt}/${HEADLESS_MAX_ATTEMPTS}`,
+                // ★終わりの関門が「途中で終わった」と残していれば、その理由を先に載せる（便 7aa63a97）。
+                //   本文を渡せなかった理由がいちばん強い（職人は何も受け取っていない）ので、それが先。
+                //   そのどちらも無く、報告の形が整っていないときは、その理由を載せる（便 461be339）。
+                detail: job.bodyHandoffError
+                    ?? residue?.text
+                    ?? delivery.note
+                    ?? `attempt ${job.attempt}/${HEADLESS_MAX_ATTEMPTS}`,
+                // ★上限に達して「途中で終わった」便だけに付ける（管制が機械で見分けられる形）。
+                ...(residue?.facts != null ? { finishGate: residue.facts } : {}),
             }, { logLine: log });
         }
         if (job.timedOut) {

package/dist/relay.js

@@ -1475,6 +1475,81 @@ export function reportPhase(commandId, env) {
     }
     return 'none';
 }
+// ───────────────────────────────────────────────────────────────────────────
+// ★報告を「届いた」と数える条件＝ファイルが在る【かつ】形が整っている（便 461be339）
+//
+// 【なぜ要るか（実地 915694ac・前便 7aa63a97 で残った穴）】従来は reportPhase() が
+//   【ファイルの有無だけ】を見ていた。だから書きかけの報告でも「届いた」と数えられ、
+//   `processed/` へ移され、管制は「終わりました・✅」になった。合流も公開もしていなかった。
+//   便の終わりの関門（`scripts/hooks/finish_gate.py`）は職人が【きれいに手番を閉じたとき】に
+//   しか鳴らないので、処理が落ちた・上限時間で止められた便では書きかけが完了の材料になる道が
+//   残っていた。そこを、中継の側でも閉じる。
+//
+// 【★見るのは「形」だけ。本文の言葉は1文字も読まない】報告の中身の言葉で完了を判定しない
+//   （えふさん確定 2026-09-23）。見るのは parseReport() が返す `missing` だけ＝
+//   囲みが閉じている／5つの欄が在る／outcome が決まった語／evidence が1件以上／本文が空でない。
+//   「作成中」「TODO」などの下書きの印を見るのは関門の側だけ（正本は finish_gate.py）。
+//
+// 【★形の決まりの正本は1か所】`scripts/hooks/report-shape.json`。
+//   REQUIRED_REPORT_FIELDS / REPORT_OUTCOMES がその値と同じであることを
+//   `test/report-shape-gate.test.ts` が固定し、同じ `cases` を関門の試験とこちらで流す
+//   ＝2か所に同じ決まりを書かない。
+//
+// 【★形が整っていないときに何をするか】
+//   ・「届いた」と数えない（`reportDelivered: false` ＋ 理由を添えて管制へ届ける）
+//   ・ファイルは【消さない・processed/ へも移さない】＝職人が書き直せる・人が読める
+//   ・管制へ本文を渡さない（書きかけを完了の材料にしない）
+//
+// 【★例外は1つだけ】「作業なし・本文未着」の申告（isInstructionMissingReport・便 7d810bbb）。
+//   職人は本文を受け取っていないので囲みを書く材料が無い。実地 af960f79 の報告は囲みが無い。
+//   これを止めると「本文未着が管制に届かない」という直した障害が戻るので、形の確かめから外す。
+// ───────────────────────────────────────────────────────────────────────────
+/** 報告の「形」で足りない物（空なら整っている）。★本文の言葉は読まない。 */
+export function reportShapeMissing(raw) {
+    return parseReport(raw).missing;
+}
+/** 形が整っていないことを人へ伝える一言（★報告の中身は1文字も写さない）。 */
+export function reportShapeNote(missing) {
+    return `報告は在りますが形が整っていません（足りない: ${missing.join(', ')}）`;
+}
+/** その指示の報告の段と形を1回で見る（★未処理 → 処理済みの順・readParsedReport と同じ順）。 */
+export function reportDelivery(commandId, env) {
+    const groups = [
+        [pendingCandidates(commandId, env), 'pending'],
+        [processedCandidates(commandId, env), 'processed'],
+    ];
+    for (const [files, phase] of groups) {
+        for (const file of files) {
+            const raw = readIfFilled(file);
+            if (raw === null)
+                continue;
+            const instructionMissing = isInstructionMissingReport(raw);
+            const missing = instructionMissing ? [] : reportShapeMissing(raw);
+            const delivered = missing.length === 0;
+            return {
+                phase,
+                raw,
+                delivered,
+                missing,
+                instructionMissing,
+                note: delivered ? null : reportShapeNote(missing),
+            };
+        }
+    }
+    return {
+        phase: 'none',
+        raw: null,
+        delivered: false,
+        missing: [],
+        instructionMissing: false,
+        note: null,
+    };
+}
+/** 管制へ渡してよい報告の本文（形が整っていなければ null＝渡さない）。 */
+export function readDeliverableReport(commandId, env) {
+    const d = reportDelivery(commandId, env);
+    return d.delivered && d.raw !== null ? parseReport(d.raw).body : null;
+}
 /**
  * 管制へ渡し終えた報告を【処理済み】にする（★消さない・削除しない）。
  *   同じ置き場の `processed/` へ移すだけ。移した時刻を更新時刻に刻む（30日の数え始め）。
@@ -1611,6 +1686,7 @@ export async function sendPendingReportFiles(cfg, opts = {}) {
     const exclude = new Set(opts.exclude ?? []);
     const rejectCounts = opts.rejectCounts ?? new Map();
     const items = [];
+    let malformed = 0;
     for (const id of pendingReportIds(env, REPORT_FILES_MAX_PER_ROUND).ids) {
         for (const file of pendingCandidates(id, env)) {
             const raw = readIfFilled(file);
@@ -1619,6 +1695,17 @@ export async function sendPendingReportFiles(cfg, opts = {}) {
                 //   （便 7d810bbb）。完了の道はその便を完了にしないので、ここで送らないと管制に届かない。
                 if (exclude.has(id) && !isInstructionMissingReport(raw))
                     break;
+                // ★形が整っていない報告は送らない（便 461be339）。未処理のまま残すので、職人が書き直せば
+                //   次の周でそのまま届く。★本文の言葉は1文字も読まない（見るのは形だけ）。
+                if (!isInstructionMissingReport(raw)) {
+                    const missing = reportShapeMissing(raw);
+                    if (missing.length > 0) {
+                        malformed++;
+                        logLine(`site=${cfg.site} id=${id} ${reportShapeNote(missing)}` +
+                            '→管制へ送らず未処理のまま残します（職人が書き直せます）');
+                        break;
+                    }
+                }
                 items.push({ id, raw });
                 break;
             }
@@ -1690,10 +1777,12 @@ export async function sendPendingReportFiles(cfg, opts = {}) {
             }
         }
     }
-    if (items.length > 0) {
-        logLine(`site=${cfg.site} 報告ファイル: 送った ${sent} 件・処理済み ${processed} 件・未処理のまま ${kept} 件${rejected > 0 ? `・退けた ${rejected} 件` : ''}`);
+    if (items.length > 0 || malformed > 0) {
+        logLine(`site=${cfg.site} 報告ファイル: 送った ${sent} 件・処理済み ${processed} 件・未処理のまま ${kept} 件` +
+            `${rejected > 0 ? `・退けた ${rejected} 件` : ''}` +
+            `${malformed > 0 ? `・形が整っていない ${malformed} 件（未処理のまま残す）` : ''}`);
     }
-    return { sent, processed, kept, rejected };
+    return { sent, processed, kept, rejected, malformed };
 }
 /**
  * 古い控えを片付ける。★消すのは `processed/` の中だけ（未処理は何日たっても消さない）。
@@ -2620,17 +2709,22 @@ export async function patchStatus(cfg, id, status, report) {
  * 返り: 追跡を畳んでよいか（true = 便の終了状態が確定した／false = 書けていない＝次の周でやり直す）。
  */
 export async function completeWithReport(cfg, id, logLine = log) {
-    // ★①報告がどの段にあるか（送る工程の話）。②③の実行可否には使わない。
-    const phase = reportPhase(id);
+    // ★①報告がどの段にあるか＋形が整っているか（送る工程の話）。②③の実行可否には使わない。
+    //   ★形が整っていない報告は「届いた」と数えない（便 461be339）。ファイルは消さず・移さず残す。
+    const delivery = reportDelivery(id);
+    const phase = delivery.phase;
     // ★②便が終わっているか（確定の工程の話）。管制の真実だけを見る。
     const already = await fetchCommandStatus(cfg, id);
     if (isCentralFinished(already)) {
         // 既に終端（職人の conductor_complete・人の操作・failed 等）。★何も書かない。
         //   控えは消さずに処理済みとして残す（MCP 経由でも同じ控えが残る・えふさん確定 2026-09-21）。
-        if (phase === 'pending')
+        //   ★形が整っていない報告は未処理のまま置く（職人が書き直せる・人が読める）。
+        if (phase === 'pending' && delivery.delivered)
             markReportProcessed(id);
         logLine(`site=${cfg.site} id=${id} は管制で既に終わっています(status=${already})` +
-            '→relayからは送りません（報告は控えとして処理済みにしました）');
+            (phase === 'pending' && !delivery.delivered
+                ? `→relayからは送りません（${delivery.note}＝未処理のまま残します）`
+                : '→relayからは送りません（報告は控えとして処理済みにしました）'));
         return true;
     }
     if (phase === 'processed') {
@@ -2641,13 +2735,19 @@ export async function completeWithReport(cfg, id, logLine = log) {
         logLine(`site=${cfg.site} id=${id} done（報告は回収の道で渡し済み＝本文は添えない）`);
         return true;
     }
-    const report = readReport(id);
+    // ★形が整っていない報告は添えない（書きかけを完了の材料にしない）。便の確定は今までどおり行う
+    //   ＝現場を占有したままにしない。「報告が届いていない」ことは終了の事実で管制へ届く。
+    const report = delivery.delivered && delivery.raw !== null ? parseReport(delivery.raw).body : null;
     if (!(await patchStatus(cfg, id, 'done', report)))
         return false;
     if (report !== null)
         markReportProcessed(id); // ★消さずに処理済みへ（同じ報告を二度送らない）
     logLine(`site=${cfg.site} id=${id} done` +
-        (report !== null ? `（報告添付・${report.length}文字）` : '（報告ファイルなし＝完了のみ）'));
+        (report !== null
+            ? `（報告添付・${report.length}文字）`
+            : phase === 'none'
+                ? '（報告ファイルなし＝完了のみ）'
+                : `（${delivery.note}＝添えずに未処理のまま残します）`));
     return true;
 }
 /**
@@ -2957,6 +3057,8 @@ export async function sendHeartbeat(cfg, id, busy = false, liveness) {
         log(`site=${cfg.site} id=${id} heartbeat -> 失敗: ${err instanceof Error ? err.message : String(err)}（縮退で安全）`);
     }
 }
+/** 便の終わりの関門の種類（控えの数え方の見出し。★`finish_gate.py` の GATE_KINDS と同じ語）。 */
+export const FINISH_GATE_KINDS = ['report', 'pr', 'background'];
 /** 届けられなかった「終了の事実」の置き場。 */
 export function jobEndDir(env = process.env) {
     const base = (env.CONDUCTOR_STATE_DIR ?? '').trim();
@@ -2993,6 +3095,10 @@ export function jobEndFactsBody(f) {
         body.startedAt = f.startedAt;
     if (f.detail !== undefined && f.detail !== '')
         body.jobEndDetail = f.detail.slice(0, JOB_END_DETAIL_MAX);
+    // ★上限に達して「途中で終わった」便だけに付ける（付いていること自体が印）。
+    //   jobEndDetail の人向けの文は今までのまま残す（管制が知らないうちは黙って捨てられる）。
+    if (f.finishGate !== undefined)
+        body.finishGate = f.finishGate;
     return body;
 }
 /** 控えを手元に書く（★届くまで消さない）。書けたか返す。 */
@@ -3310,12 +3416,16 @@ export async function keepaliveTick(cfg, state, deps = {}) {
             return;
         }
         // 完了は従来の経路（画面の DONE 行）にそのまま乗せる。★報告は職人が書いたファイルから読む。
-        const report = readReport(ka.id);
+        //   ★形が整っていない報告は添えない・移さない（便 461be339）。
+        const kaDelivery = reportDelivery(ka.id);
+        const report = kaDelivery.delivered && kaDelivery.raw !== null
+            ? parseReport(kaDelivery.raw).body
+            : null;
         if (await complete(cfg, ka.id, report)) {
             if (report !== null)
                 markReportProcessed(ka.id); // ★消さずに処理済みへ（二度送らない）
             logLine(`site=${cfg.site} id=${ka.id} 見張り終了: 画面のDONEを検出し完了を報告` +
-                `${report != null ? '（報告添付）' : ''}`);
+                `${report != null ? '（報告添付）' : kaDelivery.phase === 'none' ? '' : `（${kaDelivery.note}）`}`);
             // ★終わったら必ず手を下ろす（挙げっぱなしにすると現場が長くふさがる）。
             noteBusyEdge(state, cfg.site, ka.id, false, logLine);
             state.keepalive = null;
@@ -3501,7 +3611,8 @@ export async function relayTick(cfg, state, hooks) {
                         endKind: 'not_started',
                         exitCode: null,
                         exitSignal: null,
-                        reportDelivered: reportPhase(state.pending.id) !== 'none',
+                        // ★「作業なし・本文未着」の申告は形の確かめから外してある（便 461be339）。
+                        reportDelivered: reportDelivery(state.pending.id).delivered,
                         processEndObserved: false,
                         detail: '配り役が本文未着と申告した便（職人の処理は中継が持っていない）',
                     }, { logLine: log });
@@ -3543,7 +3654,7 @@ export async function relayTick(cfg, state, hooks) {
                             endKind: 'completed',
                             exitCode: null,
                             exitSignal: null,
-                            reportDelivered: report != null || reportPhase(state.pending.id) !== 'none',
+                            reportDelivered: report != null || reportDelivery(state.pending.id).delivered,
                             processEndObserved: false,
                             detail: '配り役が完了の知らせで確かめた（職人の処理は中継が持っていない）',
                         }, { logLine: log });
@@ -3556,7 +3667,8 @@ export async function relayTick(cfg, state, hooks) {
             if (below !== null && hasStandaloneDone(below, cfg.site)) {
                 // ★報告は画面から読まない。職人が書いたファイルを読む（relayReportDirs / readReport）。
                 //   画面を読む方式は窓が 16〜34 行しかなく、9/5 以降 162 回すべて失敗していた（調査 d1d16a56）。
-                const reportThere = reportPhase(state.pending.id) !== 'none';
+                // ★「届いた」は形まで見る（便 461be339）。整っていなければ理由も一緒に届ける。
+                const doneDelivery = reportDelivery(state.pending.id);
                 const ok = await completeWithReport(cfg, state.pending.id, log);
                 if (ok) {
                     // ★終了の事実を同じ項目で届ける（便 915694ac・1-3）。画面へ貼る方式では実プロセスを
@@ -3568,9 +3680,10 @@ export async function relayTick(cfg, state, hooks) {
                         endKind: 'completed',
                         exitCode: null,
                         exitSignal: null,
-                        reportDelivered: reportThere,
+                        reportDelivered: doneDelivery.delivered,
                         processEndObserved: false,
-                        detail: '配り役が画面の完了の合図で確かめた（職人の処理は中継が持っていない）',
+                        detail: '配り役が画面の完了の合図で確かめた（職人の処理は中継が持っていない）'
+                            + (doneDelivery.note !== null ? `／${doneDelivery.note}` : ''),
                     }, { logLine: log });
                     state.pending = null;
                 }

package/package.json

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

package/sales-template/setup.sh

@@ -859,9 +859,12 @@ GUARD_FILES=(
   policy.py                   # cc2_guard / permission_request_hook が使う判定本体
   summary_ja.py               # permission_request_hook / stop_hook が使う日本語の文言
   hook_datalink.py            # 各フックが使う管制との連絡線
+  finish_gate.py              # stop_hook が使う「便の終わりの関門」の判定（便 7aa63a97）
 )
 GUARD_ENTRYPOINTS=(cc2_guard.py permission_request_hook.py stop_hook.py notification_hook.py)
-GUARD_CONFIGS=(safe_verbs.json network_allow.json)  # 任意・あれば同居させる
+# ★この一覧は scripts/update-guard.sh の INSTALL_CONFIGS と【同じ集合】にすること。
+#   report-shape.json … 報告の「形」の決まりの正本（finish_gate.py が実行時に読む・便 461be339）。
+GUARD_CONFIGS=(safe_verbs.json network_allow.json report-shape.json)  # 任意・あれば同居させる
 
 # --- ⑤-0 見本と「配る中身」がズレていないか機械照合する（★足し忘れの再発防止） ---
 # 見本が呼ぶ入口スクリプトが (a) 正本にあるか (b) 上の GUARD_FILES に入っているか を確かめる。