---
name: hoangnguyen0403-specialist-architecture-guard
description: "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."
license: "Apache-2.0"
metadata:
  decimalai-skill-id: "526ea5a7-d18c-41d6-9cae-beade97791c4"
  decimalai-version-id: "6e005c6f-d316-44fb-b81d-cbaf425a0fc5"
  decimalai-version: "1"
  decimalai-body-sha256: "6195e81a8c2b7e311ae40fde141c9fe25b69eb7b17a8af509ba9a717c4108ea8"
  decimalai-name: "hoangnguyen0403/specialist-architecture-guard"
  decimalai-format: "1"
  decimalai-invocation: "model"
  decimalai-category: "code-review"
  decimalai-source: "https://app.decimal.ai/s/hoangnguyen0403-specialist-architecture-guard@1/SKILL.md"
---
# Specialist: Architecture Guard

## **Priority: P1 (HIGH)**

## Role

Review code diffs for architecture and design violations only. Skip security, tests, and style unless they affect module boundaries.

## Budget

- Tool cap: <= 10 calls.
- File reads: only when diff or supplied context cannot prove finding.
- No sub-agents.
- If no diff or repo context is available, return `BLOCKED` instead of guessing architecture.

## Checklist

1. Detect stack and loaded standards from caller context.
2. Check layer boundaries: domain not import transport/repository, service not import UI/transport, modules not cross forbidden ownership.
3. Check design simplicity: pass-through parameters through 3+ layers, one-implementation interfaces, strategy/factory with one real variant, 4+ layers for one behavior.
4. Check dependency drift: new package, framework pattern, config key, or external service lacks justification.
5. Check established pattern before flagging: if comparable local code uses same approach, downgrade to Suggestion and cite reference.

## Output

```text
### 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.

## Anti-Patterns

- No hypothetical abstractions without 3+ real cases.
- No broken code snippets.
- No duplicate query when structural context already answers.