@tyhld/conductor 0.11.11 → 0.11.12

package/dist/relay.js

@@ -200,13 +200,6 @@ export const BUSY_MARKERS = [
  *  '·' のような普通の出力に頻出する記号も入れない。）
  */
 export const BUSY_GLYPHS = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
-/**
- * CC2 の作業報告を囲ませる行マーカー（ADR-029 報告集約・送信側）。
- * CC2 はこの2行で報告本文を挟んで出力し、relay が間の行群を抜き出して中央へ送る。
- * BUSY_MARKERS / BUSY_GLYPHS / `DONE:` のいずれとも文字列が衝突しない（アイドル判定・完了判定を汚さない）。
- */
-export const REPORT_START = 'REPORT_START';
-export const REPORT_END = 'REPORT_END';
 /** 設定から職人の種類を読む。未指定・知らない値は 'claude'（後退させない）。 */
 export function kindOf(cfg) {
     return cfg.agentKind ?? 'claude';
@@ -940,11 +933,13 @@ export function selectOrphanToRestore(orphans, currentPending) {
  * CC2 の途中送信を防ぐため、改行はすべて空白に潰して「1行=1メッセージ」にする
  * （本文内に生 Enter を含めない＝送信は末尾の Enter 1回だけにするため）。
  *
- * 完了時に CC2 へ求める出力形式:
- *   1) 報告を ${REPORT_START} と ${REPORT_END} の2行で挟んで出力（変更ファイル・要点・チェック結果を簡潔に）。
- *   2) その後の最終行に必ず DONE:{site} とだけ出力（既存の完了運用＝relay画面スクレイプの高速パスを維持）。
- *   3) さらに devlog MCP の conductor_complete を commandId=<id> で1回呼ぶ（案D・relay非依存の確実な完了通知）。
- *      relay 再起動・画面churn で (1)(2) のスクレイプ検出が落ちても、中央が (3) で確実に done を立てる二重防御。
+ * 完了時に CC2 へ求めること（★2026-09-07 に「画面から読む」をやめ、ファイル経由にした）:
+ *   1) 報告を【ファイル】に書く（置き場と名前は relayReportDirs / reportFileName が決める）。
+ *      ★画面に長い報告を出す方式は窓が 16〜34 行しかなく、9/5 以降 162 回すべて失敗していた
+ *        （調査 d1d16a56）。ファイルなら長さの制限が無く、1文字も落ちない。
+ *   2) その後の最終行に必ず DONE:{site} とだけ出力（完了の合図。★これは今までどおり画面で見る）。
+ *   3) さらに devlog MCP の conductor_complete を commandId=<id> で1回呼ぶ（relay非依存の確実な通知）。
+ *      ★(3) が先に通っていれば relay は二重に送らない（completeWithReport が中央へ確かめる）。
  *
  * anchor: この送信に固有のアンカートークン（buildAnchor(id)＝`CONDUCTOR_JOB:<id>`）。本文の「最後」に置くことで、
  *   画面では送信エコーの末尾に出る＝完了検出(hasDoneBelowAnchor)の境界になる。
@@ -972,8 +967,11 @@ export function buildSendText(body, site, anchor) {
         `英語ではなく日本語で、「プルリクエストを作成する」「テストを実行する」のように` +
         `「何をするか」が一目で分かる簡潔な形（〜を作成する／実行する／削除する 等）で書いてください` +
         `（この説明は管制の確認カードにそのまま表示され、非エンジニアのお客さまが読んで許可を判断します）。\n\n` +
-        `作業がすべて完了したら、まず報告を ${REPORT_START} と ${REPORT_END} の2行で挟んで出力してください` +
-        `（変更ファイル・要点・チェック結果を簡潔に）。` +
+        // ★報告はファイルに書いてもらう（2026-09-07）。画面に出す方式は窓が小さすぎて
+        //   長い報告の先頭が流れ、9/5 以降 162 回すべて失敗していた（調査 d1d16a56）。
+        `作業がすべて完了したら、まず報告を次のファイルに書いてください（変更ファイル・要点・` +
+        `チェック結果を簡潔に）: ${reportFileCandidates(commandId)[0]} ` +
+        `（フォルダが無ければ作ってください。長さの制限はありません。画面に出す必要はありません）。` +
         `その後の最終行に必ず ${marker} とだけ出力してください。\n\n` +
         // ★案D（relay非依存の確実な完了通知）: 完了したら必ず devlog MCP の conductor_complete を呼ぶ。
         `そして完了したら必ず、devlog の MCP ツール conductor_complete を commandId="${commandId}" で1回だけ呼んでください` +
@@ -995,53 +993,6 @@ export function buildSendText(body, site, anchor) {
     // 改行(と連続改行)を空白1個に潰し、前後を整える。
     return full.replace(/\r?\n+/g, ' ').replace(/\s+/g, ' ').trim();
 }
-/**
- * 画面テキストから CC2 の報告本文を抜き出す純粋関数（ADR-029 報告集約・送信側）。
- *
- * 方針:
- *   - 行単位で「装飾を除くと ${REPORT_START} / ${REPORT_END} だけが残る行」をマーカー行とみなす。
- *     送信本文のエコー行は1行に両マーカー＋指示文が同居するため、装飾除去後に空にならず
- *     マーカー行と判定されない（＝誤抽出しない）。
- *   - マーカーが複数あれば「最後のペア」（最後の END と、その直前の最も近い START）を採用する。
- *     CC2 の実報告は送信本文エコーより後（画面下側）に出るため、最後のペア採用で実報告を拾える。
- *   - START/END が無い・対応が壊れている・中身が空のときは null（報告なし）。done化は妨げない。
- *
- * 限界（今回スコープ外）: tmux capture-pane は画面に見えている範囲しか取れないため、
- *   報告がスクロールで流れると全文は取れない（取れた範囲を report とする）。
- */
-export function extractReport(text) {
-    const lines = text.replace(/\r/g, '').split('\n');
-    // 装飾（ボックス罫線・行頭バレット・空白）を除いた中身を返す。
-    const stripped = (line) => line.replace(/[│|>⏺●○•*·]/g, '').trim();
-    // 最後の END マーカー行を探す。
-    let endIdx = -1;
-    for (let i = lines.length - 1; i >= 0; i--) {
-        if (stripped(lines[i]) === REPORT_END) {
-            endIdx = i;
-            break;
-        }
-    }
-    if (endIdx < 0)
-        return null;
-    // その直前にある最も近い START マーカー行を探す。
-    let startIdx = -1;
-    for (let i = endIdx - 1; i >= 0; i--) {
-        if (stripped(lines[i]) === REPORT_START) {
-            startIdx = i;
-            break;
-        }
-    }
-    if (startIdx < 0)
-        return null;
-    // 間の行群を取り出し、前後の空行を trim。
-    const inner = lines.slice(startIdx + 1, endIdx);
-    while (inner.length > 0 && inner[0].trim() === '')
-        inner.shift();
-    while (inner.length > 0 && inner[inner.length - 1].trim() === '')
-        inner.pop();
-    const report = inner.join('\n').trim();
-    return report.length > 0 ? report : null;
-}
 // ───────────────────────────────────────────────────────────────────────────
 // tmux I/O（失敗は握りつぶして null/false。送らない側に倒すため）
 // ───────────────────────────────────────────────────────────────────────────
@@ -1102,6 +1053,76 @@ export function relaySendTmpDirs() {
 }
 /** tmux バッファ名の接頭辞（他人のバッファと衝突させないため現場・PID・時刻で一意にする）。 */
 export const RELAY_SEND_BUFFER_PREFIX = 'conductor-send';
+// ───────────────────────────────────────────────────────────────────────────
+// 報告の受け渡し（★ファイル経由・えふさん決定 2026-09-07）
+// ───────────────────────────────────────────────────────────────────────────
+/**
+ * 職人が報告を書く場所（1指示につき1ファイル）。★relay はここだけを見る。
+ *
+ * 【なぜ画面から読むのをやめたか（調査 d1d16a56 で確定した事実）】
+ *   職人は全画面表示で動くので tmux から見える行は 16〜34 行しかない（実測: 代替画面=1・
+ *   履歴の行数 0〜222 行／上限 2000 行）。報告が数十行あると、終わりの目印と DONE 行が
+ *   下に残った時点で【始まりの目印はもう窓の外】になり、対が見つからず報告が null になる。
+ *   ★9/5 以降の実測: 報告を添えた完了 0 件 ／ 報告なしの完了 162 件。
+ *   ＝画面から報告を読む保険は一度も動いたことがなかった。窓の大きさの問題なので、
+ *   目印の書き方をどう工夫しても直らない。だから【画面を読むのをやめて】ファイルにする。
+ *
+ * 置き場（前から順に試す。理由は writeSendTmpFile と同じ）:
+ *   第1候補 /tmp/claude-<uid>/conductor-report … 職人の使い捨て置き場と同じ根（鉄則13）
+ *   第2候補 os.tmpdir()/conductor-report-<uid> … /tmp が使えない機体の保険
+ * 環境変数 CONDUCTOR_REPORT_DIR で1つに固定できる（テスト・特殊な機体向け）。
+ */
+export function relayReportDirs(env = process.env) {
+    const explicit = (env.CONDUCTOR_REPORT_DIR ?? '').trim();
+    if (explicit)
+        return [explicit];
+    const uid = typeof process.getuid === 'function' ? process.getuid() : 0;
+    return [
+        path.join('/tmp', `claude-${uid}`, 'conductor-report'),
+        path.join(os.tmpdir(), `conductor-report-${uid}`),
+    ];
+}
+/**
+ * その指示の報告ファイル名。★指示ID で引ける形にする（どの便の報告かを取り違えない）。
+ * 指示ID に紛れ込みうる区切り文字は落とす（別の場所を指す名前を作らせない）。
+ */
+export function reportFileName(commandId) {
+    const safe = String(commandId ?? '').replace(/[^A-Za-z0-9_-]/g, '').slice(0, 128);
+    return `${safe || 'unknown'}.md`;
+}
+/** 報告ファイルの置き場の候補（前から順に探す）。 */
+function reportFileCandidates(commandId, env) {
+    return relayReportDirs(env).map((dir) => path.join(dir, reportFileName(commandId)));
+}
+/**
+ * その指示の報告を読む（消さない）。無ければ null。
+ * ★読むだけで消さないのは、管制へ書けたことを確かめてから消すため
+ *   （書けなかったのに消すと報告が永久に失われる）。
+ */
+export function readReport(commandId, env) {
+    for (const file of reportFileCandidates(commandId, env)) {
+        try {
+            const text = readFileSync(file, 'utf8').trim();
+            if (text.length > 0)
+                return text;
+        }
+        catch {
+            // 無い・読めない → 次の候補へ（報告が無いこと自体は異常ではない）
+        }
+    }
+    return null;
+}
+/** その指示の報告ファイルを片付ける（★同じ報告を二度送らないため）。 */
+export function discardReport(commandId, env) {
+    for (const file of reportFileCandidates(commandId, env)) {
+        try {
+            unlinkSync(file);
+        }
+        catch {
+            // 無ければ何もしない
+        }
+    }
+}
 /** 本文を書いた一時ファイルを作ってパスを返す。どこにも書けなければ例外。 */
 function writeSendTmpFile(text) {
     let lastErr = new Error('一時ファイルの置き場が1つも作れませんでした');
@@ -1341,6 +1362,39 @@ export async function patchStatus(cfg, id, status, report) {
         return false;
     }
 }
+/**
+ * 完了を1回で送る（★報告と完了を同じ1回に載せる・えふさん決定 2026-09-07）。
+ *
+ * 【なぜ1回にするか】以前は「報告なしの完了」を先に送り、報告はあとから追いかけていた。
+ *   そのため管制の開発履歴が【本文なしの記録】と【あとから来た本文】の2回に分かれ、
+ *   本文が来ないまま終わると「完了したが報告なし」だけが残った。報告はファイルにあるので、
+ *   完了を送るその1回に必ず載せられる。
+ *
+ * 【★二重に送らない】職人が conductor_complete で先に完了させていたら、relay は何も送らない。
+ *   中央へ1回だけ状態を尋ねて確かめる。尋ねられない（不通）ときは送る側に倒す
+ *   ＝完了が落ちるより、同じ報告が2度書かれるほうが害が小さい（中央は上書き保存する）。
+ *
+ * 【後片付け】管制へ書けたときだけ報告ファイルを消す。書けなければ残す＝次tickでやり直せる。
+ * 返り: 追跡を畳んでよいか（true = 完了が確定した）。
+ */
+export async function completeWithReport(cfg, id, logLine = log) {
+    const already = await fetchCommandStatus(cfg, id);
+    if (isTerminalCommandStatus(already)) {
+        // 職人が自分で完了させていた。★relay は二重に送らない（報告も上書きしない）。
+        discardReport(id);
+        logLine(`site=${cfg.site} id=${id} は職人が自分で完了させていた(status=${already})` +
+            '→relayからは送りません（報告の置き場だけ片付けました）');
+        return true;
+    }
+    const report = readReport(id);
+    if (!(await patchStatus(cfg, id, 'done', report)))
+        return false;
+    if (report !== null)
+        discardReport(id); // ★書けたら片付ける（同じ報告を二度送らない）
+    logLine(`site=${cfg.site} id=${id} done` +
+        (report !== null ? `（報告添付・${report.length}文字）` : '（報告ファイルなし＝完了のみ）'));
+    return true;
+}
 /**
  * 指示 {id} 単体の現在 status を中央へ問う（外部完了検出の根治プローブ）。
  *   GET /api/conductor/commands/{id}（テナントトークン Bearer）。応答 JSON から status 文字列を拾う。
@@ -1650,9 +1704,11 @@ export async function keepaliveTick(cfg, state, deps = {}) {
         return;
     }
     if (action === 'done') {
-        // 完了は従来の経路（画面の DONE 行＋報告本文）にそのまま乗せる。報告も落とさない。
-        const report = below === null ? null : extractReport(below);
+        // 完了は従来の経路（画面の DONE 行）にそのまま乗せる。★報告は職人が書いたファイルから読む。
+        const report = readReport(ka.id);
         if (await complete(cfg, ka.id, report)) {
+            if (report !== null)
+                discardReport(ka.id); // ★書けたら片付ける（二度送らない）
             logLine(`site=${cfg.site} id=${ka.id} 見張り終了: 画面のDONEを検出し完了を報告` +
                 `${report != null ? '（報告添付）' : ''}`);
             // ★終わったら必ず手を下ろす（挙げっぱなしにすると現場が長くふさがる）。
@@ -1828,11 +1884,10 @@ export async function relayTick(cfg, state, hooks) {
             }
             const below = sliceForDoneScan(cap, state.pending.anchor, state.pending.screenDoneWide);
             if (below !== null && hasStandaloneDone(below, cfg.site)) {
-                // 報告本文も「アンカーより下」から抜き出す（今回ジョブの報告に限定。過去報告の誤添付を防ぐ）。
-                const report = extractReport(below);
-                const ok = await patchStatus(cfg, state.pending.id, 'done', report);
+                // ★報告は画面から読まない。職人が書いたファイルを読む（relayReportDirs / readReport）。
+                //   画面を読む方式は窓が 16〜34 行しかなく、9/5 以降 162 回すべて失敗していた（調査 d1d16a56）。
+                const ok = await completeWithReport(cfg, state.pending.id, log);
                 if (ok) {
-                    log(`site=${cfg.site} id=${state.pending.id} done${report != null ? '（報告添付）' : ''}`);
                     state.pending = null;
                 }
                 return; // sent 中は新規取得しない

package/package.json

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