Skip to content

Cover react/renderer/element with Stable API guards (#58178) - #58178

Closed
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D117837637
Closed

Cover react/renderer/element with Stable API guards (#58178)#58178
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D117837637

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary:

Classifies react/renderer/element:element as a private target under the C++ stable API three-tier visibility model. Adds #include <react/cxxstableapi/PrivateGuard.h> to the module's four exported headers (Element.h, ElementFragment.h, ComponentBuilder.h, testUtils.h), and wires the guard dependency into BUCK and CMake.

The guards are inert unless a consumer defines RN_STRICT_API, so there is no behavior change.

Changelog: [Internal]

Differential Revision: D117837637

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 28, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner labels Aug 28, 2026
@meta-codesync

meta-codesync Bot commented Aug 28, 2026

Copy link
Copy Markdown

@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D117837637.

@facebook-github-tools facebook-github-tools Bot added the p: Facebook Partner: Facebook label Aug 28, 2026
Summary:

Classifies `react/renderer/element:element` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's four exported headers (`Element.h`, `ElementFragment.h`, `ComponentBuilder.h`, `testUtils.h`), and wires the guard dependency into BUCK and CMake.

The guards are inert unless a consumer defines `RN_STRICT_API`, so there is no behavior change.

Changelog: [Internal]

Differential Revision: D117837637
@meta-codesync meta-codesync Bot changed the title Cover react/renderer/element with Stable API guards Cover react/renderer/element with Stable API guards (#58178) Sep 1, 2026

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in 79d9ffe Sep 2, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 2, 2026
@meta-codesync

meta-codesync Bot commented Sep 2, 2026

Copy link
Copy Markdown

This pull request has been merged in 79d9ffe.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants