refactor(skill): flatten quick-dev-new-preview step files

[alexeyv]

Summary

• Move 5 step files from steps/ subdirectory to the skill root directory
• Update path reference in workflow.md (./steps/step-01-... → ./step-01-...)
• Update template reference in step-02-plan.md (../tech-spec-template.md → ./tech-spec-template.md)

Test plan

• npm run quality passes (all checks green)
• validate:refs confirms zero broken references
• Invoke /bmad-quick-dev-new-preview and verify step loading works

🤖 Generated with Claude Code

[augmentcode]

🤖 Augment PR Summary

Summary: Flattens the bmad-quick-dev-new-preview workflow layout by moving step markdown files from steps/ into the skill root.
Changes: Updates intra-skill references in workflow.md and step-02-plan.md to point at the new relative paths.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

[coderabbitai]

📝 Walkthrough

Walkthrough

The changes update relative file path references in two workflow files within the bmad-quick-dev-new-preview directory. The tech-spec template path is changed from a parent directory reference to the current directory, and a step file reference has the "steps/" directory prefix removed.

Changes

Cohort / File(s)	Summary
Path Reference Updates src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-02-plan.md, src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/workflow.md	Updated relative file path references: tech-spec template path changed from ../tech-spec-template.md to ./tech-spec-template.md, and step reference changed from ./steps/step-01-clarify-and-route.md to ./step-01-clarify-and-route.md.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• BMAD-METHOD#1185: Both PRs modify file-path references for the tech-spec template and workflow/step files in the same bmad-quick-flow directory.
• BMAD-METHOD#1997: Both PRs adjust path references and step link locations within bmad-quick-dev-new-preview files by removing or changing directory prefixes.

Suggested reviewers

• bmadcode

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'refactor(skill): flatten quick-dev-new-preview step files' clearly and concisely summarizes the main change: flattening the step files structure by moving them from a subdirectory to the root directory.
Description check	✅ Passed	The description is directly related to the changeset, providing clear details about moving step files, updating path references, and documenting test plans with specific outcomes.
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
• Commit unit tests in branch flatten-quick-dev-steps

📝 Coding Plan

• Generate coding plan for human review comments

---

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

Tip

You can disable the changed files summary in the walkthrough.

Disable the reviews.changed_files_summary setting to disable the changed files summary in the walkthrough.

[coderabbitai]

🧹 Nitpick comments (1)

src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-02-plan.md (1)

16-16: Add an explicit missing-template HALT path

Line 16 assumes ./tech-spec-template.md exists. If flattening is partial or future edits drift, this step will fail ambiguously. Add an explicit “if missing, HALT and ask human” guard in this instruction block.

Suggested edit

-2. Read `./tech-spec-template.md` fully. Fill it out based on the intent and investigation, and write the result to `{wipFile}`.
+2. Verify `./tech-spec-template.md` exists; if missing, HALT and ask the human to repair workflow assets. Then read it fully, fill it out based on intent and investigation, and write the result to `{wipFile}`.

Based on learnings: for this specific skill format, ./ resolution is skill-root-relative and intentional, so guarding existence is the safer improvement.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-02-plan.md`
at line 16, Update the instruction in the step that reads "Read
`./tech-spec-template.md` fully. Fill it out ... write the result to
`{wipFile}`" to include an explicit existence check for
`./tech-spec-template.md` and a clear HALT path: if the template file is
missing, stop the workflow, surface a human-facing error asking the user to
place or restore `./tech-spec-template.md` in the skill root, and do not proceed
to write `{wipFile}`; reference the same `./tech-spec-template.md` and
`{wipFile}` symbols so the guard is unambiguous.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-02-plan.md`:
- Line 16: Update the instruction in the step that reads "Read
`./tech-spec-template.md` fully. Fill it out ... write the result to
`{wipFile}`" to include an explicit existence check for
`./tech-spec-template.md` and a clear HALT path: if the template file is
missing, stop the workflow, surface a human-facing error asking the user to
place or restore `./tech-spec-template.md` in the skill root, and do not proceed
to write `{wipFile}`; reference the same `./tech-spec-template.md` and
`{wipFile}` symbols so the guard is unambiguous.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5203dd3c-24ba-4db8-93e0-65cb68d2cff8

📥 Commits

Reviewing files that changed from the base of the PR and between aa2a9a2 and cad2581.

📒 Files selected for processing (6)

• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-01-clarify-and-route.md
• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-02-plan.md
• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-03-implement.md
• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-04-review.md
• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/step-05-present.md
• src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/workflow.md