@triflux/remote 10.53.0 → 10.55.0

package/scripts/lib/backup-rotation.mjs

@@ -0,0 +1,60 @@
+import {
+  existsSync,
+  readdirSync,
+  readFileSync,
+  rmSync,
+  statSync,
+  writeFileSync,
+} from "node:fs";
+import { basename, dirname, join } from "node:path";
+
+// 설정이 바뀔 때마다 백업이 쌓여 끝없이 늘었다. 같은 대상의 백업은 최근 몇 개만 남긴다.
+export const BACKUP_KEEP = 5;
+
+// 시각 suffix 만 받는다. label 이 더 긴 다른 백업(예: <label>-manual-...)은 남긴다.
+const STAMP_SUFFIX = /^[\dT]+(?:-\d+)*$/i;
+
+// 읽는 사이 다른 프로세스가 지웠으면 같은 것으로 보고 아래에서 다시 쓴다. 다른 읽기 오류는 그대로 던진다.
+function sameContent(path, content) {
+  try {
+    return readFileSync(path, "utf8") === content;
+  } catch (error) {
+    if (error?.code === "ENOENT") return true;
+    throw error;
+  }
+}
+
+// `<파일>.<label>-<시각>` 으로 백업하고, 같은 label 의 오래된 백업을 지운다.
+export function writeRotatedBackup(
+  path,
+  content,
+  { label, suffix, mode, keep = BACKUP_KEEP },
+) {
+  const directory = dirname(path);
+  const prefix = `${basename(path)}.${label}-`;
+  // 같은 시각 이름에 다른 원문이 이미 있으면 번호를 붙여 따로 남긴다.
+  let backupPath = join(directory, `${prefix}${suffix}`);
+  for (let i = 1; existsSync(backupPath); i++) {
+    if (sameContent(backupPath, content)) break;
+    backupPath = join(directory, `${prefix}${suffix}-${i}`);
+  }
+  if (!existsSync(backupPath))
+    writeFileSync(backupPath, content, { encoding: "utf8", mode });
+  const backups = readdirSync(directory)
+    .filter(
+      (name) =>
+        name.startsWith(prefix) && STAMP_SUFFIX.test(name.slice(prefix.length)),
+    )
+    .flatMap((name) => {
+      // 다른 프로세스가 같은 백업을 먼저 지웠을 수 있다.
+      try {
+        return [{ name, mtime: statSync(join(directory, name)).mtimeMs }];
+      } catch {
+        return [];
+      }
+    })
+    .sort((a, b) => b.mtime - a.mtime || b.name.localeCompare(a.name));
+  for (const { name } of backups.slice(keep))
+    rmSync(join(directory, name), { force: true });
+  return backupPath;
+}

package/scripts/lib/codex-profile-config.mjs

@@ -3,10 +3,12 @@ import {
   mkdirSync,
   readFileSync,
   renameSync,
+  statSync,
   writeFileSync,
 } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
+import { writeRotatedBackup } from "./backup-rotation.mjs";
 
 const EFFORT_BY_SUFFIX = {
   ultra: "ultra",
@@ -80,10 +82,6 @@ function readProfileScalar(raw, key) {
  * naming convention when the file is absent. Returns the keys that resolve, in
  * `key="value"` TOML-scalar form ready to be shell-quoted by the caller.
  *
- * NOTE: the Codex MCP transport does the structured-arg equivalent in
- * hub/workers/codex-mcp.mjs (resolveCodexProfileConfig → model + config object);
- * this returns the CLI `-c` string form for the same per-profile file.
- *
  * @param {string} profileName
  * @param {{ codexHome?: string }} [opts]
  * @returns {string[]} e.g. ['model="gpt-6.1-sol"', 'model_reasoning_effort="high"']
@@ -230,7 +228,8 @@ export function sanitizeCodexProfileConfig(
         skippedProfileFiles.push(profileName);
         continue;
       }
-      writeFileSync(profilePath, body, "utf8");
+      // 옮겨 온 프로필 본문에 MCP env 키가 있을 수 있다.
+      writeFileSync(profilePath, body, { encoding: "utf8", mode: 0o600 });
       migratedProfiles.push(profileName);
     }
   }
@@ -254,6 +253,15 @@ function stamp(now = () => new Date()) {
     .replace("T", "-");
 }
 
+// config.toml 에는 MCP env 키가 들어갈 수 있다. 다시 쓸 때 원본 권한을 지키고 없으면 0600 으로 만든다.
+export function privateFileMode(path) {
+  try {
+    return statSync(path).mode & 0o777;
+  } catch {
+    return 0o600;
+  }
+}
+
 export function sanitizeCodexProfileConfigFile(
   configPath,
   {
@@ -268,7 +276,12 @@ export function sanitizeCodexProfileConfigFile(
   const source = readFileSync(configPath, "utf8");
   const sanitized = sanitizeCodexProfileConfig(source, { codexHome });
   if (!sanitized.changed) return sanitized;
-  writeFileSync(`${configPath}.${backupPrefix}-${stamp(now)}`, source, "utf8");
+  const mode = privateFileMode(configPath);
+  writeRotatedBackup(configPath, source, {
+    label: backupPrefix,
+    suffix: stamp(now),
+    mode,
+  });
   // Atomic replace. config.toml is shared global state: a codex session may be
   // reading it while another entry point (tfx-route, the codex session hook, or
   // a parallel swarm worker) sanitizes it. Write the sanitized content to a
@@ -278,7 +291,7 @@ export function sanitizeCodexProfileConfigFile(
   // without corruption (the content is identical anyway since the migration is
   // deterministic). The pid keeps temp names unique across separate processes.
   const tmpPath = `${configPath}.tmp-${process.pid}-${stamp(now)}`;
-  writeFileSync(tmpPath, sanitized.toml, "utf8");
+  writeFileSync(tmpPath, sanitized.toml, { encoding: "utf8", mode });
   renameSync(tmpPath, configPath);
   return sanitized;
 }

package/scripts/lib/codex-recovery.sh

@@ -58,9 +58,8 @@ recover_codex_stdout() {
 
 # ── MCP transport 채널 사망 판정 ──
 # Codex MCP transport 채널이 실행 중 죽으면 codex exec 가 exit 0 + 빈 stdout 으로
-# 끝나는 변형이 있다(래퍼 부재로 인한 pre-flight CODEX_MCP_TRANSPORT_EXIT_CODE=70
-# 과 구분됨 — 이건 런타임 채널 사망). stderr 의 하드 크래시 서명으로 이 사망을
-# 검출한다. 아우터 가드가 exit 0 이어도 결과를 실패로 승격하는 근거로 쓴다.
+# 끝나는 변형이 있다. stderr 의 하드 크래시 서명으로 이 사망을 검출한다.
+# 아우터 가드가 exit 0 이어도 결과를 실패로 승격하는 근거로 쓴다.
 # 인자: [stderr_log] (미지정 시 $STDERR_LOG). 반환: 0=크래시 서명 있음, 1=없음.
 codex_stdout_transport_crashed() {
   local stderr_log="${1:-${STDERR_LOG:-}}"