Dr Review

DR Review

Resolve open Decision Requests.

Arguments

If $ARGUMENTS is provided, parse for:

• DR-NNN → review that specific DR directly
• --quick → use recommendations with minimal prompts

If no argument, list open DRs and prompt for selection.

Usage

/dr-review              # List open DRs, select one
/dr-review DR-001       # Review specific DR
/dr-review --quick      # Use recommendations, minimal prompts

Behavior

1. List Open DRs

Scan .sdlc/decisions/open/DR-*.md and present sorted by priority (Blocking > High > Medium > Low):

Found N open Decision Requests:

| Priority | DR | Question |
|----------|-----|----------|
| Blocking | DR-004 | AAP Pre-Flight Integration |
...

Which DR to review? (number or "next" for highest priority)

Empty state: "No open Decision Requests. All caught up!"

If >10 DRs: Show Blocking/High only, summarize: "...plus N medium/low DRs"

2. Present DR Summary

Read the selected DR file and present:

• Question: Core decision needed
• Context: Why it matters (2-3 sentences max)
• Options: Each with pros/cons
• Recommendation: Highlight if present

If no recommendation: Note "No recommendation provided" and present options neutrally.

3. Facilitate Decision

What is your decision?
1. [Option A description]
2. [Option B description]
...
D. Defer for later
M. Need more information

On option selected:

• Ask for rationale (offer recommendation text if available)
• Ask for action items (optional)

On defer:

• Ask when to revisit
• Ask for deferral reason

On "need more info":

• Ask what information is needed
• Add to DR as open question
• Keep in open/

4. Architectural Impact Check

Before recording the decision, evaluate the chosen option against the project's architectural invariants (defined in AGENTS.md).

Read the Architectural Invariants section of AGENTS.md and check the chosen option against all invariants:

Checking architectural impact of chosen option...

| Invariant | Status |
|-----------|--------|
| Validators read-only (ADR-009) | OK / ⚠ CONFLICT |
| gRPC between backend services (ADR-001) | OK / ⚠ CONFLICT |
| Async + executor discipline (ADR-007) | OK / ⚠ CONFLICT |
| Unified Validator contract | OK / ⚠ CONFLICT |
| Stateless engine / edge persistence (ADR-020) | OK / ⚠ CONFLICT |
| Scale pods, not services (ADR-012) | OK / ⚠ CONFLICT |
| Session venvs are Engine-owned (ADR-022) | OK / ⚠ CONFLICT |
| Rule ID conventions (ADR-008) | OK / ⚠ CONFLICT |
| OPA subprocess, not REST | OK / ⚠ CONFLICT |
| FixSession unified path (ADR-039) | OK / ⚠ CONFLICT |
| Engine never queries out (ADR-020) | OK / ⚠ CONFLICT |
| Built-in bundles closed (ADR-042) | OK / ⚠ CONFLICT |

Dependency direction: engine → gateway → UI preserved? [Y/N]
Engine remains caller-agnostic? [Y/N]

If conflict detected:

⚠ This decision conflicts with invariant(s): [list]

Options:
1. Redesign — modify the decision to respect the architecture
2. ADR Override — accept the conflict but require a new ADR that
   explicitly supersedes the violated invariant(s) with rationale
3. Reject — choose a different option

Architecture cannot be violated silently. Which approach?

If the user chooses "ADR Override": The DR is decided, but the decision is blocked from implementation until the corresponding ADR is accepted. Add this to the DR's Action Items.

If no conflict: Proceed to recording.

5. Record Decision

Update the DR file:

• Status: → Decided / Deferred
• Decision: → chosen option
• Rationale: → user's rationale
• Action Items: → if any
• Decided: → today's date

Preserve existing file format — only update relevant sections.

6. Move to Closed

.sdlc/decisions/
├── open/           # Remove from here
└── closed/
    ├── decided/    # Most decisions
    ├── deferred/   # Revisit later
    └── superseded/ # Replaced by another DR

7. Update README Index

Edit .sdlc/decisions/README.md:

• Remove from "Open" table
• Add to appropriate "Closed" table with decision summary

8. Offer ADR (If Architectural)

If decision affects architecture, patterns, or technology choices:

This decision affects architecture. Create an ADR? (Y/n)

If yes: Copy template to .sdlc/adrs/ADR-NNN-title.md, pre-fill from DR, set status "Proposed".

Use next available ADR number (scan existing ADRs).

9. Summary & Continue

Done! DR-001 decided: [brief summary]
- Moved to closed/decided/
- README updated
- ADR-014 created (if applicable)

Action items:
- [ ] [any recorded items]

Review another DR? (Y/n or DR number)

If "Y" or number: Loop back to step 2/1 If "n": Show remaining count: "N open DRs remaining (X blocking)"

Quick Mode

With --quick flag or when user says "just use the recommendation":

1. Show DR question and recommended option
2. Ask for confirmation only
3. Use recommendation text as rationale
4. Skip action items prompt
5. Skip ADR prompt (can always create later)

Edge Cases