ai-developer-skill-os 8.2.0 → 8.3.0

package/.agents/skills/qk-test-engineering/SKILL.md

@@ -1,139 +1,262 @@
----
-# ── Identity ───────────────────────────────────────────────
-name: qk-test-engineering
-version: 8.1.0
-status: experimental
-description: "Công cụ quyết định chiến lược kiểm thử (Unit, Integration, E2E, Coverage, Regression, Mock)."
-platforms: [antigravity, claude-code, cursor, windsurf, kilo-code]
-
-# ── V8: Classification ─────────────────────────────────────
-type: capability
-
-intent:
-  - testing-strategy
-  - unit-test
-  - integration-test
-  - e2e-test
-  - mock-strategy
-  - test-coverage
-
-complexity:
-  level: high
-  criteria:
-    files_affected: "1-5"
-    has_behavior_change: true
-    has_external_dependency: true
-    has_breaking_change: false
-
-triggers:
-  - "chiến lược test"
-  - "viết test"
-  - "mock data"
-  - "test coverage"
-
-# ── V8: References ─────────────────────────────────────────
-workflow: feature-delivery
-
-rules:
-  - global
-  - coding
-
-tools:
-  - filesystem
-  - terminal
-
-related_skills:
-  - qk-engineering-standard
-
-knowledge_scope:
-  domain:
-    - testing-strategy
-    - quality-assurance
-    - unit-testing
-    - integration-testing
-    - e2e-testing
-    - visual-regression
-  concepts:
-    - test-pyramid
-    - mock-strategy
-  references:
-    - architecture
-    - playwright
-    - vitest
-    - storybook-testing
-
-decision_boundary:
-  owns:
-    - test pyramid
-    - coverage strategy
-    - regression planning
-  does_not_own:
-    - business logic implementation
-    - writing tests
-  conflicts_with: []
-
-# ── V8: Verification ───────────────────────────────────────
-verification:
-  required: true
-  strategy: feature
-
-lifecycle:
-  promotion_gate:
-    requirements:
-      tests:
-        minimum_pass_rate: 0.95
-      usage:
-        minimum_runs: 20
-      conflicts:
-        zero_boundary_violation: true
-      evidence:
-        required:
-          - evaluation_report
-          - usage_history
-          - boundary_audit
-  demotion_gate:
-    triggers:
-      - repeated_failure
-      - boundary_violation
-      - outdated_reference
-    action:
-      change_status: "stable -> experimental"
-
-selection:
-  priority: medium
-  confidence_threshold: 0.80
-
-examples: []
-learnings: []
-
-# ── V7 Runtime ─────────────────────────────────────────────
-execution_mode: deterministic
-cost: medium
-latency: medium
-risk: low
-side_effects: read_only
-produces: [report, test_plan]
-consumes: [user-description]
-
-token_budget:
-  max_files_read: 5
-  max_lines_per_read: 150
-  max_shell_commands: 0
-  stop_early: false
-
-exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
----
-
-# qk-test-engineering — Test Engineering Strategist
-
-> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
-
-## Preconditions
-- [ ] Có code hoặc requirement rõ ràng cần kiểm thử.
-
-## Scope
-- Đưa ra quyết định chọn loại test (Unit, Integration, E2E) theo Test Pyramid.
-- Thiết kế chiến lược Mock (Mocking vs Stubbing vs Faking).
-- Đánh giá Test Coverage hiện tại và đưa ra target.
-- Quyết định Regression analysis (những phần nào cần test lại khi file A thay đổi).
-- Không tự động sinh test trừ khi user yêu cầu implement, skill này chủ yếu tư vấn architecture và strategy cho QA/Testing.
+---
+# ── Identity ───────────────────────────────────────────────
+name: qk-test-engineering
+version: 8.1.0
+status: experimental
+description: "Công cụ quyết định chiến lược kiểm thử (Unit, Integration, E2E, Coverage, Regression, Mock)."
+platforms: [antigravity, claude-code, cursor, windsurf, kilo-code]
+
+# ── V8: Classification ─────────────────────────────────────
+type: capability
+
+intent:
+  - testing-strategy
+  - unit-test-strategy
+  - integration-test-strategy
+  - e2e-test-strategy
+  - mock-strategy
+  - test-coverage-planning
+
+complexity:
+  level: high
+  criteria:
+    files_affected: "1-5"
+    has_behavior_change: true
+    has_external_dependency: true
+    has_breaking_change: false
+
+triggers:
+  - "chiến lược test"
+  - "viết test"
+  - "mock data"
+  - "test coverage"
+
+# ── V8: References ─────────────────────────────────────────
+workflow: feature-delivery
+
+rules:
+  - global
+  - coding
+
+tools:
+  - filesystem
+  - terminal
+
+related_skills:
+  - qk-engineering-standard
+
+knowledge_scope:
+  domain:
+    - testing-strategy
+    - quality-assurance
+    - unit-testing
+    - integration-testing
+    - e2e-testing
+    - visual-regression
+  concepts:
+    - test-pyramid
+    - mock-strategy
+  references:
+    - architecture
+    - anti-patterns
+    - playwright
+    - vitest
+    - storybook-testing
+
+decision_boundary:
+  owns:
+    - test pyramid
+    - coverage strategy
+    - regression planning
+  does_not_own:
+    - business logic implementation
+    - writing tests
+  conflicts_with: []
+
+# ── V8: Verification ───────────────────────────────────────
+verification:
+  required: true
+  strategy: feature
+
+lifecycle:
+  promotion_gate:
+    requirements:
+      tests:
+        minimum_pass_rate: 0.95
+      usage:
+        minimum_runs: 20
+      conflicts:
+        zero_boundary_violation: true
+      evidence:
+        required:
+          - evaluation_report
+          - usage_history
+          - boundary_audit
+  demotion_gate:
+    triggers:
+      - repeated_failure
+      - boundary_violation
+      - outdated_reference
+    action:
+      change_status: "stable -> experimental"
+
+selection:
+  priority: medium
+  confidence_threshold: 0.80
+
+examples: []
+learnings: []
+
+# ── V7 Runtime ─────────────────────────────────────────────
+execution_mode: deterministic
+cost: medium
+latency: medium
+risk: low
+side_effects: read_only
+produces: [report, test_plan]
+consumes: [user-description]
+
+token_budget:
+  max_files_read: 5
+  max_lines_per_read: 150
+  max_shell_commands: 0
+  stop_early: true
+
+exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
+---
+
+# qk-test-engineering — Test Engineering Strategist
+
+> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
+
+## Preconditions
+- [ ] Có code hoặc requirement rõ ràng cần kiểm thử.
+
+## Scope
+- Đưa ra quyết định chọn loại test (Unit, Integration, E2E) theo Test Pyramid.
+- Thiết kế chiến lược Mock (Mocking vs Stubbing vs Faking).
+- Đánh giá Test Coverage hiện tại và đưa ra target.
+- Quyết định Regression analysis (những phần nào cần test lại khi file A thay đổi).
+- **Bắt buộc thiết kế test case cho ranh giới dữ liệu (Negative Testing / Boundary Testing) để đảm bảo tuân thủ R-SEC-04.**
+- Không tự động sinh test trừ khi user yêu cầu implement, skill này chủ yếu tư vấn architecture và strategy cho QA/Testing.
+
+## Non-Goals
+- ❌ Provide implementation outside of Testing Strategy scope
+- ❌ Override explicit user directives without explanation
+- ❌ Guess ambiguous requirements without asking
+
+## Priority Order
+
+| Priority | Task | Skip Threshold |
+|----------|------|----------------|
+| P1 | Core Testing Strategy analysis and decision making | Never |
+| P2 | Validation of existing patterns | Budget < 30% |
+| P3 | Detailed documentation generation | Budget < 50% |
+| P4 | Edge case exploration | Budget < 70% |
+
+## Workflow
+
+### Phase 1 — Context Loading
+**Steps:**
+1. Read existing configuration and requirements related to Testing Strategy.
+2. Check for missing preconditions.
+
+**Decision:**
+```
+IF context is clear
+  → Confidence: HIGH → go to Phase 2
+ELSE
+  → EXIT: BLOCKED — ask user
+```
+
+### Phase 2 — Analysis & Strategy
+**Steps:**
+1. Analyze the current state against Testing Strategy best practices.
+2. Formulate strategy or audit report based on findings.
+
+**Decision:**
+```
+IF strategy/audit is complete
+  → Confidence: HIGH → go to Phase 3
+ELSE IF minor gaps exist
+  → Confidence: MEDIUM → proceed with assumptions noted
+```
+
+### Phase 3 — Finalization
+**Steps:**
+1. Generate final report or configuration.
+2. Prepare handoff data for subsequent skills.
+
+## Confidence Model
+
+| Level | Condition | Action |
+|-------|-----------|--------|
+| HIGH | All preconditions met, context fully understood | Proceed directly |
+| MEDIUM | Some context missing but safe defaults exist | Proceed and note assumptions |
+| LOW | Core requirements missing | EXIT: BLOCKED |
+
+## Severity (for findings)
+
+| Level | Definition |
+|-------|-----------|
+| CRITICAL | Severe violation of Testing Strategy principles |
+| HIGH | Significant risk or technical debt |
+| MEDIUM | Suboptimal pattern but functional |
+| LOW | Minor style or documentation issue |
+
+## Evidence Format
+
+```
+[SEVERITY] Context/File
+Issue:      [what was found]
+Confidence: HIGH
+Recommendation: [actionable advice]
+```
+
+## Retry Policy
+```
+Task fails due to missing context
+  └─ Ask user for clarification
+       ├─ Provided → Retry Phase 1
+       └─ Not provided → EXIT: BLOCKED
+```
+
+## Escalation Rules
+
+```
+BLOCKED: Missing critical context for Testing Strategy
+Missing:
+  - [Specific requirement]
+Questions:
+  1. Bạn có thể cung cấp thêm thông tin về yêu cầu này không?
+  2. Mục tiêu chính của bạn là gì?
+Recommended Assumptions: none
+```
+
+## Handoff Contract
+
+### Consumes
+```json
+{
+  "from": "user or qk-orchestrator",
+  "required_fields": ["context"],
+  "optional_fields": ["existing_config"]
+}
+```
+
+### Produces
+```json
+{
+  "to": "user or downstream skill",
+  "output_fields": ["strategy_report", "exit_code"]
+}
+```
+
+## Exit Codes
+
+| Code | Meaning | When |
+|------|---------|------|
+| SUCCESS | Testing Strategy task completed successfully | Strategy/audit generated |
+| PARTIAL | Task completed with assumptions | Medium confidence |
+| BLOCKED | Missing context | Cannot proceed |
+| FAILED | Critical conflict or error | Unresolvable constraint |

package/.agents/skills/qk-ui-audit/SKILL.md

@@ -3,7 +3,7 @@
 name: qk-ui-audit
 version: 8.0.0
 status: stable
-description: "Kiểm toán giao diện (UI) với 57-point Anti-Slop checklist — fail nếu score < 90/100."
+description: "Kiểm toán giao diện (UI) với 57-check Anti-Slop checklist — fail nếu score < 76/85 base."
 platforms: [antigravity, claude-code, cursor, windsurf, kilo-code]
 
 # ── V8: Classification ─────────────────────────────────────
@@ -48,6 +48,8 @@ knowledge_scope:
     - anti-slop
   references:
     - design-system
+    - security
+    - anti-patterns
 
 # ── V8: Verification ───────────────────────────────────────
 verification:
@@ -81,12 +83,7 @@ exit_codes: [SUCCESS, BLOCKED, FAILED, PARTIAL]
 
 # qk-ui-audit — Anti-Slop UI Inspector
 
-> **Language rule:** Code, identifiers, file names ? English. Explanations, summaries ? Vietnamese.
-
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
+> **Language rule:** Code, identifiers, file names → English. Explanations, summaries → Vietnamese.
 
 ## Preconditions
 - [ ] `DESIGN.md` exists in project root
@@ -98,11 +95,6 @@ On missing precondition:
   Message: "DESIGN.md not found. Run qk-project-bootstrap to create one, or provide design tokens manually."
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Scope
 - ✅ Score UI components against DESIGN.md tokens
 - ✅ Detect generic AI-slop aesthetics
@@ -114,11 +106,6 @@ schema_version: 2
 - ❌ Pixel-level screenshot comparison
 - ❌ Read entire CSS files — use targeted reads
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Priority Order
 
 | Priority | Category | Points | Skip Threshold |
@@ -127,16 +114,12 @@ schema_version: 2
 | P2 | Layout & Spacing (tokens, grid, whitespace) | 15pts | Budget < 30% |
 | P3 | Typography & Colors (DESIGN.md compliance) | 15pts | Budget < 50% |
 | P4 | Interactions & Animations (hover, focus, transitions) | 15pts | Budget < 60% |
-| P5 | Anti-Slop Detection (generic templates, slop colors) | 15pts | Budget < 70% |
+| P5 | Anti-Slop Detection (Tra chéo với R-C-09: thẻ card xám xịt, viền nhạt nhòa, text nhỏ...) | 15pts | Budget < 70% |
 | P6 | Performance & Best Practices | 10pts | Budget < 80% |
 | BONUS | Mobile & Cross-browser | 5pts extra | Always optional |
+| EXTENDED | Edge cases | 9pts extra | Always optional |
 
-**Total: 85 base + 5 bonus = 90 possible. Pass threshold: ≥ 90/100**
-
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
+**Total: 85 base + 14 bonus/extended = 99 max. Pass threshold: ≥ 76/85 base (90%)**
 
 ## Workflow
 
@@ -150,11 +133,6 @@ schema_version: 2
 - Tokens extracted → go to Phase 2
 - DESIGN.md empty or incomplete → EXIT: BLOCKED
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ### Phase 2 — Scan Target UI
 
 **Steps:**
@@ -164,7 +142,7 @@ schema_version: 2
 
 **Decision:**
 ```
-IF score accumulates ≥ 90 after P1+P2+P3
+IF score accumulates ≥ 76 after P1+P2+P3
   → Can skip P4–P6 if token budget < 40%
   → EXIT: PARTIAL (pass, but incomplete audit noted)
 
@@ -177,11 +155,6 @@ IF CRITICAL violation found (e.g., hardcoded password in UI, broken ARIA)
 - All categories checked → go to Phase 3
 - Token budget < 20% → go to Phase 3 with PARTIAL flag
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ### Phase 3 — Score & Report
 
 **Steps:**
@@ -191,21 +164,16 @@ schema_version: 2
 
 **Decision:**
 ```
-IF total score ≥ 90
+IF total score ≥ 76
   → EXIT: SUCCESS
 
-IF total score 70–89
+IF total score 60–75
   → EXIT: PARTIAL — list required fixes
 
-IF total score < 70
+IF total score < 60
   → EXIT: FAILED — demand redesign, not patch
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Confidence Model
 
 | Level | Condition | Action |
@@ -214,11 +182,6 @@ schema_version: 2
 | MEDIUM | Inferred from surrounding code patterns | Note assumption |
 | LOW | Cannot verify without rendering (e.g., animation timing) | Mark as "unverifiable — manual check required" |
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Severity
 
 | Level | Definition | Example |
@@ -228,11 +191,6 @@ schema_version: 2
 | MEDIUM | UX degraded, workaround exists | Missing hover state, no loading indicator |
 | LOW | Minor inconsistency | Spacing off by 1 unit |
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Evidence Format
 
 ```
@@ -254,22 +212,12 @@ Fix:        Replace with `var(--font-size-base)`
 Points:     -3 pts
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Retry Policy
 ```
 Audit is read-only — no retry needed.
 If file is inaccessible → note as PARTIAL and continue with other files.
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Escalation Rules
 
 ```
@@ -283,11 +231,6 @@ Recommended Assumptions (if proceeding):
   - Use industry-standard: 8px spacing unit, Inter font, neutral gray palette
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Handoff Contract
 
 ### Consumes
@@ -307,11 +250,6 @@ schema_version: 2
 }
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Output Format
 
 ```
@@ -328,7 +266,7 @@ Scores:
   P5 Anti-Slop:      [X/15]
   P6 Performance:    [X/10]
   ─────────────────
-  Total:             [X/85] → [PASS ≥ 90% | FAIL < 90%]
+  Total:             [X/85 base] → [PASS ≥ 76/85 | FAIL < 76/85]
 
 Violations (top priority first):
   [SEVERITY] file:LINE — reason — Fix: suggestion (-Xpts)
@@ -342,28 +280,17 @@ Verdict:     [PASS | FAIL — requires redesign]
 Exit Code:   [SUCCESS | PARTIAL | FAILED]
 ```
 
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
-
 ## Exit Codes
 
 | Code | Meaning | When |
 |------|---------|------|
-| SUCCESS | Score ≥ 90 — UI passes Anti-Slop | All checks done, no critical violations |
-| PARTIAL | Score 70–89 or incomplete audit | Some categories skipped due to token budget |
+| SUCCESS | Score ≥ 76 — UI passes Anti-Slop | All checks done, no critical violations |
+| PARTIAL | Score 60–75 or incomplete audit | Some categories skipped due to token budget |
 | BLOCKED | DESIGN.md missing or target not specified | Cannot audit without design contract |
-| FAILED | Score < 70 or CRITICAL violation found | Generic slop detected, redesign required |
-
-skill_version: 7.5.0
-runtime_version: 1
-schema_version: 2
----
+| FAILED | Score < 60 or CRITICAL violation found | Generic slop detected, redesign required |
 
 ## References
-- Full 57-point checklist: `references/anti-slop-checklist.md`
+- Full 57-check checklist: `references/anti-slop-checklist.md`
 
 ---
 
-

package/.agents/skills/qk-ui-audit/references/anti-slop-checklist.md

@@ -1,4 +1,4 @@
-# Anti-Slop Checklist — 57 Points
+# Anti-Slop Checklist — 57 Checks (99 pts max)
 > Referenced by: `qk-ui-audit/SKILL.md`
 > Purpose: Full inspection checklist for detecting generic AI-generated UI
 
@@ -132,5 +132,5 @@
 | Extended (49–57) | +9 |
 | **Maximum Possible** | **99** |
 
-**Pass threshold: ≥ 76 / 85 base (90%) — PASS**
+**Pass threshold: ≥ 76/85 base (90%)**
 **Fail: < 76 / 85 — requires redesign, not patch**