@harmonyos-arkts/opencode-acp 0.0.7 → 0.0.9

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 {
+        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 {
-        const { unlinkSync } = require("fs");
-        unlinkSync(files[i].path);
+        (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) {
@@ -20366,9 +20422,15 @@ function sanitize(obj, depth = 0) {
     } else if (val && typeof val === "object" && !Array.isArray(val)) {
       result[key] = sanitize(val, depth + 1);
     } else if (val && Array.isArray(val)) {
-      result[key] = val.map(
-        (item) => typeof item === "string" && item.length > 2e3 ? item.slice(0, 2e3) + `... [${item.length} chars total]` : item && typeof item === "object" ? sanitize(item, depth + 1) : item
-      );
+      result[key] = val.map((item) => {
+        if (typeof item === "string" && item.length > 2e3) {
+          return item.slice(0, 2e3) + `... [${item.length} chars total]`;
+        }
+        if (item && typeof item === "object") {
+          return sanitize(item, depth + 1);
+        }
+        return item;
+      });
     } else {
       result[key] = val;
     }
@@ -20406,12 +20468,12 @@ function toLocations(toolName, input) {
     case "read":
     case "edit":
     case "write":
-      return input["filePath"] ? [{ path: input["filePath"] }] : [];
+      return input.filePath ? [{ path: input.filePath }] : [];
     case "glob":
     case "grep":
-      return input["path"] ? [{ path: input["path"] }] : [];
+      return input.path ? [{ path: input.path }] : [];
     case "list":
-      return input["path"] ? [{ path: input["path"] }] : [];
+      return input.path ? [{ path: input.path }] : [];
     default:
       return [];
   }
@@ -20436,18 +20498,30 @@ function parseUri(uri) {
     return { type: "text", text: uri };
   }
 }
+function buildUpdateLogPayload(updateType, update) {
+  if (updateType === "agent_message_chunk" || updateType === "agent_thought_chunk") {
+    return { messageId: update.messageId?.slice(0, 12), delta: update.content?.text?.slice(0, 200) };
+  }
+  if (updateType === "tool_call" || updateType === "tool_call_update") {
+    return { toolCallId: update.toolCallId, tool: update.title, kind: update.kind, status: update.status };
+  }
+  if (updateType === "session_info_update") {
+    return { title: update.title, _meta: update._meta };
+  }
+  if (updateType === "usage_update") {
+    return { used: update.used, size: update.size, cost: update.cost, _meta: update._meta, error: update._meta?.error?.message };
+  }
+  if (update._meta?.error) {
+    return { error: update._meta.error?.message ?? update._meta.error };
+  }
+  return {};
+}
 async function sendToClient(connection, params) {
   const updateType = params.update.sessionUpdate;
   const update = params.update;
   acpOut(`sessionUpdate.${updateType}`, {
     sessionId: params.sessionId.slice(0, 12),
-    ...updateType === "agent_message_chunk" || updateType === "agent_thought_chunk" ? { messageId: update.messageId?.slice(0, 12), delta: update.content?.text?.slice(0, 200) } : updateType === "tool_call" || updateType === "tool_call_update" ? { toolCallId: update.toolCallId, tool: update.title, kind: update.kind, status: update.status } : updateType === "session_info_update" ? { title: update.title, _meta: update._meta } : updateType === "usage_update" ? {
-      used: update.used,
-      size: update.size,
-      cost: update.cost,
-      _meta: update._meta,
-      error: update._meta?.error?.message
-    } : update._meta?.error ? { error: update._meta.error?.message ?? update._meta.error } : {}
+    ...buildUpdateLogPayload(updateType, update)
   });
   return connection.sessionUpdate(params);
 }
@@ -20478,9 +20552,19 @@ function normalizeError(err) {
   }
   if (typeof err === "object") {
     const o = err;
-    const message = typeof o.message === "string" && o.message.trim() ? o.message.trim() : typeof o.error === "string" && o.error.trim() ? o.error.trim() : void 0;
+    let message;
+    if (typeof o.message === "string" && o.message.trim()) {
+      message = o.message.trim();
+    } else if (typeof o.error === "string" && o.error.trim()) {
+      message = o.error.trim();
+    }
     if (!message) return void 0;
-    const cause = o.cause instanceof Error ? o.cause.message : typeof o.cause === "object" && o.cause !== null && typeof o.cause.message === "string" ? o.cause.message : void 0;
+    let cause;
+    if (o.cause instanceof Error) {
+      cause = o.cause.message;
+    } else if (typeof o.cause === "object" && o.cause !== null && typeof o.cause.message === "string") {
+      cause = o.cause.message;
+    }
     return {
       code: inferCode(o),
       message,
@@ -20540,7 +20624,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" },
@@ -20702,8 +20785,8 @@ var EventHandler = class {
           }
           if (res.outcome.optionId !== "reject" && permission.permission === "edit") {
             const metadata = permission.metadata || {};
-            const filepath = typeof metadata["filepath"] === "string" ? metadata["filepath"] : "";
-            const diff = typeof metadata["diff"] === "string" ? metadata["diff"] : "";
+            const filepath = typeof metadata.filepath === "string" ? metadata.filepath : "";
+            const diff = typeof metadata.diff === "string" ? metadata.diff : "";
             if (filepath && diff) {
               const fileResult = await this.sdk.file.read({ path: filepath, directory }).then((x) => x.data).catch(() => void 0);
               const original = typeof fileResult?.content === "string" ? fileResult.content : "";
@@ -20756,17 +20839,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;
           });
@@ -20877,9 +20955,8 @@ var EventHandler = class {
         this.fileDiffStats.set(resolved.sessionId, { additions, deletions, files: diff.length });
         return;
       }
-      default: {
+      default:
         return;
-      }
     }
   }
   // ─── Child Session Announcement ──────────────────────────────────
@@ -20978,7 +21055,7 @@ var EventHandler = class {
         }
         this.bashSnapshots.delete(part.callID);
         if (part.tool === "task" && part.state.metadata) {
-          const childSessionId = typeof part.state.metadata["sessionId"] === "string" ? part.state.metadata["sessionId"] : void 0;
+          const childSessionId = typeof part.state.metadata.sessionId === "string" ? part.state.metadata.sessionId : void 0;
           if (childSessionId && this.sessionManager.tryGet(childSessionId)) {
             const tracker = this.childToolCounts.get(childSessionId);
             await this.sendChildSessionFinished(
@@ -21008,9 +21085,14 @@ var EventHandler = class {
         ];
         if (kind === "edit") {
           const input = part.state.input;
-          const filePath = typeof input["filePath"] === "string" ? input["filePath"] : "";
-          const oldText = typeof input["oldString"] === "string" ? input["oldString"] : "";
-          const newText = typeof input["newString"] === "string" ? input["newString"] : typeof input["content"] === "string" ? input["content"] : "";
+          const filePath = typeof input.filePath === "string" ? input.filePath : "";
+          const oldText = typeof input.oldString === "string" ? input.oldString : "";
+          let newText = "";
+          if (typeof input.newString === "string") {
+            newText = input.newString;
+          } else if (typeof input.content === "string") {
+            newText = input.content;
+          }
           content.push({ type: "diff", path: filePath, oldText, newText });
           this.accumulateAICodeChangeStats(sessionId, part);
         }
@@ -21059,7 +21141,7 @@ var EventHandler = class {
         }
         this.bashSnapshots.delete(part.callID);
         if (part.tool === "task" && part.state.metadata) {
-          const childSessionId = typeof part.state.metadata["sessionId"] === "string" ? part.state.metadata["sessionId"] : void 0;
+          const childSessionId = typeof part.state.metadata.sessionId === "string" ? part.state.metadata.sessionId : void 0;
           if (childSessionId && this.sessionManager.tryGet(childSessionId)) {
             const tracker = this.childToolCounts.get(childSessionId);
             const errText = typeof part.state.error === "string" && part.state.error.trim() ? part.state.error.trim() : "Sub-agent task failed";
@@ -21104,6 +21186,8 @@ var EventHandler = class {
         });
         return;
       }
+      default:
+        return;
     }
   }
   /**
@@ -21136,7 +21220,7 @@ var EventHandler = class {
   bashOutput(part) {
     if (part.tool !== "bash") return;
     if (!("metadata" in part.state) || !part.state.metadata || typeof part.state.metadata !== "object") return;
-    const output = part.state.metadata["output"];
+    const output = part.state.metadata.output;
     if (typeof output !== "string") return;
     return output;
   }
@@ -21155,7 +21239,7 @@ var EventHandler = class {
   accumulateAICodeChangeStats(sessionId, part) {
     if (part.state.status !== "completed") return;
     const meta3 = part.state.metadata ?? {};
-    const aiCodeChange = meta3["aiCodeChange"];
+    const aiCodeChange = meta3.aiCodeChange;
     if (!aiCodeChange || aiCodeChange.additions === 0 && aiCodeChange.deletions === 0) return;
     const fileMap = this.aiCodeChangeStats.get(sessionId) ?? /* @__PURE__ */ new Map();
     const existing = fileMap.get(aiCodeChange.file) ?? { additions: 0, deletions: 0 };
@@ -21296,14 +21380,19 @@ function sessionIdFromEvent(event) {
       return typeof props.sessionID === "string" ? props.sessionID : void 0;
     case "session.diff":
     case "session.updated":
-    case "session.created":
-      return typeof props.sessionID === "string" ? props.sessionID : typeof props.info?.id === "string" ? props.info.id : void 0;
+    case "session.created": {
+      if (typeof props.sessionID === "string") return props.sessionID;
+      const info = props.info;
+      if (typeof info?.id === "string") return info.id;
+      return void 0;
+    }
     case "permission.asked": {
       const permission = props;
       return typeof permission.sessionID === "string" ? permission.sessionID : void 0;
     }
+    default:
+      return void 0;
   }
-  return void 0;
 }
 var TurnTimeoutTracker = class {
   constructor(sessionManager) {
@@ -21923,7 +22012,7 @@ var Agent = class {
     const session = this.sessionManager.get(params.sessionId);
     const providers = await this.sdk.config.providers({ directory: session.cwd }).then((x) => x.data?.providers ?? []).catch(() => []);
     const sortedProviders = [...providers].sort(
-      (a, b) => a.name.toLowerCase() < b.name.toLowerCase() ? -1 : a.name.toLowerCase() > b.name.toLowerCase() ? 1 : 0
+      (a, b) => a.name.toLowerCase().localeCompare(b.name.toLowerCase())
     );
     if (params.configId === "model") {
       if (typeof params.value !== "string") throw RequestError.invalidParams("model value must be a string");
@@ -22052,7 +22141,7 @@ var Agent = class {
     const model = await this.defaultModel(params.cwd);
     const providers = await this.sdk.config.providers({ directory: params.cwd }).then((x) => x.data?.providers ?? []).catch(() => []);
     const sortedProviders = [...providers].sort(
-      (a, b) => a.name.toLowerCase() < b.name.toLowerCase() ? -1 : a.name.toLowerCase() > b.name.toLowerCase() ? 1 : 0
+      (a, b) => a.name.toLowerCase().localeCompare(b.name.toLowerCase())
     );
     const availableModels = sortedProviders.flatMap(
       (provider) => Object.values(provider.models).flatMap((modelInfo) => {
@@ -22174,7 +22263,12 @@ var Agent = class {
       if (part.type === "tool") {
         await this.replayToolPart(sessionId, part);
       } else if (part.type === "text" && part.text) {
-        const audience = part.synthetic ? ["assistant"] : part.ignored ? ["user"] : void 0;
+        let audience;
+        if (part.synthetic) {
+          audience = ["assistant"];
+        } else if (part.ignored) {
+          audience = ["user"];
+        }
         await sendToClient(this.connection, {
           sessionId,
           update: {
@@ -22274,11 +22368,17 @@ var Agent = class {
         const content = [{ type: "content", content: { type: "text", text: part.state.output } }];
         if (kind === "edit") {
           const input = part.state.input;
+          let newText = "";
+          if (typeof input.newString === "string") {
+            newText = input.newString;
+          } else if (typeof input.content === "string") {
+            newText = input.content;
+          }
           content.push({
             type: "diff",
-            path: typeof input["filePath"] === "string" ? input["filePath"] : "",
-            oldText: typeof input["oldString"] === "string" ? input["oldString"] : "",
-            newText: typeof input["newString"] === "string" ? input["newString"] : typeof input["content"] === "string" ? input["content"] : ""
+            path: typeof input.filePath === "string" ? input.filePath : "",
+            oldText: typeof input.oldString === "string" ? input.oldString : "",
+            newText
           });
           this.eventHandler.accumulateAICodeChangeStats(sessionId, part);
         }
@@ -22334,6 +22434,8 @@ var Agent = class {
         }).catch(() => {
         });
         break;
+      default:
+        break;
     }
   }
   async sendUsageUpdate(sessionId, directory, turnError) {
@@ -22423,11 +22525,18 @@ var Agent = class {
         } : { ...cs };
       }
     }
-    const totalAICodeChangeStats = parentStats ? childAICodeChangeStats ? {
-      additions: parentStats.additions + childAICodeChangeStats.additions,
-      deletions: parentStats.deletions + childAICodeChangeStats.deletions,
-      files: parentStats.files + childAICodeChangeStats.files
-    } : parentStats : childAICodeChangeStats;
+    let totalAICodeChangeStats;
+    if (parentStats && childAICodeChangeStats) {
+      totalAICodeChangeStats = {
+        additions: parentStats.additions + childAICodeChangeStats.additions,
+        deletions: parentStats.deletions + childAICodeChangeStats.deletions,
+        files: parentStats.files + childAICodeChangeStats.files
+      };
+    } else if (parentStats) {
+      totalAICodeChangeStats = parentStats;
+    } else {
+      totalAICodeChangeStats = childAICodeChangeStats;
+    }
     if (totalAICodeChangeStats) {
       _meta.aiCodeChange = totalAICodeChangeStats;
     }
@@ -22483,6 +22592,8 @@ var Agent = class {
           }
           break;
         }
+        default:
+          break;
       }
     }
     return result;