Conversation
Register all sample skills as a sample-plugins plugin and the dream weaver as a standalone plugin for installer testing.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughTwo new plugin entries were added to Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
✨ 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 |
🤖 Augment PR SummarySummary: Adds two new entries to Why: Provides stable, known manifest targets for installer/integration testing without changing the core builder plugin. 🤖 Was this summary useful? React with 👍 or 👎 |
| }, | ||
| { | ||
| "name": "bmad-dream-weaver-agent", | ||
| "source": "./samples/bmad-agent-dream-weaver", |
There was a problem hiding this comment.
.claude-plugin/marketplace.json:45: The manifest docs describe plugins[].source as the parent directory for the skill folders, but here it points at the skill directory itself; if any consumer combines source + skills for resolution, this can produce incorrect paths (e.g., double-nesting).
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Summary
sample-pluginsplugin entry with all 6 sample skillsbmad-dream-weaver-agentplugin entryTest plan
Summary by CodeRabbit