vg-coder-cli 2.0.57 → 2.0.59

package/INTEGRATION.md

@@ -66,6 +66,32 @@ Mỗi Google account muốn dùng → 1 Chrome profile riêng:
 > curl -s http://127.0.0.1:6868/api/workers | jq
 > ```
 
+### Deploy AIChat scripts (Firebase RTDB)
+
+⚠️ **Quan trọng cho dev sửa code AIChat**: file
+`vetgo-auto/scripts/aistudio.google.com/main.js` (định nghĩa `window.AIChat` —
+core scraping logic: model selection, file upload, send prompt, copy markdown)
+**KHÔNG** bundle vào npm package. Extension fetch script tại runtime từ
+**Firebase Realtime Database** (`vetgo-chrome-default-rtdb`, path
+`ENV/VGCODER`).
+
+Hệ quả:
+- Bump `vg-coder-cli` version + `npm publish` **KHÔNG** đẩy code AIChat mới.
+- Phải chạy `cd vetgo-auto && node deploy-scripts.js` để PUT script lên Firebase.
+- Test fix nhanh: deploy Firebase + restart Chromium (extension re-fetch script
+  lúc init). Không cần rebuild image / restart container.
+- CI publish workflow (v2.0.58+) đã add step `npm run deploy-scripts` → bump
+  version trên git sẽ auto-deploy Firebase.
+
+```bash
+# Deploy local (dev) — sửa main.js xong:
+cd vetgo-auto && node deploy-scripts.js
+# → PUT https://...firebaseio.../ENV/VGCODER.json
+
+# Restart chromium ở target server để re-fetch:
+ssh ... "docker exec chrome-mcp-vgcoder supervisorctl restart chromium"
+```
+
 ## REST API
 
 ### POST `/api/tasks` — submit task

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,106 @@ 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.
+
+### Round 7 — File upload race (server4 Windows-specific)
+
+Sau bug1 fix work cả 3 server cho text-only task, test multimodal:
+- server2/3 (Linux native): image + PDF Pro work ✅
+- server4 (Docker Desktop Windows): cả image + PDF → model trả "chưa upload
+  file" mặc dù chip hiển thị file đã attach trong UI.
+
+User screenshot 2 lần show:
+1. Chip "feline-xray-chest7.jpg Loading..." — file đang upload
+2. Chip "feline-xray-chest7.jpg 1,101 tokens" — upload xong, nhưng textarea
+   vẫn trống, worker chưa paste prompt
+
+DOM inspect tìm thấy chip element là `<ms-prompt-media>` /
+`[data-test-id="prompt-media-container"]` — KHÔNG match selector cũ
+`ms-prompt-chip-file, ms-file-chip, ms-attachment-chip`. Wait loop pass với
+`chips=0`, fall through 30s timeout → submit Run trước khi tokenize xong → AI
+Studio drop file silent.
+
+**Cause cụ thể**: 2 vấn đề chồng nhau:
+1. Selector outdated (AI Studio đã rename element 2026)
+2. Không có check token-count finalize (chip hiện ngay sau drop, tokenize 5-15s
+   sau đặc biệt trên Windows fs latency)
+
+**Fix v2.0.58**:
+- Update CHIP_SELECTORS thêm `ms-prompt-media`, `[data-test-id="prompt-media-container"]`
+- Add wait loop: chip text phải match `/[\d,]+ tokens/` (support comma) VÀ
+  KHÔNG có `Calculating|Processing|Uploading|Loading`
+- Timeout 60s → proceed anyway
+
+**Result**: server4 image task 105s → 23s sau fix selector + regex chuẩn. Cả 3
+server multimodal Pro work end-to-end.
+
+### Lesson learned
+
+- **Quan sát visual (noVNC) > eval DOM async**: User report "tab reload trước
+  khi chat" + screenshot chip "Loading..." → "1,101 tokens" textarea trống 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 — visual real-time mới thấy state transitions.
+- **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
+  `VG_DEFAULT_MODEL` 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".
+- **Selector update qua thời gian (round 7)**: AI Studio Angular rename
+  tag/class qua mỗi version. Match cả new + legacy selector trong array. Khi
+  wait loop pass với count=0 (warning ignored) → có thể selector đã chết.
+- **Multi-issue stacking**: Round 7 có 2 cause chồng nhau (selector outdated
+  AND tokenize race). Fix 1 cause không đủ — verify fully E2E sau mỗi fix.
+- **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.
+- **Firebase deploy không đồng bộ với npm publish**: code AIChat ở
+  `vetgo-auto/scripts/aistudio.google.com/*.js` deploy qua Firebase RTDB
+  riêng. Bump npm package version KHÔNG đẩy code này (trước v2.0.58 CI fix).
+  Test fix nhanh: `cd vetgo-auto && node deploy-scripts.js` + restart Chromium
+  — KHÔNG cần rebuild image hay update package.