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

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @atlaskit/eslint-plugin-design-system
 
+## 13.32.0
+
+### Minor Changes
+
+- [`7925336391df8`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/7925336391df8) -
+  Remove deprecated no-utility-icons ESLint rule as utility icons have been fully removed from
+  @atlaskit/icon package.
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 13.31.0
 
 ### Minor Changes

package/README.md

@@ -87,6 +87,7 @@ module.exports = {
 | <a href="./src/rules/no-margin/README.md">no-margin</a>                                                                       | Disallow using the margin CSS property.                                                                                                                                                                                                                                                                             |             |         |             |
 | <a href="./src/rules/no-nested-styles/README.md">no-nested-styles</a>                                                         | Disallows use of nested styles in `css` functions.                                                                                                                                                                                                                                                                  | Yes         |         |             |
 | <a href="./src/rules/no-physical-properties/README.md">no-physical-properties</a>                                             | Disallow physical properties and values in `css` and `cssMap` function calls.                                                                                                                                                                                                                                       |             | Yes     |             |
+| <a href="./src/rules/no-placeholder/README.md">no-placeholder</a>                                                             | 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.                                                                                                 | Yes         |         |             |
 | <a href="./src/rules/no-separator-with-list-elements/README.md">no-separator-with-list-elements</a>                           | Warn when the `separator` prop is used with `as="li"`, `as="ol"`, or `as="dl"` in the Inline component.                                                                                                                                                                                                             | Yes         |         |             |
 | <a href="./src/rules/no-styled-tagged-template-expression/README.md">no-styled-tagged-template-expression</a>                 | Disallows any `styled` tagged template expressions that originate from Emotion, Styled Components or Compiled                                                                                                                                                                                                       |             | Yes     |             |
 | <a href="./src/rules/no-to-match-snapshot/README.md">no-to-match-snapshot</a>                                                 | Disallow using toMatchSnapshot() in favor of toMatchInlineSnapshot(). See https://hello.atlassian.net/wiki/spaces/DST/pages/6105892000/DSTRFC-038+-+Removal+of+.toMatchSnapshot for rationale.                                                                                                                      |             |         |             |
@@ -95,7 +96,6 @@ module.exports = {
 | <a href="./src/rules/no-unsafe-style-overrides/README.md">no-unsafe-style-overrides</a>                                       | Discourage usage of unsafe style overrides used against the Atlassian Design System.                                                                                                                                                                                                                                | Yes         |         |             |
 | <a href="./src/rules/no-unsupported-drag-and-drop-libraries/README.md">no-unsupported-drag-and-drop-libraries</a>             | Disallow importing unsupported drag and drop modules.                                                                                                                                                                                                                                                               | Yes         |         |             |
 | <a href="./src/rules/no-unused-css-map/README.md">no-unused-css-map</a>                                                       | Detects unused styles in cssMap objects to help keep code clean.                                                                                                                                                                                                                                                    | Yes         |         |             |
-| <a href="./src/rules/no-utility-icons/README.md">no-utility-icons</a>                                                         | Disallow use of deprecated utility icons, in favor of core icons with `size="small"`.                                                                                                                                                                                                                               | Yes         | Yes     | Yes         |
 | <a href="./src/rules/prefer-primitives/README.md">prefer-primitives</a>                                                       | Increase awareness of primitive components via code hints. Strictly used for education purposes and discoverability. To enforce usage please refer to the `use-primitives` rule.                                                                                                                                    |             |         |             |
 | <a href="./src/rules/use-button-group-label/README.md">use-button-group-label</a>                                             | Ensures button groups are described to assistive technology by a direct label or by another element.                                                                                                                                                                                                                | Yes         |         | Yes         |
 | <a href="./src/rules/use-character-counter-field/README.md">use-character-counter-field</a>                                   | Suggests using CharacterCounterField or CharacterCounter when Textfield or Textarea components have maxLength or minLength props.                                                                                                                                                                                   | Yes         |         |             |

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

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 /**
  * 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
  */
 
@@ -51,6 +51,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',
@@ -59,7 +60,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/cjs/presets/all.codegen.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 /**
  * 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
  */
 
@@ -50,6 +50,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',
@@ -58,7 +59,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/cjs/presets/recommended-flat.codegen.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 /**
  * 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
  */
 
@@ -38,12 +38,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/cjs/presets/recommended.codegen.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 /**
  * 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
  */
 
@@ -37,12 +37,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/cjs/rules/index.codegen.js

@@ -43,6 +43,7 @@ var _noLegacyIcons = _interopRequireDefault(require("./no-legacy-icons"));
 var _noMargin = _interopRequireDefault(require("./no-margin"));
 var _noNestedStyles = _interopRequireDefault(require("./no-nested-styles"));
 var _noPhysicalProperties = _interopRequireDefault(require("./no-physical-properties"));
+var _noPlaceholder = _interopRequireDefault(require("./no-placeholder"));
 var _noSeparatorWithListElements = _interopRequireDefault(require("./no-separator-with-list-elements"));
 var _noStyledTaggedTemplateExpression = _interopRequireDefault(require("./no-styled-tagged-template-expression"));
 var _noToMatchSnapshot = _interopRequireDefault(require("./no-to-match-snapshot"));
@@ -51,7 +52,6 @@ var _noUnsafeInlineSnapshot = _interopRequireDefault(require("./no-unsafe-inline
 var _noUnsafeStyleOverrides = _interopRequireDefault(require("./no-unsafe-style-overrides"));
 var _noUnsupportedDragAndDropLibraries = _interopRequireDefault(require("./no-unsupported-drag-and-drop-libraries"));
 var _noUnusedCssMap = _interopRequireDefault(require("./no-unused-css-map"));
-var _noUtilityIcons = _interopRequireDefault(require("./no-utility-icons"));
 var _preferPrimitives = _interopRequireDefault(require("./prefer-primitives"));
 var _useButtonGroupLabel = _interopRequireDefault(require("./use-button-group-label"));
 var _useCharacterCounterField = _interopRequireDefault(require("./use-character-counter-field"));
@@ -80,7 +80,7 @@ var _useTokensTypography = _interopRequireDefault(require("./use-tokens-typograp
 var _useVisuallyHidden = _interopRequireDefault(require("./use-visually-hidden"));
 /**
  * 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
  */
 
@@ -123,6 +123,7 @@ var rules = exports.rules = {
   'no-margin': _noMargin.default,
   'no-nested-styles': _noNestedStyles.default,
   'no-physical-properties': _noPhysicalProperties.default,
+  'no-placeholder': _noPlaceholder.default,
   'no-separator-with-list-elements': _noSeparatorWithListElements.default,
   'no-styled-tagged-template-expression': _noStyledTaggedTemplateExpression.default,
   'no-to-match-snapshot': _noToMatchSnapshot.default,
@@ -131,7 +132,6 @@ var rules = exports.rules = {
   'no-unsafe-style-overrides': _noUnsafeStyleOverrides.default,
   'no-unsupported-drag-and-drop-libraries': _noUnsupportedDragAndDropLibraries.default,
   'no-unused-css-map': _noUnusedCssMap.default,
-  'no-utility-icons': _noUtilityIcons.default,
   'prefer-primitives': _preferPrimitives.default,
   'use-button-group-label': _useButtonGroupLabel.default,
   'use-character-counter-field': _useCharacterCounterField.default,

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

@@ -0,0 +1,162 @@
+"use strict";
+
+var _typeof = require("@babel/runtime/helpers/typeof");
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = exports.ATLASKIT_FORM_PACKAGE = exports.ATLASKIT_FIELD_IMPORT = exports.AFFECTED_HTML_ELEMENTS = exports.AFFECTED_ATLASKIT_PACKAGES = void 0;
+var _eslintCodemodUtils = require("eslint-codemod-utils");
+var ast = _interopRequireWildcard(require("../../ast-nodes"));
+var _createRule = require("../utils/create-rule");
+function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
+var AFFECTED_HTML_ELEMENTS = exports.AFFECTED_HTML_ELEMENTS = ['input', 'textarea'];
+var AFFECTED_ATLASKIT_PACKAGES = exports.AFFECTED_ATLASKIT_PACKAGES = {
+  '@atlaskit/datetime-picker': ['DatePicker', 'TimePicker', 'DateTimePicker'],
+  '@atlaskit/select': ['default', 'AsyncCreatableSelect', 'CheckboxSelect', 'CountrySelect', 'CreatableSelect', 'PopupSelect', 'RadioSelect'],
+  '@atlaskit/textarea': ['default'],
+  '@atlaskit/textfield': ['default']
+};
+var ATLASKIT_FORM_PACKAGE = exports.ATLASKIT_FORM_PACKAGE = '@atlaskit/form';
+var ATLASKIT_FIELD_IMPORT = exports.ATLASKIT_FIELD_IMPORT = 'Field';
+var rule = (0, _createRule.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 (!(0, _eslintCodemodUtils.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 (0, _eslintCodemodUtils.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 (!(0, _eslintCodemodUtils.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 ((0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXElement') && (0, _eslintCodemodUtils.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 ((0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXFragment') || (0, _eslintCodemodUtils.isNodeOfType)(_node, 'ArrowFunctionExpression') || (0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXExpressionContainer') || (0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXAttribute')) {
+                while (_node && !(0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXElement') && !(0, _eslintCodemodUtils.isNodeOfType)(_node, 'Program')) {
+                  if ((0, _eslintCodemodUtils.isNodeOfType)(_node, 'JSXAttribute')) {
+                    fieldType = 'Simple';
+                  }
+                  _node = _node.parent;
+                }
+              }
+            }
+            if (hasParentField) {
+              // if has a placeholder attribute
+              var _hasPlaceholderAttribute = node.openingElement.attributes.find(function (attr) {
+                return (0, _eslintCodemodUtils.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 (0, _eslintCodemodUtils.isNodeOfType)(attr, 'JSXAttribute') && attr.name.name === 'placeholder';
+            });
+            if (_hasPlaceholderAttribute2) {
+              return context.report({
+                node: node.openingElement,
+                messageId: 'noPlaceholder'
+              });
+            }
+          }
+        }
+      }
+    };
+  }
+});
+var _default = exports.default = rule;

package/dist/es2019/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 @@ const 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 @@ const 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/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::760141a986e4947e8a1ce74480931f43>>
+ * @codegen <<SignedSource::dcf6bf4496c97fd6ad4c1d8d65337a6b>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -44,6 +44,7 @@ const 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 @@ const 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/es2019/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 @@ const 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/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::5c119e0ab08e1f39d471b2b827bbb4de>>
+ * @codegen <<SignedSource::52a31bc6cdeb7a9818a0e225d21c88f2>>
  * @codegenCommand yarn workspace @atlaskit/eslint-plugin-design-system codegen
  */
 
@@ -31,12 +31,12 @@ const 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/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::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 const 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 const 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/es2019/rules/no-placeholder/index.js

@@ -0,0 +1,142 @@
+import { isNodeOfType } from 'eslint-codemod-utils';
+import * as ast from '../../ast-nodes';
+import { createLintRule } from '../utils/create-rule';
+export const AFFECTED_HTML_ELEMENTS = ['input', 'textarea'];
+export const 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 const ATLASKIT_FORM_PACKAGE = '@atlaskit/form';
+export const ATLASKIT_FIELD_IMPORT = 'Field';
+const 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(context) {
+    let hasFieldImport = false;
+    let fieldLocalName = null;
+    const localComponentNames = [];
+    return {
+      ImportDeclaration(node) {
+        const 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;
+        }
+        const defaultImport = node.specifiers.filter(spec => spec.type === 'ImportDefaultSpecifier');
+        const namedImport = node.specifiers.filter(spec => 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 {
+          const importNames = AFFECTED_ATLASKIT_PACKAGES[source];
+          const usesDefaultImport = importNames.includes('default');
+          const possibleNamedImports = importNames.filter(importName => 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(imp => {
+              if (imp.type === 'ImportSpecifier' && 'name' in imp.imported && possibleNamedImports.includes(imp.imported.name)) {
+                localComponentNames.push(imp.local.name);
+              }
+            });
+          }
+        }
+      },
+      JSXElement(node) {
+        if (!isNodeOfType(node, 'JSXElement')) {
+          return false;
+        }
+        const 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
+          const hasPlaceholderAttribute = node.openingElement.attributes.find(attr => 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) {
+            let _node = node;
+            let hasParentField = false;
+            let fieldType = 'Complex';
+            // If node is a Field element or if
+            while (isNodeOfType(_node, 'JSXElement') && isNodeOfType(_node.openingElement.name, 'JSXIdentifier') && !hasParentField) {
+              const 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
+              const hasPlaceholderAttribute = node.openingElement.attributes.find(attr => isNodeOfType(attr, 'JSXAttribute') && attr.name.name === 'placeholder');
+              if (hasPlaceholderAttribute) {
+                return context.report({
+                  node: node.openingElement,
+                  messageId: `noPlaceholderOn${fieldType}Field`
+                });
+              }
+            }
+          } else {
+            // if has a placeholder attribute
+            const hasPlaceholderAttribute = node.openingElement.attributes.find(attr => isNodeOfType(attr, 'JSXAttribute') && attr.name.name === 'placeholder');
+            if (hasPlaceholderAttribute) {
+              return context.report({
+                node: node.openingElement,
+                messageId: 'noPlaceholder'
+              });
+            }
+          }
+        }
+      }
+    };
+  }
+});
+export default rule;