@raolin2025/claude-code-node 2.8.20 → 2.8.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raolin2025/claude-code-node",
-  "version": "2.8.20",
+  "version": "2.8.22",
   "description": "Node.js AI Code Agent CLI - Zero dependencies, pure JavaScript, security hardened, multi-channel notifications, Telegram & QQ Bot remote programming, rich media upload, multi-account management",
   "type": "module",
   "main": "src/core/index.js",

package/src/__tests__/compact-window.test.js

@@ -320,3 +320,63 @@ test('trimToWindow：所有 system 均在开头，body 中无 system', () => {
   const afterBody = roles.slice(firstBody + 1)
   assert.ok(!afterBody.includes('system'), `trim 后 body 中不应有 system（实际 roles=${roles.join(',')}）`)
 })
+
+test('条数折叠：保留当前进行中的任务链，只折叠已完成的早期历史', () => {
+  // 场景：正在工具循环中（最近 user 之后有完整工具链），早期有已完成任务
+  const msgs = [
+    { role: 'system', content: 'SYS' },
+    // 早期已完成历史
+    { role: 'user', content: '任务1' },
+    { role: 'assistant', content: 'a1', toolCalls: [{ id: 't1', name: 'Bash', input: {} }] },
+    { role: 'tool', tool_call_id: 't1', content: 'r1' },
+    { role: 'assistant', content: '完成1' },
+    // 当前进行中的任务（工具链未完成）
+    { role: 'user', content: '按计划实现第一阶段' },
+    { role: 'assistant', content: '', toolCalls: [{ id: 'c1', name: 'Read', input: {} }] },
+    { role: 'tool', tool_call_id: 'c1', content: '计划内容' },
+    { role: 'assistant', content: '', toolCalls: [{ id: 'c2', name: 'Write', input: {} }] },
+    { role: 'tool', tool_call_id: 'c2', content: '写入成功' },
+  ]
+  const r = foldHistoryByCount(msgs, { maxMessages: 8, keepRecentTurns: 2 })
+  assert.equal(r.folded, true)
+  const all = JSON.stringify(r.messages)
+  // 当前任务（最近的 user 及其后的工具链）必须完整保留
+  assert.ok(all.includes('按计划实现第一阶段'), '当前任务 user 应保留')
+  assert.ok(all.includes('计划内容'), '当前任务的 Read 结果应保留')
+  assert.ok(all.includes('写入成功'), '当前任务的 Write 结果应保留')
+  // 早期已完成历史应被折叠
+  assert.ok(!all.includes('完成1'), '早期已完成历史应被折叠')
+})
+
+test('条数折叠：当前任务很大时按完整 user 轮次折叠，不截断工具链', () => {
+  const msgs = [{ role: 'system', content: 'SYS' }]
+  // 早期历史
+  for (let i = 0; i < 5; i++) {
+    msgs.push({ role: 'user', content: '旧任务' + i })
+    msgs.push({ role: 'assistant', content: '旧完成' + i })
+  }
+  // 当前大任务（多轮工具循环）
+  msgs.push({ role: 'user', content: '实现第一阶段' })
+  for (let i = 0; i < 15; i++) {
+    msgs.push({ role: 'assistant', content: '', toolCalls: [{ id: 'c' + i, name: 'Write', input: {} }] })
+    msgs.push({ role: 'tool', tool_call_id: 'c' + i, content: '写模块' + i })
+  }
+  const r = foldHistoryByCount(msgs, { maxMessages: 20, keepRecentTurns: 3 })
+  assert.equal(r.folded, true)
+  const all = JSON.stringify(r.messages)
+  // 当前任务 user 保留
+  assert.ok(all.includes('实现第一阶段'), '当前任务 user 应保留')
+  // 折叠后工具链连续（无中间截断）
+  const roles = r.messages.map(m => m.role).filter(x => x !== 'system')
+  let expectUser = true
+  for (const role of roles) {
+    if (role === 'user') { expectUser = false }
+  }
+  // 验证角色序列是 user,assistant,tool,assistant,tool... 的连续模式（无孤立 tool）
+  for (let i = 0; i < roles.length; i++) {
+    if (roles[i] === 'tool') {
+      // tool 之前必须是 assistant
+      assert.ok(i > 0 && roles[i - 1] === 'assistant', `第 ${i} 个 tool 前应为 assistant（实际 ${roles.join(',')}）`)
+    }
+  }
+})

package/src/__tests__/small-model.test.js

@@ -249,6 +249,17 @@ test('框架代执行：非读取任务或无路径返回 null', () => {
   assert.equal(extractFrameworkAction('运行测试', { cwd: 'D:\\x' }), null)
 })
 
+test('框架代执行：路径后带中文说明时正确截断（不吞入多余内容）', () => {
+  // 复现 error.log 的 bug：路径后跟中文说明，之前会被贪婪匹配进路径
+  const a = extractFrameworkAction(
+    '阅读D:\\workspace\\QMT_CrossPlatform\\miniQMT-trader\\COMPLETION_REPORT.md,对项目有个全盘了解',
+    { cwd: 'D:\\workspace\\QMT_CrossPlatform\\miniQMT-trader' }
+  )
+  assert.ok(a, '应识别到框架动作')
+  assert.equal(a.input.file_path, 'D:\\workspace\\QMT_CrossPlatform\\miniQMT-trader\\COMPLETION_REPORT.md',
+    `路径不应吞入中文说明（实际 ${a.input.file_path}）`)
+})
+
 // ---- _buildRequest 把 system 统一前置（防 llama.cpp 500）----
 test('_buildRequest：把中间 system 统一前置到开头', async () => {
   const { QueryEngine } = await import('../core/query-engine.js')

package/src/core/compact.js

@@ -198,9 +198,6 @@ export function foldHistoryByCount(messages, options = {}) {
   }
 
   // 分离 system 提示与普通消息
-  // 注意：把所有 system 都收集到 systemMsgs（不只是第一条）——多次折叠后 body 里
-  // 可能残留旧的摘要 system，若不全部隔离，折叠结果会把 system 挤到对话中间，
-  // 触发 llama.cpp "System message must be at the beginning" (500)。
   const systemMsgs = []
   const body = []
   for (const m of messages) {
@@ -211,18 +208,55 @@ export function foldHistoryByCount(messages, options = {}) {
     }
   }
 
-  // 找到分界点：保留最近 keepRecentTurns 轮
-  // 一轮 = user + assistant(+tool_calls) + tool 结果们 + assistant 最终回复
-  // 从末尾倒推，数到第 keepRecentTurns 个 user 即分界（splitIndex 指向该轮起点，
-  // 使得 recentMessages 恰好包含最近 keepRecentTurns 轮完整对话）
-  let turnCount = 0
-  let splitIndex = body.length
+  // 找到分界点。
+  // 核心原则：**绝不能破坏"当前正在进行的任务链"的完整性**。
+  // 小模型"工作到一半变傻"的根因，就是折叠把正在进行的任务（最近的 user 指令
+  // 及其后的 tool 调用链）折叠成模糊摘要，模型丢失了"当前任务状态"（在做什么、
+  // 做到哪、下一步干什么）而停止调用工具。
+  // 因此：
+  //   1. 优先只折叠【最近 user 指令之前】的早期历史——当前任务（最近 user 指令
+  //      之后所有消息，含进行中的工具调用链）完整保留，永不折叠。
+  //   2. 若当前任务本身消息仍过多（单个任务几十轮），再从当前任务内部按
+  //      【完整 user 轮次】为单位折叠（保留最近 keepRecentTurns 个 user 任务），
+  //      绝不从 tool/assistant 消息中间截断工具链。
+  let splitIndex = -1
+  // 找最近的 user 指令位置
+  let lastUserIdx = -1
   for (let i = body.length - 1; i >= 0; i--) {
     if (body[i].role === 'user') {
-      turnCount++
-      if (turnCount >= keepRecentTurns) {
-        splitIndex = i
-        break
+      lastUserIdx = i
+      break
+    }
+  }
+  // 最近的 user 之后的消息数（当前任务的大小）
+  const currentTaskSize = lastUserIdx === -1 ? body.length : body.length - lastUserIdx
+  if (lastUserIdx > 0 && currentTaskSize <= maxMessages) {
+    // 当前任务本身未超限 → 折叠当前任务之前的历史（保留最近 user 指令起全部）
+    splitIndex = lastUserIdx
+  } else if (lastUserIdx >= 0) {
+    // 当前任务本身也超限 → 从当前任务内部按完整 user 轮次折叠
+    let turnCount = 0
+    splitIndex = lastUserIdx
+    for (let i = body.length - 1; i >= lastUserIdx; i--) {
+      if (body[i].role === 'user') {
+        turnCount++
+        if (turnCount >= keepRecentTurns) {
+          splitIndex = i
+          break
+        }
+      }
+    }
+  } else {
+    // 没有 user 消息（异常）→ 用 keepRecentTurns 兜底
+    let turnCount = 0
+    splitIndex = body.length
+    for (let i = body.length - 1; i >= 0; i--) {
+      if (body[i].role === 'user') {
+        turnCount++
+        if (turnCount >= keepRecentTurns) {
+          splitIndex = i
+          break
+        }
       }
     }
   }
@@ -238,7 +272,7 @@ export function foldHistoryByCount(messages, options = {}) {
   // 对折叠掉的历史生成摘要（保留 Main goal / 工具 / 关键结果）
   const summary = generateSummary(earlyMessages)
 
-  // 构建折叠后的消息列表：system + (摘要 system) + 最近 N 轮完整对话
+  // 构建折叠后的消息列表：system + (摘要 system) + 当前任务完整对话
   const folded = [...systemMsgs]
   folded.push({
     role: 'system',

package/src/core/query-engine.js

@@ -306,12 +306,15 @@ export class QueryEngine {
             this.state.messages.push({ role: 'user', content: RETRY_GUIDANCE })
             continue
           }
-          // 重试后仍敷衍 → 框架代执行（不依赖模型调工具）
+          // 重试后仍敷衍 → 框架代执行，且【直接把结果作为最终答案】返回，
+          // 不再 continue 让模型总结——模型已证明它只会敷衍，继续循环只会无限重复。
+          // 这样既完成任务，又不会陷入"代执行→敷衍→再代执行"的死循环。
           const execResult = await this._frameworkExecute(userMessage.content)
           if (execResult.done) {
-            if (this.config.verbose) console.error(`[small-model] 框架代执行：${execResult.summary}`)
-            // 框架已执行工具并把结果注入对话，继续循环让模型基于结果总结
-            continue
+            if (this.config.verbose) console.error(`[small-model] 框架代执行完成，直接返回结果`)
+            // 框架已执行并把结果注入对话；直接把真实结果作为最终回复
+            finalResponse = execResult.resultText
+            break
           }
         }
 
@@ -522,31 +525,31 @@ export class QueryEngine {
    */
   async _frameworkExecute(userInput) {
     const action = extractFrameworkAction(userInput, { cwd: this.config.cwd })
-    if (!action) return { done: false, summary: '' }
+    if (!action) return { done: false, summary: '', resultText: '' }
 
     // 找到对应的工具
     const tool = this.config.tools.find(t => t.name === action.tool)
-    if (!tool) return { done: false, summary: '' }
+    if (!tool) return { done: false, summary: '', resultText: '' }
 
     // 执行工具（直接调用 handler）
     try {
       const content = await tool.handler(action.input, { cwd: this.config.cwd, engine: this, readline: this.config.readline })
 
-      // 注入框架代执行结果（作为 user 消息，避免 tool 消息无对应 assistant tool_call 报错）
-      // 说明：框架代执行不是模型发起的工具调用，不能作为 tool 角色（会缺 assistant tool_call），
-      // 因此以 user 消息承载真实结果 + 引导模型基于结果总结。
       const resultText = typeof content === 'string' ? content : JSON.stringify(content)
+
+      // 注入框架代执行结果（作为 user 消息，避免 tool 消息无对应 assistant tool_call 报错）
+      // 说明：框架代执行不是模型发起的工具调用，不能作为 tool 角色（会缺 assistant tool_call）。
       this.state.messages.push({
         role: 'user',
-        content: `[框架代执行结果] 框架已替你执行工具 ${tool.name}（参数 ${JSON.stringify(action.input)}），结果如下：\n\n${resultText.slice(0, 8000)}\n\n请基于以上【真实执行结果】，用中文直接回答用户的问题或总结内容。不要再调用工具，直接总结即可。`,
+        content: `[框架代执行结果] 框架已替你执行工具 ${tool.name}（参数 ${JSON.stringify(action.input)}），结果如下：\n\n${resultText.slice(0, 8000)}`,
       })
 
       const summary = `框架已代执行 ${tool.name}(${JSON.stringify(action.input)})`
       if (this.config.verbose) console.error(`[small-model] 框架代执行 ${tool.name} 完成`)
-      return { done: true, summary }
+      return { done: true, summary, resultText }
     } catch (err) {
       if (this.config.verbose) console.error(`[small-model] 框架代执行失败: ${err.message}`)
-      return { done: false, summary: '' }
+      return { done: false, summary: '', resultText: '' }
     }
   }
 

package/src/core/small-model.js

@@ -256,17 +256,28 @@ export function extractFrameworkAction(userInput, opts = {}) {
   const cwd = opts.cwd || ''
 
   // 1. 读取文件任务：含路径 + 读/查看/打开
-  //    匹配绝对路径（Windows: D:\... 或 /... 或 ./...）或文件名
   const readIntent = /(读|阅读|查看|打开|展示|显示|看看|浏览|看)/i.test(userInput)
-  // 提取路径：优先 Windows 绝对路径 D:\...、Unix /...、相对 ./ 或 ../，或 .md/.txt/.py/.json 文件
-  const pathMatch = userInput.match(/([A-Za-z]:\\[^\s，。；]+|(?:\/|\/\/)[^\s，。；]+|\.{1,2}\/[^\s，。；]+|[\w@.-]+\.(?:md|txt|py|json|log|yaml|yml|toml|ini|cfg|csv|xml))+/i)
 
-  if (readIntent && pathMatch) {
-    let filePath = pathMatch[1].trim()
+  // 提取路径。关键：路径字符集【只允许文件路径合法字符】，遇到逗号/分号/空格/中文
+  // 立即停止，避免把指令里的说明文字（如 ",对项目有个全盘了解"）拼进路径。
+  // 合法路径字符：字母数字、\ / . : - _、空格(在引号内罕见，排除以免误吞)、
+  // Windows 盘符。
+  // 优先匹配带扩展名的完整路径，扩展名后即停止。
+  const pathRegex =
+    /([A-Za-z]:[\\/][\w@.\-\\/ ]*\.(?:md|txt|py|json|log|yaml|yml|toml|ini|cfg|csv|xml)|[\\/][\w@.\-\\/ ]*\.(?:md|txt|py|json|log|yaml|yml|toml|ini|cfg|csv|xml)|\.{1,2}[\\/][\w@.\-\\/]*\.(?:md|txt|py|json|log|yaml|yml|toml|ini|cfg|csv|xml)|[\w@.-]+\.(?:md|txt|py|json|log|yaml|yml|toml|ini|cfg|csv|xml))/i
+
+  // 用 [^,，;；。\s] 确保遇到分隔符就停，然后取最长的、以扩展名结尾的匹配
+  const match = userInput.match(pathRegex)
+  let filePath = null
+  if (match) {
+    filePath = match[1].trim()
+  }
+
+  if (readIntent && filePath) {
     // 去掉开头的 ./ 或 ../
     filePath = filePath.replace(/^\.\.?[\\/]/, '')
     // 若是相对路径且给了 cwd，拼成绝对路径
-    if (!/^[A-Za-z]:\\/.test(filePath) && !filePath.startsWith('/') && cwd) {
+    if (!/^[A-Za-z]:[\\/]/.test(filePath) && !filePath.startsWith('/') && cwd) {
       filePath = cwd.replace(/[\\/]+$/, '') + '\\' + filePath
     }
     return { tool: 'Read', toolName: 'Read', input: { file_path: filePath } }