From 67b8d2edbb3710e1aed62ff3a630bf56ee54c6c0 Mon Sep 17 00:00:00 2001 From: Anh Tran Nguyen Minh Date: Fri, 21 Aug 2026 10:52:40 +0900 Subject: [PATCH] docs(refactor): correct the Team Duy scope block in the checklist The previous commit recorded Team Duy as owning R01/R02/R04/R10. That is wrong. Feature_Architecture_Proposal.md line 7 and DeltaTeam_prompt.md line 17 both state R01, R03, R04, R08 (Chat UI) and R10; R02 belongs to Team Nam, which is also who owns the two failing config-security tests. The completed work itself (R01, R03, R04) was already correct and is unchanged. Co-Authored-By: Claude Opus 5 (1M context) --- docs/refactor/Refactoring_Checklist.md | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/docs/refactor/Refactoring_Checklist.md b/docs/refactor/Refactoring_Checklist.md index 9d79963..3d2db2a 100644 --- a/docs/refactor/Refactoring_Checklist.md +++ b/docs/refactor/Refactoring_Checklist.md @@ -38,16 +38,19 @@ > * Mọi file production mới **< 400 dòng** (lớn nhất: `routing_application_service.py` 353 dòng) > * 2 test fail là **lỗi có sẵn từ trước**, thuộc EPIC **R02**: `config.py` vẫn hardcode `sandbox_pw = "quandh14"` ➔ `tests/test_config_security.py` đỏ > -> ### ⚠️ ĐIỀU CHỈNH PHẠM VI (theo xác nhận của Team Lead ngày 21/08) -> Team Duy nhận **R01, R02, R04, R10** (R10 làm sau cùng, chờ các team khác xong). -> * **R03 đã được làm** (6/6 task, đã push) — nằm ngoài phạm vi vừa chốt, nhưng code đã lên nhánh và đã có test bảo vệ. Cần thống nhất bàn giao hay giữ lại. -> * **R02 chưa bắt đầu** — đây là EPIC thuộc phạm vi mới của team và đang là nguyên nhân 2 test đỏ ở trên. +> ### 📍 PHẠM VI TEAM DUY & PHẦN CÒN LẠI +> Theo `Feature_Architecture_Proposal.md` (dòng 7) và `DeltaTeam_prompt.md` (dòng 17), Team Duy chủ trì **R01, R03, R04, R08 (phân hệ Chat UI), R10**. +> * ✅ **R01, R03, R04** — xong 16/16 task, đã push. +> * ⬜ **R08 (R08-T01 ➔ R08-T06)** — chưa bắt đầu: tách `ui/chat_panel.py` (1.795 dòng) thành 6 widget < 400 dòng. +> * ⬜ **R10** — làm sau cùng, chờ 3 team hoàn tất. +> * **R02 thuộc 🟣 Team Nam** (xem mục EPIC R02 bên dưới) — đây là nguyên nhân 2 test đỏ ở trên, không phải việc của Team Duy. > > ### 📌 CÒN NỢ / CẦN QUYẾT ĐỊNH > 1. `ProviderRegistry` **chưa nối** vào `state.build_provider_for` (vẫn dùng `providers/factory.py`). Nối vào sẽ sửa luôn lỗi: usage của `ollama`/`github_copilot`/`codex` hiện bị ghi nhận nhầm thành `openai_compat` trên Dashboard — nhưng làm vậy sẽ **đổi cách gom dữ liệu lịch sử**. > 2. Mode `fallback` đã hỗ trợ ở config + service nhưng **chưa có trên toggle UI** (thuộc R08). > 3. Đã sửa 2 dòng trong `config.py` (`routing_mode_for` / `set_routing_mode_for`) để dùng chung một bộ từ vựng mode — **cần báo Team Nam** vì file này đang được refactor ở R02. -> 4. Circular import `core/model_pricing.py` ↔ `core/usage_tracker.py` **chưa xử lý**. +> 4. Circular import `core/model_pricing.py` ↔ `core/usage_tracker.py` **chưa xử lý** (task ngày 28/08). +> 5. Việc kế tiếp của Team Duy là **R08 phân hệ Chat UI** (6 widget con), rồi **R10** sau cùng. ---