Skip to content

Cover react/nativemodule/microtasks:microtasks with Stable API guards (#58027) - #58027

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

Cover react/nativemodule/microtasks:microtasks with Stable API guards (#58027)#58027
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D116612641

Conversation

@j-piasecki

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

Copy link
Copy Markdown
Contributor

Summary:

Classifies react/nativemodule/microtasks:microtasks as a private target under the C++ stable API three-tier visibility model. Adds #include <react/cxxstableapi/PrivateGuard.h> to the module's only header, NativeMicrotasks.h, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into RN_STRICT_API now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D116612641

@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 20, 2026
@meta-codesync

meta-codesync Bot commented Aug 20, 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 D116612641.

@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Aug 20, 2026
@meta-codesync meta-codesync Bot changed the title Cover react/nativemodule/microtasks:microtasks with Stable API guards Cover react/nativemodule/microtasks:microtasks with Stable API guards (#58027) Aug 21, 2026
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Aug 21, 2026
…ds (react#58027)

Summary:

Classifies `react/nativemodule/microtasks:microtasks` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's only header, `NativeMicrotasks.h`, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into `RN_STRICT_API` now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Differential Revision: D116612641
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Aug 25, 2026
…ds (react#58027)

Summary:

Classifies `react/nativemodule/microtasks:microtasks` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's only header, `NativeMicrotasks.h`, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into `RN_STRICT_API` now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Differential Revision: D116612641
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Aug 26, 2026
…ds (react#58027)

Summary:

Classifies `react/nativemodule/microtasks:microtasks` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's only header, `NativeMicrotasks.h`, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into `RN_STRICT_API` now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Differential Revision: D116612641

@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.

j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Sep 2, 2026
…ds (react#58027)

Summary:

Classifies `react/nativemodule/microtasks:microtasks` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's only header, `NativeMicrotasks.h`, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into `RN_STRICT_API` now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D116612641
…ds (react#58027)

Summary:

Classifies `react/nativemodule/microtasks:microtasks` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to the module's only header, `NativeMicrotasks.h`, and declares the guard dependency in BUCK, CMake, and the podspec. Consumers that opt into `RN_STRICT_API` now get a hard error if they include it directly, with no escape hatch; without that flag the guard is inert, so no existing build changes behaviour.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D116612641
@meta-codesync meta-codesync Bot closed this in 16f8889 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 16f8889.

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