@noobdemon/noob-cli 1.1.0 → 1.1.4

package/package.json

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

package/src/agent.js

@@ -55,7 +55,7 @@ Có — cả 12 test đều pass.
 
 Follow this pattern exactly. Your very first response to a task that needs the filesystem MUST be a tool block — do not refuse or explain limitations.`;
 
-const MAX_STEPS = 30;
+const MAX_STEPS = 300;
 const MAX_PROMPT_CHARS = 80000; // ngân sách ký tự cho phần hội thoại gửi lên model
 
 // Môi trường chạy thực: model cần biết OS + shell để emit lệnh ĐÚNG. Không có
@@ -177,8 +177,13 @@ export function parseToolCall(text) {
  * @param {(msg:string)=>void} opts.onStatus      thinking/streaming status
  * @returns {Promise<string>} the final assistant answer (no tool block)
  */
-export async function runAgent({ history, model, signal, onTool, onStatus, onDelta }) {
+export async function runAgent({ history, model, signal, onTool, onStatus, onDelta, onSteer }) {
   for (let step = 0; step < MAX_STEPS; step++) {
+    // Steering: tin nhắn người dùng gõ GIỮA CHỪNG được chèn vào hội thoại TRƯỚC
+    // lần gọi model kế tiếp → model thấy và điều chỉnh ngay trong cùng task.
+    const steer = onSteer?.() || [];
+    for (const msg of steer) history.push({ role: "user", content: msg });
+
     const prompt = buildPrompt(history);
     onStatus?.("thinking");
     onDelta?.({ type: "step-start" });

package/src/i18n.js

@@ -13,6 +13,9 @@ export const t = {
   denied: "đã từ chối",
   queued: (n, txt) => `⏎ đã xếp hàng [${n}] · gửi khi model xong: ${txt}`,
   queueCleared: (n) => `(đã xoá ${n} tin đang xếp hàng)`,
+  steerHint: "💬 Gõ + Enter bất cứ lúc nào để chèn ý cho AI giữa chừng (không ngắt task đang chạy).",
+  steerWillInject: (txt) => `💬 sẽ chèn cho AI ở bước tới: ${txt}`,
+  steerInject: (txt) => `💬 đã chèn cho AI: ${txt}`,
   permRetry: "→ gõ y (đồng ý) · n (từ chối) · a (luôn cho phép)",
 
   // auth
@@ -54,6 +57,7 @@ export const t = {
   cmdSearch: "/search        bật/tắt chế độ tìm web",
   cmdChat: "/chat          quay lại chế độ chat thường",
   cmdYolo: "/yolo          bật/tắt tự duyệt (hoặc nhấn Shift+Tab)",
+  cmdKarpathy: "/karpathy [path]  rà soát code theo 4 nguyên tắc Karpathy (/kc)",
   cmdClear: "/clear /new    xoá ngữ cảnh (mở phiên mới, phiên cũ vẫn resume được)",
   cmdResume: "/resume [id]   tiếp tục phiên cũ (không id = chọn từ danh sách)",
   cmdSessions: "/sessions      liệt kê các phiên đã lưu",

package/src/repl.js

@@ -59,9 +59,11 @@ export async function startRepl(opts = {}) {
       w(line);
       return;
     }
-    // Không ai đang hỏi → đây là tin xếp hàng cho lượt kế tiếp.
+    // Không ai đang hỏi → tin xếp hàng. Nếu đang chạy task → sẽ CHÈN cho AI ở
+    // bước kế tiếp (steering). Nếu rảnh → gửi như lượt mới khi tới phiên.
     pending.push(line);
-    if (process.stdin.isTTY) console.log(c.dim("  " + t.queued(pending.length, truncate(line, 60))));
+    if (process.stdin.isTTY)
+      console.log(abort ? c.user("  " + t.steerWillInject(truncate(line, 60))) : c.dim("  " + t.queued(pending.length, truncate(line, 60))));
   }
   function endInput() {
     closed = true;
@@ -74,6 +76,10 @@ export async function startRepl(opts = {}) {
   function buildRl() {
     const r = readline.createInterface({ input: process.stdin, output: process.stdout, prompt: "" });
     r.on("line", deliver);
+    // readline ở chế độ terminal phát 'SIGINT' của RIÊNG nó (không cho process
+    // SIGINT chạy) — nối vào cùng một handler. interrupt() được hoist nên gọi
+    // được dù khai báo phía dưới.
+    r.on("SIGINT", () => interrupt());
     r.on("close", () => {
       if (exiting) return endInput(); // ta chủ động thoát
       // EOF THẬT: stdin đã end/destroy (Ctrl+Z trên Windows, Ctrl+D trên *nix),
@@ -122,7 +128,10 @@ export async function startRepl(opts = {}) {
     try {
       readline.emitKeypressEvents(process.stdin);
       process.stdin.on("keypress", (_str, key) => {
-        if (!key || key.name !== "tab" || !key.shift) return;
+        if (!key) return;
+        // Ctrl+C dạng phím thô — ở raw mode, Ctrl+C KHÔNG tự thành SIGINT nữa.
+        if (key.ctrl && key.name === "c") return interrupt();
+        if (key.name !== "tab" || !key.shift) return;
         state.yolo = !state.yolo;
         console.log(state.yolo ? c.err("  " + t.yoloOn) : c.ok("  " + t.yoloOff));
         rl.setPrompt(promptStr(false)); // cập nhật dòng trạng thái ngay lập tức
@@ -143,7 +152,19 @@ export async function startRepl(opts = {}) {
 
   let abort = null; // active turn controller
   let sigintArmed = false;
-  process.on("SIGINT", () => {
+  let sigintTimer = null;
+  let lastInterrupt = 0;
+
+  // Ctrl+C — một đường xử lý duy nhất, gọi từ 3 nguồn (xem bên dưới) vì readline
+  // ở chế độ terminal/raw NUỐT SIGINT của process trên Windows. Debounce 80ms để
+  // gộp các nguồn trùng nhau thành MỘT lần nhấn.
+  //   • đang chạy task  → 1 lần = NGỪNG task (không thoát)
+  //   • đang rảnh       → 1 lần = đếm 1.5s; thêm 1 lần trong 1.5s = THOÁT CLI
+  function interrupt() {
+    const now = Date.now();
+    if (now - lastInterrupt < 80) return; // gộp SIGINT + 'SIGINT' của rl + keypress
+    lastInterrupt = now;
+
     if (abort) {
       abort.abort();
       abort = null;
@@ -153,17 +174,33 @@ export async function startRepl(opts = {}) {
         console.log(c.dim("  " + t.queueCleared(n)));
       }
       console.log(c.err("\n  ✗ " + t.interrupted));
-      return; // the main loop will redraw the prompt
+      sigintArmed = false;
+      if (sigintTimer) {
+        clearTimeout(sigintTimer);
+        sigintTimer = null;
+      }
+      if (!closed) rl.prompt(true);
+      return;
     }
+
     if (sigintArmed) {
+      // lần 2 trong cửa sổ 1.5s → thoát
+      if (sigintTimer) clearTimeout(sigintTimer);
+      exiting = true;
+      persist();
       console.log(c.dim("\n  " + t.bye));
       process.exit(0);
     }
+    // lần 1 → vũ trang, đếm 1.5s
     sigintArmed = true;
     console.log(c.dim("\n  " + t.pressAgainToExit));
-    setTimeout(() => (sigintArmed = false), 2000);
-    rl.prompt(true);
-  });
+    sigintTimer = setTimeout(() => {
+      sigintArmed = false;
+      sigintTimer = null;
+    }, 1500);
+    if (!closed) rl.prompt(true);
+  }
+  process.on("SIGINT", interrupt);
 
   // Đừng để một lỗi bất ngờ làm "tự động tắt" CLI. Nguyên nhân hay gặp:
   // tiến trình cập nhật nền (spawn npm) phát sự kiện 'error' không ai bắt,
@@ -242,6 +279,24 @@ export async function startRepl(opts = {}) {
   }
   const startFresh = () => (session = sessions.newSession({ cwd: process.cwd(), model: state.model.id }));
 
+  // /karpathy [path] — bắt noob tự rà soát code theo 4 nguyên tắc Karpathy.
+  // Không có path → soát các file đã đổi trong phiên (model thấy qua FILES CHANGED).
+  async function runKarpathy(arg) {
+    if (!config.apiKey) return console.log(c.tool("  " + t.notLoggedIn));
+    const target = arg ? `file/đường dẫn: ${arg}` : "các file bạn đã tạo/sửa trong phiên này (xem mục FILES CHANGED)";
+    const prompt = `Đóng vai reviewer khó tính. Rà soát ${target} theo 4 nguyên tắc code của Karpathy.
+ĐỌC nội dung file thật bằng read_file trước — KHÔNG dựa vào trí nhớ.
+Với MỖI nguyên tắc, cho verdict (✅ đạt / ⚠️ cảnh báo / ❌ vi phạm) + phát hiện cụ thể kèm "tên_file:dòng":
+1. THINK FIRST — giả định ẩn nào chưa nêu? chỗ nào thiếu kiểm chứng?
+2. KEEP IT SIMPLE — over-engineer, abstraction thừa, lồng quá sâu, hàm quá dài?
+3. SURGICAL — thay đổi lạc đề, refactor tiện tay, đổi style/format vô cớ?
+4. VERIFIABLE GOAL — mục tiêu có kiểm chứng được? đã chạy build/test chưa?
+Cuối cùng: tóm tắt + danh sách sửa theo thứ tự ưu tiên. Thẳng thắn, KHÔNG nịnh.`;
+    console.log(c.tool("  ⚖ Karpathy check…"));
+    await handle(prompt);
+    persist();
+  }
+
   banner();
   printStatus(state);
   if (!config.apiKey) console.log("\n" + c.tool("  " + t.notLoggedIn) + "\n");
@@ -374,6 +429,10 @@ export async function startRepl(opts = {}) {
       }
 
       state.history.push({ role: "user", content: text });
+      if (process.stdin.isTTY && !state.steerHintShown) {
+        console.log(c.dim("  " + t.steerHint));
+        state.steerHintShown = true;
+      }
       startSpin(t.thinking);
       let printer = null;
 
@@ -382,6 +441,14 @@ export async function startRepl(opts = {}) {
         model: state.model.id,
         signal: abort.signal,
         onStatus: () => tick(t.thinking),
+        onSteer: () => {
+          if (!pending.length) return [];
+          const msgs = pending.splice(0);
+          stopSpin(); // in sạch dòng chèn rồi cho spinner chạy lại
+          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));
@@ -496,6 +563,11 @@ export async function startRepl(opts = {}) {
         state.yolo = !state.yolo;
         console.log((state.yolo ? c.err : c.ok)("  " + (state.yolo ? t.yoloOn : t.yoloOff)));
         break;
+      case "karpathy":
+      case "kcheck":
+      case "kc":
+        await runKarpathy(arg);
+        break;
       case "login":
         doLogin(arg);
         break;
@@ -718,6 +790,7 @@ function printHelp() {
         "  " + t.cmdSearch,
         "  " + t.cmdChat,
         "  " + t.cmdYolo,
+        "  " + t.cmdKarpathy,
         "  " + t.cmdLogin,
         "  " + t.cmdLogout,
         "  " + t.cmdUsage,

package/src/tools.js

@@ -39,47 +39,44 @@ export const TOOLS = {
     const file = abs(p);
     const data = await fs.readFile(file, "utf8");
     if (old_string === new_string) throw new Error("old_string and new_string are identical");
+    const useCRLF = data.includes("\r\n");
+    const adapt = (s) => {
+      const lf = s.replace(/\r\n/g, "\n");
+      return useCRLF ? lf.replace(/\n/g, "\r\n") : lf;
+    };
+    // Thay không diễn giải $&/$1… (split/slice, không dùng String.replace).
+    const applyExact = (cand, newS) => {
+      if (replace_all) return data.split(cand).join(newS);
+      const i = data.indexOf(cand);
+      return data.slice(0, i) + newS + data.slice(i + cand.length);
+    };
 
-    let oldS = old_string;
-    let newS = new_string;
-    let count = data.split(oldS).length - 1;
-
-    // Khoan dung CRLF: file Windows thường dùng \r\n, nhưng model hay gửi \n →
-    // khớp hụt. Điều chỉnh kiểu xuống dòng của old/new cho khớp file RỒI thay
-    // trên data gốc (file giữ nguyên vẹn).
-    if (count === 0) {
-      const useCRLF = data.includes("\r\n");
-      const adapt = (s) => {
-        const lf = s.replace(/\r\n/g, "\n");
-        return useCRLF ? lf.replace(/\n/g, "\r\n") : lf;
-      };
-      const a = adapt(oldS);
-      const c2 = data.split(a).length - 1;
-      if (c2 > 0) {
-        oldS = a;
-        newS = adapt(newS);
-        count = c2;
-      }
+    // Tier 1 + 2: khớp NGUYÊN VĂN, rồi khớp sau khi chỉnh CRLF cho hợp file.
+    for (const cand of old_string === adapt(old_string) ? [old_string] : [old_string, adapt(old_string)]) {
+      const count = data.split(cand).length - 1;
+      if (count === 0) continue;
+      if (count > 1 && !replace_all)
+        throw new Error(`old_string is not unique (${count} matches) in ${rel(file)}; set replace_all, or add surrounding lines to make it unique`);
+      await fs.writeFile(file, applyExact(cand, cand === old_string ? new_string : adapt(new_string)), "utf8");
+      return `Edited ${rel(file)} (${replace_all ? count : 1} replacement(s))`;
     }
 
-    if (count === 0)
-      throw new Error(
-        `old_string not found in ${rel(file)}. Read the file again with read_file and copy the target text EXACTLY — keep its indentation/whitespace and DROP the line-number prefix that read_file prints.`,
-      );
-    if (count > 1 && !replace_all)
-      throw new Error(`old_string is not unique (${count} matches) in ${rel(file)}; set replace_all, or add surrounding lines to make it unique`);
-
-    // split/join (không dùng String.replace) để chuỗi thay thế chứa $&, $1… KHÔNG
-    // bị diễn giải đặc biệt — bảo toàn nguyên văn code.
-    let next;
-    if (replace_all) {
-      next = data.split(oldS).join(newS);
-    } else {
-      const i = data.indexOf(oldS);
-      next = data.slice(0, i) + newS + data.slice(i + oldS.length);
+    // Tier 3: khoan dung khoảng trắng CUỐI dòng + CRLF — khớp theo khối DÒNG.
+    // (Thụt đầu dòng vẫn phải khớp → không bao giờ sửa nhầm chỗ.)
+    const m = matchByLines(data, old_string);
+    if (m && m.count > 1 && !replace_all)
+      throw new Error(`old_string khớp ${m.count} chỗ (sau khi bỏ qua khoảng trắng cuối dòng) trong ${rel(file)}; thêm dòng ngữ cảnh hoặc dùng replace_all`);
+    if (m && m.count === 1) {
+      const next = data.slice(0, m.start) + adapt(new_string) + data.slice(m.end);
+      await fs.writeFile(file, next, "utf8");
+      return `Edited ${rel(file)} (1 replacement(s))`;
     }
-    await fs.writeFile(file, next, "utf8");
-    return `Edited ${rel(file)} (${replace_all ? count : 1} replacement(s))`;
+
+    // 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),
+    );
   },
 
   async list_dir({ path: p = "." }) {
@@ -174,6 +171,71 @@ export const TOOLS = {
   },
 };
 
+// Khớp old_string theo KHỐI DÒNG, bỏ qua khác biệt CRLF và khoảng trắng CUỐI
+// dòng. Trả {count, start, end} (offset ký tự trong data) khi khớp đúng 1 chỗ,
+// {count>1} khi nhập nhằng, hoặc null khi không thấy.
+function matchByLines(data, oldStr) {
+  const trim = (l) => l.replace(/\r$/, "").replace(/[ \t]+$/, "");
+  const fileLines = data.split("\n");
+  const oldLines = oldStr.replace(/\r\n/g, "\n").replace(/\n$/, "").split("\n");
+  if (!oldLines.length) return null;
+  const nFile = fileLines.map(trim);
+  const nOld = oldLines.map(trim);
+  const offsets = [];
+  let acc = 0;
+  for (const l of fileLines) {
+    offsets.push(acc);
+    acc += l.length + 1; // +1 cho '\n' đã tách
+  }
+  const hits = [];
+  for (let i = 0; i + nOld.length <= nFile.length; i++) {
+    let ok = true;
+    for (let j = 0; j < nOld.length; j++)
+      if (nFile[i + j] !== nOld[j]) {
+        ok = false;
+        break;
+      }
+    if (ok) hits.push(i);
+  }
+  if (hits.length === 0) return null;
+  if (hits.length > 1) return { count: hits.length };
+  const i = hits[0];
+  const last = i + nOld.length - 1;
+  return { count: 1, start: offsets[i], end: offsets[last] + fileLines[last].length };
+}
+
+// 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) {
+  const want = (oldStr.replace(/\r\n/g, "\n").split("\n").find((l) => l.trim()) || "").trim();
+  if (!want) return "";
+  const lines = data.replace(/\r\n/g, "\n").split("\n");
+  // Dòng giống nhất = tiền tố chung dài nhất với `want` (sau khi trim).
+  const commonPrefix = (a, b) => {
+    let i = 0;
+    const n = Math.min(a.length, b.length);
+    while (i < n && a[i] === b[i]) i++;
+    return i;
+  };
+  let hit = -1;
+  let best = 0;
+  for (let i = 0; i < lines.length; i++) {
+    const s = commonPrefix(lines[i].trim(), want);
+    if (s > best) {
+      best = s;
+      hit = i;
+    }
+  }
+  if (hit < 0 || best < 6) return ` (no similar line found; the file has ${lines.length} lines — re-read it.)`;
+  const a = Math.max(0, hit - 2);
+  const b = Math.min(lines.length, hit + 3);
+  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}`;
+}
+
 function globToRegExp(glob) {
   let rx = "";
   for (let i = 0; i < glob.length; i++) {