triflux 10.13.0 → 10.13.1

package/hub/bridge.mjs

@@ -23,7 +23,6 @@ import { parseArgs as nodeParseArgs } from "node:util";
 import { getPipelineStateDbPath } from "./pipeline/state.mjs";
 import {
   createRetryStateMachine,
-  DEFAULT_ESCALATION_CHAIN,
   loadSnapshot,
   saveSnapshot,
 } from "./team/retry-state-machine.mjs";

package/hub/team/cli/commands/start/start-headless.mjs

@@ -45,7 +45,7 @@ export async function startHeadlessTeam({
   ok(`headless ${assignments.length}워커 시작`);
 
   const handle = await runHeadlessInteractive(sessionId, assignments, {
-    timeoutSec: timeoutSec || 300,
+    timeoutSec: timeoutSec || 900,
     layout,
     autoAttach: !!autoAttach,
     dashboard: !!dashboard,

package/hub/team/headless.mjs

@@ -11,6 +11,7 @@ import {
   mkdirSync,
   readFileSync,
   renameSync,
+  rmSync,
   statSync,
   writeFileSync,
 } from "node:fs";
@@ -240,16 +241,53 @@ export function buildHeadlessCommand(cli, prompt, resultFile, opts = {}) {
   return `cd '${safeCwd.replace(/'/g, "'\\''")}' && ${backendCommand}`;
 }
 
+/**
+ * 이전 run 의 stale .txt / .partial / .err 를 제거한다.
+ * Issue #118 Codex review R1 HIGH: resultFile 경로 재사용 시 (워커 restart 또는
+ * 동일 세션명 재실행) 이전 run 의 HANDOFF 가 새 run 결과로 오인될 수 있다.
+ *
+ * Issue #118 R2 MEDIUM: Windows 에서 locked/open 파일로 인한 삭제 실패를
+ * silent swallow 하면 stale artifact 가 살아남아 bug 가 non-deterministic 하게
+ * 재발한다. ENOENT 외 에러는 경고 로그 + 재시도(1회) 후 계속.
+ * @param {string} resultFile
+ */
+export function cleanStaleResultArtifacts(resultFile) {
+  for (const suffix of ["", ".partial", ".err"]) {
+    const path = `${resultFile}${suffix}`;
+    for (let attempt = 0; attempt < 2; attempt++) {
+      try {
+        rmSync(path);
+        break;
+      } catch (err) {
+        if (err?.code === "ENOENT") break; // 이미 없음 — 정상
+        if (attempt === 0) continue; // 1회 재시도
+        // 재시도도 실패: locked 파일일 수 있음. 경고 로그 후 계속 진행
+        // (dispatch 중단 시 워커 자체가 시작 못하므로 best-effort 유지)
+        console.warn(
+          `[headless] cleanStaleResultArtifacts: ${path} 삭제 실패 (${err?.code || "unknown"}). stale leak 가능.`,
+        );
+      }
+    }
+  }
+}
+
 /**
  * 결과 파일 읽기 (없으면 capture-pane fallback)
+ * Issue #118 fallback chain: .txt → .partial ([partial] prefix) → .err → capture-pane
  * @param {string} resultFile
  * @param {string} paneId
  * @returns {string}
  */
-function readResult(resultFile, paneId) {
+export function readResult(resultFile, paneId) {
   if (existsSync(resultFile)) {
     return readFileSync(resultFile, "utf8").trim();
   }
+  // Issue #118 fallback 0: timeout 직전 persist 된 부분 출력 (.partial)
+  const partialFile = `${resultFile}.partial`;
+  if (existsSync(partialFile)) {
+    const partial = readFileSync(partialFile, "utf8").trim();
+    if (partial) return `[partial] ${partial}`;
+  }
   // fallback 1: stderr 파일 (codex 실패 시 원인 추적)
   const errFile = `${resultFile}.err`;
   if (existsSync(errFile)) {
@@ -417,6 +455,17 @@ export async function waitForCompletionWithStallDetect(
 
       // 전체 타임아웃
       if (now - startedAt > completionTimeout) {
+        // Issue #118: timeout kill 전에 capture-pane 출력을 .partial 파일로 persist.
+        // resultFile(`.txt`)이 아직 생성되지 않았을 수 있으므로 readResult 가 fallback 으로 읽는다.
+        try {
+          const partialSnapshot = _capture(currentPaneId, 200);
+          if (partialSnapshot && partialSnapshot.trim().length > 0) {
+            const writer = deps.writeFileSync || writeFileSync;
+            writer(`${resultFile}.partial`, partialSnapshot, "utf8");
+          }
+        } catch {
+          /* best-effort, timeout 은 이미 확정 */
+        }
         return {
           matched: false,
           exitCode: null,
@@ -610,6 +659,8 @@ async function dispatchProgressive(sessionName, assignments, opts = {}) {
       RESULT_DIR,
       `${sessionName}-${paneName}.txt`,
     ).replace(/\\/g, "/");
+    // Issue #118 review R1 HIGH: stale artifact 제거 (이전 run / restart 잔재)
+    cleanStaleResultArtifacts(resultFile);
     const cmd = buildHeadlessCommand(
       assignment.cli,
       assignment.prompt,
@@ -686,6 +737,8 @@ async function dispatchBatch(sessionName, assignments, opts = {}) {
         RESULT_DIR,
         `${sessionName}-${paneName}.txt`,
       ).replace(/\\/g, "/");
+      // Issue #118 review R1 HIGH: stale artifact 제거 (이전 run / restart 잔재)
+      cleanStaleResultArtifacts(resultFile);
       const cmd = buildHeadlessCommand(
         assignment.cli,
         assignment.prompt,
@@ -832,9 +885,20 @@ async function awaitAll(
         );
       }
 
-      const output = completion.matched
-        ? readResult(d.resultFile, d.paneId)
-        : "";
+      // Issue #118: matched=false(timeout kill) 에도 capture-pane 스냅샷을
+      // .partial 로 persist 하여 readResult fallback chain 이 복구할 수 있도록 한다.
+      if (!completion.matched) {
+        try {
+          const snap = capturePsmuxPane(d.paneId || d.paneName, 200);
+          if (snap && snap.trim().length > 0) {
+            writeFileSync(`${d.resultFile}.partial`, snap, "utf8");
+          }
+        } catch {
+          /* best-effort */
+        }
+      }
+
+      const output = readResult(d.resultFile, d.paneId);
       unregisterHeadlessSynapseWorker(d.workerId);
 
       if (safeProgress) {
@@ -945,7 +1009,7 @@ function collectGitDiffFiles(cwd) {
  */
 export async function runHeadless(sessionName, assignments, opts = {}) {
   const {
-    timeoutSec = 300,
+    timeoutSec = 900,
     layout = "2x2",
     onProgress,
     progressIntervalSec = 0,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "triflux",
-  "version": "10.13.0",
+  "version": "10.13.1",
   "description": "CLI-first multi-model orchestrator for Claude Code — route tasks to Codex, Gemini, and Claude",
   "type": "module",
   "bin": {

package/scripts/setup.mjs

@@ -56,10 +56,45 @@ const REQUIRED_CODEX_PROFILES = [
     name: "codex53_xhigh",
     lines: ['model = "gpt-5.3-codex"', 'model_reasoning_effort = "xhigh"'],
   },
+  {
+    name: "codex53_med",
+    lines: ['model = "gpt-5.3-codex"', 'model_reasoning_effort = "medium"'],
+  },
   {
     name: "spark53_low",
     lines: ['model = "gpt-5.3-codex-spark"', 'model_reasoning_effort = "low"'],
   },
+  {
+    name: "spark53_med",
+    lines: [
+      'model = "gpt-5.3-codex-spark"',
+      'model_reasoning_effort = "medium"',
+    ],
+  },
+  {
+    name: "gpt54_xhigh",
+    lines: ['model = "gpt-5.4"', 'model_reasoning_effort = "xhigh"'],
+  },
+  {
+    name: "gpt54_high",
+    lines: ['model = "gpt-5.4"', 'model_reasoning_effort = "high"'],
+  },
+  {
+    name: "gpt54_low",
+    lines: ['model = "gpt-5.4"', 'model_reasoning_effort = "low"'],
+  },
+  {
+    name: "mini54_low",
+    lines: ['model = "gpt-5.4-mini"', 'model_reasoning_effort = "low"'],
+  },
+  {
+    name: "mini54_med",
+    lines: ['model = "gpt-5.4-mini"', 'model_reasoning_effort = "medium"'],
+  },
+  {
+    name: "mini54_high",
+    lines: ['model = "gpt-5.4-mini"', 'model_reasoning_effort = "high"'],
+  },
 ];
 
 const HUD_SYNC_EXCLUDES = new Set(["omc-hud.mjs", "omc-hud.mjs.bak"]);
@@ -556,6 +591,15 @@ function ensureCodexHubServerConfig({
   }
 }
 
+// Top-level config.toml keys that must exist with these defaults.
+// Only injected when the key is completely absent — existing user values are
+// never overwritten, regardless of what value was set.
+const REQUIRED_TOP_LEVEL_SETTINGS = [
+  { key: "model", value: '"gpt-5.4"' },
+  { key: "model_reasoning_effort", value: '"high"' },
+  { key: "service_tier", value: '"fast"' },
+];
+
 function ensureCodexProfiles() {
   try {
     if (!existsSync(CODEX_DIR)) mkdirSync(CODEX_DIR, { recursive: true });
@@ -564,9 +608,45 @@ function ensureCodexProfiles() {
       ? readFileSync(CODEX_CONFIG_PATH, "utf8")
       : "";
 
+    // Safety guard: if the file exists but is suspiciously small (< 100 bytes)
+    // skip all writes to avoid perpetuating a corrupted state.
+    if (original.length > 0 && original.length < 100) {
+      process.stderr.write(
+        `[tfx-setup] config.toml 크기 이상 (${original.length} bytes) — 쓰기 스킵. 수동 확인 필요: ${CODEX_CONFIG_PATH}\n`,
+      );
+      return { ok: false, changed: 0, message: "config too small, skipped" };
+    }
+
     let updated = original;
     let changed = 0;
 
+    // ── 1. top-level 필수 설정 주입 (없을 때만, 기존 값 보존) ──
+    // 파일 상단 [profiles.*] / [mcp_servers.*] 이전 영역만 검사한다.
+    // 프로필 섹션 내부의 동명 키(예: model = "gpt-5.3-codex")는 무시한다.
+    // 이미 존재하는 키는 절대 덮어쓰지 않는다.
+    for (const { key, value } of REQUIRED_TOP_LEVEL_SETTINGS) {
+      // top-level 영역 = 첫 번째 [profiles.*] / [mcp_servers.*] 헤더 이전
+      const firstSectionIdx = updated.search(/^\[(?:profiles|mcp_servers)\./m);
+      const topLevelRegion =
+        firstSectionIdx === -1 ? updated : updated.slice(0, firstSectionIdx);
+      const topLevelKeyRe = new RegExp(`^${key}\\s*=`, "m");
+      if (!topLevelKeyRe.test(topLevelRegion)) {
+        // firstSectionIdx already computed above for this iteration
+        const line = `${key} = ${value}\n`;
+        if (firstSectionIdx === -1) {
+          // 섹션이 없으면 파일 맨 앞에 추가
+          updated = line + updated;
+        } else {
+          updated =
+            updated.slice(0, firstSectionIdx) +
+            line +
+            updated.slice(firstSectionIdx);
+        }
+        changed++;
+      }
+    }
+
+    // ── 2. 필수 프로필 보장 ──
     for (const profile of REQUIRED_CODEX_PROFILES) {
       const desired = `[profiles.${profile.name}]\n${profile.lines.join("\n")}\n`;
 
@@ -898,6 +978,7 @@ export {
   LEGACY_CODEX_MODELS,
   PLUGIN_ROOT,
   REQUIRED_CODEX_PROFILES,
+  REQUIRED_TOP_LEVEL_SETTINGS,
   readMarker,
   replaceProfileSection,
   SETUP_MARKER_PATH,

package/scripts/tfx-route.sh

@@ -1445,6 +1445,14 @@ _codex_config_swap() {
       return 0
     fi
 
+    # Pre-validation: config.toml이 500 bytes 미만이면 이미 손상된 상태일 수 있음 — 스킵
+    local config_size
+    config_size=$(wc -c < "$config" 2>/dev/null | tr -d ' ') || config_size=0
+    if [[ "$config_size" -lt 500 ]]; then
+      echo "[tfx-route] 경고: config.toml 크기 ${config_size} bytes — 손상 의심, swap 스킵 (수동 확인 필요)" >&2
+      return 0
+    fi
+
     # 백업 생성 (이미 있으면 다른 워커가 swap 중 — 건드리지 않음)
     if [[ -f "$backup" ]]; then
       echo "[tfx-route] config.toml swap 스킵: 다른 워커가 사용 중 ($backup)" >&2
@@ -1454,6 +1462,7 @@ _codex_config_swap() {
 
     # awk로 필터링: 비허용 MCP 서버 섹션 제거, 나머지 그대로 유지.
     # keep="" 은 진입 가드에서 return 됐지만 defense-in-depth 유지.
+    local tmp_filtered="${config}.filter.$$"
     awk -v keep="$allowed_pat" '
       BEGIN { skip=0 }
       /^\[mcp_servers\./ {
@@ -1464,7 +1473,26 @@ _codex_config_swap() {
       }
       /^\[/ && !/^\[mcp_servers\./ { skip=0 }
       !skip { print }
-    ' "$backup" > "$config"
+    ' "$backup" > "$tmp_filtered"
+
+    # Output sanity check: 필터 결과가 비었거나 백업의 30% 미만이면 적용 거부
+    local filtered_size backup_size threshold
+    filtered_size=$(wc -c < "$tmp_filtered" 2>/dev/null | tr -d ' ') || filtered_size=0
+    backup_size=$(wc -c < "$backup" 2>/dev/null | tr -d ' ') || backup_size=1
+    threshold=$(( backup_size * 30 / 100 ))
+    if [[ "$filtered_size" -eq 0 || "$filtered_size" -lt "$threshold" ]]; then
+      echo "[tfx-route] 경고: 필터 결과 크기 ${filtered_size} bytes (백업 ${backup_size} bytes의 30% 미만) — 적용 거부, 백업에서 복원" >&2
+      rm -f "$tmp_filtered" 2>/dev/null
+      rm -f "$backup" 2>/dev/null
+      return 1
+    fi
+
+    # 검증 통과 — atomic rename으로 적용
+    if ! mv "$tmp_filtered" "$config"; then
+      echo "[tfx-route] 경고: 필터 결과 적용 실패 (atomic rename), 백업 보존: $backup" >&2
+      rm -f "$tmp_filtered" 2>/dev/null
+      return 1
+    fi
 
     local kept
     kept=$(echo "$allowed_pat" | tr '|' '\n' | wc -l | tr -d ' ')
@@ -1475,6 +1503,15 @@ _codex_config_swap() {
     # `cat > $config` 는 cat 실행 전에 dest 가 truncate 되어 mid-stream 실패 시
     # 빈/부분 파일이 남는다. 같은 디렉토리 내 mv 는 POSIX 상 atomic 이므로
     # 실패해도 기존 config 와 backup 모두 보존된다.
+
+    # Restore sanity check: 백업 자체가 비었거나 500 bytes 미만이면 복원 중단
+    local backup_restore_size
+    backup_restore_size=$(wc -c < "$backup" 2>/dev/null | tr -d ' ') || backup_restore_size=0
+    if [[ "$backup_restore_size" -lt 500 ]]; then
+      echo "[tfx-route] 경고: backup 크기 ${backup_restore_size} bytes — 손상 의심, 복원 중단. 수동 확인 필요: $backup" >&2
+      return 1
+    fi
+
     local tmp="${config}.restore.$$"
     if ! cp "$backup" "$tmp"; then
       echo "[tfx-route] 경고: config.toml 복원 실패 (temp copy). backup 보존: $backup" >&2