Merge branch 'main' into tts-span

Author: fabOnReact

IntegrationTests/TimersTest.js

@@ -39,7 +39,7 @@ class TimersTest extends React.Component<Props, State> {
   };
 
   setTimeout(fn: () => void, time: number): TimeoutID {
-    const id = setTimeout(() => {
+    const id: TimeoutID = setTimeout(() => {
       this._timeoutIDs.delete(id);
       fn();
     }, time);
@@ -70,7 +70,7 @@ class TimersTest extends React.Component<Props, State> {
   }
 
   setImmediate(fn: () => void): ImmediateID {
-    const id = setImmediate(() => {
+    const id: any = setImmediate(() => {
       this._immediateIDs.delete(id);
       fn();
     });
@@ -81,7 +81,7 @@ class TimersTest extends React.Component<Props, State> {
   }
 
   requestAnimationFrame(fn: () => void): AnimationFrameID {
-    const id = requestAnimationFrame(() => {
+    const id: AnimationFrameID = requestAnimationFrame(() => {
       this._animationFrameIDs.delete(id);
       fn();
     });

Libraries/Animated/AnimatedImplementation.js

@@ -415,7 +415,7 @@ const parallel = function (
       });
     },
 
-    _startNativeLoop: function () {
+    _startNativeLoop: function (): empty {
       throw new Error(
         'Loops run using the native driver cannot contain Animated.parallel animations',
       );

Libraries/BatchedBridge/MessageQueue.js

@@ -252,7 +252,7 @@ class MessageQueue {
       // folly-convertible.  As a special case, if a prop value is a
       // function it is permitted here, and special-cased in the
       // conversion.
-      const isValidArgument = (val: mixed) => {
+      const isValidArgument = (val: mixed): boolean => {
         switch (typeof val) {
           case 'undefined':
           case 'boolean':

Libraries/BugReporting/getReactData.js

@@ -165,7 +165,7 @@ function copyWithSetImpl(
   path: Array<string | number>,
   idx: number,
   value: any,
-) {
+): any {
   if (idx >= path.length) {
     return value;
   }

Libraries/Core/Timers/JSTimers.js

@@ -242,7 +242,7 @@ const JSTimers = {
    * @param {function} func Callback to be invoked before the end of the
    * current JavaScript execution loop.
    */
-  queueReactNativeMicrotask: function (func: Function, ...args: any) {
+  queueReactNativeMicrotask: function (func: Function, ...args: any): number {
     const id = _allocateCallback(
       () => func.apply(undefined, args),
       'queueReactNativeMicrotask',
@@ -254,7 +254,7 @@ const JSTimers = {
   /**
    * @param {function} func Callback to be invoked every frame.
    */
-  requestAnimationFrame: function (func: Function) {
+  requestAnimationFrame: function (func: Function): any | number {
     const id = _allocateCallback(func, 'requestAnimationFrame');
     createTimer(id, 1, Date.now(), /* recurring */ false);
     return id;
@@ -265,16 +265,19 @@ const JSTimers = {
    * with time remaining in frame.
    * @param {?object} options
    */
-  requestIdleCallback: function (func: Function, options: ?Object) {
+  requestIdleCallback: function (
+    func: Function,
+    options: ?Object,
+  ): any | number {
     if (requestIdleCallbacks.length === 0) {
       setSendIdleEvents(true);
     }
 
     const timeout = options && options.timeout;
-    const id = _allocateCallback(
+    const id: number = _allocateCallback(
       timeout != null
         ? (deadline: any) => {
-            const timeoutId = requestIdleCallbackTimeouts[id];
+            const timeoutId: number = requestIdleCallbackTimeouts[id];
             if (timeoutId) {
               JSTimers.clearTimeout(timeoutId);
               delete requestIdleCallbackTimeouts[id];
@@ -287,8 +290,8 @@ const JSTimers = {
     requestIdleCallbacks.push(id);
 
     if (timeout != null) {
-      const timeoutId = JSTimers.setTimeout(() => {
-        const index = requestIdleCallbacks.indexOf(id);
+      const timeoutId: number = JSTimers.setTimeout(() => {
+        const index: number = requestIdleCallbacks.indexOf(id);
         if (index > -1) {
           requestIdleCallbacks.splice(index, 1);
           _callTimer(id, global.performance.now(), true);
@@ -345,7 +348,7 @@ const JSTimers = {
    * This is called from the native side. We are passed an array of timerIDs,
    * and
    */
-  callTimers: function (timersToCall: Array<number>) {
+  callTimers: function (timersToCall: Array<number>): any | void {
     invariant(
       timersToCall.length !== 0,
       'Cannot call `callTimers` with an empty list of IDs.',

Libraries/Lists/ViewabilityHelper.js

@@ -234,7 +234,7 @@ class ViewabilityHelper {
     }
     this._viewableIndices = viewableIndices;
     if (this._config.minimumViewTime) {
-      const handle = setTimeout(() => {
+      const handle: TimeoutID = setTimeout(() => {
         /* $FlowFixMe[incompatible-call] (>=0.63.0 site=react_native_fb) This
          * comment suppresses an error found when Flow v0.63 was deployed. To
          * see the error delete this comment and run Flow. */

Libraries/LogBox/Data/LogBoxData.js

@@ -68,7 +68,7 @@ const observers: Set<{observer: Observer, ...}> = new Set();
 const ignorePatterns: Set<IgnorePattern> = new Set();
 let appInfo: ?() => AppInfo = null;
 let logs: LogBoxLogs = new Set();
-let updateTimeout = null;
+let updateTimeout: $FlowFixMe | null = null;
 let _isDisabled = false;
 let _selectedIndex = -1;
 

Libraries/vendor/emitter/__tests__/EventEmitter-test.js

@@ -9,6 +9,8 @@
  * @oncall react_native
  */
 
+import type {EventSubscription} from '../EventEmitter';
+
 import EventEmitter from '../EventEmitter';
 
 describe('listeners', () => {
@@ -309,15 +311,18 @@ describe('event emission', () => {
     const listenerA = jest.fn(() => {
       results.push('A');
     });
-    const listenerB = jest.fn(() => {
+    const listenerB: JestMockFn<Array<mixed>, void> = jest.fn(() => {
       results.push('B');
       subscriptionB.remove();
     });
     const listenerC = jest.fn(() => {
       results.push('C');
     });
     emitter.addListener('A', listenerA);
-    const subscriptionB = emitter.addListener('A', listenerB);
+    const subscriptionB: EventSubscription = emitter.addListener(
+      'A',
+      listenerB,
+    );
     emitter.addListener('A', listenerC);
 
     emitter.emit('A');

ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java

@@ -37,6 +37,11 @@ public class ReactFeatureFlags {
    */
   public static boolean enableBridgelessArchitecture = false;
 
+  /**
+   * Does the bridgeless architecture log soft exceptions. Could be useful for tracking down issues.
+   */
+  public static volatile boolean enableBridgelessArchitectureSoftExceptions = false;
+
   /**
    * After TurboModules and Fabric are enabled, we need to ensure that the legacy NativeModule isn't
    * isn't used. So, turn this flag on to trigger warnings whenever the legacy NativeModule system

packages/react-native-codegen/src/generators/modules/GenerateModuleObjCpp/serializeMethod.js

@@ -295,7 +295,7 @@ function getParamObjCType(
 function getReturnObjCType(
   methodName: string,
   nullableTypeAnnotation: Nullable<NativeModuleReturnTypeAnnotation>,
-) {
+): string {
   const [typeAnnotation, nullable] = unwrapNullable(nullableTypeAnnotation);
 
   function wrapIntoNullableIfNeeded(generatedType: string) {