@noobdemon/noob-cli 1.12.0 → 1.12.1

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 Tất cả thay đổi đáng kể của `@noobdemon/noob-cli` được ghi vào file này.
 
+## [1.12.1] - 2026-06-11
+
+### Fixed
+- **TDZ ReferenceError khi workflow / tool out-of-scope ask permission** (`src/repl.js`): 3 helper `askPermission`/`askAddRoot`/`askWorkflowAgentMode` trước đây khai báo `const` rải rác (line ~405/1648/1655) — SAU chỗ gọi đầu tiên trong scope `startRepl` → JS Temporal Dead Zone throw `Cannot access 'askAddRoot' before initialization`. Exception bị `uncaughtException` handler nuốt im lặng → user gõ `y` mà flow chết, workflow approve xong không chạy. Fix: gom 3 const lên line 147 ngay sau `function ask`, trước mọi function khác. Verify bằng E2E smoke spawn bin thật (`scripts/smoke-tdz.mjs`, 3/3 PASS).
+- **Todo progress bar không reset khi chuyển task** (`src/repl/todos.js`): parser cũ scan toàn lịch sử rồi dedupe theo text → todo task A vẫn đếm vào bar dù model đã viết list mới cho task B (bar 5/10 nhưng reply chỉ có 3 todo). Fix: scan reverse, dừng ở assistant message **đầu tiên** có ≥1 dòng `- [ ]`/`- [x]` — coi đó là "todo list hiện tại", bỏ qua mọi message trước. Tự nhiên reset khi model viết list mới.
+
+### Changed
+- **Todo parser hardening** (`src/repl/todos.js`): skip dòng nằm trong code fence ` ``` ` (paste README/snippet có checkbox không bị bắt nhầm), chấp nhận cả `[X]` viết hoa (cờ `/i` cho cả `RE_DONE` và `RE_TODO`), bỏ qua message có content kiểu non-string (multimodal array).
+
 ## [1.12.0] - 2026-06-11
 
 ### Added

package/package.json

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

package/src/repl/todos.js

@@ -1,38 +1,79 @@
-// Parse danh sách todo từ history hội thoại. Scan các assistant message tìm
-// pattern markdown `- [ ] task` và `- [x] task` (case-insensitive cho dấu x).
-// Pure function: chỉ phụ thuộc input `history`, không closure state.
+// Parse danh sách todo từ history hội thoại.
 //
-// Dedupe: nếu cùng một text xuất hiện nhiều lần (model lặp todo qua các lượt),
-// giữ trạng thái CUỐI cùng — phản ánh state hiện tại của model.
+// Quan trọng: chỉ parse list todo CUỐI cùng — assistant message GẦN NHẤT có
+// ít nhất 1 dòng `- [ ]` hoặc `- [x]` được coi là "todo list hiện tại". Mọi
+// message trước đó bị bỏ qua.
+//
+// Lý do: trước đây scan toàn lịch sử rồi dedupe theo text → todo từ task CŨ
+// vẫn đếm vào progress bar dù model đã chuyển sang task mới. Triệu chứng:
+// bar 5/10 nhưng reply mới nhất chỉ có 3 todo. Cách mới tự nhiên reset khi
+// model viết todo list mới, không bao giờ trộn todo từ 2 task khác nhau.
+//
+// Edge case đã xử lý:
+//   - Code fence ``` (cả với info string như ```md): bỏ qua dòng bên trong,
+//     tránh paste README/snippet có checkbox bị parser bắt nhầm thành todo.
+//   - Cả `[x]` và `[X]` (viết hoa) đều coi là done.
+//   - Multi-line message lặp cùng todo text: giữ trạng thái CUỐI.
+//
+// Pure function: chỉ phụ thuộc input `history`.
 
 /**
  * @typedef {{ text: string, done: boolean }} TodoItem
  */
 
+// Regex tách riêng cho rõ ràng + dễ test. Cả 2 dùng cờ `i` để chấp nhận `[X]`
+// (viết hoa) — markdown checkbox không phân biệt hoa thường trong thực tế.
+const RE_DONE = /^\s*-\s*\[x\]\s+(.+)/i;
+const RE_TODO = /^\s*-\s*\[\s?\]\s+(.+)/i;
+// Code fence: ``` đầu dòng, có thể kèm info string (vd ```js, ```md).
+const RE_FENCE = /^\s*```/;
+
 /**
- * Trích todo list từ history.
- * @param {Array<{role: string, content: any}>} history
+ * Parse các dòng todo trong 1 message content. Trả về [] nếu không có dòng nào.
+ * - Bỏ qua dòng nằm trong code fence (``` ... ```).
+ * - Trong cùng 1 message, nếu cùng text xuất hiện 2 lần (vd model lặp khi
+ *   format), giữ trạng thái CUỐI — phản ánh ý định gần nhất.
+ * @param {string} content
  * @returns {TodoItem[]}
  */
-export function parseTodosFromHistory(history) {
-  const todos = [];
-  for (const m of history) {
-    if (m.role !== 'assistant' || typeof m.content !== 'string') continue;
-    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 });
-      }
+function parseTodosFromMessage(content) {
+  const out = [];
+  let inFence = false;
+  for (const line of content.split('\n')) {
+    if (RE_FENCE.test(line)) {
+      inFence = !inFence;
+      continue;
     }
+    if (inFence) continue;
+    const m1 = line.match(RE_DONE);
+    if (m1) {
+      out.push({ text: m1[1].trim(), done: true });
+      continue;
+    }
+    const m2 = line.match(RE_TODO);
+    if (m2) out.push({ text: m2[1].trim(), done: false });
   }
-  // Dedupe: giữ item CUỐI cùng cho mỗi text (model có thể lặp todo qua các lượt).
+  // Dedupe trong cùng message: giữ entry CUỐI cho mỗi text. Map<1 entry rẻ
+  // như không có — không cần micro-opt skip khi out.length<=1.
   const seen = new Map();
-  for (const t of todos) seen.set(t.text, t);
+  for (const it of out) seen.set(it.text, it);
   return [...seen.values()];
 }
+
+/**
+ * Trích todo list từ history. Chỉ trả về todo từ assistant message GẦN NHẤT
+ * chứa ít nhất 1 dòng todo. Mọi message trước đó bị bỏ qua.
+ * Bỏ qua message có content kiểu non-string (vd multimodal array) — chưa hỗ trợ.
+ * @param {Array<{role: string, content: any}>} history
+ * @returns {TodoItem[]}
+ */
+export function parseTodosFromHistory(history) {
+  if (!Array.isArray(history)) return [];
+  for (let i = history.length - 1; i >= 0; i--) {
+    const m = history[i];
+    if (!m || m.role !== 'assistant' || typeof m.content !== 'string') continue;
+    const parsed = parseTodosFromMessage(m.content);
+    if (parsed.length) return parsed;
+  }
+  return [];
+}

package/src/repl.js

@@ -138,6 +138,23 @@ export async function startRepl(opts = {}) {
     return tui.read(prompt);
   }
 
+  // ── permission prompts (y/n/a) ────────────────────────────────────────
+  // PHẢI khai báo Ở ĐÂY (ngay sau `ask`), TRƯỚC bất kỳ function nào có thể
+  // gọi tới — trước đây 3 const này nằm rải rác (line ~405, ~1648, ~1655)
+  // gây TDZ ReferenceError: "Cannot access 'askAddRoot' before initialization"
+  // khi execToolCore / workflowExecute chạy lần đầu. Logic vẫn ở permission.js,
+  // chỉ wire prompt UI (tui/ask/pending/...) từ scope startRepl.
+  //
+  // ĐỪNG DI CHUYỂN khối này lên trên: dependencies cần sẵn — `tui` (~line 112),
+  // `pending` (~line 107), `ask` (~line 136), `c`/`t`/`truncate` (import top).
+  // Nếu dời lên trước `ask` sẽ lại TDZ. Vị trí hiện tại là điểm sớm nhất hợp lệ.
+  const askPermission = (name) =>
+    _askPermission(name, { tui, ask, pending, c, t, truncate });
+  const askAddRoot = (root, targetPath) =>
+    _askAddRoot(root, targetPath, { tui, ask, pending, c, t, truncate });
+  const askWorkflowAgentMode = () =>
+    _askWorkflowAgentMode({ tui, ask, pending, c, t, truncate });
+
   // NOOB_DEBUG=1: vạch trần đường thoát thật sự (xác nhận/loại bỏ giả thuyết
   // "event loop cạn"). 'beforeExit' nổ = loop cạn (stdin chết). 'exit' = thoát.
   if (process.env.NOOB_DEBUG === '1') {
@@ -399,11 +416,6 @@ Thực thi: đọc/tạo file cần thiết bằng tool, viết code production-
     await workflowExecute(trimmed);
   }
 
-  // Hỏi quyền bật agent mode để chạy workflow. CHỈ chấp nhận y/n (Enter = yes).
-  // Nếu nhận dòng lạ & dài (paste nhầm tin nhắn) → xếp hàng + hỏi lại (y hệt
-  // askPermission / askAddRoot) để user khỏi phải gõ lại /workflow.
-  const askWorkflowAgentMode = () => _askWorkflowAgentMode({ tui, ask, pending, c, t, truncate });
-
   // Chạy thật workflow prompt — chia sẻ giữa ad-hoc và `run <name>`.
   // `builtInName` (optional): nếu có thì SKIP loadSkill dynamic-workflows (prompt
   // built-in đã hardcode pattern + step cụ thể rồi, không cần model design lại).
@@ -1642,18 +1654,6 @@ NGUYÊN TẮC:
     }
   }
 
-  // Hỏi user có muốn cấp quyền folder ngoài workspace cho tool call hiện tại
-  // hay không. Trả về "y" | "n" | "a" (luôn). y = chỉ lần này; a = auto-approve
-  // mọi add-root trong phiên. Cùng style retry với askPermission (lạc → queue).
-  const askAddRoot = (root, targetPath) => _askAddRoot(root, targetPath, { tui, ask, pending, c, t, truncate });
-
-  // Đọc câu trả lời cho phép một cách CHẮC CHẮN. Chỉ chấp nhận y/n/a (hoặc
-  // Enter = đồng ý). Nếu nhận một dòng lạ & dài — gần như chắc là một tin nhắn
-  // bị paste/gõ nhầm vào đúng lúc prompt hiện ra (đây là thủ phạm làm hỏng &
-  // "tự tắt" trước đây) — thì KHÔNG coi là từ chối: xếp nó vào hàng đợi tin
-  // nhắn rồi HỎI LẠI. Nhờ vậy không thao tác nào bị quyết định bởi rác.
-  const askPermission = (name) => _askPermission(name, { tui, ask, pending, c, t, truncate });
-
   // ── slash commands ─────────────────────────────────────────────────────
   async function command(input) {
     const [cmd, ...rest] = input.slice(1).split(/\s+/);