Move remaining things to named exports (#18165)

* Move remaining things to named exports

The interesting case here is the noop renderers. The wrappers around the
reconciler now changed to use a local export that gets mutated.

ReactNoop and ReactNoopPersistent now have to destructure the object to
list out the names it's going to export. We should probably refactor
ReactNoop away from createReactNoop. Especially since it's also not Flow
typed.

* Switch interactions to star exports

This will have esModule compatibility flag on them. They should ideally
export default instead.

Author: sebmarkbage

packages/eslint-plugin-react-hooks/index.js

@@ -5,6 +5,4 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-'use strict';
-
-module.exports = require('./src/index');
+export * from './src/index';

packages/react-art/Circle.js

@@ -7,8 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const Circle = require('./npm/Circle');
-
-module.exports = Circle;
+export {default} from './npm/Circle';

packages/react-art/Rectangle.js

@@ -7,8 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const Rectangle = require('./npm/Rectangle');
-
-module.exports = Rectangle;
+export {default} from './npm/Rectangle';

packages/react-art/Wedge.js

@@ -7,8 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const Wedge = require('./npm/Wedge');
-
-module.exports = Wedge;
+export {default} from './npm/Wedge';

packages/react-art/index.js

@@ -7,8 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const ReactART = require('./src/ReactART');
-
-module.exports = ReactART;
+export * from './src/ReactART';

packages/react-art/src/__tests__/ReactART-test.js

@@ -11,23 +11,26 @@
 
 'use strict';
 
-const React = require('react');
+import * as React from 'react';
+
+import * as ReactART from 'react-art';
+import ARTSVGMode from 'art/modes/svg';
+import ARTCurrentMode from 'art/modes/current';
+// Since these are default exports, we need to import them using ESM.
+// Since they must be on top, we need to import this before ReactDOM.
+import Circle from 'react-art/Circle';
+import Rectangle from 'react-art/Rectangle';
+import Wedge from 'react-art/Wedge';
+
+// Isolate DOM renderer.
+jest.resetModules();
 const ReactDOM = require('react-dom');
 const ReactTestUtils = require('react-dom/test-utils');
 
 // Isolate test renderer.
 jest.resetModules();
 const ReactTestRenderer = require('react-test-renderer');
 
-// Isolate ART renderer.
-jest.resetModules();
-const ReactART = require('react-art');
-const ARTSVGMode = require('art/modes/svg');
-const ARTCurrentMode = require('art/modes/current');
-const Circle = require('react-art/Circle');
-const Rectangle = require('react-art/Rectangle');
-const Wedge = require('react-art/Wedge');
-
 // Isolate the noop renderer
 jest.resetModules();
 const ReactNoop = require('react-noop-renderer');

packages/react-debug-tools/index.js

@@ -5,9 +5,4 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-'use strict';
-
-const ReactDebugTools = require('./src/ReactDebugTools');
-
-// This is hacky but makes it work with both Rollup and Jest.
-module.exports = ReactDebugTools.default || ReactDebugTools;
+export * from './src/ReactDebugTools';

packages/react-flight-dom-webpack/index.js

@@ -7,10 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const ReactFlightDOMClient = require('./src/ReactFlightDOMClient');
-
-// TODO: decide on the top-level export form.
-// This is hacky but makes it work with both Rollup and Jest
-module.exports = ReactFlightDOMClient.default || ReactFlightDOMClient;
+export * from './src/ReactFlightDOMClient';

packages/react-flight-dom-webpack/server.browser.js

@@ -7,11 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-const ReactFlightDOMServerBrowser = require('./src/ReactFlightDOMServerBrowser');
-
-// TODO: decide on the top-level export form.
-// This is hacky but makes it work with both Rollup and Jest
-module.exports =
-  ReactFlightDOMServerBrowser.default || ReactFlightDOMServerBrowser;
+export * from './src/ReactFlightDOMServerBrowser';

packages/react-flight-dom-webpack/server.js

@@ -7,6 +7,4 @@
  * @flow
  */
 
-'use strict';
-
-module.exports = require('./server.node');
+export * from './server.node';