[2.4.0 stack 6/7] Public API: root exports, types, 100% coverage and type-aware lint gates - #512
Open
hyanmandian wants to merge 2 commits into
Open
[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
hyanmandian wants to merge 2 commits into
Conversation
hyanmandian
added this pull request to stack #514
September 9, 2026 17:56
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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: 202519 B to 202769 B (+0.1%, gzip 67071 B). Full import on head: 628408 B (gzip 162544 B).
New exports
|
This was referenced Sep 9, 2026
Open
hyanmandian
changed the title
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.74% 100.00% +1.25%
=======================================================
Files 151 151
Lines 2154 2154
Branches 658 658
=======================================================
+ Hits 2127 2154 +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:
|
hyanmandian
force-pushed
the
stack/06-public-api
branch
from
September 9, 2026 18:33
f4e1a60 to
e3c7bc0
Compare
hyanmandian
force-pushed
the
stack/06-public-api
branch
from
September 9, 2026 18:58
e3c7bc0 to
827b077
Compare
hyanmandian
changed the title
hyanmandian
force-pushed
the
stack/06-public-api
branch
from
September 9, 2026 21:52
827b077 to
7761122
Compare
hyanmandian
force-pushed
the
stack/06-public-api
branch
from
September 9, 2026 22:07
7761122 to
53c80ee
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.
hyanmandian
force-pushed
the
stack/06-public-api
branch
from
September 11, 2026 20:34
53c80ee to
c6d1290
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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