refactor(code-review): remove unused validation path and checklist
This commit is contained in:
parent
a8758b0393
commit
eae4ad46a1
|
|
@ -1,23 +0,0 @@
|
||||||
# Senior Developer Review - Validation Checklist
|
|
||||||
|
|
||||||
- [ ] Story file loaded from `{{story_path}}`
|
|
||||||
- [ ] Story Status verified as reviewable (review)
|
|
||||||
- [ ] Epic and Story IDs resolved ({{epic_num}}.{{story_num}})
|
|
||||||
- [ ] Story Context located or warning recorded
|
|
||||||
- [ ] Epic Tech Spec located or warning recorded
|
|
||||||
- [ ] Architecture/standards docs loaded (as available)
|
|
||||||
- [ ] Tech stack detected and documented
|
|
||||||
- [ ] MCP doc search performed (or web fallback) and references captured
|
|
||||||
- [ ] Acceptance Criteria cross-checked against implementation
|
|
||||||
- [ ] File List reviewed and validated for completeness
|
|
||||||
- [ ] Tests identified and mapped to ACs; gaps noted
|
|
||||||
- [ ] Code quality review performed on changed files
|
|
||||||
- [ ] Security review performed on changed files and dependencies
|
|
||||||
- [ ] Outcome decided (Approve/Changes Requested/Blocked)
|
|
||||||
- [ ] Review notes appended under "Senior Developer Review (AI)"
|
|
||||||
- [ ] Change Log updated with review entry
|
|
||||||
- [ ] Status updated according to settings (if enabled)
|
|
||||||
- [ ] Sprint status synced (if sprint tracking enabled)
|
|
||||||
- [ ] Story saved successfully
|
|
||||||
|
|
||||||
_Reviewer: {{user_name}} on {{date}}_
|
|
||||||
|
|
@ -30,7 +30,6 @@ Load config from `{project-root}/_bmad/bmm/config.yaml` and resolve:
|
||||||
- `installed_path` = `{project-root}/_bmad/bmm/workflows/4-implementation/code-review`
|
- `installed_path` = `{project-root}/_bmad/bmm/workflows/4-implementation/code-review`
|
||||||
- `project_context` = `**/project-context.md` (load if exists)
|
- `project_context` = `**/project-context.md` (load if exists)
|
||||||
- `sprint_status` = `{implementation_artifacts}/sprint-status.yaml`
|
- `sprint_status` = `{implementation_artifacts}/sprint-status.yaml`
|
||||||
- `validation` = `{installed_path}/checklist.md`
|
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue