Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #68 +/- ##
==========================================
+ Coverage 97.68% 98.68% +0.99%
==========================================
Files 8 8
Lines 303 304 +1
==========================================
+ Hits 296 300 +4
+ Misses 7 4 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
This PR modernizes the bundle-phobia-cli package for 2025, updating its infrastructure, improving error handling, and adding support for newer Node.js versions. The update includes migrating from Travis CI to GitHub Actions, bumping dependencies, and implementing error resilience where processing continues despite individual package failures.
Key changes:
- Consolidated entry points from
index.jsandindex-install.jsinto a singlemain.jswith command structure - Added error resilience through
fail-fastflag andstopOnErrorconfiguration - Implemented filtering of
@types/*packages from dependency lists - Updated CI to test against Node.js versions 16, 18, 20, 22, and 24
Reviewed Changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| main.js | New unified entry point consolidating stats and install commands with yargs command structure |
| index.js, index-install.js | Removed separate entry points in favor of unified main.js |
| src/npm-utils.js | Added filtering to exclude @types packages from dependency lists |
| src/core.js | Added fail-fast flag support and serial execution option for package fetching |
| src/install.js | Updated to use packages property from argv and added fail-fast support |
| test/integration/*.test.js | Updated tests to use new argv structure and normalize KB size values for stability |
| test/integration/utils.js | Added stripKb utility for normalizing size outputs in tests |
| test/e2e/*.sh | Updated shell scripts to reference main.js and normalize KB outputs |
| package.json | Updated dependencies and Node.js engine requirement to >=16.13 |
| .github/workflows/ci.yml | New GitHub Actions workflow replacing Travis CI |
| .travis.yml | Deprecated in favor of GitHub Actions |
| README.md | Updated documentation to reflect new command structure |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0d75d6e to
edc8cd8
Compare
Long landing refresh of the package