react / react/react-native

Wrong types in RCTDeviceEventEmitter

Open
#37,358 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Needs: Triage :mag: Never gets stale Newer Patch Available
Dominant language
C++
Stars
127k
Forks
25.3k
Avg merge
1d 23h
Merged PRs (30d)
4

Description

Description

On a recent update of React Native we noticed that the export of RCTDeviceEventEmitter types is wrong.

After the update, the export in .d.ts has been changed to export const:

https://github.com/facebook/react-native/blob/cbc28da666c2d103242ad36caad577c69ecc430b/packages/react-native/Libraries/EventEmitter/RCTDeviceEventEmitter.d.ts#L28-L29

But in the .js file, the export still with export default:

https://github.com/facebook/react-native/blob/cbc28da666c2d103242ad36caad577c69ecc430b/packages/react-native/Libraries/EventEmitter/RCTDeviceEventEmitter.js#L31-L32

React Native Version

0.71.8

Output of npx react-native info

System:
OS: macOS 13.2.1
CPU: (8) arm64 Apple M1
Memory: 102.47 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
Yarn: 3.3.1 - ~/.nvm/versions/node/v16.16.0/bin/yarn
npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm
Watchman: 2023.04.24.00 - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.12.0 - /Users/douglas.junior/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK:
API Levels: 30, 31, 33, 33
Build Tools: 30.0.2, 30.0.3, 31.0.0, 33.0.0, 34.0.0
System Images: android-32 | Google APIs ARM 64 v8a, android-32 | Google Play ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9477386
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 11.0.19 - /opt/homebrew/opt/openjdk@11/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.7 => 0.71.7
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

Steps to reproduce

N/A

Snack, code example, screenshot, or link to a repository

N/A

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing packages/react-native/Libraries/EventEmitter/RCTDeviceEventEmitter.d.ts with the corresponding .js file at the referenced lines. Verify how RCTDeviceEventEmitter is imported and typed, then confirm that the declaration matches the runtime export and does not produce the reported type error.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react-native, typescript
Domain
mobile-dev
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.