@strapi/admin 4.3.2 → 4.3.5

package/ee/admin/pages/SettingsPage/pages/Roles/ListPage/index.js

@@ -107,7 +107,7 @@ const useRoleActions = ({ getData, canCreate, canDelete, canUpdate }) => {
   };
 
   const onRoleDuplicate = useCallback(
-    id => {
+    (id) => {
       push(`/settings/roles/duplicate/${id}`);
     },
     [push]
@@ -115,7 +115,7 @@ const useRoleActions = ({ getData, canCreate, canDelete, canUpdate }) => {
 
   const handleNewRoleClick = () => push('/settings/roles/new');
 
-  const onRoleRemove = useCallback(roleId => {
+  const onRoleRemove = useCallback((roleId) => {
     dispatch({
       type: 'SET_ROLE_TO_DELETE',
       id: roleId,
@@ -124,10 +124,10 @@ const useRoleActions = ({ getData, canCreate, canDelete, canUpdate }) => {
     handleToggleModal();
   }, []);
 
-  const handleToggleModal = () => setIsWarningDeleteAllOpenend(prev => !prev);
+  const handleToggleModal = () => setIsWarningDeleteAllOpenend((prev) => !prev);
 
   const handleGoTo = useCallback(
-    id => {
+    (id) => {
       push(`/settings/roles/${id}`);
     },
     [push]
@@ -160,11 +160,11 @@ const useRoleActions = ({ getData, canCreate, canDelete, canUpdate }) => {
   );
 
   const getIcons = useCallback(
-    role => [
+    (role) => [
       ...(canCreate
         ? [
             {
-              onClick: e => handleClickDuplicate(e, role),
+              onClick: (e) => handleClickDuplicate(e, role),
               label: formatMessage({ id: 'app.utils.duplicate', defaultMessage: 'Duplicate' }),
               icon: <Duplicate />,
             },
@@ -182,7 +182,7 @@ const useRoleActions = ({ getData, canCreate, canDelete, canUpdate }) => {
       ...(canDelete
         ? [
             {
-              onClick: e => handleClickDelete(e, role),
+              onClick: (e) => handleClickDelete(e, role),
               label: formatMessage({ id: 'global.delete', defaultMessage: 'Delete' }),
               icon: <Trash />,
             },
@@ -339,7 +339,7 @@ const RoleListPage = () => {
               </Tr>
             </Thead>
             <Tbody>
-              {sortedRoles?.map(role => (
+              {sortedRoles?.map((role) => (
                 <BaseRoleRow
                   key={role.id}
                   id={role.id}

package/ee/admin/pages/SettingsPage/pages/Roles/ListPage/reducer.js

@@ -8,7 +8,7 @@ export const initialState = {
 };
 
 const reducer = (state, action) =>
-  produce(state, draftState => {
+  produce(state, (draftState) => {
     switch (action.type) {
       case 'ON_REMOVE_ROLES': {
         draftState.showModalConfirmButtonLoading = true;

package/ee/admin/pages/SettingsPage/utils/customRoutes.js

@@ -1,7 +1,7 @@
 const ssoRoutes = strapi.features.isEnabled(strapi.features.SSO)
   ? [
       {
-        Component: async () => {
+        async Component() {
           const component = await import(
             /* webpackChunkName: "sso-settings-page" */ '../SingleSignOn'
           );

package/ee/server/controllers/authentication/middlewares.js

@@ -90,7 +90,7 @@ const nonExistingUserScenario = (ctx, next) => async (profile, provider) => {
   return next();
 };
 
-const redirectWithAuth = ctx => {
+const redirectWithAuth = (ctx) => {
   const {
     params: { provider },
   } = ctx;

package/ee/server/controllers/authentication/utils.js

@@ -12,7 +12,7 @@ const getAdminStore = async () => strapi.store({ type: 'core', name: 'admin' });
 
 const getPrefixedRedirectUrls = () => {
   const { url: adminUrl } = strapi.config.get('admin');
-  const prefixUrl = url => `${adminUrl || '/admin'}${url}`;
+  const prefixUrl = (url) => `${adminUrl || '/admin'}${url}`;
 
   return mapValues(prefixUrl, PROVIDER_URLS_MAP);
 };

package/ee/server/controllers/role.js

@@ -6,7 +6,7 @@ const {
   validateRoleDeleteInput,
   validateRolesDeleteInput,
 } = require('../validation/role');
-const { getService } = require('../../../server//utils');
+const { getService } = require('../../../server/utils');
 const { validatedUpdatePermissionsInput } = require('../validation/permission');
 const { SUPER_ADMIN_CODE } = require('../../../server/services/constants');
 
@@ -39,7 +39,7 @@ module.exports = {
 
     const roles = await roleService.deleteByIds([id]);
 
-    const sanitizedRole = roles.map(role => roleService.sanitizeRole(role))[0] || null;
+    const sanitizedRole = roles.map((role) => roleService.sanitizeRole(role))[0] || null;
 
     return ctx.deleted({
       data: sanitizedRole,
@@ -96,8 +96,8 @@ module.exports = {
 
     const sanitizedPermissions = permissions.map(permissionService.sanitizePermission);
 
-    ctx.body = {
+    return ctx.send({
       data: sanitizedPermissions,
-    };
+    });
   },
 };

package/ee/server/services/passport/provider-registry.js

@@ -13,7 +13,7 @@ module.exports = () => {
     },
 
     registerMany(providers) {
-      providers.forEach(provider => {
+      providers.forEach((provider) => {
         this.register(provider);
       });
     },

package/ee/server/services/passport/sso.js

@@ -5,7 +5,7 @@ const createProviderRegistry = require('./provider-registry');
 
 const providerRegistry = createProviderRegistry();
 
-const getStrategyCallbackURL = providerName => `/admin/connect/${providerName}`;
+const getStrategyCallbackURL = (providerName) => `/admin/connect/${providerName}`;
 
 const syncProviderRegistryWithConfig = () => {
   const { providers = [] } = strapi.config.get('admin.auth', {});

package/ee/server/services/passport.js

@@ -18,7 +18,7 @@ const getPassportStrategies = () => {
   }
 
   const providers = sso.providerRegistry.getAll();
-  const strategies = providers.map(provider => provider.createStrategy(strapi));
+  const strategies = providers.map((provider) => provider.createStrategy(strapi));
 
   return [localStrategy, ...strategies];
 };

package/ee/server/services/role.js

@@ -3,7 +3,7 @@
 const { toString } = require('lodash/fp');
 const { ApplicationError } = require('@strapi/utils').errors;
 
-const ssoCheckRolesIdForDeletion = async ids => {
+const ssoCheckRolesIdForDeletion = async (ids) => {
   const adminStore = await strapi.store({ type: 'core', name: 'admin' });
 
   const {

package/ee/server/validation/authentication.js

@@ -7,7 +7,7 @@ const providerOptionsUpdateSchema = yup.object().shape({
   defaultRole: yup
     .strapiID()
     .required()
-    .test('is-valid-role', 'You must submit a valid default role', roleId => {
+    .test('is-valid-role', 'You must submit a valid default role', (roleId) => {
       return strapi.admin.services.role.exists({ id: roleId });
     }),
 });

package/ee/server/validation/role.js

@@ -7,10 +7,7 @@ const { features } = require('@strapi/strapi/lib/utils/ee');
 const roleCreateSchema = yup
   .object()
   .shape({
-    name: yup
-      .string()
-      .min(1)
-      .required(),
+    name: yup.string().min(1).required(),
     description: yup.string().nullable(),
   })
   .noUnknown();
@@ -23,7 +20,7 @@ const rolesDeleteSchema = yup
       .of(yup.strapiID())
       .min(1)
       .required()
-      .test('roles-deletion-checks', 'Roles deletion checks have failed', async function(ids) {
+      .test('roles-deletion-checks', 'Roles deletion checks have failed', async function (ids) {
         try {
           await strapi.admin.services.role.checkRolesIdForDeletion(ids);
 
@@ -42,7 +39,7 @@ const rolesDeleteSchema = yup
 const roleDeleteSchema = yup
   .strapiID()
   .required()
-  .test('no-admin-single-delete', 'Role deletion checks have failed', async function(id) {
+  .test('no-admin-single-delete', 'Role deletion checks have failed', async function (id) {
     try {
       await strapi.admin.services.role.checkRolesIdForDeletion([id]);
 

package/ee/server/validation/user.js

@@ -12,7 +12,7 @@ const ssoUserCreationInputExtension = yup
   })
   .noUnknown();
 
-const validateUserCreationInput = data => {
+const validateUserCreationInput = (data) => {
   let schema = schemas.userCreationSchema;
 
   if (features.isEnabled('sso')) {

package/env.js

@@ -12,9 +12,9 @@ if (fs.existsSync(dotenvFilePath)) {
 
 const STRAPI_ADMIN = /^STRAPI_ADMIN_/i;
 
-const getClientEnvironment = options => {
+const getClientEnvironment = (options) => {
   const raw = Object.keys(process.env)
-    .filter(key => STRAPI_ADMIN.test(key))
+    .filter((key) => STRAPI_ADMIN.test(key))
     .reduce(
       (acc, current) => {
         acc[current] = process.env[current];

package/index.js

@@ -36,7 +36,7 @@ async function build({ appDir, buildDestDir, env, forceBuild, optimize, options,
     ceRoot: path.resolve(cacheDir, 'admin', 'src'),
   };
 
-  const pluginsPath = Object.keys(plugins).map(pluginName => plugins[pluginName].pathToPlugin);
+  const pluginsPath = Object.keys(plugins).map((pluginName) => plugins[pluginName].pathToPlugin);
 
   // Either use the tsconfig file from the generated app or the one inside the .cache folder
   // so we can develop plugins in TS while being in a JS app
@@ -111,7 +111,7 @@ async function watchAdmin({ appDir, browser, buildDestDir, host, options, plugin
     ceRoot: path.resolve(cacheDir, 'admin', 'src'),
   };
 
-  const pluginsPath = Object.keys(plugins).map(pluginName => plugins[pluginName].pathToPlugin);
+  const pluginsPath = Object.keys(plugins).map((pluginName) => plugins[pluginName].pathToPlugin);
 
   // Either use the tsconfig file from the generated app or the one inside the .cache folder
   // so we can develop plugins in TS while being in a JS app

package/jest.config.front.js

@@ -1,10 +1,12 @@
 'use strict';
 
 const baseConfig = require('../../../jest.base-config.front');
-const pkg = require('./package');
+const pkg = require('./package.json');
 
 module.exports = {
   ...baseConfig,
   displayName: (pkg.strapi && pkg.strapi.name) || pkg.name,
   roots: [__dirname],
+  collectCoverageFrom: ['<rootDir>/packages/core/admin/admin/**/*.js'],
+  coverageDirectory: '<rootDir>/packages/core/admin/coverage',
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@strapi/admin",
-  "version": "4.3.2",
+  "version": "4.3.5",
   "description": "Strapi Admin",
   "repository": {
     "type": "git",
@@ -34,30 +34,30 @@
     "test:front": "cross-env IS_EE=true jest --config ./jest.config.front.js",
     "test:front:watch": "cross-env IS_EE=true jest --config ./jest.config.front.js --watchAll",
     "test:front:ce": "cross-env IS_EE=false jest --config ./jest.config.front.js",
-    "test:front:watch:ce": "cross-env IS_EE=false jest --config ./jest.config.front.js --watchAll"
+    "test:front:watch:ce": "cross-env IS_EE=false jest --config ./jest.config.front.js --watchAll",
+    "test:front:ce:cov": "cross-env IS_EE=false jest --config ./jest.config.front.js --coverage"
   },
   "dependencies": {
-    "@babel/core": "7.16.7",
-    "@babel/plugin-transform-runtime": "7.16.7",
-    "@babel/polyfill": "7.12.1",
-    "@babel/preset-env": "7.16.7",
+    "@babel/core": "7.18.10",
+    "@babel/plugin-transform-runtime": "7.18.10",
+    "@babel/preset-env": "7.18.10",
     "@babel/preset-react": "7.18.6",
-    "@babel/runtime": "7.16.7",
+    "@babel/runtime": "7.18.9",
     "@casl/ability": "^5.4.3",
     "@fingerprintjs/fingerprintjs": "3.3.3",
     "@fortawesome/fontawesome-free": "^5.15.3",
-    "@fortawesome/fontawesome-svg-core": "^1.2.35",
+    "@fortawesome/fontawesome-svg-core": "6.1.2",
     "@fortawesome/free-brands-svg-icons": "^5.15.3",
     "@fortawesome/free-solid-svg-icons": "^5.15.3",
     "@fortawesome/react-fontawesome": "^0.2.0",
     "@pmmmwh/react-refresh-webpack-plugin": "0.5.7",
-    "@strapi/babel-plugin-switch-ee-ce": "4.3.2",
-    "@strapi/design-system": "1.2.0",
-    "@strapi/helper-plugin": "4.3.2",
-    "@strapi/icons": "1.2.0",
-    "@strapi/typescript-utils": "4.3.2",
-    "@strapi/utils": "4.3.2",
-    "axios": "0.24.0",
+    "@strapi/babel-plugin-switch-ee-ce": "4.3.5",
+    "@strapi/design-system": "1.2.1",
+    "@strapi/helper-plugin": "4.3.5",
+    "@strapi/icons": "1.2.1",
+    "@strapi/typescript-utils": "4.3.5",
+    "@strapi/utils": "4.3.5",
+    "axios": "0.27.2",
     "babel-loader": "8.2.5",
     "babel-plugin-styled-components": "2.0.2",
     "bcryptjs": "2.4.3",
@@ -66,7 +66,7 @@
     "codemirror": "^5.65.6",
     "cross-env": "^7.0.3",
     "css-loader": "6.7.1",
-    "date-fns": "2.28.0",
+    "date-fns": "2.29.2",
     "dotenv": "8.5.1",
     "esbuild-loader": "^2.19.0",
     "execa": "^1.0.0",
@@ -100,7 +100,7 @@
     "markdown-it-sup": "1.0.0",
     "match-sorter": "^4.0.2",
     "mini-css-extract-plugin": "2.4.4",
-    "node-polyfill-webpack-plugin": "1.1.4",
+    "node-polyfill-webpack-plugin": "2.0.1",
     "p-map": "4.0.0",
     "passport-local": "1.0.0",
     "prop-types": "^15.7.2",
@@ -113,7 +113,8 @@
     "react-error-boundary": "3.1.1",
     "react-fast-compare": "^3.2.0",
     "react-helmet": "^6.1.0",
-    "react-intl": "5.20.2",
+    "react-intl": "5.25.1",
+    "react-is": "^17.0.2",
     "react-query": "3.24.3",
     "react-redux": "7.2.8",
     "react-refresh": "0.11.0",
@@ -121,14 +122,14 @@
     "react-router-dom": "5.2.0",
     "react-window": "1.8.7",
     "redux": "^4.0.1",
-    "redux-saga": "^0.16.0",
     "reselect": "^4.0.0",
     "rimraf": "3.0.2",
-    "sanitize-html": "2.7.0",
+    "sanitize-html": "2.7.1",
     "semver": "7.3.7",
     "sift": "13.5.4",
     "style-loader": "3.3.1",
     "styled-components": "5.3.3",
+    "typescript": "4.6.2",
     "webpack": "^5.73.0",
     "webpack-cli": "^4.10.0",
     "webpack-dev-server": "^4.9.3",
@@ -136,11 +137,19 @@
     "yup": "^0.32.9"
   },
   "devDependencies": {
+    "@testing-library/dom": "8.17.1",
+    "@testing-library/react": "11.2.7",
+    "@testing-library/react-hooks": "3.7.0",
+    "@testing-library/user-event": "13.5.0",
     "duplicate-dependencies-webpack-plugin": "^1.0.2",
     "glob": "8.0.3",
+    "react-test-renderer": "^17.0.2",
     "speed-measure-webpack-plugin": "1.5.0",
     "webpack-bundle-analyzer": "^4.5.0"
   },
+  "peerDependencies": {
+    "@strapi/strapi": "^4.3.4"
+  },
   "engines": {
     "node": ">=14.19.1 <=16.x.x",
     "npm": ">=6.0.0"
@@ -154,5 +163,5 @@
       }
     }
   },
-  "gitHead": "7039c0d22836c94457130515b3a55eb93d2411f8"
+  "gitHead": "a52b0535513ff35b4ef46c9daf7be48f2ba71737"
 }

package/scripts/build.js

@@ -3,7 +3,7 @@
 const path = require('path');
 const webpack = require('webpack');
 const { isObject } = require('lodash');
-// eslint-disable-next-line node/no-extraneous-require
+// eslint-disable-next-line import/no-extraneous-dependencies
 const SpeedMeasurePlugin = require('speed-measure-webpack-plugin');
 
 const webpackConfig = require('../webpack.config');
@@ -103,7 +103,7 @@ buildAdmin()
   .then(() => {
     process.exit();
   })
-  .catch(err => {
+  .catch((err) => {
     console.error(err);
     process.exit(1);
   });

package/scripts/create-dev-plugins-file.js

@@ -2,7 +2,8 @@
 
 const { join, resolve, relative } = require('path');
 const { promisify } = require('util');
-// eslint-disable-next-line node/no-extraneous-require
+
+// eslint-disable-next-line import/no-extraneous-dependencies
 const glob = promisify(require('glob').glob);
 const fs = require('fs-extra');
 const { getCorePluginsPath, createPluginsFile } = require('./create-plugins-file');
@@ -16,14 +17,11 @@ const getPluginsPackages = async () => {
   const pluginsPackageDirs = await glob(pathToPackages);
 
   return pluginsPackageDirs
-    .filter(pluginDir => {
+    .filter((pluginDir) => {
       return fs.existsSync(join(pluginDir, 'admin', 'src', 'index.js'));
     })
     .reduce((acc, current) => {
-      const depName = current
-        .replace(/\\/g, '/')
-        .split('/')
-        .slice(-1)[0];
+      const depName = current.replace(/\\/g, '/').split('/').slice(-1)[0];
 
       const adminEntryPoint = join(__dirname, '..', 'admin', 'src');
 
@@ -63,7 +61,7 @@ createFile()
     console.log('plugins.js file created');
     process.exit();
   })
-  .catch(err => {
+  .catch((err) => {
     console.error(err);
     process.exit(1);
   });

package/scripts/create-plugins-file.js

@@ -30,7 +30,7 @@ const getCorePluginsPath = () => {
  * @param {Object[]} pluginsToInstall array of plugins located in the plugins folder
  * @returns Object of plugin's paths
  */
-const getPluginToInstallPath = pluginsToInstall => {
+const getPluginToInstallPath = (pluginsToInstall) => {
   const plugins = {};
 
   for (const dep of pluginsToInstall) {
@@ -49,7 +49,7 @@ const getPluginToInstallPath = pluginsToInstall => {
  * Write the plugins.js file
  * @param {Object} plugins
  */
-const createPluginsFile = async plugins => {
+const createPluginsFile = async (plugins) => {
   const pluginFileDest = resolve(__dirname, '..', 'admin', 'src', 'plugins.js');
 
   const allPluginsArray = Object.entries(plugins).map(([plugin, pluginPath]) => {

package/server/config/admin-conditions.js

@@ -6,17 +6,17 @@ module.exports = {
       displayName: 'Is creator',
       name: 'is-creator',
       plugin: 'admin',
-      handler: user => ({ 'createdBy.id': user.id }),
+      handler: (user) => ({ 'createdBy.id': user.id }),
     },
     {
       displayName: 'Has same role as creator',
       name: 'has-same-role-as-creator',
       plugin: 'admin',
-      handler: user => ({
+      handler: (user) => ({
         'createdBy.roles': {
           $elemMatch: {
             id: {
-              $in: user.roles.map(r => r.id),
+              $in: user.roles.map((r) => r.id),
             },
           },
         },

package/server/controllers/admin.js

@@ -21,7 +21,7 @@ const PLUGIN_NAME_REGEX = /^[A-Za-z][A-Za-z0-9-_]+$/;
 /**
  * Validates a plugin name format
  */
-const isValidPluginName = plugin => {
+const isValidPluginName = (plugin) => {
   return _.isString(plugin) && !_.isEmpty(plugin) && PLUGIN_NAME_REGEX.test(plugin);
 };
 

package/server/controllers/authentication.js

@@ -37,7 +37,7 @@ module.exports = {
         return next();
       })(ctx, next);
     },
-    ctx => {
+    (ctx) => {
       const { user } = ctx.state;
 
       ctx.body = {

package/server/controllers/formatters/format-actions-by-sections.js

@@ -5,7 +5,7 @@
  * @param {Array<Action>} actions - array of actions
  * @returns {Object} "{ contentTypes, plugins, settings }"
  */
-const formatActionsBySections = actions =>
+const formatActionsBySections = (actions) =>
   actions.reduce((result, p) => {
     const checkboxItem = {
       displayName: p.displayName,
@@ -13,18 +13,21 @@ const formatActionsBySections = actions =>
     };
 
     switch (p.section) {
-      case 'contentTypes':
+      case 'contentTypes': {
         checkboxItem.subjects = p.subjects;
         break;
-      case 'plugins':
+      }
+      case 'plugins': {
         checkboxItem.subCategory = p.subCategory;
         checkboxItem.plugin = `plugin::${p.pluginName}`;
         break;
-      case 'settings':
+      }
+      case 'settings': {
         checkboxItem.category = p.category;
         checkboxItem.subCategory = p.subCategory;
         break;
-      case 'default':
+      }
+      default:
         throw new Error(`Unknown section ${p.section}`);
     }
 

package/server/controllers/role.js

@@ -117,7 +117,7 @@ module.exports = {
     let permissionsToAssign;
 
     if ([EDITOR_CODE, AUTHOR_CODE].includes(role.code)) {
-      permissionsToAssign = input.permissions.map(permission => {
+      permissionsToAssign = input.permissions.map((permission) => {
         const action = actionProvider.get(permission.action);
 
         if (action.section !== 'contentTypes') {

package/server/controllers/user.js

@@ -51,7 +51,7 @@ module.exports = {
 
     ctx.body = {
       data: {
-        results: results.map(user => userService.sanitizeUser(user)),
+        results: results.map((user) => userService.sanitizeUser(user)),
         pagination,
       },
     };

package/server/controllers/webhooks.js

@@ -3,16 +3,14 @@
 const _ = require('lodash');
 const { yup, webhook: webhookUtils, validateYupSchema } = require('@strapi/utils');
 
-const urlRegex = /^(?:([a-z0-9+.-]+):\/\/)(?:\S+(?::\S*)?@)?(?:(?:[1-9]\d?|1\d\d|2[01]\d|22[0-3])(?:\.(?:1?\d{1,2}|2[0-4]\d|25[0-5])){2}(?:\.(?:[1-9]\d?|1\d\d|2[0-4]\d|25[0-4]))|(?:(?:[a-z\u00a1-\uffff0-9_]-*)*[a-z\u00a1-\uffff0-9_]+)(?:\.(?:[a-z\u00a1-\uffff0-9_]-*)*[a-z\u00a1-\uffff0-9_]+)*\.?)(?::\d{2,5})?(?:[/?#]\S*)?$/;
+const urlRegex =
+  /^(?:([a-z0-9+.-]+):\/\/)(?:\S+(?::\S*)?@)?(?:(?:[1-9]\d?|1\d\d|2[01]\d|22[0-3])(?:\.(?:1?\d{1,2}|2[0-4]\d|25[0-5])){2}(?:\.(?:[1-9]\d?|1\d\d|2[0-4]\d|25[0-4]))|(?:(?:[a-z\u00a1-\uffff0-9_]-*)*[a-z\u00a1-\uffff0-9_]+)(?:\.(?:[a-z\u00a1-\uffff0-9_]-*)*[a-z\u00a1-\uffff0-9_]+)*\.?)(?::\d{2,5})?(?:[/?#]\S*)?$/;
 
 const webhookValidator = yup
   .object({
     name: yup.string().required(),
-    url: yup
-      .string()
-      .matches(urlRegex, 'url must be a valid URL')
-      .required(),
-    headers: yup.lazy(data => {
+    url: yup.string().matches(urlRegex, 'url must be a valid URL').required(),
+    headers: yup.lazy((data) => {
       if (typeof data !== 'object') {
         return yup.object().required();
       }
@@ -20,20 +18,12 @@ const webhookValidator = yup
       return yup
         .object(
           _.mapValues(data, () => {
-            yup
-              .string()
-              .min(1)
-              .required();
+            yup.string().min(1).required();
           })
         )
         .required();
     }),
-    events: yup.array().of(
-      yup
-        .string()
-        .oneOf(_.values(webhookUtils.webhookEvents))
-        .required()
-    ),
+    events: yup.array().of(yup.string().oneOf(_.values(webhookUtils.webhookEvents)).required()),
   })
   .noUnknown();
 

package/server/domain/action/index.js

@@ -58,7 +58,7 @@ const sanitizeActionAttributes = pick(actionFields);
  * @param {CreateActionPayload} attributes
  * @return {string}
  */
-const computeActionId = attributes => {
+const computeActionId = (attributes) => {
   const { pluginName, uid } = attributes;
 
   if (!pluginName) {
@@ -77,14 +77,14 @@ const computeActionId = attributes => {
  * @param {CreateActionPayload} attrs - Payload used to create an action
  * @return {CreateActionPayload}
  */
-const assignActionId = attrs => set('actionId', computeActionId(attrs), attrs);
+const assignActionId = (attrs) => set('actionId', computeActionId(attrs), attrs);
 
 /**
  * Transform an action by adding or removing the {@link Action.subCategory} attribute
  * @param {Action} action - The action to process
  * @return {Action}
  */
-const assignOrOmitSubCategory = action => {
+const assignOrOmitSubCategory = (action) => {
   const shouldHaveSubCategory = ['settings', 'plugins'].includes(action.section);
 
   return shouldHaveSubCategory

package/server/domain/action/provider.js

@@ -14,7 +14,7 @@ const domain = require('./index');
  * Creates a new instance of an action provider
  * @return {Provider & ActionProviderOverride}
  */
-const createActionProvider = options => {
+const createActionProvider = (options) => {
   const provider = providerFactory(options);
   const actionHooks = {
     appliesPropertyToSubject: hooks.createAsyncParallelHook(),
@@ -75,7 +75,7 @@ const createActionProvider = options => {
         subject,
       });
 
-      return results.every(result => result !== false);
+      return results.every((result) => result !== false);
     },
   };
 };