@atlaskit/eslint-plugin-design-system 13.31.0 → 13.32.0

package/dist/esm/presets/all-flat.codegen.js

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::23364c0ee38a3772c2a588841ed22c55>>
+ * @codegen <<SignedSource::4fa72535ce2bc0a19a6fb01c951150db>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -45,6 +45,7 @@ var rules = {
     '@atlaskit/design-system/no-margin': 'warn',
     '@atlaskit/design-system/no-nested-styles': 'error',
     '@atlaskit/design-system/no-physical-properties': 'error',
+    '@atlaskit/design-system/no-placeholder': 'warn',
     '@atlaskit/design-system/no-separator-with-list-elements': 'warn',
     '@atlaskit/design-system/no-styled-tagged-template-expression': 'error',
     '@atlaskit/design-system/no-to-match-snapshot': 'error',
@@ -53,7 +54,6 @@ var rules = {
     '@atlaskit/design-system/no-unsafe-style-overrides': 'warn',
     '@atlaskit/design-system/no-unsupported-drag-and-drop-libraries': 'error',
     '@atlaskit/design-system/no-unused-css-map': 'warn',
-    '@atlaskit/design-system/no-utility-icons': 'warn',
     '@atlaskit/design-system/prefer-primitives': 'warn',
     '@atlaskit/design-system/use-button-group-label': 'warn',
     '@atlaskit/design-system/use-character-counter-field': 'warn',

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::760141a986e4947e8a1ce74480931f43>>
+ * @codegen <<SignedSource::dcf6bf4496c97fd6ad4c1d8d65337a6b>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -44,6 +44,7 @@ var rules = {
     '@atlaskit/design-system/no-margin': 'warn',
     '@atlaskit/design-system/no-nested-styles': 'error',
     '@atlaskit/design-system/no-physical-properties': 'error',
+    '@atlaskit/design-system/no-placeholder': 'warn',
     '@atlaskit/design-system/no-separator-with-list-elements': 'warn',
     '@atlaskit/design-system/no-styled-tagged-template-expression': 'error',
     '@atlaskit/design-system/no-to-match-snapshot': 'error',
@@ -52,7 +53,6 @@ var rules = {
     '@atlaskit/design-system/no-unsafe-style-overrides': 'warn',
     '@atlaskit/design-system/no-unsupported-drag-and-drop-libraries': 'error',
     '@atlaskit/design-system/no-unused-css-map': 'warn',
-    '@atlaskit/design-system/no-utility-icons': 'warn',
     '@atlaskit/design-system/prefer-primitives': 'warn',
     '@atlaskit/design-system/use-button-group-label': 'warn',
     '@atlaskit/design-system/use-character-counter-field': 'warn',

package/dist/esm/presets/recommended-flat.codegen.js

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::ba71ef89188bdc20d0ec948cc00f6436>>
+ * @codegen <<SignedSource::27c9a3631c47b134bfb9da3e921e0601>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -32,12 +32,12 @@ var rules = {
       allowedFunctionCalls: [['@atlaskit/tokens', 'token']]
     }],
     '@atlaskit/design-system/no-nested-styles': 'error',
+    '@atlaskit/design-system/no-placeholder': 'warn',
     '@atlaskit/design-system/no-separator-with-list-elements': 'warn',
     '@atlaskit/design-system/no-unsafe-design-token-usage': 'error',
     '@atlaskit/design-system/no-unsafe-style-overrides': 'warn',
     '@atlaskit/design-system/no-unsupported-drag-and-drop-libraries': 'error',
     '@atlaskit/design-system/no-unused-css-map': 'warn',
-    '@atlaskit/design-system/no-utility-icons': 'warn',
     '@atlaskit/design-system/use-button-group-label': 'warn',
     '@atlaskit/design-system/use-character-counter-field': 'warn',
     '@atlaskit/design-system/use-correct-field': 'warn',

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::5c119e0ab08e1f39d471b2b827bbb4de>>
+ * @codegen <<SignedSource::52a31bc6cdeb7a9818a0e225d21c88f2>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -31,12 +31,12 @@ var rules = {
       allowedFunctionCalls: [['@atlaskit/tokens', 'token']]
     }],
     '@atlaskit/design-system/no-nested-styles': 'error',
+    '@atlaskit/design-system/no-placeholder': 'warn',
     '@atlaskit/design-system/no-separator-with-list-elements': 'warn',
     '@atlaskit/design-system/no-unsafe-design-token-usage': 'error',
     '@atlaskit/design-system/no-unsafe-style-overrides': 'warn',
     '@atlaskit/design-system/no-unsupported-drag-and-drop-libraries': 'error',
     '@atlaskit/design-system/no-unused-css-map': 'warn',
-    '@atlaskit/design-system/no-utility-icons': 'warn',
     '@atlaskit/design-system/use-button-group-label': 'warn',
     '@atlaskit/design-system/use-character-counter-field': 'warn',
     '@atlaskit/design-system/use-correct-field': 'warn',

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::7b80dfb4568d4c5f7c84d7c382b81de9>>
+ * @codegen <<SignedSource::ca89575f70ae79af2f43113bdc77c0c5>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -42,6 +42,7 @@ import noLegacyIcons from './no-legacy-icons';
 import noMargin from './no-margin';
 import noNestedStyles from './no-nested-styles';
 import noPhysicalProperties from './no-physical-properties';
+import noPlaceholder from './no-placeholder';
 import noSeparatorWithListElements from './no-separator-with-list-elements';
 import noStyledTaggedTemplateExpression from './no-styled-tagged-template-expression';
 import noToMatchSnapshot from './no-to-match-snapshot';
@@ -50,7 +51,6 @@ import noUnsafeInlineSnapshot from './no-unsafe-inline-snapshot';
 import noUnsafeStyleOverrides from './no-unsafe-style-overrides';
 import noUnsupportedDragAndDropLibraries from './no-unsupported-drag-and-drop-libraries';
 import noUnusedCssMap from './no-unused-css-map';
-import noUtilityIcons from './no-utility-icons';
 import preferPrimitives from './prefer-primitives';
 import useButtonGroupLabel from './use-button-group-label';
 import useCharacterCounterField from './use-character-counter-field';
@@ -116,6 +116,7 @@ export var rules = {
   'no-margin': noMargin,
   'no-nested-styles': noNestedStyles,
   'no-physical-properties': noPhysicalProperties,
+  'no-placeholder': noPlaceholder,
   'no-separator-with-list-elements': noSeparatorWithListElements,
   'no-styled-tagged-template-expression': noStyledTaggedTemplateExpression,
   'no-to-match-snapshot': noToMatchSnapshot,
@@ -124,7 +125,6 @@ export var rules = {
   'no-unsafe-style-overrides': noUnsafeStyleOverrides,
   'no-unsupported-drag-and-drop-libraries': noUnsupportedDragAndDropLibraries,
   'no-unused-css-map': noUnusedCssMap,
-  'no-utility-icons': noUtilityIcons,
   'prefer-primitives': preferPrimitives,
   'use-button-group-label': useButtonGroupLabel,
   'use-character-counter-field': useCharacterCounterField,

package/dist/esm/rules/no-placeholder/index.js

@@ -0,0 +1,154 @@
+import { isNodeOfType } from 'eslint-codemod-utils';
+import * as ast from '../../ast-nodes';
+import { createLintRule } from '../utils/create-rule';
+export var AFFECTED_HTML_ELEMENTS = ['input', 'textarea'];
+export var AFFECTED_ATLASKIT_PACKAGES = {
+  '@atlaskit/datetime-picker': ['DatePicker', 'TimePicker', 'DateTimePicker'],
+  '@atlaskit/select': ['default', 'AsyncCreatableSelect', 'CheckboxSelect', 'CountrySelect', 'CreatableSelect', 'PopupSelect', 'RadioSelect'],
+  '@atlaskit/textarea': ['default'],
+  '@atlaskit/textfield': ['default']
+};
+export var ATLASKIT_FORM_PACKAGE = '@atlaskit/form';
+export var ATLASKIT_FIELD_IMPORT = 'Field';
+var rule = createLintRule({
+  meta: {
+    name: 'no-placeholder',
+    type: 'problem',
+    docs: {
+      description: 'Placeholders should not be used. If information should be given to the user about the proper type or formatting of a value, this should be included using a helper message that is associated to the input instead.',
+      recommended: true,
+      severity: 'warn'
+    },
+    messages: {
+      noPlaceholder: 'Placeholders should not be used. Use separate information to help users associated using `aria-describedby` instead.',
+      noPlaceholderOnSimpleField: 'Placeholders should not be used. Use the `helperMessage` prop instead.',
+      noPlaceholderOnComplexField: 'Placeholders should not be used. Use the `HelperMessage` component instead.'
+    }
+  },
+  create: function create(context) {
+    var hasFieldImport = false;
+    var fieldLocalName = null;
+    var localComponentNames = [];
+    return {
+      ImportDeclaration: function ImportDeclaration(node) {
+        var source = node.source.value;
+        if (typeof source !== 'string') {
+          return;
+        }
+
+        // Ignore non-atlaskit input/form packages
+        if (!Object.keys(AFFECTED_ATLASKIT_PACKAGES).includes(source) && ATLASKIT_FORM_PACKAGE !== source) {
+          return;
+        }
+        if (!node.specifiers.length) {
+          return;
+        }
+        var defaultImport = node.specifiers.filter(function (spec) {
+          return spec.type === 'ImportDefaultSpecifier';
+        });
+        var namedImport = node.specifiers.filter(function (spec) {
+          return spec.type === 'ImportSpecifier';
+        });
+        if (source === ATLASKIT_FORM_PACKAGE) {
+          if (namedImport.length && namedImport[0].type === 'ImportSpecifier' && 'name' in namedImport[0].imported && namedImport[0].imported.name === ATLASKIT_FIELD_IMPORT) {
+            hasFieldImport = true;
+            fieldLocalName = namedImport[0].local.name;
+          }
+        } else {
+          var importNames = AFFECTED_ATLASKIT_PACKAGES[source];
+          var usesDefaultImport = importNames.includes('default');
+          var possibleNamedImports = importNames.filter(function (importName) {
+            return importName !== 'default';
+          });
+          if (usesDefaultImport && defaultImport.length && defaultImport[0].local) {
+            localComponentNames.push(defaultImport[0].local.name);
+            // or if popup and using a named import
+          } else if (possibleNamedImports.length >= 1 && namedImport.length) {
+            namedImport.forEach(function (imp) {
+              if (imp.type === 'ImportSpecifier' && 'name' in imp.imported && possibleNamedImports.includes(imp.imported.name)) {
+                localComponentNames.push(imp.local.name);
+              }
+            });
+          }
+        }
+      },
+      JSXElement: function JSXElement(node) {
+        if (!isNodeOfType(node, 'JSXElement')) {
+          return false;
+        }
+        var elName = ast.JSXElement.getName(node);
+        if (!elName) {
+          return false;
+        }
+
+        // If it is one of the affected native HTML elements
+        if (AFFECTED_HTML_ELEMENTS.includes(elName)) {
+          // if has a placeholder attribute
+          var hasPlaceholderAttribute = node.openingElement.attributes.find(function (attr) {
+            return isNodeOfType(attr, 'JSXAttribute') && attr.name.name === 'placeholder';
+          });
+          if (hasPlaceholderAttribute) {
+            return context.report({
+              node: node.openingElement,
+              messageId: 'noPlaceholder'
+            });
+          }
+          // Else, it is a React component
+        } else {
+          // if none of the affected packages is imported, return
+          if (localComponentNames.length === 0) {
+            return;
+          }
+          // if component name is not in the list, exit
+          if (!isNodeOfType(node.openingElement.name, 'JSXIdentifier') || !localComponentNames.includes(node.openingElement.name.name)) {
+            return;
+          }
+          if (hasFieldImport && fieldLocalName) {
+            var _node = node;
+            var hasParentField = false;
+            var fieldType = 'Complex';
+            // If node is a Field element or if
+            while (isNodeOfType(_node, 'JSXElement') && isNodeOfType(_node.openingElement.name, 'JSXIdentifier') && !hasParentField) {
+              var name = _node.openingElement.name.name;
+              hasParentField = hasParentField || name === fieldLocalName;
+              _node = _node.parent;
+              // Skip up until a JSXElement or JSXAttribute is reached
+              if (isNodeOfType(_node, 'JSXFragment') || isNodeOfType(_node, 'ArrowFunctionExpression') || isNodeOfType(_node, 'JSXExpressionContainer') || isNodeOfType(_node, 'JSXAttribute')) {
+                while (_node && !isNodeOfType(_node, 'JSXElement') && !isNodeOfType(_node, 'Program')) {
+                  if (isNodeOfType(_node, 'JSXAttribute')) {
+                    fieldType = 'Simple';
+                  }
+                  _node = _node.parent;
+                }
+              }
+            }
+            if (hasParentField) {
+              // if has a placeholder attribute
+              var _hasPlaceholderAttribute = node.openingElement.attributes.find(function (attr) {
+                return isNodeOfType(attr, 'JSXAttribute') && attr.name.name === 'placeholder';
+              });
+              if (_hasPlaceholderAttribute) {
+                return context.report({
+                  node: node.openingElement,
+                  messageId: "noPlaceholderOn".concat(fieldType, "Field")
+                });
+              }
+            }
+          } else {
+            // if has a placeholder attribute
+            var _hasPlaceholderAttribute2 = node.openingElement.attributes.find(function (attr) {
+              return isNodeOfType(attr, 'JSXAttribute') && attr.name.name === 'placeholder';
+            });
+            if (_hasPlaceholderAttribute2) {
+              return context.report({
+                node: node.openingElement,
+                messageId: 'noPlaceholder'
+              });
+            }
+          }
+        }
+      }
+    };
+  }
+});
+export default rule;

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::23364c0ee38a3772c2a588841ed22c55>>
+ * @codegen <<SignedSource::4fa72535ce2bc0a19a6fb01c951150db>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Linter } from 'eslint';

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::760141a986e4947e8a1ce74480931f43>>
+ * @codegen <<SignedSource::dcf6bf4496c97fd6ad4c1d8d65337a6b>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { ESLint } from 'eslint';

package/dist/types/presets/recommended-flat.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::ba71ef89188bdc20d0ec948cc00f6436>>
+ * @codegen <<SignedSource::27c9a3631c47b134bfb9da3e921e0601>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Linter } from 'eslint';

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::5c119e0ab08e1f39d471b2b827bbb4de>>
+ * @codegen <<SignedSource::52a31bc6cdeb7a9818a0e225d21c88f2>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { ESLint } from 'eslint';

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

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::7b80dfb4568d4c5f7c84d7c382b81de9>>
+ * @codegen <<SignedSource::ca89575f70ae79af2f43113bdc77c0c5>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Rule } from 'eslint';

package/dist/types/rules/no-placeholder/index.d.ts

@@ -0,0 +1,6 @@
+export declare const AFFECTED_HTML_ELEMENTS: string[];
+export declare const AFFECTED_ATLASKIT_PACKAGES: Record<string, string[]>;
+export declare const ATLASKIT_FORM_PACKAGE = "@atlaskit/form";
+export declare const ATLASKIT_FIELD_IMPORT = "Field";
+declare const rule: import("eslint").Rule.RuleModule;
+export default rule;

package/dist/types-ts4.5/presets/all-flat.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::23364c0ee38a3772c2a588841ed22c55>>
+ * @codegen <<SignedSource::4fa72535ce2bc0a19a6fb01c951150db>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Linter } from 'eslint';

package/dist/types-ts4.5/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::760141a986e4947e8a1ce74480931f43>>
+ * @codegen <<SignedSource::dcf6bf4496c97fd6ad4c1d8d65337a6b>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { ESLint } from 'eslint';

package/dist/types-ts4.5/presets/recommended-flat.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::ba71ef89188bdc20d0ec948cc00f6436>>
+ * @codegen <<SignedSource::27c9a3631c47b134bfb9da3e921e0601>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Linter } from 'eslint';

package/dist/types-ts4.5/presets/recommended.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::5c119e0ab08e1f39d471b2b827bbb4de>>
+ * @codegen <<SignedSource::52a31bc6cdeb7a9818a0e225d21c88f2>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { ESLint } from 'eslint';

package/dist/types-ts4.5/rules/index.codegen.d.ts

@@ -1,6 +1,6 @@
 /**
  * THIS FILE WAS CREATED VIA CODEGEN DO NOT MODIFY {@see http://go/af-codegen}
- * @codegen <<SignedSource::7b80dfb4568d4c5f7c84d7c382b81de9>>
+ * @codegen <<SignedSource::ca89575f70ae79af2f43113bdc77c0c5>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 import type { Rule } from 'eslint';

package/dist/types-ts4.5/rules/no-placeholder/index.d.ts

@@ -0,0 +1,6 @@
+export declare const AFFECTED_HTML_ELEMENTS: string[];
+export declare const AFFECTED_ATLASKIT_PACKAGES: Record<string, string[]>;
+export declare const ATLASKIT_FORM_PACKAGE = "@atlaskit/form";
+export declare const ATLASKIT_FIELD_IMPORT = "Field";
+declare const rule: import("eslint").Rule.RuleModule;
+export default rule;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@atlaskit/eslint-plugin-design-system",
 	"description": "The essential plugin for use with the Atlassian Design System.",
-	"version": "13.31.0",
+	"version": "13.32.0",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
 	"publishConfig": {
@@ -40,7 +40,7 @@
 	"dependencies": {
 		"@atlaskit/eslint-utils": "^2.0.0",
 		"@atlaskit/icon": "^29.4.0",
-		"@atlaskit/icon-lab": "^5.13.0",
+		"@atlaskit/icon-lab": "^5.14.0",
 		"@atlaskit/tokens": "^9.1.0",
 		"@babel/runtime": "^7.0.0",
 		"@typescript-eslint/utils": "^7.1.0",