improve internal mergeAll function, closes #532

Author: gcanti

CHANGELOG.md

@@ -14,6 +14,11 @@
 **Note**: Gaps between patch versions are faulty/broken releases.
 **Note**: A feature tagged as Experimental is in a high state of flux, you're at risk of it changing without notice.
 
+# 2.2.13
+
+- **Bug Fix**
+  - improve internal `mergeAll` function, closes #532 (@gcanti)
+
 # 2.2.12
 
 - **Experimental**

package.json

@@ -1,6 +1,6 @@
 {
   "name": "io-ts",
-  "version": "2.2.12",
+  "version": "2.2.13",
   "description": "TypeScript runtime type system for IO decoding/encoding",
   "main": "lib/index.js",
   "module": "es6/index.js",

src/index.ts

@@ -1416,9 +1416,13 @@ export interface IntersectionC<CS extends [Mixed, Mixed, ...Array<Mixed>]>
     unknown
   > {}
 
-const mergeAll = (base: any, us: Array<any>): any => {
+/**
+ * @internal
+ */
+export const mergeAll = (base: any, us: Array<any>): any => {
   let equal = true
   let primitive = true
+  const baseIsNotADictionary = !UnknownRecord.is(base)
   for (const u of us) {
     if (u !== base) {
       equal = false
@@ -1435,7 +1439,7 @@ const mergeAll = (base: any, us: Array<any>): any => {
   let r: any = {}
   for (const u of us) {
     for (const k in u) {
-      if (u[k] !== base[k] || !r.hasOwnProperty(k)) {
+      if (!r.hasOwnProperty(k) || baseIsNotADictionary || u[k] !== base[k]) {
         r[k] = u[k]
       }
     }

test/2.1.x/intersection.ts

@@ -3,6 +3,13 @@ import * as t from '../../src/index'
 import { assertFailure, assertStrictEqual, assertSuccess, NumberFromString } from './helpers'
 
 describe('intersection', () => {
+  it('mergeAll', () => {
+    assert.deepStrictEqual(t.mergeAll(undefined, [{ prop1: 'b', prop2: 2 }, { prop1: 'a' }, { prop2: 1 }]), {
+      prop1: 'a',
+      prop2: 1
+    })
+  })
+
   describe('name', () => {
     it('should assign a default name', () => {
       const T = t.intersection([t.type({ a: t.string }), t.type({ b: t.number })])