react-native-bundle-discovery 2.1.0 → 2.2.0-rc.1

package/README.md

@@ -3,13 +3,11 @@
 [![react-native-bundle-discovery on npm](https://badgen.net/npm/v/react-native-bundle-discovery)](https://www.npmjs.com/package/react-native-bundle-discovery)
 [![react-native-bundle-discovery downloads](https://badgen.net/npm/dm/react-native-bundle-discovery)](https://www.npmtrends.com/react-native-bundle-discovery)
 
-
 A simple package that helps developers visualize and analyze the bundle size of React Native apps.
 With this tool, you can easily explore your app's codebase, identify large or heavy packages, and inspect the structure of modules and code within your project.
 
 <img width="800" alt="" src="./assets/img.png" />
 
-
 ### Packages:
 
 - `react-native-bundle-discovery` - simple JS library to generate a JSON report of the bundle.
@@ -17,13 +15,20 @@ With this tool, you can easily explore your app's codebase, identify large or he
 - `react-native-bundle-discovery-cli` - CLI to analyze the bundle report.
 - `react-native-bundle-discovery-rozenite-plugin` - Rozenite plugin to integrate UI tool to [React Native DevTools](https://reactnative.dev/docs/react-native-devtools).
 
+---
+
 ### Setup:
 
 There are two ways to install the package:
 
-1. As in independent tool (UI + CLI) 
+1. As in independent tool (UI + CLI)
 2. Or as a [Rozenite](_rozenite/README.md) plugin (see: [_rozenite/README.md](_rozenite/README.md))
 
+
+> 🚀 If you've migrated to [Re.Pack](https://re-pack.dev/), you can still use this package to analyze 📊 JS bundles 📦 generated by Re.Pack.
+>
+> 💡 Learn more: 📖 [Re.Pack.md](./Re.Pack.md)
+
 #### 1. Install (independent tool)
 
 ```bash
@@ -62,7 +67,7 @@ See: [_rozenite/README.md](_rozenite/README.md)
 
 ### 3. Build the app
 
-As example, for iOS you can run the following command, and it will generate the `metro-stats.json` file in the root of your project:
+For example, for iOS you can run the following command, and it will generate the `metro-stats.json` file in the root of your project:
 
 ```bash
 npx react-native bundle \
@@ -112,30 +117,34 @@ npx react-native-bundle-discovery-ui build metro-stats.json
 | outputJsonPath: string | `<root>/metro-stats.json` | The path where the JSON report will be saved. Defaults to `metro-stats.json` in project root.                                            |
 | includeCode: boolean   | `true`                    | Whether to include the source and output code in the JSON report.                                                                        |
 
-### `createProcessModuleFilter(options: ProcessModuleFilterOptions)`
+**⚠️ Be cautious sharing reports with `includeCode` enabled**
+
+Enabling this option significantly increases report size and **embeds the source code** of your assets.
 
+If your code is proprietary 🔒, share these reports responsibly 🤝!
+
+---
 
-| Prop                   | Default value             | Description                   |
-| ---------------------- | ------------------------- | ----------------------------- |
-| removePromisePolyfill: boolean | `false` | Remove the Promise polyfill as Hermes provide own impl. (issue: [#57702](https://github.com/react/react-native/issues/57702)) |
-| removeOldRenderer: boolean     | `false` | Whether to remove the old renderer. Set to `true` when New Arch is enabled    |
-| removeNewRenderer: boolean     | `false` | Whether to remove the new renderer. Set to `true` when New Arch is disabled    |
-| removeUTFSequence: boolean     | `false` | Remove useless undocumented RN module.     |
+### `createResolveRequest(options: ResolveRequestOptions)`
 
+| Prop                           | Default value | Description                                                                                                                   |
+| ------------------------------ | ------------- | ----------------------------------------------------------------------------------------------------------------------------- |
+| removePromisePolyfill: boolean | `false`       | Remove the Promise polyfill as Hermes provide own impl. (issue: [#57702](https://github.com/react/react-native/issues/57702)) |
+| removeUTFSequence: boolean     | `false`       | Remove useless undocumented RN module.                                                                                        |
 
-Simple helper that devs can use to filter out unnecessary modules from the bundle report.
+Helper that creates a custom resolve request function to filter out unnecessary modules from the bundle.
 You can get recommendations during the analysis of the bundle report using the CLI tool.
 
 ```js
 // metro.config.js
-const {createProcessModuleFilter} = require('react-native-bundle-discovery');
+const { createResolveRequest } = require("react-native-bundle-discovery");
+const resolveRequest = createResolveRequest({
+  removePromisePolyfill: true,
+  removeUTFSequence: true,
+});
 const config = {
-  serializer: {
-    processModuleFilter: createProcessModuleFilter({
-      removePromisePolyfill: true,
-      removeOldRenderer: true,
-      removeUTFSequence: true,
-    }),
+  resolver: {
+    resolveRequest,
   },
 };
 ```

package/index.js

@@ -215,8 +215,6 @@ function createSerializer({
  */
 const empty = { type: "empty" };
 const createResolveRequest = ({
-  removePromisePolyfill = false, // Remove useless polyfill (Hermes already has Promise)
-  removeOldRenderer = false, // Should be true when new ARCH is enabled
   removeNewRenderer = false, // Should be true when new ARCH is disabled
   removeUTFSequence = false, // Remove useless code
 } = {}) => {
@@ -231,20 +229,6 @@ const createResolveRequest = ({
     ) {
       return empty;
     }
-    if (
-      removePromisePolyfill &&
-      result?.filePath?.endsWith("/react-native/Libraries/Promise.js")
-    ) {
-      return empty;
-    }
-    if (
-      removeOldRenderer &&
-      result?.filePath?.endsWith(
-        "/react-native/Libraries/Renderer/shims/ReactNative.js",
-      )
-    ) {
-      return empty;
-    }
     if (
       removeNewRenderer &&
       result?.filePath?.endsWith(
@@ -264,8 +248,6 @@ const createResolveRequest = ({
 const createProcessModuleFilter =
   ({
     removePromisePolyfill = false, // Remove useless polyfill (Hermes already has Promise)
-    removeOldRenderer = false, // Should be true when new ARCH is enabled
-    removeNewRenderer = false, // Should be true when new ARCH is disabled
     removeUTFSequence = false, // Remove useless code
   } = {}) =>
   (module) => {
@@ -275,22 +257,6 @@ const createProcessModuleFilter =
     ) {
       return false;
     }
-    if (
-      removeOldRenderer &&
-      module.path.endsWith(
-        "/react-native/Libraries/Renderer/shims/ReactNative.js",
-      )
-    ) {
-      return false;
-    }
-    if (
-      removeNewRenderer &&
-      module.path.endsWith(
-        "/react-native/Libraries/Renderer/shims/ReactFabric.js",
-      )
-    ) {
-      return false;
-    }
     if (
       removeUTFSequence &&
       module.path.endsWith("/react-native/Libraries/UTFSequence.js")
@@ -302,7 +268,15 @@ const createProcessModuleFilter =
   };
 
 module.exports = {
-  createResolveRequest,
   createSerializer,
-  createProcessModuleFilter,
+  createResolveRequest,
+  get createProcessModuleFilter() {
+    console.warn(
+      `[${NAME}]: ⚠️'createProcessModuleFilter' is deprecated. Use 'createResolveRequest' instead.`,
+    );
+    return createProcessModuleFilter;
+  },
+  get BundleDiscoveryPlugin() {
+    return require("./webpack").BundleDiscoveryPlugin;
+  },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-bundle-discovery",
-  "version": "2.1.0",
+  "version": "2.2.0-rc.1",
   "main": "index.js",
   "repository": "git@github.com:retyui/react-native-bundle-discovery.git",
   "author": "David <4661784+retyui@users.noreply.github.com>",
@@ -9,7 +9,9 @@
     "prepublishOnly": "cp ../README.md README.md"
   },
   "dependencies": {
-    "chalk": "^4.1.2"
+    "chalk": "^4.1.2",
+    "acorn": "^8.0.4",
+    "acorn-walk": "^8.0.0"
   },
   "peerDependencies": {
     "metro": "*"