Cross-fork lint: Support named export declaration (#20784)

Noticed this didn't work when I ran `replace-fork` and a named export
declaration in ReactFiberReconciler was not properly fixed.

Author: acdlite

packages/react-reconciler/src/ReactFiberReconciler.old.js

@@ -104,7 +104,7 @@ export {
 IdleLanePriorityasIdleEventPriority,
 }from'./ReactFiberLane.old';
 
-export{registerMutableSourceForHydration}from'./ReactMutableSource.new';
+export{registerMutableSourceForHydration}from'./ReactMutableSource.old';
 export{createPortal}from'./ReactPortal';
 export{
 createComponentSelector,

scripts/eslint-rules/__tests__/no-cross-fork-imports-test.internal.js

@@ -94,5 +94,29 @@ ruleTester.run('eslint-rules/no-cross-fork-imports', rule, {
 ],
 output: "import {scheduleUpdateOnFiber} from './ReactFiberWorkLoop.new';",
 },
+{
+code: "export {DiscreteEventPriority} from './ReactFiberLane.old.js';",
+filename: 'ReactFiberReconciler.new.js',
+errors: [
+{
+message:
+'A module that belongs to the new fork cannot import a module '+
+'from the old fork.',
+},
+],
+output: "export {DiscreteEventPriority} from './ReactFiberLane.new';",
+},
+{
+code: "export {DiscreteEventPriority} from './ReactFiberLane.new.js';",
+filename: 'ReactFiberReconciler.old.js',
+errors: [
+{
+message:
+'A module that belongs to the old fork cannot import a module '+
+'from the new fork.',
+},
+],
+output: "export {DiscreteEventPriority} from './ReactFiberLane.old';",
+},
 ],
 });

scripts/eslint-rules/no-cross-fork-imports.js

@@ -26,48 +26,59 @@ module.exports = {
 constsourceFilename=context.getFilename();
 
 if(isOldFork(sourceFilename)){
+constvisitor=node=>{
+constsourceNode=node.source;
+if(sourceNode===null){
+return;
+}
+constfilename=sourceNode.value;
+if(isNewFork(filename)){
+context.report({
+node: sourceNode,
+message:
+'A module that belongs to the old fork cannot import a module '+
+'from the new fork.',
+fix(fixer){
+returnfixer.replaceText(
+sourceNode,
+`'${filename.replace(/\.new(\.js)?$/,'.old')}'`
+);
+},
+});
+}
+};
 return{
-ImportDeclaration(node){
-constimportSourceNode=node.source;
-constfilename=importSourceNode.value;
-if(isNewFork(filename)){
-context.report({
-node: importSourceNode,
-message:
-'A module that belongs to the old fork cannot import a module '+
-'from the new fork.',
-fix(fixer){
-returnfixer.replaceText(
-importSourceNode,
-`'${filename.replace(/\.new(\.js)?$/,'.old')}'`
-);
-},
-});
-}
-},
+ImportDeclaration: visitor,
+ExportNamedDeclaration: visitor,
 };
 }
 
 if(isNewFork(sourceFilename)){
+constvisitor=node=>{
+constsourceNode=node.source;
+if(sourceNode===null){
+return;
+}
+constfilename=sourceNode.value;
+if(isOldFork(filename)){
+context.report({
+node: sourceNode,
+message:
+'A module that belongs to the new fork cannot import a module '+
+'from the old fork.',
+fix(fixer){
+returnfixer.replaceText(
+sourceNode,
+`'${filename.replace(/\.old(\.js)?$/,'.new')}'`
+);
+},
+});
+}
+};
+
 return{
-ImportDeclaration(node){
-constimportSourceNode=node.source;
-constfilename=importSourceNode.value;
-if(isOldFork(filename)){
-context.report({
-node: importSourceNode,
-message:
-'A module that belongs to the new fork cannot import a module '+
-'from the old fork.',
-fix(fixer){
-returnfixer.replaceText(
-importSourceNode,
-`'${filename.replace(/\.old(\.js)?$/,'.new')}'`
-);
-},
-});
-}
-},
+ImportDeclaration: visitor,
+ExportNamedDeclaration: visitor,
 };
 }