Architecture Review Skill
Purpose
Ensure system designs are sound, scalable, and maintainable.
Review Dimensions
| Dimension | Questions |
|---|---|
| Scalability | How does it handle 10x growth? |
| Reliability | What happens when components fail? |
| Maintainability | Can a new engineer understand it? |
| Performance | Are latency/throughput targets met? |
| Security | What are the attack vectors? |
| Cost | What's the $/request at scale? |
| Operability | Can we debug it at 3 AM? |
Review Process
- Read: Understand requirements and constraints
- Diagram: Sketch data flow and component interactions
- Question: Identify assumptions and risks
- Compare: Evaluate alternatives
- Decide: Document recommendation
ADR Template
## ADR-XXX: [Decision Title]
### Context
[What problem are we solving?]
### Decision
[What are we doing?]
### Consequences
- Positive: [...]
- Negative: [...]
- Risks: [...]
### Alternatives Considered
| Option | Pros | Cons |
|--------|------|------|
| A | [...] | [...] |
| B | [...] | [...] |
### Compliance
[How we'll verify this decision was correct]
CF-Specific
- Review against
.cursor/ANTI_PATTERNS.md - Check operator-loop compatibility (tags, harvest)
- Verify LiteLLM routing for all LLM calls
- Ensure pipeline stages are independently replaceable