@noobdemon/noob-cli 1.13.2 → 1.13.3

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 Tất cả thay đổi đáng kể của `@noobdemon/noob-cli` được ghi vào file này.
 
+## [1.13.3] - 2026-06-29
+
+### Fixed
+- **Message nhắc việc nội bộ không còn gây nhiễu mô hình** (`src/agent.js`): các message nhắc tiếp tục công việc (nhắc todo còn dở, hướng dẫn khi tool lỗi, cảnh báo gọi trùng tool) trước đây được chèn vào hội thoại như thể là tin nhắn người dùng và mở đầu bằng nhãn `[SYSTEM]`. Mô hình ở lượt sau đọc lại tưởng có người chèn lệnh giả dạng hệ thống vào cuộc trò chuyện nên sinh nghi ngờ, đi sửa lung tung. Giờ các message này đi đúng kênh kết quả tool và đổi nhãn thành `[TODO]`/`[TOOL GUIDANCE]`.
+
+### Verified
+- `npm test` 109/109 pass.
+
 ## [1.13.2] - 2026-06-28
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noobdemon/noob-cli",
-  "version": "1.13.2",
+  "version": "1.13.3",
   "publishConfig": {
     "access": "public"
   },

package/src/agent.js

@@ -479,18 +479,18 @@ export function todoContinuationMessage(toolName, ok, tasks) {
   if (!tasks?.length) return null;
   const next = tasks[0];
   const status = ok ? `đã hoàn thành` : `vừa LỖI`;
-  return `[SYSTEM] Việc "${toolName}" ${status}. Còn ${tasks.length} việc: ${tasks.map((t) => `"${t}"`).join(', ')}. Việc tiếp theo BẮT BUỘC phải làm ngay: "${next}". Gọi tool (write_file/edit_file/run_command) để làm việc này. KHÔNG dừng, KHÔNG tóm tắt.`;
+  return `[TODO] Việc "${toolName}" ${status}. Còn ${tasks.length} việc: ${tasks.map((t) => `"${t}"`).join(', ')}. Việc tiếp theo BẮT BUỘC phải làm ngay: "${next}". Gọi tool (write_file/edit_file/run_command) để làm việc này. KHÔNG dừng, KHÔNG tóm tắt.`;
 }
 
 export function toolErrorGuidance(name, { result } = {}) {
   if (name !== 'edit_file' || !String(result || '').includes('old_string not found')) return null;
   const m = String(result).match(/NEXT REQUIRED TOOL:\s*(read_file \{[^\n]+\})/);
   if (!m) return null;
-  return `[SYSTEM] edit_file thất bại vì old_string sai. Bạn PHẢI gọi đúng tool này ngay: ${m[1]}. KHÔNG gọi edit_file lại trước khi đọc file. Sau đó copy đúng text từ read_file mới để retry.`;
+  return `[TOOL GUIDANCE] edit_file thất bại vì old_string sai. Bạn PHẢI gọi đúng tool này ngay: ${m[1]}. KHÔNG gọi edit_file lại trước khi đọc file. Sau đó copy đúng text từ read_file mới để retry.`;
 }
 
 export function duplicateToolGuidance(name, input) {
-  return `[SYSTEM] Tool call vừa rồi TRÙNG HỆT lần trước: ${name} ${JSON.stringify(input || {})}. KHÔNG chạy tool này lại. Dùng kết quả đã có trong history; nếu cần tiến thêm thì gọi tool khác (grep/read_file/edit_file/run_command) hoặc trả lời Markdown nếu xong.`;
+  return `[TOOL GUIDANCE] Tool call vừa rồi TRÙNG HỆT lần trước: ${name} ${JSON.stringify(input || {})}. KHÔNG chạy tool này lại. Dùng kết quả đã có trong history; nếu cần tiến thêm thì gọi tool khác (grep/read_file/edit_file/run_command) hoặc trả lời Markdown nếu xong.`;
 }
 
 // Detect câu trả lời bị cắt giữa chừng — KHÔNG phải câu hoàn chỉnh.
@@ -722,7 +722,7 @@ export async function runAgent({
     });
     const toolOk = allow && !String(result || '').startsWith('ERROR:');
     const errorNudge = allow ? toolErrorGuidance(call.name, { result }) : null;
-    if (errorNudge) history.push({ role: 'user', content: errorNudge });
+    if (errorNudge) history.push({ role: 'tool', name: 'tool_guidance', content: errorNudge });
 
     // ── Todo continuation nudge ──────────────────────────────────────────
     // Sau mỗi tool result, inject nudge nếu còn task chưa xong.
@@ -730,7 +730,7 @@ export async function runAgent({
     {
       const tasks = pendingTasks || [];
       const msg = todoContinuationMessage(call.name, toolOk, tasks);
-      if (msg) history.push({ role: 'user', content: msg });
+      if (msg) history.push({ role: 'tool', name: 'todo_continuation', content: msg });
     }
 
     // ── Loop detection ──────────────────────────────────────────────────