Repository navigation
test: verify tutorial package consumption - #139
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe package workflow now runs an isolated tutorial package smoke test after packaging. The smoke test validates public imports, runtime behavior, dependency metadata, and CSS exports. New graph presets cover tutorial core and driver flows. ChangesTutorial package verification
Tutorial graph requests
Estimated code review effort: 3 (Moderate) | ~20 minutes Mergeability Score: ⚪ Minimal · up to This localized tooling and test-validation change has no actionable merge-blocking risk remaining after normal checks and review. Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 156715c672
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
tutorial-coreandtutorial-driverttsc graph presets and include them ingraph:preflight@mpgd/tutorialfrom its pnpm tarball in an empty offline storepack:packages:preparedso release packaging uses the same boundary checkThis PR should merge before release PR #138 so the release branch can be regenerated from the validated
mainstate.Validation
pnpm graph:tutorialpnpm graph:preflightmise exec -- pnpm pack:packages:preparedmise exec -- pnpm checkgit diff --checkChangeset
No Sampo changeset: this is tooling and test coverage only and does not change a publishable package API, runtime, metadata, or integration contract.
Summary by CodeRabbit
Tests
Tooling