@noobdemon/noob-cli 1.9.5 → 1.9.7

package/package.json

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

package/src/agent.js

@@ -33,7 +33,7 @@ Available tools (each is self-contained; pick the SMALLEST tool that answers the
 Context is finite. Don't slurp the whole repo up front. Discover information progressively: list_dir/glob to map → grep to locate → read_file (with offset+limit for big files) to inspect only what matters. Each tool result spends your attention budget — make every call earn it. When a tool returns a huge blob, extract the few facts you need, then move on; don't re-read it later (the result stays in history).
 
 # Rules
-- TODO-BASED EXECUTION: For any multi-step task (3+ actions), CREATE a todo list FIRST as your very first tool call using write_file to a temp block in your response (format: "- [ ] item"). Then WORK THROUGH EVERY ITEM, checking them off ("- [x]") as you complete each. BEFORE summarizing or claiming "done", mentally verify: "Have I checked off ALL items? Is there anything left unchecked?" If ANY item remains unchecked, CONTINUE — do not stop. If the user's request implies multiple deliverables, treat each as a TODO item. NEVER stop mid-plan. NEVER assume something is done without a tool result proving it.
+- TODO-BASED EXECUTION: For any multi-step task (3+ actions), CREATE a todo list FIRST in your response text (NOT in a file — just write them as "- [ ] task name" in your reply). Then WORK THROUGH EVERY ITEM, checking them off ("- [x] task name") as you complete each. The runtime parses your text for these markers and shows a progress bar to the user. BEFORE summarizing or claiming "done", verify ALL items are "- [x]". If ANY remain unchecked, CONTINUE — do not stop. NEVER stop mid-plan.
 - GROUND TRUTH = real TOOL RESULTs in this conversation, not your memory or what you intended to do. A file changed only if a write_file/edit_file result confirms it (see the FILES CHANGED list). A test passed / build succeeded / command worked only if a run_command result above shows it. Never narrate outcomes you didn't observe; if you haven't checked, say so and check now (read_file / list_dir / run the command). Before any "done/summary" reply, reconcile every file and result you're about to claim against the actual tool results above — if it isn't there, you didn't do it yet.
 - Investigate before editing: read the relevant files first; never invent file contents.
 - Make the smallest change that fully solves the task. Match the surrounding code style.
@@ -416,6 +416,25 @@ function buildUserMessage(history) {
   return parts.join("\n");
 }
 
+// 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) {
+  if (!text) return false;
+  const t = text.trimEnd();
+  // Kết thúc bằng dấu cuối danh sách chưa đóng: "A.", "B.", "1.", "(1)" mà không có gì sau
+  if (/[A-Z]\.\s*$/.test(t) || /\(\d+\)\s*$/.test(t) || /^\d+\.\s*$/m.test(t)) return true;
+  // Kết thúc giữa câu — không có dấu câu cuối cùng (. ! ? : ; ) và không phải markdown/code
+  const lastChar = t.slice(-1);
+  if (lastChar && !/[.!?:;)\]"'`#>\n]/.test(lastChar) && t.length > 50) {
+    // Kiểm thêm: dòng cuối chứa từ khóa bị cắt (ví dụ, vd, hay, hoặc, và, nhưng, mà)
+    const lastLine = t.split("\n").pop().trim();
+    if (/\s(ví|vd|hay|hoặc|và|nhưng|mà|hoac|or|and|but|e\.g|i\.e)\s*$/i.test(lastLine)) return true;
+    // Dòng cuối là 1 câu bắt đầu nhưng chưa xong (có chủ ngữ nhưng không có vị ngữ hoàn chỉnh)
+    if (/\s(Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn|Bạn)\s+(muốn|có|thấy|nên|cần|đã|đang|sẽ|chọn|chọn|chọn)\s*$/i.test(lastLine)) return true;
+  }
+  return false;
+}
+
 // Extract a single tool call from an assistant message, if present.
 // NOTE: we do NOT match up to a closing ``` fence — write_file content routinely
 // contains its own ```code``` fences (e.g. a README), and the first inner fence
@@ -523,6 +542,16 @@ export async function runAgent({ history, model, signal, onTool, onStatus, onDel
         });
         continue;
       }
+      // Phát hiện câu trả lời bị cắt giữa chừng (truncated stream): text kết thúc
+      // đột ngột không có dấu câu/đóng danh sách/tool block → nudge model viết tiếp.
+      if (isIncompleteResponse(text)) {
+        history.push({
+          role: "tool",
+          name: "stream_recovery",
+          content: "[STREAM CUT] Lượt vừa bị cắt giữa chừng. Hãy viết TIẾP liền mạch — nếu đang hỏi user thì hoàn tất câu hỏi, nếu đang list thì đóng danh sách, nếu đang viết tool block thì đóng đúng JSON.",
+        });
+        continue;
+      }
       return text; // final answer
     }
 

package/src/repl.js

@@ -146,6 +146,7 @@ export async function startRepl(opts = {}) {
     extraRoots: [], // thư mục ngoài cwd được /add-dir cấp quyền (UX display only;
                     // source of truth là extraRoots trong src/tools.js)
     _longSessionWarned: false, // đã in cảnh báo phiên dài chưa (reset khi /clear)
+    todos: [], // [{text, done}] — todo list parse từ model output, render trên status bar
   };
   const tokenMeter = new TokenMeter();
 
@@ -365,6 +366,9 @@ export async function startRepl(opts = {}) {
     session = sessions.newSession({ cwd: process.cwd(), model: state.model.id });
     // Reset per-session upstream memory token so the next chat starts fresh.
     resetMemoryToken();
+    // Clear todo list khi phiên mới.
+    state.todos = [];
+    tui.setTodos([]);
   };
 
   // /frontend-design <yêu cầu> — vận dụng skill frontend-design (skills/frontend-design/SKILL.md)
@@ -1326,6 +1330,9 @@ NGUYÊN TẮC:
       // Lượt cuối không stream ra gì (vd model suy luận trả 1 cục) → in fallback.
       if ((!printer || !printer.started) && answer?.trim())
         printAnswer(answer, state.model.name, providerColor(state.model.provider));
+      // Parse todo từ model output → render trên status bar.
+      state.todos = parseTodosFromHistory(state.history);
+      tui.setTodos(state.todos);
       return answer; // vòng ULTRA cần text này để dò token hoàn thành
     } catch (err) {
       stopSpin();
@@ -1386,6 +1393,29 @@ NGUYÊN TẮC:
     }
   }
 
+  // ── Todo parser ────────────────────────────────────────────────────────────
+  // Scan history assistant messages cho pattern `- [ ] task` và `- [x] task`.
+  // Trả về [{text, done}] — items mới nhất ở cuối. Dùng state mới nhất (cuối
+  // history) để反映todo hiện tại của model.
+  function parseTodosFromHistory(history) {
+    const todos = [];
+    for (const m of history) {
+      if (m.role !== "assistant" || typeof m.content !== "string") continue;
+      // Match todo items: `- [ ] task` hoặc `- [x] task` (case-insensitive)
+      const lines = m.content.split("\n");
+      for (const line of lines) {
+        const doneMatch = line.match(/^[\s]*-\s*\[x\]\s+(.+)/i);
+        if (doneMatch) { todos.push({ text: doneMatch[1].trim(), done: true }); continue; }
+        const todoMatch = line.match(/^[\s]*-\s*\[\s?\]\s+(.+)/);
+        if (todoMatch) { todos.push({ text: todoMatch[1].trim(), done: false }); }
+      }
+    }
+    // Dedupe: giữ item CUỐI cùng cho mỗi text (model có thể lặp todo)
+    const seen = new Map();
+    for (const t of todos) seen.set(t.text, t);
+    return [...seen.values()];
+  }
+
   async function execTool(name, input) {
     const desc = describe(name, input);
     const color = name === "run_command" ? "#ef4444" : "#f59e0b";

package/src/tui.js

@@ -73,6 +73,10 @@ function wrapText(text, width, maxLines) {
   return lines.map(close);
 }
 const FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+function truncStr(s, max) {
+  if (!s) return "";
+  return s.length > max ? s.slice(0, max - 1) + "…" : s;
+}
 
 export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, completer } = {}) {
   const tty = !!(process.stdout.isTTY && process.stdin.isTTY) && process.env.NOOB_TUI !== "0";
@@ -141,6 +145,7 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, completer }
   let busy = false;
   let busyLabel = "";
   let busyMeta = ""; // chuỗi phụ (elapsed + token) repl đẩy vào qua setMeta()
+  let todos = []; // [{text, done}] — danh sách todo đang chạy, repl parse từ model output
   let frame = 0;
   let frameTimer = null;
   let prevRows = 0;
@@ -311,6 +316,25 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, completer }
       return wrapText(liveOut, cols(), 2);
     }
     const spin = FRAMES[frame % FRAMES.length];
+    // Todo progress bar: hiện khi có todos, thay thế statusText/busyLabel
+    if (todos.length) {
+      const done = todos.filter((t) => t.done).length;
+      const total = todos.length;
+      const current = todos.find((t) => !t.done);
+      const pct = Math.round((done / total) * 100);
+      // Thanh progress: ████░░░░ 2/5 (40%)
+      const barW = Math.min(12, Math.floor(cols() * 0.15));
+      const filled = Math.round((done / total) * barW);
+      const bar = "█".repeat(filled) + "░".repeat(barW - filled);
+      const progress = c.ok(`${done}/${total}`) + c.dim(` ${bar} ${pct}%`);
+      // Task đang làm: ▸ ...
+      const taskLine = current
+        ? c.accent("▸ ") + c.dim(truncStr(current.text, cols() - 12))
+        : c.ok("✓ hoàn thành!");
+      const meta = busy && busyMeta ? c.dim(" · " + busyMeta) : "";
+      const tail = busy ? c.dim(" · Ctrl+C") : "";
+      return [progress + meta + tail, taskLine];
+    }
     // Khi busy, LUÔN hiện elapsed+tokens (busyMeta) cạnh statusText/busyLabel để
     // người dùng thấy phiên đang sống — kể cả lúc model im giữa các bước.
     if (statusText) {
@@ -637,6 +661,12 @@ export function createTui({ onLine, onInterrupt, onEOF, onShiftTab, completer }
       busyMeta = next;
       if (busy) draw();
     },
+    setTodos(items) {
+      // repl parse todo từ model output, đẩy vào TUI để render progress bar.
+      // items: [{text: string, done: boolean}] — empty array = ẩn todo bar.
+      todos = Array.isArray(items) ? items : [];
+      draw();
+    },
     setPrompt(label) {
       promptLabel = label || "";
       draw();