skills/specialists/specialist-architecture-guard/SKILL.md
Audits PR diffs for architecture boundary violations, design simplicity, dependency drift, and established-pattern mismatches. Use during code review when architecture, layering, or framework conventions may be affected.
npx skillsauth add hoangnguyen0403/agent-skills-standard specialist-architecture-guardInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Review code diffs for architecture and design violations only. Skip security, tests, and style unless they affect module boundaries.
BLOCKED instead of guessing architecture.### Architecture Review Findings
**Tech Stack:** [stack]
**Layers Touched:** [layers]
#### Violations
- [Severity] [file:line] - [issue + fix]
#### Design Simplicity
- [Severity] [file:line] - [issue + fix]
#### What Looks Good
- [observation]
Severity: Blocker, Major, Minor, Suggestion.
testing
Infer the requesting operator's technical fluency from message content (never ask directly) and adapt register — business, hybrid, or technical — across SDLC workflow output. Use when starting sdlc, brainstorm-feature, plan-feature, verify-work, publish-notes, or session-report, or whenever a request's phrasing signals a non-technical or cross-stack operator.
documentation
Define transaction boundaries, locking, and consistency guarantees for multi-step writes. Use when designing atomic operations, retries, idempotency, or concurrent write behavior.
development
Design relational or document schemas from access patterns, cardinality, and lifecycle. Use when modeling entities, choosing embed vs normalize, or shaping schema boundaries before implementation.
data-ai
Diagnose database latency with explain plans, index ownership, and query-shape review. Use when a query is slow, an index is missing, or scans and N+1 patterns appear.