@tyhld/conductor 0.12.37 → 0.12.38

package/dist/cli.js

@@ -13,7 +13,7 @@ import { execFileSync, spawnSync } from 'node:child_process';
 import { existsSync, readFileSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
 import os from 'node:os';
-import { RELAY_FETCH_TIMEOUT_MS, createRelayState, deliveryReport, parseAgentKind, reconnectDelayMs, relayTick, tmuxHasSession, } from './relay.js';
+import { RELAY_FETCH_TIMEOUT_MS, createRelayState, deliveryReport, parseAgentKind, reconnectDelayMs, relayTick, tmuxHasSession, pendingReportIds, sweepProcessedReports, } from './relay.js';
 import { createRelayRunner } from './relay-runner.js';
 import { installProcessGuards } from './process-guard.js';
 import { registerNudgeTarget, unregisterNudgeTarget, watchNudges, } from './nudge.js';
@@ -176,6 +176,14 @@ async function sendReport(cfg) {
         if (d.queuedCommandId !== null)
             body.queuedCommandId = d.queuedCommandId;
     }
+    // ★未処理の報告（まだ管制へ渡せていない控え）の指示IDを載せる（便 57c04e6d）。
+    //   1件も無ければ載せない＝今までと同じ本文。多すぎるときは上限までにして総数を添える。
+    const pending = pendingReportIds();
+    if (pending.ids.length > 0) {
+        body.pendingReports = pending.ids;
+        if (pending.total > pending.ids.length)
+            body.pendingReportsTotal = pending.total;
+    }
     const time = new Date().toLocaleTimeString();
     try {
         const res = await fetch(`${cfg.url}/api/conductor/report`, {
@@ -216,6 +224,23 @@ async function start(args) {
     const timer = setInterval(() => {
         void sendReport(cfg);
     }, cfg.intervalSec * 1000);
+    // ★古い控えの片付け（便 57c04e6d）。1時間に1回だけ、処理済みの控えのうち30日より古い物を消す。
+    //   ★未処理の控えは日数では消さない（渡し切れていない報告を失わないため）。
+    //   起動直後にも1回まわす＝入れ直した直後から効く。
+    const sweep = () => {
+        try {
+            const removed = sweepProcessedReports();
+            if (removed > 0)
+                console.log(`[conductor] 古い報告の控えを${removed}件片付けました（処理済み・30日超）`);
+        }
+        catch (err) {
+            // 片付けは最善努力。失敗しても生存報告は止めない。
+            console.log(`[conductor] 報告の控えの片付けに失敗: ${err instanceof Error ? err.message : String(err)}`);
+        }
+    };
+    sweep();
+    const sweepTimer = setInterval(sweep, 60 * 60 * 1000);
+    sweepTimer.unref?.();
     // ADR-029 relay。CONDUCTOR_RELAY_ENABLED=true のときだけ別インターバルで回す。
     // heartbeat とは独立。off の間は何も起きない（既存挙動を壊さない）。
     //

package/dist/relay.js

@@ -23,7 +23,7 @@
  *     メニューへ指示を流し込まないための落とし穴ガード専用。中身は読まない。
  */
 import { execFileSync } from 'node:child_process';
-import { mkdirSync, readFileSync, unlinkSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, unlinkSync, utimesSync, writeFileSync, } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 // ───────────────────────────────────────────────────────────────────────────
@@ -1010,9 +1010,22 @@ export function buildSendText(body, site, anchor) {
         `（この説明は管制の確認カードにそのまま表示され、非エンジニアのお客さまが読んで許可を判断します）。\n\n` +
         // ★報告はファイルに書いてもらう（2026-09-07）。画面に出す方式は窓が小さすぎて
         //   長い報告の先頭が流れ、9/5 以降 162 回すべて失敗していた（調査 d1d16a56）。
+        // ★2026-09-21: 頭に6つの欄（instruction_id / site / outcome / created_at / evidence / body）を
+        //   必ず書いてもらう。管制が機械で読めるようにするため（えふさん確定・便 57c04e6d）。
+        //   本文は欄の下にそのまま書けるので、長さも書き方も今までと変わらない。
         `作業がすべて完了したら、まず報告を次のファイルに書いてください（変更ファイル・要点・` +
-        `チェック結果を簡潔に）: ${reportFileCandidates(commandId)[0]} ` +
-        `（フォルダが無ければ作ってください。長さの制限はありません。画面に出す必要はありません）。` +
+        `チェック結果を簡潔に）: ${path.join(reportWriteDir(), reportFileName(commandId))} ` +
+        `（フォルダが無ければ作ってください。長さの制限はありません。画面に出す必要はありません）。\n` +
+        `★ファイルの先頭に、次の囲みをそのまま付けてください（1行目から。ここが機械の読み口です）。\n` +
+        `---\n` +
+        `instruction_id: ${commandId}\n` +
+        `site: ${site}\n` +
+        `outcome: pushed または no_change または blocked のどれか1つ\n` +
+        `created_at: 書いた日時（例 2026-09-21T04:20:00+09:00）\n` +
+        `evidence:\n` +
+        `  - 受入の証拠を1行に1つ（例 PR https://github.com/... / CI test pass / main に入った など）\n` +
+        `---\n` +
+        `この囲みの下に、いつもどおりの報告本文を書いてください（本文が空だと未完成と見なされます）。\n` +
         `その後の最終行に必ず ${marker} とだけ出力してください。\n\n` +
         // ★案D（relay非依存の確実な完了通知）: 完了したら必ず devlog MCP の conductor_complete を呼ぶ。
         `そして完了したら必ず、devlog の MCP ツール conductor_complete を commandId="${commandId}" で1回だけ呼んでください` +
@@ -1095,34 +1108,67 @@ export function relaySendTmpDirs() {
 /** tmux バッファ名の接頭辞（他人のバッファと衝突させないため現場・PID・時刻で一意にする）。 */
 export const RELAY_SEND_BUFFER_PREFIX = 'conductor-send';
 // ───────────────────────────────────────────────────────────────────────────
-// 報告の受け渡し（★ファイル経由・えふさん決定 2026-09-07）
+// 報告の受け渡し（★ファイル経由・えふさん決定 2026-09-07／置き場と作法は 2026-09-21 に改訂）
 // ───────────────────────────────────────────────────────────────────────────
+//
+// 【なぜ画面から読むのをやめたか（調査 d1d16a56 で確定した事実）】
+//   職人は全画面表示で動くので tmux から見える行は 16〜34 行しかない（実測: 代替画面=1・
+//   履歴の行数 0〜222 行／上限 2000 行）。報告が数十行あると、終わりの目印と DONE 行が
+//   下に残った時点で【始まりの目印はもう窓の外】になり、対が見つからず報告が null になる。
+//   ★9/5 以降の実測: 報告を添えた完了 0 件 ／ 報告なしの完了 162 件。
+//   ＝画面から報告を読む保険は一度も動いたことがなかった。だから【画面を読むのをやめて】ファイルにする。
+//
+// 【2026-09-21 に何を変えたか（えふさん確定・便 57c04e6d）】
+//   ・置き場を `~/.local/state/conductor/reports` にした（`/tmp` は再起動で消えるため）。
+//     旧 `/tmp/claude-<uid>/conductor-report` は【読む先としてだけ】残す＝既存の控えも拾える。
+//   ・管制へ渡した後も【消さない】。`processed/` へ移して残す（markReportProcessed）。
+//   ・職人が MCP で自分で完了させた場合も、同じ控えを残す（以前はここで捨てていた）。
+//   ・未処理の指示IDは生存報告に載せて管制へ知らせる（pendingReportIds）。
+//   実体はこのファイルの下（★relay.ts は試験から直に読まれるので、部品を別ファイルに分けない）。
+/** 処理済みの控えを置く下位フォルダの名前。 */
+export const PROCESSED_DIR = 'processed';
+/** 処理済みの控えを残す期間（既定30日）。★未処理はこの期間では消さない。 */
+export const REPORT_KEEP_PROCESSED_MS = 30 * 24 * 60 * 60 * 1000;
+/**
+ * 生存報告に載せる未処理の指示IDの上限（★管制側 ai-kana 便と同じ値にすること）。
+ * 超えた分は載せず、総数だけ `pendingReportsTotal` で伝える（本文が太らないように）。
+ */
+export const PENDING_REPORTS_MAX = 50;
+/** 報告の結末。conductor_complete の outcome と同じ語にそろえる（人も機械も同じ言い方にする）。 */
+export const REPORT_OUTCOMES = ['pushed', 'no_change', 'blocked'];
+/** 必須の欄（1つでも欠けたら「機械可読ではない」と判定する）。 */
+export const REQUIRED_REPORT_FIELDS = [
+    'instruction_id',
+    'site',
+    'outcome',
+    'created_at',
+    'evidence',
+    'body',
+];
 /**
- * 職人が報告を書く場所（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つに固定できる（テスト・特殊な機体向け）。
+ * 報告の置き場（前から順に試す）。
+ *   ① `~/.local/state/conductor/reports` … ★正本。再起動で消えない（XDG_STATE_HOME があればそちら）。
+ *   ② `/tmp/claude-<uid>/conductor-report` … 旧・読む先としてだけ残す（既存の控えを拾うため）。
+ *   ③ `os.tmpdir()/conductor-report-<uid>` … `/tmp` が使えない機体の保険（旧）。
+ * 環境変数 `CONDUCTOR_REPORT_DIR` を指定したときは、その1つだけ（試験・特殊な機体向け）。
  */
-export function relayReportDirs(env = process.env) {
+export function reportStoreDirs(env = process.env) {
     const explicit = (env.CONDUCTOR_REPORT_DIR ?? '').trim();
     if (explicit)
         return [explicit];
     const uid = typeof process.getuid === 'function' ? process.getuid() : 0;
+    const home = env.HOME ?? os.homedir();
+    const state = (env.XDG_STATE_HOME ?? '').trim() || path.join(home, '.local', 'state');
     return [
+        path.join(state, 'conductor', 'reports'),
         path.join('/tmp', `claude-${uid}`, 'conductor-report'),
         path.join(os.tmpdir(), `conductor-report-${uid}`),
     ];
 }
+/** 職人に書いてもらう置き場（★正本の1つだけ）。指示文はここを指す。 */
+export function reportWriteDir(env = process.env) {
+    return reportStoreDirs(env)[0];
+}
 /**
  * その指示の報告ファイル名。★指示ID で引ける形にする（どの便の報告かを取り違えない）。
  * 指示ID に紛れ込みうる区切り文字は落とす（別の場所を指す名前を作らせない）。
@@ -1131,39 +1177,256 @@ 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)));
+/** 未処理の置き場の候補（前から順に探す）。 */
+function pendingCandidates(commandId, env) {
+    return reportStoreDirs(env).map((dir) => path.join(dir, reportFileName(commandId)));
+}
+/** 処理済みの置き場の候補（前から順に探す）。 */
+function processedCandidates(commandId, env) {
+    return reportStoreDirs(env).map((dir) => path.join(dir, PROCESSED_DIR, reportFileName(commandId)));
+}
+/** 中身のあるファイルを読む。無い・空なら null。 */
+function readIfFilled(file) {
+    try {
+        const text = readFileSync(file, 'utf8').trim();
+        return text.length > 0 ? text : null;
+    }
+    catch {
+        return null;
+    }
 }
 /**
- * その指示の報告を読む（消さない）。無ければ null。
- * ★読むだけで消さないのは、管制へ書けたことを確かめてから消すため
- *   （書けなかったのに消すと報告が永久に失われる）。
+ * その指示の報告を読む（★消さない）。未処理が無ければ処理済みの控えも見る。
+ * 返すのは【管制へ渡す本文】＝front matter を外した本体（古い形は全文）。
  */
 export function readReport(commandId, env) {
-    for (const file of reportFileCandidates(commandId, env)) {
+    const parsed = readParsedReport(commandId, env);
+    return parsed === null ? null : parsed.body;
+}
+/** その指示の報告を、欄ごとに解析して読む（未処理→処理済みの順に探す）。 */
+export function readParsedReport(commandId, env) {
+    for (const file of [...pendingCandidates(commandId, env), ...processedCandidates(commandId, env)]) {
+        const text = readIfFilled(file);
+        if (text !== null)
+            return parseReport(text);
+    }
+    return null;
+}
+/** その指示の報告が、いまどの段にあるか。 */
+export function reportPhase(commandId, env) {
+    for (const file of pendingCandidates(commandId, env)) {
+        if (readIfFilled(file) !== null)
+            return 'pending';
+    }
+    for (const file of processedCandidates(commandId, env)) {
+        if (readIfFilled(file) !== null)
+            return 'processed';
+    }
+    return 'none';
+}
+/**
+ * 管制へ渡し終えた報告を【処理済み】にする（★消さない・削除しない）。
+ *   同じ置き場の `processed/` へ移すだけ。移した時刻を更新時刻に刻む（30日の数え始め）。
+ * 返り: 処理済みにできたら true（元から無い・既に処理済みなら false）。
+ */
+export function markReportProcessed(commandId, env, now = Date.now()) {
+    let moved = false;
+    for (const file of pendingCandidates(commandId, env)) {
+        if (!existsSync(file))
+            continue;
+        const dest = path.join(path.dirname(file), PROCESSED_DIR, path.basename(file));
         try {
-            const text = readFileSync(file, 'utf8').trim();
-            if (text.length > 0)
-                return text;
+            mkdirSync(path.dirname(dest), { recursive: true, mode: 0o700 });
+            // ★同じ置き場の中で動かす＝別のディスクをまたがないので、途中で半分になることがない。
+            renameSync(file, dest);
+            // 30日の数え始めを「処理済みにした時刻」にそろえる（書いた時刻ではない）。
+            const sec = now / 1000;
+            utimesSync(dest, sec, sec);
+            moved = true;
         }
         catch {
-            // 無い・読めない → 次の候補へ（報告が無いこと自体は異常ではない）
+            // 移せなかったら触らない（次の周でやり直せる。★消すよりは残す方へ倒す）。
         }
     }
-    return null;
+    return moved;
 }
-/** その指示の報告ファイルを片付ける（★同じ報告を二度送らないため）。 */
-export function discardReport(commandId, env) {
-    for (const file of reportFileCandidates(commandId, env)) {
+/**
+ * まだ管制へ渡していない報告の指示ID（古い順）。★生存報告に載せて管制へ知らせるための一覧。
+ *   - `processed/` の中は数えない（もう渡した物）。
+ *   - 同じ指示IDが複数の置き場に在っても1件に畳む。
+ *   - 上限を超えた分は `ids` に載せず、`total` だけで伝える。
+ */
+export function pendingReportIds(env, limit = PENDING_REPORTS_MAX) {
+    const seen = new Map(); // 指示ID → 書かれた時刻(ms)
+    for (const dir of reportStoreDirs(env)) {
+        let names;
         try {
-            unlinkSync(file);
+            names = readdirSync(dir);
         }
         catch {
-            // 無ければ何もしない
+            continue; // 置き場がまだ無い＝未処理も無い
+        }
+        for (const name of names) {
+            if (!name.endsWith('.md'))
+                continue;
+            const file = path.join(dir, name);
+            let mtime;
+            try {
+                const st = statSync(file);
+                if (!st.isFile() || st.size === 0)
+                    continue;
+                mtime = st.mtimeMs;
+            }
+            catch {
+                continue;
+            }
+            const id = name.slice(0, -3);
+            const prev = seen.get(id);
+            if (prev === undefined || mtime < prev)
+                seen.set(id, mtime);
         }
     }
+    const sorted = [...seen.entries()].sort((a, b) => a[1] - b[1]).map(([id]) => id);
+    const max = Math.max(0, Math.floor(limit));
+    return { ids: sorted.slice(0, max), total: sorted.length };
 }
+/**
+ * 古い控えを片付ける。★消すのは `processed/` の中だけ（未処理は何日たっても消さない）。
+ * 返り: 消した件数。
+ */
+export function sweepProcessedReports(env, now = Date.now(), maxAgeMs = REPORT_KEEP_PROCESSED_MS) {
+    let removed = 0;
+    for (const dir of reportStoreDirs(env)) {
+        const done = path.join(dir, PROCESSED_DIR);
+        let names;
+        try {
+            names = readdirSync(done);
+        }
+        catch {
+            continue;
+        }
+        for (const name of names) {
+            if (!name.endsWith('.md'))
+                continue;
+            const file = path.join(done, name);
+            try {
+                const st = statSync(file);
+                if (!st.isFile())
+                    continue;
+                if (now - st.mtimeMs <= maxAgeMs)
+                    continue;
+                unlinkSync(file);
+                removed += 1;
+            }
+            catch {
+                // 読めない・消せないなら触らない
+            }
+        }
+    }
+    return removed;
+}
+/**
+ * 報告の文字列を欄ごとに解く。
+ *
+ * 【なぜ front matter（`---` の囲み）にしたか＝機械が確実に読める形の選び方】
+ *   ・職人は報告を【ヒアドキュメントで一気に書く】。JSON だと本文の改行・引用符・日本語の記号で
+ *     壊れやすく、壊れた瞬間に報告そのものが読めなくなる（本文は数千〜2万字ある）。
+ *   ・front matter なら「頭の数行だけが欄・残りは全部本文」なので、本文に何が書かれていても
+ *     欄の解釈は壊れない。欄が無い古い報告も【本文だけ在る】として同じ関数で読める。
+ *   ・外部の部品（YAML 読み）を足さずに読める＝配布物が太らない。
+ */
+export function parseReport(text) {
+    const raw = String(text ?? '');
+    const lines = raw.replace(/^/, '').split('\n');
+    let i = 0;
+    while (i < lines.length && lines[i].trim() === '')
+        i += 1;
+    const empty = {
+        instructionId: null,
+        site: null,
+        outcome: null,
+        createdAt: null,
+        evidence: [],
+        body: raw.trim(),
+        missing: [],
+        hasHeader: false,
+    };
+    if (i >= lines.length || lines[i].trim() !== '---') {
+        return { ...empty, missing: missingOf(empty) };
+    }
+    const fields = new Map();
+    const evidence = [];
+    let lastKey = '';
+    let closed = false;
+    let j = i + 1;
+    for (; j < lines.length; j += 1) {
+        const line = lines[j];
+        if (line.trim() === '---') {
+            closed = true;
+            break;
+        }
+        const bullet = /^\s*-\s+(.*)$/.exec(line);
+        if (bullet !== null) {
+            // 箇条書きは直前の欄の続き。いまは evidence だけが複数行を持つ。
+            if (lastKey === 'evidence' && bullet[1].trim() !== '')
+                evidence.push(bullet[1].trim());
+            continue;
+        }
+        const kv = /^([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);
+    }
+    if (!closed) {
+        // 閉じていない＝front matter として読まない（全文を本文として扱う）。
+        return { ...empty, missing: missingOf(empty) };
+    }
+    const outcomeRaw = (fields.get('outcome') ?? '').trim();
+    const outcome = REPORT_OUTCOMES.includes(outcomeRaw)
+        ? outcomeRaw
+        : null;
+    const parsed = {
+        instructionId: nonEmpty(fields.get('instruction_id')),
+        site: nonEmpty(fields.get('site')),
+        outcome,
+        createdAt: nonEmpty(fields.get('created_at')),
+        evidence,
+        body: lines.slice(j + 1).join('\n').trim(),
+        missing: [],
+        hasHeader: true,
+    };
+    return { ...parsed, missing: missingOf(parsed) };
+}
+function nonEmpty(v) {
+    const s = (v ?? '').trim();
+    return s === '' ? null : s;
+}
+/** 欠けている必須の欄を並べる（空なら新しい形＝機械可読）。 */
+function missingOf(r) {
+    const missing = [];
+    if (r.instructionId === null)
+        missing.push('instruction_id');
+    if (r.site === null)
+        missing.push('site');
+    if (r.outcome === null)
+        missing.push('outcome');
+    if (r.createdAt === null)
+        missing.push('created_at');
+    if (r.evidence.length === 0)
+        missing.push('evidence');
+    if (r.body.trim() === '')
+        missing.push('body');
+    return missing;
+}
+/** 報告の置き場（前から順に探す）。★名前は従来のまま（中身は reportStoreDirs）。 */
+export const relayReportDirs = reportStoreDirs;
 /** 本文を書いた一時ファイルを作ってパスを返す。どこにも書けなければ例外。 */
 function writeSendTmpFile(text) {
     let lastErr = new Error('一時ファイルの置き場が1つも作れませんでした');
@@ -1415,23 +1678,33 @@ export async function patchStatus(cfg, id, status, report) {
  *   中央へ1回だけ状態を尋ねて確かめる。尋ねられない（不通）ときは送る側に倒す
  *   ＝完了が落ちるより、同じ報告が2度書かれるほうが害が小さい（中央は上書き保存する）。
  *
- * 【後片付け】管制へ書けたときだけ報告ファイルを消す。書けなければ残す＝次tickでやり直せる。
+ * 【後片付け】管制へ書けたときだけ報告を【処理済み】にする（★消さない・2026-09-21）。
+ *   書けなければ未処理のまま残す＝次tickでやり直せるし、管制も生存報告で気づける。
+ *
+ * 【★冪等（二重完了にしない）】報告が既に処理済みなら、この便はもう渡し終えている。
+ *   中央へ問い合わせる前にそこで打ち切る＝MCP と報告ファイルの二経路から同じ便が来ても1回だけ扱う。
  * 返り: 追跡を畳んでよいか（true = 完了が確定した）。
  */
 export async function completeWithReport(cfg, id, logLine = log) {
+    // ★冪等の要: 処理済みの控えが在る＝この便はもう渡してある。何もしない（二重完了にしない）。
+    if (reportPhase(id) === 'processed') {
+        logLine(`site=${cfg.site} id=${id} の報告は処理済みです→送りません（二重完了にしない）`);
+        return true;
+    }
     const already = await fetchCommandStatus(cfg, id);
     if (isTerminalCommandStatus(already)) {
         // 職人が自分で完了させていた。★relay は二重に送らない（報告も上書きしない）。
-        discardReport(id);
+        //   ★控えは消さずに処理済みとして残す（MCP 経由でも同じ控えが残る・えふさん確定 2026-09-21）。
+        markReportProcessed(id);
         logLine(`site=${cfg.site} id=${id} は職人が自分で完了させていた(status=${already})` +
-            '→relayからは送りません（報告の置き場だけ片付けました）');
+            '→relayからは送りません（報告は控えとして処理済みにしました）');
         return true;
     }
     const report = readReport(id);
     if (!(await patchStatus(cfg, id, 'done', report)))
         return false;
     if (report !== null)
-        discardReport(id); // ★書けたら片付ける（同じ報告を二度送らない）
+        markReportProcessed(id); // ★消さずに処理済みへ（同じ報告を二度送らない）
     logLine(`site=${cfg.site} id=${id} done` +
         (report !== null ? `（報告添付・${report.length}文字）` : '（報告ファイルなし＝完了のみ）'));
     return true;
@@ -1745,11 +2018,20 @@ export async function keepaliveTick(cfg, state, deps = {}) {
         return;
     }
     if (action === 'done') {
+        // ★冪等（2026-09-21）: 報告が既に処理済み＝この便はもう渡してある。送らずに畳むだけ。
+        //   実測 2026-09-20: 職人が MCP で完了した便に、見張りが画面の DONE でもう一度完了を送っていた。
+        if (reportPhase(ka.id) === 'processed') {
+            logLine(`site=${cfg.site} id=${ka.id} 見張り終了: 報告は処理済みでした` +
+                '→送らずに畳みます（二重完了にしない）');
+            noteBusyEdge(state, cfg.site, ka.id, false, logLine);
+            state.keepalive = null;
+            return;
+        }
         // 完了は従来の経路（画面の DONE 行）にそのまま乗せる。★報告は職人が書いたファイルから読む。
         const report = readReport(ka.id);
         if (await complete(cfg, ka.id, report)) {
             if (report !== null)
-                discardReport(ka.id); // ★書けたら片付ける（二度送らない）
+                markReportProcessed(ka.id); // ★消さずに処理済みへ（二度送らない）
             logLine(`site=${cfg.site} id=${ka.id} 見張り終了: 画面のDONEを検出し完了を報告` +
                 `${report != null ? '（報告添付）' : ''}`);
             // ★終わったら必ず手を下ろす（挙げっぱなしにすると現場が長くふさがる）。

package/package.json

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