Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[🐛] v20 installs subpackages as deps, regressed from v19 #7995

Open
5 tasks done
safaiyeh opened this issue Aug 29, 2024 · 6 comments
Open
5 tasks done

[🐛] v20 installs subpackages as deps, regressed from v19 #7995

safaiyeh opened this issue Aug 29, 2024 · 6 comments
Labels
Needs Attention platform: macOS (Other) plugin: app-core Firebase Apps / Core internals. type: enhancement Implements a new Feature

Comments

@safaiyeh
Copy link

Issue

In the process of integrating react-native-firebase in a project with a tight dependency security model. When integrating React Native Firebase v19, the dependencies installed were reasonable and you can selectively install the Firebase modules you need for your project.

However, in v20 setting up the project installs all the sub modules react-native-firebase offers.


Project Files

Javascript

Click To Expand

yarn.lock diff:

v20: safaiyeh/react-native-firebase-v20@db3d4f0
v19: safaiyeh/react-native-firebase-v19@f979c44


Environment

Click To Expand

react-native info output:

System:
  OS: macOS 14.6.1
  CPU: (10) arm64 Apple M1 Max
  Memory: 362.72 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.13.0
    path: ~/.nvm/versions/node/v20.13.0/bin/node
  Yarn:
    version: 3.6.4
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.5.2
    path: ~/.nvm/versions/node/v20.13.0/bin/npm
  Watchman:
    version: 2024.04.01.00
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods: Not Found
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 11.0.22
    path: /opt/homebrew/opt/openjdk@11/bin/javac
  Ruby:
    version: 2.6.10
    path: /Users/jason/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.75.2
    wanted: 0.75.2
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: false
  • Platform that you're experiencing the issue on:
    • iOS
    • Android
    • iOS but have not tested behavior on Android
    • Android but have not tested behavior on iOS
    • Both
  • react-native-firebase version you're using that has this issue:
    • 20.4.0
  • Firebase module(s) you're using that has the issue:
    • app
  • Are you using TypeScript?
    • `N


@russellwheatley
Copy link
Member

Hey @safaiyeh - thanks for reporting this. It is dues to pulling in the firebase dependency for app here which allows macOS beta support https://github.com/invertase/react-native-firebase/blob/main/packages/app/package.json#L60

I'll have to think about what we do as I'm not sure it is advisable to separate the packages as described on the npm page: https://www.npmjs.com/package/@firebase/app

@russellwheatley russellwheatley added plugin: app-core Firebase Apps / Core internals. platform: macOS (Other) and removed type: bug New bug report labels Sep 2, 2024
@russellwheatley
Copy link
Member

@safaiyeh - Are you developing on macOS platform? Assuming you are not using macOS, you might be able to stop the packages from being installed. It seems it isn't the easiest with yarn however: yarnpkg/yarn#4611

FYI; the additional packages are not RNFB packages, but they are the firebase web packages: https://www.npmjs.com/package/firebase

@safaiyeh
Copy link
Author

safaiyeh commented Sep 2, 2024

@russellwheatley That is correct, I'm not developing on macOS - just iOS / Android.

Thanks for the context, looks like there weren't any changes to the build system for RNFB. The root cause is from Firebase itself? I can open a ticket on the Firebase repo as a follow up

@russellwheatley
Copy link
Member

This isn't a firebase issue, RNFB app package has firebase web as a dependency for macOS support: https://github.com/invertase/react-native-firebase/blob/main/packages/app/package.json#L60

Copy link

github-actions bot commented Oct 1, 2024

Hello 👋, to help manage issues we automatically close stale issues.

This issue has been automatically marked as stale because it has not had activity for quite some time.Has this issue been fixed, or does it still require attention?

This issue will be closed in 15 days if no further activity occurs.

Thank you for your contributions.

@github-actions github-actions bot added the Stale label Oct 1, 2024
@safaiyeh
Copy link
Author

still require attention

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention platform: macOS (Other) plugin: app-core Firebase Apps / Core internals. type: enhancement Implements a new Feature
Projects
None yet
Development

No branches or pull requests

2 participants