Migrate cli-plugin-metro into repo #38795

huntie · 2023-08-04T16:45:16Z

Summary:

Context

RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641

Changes

Inits new package @react-native/community-cli-plugin. This migrates @react-native-community/cli-plugin-metro into the React Native repo, to enable faster iteration by the React Native core team. Specifically:

This package contains several metro dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users.
This package contains the start, bundle, and ram-bundle commands (central to the React Native development experience). The start command in particular encapsulates the React Native dev server, for which we have incoming changes to support improved debugging for 0.73.
This package now only exports commands to be attached as an RN CLI plugin. With this move, we're aiming to internalise the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: breaking: Fix ability to override commands via config react-native-community/cli#1999.)

In latest version:

(Microsoft feedback) Re-export unstable_buildBundleWithConfig, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future.

The package source has been converted from TypeScript to Flow, with a number of new flow-typed/ defs added to meet type coverage requirements.

To dos

For now, we aren't removing the existing @react-native-community/cli-plugin-metro source — until later PRs consolidate this move by changing dependencies in the react-native package.
Exported API is reduced!: I'm working with @szymonrybczak to decouple references from RN CLI packages feat: add invoking CLI's scripts for launching Metro in run-ios command react-native-community/cli#2021.

cc @robhogan @thymikee @szymonrybczak @kelset @tido64

Changelog: [Internal]

Differential Revision: D46801501

facebook-github-bot · 2023-08-04T16:46:31Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/cli-commands`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Differential Revision: D46801501 fbshipit-source-id: f78af34a6c8274386c3644a5c8a4e31c58c09993

facebook-github-bot · 2023-08-04T17:16:50Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/cli-commands`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: d420e2b927bc1c2f926263ebcffc12dd3df44e73

facebook-github-bot · 2023-08-07T09:10:34Z

This pull request was exported from Phabricator. Differential Revision: D46801501

tido64 · 2023-08-07T09:17:33Z

packages/cli-commands/src/index.flow.js

+ * @oncall react_native
+ */
+
+export {bundleCommand} from './commands/bundle/bundle';


Can you also export buildBundleWithConfig? We're using it in rnx-kit.

Looking into this! 👍🏻

Before resolving this issue, can you repeat the reasoning behind why you're not going to export buildBundleWithConfig for future reference?

From offline discussion with @tido64:

API changes to this package

We're reducing the surface area of @react-native/cli-commands (formerly @react-native-community/cli-plugin-metro) to interface as a React Native CLI commands plugin (with no other JS exports). Longer term, it may become a completely internal part of React Native.

This means no longer exporting buildBundleWithConfig as a shareable component (previously depended on by rnx-kit).

We're doing this because, combined with Metro, the React Native team is already maintaining two stable public APIs for creating a bundle:

The react-native bundle command (this package) — CLI entry point to creating a bundle, configured with a CLI config file + Metro config.

Metro's Server API — JS interface which allows implementers (e.g. Expo, rnx-kit) to bootstrap their own bundle logic.

Maintaining buildBundleWithConfig as a third function is unnecessary surface area for us. We believe it's reasonable instead for alternative CLIs/tooling to reimplement this logic themselves (using the above Metro APIs), despite some foregone code reuse today.

am I wrong or you are hard-removing this code without even a deprecation phase?

We are, given this isn't a user facing API or feature (the command/config interface to the exposed bundle command for users is unchanged).

This code removal is inline with any inter-package API changes that can be made in the CLI codebase within a major React Native cycle. This change (removal of cli-plugin-metro) is one of many breaking changes to packages in CLI 12.x, and we've been working with the CLI team on this and the related dependency decoupling.

Given rnx-kit is an API consumer of this outgoing package (which we didn't see initially), you've rightly pointed out that we should consider compatibility. After discussing with @tido64 we've compared our points of view and agreed this API removal. In particular, since rnx-kit wants to enable further dynamic configuration of config, outside of the supported CLI config entry point (which is user facing), it makes sense for rnx-kit to now own its implementation of the bundle command.

After discussing with @tido64 we've compared our points of view and agreed this API removal.

Just to clarify, I did not agree to this removal. I am pretty sure I made this crystal clear.

cli-plugin-metro was created by us so that we can share the bundle logic between @react-native-community/cli and ourselves, and contribute to a greater good. My personal opinion is that us copying all the logic almost verbatim in our own @rnx-kit/metro-service is just an overhead for us while providing no gains whatsoever. I also argued that it splits the community because any improvements we make no longer automatically accrues value to anyone not using our CLI and vice versa. If the Metro team insists on this direction, we have no other choice but to go our separate paths.

Hey @tido64 - I'm sure we all can come to an understanding here. The structure of dependencies between React Native, Metro and @react-native-community/cli is a longstanding pain point that's reduced our ability (as the React Native team at Meta) to improve the developer experience for the benefit of the entire ecosystem. Our aim in rearranging the packages is to facilitate greater sharing and more consistency across the major RN integration toolkits (Meta, RN CLI, rnx-kit, Expo). We have no intention to "split the community" or force you to do work you don't think is useful. Let's follow up in our upcoming Metro contributors meeting and make sure we have an agreed way forward.

Following up from our meeting yesterday, I have a proposal for how to proceed.

RN 0.73

Since we weren't aware of the rnx-kit call site when planning this change, I'd like to first ensure we're not forcing a sudden breaking change on rnx-kit without an agreed migration path. To that end, we'll export unstable_buildBundleWithConfig from the new @react-native/community-cli-plugin package in React Native 0.73.0, with the understanding that it will likely move or otherwise change in future versions ( = some time in the next few months). We'll discuss changes with @tido64 ahead of time (but will explicitly not support any new call sites / packages beyond existing usage).

Migration path (RN 0.74+)

For ease of maintenance and deduplicating package responsibilities, @react-native/community-cli-plugin will eventually be purely a plugin for @react-native-community/cli. That is, it will not have secondary exports. Any other useful functionality it exposes on top of being a plugin will need to move elsewhere, where it can be consumed without being tied specifically to the RN CLI. As such, unstable_buildBundleWithConfig will need to be removed, and rnx-kit will need an alternative.

My understanding of what buildBundleWithConfig does (and how we might split it up) is:

It validates the platform argument against the Metro config, which can be moved directly into Metro IMO.

It mutates the current environment according to the dev parameter, which tbh is a strange side effect I wouldn't want to include in a reusable function. rnx-kit should feel free to copy this line of code if it makes sense for its users, or we can just delete it.

It strips path information from sourceMapUrl unless --sourcemap-use-absolute-path is specified. Not sure about this one, but we might be able to bake this into Metro and remove support for --sourcemap-use-absolute-path (I doubt anyone's using this option).

It uses documented, typed Metro APIs to build a bundle and get the asset information for that bundle - this code is trivial and safe to replicate.

It writes out asset files in a platform-dependent way. This part is responsible for the bulk of additional logic that rnx-kit had to fork from the CLI before buildBundleWithConfig existed. I'll discuss this below.

Re: (5) specifically - the build logic for assets should have a single source of truth in React Native, close to the @react-native/assets-registry package which already implicitly depends on it. It is clearly useful to all apps that use @react-native/assets-registry, not just ones that happen to be built with the RN CLI. It might belong in a new package, or directly in @react-native/assets-registry for that matter (but not in @react-native/community-cli-plugin).

@tido64, please let me know whether this seems workable to you. (I can imagine a few alternative designs, but let's start here.) I would love your help in getting the right abstractions in place so that rnx-kit continues to be a valuable tool for RN users as we iterate on the core (cross-framework) developer experience.

For (5) specifically, the current logic there is currently too platform specific, and only for the built in platforms. This logic needs to be customizable for additional platforms. @react-native/assets-registry is one part of this logic, but it does not handle things like writing iOS/macOS assetCatalogs. And similar to some of the custom file location logic for android, windows will also need some custom file location logic.

I created react-native-community/cli#2002 (which after this PR will need porting to core instead), which would allow platforms to provide their own specific logic for how to save assets collected during the bundle process.

analysis-bot · 2023-08-07T09:35:43Z

Platform	Engine	Arch	Size (bytes)	Diff
android	hermes	arm64-v8a	8,891,571	-155
android	hermes	armeabi-v7a	n/a	--
android	hermes	x86	n/a	--
android	hermes	x86_64	n/a	--
android	jsc	arm64-v8a	9,483,545	+44
android	jsc	armeabi-v7a	n/a	--
android	jsc	x86	n/a	--
android	jsc	x86_64	n/a	--

Base commit: 0c483d3
Branch: main

packages/cli-commands/src/commands/bundle/bundle.js

packages/cli-commands/src/commands/start/index.js

packages/cli-commands/src/commands/start/startServerInNewWindow.js

acoates-ms · 2023-08-07T17:05:14Z

How will we integrate additional platform specific logic into this?
Currently this code has platform specific logic to deal with assets (generate assetcatalogs on ios and use android specific paths for different scale factors).

We need to add specific logic for other platforms too. react-native-community/cli#2002 was going to modify the rn-cli to allow out of tree platforms to plugin platform specific functionality for asset handing, which needs to be part of the bundle step.

The rn-cli knows about the rn-config object which has all the infra to support out of tree platforms and therefore platform specific extension points. Moving the code here means we lose out on all the infra to handle out of tree platforms. - thus breaking rn-windows and rn-macos.

huntie · 2023-08-07T17:24:39Z

@acoates-ms This is a a relocation of the cli-plugin-metro package, and theoretically your PR can be reopened against this repo after these changes!

Moving the code here means we lose out on all the infra to handle out of tree platforms

AFAIK, we will retain support for out-of-tree platforms with this move.

@react-native/cli-commands remains a React Native CLI plugin, and continues to:

Interface with the plugin command interface.
Receive the Config context object from CLI (the integration point which you've extended in your PR).
Depend on other CLI packages, e.g. @react-native-community/cli-tools.

acoates-ms · 2023-08-07T17:37:49Z

Ah, I thought the config stuff was all outside of the repo. I guess it just means there is a repo cycle of dependencies between the packages, which is annoying to deal with. But good to hear that this wouldn't block that.

huntie · 2023-08-07T17:57:28Z

@acoates-ms Yeah the cross-repo iteration friction is shifted, per the summary.

Changes to this package can now be made inside the react-native repo and will run from source 🙂 (see Add shared monorepo build setup #38718), which should provide faster iteration when working with this code.
Crucially, there is no circular package dependency, as we will apply these commands via the CLI's plugin mechanism (packages/react-native/react-native.config.js), and are detaching references in CLI in feat: add invoking CLI's scripts for launching Metro in run-ios command react-native-community/cli#2021.

Sync to the latest changes and don't depend on it for bundling. `@react-native-community/cli-plugin-metro` is being moved into the `react-native` repository. In the process, it was renamed and its API surface has been reduced to the bare minimum. `buildBundleWithConfig`, which we need to pass our custom config to the bundler, has also been axed. For more details, see facebook/react-native#38795.

facebook-github-bot · 2023-08-09T16:28:20Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: f704ff908eec1e7c2dd5aa71ee48204e5bd0a571

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: ebd78383317b9b9f806d22a1e3d0e375b3b0f0aa

facebook-github-bot · 2023-08-09T17:11:18Z

This pull request was exported from Phabricator. Differential Revision: D46801501

packages/community-cli-plugin/src/commands/bundle/bundle.js

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: 043e2aa86c23fbe8393368cfbdae2a0da3feda8e

facebook-github-bot · 2023-08-10T10:17:20Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: 0697f2d9af49034cbf39245bea59e9fe30c14dd0

facebook-github-bot · 2023-08-10T12:19:43Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: a21e7b563b0c9e0348034299fe25be7c5973e921

facebook-github-bot · 2023-08-10T12:27:40Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: 03e58ec37ef34f163a041ea67812191b75811dde

facebook-github-bot · 2023-08-10T12:34:49Z

This pull request was exported from Phabricator. Differential Revision: D46801501

Summary: Pull Request resolved: facebook#38795 ## Context RFC: Decoupling Flipper from React Native core: react-native-community/discussions-and-proposals#641 ## Changes Inits new package `react-native/community-cli-plugin`. This migrates [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) into the React Native repo, to enable faster iteration by the React Native core team. Specifically: - This package contains several `metro` dependencies, which when removed from CLI will no longer require us to ship new CLI releases to get Metro patches and features to users. - This package contains the `start`, `bundle`, and `ram-bundle` commands (central to the React Native development experience), for which we have incoming debugging-related changes. - This package now **only** exports commands to be attached via a RN CLI plugin. With this move, we're aiming to **internalise** the default implementations of these dev commands within React Native — other RN CLI plugins can continue to override these, but must do so wholesale. (See also the recent fix for this: react-native-community/cli#1999.) In V15: - (Microsoft feedback) Re-export `unstable_buildBundleWithConfig`, marking as unstable. This gives us a time buffer to consider how we repackage this functionality in future. The package source has been converted from TypeScript to Flow, with a number of new `flow-typed/` defs added to meet type coverage requirements. ## To dos - For now, we aren't removing the existing [`react-native-community/cli-plugin-metro`](https://github.com/react-native-community/cli/tree/main/packages/cli-plugin-metro) source — until later PRs consolidate this move by changing dependencies in the `react-native` package. - **Exported API is reduced!**: I'm working with szymonrybczak to decouple references from RN CLI packages react-native-community/cli#2021. Changelog: [Internal] Reviewed By: motiz88 Differential Revision: D46801501 fbshipit-source-id: b25b71677cca0cc1007ddc9f59615e1585b3bf88

facebook-github-bot · 2023-08-10T13:14:29Z

This pull request was exported from Phabricator. Differential Revision: D46801501

github-actions · 2023-08-10T19:30:56Z

This pull request was successfully merged by @huntie in e199880.

^{When will my fix make it into a release? | Upcoming Releases}

#41370) Summary: Recently, both `metro-inspector-proxy`(#39045) and `react-native-community/cli-plugin-metro`(#38795) were moved to this repo and in the process of moving these packages, the `exports` field inside package.json was added, only exporting the `index.js` file. The problem is that Expo CLI (and possibly other community packages) rely on functions and classes that are not exported in the `index.js` file, e.g. Importing the InspectorProxy class from `react-native/dev-middleware/dist/inspector-proxy/InspectorProxy`. Normally this wouldn't be a problem and we would just import from `dist/` but due to the `exports` field, attempting to import from any other file not specified on this field will result in a `ERR_PACKAGE_PATH_NOT_EXPORTED` error. As a short-term fix, we should create `unstable_`-prefixed exports of individual features Expo currently depends on. ## Changelog: [INTERNAL] [CHANGED] - Expose unstable_InspectorProxy and unstable_Device from `react-native/dev-middleware` Pull Request resolved: #41370 Test Plan: N / A Reviewed By: robhogan Differential Revision: D51163134 Pulled By: blakef fbshipit-source-id: e67adaedc4fc64131e4c9dd8383c9877b8202283

facebook#41370) Summary: Recently, both `metro-inspector-proxy`(facebook#39045) and `react-native-community/cli-plugin-metro`(facebook#38795) were moved to this repo and in the process of moving these packages, the `exports` field inside package.json was added, only exporting the `index.js` file. The problem is that Expo CLI (and possibly other community packages) rely on functions and classes that are not exported in the `index.js` file, e.g. Importing the InspectorProxy class from `react-native/dev-middleware/dist/inspector-proxy/InspectorProxy`. Normally this wouldn't be a problem and we would just import from `dist/` but due to the `exports` field, attempting to import from any other file not specified on this field will result in a `ERR_PACKAGE_PATH_NOT_EXPORTED` error. As a short-term fix, we should create `unstable_`-prefixed exports of individual features Expo currently depends on. ## Changelog: [INTERNAL] [CHANGED] - Expose unstable_InspectorProxy and unstable_Device from `react-native/dev-middleware` Pull Request resolved: facebook#41370 Test Plan: N / A Reviewed By: robhogan Differential Revision: D51163134 Pulled By: blakef fbshipit-source-id: e67adaedc4fc64131e4c9dd8383c9877b8202283

facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Aug 4, 2023

huntie force-pushed the export-D46801501 branch from e2c3bf0 to a37d065 Compare August 4, 2023 17:16

huntie force-pushed the export-D46801501 branch from a37d065 to 3d6580c Compare August 7, 2023 09:10

tido64 reviewed Aug 7, 2023

View reviewed changes

szymonrybczak reviewed Aug 7, 2023

View reviewed changes

packages/cli-commands/src/commands/bundle/bundle.js Outdated Show resolved Hide resolved

szymonrybczak reviewed Aug 7, 2023

View reviewed changes

packages/cli-commands/src/commands/bundle/bundle.js Outdated Show resolved Hide resolved

szymonrybczak reviewed Aug 7, 2023

View reviewed changes

packages/cli-commands/src/commands/start/index.js Outdated Show resolved Hide resolved

szymonrybczak reviewed Aug 7, 2023

View reviewed changes

packages/cli-commands/src/commands/start/startServerInNewWindow.js Outdated Show resolved Hide resolved

kelset mentioned this pull request Aug 7, 2023

Use @react-native-communty/cli-plugin-metro project to bundling, and remove custom code in CLI and related packages microsoft/rnx-kit#328

Closed

rnsdkbot mentioned this pull request Aug 8, 2023

Version Packages microsoft/rnx-kit#2578

Merged

huntie force-pushed the export-D46801501 branch from 3d6580c to d39004b Compare August 9, 2023 16:28

huntie force-pushed the export-D46801501 branch from d39004b to 357cd4a Compare August 9, 2023 17:11

szymonrybczak reviewed Aug 10, 2023

View reviewed changes

packages/community-cli-plugin/src/commands/bundle/bundle.js Show resolved Hide resolved

huntie force-pushed the export-D46801501 branch from 357cd4a to 9ba5916 Compare August 10, 2023 10:17

huntie force-pushed the export-D46801501 branch from 9ba5916 to 86a6ded Compare August 10, 2023 12:19

huntie force-pushed the export-D46801501 branch from 86a6ded to 6a19144 Compare August 10, 2023 12:27

huntie force-pushed the export-D46801501 branch from 6a19144 to 861cd3e Compare August 10, 2023 12:34

huntie force-pushed the export-D46801501 branch from 861cd3e to b8e7c00 Compare August 10, 2023 13:14

facebook-github-bot closed this in e199880 Aug 10, 2023

github-actions bot added the Merged This PR has been merged. label Aug 10, 2023

huntie mentioned this pull request Aug 11, 2023

breaking: Delete source for cli-plugin-metro, remove from builtin commands react-native-community/cli#2055

Merged

huntie deleted the export-D46801501 branch August 29, 2023 16:15

tido64 mentioned this pull request Sep 14, 2023

Build assets-registry for general consumption #39440

Closed

gabrieldonadel mentioned this pull request Nov 8, 2023

Expose unstable_InspectorProxy and unstable_Device from dev-middleware #41370

Closed

szymonrybczak mentioned this pull request Dec 12, 2023

fix: get commands options and description from new package callstack/repack#477

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate cli-plugin-metro into repo #38795

Migrate cli-plugin-metro into repo #38795

huntie commented Aug 4, 2023 •

edited

Loading

facebook-github-bot commented Aug 4, 2023

facebook-github-bot commented Aug 4, 2023

facebook-github-bot commented Aug 7, 2023

tido64 Aug 7, 2023

huntie Aug 7, 2023

tido64 Aug 8, 2023

huntie Aug 8, 2023

kelset Aug 8, 2023

huntie Aug 8, 2023

tido64 Aug 8, 2023 •

edited

Loading

motiz88 Aug 8, 2023

motiz88 Aug 9, 2023

acoates-ms Aug 9, 2023

analysis-bot commented Aug 7, 2023 •

edited

Loading

acoates-ms commented Aug 7, 2023

huntie commented Aug 7, 2023

acoates-ms commented Aug 7, 2023

huntie commented Aug 7, 2023 •

edited

Loading

facebook-github-bot commented Aug 9, 2023

facebook-github-bot commented Aug 9, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

github-actions bot commented Aug 10, 2023

Migrate cli-plugin-metro into repo #38795

Migrate cli-plugin-metro into repo #38795

Conversation

huntie commented Aug 4, 2023 • edited Loading

Context

Changes

To dos

facebook-github-bot commented Aug 4, 2023

facebook-github-bot commented Aug 4, 2023

facebook-github-bot commented Aug 7, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tido64 Aug 8, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

RN 0.73

Migration path (RN 0.74+)

Choose a reason for hiding this comment

analysis-bot commented Aug 7, 2023 • edited Loading

acoates-ms commented Aug 7, 2023

huntie commented Aug 7, 2023

acoates-ms commented Aug 7, 2023

huntie commented Aug 7, 2023 • edited Loading

facebook-github-bot commented Aug 9, 2023

facebook-github-bot commented Aug 9, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

facebook-github-bot commented Aug 10, 2023

github-actions bot commented Aug 10, 2023

huntie commented Aug 4, 2023 •

edited

Loading

tido64 Aug 8, 2023 •

edited

Loading

analysis-bot commented Aug 7, 2023 •

edited

Loading

huntie commented Aug 7, 2023 •

edited

Loading