Skip to content

[9.1] Remove react-is dependency (#230117)#230453

Merged
legrego merged 4 commits intoelastic:9.1from
legrego:backport/9.1/pr-230117
Aug 5, 2025
Merged

[9.1] Remove react-is dependency (#230117)#230453
legrego merged 4 commits intoelastic:9.1from
legrego:backport/9.1/pr-230117

Conversation

@legrego
Copy link
Copy Markdown
Member

@legrego legrego commented Aug 4, 2025

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

@legrego
Remove react-is dependency (elastic#230117)
b731326 
## Summary

Removes the `react-is` dependency in favor of a simple first-party
check.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 8d27fcf)

# Conflicts:
#	x-pack/platform/plugins/shared/lens/public/datasources/form_based/form_based.test.ts
@legrego legrego added the backport This PR is a backport of another PR label Aug 4, 2025
@legrego legrego requested a review from kibanamachine as a code owner August 4, 2025 16:28
@legrego legrego enabled auto-merge (squash) August 4, 2025 16:28
@legrego legrego mentioned this pull request Aug 4, 2025
Merged
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Aug 5, 2025

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
core 405 403 -2
fleet 1338 1336 -2
kibanaReact 220 218 -2
kibanaUtils 161 159 -2
security 595 593 -2
securitySolution 7772 7770 -2
total -12

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
core 63.0KB 60.9KB -2.1KB
fleet 2.1MB 2.1MB -2.1KB
kibanaReact 187.5KB 185.4KB -2.1KB
kibanaUtils 63.0KB 60.9KB -2.1KB
security 503.4KB 501.3KB -2.1KB
securitySolution 9.8MB 9.8MB -2.1KB
total -12.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-npmDll 6.4MB 6.4MB -4.3KB

History

kc13greiner
kc13greiner approved these changes Aug 5, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

Backport LGTM!

@legrego legrego merged commit 8a28214 into elastic:9.1 Aug 5, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kc13greiner kc13greiner kc13greiner approved these changes

@kibanamachine kibanamachine Awaiting requested review from kibanamachine kibanamachine is a code owner

Assignees

No one assigned

Labels

backport This PR is a backport of another PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@legrego @elasticmachine @kc13greiner