chore: remove flatMap polyfill (#18)

Uses native functionality instead.

Part of #16.

Author: 43081j

.changeset/slimy-taxes-check.md

@@ -0,0 +1,5 @@
+---
+"eslint-plugin-jsx-a11y-x": patch
+---
+
+Removes `flatMap` polyfill and uses native functionality instead.

__tests__/__util__/helpers/parsers.js

@@ -2,7 +2,6 @@ import path from 'path';
 import semver from 'semver';
 import entries from 'object.entries';
 import { version } from 'eslint/package.json';
-import flatMap from 'array.prototype.flatmap';
 
 let tsParserVersion;
 try {
@@ -26,7 +25,7 @@ function minEcmaVersion(features, parserOptions) {
     ...[]
       .concat(
         (parserOptions && parserOptions.ecmaVersion) || [],
-        flatMap(entries(minEcmaVersionForFeatures), entry => {
+        entries(minEcmaVersionForFeatures).flatMap(entry => {
           const f = entry[0];
           const y = entry[1];
           return features.has(f) ? y : [];
@@ -78,7 +77,7 @@ const parsers = {
     };
   },
   all: function all(tests) {
-    const t = flatMap(tests, test => {
+    const t = tests.flatMap(test => {
       /* eslint no-param-reassign: 0 */
       if (typeof test === 'string') {
         test = { code: test };

__tests__/__util__/ruleOptionsMapperFactory.js

@@ -1,7 +1,6 @@
 /** @flow */
 
 import entries from 'object.entries';
-import flatMap from 'array.prototype.flatmap';
 import fromEntries from 'object.fromentries';
 
 type ESLintTestRunnerTestCase = {
@@ -33,7 +32,7 @@ export default function ruleOptionsMapperFactory(
       // Flatten the array of objects in an array of one object.
       options: [
         fromEntries(
-          flatMap((options || []).concat(ruleOptions), item => entries(item)),
+          (options || []).concat(ruleOptions).flatMap(item => entries(item)),
         ),
       ],
       parserOptions,

package.json

@@ -49,7 +49,6 @@
   },
   "dependencies": {
     "aria-query": "^5.3.2",
-    "array.prototype.flatmap": "^1.3.2",
     "ast-types-flow": "^0.0.8",
     "axe-core": "^4.10.2",
     "axobject-query": "^4.1.0",

src/rules/alt-text.js

@@ -8,7 +8,6 @@
 // ----------------------------------------------------------------------------
 
 import { getProp, getPropValue, getLiteralPropValue } from 'jsx-ast-utils';
-import flatMap from 'array.prototype.flatmap';
 
 import { generateObjSchema, arraySchema } from '../util/schemas';
 import getElementType from '../util/getElementType';
@@ -218,8 +217,7 @@ export default {
     // Elements to validate for alt text.
     const elementOptions = options.elements || DEFAULT_ELEMENTS;
     // Get custom components for just the elements that will be tested.
-    const customComponents = flatMap(
-      elementOptions,
+    const customComponents = elementOptions.flatMap(
       element => options[element],
     );
     const typesToValidate = new Set(

src/rules/media-has-caption.js

@@ -10,7 +10,6 @@
 
 import type { JSXElement, JSXOpeningElement, Node } from 'ast-types-flow';
 import { getProp, getLiteralPropValue } from 'jsx-ast-utils';
-import flatMap from 'array.prototype.flatmap';
 
 import type {
   ESLintConfig,
@@ -34,7 +33,7 @@ const schema = generateObjSchema({
 const isMediaType = (context, type) => {
   const options = context.options[0] || {};
   return MEDIA_TYPES.concat(
-    flatMap(MEDIA_TYPES, mediaType => options[mediaType]),
+    MEDIA_TYPES.flatMap(mediaType => options[mediaType]),
   ).some(typeToCheck => typeToCheck === type);
 };
 

src/util/isInteractiveElement.js

@@ -2,7 +2,6 @@
 import { dom, elementRoles, roles } from 'aria-query';
 import type { Node } from 'ast-types-flow';
 import { AXObjects, elementAXObjects } from 'axobject-query';
-import flatMap from 'array.prototype.flatmap';
 
 import attributesComparator from './attributesComparator';
 
@@ -47,16 +46,14 @@ const interactiveRoles = new Set(
     ),
 );
 
-const interactiveElementRoleSchemas = flatMap(
-  elementRoleEntries,
+const interactiveElementRoleSchemas = elementRoleEntries.flatMap(
   ([elementSchema, rolesArr]) =>
     rolesArr.some((role): boolean => interactiveRoles.has(role))
       ? [elementSchema]
       : [],
 );
 
-const nonInteractiveElementRoleSchemas = flatMap(
-  elementRoleEntries,
+const nonInteractiveElementRoleSchemas = elementRoleEntries.flatMap(
   ([elementSchema, rolesArr]) =>
     rolesArr.every((role): boolean => nonInteractiveRoles.has(role))
       ? [elementSchema]
@@ -67,8 +64,7 @@ const interactiveAXObjects = new Set(
   AXObjects.keys().filter(name => AXObjects.get(name).type === 'widget'),
 );
 
-const interactiveElementAXObjectSchemas = flatMap(
-  [...elementAXObjects],
+const interactiveElementAXObjectSchemas = [...elementAXObjects].flatMap(
   ([elementSchema, AXObjectsArr]) =>
     AXObjectsArr.every((role): boolean => interactiveAXObjects.has(role))
       ? [elementSchema]

src/util/isInteractiveRole.js

@@ -2,7 +2,6 @@
 import { roles as rolesMap } from 'aria-query';
 import type { Node } from 'ast-types-flow';
 import { getProp, getLiteralPropValue } from 'jsx-ast-utils';
-import flatMap from 'array.prototype.flatmap';
 
 const roles = rolesMap.keys();
 const interactiveRoles = roles.filter(
@@ -40,7 +39,7 @@ const isInteractiveRole = (
 
   let isInteractive = false;
   const normalizedValues = String(value).toLowerCase().split(' ');
-  const validRoles = flatMap(normalizedValues, (name: string) =>
+  const validRoles = normalizedValues.flatMap((name: string) =>
     roles.includes(name) ? [name] : [],
   );
   if (validRoles.length > 0) {

src/util/isNonInteractiveElement.js

@@ -3,7 +3,6 @@
 import { dom, elementRoles, roles } from 'aria-query';
 import { AXObjects, elementAXObjects } from 'axobject-query';
 import type { Node } from 'ast-types-flow';
-import flatMap from 'array.prototype.flatmap';
 
 import attributesComparator from './attributesComparator';
 
@@ -54,16 +53,14 @@ const interactiveRoles = new Set(
     ),
 );
 
-const interactiveElementRoleSchemas = flatMap(
-  elementRoleEntries,
+const interactiveElementRoleSchemas = elementRoleEntries.flatMap(
   ([elementSchema, rolesArr]) =>
     rolesArr.some((role): boolean => interactiveRoles.has(role))
       ? [elementSchema]
       : [],
 );
 
-const nonInteractiveElementRoleSchemas = flatMap(
-  elementRoleEntries,
+const nonInteractiveElementRoleSchemas = elementRoleEntries.flatMap(
   ([elementSchema, rolesArr]) =>
     rolesArr.every((role): boolean => nonInteractiveRoles.has(role))
       ? [elementSchema]
@@ -76,8 +73,7 @@ const nonInteractiveAXObjects = new Set(
   ),
 );
 
-const nonInteractiveElementAXObjectSchemas = flatMap(
-  [...elementAXObjects],
+const nonInteractiveElementAXObjectSchemas = [...elementAXObjects].flatMap(
   ([elementSchema, AXObjectsArr]) =>
     AXObjectsArr.every((role): boolean => nonInteractiveAXObjects.has(role))
       ? [elementSchema]

src/util/isNonInteractiveRole.js

@@ -3,7 +3,6 @@
 import { dom, roles as rolesMap } from 'aria-query';
 import type { Node } from 'ast-types-flow';
 import { getProp, getLiteralPropValue } from 'jsx-ast-utils';
-import flatMap from 'array.prototype.flatmap';
 
 const nonInteractiveRoles = rolesMap
   .keys()
@@ -47,7 +46,7 @@ const isNonInteractiveRole = (
 
   let isNonInteractive = false;
   const normalizedValues = String(role).toLowerCase().split(' ');
-  const validRoles = flatMap(normalizedValues, (name: string) =>
+  const validRoles = normalizedValues.flatMap((name: string) =>
     rolesMap.has(name) ? [name] : [],
   );
   if (validRoles.length > 0) {