facebook · cipolleschi · Apr 7, 2025
diff --git a/packages/react-native/ReactCommon/react/runtime/platform/ios/ReactCommon/RCTInstance.mm b/packages/react-native/ReactCommon/react/runtime/platform/ios/ReactCommon/RCTInstance.mm
@@ -609,7 +609,7 @@ - (void)_handleJSError:(const JsErrorHandler::ProcessedError &)error withRuntime
                             name:errorData[@"name"]
                   componentStack:errorData[@"componentStack"]
                      exceptionId:error.id
-                         isFatal:errorData[@"isFatal"]
+                         isFatal:[errorData[@"isFatal"] boolValue]
                        extraData:errorData[@"extraData"]]) {
     JS::NativeExceptionsManager::ExceptionData jsErrorData{errorData};
     id<NativeExceptionsManagerSpec> exceptionsManager = [_turboModuleManager moduleForName:"ExceptionsManager"];

diff --git a/flow-typed/environment/node.js b/flow-typed/environment/node.js
    fromEnc: buffer$Encoding,
    toEnc: buffer$Encoding,
  ): Node$Buffer;
  declare var Buffer: Node$Buffer;
 }

 type child_process$execOpts = {
diff --git a/flow-typed/npm/jest.js b/flow-typed/npm/jest.js
 // Modifications are explained inline by comments beginning with `// MODIFIED`.

 // MODIFIED: Added ESLint suppression comment - no-unused-vars doesn't understand declaration files
 /* eslint-disable no-unused-vars */

 type JestMockFn<TArguments: $ReadOnlyArray<mixed>, TReturn> = {
  (...args: TArguments): TReturn,
diff --git a/packages/assets/registry.js b/packages/assets/registry.js
  return assets[assetId - 1];
 }

 module.exports = {registerAsset, getAssetByID};
diff --git a/packages/dev-middleware/src/inspector-proxy/Device.js b/packages/dev-middleware/src/inspector-proxy/Device.js
 const debug = require('debug')('Metro:InspectorProxy');

 const PAGES_POLLING_INTERVAL = 1000;
 const MIN_MESSAGE_QUEUE_BYTES_TO_REPORT = 2 * 1024 * 1024; // 2 MiB

 const WS_CLOSURE_CODE = {
  NORMAL: 1000,
diff --git a/packages/polyfills/console.js b/packages/polyfills/console.js

 'use client';

 /* eslint-disable no-shadow, eqeqeq, curly, no-unused-vars, no-void, no-control-regex  */

 /**
 * This pipes all of our console logging functions to native logging so that
diff --git a/packages/react-native-compatibility-check/src/__tests__/TypeDiffing-test.js b/packages/react-native-compatibility-check/src/__tests__/TypeDiffing-test.js
  nativeTypeDiffingTypesAliases,
  nativeTypeDiffingTypesMethodParamLookup,
  nativeTypeDiffingTypesMethodLookup,
  _,
  nativeTypeDiffingTypesEnums,
 ] = getModule(
  'native-module-type-diffing-types',
diff --git a/packages/react-native/Libraries/Animated/animations/Animation.js b/packages/react-native/Libraries/Animated/animations/Animation.js
 import type AnimatedValue from '../nodes/AnimatedValue';

 import NativeAnimatedHelper from '../../../src/private/animated/NativeAnimatedHelper';
 import * as ReactNativeFeatureFlags from '../../../src/private/featureflags/ReactNativeFeatureFlags';
 import AnimatedProps from '../nodes/AnimatedProps';

 export type EndResult = {finished: boolean, value?: number, ...};
diff --git a/packages/react-native/Libraries/Animated/nodes/AnimatedValue.js b/packages/react-native/Libraries/Animated/nodes/AnimatedValue.js
 * transform which can receive values from multiple parents.
 */
 export function flushValue(rootNode: AnimatedNode): void {
  // eslint-disable-next-line func-call-spacing
  const leaves = new Set<{update: () => void, ...}>();
  function findAnimatedStyles(node: AnimatedNode) {
    // $FlowFixMe[prop-missing]
diff --git a/packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestConfig.js b/packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestConfig.js
  remoteModuleConfig: remoteModulesConfig,
 };

 module.exports = MessageQueueTestConfig;
diff --git a/packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestModule.js b/packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestModule.js
  testHook2: function () {},
 };

 module.exports = MessageQueueTestModule;
	fromEnc: buffer$Encoding,
	toEnc: buffer$Encoding,
	): Node$Buffer;
	declare var Buffer: Node$Buffer;
Check warning on line 200 in flow-typed/environment/node.js GitHub Actions / test_js (22) `'Buffer' is already declared in the upper scope on line 53 column 15` Check warning on line 200 in flow-typed/environment/node.js GitHub Actions / test_js (20) `'Buffer' is already declared in the upper scope on line 53 column 15` Check warning on line 200 in flow-typed/environment/node.js GitHub Actions / test_js (18) `'Buffer' is already declared in the upper scope on line 53 column 15`
	}

	type child_process$execOpts = {
	// Modifications are explained inline by comments beginning with `// MODIFIED`.

	// MODIFIED: Added ESLint suppression comment - no-unused-vars doesn't understand declaration files
	/* eslint-disable no-unused-vars */
Check warning on line 15 in flow-typed/npm/jest.js GitHub Actions / test_js (22) `'no-unused-vars' rule is disabled but never reported` Check warning on line 15 in flow-typed/npm/jest.js GitHub Actions / test_js (20) `'no-unused-vars' rule is disabled but never reported` Check warning on line 15 in flow-typed/npm/jest.js GitHub Actions / test_js (18) `'no-unused-vars' rule is disabled but never reported`

	type JestMockFn<TArguments: $ReadOnlyArray<mixed>, TReturn> = {
	(...args: TArguments): TReturn,
	return assets[assetId - 1];
	}

	module.exports = {registerAsset, getAssetByID};
Check warning on line 43 in packages/assets/registry.js GitHub Actions / test_js (22) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 43 in packages/assets/registry.js GitHub Actions / test_js (20) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 43 in packages/assets/registry.js GitHub Actions / test_js (18) Use `export` syntax instead of CommonJS `module.exports`
	const debug = require('debug')('Metro:InspectorProxy');

	const PAGES_POLLING_INTERVAL = 1000;
	const MIN_MESSAGE_QUEUE_BYTES_TO_REPORT = 2 * 1024 * 1024; // 2 MiB
Check warning on line 40 in packages/dev-middleware/src/inspector-proxy/Device.js GitHub Actions / test_js (22) `'MIN_MESSAGE_QUEUE_BYTES_TO_REPORT' is assigned a value but never used` Check warning on line 40 in packages/dev-middleware/src/inspector-proxy/Device.js GitHub Actions / test_js (20) `'MIN_MESSAGE_QUEUE_BYTES_TO_REPORT' is assigned a value but never used` Check warning on line 40 in packages/dev-middleware/src/inspector-proxy/Device.js GitHub Actions / test_js (18) `'MIN_MESSAGE_QUEUE_BYTES_TO_REPORT' is assigned a value but never used`

	const WS_CLOSURE_CODE = {
	NORMAL: 1000,

	'use client';

	/* eslint-disable no-shadow, eqeqeq, curly, no-unused-vars, no-void, no-control-regex */
Check warning on line 14 in packages/polyfills/console.js GitHub Actions / test_js (22) `'curly' rule is disabled but never reported` Check warning on line 14 in packages/polyfills/console.js GitHub Actions / test_js (20) `'curly' rule is disabled but never reported` Check warning on line 14 in packages/polyfills/console.js GitHub Actions / test_js (18) `'curly' rule is disabled but never reported`

	/**
	* This pipes all of our console logging functions to native logging so that
	nativeTypeDiffingTypesAliases,
	nativeTypeDiffingTypesMethodParamLookup,
	nativeTypeDiffingTypesMethodLookup,
	_,
Check warning on line 85 in packages/react-native-compatibility-check/src/__tests__/TypeDiffing-test.js GitHub Actions / test_js (22) `'_' is assigned a value but never used` Check warning on line 85 in packages/react-native-compatibility-check/src/__tests__/TypeDiffing-test.js GitHub Actions / test_js (20) `'_' is assigned a value but never used` Check warning on line 85 in packages/react-native-compatibility-check/src/__tests__/TypeDiffing-test.js GitHub Actions / test_js (18) `'_' is assigned a value but never used`
	nativeTypeDiffingTypesEnums,
	] = getModule(
	'native-module-type-diffing-types',
	import type AnimatedValue from '../nodes/AnimatedValue';

	import NativeAnimatedHelper from '../../../src/private/animated/NativeAnimatedHelper';
	import * as ReactNativeFeatureFlags from '../../../src/private/featureflags/ReactNativeFeatureFlags';
Check warning on line 16 in packages/react-native/Libraries/Animated/animations/Animation.js GitHub Actions / test_js (22) `'ReactNativeFeatureFlags' is defined but never used` Check warning on line 16 in packages/react-native/Libraries/Animated/animations/Animation.js GitHub Actions / test_js (20) `'ReactNativeFeatureFlags' is defined but never used` Check warning on line 16 in packages/react-native/Libraries/Animated/animations/Animation.js GitHub Actions / test_js (18) `'ReactNativeFeatureFlags' is defined but never used`
	import AnimatedProps from '../nodes/AnimatedProps';

	export type EndResult = {finished: boolean, value?: number, ...};
	* transform which can receive values from multiple parents.
	*/
	export function flushValue(rootNode: AnimatedNode): void {
	// eslint-disable-next-line func-call-spacing
Check warning on line 55 in packages/react-native/Libraries/Animated/nodes/AnimatedValue.js GitHub Actions / test_js (22) `'func-call-spacing' rule is disabled but never reported` Check warning on line 55 in packages/react-native/Libraries/Animated/nodes/AnimatedValue.js GitHub Actions / test_js (20) `'func-call-spacing' rule is disabled but never reported` Check warning on line 55 in packages/react-native/Libraries/Animated/nodes/AnimatedValue.js GitHub Actions / test_js (18) `'func-call-spacing' rule is disabled but never reported`
	const leaves = new Set<{update: () => void, ...}>();
	function findAnimatedStyles(node: AnimatedNode) {
	// $FlowFixMe[prop-missing]
	remoteModuleConfig: remoteModulesConfig,
	};

	module.exports = MessageQueueTestConfig;
Check warning on line 38 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestConfig.js GitHub Actions / test_js (22) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 38 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestConfig.js GitHub Actions / test_js (20) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 38 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestConfig.js GitHub Actions / test_js (18) Use `export` syntax instead of CommonJS `module.exports`
	testHook2: function () {},
	};

	module.exports = MessageQueueTestModule;
Check warning on line 22 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestModule.js GitHub Actions / test_js (22) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 22 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestModule.js GitHub Actions / test_js (20) Use `export` syntax instead of CommonJS `module.exports` Check warning on line 22 in packages/react-native/Libraries/BatchedBridge/__mocks__/MessageQueueTestModule.js GitHub Actions / test_js (18) Use `export` syntax instead of CommonJS `module.exports`