[config] stop merging arrays from multiple configs

packages/kbn-config/src/__fixtures__/merge_1.yml

@@ -0,0 +1,10 @@
+foo: 1
+bar: 3
+arr1: [1, 2, 3]
+arr2: [42, 90]
+nested:
+  str1: "foo"
+  str2: "hello"
+obj_array:
+  - id: 1
+  - id: 2

packages/kbn-config/src/__fixtures__/merge_2.yml

@@ -0,0 +1,8 @@
+foo: 2
+arr1: [4, 5]
+arr2: []
+nested:
+  str1: "bar"
+  str3: "dolly"
+obj_array:
+  - id: 3

packages/kbn-config/src/raw/read_config.test.ts

@@ -59,6 +59,11 @@ test('throws parsing another config with forbidden paths', () => {
   ).toThrowErrorMatchingInlineSnapshot(`"Forbidden path detected: test.hello.__proto__"`);
 });
 
+test('merging two configs', () => {
+  const config = getConfigFromFiles([fixtureFile('merge_1.yml'), fixtureFile('merge_2.yml')]);
+  expect(config).toMatchSnapshot();
+});
+
 describe('different cwd()', () => {
   const originalCwd = process.cwd();
   const tempCwd = resolve(__dirname);

packages/kbn-config/src/raw/read_config.ts

@@ -27,17 +27,29 @@ function replaceEnvVarRefs(val: string) {
 }
 
 function merge(target: Record<string, any>, value: any, key?: string) {
-  if ((isPlainObject(value) || Array.isArray(value)) && Object.keys(value).length > 0) {
+  if (isPlainObject(value) && Object.keys(value).length > 0) {
     for (const [subKey, subVal] of Object.entries(value)) {
       merge(target, subVal, key ? `${key}.${subKey}` : subKey);
     }
   } else if (key !== undefined) {
-    set(target, key, typeof value === 'string' ? replaceEnvVarRefs(value) : value);
+    set(target, key, recursiveReplaceEnvVar(value));
   }
 
   return target;
 }
 
+function recursiveReplaceEnvVar(value: any) {
+  if (isPlainObject(value) || Array.isArray(value)) {
+    for (const [subKey, subVal] of Object.entries(value)) {
+      set(value, subKey, recursiveReplaceEnvVar(subVal));
+    }
+  }
+  if (typeof value === 'string') {
+    return replaceEnvVarRefs(value);
+  }
+  return value;
+}
+
 /** @internal */
 export const getConfigFromFiles = (configFiles: readonly string[]) => {
   let mergedYaml = {};