feat: add Ona as a supported platform#1968
Conversation
Add Ona (ona.com) to the BMAD installer so users can select it during `npx bmad-method install` or via `--tools ona`. Skills are installed to `.ona/skills/<name>/SKILL.md` using the default templates. Fixes bmad-code-org#1967 Co-authored-by: Ona <no-reply@ona.com>
📝 WalkthroughWalkthroughThe PR adds Ona as a supported IDE platform in the BMAD installer system, configuring it to generate Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip You can disable sequence diagrams in the walkthrough.Disable the |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@tools/platform-codes.yaml`:
- Around line 130-134: The YAML entry for the `ona` platform currently sets
category: ide but the PR intends `category: cli`; update the `ona` entry's
category field from "ide" to "cli" (i.e., change the value under the `ona:`
block) and confirm with the product owner if the category should instead remain
"ide" before committing; ensure the changed symbol is the `ona` mapping so it
matches other terminal tools like `auggie`, `claude-code`, and `gemini`.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: baacce64-7bf1-4670-abab-53a1c0293bf4
📒 Files selected for processing (3)
test/test-installation-components.jstools/cli/installers/lib/ide/platform-codes.yamltools/platform-codes.yaml
|
Re: CodeRabbit's comment on |
What
Add Ona as a supported platform in the BMAD installer.
Why
Ona is an AI development environment that supports repository-level skills via
.ona/skills/<name>/SKILL.mdfiles. The skill format (YAML frontmatter withnameanddescription, followed by Markdown content) aligns with BMAD's existingskill_format: trueoutput.Changes
tools/cli/installers/lib/ide/platform-codes.yaml— Addonaplatform entry with installer config (target_dir: .ona/skills,template_type: default,skill_format: true)tools/platform-codes.yaml— Addonametadata entry to keep both config files in synctest/test-installation-components.js— Add Test Suite 32 with 16 assertions covering config validation, install, detection, frontmatter format, and reinstallNo JavaScript changes needed —
ConfigDrivenIdeSetupandIdeManagerpick up the new entry automatically.Fixes #1967