Skip to content

[0.80] Update debugger-frontend from bc635fa...343405b (S523384 mitigation) #51564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 26, 2025

Conversation

huntie
Copy link
Member

@huntie huntie commented May 23, 2025

Summary

This is a cherry pick of facebook/react-native-devtools-frontend#164, built from the corresponding 0.80-stable branch of https://github.com/facebook/react-native-devtools-frontend.

Command run:

node scripts/debugger-frontend/sync-and-build.js --branch 0.80-stable

Context

This is a mitigation for S523384, affecting builds of react-native-devtools-frontend shipped React Native 0.78 and onwards.

Specifically, it disables all eager evaluation features from the Console panel, which, following an upstream ChromeDevTools sync, puts the frontend ahead of what Hermes currently supports under CDP.

  • Functionally, this can cause erratic and unsafe console behaviour, that executes side effects, while typing.

image

Changelog:
[General][Fixed] - React Native DevTools: Eager evaluation and Live Expressions in the Console tab have been temporarily disabled, as they were being incorrectly executed with side effects

Test Plan

See facebook/react-native-devtools-frontend#164.

@huntie huntie requested review from hezi, cortinico and cipolleschi May 23, 2025 13:16
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels May 23, 2025
@huntie huntie force-pushed the 0.80-S523384-mitigation branch from 20a7751 to 25a8338 Compare May 23, 2025 16:14
@huntie huntie changed the title [0.80] Update debugger-frontend from bc635fa...bd3b99 (S523384 mitigation) [0.80] Update debugger-frontend from bc635fa...343405b (S523384 mitigation) May 23, 2025
Copy link

@Hardanish-Singh Hardanish-Singh left a comment

Choose a reason for hiding this comment

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

LGTM!

@fabriziocucci fabriziocucci merged commit 647af1c into facebook:0.80-stable May 26, 2025
81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants