fix(quick-flow): address PR review findings for quick-dev2
- step-04-review: fix copy-paste fallback text to say "perform all three reviews inline sequentially" instead of "implement directly" - workflow.md: add missing planning_artifacts to initialization list, matching quick-spec and quick-dev siblings - quick-flow-solo-dev.agent.yaml: change QD and QQ menu entries from workflow: to exec: for .md files, matching the exec-for-md convention
This commit is contained in:
parent
60c5b12644
commit
f740cd2d17
|
|
@ -24,11 +24,11 @@ agent:
|
||||||
description: "[QS] Quick Spec: Architect a quick but complete technical spec with implementation-ready stories/specs"
|
description: "[QS] Quick Spec: Architect a quick but complete technical spec with implementation-ready stories/specs"
|
||||||
|
|
||||||
- trigger: QD or fuzzy match on quick-dev
|
- trigger: QD or fuzzy match on quick-dev
|
||||||
workflow: "{project-root}/_bmad/bmm/workflows/bmad-quick-flow/quick-dev/workflow.md"
|
exec: "{project-root}/_bmad/bmm/workflows/bmad-quick-flow/quick-dev/workflow.md"
|
||||||
description: "[QD] Quick-flow Develop: Implement a story tech spec end-to-end (Core of Quick Flow)"
|
description: "[QD] Quick-flow Develop: Implement a story tech spec end-to-end (Core of Quick Flow)"
|
||||||
|
|
||||||
- trigger: QQ or fuzzy match on quick-dev2
|
- trigger: QQ or fuzzy match on quick-dev2
|
||||||
workflow: "{project-root}/_bmad/bmm/workflows/bmad-quick-flow/quick-dev2/workflow.md"
|
exec: "{project-root}/_bmad/bmm/workflows/bmad-quick-flow/quick-dev2/workflow.md"
|
||||||
description: "[QQ] Quick Dev 2: Unified quick flow — clarify intent, plan, implement, review, present (experimental)"
|
description: "[QQ] Quick Dev 2: Unified quick flow — clarify intent, plan, implement, review, present (experimental)"
|
||||||
|
|
||||||
- trigger: CR or fuzzy match on code-review
|
- trigger: CR or fuzzy match on code-review
|
||||||
|
|
|
||||||
|
|
@ -29,7 +29,7 @@ Do NOT `git add` anything — this is read-only inspection.
|
||||||
|
|
||||||
**One-shot:** Skip diff construction. Still invoke `{adversarial_review_task}` in a subagent with the changed files — inline review invites anchoring bias.
|
**One-shot:** Skip diff construction. Still invoke `{adversarial_review_task}` in a subagent with the changed files — inline review invites anchoring bias.
|
||||||
|
|
||||||
**Plan-code-review:** Launch three subagents without conversation context. If no sub-agents are available, implement directly.
|
**Plan-code-review:** Launch three subagents without conversation context. If no sub-agents are available, perform all three reviews inline sequentially.
|
||||||
|
|
||||||
- **Blind hunter** — receives `{diff_output}` only. No spec, no context docs, no project access. Invoke via `{adversarial_review_task}`.
|
- **Blind hunter** — receives `{diff_output}` only. No spec, no context docs, no project access. Invoke via `{adversarial_review_task}`.
|
||||||
- **Edge case hunter** — receives `{diff_output}` and read access to the project. Invoke via `{edge_case_hunter_task}`.
|
- **Edge case hunter** — receives `{diff_output}` and read access to the project. Invoke via `{edge_case_hunter_task}`.
|
||||||
|
|
|
||||||
|
|
@ -71,7 +71,7 @@ This uses **step-file architecture** for disciplined execution:
|
||||||
|
|
||||||
Load and read full config from `{main_config}` and resolve:
|
Load and read full config from `{main_config}` and resolve:
|
||||||
|
|
||||||
- `project_name`, `implementation_artifacts`, `user_name`
|
- `project_name`, `planning_artifacts`, `implementation_artifacts`, `user_name`
|
||||||
- `communication_language`, `document_output_language`, `user_skill_level`
|
- `communication_language`, `document_output_language`, `user_skill_level`
|
||||||
- `date` as system-generated current datetime
|
- `date` as system-generated current datetime
|
||||||
- `project_context` = `**/project-context.md` (load if exists)
|
- `project_context` = `**/project-context.md` (load if exists)
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue