fix(native): resolve platform runtime dependencies - #249
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughPi and OpenCode Rolldown configurations now externalize Caplets core runtime modules. A Changeset records patch releases and documents loading Caplets core through its declared package exports. ChangesRuntime externalization
Estimated code review effort: 1 (Trivial) | ~5 minutes 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 |
|
| Filename | Overview |
|---|---|
| packages/opencode/rolldown.config.ts | Externalizes both core subpath imports used by OpenCode; the core package is a runtime dependency and publishes both exports. |
| packages/pi/rolldown.config.ts | Externalizes both core subpath imports used by Pi; the core package is a runtime dependency and publishes both exports. |
| .changeset/fix-native-core-runtime.md | Correctly records patch releases for both affected native integration packages. |
Reviews (1): Last reviewed commit: "fix(native): externalize core runtime im..." | Re-trigger Greptile
Summary
@caplets/coreruntime exports from the Pi bundleProblem
Bundling Caplets core into the native integration artifacts moved libSQL's dynamic native-module load into each integration bundle. Node then resolved from the integration package root and could not find
@libsql/linux-x64-gnu, preventing Pi and OpenCode from loading.Verification
pistarts with the Caplets extension enabled (caplets ✓)pnpm --filter @caplets/pi test(47 passed)pnpm --filter @caplets/pi typecheckpnpm --filter @caplets/opencode test(10 passed)pnpm --filter @caplets/opencode typecheckpnpm verifypassedSummary by CodeRabbit
Bug Fixes
Chores