ai-developer-skill-os 8.2.1 → 8.3.0

package/.agents/skills/qk-code-review/SKILL.md

@@ -1,186 +1,188 @@
----
-# ── Identity ───────────────────────────────────────────────
-name: qk-code-review
-version: 8.0.0
-status: stable
-description: "Elite AI/Code Review System: Kiểm toán, Review code và cấu hình AI với tư duy Architect, áp dụng 4-Phase Review."
-platforms: [antigravity, claude-code, cursor, windsurf, kilo-code]
-
-# ── V8: Classification ─────────────────────────────────────
-type: utility
-
-intent:
-  - code-review
-  - quality-assurance
-  - architecture-review
-  - ai-config-audit
-
-complexity:
-  level: high
-  criteria:
-    files_affected: "1-15"
-    has_behavior_change: false
-    has_external_dependency: false
-    has_breaking_change: false
-
-triggers:
-  - "review code"
-  - "code review"
-  - "kiểm tra code"
-  - "đánh giá code"
-  - "review skin"
-  - "review rule"
-  - "review ai"
-
-selection:
-  priority: high
-  confidence_threshold: 0.85
-
-# ── V8: References ─────────────────────────────────────────
-workflow: code-review
-
-rules:
-  - global
-
-tools:
-  - filesystem
-
-related_skills:
-  - qk-validation-gate
-  - qk-engineering-standard
-  - qk-project-health
-
-knowledge_scope:
-  owns:
-    - code-review-standards
-    - ai-configuration-review
-    - feedback-delivery
-  references:
-    - architecture
-    - security-best-practices
-
-# ── V8: Verification ───────────────────────────────────────
-verification:
-  required: true
-  strategy: review
-
-# ── V8: Knowledge links ────────────────────────────────────
-examples: []
-learnings: []
-
-# ── V7 Runtime ─────────────────────────────────────────────
-execution_mode: deterministic
-cost: high
-latency: medium
-risk: low
-side_effects: read_only
-produces: [report]
-consumes: [source-code, rules, workflows]
-
-token_budget:
-  max_files_read: 10
-  max_lines_per_read: 300
-  max_shell_commands: 2
-  stop_early: true
-
-exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
----
-
-# qk-code-review — Elite Review System
-
-> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
-
-Biến quá trình Code Review từ "bắt bẻ" (gatekeeping) thành "chia sẻ tri thức" (knowledge sharing) thông qua phản hồi mang tính xây dựng, phân tích có hệ thống và hợp tác cải tiến.
-
----
-
-## Preconditions
-
-- [ ] Xác định rõ đối tượng cần review: Mã nguồn phần mềm (React, Java, etc.) hay Cấu hình AI (Skin, Rules, Workflows).
-- [ ] Cung cấp ngữ cảnh hoặc mục tiêu của đoạn code/cấu hình cần review.
-
-```
-On missing precondition → EXIT: BLOCKED
-Report: "Missing: Vui lòng cung cấp ngữ cảnh hoặc chỉ định rõ file cần review."
-```
-
----
-
-## Dynamic Context Loading (Tải Ngữ Cảnh Động)
-
-> **CHỈ ĐỌC KHI CẦN (on-demand):** Chỉ mở file references/ nếu có vấn đề cụ thể cần xác minh. Đừng đọc trước khi chưa thấy vấn đề.
-
-- **Nếu review AI Config** và phát hiện vấn đề schema/anti-pattern → đọc `references/ai/v8-schema-validation.md` HOẶC `references/ai/ai-anti-patterns.md` (không cần đọc cả 2 nếu không liên quan).
-- **Nếu review Code** và gặp vấn đề ngôn ngữ cụ thể → đọc `references/languages/[tên-ngôn-ngữ].md`.
-- **Nếu review cross-cutting** và cần xác minh architecture → đọc `references/cross-cutting/architecture-review-guide.md`.
-- **Default (80% trường hợp):** Dùng kiến thức có sẵn, không đọc references.
-
----
-
-## 4-Phase Review Process
-
-Là một AI Architect, quá trình review phải tuân thủ nghiêm ngặt 4 giai đoạn sau:
-
-### Phase 1: Context Gathering & Loading (Thu thập & Nạp Ngữ Cảnh)
-- Đọc file mô tả (PR, Issue) hoặc yêu cầu của người dùng.
-- **Thực thi Dynamic Context Loading** (đọc các file reference cần thiết như đã định nghĩa ở trên).
-
-### Phase 2: High-Level Review (Kiến trúc & Chuẩn mực)
-- **Đối với AI Config:** Kiểm tra cấu trúc V8 (Identity, Intent, Complexity, Triggers). 
-- **Đối với Code Phần mềm:** Kiểm tra SOLID, Coupling/Cohesion, Performance, Security (Tham chiếu theo `cross-cutting` guides).
-
-### Phase 3: Deep-dive & Logic Review (Phân tích chi tiết)
-- **Đối với AI Config:** Kiểm tra rủi ro "ảo giác" (hallucination), token abuse, side-effects, stop_early.
-- **Đối với Code Phần mềm:** Rà soát edge cases, lỗi logic chuyên sâu theo từng ngôn ngữ (Tham chiếu theo `languages` guides).
-
-### Phase 4: Summary & Decision (Tổng hợp & Quyết định)
-- Đưa ra báo cáo theo `Output Format`.
-- Gắn nhãn `Severity Tags` rõ ràng.
-
----
-
-## Feedback Principles (Nguyên tắc Phản hồi)
-
-- **Collaborative Language (Ngôn từ hợp tác)**: Thay vì ra lệnh ("Sửa cái này thành X", "Bỏ dòng này đi"), hãy dùng câu hỏi gợi mở ("Nếu chúng ta dùng X ở đây thì có tối ưu hiệu năng hơn không?", "Có vẻ logic này bị lặp, chúng ta extract nó ra hàm riêng được không?").
-- **Differentiate Severity (Phân biệt mức độ)**: Bắt buộc sử dụng các nhãn sau để phân loại mức độ phản hồi:
-  - 🔴 `[blocking]` - Lỗi nghiêm trọng (VD: Security flaw, thiếu token_budget). Bắt buộc phải sửa.
-  - 🟡 `[important]` - Lỗi quan trọng (VD: Performance leak, trigger quá rộng). Nên sửa hoặc cần thảo luận.
-  - 🟢 `[nit]` - Cải thiện nhỏ (VD: Lỗi chính tả, format, naming). Không bắt buộc.
-  - 💡 `[suggestion]` - Cách tiếp cận hoặc thư viện thay thế để tham khảo.
-  - 📚 `[learning]` - Giải thích nguyên lý (Why) để chia sẻ kiến thức, không yêu cầu hành động.
-  - 🎉 `[praise]` - Lời khen cho đoạn code / logic thiết kế tốt.
-
----
-
-## Output Format
-
-```markdown
-## qk-code-review Report
-─────────────────────────────────────────────────
-**Target:** [Tên file / Chức năng]
-**Status:** SUCCESS | BLOCKED | FAILED | PARTIAL
-
-### 📊 Executive Summary
-[1-2 câu tóm tắt chất lượng tổng thể của mã nguồn/cấu hình. VD: "Kiến trúc rõ ràng, nhưng tiềm ẩn rủi ro lặp vô hạn ở dòng 45."]
-
-### 🔍 Findings & Recommendations
-
-[Nhóm các findings theo mức độ nghiêm trọng giảm dần. LUÔN đính kèm snippet / file line nếu có thể]
-
-- 🔴 `[blocking]`: [Vấn đề nghiêm trọng]
-  - *Location*: `file.ts:L45`
-  - *Feedback*: [Câu hỏi/Gợi ý sửa chữa]
-  
-- 🟡 `[important]`: [Vấn đề quan trọng]
-- 💡 `[suggestion]`: [Gợi ý cải thiện]
-- 🟢 `[nit]`: [Góp ý nhỏ]
-- 🎉 `[praise]`: [Khen ngợi]
-
-### 🛠️ Suggested Fixes (Optional)
-[Cung cấp Code diff hoặc YAML chuẩn xác để user dễ dàng copy & paste. TUYỆT ĐỐI KHÔNG tự sửa file nếu user chưa yêu cầu]
-
-### ✅ Verdict
-- [ ] Approve (Có thể merge/deploy ngay)
-- [ ] Changes Requested (Cần sửa các mục `[blocking]`)
-- [ ] Comment (Chỉ là gợi ý, quyền quyết định ở user)
-```
+---
+# ── Identity ───────────────────────────────────────────────
+name: qk-code-review
+version: 8.0.0
+status: stable
+description: "Elite AI/Code Review System: Kiểm toán, Review code và cấu hình AI với tư duy Architect, áp dụng 4-Phase Review."
+platforms: [antigravity, claude-code, cursor, windsurf, kilo-code]
+
+# ── V8: Classification ─────────────────────────────────────
+type: utility
+
+intent:
+  - code-review
+  - quality-assurance
+  - architecture-review
+  - ai-config-audit
+
+complexity:
+  level: high
+  criteria:
+    files_affected: "1-15"
+    has_behavior_change: false
+    has_external_dependency: false
+    has_breaking_change: false
+
+triggers:
+  - "review code"
+  - "code review"
+  - "kiểm tra code"
+  - "đánh giá code"
+  - "review skin"
+  - "review rule"
+  - "review ai"
+
+selection:
+  priority: high
+  confidence_threshold: 0.85
+
+# ── V8: References ─────────────────────────────────────────
+workflow: code-review
+
+rules:
+  - global
+
+tools:
+  - filesystem
+
+related_skills:
+  - qk-validation-gate
+  - qk-engineering-standard
+  - qk-project-health
+
+knowledge_scope:
+  owns:
+    - code-review-standards
+    - ai-configuration-review
+    - feedback-delivery
+  references:
+    - architecture
+    - security
+    - anti-patterns
+
+# ── V8: Verification ───────────────────────────────────────
+verification:
+  required: true
+  strategy: review
+
+# ── V8: Knowledge links ────────────────────────────────────
+examples: []
+learnings: []
+
+# ── V7 Runtime ─────────────────────────────────────────────
+execution_mode: deterministic
+cost: high
+latency: medium
+risk: low
+side_effects: read_only
+produces: [report]
+consumes: [source-code, rules, workflows]
+
+token_budget:
+  max_files_read: 10
+  max_lines_per_read: 300
+  max_shell_commands: 2
+  stop_early: true
+
+exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
+---
+
+# qk-code-review — Elite Review System
+
+> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
+
+Biến quá trình Code Review từ "bắt bẻ" (gatekeeping) thành "chia sẻ tri thức" (knowledge sharing) thông qua phản hồi mang tính xây dựng, phân tích có hệ thống và hợp tác cải tiến.
+
+---
+
+## Preconditions
+
+- [ ] Xác định rõ đối tượng cần review: Mã nguồn phần mềm (React, Java, etc.) hay Cấu hình AI (Skin, Rules, Workflows).
+- [ ] Cung cấp ngữ cảnh hoặc mục tiêu của đoạn code/cấu hình cần review.
+
+```
+On missing precondition → EXIT: BLOCKED
+Report: "Missing: Vui lòng cung cấp ngữ cảnh hoặc chỉ định rõ file cần review."
+```
+
+---
+
+## Dynamic Context Loading (Tải Ngữ Cảnh Động)
+
+> **CHỈ ĐỌC KHI CẦN (on-demand):** Chỉ mở file references/ nếu có vấn đề cụ thể cần xác minh. Đừng đọc trước khi chưa thấy vấn đề.
+
+- **Nếu review AI Config** và phát hiện vấn đề schema/anti-pattern → đọc `references/ai/v8-schema-validation.md` HOẶC `references/ai/ai-anti-patterns.md` (không cần đọc cả 2 nếu không liên quan).
+- **Nếu review Code** và gặp vấn đề ngôn ngữ cụ thể → đọc `references/languages/[tên-ngôn-ngữ].md`.
+- **Nếu review cross-cutting** và cần xác minh architecture → đọc `references/cross-cutting/architecture-review-guide.md`.
+- **Default (80% trường hợp):** Dùng kiến thức có sẵn, không đọc references.
+
+---
+
+## 4-Phase Review Process
+
+Là một AI Architect, quá trình review phải tuân thủ nghiêm ngặt 4 giai đoạn sau:
+
+### Phase 1: Context Gathering & Loading (Thu thập & Nạp Ngữ Cảnh)
+- Đọc file mô tả (PR, Issue) hoặc yêu cầu của người dùng.
+- **Thực thi Dynamic Context Loading** (đọc các file reference cần thiết như đã định nghĩa ở trên).
+
+### Phase 2: High-Level Review (Kiến trúc & Chuẩn mực)
+- **Đối với AI Config:** Kiểm tra cấu trúc V8 (Identity, Intent, Complexity, Triggers). 
+- **Đối với Code Phần mềm:** Kiểm tra SOLID, Coupling/Cohesion, Performance, Security. **BẮT BUỘC đối chiếu với luật chống rác mã (R-C-09) và ranh giới Zero-Trust (R-SEC-04). Vi phạm sẽ bị đánh tag `[blocking]`.**
+
+### Phase 3: Deep-dive & Logic Review (Phân tích chi tiết)
+- **Đối với AI Config:** Kiểm tra rủi ro "ảo giác" (hallucination), token abuse, side-effects, stop_early.
+- **Đối với Code Phần mềm:** Rà soát edge cases, lỗi logic chuyên sâu theo từng ngôn ngữ (Tham chiếu theo `languages` guides).
+
+### Phase 4: Summary & Decision (Tổng hợp & Quyết định)
+- Đưa ra báo cáo theo `Output Format`.
+- Gắn nhãn `Severity Tags` rõ ràng.
+
+---
+
+## Feedback Principles (Nguyên tắc Phản hồi)
+
+- **Collaborative Language (Ngôn từ hợp tác)**: Thay vì ra lệnh ("Sửa cái này thành X", "Bỏ dòng này đi"), hãy dùng câu hỏi gợi mở ("Nếu chúng ta dùng X ở đây thì có tối ưu hiệu năng hơn không?", "Có vẻ logic này bị lặp, chúng ta extract nó ra hàm riêng được không?").
+- **Differentiate Severity (Phân biệt mức độ)**: Bắt buộc sử dụng các nhãn sau để phân loại mức độ phản hồi:
+  - 🔴 `[blocking]` - Lỗi nghiêm trọng (VD: Security flaw, thiếu token_budget). Bắt buộc phải sửa.
+  - 🟡 `[important]` - Lỗi quan trọng (VD: Performance leak, trigger quá rộng). Nên sửa hoặc cần thảo luận.
+  - 🟢 `[nit]` - Cải thiện nhỏ (VD: Lỗi chính tả, format, naming). Không bắt buộc.
+  - 💡 `[suggestion]` - Cách tiếp cận hoặc thư viện thay thế để tham khảo.
+  - 📚 `[learning]` - Giải thích nguyên lý (Why) để chia sẻ kiến thức, không yêu cầu hành động.
+  - 🎉 `[praise]` - Lời khen cho đoạn code / logic thiết kế tốt.
+
+---
+
+## Output Format
+
+```markdown
+## qk-code-review Report
+─────────────────────────────────────────────────
+**Target:** [Tên file / Chức năng]
+**Status:** SUCCESS | BLOCKED | FAILED | PARTIAL
+
+### 📊 Executive Summary
+[1-2 câu tóm tắt chất lượng tổng thể của mã nguồn/cấu hình. VD: "Kiến trúc rõ ràng, nhưng tiềm ẩn rủi ro lặp vô hạn ở dòng 45."]
+
+### 🔍 Findings & Recommendations
+
+[Nhóm các findings theo mức độ nghiêm trọng giảm dần. LUÔN đính kèm snippet / file line nếu có thể]
+
+- 🔴 `[blocking]`: [Vấn đề nghiêm trọng]
+  - *Location*: `file.ts:L45`
+  - *Feedback*: [Câu hỏi/Gợi ý sửa chữa]
+  
+- 🟡 `[important]`: [Vấn đề quan trọng]
+- 💡 `[suggestion]`: [Gợi ý cải thiện]
+- 🟢 `[nit]`: [Góp ý nhỏ]
+- 🎉 `[praise]`: [Khen ngợi]
+
+### 🛠️ Suggested Fixes (Optional)
+[Cung cấp Code diff hoặc YAML chuẩn xác để user dễ dàng copy & paste. TUYỆT ĐỐI KHÔNG tự sửa file nếu user chưa yêu cầu]
+
+### ✅ Verdict
+- [ ] Approve (Có thể merge/deploy ngay)
+- [ ] Changes Requested (Cần sửa các mục `[blocking]`)
+- [ ] Comment (Chỉ là gợi ý, quyền quyết định ở user)
+```
+

package/.agents/skills/qk-context-loader/SKILL.md

@@ -54,6 +54,7 @@ knowledge_scope:
   references:
     - coding
     - repository-structure
+    - anti-patterns
 
 # ── V8: Verification ───────────────────────────────────────
 verification:
@@ -85,7 +86,6 @@ exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
 
 > **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
 
-
 ## Preconditions
 - [ ] Entry point file or module name is specified
 - [ ] Repository is accessible
@@ -96,7 +96,6 @@ On missing precondition:
   Message: "Vui lòng chỉ định entry point (file hoặc module cần map)."
 ```
 
-
 ## Scope
 - ✅ Trace imports/exports from entry point
 - ✅ Build topological dependency graph (max depth 3)
@@ -108,7 +107,6 @@ On missing precondition:
 - ❌ Guess file names — only follow explicit imports
 - ❌ Load entire repo (max 5 files)
 
-
 ## Priority Order
 
 | Priority | Task | Skip Threshold |
@@ -118,7 +116,6 @@ On missing precondition:
 | P3 | Identify shared/risky modules | Budget < 60% |
 | P4 | Annotate with risk levels | Budget < 70% |
 
-
 ## Workflow
 
 ### Phase 1 — Entry Point Analysis
@@ -137,7 +134,6 @@ ELSE IF entry file not found
   → EXIT: BLOCKED — ask for correct path
 ```
 
-
 ### Phase 2 — Graph Traversal (Max Depth 3)
 
 **Steps:**
@@ -155,14 +151,13 @@ IF circular dependency detected
   → Continue building rest of graph
 ```
 
-
 ### Phase 3 — Risk Assessment & Output
 
 **Steps:**
 1. Identify: shared modules (imported by 3+ files) = HIGH risk to change
 2. Identify: entry points with many dependents = CRITICAL to change carefully
-3. Generate graph in standard JSON + Markdown format
-
+3. **Identify: file trộn lẫn Server Logic và Client UI, hoặc vi phạm ranh giới bảo mật R-SEC-04 = Bắt buộc flag [HIGH RISK].**
+4. Generate graph in standard JSON + Markdown format
 
 ## Output Format (Mandatory Schema)
 
@@ -190,7 +185,6 @@ IF circular dependency detected
 }
 ```
 
-
 ## Evidence Format
 
 ```
@@ -200,7 +194,6 @@ Imported by: [N files]
 Exports:    [key exports]
 ```
 
-
 ## Handoff Contract
 
 ### Consumes
@@ -220,7 +213,6 @@ Exports:    [key exports]
 }
 ```
 
-
 ## Exit Codes
 
 | Code | Meaning | When |

package/.agents/skills/qk-data-lifecycle/SKILL.md

@@ -47,6 +47,8 @@ knowledge_scope:
     - migrations
   references:
     - architecture
+    - security
+    - anti-patterns
 
 # ── V8: Verification ───────────────────────────────────────
 verification:
@@ -80,7 +82,7 @@ exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
 
 # qk-data-lifecycle — Database Schema & Repository Manager
 
-> **Language rule:** Code, identifiers, file names ? English. Explanations, summaries ? Vietnamese.
+> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
 
 ---
 
@@ -134,6 +136,9 @@ drop_column:
 
 rename_table:
   strategy: create new table → migrate → drop old (never ALTER TABLE RENAME in prod)
+  
+pii_data:
+  strategy: Khi add column chứa PII (Personal Identifiable Information), BẮT BUỘC áp dụng cơ chế hashing/encryption tương ứng (tuân thủ R-SEC-04).
 ```
 
 ---

package/.agents/skills/qk-db-optimizer/SKILL.md

@@ -47,6 +47,7 @@ knowledge_scope:
     - query-optimization
   references:
     - architecture
+    - anti-patterns
 
 # ── V8: Verification ───────────────────────────────────────
 verification:
@@ -80,7 +81,7 @@ exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
 
 # qk-db-optimizer — Database Performance Tuner
 
-> **Language rule:** Code, identifiers, file names ? English. Explanations, summaries ? Vietnamese.
+> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
 
 ---
 
@@ -112,7 +113,7 @@ On missing precondition:
 | P | Issue Type | Action | Skip Threshold |
 |---|-----------|--------|----------------|
 | P1 | Sequential scan on large table (> 10k rows) | Add composite index | Never |
-| P2 | N+1 query pattern | Eager load / DataLoader | Budget < 30% |
+| P2 | N+1 query pattern | Eager load / DataLoader (Ưu tiên theo chuẩn dự án, không tự chế query lồng) | Budget < 30% |
 | P3 | Missing JOIN (multiple queries for related data) | Rewrite with JOIN | Budget < 50% |
 | P4 | SELECT * (over-fetching) | Select specific columns | Budget < 60% |