fix: add missing skill manifests for research and PRD workflows#1839
fix: add missing skill manifests for research and PRD workflows#1839alexeyv merged 1 commit intobmad-code-org:mainfrom
Conversation
These manifests were missed during the all-is-skills migration (bmad-code-org#1834), leaving 6 workflows undiscoverable by the native skills installer. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
🤖 Augment PR SummarySummary: Adds missing skill manifest sidecars so the BMM research and PRD workflow commands are surfaced by native skills installers. 🤖 Was this summary useful? React with 👍 or 👎 |
📝 WalkthroughWalkthroughTwo new YAML manifest files are added to catalog workflow definitions. The first manifest registers three research workflows (domain, market, and technical research), and the second registers three PRD-related workflows (create, edit, and validate). Each entry maps markdown workflow files to canonical IDs, types, and descriptions. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yaml (1)
14-14: Missing trailing newline at end of file.Same issue as the PRD manifest — add a trailing newline for POSIX compliance.
♻️ Add trailing newline
workflow-technical-research.md: canonicalId: bmad-bmm-technical-research type: workflow description: "Conduct technical research on technologies and architecture" +🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yaml` at line 14, The YAML manifest file bmad-skill-manifest.yaml is missing a trailing newline (POSIX requires a final newline); open the file (look for the description: "Conduct technical research on technologies and architecture" entry) and add a single newline character at the end of the file so the file ends with a blank line.src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml (1)
14-14: Missing trailing newline at end of file.POSIX convention and many tools expect files to end with a newline. This is a minor hygiene issue but can cause spurious diffs or tool warnings.
♻️ Add trailing newline
workflow-validate-prd.md: canonicalId: bmad-bmm-validate-prd type: workflow description: "Validate a PRD against standards" +🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml` at line 14, The file ending for the bmad-skill-manifest.yaml is missing a trailing newline (the line containing description: "Validate a PRD against standards") — add a single newline character at the end of the file so the manifest ends with a newline to satisfy POSIX conventions and avoid tool/diff warnings.
🤖 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/1-analysis/research/bmad-skill-manifest.yaml`:
- Line 14: The YAML manifest file bmad-skill-manifest.yaml is missing a trailing
newline (POSIX requires a final newline); open the file (look for the
description: "Conduct technical research on technologies and architecture"
entry) and add a single newline character at the end of the file so the file
ends with a blank line.
In `@src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml`:
- Line 14: The file ending for the bmad-skill-manifest.yaml is missing a
trailing newline (the line containing description: "Validate a PRD against
standards") — add a single newline character at the end of the file so the
manifest ends with a newline to satisfy POSIX conventions and avoid tool/diff
warnings.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: aa82d3d6-63c7-439a-8d8b-5c84774c0e77
📒 Files selected for processing (2)
src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yamlsrc/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml
Summary
bmad-skill-manifest.yamlforsrc/bmm/workflows/1-analysis/research/(3 research workflows)bmad-skill-manifest.yamlforsrc/bmm/workflows/2-plan-workflows/create-prd/(3 PRD workflows)Test plan
bmad-cli installand verify research and PRD workflows appear as native skills🤖 Generated with Claude Code