[2.4.0 stack 6/7] Public API: root exports, types, 100% coverage and type-aware lint gates - #512
[2.4.0 stack 6/7] Public API: root exports, types, 100% coverage and type-aware lint gates#512hyanmandian wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughThe barrel export now exposes additional helpers and public types. API tests verify the complete runtime and type surface, duplicate-free exports, and non-throwing behavior for invalid inputs. Lint type checking and 100% coverage thresholds are enabled. ChangesPublic API expansion
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to The package-root API is expanded with matching runtime and type-surface checks, while linting and coverage validation are strengthened. No merge-blocking risk remains. 🚥 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 |
Tree-shaking reportFails when a pre-existing export grows more than 20% and more than 256 B, or when importing every export that already existed on the base grows more than 5%. New exports never count as a regression. Pre-existing exports: 202515 B to 202653 B (+0.1%, gzip 66986 B). Full import on head: 627889 B (gzip 162278 B).
New exports
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## stack/05-new-utils #512 +/- ##
=======================================================
+ Coverage 98.73% 100.00% +1.26%
=======================================================
Files 150 150
Lines 2135 2135
Branches 646 646
=======================================================
+ Hits 2108 2135 +27
+ Misses 3 0 -3
+ Partials 24 0 -24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
f4e1a60 to
e3c7bc0
Compare
e3c7bc0 to
827b077
Compare
827b077 to
7761122
Compare
Export State, StateCode, StateName, AddressInfo, CepProvider, BoletoInfo, GetBoletoInfoOptions, GenerateBoletoOptions, GenerateLicensePlateFormat, Holiday, HolidayType, PhoneMask, PhoneVersion, PhoneType, ParseCurrencyOptions, ParseCnpjOptions, IsValidCnpjOptions and every new util'"'"'s public types.
Turn on oxlint's type-aware rules with type checking and require 100% statement, branch, function and line coverage now that every source file meets both.
7761122 to
53c80ee
Compare
What does this PR do?
Part 6 of 7 of the 2.4.0 release stack (main <- stack/01-tooling <- ... <- stack/07-docs). The package-root exports (
src/index.ts, 124 exports plus 10 deprecated aliases kept for compatibility), the public type re-exports and the Bun/Deno test runtime shim refresh. Closes with the commit that turns on type-aware lint with type checking and the 100% coverage thresholds, now that the whole tree meets them.Commits in this part (2)
How to review and merge
mainand merge it with a merge commit: the 56 reviewed commits land unchanged and GitHub marks [2.4.0 stack 1/7] Tooling: vp pack, npm, release-please, tree-shaking CI diff, runtime shims #507 to [2.4.0 stack 6/7] Public API: root exports, types, 100% coverage and type-aware lint gates #512 as merged.CHANGELOG.md(part 7).Stack
Summary by CodeRabbit
New Features
Tests