@tyhld/conductor 0.12.94 → 0.12.95

package/dist/relay.js

@@ -1064,6 +1064,215 @@ export function takeCodexNotice(site, env = process.env) {
     }
     return latest;
 }
+// ───────────────────────────────────────────────────────────────────────────
+// Codex の最後の返事に書いてもらう「機械で読める囲み」（便 cfa9eeec）
+//
+// 【なぜ要るか（実測 2026-10-04・codex-cli 0.160.0・便 f8ac8674 / 4c9f5049）】
+//   Codex の現場へ配る便の末尾が Claude 用のままで、Codex にはできない2つを求めていた:
+//     ① ai-kana の MCP（conductor_complete）… Codex にはつながっていない＝「利用可能なツールに無い」。
+//        毎便その断りが報告に入り、結果（outcome・PR 番号・次の独立作業候補）が管制に入らなかった。
+//     ② 作業フォルダの外（~/.local/state/conductor/reports/）への報告ファイル … 囲いが止める
+//        （Read-only file system）。毎回書こうとして失敗していた。
+//   どちらも「道具を増やす」のではなく、【いま既に通っている道】＝Codex の notify で届く
+//   「最後の返事」に機械で読める囲みを書いてもらい、配り役がそれを読んで報告ファイルを組み立てる。
+//
+// 【★受け側（管制）の口は1つも変えない】配り役が書く報告ファイルは、職人が書く物とまったく同じ形
+//   （6欄の囲み＋本文）。だから完了の道（completeWithReport）・回収の道（sendPendingReportFiles）・
+//   形の確かめ（reportDelivery）は1行も変わらない。
+//
+// 【★囲みが無い・読めないときは今までと同じ動きに倒す】最後の返事をそのまま報告として渡し、
+//   便を完了にする（実測で通っていた道を塞がない）。
+// ───────────────────────────────────────────────────────────────────────────
+/** 囲みの始まりの印（★Codex の便の配達文と同じ字面を使う）。 */
+export const CODEX_REPORT_BEGIN = '=== CONDUCTOR_REPORT ===';
+/**
+ * 囲みの終わりの印。
+ * ★`CONDUCTOR_REPORT_END` にしない理由: 画面に報告を出させていた昔の目印（`REPORT_END`）を
+ *   字面で含んでしまい、「古い指示が残っていないか」を見る検査と紛れるため（2026-09-07 に廃止した方式）。
+ */
+export const CODEX_REPORT_END = '=== END_CONDUCTOR_REPORT ===';
+/**
+ * 最後の返事から囲みの中身を切り出す（★印の行は含めない）。見つからなければ null。
+ *   - 囲みが閉じていなければ、返事の最後までを囲みの中身として読む（途中で切れた返事を救う）。
+ *   - 印の行に飾り（``` ・ ** ・ # ・ 引用の >）が付いていても読む（会話の返事は飾られやすい）。
+ * 返り: inner＝囲みの中の行／rest＝囲みを外した残り（＝報告の本文になる）。
+ */
+export function sliceCodexReportBlock(reply) {
+    const lines = String(reply ?? '').replace(/\r/g, '').split('\n');
+    const begin = lines.findIndex((l) => isCodexReportMark(l, 'CONDUCTOR_REPORT'));
+    if (begin < 0)
+        return null;
+    let end = -1;
+    for (let i = begin + 1; i < lines.length; i += 1) {
+        if (isCodexReportMark(lines[i], 'END_CONDUCTOR_REPORT')) {
+            end = i;
+            break;
+        }
+    }
+    const inner = lines.slice(begin + 1, end < 0 ? lines.length : end);
+    const rest = [...lines.slice(0, begin), ...(end < 0 ? [] : lines.slice(end + 1))]
+        .join('\n')
+        .trim();
+    return { inner, rest };
+}
+/** 印の行か（飾り・前後の `=` を落として、語そのものと突き合わせる）。 */
+function isCodexReportMark(line, word) {
+    const bare = String(line ?? '')
+        .trim()
+        .replace(/^[`*#>\s]+/, '')
+        .replace(/[`*\s]+$/, '')
+        .replace(/^=+/, '')
+        .replace(/=+$/, '')
+        .trim()
+        .toUpperCase();
+    return bare === word;
+}
+/**
+ * 囲みの中身を欄ごとに読む（★読み方は parseReport の囲みと同じ＝「key: 値」と「- 」の箇条書き）。
+ * 囲みが無ければ null。
+ */
+export function parseCodexReportBlock(reply) {
+    const block = sliceCodexReportBlock(reply);
+    if (block === null)
+        return null;
+    const fields = new Map();
+    const evidence = [];
+    let lastKey = '';
+    for (const line of block.inner) {
+        const bullet = /^\s*[-*]\s+(.*)$/.exec(line);
+        if (bullet !== null) {
+            if (lastKey === 'evidence' && bullet[1].trim() !== '')
+                evidence.push(bullet[1].trim());
+            continue;
+        }
+        const kv = /^\s*([A-Za-z_][A-Za-z0-9_]*)\s*:\s*(.*)$/.exec(line);
+        if (kv === null)
+            continue; // 読めない行は捨てる（欄の解釈を壊さない）
+        lastKey = kv[1].toLowerCase();
+        const value = kv[2].trim();
+        if (lastKey === 'evidence') {
+            if (value !== '')
+                evidence.push(value);
+            continue;
+        }
+        fields.set(lastKey, value);
+    }
+    return {
+        instructionId: nonEmpty(fields.get('instruction_id')),
+        site: nonEmpty(fields.get('site')),
+        outcome: pickOutcome(fields.get('outcome')),
+        createdAt: nonEmpty(fields.get('created_at')),
+        evidence,
+        prNumber: pickPrNumber(fields.get('pr_number')),
+        nextWork: nonEmpty(fields.get('next_work')),
+        rest: block.rest,
+    };
+}
+/**
+ * outcome の語を1つだけ取り出す（★2つ以上書かれていたら「読めない」に倒す）。
+ * 配達文の説明文（「pushed か no_change か blocked のどれか1つ」）をそのまま写した返事を
+ * pushed と読んでしまわないため。
+ */
+function pickOutcome(raw) {
+    const text = String(raw ?? '');
+    const hit = REPORT_OUTCOMES.filter((o) => new RegExp(`(^|[^a-z_])${o}([^a-z_]|$)`).test(text));
+    return hit.length === 1 ? hit[0] : null;
+}
+/** PR 番号（数字だけ）。「なし」「-」など数字が無ければ null。 */
+function pickPrNumber(raw) {
+    const m = /(\d{1,7})/.exec(String(raw ?? ''));
+    if (m === null)
+        return null;
+    const n = Number(m[1]);
+    return Number.isSafeInteger(n) && n > 0 ? n : null;
+}
+/** 囲みの instruction_id が、いま追っている便の物か（頭だけ書かれていても通す）。 */
+export function codexReportIdMatches(blockId, commandId) {
+    if (blockId === null)
+        return true; // 書かれていない＝配り役が知っている便IDで埋める
+    const got = blockId.trim();
+    if (got === commandId)
+        return true;
+    return got.length >= 8 && commandId.startsWith(got);
+}
+/**
+ * 囲みから【職人が書く物とまったく同じ形の報告】を組み立てる。読めなければ null。
+ *
+ * 【★読めたと認める条件】囲みが在る／instruction_id が食い違わない／site が食い違わない／
+ *   outcome が3つの語のどれか1つ／evidence が1件以上。
+ *   ★instruction_id・site は【配り役が知っている値】で書く（職人の写し間違いで別の便の報告にしない）。
+ * 【created_at が無いとき】配り役が受け取った時刻を入れ、入れたことを本文に1行で書く（作り話にしない）。
+ */
+export function buildCodexReportText(reply, opts) {
+    const block = parseCodexReportBlock(reply);
+    if (block === null)
+        return null;
+    if (!codexReportIdMatches(block.instructionId, opts.commandId))
+        return null;
+    if (block.site !== null && block.site !== opts.site)
+        return null;
+    if (block.outcome === null)
+        return null;
+    if (block.evidence.length === 0)
+        return null;
+    const filledAt = block.createdAt === null ? new Date(opts.now ?? Date.now()).toISOString() : null;
+    const head = [
+        '---',
+        `instruction_id: ${opts.commandId}`,
+        `site: ${opts.site}`,
+        `outcome: ${block.outcome}`,
+        `created_at: ${block.createdAt ?? filledAt}`,
+        'evidence:',
+        ...block.evidence.map((e) => `  - ${e}`),
+        '---',
+        '',
+    ];
+    const body = [
+        '（この報告は、職人の最後の返事の囲みから配り役が組み立てました）',
+        `結末（outcome）: ${block.outcome}`,
+        `PR番号: ${block.prNumber === null ? 'なし' : block.prNumber}`,
+        `次の独立作業候補: ${block.nextWork ?? '（囲みに書かれていません）'}`,
+        ...(filledAt === null ? [] : [`created_at は囲みに無かったので、配り役が受け取った時刻を入れました`]),
+        '',
+        block.rest,
+    ];
+    const text = `${head.join('\n')}${body.join('\n')}`.trim();
+    // ★形が整っていない物は書かない（書きかけを完了の材料にしない・便 461be339 と同じ判定を使う）。
+    return reportShapeMissing(text).length === 0 ? text : null;
+}
+/**
+ * Codex の最後の返事の囲みを読み、報告ファイル（職人が書く物と同じ形）を置く。
+ *   - 既に報告ファイルが在るときは【上書きしない】（職人が書けた現場を壊さない）。
+ *   - 囲みが無い・読めない・書けないときは wrote:false＝呼び出し側は今までの動きに倒す。
+ */
+export function intakeCodexReport(reply, opts) {
+    const env = opts.env;
+    if (reportPhase(opts.commandId, env) !== 'none') {
+        return { wrote: false, file: null, note: '報告ファイルが既にあるので囲みからは作りません' };
+    }
+    const text = buildCodexReportText(reply, { commandId: opts.commandId, site: opts.site, now: opts.now });
+    if (text === null) {
+        return {
+            wrote: false,
+            file: null,
+            note: '最後の返事に機械で読める囲みが無い（または読めない）ので、返事をそのまま報告として渡します',
+        };
+    }
+    const file = path.join(reportWriteDir(env), reportFileName(opts.commandId));
+    try {
+        mkdirSync(path.dirname(file), { recursive: true, mode: 0o700 });
+        writeFileSync(file, `${text}\n`, { encoding: 'utf8', mode: 0o600 });
+    }
+    catch (err) {
+        return {
+            wrote: false,
+            file: null,
+            note: '囲みは読めましたが報告ファイルを置けませんでした（返事をそのまま報告として渡します）: ' +
+                (err instanceof Error ? err.message : String(err)),
+        };
+    }
+    return { wrote: true, file, note: `最後の返事の囲みから報告ファイルを組み立てました: ${file}` };
+}
 export function sliceForDoneScan(text, anchor, wide) {
     const below = sliceBelowAnchor(text, anchor);
     if (below !== null)
@@ -1243,8 +1452,9 @@ export function buildSendText(body, site, anchor, kind = 'claude', context) {
         ? anchor.slice(SEND_ANCHOR_PREFIX.length + 1)
         : anchor;
     // ★Claude の職人には「便の終え方」の1行だけを付ける（書き方の正本は鉄則の「便の終え方」の節）。
-    //   ★Codex の職人には今までどおり全文を付ける。鉄則（~/.claude/skills）は Codex には届かないため。
-    const guide = kind === 'codex' ? fullCompletionGuide(site, commandId) : shortCompletionLine(site, commandId);
+    //   ★Codex の職人には Codex にできることだけを頼む全文を付ける（便 cfa9eeec・codexCompletionGuide）。
+    //     鉄則（~/.claude/skills）は Codex には届かないので、全文で渡すのは今までどおり。
+    const guide = kind === 'codex' ? codexCompletionGuide(site, commandId) : shortCompletionLine(site, commandId);
     const full = `${body}\n\n${dispatchContextBlock(context)}${guide}` +
         // アンカー（管理用ジョブ印）は必ず本文の最後に置く。完了検出の境界として使う。
         `（管理用ジョブ印・変更不要: ${anchor}）`;
@@ -1290,9 +1500,14 @@ export function shortCompletionLine(site, commandId) {
         `完了の知らせ: conductor_complete を commandId="${commandId}" で1回だけ（nextWork は必ず書く）。`);
 }
 /**
- * 全文の完了案内（★Codex の職人向け・Claude 向けは shortCompletionLine）。
- * ★ここに書いてある決まりは、鉄則の「便の終え方」の節と中身を同じに保つ
- *   （試験 relay-finish-guide-equivalence が1つずつ突き合わせる）。
+ * 便の終え方の【全文＝正本の物差し】。
+ *
+ * ★この全文は、もう誰にも送らない（便 cfa9eeec）。本物は次の2つ:
+ *   ・Claude の職人 … 1行（shortCompletionLine）＋鉄則の「便の終え方」の節
+ *   ・Codex の職人 … Codex にできることだけを頼む全文（codexCompletionGuide）
+ * ★残してある理由: 「寄せる前の決まりを1つも失っていない」ことを測る物差しだから
+ *   （試験 relay-finish-guide-equivalence が、この全文と「1行＋鉄則の節」を1つずつ突き合わせる）。
+ *   ここに書いてある決まりは、鉄則の「便の終え方」の節と中身を同じに保つ。
  */
 export function fullCompletionGuide(site, commandId) {
     const marker = `DONE:${site}`;
@@ -1348,6 +1563,60 @@ export function fullCompletionGuide(site, commandId) {
         // ★nextWork（次の独立作業候補）は必ず書く（conductor_complete の説明と同じ・便 a5306df2 で両方にそろえた）。
         `nextWork（次の独立作業候補）は必ず書いてください（無ければ「なし」と理由）。\n\n`);
 }
+/**
+ * Codex の職人へ付ける完了案内（便 cfa9eeec・★Claude 向けは shortCompletionLine）。
+ *
+ * 【なぜ分けたか（実測 2026-10-04・codex-cli 0.160.0・便 f8ac8674 / 4c9f5049）】
+ *   Claude 用の全文は、Codex にできない2つを求めていた＝毎便、無駄な失敗になっていた:
+ *     ① ai-kana の MCP（完了の知らせの道具）… Codex にはつながっていない＝呼べない。
+ *     ② 作業フォルダの外への報告ファイル … 囲いが止める（Read-only file system）。
+ *   どちらも頼まない。かわりに【いま既に通っている道】＝Codex の notify で届く「最後の返事」に、
+ *   機械で読める囲みを書いてもらう。配り役がそれを読んで、職人が書く物と同じ形の報告ファイルを
+ *   組み立て、いまの完了の道へそのまま流す（intakeCodexReport・受け側の口は1つも変えない）。
+ *
+ * 【★.git への書き込み（実測の違い）】便 f8ac8674 は「止められたら回避せず報告」だけを書いていたので、
+ *   Codex は承認を求めずに blocked で終えた。便 4c9f5049 で「正規の手順で承認を1回だけ求めてよい」と
+ *   書いたら、承認されて記録（コミット 04c108f）まで通った。その1〜2文をここに入れておく。
+ *
+ * 【★この文に入れない物（試験で固定）】完了の知らせの道具の名前・報告ファイルの置き場。
+ *   どちらも Codex にはできない＝書くと毎便失敗して報告が汚れる。
+ */
+export function codexCompletionGuide(site, commandId) {
+    const marker = `DONE:${site}`;
+    return (
+    // ★確認カードを非エンジニアが読める日本語にする（調査 2987cb8f）。Codex でも確認は同じ形で出る。
+    `作業中にユーザー確認（許可を求めるプロンプト）が出る操作では、その一言説明（description）を` +
+        `英語ではなく日本語で、「プルリクエストを作成する」「テストを実行する」のように` +
+        `「何をするか」が一目で分かる簡潔な形（〜を作成する／実行する／削除する 等）で書いてください` +
+        `（この説明は管制の確認カードにそのまま表示され、非エンジニアのお客さまが読んで許可を判断します）。\n\n` +
+        // ★記録（.git への書き込み）が囲いに止められたときの進み方（実測 f8ac8674 → 4c9f5049）。
+        `記録（git add・git commit）が囲い（サンドボックス）に止められたときは、正規の手順で承認を1回だけ求めてよいです。` +
+        `断られた・求める手段が無かったときは、別のやり方（設定の書き換え・囲いを外す起動・.git を別の場所に作る 等）で` +
+        `通そうとせず、そのまま報告して終えてください。\n\n` +
+        // ★報告は「最後の返事」に書いてもらう（作業フォルダの外へは書けない＝囲いが止める）。
+        `作業がすべて完了したら、報告を【この会話の最後の返事】に書いてください` +
+        `（変更ファイル・要点・チェック結果を簡潔に）。` +
+        `★報告を作業フォルダの外のファイルへ書こうとしないでください（囲いが止めます）。\n` +
+        `★最後の返事の先頭に、次の囲みをそのまま付けてください（ここが機械の読み口です。1行に1項目・左の語と「: 」は変えない）。\n` +
+        `1行目: ${CODEX_REPORT_BEGIN}\n` +
+        `2行目: instruction_id: ${commandId}\n` +
+        `3行目: site: ${site}\n` +
+        `4行目: outcome: pushed / no_change / blocked のどれか1つだけを書く` +
+        `（pushed＝変更して送った、no_change＝変更不要だった、blocked＝衝突・要判断・許可待ち等で完了できなかった。★選んだ1語だけを書く）\n` +
+        `5行目: created_at: 書いた日時（例 2026-10-04T18:00:00+09:00）\n` +
+        `6行目: evidence:（この行は「evidence:」だけにして、次の行から「- 」で始めて1行に1つ書く。` +
+        `例「- PR https://github.com/...」「- CI test pass」「- main に入った」。★1件以上必ず書く）\n` +
+        `次の行: pr_number: PR 番号（数字だけ。無ければ なし）\n` +
+        `次の行: next_work: 次の独立作業候補（無ければ「なし」と、なぜ無いかの理由）\n` +
+        `最後の行: ${CODEX_REPORT_END}\n` +
+        `この囲みの下に、いつもどおりの報告本文を書いてください（本文が空だと未完成と見なされます）。\n` +
+        `そして返事のいちばん最後の行に ${marker} とだけ出力してください。\n\n` +
+        // ★道具は呼ばせない（Codex にはつながっていない）。囲みが書けなかったときの倒れ方も書いておく。
+        `★完了の知らせと、管制へ結果（outcome・PR 番号・次の独立作業候補）を入れる作業は、配り役が行います。` +
+        `あなたは上の囲みと ${marker} を書くだけでよく、道具（MCP）を呼ぶ必要はありません。` +
+        `囲みが書けなかったときも、いつもどおり最後の返事に報告を書いて ${marker} で終えてください` +
+        `（配り役がその返事をそのまま報告として受け取ります）。\n\n`);
+}
 // ───────────────────────────────────────────────────────────────────────────
 // tmux I/O（失敗は握りつぶして null/false。送らない側に倒すため）
 // ───────────────────────────────────────────────────────────────────────────
@@ -3775,9 +4044,21 @@ export async function relayTick(cfg, state, hooks) {
             if (kindOf(cfg) === 'codex') {
                 const notice = takeCodexNotice(cfg.site);
                 if (notice !== null) {
+                    // ★最後の返事の囲みを読み、職人が書く物と同じ形の報告ファイルを組み立てる（便 cfa9eeec）。
+                    //   書けたら【いまの完了の道】へそのまま流す（受け側の口は1つも変えない）。
+                    //   囲みが無い・読めないときは、今までと同じ動き（返事をそのまま報告にして完了）に倒す。
+                    const intake = intakeCodexReport(notice.report, {
+                        commandId: state.pending.id,
+                        site: cfg.site,
+                    });
+                    log(`site=${cfg.site} id=${state.pending.id} ${intake.note}`);
                     const report = notice.report.trim().length > 0 ? notice.report : null;
-                    if (await patchStatus(cfg, state.pending.id, 'done', report)) {
-                        log(`site=${cfg.site} id=${state.pending.id} done（Codexの通知${report != null ? '・報告添付' : ''}）`);
+                    const written = intake.wrote
+                        ? await completeWithReport(cfg, state.pending.id, log)
+                        : await patchStatus(cfg, state.pending.id, 'done', report);
+                    if (written) {
+                        log(`site=${cfg.site} id=${state.pending.id} done（Codexの通知` +
+                            `${intake.wrote ? '・囲みから報告ファイル' : report != null ? '・報告添付' : ''}）`);
                         // ★終了の事実を同じ項目で届ける（便 915694ac・1-3）。この配り方では実プロセスを
                         //   中継が持っていないので、終わりは【つかめていない】と伝える（終了コードは null）。
                         //   届けるのは「中継が完了を確かめた」という事実だけ。★作り話の終了を送らない。
@@ -3790,7 +4071,8 @@ export async function relayTick(cfg, state, hooks) {
                             exitSignal: null,
                             reportDelivered: report != null || reportDelivery(state.pending.id).delivered,
                             processEndObserved: false,
-                            detail: '配り役が完了の知らせで確かめた（職人の処理は中継が持っていない）',
+                            detail: '配り役が完了の知らせで確かめた（職人の処理は中継が持っていない）' +
+                                (intake.wrote ? '／最後の返事の囲みから報告ファイルを組み立てた' : ''),
                         }, { logLine: log });
                         state.pending = null;
                     }

package/package.json

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