Repository navigation
Install BMad Method v6.2.0 with Claude Code integration - #1
Merged
Merged
Conversation
Sets up BMad Method (Agile AI-Driven Development framework) with the BMM module, 36 skills and 9 agents configured for Claude Code. https://claude.ai/code/session_01VY2xiQ7rg51abGxTBCfcVj
There was a problem hiding this comment.
Pull request overview
Installs the BMAD Method v6.2.0 “skills” bundle for Claude Code by adding a large set of workflow/step markdowns, templates, and supporting CSV/JSON schema files under .claude/skills/.
Changes:
- Added multiple BMAD skills (document-project, PRD creation, architecture, epics/stories, code review, readiness checks, brainstorming, etc.) with step-file workflow instructions.
- Added templates/schemas/resources (Handlebars-style markdown templates, JSON schema, CSV registries) to support generation and resumability.
- Added distillation agents/resources for producing and validating “distillate” outputs.
Reviewed changes
Copilot reviewed 148 out of 656 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| .claude/skills/bmad-document-project/workflows/deep-dive-workflow.md | Adds deep-dive documentation sub-workflow routing/instructions. |
| .claude/skills/bmad-document-project/workflow.md | Adds top-level document-project workflow definition. |
| .claude/skills/bmad-document-project/templates/source-tree-template.md | Adds source tree analysis output template. |
| .claude/skills/bmad-document-project/templates/project-scan-report-schema.json | Adds JSON schema for resumable scan state tracking. |
| .claude/skills/bmad-document-project/templates/project-overview-template.md | Adds project overview output template. |
| .claude/skills/bmad-document-project/templates/index-template.md | Adds documentation index output template. |
| .claude/skills/bmad-document-project/instructions.md | Adds router logic for resume/full/deep-dive scan modes. |
| .claude/skills/bmad-document-project/bmad-skill-manifest.yaml | Adds skill manifest stub for document-project. |
| .claude/skills/bmad-document-project/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-distillator/resources/splitting-strategy.md | Adds semantic splitting guidance for distillation. |
| .claude/skills/bmad-distillator/resources/compression-rules.md | Adds compression/deduplication rules for distillation. |
| .claude/skills/bmad-distillator/bmad-skill-manifest.yaml | Adds distillator skill manifest with capabilities section. |
| .claude/skills/bmad-distillator/agents/round-trip-reconstructor.md | Adds reconstructor agent spec for completeness checking. |
| .claude/skills/bmad-distillator/agents/distillate-compressor.md | Adds compressor agent spec and return contract. |
| .claude/skills/bmad-dev/SKILL.md | Adds dev agent activation wrapper for BMAD dev agent. |
| .claude/skills/bmad-dev-story/checklist.md | Adds enhanced DoD checklist for dev-story execution. |
| .claude/skills/bmad-dev-story/bmad-skill-manifest.yaml | Adds skill manifest stub for dev-story. |
| .claude/skills/bmad-dev-story/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-ux-design/workflow.md | Adds UX design workflow definition and initialization rules. |
| .claude/skills/bmad-create-ux-design/ux-design-template.md | Adds UX design spec document template. |
| .claude/skills/bmad-create-ux-design/steps/step-14-complete.md | Adds UX workflow completion step instructions. |
| .claude/skills/bmad-create-ux-design/steps/step-02-discovery.md | Adds UX project understanding/discovery step instructions. |
| .claude/skills/bmad-create-ux-design/steps/step-01b-continue.md | Adds UX continuation/resume step instructions. |
| .claude/skills/bmad-create-ux-design/steps/step-01-init.md | Adds UX workflow initialization + input discovery rules. |
| .claude/skills/bmad-create-ux-design/bmad-skill-manifest.yaml | Adds skill manifest stub for create-ux-design. |
| .claude/skills/bmad-create-ux-design/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-story/template.md | Adds story file template for later dev execution. |
| .claude/skills/bmad-create-story/discover-inputs.md | Adds “smart discovery” protocol for loading inputs. |
| .claude/skills/bmad-create-story/bmad-skill-manifest.yaml | Adds skill manifest stub for create-story. |
| .claude/skills/bmad-create-story/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-product-brief/workflow.md | Adds product brief workflow definition. |
| .claude/skills/bmad-create-product-brief/steps/step-06-complete.md | Adds product brief completion step. |
| .claude/skills/bmad-create-product-brief/steps/step-04-metrics.md | Adds success metrics definition step. |
| .claude/skills/bmad-create-product-brief/steps/step-03-users.md | Adds target users/journey step. |
| .claude/skills/bmad-create-product-brief/steps/step-02-vision.md | Adds product vision discovery step. |
| .claude/skills/bmad-create-product-brief/steps/step-01b-continue.md | Adds continuation handler for product brief workflow. |
| .claude/skills/bmad-create-product-brief/steps/step-01-init.md | Adds initialization + input discovery rules for product brief. |
| .claude/skills/bmad-create-product-brief/product-brief.template.md | Adds product brief output document template. |
| .claude/skills/bmad-create-product-brief/bmad-skill-manifest.yaml | Adds skill manifest stub for create-product-brief. |
| .claude/skills/bmad-create-product-brief/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-prd/workflow.md | Adds PRD creation workflow definition (create mode). |
| .claude/skills/bmad-create-prd/templates/prd-template.md | Adds PRD output template. |
| .claude/skills/bmad-create-prd/steps-c/step-12-complete.md | Adds PRD completion step. |
| .claude/skills/bmad-create-prd/steps-c/step-05-domain.md | Adds optional domain-specific requirements step. |
| .claude/skills/bmad-create-prd/steps-c/step-02c-executive-summary.md | Adds executive summary generation step. |
| .claude/skills/bmad-create-prd/steps-c/step-02b-vision.md | Adds PRD “vision discovery” step. |
| .claude/skills/bmad-create-prd/steps-c/step-01b-continue.md | Adds PRD continuation handler. |
| .claude/skills/bmad-create-prd/steps-c/step-01-init.md | Adds PRD initialization + input discovery step. |
| .claude/skills/bmad-create-prd/data/project-types.csv | Adds project type detection and required sections registry. |
| .claude/skills/bmad-create-prd/data/prd-purpose.md | Adds PRD purpose/standards guidance document. |
| .claude/skills/bmad-create-prd/data/domain-complexity.csv | Adds domain complexity registry for optional domain step. |
| .claude/skills/bmad-create-prd/bmad-skill-manifest.yaml | Adds skill manifest stub for create-prd. |
| .claude/skills/bmad-create-prd/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-epics-and-stories/workflow.md | Adds epics/stories workflow definition. |
| .claude/skills/bmad-create-epics-and-stories/templates/epics-template.md | Adds epics/stories output template. |
| .claude/skills/bmad-create-epics-and-stories/steps/step-04-final-validation.md | Adds final validation step for epic/story quality. |
| .claude/skills/bmad-create-epics-and-stories/bmad-skill-manifest.yaml | Adds skill manifest stub for create-epics-and-stories. |
| .claude/skills/bmad-create-epics-and-stories/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-create-architecture/workflow.md | Adds architecture workflow definition. |
| .claude/skills/bmad-create-architecture/steps/step-08-complete.md | Adds architecture completion/handoff step. |
| .claude/skills/bmad-create-architecture/steps/step-01b-continue.md | Adds architecture continuation handler. |
| .claude/skills/bmad-create-architecture/steps/step-01-init.md | Adds architecture initialization + input discovery step. |
| .claude/skills/bmad-create-architecture/data/project-types.csv | Adds architecture-side project type registry. |
| .claude/skills/bmad-create-architecture/data/domain-complexity.csv | Adds architecture-side domain complexity hints. |
| .claude/skills/bmad-create-architecture/bmad-skill-manifest.yaml | Adds skill manifest stub for create-architecture. |
| .claude/skills/bmad-create-architecture/architecture-decision-template.md | Adds architecture decision doc template. |
| .claude/skills/bmad-create-architecture/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-correct-course/bmad-skill-manifest.yaml | Adds skill manifest stub for correct-course. |
| .claude/skills/bmad-correct-course/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-code-review/workflow.md | Adds code review workflow definition + step-based process. |
| .claude/skills/bmad-code-review/steps/step-04-present.md | Adds “present findings” step. |
| .claude/skills/bmad-code-review/steps/step-03-triage.md | Adds triage/normalization step for reviewer outputs. |
| .claude/skills/bmad-code-review/steps/step-02-review.md | Adds parallel reviewer launch step definition. |
| .claude/skills/bmad-code-review/steps/step-01-gather-context.md | Adds diff/spec selection and context gathering step. |
| .claude/skills/bmad-code-review/bmad-skill-manifest.yaml | Adds skill manifest stub for code-review. |
| .claude/skills/bmad-code-review/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-check-implementation-readiness/workflow.md | Adds readiness validation workflow definition. |
| .claude/skills/bmad-check-implementation-readiness/templates/readiness-report-template.md | Adds readiness report template. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-06-final-assessment.md | Adds final assessment/reporting step. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-05-epic-quality-review.md | Adds epic/story quality enforcement step. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-04-ux-alignment.md | Adds UX alignment check step. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-03-epic-coverage-validation.md | Adds FR coverage validation step. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-02-prd-analysis.md | Adds PRD requirement extraction step. |
| .claude/skills/bmad-check-implementation-readiness/steps/step-01-document-discovery.md | Adds document inventory + conflict detection step. |
| .claude/skills/bmad-check-implementation-readiness/bmad-skill-manifest.yaml | Adds skill manifest stub for readiness check. |
| .claude/skills/bmad-check-implementation-readiness/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-brainstorming/workflow.md | Adds brainstorming workflow definition. |
| .claude/skills/bmad-brainstorming/template.md | Adds brainstorming session output template. |
| .claude/skills/bmad-brainstorming/steps/step-02c-random-selection.md | Adds random technique selection step. |
| .claude/skills/bmad-brainstorming/steps/step-01b-continue.md | Adds brainstorming continuation handler. |
| .claude/skills/bmad-brainstorming/bmad-skill-manifest.yaml | Adds skill manifest stub for brainstorming. |
| .claude/skills/bmad-brainstorming/SKILL.md | Adds skill metadata + pointer to workflow.md. |
| .claude/skills/bmad-architect/SKILL.md | Adds architect agent activation wrapper. |
| .claude/skills/bmad-analyst/SKILL.md | Adds analyst agent activation wrapper. |
| .claude/skills/bmad-advanced-elicitation/workflow.md | Adds advanced elicitation workflow definition. |
| .claude/skills/bmad-advanced-elicitation/methods.csv | Adds elicitation method registry for selection/execution. |
| .claude/skills/bmad-advanced-elicitation/bmad-skill-manifest.yaml | Adds skill manifest stub for advanced-elicitation. |
| .claude/skills/bmad-advanced-elicitation/SKILL.md | Adds skill metadata + pointer to workflow.md. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+1
to
+3
| --- | ||
| --- | ||
|
|
There was a problem hiding this comment.
Same issue as step-04-present.md: the file begins with two consecutive --- lines and no YAML content. This can break frontmatter parsing and downstream tooling; replace with a valid single frontmatter block or remove the delimiters.
Suggested change
| --- | |
| --- |
This was referenced Jul 1, 2026
Open
don-petry
added a commit
that referenced
this pull request
Oct 11, 2026
…to-review - Use explicit .length == 1 check instead of !pull_requests[1] to robustly detect multiple PRs (fixes issue #1) - Add head_commit.id == github.sha check to prevent delayed old events from cancelling newer runs (fixes issue #2) - Both fixes apply to check_suite and workflow_run event conditions Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
don-petry
added a commit
that referenced
this pull request
Oct 11, 2026
…eview.yml-concurrency (#521) * feat: implement issue #508 — Compliance: stub-surface-drift-pr-auto-review.yml-concurrency * chore: dev-lead update (review-changes) [skip ci-relay] * fix(bot): improve docstring coverage in test-pr-auto-review-workflow.test.sh Add single-line docstrings to 11 shell functions to improve docstring coverage from 63.64% to above 80% threshold, as required by CodeRabbit's docstring coverage check. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(bot): improve docstring coverage and validation logic in workflow guard scripts - Add docstrings to 11 test helper functions (63.64% → 80% coverage) - Improve concurrency.group validation checks (3, 4, 5) to verify complete, properly guarded expression branches using grep pattern matching, rather than accepting required references found in unrelated text - Use grep to check for check_suite/workflow_run patterns with both PR number and head_sha in the same event block, ensuring commit-scoped isolation - All guard checks and test fixtures continue to pass Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(reviews): validate full cancel-in-progress expression in pr-auto-review guard Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(bot): improve multi-PR detection and race condition prevention in pr-auto-review - Changed array existence check from `!pull_requests[1]` to explicit `(pull_requests[1] == null)` for more reliable multi-PR detection - Fixed cancel-in-progress to only cancel when using head_sha-based groups (single PR case), preventing delayed completion events from canceling unrelated runs - These fixes prevent runs from different PRs sharing the same cancellable group when a commit belongs to multiple PRs Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(reviews): accept single-PR cancel-in-progress expression in pr-auto-review guard Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(bot): add explicit null-safety checks in pr-auto-review concurrency expressions - Added explicit checks for pull_requests[0] existence before accessing .number property - This prevents potential issues when accessing array elements that may not exist - Updated validation script to accept the more defensive null-safety form - These changes address CodeAnt's concerns about multi-PR detection logic Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(bot): restore pr-auto-review concurrency block to match canonical template Replace custom head_sha and multi-PR logic with the canonical template's simpler PR-number-based grouping and event-conditional cancellation per issue #508. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(reviews): align pr-auto-review guard with canonical PR-number-only concurrency The stub is synced verbatim from the org template (PR-number-keyed group), so the guard now accepts it instead of requiring head_sha / pull_requests[1]. Checks 3-4 are scoped per event branch. Test-Change-Justification: the canonical stub restored in ff1f0ef is PR-number-only; the old Case 3/4 rejected exactly that policy, which made CI fail (review threads PRRT_kwDORsnIRM6rJpgj, PRRT_kwDORsnIRM6rKUi4 on PR #521, #521) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): require PR number in format() args of each concurrency branch Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): require PR number as {0} arg and run_id as terminal fallback in concurrency guard Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): reject run-unique values in per-PR concurrency branches Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): harden pr-auto-review concurrency guard Require PR-number gating per branch, shared group format, ${{ }} delimiters, and reject event-specific ids in per-PR keys. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): tighten pr-auto-review concurrency guard checks Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(reviews): reject escaped fallback placeholder and unwrapped cancel-in-progress in guard Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(bot): add multi-PR guards to pr-auto-review concurrency conditions Add explicit checks to reject multi-PR events from the per-PR concurrency groups, ensuring only single-PR events use PR-number deduplication while multi-PR and no-PR events fall back to unique run_id groups. Fixes: codeant-ai concern about concurrent events with multiple PRs potentially suppressing review dispatch for unrelated PRs. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(bot): resolve concurrency guard logic and race condition in pr-auto-review - Use explicit .length == 1 check instead of !pull_requests[1] to robustly detect multiple PRs (fixes issue #1) - Add head_commit.id == github.sha check to prevent delayed old events from cancelling newer runs (fixes issue #2) - Both fixes apply to check_suite and workflow_run event conditions Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(reviews): gate pr-auto-review concurrency on pull_requests[0].number Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> * fix(compliance): restore canonical pr-auto-review concurrency block Issue #508 asks for the concurrency surface to be re-synced verbatim from petry-projects/.github standards/workflows/pr-auto-review.yml. Earlier review passes added a pull_requests[1] guard to the check_suite and workflow_run branches of the group expression, which the compliance audit's stub_surface_drift comparison still reports as drift. Drop the two guards so the block matches the canonical template; a multi-PR guard, if wanted, belongs in the template. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U1bsmUpeti2XUHnhQxWHU9 * test(compliance): key the event-name mutation on the canonical group text The self-test's "check_suite branch not gated on its event name" mutation matched the pull_requests[1] guard that the previous commit removed, so the sed no longer applied and the case failed with "mutation did not apply". Match the canonical line instead; the guard still rejects the mutant. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U1bsmUpeti2XUHnhQxWHU9 * fix(bot): add multi-PR guard to prevent cross-PR group collisions Strengthen the concurrency control by adding !pull_requests[1] checks to ensure that events affecting multiple PRs fall back to the run-unique group instead of mixing work across PRs. Update the regression guard to enforce this constraint and add a test case verifying rejection of configurations without the multi-PR guard. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(compliance): restore canonical pr-auto-review concurrency block (again) Reverts ec9ff01, which re-added the pull_requests[1] guard to the group expression after a reviewer-bot comment. Issue #508 requires the concurrency surface to match petry-projects/.github standards/workflows/pr-auto-review.yml verbatim; with the guard the compliance audit's stub_surface_drift comparison still reports drift. The tree is identical to 28d9085, on which all required checks and the workflow regression guards passed. The maintainer has chosen to merge this PR by hand; it carries dev-lead:hands-off so the guard is not added back. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U1bsmUpeti2XUHnhQxWHU9 --------- Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Installing BMAD method setup.