architecture-review

Featured

Reviews a technical proposal before implementation. Use for specs, designs, RFCs, ADRs, architecture proposals, and issues that specify how a system change should work. Finds material ambiguity and flaws in correctness, scalability, performance, security, operations, and proof.

Code & Development 390 stars 68 forks Updated 4 days ago MIT

Install

View on GitHub

Quality Score: 93/100

Stars 20%
86
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
50
License 10%
100
Description 5%
100

Skill Content

# Architecture Review Review a technical proposal before implementation. It may describe a large system change or a small implementation-level design. Review the proposed choices, not the document type or size. Use one fresh subagent that did not write the proposal. Give it the complete review context and tell it to review directly without delegating. If you are that reviewer, review directly. Stay read-only. ## Workflow 1. Read the goal, proposal, repository instructions, relevant current code, tests, schemas, configuration, and linked material. Treat claims about the current system as unverified until code, tests, schemas, configuration, infrastructure, or relevant runtime evidence supports them. 2. State the problem, affected user, intended outcome, success measure, scope, constraints, and main tradeoff. Report any that the proposal leaves unclear. 3. Trace one real case from input to observable outcome. Include ownership, validation, state changes, side effects, response timing, failure, retry, cleanup, and what the user sees where they matter. 4. Challenge the chosen design with the review lenses below. Look for a simpler choice that reaches the same outcome and proof with less state, coupling, duplication, or operational work. 5. Surface material open questions. Recommend an answer when evidence supports one. Do not invent questions that cannot change the design. 6. Return findings, open questions, a short assessment, and one verdict. Do n...

Details

Author
owainlewis
Repository
owainlewis/blueprint
Created
6 months ago
Last Updated
4 days ago
Language
Python
License
MIT

Similar Skills

Semantically similar based on skill content — not just same category

Code & Development Listed

architecture-review

Review a proposed architecture/design before execution planning. Use for cross-component, hard-to-reverse, data, security, performance, migration, or long-lived design decisions.

3 Updated today
xiongxianfei
Code & Development Listed

review-arch

Conduct a principal-level architecture review of a system, specification, diagram, or set of design decisions. Use when asked to review an architecture, evaluate a design, assess coupling/cohesion, check for anti-patterns, audit system boundaries, or produce an Architecture Review Board (ARB) style verdict. Also use when someone says 'review this architecture', 'is this design sound', 'what's wrong with this topology', 'evaluate these tradeoffs', or 'audit this system'. Produces a structured verdict organized around critical risks, significant concerns, observations, strengths, and open questions, grounded in evidence from the codebase and cited against established methodology (ATAM, ISO/IEC 25010, documented anti-pattern catalogues). Do NOT use for line-by-line code review, spec-vs-implementation verification, or PR-level correctness checks. Name the system, spec, or design to be reviewed in the arguments; the skill does not infer the target from the conversation.

5 Updated today
sergeyklay
Code & Development Listed

reviewing-architecture

Reviews system/module architecture, module boundaries, dependency direction, and technical debt/scalability risk, and drafts ADRs — for existing systems or a proposed design. Advisory only; never implements or refactors. Use when asked to review architecture, draft an ADR, analyze technical debt, or decide how to modularize something.

0 Updated 4 days ago
JavierQuinan