ScrollView.d.ts does not match ScrollView.js

[edkimmel]

Description

There has been recent changes to ScrollView's props and types in ScrollView.js, but the declaration file has not been updated.

For example.

  /**
   * A ref to the inner View element of the ScrollView. This should be used
   * instead of calling `getInnerViewRef`.
   */
  innerViewRef?: React.RefSetter<InnerViewInstance>,
  /**
   * A ref to the Native ScrollView component. This ref can be used to call
   * all of ScrollView's public methods, in addition to native methods like
   * measure, measureLayout, etc.
   */
  scrollViewRef?: React.RefSetter<PublicScrollViewInstance>,

These props are not visible to the caller, because they are missing in the declaration file.

Steps to reproduce

Compare the props exposed by the declaration file

compared to the source

The declaration file is also one year old (as of 0.79.2) where as ScrollView was changed more recently.

React Native Version

0.79.2

Affected Platforms

Build - MacOS

Output of npx @react-native-community/cli info

N/A

Stacktrace or Logs

N/A

MANDATORY Reproducer

https://github.com/edkimmel/scroll-view-props

Screenshots and Videos

[cortinico]

There has been recent changes to ScrollView's props and types in ScrollView.js, but the declaration file has not been updated.

mmm I'm not sure what you mean with "recent changes" as innerViewRef was introduced 5 years ago:
10254a9

[edkimmel]

There has been recent changes to ScrollView's props and types in ScrollView.js, but the declaration file has not been updated.

mmm I'm not sure what you mean with "recent changes" as innerViewRef was introduced 5 years ago: 10254a9

Yes you're right, I was mistaking the changes to the imperative code 3 months ago as adding it.

In any case, the props exposed by ScrollView.d do not align to the implementation, and the suggested refs to use are not available.