Quality Standards Skill

Context (Input)

•Need to understand protected quality thresholds
•Running comprehensive quality checks before commit
•Determining which specialized skill to use for specific issues
•Quick reference for quality tool commands

Task (Function)

Understand quality metrics and route to appropriate specialized skill for fixes.

Success Criteria: Know which skill to use for your specific quality issue.

Protected Quality Thresholds

CRITICAL: These thresholds are protected and must NEVER be lowered.

PHPInsights (Source Code)

Metric	Required	Fix With
Quality	100%	complexity-management
Complexity	94%	complexity-management
Architecture	100%	deptrac-fixer
Style	100%	Run `make phpcsfixer`

PHPInsights (Tests)

Metric	Required	Fix With
Quality	95%	complexity-management
Complexity	95%	complexity-management
Architecture	90%	deptrac-fixer
Style	95%	Run `make phpcsfixer`

Other Tools

Tool	Metric	Required	Fix With
Deptrac	Violations	0	deptrac-fixer
Psalm	Errors	0	Fix reported issues
Psalm	Security Issues	0	Fix tainted flows
Infection	MSI	100%	testing-workflow
PHPUnit	Coverage	100%	testing-workflow

Quick Reference Commands

Comprehensive Checks

bash

# Run all CI checks (recommended before commit)
make ci

Success: Must output "✅ CI checks successfully passed!"

Individual Quality Checks

Check	Command	Purpose
Code quality	`make phpinsights`	All PHPInsights metrics
Complexity analysis	`make phpmd`	Find high-complexity methods
Static analysis	`make psalm`	Type checking and errors
Security taint	`make psalm-security`	Security vulnerability scan
Architecture	`make deptrac`	Layer boundary validation
Code style	`make phpcsfixer`	Auto-fix PSR-12 style
Composer validation	`make composer-validate`	Validate composer.json

Testing Commands

Check	Command	Purpose
Unit tests	`make unit-tests`	Domain/Application logic
Integration tests	`make integration-tests`	Component interactions
E2E tests	`make behat`	Full user scenarios (Behat)
All tests	`make all-tests`	Unit + Integration + E2E
Test coverage	`make tests-with-coverage`	Generate coverage report
Mutation tests	`make infection`	Test quality validation

Routing to Specialized Skills

When quality checks fail, use the appropriate specialized skill:

Architecture Issues

•
Deptrac violations → deptrac-fixer
- •Domain depends on Infrastructure
- •Layer boundary violations
- •"must not depend on" errors
•
DDD architecture patterns → implementing-ddd-architecture
- •Creating new entities/value objects
- •Implementing CQRS patterns
- •Understanding layer responsibilities

Code Quality Issues

•
High cyclomatic complexity → complexity-management
- •PHPInsights complexity < 94%
- •PHPMD reports high CCN
- •Methods too complex
•
Code style issues → Run make phpcsfixer
- •PSR-12 violations
- •Line length > 100 chars
- •Formatting issues

Testing Issues

•
Test failures → testing-workflow
- •Unit/Integration/E2E failures
- •Mutation testing (Infection)
- •Test coverage < 100%

Workflow Integration

•
Before committing → ci-workflow
- •Run all checks systematically
- •Fix failures in priority order
- •Ensure all checks pass
•
PR review feedback → code-review
- •Fetch and address PR comments
- •Systematic comment resolution

Quality Improvement Workflow

Step 1: Run Comprehensive Checks

bash

make ci

Step 2: Identify Failing Check

Check output for specific failure:

code

❌ CI checks failed:
  - phpinsights: Complexity score too low (93.5% < 94%)

Step 3: Use Specialized Skill

Based on failure type, use appropriate skill:

Failure Pattern	Skill to Use
"Complexity score too low"	complexity-management
"Deptrac violations"	deptrac-fixer
"must not depend on"	deptrac-fixer
"tests failed"	testing-workflow
"Psalm found errors"	Fix type errors directly
"escaped mutants"	testing-workflow

Step 4: Re-run CI

bash

make ci

Repeat until: "✅ CI checks successfully passed!"

Constraints (Parameters)

NEVER

•Lower quality thresholds in config files (phpinsights.php, infection.json5, etc.)
•Skip failing checks to "save time"
•Commit code without all CI checks passing
•Modify deptrac.yaml to allow violations (fix code, not config)
•Disable security checks

ALWAYS

•Fix code to meet standards (not config to meet code)
•Run make ci before creating commits
•Use specialized skills for specific quality issues
•Maintain 100% test coverage
•Keep cyclomatic complexity low (target: < 5 per method)
•Respect hexagonal architecture boundaries

Format (Output)

Expected CI Output

code

✅ CI checks successfully passed!

Expected PHPInsights Output

code

[CODE] 100.0 pts       ✅ Target: 100%
[COMPLEXITY] 94.0 pts  ✅ Target: 94%
[ARCHITECTURE] 100 pts ✅ Target: 100%
[STYLE] 100.0 pts      ✅ Target: 100%

Expected Deptrac Output

code

✅ No violations found

Expected Infection Output

code

Mutation Score Indicator (MSI): 100%

Verification Checklist

After using this skill:

• Identified which quality check is failing
• Selected appropriate specialized skill for the issue
• Ready to execute specialized skill workflow
• Understand which threshold applies to the failure
• Know the command to re-run the check after fixes

Related Skills

•ci-workflow - Run comprehensive CI validation
•complexity-management - Reduce complexity, improve quality
•deptrac-fixer - Fix architectural violations
•implementing-ddd-architecture - Understand DDD patterns
•testing-workflow - Fix test failures, improve coverage

Reference Documentation

For detailed examples and patterns, see:

•Refactoring patterns → complexity-management skill
•Architecture rules → implementing-ddd-architecture skill
•Layer boundaries → deptrac-fixer skill
•Testing strategies → testing-workflow skill