Install
$ agentstack add skill-genkovich-sdd-decide-adr ✓ scanned · ✓ verified — works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
About
Skill: decide-adr
The post-hoc / asynchronous ADR path (pipeline stage 8a). design spawns ADRs synchronously and Accepted while you walk it Socratically; decide-adr records a decision that missed that pass — already in code, agreed in a chat, sketched on a whiteboard, or flagged by a tasks/review gate as a contract with no ADR behind it. It can also run a Proposed → Accepted review flow when the decision still needs a reviewer's sign-off. One file = one decision; it reuses design's MADR template, so there is no second ADR format here.
It is a recording utility, not a Socratic design stage — it does not run the shared Socratic loop or critic. The two shared dependencies are question phrasing and the worthiness gate: → [../_shared/ask-style.md](../_shared/ask-style.md) · [../design/references/blast-radius.md](../design/references/blast-radius.md)
Owner
Decision author (usually the Architect or Tech Lead). A reviewer (Tech Lead, plus Security when relevant) signs off the Proposed → Accepted transition.
Inputs
- `` — the feature slug, same as every earlier stage.
- `
— kebab-case, describes the **decision**, not the problem (time-sortable-ids, notid-strategy`). - The decision itself + its alternatives — pulled from
sad.md§4 Solution strategy / §9 ADR index / §11 Risks, or supplied by the user. - Input gate (soft). Expects
docs/features//to exist, ideally withsad.md(decide-adr reads its §4/§9/§11 for context and drivers). If the decision is genuinely standalone — no feature folder yet — allow it, but note the missing design context in the ADR's Context section and warn the user, rather than refusing.
Protocol
- Locate the feature.
test -d docs/features/. Missing → ask whether to proceed standalone (AskUserQuestion, phrasing per [../_shared/ask-style.md](../_shared/ask-style.md)). On «yes», createdocs/features//adr/and flag that design context is absent.sad.mdpresent → read its §4 (strategy), §9 (existing ADR index), §11 (risks); absent → note it and source context from the user. - Worthiness check. Score the decision against the blast-radius gate — [
../design/references/blast-radius.md](../design/references/blast-radius.md) (irreversible / multi-module / has legitimate alternatives). 2-of-3 → proceed. Below the bar → tell the user it's likely inline-in-sad.mdmaterial, not an ADR, and confirm before writing one anyway. - Dedup.
ls docs/features//adr/*.md 2>/dev/null. An Accepted ADR on the same topic exists → don't duplicate: propose either editing it or a new ADR that marks the old oneSuperseded by NNNN(and stamps the old one'sstatus+updated_at). - Pick the number.
NNNN= (count of existingadr/*.md) + 1, zero-padded to 4 digits (0001,0002, …). Never reuse a number. - Copy the template. Copy [
../design/templates/adr.md](../design/templates/adr.md) →docs/features//adr/NNNN-.md. This is the canonical MADR shape, owned bydesignand referenced here — do not invent a variant. Patch frontmatter:owner,updated_at:,feature_size(from.sizeif present),ticket. - Context. 2–4 sentences: what forced this decision (an NFR, an incident, a constraint), and — if
sad.mdis absent — an explicit note that there is no design document, so the context is reconstructed from the author. - Decision drivers. Bullets — the quality goals / constraints that pushed the choice, each traceable to a real source (spec §6 NFR, sad.md §2 Constraints, a §1 top-3 quality goal). Don't invent drivers; they filter out pet decisions.
- Considered options. List all serious options (≥2 — one option is a declaration, not a decision), one line each with its trade-off. No strawman (an option an existing constraint already rules out).
- Decision outcome. «Chosen: » + 1–2 sentences on why it won, citing the drivers above.
- Consequences. Positive and Negative and Neutral — cons included, or it's rationalisation, not a record. Name what changes in the codebase, ops, monitoring, onboarding. `` only where a number honestly needs a spike.
- Status.
Proposedwhile a reviewer still has to sign off;Acceptedonce final. Run the review flow when needed: writeProposed, fillreviewers, and on sign-off flip toAcceptedand bumpupdated_at. A reader six months on must be able to tell a live plan from a settled fact. - Close the loop. Add a row to
sad.md§9 ADR index (and link fromtasks/_epic.mdif the ADR scopes a specific task). The ADR's own## Linksmust point up to the spec + the relevantsad.md§N — no orphans. - Propose commit + handoff.
adr: NNNN. Then emit the stage-handoff block per [../_shared/handoff.md](../_shared/handoff.md) (utility variant) — What I did + Review (adr/NNNN-.md) + Run next: resume the gate that needed it (/sdd:tasksor/sdd:plan-tests);/clearoptional.
Definition of Done
docs/features//adr/NNNN-.mdexists in design's MADR format (frontmatter + Context + Decision drivers + Considered options + Decision outcome + Consequences + Links).NNNNis correct (existing count + 1, 4-digit) and the title is in decision-form (0007-time-sortable-ids.md✓ vs0007-id-strategy.md✗).statusis explicit —Accepted(final) orProposed(reviewer pending,reviewersfilled).- ≥2 considered options, no strawman; Consequences carry real Negatives, not only Positives.
- Linked both ways: a row in
sad.md§9 (whensad.mdexists) and the ADR's## Linkspoint up to spec + sad §N. A genuinely standalone ADR notes the missing design context instead. - Dedup ran; a same-topic prior decision is
Superseded by NNNN, never silently duplicated.
Anti-patterns
- Using this for a live decision. A choice made now with the user belongs in
design(spawned inline,Accepted);decide-adris for what missed that pass. - ADR without options. «We chose X» with no alternatives is a declaration. List ≥2 serious options; no strawman.
- ADR as a changelog. «Tried it, didn't work» is a news feed, not a decision record.
- ADR as a spec. Acceptance criteria / NFRs don't live here — an ADR is trade-off and reasoning. That detail is in
spec.md/data-model/api. - No status. A reader six months on can't tell a current plan from an old decision.
- Problem-form title (
0007-id-strategy.md) — in the §9 index it's unclear which decision exists. Use the decision (0007-time-sortable-ids.md). - All-positive consequences — an honest ADR names its Negatives and Neutrals too.
- Reinventing the template. Reuse [
../design/templates/adr.md](../design/templates/adr.md); a second ADR format fragments the genre. - Orphan ADR. Written but not in §9 and with no
## Linksup to spec/sad — six-month archaeology can't find it.
References & template
- [
../design/templates/adr.md](../design/templates/adr.md) — the canonical MADR scaffold this skill copies and fills (owned bydesign; do not duplicate it here). - [
../design/references/blast-radius.md](../design/references/blast-radius.md) — the 3-criteria worthiness gate (irreversible / multi-module / legitimate alternatives); same gatedesignruns, applied here to confirm the decision earns an ADR. - [
../_shared/ask-style.md](../_shared/ask-style.md) — phrasing for the standalone-confirm, worthiness-borderline, and Superseded questions.
Example invocation
> User: «ADR for time-sortable-ids on checkout-discounts — tasks flagged a missing ADR for the id-generation choice» > Skill: docs/features/checkout-discounts/ exists with sad.md → reads §4/§9/§11. Blast-radius: id strategy is irreversible (a switch later needs a backfill across every row) + multi-module (other modules read the ids) → 2-of-3, proceed. ls adr/ → 0001, 0002 exist → NNNN = 0003. Title 0003-time-sortable-ids.md (decision-form). Copies ../design/templates/adr.md. Context: the §11 risk on hot-row contention forced an explicit id choice. Drivers: spec §6 NFR (predictable ordering) + the existing capability in the stack. Options: (a) time-sortable ids generated in the app; (b) database auto-increment; (c) random ids. Outcome: «Chosen: (a)» — keeps natural ordering without a central sequence; (b) couples to one writer, (c) loses ordering. Consequences: + ordered without coordination; − slightly larger ids than auto-increment; neutral: switching later needs a backfill. Status Accepted. Adds a §9 row, fills ## Links. Commit adr: checkout-discounts 0003 time-sortable-ids.
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: genkovich
- Source: genkovich/sdd
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet — be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.