@atlaskit/eslint-plugin-design-system 10.12.5 → 10.14.0

package/dist/cjs/rules/use-primitives-text/transformers/span-elements.js

@@ -24,58 +24,82 @@ var SpanElements = exports.SpanElements = {
     }
 
     // Check whether all criteria needed to make a transformation are met
-    if (!SpanElements._check(node, {
-      context: context,
-      config: config
-    })) {
-      return;
-    }
-    var fix = SpanElements._fix(node, {
-      context: context,
-      config: config
-    });
-    context.report(_objectSpread({
-      node: node.openingElement,
-      messageId: 'preferPrimitivesText'
-    }, config.enableUnsafeAutofix ? {
-      fix: fix
-    } : {
-      suggest: [{
-        desc: "Convert to Text",
+    var _SpanElements$_check = SpanElements._check(node, {
+        context: context,
+        config: config
+      }),
+      success = _SpanElements$_check.success,
+      autoFixable = _SpanElements$_check.autoFixable;
+    if (success && autoFixable) {
+      var fix = SpanElements._fix(node, {
+        context: context,
+        config: config
+      });
+      context.report(_objectSpread({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText'
+      }, config.enableUnsafeAutofix ? {
         fix: fix
-      }]
-    }));
+      } : {
+        suggest: [{
+          desc: "Convert to Text",
+          fix: fix
+        }]
+      }));
+    } else if (success && config.enableUnsafeReport) {
+      context.report({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText'
+      });
+    }
   },
   _check: function _check(node, _ref2) {
     var context = _ref2.context,
       config = _ref2.config;
     if (!config.patterns.includes('span-elements')) {
-      return false;
+      return {
+        success: false
+      };
     }
     var elementName = ast.JSXElement.getName(node);
     if (elementName !== 'span') {
-      return false;
+      return {
+        success: false
+      };
     }
     if (!node.children.length) {
-      return false;
-    }
-
-    // Element has no unallowed props
-    if (!ast.JSXElement.hasAllowedAttrsOnly(node, _common.allowedAttrs)) {
-      return false;
+      return {
+        success: false
+      };
     }
 
     // Only allow elements with strings as children
     if (!(0, _common.hasTextChildrenOnly)(node)) {
-      return false;
+      return {
+        success: false
+      };
+    }
+
+    // Element has no unallowed props
+    if (!ast.JSXElement.hasAllowedAttrsOnly(node, _common.allowedAttrs)) {
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
-    var importDeclaration = ast.Root.findImportsByModule(context.getSourceCode().ast.body, '@atlaskit/primitives');
 
     // If there is more than one `@atlaskit/primitives` import, then it becomes difficult to determine which import to transform
+    var importDeclaration = ast.Root.findImportsByModule(context.getSourceCode().ast.body, '@atlaskit/primitives');
     if (importDeclaration.length > 1) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
-    return true;
+    return {
+      success: true,
+      autoFixable: true
+    };
   },
   _fix: function _fix(node, _ref3) {
     var context = _ref3.context,

package/dist/cjs/rules/use-primitives-text/transformers/strong-elements.js

@@ -24,53 +24,75 @@ var StrongElements = exports.StrongElements = {
     }
 
     // Check whether all criteria needed to make a transformation are met
-    if (!StrongElements._check(node, {
-      context: context,
-      config: config
-    })) {
-      return;
-    }
-    var fix = StrongElements._fix(node, {
-      context: context,
-      config: config
-    });
-    context.report(_objectSpread({
-      node: node.openingElement,
-      messageId: 'preferPrimitivesText'
-    }, config.enableUnsafeAutofix ? {
-      fix: fix
-    } : {
-      suggest: [{
-        desc: "Convert to Text",
+    var _StrongElements$_chec = StrongElements._check(node, {
+        context: context,
+        config: config
+      }),
+      success = _StrongElements$_chec.success,
+      autoFixable = _StrongElements$_chec.autoFixable;
+    if (success && autoFixable) {
+      var fix = StrongElements._fix(node, {
+        context: context,
+        config: config
+      });
+      context.report(_objectSpread({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText'
+      }, config.enableUnsafeAutofix ? {
         fix: fix
-      }]
-    }));
+      } : {
+        suggest: [{
+          desc: "Convert to Text",
+          fix: fix
+        }]
+      }));
+    } else if (success && config.enableUnsafeReport) {
+      context.report({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText'
+      });
+    }
   },
   _check: function _check(node, _ref2) {
     var context = _ref2.context,
       config = _ref2.config;
     if (!config.patterns.includes('strong-elements')) {
-      return false;
+      return {
+        success: false
+      };
     }
     var elementName = ast.JSXElement.getName(node);
     if (elementName !== 'strong') {
-      return false;
+      return {
+        success: false
+      };
     }
     if (!node.children.length) {
-      return false;
+      return {
+        success: false
+      };
     }
 
     // Element has no unallowed props
     if (!ast.JSXElement.hasAllowedAttrsOnly(node, _common.allowedAttrs)) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
 
     // If there is more than one `@atlaskit/primitives` import, then it becomes difficult to determine which import to transform
     var importDeclaration = ast.Root.findImportsByModule(context.getSourceCode().ast.body, '@atlaskit/primitives');
     if (importDeclaration.length > 1) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
-    return true;
+    return {
+      success: true,
+      autoFixable: true
+    };
   },
   _fix: function _fix(node, _ref3) {
     var context = _ref3.context,

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::ab43b6e1a867d07b9a27eae78d48834a>>
+ * @codegen <<SignedSource::b6947ac630ea512fe3f4e3f44abb6783>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 export default {
@@ -17,12 +17,11 @@ export default {
     '@atlaskit/design-system/no-deprecated-design-token-usage': 'warn',
     '@atlaskit/design-system/no-deprecated-imports': 'error',
     '@atlaskit/design-system/no-direct-use-of-web-platform-drag-and-drop': 'error',
-    '@atlaskit/design-system/no-empty-styled-expression': 'warn',
-    '@atlaskit/design-system/no-exported-css': 'warn',
-    '@atlaskit/design-system/no-exported-keyframes': 'warn',
     '@atlaskit/design-system/no-html-anchor': 'warn',
     '@atlaskit/design-system/no-html-button': 'warn',
-    '@atlaskit/design-system/no-invalid-css-map': 'error',
+    '@atlaskit/design-system/no-invalid-css-map': ['error', {
+      allowedFunctionCalls: [['@atlaskit/tokens', 'token']]
+    }],
     '@atlaskit/design-system/no-keyframes-tagged-template-expression': 'error',
     '@atlaskit/design-system/no-legacy-icons': 'warn',
     '@atlaskit/design-system/no-margin': 'warn',

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::a620cfa5339ecfe9527af26afe5121e4>>
+ * @codegen <<SignedSource::9b6bccf688f4ee18dd0d49dbce48e475>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 export default {
@@ -14,12 +14,11 @@ export default {
     '@atlaskit/design-system/no-deprecated-design-token-usage': 'warn',
     '@atlaskit/design-system/no-deprecated-imports': 'error',
     '@atlaskit/design-system/no-direct-use-of-web-platform-drag-and-drop': 'error',
-    '@atlaskit/design-system/no-empty-styled-expression': 'warn',
-    '@atlaskit/design-system/no-exported-css': 'warn',
-    '@atlaskit/design-system/no-exported-keyframes': 'warn',
     '@atlaskit/design-system/no-html-anchor': 'warn',
     '@atlaskit/design-system/no-html-button': 'warn',
-    '@atlaskit/design-system/no-invalid-css-map': 'error',
+    '@atlaskit/design-system/no-invalid-css-map': ['error', {
+      allowedFunctionCalls: [['@atlaskit/tokens', 'token']]
+    }],
     '@atlaskit/design-system/no-nested-styles': 'error',
     '@atlaskit/design-system/no-unsafe-design-token-usage': 'error',
     '@atlaskit/design-system/no-unsafe-style-overrides': 'warn',

package/dist/es2019/rules/no-empty-styled-expression/index.js

@@ -40,8 +40,7 @@ const noEmptyStyledExpressionRule = createLintRule({
     name: 'no-empty-styled-expression',
     docs: {
       description: 'Forbids any styled expression to be used when passing empty arguments to styled.div() (or other JSX elements).',
-      recommended: true,
-      severity: 'warn'
+      removeFromPresets: true // effectively disable this rule here, this is configured by `@atlaskit/ui-styling-standard` instead
     },
     messages: {
       unexpected: 'Found an empty expression, or empty object argument passed to `styled` function call. This unnecessarily causes a major performance penalty - please use a plain JSX element or a React fragment instead (e.g. `<div>Hello</div>` or `<>Hello</>`).'

package/dist/es2019/rules/no-exported-css/index.js

@@ -7,8 +7,7 @@ const noExportedCssRule = createLintRule({
     type: 'problem',
     docs: {
       description: 'Forbid exporting `css` function calls. Exporting `css` function calls can result in unexpected behaviour at runtime, and is not statically analysable.',
-      recommended: true,
-      severity: 'warn'
+      removeFromPresets: true // effectively disable this rule here, this is overriden by `@atlaskit/ui-styling-standard` instead
     },
     messages: {
       unexpected: "`css` can't be exported - this will cause unexpected behaviour at runtime. Instead, please move your `css(...)` code to the same file where these styles are being used."

package/dist/es2019/rules/no-exported-keyframes/index.js

@@ -7,8 +7,7 @@ const noExportedKeyframesRule = createLintRule({
     type: 'problem',
     docs: {
       description: 'Forbid exporting `keyframes` function calls. Exporting `css` function calls can result in unexpected behaviour at runtime, and is not statically analysable.',
-      recommended: true,
-      severity: 'warn'
+      removeFromPresets: true // effectively disable this rule here, this is overriden by `@atlaskit/ui-styling-standard` instead
     },
     messages: {
       unexpected: "`keyframes` can't be exported - this will cause unexpected behaviour at runtime. Instead, please move your `keyframes(...)` code to the same file where these styles are being used."

package/dist/es2019/rules/no-invalid-css-map/index.js

@@ -58,7 +58,10 @@ const noInvalidCssMapRule = createLintRule({
     docs: {
       description: "Checks the validity of a CSS map created through cssMap. This is intended to be used alongside TypeScript's type-checking.",
       recommended: true,
-      severity: 'error'
+      severity: 'error',
+      pluginConfig: {
+        allowedFunctionCalls: [['@atlaskit/tokens', 'token']]
+      }
     },
     messages: {
       mustBeTopLevelScope: 'cssMap must only be used in the top-most scope of the module.',

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

@@ -121,12 +121,12 @@ export const createGuidance = (iconPackage, insideNewButton = false, size) => {
  * @returns True if the color can be migrated, false otherwise
  */
 export const canMigrateColor = color => {
-  if (color.match(/^color\.text$/)) {
-    return true;
-  } else if (color.match(/^color\.icon/)) {
+  if (color.match(/^color\.icon/)) {
     return true;
   } else if (color.match(/^color\.link/)) {
     return true;
+  } else if (color.match(/^color\.text/)) {
+    return true;
   } else if (color === 'currentColor') {
     return true;
   } else {

package/dist/es2019/rules/use-heading/config/index.js

@@ -1,7 +1,8 @@
 const defaults = {
   failSilently: false,
   patterns: ['native-elements'],
-  enableUnsafeAutofix: false
+  enableUnsafeAutofix: false,
+  enableUnsafeReport: false
 };
 export const getConfig = overrides => {
   return Object.assign({}, defaults, overrides);

package/dist/es2019/rules/use-heading/transformers/native-elements.js

@@ -17,65 +17,96 @@ export const NativeElements = {
     config
   }) {
     // Check whether all criteria needed to make a transformation are met
-    if (!NativeElements._check(node, {
-      context,
-      config
-    })) {
-      return;
-    }
-    const fix = NativeElements._fix(node, {
+    const {
+      success,
+      autoFixable
+    } = NativeElements._check(node, {
       context,
       config
     });
-    context.report({
-      node,
-      messageId: 'preferHeading',
-      ...(config.enableUnsafeAutofix ? {
-        fix
-      } : {
-        suggest: [{
-          desc: `Convert to Heading`,
+    if (success && autoFixable) {
+      const fix = NativeElements._fix(node, {
+        context,
+        config
+      });
+      context.report({
+        node,
+        messageId: 'preferHeading',
+        ...(config.enableUnsafeAutofix ? {
           fix
-        }]
-      })
-    });
+        } : {
+          suggest: [{
+            desc: `Convert to Heading`,
+            fix
+          }]
+        })
+      });
+    } else if (success && config.enableUnsafeReport) {
+      context.report({
+        node,
+        messageId: 'preferHeading'
+      });
+    }
   },
   _check(node, {
     config
   }) {
     if (!config.patterns.includes('native-elements')) {
-      return false;
+      return {
+        success: false
+      };
     }
     if (!isNodeOfType(node, 'JSXElement')) {
-      return false;
+      return {
+        success: false
+      };
     }
     if (!node.children.length) {
-      return false;
-    }
-    if (!node.parent) {
-      return false;
+      return {
+        success: false
+      };
     }
     const elementName = ast.JSXElement.getName(node);
     if (!Object.keys(tagSizeMap).includes(elementName)) {
-      return false;
+      return {
+        success: false
+      };
+    }
+    if (!node.parent) {
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
 
     // Element has to be first element of its siblings
     if (!(isNodeOfType(node.parent, 'JSXElement') || isNodeOfType(node.parent, 'JSXFragment'))) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
     const siblings = ast.JSXElement.getChildren(node.parent);
     if (siblings.length > 1) {
       var _siblings$0$range, _node$range, _siblings$0$range2, _node$range2;
       // Only report if element is first child element
       if (((_siblings$0$range = siblings[0].range) === null || _siblings$0$range === void 0 ? void 0 : _siblings$0$range[0]) !== ((_node$range = node.range) === null || _node$range === void 0 ? void 0 : _node$range[0]) || ((_siblings$0$range2 = siblings[0].range) === null || _siblings$0$range2 === void 0 ? void 0 : _siblings$0$range2[1]) !== ((_node$range2 = node.range) === null || _node$range2 === void 0 ? void 0 : _node$range2[1])) {
-        return false;
+        return {
+          success: true,
+          autoFixable: false
+        };
       }
     }
     if (!ast.JSXElement.hasAllowedAttrsOnly(node, allowedAttrs)) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
-    return true;
+    return {
+      success: true,
+      autoFixable: true
+    };
   },
   _fix(node, {
     context

package/dist/es2019/rules/use-primitives-text/config/index.js

@@ -2,7 +2,8 @@ const defaults = {
   failSilently: false,
   patterns: ['paragraph-elements', 'span-elements', 'strong-elements', 'emphasis-elements'],
   inheritColor: false,
-  enableUnsafeAutofix: false
+  enableUnsafeAutofix: false,
+  enableUnsafeReport: false
 };
 export const getConfig = overrides => {
   return Object.assign({}, defaults, overrides);

package/dist/es2019/rules/use-primitives-text/transformers/emphasis-elements.js

@@ -13,55 +13,78 @@ export const EmphasisElements = {
     }
 
     // Check whether all criteria needed to make a transformation are met
-    if (!EmphasisElements._check(node, {
-      context,
-      config
-    })) {
-      return;
-    }
-    const fix = EmphasisElements._fix(node, {
+    const {
+      success,
+      autoFixable
+    } = EmphasisElements._check(node, {
       context,
       config
     });
-    context.report({
-      node: node.openingElement,
-      messageId: 'preferPrimitivesText',
-      ...(config.enableUnsafeAutofix ? {
-        fix
-      } : {
-        suggest: [{
-          desc: `Convert to Text`,
+    if (success && autoFixable) {
+      const fix = EmphasisElements._fix(node, {
+        context,
+        config
+      });
+      context.report({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText',
+        ...(config.enableUnsafeAutofix ? {
           fix
-        }]
-      })
-    });
+        } : {
+          suggest: [{
+            desc: `Convert to Text`,
+            fix
+          }]
+        })
+      });
+    } else if (success && config.enableUnsafeReport) {
+      context.report({
+        node: node.openingElement,
+        messageId: 'preferPrimitivesText'
+      });
+    }
   },
   _check(node, {
     context,
     config
   }) {
     if (!config.patterns.includes('emphasis-elements')) {
-      return false;
+      return {
+        success: false
+      };
     }
     const elementName = ast.JSXElement.getName(node);
     if (elementName !== 'em') {
-      return false;
+      return {
+        success: false
+      };
     }
     if (!node.children.length) {
-      return false;
+      return {
+        success: false
+      };
     }
 
     // Element has no unallowed props
     if (!ast.JSXElement.hasAllowedAttrsOnly(node, allowedAttrs)) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
 
     // If there is more than one `@atlaskit/primitives` import, then it becomes difficult to determine which import to transform
     const importDeclaration = ast.Root.findImportsByModule(context.getSourceCode().ast.body, '@atlaskit/primitives');
     if (importDeclaration.length > 1) {
-      return false;
+      return {
+        success: true,
+        autoFixable: false
+      };
     }
-    return true;
+    return {
+      success: true,
+      autoFixable: true
+    };
   },
   _fix(node, {
     context,