Skip to content

chore(react-icons): add bundle-size metrics,gates#877

Merged
Hotell merged 12 commits intomicrosoft:mainfrom
Hotell:react-icons/bundle-size-check
Jan 14, 2026
Merged

chore(react-icons): add bundle-size metrics,gates#877
Hotell merged 12 commits intomicrosoft:mainfrom
Hotell:react-icons/bundle-size-check

Conversation

@Hotell
Copy link
Copy Markdown
Collaborator

@Hotell Hotell commented Aug 4, 2025

Adds bundle size measurement and reporting for monorepo via monosize tool.

  1. new PR workflow step
  • runs monosize measure for existing fixtures
  • runs monosize compare-reports comparing repo stored storage vs current PR measurement
image
  1. new baseline workflow
  • runs on merge to main if react-icons or assets have been modified in order to update baseline metrics
  1. implements custom GitHub createLocalStorage monosize adapter which operates on physical repo files
  • compares PR against baseline stored in default branch
  • stores baseline via default branch main pipeline to repo
  1. nx
  • setups cache for build target on react-icons

@Hotell Hotell changed the title React icons/bundle size check chore(react-icons): add bundle-size safe guards Aug 4, 2025
@Hotell Hotell marked this pull request as ready for review August 5, 2025 10:21
@Hotell Hotell requested review from a team as code owners August 5, 2025 10:21
@Hotell Hotell requested a review from dmytrokirpa August 5, 2025 12:07
dmytrokirpa
dmytrokirpa previously approved these changes Aug 6, 2025
@Hotell Hotell force-pushed the react-icons/bundle-size-check branch from 3154793 to 41b98d6 Compare August 7, 2025 09:10
@Hotell Hotell force-pushed the react-icons/bundle-size-check branch from 41b98d6 to 88595e5 Compare January 13, 2026 10:23
* Creates a local storage adapter for Monosize.
* This adapter reads and writes bundle size reports to a local JSON file.
* It is used to store the bundle size reports locally for the package.
* TODO: needs to be reworked to leverage GHA artifacts because pushing from CI to protected branch is not allowed.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should no longer be the case, will remove once validated after merge

@Hotell Hotell changed the title chore(react-icons): add bundle-size safe guards chore(react-icons): add bundle-size metrics,gates Jan 13, 2026
@Hotell Hotell closed this Jan 13, 2026
@Hotell Hotell reopened this Jan 13, 2026
@Hotell Hotell force-pushed the react-icons/bundle-size-check branch from f8f05e2 to e75c684 Compare January 14, 2026 12:09
@Hotell Hotell requested a review from dmytrokirpa January 14, 2026 14:12
@Hotell
Copy link
Copy Markdown
Collaborator Author

Hotell commented Jan 14, 2026

@dmytrokirpa please re-review. I revamped the storage mechanism with additional infra which should work as expected for all scenarios. ty !

Comment thread packages/react-icons/BUNDLE-SIZE-ANALYSIS.md Outdated
@Hotell Hotell force-pushed the react-icons/bundle-size-check branch from 15b065d to 250a662 Compare January 14, 2026 14:38
@Hotell Hotell requested a review from dmytrokirpa January 14, 2026 14:42
@Hotell Hotell merged commit b1cadca into microsoft:main Jan 14, 2026
7 checks passed
@Hotell Hotell deleted the react-icons/bundle-size-check branch January 14, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants