@noobdemon/noob-cli 1.10.20 → 1.11.0

package/README.md

@@ -41,12 +41,12 @@ Trong phiên: `/login <key>`, `/usage`, `/logout`.
 
 **Các gói:**
 
-| Gói | Hạn mức |
-|---|---|
-| `pro` | 5 000 request / 5 giờ (cửa sổ trượt) |
-| `proplus` | 10 000 request / 5 giờ |
-| `admin` | không giới hạn |
-| `trial` | 200 request dùng thử, hết là key **dead** |
+| Gói       | Hạn mức                                   |
+| --------- | ----------------------------------------- |
+| `pro`     | 5 000 request / 5 giờ (cửa sổ trượt)      |
+| `proplus` | 10 000 request / 5 giờ                    |
+| `admin`   | không giới hạn                            |
+| `trial`   | 200 request dùng thử, hết là key **dead** |
 
 > Mỗi lệnh gọi mô hình (kể cả từng bước tool trong một tác vụ) tính là 1 request.
 
@@ -71,17 +71,17 @@ Without `npm link`, run `node bin/noob.js …`.
 
 ### In-session commands
 
-| Command | What it does |
-|---|---|
-| `/model [name]` | switch model (fuzzy match), or list all |
-| `/models` | list every model grouped by provider |
-| `/merge` | toggle **Merge AI** — synthesizes GPT-5.5 + Claude + Gemini via o3 |
-| `/search` | toggle **web search** mode |
-| `/chat` | back to normal chat mode |
-| `/yolo` | toggle auto-approve for file/command tools (or **Shift+Tab**) |
-| `/clear` `/new` | wipe conversation context |
-| `/status` | show current model + working directory |
-| `/exit` | quit (Ctrl+C once = stop turn, twice = quit) |
+| Command         | What it does                                                       |
+| --------------- | ------------------------------------------------------------------ |
+| `/model [name]` | switch model (fuzzy match), or list all                            |
+| `/models`       | list every model grouped by provider                               |
+| `/merge`        | toggle **Merge AI** — synthesizes GPT-5.5 + Claude + Gemini via o3 |
+| `/search`       | toggle **web search** mode                                         |
+| `/chat`         | back to normal chat mode                                           |
+| `/yolo`         | toggle auto-approve for file/command tools (or **Shift+Tab**)      |
+| `/clear` `/new` | wipe conversation context                                          |
+| `/status`       | show current model + working directory                             |
+| `/exit`         | quit (Ctrl+C once = stop turn, twice = quit)                       |
 
 ## How it works
 
@@ -94,6 +94,26 @@ function-calling. noob layers an agent on top:
 3. noob parses the tool call, asks permission if it's destructive, executes it, and
    feeds the result back — looping until the model answers without a tool block.
 
+### Architecture (high-level)
+
+```mermaid
+flowchart LR
+  User[User<br/>terminal] -->|prompt| REPL[repl.js<br/>input loop + slash commands]
+  REPL -->|message + history| Agent[agent.js<br/>tool-loop driver]
+  Agent -->|/api/chat| API[api.js<br/>gateway client]
+  API -->|HTTPS stream| GW[(Noob Demon<br/>Gateway)]
+  GW -->|delta tokens| API
+  API -->|onDelta callbacks| Agent
+  Agent -->|parse tool block| Tools[tools.js<br/>read/write/edit/run]
+  Tools -->|fs / spawn| FS[(workspace)]
+  Tools -->|result text| Agent
+  Agent -->|final answer| REPL
+  REPL -->|persist| Sessions[(~/.noob/sessions/)]
+  REPL -->|memory| Memory[(noob.md per project)]
+  Agent -.optional.-> Subagent[subagent.js<br/>spawn_agent]
+  Subagent -->|isolated context| API
+```
+
 ### Tools the agent can call
 
 `read_file` · `write_file` · `edit_file` · `list_dir` · `glob` · `grep` · `run_command`
@@ -103,10 +123,10 @@ Destructive tools (`write_file`, `edit_file`, `run_command`) prompt for approval
 
 ## Configuration
 
-| Env var | Effect |
-|---|---|
-| `NOOB_API_BASE` | override the gateway URL |
-| `NOOB_API_KEY` | API key (overrides `~/.noob/config.json`) |
+| Env var               | Effect                                                                                                                                   |
+| --------------------- | ---------------------------------------------------------------------------------------------------------------------------------------- |
+| `NOOB_API_BASE`       | override the gateway URL                                                                                                                 |
+| `NOOB_API_KEY`        | API key (overrides `~/.noob/config.json`)                                                                                                |
 | `NOOB_INSECURE_TLS=1` | disable TLS verification — **last resort** for machines behind a TLS-intercepting proxy. Prefer adding your proxy CA to the trust store. |
 
 ## Model compatibility
@@ -114,16 +134,82 @@ Destructive tools (`write_file`, `edit_file`, `run_command`) prompt for approval
 The agent drives tools through a text protocol (the proxy has no native
 function-calling). Models differ in how willingly they follow it:
 
-| Provider | Agentic tools | Notes |
-|---|---|---|
-| **Anthropic** (Claude) | ✅ best | default — `gateway-claude-opus-4-7` |
-| **DeepSeek** | ✅ works | good free alternative |
-| OpenAI (GPT/o-series) | ⚠️ often refuses | replies "I can't access your filesystem" |
-| Google (Gemini) | ⚠️ often refuses | same |
+| Provider               | Agentic tools    | Notes                                    |
+| ---------------------- | ---------------- | ---------------------------------------- |
+| **Anthropic** (Claude) | ✅ best          | default — `gateway-claude-opus-4-7`      |
+| **DeepSeek**           | ✅ works         | good free alternative                    |
+| OpenAI (GPT/o-series)  | ⚠️ often refuses | replies "I can't access your filesystem" |
+| Google (Gemini)        | ⚠️ often refuses | same                                     |
 
 Stick with Claude or DeepSeek for file edits & commands. Any model is fine for
 plain chat, `/merge`, and `/search`.
 
+## Project structure
+
+```
+src/
+├── api.js              # gateway client + stream parser + memory token
+├── agent.js            # tool-loop driver, summarization, prompt assembly
+├── tools.js            # read/write/edit/list_dir/glob/grep/run_command + bg
+├── repl.js             # input loop, slash commands, session state (88 KB)
+├── repl/
+│   ├── complete.js     # SLASH catalog + autocomplete
+│   ├── todos.js        # parseTodosFromHistory (pure)
+│   ├── ultra.js        # ULTRA mode constants + prompt templates
+│   └── workflow-commands.js  # /workflow help/list/load/delete (pure)
+├── subagent.js         # spawn_agent / spawn_agents — sub-agent isolation
+├── tui.js              # terminal UI (input, status bar, prompt rendering)
+├── ui.js               # color helpers, markdown renderer, banner
+├── config.js           # ~/.noob/config.json (api key, gateway, model)
+├── sessions.js         # ~/.noob/sessions/ — save/list/resume
+├── skills.js           # ~/.noob/skills/ — user-defined skill loader
+├── workflows.js        # ~/.noob/workflows/ — user-saved workflows
+├── workflows-builtin.js # ship-with-binary workflows (deep-research, etc)
+├── memory.js           # noob.md per-project agent memory
+├── tokens.js           # local token counter (o200k_base / cl100k_base)
+├── models.js           # 34-model catalog + fuzzy resolver
+├── i18n.js             # Vietnamese string table
+├── update.js           # `noob update` self-updater
+└── prompts/
+    └── system.md       # base system prompt template
+
+tests/                   # vitest unit tests (96 tests)
+scripts/                 # check-imports, notify-discord, release.ps1
+```
+
+## Development
+
+```bash
+npm test                 # run vitest unit tests (96 tests across 7 files)
+npm run lint             # eslint --check
+npm run lint:fix         # eslint --fix
+npm run format           # prettier --write
+npm run format:check     # prettier --check
+npm run check            # lint + format:check + test (pre-commit gate)
+```
+
+Husky chạy `lint-staged` + `npm test` trước mỗi commit. Bỏ qua bằng
+`git commit --no-verify` nếu thực sự cần (vd commit WIP).
+
+## Troubleshooting
+
+**"CLI tự thoát sau khi hỏi quyền"** — thường là readline trên Windows phát sự
+kiện `close` từ tiến trình con chạm vào console. Đã có workaround trong
+`repl.js` (input layer tự re-arm). Nếu còn xảy ra, mở issue kèm OS + Node version.
+
+**"TLS error / certificate"** — máy bạn sau proxy chặn TLS (Zscaler, Cisco
+Umbrella…). Ưu tiên add CA của proxy vào trust store. Cuối cùng mới dùng
+`NOOB_INSECURE_TLS=1` (tắt verify TLS toàn process, MITM-vulnerable).
+
+**"Model OpenAI/Gemini từ chối tool"** — bình thường. GPT-o-series và Gemini
+thường refuse access filesystem qua text protocol. Dùng `/model claude-opus` hoặc
+`/model deepseek-v3` cho tasks cần tool.
+
+**"Token count có vẻ sai"** — `tokens.js` chọn encoder theo model id
+(`o200k_base` cho GPT-4o/5/o-series, `cl100k_base` cho phần còn lại). Đếm chính
+xác cho OpenAI, xấp xỉ ±5-15% cho Claude/Gemini/khác (các provider này không
+publish tokenizer public).
+
 ## Notes & limits
 
 - Free shared proxy: rate limits and occasional hiccups are expected.

package/bin/noob.js

@@ -1,25 +1,32 @@
 #!/usr/bin/env node
-import { startRepl } from "../src/repl.js";
-import { config } from "../src/config.js";
-import { usage, ApiError, applyInsecureTLS } from "../src/api.js";
-import { c } from "../src/ui.js";
-import { t } from "../src/i18n.js";
-import { checkLatest, runUpdate, CURRENT } from "../src/update.js";
+import { startRepl } from '../src/repl.js';
+import { config } from '../src/config.js';
+import { usage, ApiError, applyInsecureTLS } from '../src/api.js';
+import { c } from '../src/ui.js';
+import { t } from '../src/i18n.js';
+import { checkLatest, runUpdate, CURRENT } from '../src/update.js';
 
 const argv = process.argv.slice(2);
-const opts = { yolo: false, ultra: false, model: undefined, prompt: undefined, continue: false, resume: false };
+const opts = {
+  yolo: false,
+  ultra: false,
+  model: undefined,
+  prompt: undefined,
+  continue: false,
+  resume: false,
+};
 const positional = [];
 
 for (let i = 0; i < argv.length; i++) {
   const a = argv[i];
-  if (a === "--yolo" || a === "-y") opts.yolo = true;
-  else if (a === "--ultra" || a === "-u") opts.ultra = true;
-  else if (a === "--insecure-tls") process.env.NOOB_INSECURE_TLS = "1";
-  else if (a === "--model" || a === "-m") opts.model = argv[++i];
-  else if (a === "--continue" || a === "-c") opts.continue = true;
-  else if (a === "--resume" || a === "-r") opts.resume = true;
-  else if (a.startsWith("--resume=")) opts.resume = a.slice("--resume=".length);
-  else if (a === "--help" || a === "-h") {
+  if (a === '--yolo' || a === '-y') opts.yolo = true;
+  else if (a === '--ultra' || a === '-u') opts.ultra = true;
+  else if (a === '--insecure-tls') process.env.NOOB_INSECURE_TLS = '1';
+  else if (a === '--model' || a === '-m') opts.model = argv[++i];
+  else if (a === '--continue' || a === '-c') opts.continue = true;
+  else if (a === '--resume' || a === '-r') opts.resume = true;
+  else if (a.startsWith('--resume=')) opts.resume = a.slice('--resume='.length);
+  else if (a === '--help' || a === '-h') {
     printHelp();
     process.exit(0);
   } else positional.push(a);
@@ -32,35 +39,41 @@ const sub = positional[0];
 applyInsecureTLS();
 
 // ── subcommands ──────────────────────────────────────────────────────────
-if (sub === "login") {
+if (sub === 'login') {
   const key = positional[1];
   if (!key) {
     console.log(c.err(t.needKeyArg));
     process.exit(1);
   }
   config.setKey(key);
-  console.log(c.ok("✓ ") + t.loginSaved(config.path));
+  console.log(c.ok('✓ ') + t.loginSaved(config.path));
   try {
     const u = await usage();
     if (u.ok) {
-      console.log(c.ok(t.loginOk({ pro: "Pro", proplus: "Pro+", admin: "Admin", trial: "Trial" }[u.plan] || u.plan)));
+      console.log(
+        c.ok(
+          t.loginOk(
+            { pro: 'Pro', proplus: 'Pro+', admin: 'Admin', trial: 'Trial' }[u.plan] || u.plan
+          )
+        )
+      );
       process.exit(0);
     } else {
       // Key đã lưu xuống đĩa nhưng gateway từ chối — exit != 0 để user biết, script CI cũng bắt được.
-      console.log(c.err("✗ " + t.errInvalidKey));
+      console.log(c.err('✗ ' + t.errInvalidKey));
       process.exit(2);
     }
   } catch (err) {
     // Mạng/gateway down: giữ key (user có thể đang offline), nhưng KHÔNG im lặng.
-    console.log(c.err("⚠ không verify được key (mạng/gateway): " + (err?.message || err)));
-    console.log(c.dim("  key vẫn được lưu — chạy `noob usage` khi có mạng để xác nhận."));
+    console.log(c.err('⚠ không verify được key (mạng/gateway): ' + (err?.message || err)));
+    console.log(c.dim('  key vẫn được lưu — chạy `noob usage` khi có mạng để xác nhận.'));
     process.exit(3);
   }
-} else if (sub === "logout") {
+} else if (sub === 'logout') {
   config.clearKey();
   console.log(c.ok(t.loggedOut));
   process.exit(0);
-} else if (sub === "update") {
+} else if (sub === 'update') {
   console.log(c.dim(t.updateChecking));
   const v = await checkLatest({ throttle: false });
   if (!v) {
@@ -71,7 +84,7 @@ if (sub === "login") {
   const ok = await runUpdate({ background: false });
   console.log(ok ? c.ok(t.updateOk) : c.err(t.updateFail));
   process.exit(ok ? 0 : 1);
-} else if (sub === "usage") {
+} else if (sub === 'usage') {
   if (!config.apiKey) {
     console.log(c.tool(t.notLoggedIn));
     process.exit(1);
@@ -80,16 +93,16 @@ if (sub === "login") {
     const u = await usage();
     console.log(JSON.stringify(u, null, 2));
   } catch (err) {
-    console.log(c.err("✗ " + (err.message || t.errConn)));
+    console.log(c.err('✗ ' + (err.message || t.errConn)));
     process.exit(1);
   }
   process.exit(0);
 }
 
-if (positional.length) opts.prompt = positional.join(" ");
+if (positional.length) opts.prompt = positional.join(' ');
 
 startRepl(opts).catch((err) => {
-  console.error(c.err("lỗi nghiêm trọng: " + (err?.stack || err?.message || err)));
+  console.error(c.err('lỗi nghiêm trọng: ' + (err?.stack || err?.message || err)));
   process.exit(1);
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noobdemon/noob-cli",
-  "version": "1.10.20",
+  "version": "1.11.0",
   "publishConfig": {
     "access": "public"
   },
@@ -14,11 +14,20 @@
     "src/",
     "skills/",
     "README.md",
+    "CHANGELOG.md",
     "LICENSE"
   ],
   "scripts": {
     "start": "node bin/noob.js",
-    "postpublish": "node scripts/notify-discord.js"
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "format": "prettier --write .",
+    "format:check": "prettier --check .",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "check": "npm run lint && npm run format:check && npm run test",
+    "postpublish": "node scripts/notify-discord.js",
+    "prepare": "husky"
   },
   "engines": {
     "node": ">=18"
@@ -44,5 +53,24 @@
     "gradient-string": "^3.0.0",
     "marked": "^15.0.12",
     "marked-terminal": "^7.3.0"
+  },
+  "devDependencies": {
+    "@eslint/js": "^9.39.4",
+    "eslint": "^9.39.4",
+    "eslint-config-prettier": "^10.1.8",
+    "globals": "^17.6.0",
+    "husky": "^9.1.7",
+    "lint-staged": "^16.4.0",
+    "prettier": "^3.8.4",
+    "vitest": "^4.1.8"
+  },
+  "lint-staged": {
+    "*.{js,mjs,cjs}": [
+      "eslint --fix",
+      "prettier --write"
+    ],
+    "*.{json,md,yml,yaml}": [
+      "prettier --write"
+    ]
   }
 }