Skip to content

fix: add missing skill manifests for research and PRD workflows#1839

Merged
alexeyv merged 1 commit intobmad-code-org:mainfrom
alexeyv:fix/missing-skill-manifests
Mar 7, 2026
Merged

fix: add missing skill manifests for research and PRD workflows#1839
alexeyv merged 1 commit intobmad-code-org:mainfrom
alexeyv:fix/missing-skill-manifests

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Mar 7, 2026

Summary

  • Adds bmad-skill-manifest.yaml for src/bmm/workflows/1-analysis/research/ (3 research workflows)
  • Adds bmad-skill-manifest.yaml for src/bmm/workflows/2-plan-workflows/create-prd/ (3 PRD workflows)
  • These were missed during the all-is-skills migration (refactor: all-is-skills - Convert BMAD to skills-based architecture #1834), leaving 6 workflows undiscoverable by the native skills installer

Test plan

  • Run bmad-cli install and verify research and PRD workflows appear as native skills
  • Confirm no regressions in other skill manifest discovery

🤖 Generated with Claude Code

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>
@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Mar 7, 2026

🤖 Augment PR Summary

Summary: Adds missing skill manifest sidecars so the BMM research and PRD workflow commands are surfaced by native skills installers.
Changes: Introduces multi-entry bmad-skill-manifest.yaml files mapping six workflow markdown files to canonical IDs and descriptions.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 7, 2026

📝 Walkthrough

Walkthrough

Two 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

Cohort / File(s) Summary
Workflow Manifests
src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yaml, src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml
Adds YAML manifest files cataloging workflow definitions with canonical IDs, types, and descriptions for research workflows and PRD creation workflows.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • bmad-code-org/BMAD-METHOD#1619: Related through the validate-prd workflow entry, which is both manifested in this PR and modified in that PR's auto-discovery implementation.

Suggested reviewers

  • bmadcode
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding missing skill manifests for research and PRD workflows that were overlooked during a migration.
Description check ✅ Passed The description is directly related to the changeset, explaining what manifests were added, why they were needed, and providing a clear test plan.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 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

📥 Commits

Reviewing files that changed from the base of the PR and between 0d3b317 and b6affda.

📒 Files selected for processing (2)
  • src/bmm/workflows/1-analysis/research/bmad-skill-manifest.yaml
  • src/bmm/workflows/2-plan-workflows/create-prd/bmad-skill-manifest.yaml

@alexeyv alexeyv merged commit 434e7ef into bmad-code-org:main Mar 7, 2026
7 checks passed
@alexeyv alexeyv deleted the fix/missing-skill-manifests branch March 7, 2026 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant