Assesses PopKit plugin architecture using Python scripts and checklists for DRY, coupling, cohesion, error handling, SOLID principles, and tool usage. Outputs JSON quality scores and refactoring suggestions.
From popkit-opsnpx claudepluginhub jrc1883/popkit-ai --plugin popkit-opsThis skill uses the workspace's default tool permissions.
checklists/architecture-review.jsonchecklists/pattern-catalog.jsonscripts/analyze_structure.pyscripts/detect_patterns.pystandards/clean-architecture.mdstandards/separation-of-concerns.mdDesigns and optimizes AI agent action spaces, tool definitions, observation formats, error recovery, and context for higher task completion rates.
Implements structured self-debugging workflow for AI agent failures: capture errors, diagnose patterns like loops or context overflow, apply contained recoveries, and generate introspection reports.
Compares coding agents like Claude Code and Aider on custom YAML-defined codebase tasks using git worktrees, measuring pass rate, cost, time, and consistency.
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