@noobdemon/noob-cli 1.10.6 → 1.10.8

package/package.json

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

package/src/agent.js

@@ -496,6 +496,8 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
   // 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
+  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
   // 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 || "");
@@ -586,8 +588,8 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
 
     // ── Loop detection ──────────────────────────────────────────────────
     // Theo dõi N tool call gần nhất. Nếu cùng tên + input giống nhau liên tiếp
-    // quá ngưỡng → model bị kẹt. Inject thông báo nhắc model chuyển bước,
-    // KHÔNG ngắt task (vẫn có thể Ctrl+C thủ công).
+    // quá ngưỡng → model bị kẹt. Inject thông báo nhắc model chuyển bước.
+    // Nếu model vẫn lặp → force stop sau MAX_LOOP_DETECTIONS lần.
     const inputStr = JSON.stringify(call.input || {});
     recentCalls.push({ name: call.name, inputStr });
     if (recentCalls.length > LOOP_DETECT_WINDOW) recentCalls.shift();
@@ -595,13 +597,25 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
       const lastN = recentCalls.slice(-LOOP_DETECT_THRESHOLD);
       const allSame = lastN.every((c) => c.name === lastN[0].name && c.inputStr === lastN[0].inputStr);
       if (allSame) {
+        loopDetectedCount++;
+        // Đã lặp quá nhiều lần liên tiếp → force stop
+        if (loopDetectedCount >= MAX_LOOP_DETECTIONS) {
+          history.push({
+            role: "tool",
+            name: "loop_detection",
+            content: `[LOOP DETECTED × ${loopDetectedCount}] Bạn đã gọi ${lastN[0].name} nhiều lần liên tiếp với cùng input — KHÔNG THỂ TIẾP TỤC. Dừng ngay lập tức.`,
+          });
+          return `[LOOP STOPPED] Đã dừng vì model bị kẹt trong vòng lặp gọi ${lastN[0].name} liên tục.`;
+        }
         // Đã lặp — inject cảnh báo, xoá calls để tránh trigger liên tục
         recentCalls.length = 0;
         history.push({
           role: "tool",
           name: "loop_detection",
-          content: `[LOOP DETECTED] Bạn vừa gọi ${lastN[0].name} ${LOOP_DETECT_THRESHOLD} lần liên tiếp với cùng input — có vẻ bạn đang bị kẹt trong vòng lặp. HÃY CHUYỂN BƯỚC: gọi tool khác, hoặc trả lời Markdown nếu đã xong. KHÔNG được gọi cùng tool cùng input lần nữa.`,
+          content: `[LOOP DETECTED × ${loopDetectedCount}] Bạn vừa gọi ${lastN[0].name} ${LOOP_DETECT_THRESHOLD} lần liên tiếp với cùng input — có vẻ bạn đang bị kẹt trong vòng lặp. HÃY CHUYỂN BƯỚC: gọi tool khác, hoặc trả lời Markdown nếu đã xong. KHÔNG được gọi cùng tool cùng input lần nữa. Nếu tiếp tục lặp, task sẽ bị dừng.`,
         });
+      } else {
+        loopDetectedCount = 0; // model gọi tool khác → reset counter
       }
     }
   }

package/src/api.js

@@ -116,7 +116,7 @@ function hasUnclosedToolBlock(text) {
  *
  * @returns {Promise<{text:string, reasoning:string}>}
  */
-export async function stream({ mode = "chat", message, model, system, conversation, effort, signal, onDelta, onReasoning, onStatus, idleMs = 45000, maxContinues = Infinity }) {
+export async function stream({ mode = "chat", message, model, system, conversation, effort, signal, onDelta, onReasoning, onStatus, idleMs = 240000, maxContinues = Infinity }) {
   const endpoint = mode === "search" ? "/api/search" : mode === "merge" ? "/api/merge" : "/api/chat";
 
   let fullText = "";

package/src/repl.js

@@ -1351,75 +1351,6 @@ NGUYÊN TẮC:
       if ((!printer || !printer.started) && answer?.trim())
         printAnswer(answer, state.model.name, providerColor(state.model.provider));
 
-      // ── Idle verification: model dừng → gọi model check task hoàn thành chưa ───
-      // Nếu chưa xong → chạy runAgent tiếp với prompt verify
-      // Giới hạn 3 lần để tránh infinite loop
-      const MAX_IDLE_VERIFY = 3;
-      let idleVerifyCount = 0;
-      while (idleVerifyCount < MAX_IDLE_VERIFY) {
-        // Kiểm tra xem model có thực sự xong không
-        const verifyPrompt = `[SYSTEM] Bạn vừa dừng lại. Hãy KIỂM TRA LẠI xem task đã hoàn thành chưa:
-1. Đọc file kết quả (nếu có) bằng read_file để xác nhận nội dung
-2. Chạy test/build nếu task liên quan code
-3. So sánh với yêu cầu ban đầu của user
-
-Nếu thực sự HOÀN THÀNH → trả lời "XÁC NHẬN: [tóm tắt ngắn những gì đã làm]"
-Nếu CHƯA XONG → gọi tool (write_file/edit_file/run_command) để hoàn thành tiếp. KHÔNG dừng.`;
-
-        state.history.push({ role: "user", content: verifyPrompt });
-        
-        const nextAnswer = await runAgent({
-          history: state.history,
-          model: state.model.id,
-          signal: abort.signal,
-          tokenMeter,
-          goal: state.goal,
-          recentSessions: sessions.list(5, process.cwd()).filter((s) => s.id !== session?.id),
-          extraToolsDoc: state.agentMode ? spawnAgentToolsDoc(0) : "",
-          pendingTasks: (state.todos || []).filter((t) => !t.done).map((t) => t.text),
-          onStatus: () => tick(t.thinking),
-          onSteer: () => {
-            if (!pending.length) return [];
-            const msgs = pending.splice(0);
-            stopSpin();
-            for (const msg of msgs) console.log(c.user("  " + t.steerInject(truncate(msg, 70))));
-            startSpin(t.thinking);
-            return msgs;
-          },
-          onDelta: (ev) => {
-            if (ev.type === "step-start") {
-              printer = makeStreamPrinter(state.model.name, providerColor(state.model.provider));
-            } else if (ev.type === "delta") {
-              if (printer.suppressing) return printer.push(ev.text);
-              stopSpin();
-              printer.push(ev.text);
-              if (printer.suppressing) startSpin(t.thinking);
-            } else if (ev.type === "step-end") {
-              printer?.flush();
-            }
-          },
-          onTool: (name, input) => dispatchTool(name, input, 0),
-        });
-
-        if (!nextAnswer?.trim()) {
-          idleVerifyCount++;
-          continue;
-        }
-
-        // In response nếu có
-        if ((!printer || !printer.started) && nextAnswer?.trim())
-          printAnswer(nextAnswer, state.model.name, providerColor(state.model.provider));
-
-        // Check nếu model confirm "XÁC NHẬN"
-        if (/XÁC NHẬN/i.test(nextAnswer)) {
-          answer = nextAnswer;
-          break;
-        }
-
-        idleVerifyCount++;
-        // Nếu model gọi tool thì tiếp tục loop (sẽ được xử lý bởi runAgent tiếp theo)
-      }
-
       // Parse todo từ model output → render trên status bar.
       state.todos = parseTodosFromHistory(state.history);
       tui.setTodos(state.todos);