8680f4c849
User chốt MD wrap-up Session 17 (13 commit từc847dc0→de0f38d) — PE Workflow V2 schema + Service wire + UX iteration đầy đủ. MD updates: - STATUS.md — header counts (24 mig, 58 tables, 81 test, 43 gotcha) + Recently Done row consolidate Session 17 wrap-up (gộp 4 row iter cũ thành 1 row tổng, không cắt narrative quan trọng) - HANDOFF.md — TL;DR Session 17 + Chunk E (Designer iter + State machine + Service wire + UX) + Pending Session 18+ - CLAUDE.md — count (22→24 mig, 77→81 test) + V2 schema overview - changelog/migration-todos.md — section ✅ Session 17 done với 7 task ticked + Defer Session 18+ explicit - database/schema-diagram.md — §14 ApprovalWorkflow V2 schema (3 bảng + 2 column PE + state transitions + Service branch + Designer constraints + match approver V2 vs V1 table) - gotchas.md — +#42 Dual schema branch + #43 Step.Order ≠ index 0-based - skill contract-workflow — +section "Phase 9+ done Mig 22-24" với V2 spec + Service branch + match logic table + Designer constraints + UX V2-aware + Defer Session 18+ - changelog/sessions/2026-05-08-1100-pe-workflow-v2-end-to-end.md (NEW) — full session log với 13 commit timeline + stats + gotchas + pending Memory: - project_solution_erp.md — entry Session 17 wrap-up đầy đủ (KHÔNG tạo file feedback mới — decisions specific cho project, không reusable cross-project per §9.5 anti-pattern) Verify: - 81 test pass (58 Domain + 23 Infra) — không thay đổi sau wrap-up - 0 BE error - 2 FE builds OK (đã verify ở các commit trước) - Quy tắc consolidate §6.5: KHÔNG cắt narrative, chỉ phân tầng + remove duplicate. Session 17 row dài cố ý — cover full 13 commit context cho Session 18+ đọc lại.
146 lines
9.1 KiB
Markdown
146 lines
9.1 KiB
Markdown
# CLAUDE.md — AI Agent Context
|
||
|
||
> **Full content:** [`docs/CLAUDE.md`](docs/CLAUDE.md)
|
||
|
||
---
|
||
|
||
**SOLUTION_ERP** — Hệ thống Quản lý Hợp đồng Nhà cung cấp / Thầu phụ / Tổ đội + **Phiếu Duyệt NCC tiền-HĐ** cho Công ty TNHH Xây dựng Solutions.
|
||
|
||
Kiến trúc: **.NET 10 Clean Architecture + 2 React FE (admin + user) + SQL Server + IIS**.
|
||
|
||
## 🚀 BẮT ĐẦU SESSION — 5 file đọc trước tiên
|
||
|
||
```
|
||
1. docs/STATUS.md ← Snapshot HIỆN TẠI (phase nào, việc gì đang làm — PE module CÒN CHỈNH NHIỀU)
|
||
2. docs/HANDOFF.md ← Brief 5 phút: session trước làm gì + cảnh báo session tiếp
|
||
3. docs/PROJECT-MAP.md ← Bản đồ toàn cảnh
|
||
4. docs/changelog/migration-todos.md ← Atomic tasks theo phase (Phase 7 PE refinement mới)
|
||
5. docs/workflow-contract.md ← ⭐ State machine 9 phase HĐ — base pattern cho PE workflow
|
||
```
|
||
|
||
## ⚡ Quick Rules
|
||
|
||
### Backend — `.NET 10` Clean Architecture
|
||
|
||
- Solution: `SolutionErp.slnx` ở root, projects ở `src/Backend/`
|
||
- **4 layer:** `Api → Application ← Domain` + `Infrastructure → Application`
|
||
- Pattern: **CQRS + MediatR**, **FluentValidation**, **AutoMapper**
|
||
- Repository qua `IApplicationDbContext` interface (Application layer)
|
||
- Auth: **JWT Bearer + ASP.NET Identity**
|
||
- DB: **SQL Server** (LocalDB dev / SQL Server prod), **EF Core 10 Code-First migrations**
|
||
- Error handling: `GlobalExceptionMiddleware` map exception → HTTP status
|
||
- Commit scope tech stack: `Api` · `App` · `Domain` · `Infra`
|
||
|
||
### Frontend — 2 app React 18 + Vite + TS + shadcn/ui + TanStack Query
|
||
|
||
- `fe-admin/` (port **8082**) · `fe-user/` (port **8080**)
|
||
- Vite proxy `/api → http://localhost:5443` (SolutionErp.Api)
|
||
- **Named export**, không default export (trừ `App`)
|
||
- shadcn/ui copy-paste, **duplicate giữa 2 app là CÓ CHỦ ĐÍCH** (mỗi app UX riêng)
|
||
- Auth context: `solution-erp-admin-token` / `solution-erp-user-token` ở `localStorage`
|
||
- TanStack Query cho data fetching
|
||
- **Node pin `>=20`** trong `engines`; CI pin `20.x` qua `.nvmrc` (bài học NamGroup — KHÔNG dùng Node latest trên CI)
|
||
- UI **100% tiếng Việt**
|
||
|
||
### Database conventions
|
||
|
||
- Schema: `dbo` (single schema)
|
||
- Table: **PascalCase tiếng Anh** (Contracts, Suppliers, Projects, ContractApprovals, PurchaseEvaluations, ...)
|
||
- PK: `Id` (Guid), FK: `{Entity}Id`
|
||
- Audit fields: `CreatedAt`, `UpdatedAt`, `CreatedBy`, `UpdatedBy` (`BaseEntity`)
|
||
- Soft delete: `IsDeleted`, `DeletedAt`, `DeletedBy` (`AuditableEntity`)
|
||
- Migrations: `dotnet ef migrations add <Name> --project src/Backend/SolutionErp.Infrastructure --startup-project src/Backend/SolutionErp.Api`
|
||
- **Hiện có 24 migration → 58 bảng** (Phase 9+ Session 17 wrap-up — Mig 22-24 V2 schema PE end-to-end: Quy trình > Bước (Phòng) > Cấp (N NV cụ thể qua ApproverUserId, OR-of-N cùng cấp). 3 bảng mới `ApprovalWorkflows` + Steps + Levels. Mig 23 pin `PE.ApprovalWorkflowId`. Mig 24 `PE.CurrentApprovalLevelOrder` track Cấp đang chờ. State machine 5 trạng thái: Nháp / Đã gửi duyệt / Trả lại (Phase riêng TraLai=98) / Từ chối / Đã duyệt. PE Service V2 wire: iterate Steps/Levels match `actor.Id == ApproverUserId`. Contract V2 chưa wire (defer session sau). 81 test pass. Mig 21 V1 flat workflow Mig 21 vẫn live cho phiếu cũ.)
|
||
|
||
### Modules
|
||
|
||
| Module | Namespace | Migration | Trạng thái |
|
||
|---|---|---|---|
|
||
| Contract (HĐ) | `Domain/Contracts/` | 1-11 | Feature-complete (7 ContractType × 9 phase) |
|
||
| PurchaseEvaluation (Duyệt NCC tiền-HĐ) | `Domain/PurchaseEvaluations/` | 12, 13, 15 | Feature-complete — chỉ Export PDF còn pending (không quan trọng) |
|
||
| Budget (Ngân sách dự án) | `Domain/Budgets/` | 14 | **Feature-complete** — BE + FE 3-panel + integration với PE/HD |
|
||
| Master (Supplier/Project/Department) | `Domain/Master/` | 2, 10 | Feature-complete |
|
||
| Identity (User/Role/Permission/MenuItem) | `Domain/Identity/` | 1, 3, 11 | Feature-complete (30 demo user — 16 sample + 14 Solutions thật) |
|
||
| Forms (Template + Clause) | `Domain/Forms/` | 4 | Feature-complete |
|
||
| Notifications | `Domain/Notifications/` | 6 | In-app + SignalR OK, email SMTP TODO |
|
||
| **Tests** | `tests/SolutionErp.{Domain,Infrastructure}.Tests/` | — | **81 test pass** (58 Domain + 23 Infra: 17 codegen + 6 PE WF) — CI gate + path filter docs-only skip |
|
||
|
||
### Commit convention
|
||
|
||
```
|
||
[CLAUDE] <scope>: <imperative message>
|
||
```
|
||
|
||
**Scope:** `Contract` · `PurchaseEvaluation` · `Budget` · `Form` · `Workflow` · `Supplier` · `Auth` · `Admin` · `Api` · `App` · `Domain` · `Infra` · `FE-Admin` · `FE-User` · `Tests` · `Docs` · `CICD` · `Scripts` · `Skill`
|
||
|
||
## 🧪 Tests (Phase 9 — Session 9 +6)
|
||
|
||
```
|
||
tests/
|
||
├── SolutionErp.Domain.Tests/ (54 test - Phase 1: WorkflowPolicy / PEPolicy / BudgetPolicy)
|
||
└── SolutionErp.Infrastructure.Tests/ (17 + 6 + 6 = 29 test)
|
||
├── Common/ (SqliteDbFixture + TestApplicationDbContext + IdentityFixture S9)
|
||
├── Services/ (17 codegen + 6 PE 2-stage approval S9)
|
||
└── Application/ (6 test - PeWorkflowDefinition versioning)
|
||
```
|
||
|
||
**81 unit test pass** / ~3s (58 Domain + 23 Infra: 17 codegen + 6 PE WF versioning). +4 TraLai entry-point tests Session 17 (Standard_TraLai_To_DangGopY + BothPolicies_TraLai_To_ChoPurchasing + NextPhasesFrom_TraLai). Mig 21 drop 19 legacy N-stage/2-stage tests. CI gate + path filter live.
|
||
|
||
```bash
|
||
dotnet test SolutionErp.slnx # chạy cả 2 test project
|
||
```
|
||
|
||
**Quy tắc timing test:** feature mới = test-after (UAT ổn → viết). Bug fix = test-before BẮT BUỘC (reproduce → fix). Critical algorithm (codegen/guard/financial/security) = test-before merge. Spec change = update test cũ + code chung commit. Skip: DTO mapping, CRUD master, FE snapshot. Detail `docs/rules.md §7`.
|
||
|
||
### CI/CD pipeline (3 fix lớn 29/04)
|
||
|
||
- ✅ Manual checkout bypass github.com (fix gotcha #39 TCP timeout)
|
||
- ✅ Path filter docs-only skip — `paths-ignore` (gotcha #41)
|
||
- ⏸️ npm cache (gotcha #40 — failed, rolled back)
|
||
|
||
**Tốc độ:** code commit ~3 phút / docs-only commit **0s** (skip).
|
||
|
||
## 🛠️ Skills (.claude/skills/) — 6 skill PHẢI dùng khi task khớp
|
||
|
||
| Domain (3) | Ops (3) |
|
||
|---|---|
|
||
| `contract-workflow` — state machine + versioned WF | `dependency-audit-erp` — npm/dotnet CVE scan |
|
||
| `form-engine` — render docx/xlsx + PDF | `ef-core-migration` — EF migration + 3-file rule |
|
||
| `permission-matrix` — role × menu × CRUD | `iis-deploy-runbook` — 3 site IIS + win-acme + runner |
|
||
|
||
**Audit định kỳ:** đầu mỗi tháng — combined skill + doc drift audit theo `docs/rules.md §6.4 + §9.4`. Cron `solution-erp-skill-audit-monthly` fire 9:00 ngày 1. Lần kế: **2026-05-01**.
|
||
|
||
Quy tắc:
|
||
- KHÔNG bulk-clone repo skill 3rd party. Chỉ thêm skill PROJECT-SPECIFIC. Xem `docs/rules.md §9` đầy đủ.
|
||
- KHÔNG rewrite toàn bộ MD định kỳ. Chỉ compact + patch drift. Xem `docs/rules.md §6.4` đầy đủ.
|
||
|
||
## 📖 Tài liệu quan trọng
|
||
|
||
| File | Nội dung |
|
||
|---|---|
|
||
| [`docs/STATUS.md`](docs/STATUS.md) | **🔥 Current state** — đọc đầu tiên |
|
||
| [`docs/HANDOFF.md`](docs/HANDOFF.md) | Brief 5 phút: session trước làm gì + next tasks |
|
||
| [`docs/rules.md`](docs/rules.md) | ⭐ Coding conventions (BE Clean Arch, FE React, DB, Git, Docs) |
|
||
| [`docs/architecture.md`](docs/architecture.md) | ⭐ Layered architecture + request lifecycle + deployment |
|
||
| [`docs/PROJECT-MAP.md`](docs/PROJECT-MAP.md) | Bản đồ tổng quan |
|
||
| [`docs/changelog/migration-todos.md`](docs/changelog/migration-todos.md) | Roadmap 5 phase + atomic tasks |
|
||
| [`docs/CLAUDE.md`](docs/CLAUDE.md) | Full context — tech stack chi tiết |
|
||
| [`docs/workflow-contract.md`](docs/workflow-contract.md) | State machine 9 phase HĐ + role matrix |
|
||
| [`docs/forms-spec.md`](docs/forms-spec.md) | Catalog 8 form + quy định mã HĐ RG-001 |
|
||
| [`docs/database/database-guide.md`](docs/database/database-guide.md) | DB conventions + migration workflow + cheatsheet |
|
||
| [`docs/database/schema-diagram.md`](docs/database/schema-diagram.md) | ⭐ ERD + luồng DB + data flow 52 table (+ §11 PE module + §12 Budget module + §13 PEDeptOpinions) |
|
||
| [`docs/flows/README.md`](docs/flows/README.md) | Index 6 flow (auth, permission, contract, form, SLA) |
|
||
| [`docs/gotchas.md`](docs/gotchas.md) | ⭐ 26 bẫy đã gặp — đọc trước khi debug tương tự |
|
||
| [`.claude/skills/`](.claude/skills/README.md) | 6 skill: contract-workflow, form-engine, permission-matrix, dependency-audit-erp, ef-core-migration, iis-deploy-runbook |
|
||
| [`docs/guides/vps-setup.md`](docs/guides/vps-setup.md) | ⭐ Master runbook deploy VPS shared với VIETREPORT |
|
||
|
||
## ⚠️ Kết thúc session
|
||
|
||
1. Update `docs/STATUS.md` (`In Progress` → `Recently Done`)
|
||
2. Tick checklist tương ứng trong `docs/changelog/migration-todos.md`
|
||
3. Tạo session log `docs/changelog/sessions/YYYY-MM-DD-HHMM-{topic}.md` nếu đáng ghi
|
||
4. Commit `[CLAUDE] <scope>: <message>`
|
||
5. ⚠️ **Update `SolutionErp.slnx`** nếu có `.cs/.csproj` mới
|
||
|
||
> Bỏ qua nếu chỉ trả lời câu hỏi, không sửa file nào.
|