Fix sensor listener leak on Android

[graszka22]

Summary

Fixes #3896

In the scenario presented in the issue every time the component was mounted two ReanimatedSensor were created: one was just initialized and the second one added to the hashmap. Then, when unregistering, only the second one was unregistered, the first one was kept registered. After some time the max limit of registered listeners is hit.

Test plan

import * as React from 'react';
import { Text, View, StyleSheet } from 'react-native';
import Constants from 'expo-constants';
import { useAnimatedSensor, SensorType } from 'react-native-reanimated';

export function BasicLayoutAnimation() {
  return (
    <View style={styles.container}>
      <Parent />
    </View>
  );
}

const Parent = () => {
  const [state, setState] = React.useState(0);

  React.useEffect(() => {
    setInterval(() => setState((prevState) => ++prevState), 100);
  }, []);
  return (
    <View style={[styles.container, styles.parent]}>
      <Text>Parent</Text>
      <Child />
    </View>
  );
};

const Child = () => {
  useAnimatedSensor(SensorType.MAGNETIC_FIELD, {
    interval: 2000,
  });
  const count = React.useRef(0);
  React.useEffect(() => {
    console.log('render count: ', ++count.current);
  });
  return (
    <View style={styles.container}>
      <Text>Child render: {count.current}</Text>
    </View>
  );
};

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    paddingTop: Constants.statusBarHeight,
    backgroundColor: '#ecf0f1',
    padding: 20,
  },
  parent: {
    backgroundColor: 'red',
  },
});

[ashuvssut]

will this fix be available in v2.13.1? or which package ver should I install to get this fix?

[tomekzaw]

@ashuvssut We will ship this fix in 2.14.0 and 3.0.0-rc.10 next week.