From 5fd9db732dff1b99c096bffef9a565b594c788de Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Markb=C3=A5ge?= Date: Tue, 8 Dec 2020 08:08:57 -0500 Subject: [PATCH] [Flight] Rename react-transport-... packages to react-server-... (#20403) * Move files * Update paths * Rename import variables * Rename /server to /writer This is mainly because "React Server Server" is weird so we need another dimension. * Use "react-server" convention to enforce that writer is only loaded in a server --- .eslintrc.js | 4 +- fixtures/flight-browser/index.html | 10 ++-- fixtures/flight/config/webpack.config.js | 2 +- fixtures/flight/loader/index.js | 2 +- fixtures/flight/server/cli.server.js | 2 +- fixtures/flight/server/handler.server.js | 2 +- fixtures/flight/src/index.js | 6 +- ...ReactFlightClientHostConfig.dom-browser.js | 2 +- .../ReactFlightClientHostConfig.dom-relay.js | 2 +- .../forks/ReactFlightClientHostConfig.dom.js | 2 +- ...eactFlightClientHostConfig.native-relay.js | 2 +- .../index.js | 0 .../package.json | 4 +- .../server.js | 0 .../src/ReactFlightDOMRelayClient.js | 0 .../ReactFlightDOMRelayClientHostConfig.js | 0 .../src/ReactFlightDOMRelayProtocol.js | 0 .../src/ReactFlightDOMRelayServer.js | 0 .../ReactFlightDOMRelayServerHostConfig.js | 0 .../src/__mocks__/JSResourceReference.js | 0 .../ReactFlightDOMRelayClientIntegration.js | 0 .../ReactFlightDOMRelayServerIntegration.js | 0 .../ReactFlightDOMRelay-test.internal.js | 4 +- .../README.md | 2 +- .../esm/package.json | 0 .../react-server-dom-webpack-node-loader.js} | 0 .../index.js | 0 .../node-register.js | 0 .../npm/esm/package.json | 0 .../react-server-dom-webpack/npm/index.js | 7 +++ .../npm/node-register.js | 3 + .../react-server-dom-webpack/npm/plugin.js | 3 + .../npm/writer.browser.server.js | 7 +++ .../react-server-dom-webpack/npm/writer.js | 6 ++ .../npm/writer.node.server.js | 7 +++ .../package.json | 28 ++++++---- .../plugin.js | 0 .../ReactFlightClientWebpackBundlerConfig.js | 0 .../src/ReactFlightDOMClient.js | 0 .../src/ReactFlightDOMServerBrowser.js | 0 .../src/ReactFlightDOMServerNode.js | 0 .../ReactFlightServerWebpackBundlerConfig.js | 0 .../src/ReactFlightWebpackNodeLoader.js | 0 .../src/ReactFlightWebpackNodeRegister.js | 0 .../src/ReactFlightWebpackPlugin.js | 2 +- .../src/__tests__/ReactFlightDOM-test.js | 36 ++++++------ .../__tests__/ReactFlightDOMBrowser-test.js | 12 ++-- .../writer.browser.server.js} | 0 .../writer.js} | 5 +- .../writer.node.server.js} | 0 .../index.js | 0 .../package.json | 4 +- .../server.js | 0 .../src/ReactFlightNativeRelayClient.js | 0 .../ReactFlightNativeRelayClientHostConfig.js | 0 .../src/ReactFlightNativeRelayProtocol.js | 0 .../src/ReactFlightNativeRelayServer.js | 0 .../ReactFlightNativeRelayServerHostConfig.js | 0 .../src/__mocks__/JSResourceReferenceImpl.js | 0 ...ReactFlightNativeRelayClientIntegration.js | 0 ...ReactFlightNativeRelayServerIntegration.js | 0 .../ReactFlightNativeRelay-test.internal.js | 4 +- ...actFlightNativeRelay-test.internal.js.snap | 0 .../ReactFlightServerConfig.dom-browser.js | 2 +- .../ReactFlightServerConfig.dom-relay.js | 2 +- .../src/forks/ReactFlightServerConfig.dom.js | 2 +- .../ReactFlightServerConfig.native-relay.js | 2 +- .../react-transport-dom-webpack/npm/index.js | 7 --- .../npm/node-register.js | 3 - .../react-transport-dom-webpack/npm/plugin.js | 3 - .../npm/server.browser.js | 7 --- .../react-transport-dom-webpack/npm/server.js | 3 - .../npm/server.node.js | 7 --- scripts/rollup/bundles.js | 56 +++++++++---------- scripts/rollup/packaging.js | 2 +- scripts/shared/inlinedHostConfigs.js | 35 ++++++------ 76 files changed, 154 insertions(+), 147 deletions(-) rename packages/{react-transport-dom-relay => react-server-dom-relay}/index.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/package.json (76%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/server.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/ReactFlightDOMRelayClient.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/ReactFlightDOMRelayClientHostConfig.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/ReactFlightDOMRelayProtocol.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/ReactFlightDOMRelayServer.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/ReactFlightDOMRelayServerHostConfig.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/__mocks__/JSResourceReference.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/__mocks__/ReactFlightDOMRelayClientIntegration.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/__mocks__/ReactFlightDOMRelayServerIntegration.js (100%) rename packages/{react-transport-dom-relay => react-server-dom-relay}/src/__tests__/ReactFlightDOMRelay-test.internal.js (97%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/README.md (74%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/esm/package.json (100%) rename packages/{react-transport-dom-webpack/esm/react-transport-dom-webpack-node-loader.js => react-server-dom-webpack/esm/react-server-dom-webpack-node-loader.js} (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/index.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/node-register.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/npm/esm/package.json (100%) create mode 100644 packages/react-server-dom-webpack/npm/index.js create mode 100644 packages/react-server-dom-webpack/npm/node-register.js create mode 100644 packages/react-server-dom-webpack/npm/plugin.js create mode 100644 packages/react-server-dom-webpack/npm/writer.browser.server.js create mode 100644 packages/react-server-dom-webpack/npm/writer.js create mode 100644 packages/react-server-dom-webpack/npm/writer.node.server.js rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/package.json (56%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/plugin.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightClientWebpackBundlerConfig.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightDOMClient.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightDOMServerBrowser.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightDOMServerNode.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightServerWebpackBundlerConfig.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightWebpackNodeLoader.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightWebpackNodeRegister.js (100%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/ReactFlightWebpackPlugin.js (99%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/__tests__/ReactFlightDOM-test.js (91%) rename packages/{react-transport-dom-webpack => react-server-dom-webpack}/src/__tests__/ReactFlightDOMBrowser-test.js (80%) rename packages/{react-transport-dom-webpack/server.browser.js => react-server-dom-webpack/writer.browser.server.js} (100%) rename packages/{react-transport-dom-webpack/server.js => react-server-dom-webpack/writer.js} (52%) rename packages/{react-transport-dom-webpack/server.node.js => react-server-dom-webpack/writer.node.server.js} (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/index.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/package.json (76%) rename packages/{react-transport-native-relay => react-server-native-relay}/server.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/ReactFlightNativeRelayClient.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/ReactFlightNativeRelayClientHostConfig.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/ReactFlightNativeRelayProtocol.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/ReactFlightNativeRelayServer.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/ReactFlightNativeRelayServerHostConfig.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/__mocks__/JSResourceReferenceImpl.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/__mocks__/ReactFlightNativeRelayClientIntegration.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/__mocks__/ReactFlightNativeRelayServerIntegration.js (100%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/__tests__/ReactFlightNativeRelay-test.internal.js (95%) rename packages/{react-transport-native-relay => react-server-native-relay}/src/__tests__/__snapshots__/ReactFlightNativeRelay-test.internal.js.snap (100%) delete mode 100644 packages/react-transport-dom-webpack/npm/index.js delete mode 100644 packages/react-transport-dom-webpack/npm/node-register.js delete mode 100644 packages/react-transport-dom-webpack/npm/plugin.js delete mode 100644 packages/react-transport-dom-webpack/npm/server.browser.js delete mode 100644 packages/react-transport-dom-webpack/npm/server.js delete mode 100644 packages/react-transport-dom-webpack/npm/server.node.js diff --git a/.eslintrc.js b/.eslintrc.js index 715c6eb96074a..a456964bdbdb0 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -185,14 +185,14 @@ module.exports = { { files: [ 'packages/react-native-renderer/**/*.js', - 'packages/react-transport-native-relay/**/*.js', + 'packages/react-server-native-relay/**/*.js', ], globals: { nativeFabricUIManager: true, }, }, { - files: ['packages/react-transport-dom-webpack/**/*.js'], + files: ['packages/react-server-dom-webpack/**/*.js'], globals: { __webpack_chunk_load__: true, __webpack_require__: true, diff --git a/fixtures/flight-browser/index.html b/fixtures/flight-browser/index.html index 1fc2a4e195284..c48e3e4c9e63f 100644 --- a/fixtures/flight-browser/index.html +++ b/fixtures/flight-browser/index.html @@ -19,8 +19,8 @@

Flight Example

- - + +