@noobdemon/noob-cli 1.13.0 → 1.13.1

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 Tất cả thay đổi đáng kể của `@noobdemon/noob-cli` được ghi vào file này.
 
+## [1.13.1] - 2026-06-28
+
+### Fixed
+- **Text-loop detection** (`src/agent.js`): model lặp cùng 1 câu liên tiếp mà không gọi tool (vd "Tôi sẽ bắt đầu bằng việc khảo sát workspace" 40 lần) — phát hiện ≥2 lần trùng → nudge `[TEXT LOOP]`, ≥3 lần → force stop `[LOOP STOPPED]`. Reset khi model gọi tool thành công.
+- **Duplicate tool call block trước execution** (`src/agent.js`): kiểm tra `duplicateToolGuidance` chạy TRƯỚC `onTool()` (chỉ post-execution trước đó).
+- **Edit-file retry guidance** (`src/tools.js` + `src/agent.js`): lỗi `old_string not found` giờ inject `NEXT REQUIRED TOOL: read_file {path,offset,limit}` + `toolErrorGuidance` system nudge ép model đọc lại file trước khi retry.
+- **Todo nudge phân biệt success/failure** (`src/agent.js`): `todoContinuationMessage` nói "vừa LỖI" khi tool trả `ERROR:`, không đẩy model lặp lại thao tác thất bại.
+- **nearbyContext mở rộng** (`src/tools.js`): context lines khi edit_file lỗi scales với độ dài `old_string` (`Math.max(8, oldLineCount + 5)`).
+
+### Verified
+- `npm test` 109/109 pass.
+
 ## [1.13.0] - 2026-06-27
 
 ### Fixed

package/package.json

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

package/src/agent.js

@@ -475,6 +475,24 @@ export function buildUserMessage(history) {
   return parts.join('\n');
 }
 
+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.`;
+}
+
+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.`;
+}
+
+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.`;
+}
+
 // Detect câu trả lời bị cắt giữa chừng — KHÔNG phải câu hoàn chỉnh.
 // Trả true nếu text kết thúc đột ngột (thiếu dấu câu, list chưa đóng, v.v.).
 function isIncompleteResponse(text) {
@@ -578,9 +596,13 @@ export async function runAgent({
   // [GỠ BUDGET 2026-06-06] Không còn token budget enforcement. Agent/loop/sub-agent
   // chạy không giới hạn token. Dừng theo: GOAL đạt, <<LOOP_DONE>>, <<ULTRA_DONE>>,
   // model tự kết thúc reply không có tool block, hoặc user Ctrl+C.
-  const recentCalls = []; // {name, inputStr} — theo dõi vòng lặp
+  const recentCalls = []; // {name, inputStr} — theo dõi vòng lặp tool call
   let loopDetectedCount = 0; // số lần loop detection liên tiếp — reset khi model gọi tool khác
   const MAX_LOOP_DETECTIONS = 3; // sau 3 lần loop detection liên tiếp → force stop
+  // ponytail: text-loop detection — model lặp cùng 1 câu mà không gọi tool
+  let lastTextHash = '';
+  let textRepeatCount = 0;
+  const MAX_TEXT_REPEATS = 3; // cùng text 3 lần liên tiếp → force stop
   // Effort classifier: phân loại task từ user message gốc → set effort level.
   // Chỉ classify 1 lần ở bước đầu, giữ nguyên suốt task (thay đổi giữa chừng gây bất ổn).
   const effort = classifyEffort(history.find((m) => m.role === 'user')?.content || '');
@@ -654,29 +676,59 @@ export async function runAgent({
         });
         continue;
       }
+      // ponytail: text-loop detection — model lặp cùng 1 câu không tool
+      const textHash = text.trim().slice(0, 200);
+      if (textHash === lastTextHash) {
+        textRepeatCount++;
+        if (textRepeatCount >= MAX_TEXT_REPEATS) {
+          history.push({
+            role: 'tool',
+            name: 'loop_detection',
+            content: `[TEXT LOOP × ${textRepeatCount}] Bạn vừa nói cùng 1 câu ${textRepeatCount} lần liên tiếp mà KHÔNG gọi tool. DỪNG NGAY. Gọi read_file/list_dir/grep để thực sự bắt đầu, hoặc trả lời Markdown nếu đã xong.`,
+          });
+          return `[LOOP STOPPED] Model kẹt trong vòng lặp text: "${text.slice(0, 80)}…"`;
+        }
+        history.push({
+          role: 'tool',
+          name: 'loop_detection',
+          content: `[TEXT LOOP × ${textRepeatCount}] Bạn vừa lặp lại câu trả lời giống hệt. Hãy gọi tool (read_file/grep/list_dir) hoặc trả lời Markdown khác đi. KHÔNG nói lại câu cũ.`,
+        });
+        continue; // quay lại loop, ép model gọi tool
+      }
+      lastTextHash = textHash;
+      textRepeatCount = 0;
       // Model dừng (không tool call, không incomplete) → return để repl quyết định tiếp tục hay không
       return text; // final answer
     }
 
+    const inputStr = JSON.stringify(call.input || {});
+    const prev = recentCalls[recentCalls.length - 1];
+    if (prev && prev.name === call.name && prev.inputStr === inputStr) {
+      history.push({
+        role: 'tool',
+        name: 'loop_detection',
+        content: duplicateToolGuidance(call.name, call.input),
+      });
+      continue;
+    }
+
     const { allow, result } = await onTool(call.name, call.input);
     history.push({
       role: 'tool',
       name: call.name,
       content: allow ? result : t.toolDenied,
     });
+    const toolOk = allow && !String(result || '').startsWith('ERROR:');
+    const errorNudge = allow ? toolErrorGuidance(call.name, { result }) : null;
+    if (errorNudge) history.push({ role: 'user', content: errorNudge });
 
     // ── Todo continuation nudge ──────────────────────────────────────────
     // Sau mỗi tool result, inject nudge nếu còn task chưa xong.
     // Dùng pendingTasks (caller gửi vào) thay vì parse output của model.
     {
       const tasks = pendingTasks || [];
-      if (tasks.length > 0) {
-        const next = tasks[0];
-        history.push({
-          role: 'user',
-          content: `[SYSTEM] Việc "${call.name}" đã hoàn thành. 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.`,
-        });
-      }
+      const msg = todoContinuationMessage(call.name, toolOk, tasks);
+      if (msg) history.push({ role: 'user', content: msg });
     }
 
     // ── Loop detection ──────────────────────────────────────────────────
@@ -685,7 +737,6 @@ export async function runAgent({
     //   (B) Pattern vòng (A-B-A-B, A-B-C-A-B-C) — model xen kẽ 2-3 tool
     //       khác nhau để tránh phát hiện cũ. So nửa đầu vs nửa cuối window.
     // Nếu phát hiện → inject cảnh báo. Nếu tái diễn → force stop.
-    const inputStr = JSON.stringify(call.input || {});
     recentCalls.push({ name: call.name, inputStr });
     if (recentCalls.length > LOOP_DETECT_WINDOW) recentCalls.shift();
     let loopType = null; // 'consecutive' | 'pattern' | null
@@ -749,6 +800,8 @@ export async function runAgent({
       });
     } else {
       loopDetectedCount = 0; // tool khác → reset
+      lastTextHash = ''; // text-loop: reset sau tool call thành công
+      textRepeatCount = 0;
     }
   }
   return t.maxSteps;

package/src/tools.js

@@ -408,7 +408,7 @@ export const TOOLS = {
     // Không thấy → lỗi GIÀU THÔNG TIN: cho model thấy đúng byte trong file để sửa.
     throw new Error(
       `old_string not found in ${rel(file)}. Copy the target text EXACTLY (indentation/whitespace included, NO line-number prefix).` +
-        nearbyContext(data, old_string)
+        nearbyContext(data, old_string, rel(file))
     );
   },
 
@@ -730,7 +730,8 @@ function matchByLines(data, oldStr) {
 
 // Khi không khớp: in vùng file gần dòng giống nhất, dạng JSON-escaped để model
 // thấy rõ tab/space → sửa old_string cho khớp ngay lần sau.
-function nearbyContext(data, oldStr) {
+function nearbyContext(data, oldStr, p) {
+  const oldLineCount = oldStr.replace(/\r\n/g, '\n').split('\n').length;
   const want = (
     oldStr
       .replace(/\r\n/g, '\n')
@@ -756,14 +757,19 @@ function nearbyContext(data, oldStr) {
     }
   }
   if (hit < 0 || best < 6)
-    return ` (no similar line found; the file has ${lines.length} lines — re-read it.)`;
+    return ` (no similar line found; the file has ${lines.length} lines — re-read it.)${readFileRetryHint(p, 1, Math.min(lines.length, 80))}`;
   const a = Math.max(0, hit - 2);
-  const b = Math.min(lines.length, hit + 3);
+  const b = Math.min(lines.length, hit + Math.max(8, oldLineCount + 5));
   const snippet = lines
     .slice(a, b)
     .map((l, k) => `  ${a + k + 1}: ${JSON.stringify(l)}`)
     .join('\n');
-  return `\nActual file lines near the closest match (JSON-escaped — match these bytes EXACTLY):\n${snippet}`;
+  return `\nActual file lines near the closest match (JSON-escaped — match these bytes EXACTLY):\n${snippet}${readFileRetryHint(p, a + 1, b - a)}`;
+}
+
+function readFileRetryHint(pathHint, offset, limit) {
+  const pathPart = `"path":${JSON.stringify(pathHint || '<same path>')},`;
+  return `\nNEXT REQUIRED TOOL: read_file {${pathPart}"offset":${offset},"limit":${limit}}. Then retry edit_file using text copied from that fresh read_file result.`;
 }
 
 function globToRegExp(glob) {