Skip to content

Conversation

@sebmarkbage
Copy link
Collaborator

The only thing that uses memoizedState as a public API is ClassComponents. Everything else uses it as internals. We shouldn't ever show those internals.

Before those internals showed up for example on a suspended Suspense boundary:

Screenshot 2025-08-03 at 8 13 37 PM

@meta-cla meta-cla bot added the CLA Signed label Aug 4, 2025
@sebmarkbage sebmarkbage requested review from eps1lon and hoxyq August 4, 2025 00:16
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Aug 4, 2025
@sebmarkbage sebmarkbage merged commit 0417546 into facebook:main Aug 4, 2025
248 checks passed
eps1lon pushed a commit to eps1lon/react that referenced this pull request Aug 4, 2025
The only thing that uses `memoizedState` as a public API is
ClassComponents. Everything else uses it as internals. We shouldn't ever
show those internals.

Before those internals showed up for example on a suspended Suspense
boundary:

<img width="436" height="370" alt="Screenshot 2025-08-03 at 8 13 37 PM"
src="https://github.com/user-attachments/assets/7fe275a7-d5da-421d-a000-523825916630"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants