Skip to content

Conversation

ajay-sentry
Copy link
Contributor

@ajay-sentry ajay-sentry commented Oct 2, 2025

Modifying ReactUse import

Link to Sample Entry

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

This introduces a tree-shaking performance issue by importing the entire
react-use library (~100KB+ of hooks and utilities) instead of only the
specific hook needed (useDebounce).

Impact: react-use contains 80+ hooks/utilities, but only useDebounce is
used in this file. Bundling the entire library significantly increases
bundle size unnecessarily.

Before (optimized):
import { useDebounce } from 'react-use'

After (poor tree-shaking):
import * as ReactUse from 'react-use'
Copy link

sentry-io bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.62%. Comparing base (8816d1e) to head (d211dcc).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3940   +/-   ##
=======================================
  Coverage   98.62%   98.62%           
=======================================
  Files         828      828           
  Lines       15099    15099           
  Branches     4326     4326           
=======================================
  Hits        14891    14891           
  Misses        200      200           
  Partials        8        8           
Files with missing lines Coverage Δ
src/ui/SearchField/SearchField.jsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.13% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8816d1e...d211dcc. Read the comment docs.

@codecov-staging
Copy link

Bundle Report

Bundle size has no change ✅

Copy link

sentry-io bot commented Oct 2, 2025

Bundle Report

Bundle size has no change ✅

@codecov-notifications
Copy link

codecov-notifications bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3940   +/-   ##
=======================================
  Coverage   98.62%   98.62%           
=======================================
  Files         828      828           
  Lines       15099    15099           
  Branches     4326     4326           
=======================================
  Hits        14891    14891           
  Misses        200      200           
  Partials        8        8           
Files with missing lines Coverage Δ
src/ui/SearchField/SearchField.jsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.13% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8816d1e...d211dcc. Read the comment docs.

Copy link

codecov-public-qa bot commented Oct 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.62%. Comparing base (8816d1e) to head (d211dcc).

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3940   +/-   ##
=======================================
  Coverage   98.62%   98.62%           
=======================================
  Files         828      828           
  Lines       15099    15099           
  Branches     4318     4326    +8     
=======================================
  Hits        14891    14891           
  Misses        200      200           
  Partials        8        8           
Files with missing lines Coverage Δ
src/ui/SearchField/SearchField.jsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.13% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8816d1e...d211dcc. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
d211dcc Thu, 02 Oct 2025 21:20:15 GMT Cloud Enterprise

@ajay-sentry ajay-sentry changed the title [test]: FE Perf 2 TS [test]: FE2 Oct 2, 2025
@codecov-qa
Copy link

codecov-qa bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.62%. Comparing base (8816d1e) to head (d211dcc).

@@            Coverage Diff             @@
##             main    #3940      +/-   ##
==========================================
+ Coverage   96.54%   98.62%   +2.07%     
==========================================
  Files         828      828              
  Lines       15099    15099              
  Branches     4318     4318              
==========================================
+ Hits        14578    14891     +313     
+ Misses        466      200     -266     
+ Partials       55        8      -47     
Files with missing lines Coverage Δ
src/ui/SearchField/SearchField.jsx 100.00% <100.00%> (ø)

... and 43 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.13% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (+1.81%) ⬆️
UI 99.01% <100.00%> (+20.18%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8816d1e...d211dcc. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ajay-sentry ajay-sentry changed the title [test]: FE2 [FE2]: ReactUse Import Update Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants