@atlaskit/eslint-plugin-platform 2.8.0 → 2.9.0

package/dist/cjs/rules/import/no-jest-mock-barrel-files/index.js

@@ -232,6 +232,7 @@ function getImportPathForSourceFile(_ref) {
     fs = _ref.fs;
   var crossPackageName = exportInfo === null || exportInfo === void 0 || (_exportInfo$crossPack = exportInfo.crossPackageSource) === null || _exportInfo$crossPack === void 0 ? void 0 : _exportInfo$crossPack.packageName;
   if (crossPackageName) {
+    var _findExportForSourceF;
     var sourcePackageExportsMaps = getSourcePackageExportsMaps(fs);
     var exportsMap = sourcePackageExportsMaps.get(crossPackageName);
     if (!exportsMap) {
@@ -248,10 +249,10 @@ function getImportPathForSourceFile(_ref) {
         sourcePackageExportsMaps.set(crossPackageName, exportsMap);
       }
     }
-    var targetExportPath = exportsMap ? (0, _packageResolution.findExportForSourceFile)({
+    var targetExportPath = exportsMap ? (_findExportForSourceF = (0, _packageResolution.findExportForSourceFile)({
       sourceFilePath: sourceFilePath,
       exportsMap: exportsMap
-    }) : null;
+    })) === null || _findExportForSourceF === void 0 ? void 0 : _findExportForSourceF.exportPath : null;
     return targetExportPath ? crossPackageName + targetExportPath.slice(1) : crossPackageName;
   }
   return getRelativeImportPath({

package/dist/cjs/rules/import/no-relative-barrel-file-imports/index.js

@@ -68,6 +68,7 @@ function getImportPathForSourceFile(_ref) {
     fs = _ref.fs;
   var crossPackageName = exportInfo === null || exportInfo === void 0 || (_exportInfo$crossPack = exportInfo.crossPackageSource) === null || _exportInfo$crossPack === void 0 ? void 0 : _exportInfo$crossPack.packageName;
   if (crossPackageName) {
+    var _findExportForSourceF;
     var sourcePackageExportsMaps = getSourcePackageExportsMaps(fs);
     var exportsMap = sourcePackageExportsMaps.get(crossPackageName);
     if (!exportsMap) {
@@ -84,10 +85,10 @@ function getImportPathForSourceFile(_ref) {
         sourcePackageExportsMaps.set(crossPackageName, exportsMap);
       }
     }
-    var targetExportPath = exportsMap ? (0, _packageResolution.findExportForSourceFile)({
+    var targetExportPath = exportsMap ? (_findExportForSourceF = (0, _packageResolution.findExportForSourceFile)({
       sourceFilePath: sourceFilePath,
       exportsMap: exportsMap
-    }) : null;
+    })) === null || _findExportForSourceF === void 0 ? void 0 : _findExportForSourceF.exportPath : null;
     return targetExportPath ? crossPackageName + targetExportPath.slice(1) : crossPackageName;
   }
   return getRelativeImportPath({
@@ -466,10 +467,13 @@ function transformExportSpecifiers(_ref1) {
   var specsWithOriginal = _ref1.specsWithOriginal;
   return specsWithOriginal.map(function (_ref10) {
     var originalName = _ref10.originalName,
+      nameInSource = _ref10.nameInSource,
       nameInLocal = _ref10.nameInLocal,
       kind = _ref10.kind;
     return {
-      nameInSource: originalName || 'default',
+      // Use originalName if available (means there was an alias in the barrel),
+      // otherwise use nameInSource (the direct export name from the barrel)
+      nameInSource: originalName !== null && originalName !== void 0 ? originalName : nameInSource,
       nameInLocal: nameInLocal,
       kind: kind
     };

package/dist/cjs/rules/import/shared/jest-utils.js

@@ -5,6 +5,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.extractImportPath = extractImportPath;
+exports.findJestRequireActualCalls = findJestRequireActualCalls;
 exports.findJestRequireMockCalls = findJestRequireMockCalls;
 exports.isJestMockCall = isJestMockCall;
 exports.isJestRequireActual = isJestRequireActual;
@@ -136,6 +137,58 @@ function findJestRequireMockCalls(_ref) {
   return results;
 }
 
+/**
+ * Find all jest.requireActual() calls in the AST whose import path matches a given target.
+ * Works identically to findJestRequireMockCalls but for requireActual.
+ */
+function findJestRequireActualCalls(_ref2) {
+  var ast = _ref2.ast,
+    matchPath = _ref2.matchPath;
+  var results = [];
+  var visited = new WeakSet();
+  var skipKeys = new Set(['parent', 'loc', 'range', 'tokens', 'comments']);
+  function visit(node) {
+    if (visited.has(node)) {
+      return;
+    }
+    visited.add(node);
+    if (node.type === 'CallExpression' && isJestRequireActual(node)) {
+      var path = extractImportPath(node);
+      if (path && matchPath(path)) {
+        results.push(node);
+      }
+    }
+    for (var key in node) {
+      if (skipKeys.has(key)) {
+        continue;
+      }
+      var value = node[key];
+      if (value && (0, _typeof2.default)(value) === 'object') {
+        if (Array.isArray(value)) {
+          var _iterator2 = _createForOfIteratorHelper(value),
+            _step2;
+          try {
+            for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
+              var child = _step2.value;
+              if (child && (0, _typeof2.default)(child) === 'object' && 'type' in child) {
+                visit(child);
+              }
+            }
+          } catch (err) {
+            _iterator2.e(err);
+          } finally {
+            _iterator2.f();
+          }
+        } else if ('type' in value) {
+          visit(value);
+        }
+      }
+    }
+  }
+  visit(ast);
+  return results;
+}
+
 /**
  * Determine the best new import path for a jest.requireMock() call by inspecting
  * the destructured symbols or property access at the call site.
@@ -144,20 +197,20 @@ function findJestRequireMockCalls(_ref) {
  * @param symbolToNewPath - Map from symbol name to the new mock path that provides it
  * @returns The resolved new path, or null if it cannot be determined
  */
-function resolveNewPathForRequireMock(_ref2) {
-  var requireMockNode = _ref2.requireMockNode,
-    symbolToNewPath = _ref2.symbolToNewPath;
+function resolveNewPathForRequireMock(_ref3) {
+  var requireMockNode = _ref3.requireMockNode,
+    symbolToNewPath = _ref3.symbolToNewPath;
   var parent = requireMockNode.parent;
   if (parent) {
     var _parent;
     // Check for destructuring pattern: const { foo } = jest.requireMock('...')
     var declarator = parent.type === 'VariableDeclarator' ? parent : ((_parent = parent.parent) === null || _parent === void 0 ? void 0 : _parent.type) === 'VariableDeclarator' ? parent.parent : null;
     if (declarator && declarator.type === 'VariableDeclarator' && declarator.id.type === 'ObjectPattern') {
-      var _iterator2 = _createForOfIteratorHelper(declarator.id.properties),
-        _step2;
+      var _iterator3 = _createForOfIteratorHelper(declarator.id.properties),
+        _step3;
       try {
-        for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
-          var prop = _step2.value;
+        for (_iterator3.s(); !(_step3 = _iterator3.n()).done;) {
+          var prop = _step3.value;
           if (prop.type === 'Property' && prop.key.type === 'Identifier') {
             var matchedPath = symbolToNewPath.get(prop.key.name);
             if (matchedPath) {
@@ -166,9 +219,9 @@ function resolveNewPathForRequireMock(_ref2) {
           }
         }
       } catch (err) {
-        _iterator2.e(err);
+        _iterator3.e(err);
       } finally {
-        _iterator2.f();
+        _iterator3.f();
       }
     }
 

package/dist/cjs/rules/import/shared/package-resolution.js

@@ -1,6 +1,7 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
+var _typeof3 = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -11,17 +12,96 @@ exports.resolveCrossPackageImport = resolveCrossPackageImport;
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _typeof2 = _interopRequireDefault(require("@babel/runtime/helpers/typeof"));
 var _path = require("path");
+var ts = _interopRequireWildcard(require("typescript"));
 var _fileSystem = require("./file-system");
 var _packageRegistry = require("./package-registry");
+function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof3(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
 function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
 function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
+var ENTRY_POINT_FOLDER_NAMES = new Set(['entry-points', 'entrypoints', 'entrypoint', 'entry-point']);
+function isInEntryPointsFolder(filePath) {
+  var parts = filePath.split(/[/\\]/);
+  return parts.some(function (part) {
+    return ENTRY_POINT_FOLDER_NAMES.has(part);
+  });
+}
 /**
- * Parse the package.json exports field and return a map of export paths to resolved file paths.
+ * Parse an entry-point wrapper file and resolve the source files it re-exports from,
+ * along with name mappings (source export name → entry-point export name).
  */
-function parsePackageExports(_ref) {
-  var packageDir = _ref.packageDir,
+function resolveEntryPointReExports(_ref) {
+  var entryPointFilePath = _ref.entryPointFilePath,
     fs = _ref.fs;
+  var content = (0, _fileSystem.readFileContent)({
+    filePath: entryPointFilePath,
+    fs: fs
+  });
+  if (!content) {
+    return [];
+  }
+  try {
+    var sourceFile = ts.createSourceFile(entryPointFilePath, content, ts.ScriptTarget.Latest, true);
+    var basedir = (0, _path.dirname)(entryPointFilePath);
+    var results = [];
+    var _iterator = _createForOfIteratorHelper(sourceFile.statements),
+      _step;
+    try {
+      for (_iterator.s(); !(_step = _iterator.n()).done;) {
+        var statement = _step.value;
+        if (ts.isExportDeclaration(statement) && statement.moduleSpecifier && ts.isStringLiteral(statement.moduleSpecifier)) {
+          var modulePath = statement.moduleSpecifier.text;
+          if (!(0, _fileSystem.isRelativeImport)(modulePath)) {
+            continue;
+          }
+          var resolved = (0, _fileSystem.resolveImportPath)({
+            basedir: basedir,
+            importPath: modulePath,
+            fs: fs
+          });
+          if (!resolved) {
+            continue;
+          }
+          var nameMap = new Map();
+          if (statement.exportClause && ts.isNamedExports(statement.exportClause)) {
+            var _iterator2 = _createForOfIteratorHelper(statement.exportClause.elements),
+              _step2;
+            try {
+              for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
+                var element = _step2.value;
+                var exportedName = element.name.text;
+                var sourceName = element.propertyName ? element.propertyName.text : exportedName;
+                nameMap.set(sourceName, exportedName);
+              }
+            } catch (err) {
+              _iterator2.e(err);
+            } finally {
+              _iterator2.f();
+            }
+          }
+          results.push({
+            sourcePath: resolved,
+            nameMap: nameMap
+          });
+        }
+      }
+    } catch (err) {
+      _iterator.e(err);
+    } finally {
+      _iterator.f();
+    }
+    return results;
+  } catch (_unused) {
+    return [];
+  }
+}
+
+/**
+ * Parse the package.json exports field and return a map of export paths to resolved file paths.
+ */
+function parsePackageExports(_ref2) {
+  var packageDir = _ref2.packageDir,
+    fs = _ref2.fs;
   // Memoize per-package to avoid repeated reads/parses during IDE lint runs.
   // Additionally, invalidate per-package if the package.json mtime changes
   // (covers unstaged local edits in IDE).
@@ -33,7 +113,7 @@ function parsePackageExports(_ref) {
   try {
     var _fs$statSync$mtimeMs;
     currentMtimeMs = (_fs$statSync$mtimeMs = fs.statSync(packageJsonPath).mtimeMs) !== null && _fs$statSync$mtimeMs !== void 0 ? _fs$statSync$mtimeMs : null;
-  } catch (_unused) {
+  } catch (_unused2) {
     // If package.json can't be stat'ed (missing/inaccessible), use null to force re-read
     currentMtimeMs = null;
   }
@@ -82,7 +162,7 @@ function parsePackageExports(_ref) {
         }
       }
     }
-  } catch (_unused2) {
+  } catch (_unused3) {
     // Ignore parsing errors
   }
 
@@ -93,29 +173,82 @@ function parsePackageExports(_ref) {
   });
   return exportsMap;
 }
-
 /**
  * Find a matching export entry for a given source file path.
  * Returns the export path (e.g., "./controllers/analytics") or null if not found.
+ *
+ * When `fs` is provided, also checks entry-point wrapper files. If an export resolves
+ * to a file inside a recognized entry-points folder (entry-points, entrypoints, etc.),
+ * the wrapper is parsed to see if it re-exports from `sourceFilePath`.
+ *
+ * `sourceExportName` is the name under which the symbol is exported from the source file
+ * (e.g. `'default'`). Used to look up the corresponding entry-point export name so the
+ * caller can generate the correct import style.
  */
-function findExportForSourceFile(_ref2) {
-  var sourceFilePath = _ref2.sourceFilePath,
-    exportsMap = _ref2.exportsMap;
-  var _iterator = _createForOfIteratorHelper(exportsMap),
-    _step;
+function findExportForSourceFile(_ref3) {
+  var sourceFilePath = _ref3.sourceFilePath,
+    exportsMap = _ref3.exportsMap,
+    fs = _ref3.fs,
+    sourceExportName = _ref3.sourceExportName;
+  var _iterator3 = _createForOfIteratorHelper(exportsMap),
+    _step3;
   try {
-    for (_iterator.s(); !(_step = _iterator.n()).done;) {
-      var _step$value = (0, _slicedToArray2.default)(_step.value, 2),
-        exportPath = _step$value[0],
-        resolvedPath = _step$value[1];
-      if (resolvedPath === sourceFilePath) {
-        return exportPath;
+    for (_iterator3.s(); !(_step3 = _iterator3.n()).done;) {
+      var _step3$value = (0, _slicedToArray2.default)(_step3.value, 2),
+        _exportPath = _step3$value[0],
+        _resolvedPath = _step3$value[1];
+      if (_resolvedPath === sourceFilePath) {
+        return {
+          exportPath: _exportPath
+        };
       }
     }
   } catch (err) {
-    _iterator.e(err);
+    _iterator3.e(err);
   } finally {
-    _iterator.f();
+    _iterator3.f();
+  }
+  if (fs) {
+    var _iterator4 = _createForOfIteratorHelper(exportsMap),
+      _step4;
+    try {
+      for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
+        var _step4$value = (0, _slicedToArray2.default)(_step4.value, 2),
+          exportPath = _step4$value[0],
+          resolvedPath = _step4$value[1];
+        if (isInEntryPointsFolder(resolvedPath)) {
+          var reExports = resolveEntryPointReExports({
+            entryPointFilePath: resolvedPath,
+            fs: fs
+          });
+          var _iterator5 = _createForOfIteratorHelper(reExports),
+            _step5;
+          try {
+            for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
+              var reExport = _step5.value;
+              if (reExport.sourcePath === sourceFilePath) {
+                var entryPointExportName = void 0;
+                if (sourceExportName !== undefined && reExport.nameMap.has(sourceExportName)) {
+                  entryPointExportName = reExport.nameMap.get(sourceExportName);
+                }
+                return {
+                  exportPath: exportPath,
+                  entryPointExportName: entryPointExportName
+                };
+              }
+            }
+          } catch (err) {
+            _iterator5.e(err);
+          } finally {
+            _iterator5.f();
+          }
+        }
+      }
+    } catch (err) {
+      _iterator4.e(err);
+    } finally {
+      _iterator4.f();
+    }
   }
   return null;
 }
@@ -139,10 +272,10 @@ function extractPackageNameFromImport(moduleSpecifier) {
  * Resolve a cross-package import to its package directory and export info.
  * Returns null if the package is not in the target folder or cannot be resolved.
  */
-function resolveCrossPackageImport(_ref3) {
-  var moduleSpecifier = _ref3.moduleSpecifier,
-    workspaceRoot = _ref3.workspaceRoot,
-    fs = _ref3.fs;
+function resolveCrossPackageImport(_ref4) {
+  var moduleSpecifier = _ref4.moduleSpecifier,
+    workspaceRoot = _ref4.workspaceRoot,
+    fs = _ref4.fs;
   // Only handle @atlassian/* scoped packages
   var parsed = extractPackageNameFromImport(moduleSpecifier);
   if (!parsed) {