@raolin2025/claude-code-node 2.8.16 → 2.8.18

package/package.json

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

@@ -284,3 +284,39 @@ test('工具结果截断：非 tool 消息与空内容不受影响', () => {
   assert.equal(r[0].content.length, 10000, 'user 消息不应被截断')
   assert.equal(r[1].content, '')
 })
+
+test('折叠/裁剪：多次折叠后所有 system 仍在开头（防 llama.cpp 500）', () => {
+  // 模拟：一次折叠后 body 里有摘要 system 残留的场景
+  const msgs = [
+    { role: 'system', content: 'ROOT SYS' },
+    { role: 'system', content: '[Context Summary — 旧的折叠摘要]' }, // 已存在第二条 system
+    { role: 'user', content: '任务1' },
+    { role: 'assistant', content: '结果1' },
+    { role: 'user', content: '任务2' },
+    { role: 'assistant', content: '结果2' },
+  ]
+  // foldHistoryByCount 折叠后，所有 system 都应在最前，body 里无 system
+  const r = foldHistoryByCount(msgs, { maxMessages: 5, keepRecentTurns: 1 })
+  const roles = r.messages.map(m => m.role)
+  // 找到第一个非 system 的索引，之后不应再有 system
+  const firstBody = roles.findIndex(x => x !== 'system')
+  const afterBody = roles.slice(firstBody + 1)
+  assert.ok(!afterBody.includes('system'), `折叠后 body 中不应有 system（实际 roles=${roles.join(',')}）`)
+})
+
+test('trimToWindow：所有 system 均在开头，body 中无 system', () => {
+  const msgs = [
+    { role: 'system', content: 'ROOT SYS' },
+    { role: 'system', content: '[Context Summary — 旧摘要]' },
+    { role: 'user', content: 'a'.repeat(50) },
+    { role: 'assistant', content: 'b'.repeat(50) },
+    { role: 'user', content: 'c'.repeat(50) },
+    { role: 'assistant', content: 'd'.repeat(50) },
+  ]
+  const tb = makeBudget(200, 0)
+  const r = trimToWindow(msgs, { tokenBudget: tb, maxTokens: tb.maxTokens })
+  const roles = r.messages.map(m => m.role)
+  const firstBody = roles.findIndex(x => x !== 'system')
+  const afterBody = roles.slice(firstBody + 1)
+  assert.ok(!afterBody.includes('system'), `trim 后 body 中不应有 system（实际 roles=${roles.join(',')}）`)
+})

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

@@ -226,3 +226,29 @@ test('max_tokens：绝不超过窗口一半（防超窗）', async () => {
   const qe2 = new QueryEngine({ tokenBudget: new TokenBudget({ maxTokens: 10000 }), outputRatio: 1.0 })
   assert.ok(qe2._computeMaxOutputTokens() <= 5000, `max_tokens=${qe2._computeMaxOutputTokens()} 不应超过窗口一半 5000`)
 })
+
+// ---- _buildRequest 把 system 统一前置（防 llama.cpp 500）----
+test('_buildRequest：把中间 system 统一前置到开头', async () => {
+  const { QueryEngine } = await import('../core/query-engine.js')
+  const qe = new QueryEngine({ systemPrompt: 'SYS-ROOT', tools: [] })
+  // 模拟 state.messages 里 system 摘要被插到中间（折叠/恢复导致）
+  const messages = [
+    { role: 'user', content: 'u1' },
+    { role: 'assistant', content: 'a1' },
+    { role: 'system', content: '[Context Summary] 中间摘要' },
+    { role: 'user', content: 'u2' },
+  ]
+  const req = qe._buildRequest(messages)
+  // 验证：所有 system 在开头，中间无 system
+  let seenBody = false, badSystem = false
+  for (const m of req) {
+    if (m.role !== 'system') seenBody = true
+    else if (seenBody) badSystem = true
+  }
+  assert.equal(badSystem, false, '中间不应出现 system（否则 llama.cpp 报 500）')
+  // 第一条是 system
+  assert.equal(req[0].role, 'system')
+  // 原始顺序（非 system 部分）应保持
+  const nonSys = req.filter(m => m.role !== 'system').map(m => m.role)
+  assert.deepEqual(nonSys, ['user', 'assistant', 'user'])
+})

package/src/core/compact.js

@@ -197,11 +197,14 @@ export function foldHistoryByCount(messages, options = {}) {
     return { folded: false, messages, removed: 0, summary: null }
   }
 
-  // 分离 system 提示（首条 system 永不折叠）与普通消息
+  // 分离 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) {
-    if (m.role === 'system' && systemMsgs.length === 0) {
+    if (m.role === 'system') {
       systemMsgs.push(m)
     } else {
       body.push(m)
@@ -400,11 +403,13 @@ export function trimToWindow(messages, options = {}) {
     return { trimmed: false, messages, removed: 0, summary: null }
   }
 
-  // 分离 system 提示（首条 system 永不裁剪）与普通消息
+  // 分离 system 提示与普通消息
+  // 注意：收集【所有】 system（不只第一条），避免多次裁剪后旧的摘要 system
+  // 残留在 body 里、被挤到对话中间，触发 llama.cpp "System message must be at the beginning" (500)。
   const systemMsgs = []
   const body = []
   for (const m of messages) {
-    if (m.role === 'system' && systemMsgs.length === 0) {
+    if (m.role === 'system') {
       systemMsgs.push(m)
     } else {
       body.push(m)

package/src/core/query-engine.js

@@ -358,10 +358,25 @@ export class QueryEngine {
     }
 
     // 历史消息 — 转换为 OpenAI 兼容格式
+    // 关键：把所有 system 消息【收集到一起放到最前面】，而不是原样透传。
+    // 原因：llama.cpp/Jinja 严格要求 system 消息必须在对话开头，中间出现 system
+    // 会报 "System message must be at the beginning" (500)。
+    // 折叠/压缩/引导注入可能在 state.messages 中间留下 system 摘要消息，
+    // 这里统一前置，彻底规避该错误。
+    const systemMsgs = []
+    const bodyMsgs = []
     for (const msg of messages) {
       if (msg.role === 'system') {
-        request.push({ role: 'system', content: msg.content })
-      } else if (msg.role === 'user') {
+        systemMsgs.push(msg)
+      } else {
+        bodyMsgs.push(msg)
+      }
+    }
+    for (const msg of systemMsgs) {
+      request.push({ role: 'system', content: msg.content })
+    }
+    for (const msg of bodyMsgs) {
+      if (msg.role === 'user') {
         request.push({ role: 'user', content: this._buildUserContent(msg) })
       } else if (msg.role === 'assistant') {
         // 构建 assistant 消息基础