@noobdemon/noob-cli 1.9.1 → 1.9.3

package/package.json

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

package/src/agent.js

@@ -43,6 +43,7 @@ Context is finite. Don't slurp the whole repo up front. Discover information pro
   • A server/service the USER wants to keep using (they asked to "run the app/server") → leave it running, and tell the user it is up: its id and the URL/port, and that they can ask you to stop it when done (you will kill_bg it). It also stops automatically when noob exits.
   • If bg_output shows it exited with a non-zero code or an error, treat it like a failed command: read the output and fix, don't silently move on.
 - Keep prose tight. Explain what you did and why, not how to use a tool.
+- Do NOT call the same tool with the same input repeatedly. The runtime detects loops and will force a step change. If you need to re-read a file, use the data already in your history.
 - JSON in the tool block must be valid: escape newlines as \\n inside string values.
 - LANGUAGE: Always write your prose answers to the user in Vietnamese (tiếng Việt), unless the user explicitly writes in another language. Keep code, file paths, commands, and tool JSON unchanged.
 
@@ -77,6 +78,11 @@ Follow this pattern exactly. Your very first response to a task that needs the f
 // Số bước tool tối đa cho một lượt. Đặt rất cao theo yêu cầu người dùng: task
 // dài cứ chạy, đừng tự dừng. Người dùng vẫn có thể Ctrl+C bất cứ lúc nào.
 const MAX_STEPS = 10000;
+
+// Loop detection: nếu model gọi cùng 1 tool với input giống nhau liên tiếp
+// quá ngưỡng này → coi là bị kẹt, inject thông báo để model chuyển bước.
+const LOOP_DETECT_WINDOW = 3;
+const LOOP_DETECT_THRESHOLD = 2;
 const MAX_PROMPT_CHARS = 80000; // ngân sách ký tự cho phần hội thoại gửi lên model
 // Khi history vượt ngưỡng này, gọi model phụ tóm tắt các lượt cũ thay vì cắt cụt
 // → giữ được "trí nhớ dài hạn" trong phiên mà không nổ context.
@@ -408,6 +414,7 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
   // [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
   for (let step = 0; step < MAX_STEPS; step++) {
     // Mỗi 100 bước log một mốc để người dùng biết noob vẫn đang chạy (task dài).
     if (step > 0 && step % 100 === 0) onStatus?.(`đã chạy ${step} bước…`);
@@ -460,6 +467,27 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
       name: call.name,
       content: allow ? result : t.toolDenied,
     });
+
+    // ── 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).
+    const inputStr = JSON.stringify(call.input || {});
+    recentCalls.push({ name: call.name, inputStr });
+    if (recentCalls.length > LOOP_DETECT_WINDOW) recentCalls.shift();
+    if (recentCalls.length >= LOOP_DETECT_THRESHOLD + 1) {
+      const lastN = recentCalls.slice(-LOOP_DETECT_THRESHOLD);
+      const allSame = lastN.every((c) => c.name === lastN[0].name && c.inputStr === lastN[0].inputStr);
+      if (allSame) {
+        // Đã 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.`,
+        });
+      }
+    }
   }
   return t.maxSteps;
 }
@@ -494,7 +522,7 @@ async function streamWithRetry({ model, message, system, signal, tokenMeter, onD
       lastErr = err;
       if (attempt >= MAX_RETRIES) break;
       const backoff = Math.min(30000, 1000 * Math.pow(2, attempt));
-      onStatus?.(`mạng lỗi (${err.message}) — thử lại sau ${(backoff/1000)|0}s [${attempt+1}/${MAX_RETRIES}]`);
+      onStatus?.(`Lỗi kết nối — thử lại sau ${(backoff/1000)|0}s [${attempt+1}/${MAX_RETRIES}]…`);
       await sleep(backoff, signal);
     }
   }

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 = 25000, maxContinues = Infinity }) {
+export async function stream({ mode = "chat", message, model, system, conversation, effort, signal, onDelta, onReasoning, onStatus, idleMs = 45000, maxContinues = Infinity }) {
   const endpoint = mode === "search" ? "/api/search" : mode === "merge" ? "/api/merge" : "/api/chat";
 
   let fullText = "";
@@ -209,9 +209,9 @@ async function streamOnce({ endpoint, mode, message, model, system, conversation
   let warnTimer = null;
   let probeTimer = null;
   let probeInFlight = false;
-  const WARN_MS = Math.min(8000, idleMs / 3);
-  const PROBE_MS = Math.min(12000, (idleMs * 2) / 3);
-  const WIRE_MS = idleMs * 2; // socket dead-cứng (mất cả heartbeat) — ngưỡng rộng
+  const WARN_MS = Math.min(15000, idleMs / 3);
+  const PROBE_MS = Math.min(25000, (idleMs * 2) / 3);
+  const WIRE_MS = idleMs * 3; // socket dead-cứng (mất cả heartbeat) — ngưỡng rất rộng
   const clearContentTimers = () => {
     clearTimeout(warnTimer); warnTimer = null;
     clearTimeout(probeTimer); probeTimer = null;
@@ -220,7 +220,7 @@ async function streamOnce({ endpoint, mode, message, model, system, conversation
   const armContent = () => {
     clearContentTimers();
     warnTimer = setTimeout(() => {
-      if (onStatus) onStatus("Đang chờ proxy phản hồi…");
+      if (onStatus) onStatus("Model đang suy nghĩ… (đợi thêm nếu task phức tạp)");
     }, WARN_MS);
     probeTimer = setTimeout(async () => {
       if (probeInFlight || ctrl.signal.aborted) return;
@@ -234,7 +234,7 @@ async function streamOnce({ endpoint, mode, message, model, system, conversation
       } catch {
         if (!ctrl.signal.aborted) {
           timedOut = true;
-          if (onStatus) onStatus("Proxy không phản hồi — đang gọi lại model…");
+          if (onStatus) onStatus("Model không phản hồi — đang kết nối lại…");
           ctrl.abort();
         }
       } finally {
@@ -328,7 +328,7 @@ async function streamOnce({ endpoint, mode, message, model, system, conversation
     return { text, reasoning, truncated };
   } catch (err) {
     if (signal?.aborted) throw err; // người dùng bấm Ctrl+C → huỷ thật, không nối tiếp
-    if (timedOut) throw new ApiError("Kết nối tới máy chủ quá thời gian chờ (treo).", { code: "timeout" });
+    if (timedOut) throw new ApiError("Kết nối tới máy chủ quá thời gian chờ (treo). Đang thử lại…", { code: "timeout" });
     // Rớt mạng giữa chừng (không phải huỷ, không phải treo): với chat, nếu đã có
     // chữ thì trả phần đã nhận + cờ truncated để lớp trên nối tiếp.
     if (mode === "chat" && text) return { text, reasoning, truncated: true };