From 79959e75acd23e0f744093afc36e5b3accfeeaa1 Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Sat, 24 Jan 2026 19:15:01 -0800 Subject: [PATCH 1/2] fix(quick-flow): standardize menu shortcuts for intuitive UX (#1401) * fix(quick-spec): change menu shortcuts to avoid Approve/Advanced confusion Users were typing 'a' expecting to Approve (since it starts with A) but triggering Advanced Elicitation instead. Changed shortcuts to: - [C] Continue (was [Y] Approve) - [E] Edit (was [C] Changes) This keeps [A] for Advanced Elicitation consistent with other workflows. Fixes user-reported UX issue with confusing menu shortcuts. Co-Authored-By: Claude Opus 4.5 * fix(quick-dev): standardize menu shortcuts to use intuitive letters - Change [T] to [P] for "Plan first" (P matches the label) - Change [1][2][3] to [W][F][S] for findings resolution: - [W] Walk through - [F] Fix automatically - [S] Skip Consistent with letter-based menu pattern used elsewhere. Co-Authored-By: Claude Opus 4.5 --------- Co-authored-by: Claude Opus 4.5 --- .../quick-dev/steps/step-01-mode-detection.md | 14 +++++++------- .../steps/step-06-resolve-findings.md | 18 +++++++++--------- .../quick-spec/steps/step-04-review.md | 8 ++++---- 3 files changed, 20 insertions(+), 20 deletions(-) diff --git a/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-01-mode-detection.md b/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-01-mode-detection.md index dc1b955a..4ea630b1 100644 --- a/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-01-mode-detection.md +++ b/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-01-mode-detection.md @@ -88,11 +88,11 @@ Use holistic judgment, not mechanical keyword matching. ### No Escalation (simple request) -Display: "**Select:** [T] Plan first (tech-spec) [E] Execute directly" +Display: "**Select:** [P] Plan first (tech-spec) [E] Execute directly" #### Menu Handling Logic: -- IF T: Direct user to `{quick_spec_workflow}`. **EXIT Quick Dev.** +- IF P: Direct user to `{quick_spec_workflow}`. **EXIT Quick Dev.** - IF E: Ask for any additional guidance, then **NEXT:** Read fully and follow: `step-02-context-gathering.md` #### EXECUTION RULES: @@ -108,13 +108,13 @@ Present: "This looks like a focused feature with multiple components." Display: -**[T] Create tech-spec first** (recommended) +**[P] Plan first (tech-spec)** (recommended) **[W] Seems bigger than quick-dev** - Recommend the Full BMad Flow PRD Process **[E] Execute directly** #### Menu Handling Logic: -- IF T: Direct to `{quick_spec_workflow}`. **EXIT Quick Dev.** +- IF P: Direct to `{quick_spec_workflow}`. **EXIT Quick Dev.** - IF W: Direct user to run the PRD workflow instead. **EXIT Quick Dev.** - IF E: Ask for guidance, then **NEXT:** Read fully and follow: `step-02-context-gathering.md` @@ -132,12 +132,12 @@ Present: "This sounds like platform/system work." Display: **[W] Start BMad Method** (recommended) -**[T] Create tech-spec** (lighter planning) +**[P] Plan first (tech-spec)** (lighter planning) **[E] Execute directly** - feeling lucky #### Menu Handling Logic: -- IF T: Direct to `{quick_spec_workflow}`. **EXIT Quick Dev.** +- IF P: Direct to `{quick_spec_workflow}`. **EXIT Quick Dev.** - IF W: Direct user to run the PRD workflow instead. **EXIT Quick Dev.** - IF E: Ask for guidance, then **NEXT:** Read fully and follow: `step-02-context-gathering.md` @@ -154,7 +154,7 @@ Display: - Mode A (tech-spec): "**NEXT:** read fully and follow: `step-03-execute.md`" - Mode B (direct, [E] selected): "**NEXT:** Read fully and follow: `step-02-context-gathering.md`" -- Escalation ([T] or [W]): "**EXITING Quick Dev.** Follow the directed workflow." +- Escalation ([P] or [W]): "**EXITING Quick Dev.** Follow the directed workflow." --- diff --git a/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-06-resolve-findings.md b/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-06-resolve-findings.md index 7f9457fa..4ab367c6 100644 --- a/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-06-resolve-findings.md +++ b/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-06-resolve-findings.md @@ -29,15 +29,15 @@ Present: "How would you like to handle these findings?" Display: -**[1] Walk through** - Discuss each finding individually -**[2] Auto-fix** - Automatically fix issues classified as "real" -**[3] Skip** - Acknowledge and proceed to commit +**[W] Walk through** - Discuss each finding individually +**[F] Fix automatically** - Automatically fix issues classified as "real" +**[S] Skip** - Acknowledge and proceed to commit ### Menu Handling Logic: -- IF 1: Execute OPTION 1 (Walk Through) below -- IF 2: Execute OPTION 2 (Auto-fix) below -- IF 3: Execute OPTION 3 (Skip) below +- IF W: Execute WALK THROUGH section below +- IF F: Execute FIX AUTOMATICALLY section below +- IF S: Execute SKIP section below ### EXECUTION RULES: @@ -46,7 +46,7 @@ Display: --- -## OPTION 1: WALK THROUGH +## WALK THROUGH [W] For each finding in order: @@ -61,7 +61,7 @@ After all findings processed, summarize what was fixed/skipped. --- -## OPTION 2: AUTO-FIX +## FIX AUTOMATICALLY [F] 1. Filter findings to only those classified as "real" 2. Apply fixes for each real finding @@ -78,7 +78,7 @@ Skipped (noise/uncertain): F2, F4 --- -## OPTION 3: SKIP +## SKIP [S] 1. Acknowledge all findings were reviewed 2. Note that user chose to proceed without fixes diff --git a/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md b/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md index 848df729..6f234612 100644 --- a/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md +++ b/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md @@ -43,14 +43,14 @@ wipFile: '{implementation_artifacts}/tech-spec-wip.md' **Present review menu:** -Display: "**Select:** [Y] Approve [C] Changes [Q] Questions [A] Advanced Elicitation [P] Party Mode" +Display: "**Select:** [C] Continue [E] Edit [Q] Questions [A] Advanced Elicitation [P] Party Mode" **HALT and wait for user selection.** #### Menu Handling Logic: -- IF Y: Proceed to Section 3 (Finalize the Spec) -- IF C: Proceed to Section 2 (Handle Review Feedback), then return here and redisplay menu +- IF C: Proceed to Section 3 (Finalize the Spec) +- IF E: Proceed to Section 2 (Handle Review Feedback), then return here and redisplay menu - IF Q: Answer questions, then redisplay this menu - IF A: Read fully and follow: `{advanced_elicitation}` with current spec content, process enhanced insights, ask user "Accept improvements? (y/n)", if yes update spec then redisplay menu, if no keep original then redisplay menu - IF P: Read fully and follow: `{party_mode_exec}` with current spec content, process collaborative insights, ask user "Accept changes? (y/n)", if yes update spec then redisplay menu, if no keep original then redisplay menu @@ -59,7 +59,7 @@ Display: "**Select:** [Y] Approve [C] Changes [Q] Questions [A] Advanced Elicita #### EXECUTION RULES: - ALWAYS halt and wait for user input after presenting menu -- ONLY proceed to finalize when user selects 'Y' +- ONLY proceed to finalize when user selects 'C' - After other menu items execution, return to this menu ### 2. Handle Review Feedback From 9b8ce69f37b525a9fe8632198cb7d40ab170f3e9 Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Sat, 24 Jan 2026 21:57:08 -0800 Subject: [PATCH 2/2] fix(quick-spec): reorder final menu to D-B-A-P-R (#1402) Group "go-on" options first (Done, Begin Dev), then reasoning options (Advanced Elicitation, Party Mode, Adversarial Review). Follows established pattern: most common action first, related options grouped. Co-authored-by: Claude Opus 4.5 --- .../bmad-quick-flow/quick-spec/steps/step-04-review.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md b/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md index 6f234612..105c049d 100644 --- a/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md +++ b/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md @@ -115,11 +115,11 @@ Saved to: {finalFile} **Next Steps:** -[A] Advanced Elicitation - refine further -[R] Adversarial Review - critique of the spec (highly recommended) -[B] Begin Development - start implementing now (not recommended) [D] Done - exit workflow +[B] Begin Development - start implementing now (not recommended) +[A] Advanced Elicitation - refine further [P] Party Mode - get expert feedback before dev +[R] Adversarial Review again - critique of the spec (highly recommended) --- @@ -138,9 +138,9 @@ b) **HALT and wait for user selection.** #### Menu Handling Logic: -- IF A: Read fully and follow: `{advanced_elicitation}` with current spec content, process enhanced insights, ask user "Accept improvements? (y/n)", if yes update spec then redisplay menu, if no keep original then redisplay menu -- IF B: Load and execute `{quick_dev_workflow}` with the final spec file (warn: fresh context is better) - IF D: Exit workflow - display final confirmation and path to spec +- IF B: Load and execute `{quick_dev_workflow}` with the final spec file (warn: fresh context is better) +- IF A: Read fully and follow: `{advanced_elicitation}` with current spec content, process enhanced insights, ask user "Accept improvements? (y/n)", if yes update spec then redisplay menu, if no keep original then redisplay menu - IF P: Read fully and follow: `{party_mode_exec}` with current spec content, process collaborative insights, ask user "Accept changes? (y/n)", if yes update spec then redisplay menu, if no keep original then redisplay menu - IF R: Execute Adversarial Review (see below) - IF Any other comments or queries: respond helpfully then redisplay menu