Add private-deep-imports lint (disallow metro*/src/...) (#1528)

Summary: Pull Request resolved: #1528

Differential Revision: D77832800

Author: robhogan

scripts/eslint/base.js

@@ -59,6 +59,8 @@ module.exports = {
     'flowtype/use-flow-type': 0,
     // flow handles this check for us, so it's not required
     'no-undef': 0,
+
+    'lint/metro-deep-imports': 'warn',
   },
   overrides: [
     {

scripts/eslint/rules/__tests__/metro-deep-imports-test.js

@@ -0,0 +1,55 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @flow
+ * @format
+ * @oncall react_native
+ */
+
+'use strict';
+
+const rule = require('../metro-deep-imports.js');
+const ESLintTester = require('eslint').RuleTester;
+
+ESLintTester.setDefaultConfig({
+  parser: require.resolve('hermes-eslint'),
+  parserOptions: {
+    ecmaVersion: 6,
+    sourceType: 'module',
+  },
+});
+
+const eslintTester = new ESLintTester();
+
+eslintTester.run('../metro-deep-imports', rule, {
+  valid: [
+    'require("metro")',
+    'const Foo = require("metro-subpkg")',
+    'require("metro/private/Bar")',
+    'import Baz from "metro-baz/private/Baz"',
+    'import NotMetro from "foo/src/bar"',
+
+    // metro-runtime allows subpath imports. We can't rely on package#exports
+    // redirections as they may be disabled under Metro, and we must be able
+    // to import single modules as polyfills are side-effectful.
+    'import Polyfill from "metro-runtime/src/polyfills/foo"',
+    'const Polyfill = require("metro-runtime/src/polyfills/foo")',
+  ],
+  invalid: [
+    {
+      code: 'const myLib = require("metro/src/lib")',
+      output: "const myLib = require('metro/private/lib')",
+    },
+    {
+      code: "import MetroInternal from 'metro-pkg/src/internal'",
+      output: "import MetroInternal from 'metro-pkg/private/internal'",
+    },
+    {
+      code: "import type {Bar} from 'metro-types/src/bar'",
+      output: "import type {Bar} from 'metro-types/private/bar'",
+    },
+  ].map(obj => ({...obj, errors: [{messageId: 'METRO_DEEP_IMPORT'}]})),
+});

scripts/eslint/rules/metro-deep-imports.js

@@ -0,0 +1,83 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @format
+ * @flow strict-local
+ */
+
+'use strict';
+
+// $FlowExpectedError[untyped-type-import] - eslint not typed in OSS
+import type {RuleModule, SuggestionReportDescriptor} from 'eslint';
+import type {StringLiteral} from 'hermes-estree';
+
+const METRO_DEEP_IMPORT_RE = /^(metro(?!-runtime)(?:-[a-z\-]+)?)\/src\//;
+const messageId = 'METRO_DEEP_IMPORT';
+
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description:
+        'Deep imports from Metro must use explicitly-private subpaths',
+    },
+    messages: {
+      METRO_DEEP_IMPORT:
+        "Metro deep imports from src ('{{originalImport.value}}') are deprecated. Prefer top level imports, or replace '/src/' with '/private/'.",
+    },
+    schema: [] as const,
+    fixable: 'code',
+  },
+
+  create(context) {
+    return {
+      ImportDeclaration(node) {
+        if (
+          typeof node.source.value !== 'string' ||
+          !METRO_DEEP_IMPORT_RE.test(node.source.value)
+        ) {
+          return;
+        }
+        const stringNode = node.source;
+        context.report({
+          node: node.source,
+          messageId,
+          data: {originalImport: stringNode.value},
+          fix: getFix(stringNode),
+        });
+      },
+      CallExpression(node) {
+        if (
+          node.callee.type !== 'Identifier' ||
+          node.callee.name !== 'require' ||
+          node.arguments.length < 1 ||
+          node.arguments[0].type !== 'Literal' ||
+          node.arguments[0].literalType !== 'string' ||
+          !METRO_DEEP_IMPORT_RE.test(node.arguments[0].value)
+        ) {
+          return;
+        }
+        const stringNode = node.arguments[0];
+        context.report({
+          node,
+          messageId,
+          data: {originalImport: stringNode.value},
+          fix: getFix(stringNode),
+        });
+      },
+    };
+  },
+} as RuleModule;
+
+function getFix(
+  nodeToReplace: StringLiteral,
+): SuggestionReportDescriptor['fix'] {
+  return fixer =>
+    fixer.replaceText(
+      nodeToReplace,
+      `'${nodeToReplace.value.replace(METRO_DEEP_IMPORT_RE, '$1/private/')}'`,
+    );
+}