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

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

@@ -3,7 +3,7 @@ function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol
 function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
 function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
 import { isNodeOfType } from 'eslint-codemod-utils';
-import { canAutoMigrateNewIconBasedOnSize, canMigrateColor, createAutoMigrationError, createCantFindSuitableReplacementError, createCantMigrateColorError, createCantMigrateFunctionUnknownError, createCantMigrateIdentifierError, createCantMigrateReExportError, createCantMigrateSizeUnknown, createCantMigrateSpreadPropsError, createCantMigrateUnsafeProp, createGuidance, createHelpers, findUNSAFEProp, getIconKey, getMigrationMapObject, locToString } from './helpers';
+import { canAutoMigrateNewIconBasedOnSize, canMigrateColor, createAutoMigrationError, createCantFindSuitableReplacementError, createCantMigrateColorError, createCantMigrateFunctionUnknownError, createCantMigrateIdentifierError, createCantMigrateReExportError, createCantMigrateSizeUnknown, createCantMigrateSpreadPropsError, createCantMigrateUnsafeProp, createGuidance, createHelpers, findUNSAFEProp, getMigrationMapObject, locToString } from './helpers';
 import { isSize } from './migration-map-temp';
 export var createChecks = function createChecks(context) {
   //create global variables to be shared by the checks
@@ -134,25 +134,36 @@ export var 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) {
-        createCantMigrateReExportError(node, moduleSource, node.specifiers[0].exported.name, errorsManual);
-        guidance[locToString(node)] = createGuidance(moduleSource);
+        var _iterator4 = _createForOfIteratorHelper(node.specifiers),
+          _step4;
+        try {
+          for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
+            var spec = _step4.value;
+            createCantMigrateReExportError(spec, moduleSource, spec.exported.name, errorsManual);
+            guidance[locToString(spec)] = createGuidance(moduleSource);
+          }
+        } catch (err) {
+          _iterator4.e(err);
+        } finally {
+          _iterator4.f();
+        }
       }
     } else if (node.declaration && 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 (isNodeOfType(decl, 'VariableDeclarator') && 'init' in decl && decl.init && isNodeOfType(decl.init, 'Identifier') && decl.init.name in legacyIconImports) {
             createCantMigrateReExportError(node, legacyIconImports[decl.init.name].packageName, decl.init.name, errorsManual);
-            guidance[locToString(node)] = createGuidance(createGuidance(legacyIconImports[decl.init.name].packageName));
+            guidance[locToString(node)] = 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) {
       /**
@@ -161,25 +172,25 @@ export var 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
             };
-            createCantMigrateReExportError(spec, legacyIconImports[spec.local.name].packageName, spec.exported.name, errorsManual);
-            guidance[locToString(spec)] = createGuidance(legacyIconImports[spec.local.name].packageName);
+            createCantMigrateReExportError(_spec2, legacyIconImports[_spec2.local.name].packageName, _spec2.exported.name, errorsManual);
+            guidance[locToString(_spec2)] = createGuidance(legacyIconImports[_spec2.local.name].packageName);
           }
         }
       } catch (err) {
-        _iterator5.e(err);
+        _iterator6.e(err);
       } finally {
-        _iterator5.f();
+        _iterator6.f();
       }
     }
   };
@@ -226,18 +237,17 @@ export var createChecks = function createChecks(context) {
         UNSAFE_propName = result.UNSAFE_propName;
       }
       if (newIcon && isInNewButton && isNewIconMigratable && UNSAFE_size !== 'large' && UNSAFE_size !== 'xlarge') {
-        var iconKey = getIconKey(legacyIconImports[node.name].packageName);
-        createAutoMigrationError(node, legacyIconImports[node.name].packageName, node.name, newIcon, iconKey, errorsAuto);
-        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, 'medium');
+        createAutoMigrationError(node, legacyIconImports[node.name].packageName, node.name, errorsAuto);
+        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, isInNewButton, 'medium');
       } else if ((!newIcon || !isNewIconMigratable) && !UNSAFE_size) {
         createCantFindSuitableReplacementError(node, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName);
+        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, isInNewButton);
       } else if ((UNSAFE_size === 'large' || UNSAFE_size === 'xlarge') && UNSAFE_propName) {
         createCantMigrateUnsafeProp(node, UNSAFE_propName, UNSAFE_size, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, UNSAFE_size);
+        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, isInNewButton, UNSAFE_size);
       } else if (!isInNewButton) {
         createCantMigrateFunctionUnknownError(node, legacyIconImports[node.name].packageName, node.name, errorsManual);
-        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName);
+        guidance[locToString(node)] = createGuidance(legacyIconImports[node.name].packageName, isInNewButton);
       }
     }
   };
@@ -280,11 +290,11 @@ export var 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 (isNodeOfType(attr, 'JSXSpreadAttribute')) {
             // In case there are more spread props
@@ -317,9 +327,9 @@ export var createChecks = function createChecks(context) {
           }
         }
       } catch (err) {
-        _iterator6.e(err);
+        _iterator7.e(err);
       } finally {
-        _iterator6.f();
+        _iterator7.f();
       }
       var hasManualMigration = false;
       if (primaryColor && !canMigrateColor(primaryColor) || hasPrimaryColorProp && !primaryColor) {
@@ -333,7 +343,6 @@ export var createChecks = function createChecks(context) {
         createCantMigrateSizeUnknown(node, errorsManual, legacyIconImports[name].packageName, name);
         hasManualMigration = true;
       }
-      guidance[locToString(node)] = 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')) {
         createCantMigrateUnsafeProp(node, UNSAFE_propName, UNSAFE_size, legacyIconImports[name].packageName, name, errorsManual);
@@ -355,14 +364,14 @@ export var createChecks = function createChecks(context) {
         hasManualMigration = true;
       }
       var migrationMapObject = getMigrationMapObject(legacyIconImports[name].packageName);
-      var iconKey = getIconKey(legacyIconImports[name].packageName);
       var newIcon = migrationMapObject === null || migrationMapObject === void 0 ? void 0 : migrationMapObject.newIcon;
       var isNewIconMigratable = canAutoMigrateNewIconBasedOnSize(migrationMapObject === null || migrationMapObject === void 0 ? void 0 : migrationMapObject.sizeGuidance[(_size = size) !== null && _size !== void 0 ? _size : 'medium']);
       if (!hasManualMigration && newIcon && isNewIconMigratable) {
-        createAutoMigrationError(node, legacyIconImports[name].packageName, name, newIcon, iconKey, errorsAuto);
+        createAutoMigrationError(node, legacyIconImports[name].packageName, name, errorsAuto);
       } else if ((!newIcon || !isNewIconMigratable) && size) {
-        createCantFindSuitableReplacementError(node, legacyIconImports[name].packageName, name, errorsManual);
+        createCantFindSuitableReplacementError(node, legacyIconImports[name].packageName, name, errorsManual, migrationMapObject ? true : false);
       }
+      guidance[locToString(node)] = createGuidance(legacyIconImports[name].packageName, insideNewButton, size && isSize(size) ? size : undefined);
     }
   };
 
@@ -372,20 +381,20 @@ export var 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 (isNodeOfType(arg, 'Identifier') && arg.name in legacyIconImports && legacyIconImports[arg.name].packageName) {
             createCantMigrateFunctionUnknownError(node, legacyIconImports[arg.name].packageName, arg.name, errorsManual);
             guidance[locToString(node)] = createGuidance(legacyIconImports[arg.name].packageName);
           }
         }
       } catch (err) {
-        _iterator7.e(err);
+        _iterator8.e(err);
       } finally {
-        _iterator7.f();
+        _iterator8.f();
       }
     }
   };
@@ -401,36 +410,28 @@ export var 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();
             }
           }
         }
@@ -449,16 +450,11 @@ export var 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/esm/rules/no-legacy-icons/helpers.js

@@ -20,7 +20,7 @@ export var getMigrationMapObject = function getMigrationMapObject(iconPackage) {
  * @param iconPackage The name of the legacy icon package
  * @returns The unique identifier for the icon (the part after "@atlaskit/icon/glyph")
  */
-export var getIconKey = function getIconKey(iconPackage) {
+var getIconKey = function getIconKey(iconPackage) {
   var key = iconPackage.replace(/^@atlaskit\/icon\/glyph\//, '');
   return key;
 };
@@ -35,39 +35,48 @@ export var canAutoMigrateNewIconBasedOnSize = function canAutoMigrateNewIconBase
 /**
  * Creates the written guidance for migrating a legacy icon to a new icon
  */
-export var createGuidance = function createGuidance(iconPackage, size) {
+export var 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 outcomeDescriptionMap) {
-        guidance += " Please: ".concat(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 outcomeDescriptionMap ? " Please: ".concat(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 = _slicedToArray(_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 outcomeDescriptionMap)) {
-          _guidance += 'No migration advice given.\n';
+          guidance += 'No migration advice given.\n';
         } else {
-          _guidance += "".concat(outcomeDescriptionMap[value], "\n");
+          guidance += "".concat(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");
   }
 };
 
@@ -145,13 +154,14 @@ export var createCantMigrateUnsafeProp = function createCantMigrateUnsafeProp(no
   };
   pushManualError(locToString(node), errors, myError, packageName, iconName);
 };
-export var createCantFindSuitableReplacementError = function createCantFindSuitableReplacementError(node, importSource, iconName, errors) {
+export var 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);
@@ -194,18 +204,13 @@ export var createCantMigrateSizeUnknown = function createCantMigrateSizeUnknown(
   };
   pushManualError(locToString(node), errors, myError, importSource, iconName);
 };
-export var 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);
+export var 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/esm/rules/no-legacy-icons/index.js

@@ -27,17 +27,16 @@ var rule = 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) {
@@ -54,95 +53,23 @@ var rule = createLintRule({
       checkJSXElement = _createChecks.checkJSXElement,
       checkCallExpression = _createChecks.checkCallExpression,
       throwErrors = _createChecks.throwErrors;
-    return {
+    return errorBoundary({
       // Track imports of relevant components
-      ImportDeclaration: function ImportDeclaration(node) {
-        errorBoundary(function () {
-          return checkImportDeclarations(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      ImportDeclaration: checkImportDeclarations,
       // Keep track of the relevant variable declarations and renames
-      VariableDeclaration: function VariableDeclaration(node) {
-        errorBoundary(function () {
-          return checkVariableDeclarations(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
+      VariableDeclaration: checkVariableDeclarations,
       // Case: default re-exports. Can't be auto-migrated
-      ExportDefaultDeclaration: function ExportDefaultDeclaration(node) {
-        errorBoundary(function () {
-          return checkExportDefaultDeclaration(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      ExportNamedDeclaration: function ExportNamedDeclaration(node) {
-        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) {
-        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) {
-        errorBoundary(function () {
-          return checkIconAsProp(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      JSXElement: function JSXElement(node) {
-        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) {
-        errorBoundary(function () {
-          return checkCallExpression(node);
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      },
-      'Program:exit': function ProgramExit() {
-        errorBoundary(function () {
-          return throwErrors();
-        }, {
-          config: {
-            failSilently: failSilently
-          }
-        });
-      }
-    };
+      CallExpression: checkCallExpression,
+      'Program:exit': throwErrors
+    }, failSilently);
   }
 });
 export default rule;

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

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

package/dist/esm/rules/utils/error-boundary.js

@@ -1,3 +1,11 @@
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
+import _typeof from "@babel/runtime/helpers/typeof";
+// eslint-disable-next-line import/no-extraneous-dependencies
+
+// Need to intersect type RuleListener with a generic function to allow use of Parameters<...> to be used
+
+// Allow config to be to be easily passed from rules
+
 /**
  * ESLint rules should NEVER throw exceptions, because that breaks the VSCode ESLint server
  * (and probably the IntelliJ one too), which causes linting to fail in a file.
@@ -5,14 +13,52 @@
  * It also breaks CI, which was the reason this error boundary was added. It's a final
  * catch all.
  */
-export var errorBoundary = function errorBoundary(func, _ref) {
-  var config = _ref.config;
-  try {
-    func();
-  } catch (err) {
-    if (!config.failSilently) {
-      // eslint-disable-next-line no-console
-      console.warn(err);
-    }
+export function errorBoundary(ruleOrRules) {
+  var config = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
+  var failSilently = failSilentlyFromConfig(config);
+  if (isSingleRuleListener(ruleOrRules)) {
+    return wrapSingleRuleListener(ruleOrRules, failSilently);
+  }
+  return wrapRuleListener(ruleOrRules, failSilently);
+}
+function isSingleRuleListener(rule) {
+  return typeof rule === 'function';
+}
+function failSilentlyFromConfig(c) {
+  switch (_typeof(c)) {
+    case 'undefined':
+      return false;
+    case 'boolean':
+      return c;
+    case 'object':
+      if ('failSilently' in c) {
+        var _c$failSilently;
+        return (_c$failSilently = c.failSilently) !== null && _c$failSilently !== void 0 ? _c$failSilently : false;
+      } else if ('config' in c) {
+        var _c$config$failSilentl;
+        return (_c$config$failSilentl = c.config.failSilently) !== null && _c$config$failSilentl !== void 0 ? _c$config$failSilentl : false;
+      }
+      return false;
+    default:
+      throw new Error('Invalid config');
   }
-};
+}
+function wrapSingleRuleListener(rule, failSilently) {
+  return function () {
+    try {
+      rule.apply(void 0, arguments);
+    } catch (err) {
+      if (!failSilently) {
+        // eslint-disable-next-line no-console
+        console.warn(err);
+      }
+    }
+  };
+}
+function wrapRuleListener(ruleListener, failSilently) {
+  return Object.entries(ruleListener).reduce(function (wrappedRuleListener, e) {
+    var ruleName = e[0];
+    var rule = e[1];
+    return Object.assign(wrappedRuleListener, _defineProperty({}, ruleName, wrapSingleRuleListener(rule, failSilently)));
+  }, {});
+}

package/dist/types/index.codegen.d.ts

@@ -9,6 +9,7 @@ export declare const configs: {
             '@atlaskit/design-system/icon-label': string;
             '@atlaskit/design-system/no-banned-imports': string;
             '@atlaskit/design-system/no-css-tagged-template-expression': string;
+            '@atlaskit/design-system/no-custom-icons': string;
             '@atlaskit/design-system/no-deprecated-apis': string;
             '@atlaskit/design-system/no-deprecated-design-token-usage': string;
             '@atlaskit/design-system/no-deprecated-imports': string;

package/dist/types/presets/all.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::9bb5329713677543f16019954484ae2f>>
+ * @codegen <<SignedSource::ab43b6e1a867d07b9a27eae78d48834a>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 declare const _default: {
@@ -12,6 +12,7 @@ declare const _default: {
         '@atlaskit/design-system/icon-label': string;
         '@atlaskit/design-system/no-banned-imports': string;
         '@atlaskit/design-system/no-css-tagged-template-expression': string;
+        '@atlaskit/design-system/no-custom-icons': string;
         '@atlaskit/design-system/no-deprecated-apis': string;
         '@atlaskit/design-system/no-deprecated-design-token-usage': string;
         '@atlaskit/design-system/no-deprecated-imports': string;

package/dist/types/rules/index.codegen.d.ts

@@ -5,6 +5,7 @@ declare const _default: {
     'icon-label': import("eslint").Rule.RuleModule;
     'no-banned-imports': import("eslint").Rule.RuleModule;
     'no-css-tagged-template-expression': import("eslint").Rule.RuleModule;
+    'no-custom-icons': import("eslint").Rule.RuleModule;
     'no-deprecated-apis': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<string, [{
         deprecatedConfig: import("./utils/types").DeprecatedConfig;
     }], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;

package/dist/types/rules/no-custom-icons/checks/has-prop.d.ts

@@ -0,0 +1,2 @@
+import { type JSXElement } from 'eslint-codemod-utils';
+export declare function hasProp(node: JSXElement, propName: string): boolean;

package/dist/types/rules/no-custom-icons/checks/is-from-import-source.d.ts

@@ -0,0 +1,8 @@
+import type { Rule } from 'eslint';
+import type { ImportDeclaration } from 'eslint-codemod-utils';
+import { type ImportedJSXElement } from './is-imported-jsx-element';
+export declare function createIsFromImportSourceFor(...importSources: (string | RegExp)[]): {
+    (node: Rule.Node): node is ImportedJSXElement;
+    importDeclarationHook(node: ImportDeclaration): void;
+    getImportSource(node: Rule.Node): string;
+};