Validates PopKit code quality using concrete metrics for DRY, coupling, cohesion, and architectural patterns
Inherits all available tools
Additional assets for this skill
This skill inherits all available tools. When active, it can use any tool Claude has access to.
checklists/architecture-review.jsonchecklists/pattern-catalog.jsonscripts/analyze_structure.pyscripts/detect_patterns.pystandards/clean-architecture.mdstandards/separation-of-concerns.mdname: pop-assessment-architecture description: "Validates PopKit code quality using concrete metrics for DRY, coupling, cohesion, and architectural patterns" triggers:
Provides concrete, reproducible architecture assessment for PopKit plugins using:
python skills/pop-assessment-architecture/scripts/detect_duplication.py packages/plugin/
python skills/pop-assessment-architecture/scripts/analyze_coupling.py packages/plugin/
python skills/pop-assessment-architecture/scripts/calculate_quality.py packages/plugin/
Read and apply checklists in order:
checklists/dry-principles.json - Duplication detectionchecklists/separation-of-concerns.json - Module boundarieschecklists/error-handling.json - Error coveragechecklists/tool-selection.json - Appropriate tool usageCombine automated analysis with checklist results for final architecture report.
| Standard | File | Key Checks |
|---|---|---|
| DRY Principles | standards/dry-principles.md | DRY-001 through DRY-008 |
| Separation of Concerns | standards/separation-of-concerns.md | SOC-001 through SOC-008 |
| Error Handling | standards/error-handling.md | EH-001 through EH-010 |
| Tool Selection | standards/tool-selection.md | TS-001 through TS-008 |
| Metric | Good | Warning | Critical |
|---|---|---|---|
| Code Duplication | <5% | 5-15% | >15% |
| Cyclomatic Complexity | <10 | 10-20 | >20 |
| Module Coupling | Low | Medium | High |
| Module Cohesion | High | Medium | Low |
| Error Coverage | >80% | 50-80% | <50% |
| Principle | Check ID | Description |
|---|---|---|
| Single Responsibility | SOLID-001 | One reason to change |
| Open/Closed | SOLID-002 | Open for extension |
| Liskov Substitution | SOLID-003 | Proper inheritance |
| Interface Segregation | SOLID-004 | Minimal interfaces |
| Dependency Inversion | SOLID-005 | Depend on abstractions |
Returns JSON with:
quality_score: 0-100 (higher = better)duplication_percent: Code duplication levelcoupling_level: Low/Medium/Hightechnical_debt: List of debt itemsrefactoring_suggestions: Prioritized improvements