Clean Code

Clean Code - Pragmatic AI Coding Standards

CRITICAL SKILL - Be concise, direct, and solution-focused.

Core Principles

Naming Rules

Rule: If you need a comment to explain a name, rename it.

Function Rules

Code Structure

AI Coding Style

Anti-Patterns (DON'T)

🔴 Before Editing ANY File (THINK FIRST!)

Before changing a file, ask yourself:

Quick Check:

File to edit: UserService.ts└── Who imports this? → UserController.ts, AuthController.ts└── Do they need changes too? → Check function signatures

🔴 Rule: Edit the file + all dependent files in the SAME task. 🔴 Never leave broken imports or missing updates.

Summary

Remember: The user wants working code, not a programming lesson.

🔴 Self-Check Before Completing (MANDATORY)

Before saying "task complete", verify:

🔴 Rule: If ANY check fails, fix it before completing.

Verification Scripts (MANDATORY)

🔴 CRITICAL: Each agent runs ONLY their own skill's scripts after completing work.

Agent → Script Mapping

❌ WRONG: test-engineer running ux_audit.py ✅ CORRECT: frontend-specialist running ux_audit.py

🔴 Script Output Handling (READ → SUMMARIZE → ASK)

When running a validation script, you MUST:

1. Run the script and capture ALL output
2. Parse the output - identify errors, warnings, and passes
3. Summarize to user in this format:

## Script Results: [script_name.py]
### ❌ Errors Found (X items)- [File:Line] Error description 1- [File:Line] Error description 2
### ⚠️ Warnings (Y items)- [File:Line] Warning description
### ✅ Passed (Z items)- Check 1 passed- Check 2 passed
**Should I fix the X errors?**

4. Wait for user confirmation before fixing
5. After fixing → Re-run script to confirm

🔴 VIOLATION: Running script and ignoring output = FAILED task. 🔴 VIOLATION: Auto-fixing without asking = Not allowed. 🔴 Rule: Always READ output → SUMMARIZE → ASK → then fix.