feat(cli): cover hono in the sandbox and run its smoke in ci - #632
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
Thank you for following the naming conventions! 🙏 |
🔗 Linked issue
Stacked on #630 (itself on #629). Third layer of the Hono CLI stack.
📚 Description
scripts/cli-sandbox.mjsalready solves "test the CLI against real apps and put them back": the committed base is themapfixtures plus generators, the working copies live under the gitignored.sandbox/, and each app is its own git repo whose initial commit is the pristine state —--resetis acheckout+clean. This PR makes that workflow cover the new framework and makes it public instead of tribal:APPS(reusing thehono-basicfixture from feat(cli): add Hono adapter to evlog map #629), sopnpm cli:sandboxbuilds it and--smokedrives all 18 non-interactive checks against it — the full matrix passes at 18 × 5. One smoke assertion learned Hono'ssrc/evlog.tsshape.packages/clior the script, so a docs-only PR pays nothing. This is the end-to-end seam (detect → init → doctor → map → baseline) that would have caught the init crash the stack's base PR fixed.pnpm cli:sandboxline in the root AGENTS.md commands, and thecreate-map-ruleskill now lists the sandbox as an adapter touchpoint and uses it as the verification step.No changeset: nothing published changes.
📝 Checklist