@kairyou/agent-tools 0.13.1 → 0.13.3

package/dist/log/hook.mjs

@@ -897,21 +897,75 @@ async function main() {
   const statePath = path.join(CACHE_ROOT, `${day}.state.json`);
   const snapshotsRoot = path.join(CACHE_ROOT, `${day}.snapshots`);
   await fs.mkdir(CACHE_ROOT, { recursive: true });
-  await cleanupCache(day);
-  const state = await loadState(statePath, day);
-  await handleEvent(state, hookInput, { now, snapshotsRoot, effective, scopeKey, scopePath: scope?.path || "" });
-  await writeFileAtomic(statePath, `${JSON.stringify(state, null, 2)}
-`);
-  if (effective.format === "detailed") {
-    const report = await renderDetailedReport(state, {
+  const release = await acquireLock(path.join(CACHE_ROOT, `${day}.lock`));
+  if (!release) {
+    await debugLog(`dropped ${eventName}: could not acquire ${day}.lock`);
+    return;
+  }
+  try {
+    await cleanupCache(day);
+    const state = await loadState(statePath, day);
+    await handleEvent(state, hookInput, {
+      now,
       snapshotsRoot,
-      language: effective.language,
-      scopeKey
+      effective,
+      scopeKey,
+      scopePath: scope?.path || ""
     });
-    await fs.mkdir(effective.output, { recursive: true });
-    await writeFileAtomic(path.join(effective.output, `${day}.md`), report);
-  } else {
-    await updateDailyFile(effective.output, day, buildDailyItems(state, scopeKey));
+    await writeFileAtomic(statePath, `${JSON.stringify(state, null, 2)}
+`);
+    if (effective.format === "detailed") {
+      const report = await renderDetailedReport(state, {
+        snapshotsRoot,
+        language: effective.language,
+        scopeKey
+      });
+      await fs.mkdir(effective.output, { recursive: true });
+      await writeFileAtomic(path.join(effective.output, `${day}.md`), report);
+    } else {
+      await updateDailyFile(effective.output, day, buildDailyItems(state, scopeKey));
+    }
+  } finally {
+    await release();
+  }
+}
+var LOCK_TIMEOUT_MS = 5e3;
+var LOCK_RETRY_MS = 25;
+var LOCK_STALE_MS = 3e4;
+async function acquireLock(lockPath) {
+  const deadline = Date.now() + LOCK_TIMEOUT_MS;
+  while (Date.now() < deadline) {
+    try {
+      const handle = await fs.open(lockPath, "wx");
+      await handle.close();
+      return async () => {
+        try {
+          await fs.unlink(lockPath);
+        } catch {
+        }
+      };
+    } catch (error) {
+      if (error?.code !== "EEXIST") return null;
+      const age = await fs.stat(lockPath).then(({ mtimeMs }) => Date.now() - mtimeMs).catch(() => 0);
+      if (age > LOCK_STALE_MS) {
+        await fs.unlink(lockPath).catch(() => {
+        });
+        continue;
+      }
+      await sleep(LOCK_RETRY_MS);
+    }
+  }
+  return null;
+}
+function sleep(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
+async function debugLog(message) {
+  if (process.env.AGENT_TOOLS_LOG_DEBUG !== "1") return;
+  try {
+    await fs.appendFile(path.join(CACHE_ROOT, "debug.log"), `${(/* @__PURE__ */ new Date()).toISOString()} ${message}
+`);
+  } catch {
   }
 }
 function eventCwd(input) {
@@ -980,6 +1034,11 @@ async function handleEvent(state, input, context) {
   const session = ensureSession(state, sessionId, timestamp);
   session.last_time = timestamp;
   if (eventName === "UserPromptSubmit") {
+    const last = session.turns[session.turns.length - 1];
+    if (last && !last.request_text && last.result_summary) {
+      last.request_text = excerptMultiline(getPromptText(input), 1600);
+      return;
+    }
     session.turns.push(
       createTurn(session, timestamp, getPromptText(input), resolveProjectLabel(input, context.scopePath), context.scopeKey)
     );
@@ -1028,7 +1087,7 @@ async function cleanupCache(day) {
   }
   await Promise.all(
     entries.map(async (entry) => {
-      const isState = entry.isFile() && /^\d{4}-\d{2}-\d{2}\.state\.json$/.test(entry.name);
+      const isState = entry.isFile() && /^\d{4}-\d{2}-\d{2}\.(state\.json|lock)$/.test(entry.name);
       const isSnapshots = entry.isDirectory() && /^\d{4}-\d{2}-\d{2}\.snapshots$/.test(entry.name);
       if (!isState && !isSnapshots || entry.name.startsWith(day)) return;
       try {
@@ -1182,9 +1241,9 @@ function dailyItemText(turn) {
   const outcome = String(turn.result_summary || "");
   if (!hasSubstantiveTurn(request, outcome) || isTrivialTurn(request, outcome)) return "";
   const source = outcome || request;
-  const firstLine = source.split("\n").map((line) => line.replace(/^[#>*\-\s`]+/, "").trim()).find((line) => line.length > 0);
-  if (!firstLine) return "";
-  return firstLine.length > DAILY_ITEM_MAX_CHARS ? `${firstLine.slice(0, DAILY_ITEM_MAX_CHARS - 3)}...` : firstLine;
+  const flattened = source.split("\n").map((line) => line.replace(/^[#>*\-\s`|]+/, "").trim()).filter(Boolean).join(" ");
+  if (!flattened) return "";
+  return flattened.length > DAILY_ITEM_MAX_CHARS ? `${flattened.slice(0, DAILY_ITEM_MAX_CHARS - 3)}...` : flattened;
 }
 async function updateDailyFile(outputFile, day, items) {
   if (items.length === 0) return;
@@ -1528,7 +1587,19 @@ function getShellCommand(input) {
 async function writeFileAtomic(file, text) {
   const temp = `${file}.${process.pid}.tmp`;
   await fs.writeFile(temp, text, "utf8");
-  await fs.rename(temp, file);
+  for (let attempt = 0; ; attempt += 1) {
+    try {
+      await fs.rename(temp, file);
+      return;
+    } catch (error) {
+      if (attempt >= 5 || !["EPERM", "EACCES", "EBUSY"].includes(error?.code)) {
+        await fs.unlink(temp).catch(() => {
+        });
+        throw error;
+      }
+      await sleep(20 * (attempt + 1));
+    }
+  }
 }
 function excerptMultiline(text, limit) {
   const value = String(text || "").replace(/\r\n/g, "\n").trim();

package/docs/en/extras.md

@@ -78,14 +78,15 @@ npx -y @kairyou/agent-tools@latest log -a claude codex opencode
 ```markdown
 + 2026-08-03
 <!-- log:2026-08-03:start -->
-  1. project-a: traced the login timeout to sessions never renewing.
-  2. project-a: fixed the login timeout, added regression tests.
-  3. project-b: traced empty report exports to an inverted permission filter.
+  1. project-a: Found the cause of the login timeout: the renewal branch never updated the cache expiry, so the second request still read the old value. Patched session.ts and verified...
+  2. project-a: Fix is in, with 3 regression tests covering the renewal path; all passing.
+  3. project-b: Empty report exports came from an inverted permission filter, now corrected and confirmed working.
 <!-- log:2026-08-03:end -->
 ```
 
-One line per turn, written from the AI's own summary at the end of that turn; no
-cross-turn consolidation. Updates likewise rewrite only what sits between the markers.
+One line per turn, taken verbatim from the closing text of that turn's AI reply
+(truncated when long); no distilling, no cross-turn consolidation. Updates likewise
+rewrite only what sits between the markers.
 
 `detailed` output example (excerpt):
 

package/docs/zh-CN/extras.md

@@ -76,14 +76,14 @@ npx -y @kairyou/agent-tools@latest log -a claude codex opencode
 ```markdown
 + 2026-08-03
 <!-- log:2026-08-03:start -->
-  1. project-a: 排查登录超时, 定位到会话缓存没有续期.
-  2. project-a: 修复登录超时, 补充回归用例.
-  3. project-b: 定位报表导出为空的问题, 原因是权限过滤条件写反.
+  1. project-a: 已定位登录超时的原因: 会话缓存在续期分支上没有更新过期时间, 第二次请求拿到的还是旧值。已经在 session.ts 补上续期并本地验证通过, 接下来...
+  2. project-a: 修复完成, 新增 3 个回归用例覆盖续期路径, 全部通过。
+  3. project-b: 报表导出为空定位到权限过滤条件写反, 已修正并确认导出恢复正常。
 <!-- log:2026-08-03:end -->
 ```
 
-每轮一行, 内容是那一轮结束时 AI 自己写的总结, 不做跨轮归纳; 同样只重写标记之间
-的部分, 文件里的其他内容不会被碰.
+每轮一行, 直接摘录那一轮 AI 回复的收尾内容(过长会截断), 不做提炼也不跨轮归纳;
+同样只重写标记之间的部分, 文件里的其他内容不会被碰.
 
 `detailed` 输出示例(节选):
 

package/integrations/log/hook.mjs

@@ -13,12 +13,10 @@
 //             recorded, and an entry may override format/output/language
 //
 // State and diff snapshots live in ~/.agent-tools/cache/log/ and only the
-// current day is kept. Concurrent sessions share the day state without a lock,
-// last-writer-wins: a simultaneous write from another session can drop that
-// event's update, up to a whole turn with its outcome and file records.
-// Accepted as best effort — the log is generated data, never user content.
-// The opencode adapter serializes its own sends, so single-session events
-// there never race each other.
+// current day is kept. Sessions across agents share that state, so each run
+// takes ~/.agent-tools/cache/log/<date>.lock for the whole read-update-render
+// cycle; an event that cannot get the lock within seconds is dropped rather
+// than corrupting the day.
 
 import { spawnSync } from "node:child_process";
 import fs from "node:fs/promises";
@@ -67,22 +65,89 @@ async function main() {
   const snapshotsRoot = path.join(CACHE_ROOT, `${day}.snapshots`);
 
   await fs.mkdir(CACHE_ROOT, { recursive: true });
-  await cleanupCache(day);
 
-  const state = await loadState(statePath, day);
-  await handleEvent(state, hookInput, { now, snapshotsRoot, effective, scopeKey, scopePath: scope?.path || "" });
-  await writeFileAtomic(statePath, `${JSON.stringify(state, null, 2)}\n`);
-
-  if (effective.format === "detailed") {
-    const report = await renderDetailedReport(state, {
+  // Read, update and render under one lock: concurrent sessions across agents
+  // otherwise overwrite each other's turns, and on Windows their simultaneous
+  // renames fail outright with EPERM.
+  const release = await acquireLock(path.join(CACHE_ROOT, `${day}.lock`));
+  if (!release) {
+    await debugLog(`dropped ${eventName}: could not acquire ${day}.lock`);
+    return;
+  }
+  try {
+    await cleanupCache(day);
+    const state = await loadState(statePath, day);
+    await handleEvent(state, hookInput, {
+      now,
       snapshotsRoot,
-      language: effective.language,
+      effective,
       scopeKey,
+      scopePath: scope?.path || "",
     });
-    await fs.mkdir(effective.output, { recursive: true });
-    await writeFileAtomic(path.join(effective.output, `${day}.md`), report);
-  } else {
-    await updateDailyFile(effective.output, day, buildDailyItems(state, scopeKey));
+    await writeFileAtomic(statePath, `${JSON.stringify(state, null, 2)}\n`);
+
+    if (effective.format === "detailed") {
+      const report = await renderDetailedReport(state, {
+        snapshotsRoot,
+        language: effective.language,
+        scopeKey,
+      });
+      await fs.mkdir(effective.output, { recursive: true });
+      await writeFileAtomic(path.join(effective.output, `${day}.md`), report);
+    } else {
+      await updateDailyFile(effective.output, day, buildDailyItems(state, scopeKey));
+    }
+  } finally {
+    await release();
+  }
+}
+
+// ---- Cross-process lock. ----
+
+const LOCK_TIMEOUT_MS = 5_000;
+const LOCK_RETRY_MS = 25;
+// A hook run is short; a lock older than this belongs to a process that died.
+const LOCK_STALE_MS = 30_000;
+
+async function acquireLock(lockPath) {
+  const deadline = Date.now() + LOCK_TIMEOUT_MS;
+  while (Date.now() < deadline) {
+    try {
+      const handle = await fs.open(lockPath, "wx");
+      await handle.close();
+      return async () => {
+        try {
+          await fs.unlink(lockPath);
+        } catch {
+          // Already reclaimed as stale by another process.
+        }
+      };
+    } catch (error) {
+      if (error?.code !== "EEXIST") return null;
+      const age = await fs
+        .stat(lockPath)
+        .then(({ mtimeMs }) => Date.now() - mtimeMs)
+        .catch(() => 0);
+      if (age > LOCK_STALE_MS) {
+        await fs.unlink(lockPath).catch(() => {});
+        continue;
+      }
+      await sleep(LOCK_RETRY_MS);
+    }
+  }
+  return null;
+}
+
+function sleep(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
+
+async function debugLog(message) {
+  if (process.env.AGENT_TOOLS_LOG_DEBUG !== "1") return;
+  try {
+    await fs.appendFile(path.join(CACHE_ROOT, "debug.log"), `${new Date().toISOString()} ${message}\n`);
+  } catch {
+    // Diagnostics must never break logging.
   }
 }
 
@@ -170,9 +235,16 @@ async function handleEvent(state, input, context) {
   session.last_time = timestamp;
 
   if (eventName === "UserPromptSubmit") {
-    // Always start a turn, even for greetings: reusing the previous turn would
-    // let this prompt's Stop overwrite the previous result summary. Trivial
-    // turns are filtered at render time instead.
+    const last = session.turns[session.turns.length - 1];
+    // A Stop that arrived before its prompt left a turn holding only an
+    // outcome; fill this prompt in rather than splitting one exchange in two.
+    if (last && !last.request_text && last.result_summary) {
+      last.request_text = excerptMultiline(getPromptText(input), 1600);
+      return;
+    }
+    // Otherwise always start a turn, even for greetings: reusing the previous
+    // turn would let this prompt's Stop overwrite the previous result summary.
+    // Trivial turns are filtered at render time instead.
     session.turns.push(
       createTurn(session, timestamp, getPromptText(input), resolveProjectLabel(input, context.scopePath), context.scopeKey)
     );
@@ -231,7 +303,7 @@ async function cleanupCache(day) {
   }
   await Promise.all(
     entries.map(async (entry) => {
-      const isState = entry.isFile() && /^\d{4}-\d{2}-\d{2}\.state\.json$/.test(entry.name);
+      const isState = entry.isFile() && /^\d{4}-\d{2}-\d{2}\.(state\.json|lock)$/.test(entry.name);
       const isSnapshots = entry.isDirectory() && /^\d{4}-\d{2}-\d{2}\.snapshots$/.test(entry.name);
       if ((!isState && !isSnapshots) || entry.name.startsWith(day)) return;
       try {
@@ -427,14 +499,17 @@ function dailyItemText(turn) {
   const outcome = String(turn.result_summary || "");
   if (!hasSubstantiveTurn(request, outcome) || isTrivialTurn(request, outcome)) return "";
   const source = outcome || request;
-  const firstLine = source
+  // Flattened rather than first-line: a structured summary often opens with a
+  // preamble line, and the substance sits in the lines after it.
+  const flattened = source
     .split("\n")
-    .map((line) => line.replace(/^[#>*\-\s`]+/, "").trim())
-    .find((line) => line.length > 0);
-  if (!firstLine) return "";
-  return firstLine.length > DAILY_ITEM_MAX_CHARS
-    ? `${firstLine.slice(0, DAILY_ITEM_MAX_CHARS - 3)}...`
-    : firstLine;
+    .map((line) => line.replace(/^[#>*\-\s`|]+/, "").trim())
+    .filter(Boolean)
+    .join(" ");
+  if (!flattened) return "";
+  return flattened.length > DAILY_ITEM_MAX_CHARS
+    ? `${flattened.slice(0, DAILY_ITEM_MAX_CHARS - 3)}...`
+    : flattened;
 }
 
 async function updateDailyFile(outputFile, day, items) {
@@ -859,10 +934,23 @@ function getShellCommand(input) {
 
 // ---- Small helpers. ----
 
+// Windows fails the rename with EPERM when another process holds the target
+// open, so a lost race retries briefly instead of dropping the write.
 async function writeFileAtomic(file, text) {
   const temp = `${file}.${process.pid}.tmp`;
   await fs.writeFile(temp, text, "utf8");
-  await fs.rename(temp, file);
+  for (let attempt = 0; ; attempt += 1) {
+    try {
+      await fs.rename(temp, file);
+      return;
+    } catch (error) {
+      if (attempt >= 5 || !["EPERM", "EACCES", "EBUSY"].includes(error?.code)) {
+        await fs.unlink(temp).catch(() => {});
+        throw error;
+      }
+      await sleep(20 * (attempt + 1));
+    }
+  }
 }
 
 function excerptMultiline(text, limit) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kairyou/agent-tools",
-  "version": "0.13.1",
+  "version": "0.13.3",
   "description": "Reusable Agent Skills, plus integrations (statusline, provider usage, vision) that install into Codex, Claude Code, and opencode.",
   "license": "MIT",
   "repository": {