4a1c2a9ece
Summary: Added infra for writing and using custom eslint rules and created a rule for disallowing cross-package file imports. Such imports are anti-pattern and they also break standalone plugin bundling. We still have a bunch of places where Flipper core references code directly from plugins. I've ignored these places for now and added task T71355623 to revisit them. Reviewed By: passy Differential Revision: D22998955 fbshipit-source-id: d04cff8fc115ba1300a7e6830306ec134046e927
44 lines
1.3 KiB
TypeScript
44 lines
1.3 KiB
TypeScript
/**
|
|
* 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.
|
|
*
|
|
* @format
|
|
*/
|
|
|
|
import {CallExpression} from '@babel/types';
|
|
import {NodePath} from '@babel/traverse';
|
|
|
|
module.exports = () => ({
|
|
name: 'change-electron-to-electronRequire-in-main',
|
|
visitor: {
|
|
CallExpression(path: NodePath<CallExpression>) {
|
|
const node = path.node;
|
|
if (
|
|
node.type === 'CallExpression' &&
|
|
node.callee.type === 'Identifier' &&
|
|
node.callee.name === 'require' &&
|
|
node.arguments.length === 1 &&
|
|
node.arguments[0].type === 'StringLiteral'
|
|
) {
|
|
const source = node.arguments[0].value;
|
|
if (!source.startsWith('./') && !source.startsWith('../')) {
|
|
node.callee.name = 'electronRequire';
|
|
}
|
|
}
|
|
if (
|
|
node.callee.type === 'MemberExpression' &&
|
|
node.callee.object.type === 'Identifier' &&
|
|
node.callee.object.name === 'require' &&
|
|
node.callee.property.type === 'Identifier' &&
|
|
node.callee.property.name === 'resolve' &&
|
|
node.arguments.length === 1 &&
|
|
node.arguments[0].type == 'StringLiteral'
|
|
) {
|
|
node.callee.object.name = 'electronRequire';
|
|
}
|
|
},
|
|
},
|
|
});
|