@raolin2025/claude-code-node 2.8.21 → 2.8.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raolin2025/claude-code-node",
-  "version": "2.8.21",
+  "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__/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/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 } }