Move unstable_batchedUpdates out of the renderer implementation API (facebook#56567)

Summary:
Pull Request resolved: facebook#56567

Changelog: [internal]

`unstable_batchedUpdates` was only ever defined in Paper, where it batched state updates triggered by an event handler. Some product code in apps using only Fabric imported it directly from Paper, where it was a no-op. After Paper was removed from React Native, the no-op was kept in `RendererImplementation` (and forwarded through `RendererProxy`) purely to preserve that compatibility — it has nothing to do with the Fabric renderer indirection that the rest of `RendererImplementation` is for.

Drop it from the renderer implementation API and define the no-op directly at the `react-native` package boundary:
- `index.js.flow` and `index.js` now define `unstable_batchedUpdates` inline (the public re-export already came from the package, so consumers using `import {unstable_batchedUpdates} from 'react-native'` are unaffected).
- Remove the corresponding `unstable_batchedUpdates` definitions / re-exports from `RendererImplementation.js`, `RendererProxy.fb.js`, and the `jest-preset` mock.

Reviewed By: javache

Differential Revision: D101989236

fbshipit-source-id: 9d83668f0dbc108ec24371fba37ed7ce7b452978

Author: rubennorte

packages/jest-preset/jest/mocks/RendererProxy.js

@@ -23,7 +23,6 @@ const {
   isProfilingRenderer,
   renderElement,
   sendAccessibilityEvent,
-  unstable_batchedUpdates,
 } = jest.requireActual<TRendererImplementation>(
   'react-native/Libraries/ReactNative/RendererImplementation',
 ) as TRendererImplementation;
@@ -39,5 +38,4 @@ export {
   isProfilingRenderer,
   renderElement,
   sendAccessibilityEvent,
-  unstable_batchedUpdates,
 };

packages/react-native/Libraries/ReactNative/RendererImplementation.js

@@ -53,13 +53,6 @@ export const getPublicInstanceFromInternalInstanceHandle =
 export const getPublicInstanceFromRootTag =
   ReactFabric.getPublicInstanceFromRootTag;
 
-export function unstable_batchedUpdates<T>(
-  fn: (bookkeeping: T) => void,
-  bookkeeping: T,
-): void {
-  fn(bookkeeping);
-}
-
 export function isProfilingRenderer(): boolean {
   return Boolean(__DEV__);
 }

packages/react-native/index.js

@@ -362,9 +362,8 @@ module.exports = {
   get UIManager() {
     return require('./Libraries/ReactNative/UIManager').default;
   },
-  get unstable_batchedUpdates() {
-    return require('./Libraries/ReactNative/RendererProxy')
-      .unstable_batchedUpdates;
+  unstable_batchedUpdates<T>(fn: (bookkeeping: T) => void, bookkeeping: T) {
+    fn(bookkeeping);
   },
   get useAnimatedValue() {
     return require('./Libraries/Animated/useAnimatedValue').default;

packages/react-native/index.js.flow

@@ -405,7 +405,12 @@ export * as Systrace from './Libraries/Performance/Systrace';
 export {default as ToastAndroid} from './Libraries/Components/ToastAndroid/ToastAndroid';
 export * as TurboModuleRegistry from './Libraries/TurboModule/TurboModuleRegistry';
 export {default as UIManager} from './Libraries/ReactNative/UIManager';
-export {unstable_batchedUpdates} from './Libraries/ReactNative/RendererProxy';
+export function unstable_batchedUpdates<T>(
+  fn: (bookkeeping: T) => void,
+  bookkeeping: T,
+): void {
+  fn(bookkeeping);
+}
 export {default as useAnimatedValue} from './Libraries/Animated/useAnimatedValue';
 export {default as useAnimatedValueXY} from './Libraries/Animated/useAnimatedValueXY';
 export {default as useAnimatedColor} from './Libraries/Animated/useAnimatedColor';