aibot88 Perform a full code review of changed files, validating against docs/ARCHITECTURE.md.
Scope: $ARGUMENTS (if empty, review files changed in git)
Steps
- Identify changed files:
git diff --name-only HEAD~1 2>/dev/null || git diff --name-only --cached 2>/dev/null || echo "Please specify the files"
- Run automated checks:
npx tsc --noEmit
ng lint
ng test --watch=false
Check ARCHITECTURE.md patterns:
- Services: HttpClient only, no try/catch, no transformation, inject(HttpClient)
- Adapters: pure functions
- Stores: signal-based, private WritableSignal, public asReadonly()
- Components: standalone, input()/output() signals, OnPush, < 200 lines
- Naming: kebab-case files, PascalCase classes
- Boundaries: no cross-module imports
- DI: inject() everywhere, no constructor DI
Classify:
- VIOLATION - breaks ARCHITECTURE.md
- ATTENTION - recommended improvement
- COMPLIANT - correct
- HIGHLIGHT - positive highlight
Produce report with verdict: Approved | With caveats | Requires changes