@noobdemon/noob-cli 1.13.2 → 1.13.4

package/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 Tất cả thay đổi đáng kể của `@noobdemon/noob-cli` được ghi vào file này.
 
+## [1.13.4] - 2026-06-29
+
+### Added
+- **Ctrl+Backspace / Ctrl+W xoá nguyên từ** (`src/tui.js`): trên thanh nhập, Ctrl+Backspace (`\x08`/Ctrl+H ở phần lớn terminal) và Ctrl+W xoá cả từ trước con trỏ thay vì từng ký tự. Dùng cùng quy ước ranh giới từ với di chuyển từ (khoảng trắng + chip ảnh dán tính là ranh giới).
+
+### Changed
+- **Chặn slash command khi model đang chạy** (`src/repl.js` + `src/i18n.js`): gõ slash (`/kc`, `/model`, …) lúc task đang chạy không còn bị xếp hàng / chèn nhầm cho AI — in cảnh báo `⏳ model đang chạy — chờ xong task mới dùng được lệnh` và bỏ qua. Tin nhắn thường vẫn xếp hàng/chèn cho AI như cũ.
+
+### Verified
+- `npm test` 109/109 pass.
+
+## [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.4",
   "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 ──────────────────────────────────────────────────

package/src/i18n.js

@@ -12,6 +12,7 @@ export const t = {
   running: 'đang chạy…',
   denied: 'đã từ chối',
   queued: (n, txt) => `⏎ đã xếp hàng [${n}] · gửi khi model xong: ${txt}`,
+  slashBusy: '⏳ model đang chạy — chờ xong task mới dùng được lệnh (Ctrl+C để dừng)',
   queueCleared: (n) => `(đã xoá ${n} tin đang xếp hàng)`,
   steerHint:
     '💬 Gõ + Enter bất cứ lúc nào để chèn ý cho AI giữa chừng (không ngắt task đang chạy).',

package/src/repl.js

@@ -164,6 +164,12 @@ export async function startRepl(opts = {}) {
     onLine: (line) => {
       // Submit khi KHÔNG có read() đang chờ = tin xếp hàng. Đang chạy task → sẽ
       // CHÈN cho AI ở bước kế tiếp (steering); rảnh → gửi như lượt mới.
+      // ponytail: slash command lúc busy KHÔNG vào pending — chặn, bắt user chờ
+      // model xong task mới dùng lệnh (tránh lệnh bị xếp hàng / chèn nhầm cho AI).
+      if (abort && line.trim().startsWith('/')) {
+        tui.print(c.dim('  ' + t.slashBusy));
+        return;
+      }
       pending.push(line);
       tui.print(
         abort
@@ -1869,7 +1875,16 @@ NGUYÊN TẮC:
         return await execToolCore(name, input, { retried: true });
       }
       console.log(c.err('  ✗ ' + err.message));
-      return { allow: true, result: 'ERROR: ' + err.message };
+      // Framing: lỗi tool = GỌI SAI CÁCH, không phải mất quyền. Model có lịch sử
+      // diễn giải nhầm 'ERROR:' thành 'mình không có quyền truy cập tool' rồi bỏ
+      // cuộc. Nói rõ model VẪN có quyền, chỉ cần sửa tham số rồi gọi lại.
+      return {
+        allow: true,
+        result:
+          'ERROR: ' +
+          err.message +
+          '\n[Đây là lỗi do tham số gọi tool chưa đúng, KHÔNG phải mất quyền truy cập. Bạn VẪN có quyền chạy tool này — đọc kỹ thông báo lỗi, sửa tham số (path/old_string/...) rồi GỌI LẠI. Đừng kết luận là không truy cập được.]',
+      };
     }
   }
 

package/src/tools.js

@@ -373,7 +373,10 @@ export const TOOLS = {
     checkAbort(signal);
     const file = abs(p);
     const data = await fs.readFile(file, 'utf8');
-    if (old_string === new_string) throw new Error('old_string and new_string are identical');
+    if (old_string === new_string)
+      throw new Error(
+        `old_string and new_string are identical in ${rel(file)} — no change to make. This is a CALL mistake, NOT a permissions/access problem: you DO have edit access. Set new_string to the DIFFERENT text you want, then call edit_file again.`
+      );
     const useCRLF = data.includes('\r\n');
     const adapt = (s) => {
       const lf = s.replace(/\r\n/g, '\n');

package/src/tui.js

@@ -322,9 +322,7 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, onCtrlT, com
     if (x.image) return c.dim(imageLabel(x));
     if (x.paste === undefined) return x.c;
     const preview = pastePreview(x.paste);
-    const label = preview
-      ? `[pasted ${x.lines} lines: "${preview}"]`
-      : `[pasted ${x.lines} lines]`;
+    const label = preview ? `[pasted ${x.lines} lines: "${preview}"]` : `[pasted ${x.lines} lines]`;
     return c.dim(label);
   };
   const coloredInput = () => {
@@ -424,7 +422,8 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, onCtrlT, com
         let body = l;
         if (i === curLine) {
           if (colInLine >= l.length) body = l + `${REV} ${UNREV}`;
-          else body = l.slice(0, colInLine) + `${REV}${l[colInLine]}${UNREV}` + l.slice(colInLine + 1);
+          else
+            body = l.slice(0, colInLine) + `${REV}${l[colInLine]}${UNREV}` + l.slice(colInLine + 1);
         }
         return (i === 0 ? promptLabel : indent) + body;
       })
@@ -716,6 +715,19 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, onCtrlT, com
       histPos = null;
     }
   }
+  // Ctrl+Backspace / Ctrl+W: xoá nguyên từ trước con trỏ (cùng quy ước ranh giới
+  // từ với moveWordLeft — khoảng trắng + chip dán tính là ranh giới).
+  function deleteWordLeft() {
+    const start = cur;
+    while (cur > 0 && charAt(cur - 1) === ' ') cur -= 1;
+    if (cur > 0 && charAt(cur - 1) === null)
+      cur -= 1; // qua 1 chip
+    else while (cur > 0 && charAt(cur - 1) !== null && charAt(cur - 1) !== ' ') cur -= 1;
+    if (cur < start) {
+      cells.splice(cur, start - cur);
+      histPos = null;
+    }
+  }
   // null = ô dán (coi như ranh giới từ); ngược lại trả ký tự của ô.
   const charAt = (k) =>
     cells[k] && cells[k].paste === undefined && !cells[k].image ? cells[k].c : null;
@@ -923,7 +935,15 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, onCtrlT, com
         i++;
         continue;
       }
-      if (ch === '\x7f' || ch === '\b') {
+      if (ch === '\x08' || ch === '\x17') {
+        // Ctrl+Backspace (nhiều terminal gửi \x08 = Ctrl+H) và Ctrl+W: xoá 1 từ.
+        deleteWordLeft();
+        refreshMenu();
+        draw();
+        i++;
+        continue;
+      } // Ctrl+Backspace / Ctrl+W
+      if (ch === '\x7f') {
         backspace();
         refreshMenu();
         draw();