code-quality by bonny

---
name: code-quality
description: Runs linting and static analysis on PHP/CSS/JS using phpcs, phpstan, and rector. Use when checking code style, fixing lint errors, or running static analysis.
allowed-tools: Read, Grep, Glob, Bash
---

# Code Quality Standards

## Quick Commands

```bash
npm run php:lint        # Check PHP style
npm run php:lint-fix    # Auto-fix PHP issues
npm run php:phpstan     # Static analysis
npm run lint:js         # JavaScript
npm run lint:css        # CSS
```

## Project-Specific Rules

| Area               | Standard                                   |
| ------------------ | ------------------------------------------ |
| PHP                | 7.4+, WordPress Coding Standards           |
| Prefixes           | `sh`, `simplehistory`, `simple_history`    |
| Text domain        | `simple-history`                           |
| CSS naming         | SuitCSS: `sh-ComponentName-subpart`        |
| Array syntax       | Short `[]` not `array()`                   |
| Control structures | Always use braces `{}`, never colon syntax |

## Essential Principles

1. **Always escape output** - Use WordPress escaping functions
2. **Prefix everything** - All hooks, functions, classes
3. **Run tools after changes** - phpcs/phpstan before committing

## Detailed Guidelines

-   [php-standards.md](php-standards.md) - PHP style, happy path, early returns
-   [css-standards.md](css-standards.md) - SuitCSS naming conventions
-   [js-standards.md](js-standards.md) - JavaScript conventions
-   [tooling.md](tooling.md) - phpcs, phpstan, rector usage
-   [design-principles.md](design-principles.md) - DRY, YAGNI, refactoring patterns

## Related Files

-   `phpcs.xml.dist` - PHP_CodeSniffer config
-   `phpstan.neon` - PHPStan config