fix: add missing review menu and HALT to quick-spec step 4

[alexeyv]

Summary

• Adds structured review menu with explicit HALT to quick-spec step 4
• Replaces confusing open-ended question with clear options
• Aligns step 4 with the menu-driven pattern used in steps 1-3
• Bumps prettier from ^3.5.3 to ^3.7.4 to match lockfile (fixes CI mismatch)

Fixes #1304

Test plan

• Run quick-spec workflow through step 4
• Verify menu is displayed after spec presentation
• Verify workflow halts and waits for user selection

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR modifies a workflow step to replace a simple yes/no question with an interactive multi-option review menu featuring Approve, Changes, Questions, Advanced Elicitation, and Party Mode options, with loop logic returning to the menu after actions. A blank line is also removed from a configuration file for formatting.

Changes

Cohort / File(s)	Summary
Workflow Step Review Menu src/modules/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md	Replaced simple closing question with interactive multi-option review menu (Approve, Changes, Questions, Advanced Elicitation, Party Mode) and menu handling logic with loop flow that returns to menu after actions, addressing ambiguous prompt wording from issue #1304
Configuration Formatting src/modules/cis/module.yaml	Removed blank line before "Variables from Core Config inserted" block; no semantic or configuration changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• refactor: rename create-tech-spec to quick-spec for QuickFlow branding #1290: Also modifies the same workflow step file (step-04-review.md) but updates workflow path/name references rather than review control flow logic

Suggested reviewers

• bmadcode
• pbean

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding a review menu and HALT instruction to quick-spec step 4, which is the primary focus of the changeset.
Linked Issues check	✅ Passed	The PR successfully addresses issue #1304 by replacing the confusing mixed yes/no question with a structured review menu offering clear options (Approve, Changes, Questions, Advanced Elicitation, Party Mode) and explicit HALT instruction.
Out of Scope Changes check	✅ Passed	The changes are focused on the primary objective: adding a review menu and HALT to step 4. A minor formatting fix in module.yaml (removing a blank line) is a pre-existing issue mentioned in the commit message, not a significant out-of-scope change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The PR description clearly relates to the changeset, describing the addition of a structured review menu to step 4 and mentioning alignment with the menu-driven pattern.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 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.}