@lark-codem/codem-cli 0.1.237 → 0.1.238

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lark-codem/codem-cli",
-  "version": "0.1.237",
+  "version": "0.1.238",
   "codemServiceManagementProtocol": 3,
   "codemProfileProtocol": 1,
   "type": "module",
@@ -29,16 +29,16 @@
     "access": "public"
   },
   "dependencies": {
-    "@lark-codem/codem-core": "0.8.69",
+    "@lark-codem/codem-core": "0.8.70",
     "zod": "3.25.76"
   },
   "optionalDependencies": {
-    "@lark-codem/codem-cli-darwin-arm64": "0.1.237",
-    "@lark-codem/codem-cli-darwin-x64": "0.1.237",
-    "@lark-codem/codem-cli-linux-arm64": "0.1.237",
-    "@lark-codem/codem-cli-linux-x64": "0.1.237",
-    "@lark-codem/codem-cli-win32-arm64": "0.1.237",
-    "@lark-codem/codem-cli-win32-x64": "0.1.237"
+    "@lark-codem/codem-cli-darwin-arm64": "0.1.238",
+    "@lark-codem/codem-cli-darwin-x64": "0.1.238",
+    "@lark-codem/codem-cli-linux-arm64": "0.1.238",
+    "@lark-codem/codem-cli-linux-x64": "0.1.238",
+    "@lark-codem/codem-cli-win32-arm64": "0.1.238",
+    "@lark-codem/codem-cli-win32-x64": "0.1.238"
   },
   "engines": {
     "node": ">=18"

package/profile/config-lock.js

@@ -61,6 +61,9 @@ import { debugLog } from "./debug.js";
 //   ——PID 复用等边界最多让锁延迟释放，绝不以破坏互斥换可用性。回收者通过
 //   `<lock>.reap/` 下的唯一候选做 Bakery 选举，把 token 复核与 unlink 串成单回收者
 //   临界区；候选绑定 PID，进程崩溃后可被后续回收者安全清理。
+// - tmp 清扫：获取时顺带扫描同目录的 `<lock>.tmp-*` 兄弟，仅回收「可证明无主」
+//   （body 可读 + 本机 host + kill(pid,0) ESRCH）的残留——等待窗口内被 SIGKILL/Ctrl+C/
+//   断电终止的进程留下的 tmp 由此收敛，body 读不出的一律不动（见 sweepAbandonedTmpFiles）。
 //
 //   ⚠️ 明确不做「回收 0 字节锁」的自愈：新代码永不产生 0 字节锁，但 rolling upgrade 期间
 //   旧版进程仍可能瞬时留下 0 字节锁，且无法与旧版崩溃残骸区分——回收它就会误删活持有者。
@@ -157,6 +160,54 @@ async function readOwner(filePath) {
   }
 }
 
+// 机会性清扫残留 tmp（Bug 7384601509）：tmp 的生命周期原本只挂在「获取进程走完所有
+// 出口」上——等待窗口内被 SIGINT/SIGTERM/SIGKILL/崩溃/断电终止的进程，其 tmp 因文件名
+// 内嵌 randomUUID 永不复用而永久残留。这里在每次获取时顺带回收同目录的 `<lock>.tmp-*`
+// 兄弟文件，判定与锁文件回收完全同一不变量：**可证明无主才删**（body 可读 + host 为
+// 本机 + isPidAlive 判定 PID 不存在）。body 读不出的 tmp 一律不动（可能是正在写的活
+// 等待者，误删会让其 fs.link 失败于 ENOENT 而不是 EEXIST，错误直接抛给调用方）；异机
+// host 不动（共享 HOME 场景不替别机做主）。清扫 best-effort：任何失败只留 debugLog，
+// 绝不影响获取主流程。
+async function sweepAbandonedTmpFiles(lockFile, { isPidAlive, ownTmp }) {
+  const dir = path.dirname(lockFile);
+  const prefix = `${path.basename(lockFile)}.tmp-`;
+  let entries;
+  try {
+    entries = await fs.readdir(dir, { withFileTypes: true });
+  } catch (error) {
+    if (error?.code !== "ENOENT") {
+      debugLog("config-lock.tmp_sweep_failed", { lockFile, code: error?.code, message: error?.message });
+    }
+    return;
+  }
+  for (const entry of entries) {
+    if (!entry.isFile()) continue;
+    const name = entry.name;
+    // ownTmp 显式跳过：调用方可能注入「一律判死」的 isPidAlive 测试桩，靠探活区分
+    // 不了自己的 tmp；误删自己的 tmp 会让 fs.link 失败于 ENOENT。
+    if (!name.startsWith(prefix) || name === ownTmp) continue;
+    const sibling = path.join(dir, name);
+    const owner = await readOwner(sibling);
+    if (!owner) continue; // body 读不出：无法证明无主，不动
+    if (owner.host !== os.hostname()) continue; // 异机无法探活，不动
+    // process.kill 接受正的 int32 PID；范围外的元数据不能作为死亡证明。
+    if (!Number.isInteger(owner.pid) || owner.pid <= 0 || owner.pid > 0x7fff_ffff) continue;
+    try {
+      if (isPidAlive(owner.pid)) continue; // 存活（含僵尸/PID 复用）一律不删
+    } catch (error) {
+      debugLog("config-lock.tmp_sweep_failed", { tmpFile: sibling, code: error?.code, message: error?.message });
+      continue;
+    }
+    try {
+      await fs.rm(sibling, { force: true });
+    } catch (error) {
+      debugLog("config-lock.tmp_remove_failed", { tmpFile: sibling, code: error?.code, message: error?.message });
+      continue;
+    }
+    debugLog("config-lock.tmp_swept", { tmpFile: sibling, pid: owner.pid });
+  }
+}
+
 /**
  * 获取 `<targetPath>.lock` 跨进程锁。返回一个幂等的、ownership-checked 的 `release()`。
  *
@@ -196,6 +247,10 @@ export async function acquireConfigLock(targetPath, {
   }
 
   try {
+    // 清扫处于自己的 tmp 清理保护区内，且失败不阻断锁获取。
+    await sweepAbandonedTmpFiles(lockFile, { isPidAlive, ownTmp: path.basename(tmpFile) }).catch((error) => {
+      debugLog("config-lock.tmp_sweep_failed", { lockFile, code: error?.code, message: error?.message });
+    });
     for (;;) {
       // 原子上位：lockFile 要么指向这个已写满的 inode，要么 EEXIST。link 之后
       // lockFile 与 tmpFile 是同一 inode 的两个名字，删 tmp 不影响 lockFile。
@@ -255,7 +310,11 @@ export async function acquireConfigLock(targetPath, {
   } finally {
     // tmp 在所有出口（成功 return / 超时 throw / 其它 error）都不再需要：成功时
     // lockFile 已是独立的第二个名字，删 tmp 只是丢一个多余的 link，inode 由 lockFile 存续。
-    await fs.rm(tmpFile, { force: true }).catch(() => {});
+    // 删失败不再静默吞掉（残留 tmp 正是 Bug 7384601509 的累积源之一），与
+    // config-lock.hardlink_unsupported 同风格留痕，便于事后定位。
+    await fs.rm(tmpFile, { force: true }).catch((error) => {
+      debugLog("config-lock.tmp_remove_failed", { tmpFile, code: error?.code, message: error?.message });
+    });
   }
 }