fix: clarify onboarding across tool syntax, workflows, and delivery#962
fix: clarify onboarding across tool syntax, workflows, and delivery#962alfred-openspec wants to merge 1 commit intomainfrom
Conversation
|
Task completed. I'll review this PR by examining the changes. Let me start by understanding the diff and the relevant files. Powered by 1Code |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
Summary\n- make onboarding guidance tool-aware, profile-aware, and delivery-aware across init, update, and migration flows\n- fix first-run docs and welcome copy so the default core workflow is clear and command syntax is explained upfront\n- add tests for the new onboarding guidance helper and updated CLI messaging\n\n## Why\nOPE-20 is an onboarding trust issue across supported integrations, not just an OpenCode docs mismatch. This PR makes the default workflow set, command syntax differences, and skills-vs-commands delivery model explicit so installs feel correct instead of flaky.\n\n## Testing\n- pnpm install\n- pnpm test -- test/core/onboarding-guidance.test.ts test/core/init.test.ts test/core/update.test.ts test/core/migration.test.ts\n- pnpm build