ai-developer-skill-os 6.0.3 → 7.0.1

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

@@ -1,36 +1,42 @@
 ---
 name: qk-ui-audit
 category: qa
-version: 6.0.3
-description: "Kiểm toán giao diện (UI) để đảm bảo tính nhất quán, khả năng truy cập (A11y), độ phản hồi (Responsive) và hiệu suất."
+version: 7.0.0
+description: "Kiểm toán giao diện (UI) bằng checklist chống rác (Anti-slop) khắt khe."
 ---
 
 # qk-ui-audit
 
 ## Scope
-- UI consistency, accessibility (A11y), responsiveness, and performance auditing (Evaluate)
+- UI consistency, Anti-Slop validation, accessibility, and performance auditing (Evaluate)
+
+## Verbs
+- `[AUDIT]`: Score the UI against the 57-point strict checklist (Anti-slop).
 
 ## Constraints
 ```yaml
 must:
-  - Check UI against established design tokens and guidelines
-  - Verify ARIA roles and contrast ratios
+  - Score the UI components against `DESIGN.md` tokens.
+  - Fail the audit if generic, lazy AI aesthetics are detected (e.g., default Tailwind blue/red, cluttered layouts).
+  - Verify ARIA roles and contrast ratios.
+  - Require micro-interactions (hover, active states) for all interactive elements.
 must_not:
-  - Ignore mobile viewport constraints
+  - Pass the audit if the UI looks like a "cheap template".
+  - Ignore mobile viewport constraints.
 ```
 
 ## Policies
 ```yaml
 prefer:
-  - Identifying systemic UI issues over isolated pixel tweaks
+  - Identifying systemic UI issues and demanding complete REDESIGN over isolated pixel tweaks.
 ```
 
 ## Escalation
 ```yaml
 stop:
-  - UI components completely lack semantic structure
+  - UI fails the Anti-Slop Audit (Score < 90/100). The agent MUST self-correct or ask the user to reject the code.
+  - UI components completely lack semantic structure.
 ```
 
 ## Output
-- UI audit report with actionable fixes
-```
+- Strict UI audit report with actionable fixes, demanding redesign if slop is detected.

package/skills/qk-ui-system-builder/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: qk-ui-system-builder
 category: frontend
-version: 6.0.3
+version: 7.0.0
 description: "Xây dựng và duy trì Design Systems, Token extraction, và các thư viện UI dùng chung."
 ---
 
@@ -9,28 +9,35 @@ description: "Xây dựng và duy trì Design Systems, Token extraction, và cá
 
 ## Scope
 - Building and maintaining Design Systems and shared UI tokens (Plan & Execute)
+- Enforcing Open Design constraints via `DESIGN.md`
+
+## Verbs
+- `[EXTRACT]`: Read `DESIGN.md` and generate CSS variables/tokens.
+- `[ENFORCE]`: Ensure no generic "AI-slop" tokens are created.
 
 ## Constraints
 ```yaml
 must:
-  - Extract reusable design tokens (colors, spacing, typography)
-  - Ensure components are accessible (A11y)
+  - "Extract reusable design tokens (colors, spacing, typography) from DESIGN.md ONLY"
+  - "Ensure components are accessible (A11y)"
 must_not:
-  - Create duplicate tokens for the same visual value
+  - "Create duplicate tokens for the same visual value"
+  - "Invent tokens (e.g., standard Tailwind colors) not explicitly defined in DESIGN.md"
 ```
 
 ## Policies
 ```yaml
 prefer:
+  - Strict adherence to HSL/Oklch over generic HEX colors
   - Composable components over rigid layouts
 ```
 
 ## Escalation
 ```yaml
 stop:
-  - Design tokens conflict with existing design guidelines
+  - "DESIGN.md is missing (Prompt user to run bootstrap or provide one)"
+  - "Design tokens conflict with existing DESIGN.md guidelines"
 ```
 
 ## Output
-- UI component library or Design System tokens
-```
+- Strict UI component library or Design System tokens mapped to `DESIGN.md`

package/skills/qk-validation-gate/SKILL.md

@@ -1,37 +1,40 @@
 ---
 name: qk-validation-gate
 category: qa
-version: 6.0.3
-description: "Cổng kiểm tra chất lượng bắt buộc (Test, Lint, Security)."
+version: 7.0.0
+description: "Cổng kiểm tra chất lượng bắt buộc (Test, Lint, Security) chặn đứng mọi mã nguồn lỗi."
 ---
 
 # qk-validation-gate
 
 ## Scope
-- Mandatory quality checks (Test, Lint, Security) before merge/commit (Evaluate)
+- Pre-commit/Pre-push Quality Gates and Test execution (Evaluate)
+
+## Verbs
+- `[VALIDATE]`: Run tests, linters, and security checks before any merge or commit.
 
 ## Constraints
 ```yaml
 must:
-  - Verify acceptance criteria and test coverage
-  - Fail early if critical errors or lint violations are found
+  - "Run all unit/integration tests before passing the gate"
+  - "Run static analysis (Linting, Type-checking) on modified files"
 must_not:
-  - Approve incomplete work or failing tests
-  - Bypass security scans
+  - "Bypass failing tests with generic 'ignore' flags"
+  - "Allow commits with unaddressed security warnings"
 ```
 
 ## Policies
 ```yaml
 prefer:
-  - Automated testing over manual verification
+  - "Strict failure over silent continuation"
 ```
 
 ## Escalation
 ```yaml
 stop:
-  - Artifact is absent or tests are failing
+  - "Tests fail or Code coverage drops below threshold"
+  - "Linter reports errors (not just warnings)"
 ```
 
 ## Output
-- Pass/Fail decision and validation report
-```
+- Validation report (Pass/Fail) and execution logs.

package/templates/DESIGN.md

@@ -0,0 +1,38 @@
+# DESIGN.md - The Brand Contract
+
+> **CRITICAL RULE:** This file is the Single Source of Truth for all UI/UX in this project. 
+> Any AI agent performing frontend work MUST adhere strictly to the rules, tokens, and components defined here. 
+> DO NOT invent generic Tailwind colors or default layouts. DO NOT use "slop" AI aesthetics.
+
+## 1. Typography
+- **Primary Font:** [Specify Font Family, e.g., Inter, Roboto]
+- **Heading Font:** [Specify Font Family if different]
+- **Base Size:** [e.g., 16px]
+- **Scale:** [e.g., Major Third (1.25) or Perfect Fourth (1.333)]
+
+## 2. Color Palette (Strict Constraints)
+- **Primary:** HSL(xxx, xx%, xx%)
+- **Secondary:** HSL(xxx, xx%, xx%)
+- **Background (Light):** HSL(xxx, xx%, xx%)
+- **Background (Dark):** HSL(xxx, xx%, xx%)
+- **Text (Light/Dark):** HSL(xxx, xx%, xx%)
+- **Forbidden Colors:** [e.g., Default Tailwind `blue-500`, `red-500`. NEVER use default primary colors].
+
+## 3. Spacing & Grid System
+- **Base Unit:** [e.g., 4px / 0.25rem]
+- **Grid Layout:** [e.g., 12-column grid, max-width: 1280px]
+- **Whitespace Rule:** Always favor generous whitespace (whitespace is cheap, clutter is expensive).
+
+## 4. UI Components & Micro-interactions
+- **Border Radius:** [e.g., 8px for cards, 4px for buttons, 9999px for pills]
+- **Shadows/Elevation:** [Define soft, organic shadows; avoid harsh, generic drop shadows]
+- **Animations:** [e.g., 200ms ease-out for hover states, use micro-animations for button presses].
+
+## 5. Anti-Slop Guidelines (Hallmark Rules)
+- **NO Default Borders:** Avoid adding a 1px solid border to everything. Use background color shifts or soft shadows instead.
+- **NO Cluttered Forms:** Forms must have clear visual hierarchy, grouped inputs, and explicit focus states.
+- **NO Floating Elements:** Elements must be anchored visually to a grid.
+- **MUST Check Contrast:** All text must meet WCAG AA contrast ratios.
+
+---
+*Note to AI Agent: If you are about to emit code that violates these rules, STOP. Redesign it, run an [AUDIT], and apply the correct constraints.*

package/tests/behavior-conformance.test.js

@@ -22,12 +22,14 @@ function parseBSF(skillPath) {
   const fmMatch = content.match(/^---\s*\n([\s\S]*?)\n---/);
   if (!fmMatch) return null;
   const frontmatter = YAML.load(fmMatch[1]);
-  if (frontmatter.version !== '6.0.0') return { type: 'v5', frontmatter };
-
-  const bsf = { type: 'v6', frontmatter };
+  let type = 'legacy';
+  if (frontmatter.version && frontmatter.version.startsWith('7.')) type = 'v7';
+  else if (frontmatter.version && frontmatter.version.startsWith('6.')) type = 'v6';
+  
+  const bsf = { type, frontmatter };
   
   // Parse YAML blocks
-  const yamlBlocks = [...content.matchAll(/```yaml\n([\s\S]*?)\n```/g)];
+  const yamlBlocks = [...content.matchAll(/```yaml\r?\n([\s\S]*?)\r?\n```/g)];
   
   try {
     let parsedBlocks = {};
@@ -61,24 +63,24 @@ function parseBSF(skillPath) {
 describe('Behavior Validation Framework', () => {
 
   describe('Level 1: Specification Valid', () => {
-    it('Every v6 skill must have valid Schema (Metadata, Scope, Constraints)', () => {
+    it('Every v6/v7 skill must have valid Schema (Metadata, Scope, Constraints)', () => {
       const dirs = getActiveSkillDirs();
-      const v6Skills = [];
+      const modernSkills = [];
 
       dirs.forEach(dir => {
         const skillPath = path.join(dir, 'SKILL.md');
         if (!fs.existsSync(skillPath)) return;
         const bsf = parseBSF(skillPath);
         
-        if (bsf && bsf.type === 'v6') {
-          v6Skills.push(dir);
+        if (bsf && bsf.type === 'v7') {
+          modernSkills.push(dir);
           expect(bsf.error, `Parser error in ${path.basename(dir)}: ${bsf.error}`).toBeUndefined();
           expect(bsf.frontmatter.category, `Missing category metadata in ${path.basename(dir)}`).toBeDefined();
           expect(bsf.constraints, `Missing Constraints in ${path.basename(dir)}`).toBeDefined();
         }
       });
 
-      expect(v6Skills.length).toBeGreaterThan(0);
+      expect(modernSkills.length).toBeGreaterThan(0);
     });
   });
 
@@ -91,12 +93,13 @@ describe('Behavior Validation Framework', () => {
         if (!fs.existsSync(skillPath)) return;
         const bsf = parseBSF(skillPath);
         
-        if (bsf && bsf.type === 'v6' && !bsf.error) {
+        if (bsf && bsf.type === 'v7' && !bsf.error) {
           const must = bsf.constraints.must || [];
           const must_not = bsf.constraints.must_not || [];
           
           must.forEach(rule => {
-            const conflict = must_not.some(c => c.toLowerCase() === rule.toLowerCase());
+            if (typeof rule !== 'string') return;
+            const conflict = must_not.some(c => typeof c === 'string' && c.toLowerCase() === rule.toLowerCase());
             expect(conflict, `Conflict detected in ${path.basename(dir)}: '${rule}' is in both must and must_not`).toBe(false);
           });
         }

package/scratch_update.cjs

@@ -1,66 +0,0 @@
-const fs = require('fs');
-const path = require('path');
-
-const repoDir = 'd:\\ai-code-skin-mcp\\rules-skill';
-const skillsJsonPath = path.join(repoDir, 'skills.json');
-
-try {
-  const data = fs.readFileSync(skillsJsonPath, 'utf8');
-  const skillsConfig = JSON.parse(data);
-
-  skillsConfig.skills.forEach(skill => {
-    if (skill.path && skill.description) {
-      const skillMdPath = path.join(repoDir, skill.path);
-      if (fs.existsSync(skillMdPath)) {
-        let content = fs.readFileSync(skillMdPath, 'utf8');
-        const frontmatterRegex = /^---\r?\n([\s\S]*?)\r?\n---/;
-        const match = content.match(frontmatterRegex);
-
-        if (match) {
-          let frontmatter = match[1];
-          // Update version to 6.0.3 in frontmatter
-          if (/^version:/m.test(frontmatter)) {
-            frontmatter = frontmatter.replace(/^version:.*$/m, `version: 6.0.3`);
-          } else {
-            frontmatter += `\nversion: 6.0.3`;
-          }
-
-          // Check if description already exists
-          if (/^description:/m.test(frontmatter)) {
-            frontmatter = frontmatter.replace(/^description:.*$/m, `description: "${skill.description}"`);
-          } else {
-            frontmatter += `\ndescription: "${skill.description}"`;
-          }
-          
-          const newContent = content.replace(frontmatterRegex, `---\n${frontmatter}\n---`);
-          fs.writeFileSync(skillMdPath, newContent, 'utf8');
-          console.log(`Updated ${skillMdPath}`);
-        } else {
-          console.log(`No frontmatter found in ${skillMdPath}`);
-        }
-      } else {
-        console.log(`File not found: ${skillMdPath}`);
-      }
-    }
-  });
-
-  // Also update package.json version
-  const packageJsonPath = path.join(repoDir, 'package.json');
-  if (fs.existsSync(packageJsonPath)) {
-    const pkg = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-    pkg.version = "6.0.3";
-    fs.writeFileSync(packageJsonPath, JSON.stringify(pkg, null, 2), 'utf8');
-    console.log('Updated package.json to 6.0.3');
-  }
-
-  // Also update skills.json version if needed
-  if (skillsConfig.version) {
-    skillsConfig.version = "6.0.3";
-    fs.writeFileSync(skillsJsonPath, JSON.stringify(skillsConfig, null, 2), 'utf8');
-    console.log('Updated skills.json version to 6.0.3');
-  }
-
-  console.log('Done!');
-} catch (err) {
-  console.error('Error:', err);
-}