Add environment-safe chart barrels - #8
Conversation
|
Warning Review limit reached
Next review available in: 21 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (27)
📝 WalkthroughWalkthroughThe package adds ChangesPortable package boundaries
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant PortableConsumer
participant PortableEntry
participant PackedConsumerChecks
PortableConsumer->>PortableEntry: Import portable runtime and types
PackedConsumerChecks->>PortableEntry: Resolve packed declarations and bundle entry
PackedConsumerChecks->>PackedConsumerChecks: Reject browser-host modules in portable output
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 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 |
ed7bcde to
059faf0
Compare
Summary
@tanstack/charts/portablefor common authoring, runtime, scene, and static SVG values without browser hosts or adapters@tanstack/charts/typesfor portable public contractsCompatibility
The existing
@tanstack/chartsruntime and type API is unchanged. Both new subpaths are additive, and no platform-specific export conditions are introduced.All 175 existing root exports remain present. The 12 relocated DOM types retain identical declarations and root exports, and transpiled JavaScript for every modified existing runtime module is byte-identical.
Unsupported deep imports from
@tanstack/charts/dist/typesare outside the package export map and are not preserved.The packed portable barrel is 55.27 kB minified / 17.04 kB gzip. Granular subpaths remain available for bundle-sensitive consumers.
Validation
pnpm run validate— all 17 Nx targets passpnpm changeset status