@tyhld/conductor 0.12.74 → 0.12.77

package/dist/headless.js

@@ -16,7 +16,7 @@
  */
 import { execFileSync, spawn } from 'node:child_process';
 import { createInterface } from 'node:readline';
-import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, unlinkSync, writeFileSync } from 'node:fs';
+import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, 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, deliveryReport, fetchCommand, fetchCommandStatus, fetchControl, fetchOrphans, isTerminalCommandStatus, kindOf, markDelivered, noteCentralDown, noteCentralUp, noteReadiness, patchStatus, reconnectDelayMs, selectOrphanToRestore, sendHeartbeat, } from './relay.js';
@@ -546,6 +546,107 @@ export function isAuthError(job) {
 export function eventsLogPath(commandId, attempt) {
     return path.join(RUNS_DIR, commandId, `attempt-${attempt}.events.jsonl`);
 }
+// ─────────────────────────────────────────────────────────────────────────────
+// 実行ログの後片付け（便ごとに増え続けるので、古い便の分を消す）
+//
+// 【なぜ要るか】1便ごとに runs/<便ID>/ が1つ増え、中に生の記録（attempt-N.jsonl・
+//   実物は1行が数千文字）と進捗（attempt-N.events.jsonl）と stderr が入る。
+//   消す仕組みが無いと、動いているPCで際限なく増える（前便 fbaba4b1 の残件）。
+// ─────────────────────────────────────────────────────────────────────────────
+/**
+ * 実行ログを残す期間(ms)。これより古い便のフォルダは消す。
+ * ★線引きはこの2つの定数だけ（1か所）。
+ *
+ * 【なぜ14日か】この記録を読むのは「あの便は何をしたのか」を後から確かめるとき。
+ *   受入判定（CC1）は当日〜数日、えふさんの確認と作り直しを挟んでも2週あれば足りる。
+ *   それを過ぎた便は報告（reports/<指示ID>.md）と devlog に残っているので、
+ *   生の記録そのものは要らなくなる。
+ */
+export const RUNS_RETENTION_MS = 14 * 24 * 60 * 60_000;
+/**
+ * 残す便のフォルダの最大数（新しい順）。期間内でも、これを超えた分は古い方から消す。
+ *
+ * 【なぜ50件か】1便の生の記録は実測で数MB〜十数MB。50件なら最悪でも数百MBに収まり、
+ *   かつ「直近の数日ぶんの便」は必ず残る（1日に50便を超える現場は今のところ無い）。
+ *   ★期間だけだと短期間に大量の便が来たPCで膨らみ、上限だけだと静かなPCで古い物が
+ *   いつまでも残る。両方を掛けて初めて上限が決まる。
+ */
+export const RUNS_RETENTION_MAX_DIRS = 50;
+/** 便IDらしい名前だけを消す対象にする（★runs/ の中の見知らぬ物を消さない）。 */
+const RUN_DIR_NAME_RE = /^[A-Za-z0-9][A-Za-z0-9_-]*$/;
+/** そのフォルダの中で、いちばん新しい更新時刻(ms)を返す。読めなければ 0。 */
+function newestMtimeMs(dir) {
+    let newest = 0;
+    try {
+        newest = statSync(dir).mtimeMs;
+        for (const name of readdirSync(dir)) {
+            try {
+                newest = Math.max(newest, statSync(path.join(dir, name)).mtimeMs);
+            }
+            catch { /* 読めない物は飛ばす */ }
+        }
+    }
+    catch { /* フォルダが読めなければ 0 のまま */ }
+    return newest;
+}
+/**
+ * 古い便の実行ログを消す。消したフォルダ名を返す。
+ *
+ * ★消さない物（Fail Safe）：
+ *   - keep に挙げた便（いま作業中の便・直前に終わった便・これから書く便）
+ *   - 期間内（RUNS_RETENTION_MS）の便
+ *   - 新しい順に RUNS_RETENTION_MAX_DIRS 件まで
+ *   - 便IDらしくない名前・フォルダでない物
+ * ★失敗しても作業は止めない（後片付けは本業ではない）。
+ */
+export function cleanupRunLogs(opts = {}) {
+    const root = opts.root ?? RUNS_DIR;
+    const nowMs = opts.nowMs ?? Date.now();
+    const keep = new Set(opts.keep ?? []);
+    const removed = [];
+    let names;
+    try {
+        names = readdirSync(root);
+    }
+    catch {
+        return removed; // まだ1便も走っていない
+    }
+    const dirs = [];
+    for (const name of names) {
+        if (!RUN_DIR_NAME_RE.test(name))
+            continue;
+        const p = path.join(root, name);
+        try {
+            if (!statSync(p).isDirectory())
+                continue;
+        }
+        catch {
+            continue;
+        }
+        dirs.push({ name, at: newestMtimeMs(p) });
+    }
+    dirs.sort((a, b) => b.at - a.at); // 新しい順
+    for (let i = 0; i < dirs.length; i++) {
+        const d = dirs[i];
+        if (keep.has(d.name))
+            continue;
+        const tooOld = nowMs - d.at > RUNS_RETENTION_MS;
+        const overflow = i >= RUNS_RETENTION_MAX_DIRS;
+        if (!tooOld && !overflow)
+            continue;
+        try {
+            rmSync(path.join(root, d.name), { recursive: true, force: true });
+            removed.push(d.name);
+        }
+        catch (err) {
+            log(`WARN 古い実行ログを消せません (${d.name}): ${err instanceof Error ? err.message : String(err)}`);
+        }
+    }
+    if (removed.length > 0) {
+        log(`古い実行ログを ${removed.length} 件片付けました（${RUNS_RETENTION_MS / 86_400_000}日・最大${RUNS_RETENTION_MAX_DIRS}件を残す）`);
+    }
+    return removed;
+}
 /** 実行ログのディレクトリを作り、stdout / stderr / 進捗 の WriteStream を返す。 */
 function openRunLogs(commandId, attempt) {
     try {
@@ -649,6 +750,12 @@ export function spawnClaudeProcess(cfg, cmd, sessionId, attempt, isResume) {
         return null;
     }
     const logs = openRunLogs(cmd.id, attempt);
+    // ★後片付けはフォルダが増える瞬間にだけ行う（便が1つ増えたら古い便を1つ片付ける）。
+    //   これから書く便・いま手元に記録がある便・直前に終わった便は、名指しで残す。
+    cleanupRunLogs({
+        keep: [cmd.id, loadActiveJob(cfg.site)?.commandId, loadLastJob(cfg.site)?.commandId]
+            .filter((id) => typeof id === 'string' && id.length > 0),
+    });
     const job = {
         commandId: cmd.id,
         process: proc,

package/dist/view.js

@@ -26,11 +26,25 @@ import { RUNS_DIR, eventsLogPath, isProcessAlive, loadActiveJob, loadLastJob, ma
 // 定数
 // ─────────────────────────────────────────────────────────────────────────────
 /**
- * 「最近の作業記録がある」と見なす上限(ms)。これを超えて記録が出ていなければ
- * 実プロセスが動いていても「作業中」とは出さない（えふさん確定②）。
- * ★長い1操作（試験の実行など）で記録が止まることがあるので、短くし過ぎない。
+ * 「最近」の線引き(ms)。★線引きはこの1か所だけ。
+ *
+ * 【使い所は2つだけ（便 062431f2 の直し）】
+ *   ① 見守り役の生死が確かめられないときに限り、「作業記録が最近あるか」の判定に使う。
+ *   ② 🟠 のまま「記録が N分 出ていません」と1行添えるかの線引き。
+ *
+ * 【★これを「止まった（🟡）」の判定には使わない（えふさん確定 2026-10-02 の直し）】
+ *   長い試験（建造くんは4,000件超・TYHLD-platform は8分級）や長い考えごとの間は、
+ *   実プロセスが動いていても記録は何分も出ない。その間に「職人は止まった／終了しました」と
+ *   出すのは、動いている物を「止まった」と言う嘘になる。
+ *   確定の文言は「最近のheartbeat/作業記録」＝**どちらかが最近あればよい**。
+ *
+ * 【なぜ5分か】①は見守り役の生死が分からない機体（systemd も launchd も無い）でだけ働く
+ *   予備の判定で、そこでは「記録が出ている」しか手がかりが無い。②の注記は、読む人が
+ *   「止まったのでは」と不安になる前に理由を添えるのが目的なので、ふつうの1操作
+ *   （ファイルを読む・編集する＝数十秒）より十分長く、長い試験（8分級）より短い所に置く。
+ *   ★この値は表示の言葉を変えるだけで、状態（🟠/🟡）を変えない。
  */
-export const STALE_ACTIVITY_MS = 5 * 60_000;
+export const QUIET_ACTIVITY_MS = 5 * 60_000;
 /** 通常表示に出す「流れ」の行数の上限。 */
 export const FLOW_LINES_MAX = 20;
 /** 詳細表示に出す生ログの行数の上限。 */
@@ -58,11 +72,20 @@ export const STATUS_NOTE = {
 /**
  * 6つの状態のどれかを決める純粋関数（★「作業中」の条件を厳密にする所）。
  *
+ * 【えふさん確定（2026-10-02・1文字も変えない）】
+ *   🟠 作業中 と表示してよいのは「Claude Codeの実プロセスが動いている ＋
+ *   最近のheartbeat/作業記録がある」場合だけ。
+ *   ＝実プロセスは必須。もう一方は **heartbeat か 作業記録 のどちらかが最近あればよい**。
+ *
  * ★順番に意味がある。
  *   1. 見守り役が止まっていれば、何も管制へ届かない＝💤。
  *   2. えふさんの手番は、動いているかどうかより先に出す（人を待たせないため）。
- *   3. 実行中の仕事があるときは、実プロセスと最近の記録の【両方】がそろって初めて 🟠。
- *      そろわなければ 🟡（勝手に「作業中」と出さない）。
+ *   3. 実行中の仕事があるとき：
+ *      a. 実プロセスが消えた・終了した・生死を確かめられない → 🟡 未確認。
+ *      b. 実プロセスが生きていて、見守り役も生きている → 🟠（★記録が止まっていても 🟠）。
+ *         長い試験や長い考えごとの最中に「止まった」と言わないため（便 062431f2 の直し）。
+ *      c. 実プロセスが生きていて、見守り役の生死が確かめられない（null）→
+ *         手がかりは記録だけ。最近の記録があれば 🟠、無ければ 🟡。
  *   4. 実行中の仕事が無ければ、直前の結果が在れば ✅、無ければ ⏳。
  */
 export function decideSiteStatus(input) {
@@ -73,16 +96,26 @@ export function decideSiteStatus(input) {
     if (input.hasActiveJob) {
         if (!input.processAlive)
             return 'unconfirmed';
-        if (input.sinceActivityMs === null)
-            return 'unconfirmed';
-        if (input.sinceActivityMs > STALE_ACTIVITY_MS)
-            return 'unconfirmed';
-        return 'working';
+        if (input.relayRunning === true)
+            return 'working';
+        const activityFresh = input.sinceActivityMs !== null && input.sinceActivityMs <= QUIET_ACTIVITY_MS;
+        return activityFresh ? 'working' : 'unconfirmed';
     }
     if (input.hasLastJob)
         return 'done';
     return 'queued';
 }
+/**
+ * 🟠 のまま添える注記を返す（記録が長く止まっているときだけ1行）。
+ * ★状態は変えない。「止まったのでは」と読まれる前に、理由の候補を添えるだけ。
+ */
+export function quietActivityNote(sinceActivityMs) {
+    if (sinceActivityMs === null)
+        return null;
+    if (sinceActivityMs <= QUIET_ACTIVITY_MS)
+        return null;
+    return `記録が ${friendlyDuration(sinceActivityMs)} 出ていません（長い試験の最中かもしれません）`;
+}
 // ─────────────────────────────────────────────────────────────────────────────
 // 書式
 // ─────────────────────────────────────────────────────────────────────────────
@@ -347,9 +380,19 @@ export function renderNormalView(v) {
     else {
         out.push('実作業：—');
     }
-    out.push(`最後に動いた：${v.lastActivityAt !== null ? friendlyTime(v.lastActivityAt, v.nowMs) : unknown}`);
+    // ★「いつ」だけでなく「どれだけ前か」も出す（止まっている時間が一目で分かる）。
+    const sinceActivityMs = v.lastActivityAt === null ? null : v.nowMs - v.lastActivityAt;
+    out.push(v.lastActivityAt !== null
+        ? `最後に動いた：${friendlyTime(v.lastActivityAt, v.nowMs)}（${friendlyDuration(sinceActivityMs ?? 0)}前）`
+        : `最後に動いた：${unknown}`);
     const current = v.flow.length > 0 ? v.flow[v.flow.length - 1].text : '';
     out.push(`現在：${current.length > 0 ? current : unknown}`);
+    // ★作業中のまま記録が長く止まっているときは、理由の候補を1行添える（状態は変えない）。
+    if (v.status === 'working') {
+        const note = quietActivityNote(sinceActivityMs);
+        if (note !== null)
+            out.push(`※${note}`);
+    }
     out.push(RULE);
     if (v.flow.length > 0) {
         for (const l of v.flow)

package/package.json

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

package/scripts/hooks/policy.py

@@ -2065,12 +2065,23 @@ def _prod_db_change_result(reason):
 # 危険でない作業フォルダの絶対パス先頭成分（/tmp/xxx は限定削除として通す）。
 _SAFE_ABS_TOP = frozenset({'tmp'})
 
+# rm で保護するディレクトリ名（workDir の中であっても削除禁止）。
+# .git … リポジトリの履歴、.claude … エージェント設定、.tyhld … 番人・道具の設定。
+_RM_PROTECTED_DIRS = frozenset({'.git', '.claude', '.tyhld'})
+
+
+def _rm_hits_protected(rel_comps: list) -> bool:
+    """相対パスの成分リストが保護対象を含むか（.git・.claude・.tyhld のいずれかに当たるか）。"""
+    return bool(_RM_PROTECTED_DIRS.intersection(rel_comps))
+
 
 def rm_path_risk(path: str, work_dir: str = None) -> str:
     """rm の対象パスの危険度: 'dangerous' / 'safe' / 'unknown'。
 
     work_dir が渡されたとき、絶対パスが work_dir の中を指すなら 'safe'（相対パスと同じ扱い）。
-    ただし work_dir 自体・.git を含むパス・.. やシンボリックリンクで外に出るパスは 'dangerous'。
+    ただし work_dir 自体・保護対象（.git/.claude/.tyhld）を含むパス・
+    .. やシンボリックリンクで外に出るパスは 'dangerous'。
+    相対パスでも保護対象を含むなら 'dangerous'。
     work_dir が None のとき（非 conductor 環境）は従来どおり。
     """
     p = path.strip().strip('"').strip("'")
@@ -2092,10 +2103,8 @@ def rm_path_risk(path: str, work_dir: str = None) -> str:
         if not comps:
             return 'dangerous'
         # ★workDir 内の絶対パスは相対パスと同じ扱い（便 24c9b6b6）。
-        #   workDir 自体・.git・.. やシンボリックリンクで外に出るパスは引き続き dangerous。
+        #   workDir 自体・保護対象（.git/.claude/.tyhld）・.. やシンボリックリンクで外に出るパスは dangerous。
         #   workDir を知れないときは従来どおり止める（Fail Closed）。
-        #   ★秘密（.env 等）と設定（.claude 等）は別の検査（_secret_danger / _guard_write_scan）が
-        #     引き続き止めるので、ここでは .git だけを追加で守る。
         if work_dir:
             try:
                 resolved = os.path.realpath(p)
@@ -2104,14 +2113,37 @@ def rm_path_risk(path: str, work_dir: str = None) -> str:
                     pass  # workDir 自体 → fall through to 'dangerous'
                 elif resolved.startswith(work_resolved + os.sep):
                     rel_comps = resolved[len(work_resolved) + 1:].split(os.sep)
-                    if '.git' not in rel_comps:
+                    if not _rm_hits_protected(rel_comps):
                         return 'safe'
-                    # .git を含む → fall through to 'dangerous'
+                    # 保護対象を含む → fall through to 'dangerous'
             except (OSError, ValueError):
                 pass  # 解決できない → fall through（Fail Closed）
         if comps[0] in _SAFE_ABS_TOP and len(comps) >= 2:
             return 'safe'
         return 'dangerous'
+    # 相対パス: 保護対象を含むなら dangerous（書き方に依らず最終的に消される先で判定）。
+    # work_dir があれば解決して判定、なければ成分名で判定（Fail Closed）。
+    rel_comps = [c for c in p.replace('\\', '/').split('/') if c and c != '.']
+    if '..' in rel_comps:
+        # .. を含む相対パスは解決しないと行き先が分からない
+        if work_dir:
+            try:
+                resolved = os.path.realpath(os.path.join(work_dir, p))
+                work_resolved = os.path.realpath(work_dir)
+                if resolved == work_resolved or not resolved.startswith(work_resolved + os.sep):
+                    return 'dangerous'
+                inner = resolved[len(work_resolved) + 1:].split(os.sep)
+                if _rm_hits_protected(inner):
+                    return 'dangerous'
+            except (OSError, ValueError):
+                # 解決できない＋保護対象の名前を含む → dangerous（Fail Closed）
+                if _RM_PROTECTED_DIRS.intersection(rel_comps):
+                    return 'dangerous'
+        else:
+            if _RM_PROTECTED_DIRS.intersection(rel_comps):
+                return 'dangerous'
+    elif _rm_hits_protected(rel_comps):
+        return 'dangerous'
     return 'safe'
 
 

package/scripts/hooks/test_policy.py

@@ -1825,6 +1825,82 @@ finally:
     else:
         os.environ['CONDUCTOR_WORK_DIR'] = _wd_env
 
+# ─────────────────────────────────────────────────────────────────────────────
+# workDir 内の rm: .claude/.tyhld/.git を絶対パス・相対パス両方で保護（便 13954515）
+# ─────────────────────────────────────────────────────────────────────────────
+print('\n=== workDir 内の rm: 環境設定の保護（絶対パス） ===')
+check('workDir 内の .claude/hooks (絶対パス)',
+      policy.rm_path_risk(f'{_WD}/.claude/hooks', _WD), 'dangerous')
+check('workDir 内の .claude/任意ファイル (絶対パス)',
+      policy.rm_path_risk(f'{_WD}/.claude/any-file.json', _WD), 'dangerous')
+check('workDir 内の .tyhld (絶対パス)',
+      policy.rm_path_risk(f'{_WD}/.tyhld', _WD), 'dangerous')
+check('workDir 内の .tyhld/hooks/p.py (絶対パス)',
+      policy.rm_path_risk(f'{_WD}/.tyhld/hooks/p.py', _WD), 'dangerous')
+check('workDir 内の作業ファイルは引き続き safe (絶対パス)',
+      policy.rm_path_risk(f'{_WD}/src/main.ts', _WD), 'safe')
+
+print('\n=== workDir 内の rm: 環境設定の保護（相対パス） ===')
+check('rm -rf .git (相対パス)',
+      policy.rm_path_risk('.git', _WD), 'dangerous')
+check('rm -rf ./.git (相対パス)',
+      policy.rm_path_risk('./.git', _WD), 'dangerous')
+check('rm -rf .git/config (相対パス)',
+      policy.rm_path_risk('.git/config', _WD), 'dangerous')
+check('rm -rf .claude/hooks (相対パス)',
+      policy.rm_path_risk('.claude/hooks', _WD), 'dangerous')
+check('rm -rf .claude/settings.json (相対パス)',
+      policy.rm_path_risk('.claude/settings.json', _WD), 'dangerous')
+check('rm -rf .tyhld (相対パス)',
+      policy.rm_path_risk('.tyhld', _WD), 'dangerous')
+check('rm -rf .tyhld/hooks (相対パス)',
+      policy.rm_path_risk('.tyhld/hooks', _WD), 'dangerous')
+# .. で保護対象を指す
+check('sub/../.git (相対パス)',
+      policy.rm_path_risk('sub/../.git', _WD), 'dangerous')
+check('sub/../.claude/hooks (相対パス)',
+      policy.rm_path_risk('sub/../.claude/hooks', _WD), 'dangerous')
+check('sub/../.tyhld (相対パス)',
+      policy.rm_path_risk('sub/../.tyhld', _WD), 'dangerous')
+# 普通の相対パスは引き続き safe
+check('rm -rf node_modules (相対パス) → safe',
+      policy.rm_path_risk('node_modules', _WD), 'safe')
+check('rm -rf dist (相対パス) → safe',
+      policy.rm_path_risk('dist', _WD), 'safe')
+check('rm foo.txt (相対パス) → safe',
+      policy.rm_path_risk('foo.txt', _WD), 'safe')
+check('rm -rf .next (相対パス) → safe',
+      policy.rm_path_risk('.next', _WD), 'safe')
+
+print('\n=== workDir 内の rm: decide_bash 経由で環境設定の保護 ===')
+_wd_env2 = os.environ.get('CONDUCTOR_WORK_DIR')
+try:
+    os.environ['CONDUCTOR_WORK_DIR'] = _WD
+    # 絶対パス
+    check('decide_bash: rm -rf .claude/hooks (絶対パス) → deny',
+          policy.decide_bash(f'rm -rf {_WD}/.claude/hooks')['decision'], policy.DENY)
+    check('decide_bash: rm -rf .tyhld (絶対パス) → deny',
+          policy.decide_bash(f'rm -rf {_WD}/.tyhld')['decision'], policy.DENY)
+    # 相対パス
+    check('decide_bash: rm -rf .git (相対パス) → deny',
+          policy.decide_bash('rm -rf .git')['decision'], policy.DENY)
+    check('decide_bash: rm -rf .claude/hooks (相対パス) → deny',
+          policy.decide_bash('rm -rf .claude/hooks')['decision'], policy.DENY)
+    check('decide_bash: rm -rf .tyhld (相対パス) → deny',
+          policy.decide_bash('rm -rf .tyhld')['decision'], policy.DENY)
+    # 普通の削除は引き続き allow
+    check('decide_bash: rm -rf node_modules (相対パス) → allow',
+          policy.decide_bash('rm -rf node_modules')['decision'], policy.ALLOW)
+    check('decide_bash: rm -rf dist (相対パス) → allow',
+          policy.decide_bash('rm -rf dist')['decision'], policy.ALLOW)
+    check('decide_bash: rm foo.txt → allow',
+          policy.decide_bash('rm foo.txt')['decision'], policy.ALLOW)
+finally:
+    if _wd_env2 is None:
+        os.environ.pop('CONDUCTOR_WORK_DIR', None)
+    else:
+        os.environ['CONDUCTOR_WORK_DIR'] = _wd_env2
+
 # ─────────────────────────────────────────────────────────────────────────────
 print('\n' + '=' * 72)
 if failures: