@harmonyos-arkts/opencode-acp 0.0.7 → 0.0.8

package/CHANGELOG.md

@@ -6,6 +6,11 @@ All notable changes to this project will be documented in this file.
 
 ### Added
 
+- 日志老化机制：基于 harmony-code 的日志管理策略
+  - 文件大小轮转：单个日志文件超过 5MB 时自动重命名（加时间戳后缀）
+  - 存留期清理：启动时删除超过 7 天的日志文件
+  - 数量上限：最多保留 30 个日志文件
+  - 缓冲写入：日志条目缓冲后每 500ms 或每 50 条批量刷盘，减少 I/O 开销
 - MCP extMethod API：通过 ACP `extMethod("mcp/*")` 向客户端暴露 MCP 服务器运行时管理能力
   - `mcp/status` — 获取所有 MCP 服务器连接状态
   - `mcp/add` — 动态添加 MCP 服务器（local/remote）

package/README.md

@@ -128,7 +128,12 @@ Options:
 
 ## Traffic Logging
 
-All communication is logged to `~/.harmony-acp/logs/` as JSONL files (auto-rotated, max 30 files).
+All communication is logged to `~/.harmony-acp/logs/` as JSONL files with automatic rotation and aging:
+
+- **Size rotation**: log files are renamed with a timestamp when they exceed 5MB
+- **Age cleanup**: files older than 7 days are deleted on startup
+- **Count cap**: at most 30 log files are retained
+- **Buffered writes**: log entries are buffered and flushed every 500ms or every 50 entries for performance
 
 ### Log Categories
 

package/dist/index.cjs

@@ -374,15 +374,15 @@ var require_readShebang = __commonJS({
     var shebangCommand = require_shebang_command();
     function readShebang(command) {
       const size = 150;
-      const buffer = Buffer.alloc(size);
+      const buffer2 = Buffer.alloc(size);
       let fd;
       try {
         fd = fs.openSync(command, "r");
-        fs.readSync(fd, buffer, 0, size, 0);
+        fs.readSync(fd, buffer2, 0, size, 0);
         fs.closeSync(fd);
       } catch (e) {
       }
-      return shebangCommand(buffer.toString());
+      return shebangCommand(buffer2.toString());
     }
     module2.exports = readShebang;
   }
@@ -15846,7 +15846,7 @@ var createSseClient = ({ onRequest, onSseError, onSseEvent, responseTransformer,
         if (!response.body)
           throw new Error("No body in SSE response");
         const reader = response.body.pipeThrough(new TextDecoderStream()).getReader();
-        let buffer = "";
+        let buffer2 = "";
         const abortHandler = () => {
           try {
             reader.cancel();
@@ -15859,10 +15859,10 @@ var createSseClient = ({ onRequest, onSseError, onSseEvent, responseTransformer,
             const { done, value } = await reader.read();
             if (done)
               break;
-            buffer += value;
-            buffer = buffer.replace(/\r\n/g, "\n").replace(/\r/g, "\n");
-            const chunks = buffer.split("\n\n");
-            buffer = chunks.pop() ?? "";
+            buffer2 += value;
+            buffer2 = buffer2.replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+            const chunks = buffer2.split("\n\n");
+            buffer2 = chunks.pop() ?? "";
             for (const chunk of chunks) {
               const lines = chunk.split("\n");
               const dataLines = [];
@@ -20298,9 +20298,15 @@ var import_fs = require("fs");
 var import_path = require("path");
 var import_os = require("os");
 var LOG_DIR = (0, import_path.join)((0, import_os.homedir)(), ".harmony-acp", "logs");
+var MAX_LOG_SIZE_BYTES = 5 * 1024 * 1024;
+var MAX_LOG_AGE_DAYS = 7;
 var MAX_LOG_FILES = 30;
+var FLUSH_INTERVAL_MS = 500;
+var BUFFER_SIZE_LIMIT = 50;
 var logFile = "";
 var enabled = false;
+var buffer = [];
+var flushTimer = null;
 function initLogger() {
   const ts = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
   logFile = (0, import_path.join)(LOG_DIR, `${ts}.log`);
@@ -20310,24 +20316,72 @@ function initLogger() {
 }
 function setLogEnabled(v) {
   enabled = v;
+  if (!v) flush();
 }
 function cleanOldLogs() {
   try {
-    const files = (0, import_fs.readdirSync)(LOG_DIR).filter((f) => f.endsWith(".log")).map((f) => ({
-      name: f,
-      path: (0, import_path.join)(LOG_DIR, f),
-      mtime: (0, import_fs.statSync)((0, import_path.join)(LOG_DIR, f)).mtime.getTime()
-    })).sort((a, b) => b.mtime - a.mtime);
-    for (let i = MAX_LOG_FILES; i < files.length; i++) {
+    const now = Date.now();
+    const maxAgeMs = MAX_LOG_AGE_DAYS * 24 * 60 * 60 * 1e3;
+    const files = (0, import_fs.readdirSync)(LOG_DIR).filter((f) => f.endsWith(".log")).map((f) => {
+      const full = (0, import_path.join)(LOG_DIR, f);
       try {
-        const { unlinkSync } = require("fs");
-        unlinkSync(files[i].path);
+        return { name: f, path: full, mtime: (0, import_fs.statSync)(full).mtime.getTime() };
+      } catch {
+        return null;
+      }
+    }).filter((e) => e !== null);
+    for (const f of files) {
+      if (now - f.mtime > maxAgeMs) {
+        try {
+          (0, import_fs.unlinkSync)(f.path);
+        } catch {
+        }
+      }
+    }
+    const surviving = files.filter((f) => now - f.mtime <= maxAgeMs).sort((a, b) => b.mtime - a.mtime);
+    for (let i = MAX_LOG_FILES; i < surviving.length; i++) {
+      try {
+        (0, import_fs.unlinkSync)(surviving[i].path);
       } catch {
       }
     }
   } catch {
   }
 }
+function rotateIfNeeded() {
+  try {
+    const stat = (0, import_fs.statSync)(logFile);
+    if (stat.size >= MAX_LOG_SIZE_BYTES) {
+      const now = /* @__PURE__ */ new Date();
+      const pad = (n) => String(n).padStart(2, "0");
+      const ts = `${now.getFullYear()}${pad(now.getMonth() + 1)}${pad(now.getDate())}-${pad(now.getHours())}${pad(now.getMinutes())}${pad(now.getSeconds())}`;
+      const baseName = logFile.replace(/\.log$/, "");
+      (0, import_fs.renameSync)(logFile, `${baseName}.${ts}.log`);
+    }
+  } catch {
+  }
+}
+function flush() {
+  if (flushTimer) {
+    clearTimeout(flushTimer);
+    flushTimer = null;
+  }
+  if (buffer.length === 0) return;
+  const data = buffer.join("");
+  buffer = [];
+  try {
+    (0, import_fs.appendFileSync)(logFile, data);
+    rotateIfNeeded();
+  } catch {
+  }
+}
+function scheduleFlush() {
+  if (flushTimer) return;
+  flushTimer = setTimeout(() => {
+    flushTimer = null;
+    flush();
+  }, FLUSH_INTERVAL_MS);
+}
 function write(category, action, data) {
   if (!enabled) return;
   const entry = {
@@ -20336,9 +20390,11 @@ function write(category, action, data) {
     action,
     ...data && { data }
   };
-  try {
-    (0, import_fs.appendFileSync)(logFile, JSON.stringify(entry) + "\n");
-  } catch {
+  buffer.push(JSON.stringify(entry) + "\n");
+  if (buffer.length >= BUFFER_SIZE_LIMIT) {
+    flush();
+  } else {
+    scheduleFlush();
   }
 }
 function acpIn(method, params) {
@@ -20540,7 +20596,6 @@ async function emitSessionError(connection, sessionId, payload, options) {
 // src/event-handler.ts
 var SUBSCRIPTION_RETRY_MS = 3e3;
 var SUBSCRIPTION_ERROR_BROADCAST_MS = 3e4;
-var QUESTION_TIMEOUT_MS = 3e5;
 var permissionOptions = [
   { optionId: "once", kind: "allow_once", name: "Allow once" },
   { optionId: "always", kind: "allow_always", name: "Always allow" },
@@ -20756,17 +20811,12 @@ var EventHandler = class {
         const sessionId = resolved.sessionId;
         const prev = this.questionQueues.get(q.sessionID) ?? Promise.resolve();
         const next = prev.then(async () => {
-          const extResult = await Promise.race([
-            this.connection.extMethod("questionAsked", {
-              sessionId,
-              questionId: q.id,
-              questions: q.questions,
-              ...q.tool && { tool: q.tool }
-            }),
-            new Promise(
-              (_, reject) => setTimeout(() => reject(new Error("questionAsked timeout")), QUESTION_TIMEOUT_MS)
-            )
-          ]).catch((err) => {
+          const extResult = await this.connection.extMethod("questionAsked", {
+            sessionId,
+            questionId: q.id,
+            questions: q.questions,
+            ...q.tool && { tool: q.tool }
+          }).catch((err) => {
             console.error("[event-handler] extMethod questionAsked failed:", err?.message ?? err);
             return void 0;
           });