dw-kit 1.9.0 → 1.9.2

package/.claude/agents/reviewer.md

@@ -1,126 +1,126 @@
----
-name: reviewer
-description: "Agent chuyên review code. Kiểm tra correctness, security, conventions, test coverage. Tạo báo cáo phân loại theo mức độ nghiêm trọng."
-tools:
-  - Read
-  - Grep
-  - Glob
-  - Bash
-disallowedTools:
-  - Write
-  - Edit
-  - NotebookEdit
-model: sonnet
----
-
-# Reviewer Agent
-
-Bạn là Senior Software Engineer kiêm Security-conscious Code Reviewer. Nhiệm vụ: kiểm tra code thay đổi và đưa ra feedback cụ thể, actionable, phân loại theo mức độ.
-
-## Nguyên Tắc
-
-1. **CHỈ ĐỌC** — Không sửa code, chỉ tạo báo cáo
-2. **Cụ thể, có dẫn chứng** — Mọi issue phải kèm file:line và lý do
-3. **Actionable** — Mỗi issue phải có hướng fix rõ ràng
-4. **Cân bằng** — Ghi nhận điểm tốt, không chỉ chỉ trích
-5. **Bash**: chỉ dùng `git diff`, `git log`, `git show`
-
-## Tiêu Chí Review
-
-### 🔴 CRITICAL — Phải sửa trước khi merge
-- Logic errors, wrong business behavior
-- Security vulnerabilities (injection, auth bypass, data exposure)
-- Data loss risk
-- Breaking API contracts không có migration
-- Missing critical tests
-
-### 🟡 WARNING — Nên sửa
-- Performance issues (N+1 queries, missing indexes)
-- Missing error handling
-- Code smells làm giảm maintainability
-- Test cases chưa đủ coverage
-- Naming không rõ ràng gây hiểu nhầm
-
-### 🔵 SUGGESTION — Cải thiện
-- Refactoring nhỏ
-- DRY improvements
-- Comment/documentation
-- Style nhất quán hơn
-
-## Checklist Review
-
-```
-[ ] Correctness: Logic đúng? Edge cases handled?
-[ ] Security: Input validation? SQL/XSS injection? Auth check?
-[ ] Performance: N+1? Unnecessary DB calls? Loop complexity?
-[ ] Error handling: Errors caught? Logged đủ context?
-[ ] Tests: Unit tests? Integration tests? Edge cases tested?
-[ ] Conventions: Naming? File structure? Code style?
-[ ] Documentation: Complex logic có comment? Public API documented?
-[ ] Breaking changes: API contract thay đổi? Migration cần thiết?
-```
-
-## Output Format
-
-Tạo ĐẦY ĐỦ cả hai phần: markdown cho human, JSON cho machine.
-
-```markdown
-# Code Review: [PR/Branch/Task]
-
-## Tóm Tắt Thay Đổi
-[Mô tả ngắn những gì changed]
-
-## Đánh Giá Tổng
-- Code Quality: ⭐⭐⭐⭐☆ (4/5)
-- Test Coverage: Đủ / Thiếu / Không có
-- Security: ✅ OK / ⚠️ Có concern / 🚫 Vấn đề nghiêm trọng
-- Performance: ✅ OK / ⚠️ Cần xem xét
-
-## 🔴 Critical Issues (Phải sửa)
-- [ ] **[file.ts:42]** — [mô tả vấn đề] → Fix: [hướng dẫn cụ thể]
-
-## 🟡 Warnings (Nên sửa)
-- [ ] **[file.ts:15]** — [mô tả] → Suggestion: [gợi ý]
-
-## 🔵 Suggestions (Cân nhắc)
-- [ ] **[file.ts:80]** — [mô tả]
-
-## ✅ Điểm Tốt
-- [Ghi nhận pattern/approach tốt]
-
-## Kết Luận
-[Approve / Request Changes / Needs Discussion]
-```
-
-Sau phần markdown, thêm JSON block để CI/CD parse:
-
-```json
-{
-  "approved": false,
-  "score": 7.5,
-  "conclusion": "request_changes",
-  "critical": [
-    {
-      "file": "src/auth/service.ts",
-      "line": 42,
-      "issue": "MD5 used for password hashing",
-      "fix": "Replace with bcrypt, minimum 12 rounds"
-    }
-  ],
-  "warnings": [
-    {
-      "file": "src/users/repo.ts",
-      "line": 18,
-      "issue": "Missing error handling in DB call",
-      "fix": "Wrap in try/catch, log error with context"
-    }
-  ],
-  "suggestions": [],
-  "positives": [
-    "Good separation of concerns in service layer"
-  ]
-}
-```
-
-**Lưu ý**: JSON phải valid. Nếu không có issues ở một mức độ, dùng array rỗng `[]`.
-`conclusion` values: `"approve"` | `"request_changes"` | `"needs_discussion"`
+---
+name: reviewer
+description: "Agent chuyên review code. Kiểm tra correctness, security, conventions, test coverage. Tạo báo cáo phân loại theo mức độ nghiêm trọng."
+tools:
+  - Read
+  - Grep
+  - Glob
+  - Bash
+disallowedTools:
+  - Write
+  - Edit
+  - NotebookEdit
+model: sonnet
+---
+
+# Reviewer Agent
+
+Bạn là Senior Software Engineer kiêm Security-conscious Code Reviewer. Nhiệm vụ: kiểm tra code thay đổi và đưa ra feedback cụ thể, actionable, phân loại theo mức độ.
+
+## Nguyên Tắc
+
+1. **CHỈ ĐỌC** — Không sửa code, chỉ tạo báo cáo
+2. **Cụ thể, có dẫn chứng** — Mọi issue phải kèm file:line và lý do
+3. **Actionable** — Mỗi issue phải có hướng fix rõ ràng
+4. **Cân bằng** — Ghi nhận điểm tốt, không chỉ chỉ trích
+5. **Bash**: chỉ dùng `git diff`, `git log`, `git show`
+
+## Tiêu Chí Review
+
+### 🔴 CRITICAL — Phải sửa trước khi merge
+- Logic errors, wrong business behavior
+- Security vulnerabilities (injection, auth bypass, data exposure)
+- Data loss risk
+- Breaking API contracts không có migration
+- Missing critical tests
+
+### 🟡 WARNING — Nên sửa
+- Performance issues (N+1 queries, missing indexes)
+- Missing error handling
+- Code smells làm giảm maintainability
+- Test cases chưa đủ coverage
+- Naming không rõ ràng gây hiểu nhầm
+
+### 🔵 SUGGESTION — Cải thiện
+- Refactoring nhỏ
+- DRY improvements
+- Comment/documentation
+- Style nhất quán hơn
+
+## Checklist Review
+
+```
+[ ] Correctness: Logic đúng? Edge cases handled?
+[ ] Security: Input validation? SQL/XSS injection? Auth check?
+[ ] Performance: N+1? Unnecessary DB calls? Loop complexity?
+[ ] Error handling: Errors caught? Logged đủ context?
+[ ] Tests: Unit tests? Integration tests? Edge cases tested?
+[ ] Conventions: Naming? File structure? Code style?
+[ ] Documentation: Complex logic có comment? Public API documented?
+[ ] Breaking changes: API contract thay đổi? Migration cần thiết?
+```
+
+## Output Format
+
+Tạo ĐẦY ĐỦ cả hai phần: markdown cho human, JSON cho machine.
+
+```markdown
+# Code Review: [PR/Branch/Task]
+
+## Tóm Tắt Thay Đổi
+[Mô tả ngắn những gì changed]
+
+## Đánh Giá Tổng
+- Code Quality: ⭐⭐⭐⭐☆ (4/5)
+- Test Coverage: Đủ / Thiếu / Không có
+- Security: ✅ OK / ⚠️ Có concern / 🚫 Vấn đề nghiêm trọng
+- Performance: ✅ OK / ⚠️ Cần xem xét
+
+## 🔴 Critical Issues (Phải sửa)
+- [ ] **[file.ts:42]** — [mô tả vấn đề] → Fix: [hướng dẫn cụ thể]
+
+## 🟡 Warnings (Nên sửa)
+- [ ] **[file.ts:15]** — [mô tả] → Suggestion: [gợi ý]
+
+## 🔵 Suggestions (Cân nhắc)
+- [ ] **[file.ts:80]** — [mô tả]
+
+## ✅ Điểm Tốt
+- [Ghi nhận pattern/approach tốt]
+
+## Kết Luận
+[Approve / Request Changes / Needs Discussion]
+```
+
+Sau phần markdown, thêm JSON block để CI/CD parse:
+
+```json
+{
+  "approved": false,
+  "score": 7.5,
+  "conclusion": "request_changes",
+  "critical": [
+    {
+      "file": "src/auth/service.ts",
+      "line": 42,
+      "issue": "MD5 used for password hashing",
+      "fix": "Replace with bcrypt, minimum 12 rounds"
+    }
+  ],
+  "warnings": [
+    {
+      "file": "src/users/repo.ts",
+      "line": 18,
+      "issue": "Missing error handling in DB call",
+      "fix": "Wrap in try/catch, log error with context"
+    }
+  ],
+  "suggestions": [],
+  "positives": [
+    "Good separation of concerns in service layer"
+  ]
+}
+```
+
+**Lưu ý**: JSON phải valid. Nếu không có issues ở một mức độ, dùng array rỗng `[]`.
+`conclusion` values: `"approve"` | `"request_changes"` | `"needs_discussion"`

package/.claude/rules/code-style.md

@@ -1,37 +1,37 @@
-# Code Style & Conventions
-
-## Nguyên tắc chung
-- Đặt tên biến/hàm rõ ràng, tự giải thích (self-documenting)
-- Ưu tiên đơn giản, dễ đọc hơn clever code
-- Mỗi function làm MỘT việc
-- Comments giải thích WHY, không phải WHAT
-- Xử lý errors ở đầu function (guard clauses / early return)
-
-## Naming Conventions
-- Variables/Functions: camelCase
-- Classes/Components: PascalCase
-- Constants: UPPER_SNAKE_CASE
-- Files: kebab-case hoặc theo convention của framework
-- Directories: kebab-case
-
-## File Organization
-- 1 component/class per file (trừ khi strongly related)
-- Group imports: external → internal → relative
-- Export ở cuối file hoặc inline (nhất quán trong project)
-
-## Error Handling
-- KHÔNG swallow errors (catch rỗng)
-- Log đủ context để debug (error message, input data, stack)
-- Dùng custom error types cho domain errors
-- Validate input ở boundary (API, form, external data)
-
-## Testing
-- Test file cùng tên với source: `foo.ts` → `foo.test.ts` hoặc `foo.spec.ts`
-- Mỗi test case kiểm tra MỘT behavior
-- Test name mô tả expected behavior: "should return error when input is empty"
-- Arrange → Act → Assert pattern
-- KHÔNG mock internal implementation details
-
-## NOTE
-Đây là quy tắc mặc định. Team tùy chỉnh theo stack cụ thể của dự án.
-Thêm framework-specific rules vào file này hoặc tạo file riêng trong `.claude/rules/`.
+# Code Style & Conventions
+
+## Nguyên tắc chung
+- Đặt tên biến/hàm rõ ràng, tự giải thích (self-documenting)
+- Ưu tiên đơn giản, dễ đọc hơn clever code
+- Mỗi function làm MỘT việc
+- Comments giải thích WHY, không phải WHAT
+- Xử lý errors ở đầu function (guard clauses / early return)
+
+## Naming Conventions
+- Variables/Functions: camelCase
+- Classes/Components: PascalCase
+- Constants: UPPER_SNAKE_CASE
+- Files: kebab-case hoặc theo convention của framework
+- Directories: kebab-case
+
+## File Organization
+- 1 component/class per file (trừ khi strongly related)
+- Group imports: external → internal → relative
+- Export ở cuối file hoặc inline (nhất quán trong project)
+
+## Error Handling
+- KHÔNG swallow errors (catch rỗng)
+- Log đủ context để debug (error message, input data, stack)
+- Dùng custom error types cho domain errors
+- Validate input ở boundary (API, form, external data)
+
+## Testing
+- Test file cùng tên với source: `foo.ts` → `foo.test.ts` hoặc `foo.spec.ts`
+- Mỗi test case kiểm tra MỘT behavior
+- Test name mô tả expected behavior: "should return error when input is empty"
+- Arrange → Act → Assert pattern
+- KHÔNG mock internal implementation details
+
+## NOTE
+Đây là quy tắc mặc định. Team tùy chỉnh theo stack cụ thể của dự án.
+Thêm framework-specific rules vào file này hoặc tạo file riêng trong `.claude/rules/`.

package/.claude/settings.json

@@ -11,36 +11,10 @@
       "Bash(git status)",
       "Bash(git blame *)",
       "Bash(git stash list)",
+      "Bash(git add *)",
       "Bash(ls *)",
       "Bash(wc *)",
-      "Bash(node src/smoke-test.mjs)",
-      "Bash(xargs echo *)",
-      "Edit(/.claude/skills/dw-decision/**)",
-      "Bash(node bin/dw.mjs active)",
-      "Bash(node bin/dw.mjs metrics show)",
-      "Bash(node bin/dw.mjs dashboard)",
-      "Bash(node bin/dw.mjs init --help)",
-      "Bash(mkdir -p /tmp/test-solo)",
-      "Read(//tmp/**)",
-      "Bash(DW_NAME=test node /d/huygdv_projects/dw-kit/bin/dw.mjs init --solo)",
-      "Bash(npm test *)",
-      "Bash(node bin/dw.mjs metrics cut-analysis)",
-      "Bash(bash -n .claude/hooks/pre-commit-gate.sh)",
-      "Bash(bash -n /dev/stdin)",
-      "Bash(dos2unix .claude/hooks/pre-commit-gate.sh)",
-      "Bash(sed -i 's/\\\\r$//' .claude/hooks/pre-commit-gate.sh)",
-      "Bash(git add *)",
-      "Edit(/.claude/skills/dw-task-init/**)",
-      "Edit(/.claude/skills/dw-retroactive/**)",
-      "Edit(/.claude/skills/dw-research/**)",
-      "Edit(/.claude/skills/dw-plan/**)",
-      "Edit(/.claude/skills/dw-execute/**)",
-      "Edit(/.claude/skills/dw-handoff/**)",
-      "Bash(git commit -m 'fix\\(v1.3.3\\): writer skills v1/v2 compat + cleanup legacy slash-command refs *)",
-      "Bash(git commit -m 'feat\\(v1.3.4\\): /dw:plan integrated Quick Debate \\(red/blue team\\) - depth-driven *)"
-    ],
-    "additionalDirectories": [
-      "D:\\huygdv_projects\\dw-kit\\.claude\\rules"
+      "Bash(npm test *)"
     ]
   },
   "hooks": {

package/.claude/skills/dw-kit-report/SKILL.md

@@ -17,6 +17,18 @@ $ARGUMENTS
 
 ---
 
+## Nguyên tắc riêng tư (ĐỌC TRƯỚC — bắt buộc)
+
+Skill này tạo issue trên **repo bên thứ ba `dv-workflow/dv-workflow`** (ngoài project của bạn). Vì vậy **KHÔNG bao giờ tự động đưa thông tin nhận dạng của project người report vào issue**:
+
+- ❌ KHÔNG kèm `project.name` từ config (đây là tên codebase/đơn vị của người dùng).
+- ❌ KHÔNG kèm tên task, mã ticket nội bộ (vd `PROJ-123`), đường dẫn file độc quyền, hay code snippet.
+- ✅ CHỈ gửi: loại feedback, component của **dw-kit**, mô tả vấn đề (từ `$ARGUMENTS`), và môi trường tối thiểu (OS, shell, dw version).
+
+Nếu chính `$ARGUMENTS` chứa thông tin nhận dạng nội bộ → cảnh báo người dùng và đề nghị lược bỏ. **Luôn preview + xin xác nhận trước khi gửi (Bước 4).**
+
+---
+
 ## Bước 1: Thu Thập Context
 
 **OS detection:**
@@ -24,9 +36,9 @@ $ARGUMENTS
 uname -s 2>/dev/null || echo "Windows"
 ```
 
-**dw version:** Đọc `_toolkit.core_version` từ `.dw/config/dw.config.yml`
+**dw version:** Đọc `_toolkit.core_version` từ `.dw/config/dw.config.yml` (KHÔNG đọc `project.name`).
 
-**Task context:** Kiểm tra `.dw/tasks/` — task nào đang In Progress?
+**Workflow context:** Có task In Progress không, và depth của nó (quick/standard/thorough) — chỉ để giúp reproduce. Ghi nhận CÓ/KHÔNG + depth, **KHÔNG lấy tên task**.
 
 ---
 
@@ -83,8 +95,8 @@ Ví dụ:
 [Nội dung từ $ARGUMENTS — đầy đủ, rõ ràng]
 
 ## Context
-- Task khi gặp vấn đề: [task name hoặc "general usage"]
-- Command/skill liên quan: [nếu biết]
+- Hoàn cảnh: ["general usage" hoặc "trong một task depth=X" — KHÔNG ghi tên task/ticket]
+- Command/skill dw-kit liên quan: [nếu biết]
 - Bước reproduce (nếu là bug):
   1. ...
   2. ...
@@ -95,12 +107,31 @@ Ví dụ:
 - [ ] Minor — annoying, có workaround dễ
 
 ---
-*Reported via `/dw:kit-report` | Project: [project.name từ config]*
+*Reported via `/dw:kit-report`*
 ```
 
+> Không thêm dòng `Project:` hay bất kỳ tên project/ticket/path nội bộ nào (xem Nguyên tắc riêng tư).
+
+---
+
+## Bước 4: Preview + Xác Nhận (BẮT BUỘC trước khi gửi)
+
+Issue sẽ được tạo trên repo bên thứ ba **công khai với dw-kit team**. Trước khi gửi, hiện đầy đủ title + body sẽ post và CHỜ người dùng đồng ý:
+
+```
+┌─ Sẽ gửi lên github.com/dv-workflow/dv-workflow ─────────────┐
+ TITLE: [title]
+
+ BODY:
+ [body đầy đủ]
+└─ Gõ "ok" để gửi · hoặc sửa nội dung trước ──────────────────┘
+```
+
+Trước khi hiện preview, **rà soát lần cuối**: nếu title/body (kể cả phần người dùng tự nhập trong `$ARGUMENTS`) còn chứa tên project, mã ticket nội bộ, đường dẫn file độc quyền, hay code snippet → cảnh báo rõ và đề nghị lược bỏ. **Chỉ sang Bước 5 sau khi người dùng xác nhận.**
+
 ---
 
-## Bước 4: Gửi Lên GitHub
+## Bước 5: Gửi Lên GitHub
 
 **Kiểm tra `gh` CLI:**
 ```bash
@@ -141,7 +172,7 @@ In ra:
 
 ---
 
-## Bước 5: Xác Nhận
+## Bước 6: Xác Nhận
 
 ```
 ✓ Issue đã được gửi: https://github.com/dv-workflow/dv-workflow/issues/[N]

package/.claude/skills/dw-plan/template-plan.md

@@ -1,47 +1,47 @@
-# Plan: [Task Name]
-
-## Ngày: [date] | Trạng thái: Draft → Approved
-
----
-
-## Tóm Tắt
-
-[1-2 câu: mục tiêu và approach được chọn]
-
-## Phương Án So Sánh
-
-| # | Phương án | Ưu điểm | Nhược điểm | Chọn? |
-|---|-----------|---------|------------|-------|
-| 1 | | | | **Chọn** |
-| 2 | | | | Loại — lý do: |
-
-## Subtasks
-
-### ST-1: [Tên]
-- **Mô tả**: [cụ thể]
-- **Files**: [danh sách]
-- **Criteria**: [ ] [điều kiện pass]
-- **Dependencies**: none
-- **Estimate**: [nếu bật]
-
-### ST-2: [Tên]
-- **Mô tả**: ...
-- **Dependencies**: ST-1
-
-## Rủi Ro & Giả Định
-
-| # | Loại | Mô tả | Xác suất | Tác động | Mitigation |
-|---|------|--------|----------|----------|------------|
-| 1 | Giả định | | | | |
-| 2 | Rủi ro | | Cao/TB/Thấp | | |
-
-## Edge Cases
-
-- [ ] [Edge case cần xử lý]
-
-## Tác Động Hệ Thống
-
-- Modules: [danh sách]
-- API changes: [Có / Không]
-- Migration: [Có / Không]
-- Breaking: [Có / Không]
+# Plan: [Task Name]
+
+## Ngày: [date] | Trạng thái: Draft → Approved
+
+---
+
+## Tóm Tắt
+
+[1-2 câu: mục tiêu và approach được chọn]
+
+## Phương Án So Sánh
+
+| # | Phương án | Ưu điểm | Nhược điểm | Chọn? |
+|---|-----------|---------|------------|-------|
+| 1 | | | | **Chọn** |
+| 2 | | | | Loại — lý do: |
+
+## Subtasks
+
+### ST-1: [Tên]
+- **Mô tả**: [cụ thể]
+- **Files**: [danh sách]
+- **Criteria**: [ ] [điều kiện pass]
+- **Dependencies**: none
+- **Estimate**: [nếu bật]
+
+### ST-2: [Tên]
+- **Mô tả**: ...
+- **Dependencies**: ST-1
+
+## Rủi Ro & Giả Định
+
+| # | Loại | Mô tả | Xác suất | Tác động | Mitigation |
+|---|------|--------|----------|----------|------------|
+| 1 | Giả định | | | | |
+| 2 | Rủi ro | | Cao/TB/Thấp | | |
+
+## Edge Cases
+
+- [ ] [Edge case cần xử lý]
+
+## Tác Động Hệ Thống
+
+- Modules: [danh sách]
+- API changes: [Có / Không]
+- Migration: [Có / Không]
+- Breaking: [Có / Không]

package/.claude/skills/dw-research/template-research.md

@@ -1,51 +1,51 @@
-# Research: [Task Name]
-
-## Ngày: [date] | Agent: researcher
-
----
-
-## Yêu Cầu
-
-[Mô tả ngắn gọn task / yêu cầu cần khảo sát]
-
-## Files Liên Quan
-
-| # | File | Vai trò | Cần thay đổi? |
-|---|------|---------|----------------|
-| 1 | | | Có / Không |
-
-## Kiến Trúc Hiện Tại
-
-```
-[Sơ đồ ASCII đơn giản hoặc mô tả luồng hiện tại]
-```
-
-## Dependencies
-
-**Upstream** (phụ thuộc vào):
-- [Module/API]
-
-**Downstream** (bị ảnh hưởng):
-- [Module/API]
-
-## Patterns Phát Hiện
-
-| Pattern | Mô tả | Ví dụ (file) |
-|---------|--------|--------------|
-| | | |
-
-## Test Coverage
-
-- Tests hiện có: [Có / Không / Partial]
-- Files test: [danh sách nếu có]
-- Gaps: [thiếu test ở đâu]
-
-## Giả Định & Hạn Chế
-
-- **Giả định**: [điều đang coi là đúng]
-- **Hạn chế**: [giới hạn đã biết]
-- **Chưa rõ**: [cần làm rõ thêm]
-
-## Ghi Chú
-
-[Bất kỳ context quan trọng: gotchas, tech debt, lịch sử thay đổi gần đây]
+# Research: [Task Name]
+
+## Ngày: [date] | Agent: researcher
+
+---
+
+## Yêu Cầu
+
+[Mô tả ngắn gọn task / yêu cầu cần khảo sát]
+
+## Files Liên Quan
+
+| # | File | Vai trò | Cần thay đổi? |
+|---|------|---------|----------------|
+| 1 | | | Có / Không |
+
+## Kiến Trúc Hiện Tại
+
+```
+[Sơ đồ ASCII đơn giản hoặc mô tả luồng hiện tại]
+```
+
+## Dependencies
+
+**Upstream** (phụ thuộc vào):
+- [Module/API]
+
+**Downstream** (bị ảnh hưởng):
+- [Module/API]
+
+## Patterns Phát Hiện
+
+| Pattern | Mô tả | Ví dụ (file) |
+|---------|--------|--------------|
+| | | |
+
+## Test Coverage
+
+- Tests hiện có: [Có / Không / Partial]
+- Files test: [danh sách nếu có]
+- Gaps: [thiếu test ở đâu]
+
+## Giả Định & Hạn Chế
+
+- **Giả định**: [điều đang coi là đúng]
+- **Hạn chế**: [giới hạn đã biết]
+- **Chưa rõ**: [cần làm rõ thêm]
+
+## Ghi Chú
+
+[Bất kỳ context quan trọng: gotchas, tech debt, lịch sử thay đổi gần đây]