diff --git a/CHANGELOG.md b/CHANGELOG.md index 859b3ff..ddb3d91 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,43 @@ và dự án này tuân theo [Semantic Versioning](https://semver.org/spec/v2.0. ## [Unreleased] +### GOO-33 Documentation Sprint (2026-04-22) + +#### Đã hoàn thành +- QA_TRACKER.md — cập nhật test status baseline + Sprint 1-2 test plans +- CHANGELOG.md — cập nhật changelog lần cuối (2026-04-22) +- PROJECT_TRACKER.md — cập nhật GOO-33 status → in_progress +- CONTRIBUTING.md — thêm branching strategy, PR conventions, commit message format +- docs/ci-cd.md — tài liệu đầy đủ GitHub Actions pipeline (lint → typecheck → test → build) +- docs/onboarding.md — hướng dẫn setup dành cho developer mới +- docs/mcp-servers.md — tài liệu 3 MCP servers (search, analytics, valuation) +- docs/audits/ — curate từ ~103 → 12 canonical audit reports + archive old files + +### GOO-2 Lead Orchestrator Audit (2026-04-22) + +#### Audit & Planning +- Kiểm tra toàn diện codebase: 51 findings (4 blockers, 24 high, 13 medium, 10 low) +- Nghiên cứu thị trường BĐS VN: 23 findings (3 P0, 10 P1, 8 P2, 1 P3) +- Ma trận đề xuất: 25 cải thiện (Nhóm A) + 20 tính năng mới (Nhóm B) + 10 docs gaps (Nhóm C) +- Tạo 32 subtasks (GOO-3 → GOO-34) phân theo 6 sprints +- Tạo QA_TRACKER.md, cập nhật PROJECT_TRACKER.md + +#### Đã sửa +- GOO-3: Fix double CSRF middleware — login/register/payment callbacks hoạt động (Sprint 1) ✅ + +#### Đang triển khai (Sprint 1 Blockers) +- GOO-4: UsageRecord atomic metering (fix quota bypass) +- GOO-5: Rate-limit POST /auth/exchange-token +- GOO-6: Fix MoMo IPN URL (tách ipnUrl khỏi redirectUrl) +- GOO-7: JWT validate user status (isActive + deletedAt) + +#### Phát hiện chính (P0 — Launch Blockers) +- Thiếu Phone-OTP login (phương thức auth chính ở VN) +- legalStatus là free-text, không phải enum (tín hiệu tin cậy #1) +- Typesense không hỗ trợ tìm kiếm dấu tiếng Việt +- Thiếu phòng trọ (ROOM_RENTAL) trong PropertyType enum +- Quận 2/9 đã bị xóa (→ Thủ Đức) nhưng vẫn hardcoded trong UI + ### Đã thêm (CEO Audit Wave 13 — 2026-04-12) - Quy trình kiểm tra CEO (TEC-1915) — kiểm tra toàn bộ codebase + xem xét trạng thái dự án - Tài liệu kế hoạch với báo cáo 7 phần: tóm tắt kiểm tra, các vấn đề quan trọng, ưu tiên, khuyến nghị diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 9662554..bd07af1 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,5 +1,209 @@ # Hướng Dẫn Đóng Góp +## Quy Trình Git & Branching + +### Nhánh Chính + +| Nhánh | Mục đích | Protected | +|-------|---------|-----------| +| `main` / `master` | Production branch — stable releases | ✅ Yes | +| `develop` | Development branch — integration point | ✅ Yes | +| `feature/*` | Feature branches — phát triển tính năng mới | ❌ No | +| `fix/*` | Bug fix branches | ❌ No | +| `docs/*` | Documentation updates | ❌ No | +| `refactor/*` | Code refactoring, cleanup | ❌ No | + +### Quy Trình Tạo Feature Branch + +```bash +# 1. Cập nhật branch chính +git checkout develop +git pull origin develop + +# 2. Tạo feature branch +git checkout -b feature/awesome-feature + +# Naming convention: +# feature/user-authentication +# fix/csrf-middleware-double-middleware +# docs/api-documentation +# refactor/remove-dead-code +``` + +### Pull Request Workflow + +```bash +# 1. Commit changes +git add . +git commit -m "feat(auth): implement phone OTP login" + +# 2. Push to origin +git push origin feature/awesome-feature + +# 3. Open PR on GitHub +# - Title: Short summary (max 70 chars) +# - Description: Why, what changed, how to test +# - Link related issues: Fixes #GOO-7 +# - Request reviewers: team lead, domain expert + +# 4. Address review feedback +git add . +git commit -m "refactor(auth): address PR feedback" +git push + +# 5. Merge when approved +# - Squash commits if many small fixes +# - Delete branch after merge +``` + +### Protected Branch Rules + +`main` và `develop` branches yêu cầu: + +- ✅ All CI checks pass (lint, typecheck, test, build) +- ✅ 1 approval từ code owner +- ✅ Dismiss stale PR approvals +- ✅ Branches must be up to date before merge +- ❌ Force push không được phép + +--- + +## Quy Ước Commit + +Theo chuẩn **[Conventional Commits](https://www.conventionalcommits.org/)**: + +``` +(): + + + +