react-native-bundle-discovery 2.0.1 → 2.1.0-rc.2

package/README.md

@@ -99,6 +99,20 @@ npx react-native-bundle-discovery-ui metro-stats.json [--port <port>]
 npx react-native-bundle-discovery-ui build metro-stats.json
 ```
 
+---
+
+
+### Re.Pack Support
+
+If you've migrated to [Re.Pack](https://re-pack.dev/docs/guides/bundle-analysis), you can still use this package to analyze bundles generated by Re.Pack.
+
+Learn more: [Rsdoctor.md](./Rsdoctor.md)
+
+```bash
+npx react-native-bundle-discovery-ui rsdoctor-data.json --format rsdoctor
+```
+
+
 ---
 
 ### `react-native-bundle-discovery` JS API
@@ -112,9 +126,18 @@ 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 🤝!
+
+---
+
+### `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)) |
@@ -122,20 +145,20 @@ npx react-native-bundle-discovery-ui build metro-stats.json
 | 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.     |
 
-
-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,
+  removeOldRenderer: true,
+  removeUTFSequence: true,
+});
 const config = {
-  serializer: {
-    processModuleFilter: createProcessModuleFilter({
-      removePromisePolyfill: true,
-      removeOldRenderer: true,
-      removeUTFSequence: true,
-    }),
+  resolver: {
+    resolveRequest,
   },
 };
 ```

package/index.js

@@ -197,6 +197,70 @@ function createSerializer({
   return customSerializer;
 }
 
+/**
+ * Usage:
+ *
+ * ```js
+ * // metro.config.js
+ * const { createResolveRequest } = require("react-native-bundle-discovery");
+ * const resolveRequest = createResolveRequest({
+ *   removePromisePolyfill: true,
+ * });
+ * const config = {
+ *   resolver: {
+ *     resolveRequest,
+ *   },
+ * };
+ * ```
+ */
+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
+} = {}) => {
+  const resolveRequest = (context, ...rest) => {
+    const result = context.resolveRequest(context, ...rest);
+    if (context.dev) {
+      return result;
+    }
+    if (
+      removeUTFSequence &&
+      result?.filePath?.endsWith("/react-native/Libraries/UTFSequence.js")
+    ) {
+      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(
+        "/react-native/Libraries/Renderer/shims/ReactFabric.js",
+      )
+    ) {
+      return empty;
+    }
+    return result;
+  };
+  return resolveRequest;
+};
+
+/**
+ * @deprecated Use `createResolveRequest` instead.
+ */
 const createProcessModuleFilter =
   ({
     removePromisePolyfill = false, // Remove useless polyfill (Hermes already has Promise)
@@ -237,4 +301,8 @@ const createProcessModuleFilter =
     return true;
   };
 
-module.exports = { createSerializer, createProcessModuleFilter };
+module.exports = {
+  createResolveRequest,
+  createSerializer,
+  createProcessModuleFilter,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-bundle-discovery",
-  "version": "2.0.1",
+  "version": "2.1.0-rc.2",
   "main": "index.js",
   "repository": "git@github.com:retyui/react-native-bundle-discovery.git",
   "author": "David <4661784+retyui@users.noreply.github.com>",