diff --git a/packages/react-art/src/ReactART.js b/packages/react-art/src/ReactART.js index 71414f607c029..9e0778c1c5730 100644 --- a/packages/react-art/src/ReactART.js +++ b/packages/react-art/src/ReactART.js @@ -12,7 +12,7 @@ import { createContainer, updateContainer, injectIntoDevTools, -} from 'react-reconciler/inline.art'; +} from 'react-reconciler/src/ReactFiberReconciler'; import Transform from 'art/core/transform'; import Mode from 'art/modes/current'; import FastNoSideEffects from 'art/modes/fast-noSideEffects'; diff --git a/packages/react-dom/src/client/ReactDOM.js b/packages/react-dom/src/client/ReactDOM.js index 340cba54773e3..4056f0a2a9be1 100644 --- a/packages/react-dom/src/client/ReactDOM.js +++ b/packages/react-dom/src/client/ReactDOM.js @@ -35,7 +35,7 @@ import { attemptUserBlockingHydration, attemptContinuousHydration, attemptHydrationAtCurrentPriority, -} from 'react-reconciler/inline.dom'; +} from 'react-reconciler/src/ReactFiberReconciler'; import {createPortal as createPortalImpl} from 'shared/ReactPortal'; import {canUseDOM} from 'shared/ExecutionEnvironment'; import {setBatchingImplementation} from 'legacy-events/ReactGenericBatching'; diff --git a/packages/react-dom/src/client/ReactDOMLegacy.js b/packages/react-dom/src/client/ReactDOMLegacy.js index 97fbac293578a..c3ab91f8f932e 100644 --- a/packages/react-dom/src/client/ReactDOMLegacy.js +++ b/packages/react-dom/src/client/ReactDOMLegacy.js @@ -31,7 +31,7 @@ import { getPublicRootInstance, findHostInstance, findHostInstanceWithWarning, -} from 'react-reconciler/inline.dom'; +} from 'react-reconciler/src/ReactFiberReconciler'; import getComponentName from 'shared/getComponentName'; import invariant from 'shared/invariant'; import ReactSharedInternals from 'shared/ReactSharedInternals'; diff --git a/packages/react-dom/src/client/ReactDOMRoot.js b/packages/react-dom/src/client/ReactDOMRoot.js index 7cbbbbd939201..62cd9fcef034d 100644 --- a/packages/react-dom/src/client/ReactDOMRoot.js +++ b/packages/react-dom/src/client/ReactDOMRoot.js @@ -13,7 +13,7 @@ import type {ReactNodeList} from 'shared/ReactTypes'; // TODO: This type is shared between the reconciler and ReactDOM, but will // eventually be lifted out to the renderer. import type {FiberRoot} from 'react-reconciler/src/ReactFiberRoot'; -import {findHostInstanceWithNoPortals} from 'react-reconciler/inline.dom'; +import {findHostInstanceWithNoPortals} from 'react-reconciler/src/ReactFiberReconciler'; export type RootType = { render(children: ReactNodeList): void, @@ -45,7 +45,10 @@ import { DOCUMENT_FRAGMENT_NODE, } from '../shared/HTMLNodeType'; -import {createContainer, updateContainer} from 'react-reconciler/inline.dom'; +import { + createContainer, + updateContainer, +} from 'react-reconciler/src/ReactFiberReconciler'; import invariant from 'shared/invariant'; import {BlockingRoot, ConcurrentRoot, LegacyRoot} from 'shared/ReactRootTags'; diff --git a/packages/react-dom/src/events/EnterLeaveEventPlugin.js b/packages/react-dom/src/events/EnterLeaveEventPlugin.js index 4130cd709ad20..b996973b69177 100644 --- a/packages/react-dom/src/events/EnterLeaveEventPlugin.js +++ b/packages/react-dom/src/events/EnterLeaveEventPlugin.js @@ -21,7 +21,7 @@ import { getNodeFromInstance, } from '../client/ReactDOMComponentTree'; import {HostComponent, HostText} from 'shared/ReactWorkTags'; -import {getNearestMountedFiber} from 'react-reconciler/reflection'; +import {getNearestMountedFiber} from 'react-reconciler/src/ReactFiberTreeReflection'; import {enableModernEventSystem} from 'shared/ReactFeatureFlags'; const eventTypes = { diff --git a/packages/react-dom/src/events/ReactDOMEventListener.js b/packages/react-dom/src/events/ReactDOMEventListener.js index 647b7dc239297..040029c46d7f4 100644 --- a/packages/react-dom/src/events/ReactDOMEventListener.js +++ b/packages/react-dom/src/events/ReactDOMEventListener.js @@ -32,7 +32,7 @@ import { getNearestMountedFiber, getContainerFromFiber, getSuspenseInstanceFromFiber, -} from 'react-reconciler/reflection'; +} from 'react-reconciler/src/ReactFiberTreeReflection'; import {HostRoot, SuspenseComponent} from 'shared/ReactWorkTags'; import { type EventSystemFlags, diff --git a/packages/react-dom/src/events/ReactDOMEventReplaying.js b/packages/react-dom/src/events/ReactDOMEventReplaying.js index a72fcdd2c5cde..0f1f705169116 100644 --- a/packages/react-dom/src/events/ReactDOMEventReplaying.js +++ b/packages/react-dom/src/events/ReactDOMEventReplaying.js @@ -28,7 +28,7 @@ import { getNearestMountedFiber, getContainerFromFiber, getSuspenseInstanceFromFiber, -} from 'react-reconciler/reflection'; +} from 'react-reconciler/src/ReactFiberTreeReflection'; import { attemptToDispatchEvent, addResponderEventSystemEvent, diff --git a/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js b/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js index cbf83b1b71aed..0a3b41db0cc69 100644 --- a/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js +++ b/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js @@ -13,7 +13,7 @@ import { createRequest, startWork, startFlowing, -} from 'react-server/inline.dom-browser'; +} from 'react-server/src/ReactFizzStreamer'; function renderToReadableStream(children: ReactNodeList): ReadableStream { let request; diff --git a/packages/react-dom/src/server/ReactDOMFizzServerNode.js b/packages/react-dom/src/server/ReactDOMFizzServerNode.js index 154b4fff204a6..538becc925bb2 100644 --- a/packages/react-dom/src/server/ReactDOMFizzServerNode.js +++ b/packages/react-dom/src/server/ReactDOMFizzServerNode.js @@ -10,7 +10,11 @@ import type {ReactNodeList} from 'shared/ReactTypes'; import type {Writable} from 'stream'; -import {createRequest, startWork, startFlowing} from 'react-server/inline.dom'; +import { + createRequest, + startWork, + startFlowing, +} from 'react-server/src/ReactFizzStreamer'; function createDrainHandler(destination, request) { return () => startFlowing(request); diff --git a/packages/react-dom/src/test-utils/ReactTestUtils.js b/packages/react-dom/src/test-utils/ReactTestUtils.js index 9449880c83ed4..99dcf67a47af5 100644 --- a/packages/react-dom/src/test-utils/ReactTestUtils.js +++ b/packages/react-dom/src/test-utils/ReactTestUtils.js @@ -7,7 +7,7 @@ import * as React from 'react'; import * as ReactDOM from 'react-dom'; -import {findCurrentFiberUsingSlowPath} from 'react-reconciler/reflection'; +import {findCurrentFiberUsingSlowPath} from 'react-reconciler/src/ReactFiberTreeReflection'; import {get as getInstance} from 'shared/ReactInstanceMap'; import { ClassComponent, diff --git a/packages/react-dom/testing.classic.fb.js b/packages/react-dom/testing.classic.fb.js index 841f6a1d8f8db..0e2cfec9da22e 100644 --- a/packages/react-dom/testing.classic.fb.js +++ b/packages/react-dom/testing.classic.fb.js @@ -8,4 +8,4 @@ */ export * from './index.classic.fb.js'; -export {act} from 'react-reconciler/inline.dom'; +export {act} from 'react-reconciler/src/ReactFiberReconciler'; diff --git a/packages/react-dom/testing.experimental.js b/packages/react-dom/testing.experimental.js index 0727208336ccd..4d3f2d2a7f4f2 100644 --- a/packages/react-dom/testing.experimental.js +++ b/packages/react-dom/testing.experimental.js @@ -8,4 +8,4 @@ */ export * from './index.experimental.js'; -export {act} from 'react-reconciler/inline.dom'; +export {act} from 'react-reconciler/src/ReactFiberReconciler'; diff --git a/packages/react-dom/testing.js b/packages/react-dom/testing.js index 6169f8727267a..bf3fceca62152 100644 --- a/packages/react-dom/testing.js +++ b/packages/react-dom/testing.js @@ -8,4 +8,4 @@ */ export * from './index.js'; -export {act} from 'react-reconciler/inline.dom'; +export {act} from 'react-reconciler/src/ReactFiberReconciler'; diff --git a/packages/react-dom/testing.modern.fb.js b/packages/react-dom/testing.modern.fb.js index 4eca948d49902..dfd6d75097fc5 100644 --- a/packages/react-dom/testing.modern.fb.js +++ b/packages/react-dom/testing.modern.fb.js @@ -8,4 +8,4 @@ */ export * from './index.modern.fb.js'; -export {act} from 'react-reconciler/inline.dom'; +export {act} from 'react-reconciler/src/ReactFiberReconciler'; diff --git a/packages/react-dom/testing.stable.js b/packages/react-dom/testing.stable.js index 474972b70d816..47161fadd7f88 100644 --- a/packages/react-dom/testing.stable.js +++ b/packages/react-dom/testing.stable.js @@ -8,4 +8,4 @@ */ export * from './index.stable.js'; -export {act} from 'react-reconciler/inline.dom'; +export {act} from 'react-reconciler/src/ReactFiberReconciler'; diff --git a/packages/react-flight-dom-webpack/src/ReactFlightDOMClient.js b/packages/react-flight-dom-webpack/src/ReactFlightDOMClient.js index 908ffee1a4ba7..d5712e281bb50 100644 --- a/packages/react-flight-dom-webpack/src/ReactFlightDOMClient.js +++ b/packages/react-flight-dom-webpack/src/ReactFlightDOMClient.js @@ -16,7 +16,7 @@ import { processStringChunk, processBinaryChunk, complete, -} from 'react-flight/inline.dom'; +} from 'react-flight/src/ReactFlightClient'; function startReadingFromStream(response, stream: ReadableStream): void { let reader = stream.getReader(); diff --git a/packages/react-flight-dom-webpack/src/ReactFlightDOMServerBrowser.js b/packages/react-flight-dom-webpack/src/ReactFlightDOMServerBrowser.js index b7538e48a2bdf..0d9468bbe6260 100644 --- a/packages/react-flight-dom-webpack/src/ReactFlightDOMServerBrowser.js +++ b/packages/react-flight-dom-webpack/src/ReactFlightDOMServerBrowser.js @@ -7,13 +7,13 @@ * @flow */ -import type {ReactModel} from 'react-server/flight.inline-typed'; +import type {ReactModel} from 'react-server/src/ReactFlightServer'; import { createRequest, startWork, startFlowing, -} from 'react-server/flight.inline.dom-browser'; +} from 'react-server/src/ReactFlightServer'; function renderToReadableStream(model: ReactModel): ReadableStream { let request; diff --git a/packages/react-flight-dom-webpack/src/ReactFlightDOMServerNode.js b/packages/react-flight-dom-webpack/src/ReactFlightDOMServerNode.js index 6fde6197f4a31..8bc5e20d4ed55 100644 --- a/packages/react-flight-dom-webpack/src/ReactFlightDOMServerNode.js +++ b/packages/react-flight-dom-webpack/src/ReactFlightDOMServerNode.js @@ -7,14 +7,14 @@ * @flow */ -import type {ReactModel} from 'react-server/flight.inline-typed'; +import type {ReactModel} from 'react-server/src/ReactFlightServer'; import type {Writable} from 'stream'; import { createRequest, startWork, startFlowing, -} from 'react-server/flight.inline.dom'; +} from 'react-server/src/ReactFlightServer'; function createDrainHandler(destination, request) { return () => startFlowing(request); diff --git a/packages/react-flight/index.js b/packages/react-flight/index.js index 67fd8e660489a..7d0a0b03ba920 100644 --- a/packages/react-flight/index.js +++ b/packages/react-flight/index.js @@ -3,18 +3,8 @@ * * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. + * + * @flow */ -// This entry point is intentionally not typed. It exists only for third-party -// renderers. The renderers we ship (such as React DOM) instead import a named -// "inline" entry point (for example, `react-server/inline.dom`). It uses -// the same code, but the Flow configuration redirects the host config to its -// real implementation so we can check it against exact intended host types. -// -// Only one renderer (the one you passed to `yarn flow `) is fully -// type-checked at any given time. The Flow config maps the -// `react-server/inline.` import (which is *not* Flow typed) to -// `react-server/inline-typed` (which *is*) for the current renderer. -// On CI, we run Flow checks for each renderer separately. - export * from './src/ReactFlightClient'; diff --git a/packages/react-flight/inline-typed.js b/packages/react-flight/inline-typed.js deleted file mode 100644 index 70a74cb38eb4f..0000000000000 --- a/packages/react-flight/inline-typed.js +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - * - * @flow - */ - -// This file must have the Flow annotation. -// -// This is the Flow-typed entry point for the renderer. It should not be -// imported directly in code. Instead, our Flow configuration uses this entry -// point for the currently checked renderer (the one you passed to `yarn flow`). -// -// For example, if you run `yarn flow dom`, `react-server/inline.dom` points -// to this module (and thus will be considered Flow-typed). But other renderers -// (e.g. `react-test-renderer`) will see stream as untyped during the check. -// -// We can't make all entry points typed at the same time because different -// renderers have different host config types. So we check them one by one. -// We run Flow on all renderers on CI. - -export * from './src/ReactFlightClient'; diff --git a/packages/react-flight/inline.dom-browser.js b/packages/react-flight/inline.dom-browser.js deleted file mode 100644 index c42c00289e3a2..0000000000000 --- a/packages/react-flight/inline.dom-browser.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFlightClient'; diff --git a/packages/react-flight/inline.dom.js b/packages/react-flight/inline.dom.js deleted file mode 100644 index c42c00289e3a2..0000000000000 --- a/packages/react-flight/inline.dom.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFlightClient'; diff --git a/packages/react-native-renderer/src/ReactFabric.js b/packages/react-native-renderer/src/ReactFabric.js index 54da153f84024..c5d49f81283d7 100644 --- a/packages/react-native-renderer/src/ReactFabric.js +++ b/packages/react-native-renderer/src/ReactFabric.js @@ -24,7 +24,7 @@ import { updateContainer, injectIntoDevTools, getPublicRootInstance, -} from 'react-reconciler/inline.fabric'; +} from 'react-reconciler/src/ReactFiberReconciler'; import {createPortal as createPortalImpl} from 'shared/ReactPortal'; import {setBatchingImplementation} from 'legacy-events/ReactGenericBatching'; diff --git a/packages/react-native-renderer/src/ReactNativeFiberInspector.js b/packages/react-native-renderer/src/ReactNativeFiberInspector.js index 5693a8edd892d..bdd4cdef78280 100644 --- a/packages/react-native-renderer/src/ReactNativeFiberInspector.js +++ b/packages/react-native-renderer/src/ReactNativeFiberInspector.js @@ -12,7 +12,7 @@ import type {Fiber} from 'react-reconciler/src/ReactFiber'; import { findCurrentHostFiber, findCurrentFiberUsingSlowPath, -} from 'react-reconciler/reflection'; +} from 'react-reconciler/src/ReactFiberTreeReflection'; import getComponentName from 'shared/getComponentName'; import {HostComponent} from 'shared/ReactWorkTags'; import invariant from 'shared/invariant'; diff --git a/packages/react-native-renderer/src/ReactNativeRenderer.js b/packages/react-native-renderer/src/ReactNativeRenderer.js index 9d156026eb28c..0d587355cffbc 100644 --- a/packages/react-native-renderer/src/ReactNativeRenderer.js +++ b/packages/react-native-renderer/src/ReactNativeRenderer.js @@ -23,7 +23,7 @@ import { updateContainer, injectIntoDevTools, getPublicRootInstance, -} from 'react-reconciler/inline.native'; +} from 'react-reconciler/src/ReactFiberReconciler'; // TODO: direct imports like some-package/src/* are bad. Fix me. import {getStackByFiberInDevAndProd} from 'react-reconciler/src/ReactCurrentFiber'; import {createPortal as createPortalImpl} from 'shared/ReactPortal'; diff --git a/packages/react-noop-renderer/src/ReactNoopFlightClient.js b/packages/react-noop-renderer/src/ReactNoopFlightClient.js index eca91cfc46711..511c7004bf851 100644 --- a/packages/react-noop-renderer/src/ReactNoopFlightClient.js +++ b/packages/react-noop-renderer/src/ReactNoopFlightClient.js @@ -14,7 +14,7 @@ * environment. */ -import type {ReactModelRoot} from 'react-flight/inline-typed'; +import type {ReactModelRoot} from 'react-flight'; import ReactFlightClient from 'react-flight'; diff --git a/packages/react-noop-renderer/src/ReactNoopFlightServer.js b/packages/react-noop-renderer/src/ReactNoopFlightServer.js index 6377812dd5413..8acf4307b9ff7 100644 --- a/packages/react-noop-renderer/src/ReactNoopFlightServer.js +++ b/packages/react-noop-renderer/src/ReactNoopFlightServer.js @@ -14,7 +14,7 @@ * environment. */ -import type {ReactModel} from 'react-server/flight.inline-typed'; +import type {ReactModel} from 'react-server/src/ReactFlightServer'; import ReactFlightServer from 'react-server/flight'; diff --git a/packages/react-noop-renderer/src/ReactNoopPersistent.js b/packages/react-noop-renderer/src/ReactNoopPersistent.js index b093b627dad1e..3fc7f64445b84 100644 --- a/packages/react-noop-renderer/src/ReactNoopPersistent.js +++ b/packages/react-noop-renderer/src/ReactNoopPersistent.js @@ -14,7 +14,7 @@ * environment. */ -import ReactFiberPersistentReconciler from 'react-reconciler/persistent'; +import ReactFiberReconciler from 'react-reconciler'; import createReactNoop from './createReactNoop'; export const { @@ -47,6 +47,6 @@ export const { dumpTree, getRoot, } = createReactNoop( - ReactFiberPersistentReconciler, // reconciler + ReactFiberReconciler, // reconciler false, // useMutation ); diff --git a/packages/react-reconciler/index.js b/packages/react-reconciler/index.js index 2fa37357c88eb..ef6b0a4fe3c41 100644 --- a/packages/react-reconciler/index.js +++ b/packages/react-reconciler/index.js @@ -3,18 +3,8 @@ * * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. + * + * @flow */ -// This entry point is intentionally not typed. It exists only for third-party -// renderers. The renderers we ship (such as React DOM) instead import a named -// "inline" entry point (for example, `react-reconciler/inline.dom`). It uses -// the same code, but the Flow configuration redirects the host config to its -// real implementation so we can check it against exact intended host types. -// -// Only one renderer (the one you passed to `yarn flow `) is fully -// type-checked at any given time. The Flow config maps the -// `react-reconciler/inline.` import (which is *not* Flow typed) to -// `react-reconciler/inline-typed` (which *is*) for the current renderer. -// On CI, we run Flow checks for each renderer separately. - export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline-typed.js b/packages/react-reconciler/inline-typed.js deleted file mode 100644 index 0b13d9b523ae9..0000000000000 --- a/packages/react-reconciler/inline-typed.js +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - * - * @flow - */ - -// This file must have the Flow annotation. -// -// This is the Flow-typed entry point for the reconciler. It should not be -// imported directly in code. Instead, our Flow configuration uses this entry -// point for the currently checked renderer (the one you passed to `yarn flow`). -// -// For example, if you run `yarn flow dom`, `react-reconciler/inline.dom` points -// to this module (and thus will be considered Flow-typed). But other renderers -// (e.g. `react-test-renderer`) will see reconciler as untyped during the check. -// -// We can't make all entry points typed at the same time because different -// renderers have different host config types. So we check them one by one. -// We run Flow on all renderers on CI. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.art.js b/packages/react-reconciler/inline.art.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.art.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.dom-browser.js b/packages/react-reconciler/inline.dom-browser.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.dom-browser.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.dom.js b/packages/react-reconciler/inline.dom.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.dom.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.fabric.js b/packages/react-reconciler/inline.fabric.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.fabric.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.fire.js b/packages/react-reconciler/inline.fire.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.fire.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.native.js b/packages/react-reconciler/inline.native.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.native.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/inline.test.js b/packages/react-reconciler/inline.test.js deleted file mode 100644 index 0d0c1546cc9a2..0000000000000 --- a/packages/react-reconciler/inline.test.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/npm/persistent.js b/packages/react-reconciler/npm/persistent.js deleted file mode 100644 index c428b4b664303..0000000000000 --- a/packages/react-reconciler/npm/persistent.js +++ /dev/null @@ -1,7 +0,0 @@ -'use strict'; - -if (process.env.NODE_ENV === 'production') { - module.exports = require('./cjs/react-reconciler-persistent.production.min.js'); -} else { - module.exports = require('./cjs/react-reconciler-persistent.development.js'); -} diff --git a/packages/react-reconciler/package.json b/packages/react-reconciler/package.json index 1946a83f4f2f1..bb90b2401a690 100644 --- a/packages/react-reconciler/package.json +++ b/packages/react-reconciler/package.json @@ -13,7 +13,6 @@ "README.md", "build-info.json", "index.js", - "persistent.js", "reflection.js", "cjs/" ], diff --git a/packages/react-reconciler/persistent.js b/packages/react-reconciler/persistent.js deleted file mode 100644 index 59906bfe44c1f..0000000000000 --- a/packages/react-reconciler/persistent.js +++ /dev/null @@ -1,12 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its 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'; - -// This is the same export as in index.js, -// with persistent reconciler flags turned on. -export * from './src/ReactFiberReconciler'; diff --git a/packages/react-reconciler/src/ReactFiberClassComponent.js b/packages/react-reconciler/src/ReactFiberClassComponent.js index 84c0625c51095..ffe007e437b18 100644 --- a/packages/react-reconciler/src/ReactFiberClassComponent.js +++ b/packages/react-reconciler/src/ReactFiberClassComponent.js @@ -19,7 +19,7 @@ import { warnAboutDeprecatedLifecycles, } from 'shared/ReactFeatureFlags'; import ReactStrictModeWarnings from './ReactStrictModeWarnings'; -import {isMounted} from 'react-reconciler/reflection'; +import {isMounted} from 'react-reconciler/src/ReactFiberTreeReflection'; import {get as getInstance, set as setInstance} from 'shared/ReactInstanceMap'; import shallowEqual from 'shared/shallowEqual'; import getComponentName from 'shared/getComponentName'; diff --git a/packages/react-reconciler/src/ReactFiberContext.js b/packages/react-reconciler/src/ReactFiberContext.js index f6009c2b9d3dc..27c1b1c3fd404 100644 --- a/packages/react-reconciler/src/ReactFiberContext.js +++ b/packages/react-reconciler/src/ReactFiberContext.js @@ -10,7 +10,7 @@ import type {Fiber} from './ReactFiber'; import type {StackCursor} from './ReactFiberStack'; -import {isFiberMounted} from 'react-reconciler/reflection'; +import {isFiberMounted} from 'react-reconciler/src/ReactFiberTreeReflection'; import {disableLegacyContext} from 'shared/ReactFeatureFlags'; import {ClassComponent, HostRoot} from 'shared/ReactWorkTags'; import getComponentName from 'shared/getComponentName'; diff --git a/packages/react-reconciler/src/ReactFiberReconciler.js b/packages/react-reconciler/src/ReactFiberReconciler.js index 816a3946562e1..fbb258507d314 100644 --- a/packages/react-reconciler/src/ReactFiberReconciler.js +++ b/packages/react-reconciler/src/ReactFiberReconciler.js @@ -27,7 +27,7 @@ import type { import { findCurrentHostFiber, findCurrentHostFiberWithNoPortals, -} from 'react-reconciler/reflection'; +} from 'react-reconciler/src/ReactFiberTreeReflection'; import {get as getInstance} from 'shared/ReactInstanceMap'; import { HostComponent, diff --git a/packages/react-server/flight.inline-typed.js b/packages/react-server/flight.inline-typed.js deleted file mode 100644 index 8afc4525e9559..0000000000000 --- a/packages/react-server/flight.inline-typed.js +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - * - * @flow - */ - -// This file must have the Flow annotation. -// -// This is the Flow-typed entry point for the renderer. It should not be -// imported directly in code. Instead, our Flow configuration uses this entry -// point for the currently checked renderer (the one you passed to `yarn flow`). -// -// For example, if you run `yarn flow dom`, `react-server/flight.inline.dom` points -// to this module (and thus will be considered Flow-typed). But other renderers -// (e.g. `react-test-renderer`) will see stream as untyped during the check. -// -// We can't make all entry points typed at the same time because different -// renderers have different host config types. So we check them one by one. -// We run Flow on all renderers on CI. - -export * from './src/ReactFlightServer'; diff --git a/packages/react-server/flight.inline.dom-browser.js b/packages/react-server/flight.inline.dom-browser.js deleted file mode 100644 index 2c140b0a7e498..0000000000000 --- a/packages/react-server/flight.inline.dom-browser.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFlightServer'; diff --git a/packages/react-server/flight.inline.dom.js b/packages/react-server/flight.inline.dom.js deleted file mode 100644 index 2c140b0a7e498..0000000000000 --- a/packages/react-server/flight.inline.dom.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFlightServer'; diff --git a/packages/react-server/flight.js b/packages/react-server/flight.js index 74fce7fbe8dd5..d09300cc5fdb6 100644 --- a/packages/react-server/flight.js +++ b/packages/react-server/flight.js @@ -3,18 +3,8 @@ * * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. + * + * @flow */ -// This entry point is intentionally not typed. It exists only for third-party -// renderers. The renderers we ship (such as React DOM) instead import a named -// "inline" entry point (for example, `react-server/flight.inline.dom`). It uses -// the same code, but the Flow configuration redirects the host config to its -// real implementation so we can check it against exact intended host types. -// -// Only one renderer (the one you passed to `yarn flow `) is fully -// type-checked at any given time. The Flow config maps the -// `react-server/flight.inline.` import (which is *not* Flow typed) to -// `react-server/flight.inline-typed` (which *is*) for the current renderer. -// On CI, we run Flow checks for each renderer separately. - export * from './src/ReactFlightServer'; diff --git a/packages/react-server/index.js b/packages/react-server/index.js index 8f547166be4b8..b13c062b44692 100644 --- a/packages/react-server/index.js +++ b/packages/react-server/index.js @@ -3,18 +3,8 @@ * * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. + * + * @flow */ -// This entry point is intentionally not typed. It exists only for third-party -// renderers. The renderers we ship (such as React DOM) instead import a named -// "inline" entry point (for example, `react-server/inline.dom`). It uses -// the same code, but the Flow configuration redirects the host config to its -// real implementation so we can check it against exact intended host types. -// -// Only one renderer (the one you passed to `yarn flow `) is fully -// type-checked at any given time. The Flow config maps the -// `react-server/inline.` import (which is *not* Flow typed) to -// `react-server/inline-typed` (which *is*) for the current renderer. -// On CI, we run Flow checks for each renderer separately. - export * from './src/ReactFizzStreamer'; diff --git a/packages/react-server/inline-typed.js b/packages/react-server/inline-typed.js deleted file mode 100644 index 27f5e86c80780..0000000000000 --- a/packages/react-server/inline-typed.js +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - * - * @flow - */ - -// This file must have the Flow annotation. -// -// This is the Flow-typed entry point for the renderer. It should not be -// imported directly in code. Instead, our Flow configuration uses this entry -// point for the currently checked renderer (the one you passed to `yarn flow`). -// -// For example, if you run `yarn flow dom`, `react-server/inline.dom` points -// to this module (and thus will be considered Flow-typed). But other renderers -// (e.g. `react-test-renderer`) will see stream as untyped during the check. -// -// We can't make all entry points typed at the same time because different -// renderers have different host config types. So we check them one by one. -// We run Flow on all renderers on CI. - -export * from './src/ReactFizzStreamer'; diff --git a/packages/react-server/inline.dom-browser.js b/packages/react-server/inline.dom-browser.js deleted file mode 100644 index 9bae815dca122..0000000000000 --- a/packages/react-server/inline.dom-browser.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFizzStreamer'; diff --git a/packages/react-server/inline.dom.js b/packages/react-server/inline.dom.js deleted file mode 100644 index 9bae815dca122..0000000000000 --- a/packages/react-server/inline.dom.js +++ /dev/null @@ -1,11 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -// This file intentionally does *not* have the Flow annotation. -// Don't add it. See `./inline-typed.js` for an explanation. - -export * from './src/ReactFizzStreamer'; diff --git a/packages/react-test-renderer/src/ReactTestRenderer.js b/packages/react-test-renderer/src/ReactTestRenderer.js index 6cebfab39ce60..3bb1001b84432 100644 --- a/packages/react-test-renderer/src/ReactTestRenderer.js +++ b/packages/react-test-renderer/src/ReactTestRenderer.js @@ -20,8 +20,8 @@ import { injectIntoDevTools, batchedUpdates, act, -} from 'react-reconciler/inline.test'; -import {findCurrentFiberUsingSlowPath} from 'react-reconciler/reflection'; +} from 'react-reconciler/src/ReactFiberReconciler'; +import {findCurrentFiberUsingSlowPath} from 'react-reconciler/src/ReactFiberTreeReflection'; import { Fragment, FunctionComponent, diff --git a/packages/shared/forks/ReactFeatureFlags.persistent.js b/packages/shared/forks/ReactFeatureFlags.persistent.js deleted file mode 100644 index 68055d71fcd15..0000000000000 --- a/packages/shared/forks/ReactFeatureFlags.persistent.js +++ /dev/null @@ -1,62 +0,0 @@ -/** - * Copyright (c) Facebook, Inc. and its affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - * - * @flow - */ - -import invariant from 'shared/invariant'; - -import typeof * as FeatureFlagsType from 'shared/ReactFeatureFlags'; -import typeof * as ExportsType from './ReactFeatureFlags.persistent'; - -export const debugRenderPhaseSideEffectsForStrictMode = false; -export const enableUserTimingAPI = __DEV__; -export const warnAboutDeprecatedLifecycles = true; -export const replayFailedUnitOfWorkWithInvokeGuardedCallback = __DEV__; -export const enableProfilerTimer = __PROFILE__; -export const enableProfilerCommitHooks = false; -export const enableSchedulerTracing = __PROFILE__; -export const enableSuspenseServerRenderer = false; -export const enableSelectiveHydration = false; -export const enableBlocksAPI = false; -export const disableJavaScriptURLs = false; -export const disableInputAttributeSyncing = false; -export const warnAboutShorthandPropertyCollision = true; -export const enableSchedulerDebugging = false; -export const enableDeprecatedFlareAPI = false; -export const enableFundamentalAPI = false; -export const enableScopeAPI = false; -export const warnAboutUnmockedScheduler = true; -export const flushSuspenseFallbacksInTests = true; -export const enableSuspenseCallback = false; -export const warnAboutDefaultPropsOnFunctionComponents = false; -export const warnAboutStringRefs = false; -export const disableLegacyContext = false; -export const disableSchedulerTimeoutBasedOnReactExpirationTime = false; -export const enableTrustedTypesIntegration = false; -export const disableTextareaChildren = false; -export const disableMapsAsChildren = false; -export const disableModulePatternComponents = false; -export const warnUnstableRenderSubtreeIntoContainer = false; -export const deferPassiveEffectCleanupDuringUnmount = false; -export const runAllPassiveEffectDestroysBeforeCreates = false; -export const enableModernEventSystem = false; -export const warnAboutSpreadingKeyToJSX = false; -export const enableLegacyFBPrimerSupport = false; - -// Internal-only attempt to debug a React Native issue. See D20130868. -export const throwEarlyForMysteriousError = false; - -// Only used in www builds. -export function addUserTimingListener() { - invariant(false, 'Not implemented.'); -} - -// Flow magic to verify the exports of this file match the original version. -// eslint-disable-next-line no-unused-vars -type Check<_X, Y: _X, X: Y = _X> = null; -// eslint-disable-next-line no-unused-expressions -(null: Check); diff --git a/scripts/flow/config/flowconfig b/scripts/flow/config/flowconfig index a7f536c23b24c..4790c78cf38e3 100644 --- a/scripts/flow/config/flowconfig +++ b/scripts/flow/config/flowconfig @@ -14,6 +14,9 @@ .*/__mocks__/.* .*/__tests__/.* +# TODO: noop should get its own inlinedHostConfig entry +.*/packages/react-noop-renderer/.* + %REACT_RENDERER_FLOW_IGNORES% [include] diff --git a/scripts/flow/createFlowConfigs.js b/scripts/flow/createFlowConfigs.js index a0a5e7a323039..0c5e8f4dce984 100644 --- a/scripts/flow/createFlowConfigs.js +++ b/scripts/flow/createFlowConfigs.js @@ -18,37 +18,43 @@ const configTemplate = fs .readFileSync(__dirname + '/config/flowconfig') .toString(); -function writeConfig(renderer, isServerSupported) { +function writeConfig(renderer, rendererInfo, isServerSupported) { const folder = __dirname + '/' + renderer; mkdirp.sync(folder); const serverRenderer = isServerSupported ? renderer : 'custom'; + + let ignoredPaths = []; + + inlinedHostConfigs.forEach(otherRenderer => { + if (otherRenderer === rendererInfo) { + return; + } + otherRenderer.paths.forEach(otherPath => { + if (rendererInfo.paths.indexOf(otherPath) !== -1) { + return; + } + ignoredPaths.push(`.*/packages/${otherPath}`); + }); + + if (otherRenderer.shortName !== serverRenderer) { + ignoredPaths.push( + `.*/packages/.*/forks/.*.${otherRenderer.shortName}.js`, + ); + } + }); + const config = configTemplate .replace( '%REACT_RENDERER_FLOW_OPTIONS%', ` -module.name_mapper='react-reconciler/inline.${renderer}$$' -> 'react-reconciler/inline-typed' module.name_mapper='ReactFiberHostConfig$$' -> 'forks/ReactFiberHostConfig.${renderer}' -module.name_mapper='react-server/inline.${renderer}$$' -> 'react-server/inline-typed' -module.name_mapper='react-server/flight.inline.${renderer}$$' -> 'react-server/flight.inline-typed' module.name_mapper='ReactServerHostConfig$$' -> 'forks/ReactServerHostConfig.${serverRenderer}' module.name_mapper='ReactServerFormatConfig$$' -> 'forks/ReactServerFormatConfig.${serverRenderer}' -module.name_mapper='react-flight/inline.${renderer}$$' -> 'react-flight/inline-typed' module.name_mapper='ReactFlightClientHostConfig$$' -> 'forks/ReactFlightClientHostConfig.${serverRenderer}' `.trim(), ) - .replace( - '%REACT_RENDERER_FLOW_IGNORES%', - renderer === 'dom' || renderer === 'dom-browser' - ? '' - : // If we're not checking DOM, ignore the DOM package since it - // won't be consistent. - ` - .*/packages/react-dom/.* - .*/packages/.*/forks/.*.dom.js - .*/packages/.*/forks/.*.dom-browser.js - `.trim(), - ); + .replace('%REACT_RENDERER_FLOW_IGNORES%', ignoredPaths.join('\n')); const disclaimer = ` # ---------------------------------------------------------------# @@ -81,6 +87,10 @@ ${disclaimer} // so that we can run those checks in parallel if we want. inlinedHostConfigs.forEach(rendererInfo => { if (rendererInfo.isFlowTyped) { - writeConfig(rendererInfo.shortName, rendererInfo.isServerSupported); + writeConfig( + rendererInfo.shortName, + rendererInfo, + rendererInfo.isServerSupported, + ); } }); diff --git a/scripts/jest/setupHostConfigs.js b/scripts/jest/setupHostConfigs.js index 93078a03cdee0..e34279b0786f8 100644 --- a/scripts/jest/setupHostConfigs.js +++ b/scripts/jest/setupHostConfigs.js @@ -11,12 +11,6 @@ jest.mock('react-reconciler', () => { return require.requireActual('react-reconciler'); }; }); -jest.mock('react-reconciler/persistent', () => { - return config => { - jest.mock(shimHostConfigPath, () => config); - return require.requireActual('react-reconciler/persistent'); - }; -}); const shimServerHostConfigPath = 'react-server/src/ReactServerHostConfig'; const shimServerFormatConfigPath = 'react-server/src/ReactServerFormatConfig'; jest.mock('react-server', () => { @@ -42,6 +36,24 @@ jest.mock('react-flight', () => { }; }); +const configPaths = [ + 'react-reconciler/src/ReactFiberHostConfig', + 'react-flight/src/ReactFlightClientHostConfig', + 'react-server/src/ReactServerHostConfig', + 'react-server/src/ReactServerFormatConfig', +]; + +function mockAllConfigs(rendererInfo) { + configPaths.forEach(path => { + // We want the reconciler to pick up the host config for this renderer. + jest.mock(path, () => { + let idx = path.lastIndexOf('/'); + let forkPath = path.substr(0, idx) + '/forks' + path.substr(idx); + return require.requireActual(`${forkPath}.${rendererInfo.shortName}.js`); + }); + }); +} + // But for inlined host configs (such as React DOM, Native, etc), we // mock their named entry points to establish a host config mapping. inlinedHostConfigs.forEach(rendererInfo => { @@ -50,131 +62,12 @@ inlinedHostConfigs.forEach(rendererInfo => { // Instead, it's handled by the generic `react-reconciler` entry point above. return; } - jest.mock(`react-reconciler/inline.${rendererInfo.shortName}`, () => { - let hasImportedShimmedConfig = false; - - // We want the reconciler to pick up the host config for this renderer. - jest.mock(shimHostConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-reconciler/src/forks/ReactFiberHostConfig.${rendererInfo.shortName}.js` - ); + rendererInfo.entryPoints.forEach(entryPoint => { + jest.mock(entryPoint, () => { + mockAllConfigs(rendererInfo); + return require.requireActual(entryPoint); }); - - const renderer = require.requireActual('react-reconciler'); - // If the shimmed config factory function above has not run, - // it means this test file loads more than one renderer - // but doesn't reset modules between them. This won't work. - if (!hasImportedShimmedConfig) { - throw new Error( - `Could not import the "${rendererInfo.shortName}" renderer ` + - `in this suite because another renderer has already been ` + - `loaded earlier. Call jest.resetModules() before importing any ` + - `of the following entry points:\n\n` + - rendererInfo.entryPoints.map(entry => ` * ${entry}`) - ); - } - - return renderer; }); - - if (rendererInfo.isServerSupported) { - jest.mock(`react-server/inline.${rendererInfo.shortName}`, () => { - let hasImportedShimmedConfig = false; - - // We want the renderer to pick up the host config for this renderer. - jest.mock(shimServerHostConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-server/src/forks/ReactServerHostConfig.${rendererInfo.shortName}.js` - ); - }); - jest.mock(shimServerFormatConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-server/src/forks/ReactServerFormatConfig.${rendererInfo.shortName}.js` - ); - }); - - const renderer = require.requireActual('react-server'); - // If the shimmed config factory function above has not run, - // it means this test file loads more than one renderer - // but doesn't reset modules between them. This won't work. - if (!hasImportedShimmedConfig) { - throw new Error( - `Could not import the "${rendererInfo.shortName}" renderer ` + - `in this suite because another renderer has already been ` + - `loaded earlier. Call jest.resetModules() before importing any ` + - `of the following entry points:\n\n` + - rendererInfo.entryPoints.map(entry => ` * ${entry}`) - ); - } - - return renderer; - }); - - jest.mock(`react-server/flight.inline.${rendererInfo.shortName}`, () => { - let hasImportedShimmedConfig = false; - - // We want the renderer to pick up the host config for this renderer. - jest.mock(shimServerHostConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-server/src/forks/ReactServerHostConfig.${rendererInfo.shortName}.js` - ); - }); - jest.mock(shimServerFormatConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-server/src/forks/ReactServerFormatConfig.${rendererInfo.shortName}.js` - ); - }); - - const renderer = require.requireActual('react-server/flight'); - // If the shimmed config factory function above has not run, - // it means this test file loads more than one renderer - // but doesn't reset modules between them. This won't work. - if (!hasImportedShimmedConfig) { - throw new Error( - `Could not import the "${rendererInfo.shortName}" renderer ` + - `in this suite because another renderer has already been ` + - `loaded earlier. Call jest.resetModules() before importing any ` + - `of the following entry points:\n\n` + - rendererInfo.entryPoints.map(entry => ` * ${entry}`) - ); - } - - return renderer; - }); - - jest.mock(`react-flight/inline.${rendererInfo.shortName}`, () => { - let hasImportedShimmedConfig = false; - - // We want the renderer to pick up the host config for this renderer. - jest.mock(shimFlightClientHostConfigPath, () => { - hasImportedShimmedConfig = true; - return require.requireActual( - `react-flight/src/forks/ReactFlightClientHostConfig.${rendererInfo.shortName}.js` - ); - }); - - const renderer = require.requireActual('react-flight'); - // If the shimmed config factory function above has not run, - // it means this test file loads more than one renderer - // but doesn't reset modules between them. This won't work. - if (!hasImportedShimmedConfig) { - throw new Error( - `Could not import the "${rendererInfo.shortName}" renderer ` + - `in this suite because another renderer has already been ` + - `loaded earlier. Call jest.resetModules() before importing any ` + - `of the following entry points:\n\n` + - rendererInfo.entryPoints.map(entry => ` * ${entry}`) - ); - } - - return renderer; - }); - } }); // Make it possible to import this module inside diff --git a/scripts/rollup/bundles.js b/scripts/rollup/bundles.js index 41b9e30b43a5f..e4d48638f798c 100644 --- a/scripts/rollup/bundles.js +++ b/scripts/rollup/bundles.js @@ -362,15 +362,6 @@ const bundles = [ externals: ['react'], }, - /******* React Persistent Reconciler *******/ - { - bundleTypes: [NODE_DEV, NODE_PROD], - moduleType: RECONCILER, - entry: 'react-reconciler/persistent', - global: 'ReactPersistentReconciler', - externals: ['react'], - }, - /******* React Server *******/ { bundleTypes: [NODE_DEV, NODE_PROD], diff --git a/scripts/rollup/forks.js b/scripts/rollup/forks.js index 1fbbd0ffed8dc..a69629936bf5c 100644 --- a/scripts/rollup/forks.js +++ b/scripts/rollup/forks.js @@ -100,8 +100,6 @@ const forks = Object.freeze({ `Unexpected entry (${entry}) and bundleType (${bundleType})` ); } - case 'react-reconciler/persistent': - return 'shared/forks/ReactFeatureFlags.persistent.js'; case 'react-test-renderer': switch (bundleType) { case FB_WWW_DEV: diff --git a/scripts/shared/inlinedHostConfigs.js b/scripts/shared/inlinedHostConfigs.js index c92deb2f24f37..0bef020e278c3 100644 --- a/scripts/shared/inlinedHostConfigs.js +++ b/scripts/shared/inlinedHostConfigs.js @@ -16,40 +16,65 @@ module.exports = [ 'react-flight-dom-webpack/server.node', 'react-flight-dom-webpack', ], + paths: [ + 'react-dom', + 'react-dom/unstable-fizz', + 'react-dom/unstable-fizz.node', + 'react-dom/src/server/ReactDOMFizzServerNode.js', // react-dom/unstable-fizz.node + 'react-flight-dom-webpack', + 'react-flight-dom-webpack/server', + 'react-flight-dom-webpack/server.node', + 'react-flight-dom-webpack/src/ReactFlightDOMServerNode.js', // react-flight-dom-webpack/server.browser + ], isFlowTyped: true, isServerSupported: true, }, { shortName: 'dom-browser', entryPoints: [ + 'react-dom', + 'react-dom/testing', 'react-dom/unstable-fizz.browser', 'react-flight-dom-webpack/server.browser', 'react-flight-dom-webpack', ], + paths: [ + 'react-dom', + 'react-dom/testing', + 'react-dom/unstable-fizz.browser', + 'react-dom/src/server/ReactDOMFizzServerBrowser.js', // react-dom/unstable-fizz.browser + 'react-flight-dom-webpack', + 'react-flight-dom-webpack/server.browser', + 'react-flight-dom-webpack/src/ReactFlightDOMServerBrowser.js', // react-flight-dom-webpack/server.browser + ], isFlowTyped: true, isServerSupported: true, }, { shortName: 'art', entryPoints: ['react-art'], + paths: ['react-art'], isFlowTyped: false, // TODO: type it. isServerSupported: false, }, { shortName: 'native', entryPoints: ['react-native-renderer'], + paths: ['react-native-renderer'], isFlowTyped: true, isServerSupported: false, }, { shortName: 'fabric', entryPoints: ['react-native-renderer/fabric'], + paths: ['react-native-renderer'], isFlowTyped: true, isServerSupported: false, }, { shortName: 'test', entryPoints: ['react-test-renderer'], + paths: ['react-test-renderer'], isFlowTyped: true, isServerSupported: false, }, @@ -57,11 +82,11 @@ module.exports = [ shortName: 'custom', entryPoints: [ 'react-reconciler', - 'react-reconciler/persistent', 'react-flight', 'react-server', 'react-server/flight', ], + paths: [], isFlowTyped: true, isServerSupported: true, },