fix(quick-spec): reorder final menu to D-B-A-P-R

[alexeyv]

Summary

Reorders the final menu in quick-spec step-04-review to group related options:

Before: [A] [R] [B] [D] [P] — scattered, no logic

After: [D] [B] [A] [P] [R] — grouped by purpose:

• "Go-on" options first: Done, Begin Development
• "Reasoning" options last: Advanced Elicitation, Party Mode, Adversarial Review

Also adds "again" to Adversarial Review option for clarity.

Follows established pattern: most common action first, related options grouped together.

Test plan

• Complete a quick-spec workflow to the final menu
• Verify menu displays in D-B-A-P-R order
• Verify R option says "Adversarial Review again"

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

A single documentation file for a workflow step is reordered to adjust the finalization menu options, moving Begin Development before Advanced Elicitation, adding Party Mode and Adversarial Review labels, and removing the Done exit option from this menu stage.

Changes

Cohort / File(s)	Summary
Workflow Menu Reordering src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md	Reordered finalization menu options: [B] Begin Development now precedes [A] Advanced Elicitation; [P] Party Mode and [R] Adversarial Review again added; [D] Done option removed. Control flow lines for B and A swapped to reflect new sequence; D-handling branch eliminated.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• refactor: convert create-tech-spec to sharded workflow with orient-first pattern #1185: Modifies the same final review step with adjustments to menu and adversarial-review behavior
• fix: add missing review menu and HALT to quick-spec step 4 #1305: Modifies the same review menu and control-flow logic in step-04-review.md
• refactor: rename create-tech-spec to quick-spec for QuickFlow branding #1290: Modifies the same file with workflow path and rename changes

Suggested reviewers

• bmadcode

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: reordering the final menu in quick-spec from the previous order to D-B-A-P-R sequence.
Description check	✅ Passed	The description is directly related to the changeset, clearly explaining the menu reordering, the rationale for grouping options by purpose, and providing a test plan.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}