RuntimeScheduler umbrella POC

Differential Revision: D107878704

Author: j-piasecki

packages/react-native/ReactCommon/react/renderer/runtimescheduler/CMakeLists.txt

@@ -13,6 +13,18 @@ add_library(react_renderer_runtimescheduler STATIC ${react_renderer_runtimesched
 
 target_include_directories(react_renderer_runtimescheduler PUBLIC ${REACT_COMMON_DIR})
 
+# POC: vend the umbrella as <React/RuntimeScheduler.h>. The umbrella physically
+# lives at this module's `React/` subdir, so adding the module dir as a public
+# include dir makes `<React/RuntimeScheduler.h>` resolve for dependents.
+# NOTE (macOS, case-insensitive FS): the `React/` prefix differs from the
+# existing lowercase `react/` tree only by case. They do not collide here (the
+# two RuntimeScheduler.h files sit at different depths), but this is a known
+# case-insensitive-FS fragility tracked in
+# __docs__/RuntimeSchedulerUmbrellaPOC.fb.md.
+# NOTE: adding the module dir also exposes the other headers as bare includes; a
+# real rollout would scope this to a dedicated public-include subdir.
+target_include_directories(react_renderer_runtimescheduler PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
+
 target_link_libraries(react_renderer_runtimescheduler
         callinvoker
         jsi
@@ -29,7 +41,7 @@ target_compile_reactnative_options(react_renderer_runtimescheduler PRIVATE)
 target_compile_options(react_renderer_runtimescheduler PRIVATE -Wpedantic)
 
 # POC (public C++ API surface reduction): this module exposes a single public
-# umbrella header, react/renderer/runtimescheduler/RuntimeScheduler.umbrella.h.
+# umbrella header, vended top-level as <React/RuntimeScheduler.h>.
 # Including any other module header directly is gated by an opt-in guard
 # (RuntimeSchedulerUmbrellaGuard.h) that is inert unless
 # REACT_RUNTIMESCHEDULER_ENFORCE_UMBRELLA is defined.

packages/react-native/ReactCommon/react/renderer/runtimescheduler/React-runtimescheduler.podspec

@@ -33,10 +33,12 @@ Pod::Spec.new do |s|
   s.source                 = source
   s.source_files           = podspec_sources("**/*.{cpp,h}", "**/*.h")
   s.header_dir             = "react/renderer/runtimescheduler"
-  s.exclude_files          = "tests"
+  # The umbrella is vended top-level as <React/RuntimeScheduler.h> via the
+  # "Umbrella" subspec below, so exclude it from the main (react/...) mapping.
+  s.exclude_files          = ["tests", "React"]
   # POC (public C++ API surface reduction): RuntimeScheduler exposes a single
-  # public umbrella header (RuntimeScheduler.umbrella.h). Including any other
-  # module header directly is gated by an opt-in guard that is inert unless
+  # public umbrella header, vended top-level as <React/RuntimeScheduler.h>.
+  # Including any other module header directly is gated by an opt-in guard that is inert unless
   # REACT_RUNTIMESCHEDULER_ENFORCE_UMBRELLA is defined. CocoaPods exposes the
   # whole header_dir, so the individual headers remain includable here; the guard
   # macro is the only way to enforce "umbrella only" in the pods build.
@@ -69,4 +71,22 @@ Pod::Spec.new do |s|
   depend_on_js_engine(s)
   add_rn_third_party_dependencies(s)
   add_rncore_dependency(s)
+
+  # POC: vend the umbrella under the shared top-level `React/` prefix, i.e.
+  # `#include <React/RuntimeScheduler.h>`. header_mappings_dir = "React" maps the
+  # physical react/renderer/runtimescheduler/React/RuntimeScheduler.h to a
+  # leaf-level RuntimeScheduler.h, and header_dir = "React" places it under React/.
+  #
+  # NOTE (untested): CocoaPods cannot be run in the internal build, so the exact
+  # header_dir / header_mappings_dir interaction needs validation with `pod install`.
+  # NOTE (macOS): under use_frameworks!, this pod's Headers would contain both
+  # `React/` (umbrella) and `react/...` (interface headers); on a case-insensitive
+  # filesystem those fold to one directory. They do not overwrite (different
+  # depths), but this is a known fragility tracked in
+  # __docs__/RuntimeSchedulerUmbrellaPOC.fb.md.
+  s.subspec "Umbrella" do |ss|
+    ss.source_files        = "React/*.h"
+    ss.header_dir          = "React"
+    ss.header_mappings_dir = "React"
+  end
 end

packages/react-native/ReactCommon/react/renderer/runtimescheduler/React/RuntimeScheduler.h

@@ -0,0 +1,44 @@
+/*
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+#pragma once
+
+// =============================================================================
+// Umbrella header for the `runtimescheduler` module — public entry point.
+//
+// POC (public C++ API surface reduction). Consumers include ONLY this header,
+// via the shared top-level `React/` prefix:
+//
+//   #include <React/RuntimeScheduler.h>
+//
+// The individual interface headers below are re-exported here. The fork
+// implementations `RuntimeScheduler_Modern` / `RuntimeScheduler_Legacy` are
+// deliberately NOT included: they are private to the build unit.
+//
+// The `React/` prefix is a shared namespace (also used by React-Core's Obj-C
+// headers). Naming policy: one umbrella per module, named after the module, to
+// avoid leaf collisions in the shared namespace.
+//
+// NOTE: this file lives at `react/renderer/runtimescheduler/React/` so the
+// `<React/...>` include path can be produced physically (CMake) as well as via
+// header maps (Buck) and `header_dir` (CocoaPods). RN-internal code should keep
+// using the fine-grained `<react/renderer/runtimescheduler/...>` includes; only
+// outside consumers use this umbrella.
+// =============================================================================
+
+// Mark that subsequent module headers are pulled in through the umbrella. This
+// satisfies the per-header umbrella guard (see RuntimeSchedulerUmbrellaGuard.h).
+#define REACT_RUNTIMESCHEDULER_UMBRELLA_INCLUDE
+
+#include <react/renderer/runtimescheduler/RuntimeScheduler.h>
+#include <react/renderer/runtimescheduler/RuntimeSchedulerBinding.h>
+#include <react/renderer/runtimescheduler/RuntimeSchedulerCallInvoker.h>
+#include <react/renderer/runtimescheduler/RuntimeSchedulerEventTimingDelegate.h>
+#include <react/renderer/runtimescheduler/RuntimeSchedulerIntersectionObserverDelegate.h>
+#include <react/renderer/runtimescheduler/SchedulerPriorityUtils.h>
+#include <react/renderer/runtimescheduler/Task.h>
+#include <react/renderer/runtimescheduler/primitives.h>

packages/react-native/ReactCommon/react/renderer/runtimescheduler/RuntimeScheduler.umbrella.h

@@ -5,13 +5,13 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+#include <React/RuntimeScheduler.h>
 #include <gtest/gtest.h>
 #include <hermes/hermes.h>
 #include <jsi/jsi.h>
 #include <react/featureflags/ReactNativeFeatureFlags.h>
 #include <react/featureflags/ReactNativeFeatureFlagsDefaults.h>
 #include <react/performance/timeline/PerformanceEntryReporter.h>
-#include <react/renderer/runtimescheduler/RuntimeScheduler.umbrella.h>
 #include <chrono>
 #include <memory>
 #include <semaphore>

packages/react-native/ReactCommon/react/renderer/runtimescheduler/tests/RuntimeSchedulerTest.cpp

@@ -5,8 +5,8 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+#include <React/RuntimeScheduler.h>
 #include <gtest/gtest.h>
-#include <react/renderer/runtimescheduler/RuntimeScheduler.umbrella.h>
 #include <chrono>
 
 using namespace facebook::react;