@cocorograph/hub-agent 0.8.107 → 0.8.108

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.107",
+  "version": "0.8.108",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの Claude / Codex チャットと tmux セッションを中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/hub-bundle.mjs

@@ -322,18 +322,50 @@ function _postInstallArgv(cmd, win) {
   return null
 }
 
+// stdout/stderr を継承した子プロセスの数。agent.log の回転はこの子の追記を保護できないため、
+// 0 でない間は回転を延期する (log-rotation.mjs)。
+let inheritedOutputChildren = 0
+
+export function hasInheritedOutputChild() {
+  return inheritedOutputChildren > 0
+}
+
 function runOnce(argv, { logger } = {}) {
   return new Promise((resolve, reject) => {
     const [bin, ...args] = argv
-    const child = spawn(bin, args, { stdio: "inherit" })
+    inheritedOutputChildren++
+    let released = false
+    const release = () => {
+      if (released) return
+      released = true
+      inheritedOutputChildren--
+    }
+    let child
+    try {
+      child = spawn(bin, args, { stdio: "inherit" })
+    } catch (err) {
+      release()
+      throw err
+    }
+    // spawn 失敗 (ENOENT 等) は exit が来ないことがある。error でも必ず settle させ、
+    // bundle 同期を未完了のまま残さない (resolve/reject は最初の 1 回だけ有効)。
+    // 診断ログは settle の後に出し、ログ書き込みの失敗 (ENOSPC 等) が settle を妨げないようにする。
+    const safeLog = (level, fields, message) => {
+      try { logger?.[level]?.(fields, message) } catch {}
+    }
+    child.once("error", (err) => {
+      release()
+      reject(err)
+      safeLog("warn", { cmd: argv.join(" "), error: err.message }, "hub-bundle post_install failed")
+    })
     child.on("exit", (code) => {
+      release()
       if (code === 0) {
-        logger?.info({ cmd: argv.join(" ") }, "hub-bundle post_install ok")
         resolve(code)
+        safeLog("info", { cmd: argv.join(" ") }, "hub-bundle post_install ok")
       } else {
-        const err = new Error(`post_install command failed (exit ${code}): ${argv.join(" ")}`)
-        logger?.warn({ cmd: argv.join(" "), code }, "hub-bundle post_install failed")
-        reject(err)
+        reject(new Error(`post_install command failed (exit ${code}): ${argv.join(" ")}`))
+        safeLog("warn", { cmd: argv.join(" "), code }, "hub-bundle post_install failed")
       }
     })
   })

package/src/log-rotation.mjs

@@ -0,0 +1,150 @@
+import { randomBytes } from "node:crypto"
+import { closeSync, fstatSync, ftruncateSync, openSync, readdirSync, readSync, renameSync, statSync, unlinkSync, writeSync } from "node:fs"
+import os from "node:os"
+import path from "node:path"
+
+export const DEFAULT_LOG_MAX_BYTES = 50 * 1024 * 1024
+export const LOG_CHECK_INTERVAL_MS = 10 * 60 * 1000
+// 回転は同期 I/O で常駐処理を止めるため、設定値で無制限に大きくさせない。
+export const MAX_LOG_MAX_BYTES = 512 * 1024 * 1024
+
+export function resolveLogMaxBytes(value = process.env.HUB_AGENT_LOG_MAX_BYTES) {
+  if (typeof value !== "string" || !/^\d+$/.test(value)) return DEFAULT_LOG_MAX_BYTES
+  const bytes = Number(value)
+  if (!Number.isSafeInteger(bytes) || bytes <= 0) return DEFAULT_LOG_MAX_BYTES
+  return Math.min(bytes, MAX_LOG_MAX_BYTES)
+}
+
+function sameFile(a, b) {
+  return a.dev === b.dev && a.ino === b.ino
+}
+
+// 削除判断に使うので、確実に居ないと分かる ESRCH のときだけ死亡扱いにする
+// (hub-bundle.mjs の _pidAlive はロック回収用で、EPERM 以外を死亡扱いにする別契約)。
+function processAlive(pid) {
+  try {
+    process.kill(pid, 0)
+    return true
+  } catch (err) {
+    return err?.code !== "ESRCH"
+  }
+}
+
+/** 異常終了で残った一時ファイルを、作成したプロセスが既に居ない場合だけ消す。 */
+function removeStaleTemps(dir, prefix) {
+  for (const name of readdirSync(dir)) {
+    if (!name.startsWith(prefix)) continue
+    const pid = Number(name.slice(prefix.length).split("-")[0])
+    if (!Number.isSafeInteger(pid) || pid <= 0 || pid === process.pid || processAlive(pid)) continue
+    try { unlinkSync(path.join(dir, name)) } catch {}
+  }
+}
+
+/**
+ * logrotate の copytruncate と同じ方式。fd は launchd が開いたまま使うため rename はしない。
+ * 前提: この fd へ書く writer は同期 write だけ (main.mjs の pino は sync destination)。
+ * 同期区間の中でサイズ取得・コピー・truncate を行うので、同じプロセスの write とは競合しない。
+ * stdout を継承した子プロセスが動いている間は isDeferred で延期する。
+ */
+export function rotateLogIfNeeded(logger, {
+  fd = 1,
+  home = os.homedir(),
+  maxBytes = resolveLogMaxBytes(),
+  isDeferred = () => false,
+} = {}) {
+  try {
+    if (isDeferred()) return false
+    const before = fstatSync(fd)
+    if (!before.isFile()) return false
+    const logPath = path.join(home, ".hub", "agent.log")
+    let target
+    try {
+      target = statSync(logPath)
+    } catch (err) {
+      if (err.code === "ENOENT") return false
+      throw err
+    }
+    if (!sameFile(before, target) || before.size <= maxBytes) return false
+
+    const source = openSync(logPath, "r")
+    try {
+      if (!sameFile(before, fstatSync(source))) return false
+      // 上限値に比例してメモリを確保せず、末尾だけを小さな塊でコピーする。
+      const buffer = Buffer.alloc(64 * 1024)
+      let position = before.size - maxBytes
+      if (readSync(source, buffer, 0, 1, position - 1) !== 1) {
+        throw new Error("log changed before rotation")
+      }
+      let atLineStart = buffer[0] === 10
+      // 旧 .1 はコピー完了まで壊さない。一時ファイルを排他作成して rename で置き換える
+      // (unlink はリンク自体を消すだけで参照先を辿らない。wx は既存のリンクや FIFO を開かない)。
+      const backupPath = `${logPath}.1`
+      const tempPrefix = `${path.basename(backupPath)}.tmp-`
+      removeStaleTemps(path.dirname(logPath), tempPrefix)
+      const tempPath = path.join(
+        path.dirname(logPath),
+        `${tempPrefix}${process.pid}-${randomBytes(6).toString("hex")}`,
+      )
+      const backup = openSync(tempPath, "wx", 0o600)
+      let completed = false
+      let closeError = null
+      try {
+        while (position < before.size) {
+          const length = readSync(source, buffer, 0, Math.min(buffer.length, before.size - position), position)
+          if (length === 0) throw new Error("log changed during rotation")
+          position += length
+          let offset = 0
+          if (!atLineStart) {
+            const newline = buffer.subarray(0, length).indexOf(10)
+            if (newline === -1) continue
+            offset = newline + 1
+            atLineStart = true
+          }
+          while (offset < length) {
+            const written = writeSync(backup, buffer, offset, length - offset)
+            if (written === 0) throw new Error("log backup write made no progress")
+            offset += written
+          }
+        }
+        completed = true
+      } finally {
+        // close が失敗しても未確定の一時ファイルは消す (稼働中は残骸回収の対象外になるため)。
+        try {
+          closeSync(backup)
+        } catch (err) {
+          if (completed) {
+            completed = false
+            closeError = err
+          }
+        }
+        if (!completed) {
+          try { unlinkSync(tempPath) } catch {}
+        }
+      }
+      if (closeError) throw closeError
+      try {
+        renameSync(tempPath, backupPath)
+      } catch (err) {
+        try { unlinkSync(tempPath) } catch {}
+        throw err
+      }
+    } finally {
+      closeSync(source)
+    }
+    ftruncateSync(fd, 0)
+    logger?.info?.({ previousBytes: before.size, maxBytes }, "agent log rotated")
+    return true
+  } catch (err) {
+    // ログ出力自体の失敗も agent を停止させない。
+    try { logger?.warn?.({ error: err.message }, "agent log rotation failed") } catch {}
+    return false
+  }
+}
+
+export function startLogRotation(logger, { schedule = setInterval, ...options } = {}) {
+  const check = () => rotateLogIfNeeded(logger, options)
+  check()
+  const timer = schedule(check, LOG_CHECK_INTERVAL_MS)
+  timer.unref()
+  return timer
+}

package/src/main.mjs

@@ -1,6 +1,7 @@
 import { performance } from "node:perf_hooks"
 import { createHash } from "node:crypto"
 import { logPhaseTiming, startEventLoopMetrics } from "./startup-metrics.mjs"
+import { startLogRotation } from "./log-rotation.mjs"
 /**
  * hub-agent daemon 本体。
  *
@@ -37,6 +38,7 @@ import {
 import {
   codexTerminalSkipVersion,
   compareBundleVersions,
+  hasInheritedOutputChild,
   readCliBundleVersionSync,
   readCliInstalledVersionSync,
   readCodexApprovalStatusSync,
@@ -150,7 +152,10 @@ export function codexAvailableForDispatch(ctx) {
   return (ctx?._resolveCodexBinary || resolveCodexBinary)().available
 }
 
-const logger = pino({ name: "hub-agent" })
+// 同期書き込みにする。既定の非同期 destination は投入済み write の完了を待つ手段が無く
+// (minLength 0 の flush は即時 callback)、agent.log の copy+truncate 回転の間に完了した
+// write が消えるため。
+const logger = pino({ name: "hub-agent" }, pino.destination({ dest: 1, sync: true }))
 
 
 const BUNDLE_MANIFEST_PATH =
@@ -908,6 +913,7 @@ export function wireBridgeEvents(client, { claudeBridge, codexBridge }) {
 }
 
 export async function startDaemon({ version, claudeSdk } = {}) {
+  const logRotationTimer = startLogRotation(logger, { isDeferred: hasInheritedOutputChild })
   startEventLoopMetrics(logger)
   const config = await readConfig()
   if (!config) {
@@ -1190,6 +1196,7 @@ export async function startDaemon({ version, claudeSdk } = {}) {
     client.stop()
     // 0.6.2 fix: watchdog を解放して event loop を抜けられるようにする
     clearInterval(keepaliveTimer)
+    clearInterval(logRotationTimer)
     process.exit(0)
   }
   restartCoordinator = new DeferredRestartCoordinator({