react-native-bundle-discovery-ui 2.1.0-rc.2 → 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 \
@@ -99,20 +104,6 @@ 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
@@ -126,8 +117,6 @@ npx react-native-bundle-discovery-ui rsdoctor-data.json --format rsdoctor
 | 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.                                                                        |
 
-
-
 **⚠️ Be cautious sharing reports with `includeCode` enabled**
 
 Enabling this option significantly increases report size and **embeds the source code** of your assets.
@@ -138,22 +127,19 @@ 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)) |
-| 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.     |
+| 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.                                                                                        |
 
 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 {createResolveRequest} = require('react-native-bundle-discovery');
+const { createResolveRequest } = require("react-native-bundle-discovery");
 const resolveRequest = createResolveRequest({
   removePromisePolyfill: true,
-  removeOldRenderer: true,
   removeUTFSequence: true,
 });
 const config = {

package/lib/bin.js

@@ -5,10 +5,10 @@ function printHelp() {
   console.log(`react-native-bundle-discovery-ui
 
 Usage:
-  react-native-bundle-discovery-ui <file> [port] [--format <rsdoctor|metro>] [--verbose] # <- server can be omitted
-  react-native-bundle-discovery-ui server <file> [port] [--format <rsdoctor|metro>] [--verbose]
+  react-native-bundle-discovery-ui <file> [port] [--verbose] # <- server can be omitted
+  react-native-bundle-discovery-ui server <file> [port] [--verbose]
 
-  react-native-bundle-discovery-ui build <file> [--output <path>] [--clean] [--single-file] [--format <rsdoctor|metro>] [--verbose]
+  react-native-bundle-discovery-ui build <file> [--output <path>] [--clean] [--single-file] [--verbose]
 
 Commands:
   server <file> [port]  Run a web server to show a Metro bundler stat report
@@ -19,7 +19,6 @@ Options:
   -o, --output <path>    Path for a build result (default: .bundle-discovery)
   -c, --clean            Clean output directory before writing build files (default: true)
   -s, --single-file      Output report build as a single HTML file (default: true)
-  -f, --format <name>    Report format: rsdoctor | metro (default: metro)
   -p, --port <port>      Port for server command (same as [port], default: 8079)
   -h, --help             Show help
 `);
@@ -38,24 +37,16 @@ const argv = minimist(process.argv.slice(2), {
     o: "output",
     c: "clean",
     s: "single-file",
-    f: "format",
     p: "port",
   },
   boolean: ["verbose", "help", "clean", "single-file"],
-  string: ["output", "format"],
+  string: ["output"],
   default: {
     clean: true,
     "single-file": true,
-    format: "metro",
   },
 });
 
-const SUPPORTED_FORMATS = new Set(["rsdoctor", "metro"]);
-const format = argv.format;
-if (!SUPPORTED_FORMATS.has(format)) {
-  fail(`Invalid format: ${format}. Supported formats: rsdoctor | metro`);
-}
-
 const command = argv._[0];
 const isServerCommand = command === "server";
 const isBuildCommand = command === "build";
@@ -81,7 +72,7 @@ if (isServerCommand || isImplicitServerCommand) {
   }
 
   const { serve } = require("./server.js");
-  return serve(file, port, format, Boolean(argv.verbose));
+  return serve(file, port, Boolean(argv.verbose));
 }
 
 if (isBuildCommand) {
@@ -96,7 +87,6 @@ if (isBuildCommand) {
     argv.output,
     argv.clean,
     argv["single-file"],
-    format,
     Boolean(argv.verbose),
   );
 }

package/lib/build.js

@@ -3,8 +3,9 @@ const path = require("path");
 const chalk = require("chalk");
 const { build } = require("@discoveryjs/cli");
 const config = require("../.discoveryrc.js");
+const { isRsdoctorReport } = require("./utils.js");
 
-function buildHtmlPage(filePath, output, clean, singleFile, format, verbose) {
+function buildHtmlPage(filePath, output, clean, singleFile, verbose) {
   if (verbose) {
     console.info(
       `Building HTML page from file: ${chalk.green(filePath)} options: ${JSON.stringify(
@@ -56,7 +57,7 @@ module.exports = ${JSON.stringify(
       1,
     ).replace(
       `"<tmp>"`,
-      format === "rsdoctor"
+      isRsdoctorReport(fullJsonPath)
         ? `() => transformRSDoctorData(require("${fullJsonPath}"))`
         : `() => require("${fullJsonPath}")`,
     )};`,

package/lib/server.js

@@ -4,8 +4,9 @@ const chalk = require("chalk");
 const { createServer } = require("@discoveryjs/cli");
 const { silent } = require("@discoveryjs/cli/lib/shared/utils.js");
 const config = require("../.discoveryrc.js");
+const { isRsdoctorReport } = require("./utils.js");
 
-function serve(filePath, port, format, verbose) {
+function serve(filePath, port, verbose) {
   const PORT = process.env.PORT || port;
 
   if (verbose) {
@@ -55,8 +56,7 @@ module.exports = ${JSON.stringify(
       1,
     ).replace(
       `"<tmp>"`,
-
-      format === "rsdoctor"
+      isRsdoctorReport(fullJsonPath)
         ? `() => transformRSDoctorData(require("${fullJsonPath}"))`
         : `() => require("${fullJsonPath}")`,
     )};`,

package/lib/utils.js

@@ -0,0 +1,23 @@
+const chalk = require("chalk");
+
+const isRsdoctorReport = (fullJsonPath) => {
+  try {
+    const json = require(fullJsonPath);
+    if (
+      Array.isArray(json?.data?.moduleGraph?.modules) ||
+      Array.isArray(json?.data?.packageGraph?.packages)
+    ) {
+      console.info(
+        `Detected ${chalk.red("rsdoctor")} format in the input file: ${chalk.green(fullJsonPath)}`,
+      );
+      return true;
+    }
+    return false;
+  } catch {
+    return false;
+  }
+};
+
+module.exports = {
+  isRsdoctorReport,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-bundle-discovery-ui",
-  "version": "2.1.0-rc.2",
+  "version": "2.2.0-rc.1",
   "main": "lib/bin.js",
   "bin": "lib/bin.js",
   "repository": "git@github.com:retyui/react-native-bundle-discovery.git",
@@ -38,5 +38,9 @@
     "queryHelpers.js",
     ".discoveryrc.js"
   ],
-  "packageManager": "yarn@4.18.0"
+  "packageManager": "yarn@4.18.0",
+  "devDependencies": {
+    "acorn": "^8.18.0",
+    "acorn-walk": "^8.3.5"
+  }
 }

package/queryHelpers.js

@@ -17,6 +17,9 @@ const helpers = {
       .then((formatted) => {
         helpers.prettifyMap.set(sourceStr, formatted);
         return formatted;
+      })
+      .catch(() => {
+        return sourceStr;
       });
   },
   askChatGPTAboutPackages() {