vg-coder-cli 2.0.57 → 2.0.58

package/bugs/bug1.md

@@ -8,32 +8,49 @@
 
 ---
 
-## Status: 🟢 RESOLVED (2026-05-10, v2.0.53+)
+## Status: 🟢 RESOLVED (2026-05-10, v2.0.57)
 
-**Root cause confirmed**: AI Studio silently fallback model khi account thiếu
-access. AI Studio versions mới (verified 2026-05-10) KHÔNG redirect URL — URL
-param giữ nguyên request, chỉ DOM `<ms-model-selector>` phản ánh model thực.
+**Real root cause** (xác định cuối cùng sau 5 round debug):
+
+`vetgo-auto/scripts/aistudio.google.com/main.js` hardcode
+`VG_DEFAULT_MODEL='gemini-3-flash-preview'`. `task-worker.js:handleTaskExecute`
+gọi `startNewChat()` ở đầu mỗi task → navigate
+`/prompts/new_chat?model=gemini-3-flash-preview` (drop `?model=` của caller pin).
+Task chạy trên Flash bất kể caller request gì.
+
+**Bằng chứng** (verified server2 ULTRA, 2026-05-10):
+- Trước v2.0.57: pin `gemini-3.1-pro-preview` → quan sát qua noVNC: tab navigate
+  Pro → reload thành Flash → `actualModel: gemini-3-flash-preview`
+- Sau v2.0.57: pin `gemini-3.1-pro-preview` → tab giữ Pro → `actualModel:
+  gemini-3.1-pro-preview` ✅
 
 **Fix shipped**:
 - v2.0.52: `open-tab` response thêm `requested_model` / `actual_model` /
-  `fallback_occurred` (URL-based — kept for backward-compat nhưng KHÔNG
-  reliable với AI Studio versions mới).
-- v2.0.53: **`task.result.actualModel`** — worker scrape DOM
-  `<ms-model-selector> [data-test-id="model-name"]` sau khi task done. Đây
-  là **source of truth** cho client. Verified work với:
-  - Request `gemini-99-fake` → `actualModel: "gemini-3.1-pro-preview"` (AI Studio fallback rõ ràng).
-
-**Client pattern recommend**: bỏ qua `actual_model` ở open-tab response, check
-`task.result.actualModel === requestedModel` sau task done; nếu khác → fallback
-xảy ra, retry hoặc fail-fast tùy context.
+  `fallback_occurred` (URL-based, partial detection)
+- v2.0.53: `task.result.actualModel` — worker scrape DOM `<ms-model-selector>`
+  sau task done (detection accurate)
+- v2.0.55: `_pinnedModelByEmail` Map — `_recycleWorkerTab` reopen với pinned
+  model (giữ pin qua nhiều task)
+- **v2.0.57: REAL FIX** — `getTargetModel()` đọc model từ URL hiện tại +
+  sessionStorage cache, thay tất cả hardcode `VG_DEFAULT_MODEL`. `startNewChat`
+  + `pinPromptModel` không còn override pin của caller.
+
+**Client pattern recommend**:
+1. Pin model qua `POST /api/launcher/open-tab` body `{model: "..."}`
+2. Submit task → check `task.result.actualModel === expectedModel`
+3. Nếu khác → AI Studio fallback (account không có access). Decide retry /
+   fail-fast / accept tùy use-case.
 
 **Code refs**:
-- `vetgo-auto/chrome/src/launcher.ts` — open-tab handler (URL-based detection,
-  legacy)
-- `src/server/views/js/features/task-worker.js:201-235` — `readActualModel()`
-  scrape DOM, retry without/with toggle panel
-- `src/server/task-queue.js:417-422` — persist `actualModel` vào `task.result`
-- `INTEGRATION.md` — client docs với recommend pattern
+- `vetgo-auto/scripts/aistudio.google.com/main.js:20-37` — `getTargetModel()`
+  (real fix — primary)
+- `src/server/views/js/features/task-worker.js:210-236` — `readActualModel()`
+  scrape DOM (verification layer)
+- `src/server/task-queue.js:42-44, 491-494` — `_pinnedModelByEmail` persist
+  qua recycle (secondary fix)
+- `vetgo-auto/chrome/src/launcher.ts` — open-tab handler URL-based detection
+  (legacy — không reliable với AI Studio versions mới)
+- `INTEGRATION.md "Verify model thực"` — client docs với recommend pattern
 
 ---
 
@@ -371,34 +388,63 @@ For the AI agent fixing this bug:
 
 ---
 
-## Update 2026-05-10 — Deep root cause discovered
-
-Sau khi ship v2.0.55 (`_pinnedModelByEmail`), test với account ULTRA có Pro
-access vẫn cho `actualModel: gemini-3-flash-preview`. Debug DOM trong khi task
-đang chạy phát hiện:
-
-```
-URL during task: https://aistudio.google.com/prompts/new_chat   ← KHÔNG có ?model=
-DOM ms-model-selector: "gemini-3-flash-preview"
-```
-
-**AI Studio strip `?model=` query khỏi URL sau khi user/script trigger run.**
-URL pin model chỉ work cho **initial load** — sau khi prompt submit, AI Studio
-clean URL → model selector revert về default Flash → task chạy thực sự trên
-Flash bất kể request gì.
-
-### Real fix cần thiết (ngoài scope hôm nay)
-
-1. Worker phải **click model dropdown UI + select model** trước khi submit prompt
-   (không thể chỉ rely vào URL param)
-2. Hoặc: dùng AI Studio **Run settings panel** → model picker thay vì URL trick
-
-Cả 2 cách yêu cầu DOM scripting thêm trong `task-worker.js` `handleTaskExecute`,
-prerun step "ensureModel(targetModel)". Phức tạp hơn fix hiện tại.
-
-### Tạm thời (workaround)
-
-- `actualModel` field vẫn correct — phản ánh model THỰC SỰ chạy (không phải
-  request). Client dùng để fail-fast: `if (actualModel !== expected) abort`.
-- Default model `gemini-3-flash-preview` chấp nhận được cho most use cases.
-- Nếu cần Pro: manual select via noVNC trước khi run task pipeline.
+## Debug timeline (2026-05-10)
+
+Bug fix mất 5 round vì đoán sai root cause vài lần. Lưu lại để tránh lặp:
+
+### Round 1 (v2.0.52) — URL-based detection ❌
+Hypothesis: AI Studio redirect URL khi fallback. Add `requested_model` /
+`actual_model` từ URL ở open-tab response.
+**Sai vì**: AI Studio versions hiện tại **không** redirect URL (từng làm trong
+quá khứ?). URL giữ nguyên dù fallback.
+
+### Round 2 (v2.0.53) — DOM scrape sau task ✅ partial
+Worker `readActualModel()` scrape `<ms-model-selector>` sau task done. Verified
+work với account free tier (request fake model → DOM trả model thực).
+
+### Round 3 — Test với account ULTRA, vẫn Flash
+Hypothesis: pin model bị mất sau `_recycleWorkerTab` (close+reopen với default).
+v2.0.55 add `_pinnedModelByEmail` Map.
+**Đúng 1 phần**: pin persist qua recycle. Nhưng test vẫn Flash.
+
+### Round 4 — Hypothesis "AI Studio strip ?model="
+Quan sát: URL during task = `/prompts/new_chat` (no query). Đoán AI Studio
+auto-clean URL sau prompt submit.
+**Sai vì**: chính code vg-coder navigate URL (không phải AI Studio).
+
+### Round 5 (v2.0.57) — REAL ROOT CAUSE ✅
+User quan sát qua noVNC: tab navigate đúng Pro → reload thành Flash trước khi
+chat. Grep code → tìm thấy `vetgo-auto/scripts/aistudio.google.com/main.js`
+hardcode `VG_DEFAULT_MODEL='gemini-3-flash-preview'`. `startNewChat()` ở đầu
+mỗi task navigate `/prompts/new_chat?model=Flash` → override pin caller.
+Fix: `getTargetModel()` dynamic từ URL + sessionStorage. **Verified work.**
+
+### Round 6 — CI miss step (real-real fix)
+
+Sau v2.0.57 deployed, test server3 vẫn fail. Verify bundle trên server:
+- `grep getTargetModel /usr/local/lib/node_modules/vg-coder-cli/dist/vg-coder-bundle.js` → 0 match
+- v2.0.57 publish thành công nhưng code mới **không có** trong bundle
+
+**Cause**: `vetgo-auto/scripts/aistudio.google.com/main.js` deploy qua Firebase
+RTDB (`ENV/VGCODER`), KHÔNG bundle vào npm package. Extension fetch script tại
+runtime từ Firebase. CI `publish.yml` chỉ chạy `build:extension` + `build:copy`
++ `build:inject` — bỏ qua `deploy-scripts`. Phải chạy thủ công
+`cd vetgo-auto && node deploy-scripts.js` từ máy local để push code mới.
+
+**Fix CI**: add `npm run deploy-scripts` step vào `publish.yml` (commit
+83186ba). Lần sau bump version → CI tự push Firebase.
+
+### Lesson learned
+
+- **Quan sát visual (noVNC) > eval DOM async**: User report "tab reload trước
+  khi chat" là clue quyết định. Eval DOM tại các thời điểm khác nhau cho data
+  rời rạc khó ráp.
+- **Grep hardcode constant trước khi đoán external behavior**: 4 round đầu đoán
+  AI Studio làm gì đó (redirect, strip query). Round 5 tìm thấy hardcode trong
+  chính code mình.
+- **Verify deploy artifact trên target server**: Round 6 chỉ ra version bump
+  KHÔNG đảm bảo code mới chạy nếu deploy pipeline có gap. Sau mỗi fix, grep
+  symbol mới trong file production thực — không tin "CI passed = code chạy".
+- **Multiple layer detection có giá trị**: `actualModel` (DOM scrape) là
+  source-of-truth đúng đắn ngay từ v2.0.53 — confirm bug có thật, định nghĩa
+  expected behavior cho fix, không phụ thuộc fix nào fail.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vg-coder-cli",
-  "version": "2.0.57",
+  "version": "2.0.58",
   "description": "🚀 CLI tool to analyze projects, concatenate source files, count tokens, and export HTML with syntax highlighting and copy functionality",
   "main": "src/index.js",
   "bin": {