@atlaskit/eslint-plugin-design-system 10.8.2 → 10.10.0

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/index.js

@@ -60,95 +60,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)({

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

@@ -1,9 +1,18 @@
 "use strict";
 
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.errorBoundary = void 0;
+exports.errorBoundary = errorBoundary;
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _typeof2 = _interopRequireDefault(require("@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.
@@ -11,14 +20,52 @@ exports.errorBoundary = void 0;
  * It also breaks CI, which was the reason this error boundary was added. It's a final
  * catch all.
  */
-var errorBoundary = exports.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);
-    }
+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 ((0, _typeof2.default)(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, (0, _defineProperty2.default)({}, ruleName, wrapSingleRuleListener(rule, failSilently)));
+  }, {});
+}

package/dist/es2019/presets/all.codegen.js

@@ -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
  */
 export default {
@@ -12,6 +12,7 @@ export default {
     '@atlaskit/design-system/icon-label': 'warn',
     '@atlaskit/design-system/no-banned-imports': 'error',
     '@atlaskit/design-system/no-css-tagged-template-expression': 'error',
+    '@atlaskit/design-system/no-custom-icons': 'warn',
     '@atlaskit/design-system/no-deprecated-apis': 'error',
     '@atlaskit/design-system/no-deprecated-design-token-usage': 'warn',
     '@atlaskit/design-system/no-deprecated-imports': 'error',

package/dist/es2019/rules/consistent-css-prop-usage/index.js

@@ -449,7 +449,8 @@ const defaultConfig = {
   cssImportSource: CSS_IN_JS_IMPORTS.compiled,
   xcssImportSource: CSS_IN_JS_IMPORTS.atlaskitPrimitives,
   excludeReactComponents: false,
-  autoFix: true
+  autoFix: true,
+  shouldAlwaysCheckXcss: false
 };
 const rule = createLintRule({
   meta: {
@@ -492,6 +493,9 @@ const rule = createLintRule({
         excludeReactComponents: {
           type: 'boolean'
         },
+        shouldAlwaysCheckXcss: {
+          type: 'boolean'
+        },
         autoFix: {
           type: 'boolean'
         }
@@ -508,7 +512,20 @@ const rule = createLintRule({
           name,
           value
         } = node;
-        if (mergedConfig.excludeReactComponents && node.parent.type === 'JSXOpeningElement') {
+
+        /**
+         * We skip linting `xcss` attributes if:
+         *
+         * - excludeReactComponents === true
+         * - shouldAlwaysCheckXcss === false
+         *
+         * In the future we may want to remove `shouldAlwaysCheckXcss`
+         * and just always lint `xcss`, regardless of `excludeReactComponents`
+         */
+        if (mergedConfig.excludeReactComponents && name.name === 'xcss' && !mergedConfig.shouldAlwaysCheckXcss) {
+          return;
+        }
+        if (mergedConfig.excludeReactComponents && node.parent.type === 'JSXOpeningElement' && name.name === 'css') {
           // e.g. <item.before />
           if (node.parent.name.type === 'JSXMemberExpression') {
             return;

package/dist/es2019/rules/ensure-design-token-usage/index.js

@@ -27,25 +27,21 @@ const createWithConfig = initialConfig => context => {
     failSilently: (userConfig === null || userConfig === void 0 ? void 0 : userConfig.failSilently) || defaultConfig.failSilently
   };
   let tokenNode = null;
-  return {
-    ImportDeclaration: node => errorBoundary(() => {
+  return errorBoundary({
+    ImportDeclaration: node => {
       if (node.source.value === '@atlaskit/tokens' && config.applyImport) {
         tokenNode = node;
       }
-    }, {
-      config
-    }),
+    },
     // For expressions within template literals (e.g. `color: ${red}`) - color only
-    'TemplateLiteral > Identifier': node => errorBoundary(() => {
+    'TemplateLiteral > Identifier': node => {
       if (config.domains.includes('color')) {
         return lintTemplateIdentifierForColor(node, context, config);
       }
       return;
-    }, {
-      config
-    }),
+    },
     // const styles = css({ color: 'red', margin: '4px' }), styled.div({ color: 'red', margin: '4px' })
-    ObjectExpression: parentNode => errorBoundary(() => {
+    ObjectExpression: parentNode => {
       const {
         references
       } = context.getScope();
@@ -115,13 +111,11 @@ const createWithConfig = initialConfig => context => {
         }
       }
       parentNode.properties.forEach(findObjectStyles);
-    }, {
-      config
-    }),
+    },
     // CSSTemplateLiteral and StyledTemplateLiteral
     // const cssTemplateLiteral = css`color: red; padding: 12px`;
     // const styledTemplateLiteral = styled.p`color: red; padding: 8px`;
-    'TaggedTemplateExpression[tag.name="css"],TaggedTemplateExpression[tag.object.name="styled"],TaggedTemplateExpression[tag.callee.name="styled"]': node => errorBoundary(() => {
+    'TaggedTemplateExpression[tag.name="css"],TaggedTemplateExpression[tag.object.name="styled"],TaggedTemplateExpression[tag.callee.name="styled"]': node => {
       // To force the correct node type
       if (!isNodeOfType(node, 'TaggedTemplateExpression')) {
         return;
@@ -249,37 +243,29 @@ const createWithConfig = initialConfig => context => {
           }
         });
       }
-    }, {
-      config
-    }),
+    },
     // For inline JSX styles - literals (e.g. <Test color="red"/>) - color only
-    'JSXAttribute > Literal': node => errorBoundary(() => {
+    'JSXAttribute > Literal': node => {
       if (config.domains.includes('color')) {
         return lintJSXLiteralForColor(node, context, config);
       }
       return;
-    }, {
-      config
-    }),
+    },
     // For inline JSX styles - members (e.g. <Test color={color.red}/>) - color only
-    'JSXExpressionContainer > MemberExpression': node => errorBoundary(() => {
+    'JSXExpressionContainer > MemberExpression': node => {
       if (config.domains.includes('color')) {
         return lintJSXMemberForColor(node, context, config);
       }
       return;
-    }, {
-      config
-    }),
+    },
     // For inline JSX styles - identifiers (e.g. <Test color={red}/>) - color only
-    'JSXExpressionContainer > Identifier': node => errorBoundary(() => {
+    'JSXExpressionContainer > Identifier': node => {
       if (config.domains.includes('color')) {
         return lintJSXIdentifierForColor(node, context, config);
       }
       return;
-    }, {
-      config
-    })
-  };
+    }
+  }, config);
 };
 const rule = createLintRule({
   meta: ruleMeta,

package/dist/es2019/rules/index.codegen.js

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::b359d356c482db0087ddfce5bd105403>>
+ * @codegen <<SignedSource::402e6eb5433560b1032e5ed926bb5564>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import consistentCssPropUsage from './consistent-css-prop-usage';
@@ -9,6 +9,7 @@ import ensureDesignTokenUsagePreview from './ensure-design-token-usage-preview';
 import iconLabel from './icon-label';
 import noBannedImports from './no-banned-imports';
 import noCssTaggedTemplateExpression from './no-css-tagged-template-expression';
+import noCustomIcons from './no-custom-icons';
 import noDeprecatedApis from './no-deprecated-apis';
 import noDeprecatedDesignTokenUsage from './no-deprecated-design-token-usage';
 import noDeprecatedImports from './no-deprecated-imports';
@@ -50,6 +51,7 @@ export default {
   'icon-label': iconLabel,
   'no-banned-imports': noBannedImports,
   'no-css-tagged-template-expression': noCssTaggedTemplateExpression,
+  'no-custom-icons': noCustomIcons,
   'no-deprecated-apis': noDeprecatedApis,
   'no-deprecated-design-token-usage': noDeprecatedDesignTokenUsage,
   'no-deprecated-imports': noDeprecatedImports,

package/dist/es2019/rules/no-custom-icons/checks/has-prop.js

@@ -0,0 +1,4 @@
+import { isNodeOfType } from 'eslint-codemod-utils';
+export function hasProp(node, propName) {
+  return isNodeOfType(node.openingElement, 'JSXOpeningElement') && node.openingElement.attributes.some(a => a.type === 'JSXAttribute' && a.name.name === propName);
+}

package/dist/es2019/rules/no-custom-icons/checks/is-from-import-source.js

@@ -0,0 +1,23 @@
+import { isImportedJSXElement } from './is-imported-jsx-element';
+export function createIsFromImportSourceFor(...importSources) {
+  const literalImportSources = importSources.filter(s => typeof s === 'string');
+  const matchImportSources = importSources.filter(s => s instanceof RegExp);
+  const varImportSourceMap = new Map();
+  function isFromImportSource(node) {
+    return isImportedJSXElement(node) && varImportSourceMap.has(node.openingElement.name.name);
+  }
+  isFromImportSource.importDeclarationHook = node => {
+    const source = node.source.value;
+    if (typeof source !== 'string' || !(literalImportSources.includes(source) || matchImportSources.some(r => r.test(source)))) {
+      return;
+    }
+    node.specifiers.filter(spec => ['ImportSpecifier', 'ImportDefaultSpecifier'].includes(spec.type)).forEach(spec => varImportSourceMap.set(spec.local.name, source));
+  };
+  isFromImportSource.getImportSource = node => {
+    if (!isFromImportSource(node)) {
+      throw new Error('Node is not an imported JSX element');
+    }
+    return varImportSourceMap.get(node.openingElement.name.name);
+  };
+  return isFromImportSource;
+}

package/dist/es2019/rules/no-custom-icons/checks/is-imported-jsx-element.js

@@ -0,0 +1,4 @@
+import { isNodeOfType } from 'eslint-codemod-utils';
+export function isImportedJSXElement(node) {
+  return isNodeOfType(node, 'JSXElement') && isNodeOfType(node.openingElement.name, 'JSXIdentifier');
+}

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

@@ -0,0 +1,61 @@
+// eslint-disable-next-line import/no-extraneous-dependencies
+
+import { createLintRule } from '../utils/create-rule';
+import { errorBoundary } from '../utils/error-boundary';
+import { hasProp } from './checks/has-prop';
+import { createIsFromImportSourceFor } from './checks/is-from-import-source';
+const rule = 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}}.
+[Migration guide](https://hello.atlassian.net/wiki/spaces/DST/pages/3748692796/New+ADS+iconography+-+Code+migration+guide).`
+    }
+  },
+  create(context) {
+    var _context$options$;
+    const isIconBase = createIsFromImportSourceFor('@atlaskit/icon', '@atlaskit/icon/base');
+    const {
+      centralLocation = '',
+      failSilently = false
+    } = (_context$options$ = context.options[0]) !== null && _context$options$ !== void 0 ? _context$options$ : {};
+    const locationMessage = centralLocation ? ` or move the icon to '${centralLocation}'` : '';
+    return errorBoundary({
+      JSXElement(node) {
+        var _isIconBase$getImport;
+        if (!isIconBase(node) || !hasProp(node, 'glyph')) {
+          return;
+        }
+        const importSource = (_isIconBase$getImport = isIconBase.getImportSource(node)) !== null && _isIconBase$getImport !== void 0 ? _isIconBase$getImport : '';
+        context.report({
+          node: node.openingElement,
+          messageId: 'noCustomIcons',
+          data: {
+            importSource,
+            locationMessage
+          }
+        });
+      },
+      ImportDeclaration: isIconBase.importDeclarationHook
+    }, failSilently);
+  }
+});
+export default rule;

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

@@ -56,77 +56,23 @@ const rule = createLintRule({
       checkCallExpression,
       throwErrors
     } = createChecks(context);
-    return {
+    return errorBoundary({
       // Track imports of relevant components
-      ImportDeclaration(node) {
-        errorBoundary(() => checkImportDeclarations(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
+      ImportDeclaration: checkImportDeclarations,
       // Keep track of the relevant variable declarations and renames
-      VariableDeclaration(node) {
-        errorBoundary(() => checkVariableDeclarations(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
+      VariableDeclaration: checkVariableDeclarations,
       // Case: default re-exports. Can't be auto-migrated
-      ExportDefaultDeclaration(node) {
-        errorBoundary(() => checkExportDefaultDeclaration(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
-      ExportNamedDeclaration(node) {
-        errorBoundary(() => checkExportNamedVariables(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
+      ExportDefaultDeclaration: checkExportDefaultDeclaration,
+      ExportNamedDeclaration: checkExportNamedVariables,
       // Legacy icons found in arrays/objects
-      'ObjectExpression > Property > Identifier, ArrayExpression > Identifier ': node => {
-        errorBoundary(() => checkArrayOrMap(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
+      'ObjectExpression > Property > Identifier, ArrayExpression > Identifier ': checkArrayOrMap,
       // Legacy icons passed in via props, as JSX identifier (i.e. icon={AddIcon})
-      'JSXOpeningElement > JSXAttribute > JSXExpressionContainer > Identifier': node => {
-        errorBoundary(() => checkIconAsProp(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
-      JSXElement(node) {
-        errorBoundary(() => checkJSXElement(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
+      'JSXOpeningElement > JSXAttribute > JSXExpressionContainer > Identifier': checkIconAsProp,
+      JSXElement: checkJSXElement,
       // Icons called as an argument of a function (i.e. icon={DefaultIcon(AddIcon)})
-      CallExpression: node => {
-        errorBoundary(() => checkCallExpression(node), {
-          config: {
-            failSilently
-          }
-        });
-      },
-      'Program:exit': () => {
-        errorBoundary(() => throwErrors(), {
-          config: {
-            failSilently
-          }
-        });
-      }
-    };
+      CallExpression: checkCallExpression,
+      'Program:exit': throwErrors
+    }, failSilently);
   }
 });
 export default rule;

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

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