Architecture decision review
Skill yangshu2087/Codex/.agents/skills/architecture-decision-review
Meta workspace for Codex configuration, shared skills, and maintenance scripts.
npx -y skills add yangshu2087/Codex --skill architecture-decision-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
2 things to look at
- no licenseNo license file was found in the repository. Code published without one is not open source by default, so using it at work is a question for whoever answers licensing questions where you are.
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Use when a task involves architecture decisions, migrations, rollout planning, system boundaries, technical tradeoffs, or rollback risk.
SKILL.md
2.1 KB, as published. Nobody here has run it
Architecture Decision Review
Use this skill when the cost of a wrong design decision is higher than the cost of planning first.
Required inputs
- Current repository or service boundary
- Existing architecture facts and constraints
- Target behavior or decision to make
- Compatibility, migration, and operational limits
- Known failure modes, observability needs, and rollback requirements
Workflow
- Write the task card: Goal, Constraints, Non-goals, Done criteria, Verification commands.
- Compare at least two viable options unless only one is technically viable.
- For each option, evaluate correctness, compatibility, complexity, migration cost, operational risk, observability, and rollback path.
- Call out API, auth/permission, data consistency, deployment, queue, or external-service impact when any of those boundaries are touched.
- Recommend one option and explain why it best fits the constraints.
- Propose implementation milestones in lowest-risk order; prefer reversible checkpoints over broad rewrites.
- For changes that touch data, auth, queues, external APIs, or deployment, include rollout and rollback notes before implementation.
Evidence requirements
- Option comparison and selected approach
- Tradeoffs and rejected alternatives
- Rollout or migration path
- Rollback path or explicit reason rollback is not applicable
- Observability / operational risk notes
- Verification plan with dry-run, tests, smoke, or targeted regression checks
Output standard
- Task card
- Option comparison table
- Recommendation
- Migration / rollout plan
- Rollback plan
- API/data/auth impact
- Observability and risk notes
- Verification plan
- Residual risks
Common mistakes
- Treating a refactor as architecture without stating migration cost.
- Selecting an option without rollback or observability.
- Coding first and justifying the design afterward.
- Calling a design “done” when it only describes the happy path.