search

codecov / codecov-javascript-bundler-plugins

Set of bundler plugins that allow users to upload bundle stats to Codecov.
MIT License
4 stars 1 forks source link

ref: Bundler plugin core to just export helper utils #116

Closed nicholas-codecov closed 2 months ago

nicholas-codecov commented 2 months ago

Description

This PR refactors how we currently setup our plugins to a more composable way, and also exposes the unplugin plugins so we can create meta-framework specific plugins with ease.

Notable Changes

codecov-staging[bot] commented 2 months ago

Bundle Report

Changes will decrease total bundle size by 138.43kB :arrow_down:

Bundle name Size Change
@codecov/webpack-plugin-cjs 14.8kB 0 bytes
@codecov/bundler-plugin-core-es 6.29kB 140.69kB :arrow_down:
@codecov/webpack-plugin-es 904 bytes 0 bytes
@codecov/example-vite-app-esm 150.59kB 0 bytes
@codecov/example-next-app-server 325.59kB 0 bytes
@codecov/vite-plugin-cjs 13.66kB 1.53kB :arrow_up:
@codecov/rollup-plugin-es 794 bytes 0 bytes
@codecov/example-next-app-edge-server 306 bytes 0 bytes
@codecov/vite-plugin-es 864 bytes 0 bytes
@codecov/rollup-plugin-cjs 12.1kB 0 bytes
@codecov/example-webpack-app-array-push 71.19kB 0 bytes
@codecov/example-next-app-client 894.27kB 0 bytes
@codecov/example-rollup-app-iife 95.45kB 0 bytes
@codecov/bundler-plugin-core-cjs 151.97kB 724 bytes :arrow_up:
codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 25.19084% with 98 lines in your changes are missing coverage. Please review.

Project coverage is 78.80%. Comparing base (389e3e1) to head (46b68ca).

:white_check_mark: All tests successful. No failed tests found :relaxed:

Files Patch % Lines
packages/rollup-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
packages/vite-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
...c/vite-bundle-analysis/viteBundleAnalysisPlugin.ts 0.00% 17 Missing :warning:
packages/webpack-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
...llup-bundle-analysis/rollupBundleAnalysisPlugin.ts 0.00% 12 Missing :warning:
...ack-bundle-analysis/webpackBundleAnalysisPlugin.ts 0.00% 10 Missing and 2 partials :warning:
.../bundler-plugin-core/src/utils/checkNodeVersion.ts 0.00% 6 Missing :warning:
Additional details and impacted files | [Components](https://app.codecov.io/gh/codecov/codecov-javascript-bundler-plugins/pull/116/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=codecov) | Coverage Δ | | |---|---|---| | [Plugin core](https://app.codecov.io/gh/codecov/codecov-javascript-bundler-plugins/pull/116/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=codecov) | `97.22% <84.61%> (+1.98%)` | :arrow_up: | | [Rollup plugin](https://app.codecov.io/gh/codecov/codecov-javascript-bundler-plugins/pull/116/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=codecov) | `4.08% <0.00%> (-2.50%)` | :arrow_down: | | [Vite plugin](https://app.codecov.io/gh/codecov/codecov-javascript-bundler-plugins/pull/116/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=codecov) | `3.88% <0.00%> (-2.70%)` | :arrow_down: | | [Webpack plugin](https://app.codecov.io/gh/codecov/codecov-javascript-bundler-plugins/pull/116/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=codecov) | `24.40% <0.00%> (-6.07%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] commented 2 months ago

Bundle Report

Changes will decrease total bundle size by 1.18MB :arrow_down:

Bundle name Size Change
@codecov/vite-plugin-cjs 15.07kB 2.94kB :arrow_up:
@codecov/vite-plugin-es 1.08kB 10.69kB :arrow_down:
@codecov/example-rollup-app-iife 95.45kB 0 bytes
@codecov/example-webpack-app-array-push 71.19kB 0 bytes
@codecov/example-vite-app-esm 150.59kB 0 bytes
@codecov/example-next-app-server 325.59kB 0 bytes
@codecov/example-next-app-edge-server 306 bytes 0 bytes
@codecov/webpack-plugin-cjs 17.32kB 2.52kB :arrow_up:
@codecov/rollup-plugin-cjs 14.58kB 2.48kB :arrow_up:
@codecov/webpack-plugin-es 16.64kB 15.74kB :arrow_up:
@codecov/rollup-plugin-es 14.08kB 2.37kB :arrow_up:
@codecov/bundler-plugin-core-es (removed) 147.5kB :arrow_down:
@codecov/bundler-plugin-core-cjs (removed) 151.8kB :arrow_down:
@codecov/example-next-app-client (removed) 894.27kB :arrow_down:
codecov-staging[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 120 lines in your changes are missing coverage. Please review.

:white_check_mark: All tests successful. No failed tests found :relaxed:

Files Patch % Lines
packages/bundler-plugin-core/src/utils/Output.ts 0.00% 27 Missing and 1 partial :warning:
packages/rollup-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
packages/vite-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
packages/webpack-plugin/src/index.ts 0.00% 14 Missing and 3 partials :warning:
...c/vite-bundle-analysis/viteBundleAnalysisPlugin.ts 0.00% 15 Missing :warning:
...llup-bundle-analysis/rollupBundleAnalysisPlugin.ts 0.00% 9 Missing and 1 partial :warning:
...ack-bundle-analysis/webpackBundleAnalysisPlugin.ts 0.00% 10 Missing :warning:
.../bundler-plugin-core/src/utils/checkNodeVersion.ts 0.00% 5 Missing and 1 partial :warning:
Components Coverage Δ
Plugin core 93.25% <0.00%> (-2.00%) :arrow_down:
Rollup plugin 4.16% <0.00%> (-2.42%) :arrow_down:
Vite plugin 3.96% <0.00%> (-2.62%) :arrow_down:
Webpack plugin 24.80% <0.00%> (-5.68%) :arrow_down:

:loudspeaker: Thoughts on this report? Let us know!