.eslintrc.js

@@ -25,7 +25,6 @@ module.exports = {
     'fp/no-this': 0,
     'import/max-dependencies': 0,
     'node/no-sync': 0,
-    'unicorn/prefer-spread': 0,
     'unicorn/consistent-destructuring': 0,
     // TODO: harmonize with filename snake_case in other Netlify Dev projects
     'unicorn/filename-case': [2, { case: 'kebabCase' }],

CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+### [4.4.1](https://www.github.com/netlify/cli/compare/v4.4.0...v4.4.1) (2021-07-15)
+
+
+### Bug Fixes
+
+* enable and fix unicorn/prefer-spread ([#2943](https://www.github.com/netlify/cli/issues/2943)) ([77c6ebb](https://www.github.com/netlify/cli/commit/77c6ebbbbb3608312183d1654348251cc7aeca97))
+
 ## [4.4.0](https://www.github.com/netlify/cli/compare/v4.3.3...v4.4.0) (2021-07-15)
 
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "netlify-cli",
   "description": "Netlify command line tool",
-  "version": "4.4.0",
+  "version": "4.4.1",
   "author": "Netlify Inc.",
   "contributors": [
     "Mathias Biilmann <matt@netlify.com> (https://twitter.com/biilmann)",

src/commands/dev/trace.js

@@ -12,7 +12,7 @@ class TraceCommand extends Command {
   async run() {
     this.parse(TraceCommand)
 
-    const args = ['trace'].concat(this.argv)
+    const args = ['trace', ...this.argv]
     const { subprocess } = runProcess({ log: this.log, args })
     await subprocess
   }

src/utils/addons/compare.js

@@ -12,14 +12,14 @@ module.exports = function compare(oldValues, newValues) {
 
   const oldKeys = Object.keys(oldValues)
   const newKeys = Object.keys(newValues)
-  const set = new Set(newKeys.concat(oldKeys))
+  const set = new Set([...newKeys, ...oldKeys])
 
   return [...set].reduce((acc, current) => {
     // if values not deep equal. There are changes
     if (!isEqual(newValues[current], oldValues[current])) {
       return {
         isEqual: false,
-        keys: acc.keys.concat(current),
+        keys: [...acc.keys, current],
         diffs: {
           ...acc.diffs,
           [`${current}`]: {

src/utils/deploy/util.js

@@ -96,6 +96,7 @@ const waitForDeploy = async (api, deployId, siteId, timeout) => {
 const getUploadList = (required, shaMap) => {
   if (!required || !shaMap) return []
   // TODO: use `Array.flatMap()` instead once we remove support for Node <11.0.0
+  // eslint-disable-next-line unicorn/prefer-spread
   return [].concat(...required.map((sha) => shaMap[sha]))
 }
 

src/utils/detect-server-settings.js

@@ -310,15 +310,16 @@ const filterSettings = function (scriptInquirerOptions, input) {
 }
 
 const formatSettingsArrForInquirer = function (frameworks) {
-  return [].concat(
-    ...frameworks.map((framework) =>
-      framework.dev.commands.map((command) => ({
-        name: `[${chalk.yellow(framework.name)}] '${command}'`,
-        value: { ...framework, commands: [command] },
-        short: `${framework.name}-${command}`,
-      })),
-    ),
+  const formattedArr = frameworks.map((framework) =>
+    framework.dev.commands.map((command) => ({
+      name: `[${chalk.yellow(framework.name)}] '${command}'`,
+      value: { ...framework, commands: [command] },
+      short: `${framework.name}-${command}`,
+    })),
   )
+  // Replace by .flatMap() when Node.js support >= 11.0.0
+  // eslint-disable-next-line unicorn/prefer-spread
+  return [].concat(...formattedArr)
 }
 
 module.exports = {

src/utils/proxy.js

@@ -201,7 +201,7 @@ const serveRedirect = async function ({ req, res, proxy, match, options }) {
     // We pass through request params in one of the following cases:
     // 1. The redirect rule doesn't have any query params
     // 2. This is a function redirect https://github.com/netlify/cli/issues/1605
-    if (Array.from(dest.searchParams).length === 0 || isFunction(options.functionsPort, stripOrigin(dest))) {
+    if ([...dest.searchParams].length === 0 || isFunction(options.functionsPort, stripOrigin(dest))) {
       dest.searchParams.forEach((_, key) => {
         dest.searchParams.delete(key)
       })