⚡ Bolt: Refactor IconifyIcon to remove bundled icon sets#28
⚡ Bolt: Refactor IconifyIcon to remove bundled icon sets#28sshahriazz wants to merge 1 commit intomainfrom
Conversation
- Removes static imports of full icon sets (material-symbols, mdi, etc) which were causing massive bundle bloat. - Updates IconifyIcon to use on-demand fetching via @iconify/react. - Reduces initial bundle size by ~9MB. - Improves initial load time significantly. Journal: Recorded learning about bundle size bloat from Iconify imports. Co-authored-by: sshahriazz <34005640+sshahriazz@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
⚡ Bolt: Refactor IconifyIcon to remove bundled icon sets
💡 What: Refactored
apps/client/src/components/base/IconifyIcon.tsxto remove static imports of large icon sets (likematerial-symbols,mdi, etc.) and the logic that searched through them. The component now passes the icon string directly to@iconify/react, which handles fetching on demand.🎯 Why: The previous implementation statically bundled multiple full icon sets into the application, resulting in a main bundle size that was ~9MB larger than necessary. This severely impacted initial load time and performance.
📊 Impact:
First Load JSfor pages like Login dropped from ~8.96 MB to ~1.24 MB.🔬 Measurement:
pnpm --filter client build.PR created automatically by Jules for task 7727284002569646124 started by @sshahriazz