@@ -303,8 +303,7 @@ const all = [
   },
 ];
 
-module.exports = {};
-
-all.forEach((error) => {
-  module.exports[error.errno] = error.description;
-});
+module.exports = all.reduce((obj, error) => ({
+  ...obj,
+  [error.errno]: error.description,
+}));
@@ -98,26 +98,23 @@ async function processDir(dirPath, opts) {
         return arr;
       }
 
-      if (value.constructor === Array) {
-        return arr.concat(value);
+      if (Array.isArray(value)) {
+        return [...arr, ...value];
       }
 
-      arr.push(value);
-      return arr;
+      return [...arr, value];
     }, [{ dirPath, success: true }]);
   } catch (err) {
     return [{ dirPath, err, success: false }];
   }
 }
 
-async function removeNPMAbsolutePaths(filePath, opts) {
-  opts = opts || {}; // eslint-disable-line no-param-reassign
-
+async function removeNPMAbsolutePaths(filePath, opts = {}) {
   if (!filePath) {
     throw new ProcessingError('Missing path.\nThe first argument should be the path to a directory or a package.json file.');
   }
 
-  if (opts.fields && (opts.fields.constructor !== Array || opts.fields.length === 0)) {
+  if (opts.fields && (!Array.isArray(opts.fields) || opts.fields.length === 0)) {
     throw new ProcessingError('Invalid option: fields.\nThe fields option should be an array cotaining the names of the specific fields that should be removed.');
   }
 
 
@@ -52,7 +52,7 @@ describe('cli', function () {
       });
 
       it('set the fields option if --field flag and some fields are passed', function () {
-        const args = [path, '--fields'].concat(testFields);
+        const args = [path, '--fields', ...testFields];
         const parsedArgs = cli.parseArguments(args);
         expect(parsedArgs.path).to.equal(path);
         expect(parsedArgs.opts.fields).to.deep.equal(testFields);
@@ -61,17 +61,17 @@ describe('cli', function () {
     });
 
     it('ignore unknown arguments', function () {
-      const args1 = [path, unknownArg1, '--force', unknownArg2, '--fields'].concat(testFields);
-      const args2 = [path, '--fields'].concat(testFields).concat(['--force', unknownArg1, unknownArg2]);
+      const args1 = [path, unknownArg1, '--force', unknownArg2, '--fields', ...testFields];
+      const args2 = [path, '--fields', ...testFields, '--force', unknownArg1, unknownArg2];
       const parsedArgs1 = cli.parseArguments(args1);
       const parsedArgs2 = cli.parseArguments(args2);
       expect(parsedArgs1.ignored).to.deep.equal(parsedArgs2.ignored);
       expect(parsedArgs1.ignored).to.deep.equal([unknownArg1, unknownArg2]);
     });
 
     it('produce the same result regardless of the arguments passed', function () {
-      const args1 = [path, unknownArg1, '--force', unknownArg2, '--fields'].concat(testFields);
-      const args2 = [path, '--fields'].concat(testFields).concat(['--force', unknownArg1, unknownArg2]);
+      const args1 = [path, unknownArg1, '--force', unknownArg2, '--fields', ...testFields];
+      const args2 = [path, '--fields', ...testFields, '--force', unknownArg1, unknownArg2];
       const parsedArgs1 = cli.parseArguments(args1);
       const parsedArgs2 = cli.parseArguments(args2);
       expect(parsedArgs1).to.deep.equal(parsedArgs2);
 
@@ -209,7 +209,7 @@ describe('removeNPMAbsolutePaths.js', function () {
         });
 
         it('return error if fields option is passed but is empty', function () {
-          const dirPath = path.join(__dirname, 'data' ,'underscore_fields');
+          const dirPath = path.join(__dirname, 'data', 'underscore_fields');
           const opts = {
             fields: [],
           };
@@ -324,7 +324,7 @@ describe('removeNPMAbsolutePaths.js', function () {
 
       describe('fields', function () {
         it('return error if fields option is passed but is not an array', function () {
-          const filePath = path.join(__dirname, 'data' , 'underscore_fields' , 'module' , 'package.json');
+          const filePath = path.join(__dirname, 'data', 'underscore_fields', 'module', 'package.json');
           const opts = {
             fields: 'string_value',
           };
Original file line number	Diff line number	Diff line change
`@@ -98,26 +98,23 @@ async function processDir(dirPath, opts) {`
`98`	`98`	`return arr;`
`99`	`99`	`}`
`100`	`100`
`101`		`- if (value.constructor === Array) {`
`102`		`- return arr.concat(value);`
	`101`	`+ if (Array.isArray(value)) {`
	`102`	`+ return [...arr, ...value];`
`103`	`103`	`}`
`104`	`104`
`105`		`- arr.push(value);`
`106`		`- return arr;`
	`105`	`+ return [...arr, value];`
`107`	`106`	`}, [{ dirPath, success: true }]);`
`108`	`107`	`} catch (err) {`
`109`	`108`	`return [{ dirPath, err, success: false }];`
`110`	`109`	`}`
`111`	`110`	`}`
`112`	`111`
`113`		`-async function removeNPMAbsolutePaths(filePath, opts) {`
`114`		`- opts = opts \|\| {}; // eslint-disable-line no-param-reassign`
`115`		`-`
	`112`	`+async function removeNPMAbsolutePaths(filePath, opts = {}) {`
`116`	`113`	`if (!filePath) {`
`117`	`114`	`throw new ProcessingError('Missing path.\nThe first argument should be the path to a directory or a package.json file.');`
`118`	`115`	`}`
`119`	`116`
`120`		`- if (opts.fields && (opts.fields.constructor !== Array \|\| opts.fields.length === 0)) {`
	`117`	`+ if (opts.fields && (!Array.isArray(opts.fields) \|\| opts.fields.length === 0)) {`
`121`	`118`	`throw new ProcessingError('Invalid option: fields.\nThe fields option should be an array cotaining the names of the specific fields that should be removed.');`
`122`	`119`	`}`
`123`	`120`