@atlaskit/eslint-plugin-design-system 11.1.0 → 11.3.0

package/CHANGELOG.md

@@ -1,5 +1,44 @@
 # @atlaskit/eslint-plugin-design-system
 
+## 11.3.0
+
+### Minor Changes
+
+- [#175432](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/175432)
+  [`0a1070983ac76`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/0a1070983ac76) -
+  `use-tokens-typography` rule now reports on `fontWeight` used with raw values (e.g. 600, 'bold').
+  If a matching token is found, a fix is suggested. This rule is disabled by default but can be
+  enabled under the name `font-weight`.
+
+### Patch Changes
+
+- [#173274](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/173274)
+  [`fd5bac37c73ba`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/fd5bac37c73ba) -
+  Fix eslint type error.
+
+## 11.2.0
+
+### Minor Changes
+
+- [#175583](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/175583)
+  [`75911cb003bd5`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/75911cb003bd5) - ####
+  no-legacy-icons
+
+  Add new `shouldUseSafeMigrationMode` flag to no-legacy-icons rule. When set to true, the autofixer
+  will only attempt to migrate icons that are visually similar and do not include secondary colors
+  or sizes other than medium.
+
+  Additionally, the autofixer will no longer attempt to explicity add `color="currentColor"` for
+  every migration as this is now the default.
+
+  #### no-deprecated-apis
+
+  Refactored to fix type errors and match code style of other rules.
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 11.1.0
 
 ### Minor Changes

package/dist/cjs/rules/no-deprecated-apis/index.js

@@ -1,43 +1,36 @@
 "use strict";
 
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.noDeprecatedJSXAttributeMessageId = exports.name = exports.default = void 0;
-var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
-var _utils = require("@typescript-eslint/utils");
+var _eslintCodemodUtils = require("eslint-codemod-utils");
 var _createRule = require("../utils/create-rule");
 var _getDeprecatedConfig = require("../utils/get-deprecated-config");
 var _types = require("../utils/types");
 var noDeprecatedJSXAttributeMessageId = exports.noDeprecatedJSXAttributeMessageId = 'noDeprecatedJSXAttributes';
-var isNodeOfType = function isNodeOfType(node, nodeType) {
-  return _utils.ASTUtils.isNodeOfType(nodeType)(node);
-};
 var isImportDeclaration = function isImportDeclaration(programStatement) {
   return (programStatement === null || programStatement === void 0 ? void 0 : programStatement.type) === 'ImportDeclaration';
 };
 var findJSXElementName = function findJSXElementName(jsxAttributeNode) {
-  if (!jsxAttributeNode.parent || !isNodeOfType(jsxAttributeNode.parent, _utils.AST_NODE_TYPES.JSXOpeningElement)) {
+  if (!jsxAttributeNode.parent || !(0, _eslintCodemodUtils.isNodeOfType)(jsxAttributeNode === null || jsxAttributeNode === void 0 ? void 0 : jsxAttributeNode.parent, 'JSXOpeningElement')) {
     return;
   }
   var openingElement = jsxAttributeNode.parent;
-  if (!isNodeOfType(openingElement.name, _utils.AST_NODE_TYPES.JSXIdentifier)) {
+  if (!(0, _eslintCodemodUtils.isNodeOfType)(openingElement.name, 'JSXIdentifier')) {
     return;
   }
   return openingElement.name.name;
 };
 var name = exports.name = 'no-deprecated-apis';
-var rule = (0, _createRule.createRule)({
-  name: name,
-  defaultOptions: [{
-    deprecatedConfig: (0, _getDeprecatedConfig.getConfig)('jsxAttributes')
-  }],
+var rule = (0, _createRule.createLintRule)({
   meta: {
+    name: name,
     type: 'suggestion',
     docs: {
       description: 'Disallow using deprecated APIs.',
-      recommended: 'strict'
+      recommended: true,
+      severity: 'error'
     },
     messages: {
       noDeprecatedJSXAttributes: 'The JSX attribute {{propName}} has been deprecated.'
@@ -75,24 +68,18 @@ var rule = (0, _createRule.createRule)({
       }
     }]
   },
-  create: function create(context, _ref) {
+  create: function create(context) {
     var _context$options$;
-    var _ref2 = (0, _slicedToArray2.default)(_ref, 1),
-      options = _ref2[0];
-    // Get rule configuration
-    var defaultDeprecatedConfig = options.deprecatedConfig;
-
     // Get the rule configuration specified otherwise use default config.
     // A bit confusing as it seems that the default options have precedence over the user specified options.
-    var deprecatedConfig = ((_context$options$ = context.options[0]) === null || _context$options$ === void 0 ? void 0 : _context$options$.deprecatedConfig) || defaultDeprecatedConfig;
+    var deprecatedConfig = ((_context$options$ = context.options[0]) === null || _context$options$ === void 0 ? void 0 : _context$options$.deprecatedConfig) || (0, _getDeprecatedConfig.getConfig)('jsxAttributes');
     return {
       // find JSX atribute - find name of attribute - get source and find relevant identifiers.
       JSXAttribute: function JSXAttribute(node) {
-        var jsxAttributeIdentifier = node.name;
-        if (!isNodeOfType(jsxAttributeIdentifier, _utils.AST_NODE_TYPES.JSXIdentifier)) {
+        if (!(0, _eslintCodemodUtils.isNodeOfType)(node, 'JSXAttribute') || !(0, _eslintCodemodUtils.isNodeOfType)(node.name, 'JSXIdentifier')) {
           return;
         }
-        var jsxAttributeName = jsxAttributeIdentifier.name;
+        var jsxAttributeName = node.name.name;
         if (!(0, _types.isDeprecatedJSXAttributeConfig)(deprecatedConfig) || !deprecatedConfig[jsxAttributeName]) {
           return;
         }
@@ -100,13 +87,13 @@ var rule = (0, _createRule.createRule)({
         if (!jsxElementName) {
           return;
         }
-        var source = context.getSourceCode();
+        var source = context.sourceCode;
 
         // find an import for the path of the banned api
         deprecatedConfig[jsxAttributeName].forEach(function (importItem) {
           var _importItem$namedSpec;
           var importNode = source.ast.body.filter(isImportDeclaration).find(function (node) {
-            return node.source.value.includes(importItem.moduleSpecifier);
+            return node && node.source.value && typeof node.source.value === 'string' && node.source.value.includes(importItem.moduleSpecifier);
           });
           if (!importNode) {
             return;

package/dist/cjs/rules/no-legacy-icons/checks.js

@@ -10,7 +10,7 @@ function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol
 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 createChecks = exports.createChecks = function createChecks(context) {
-  //create global variables to be shared by the checks
+  // Create global variables to be shared by the checks
   var _createHelpers = (0, _helpers.createHelpers)(context),
     getPrimaryColor = _createHelpers.getPrimaryColor,
     getConfigFlag = _createHelpers.getConfigFlag;
@@ -27,6 +27,7 @@ var createChecks = exports.createChecks = function createChecks(context) {
   var shouldErrorForAutoMigration = getConfigFlag('shouldErrorForAutoMigration', true);
   var isQuietMode = getConfigFlag('quiet', false);
   var shouldUseMigrationPath = getConfigFlag('shouldUseMigrationPath', true);
+  var shouldUseSafeMigrationMode = getConfigFlag('shouldUseSafeMigrationMode', false);
 
   // Sorted list of ranges
   var errorRanges = [];
@@ -411,10 +412,11 @@ var createChecks = exports.createChecks = function createChecks(context) {
       // Find size prop on node
       var size = 'medium';
       var primaryColor = null;
+      var hasPrimaryColorProp = false;
+      var hasSecondaryColorProp = false;
       var afterSpreadSet = new Set();
       var requiredAttributesAfterSpread = new Set(['size', 'primaryColor', 'secondaryColor']);
       var hasSpread = false;
-      var hasPrimaryColorProp = false;
       var _iterator8 = _createForOfIteratorHelper(node.openingElement.attributes),
         _step8;
       try {
@@ -449,6 +451,9 @@ var createChecks = exports.createChecks = function createChecks(context) {
               primaryColor = getPrimaryColor(attr);
               hasPrimaryColorProp = true;
               break;
+            case 'secondaryColor':
+              hasSecondaryColorProp = true;
+              break;
           }
         }
       } catch (err) {
@@ -457,6 +462,8 @@ var createChecks = exports.createChecks = function createChecks(context) {
         _iterator8.f();
       }
       var hasManualMigration = false;
+
+      // Flag manual migration if primary color cannot be migrated
       if (primaryColor && !(0, _helpers.canMigrateColor)(primaryColor) || hasPrimaryColorProp && !primaryColor) {
         (0, _helpers.createCantMigrateColorError)(node, primaryColor ? "the value of '".concat(primaryColor, "'") : 'a statically unknown value', errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
@@ -468,6 +475,7 @@ var createChecks = exports.createChecks = function createChecks(context) {
         (0, _helpers.createCantMigrateSizeUnknown)(node, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
+
       // Do a set comparison - is requiredAttributesAfterSpread a subset of afterSpreadSet?
       if (hasSpread === true && !Array.from(requiredAttributesAfterSpread).every(function (val) {
         return afterSpreadSet.has(val);
@@ -478,6 +486,7 @@ var createChecks = exports.createChecks = function createChecks(context) {
         (0, _helpers.createCantMigrateSpreadPropsError)(node, missingProps, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
+
       // Check if it is an exported component?
       if (legacyIconImports[name].exported) {
         (0, _helpers.createCantMigrateReExportError)(node, legacyIconImports[name].packageName, name, errorsManual);
@@ -507,7 +516,9 @@ var createChecks = exports.createChecks = function createChecks(context) {
           spacing = 'spacious';
         }
       }
-      if (!hasManualMigration && (newIcon || upcomingIcon) && isNewIconMigratable) {
+      if (shouldUseSafeMigrationMode && !hasManualMigration && (newIcon !== null && newIcon !== void 0 && newIcon.isMigrationUnsafe || size !== 'medium' || hasSecondaryColorProp)) {
+        (0, _helpers.createCantFindSuitableReplacementError)(node, legacyIconImports[name].packageName, name, errorsManual, upcomingIcon ? true : migrationMapObject ? true : false);
+      } else if (!hasManualMigration && (newIcon || upcomingIcon) && isNewIconMigratable) {
         (0, _helpers.createAutoMigrationError)({
           node: node,
           importSource: legacyIconImports[name].packageName,

package/dist/cjs/rules/no-legacy-icons/helpers.js

@@ -480,8 +480,7 @@ var createPropFixes = function createPropFixes(_ref7) {
     migrationImportNode = _ref7.migrationImportNode,
     newIconName = _ref7.newIconName;
   var fixes = [];
-  var spacing = metadata.spacing,
-    insideNewButton = metadata.insideNewButton;
+  var spacing = metadata.spacing;
   if (shouldUseMigrationPath && !legacyImportNode) {
     return fixes;
   }
@@ -498,16 +497,6 @@ var createPropFixes = function createPropFixes(_ref7) {
       fixes.push(fixer.replaceText(primaryColor.name, 'color'));
     }
 
-    // add color="currentColor" if
-    // 1. primaryColor prop is not set
-    // 2. icon is not imported from migration entrypoint
-    // 3. icon element is not inside a new button
-    if (legacyImportNode && !primaryColor && !migrationImportNode &&
-    // value type need to be a string in Rule.ReportDescriptor
-    insideNewButton !== 'true') {
-      fixes.push(fixer.insertTextAfter(openingElement.name, " color=\"currentColor\""));
-    }
-
     // rename or remove size prop based on shouldUseMigrationPath,
     // add spacing="spacious" if
     // 1. it's in error metadata, which means size is medium
@@ -637,12 +626,12 @@ var throwAutoErrors = exports.throwAutoErrors = function throwAutoErrors(_ref10)
     }
     return result;
   }, new Set());
-  //group errors by import source and remove any unwanted errors
+  // Group errors by import source and remove any unwanted errors
   var groupedErrorList = Object.entries(errorsAuto).reduce(function (result, option) {
     var _option2 = (0, _slicedToArray2.default)(option, 2),
       key = _option2[0],
       error = _option2[1];
-    //return early if no data
+    // Return early if no data
     if (!error.data) {
       return result;
     }

package/dist/cjs/rules/no-legacy-icons/index.js

@@ -28,6 +28,9 @@ var rule = (0, _createRule.createLintRule)({
         shouldErrorForAutoMigration: {
           type: 'boolean'
         },
+        shouldUseSafeMigrationMode: {
+          type: 'boolean'
+        },
         quiet: {
           type: 'boolean'
         },

package/dist/cjs/rules/use-tokens-typography/config/index.js

@@ -26,7 +26,8 @@ var ruleSchema = exports.ruleSchema = {
 var defaultConfig = {
   failSilently: false,
   shouldEnforceFallbacks: false,
-  enableUnsafeAutofix: false
+  enableUnsafeAutofix: false,
+  patterns: ['style-object']
 };
 var getConfig = exports.getConfig = function getConfig(overrides) {
   return Object.assign({}, defaultConfig, overrides);

package/dist/cjs/rules/use-tokens-typography/index.js

@@ -7,18 +7,26 @@ exports.default = void 0;
 var _createRule = require("../utils/create-rule");
 var _errorBoundary = require("../utils/error-boundary");
 var _config = require("./config");
+var _fontWeight = require("./transformers/font-weight");
 var _styleObject = require("./transformers/style-object");
 var create = function create(context) {
   var config = (0, _config.getConfig)(context.options[0]);
-  return {
-    // const styles = css({ fontSize: '14px, ... }), styled.div({ fontSize: 14, ... })
-    ObjectExpression: (0, _errorBoundary.errorBoundary)(function (node) {
+  return (0, _errorBoundary.errorBoundary)({
+    // const styles = css({ fontSize: '14px', ... }), styled.div({ fontSize: 14, ... })
+    ObjectExpression: function ObjectExpression(node) {
       return _styleObject.StyleObject.lint(node, {
         context: context,
         config: config
       });
-    }, config)
-  };
+    },
+    // const styles = css({ fontWeight: 600, 'bold', ... })
+    'ObjectExpression > Property > Identifier[name=/fontWeight/]': function ObjectExpressionPropertyIdentifierNameFontWeight(node) {
+      return _fontWeight.FontWeight.lint(node, {
+        context: context,
+        config: config
+      });
+    }
+  }, config);
 };
 var rule = (0, _createRule.createLintRule)({
   meta: {
@@ -32,7 +40,8 @@ var rule = (0, _createRule.createLintRule)({
       severity: 'warn'
     },
     messages: {
-      noRawTypographyValues: 'Typography primitives or tokens should be used instead of hard-coded values.\n\n@meta <<{{payload}}>>.\n\nNOTE: Using tokens with the `fontSize` property is invalid. Any `font.heading` or `font.body` tokens must use the CSS `font` property.'
+      noRawTypographyValues: 'Typography primitives or tokens should be used instead of hard-coded values.\n\n@meta <<{{payload}}>>.\n\nNOTE: Using tokens with the `fontSize` property is invalid. Any `font.heading` or `font.body` tokens must use the CSS `font` property.',
+      noRawFontWeightValues: 'Font weight tokens should be used instead of hard-coded values.'
     },
     schema: _config.ruleSchema
   },

package/dist/cjs/rules/use-tokens-typography/transformers/font-weight.js

@@ -0,0 +1,96 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.FontWeight = void 0;
+var _eslintCodemodUtils = require("eslint-codemod-utils");
+var _astNodes = require("../../../ast-nodes");
+var _utils = require("../../ensure-design-token-usage/utils");
+var _isNode = require("../../utils/is-node");
+var _utils2 = require("../utils");
+/* eslint-disable @repo/internal/react/require-jsdoc */
+
+var FontWeight = exports.FontWeight = {
+  lint: function lint(node, _ref) {
+    var context = _ref.context,
+      config = _ref.config;
+    // To force the correct node type
+    if (!(0, _eslintCodemodUtils.isNodeOfType)(node, 'Identifier')) {
+      return;
+    }
+
+    // Check whether all criteria needed to make a transformation are met
+    var success = FontWeight._check(node, {
+      context: context,
+      config: config
+    });
+    if (success) {
+      return context.report({
+        node: node,
+        messageId: 'noRawFontWeightValues',
+        fix: FontWeight._fix(node, context)
+      });
+    }
+  },
+  _check: function _check(node, _ref2) {
+    var context = _ref2.context,
+      config = _ref2.config;
+    if (!config.patterns.includes('font-weight')) {
+      return false;
+    }
+    if (!(0, _isNode.isDecendantOfStyleBlock)(node) && !(0, _isNode.isDecendantOfType)(node, 'JSXExpressionContainer')) {
+      return false;
+    }
+    if (!(0, _eslintCodemodUtils.isNodeOfType)(node.parent, 'Property')) {
+      return false;
+    }
+    var fontWeightValue = (0, _utils.getValueForPropertyNode)(node.parent, context);
+    if (typeof fontWeightValue === 'string' && fontWeightValue.includes('font.weight.')) {
+      return false;
+    }
+    return true;
+  },
+  _fix: function _fix(node, context) {
+    return function (fixer) {
+      var _findFontWeightTokenF;
+      var fixes = [];
+
+      // -- Type assertions to force the correct node type --
+
+      if (!(0, _eslintCodemodUtils.isNodeOfType)(node.parent, 'Property')) {
+        return [];
+      }
+      if (!(0, _eslintCodemodUtils.isNodeOfType)(node.parent.value, 'Literal')) {
+        return [];
+      }
+      if (!node.parent.value.raw) {
+        return [];
+      }
+
+      // -- Fix: Replace raw value with token --
+
+      var matchingToken = (_findFontWeightTokenF = (0, _utils2.findFontWeightTokenForValue)(node.parent.value.raw)) === null || _findFontWeightTokenF === void 0 ? void 0 : _findFontWeightTokenF.tokenName;
+      if (!matchingToken) {
+        return [];
+      }
+      var fontWeightValueFix = fixer.replaceText(node.parent.value, "token('".concat(matchingToken, "')"));
+      fixes.push(fontWeightValueFix);
+
+      // -- Fix: Add import if it doesn't exist --
+
+      var body = context.sourceCode.ast.body;
+      var tokensImportDeclarations = _astNodes.Root.findImportsByModule(body, '@atlaskit/tokens');
+
+      // If there is more than one `@atlaskit/tokens` import, then it becomes difficult to determine which import to transform
+      if (tokensImportDeclarations.length > 1) {
+        return fixes;
+      }
+      var tokensImportDeclaration = tokensImportDeclarations[0];
+      if (!tokensImportDeclaration) {
+        fixes.push((0, _utils2.insertTokensImport)(body, fixer));
+      }
+      return fixes;
+    };
+  }
+};

package/dist/cjs/rules/use-tokens-typography/transformers/style-object.js

@@ -204,6 +204,11 @@ var StyleObject = exports.StyleObject = {
   _check: function _check(node, _ref2) {
     var context = _ref2.context,
       config = _ref2.config;
+    if (!config.patterns.includes('style-object')) {
+      return {
+        success: false
+      };
+    }
     if (!(0, _isNode.isDecendantOfStyleBlock)(node) && !(0, _isNode.isDecendantOfType)(node, 'JSXExpressionContainer')) {
       return {
         success: false

package/dist/cjs/rules/use-tokens-typography/utils.js

@@ -99,6 +99,12 @@ var fontWeightTokens = exports.fontWeightTokens = _tokensRaw.typographyAdg3.filt
   };
 });
 function findFontWeightTokenForValue(fontWeight) {
+  if (fontWeight === "'normal'") {
+    fontWeight = '400';
+  }
+  if (fontWeight === "'bold'") {
+    fontWeight = '700';
+  }
   return fontWeightTokens.find(function (token) {
     return token.tokenValue === fontWeight;
   });

package/dist/es2019/rules/no-deprecated-apis/index.js

@@ -1,33 +1,30 @@
-import { AST_NODE_TYPES, ASTUtils } from '@typescript-eslint/utils';
-import { createRule } from '../utils/create-rule';
+import { isNodeOfType } from 'eslint-codemod-utils';
+import { createLintRule } from '../utils/create-rule';
 import { getConfig } from '../utils/get-deprecated-config';
 import { isDeprecatedJSXAttributeConfig } from '../utils/types';
 export const noDeprecatedJSXAttributeMessageId = 'noDeprecatedJSXAttributes';
-const isNodeOfType = (node, nodeType) => ASTUtils.isNodeOfType(nodeType)(node);
 const isImportDeclaration = programStatement => {
   return (programStatement === null || programStatement === void 0 ? void 0 : programStatement.type) === 'ImportDeclaration';
 };
 const findJSXElementName = jsxAttributeNode => {
-  if (!jsxAttributeNode.parent || !isNodeOfType(jsxAttributeNode.parent, AST_NODE_TYPES.JSXOpeningElement)) {
+  if (!jsxAttributeNode.parent || !isNodeOfType(jsxAttributeNode === null || jsxAttributeNode === void 0 ? void 0 : jsxAttributeNode.parent, 'JSXOpeningElement')) {
     return;
   }
   const openingElement = jsxAttributeNode.parent;
-  if (!isNodeOfType(openingElement.name, AST_NODE_TYPES.JSXIdentifier)) {
+  if (!isNodeOfType(openingElement.name, 'JSXIdentifier')) {
     return;
   }
   return openingElement.name.name;
 };
 export const name = 'no-deprecated-apis';
-const rule = createRule({
-  name,
-  defaultOptions: [{
-    deprecatedConfig: getConfig('jsxAttributes')
-  }],
+const rule = createLintRule({
   meta: {
+    name,
     type: 'suggestion',
     docs: {
       description: 'Disallow using deprecated APIs.',
-      recommended: 'strict'
+      recommended: true,
+      severity: 'error'
     },
     messages: {
       noDeprecatedJSXAttributes: 'The JSX attribute {{propName}} has been deprecated.'
@@ -65,24 +62,18 @@ const rule = createRule({
       }
     }]
   },
-  create(context, [options]) {
+  create(context) {
     var _context$options$;
-    // Get rule configuration
-    const {
-      deprecatedConfig: defaultDeprecatedConfig
-    } = options;
-
     // Get the rule configuration specified otherwise use default config.
     // A bit confusing as it seems that the default options have precedence over the user specified options.
-    const deprecatedConfig = ((_context$options$ = context.options[0]) === null || _context$options$ === void 0 ? void 0 : _context$options$.deprecatedConfig) || defaultDeprecatedConfig;
+    const deprecatedConfig = ((_context$options$ = context.options[0]) === null || _context$options$ === void 0 ? void 0 : _context$options$.deprecatedConfig) || getConfig('jsxAttributes');
     return {
       // find JSX atribute - find name of attribute - get source and find relevant identifiers.
       JSXAttribute(node) {
-        const jsxAttributeIdentifier = node.name;
-        if (!isNodeOfType(jsxAttributeIdentifier, AST_NODE_TYPES.JSXIdentifier)) {
+        if (!isNodeOfType(node, 'JSXAttribute') || !isNodeOfType(node.name, 'JSXIdentifier')) {
           return;
         }
-        const jsxAttributeName = jsxAttributeIdentifier.name;
+        const jsxAttributeName = node.name.name;
         if (!isDeprecatedJSXAttributeConfig(deprecatedConfig) || !deprecatedConfig[jsxAttributeName]) {
           return;
         }
@@ -90,12 +81,12 @@ const rule = createRule({
         if (!jsxElementName) {
           return;
         }
-        const source = context.getSourceCode();
+        const source = context.sourceCode;
 
         // find an import for the path of the banned api
         deprecatedConfig[jsxAttributeName].forEach(importItem => {
           var _importItem$namedSpec;
-          const importNode = source.ast.body.filter(isImportDeclaration).find(node => node.source.value.includes(importItem.moduleSpecifier));
+          const importNode = source.ast.body.filter(isImportDeclaration).find(node => node && node.source.value && typeof node.source.value === 'string' && node.source.value.includes(importItem.moduleSpecifier));
           if (!importNode) {
             return;
           }

package/dist/es2019/rules/no-legacy-icons/checks.js

@@ -1,7 +1,7 @@
 import { isNodeOfType } from 'eslint-codemod-utils';
 import { addToListOfRanges, canAutoMigrateNewIconBasedOnSize, canMigrateColor, createAutoMigrationError, createCantFindSuitableReplacementError, createCantMigrateColorError, createCantMigrateFunctionUnknownError, createCantMigrateIdentifierError, createCantMigrateIdentifierMapOrArrayError, createCantMigrateReExportError, createCantMigrateSizeUnknown, createCantMigrateSpreadPropsError, createGuidance, createHelpers, getMigrationMapObject, getUpcomingIcons, isInsideLegacyButton, isInsideNewButton, isSize, locToString, throwAutoErrors, throwManualErrors } from './helpers';
 export const createChecks = context => {
-  //create global variables to be shared by the checks
+  // Create global variables to be shared by the checks
   const {
     getPrimaryColor,
     getConfigFlag
@@ -19,6 +19,7 @@ export const createChecks = context => {
   const shouldErrorForAutoMigration = getConfigFlag('shouldErrorForAutoMigration', true);
   const isQuietMode = getConfigFlag('quiet', false);
   const shouldUseMigrationPath = getConfigFlag('shouldUseMigrationPath', true);
+  const shouldUseSafeMigrationMode = getConfigFlag('shouldUseSafeMigrationMode', false);
 
   // Sorted list of ranges
   let errorRanges = [];
@@ -334,10 +335,11 @@ export const createChecks = context => {
       // Find size prop on node
       let size = 'medium';
       let primaryColor = null;
+      let hasPrimaryColorProp = false;
+      let hasSecondaryColorProp = false;
       let afterSpreadSet = new Set();
       let requiredAttributesAfterSpread = new Set(['size', 'primaryColor', 'secondaryColor']);
       let hasSpread = false;
-      let hasPrimaryColorProp = false;
       for (const attr of node.openingElement.attributes) {
         // Detect spread props
         if (isNodeOfType(attr, 'JSXSpreadAttribute')) {
@@ -368,9 +370,14 @@ export const createChecks = context => {
             primaryColor = getPrimaryColor(attr);
             hasPrimaryColorProp = true;
             break;
+          case 'secondaryColor':
+            hasSecondaryColorProp = true;
+            break;
         }
       }
       let hasManualMigration = false;
+
+      // Flag manual migration if primary color cannot be migrated
       if (primaryColor && !canMigrateColor(primaryColor) || hasPrimaryColorProp && !primaryColor) {
         createCantMigrateColorError(node, primaryColor ? `the value of '${primaryColor}'` : 'a statically unknown value', errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
@@ -382,12 +389,14 @@ export const createChecks = context => {
         createCantMigrateSizeUnknown(node, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
+
       // Do a set comparison - is requiredAttributesAfterSpread a subset of afterSpreadSet?
       if (hasSpread === true && !Array.from(requiredAttributesAfterSpread).every(val => afterSpreadSet.has(val)) && !insideNewButton) {
         const missingProps = Array.from(requiredAttributesAfterSpread).filter(val => !afterSpreadSet.has(val));
         createCantMigrateSpreadPropsError(node, missingProps, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
+
       // Check if it is an exported component?
       if (legacyIconImports[name].exported) {
         createCantMigrateReExportError(node, legacyIconImports[name].packageName, name, errorsManual);
@@ -415,7 +424,9 @@ export const createChecks = context => {
           spacing = 'spacious';
         }
       }
-      if (!hasManualMigration && (newIcon || upcomingIcon) && isNewIconMigratable) {
+      if (shouldUseSafeMigrationMode && !hasManualMigration && (newIcon !== null && newIcon !== void 0 && newIcon.isMigrationUnsafe || size !== 'medium' || hasSecondaryColorProp)) {
+        createCantFindSuitableReplacementError(node, legacyIconImports[name].packageName, name, errorsManual, upcomingIcon ? true : migrationMapObject ? true : false);
+      } else if (!hasManualMigration && (newIcon || upcomingIcon) && isNewIconMigratable) {
         createAutoMigrationError({
           node,
           importSource: legacyIconImports[name].packageName,

package/dist/es2019/rules/no-legacy-icons/helpers.js

@@ -462,8 +462,7 @@ const createPropFixes = ({
 }) => {
   const fixes = [];
   const {
-    spacing,
-    insideNewButton
+    spacing
   } = metadata;
   if (shouldUseMigrationPath && !legacyImportNode) {
     return fixes;
@@ -485,16 +484,6 @@ const createPropFixes = ({
       fixes.push(fixer.replaceText(primaryColor.name, 'color'));
     }
 
-    // add color="currentColor" if
-    // 1. primaryColor prop is not set
-    // 2. icon is not imported from migration entrypoint
-    // 3. icon element is not inside a new button
-    if (legacyImportNode && !primaryColor && !migrationImportNode &&
-    // value type need to be a string in Rule.ReportDescriptor
-    insideNewButton !== 'true') {
-      fixes.push(fixer.insertTextAfter(openingElement.name, ` color="currentColor"`));
-    }
-
     // rename or remove size prop based on shouldUseMigrationPath,
     // add spacing="spacious" if
     // 1. it's in error metadata, which means size is medium
@@ -611,10 +600,10 @@ export const throwAutoErrors = ({
     }
     return result;
   }, new Set());
-  //group errors by import source and remove any unwanted errors
+  // Group errors by import source and remove any unwanted errors
   const groupedErrorList = Object.entries(errorsAuto).reduce((result, option) => {
     const [key, error] = option;
-    //return early if no data
+    // Return early if no data
     if (!error.data) {
       return result;
     }

package/dist/es2019/rules/no-legacy-icons/index.js

@@ -22,6 +22,9 @@ const rule = createLintRule({
         shouldErrorForAutoMigration: {
           type: 'boolean'
         },
+        shouldUseSafeMigrationMode: {
+          type: 'boolean'
+        },
         quiet: {
           type: 'boolean'
         },