jinzd-ai-cli 0.4.65 → 0.4.67

package/dist/{chunk-SKN6HBBT.js → chunk-BVLQ3FRA.js}

@@ -10,7 +10,7 @@ import {
   SUBAGENT_DEFAULT_MAX_ROUNDS,
   SUBAGENT_MAX_ROUNDS_LIMIT,
   runTestsTool
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 
 // src/tools/builtin/bash.ts
 import { execSync } from "child_process";
@@ -793,6 +793,11 @@ import { dirname as dirname2 } from "path";
 import chalk3 from "chalk";
 import { existsSync as existsSync4, readFileSync as readFileSync3 } from "fs";
 
+// src/core/readline-internal.ts
+function rlInternal(rl) {
+  return rl;
+}
+
 // src/tools/types.ts
 function isFileWriteTool(name) {
   return name === "write_file" || name === "edit_file" || name === "notebook_edit";
@@ -842,6 +847,51 @@ function schemaToJsonSchema(schema) {
   return result;
 }
 
+// src/tools/executor-phases.ts
+function groupCallsByPhase(calls) {
+  const safeParallel = [];
+  const safeBash = [];
+  const fileWriteCalls = [];
+  const otherCalls = [];
+  for (let i = 0; i < calls.length; i++) {
+    const call = calls[i];
+    const level = getDangerLevel(call.name, call.arguments);
+    if (level === "safe") {
+      if (call.name === "bash") {
+        safeBash.push({ idx: i, call });
+      } else {
+        safeParallel.push({ idx: i, call });
+      }
+    } else if (isFileWriteTool(call.name) && level === "write") {
+      fileWriteCalls.push({ idx: i, call });
+    } else {
+      otherCalls.push({ idx: i, call });
+    }
+  }
+  return { safeParallel, safeBash, fileWriteCalls, otherCalls };
+}
+async function runSafePhases(phase, execute, results, logTag = "executor") {
+  const t0 = Date.now();
+  const { safeParallel, safeBash } = phase;
+  const wrapExec = async (idx, call) => {
+    try {
+      results[idx] = await execute(call);
+    } catch (err) {
+      const msg = err instanceof Error ? err.message : String(err);
+      console.error(`[${logTag}] Unexpected error in tool "${call.name}":`, err);
+      results[idx] = { callId: call.id, content: `Tool execution failed: ${msg}`, isError: true };
+    }
+  };
+  const parallelPhase = safeParallel.length > 0 ? Promise.all(safeParallel.map(({ idx, call }) => wrapExec(idx, call))) : Promise.resolve();
+  const bashPhase = (async () => {
+    for (const { idx, call } of safeBash) {
+      await wrapExec(idx, call);
+    }
+  })();
+  await Promise.all([parallelPhase, bashPhase]);
+  return Date.now() - t0;
+}
+
 // src/tools/diff-utils.ts
 import chalk from "chalk";
 function renderDiff(oldText, newText, opts = {}) {
@@ -1365,57 +1415,22 @@ var ToolExecutor = class {
     }
   }
   async executeAll(calls) {
-    const safeParallel = [];
-    const safeBash = [];
-    const fileWriteCalls = [];
-    const otherCalls = [];
-    for (let i = 0; i < calls.length; i++) {
-      const call = calls[i];
-      const level = getDangerLevel(call.name, call.arguments);
-      if (level === "safe") {
-        if (call.name === "bash") {
-          safeBash.push({ idx: i, call });
-        } else {
-          safeParallel.push({ idx: i, call });
-        }
-      } else if (isFileWriteTool(call.name) && level === "write") {
-        fileWriteCalls.push({ idx: i, call });
-      } else {
-        otherCalls.push({ idx: i, call });
-      }
-    }
+    const phase = groupCallsByPhase(calls);
     const results = new Array(calls.length);
-    const t0 = Date.now();
-    const wrapExec = async (idx, call) => {
-      try {
-        results[idx] = await this.execute(call);
-      } catch (err) {
-        const msg = err instanceof Error ? err.message : String(err);
-        console.error(`[executor] Unexpected error in tool "${call.name}":`, err);
-        results[idx] = { callId: call.id, content: `Tool execution failed: ${msg}`, isError: true };
-      }
-    };
-    const parallelPhase = safeParallel.length > 0 ? Promise.all(safeParallel.map(({ idx, call }) => wrapExec(idx, call))) : Promise.resolve();
-    const bashPhase = (async () => {
-      for (const { idx, call } of safeBash) {
-        await wrapExec(idx, call);
-      }
-    })();
-    await Promise.all([parallelPhase, bashPhase]);
-    if (safeParallel.length >= 2) {
-      const elapsed = Date.now() - t0;
-      console.log(theme.dim(`  \u26A1 ${safeParallel.length} tools executed in parallel (${elapsed}ms)`));
-    }
-    if (fileWriteCalls.length === 1) {
-      const { idx, call } = fileWriteCalls[0];
+    const elapsed = await runSafePhases(phase, (c) => this.execute(c), results, "executor");
+    if (phase.safeParallel.length >= 2) {
+      console.log(theme.dim(`  \u26A1 ${phase.safeParallel.length} tools executed in parallel (${elapsed}ms)`));
+    }
+    if (phase.fileWriteCalls.length === 1) {
+      const { idx, call } = phase.fileWriteCalls[0];
       results[idx] = await this.execute(call);
-    } else if (fileWriteCalls.length >= 2) {
-      const batchResults = await this.executeBatchFileWrites(fileWriteCalls.map((f) => f.call));
-      for (let i = 0; i < fileWriteCalls.length; i++) {
-        results[fileWriteCalls[i].idx] = batchResults[i];
+    } else if (phase.fileWriteCalls.length >= 2) {
+      const batchResults = await this.executeBatchFileWrites(phase.fileWriteCalls.map((f) => f.call));
+      for (let i = 0; i < phase.fileWriteCalls.length; i++) {
+        results[phase.fileWriteCalls[i].idx] = batchResults[i];
       }
     }
-    for (const { idx, call } of otherCalls) {
+    for (const { idx, call } of phase.otherCalls) {
       results[idx] = await this.execute(call);
     }
     return results;
@@ -1450,6 +1465,10 @@ var ToolExecutor = class {
         results[i] = { callId: calls[i].id, content: `[User rejected] The user rejected this ${calls[i].name} operation. Do not retry without asking.`, isError: true };
       }
     }
+    if (approvedIndices.length >= 2) {
+      const labels = approvedIndices.map((i) => `[${i + 1}]`).join(" ");
+      console.log(theme.dim(`  \u26A1 Writing ${labels} in parallel (results may interleave)`));
+    }
     const t0 = Date.now();
     await Promise.all(
       approvedIndices.map(async (i) => {
@@ -1488,7 +1507,7 @@ var ToolExecutor = class {
       return Promise.resolve("none");
     }
     const rl = this.rl;
-    const rlAny = rl;
+    const rlAny = rlInternal(rl);
     const savedOutput = rlAny.output;
     rlAny.output = process.stdout;
     rl.resume();
@@ -1668,7 +1687,7 @@ var ToolExecutor = class {
       return Promise.resolve(false);
     }
     const rl = this.rl;
-    const rlAny = rl;
+    const rlAny = rlInternal(rl);
     const savedOutput = rlAny.output;
     rlAny.output = process.stdout;
     rl.resume();
@@ -2930,7 +2949,7 @@ var askUserTool = {
   }
 };
 function promptUser(rl, question) {
-  const rlAny = rl;
+  const rlAny = rlInternal(rl);
   const savedOutput = rlAny.output;
   rlAny.output = process.stdout;
   rl.resume();
@@ -4332,13 +4351,15 @@ export {
   RateLimitError,
   ConfigError,
   ProviderNotFoundError,
-  isFileWriteTool,
   getDangerLevel,
   schemaToJsonSchema,
   initTheme,
   theme,
   undoStack,
   renderDiff,
+  rlInternal,
+  groupCallsByPhase,
+  runSafePhases,
   runHook,
   checkPermission,
   setMaxOutputCap,

package/dist/{chunk-AQNAV4H4.js → chunk-F44OKMB2.js}

@@ -8,7 +8,7 @@ import { platform } from "os";
 import chalk from "chalk";
 
 // src/core/constants.ts
-var VERSION = "0.4.65";
+var VERSION = "0.4.67";
 var APP_NAME = "ai-cli";
 var CONFIG_DIR_NAME = ".aicli";
 var CONFIG_FILE_NAME = "config.json";

package/dist/{chunk-UNLZJRCY.js → chunk-X7NVAEFI.js}

@@ -6,7 +6,7 @@ import { platform } from "os";
 import chalk from "chalk";
 
 // src/core/constants.ts
-var VERSION = "0.4.65";
+var VERSION = "0.4.67";
 var APP_NAME = "ai-cli";
 var CONFIG_DIR_NAME = ".aicli";
 var CONFIG_FILE_NAME = "config.json";

package/dist/{chunk-2B2EVU6Z.js → chunk-XSIVGDCN.js}

@@ -8,7 +8,7 @@ import {
   RateLimitError,
   schemaToJsonSchema,
   truncateForPersist
-} from "./chunk-SKN6HBBT.js";
+} from "./chunk-BVLQ3FRA.js";
 import {
   APP_NAME,
   CONFIG_DIR_NAME,
@@ -21,7 +21,7 @@ import {
   MCP_TOOL_PREFIX,
   PLUGINS_DIR_NAME,
   VERSION
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 
 // src/config/config-manager.ts
 import { readFileSync, writeFileSync, existsSync, mkdirSync } from "fs";
@@ -3819,18 +3819,34 @@ function trimOldToolOutput(messages, keepRecentRounds = 10) {
   }
   return trimmed;
 }
+function cloneMessages(messages) {
+  return messages.map((m) => ({ ...m }));
+}
 function autoTrimSessionIfNeeded(session, sizeLimit = SESSION_SIZE_LIMIT) {
-  const json = JSON.stringify(session.toJSON());
-  if (json.length <= sizeLimit) return false;
+  const originalSize = JSON.stringify(session.toJSON()).length;
+  if (originalSize <= sizeLimit) return false;
+  const snapshotLen = session.messages.length;
+  let working = cloneMessages(session.messages.slice(0, snapshotLen));
+  let committedSize = originalSize;
+  let anyTrim = false;
   let keepRecent = 10;
   while (keepRecent >= 2) {
-    const trimmed = trimOldToolOutput(session.messages, keepRecent);
-    if (trimmed === 0) break;
+    const trimmedCount = trimOldToolOutput(working, keepRecent);
+    if (trimmedCount === 0) break;
+    anyTrim = true;
+    const originalMessages = session.messages;
+    session.messages = [...working, ...originalMessages.slice(snapshotLen)];
     const newSize = JSON.stringify(session.toJSON()).length;
-    if (newSize <= sizeLimit) return true;
+    session.messages = originalMessages;
+    committedSize = newSize;
+    if (newSize <= sizeLimit) break;
     keepRecent = Math.max(2, Math.floor(keepRecent / 2));
+    if (keepRecent < 2) break;
   }
-  return true;
+  if (!anyTrim) return false;
+  const appended = session.messages.slice(snapshotLen);
+  session.messages.splice(0, session.messages.length, ...working, ...appended);
+  return committedSize < originalSize;
 }
 
 // src/repl/dev-state.ts

package/dist/{hub-TKYPJZEX.js → hub-RGK325NQ.js}

@@ -385,7 +385,7 @@ ${content}`);
   }
 }
 async function runTaskMode(config, providers, configManager, topic) {
-  const { TaskOrchestrator } = await import("./task-orchestrator-4DCUYIRI.js");
+  const { TaskOrchestrator } = await import("./task-orchestrator-4PVBMQJ7.js");
   const orchestrator = new TaskOrchestrator(config, providers, configManager);
   let interrupted = false;
   const onSigint = () => {

package/dist/index.js

@@ -31,7 +31,7 @@ import {
   saveDevState,
   sessionHasMeaningfulContent,
   setupProxy
-} from "./chunk-2B2EVU6Z.js";
+} from "./chunk-XSIVGDCN.js";
 import {
   ToolExecutor,
   ToolRegistry,
@@ -41,12 +41,13 @@ import {
   initTheme,
   lastResponseStore,
   renderDiff,
+  rlInternal,
   setContextWindow,
   setMaxOutputCap,
   spawnAgentContext,
   theme,
   undoStack
-} from "./chunk-SKN6HBBT.js";
+} from "./chunk-BVLQ3FRA.js";
 import {
   fileCheckpoints
 } from "./chunk-4BKXL7SM.js";
@@ -71,7 +72,7 @@ import {
   SKILLS_DIR_NAME,
   VERSION,
   buildUserIdentityPrompt
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 
 // src/index.ts
 import { program } from "commander";
@@ -2193,7 +2194,7 @@ ${hint}` : "")
       usage: "/test [command|filter]",
       async execute(args, ctx) {
         try {
-          const { executeTests } = await import("./run-tests-C4CEOYZP.js");
+          const { executeTests } = await import("./run-tests-NUF7CNM4.js");
           const argStr = args.join(" ").trim();
           let testArgs = {};
           if (argStr) {
@@ -3273,21 +3274,38 @@ var CostTracker = class {
       if (existsSync4(this.filePath)) {
         const data = JSON.parse(readFileSync3(this.filePath, "utf-8"));
         if (data.version === 1 && Array.isArray(data.records)) {
-          this.records = data.records;
+          this.records = [...data.records].sort((a, b) => a.date.localeCompare(b.date));
         }
       }
     } catch {
       this.records = [];
     }
   }
-  /** Save to disk (atomic write). */
+  /**
+   * Save to disk (atomic write).
+   *
+   * H2: Snapshot records before writing and only clear the dirty flag after
+   * the write succeeds. If writeFileSync/renameSync throws, dirty remains
+   * true so the next save() retries — prevents silent data loss on transient
+   * disk errors. writeFileSync is sync in Node.js so there's no interleaving
+   * risk within a single save() call; the snapshot mainly protects against
+   * future refactors to async I/O.
+   */
   save() {
     if (!this.dirty) return;
-    const data = { version: 1, records: this.records };
-    const tmp = this.filePath + ".tmp";
-    writeFileSync2(tmp, JSON.stringify(data, null, 2), "utf-8");
-    renameSync(tmp, this.filePath);
-    this.dirty = false;
+    const snapshot = {
+      version: 1,
+      records: [...this.records]
+      // shallow copy — records are plain data
+    };
+    try {
+      const tmp = this.filePath + ".tmp";
+      writeFileSync2(tmp, JSON.stringify(snapshot, null, 2), "utf-8");
+      renameSync(tmp, this.filePath);
+      this.dirty = false;
+    } catch (err) {
+      console.error("[cost-tracker] Failed to persist cost history:", err);
+    }
   }
   /**
    * Record cost from a completed session/interaction.
@@ -4371,7 +4389,7 @@ Session '${this.resumeSessionId}' not found.
         }
         processing = true;
         this.rl.pause();
-        const rlAny = this.rl;
+        const rlAny = rlInternal(this.rl);
         const savedOutput = rlAny.output;
         rlAny.output = null;
         try {
@@ -4386,10 +4404,10 @@ Session '${this.resumeSessionId}' not found.
           processing = false;
           if (this.running) {
             rlAny.output = savedOutput;
-            const rlInternal = this.rl;
-            rlInternal.line = "";
-            rlInternal.cursor = 0;
-            rlInternal.paused = false;
+            const rlInternal2 = this.rl;
+            rlInternal2.line = "";
+            rlInternal2.cursor = 0;
+            rlInternal2.paused = false;
             process.stdin.resume();
             this.showPrompt();
           } else {
@@ -5542,7 +5560,7 @@ ${mcpBudgetNote}` : "");
           process.stdout.write(theme.dim("  (Press ") + theme.info("n") + theme.dim(" or ") + theme.info("Esc") + theme.dim(" to stop)\n"));
           this.teardownInterjectionListener();
           const pauseResponse = await new Promise((resolve3) => {
-            const rlWithOutput = this.rl;
+            const rlWithOutput = rlInternal(this.rl);
             const savedOutput = rlWithOutput.output;
             rlWithOutput.output = process.stdout;
             this.rl.question(theme.warning("  \u25B8 "), (answer) => {
@@ -5685,9 +5703,9 @@ Tip: You can continue the conversation by asking the AI to proceed.`
       select: (prompt, items, initialIndex) => {
         this.selecting = true;
         return selectFromList(prompt, items, initialIndex).finally(() => {
-          const rlInternal = this.rl;
-          rlInternal.line = "";
-          rlInternal.cursor = 0;
+          const rlInternal2 = this.rl;
+          rlInternal2.line = "";
+          rlInternal2.cursor = 0;
           process.stdin.pause();
           setImmediate(() => {
             this.selecting = false;
@@ -5732,7 +5750,7 @@ Tip: You can continue the conversation by asking the AI to proceed.`
       getGitBranch: () => this.gitBranch,
       getLastResponse: () => lastResponseStore.content,
       runSetupWizard: async () => {
-        const rlAny = this.rl;
+        const rlAny = rlInternal(this.rl);
         rlAny.output = process.stdout;
         process.stdin.resume();
         try {
@@ -5905,7 +5923,7 @@ program.command("web").description("Start Web UI server with browser-based chat
     console.error("Error: Invalid port number. Must be between 1 and 65535.");
     process.exit(1);
   }
-  const { startWebServer } = await import("./server-CDRR3DC7.js");
+  const { startWebServer } = await import("./server-5HLNHRKM.js");
   await startWebServer({ port, host: options.host });
 });
 program.command("user [action] [username]").description("Manage Web UI users (list | create <name> | delete <name> | reset-password <name> | migrate <name>)").action(async (action, username) => {
@@ -6138,7 +6156,7 @@ program.command("hub [topic]").description("Start multi-agent hub (discuss / bra
     }),
     config.get("customProviders")
   );
-  const { startHub } = await import("./hub-TKYPJZEX.js");
+  const { startHub } = await import("./hub-RGK325NQ.js");
   await startHub(
     {
       topic: topic ?? "",

package/dist/{run-tests-KATYWRVQ.js → run-tests-KGUJL6L5.js}

@@ -1,7 +1,7 @@
 import {
   executeTests,
   runTestsTool
-} from "./chunk-UNLZJRCY.js";
+} from "./chunk-X7NVAEFI.js";
 export {
   executeTests,
   runTestsTool

package/dist/{run-tests-C4CEOYZP.js → run-tests-NUF7CNM4.js}

@@ -2,7 +2,7 @@
 import {
   executeTests,
   runTestsTool
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 export {
   executeTests,
   runTestsTool

package/dist/{server-CDRR3DC7.js → server-5HLNHRKM.js}

@@ -21,7 +21,7 @@ import {
   persistToolRound,
   rebuildExtraMessages,
   setupProxy
-} from "./chunk-2B2EVU6Z.js";
+} from "./chunk-XSIVGDCN.js";
 import {
   AuthManager
 } from "./chunk-BYNY5JPB.js";
@@ -33,15 +33,16 @@ import {
   estimateTokens,
   getDangerLevel,
   googleSearchContext,
-  isFileWriteTool,
+  groupCallsByPhase,
   renderDiff,
   runHook,
+  runSafePhases,
   setContextWindow,
   setMaxOutputCap,
   spawnAgentContext,
   truncateOutput,
   undoStack
-} from "./chunk-SKN6HBBT.js";
+} from "./chunk-BVLQ3FRA.js";
 import "./chunk-4BKXL7SM.js";
 import {
   AGENTIC_BEHAVIOR_GUIDELINE,
@@ -61,7 +62,7 @@ import {
   SKILLS_DIR_NAME,
   VERSION,
   buildUserIdentityPrompt
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 
 // src/web/server.ts
 import express from "express";
@@ -325,52 +326,19 @@ var ToolExecutorWeb = class _ToolExecutorWeb {
     }
   }
   async executeAll(calls) {
-    const safeParallel = [];
-    const safeBash = [];
-    const fileWriteCalls = [];
-    const otherCalls = [];
-    for (let i = 0; i < calls.length; i++) {
-      const call = calls[i];
-      const level = getDangerLevel(call.name, call.arguments);
-      if (level === "safe") {
-        if (call.name === "bash") {
-          safeBash.push({ idx: i, call });
-        } else {
-          safeParallel.push({ idx: i, call });
-        }
-      } else if (isFileWriteTool(call.name) && level === "write") {
-        fileWriteCalls.push({ idx: i, call });
-      } else {
-        otherCalls.push({ idx: i, call });
-      }
-    }
+    const phase = groupCallsByPhase(calls);
     const results = new Array(calls.length);
-    const wrapExec = async (idx, call) => {
-      try {
-        results[idx] = await this.execute(call);
-      } catch (err) {
-        const msg = err instanceof Error ? err.message : String(err);
-        console.error(`[tool-executor-web] Unexpected error in tool "${call.name}":`, err);
-        results[idx] = { callId: call.id, content: `Tool execution failed: ${msg}`, isError: true };
-      }
-    };
-    const parallelPhase = safeParallel.length > 0 ? Promise.all(safeParallel.map(({ idx, call }) => wrapExec(idx, call))) : Promise.resolve();
-    const bashPhase = (async () => {
-      for (const { idx, call } of safeBash) {
-        await wrapExec(idx, call);
-      }
-    })();
-    await Promise.all([parallelPhase, bashPhase]);
-    if (fileWriteCalls.length === 1) {
-      const { idx, call } = fileWriteCalls[0];
+    await runSafePhases(phase, (c) => this.execute(c), results, "tool-executor-web");
+    if (phase.fileWriteCalls.length === 1) {
+      const { idx, call } = phase.fileWriteCalls[0];
       results[idx] = await this.execute(call);
-    } else if (fileWriteCalls.length >= 2) {
-      const batchResult = await this.executeBatchFileWrites(fileWriteCalls);
-      for (let i = 0; i < fileWriteCalls.length; i++) {
-        results[fileWriteCalls[i].idx] = batchResult[i];
+    } else if (phase.fileWriteCalls.length >= 2) {
+      const batchResult = await this.executeBatchFileWrites(phase.fileWriteCalls);
+      for (let i = 0; i < phase.fileWriteCalls.length; i++) {
+        results[phase.fileWriteCalls[i].idx] = batchResult[i];
       }
     }
-    for (const { idx, call } of otherCalls) {
+    for (const { idx, call } of phase.otherCalls) {
       results[idx] = await this.execute(call);
     }
     return results;
@@ -391,6 +359,13 @@ var ToolExecutorWeb = class _ToolExecutorWeb {
         };
       }
     }
+    if (approvedIndices.length >= 2) {
+      const labels = approvedIndices.map((i) => `[${i + 1}]`).join(" ");
+      this.send({
+        type: "info",
+        message: `\u26A1 Writing ${labels} in parallel (results may interleave)`
+      });
+    }
     await Promise.all(
       approvedIndices.map(async (i) => {
         const call = calls[i];
@@ -1971,7 +1946,7 @@ ${undoResults.map((r) => `  \u2022 ${r}`).join("\n")}` });
       case "test": {
         this.send({ type: "info", message: "\u{1F9EA} Running tests..." });
         try {
-          const { executeTests } = await import("./run-tests-C4CEOYZP.js");
+          const { executeTests } = await import("./run-tests-NUF7CNM4.js");
           const argStr = args.join(" ").trim();
           let testArgs = {};
           if (argStr) {

package/dist/{task-orchestrator-4DCUYIRI.js → task-orchestrator-4PVBMQJ7.js}

@@ -4,11 +4,11 @@ import {
   getDangerLevel,
   googleSearchContext,
   truncateOutput
-} from "./chunk-SKN6HBBT.js";
+} from "./chunk-BVLQ3FRA.js";
 import "./chunk-4BKXL7SM.js";
 import {
   SUBAGENT_ALLOWED_TOOLS
-} from "./chunk-AQNAV4H4.js";
+} from "./chunk-F44OKMB2.js";
 
 // src/hub/task-orchestrator.ts
 import { createInterface } from "readline";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jinzd-ai-cli",
-  "version": "0.4.65",
+  "version": "0.4.67",
   "description": "Cross-platform REPL-style AI CLI with multi-provider support",
   "type": "module",
   "main": "./dist/index.js",