fix: clone arrays in merge utils

Netcentric · Jan 31, 2024 · 5c432f8 · 5c432f8
1 parent 60eebbf
commit 5c432f8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/utils/extendConfig.js b/utils/extendConfig.js
@@ -34,6 +34,7 @@ module.exports = (configPath, config) => {
  }
 
  // config merge
- const extendedConfig = merge(config, override);
+ const copyConfig = merge({}, config);
+ const extendedConfig = merge(copyConfig, override);
  return extendedConfig;
 };
diff --git a/utils/merge.js b/utils/merge.js
@@ -10,6 +10,8 @@ module.exports = function merge(original = {}, newObject) {
  && !Array.isArray(newObject[prop])
  && !(newObject[prop] instanceof RegExp)) {
  copy[prop] = merge(original[prop], newObject[prop]);
+ } else if (Array.isArray(newObject[prop])) {
+ copy[prop] = newObject[prop] ? [...newObject[prop]] : [...original[prop]];
  } else {
  copy[prop] = newObject[prop] || typeof newObject[prop] === 'boolean' ? newObject[prop] : original[prop];
  }