bmad-method 6.0.4 â 6.0.5-next.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.augment/code_review_guidelines.yaml +2 -42
- package/.claude/skills/bmad-os-findings-triage/SKILL.md +6 -0
- package/.claude/skills/bmad-os-findings-triage/prompts/agent-prompt.md +104 -0
- package/.claude/skills/bmad-os-findings-triage/prompts/instructions.md +286 -0
- package/.claude/skills/bmad-os-review-pr/SKILL.md +1 -1
- package/.claude/skills/bmad-os-review-pr/prompts/instructions.md +63 -6
- package/.claude/skills/bmad-os-review-prompt/SKILL.md +177 -0
- package/.github/workflows/publish.yaml +243 -0
- package/CONTRIBUTING.md +1 -1
- package/README_CN.md +121 -0
- package/docs/_STYLE_GUIDE.md +10 -10
- package/docs/explanation/brainstorming.md +1 -1
- package/docs/explanation/party-mode.md +1 -1
- package/docs/explanation/preventing-agent-conflicts.md +1 -1
- package/docs/explanation/project-context.md +15 -15
- package/docs/explanation/quick-flow.md +9 -9
- package/docs/how-to/established-projects.md +7 -7
- package/docs/how-to/get-answers-about-bmad.md +2 -2
- package/docs/how-to/install-bmad.md +16 -6
- package/docs/how-to/project-context.md +2 -2
- package/docs/how-to/quick-fixes.md +5 -5
- package/docs/how-to/shard-large-documents.md +1 -1
- package/docs/how-to/upgrade-to-v6.md +8 -5
- package/docs/index.md +1 -1
- package/docs/reference/agents.md +14 -14
- package/docs/reference/commands.md +64 -70
- package/docs/reference/testing.md +1 -1
- package/docs/reference/workflow-map.md +19 -19
- package/docs/tutorials/getting-started.md +34 -34
- package/docs/zh-cn/404.md +9 -0
- package/docs/zh-cn/_STYLE_GUIDE.md +370 -0
- package/docs/zh-cn/explanation/advanced-elicitation.md +62 -0
- package/docs/zh-cn/explanation/adversarial-review.md +71 -0
- package/docs/zh-cn/explanation/brainstorming.md +43 -0
- package/docs/zh-cn/explanation/established-projects-faq.md +60 -0
- package/docs/zh-cn/explanation/party-mode.md +79 -0
- package/docs/zh-cn/explanation/preventing-agent-conflicts.md +137 -0
- package/docs/zh-cn/explanation/project-context.md +176 -0
- package/docs/zh-cn/explanation/quick-flow.md +93 -0
- package/docs/zh-cn/explanation/why-solutioning-matters.md +90 -0
- package/docs/zh-cn/how-to/customize-bmad.md +182 -0
- package/docs/zh-cn/how-to/established-projects.md +134 -0
- package/docs/zh-cn/how-to/get-answers-about-bmad.md +144 -0
- package/docs/zh-cn/how-to/install-bmad.md +105 -0
- package/docs/zh-cn/how-to/non-interactive-installation.md +181 -0
- package/docs/zh-cn/how-to/project-context.md +152 -0
- package/docs/zh-cn/how-to/quick-fixes.md +140 -0
- package/docs/zh-cn/how-to/shard-large-documents.md +86 -0
- package/docs/zh-cn/how-to/upgrade-to-v6.md +120 -0
- package/docs/zh-cn/index.md +69 -0
- package/docs/zh-cn/reference/agents.md +41 -0
- package/docs/zh-cn/reference/commands.md +166 -0
- package/docs/zh-cn/reference/modules.md +94 -0
- package/docs/zh-cn/reference/testing.md +122 -0
- package/docs/zh-cn/reference/workflow-map.md +104 -0
- package/docs/zh-cn/roadmap.mdx +152 -0
- package/docs/zh-cn/tutorials/getting-started.md +300 -0
- package/package.json +1 -1
- package/src/bmm/agents/analyst.agent.yaml +1 -1
- package/src/bmm/agents/bmad-skill-manifest.yaml +39 -0
- package/src/bmm/agents/dev.agent.yaml +2 -2
- package/src/bmm/agents/pm.agent.yaml +1 -1
- package/src/bmm/agents/qa.agent.yaml +1 -1
- package/src/bmm/agents/quick-flow-solo-dev.agent.yaml +6 -2
- package/src/bmm/agents/sm.agent.yaml +4 -4
- package/src/bmm/agents/tech-writer/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/agents/tech-writer/tech-writer.agent.yaml +1 -1
- package/src/bmm/module-help.csv +11 -10
- package/src/bmm/workflows/1-analysis/create-product-brief/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/1-analysis/create-product-brief/steps/step-02-vision.md +1 -1
- package/src/bmm/workflows/1-analysis/create-product-brief/steps/step-03-users.md +1 -1
- package/src/bmm/workflows/1-analysis/create-product-brief/steps/step-04-metrics.md +1 -1
- package/src/bmm/workflows/1-analysis/create-product-brief/steps/step-05-scope.md +1 -1
- package/src/bmm/workflows/1-analysis/create-product-brief/steps/step-06-complete.md +1 -1
- package/src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yaml +14 -0
- package/src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml +14 -0
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-02-discovery.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-02b-vision.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-02c-executive-summary.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-03-success.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-04-journeys.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-05-domain.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-06-innovation.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-07-project-type.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-08-scoping.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-09-functional.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-10-nonfunctional.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-11-polish.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-12-complete.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-e/step-e-01-discovery.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-e/step-e-02-review.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-v/step-v-01-discovery.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-v/step-v-10-smart-validation.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-v/step-v-11-holistic-quality-validation.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-prd/steps-v/step-v-13-report-complete.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-02-discovery.md +1 -1
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-03-core-experience.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-04-emotional-response.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-05-inspiration.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-06-design-system.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-07-defining-experience.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-08-visual-foundation.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-09-design-directions.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-10-user-journeys.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-11-component-strategy.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-12-ux-patterns.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-13-responsive-accessibility.md +2 -2
- package/src/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-14-complete.md +1 -1
- package/src/bmm/workflows/3-solutioning/check-implementation-readiness/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/3-solutioning/check-implementation-readiness/steps/step-06-final-assessment.md +1 -1
- package/src/bmm/workflows/3-solutioning/create-architecture/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-02-context.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-03-starter.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-04-decisions.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-05-patterns.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-06-structure.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-07-validation.md +2 -2
- package/src/bmm/workflows/3-solutioning/create-architecture/steps/step-08-complete.md +1 -1
- package/src/bmm/workflows/3-solutioning/create-epics-and-stories/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-01-validate-prerequisites.md +1 -1
- package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-02-design-epics.md +1 -1
- package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-03-create-stories.md +1 -1
- package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-04-final-validation.md +2 -2
- package/src/bmm/workflows/4-implementation/code-review/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/code-review/discover-inputs.md +88 -0
- package/src/bmm/workflows/4-implementation/code-review/workflow.md +271 -0
- package/src/bmm/workflows/4-implementation/correct-course/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/correct-course/checklist.md +1 -1
- package/src/bmm/workflows/4-implementation/correct-course/{instructions.md â workflow.md} +79 -12
- package/src/bmm/workflows/4-implementation/create-story/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/create-story/checklist.md +9 -10
- package/src/bmm/workflows/4-implementation/create-story/discover-inputs.md +88 -0
- package/src/bmm/workflows/4-implementation/create-story/workflow.md +388 -0
- package/src/bmm/workflows/4-implementation/dev-story/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/dev-story/{instructions.xml â workflow.md} +49 -2
- package/src/bmm/workflows/4-implementation/retrospective/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/retrospective/{instructions.md â workflow.md} +64 -23
- package/src/bmm/workflows/4-implementation/sprint-planning/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/sprint-planning/sprint-status-template.yaml +1 -0
- package/src/bmm/workflows/4-implementation/sprint-planning/{instructions.md â workflow.md} +55 -10
- package/src/bmm/workflows/4-implementation/sprint-status/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/4-implementation/sprint-status/{instructions.md â workflow.md} +45 -8
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/SKILL.md +6 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/bmad-skill-manifest.yaml +1 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/steps/step-01-clarify-and-route.md +54 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/steps/step-02-plan.md +39 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/steps/step-03-implement.md +35 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/steps/step-04-review.md +55 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/steps/step-05-present.md +19 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/tech-spec-template.md +90 -0
- package/src/bmm/workflows/bmad-quick-flow/bmad-quick-dev-new-preview/workflow.md +84 -0
- package/src/bmm/workflows/bmad-quick-flow/quick-dev/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/bmad-quick-flow/quick-dev/steps/step-05-adversarial-review.md +8 -14
- package/src/bmm/workflows/bmad-quick-flow/quick-dev/workflow.md +1 -1
- package/src/bmm/workflows/bmad-quick-flow/quick-spec/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/bmad-quick-flow/quick-spec/steps/step-04-review.md +4 -6
- package/src/bmm/workflows/bmad-quick-flow/quick-spec/workflow.md +1 -1
- package/src/bmm/workflows/document-project/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/document-project/instructions.md +5 -7
- package/src/bmm/workflows/document-project/workflow.md +39 -0
- package/src/bmm/workflows/document-project/workflows/deep-dive-instructions.md +0 -1
- package/src/bmm/workflows/document-project/workflows/deep-dive-workflow.md +42 -0
- package/src/bmm/workflows/document-project/workflows/full-scan-instructions.md +0 -1
- package/src/bmm/workflows/document-project/workflows/full-scan-workflow.md +42 -0
- package/src/bmm/workflows/generate-project-context/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/generate-project-context/steps/step-02-generate.md +2 -2
- package/src/bmm/workflows/qa-generate-e2e-tests/bmad-skill-manifest.yaml +3 -0
- package/src/bmm/workflows/qa-generate-e2e-tests/checklist.md +1 -1
- package/src/bmm/workflows/qa-generate-e2e-tests/{instructions.md â workflow.md} +40 -7
- package/src/core/agents/bmad-master.agent.yaml +1 -1
- package/src/core/agents/bmad-skill-manifest.yaml +3 -0
- package/src/core/module-help.csv +3 -3
- package/src/core/module.yaml +1 -1
- package/src/core/tasks/bmad-help/SKILL.md +6 -0
- package/src/core/tasks/bmad-help/bmad-skill-manifest.yaml +1 -0
- package/src/core/tasks/{help.md â bmad-help/workflow.md} +6 -4
- package/src/core/tasks/bmad-review-adversarial-general/SKILL.md +6 -0
- package/src/core/tasks/bmad-review-adversarial-general/bmad-skill-manifest.yaml +1 -0
- package/src/core/tasks/bmad-review-adversarial-general/workflow.md +32 -0
- package/src/core/tasks/bmad-review-edge-case-hunter/SKILL.md +6 -0
- package/src/core/tasks/bmad-review-edge-case-hunter/bmad-skill-manifest.yaml +1 -0
- package/src/core/tasks/bmad-review-edge-case-hunter/workflow.md +62 -0
- package/src/core/tasks/bmad-skill-manifest.yaml +19 -0
- package/src/core/workflows/advanced-elicitation/bmad-skill-manifest.yaml +3 -0
- package/src/core/workflows/advanced-elicitation/workflow.md +138 -0
- package/src/core/workflows/brainstorming/bmad-skill-manifest.yaml +3 -0
- package/src/core/workflows/brainstorming/steps/step-03-technique-execution.md +1 -1
- package/src/core/workflows/brainstorming/workflow.md +1 -1
- package/src/core/workflows/party-mode/bmad-skill-manifest.yaml +3 -0
- package/src/utility/agent-components/activation-steps.txt +2 -2
- package/src/utility/agent-components/handler-multi.txt +1 -2
- package/test/adversarial-review-tests/README.md +3 -3
- package/test/adversarial-review-tests/test-cases.yaml +2 -2
- package/test/fixtures/agent-schema/valid/menu/multiple-menu-items.agent.yaml +1 -1
- package/test/fixtures/agent-schema/valid/menu-commands/all-command-types.agent.yaml +1 -1
- package/test/fixtures/agent-schema/valid/menu-commands/multiple-commands.agent.yaml +1 -2
- package/test/fixtures/file-refs-csv/valid/bmm-style.csv +1 -1
- package/test/test-file-refs-csv.js +1 -1
- package/test/test-install-to-bmad.js +154 -0
- package/test/test-installation-components.js +1586 -2
- package/test/test-workflow-path-regex.js +88 -0
- package/tools/cli/installers/lib/core/installer.js +34 -1
- package/tools/cli/installers/lib/core/manifest-generator.js +328 -35
- package/tools/cli/installers/lib/ide/_base-ide.js +24 -15
- package/tools/cli/installers/lib/ide/_config-driven.js +472 -53
- package/tools/cli/installers/lib/ide/manager.js +23 -61
- package/tools/cli/installers/lib/ide/platform-codes.yaml +108 -30
- package/tools/cli/installers/lib/ide/shared/agent-command-generator.js +1 -0
- package/tools/cli/installers/lib/ide/shared/bmad-artifacts.js +7 -0
- package/tools/cli/installers/lib/ide/shared/path-utils.js +68 -3
- package/tools/cli/installers/lib/ide/shared/skill-manifest.js +90 -0
- package/tools/cli/installers/lib/ide/shared/task-tool-command-generator.js +2 -0
- package/tools/cli/installers/lib/ide/shared/workflow-command-generator.js +6 -145
- package/tools/cli/installers/lib/modules/manager.js +9 -132
- package/tools/cli/lib/agent/compiler.js +1 -10
- package/tools/cli/lib/agent-analyzer.js +2 -14
- package/tools/cli/lib/yaml-xml-builder.js +1 -18
- package/tools/docs/native-skills-migration-checklist.md +281 -0
- package/tools/platform-codes.yaml +1 -1
- package/tools/schema/agent.js +1 -3
- package/tools/validate-file-refs.js +2 -0
- package/website/astro.config.mjs +24 -3
- package/website/src/content/config.ts +2 -1
- package/website/src/content/i18n/zh-CN.json +28 -0
- package/src/bmm/workflows/4-implementation/code-review/instructions.xml +0 -227
- package/src/bmm/workflows/4-implementation/code-review/workflow.yaml +0 -43
- package/src/bmm/workflows/4-implementation/correct-course/workflow.yaml +0 -53
- package/src/bmm/workflows/4-implementation/create-story/instructions.xml +0 -346
- package/src/bmm/workflows/4-implementation/create-story/workflow.yaml +0 -52
- package/src/bmm/workflows/4-implementation/dev-story/workflow.yaml +0 -20
- package/src/bmm/workflows/4-implementation/retrospective/workflow.yaml +0 -52
- package/src/bmm/workflows/4-implementation/sprint-planning/workflow.yaml +0 -47
- package/src/bmm/workflows/4-implementation/sprint-status/workflow.yaml +0 -25
- package/src/bmm/workflows/document-project/workflow.yaml +0 -22
- package/src/bmm/workflows/document-project/workflows/deep-dive.yaml +0 -31
- package/src/bmm/workflows/document-project/workflows/full-scan.yaml +0 -31
- package/src/bmm/workflows/qa-generate-e2e-tests/workflow.yaml +0 -42
- package/src/core/tasks/review-adversarial-general.xml +0 -49
- package/src/core/tasks/review-edge-case-hunter.xml +0 -63
- package/src/core/tasks/workflow.xml +0 -235
- package/src/core/workflows/advanced-elicitation/workflow.xml +0 -118
- package/src/utility/agent-components/handler-validate-workflow.txt +0 -7
- package/src/utility/agent-components/handler-workflow.txt +0 -10
- package/tools/cli/installers/lib/ide/codex.js +0 -440
- package/tools/cli/installers/lib/ide/github-copilot.js +0 -699
- package/tools/cli/installers/lib/ide/kilo.js +0 -269
- package/tools/cli/installers/lib/ide/rovodev.js +0 -257
- package/tools/cli/installers/lib/ide/templates/combined/default-workflow-yaml.md +0 -14
- package/tools/cli/installers/lib/ide/templates/combined/kiro-workflow-yaml.md +0 -15
- package/tools/cli/installers/lib/ide/templates/workflow-command-template.md +0 -13
- package/tools/cli/installers/lib/ide/templates/workflow-commander.md +0 -5
|
@@ -30,7 +30,7 @@ This step will generate content and present choices:
|
|
|
30
30
|
|
|
31
31
|
## PROTOCOL INTEGRATION:
|
|
32
32
|
|
|
33
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
33
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
34
34
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
35
35
|
- PROTOCOLS always return to this step's A/P/C menu
|
|
36
36
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -209,7 +209,7 @@ Show the generated responsive and accessibility content and present choices:
|
|
|
209
209
|
|
|
210
210
|
#### If 'A' (Advanced Elicitation):
|
|
211
211
|
|
|
212
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
212
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with the current responsive/accessibility content
|
|
213
213
|
- Process the enhanced insights that come back
|
|
214
214
|
- Ask user: "Accept these improvements to the responsive/accessibility strategy? (y/n)"
|
|
215
215
|
- If yes: Update content with improvements, then return to A/P/C menu
|
|
@@ -109,7 +109,7 @@ The assessment found [number] issues requiring attention. Review the detailed re
|
|
|
109
109
|
|
|
110
110
|
The implementation readiness workflow is now complete. The report contains all findings and recommendations for the user to consider.
|
|
111
111
|
|
|
112
|
-
Implementation Readiness complete.
|
|
112
|
+
Implementation Readiness complete. Invoke the `bmad-help` skill.
|
|
113
113
|
|
|
114
114
|
---
|
|
115
115
|
|
|
@@ -31,7 +31,7 @@ This step will generate content and present choices:
|
|
|
31
31
|
|
|
32
32
|
## PROTOCOL INTEGRATION:
|
|
33
33
|
|
|
34
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
34
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
35
35
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
36
36
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
37
37
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -170,7 +170,7 @@ Show the generated content and present choices:
|
|
|
170
170
|
|
|
171
171
|
#### If 'A' (Advanced Elicitation):
|
|
172
172
|
|
|
173
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
173
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with the current context analysis
|
|
174
174
|
- Process the enhanced architectural insights that come back
|
|
175
175
|
- Ask user: "Accept these enhancements to the project context analysis? (y/n)"
|
|
176
176
|
- If yes: Update content with improvements, then return to A/P/C menu
|
|
@@ -31,7 +31,7 @@ This step will generate content and present choices:
|
|
|
31
31
|
|
|
32
32
|
## PROTOCOL INTEGRATION:
|
|
33
33
|
|
|
34
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
34
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
35
35
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
36
36
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
37
37
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -276,7 +276,7 @@ Show the generated content and present choices:
|
|
|
276
276
|
|
|
277
277
|
#### If 'A' (Advanced Elicitation):
|
|
278
278
|
|
|
279
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
279
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with current starter analysis
|
|
280
280
|
- Process enhanced insights about starter options or custom approaches
|
|
281
281
|
- Ask user: "Accept these changes to the starter template evaluation? (y/n)"
|
|
282
282
|
- If yes: Update content, then return to A/P/C menu
|
|
@@ -32,7 +32,7 @@ This step will generate content and present choices for each decision category:
|
|
|
32
32
|
|
|
33
33
|
## PROTOCOL INTEGRATION:
|
|
34
34
|
|
|
35
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
35
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
36
36
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
37
37
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
38
38
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -264,7 +264,7 @@ Show the generated decisions content and present choices:
|
|
|
264
264
|
|
|
265
265
|
#### If 'A' (Advanced Elicitation):
|
|
266
266
|
|
|
267
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
267
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with specific decision categories
|
|
268
268
|
- Process enhanced insights about particular decisions
|
|
269
269
|
- Ask user: "Accept these enhancements to the architectural decisions? (y/n)"
|
|
270
270
|
- If yes: Update content, then return to A/P/C menu
|
|
@@ -32,7 +32,7 @@ This step will generate content and present choices:
|
|
|
32
32
|
|
|
33
33
|
## PROTOCOL INTEGRATION:
|
|
34
34
|
|
|
35
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
35
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
36
36
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
37
37
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
38
38
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -305,7 +305,7 @@ Show the generated patterns content and present choices:
|
|
|
305
305
|
|
|
306
306
|
#### If 'A' (Advanced Elicitation):
|
|
307
307
|
|
|
308
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
308
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with current patterns
|
|
309
309
|
- Process enhanced consistency rules that come back
|
|
310
310
|
- Ask user: "Accept these additional pattern refinements? (y/n)"
|
|
311
311
|
- If yes: Update content, then return to A/P/C menu
|
|
@@ -32,7 +32,7 @@ This step will generate content and present choices:
|
|
|
32
32
|
|
|
33
33
|
## PROTOCOL INTEGRATION:
|
|
34
34
|
|
|
35
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
35
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
36
36
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
37
37
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
38
38
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -325,7 +325,7 @@ Show the generated project structure content and present choices:
|
|
|
325
325
|
|
|
326
326
|
#### If 'A' (Advanced Elicitation):
|
|
327
327
|
|
|
328
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
328
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with current project structure
|
|
329
329
|
- Process enhanced organizational insights that come back
|
|
330
330
|
- Ask user: "Accept these changes to the project structure? (y/n)"
|
|
331
331
|
- If yes: Update content, then return to A/P/C menu
|
|
@@ -32,7 +32,7 @@ This step will generate content and present choices:
|
|
|
32
32
|
|
|
33
33
|
## PROTOCOL INTEGRATION:
|
|
34
34
|
|
|
35
|
-
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
35
|
+
- When 'A' selected: Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md
|
|
36
36
|
- When 'P' selected: Read fully and follow: {project-root}/_bmad/core/workflows/party-mode/workflow.md
|
|
37
37
|
- PROTOCOLS always return to display this step's A/P/C menu after the A or P have completed
|
|
38
38
|
- User accepts/rejects protocol changes before proceeding
|
|
@@ -305,7 +305,7 @@ Show the validation results and present choices:
|
|
|
305
305
|
|
|
306
306
|
#### If 'A' (Advanced Elicitation):
|
|
307
307
|
|
|
308
|
-
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
308
|
+
- Read fully and follow: {project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md with validation issues
|
|
309
309
|
- Process enhanced solutions for complex concerns
|
|
310
310
|
- Ask user: "Accept these architectural improvements? (y/n)"
|
|
311
311
|
- If yes: Update content, then return to A/P/C menu
|
|
@@ -41,7 +41,7 @@ completedAt: '{{current_date}}'
|
|
|
41
41
|
|
|
42
42
|
### 3. Next Steps Guidance
|
|
43
43
|
|
|
44
|
-
Architecture complete.
|
|
44
|
+
Architecture complete. Invoke the `bmad-help` skill.
|
|
45
45
|
|
|
46
46
|
Upon Completion of task output: offer to answer any questions about the Architecture Document.
|
|
47
47
|
|
|
@@ -13,7 +13,7 @@ outputFile: '{planning_artifacts}/epics.md'
|
|
|
13
13
|
epicsTemplate: '{workflow_path}/templates/epics-template.md'
|
|
14
14
|
|
|
15
15
|
# Task References
|
|
16
|
-
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
16
|
+
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md'
|
|
17
17
|
partyModeWorkflow: '{project-root}/_bmad/core/workflows/party-mode/workflow.md'
|
|
18
18
|
|
|
19
19
|
# Template References
|
package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-02-design-epics.md
CHANGED
|
@@ -12,7 +12,7 @@ workflowFile: '{workflow_path}/workflow.md'
|
|
|
12
12
|
outputFile: '{planning_artifacts}/epics.md'
|
|
13
13
|
|
|
14
14
|
# Task References
|
|
15
|
-
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
15
|
+
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md'
|
|
16
16
|
partyModeWorkflow: '{project-root}/_bmad/core/workflows/party-mode/workflow.md'
|
|
17
17
|
|
|
18
18
|
# Template References
|
package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-03-create-stories.md
CHANGED
|
@@ -12,7 +12,7 @@ workflowFile: '{workflow_path}/workflow.md'
|
|
|
12
12
|
outputFile: '{planning_artifacts}/epics.md'
|
|
13
13
|
|
|
14
14
|
# Task References
|
|
15
|
-
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
15
|
+
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md'
|
|
16
16
|
partyModeWorkflow: '{project-root}/_bmad/core/workflows/party-mode/workflow.md'
|
|
17
17
|
|
|
18
18
|
# Template References
|
package/src/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-04-final-validation.md
CHANGED
|
@@ -11,7 +11,7 @@ workflowFile: '{workflow_path}/workflow.md'
|
|
|
11
11
|
outputFile: '{planning_artifacts}/epics.md'
|
|
12
12
|
|
|
13
13
|
# Task References
|
|
14
|
-
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.
|
|
14
|
+
advancedElicitationTask: '{project-root}/_bmad/core/workflows/advanced-elicitation/workflow.md'
|
|
15
15
|
partyModeWorkflow: '{project-root}/_bmad/core/workflows/party-mode/workflow.md'
|
|
16
16
|
|
|
17
17
|
# Template References
|
|
@@ -144,6 +144,6 @@ If all validations pass:
|
|
|
144
144
|
|
|
145
145
|
When C is selected, the workflow is complete and the epics.md is ready for development.
|
|
146
146
|
|
|
147
|
-
Epics and Stories complete.
|
|
147
|
+
Epics and Stories complete. Invoke the `bmad-help` skill.
|
|
148
148
|
|
|
149
149
|
Upon Completion of task output: offer to answer any questions about the Epics and Stories.
|
|
@@ -0,0 +1,88 @@
|
|
|
1
|
+
# Discover Inputs Protocol
|
|
2
|
+
|
|
3
|
+
**Objective:** Intelligently load project files (whole or sharded) based on the workflow's Input Files configuration.
|
|
4
|
+
|
|
5
|
+
**Prerequisite:** Only execute this protocol if the workflow defines an Input Files section. If no input file patterns are configured, skip this entirely.
|
|
6
|
+
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
## Step 1: Parse Input File Patterns
|
|
10
|
+
|
|
11
|
+
- Read the Input Files table from the workflow configuration.
|
|
12
|
+
- For each input group (prd, architecture, epics, ux, etc.), note the **load strategy** if specified.
|
|
13
|
+
|
|
14
|
+
## Step 2: Load Files Using Smart Strategies
|
|
15
|
+
|
|
16
|
+
For each pattern in the Input Files table, work through the following substeps in order:
|
|
17
|
+
|
|
18
|
+
### 2a: Try Sharded Documents First
|
|
19
|
+
|
|
20
|
+
If a sharded pattern exists for this input, determine the load strategy (defaults to **FULL_LOAD** if not specified), then apply the matching strategy:
|
|
21
|
+
|
|
22
|
+
#### FULL_LOAD Strategy
|
|
23
|
+
|
|
24
|
+
Load ALL files in the sharded directory. Use this for PRD, Architecture, UX, brownfield docs, or whenever the full picture is needed.
|
|
25
|
+
|
|
26
|
+
1. Use the glob pattern to find ALL `.md` files (e.g., `{planning_artifacts}/*architecture*/*.md`).
|
|
27
|
+
2. Load EVERY matching file completely.
|
|
28
|
+
3. Concatenate content in logical order: `index.md` first if it exists, then alphabetical.
|
|
29
|
+
4. Store the combined result in a variable named `{pattern_name_content}` (e.g., `{architecture_content}`).
|
|
30
|
+
|
|
31
|
+
#### SELECTIVE_LOAD Strategy
|
|
32
|
+
|
|
33
|
+
Load a specific shard using a template variable. Example: used for epics with `{{epic_num}}`.
|
|
34
|
+
|
|
35
|
+
1. Check for template variables in the sharded pattern (e.g., `{{epic_num}}`).
|
|
36
|
+
2. If the variable is undefined, ask the user for the value OR infer it from context.
|
|
37
|
+
3. Resolve the template to a specific file path.
|
|
38
|
+
4. Load that specific file.
|
|
39
|
+
5. Store in variable: `{pattern_name_content}`.
|
|
40
|
+
|
|
41
|
+
#### INDEX_GUIDED Strategy
|
|
42
|
+
|
|
43
|
+
Load index.md, analyze the structure and description of each doc in the index, then intelligently load relevant docs.
|
|
44
|
+
|
|
45
|
+
**DO NOT BE LAZY** -- use best judgment to load documents that might have relevant information, even if there is only a 5% chance of relevance.
|
|
46
|
+
|
|
47
|
+
1. Load `index.md` from the sharded directory.
|
|
48
|
+
2. Parse the table of contents, links, and section headers.
|
|
49
|
+
3. Analyze the workflow's purpose and objective.
|
|
50
|
+
4. Identify which linked/referenced documents are likely relevant.
|
|
51
|
+
- *Example:* If the workflow is about authentication and the index shows "Auth Overview", "Payment Setup", "Deployment" -- load the auth docs, consider deployment docs, skip payment.
|
|
52
|
+
5. Load all identified relevant documents.
|
|
53
|
+
6. Store combined content in variable: `{pattern_name_content}`.
|
|
54
|
+
|
|
55
|
+
**When in doubt, LOAD IT** -- context is valuable, and being thorough is better than missing critical info.
|
|
56
|
+
|
|
57
|
+
---
|
|
58
|
+
|
|
59
|
+
After applying the matching strategy, mark the pattern as **RESOLVED** and move to the next pattern.
|
|
60
|
+
|
|
61
|
+
### 2b: Try Whole Document if No Sharded Found
|
|
62
|
+
|
|
63
|
+
If no sharded matches were found OR no sharded pattern exists for this input:
|
|
64
|
+
|
|
65
|
+
1. Attempt a glob match on the "whole" pattern (e.g., `{planning_artifacts}/*prd*.md`).
|
|
66
|
+
2. If matches are found, load ALL matching files completely (no offset/limit).
|
|
67
|
+
3. Store content in variable: `{pattern_name_content}` (e.g., `{prd_content}`).
|
|
68
|
+
4. Mark pattern as **RESOLVED** and move to the next pattern.
|
|
69
|
+
|
|
70
|
+
### 2c: Handle Not Found
|
|
71
|
+
|
|
72
|
+
If no matches were found for either sharded or whole patterns:
|
|
73
|
+
|
|
74
|
+
1. Set `{pattern_name_content}` to empty string.
|
|
75
|
+
2. Note in session: "No {pattern_name} files found" -- this is not an error, just unavailable. Offer the user a chance to provide the file.
|
|
76
|
+
|
|
77
|
+
## Step 3: Report Discovery Results
|
|
78
|
+
|
|
79
|
+
List all loaded content variables with file counts. Example:
|
|
80
|
+
|
|
81
|
+
```
|
|
82
|
+
OK Loaded {prd_content} from 5 sharded files: prd/index.md, prd/requirements.md, ...
|
|
83
|
+
OK Loaded {architecture_content} from 1 file: Architecture.md
|
|
84
|
+
OK Loaded {epics_content} from selective load: epics/epic-3.md
|
|
85
|
+
-- No ux_design files found
|
|
86
|
+
```
|
|
87
|
+
|
|
88
|
+
This gives the workflow transparency into what context is available.
|
|
@@ -0,0 +1,271 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: code-review
|
|
3
|
+
description: 'Perform adversarial code review finding specific issues. Use when the user says "run code review" or "review this code"'
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Code Review Workflow
|
|
7
|
+
|
|
8
|
+
**Goal:** Perform adversarial code review finding specific issues.
|
|
9
|
+
|
|
10
|
+
**Your Role:** Adversarial Code Reviewer.
|
|
11
|
+
- YOU ARE AN ADVERSARIAL CODE REVIEWER - Find what's wrong or missing!
|
|
12
|
+
- Communicate all responses in {communication_language} and language MUST be tailored to {user_skill_level}
|
|
13
|
+
- Generate all documents in {document_output_language}
|
|
14
|
+
- Your purpose: Validate story file claims against actual implementation
|
|
15
|
+
- Challenge everything: Are tasks marked [x] actually done? Are ACs really implemented?
|
|
16
|
+
- Find 3-10 specific issues in every review minimum - no lazy "looks good" reviews - YOU are so much better than the dev agent that wrote this slop
|
|
17
|
+
- Read EVERY file in the File List - verify implementation against story requirements
|
|
18
|
+
- Tasks marked complete but not done = CRITICAL finding
|
|
19
|
+
- Acceptance Criteria not implemented = HIGH severity finding
|
|
20
|
+
- Do not review files that are not part of the application's source code. Always exclude the `_bmad/` and `_bmad-output/` folders from the review. Always exclude IDE and CLI configuration folders like `.cursor/` and `.windsurf/` and `.claude/`
|
|
21
|
+
|
|
22
|
+
---
|
|
23
|
+
|
|
24
|
+
## INITIALIZATION
|
|
25
|
+
|
|
26
|
+
### Configuration Loading
|
|
27
|
+
|
|
28
|
+
Load config from `{project-root}/_bmad/bmm/config.yaml` and resolve:
|
|
29
|
+
|
|
30
|
+
- `project_name`, `user_name`
|
|
31
|
+
- `communication_language`, `document_output_language`
|
|
32
|
+
- `user_skill_level`
|
|
33
|
+
- `planning_artifacts`, `implementation_artifacts`
|
|
34
|
+
- `date` as system-generated current datetime
|
|
35
|
+
|
|
36
|
+
### Paths
|
|
37
|
+
|
|
38
|
+
- `installed_path` = `{project-root}/_bmad/bmm/workflows/4-implementation/code-review`
|
|
39
|
+
- `sprint_status` = `{implementation_artifacts}/sprint-status.yaml`
|
|
40
|
+
- `validation` = `{installed_path}/checklist.md`
|
|
41
|
+
|
|
42
|
+
### Input Files
|
|
43
|
+
|
|
44
|
+
| Input | Description | Path Pattern(s) | Load Strategy |
|
|
45
|
+
|-------|-------------|------------------|---------------|
|
|
46
|
+
| architecture | System architecture for review context | whole: `{planning_artifacts}/*architecture*.md`, sharded: `{planning_artifacts}/*architecture*/*.md` | FULL_LOAD |
|
|
47
|
+
| ux_design | UX design specification (if UI review) | whole: `{planning_artifacts}/*ux*.md`, sharded: `{planning_artifacts}/*ux*/*.md` | FULL_LOAD |
|
|
48
|
+
| epics | Epic containing story being reviewed | whole: `{planning_artifacts}/*epic*.md`, sharded_index: `{planning_artifacts}/*epic*/index.md`, sharded_single: `{planning_artifacts}/*epic*/epic-{{epic_num}}.md` | SELECTIVE_LOAD |
|
|
49
|
+
|
|
50
|
+
### Context
|
|
51
|
+
|
|
52
|
+
- `project_context` = `**/project-context.md` (load if exists)
|
|
53
|
+
|
|
54
|
+
---
|
|
55
|
+
|
|
56
|
+
## EXECUTION
|
|
57
|
+
|
|
58
|
+
<workflow>
|
|
59
|
+
|
|
60
|
+
<step n="1" goal="Load story and discover changes">
|
|
61
|
+
<action>Use provided {{story_path}} or ask user which story file to review</action>
|
|
62
|
+
<action>Read COMPLETE story file</action>
|
|
63
|
+
<action>Set {{story_key}} = extracted key from filename (e.g., "1-2-user-authentication.md" â "1-2-user-authentication") or story
|
|
64
|
+
metadata</action>
|
|
65
|
+
<action>Parse sections: Story, Acceptance Criteria, Tasks/Subtasks, Dev Agent Record â File List, Change Log</action>
|
|
66
|
+
|
|
67
|
+
<!-- Discover actual changes via git -->
|
|
68
|
+
<action>Check if git repository detected in current directory</action>
|
|
69
|
+
<check if="git repository exists">
|
|
70
|
+
<action>Run `git status --porcelain` to find uncommitted changes</action>
|
|
71
|
+
<action>Run `git diff --name-only` to see modified files</action>
|
|
72
|
+
<action>Run `git diff --cached --name-only` to see staged files</action>
|
|
73
|
+
<action>Compile list of actually changed files from git output</action>
|
|
74
|
+
</check>
|
|
75
|
+
|
|
76
|
+
<!-- Cross-reference story File List vs git reality -->
|
|
77
|
+
<action>Compare story's Dev Agent Record â File List with actual git changes</action>
|
|
78
|
+
<action>Note discrepancies:
|
|
79
|
+
- Files in git but not in story File List
|
|
80
|
+
- Files in story File List but no git changes
|
|
81
|
+
- Missing documentation of what was actually changed
|
|
82
|
+
</action>
|
|
83
|
+
|
|
84
|
+
<action>Read fully and follow `{installed_path}/discover-inputs.md` to load all input files</action>
|
|
85
|
+
<action>Load {project_context} for coding standards (if exists)</action>
|
|
86
|
+
</step>
|
|
87
|
+
|
|
88
|
+
<step n="2" goal="Build review attack plan">
|
|
89
|
+
<action>Extract ALL Acceptance Criteria from story</action>
|
|
90
|
+
<action>Extract ALL Tasks/Subtasks with completion status ([x] vs [ ])</action>
|
|
91
|
+
<action>From Dev Agent Record â File List, compile list of claimed changes</action>
|
|
92
|
+
|
|
93
|
+
<action>Create review plan:
|
|
94
|
+
1. **AC Validation**: Verify each AC is actually implemented
|
|
95
|
+
2. **Task Audit**: Verify each [x] task is really done
|
|
96
|
+
3. **Code Quality**: Security, performance, maintainability
|
|
97
|
+
4. **Test Quality**: Real tests vs placeholder bullshit
|
|
98
|
+
</action>
|
|
99
|
+
</step>
|
|
100
|
+
|
|
101
|
+
<step n="3" goal="Execute adversarial review">
|
|
102
|
+
<critical>VALIDATE EVERY CLAIM - Check git reality vs story claims</critical>
|
|
103
|
+
|
|
104
|
+
<!-- Git vs Story Discrepancies -->
|
|
105
|
+
<action>Review git vs story File List discrepancies:
|
|
106
|
+
1. **Files changed but not in story File List** â MEDIUM finding (incomplete documentation)
|
|
107
|
+
2. **Story lists files but no git changes** â HIGH finding (false claims)
|
|
108
|
+
3. **Uncommitted changes not documented** â MEDIUM finding (transparency issue)
|
|
109
|
+
</action>
|
|
110
|
+
|
|
111
|
+
<!-- Use combined file list: story File List + git discovered files -->
|
|
112
|
+
<action>Create comprehensive review file list from story File List and git changes</action>
|
|
113
|
+
|
|
114
|
+
<!-- AC Validation -->
|
|
115
|
+
<action>For EACH Acceptance Criterion:
|
|
116
|
+
1. Read the AC requirement
|
|
117
|
+
2. Search implementation files for evidence
|
|
118
|
+
3. Determine: IMPLEMENTED, PARTIAL, or MISSING
|
|
119
|
+
4. If MISSING/PARTIAL â HIGH SEVERITY finding
|
|
120
|
+
</action>
|
|
121
|
+
|
|
122
|
+
<!-- Task Completion Audit -->
|
|
123
|
+
<action>For EACH task marked [x]:
|
|
124
|
+
1. Read the task description
|
|
125
|
+
2. Search files for evidence it was actually done
|
|
126
|
+
3. **CRITICAL**: If marked [x] but NOT DONE â CRITICAL finding
|
|
127
|
+
4. Record specific proof (file:line)
|
|
128
|
+
</action>
|
|
129
|
+
|
|
130
|
+
<!-- Code Quality Deep Dive -->
|
|
131
|
+
<action>For EACH file in comprehensive review list:
|
|
132
|
+
1. **Security**: Look for injection risks, missing validation, auth issues
|
|
133
|
+
2. **Performance**: N+1 queries, inefficient loops, missing caching
|
|
134
|
+
3. **Error Handling**: Missing try/catch, poor error messages
|
|
135
|
+
4. **Code Quality**: Complex functions, magic numbers, poor naming
|
|
136
|
+
5. **Test Quality**: Are tests real assertions or placeholders?
|
|
137
|
+
</action>
|
|
138
|
+
|
|
139
|
+
<check if="total_issues_found lt 3">
|
|
140
|
+
<critical>NOT LOOKING HARD ENOUGH - Find more problems!</critical>
|
|
141
|
+
<action>Re-examine code for:
|
|
142
|
+
- Edge cases and null handling
|
|
143
|
+
- Architecture violations
|
|
144
|
+
- Documentation gaps
|
|
145
|
+
- Integration issues
|
|
146
|
+
- Dependency problems
|
|
147
|
+
- Git commit message quality (if applicable)
|
|
148
|
+
</action>
|
|
149
|
+
<action>Find at least 3 more specific, actionable issues</action>
|
|
150
|
+
</check>
|
|
151
|
+
</step>
|
|
152
|
+
|
|
153
|
+
<step n="4" goal="Present findings and fix them">
|
|
154
|
+
<action>Categorize findings: HIGH (must fix), MEDIUM (should fix), LOW (nice to fix)</action>
|
|
155
|
+
<action>Set {{fixed_count}} = 0</action>
|
|
156
|
+
<action>Set {{action_count}} = 0</action>
|
|
157
|
+
|
|
158
|
+
<output>**đĨ CODE REVIEW FINDINGS, {user_name}!**
|
|
159
|
+
|
|
160
|
+
**Story:** {{story_file}}
|
|
161
|
+
**Git vs Story Discrepancies:** {{git_discrepancy_count}} found
|
|
162
|
+
**Issues Found:** {{high_count}} High, {{medium_count}} Medium, {{low_count}} Low
|
|
163
|
+
|
|
164
|
+
## đ´ CRITICAL ISSUES
|
|
165
|
+
- Tasks marked [x] but not actually implemented
|
|
166
|
+
- Acceptance Criteria not implemented
|
|
167
|
+
- Story claims files changed but no git evidence
|
|
168
|
+
- Security vulnerabilities
|
|
169
|
+
|
|
170
|
+
## đĄ MEDIUM ISSUES
|
|
171
|
+
- Files changed but not documented in story File List
|
|
172
|
+
- Uncommitted changes not tracked
|
|
173
|
+
- Performance problems
|
|
174
|
+
- Poor test coverage/quality
|
|
175
|
+
- Code maintainability issues
|
|
176
|
+
|
|
177
|
+
## đĸ LOW ISSUES
|
|
178
|
+
- Code style improvements
|
|
179
|
+
- Documentation gaps
|
|
180
|
+
- Git commit message quality
|
|
181
|
+
</output>
|
|
182
|
+
|
|
183
|
+
<ask>What should I do with these issues?
|
|
184
|
+
|
|
185
|
+
1. **Fix them automatically** - I'll update the code and tests
|
|
186
|
+
2. **Create action items** - Add to story Tasks/Subtasks for later
|
|
187
|
+
3. **Show me details** - Deep dive into specific issues
|
|
188
|
+
|
|
189
|
+
Choose [1], [2], or specify which issue to examine:</ask>
|
|
190
|
+
|
|
191
|
+
<check if="user chooses 1">
|
|
192
|
+
<action>Fix all HIGH and MEDIUM issues in the code</action>
|
|
193
|
+
<action>Add/update tests as needed</action>
|
|
194
|
+
<action>Update File List in story if files changed</action>
|
|
195
|
+
<action>Update story Dev Agent Record with fixes applied</action>
|
|
196
|
+
<action>Set {{fixed_count}} = number of HIGH and MEDIUM issues fixed</action>
|
|
197
|
+
<action>Set {{action_count}} = 0</action>
|
|
198
|
+
</check>
|
|
199
|
+
|
|
200
|
+
<check if="user chooses 2">
|
|
201
|
+
<action>Add "Review Follow-ups (AI)" subsection to Tasks/Subtasks</action>
|
|
202
|
+
<action>For each issue: `- [ ] [AI-Review][Severity] Description [file:line]`</action>
|
|
203
|
+
<action>Set {{action_count}} = number of action items created</action>
|
|
204
|
+
<action>Set {{fixed_count}} = 0</action>
|
|
205
|
+
</check>
|
|
206
|
+
|
|
207
|
+
<check if="user chooses 3">
|
|
208
|
+
<action>Show detailed explanation with code examples</action>
|
|
209
|
+
<action>Return to fix decision</action>
|
|
210
|
+
</check>
|
|
211
|
+
</step>
|
|
212
|
+
|
|
213
|
+
<step n="5" goal="Update story status and sync sprint tracking">
|
|
214
|
+
<!-- Determine new status based on review outcome -->
|
|
215
|
+
<check if="all HIGH and MEDIUM issues fixed AND all ACs implemented">
|
|
216
|
+
<action>Set {{new_status}} = "done"</action>
|
|
217
|
+
<action>Update story Status field to "done"</action>
|
|
218
|
+
</check>
|
|
219
|
+
<check if="HIGH or MEDIUM issues remain OR ACs not fully implemented">
|
|
220
|
+
<action>Set {{new_status}} = "in-progress"</action>
|
|
221
|
+
<action>Update story Status field to "in-progress"</action>
|
|
222
|
+
</check>
|
|
223
|
+
<action>Save story file</action>
|
|
224
|
+
|
|
225
|
+
<!-- Determine sprint tracking status -->
|
|
226
|
+
<check if="{sprint_status} file exists">
|
|
227
|
+
<action>Set {{current_sprint_status}} = "enabled"</action>
|
|
228
|
+
</check>
|
|
229
|
+
<check if="{sprint_status} file does NOT exist">
|
|
230
|
+
<action>Set {{current_sprint_status}} = "no-sprint-tracking"</action>
|
|
231
|
+
</check>
|
|
232
|
+
|
|
233
|
+
<!-- Sync sprint-status.yaml when story status changes (only if sprint tracking enabled) -->
|
|
234
|
+
<check if="{{current_sprint_status}} != 'no-sprint-tracking'">
|
|
235
|
+
<action>Load the FULL file: {sprint_status}</action>
|
|
236
|
+
<action>Find development_status key matching {{story_key}}</action>
|
|
237
|
+
|
|
238
|
+
<check if="{{new_status}} == 'done'">
|
|
239
|
+
<action>Update development_status[{{story_key}}] = "done"</action>
|
|
240
|
+
<action>Update last_updated field to current date</action>
|
|
241
|
+
<action>Save file, preserving ALL comments and structure</action>
|
|
242
|
+
<output>â
Sprint status synced: {{story_key}} â done</output>
|
|
243
|
+
</check>
|
|
244
|
+
|
|
245
|
+
<check if="{{new_status}} == 'in-progress'">
|
|
246
|
+
<action>Update development_status[{{story_key}}] = "in-progress"</action>
|
|
247
|
+
<action>Update last_updated field to current date</action>
|
|
248
|
+
<action>Save file, preserving ALL comments and structure</action>
|
|
249
|
+
<output>đ Sprint status synced: {{story_key}} â in-progress</output>
|
|
250
|
+
</check>
|
|
251
|
+
|
|
252
|
+
<check if="story key not found in sprint status">
|
|
253
|
+
<output>â ī¸ Story file updated, but sprint-status sync failed: {{story_key}} not found in sprint-status.yaml</output>
|
|
254
|
+
</check>
|
|
255
|
+
</check>
|
|
256
|
+
|
|
257
|
+
<check if="{{current_sprint_status}} == 'no-sprint-tracking'">
|
|
258
|
+
<output>âšī¸ Story status updated (no sprint tracking configured)</output>
|
|
259
|
+
</check>
|
|
260
|
+
|
|
261
|
+
<output>**â
Review Complete!**
|
|
262
|
+
|
|
263
|
+
**Story Status:** {{new_status}}
|
|
264
|
+
**Issues Fixed:** {{fixed_count}}
|
|
265
|
+
**Action Items Created:** {{action_count}}
|
|
266
|
+
|
|
267
|
+
{{#if new_status == "done"}}Code review complete!{{else}}Address the action items and continue development.{{/if}}
|
|
268
|
+
</output>
|
|
269
|
+
</step>
|
|
270
|
+
|
|
271
|
+
</workflow>
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
# Change Navigation Checklist
|
|
2
2
|
|
|
3
|
-
<critical>This checklist is executed as part of: {project-root}/_bmad/bmm/workflows/4-implementation/correct-course/workflow.
|
|
3
|
+
<critical>This checklist is executed as part of: {project-root}/_bmad/bmm/workflows/4-implementation/correct-course/workflow.md</critical>
|
|
4
4
|
<critical>Work through each section systematically with the user, recording findings and impacts</critical>
|
|
5
5
|
|
|
6
6
|
<checklist>
|