@atlaskit/eslint-plugin-design-system 10.9.0 → 10.10.1

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

@@ -0,0 +1,67 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = void 0;
+var _createRule = require("../utils/create-rule");
+var _errorBoundary = require("../utils/error-boundary");
+var _hasProp = require("./checks/has-prop");
+var _isFromImportSource = require("./checks/is-from-import-source");
+// eslint-disable-next-line import/no-extraneous-dependencies
+
+var rule = (0, _createRule.createLintRule)({
+  meta: {
+    name: 'no-custom-icons',
+    type: 'problem',
+    docs: {
+      description: 'Enforces custom glyph icons are used.',
+      recommended: false,
+      severity: 'warn'
+    },
+    schema: [{
+      type: 'object',
+      properties: {
+        centralLocation: {
+          type: 'string'
+        },
+        failSilently: {
+          type: 'boolean'
+        }
+      },
+      additionalProperties: false
+    }],
+    messages: {
+      noCustomIcons: "Custom icons from {{importSource}} are no longer supported. Migrate to an icon from '@atlaskit/(icon-labs|icon/core|icon/utility)'{{locationMessage}}.\n[Migration guide](https://hello.atlassian.net/wiki/spaces/DST/pages/3748692796/New+ADS+iconography+-+Code+migration+guide)."
+    }
+  },
+  create: function create(context) {
+    var _context$options$;
+    var isIconBase = (0, _isFromImportSource.createIsFromImportSourceFor)('@atlaskit/icon', '@atlaskit/icon/base');
+    var _ref = (_context$options$ = context.options[0]) !== null && _context$options$ !== void 0 ? _context$options$ : {},
+      _ref$centralLocation = _ref.centralLocation,
+      centralLocation = _ref$centralLocation === void 0 ? '' : _ref$centralLocation,
+      _ref$failSilently = _ref.failSilently,
+      failSilently = _ref$failSilently === void 0 ? false : _ref$failSilently;
+    var locationMessage = centralLocation ? " or move the icon to '".concat(centralLocation, "'") : '';
+    return (0, _errorBoundary.errorBoundary)({
+      JSXElement: function JSXElement(node) {
+        var _isIconBase$getImport;
+        if (!isIconBase(node) || !(0, _hasProp.hasProp)(node, 'glyph')) {
+          return;
+        }
+        var importSource = (_isIconBase$getImport = isIconBase.getImportSource(node)) !== null && _isIconBase$getImport !== void 0 ? _isIconBase$getImport : '';
+        context.report({
+          node: node.openingElement,
+          messageId: 'noCustomIcons',
+          data: {
+            importSource: importSource,
+            locationMessage: locationMessage
+          }
+        });
+      },
+      ImportDeclaration: isIconBase.importDeclarationHook
+    }, failSilently);
+  }
+});
+var _default = exports.default = rule;

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

@@ -141,25 +141,36 @@ var createChecks = exports.createChecks = function createChecks(context) {
       if (typeof moduleSource === 'string' && ['@atlaskit/icon/glyph/', '@atlaskit/icon-object/glyph/'].find(function (val) {
         return moduleSource.startsWith(val);
       }) && node.specifiers.length) {
-        (0, _helpers.createCantMigrateReExportError)(node, moduleSource, node.specifiers[0].exported.name, errorsManual);
-        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(moduleSource);
+        var _iterator4 = _createForOfIteratorHelper(node.specifiers),
+          _step4;
+        try {
+          for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
+            var spec = _step4.value;
+            (0, _helpers.createCantMigrateReExportError)(spec, moduleSource, spec.exported.name, errorsManual);
+            guidance[(0, _helpers.locToString)(spec)] = (0, _helpers.createGuidance)(moduleSource);
+          }
+        } catch (err) {
+          _iterator4.e(err);
+        } finally {
+          _iterator4.f();
+        }
       }
     } else if (node.declaration && (0, _eslintCodemodUtils.isNodeOfType)(node.declaration, 'VariableDeclaration')) {
       // export const Icon = AddIcon;
-      var _iterator4 = _createForOfIteratorHelper(node.declaration.declarations),
-        _step4;
+      var _iterator5 = _createForOfIteratorHelper(node.declaration.declarations),
+        _step5;
       try {
-        for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
-          var decl = _step4.value;
+        for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
+          var decl = _step5.value;
           if ((0, _eslintCodemodUtils.isNodeOfType)(decl, 'VariableDeclarator') && 'init' in decl && decl.init && (0, _eslintCodemodUtils.isNodeOfType)(decl.init, 'Identifier') && decl.init.name in legacyIconImports) {
             (0, _helpers.createCantMigrateReExportError)(node, legacyIconImports[decl.init.name].packageName, decl.init.name, errorsManual);
-            guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)((0, _helpers.createGuidance)(legacyIconImports[decl.init.name].packageName));
+            guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[decl.init.name].packageName);
           }
         }
       } catch (err) {
-        _iterator4.e(err);
+        _iterator5.e(err);
       } finally {
-        _iterator4.f();
+        _iterator5.f();
       }
     } else if (!node.source && node.specifiers && node.specifiers.length > 0) {
       /**
@@ -168,25 +179,25 @@ var createChecks = exports.createChecks = function createChecks(context) {
        * const crossIcon = LegacyIcon2;
        * export { AddIcon, CrossIcon as default }
        */
-      var _iterator5 = _createForOfIteratorHelper(node.specifiers),
-        _step5;
+      var _iterator6 = _createForOfIteratorHelper(node.specifiers),
+        _step6;
       try {
-        for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
-          var spec = _step5.value;
-          if (spec.local.name in legacyIconImports) {
+        for (_iterator6.s(); !(_step6 = _iterator6.n()).done;) {
+          var _spec2 = _step6.value;
+          if (_spec2.local.name in legacyIconImports) {
             //update legacy imports to be exported
-            legacyIconImports[spec.local.name] = {
-              packageName: legacyIconImports[spec.local.name].packageName,
+            legacyIconImports[_spec2.local.name] = {
+              packageName: legacyIconImports[_spec2.local.name].packageName,
               exported: true
             };
-            (0, _helpers.createCantMigrateReExportError)(spec, legacyIconImports[spec.local.name].packageName, spec.exported.name, errorsManual);
-            guidance[(0, _helpers.locToString)(spec)] = (0, _helpers.createGuidance)(legacyIconImports[spec.local.name].packageName);
+            (0, _helpers.createCantMigrateReExportError)(_spec2, legacyIconImports[_spec2.local.name].packageName, _spec2.exported.name, errorsManual);
+            guidance[(0, _helpers.locToString)(_spec2)] = (0, _helpers.createGuidance)(legacyIconImports[_spec2.local.name].packageName);
           }
         }
       } catch (err) {
-        _iterator5.e(err);
+        _iterator6.e(err);
       } finally {
-        _iterator5.f();
+        _iterator6.f();
       }
     }
   };
@@ -233,18 +244,17 @@ var createChecks = exports.createChecks = function createChecks(context) {
         UNSAFE_propName = result.UNSAFE_propName;
       }
       if (newIcon && isInNewButton && isNewIconMigratable && UNSAFE_size !== 'large' && UNSAFE_size !== 'xlarge') {
-        var iconKey = (0, _helpers.getIconKey)(legacyIconImports[node.name].packageName);
-        (0, _helpers.createAutoMigrationError)(node, legacyIconImports[node.name].packageName, node.name, newIcon, iconKey, errorsAuto);
-        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, 'medium');
+        (0, _helpers.createAutoMigrationError)(node, legacyIconImports[node.name].packageName, node.name, errorsAuto);
+        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, isInNewButton, 'medium');
       } else if ((!newIcon || !isNewIconMigratable) && !UNSAFE_size) {
         (0, _helpers.createCantFindSuitableReplacementError)(node, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName);
+        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, isInNewButton);
       } else if ((UNSAFE_size === 'large' || UNSAFE_size === 'xlarge') && UNSAFE_propName) {
         (0, _helpers.createCantMigrateUnsafeProp)(node, UNSAFE_propName, UNSAFE_size, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, UNSAFE_size);
+        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, isInNewButton, UNSAFE_size);
       } else if (!isInNewButton) {
         (0, _helpers.createCantMigrateFunctionUnknownError)(node, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName);
+        guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[node.name].packageName, isInNewButton);
       }
     }
   };
@@ -287,11 +297,11 @@ var createChecks = exports.createChecks = function createChecks(context) {
       var requiredAttributesAfterSpread = new Set(['size', 'primaryColor', 'secondaryColor']);
       var hasSpread = false;
       var hasPrimaryColorProp = false;
-      var _iterator6 = _createForOfIteratorHelper(node.openingElement.attributes),
-        _step6;
+      var _iterator7 = _createForOfIteratorHelper(node.openingElement.attributes),
+        _step7;
       try {
-        for (_iterator6.s(); !(_step6 = _iterator6.n()).done;) {
-          var attr = _step6.value;
+        for (_iterator7.s(); !(_step7 = _iterator7.n()).done;) {
+          var attr = _step7.value;
           // Detect spread props
           if ((0, _eslintCodemodUtils.isNodeOfType)(attr, 'JSXSpreadAttribute')) {
             // In case there are more spread props
@@ -324,9 +334,9 @@ var createChecks = exports.createChecks = function createChecks(context) {
           }
         }
       } catch (err) {
-        _iterator6.e(err);
+        _iterator7.e(err);
       } finally {
-        _iterator6.f();
+        _iterator7.f();
       }
       var hasManualMigration = false;
       if (primaryColor && !(0, _helpers.canMigrateColor)(primaryColor) || hasPrimaryColorProp && !primaryColor) {
@@ -340,7 +350,6 @@ var createChecks = exports.createChecks = function createChecks(context) {
         (0, _helpers.createCantMigrateSizeUnknown)(node, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
-      guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[name].packageName, size ? size : undefined);
       // Check for unsafe size
       if ((UNSAFE_size === 'large' || UNSAFE_size === 'xlarge') && (UNSAFE_propName === 'UNSAFE_iconAfter_size' || UNSAFE_propName === 'UNSAFE_iconBefore_size' || UNSAFE_propName === 'UNSAFE_size')) {
         (0, _helpers.createCantMigrateUnsafeProp)(node, UNSAFE_propName, UNSAFE_size, legacyIconImports[name].packageName, name, errorsManual);
@@ -362,14 +371,14 @@ var createChecks = exports.createChecks = function createChecks(context) {
         hasManualMigration = true;
       }
       var migrationMapObject = (0, _helpers.getMigrationMapObject)(legacyIconImports[name].packageName);
-      var iconKey = (0, _helpers.getIconKey)(legacyIconImports[name].packageName);
       var newIcon = migrationMapObject === null || migrationMapObject === void 0 ? void 0 : migrationMapObject.newIcon;
       var isNewIconMigratable = (0, _helpers.canAutoMigrateNewIconBasedOnSize)(migrationMapObject === null || migrationMapObject === void 0 ? void 0 : migrationMapObject.sizeGuidance[(_size = size) !== null && _size !== void 0 ? _size : 'medium']);
       if (!hasManualMigration && newIcon && isNewIconMigratable) {
-        (0, _helpers.createAutoMigrationError)(node, legacyIconImports[name].packageName, name, newIcon, iconKey, errorsAuto);
+        (0, _helpers.createAutoMigrationError)(node, legacyIconImports[name].packageName, name, errorsAuto);
       } else if ((!newIcon || !isNewIconMigratable) && size) {
-        (0, _helpers.createCantFindSuitableReplacementError)(node, legacyIconImports[name].packageName, name, errorsManual);
+        (0, _helpers.createCantFindSuitableReplacementError)(node, legacyIconImports[name].packageName, name, errorsManual, migrationMapObject ? true : false);
       }
+      guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[name].packageName, insideNewButton, size && (0, _migrationMapTemp.isSize)(size) ? size : undefined);
     }
   };
 
@@ -379,20 +388,20 @@ var createChecks = exports.createChecks = function createChecks(context) {
    */
   var checkCallExpression = function checkCallExpression(node) {
     if ('arguments' in node && node.arguments.length) {
-      var _iterator7 = _createForOfIteratorHelper(node.arguments),
-        _step7;
+      var _iterator8 = _createForOfIteratorHelper(node.arguments),
+        _step8;
       try {
-        for (_iterator7.s(); !(_step7 = _iterator7.n()).done;) {
-          var arg = _step7.value;
+        for (_iterator8.s(); !(_step8 = _iterator8.n()).done;) {
+          var arg = _step8.value;
           if ((0, _eslintCodemodUtils.isNodeOfType)(arg, 'Identifier') && arg.name in legacyIconImports && legacyIconImports[arg.name].packageName) {
             (0, _helpers.createCantMigrateFunctionUnknownError)(node, legacyIconImports[arg.name].packageName, arg.name, errorsManual);
             guidance[(0, _helpers.locToString)(node)] = (0, _helpers.createGuidance)(legacyIconImports[arg.name].packageName);
           }
         }
       } catch (err) {
-        _iterator7.e(err);
+        _iterator8.e(err);
       } finally {
-        _iterator7.f();
+        _iterator8.f();
       }
     }
   };
@@ -408,36 +417,28 @@ var createChecks = exports.createChecks = function createChecks(context) {
           errorList = _Object$entries$_i[1];
         var node = 'node' in errorList.errors[0] ? errorList.errors[0].node : null;
         if (node) {
+          var guidanceMessage = _key in guidance ? guidance[_key] : '';
           context.report({
             node: node,
             messageId: 'noLegacyIconsManualMigration',
             data: {
               iconName: errorList.iconName,
               importSource: errorList.importSource,
-              quietModeGuidance: isQuietMode ? 'For more information see the below errors:' : ''
+              guidance: isQuietMode ? guidanceMessage : "".concat(guidanceMessage, "For more information see the below errors.\n")
             }
           });
           if (!isQuietMode) {
-            var _iterator8 = _createForOfIteratorHelper(errorList.errors),
-              _step8;
+            var _iterator9 = _createForOfIteratorHelper(errorList.errors),
+              _step9;
             try {
-              for (_iterator8.s(); !(_step8 = _iterator8.n()).done;) {
-                var error = _step8.value;
+              for (_iterator9.s(); !(_step9 = _iterator9.n()).done;) {
+                var error = _step9.value;
                 context.report(error);
               }
             } catch (err) {
-              _iterator8.e(err);
+              _iterator9.e(err);
             } finally {
-              _iterator8.f();
-            }
-            if (_key in guidance) {
-              context.report({
-                node: node,
-                messageId: 'guidance',
-                data: {
-                  guidance: guidance[_key]
-                }
-              });
+              _iterator9.f();
             }
           }
         }
@@ -456,16 +457,11 @@ var createChecks = exports.createChecks = function createChecks(context) {
         }
         var _node = 'node' in _error ? _error.node : null;
         if (_node) {
-          context.report(_error);
-          if (_key2 in guidance && !isQuietMode) {
-            context.report({
-              node: _node,
-              messageId: 'guidance',
-              data: {
-                guidance: guidance[_key2]
-              }
-            });
+          var _guidanceMessage = _key2 in guidance ? guidance[_key2] : '';
+          if ('data' in _error && _error.data) {
+            _error.data.guidance = _guidanceMessage;
           }
+          context.report(_error);
         }
       }
     }

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

@@ -5,7 +5,7 @@ var _typeof = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.locToString = exports.getMigrationMapObject = exports.getLiteralStringValue = exports.getIconKey = exports.findUNSAFEProp = exports.createHelpers = exports.createGuidance = exports.createCantMigrateUnsafeProp = exports.createCantMigrateSpreadPropsError = exports.createCantMigrateSizeUnknown = exports.createCantMigrateReExportError = exports.createCantMigrateIdentifierError = exports.createCantMigrateFunctionUnknownError = exports.createCantMigrateColorError = exports.createCantFindSuitableReplacementError = exports.createAutoMigrationError = exports.canMigrateColor = exports.canAutoMigrateNewIconBasedOnSize = void 0;
+exports.locToString = exports.getMigrationMapObject = exports.getLiteralStringValue = exports.findUNSAFEProp = exports.createHelpers = exports.createGuidance = exports.createCantMigrateUnsafeProp = exports.createCantMigrateSpreadPropsError = exports.createCantMigrateSizeUnknown = exports.createCantMigrateReExportError = exports.createCantMigrateIdentifierError = exports.createCantMigrateFunctionUnknownError = exports.createCantMigrateColorError = exports.createCantFindSuitableReplacementError = exports.createAutoMigrationError = exports.canMigrateColor = exports.canAutoMigrateNewIconBasedOnSize = void 0;
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _eslintCodemodUtils = require("eslint-codemod-utils");
 var _getImportName = require("../utils/get-import-name");
@@ -30,7 +30,7 @@ var getMigrationMapObject = exports.getMigrationMapObject = function getMigratio
  * @param iconPackage The name of the legacy icon package
  * @returns The unique identifier for the icon (the part after "@atlaskit/icon/glyph")
  */
-var getIconKey = exports.getIconKey = function getIconKey(iconPackage) {
+var getIconKey = function getIconKey(iconPackage) {
   var key = iconPackage.replace(/^@atlaskit\/icon\/glyph\//, '');
   return key;
 };
@@ -45,39 +45,48 @@ var canAutoMigrateNewIconBasedOnSize = exports.canAutoMigrateNewIconBasedOnSize
 /**
  * Creates the written guidance for migrating a legacy icon to a new icon
  */
-var createGuidance = exports.createGuidance = function createGuidance(iconPackage, size) {
+var createGuidance = exports.createGuidance = function createGuidance(iconPackage) {
+  var insideNewButton = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
+  var size = arguments.length > 2 ? arguments[2] : undefined;
   var migrationMapObject = getMigrationMapObject(iconPackage);
   if (migrationMapObject) {
     var newIcon = migrationMapObject.newIcon;
     if (!newIcon) {
       return 'No equivalent icon in new set. An option is to contribute a custom icon into icon-labs package instead.\n';
     }
+    var buttonGuidanceStr = "Please set 'spacing' property of the new icon to 'none', to ensure appropriate spacing inside `@atlaskit/button`.\n";
+    var guidance = '';
     if (size) {
-      var guidance = "Fix: Use ".concat(newIcon.name, " from @atlaskit/").concat(newIcon.library, "/").concat(newIcon.type, "/").concat(newIcon.name, " instead.");
-      if (migrationMapObject.sizeGuidance[size] in _migrationMapTemp.outcomeDescriptionMap) {
-        guidance += " Please: ".concat(_migrationMapTemp.outcomeDescriptionMap[migrationMapObject.sizeGuidance[size]], "\n");
+      if (migrationMapObject.sizeGuidance[size] && canAutoMigrateNewIconBasedOnSize(migrationMapObject.sizeGuidance[size])) {
+        guidance += "Fix: Use ".concat(newIcon.name, " from @atlaskit/").concat(newIcon.library, "/").concat(newIcon.type, "/").concat(newIcon.name, " instead.");
       } else {
-        guidance += ' No migration advice given.\n';
+        guidance += "No equivalent icon for this size, ".concat(size, ", in new set.");
       }
-      return guidance;
+      guidance += "".concat(migrationMapObject.sizeGuidance[size] in _migrationMapTemp.outcomeDescriptionMap ? " Please: ".concat(_migrationMapTemp.outcomeDescriptionMap[migrationMapObject.sizeGuidance[size]]) : ' No migration size advice given.', "\n");
     } else {
-      var _guidance = "Use ".concat(newIcon.name, " from @atlaskit/").concat(newIcon.library, "/").concat(newIcon.type, "/").concat(newIcon.name, " instead.\nMigration suggestions, depending on the legacy icon size:\n");
+      guidance = "Use ".concat(newIcon.name, " from @atlaskit/").concat(newIcon.library, "/").concat(newIcon.type, "/").concat(newIcon.name, " instead.\nMigration suggestions, depending on the legacy icon size:\n");
       for (var _i = 0, _Object$entries = Object.entries(migrationMapObject.sizeGuidance); _i < _Object$entries.length; _i++) {
         var _Object$entries$_i = (0, _slicedToArray2.default)(_Object$entries[_i], 2),
           _size = _Object$entries$_i[0],
           value = _Object$entries$_i[1];
-        _guidance += "\t- ".concat(_size, ": ");
+        guidance += "\t- ".concat(_size, ": ");
         if (!(value in _migrationMapTemp.outcomeDescriptionMap)) {
-          _guidance += 'No migration advice given.\n';
+          guidance += 'No migration advice given.\n';
         } else {
-          _guidance += "".concat(_migrationMapTemp.outcomeDescriptionMap[value], "\n");
+          guidance += "".concat(_migrationMapTemp.outcomeDescriptionMap[value], ".\n");
         }
-        _guidance;
       }
-      return _guidance;
     }
+    if (insideNewButton) {
+      guidance += buttonGuidanceStr;
+    } else if (size && size === 'medium') {
+      guidance += "Setting the spacing property to 'spacious' will maintain the icon's box dimensions - but consider setting spacing='none' as it allows for easier control of spacing by parent elements.\n";
+    } else if (size) {
+      guidance += "In the new icon, please use spacing='none'.\n";
+    }
+    return guidance;
   } else {
-    return 'Migration suggestions not found\n';
+    return "Migration suggestions not found for \"".concat(iconPackage, "\".\n");
   }
 };
 
@@ -155,13 +164,14 @@ var createCantMigrateUnsafeProp = exports.createCantMigrateUnsafeProp = function
   };
   pushManualError(locToString(node), errors, myError, packageName, iconName);
 };
-var createCantFindSuitableReplacementError = exports.createCantFindSuitableReplacementError = function createCantFindSuitableReplacementError(node, importSource, iconName, errors) {
+var createCantFindSuitableReplacementError = exports.createCantFindSuitableReplacementError = function createCantFindSuitableReplacementError(node, importSource, iconName, errors, sizeIssue) {
   var myError = {
     node: node,
     messageId: 'cantFindSuitableReplacement',
     data: {
       importSource: importSource,
-      iconName: iconName
+      iconName: iconName,
+      sizeGuidance: sizeIssue ? ' at the current size' : ''
     }
   };
   pushManualError(locToString(node), errors, myError, importSource, iconName);
@@ -204,18 +214,13 @@ var createCantMigrateSizeUnknown = exports.createCantMigrateSizeUnknown = functi
   };
   pushManualError(locToString(node), errors, myError, importSource, iconName);
 };
-var createAutoMigrationError = exports.createAutoMigrationError = function createAutoMigrationError(node, importSource, iconName, newIcon, oldIconName, errors) {
-  var migrationKey = newIcon.name === oldIconName ? newIcon.name : "".concat(newIcon.name, "--").concat(oldIconName);
-  var newMigrationIconPackage = "@atlaskit/".concat(newIcon.library, "/").concat(newIcon.type, "/migration/").concat(migrationKey);
+var createAutoMigrationError = exports.createAutoMigrationError = function createAutoMigrationError(node, importSource, iconName, errors) {
   var myError = {
     node: node,
     messageId: 'noLegacyIconsAutoMigration',
     data: {
       importSource: importSource,
-      iconName: iconName,
-      newIcon: newIcon.name,
-      //TODO: provide size guidance
-      newPackage: newMigrationIconPackage
+      iconName: iconName
     }
   };
   errors[locToString(node)] = myError;

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

@@ -33,17 +33,16 @@ var rule = (0, _createRule.createLintRule)({
       additionalProperties: false
     }],
     messages: {
-      noLegacyIconsAutoMigration: "Auto Migration:\nLegacy icon '{{iconName}}' from '{{importSource}}' detected.\nAutomatic migration is possible to '{{newIcon}}' from '{{newPackage}}.\nAtlassians: See https://go.atlassian.com/icon-migration-guide for details.'",
-      noLegacyIconsManualMigration: "Manual Migration:\nLegacy icon '{{iconName}}' from '{{importSource}}' detected.\nAutomatic migration not possible.{{quietModeGuidance}}.\nAtlassians: See https://go.atlassian.com/icon-migration-guide for details.",
-      cantFindSuitableReplacement: "No suitable replacement found for '{{iconName}}' from '{{importSource}}' at the size listed. Please manually migrate this icon.",
+      noLegacyIconsAutoMigration: "Auto Migration:\nLegacy icon '{{iconName}}' from '{{importSource}}' detected.\nAutomatic migration is possible.\n\n{{guidance}}\nAtlassians: See https://go.atlassian.com/icon-migration-guide for details.'",
+      noLegacyIconsManualMigration: "Manual Migration:\nLegacy icon '{{iconName}}' from '{{importSource}}' detected.\nAutomatic migration not possible.\n\n{{guidance}}\nAtlassians: See https://go.atlassian.com/icon-migration-guide for details.",
+      cantFindSuitableReplacement: "No suitable replacement found for '{{iconName}}' from '{{importSource}}'{{sizeGuidance}}. Please manually migrate this icon.",
       cantMigrateReExport: "'{{exportName}}' is a re-export of icon from '{{packageName}}' and cannot be automatically migrated. Please utilize the export from icons directly.",
       cantMigrateColor: "Primary Color prop has {{colorValue}} which cannot be automatically migrated to color prop in the new Icon API. Please use either an appropriate color.icon, color.link token, or currentColor (as a last resort).",
       cantMigrateSpreadProps: "This usage of Icon uses spread props in a way that can't be automatically migrated. Please explicitly define the following props after spread: '{{missingProps}}' ",
       cantMigrateSizeUnknown: "This usage of Icon sets the size via a variable or function that can't be determined.",
       cantMigrateFunctionUnknown: "'{{iconName}}' from legacy '{{importSource}}' is used in unknown function/component. Please manually migrate this icon.",
       cantMigrateIdentifier: "This icon is passed to other components via a map or array, in a way that can't be automatically migrated. Please manually migrate wherever this expression is used and use the icon components directly.",
-      cantMigrateUnsafeProp: "Property '{{propName}}' with value of '{{value}}' is unable to be auto-migrated to the new button. Please manually migrate this icon.",
-      guidance: "Guidance:\n'{{guidance}}'"
+      cantMigrateUnsafeProp: "Property '{{propName}}' with value of '{{value}}' is unable to be auto-migrated to the new button. Please manually migrate this icon."
     }
   },
   create: function create(context) {
@@ -60,95 +59,23 @@ var rule = (0, _createRule.createLintRule)({
       checkJSXElement = _createChecks.checkJSXElement,
       checkCallExpression = _createChecks.checkCallExpression,
       throwErrors = _createChecks.throwErrors;
-    return {
+    return (0, _errorBoundary.errorBoundary)({
       // Track imports of relevant components
-      ImportDeclaration: function ImportDeclaration(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkImportDeclarations(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      ImportDeclaration: checkImportDeclarations,
       // Keep track of the relevant variable declarations and renames
-      VariableDeclaration: function VariableDeclaration(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkVariableDeclarations(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      VariableDeclaration: checkVariableDeclarations,
       // Case: default re-exports. Can't be auto-migrated
-      ExportDefaultDeclaration: function ExportDefaultDeclaration(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkExportDefaultDeclaration(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      ExportNamedDeclaration: function ExportNamedDeclaration(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkExportNamedVariables(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      ExportDefaultDeclaration: checkExportDefaultDeclaration,
+      ExportNamedDeclaration: checkExportNamedVariables,
       // Legacy icons found in arrays/objects
-      'ObjectExpression > Property > Identifier, ArrayExpression > Identifier ': function ObjectExpressionPropertyIdentifierArrayExpressionIdentifier(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkArrayOrMap(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      'ObjectExpression > Property > Identifier, ArrayExpression > Identifier ': checkArrayOrMap,
       // Legacy icons passed in via props, as JSX identifier (i.e. icon={AddIcon})
-      'JSXOpeningElement > JSXAttribute > JSXExpressionContainer > Identifier': function JSXOpeningElementJSXAttributeJSXExpressionContainerIdentifier(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkIconAsProp(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      JSXElement: function JSXElement(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkJSXElement(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      'JSXOpeningElement > JSXAttribute > JSXExpressionContainer > Identifier': checkIconAsProp,
+      JSXElement: checkJSXElement,
       // Icons called as an argument of a function (i.e. icon={DefaultIcon(AddIcon)})
-      CallExpression: function CallExpression(node) {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return checkCallExpression(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      'Program:exit': function ProgramExit() {
-        (0, _errorBoundary.errorBoundary)(function () {
-          return throwErrors();
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      }
-    };
+      CallExpression: checkCallExpression,
+      'Program:exit': throwErrors
+    }, failSilently);
   }
 });
 var _default = exports.default = rule;

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

@@ -12,16 +12,12 @@ var create = function create(context) {
   var config = (0, _config.getConfig)(context.options[0]);
   return {
     // const styles = css({ fontSize: '14px, ... }), styled.div({ fontSize: 14, ... })
-    ObjectExpression: function ObjectExpression(node) {
-      return (0, _errorBoundary.errorBoundary)(function () {
-        return _styleObject.StyleObject.lint(node, {
-          context: context,
-          config: config
-        });
-      }, {
+    ObjectExpression: (0, _errorBoundary.errorBoundary)(function (node) {
+      return _styleObject.StyleObject.lint(node, {
+        context: context,
         config: config
       });
-    }
+    }, config)
   };
 };
 var rule = (0, _createRule.createLintRule)({