Revert to CJS but allow dynamic imports in CJS

Author: sebmarkbage

packages/react-server-dom-webpack/esm/react-server-dom-webpack-client.node.unbundled.development.js renamed to packages/react-server-dom-webpack/client.node.unbundled.js

@@ -7,4 +7,4 @@
  * @flow
  */
 
-export * from '../src/ReactFlightDOMClientNode.js';
+export * from './src/ReactFlightDOMClientNode';

packages/react-server-dom-webpack/esm/react-server-dom-webpack-server.node.unbundled.development.js

@@ -0,0 +1,7 @@
+'use strict';
+
+if (process.env.NODE_ENV === 'production') {
+  module.exports = require('./cjs/react-server-dom-webpack-client.node.unbundled.production.min.js');
+} else {
+  module.exports = require('./cjs/react-server-dom-webpack-client.node.unbundled.development.js');
+}

packages/react-server-dom-webpack/esm/react-server-dom-webpack-server.node.unbundled.production.min.js

@@ -0,0 +1,7 @@
+'use strict';
+
+if (process.env.NODE_ENV === 'production') {
+  module.exports = require('./cjs/react-server-dom-webpack-server.node.unbundled.production.min.js');
+} else {
+  module.exports = require('./cjs/react-server-dom-webpack-server.node.unbundled.development.js');
+}

packages/react-server-dom-webpack/npm/client.node.unbundled.js

@@ -38,28 +38,23 @@
       "worker": "./client.edge.js",
       "node": {
         "webpack": "./client.node.js",
-        "production": "./esm/react-server-dom-webpack-client.node.unbundled.production.min.js",
-        "default": "./esm/react-server-dom-webpack-client.node.unbundled.development.js"
+        "default": "./client.node.unbundled.js"
       },
       "browser": "./client.browser.js",
       "default": "./client.browser.js"
     },
     "./client.browser": "./client.browser.js",
     "./client.edge": "./client.edge.js",
     "./client.node": "./client.node.js",
-    "./client.node.unbundled": {
-      "production": "./esm/react-server-dom-webpack-client.node.unbundled.production.min.js",
-      "default": "./esm/react-server-dom-webpack-client.node.unbundled.development.js"
-    },
+    "./client.node.unbundled": "./client.node.unbundled.js",
     "./server": {
       "react-server": {
         "workerd": "./server.edge.js",
         "edge-light": "./server.edge.js",
         "deno": "./server.browser.js",
         "node": {
           "webpack": "./server.node.js",
-          "production": "./esm/react-server-dom-webpack-server.node.unbundled.production.min.js",
-          "default": "./esm/react-server-dom-webpack-server.node.unbundled.development.js"
+          "default": "./server.node.unbundled.js"
         },
         "browser": "./server.browser.js"
       },
@@ -68,10 +63,7 @@
     "./server.browser": "./server.browser.js",
     "./server.edge": "./server.edge.js",
     "./server.node": "./server.node.js",
-    "./server.node.unbundled": {
-      "production": "./esm/react-server-dom-webpack-server.node.unbundled.production.min.js",
-      "default": "./esm/react-server-dom-webpack-server.node.unbundled.development.js"
-    },
+    "./server.node.unbundled": "./server.node.unbundled.js",
     "./node-loader": "./esm/react-server-dom-webpack-node-loader.production.min.js",
     "./node-register": "./node-register.js",
     "./src/*": "./src/*",

packages/react-server-dom-webpack/npm/server.node.unbundled.js

@@ -7,4 +7,4 @@
  * @flow
  */
 
-export * from '../src/ReactFlightDOMClientNode.js';
+export * from './src/ReactFlightDOMServerNode';

packages/react-server-dom-webpack/package.json

@@ -19,6 +19,7 @@ const Sync = require('./sync');
 const sizes = require('./plugins/sizes-plugin');
 const useForks = require('./plugins/use-forks-plugin');
 const stripUnusedImports = require('./plugins/strip-unused-imports');
+const dynamicImports = require('./plugins/dynamic-imports');
 const Packaging = require('./packaging');
 const {asyncRimRaf} = require('./utils');
 const codeFrame = require('@babel/code-frame');
@@ -332,6 +333,8 @@ function getPlugins(
     bundleType === RN_FB_PROFILING;
   const shouldStayReadable = isFBWWWBundle || isRNBundle || forcePrettyOutput;
   return [
+    // Keep dynamic imports as externals
+    dynamicImports(),
     {
       name: 'rollup-plugin-flow-remove-types',
       transform(code) {
@@ -410,6 +413,7 @@ function getPlugins(
         process_common_js_modules: false,
         rewrite_polyfills: false,
         inject_libraries: false,
+        allow_dynamic_import: true,
 
         // Don't let it create global variables in the browser.
         // https://github.com/facebook/react/issues/10909

packages/react-server-dom-webpack/esm/react-server-dom-webpack-client.node.unbundled.production.min.js renamed to packages/react-server-dom-webpack/server.node.unbundled.js

@@ -360,7 +360,7 @@ const bundles = [
     externals: ['react', 'util', 'async_hooks', 'react-dom'],
   },
   {
-    bundleTypes: [ESM_DEV, ESM_PROD],
+    bundleTypes: [NODE_DEV, NODE_PROD],
     moduleType: RENDERER,
     entry: 'react-server-dom-webpack/server.node.unbundled',
     global: 'ReactServerDOMServer',
@@ -398,7 +398,7 @@ const bundles = [
     externals: ['react', 'util'],
   },
   {
-    bundleTypes: [ESM_DEV, ESM_PROD],
+    bundleTypes: [NODE_DEV, NODE_PROD],
     moduleType: RENDERER,
     entry: 'react-server-dom-webpack/client.node.unbundled',
     global: 'ReactServerDOMClient',

scripts/rollup/build.js

@@ -0,0 +1,19 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+'use strict';
+
+module.exports = function dynamicImports() {
+  return {
+    name: 'scripts/rollup/plugins/dynamic-imports',
+    renderDynamicImport({targetModuleId}) {
+      if (targetModuleId === null) {
+        return {left: 'import(', right: ')'};
+      }
+      return null;
+    },
+  };
+};