@noobdemon/noob-cli 1.1.0 → 1.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noobdemon/noob-cli",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "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ó

package/src/i18n.js

@@ -54,6 +54,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

@@ -242,6 +242,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");
@@ -496,6 +514,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 +741,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++) {