@raolin2025/claude-code-node 2.8.21 → 2.8.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raolin2025/claude-code-node",
-  "version": "2.8.21",
+  "version": "2.8.23",
   "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__/small-model.test.js

@@ -127,10 +127,19 @@ test('工具精简：未启用返回全部', () => {
   assert.equal(selectRelevantTools(fakeTools, 'xx', { enable: false }), fakeTools)
 })
 
-test('工具精简：无明确意图时返回核心文件工具', () => {
+test('工具精简：无明确意图时返回全部工具（让模型自己挑）', () => {
   const reduced = selectRelevantTools(fakeTools, '你好', { enable: true })
-  assert.ok(reduced.length <= 6, '应只保留核心工具')
-  assert.ok(reduced.some(t => t.name === 'Bash'))
+  // 实测经验：小模型不能过度精简工具，无明确适配时干脆全部给
+  assert.equal(reduced.length, fakeTools.length, '无明确意图应返回全部工具')
+})
+
+test('工具精简：有明确意图时至少保留 6 个核心工具', () => {
+  const reduced = selectRelevantTools(fakeTools, '阅读D:\\x\\COMPLETION_REPORT.md', { enable: true })
+  // 至少 6 个核心工具，绝不精简到 2 个
+  assert.ok(reduced.length >= 6, `应至少保留 6 个工具（实际 ${reduced.length}）`)
+  assert.ok(reduced.some(t => t.name === 'Read'))
+  assert.ok(reduced.some(t => t.name === 'Bash'), '应保留 Bash')
+  assert.ok(reduced.some(t => t.name === 'Write'), '应保留 Write')
 })
 
 // ---- system prompt 强化 ----
@@ -249,6 +258,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/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 } }
@@ -366,32 +377,53 @@ export function buildCombinedGuidance(userInput, opts = {}) {
 }
 
 /**
- * 精简工具列表（层 A）— 小模型一次看太多工具会混乱
+ * 工具选择（层 A）— 小模型适配的关键经验
  *
- * 从完整工具列表里，按用户指令筛选出最相关的核心工具子集。
- * 这样模型的工具选择负担小，更不容易乱调/漏调。
+ * 重要结论（来自实测）：**小模型工具调用不能过度精简**。
+ * v2.8.12 之前（16 个工具全上）模型能正常调用工具；v2.8.13 之后我做了"按意图
+ * 精简成 2-6 个"，结果模型反而不调用工具、只会瞎编。因此策略改为：
+ *
+ *   1. **保底至少 6 个核心工具**：Bash, Read, Edit, Write, Glob, Grep（每次必给）；
+ *   2. **按任务适配追加**：若意图明确需要额外工具（联网→WebSearch/WebFetch 等），
+ *      在核心 6 个之上追加，绝不减少核心集；
+ *   3. **找不到合适的适配，就全部给**：宁可全给 16 个，也不要精简到很少。
+ *
+ * 原则：**工具给全，让模型自己挑；而不是替它精简，导致它无工具可用而瞎编。**
  *
  * @param {Array} allTools — 完整 ToolDef 列表
  * @param {string} userInput — 用户指令
  * @param {object} [opts]
- * @param {boolean} [opts.enable=true] — 是否启用精简
- * @returns {Array} 精简后的工具列表
+ * @param {boolean} [opts.enable=true] — 是否启用适配
+ * @returns {Array} 选择后的工具列表（至少 6 个核心；找不到适配则全部）
  */
 export function selectRelevantTools(allTools, userInput, opts = {}) {
   if (opts.enable === false) return allTools
   if (!allTools || allTools.length === 0) return allTools
 
+  // 核心工具保底集（每次必给）：小模型不能过度精简工具，否则不会调用只会瞎编
+  const CORE = ['Bash', 'Read', 'Edit', 'Write', 'Glob', 'Grep']
   const intent = detectIntent(userInput)
-  // 未识别到明确意图 → 返回核心工具（不塞满 16 个）
-  const coreNames = intent
-    ? intent.toolHint.split(',').map(s => s.trim())
-    : ['Bash', 'Read', 'Edit', 'Write', 'Glob', 'Grep']
-
-  // 核心工具优先，保留指定工具；若一个都没命中则退回首屏核心集
-  const selected = allTools.filter(t => coreNames.includes(t.name))
-  if (selected.length === 0) {
-    return allTools.filter(t => ['Bash', 'Read', 'Edit', 'Write', 'Glob', 'Grep'].includes(t.name))
+
+  // 找不到合适的适配（无明确意图）→ 干脆全部调用，让模型自己挑。
+  // 这是实测经验：v2.8.12 之前全 16 个工具模型能正常调用；精简后反而变傻。
+  if (!intent) {
+    return allTools
   }
+
+  // 有明确意图 → 核心 6 个（保底）+ 意图命中的额外工具（追加，不减少核心）
+  const toolNames = new Set(CORE)
+  if (intent.toolHint) {
+    for (const name of intent.toolHint.split(',').map(s => s.trim())) {
+      toolNames.add(name)
+    }
+  }
+  const selected = allTools.filter(t => toolNames.has(t.name))
+
+  // 兜底：筛选不足 6 个（核心工具不全）→ 全给
+  if (selected.length < 6) {
+    return allTools
+  }
+
   return selected
 }