From ceecfbba1a2f3b755e835fc1de4be7068bcc891f Mon Sep 17 00:00:00 2001
From: Nick the Sick <nicholas.perez@tiptap.dev>
Date: Wed, 12 Jun 2024 08:25:32 +0200
Subject: [PATCH] fix(core): mergeDeep handles nulls

---
 packages/core/src/utilities/mergeDeep.ts      | 10 ++-----
 .../integration/core/mergeDeep.spec.ts        | 30 +++++++++++++++++++
 2 files changed, 33 insertions(+), 7 deletions(-)

diff --git a/packages/core/src/utilities/mergeDeep.ts b/packages/core/src/utilities/mergeDeep.ts
index 4b54c3260d9..50befec51c0 100644
--- a/packages/core/src/utilities/mergeDeep.ts
+++ b/packages/core/src/utilities/mergeDeep.ts
@@ -5,14 +5,10 @@ export function mergeDeep(target: Record<string, any>, source: Record<string, an
 
   if (isPlainObject(target) && isPlainObject(source)) {
     Object.keys(source).forEach(key => {
-      if (isPlainObject(source[key])) {
-        if (!(key in target)) {
-          Object.assign(output, { [key]: source[key] })
-        } else {
-          output[key] = mergeDeep(target[key], source[key])
-        }
+      if (isPlainObject(source[key]) && isPlainObject(target[key])) {
+        output[key] = mergeDeep(target[key], source[key])
       } else {
-        Object.assign(output, { [key]: source[key] })
+        output[key] = source[key]
       }
     })
   }
diff --git a/tests/cypress/integration/core/mergeDeep.spec.ts b/tests/cypress/integration/core/mergeDeep.spec.ts
index 4e7e4979ee2..31e18e7416d 100644
--- a/tests/cypress/integration/core/mergeDeep.spec.ts
+++ b/tests/cypress/integration/core/mergeDeep.spec.ts
@@ -49,6 +49,36 @@ describe('mergeDeep', () => {
     expect(merged).to.deep.eq(result)
   })
 
+  it('should merge when source has null value', () => {
+    const one = {
+      a: null,
+    }
+    const two = {
+      a: { c: 3 },
+    }
+    const result = {
+      a: { c: 3 },
+    }
+    const merged = mergeDeep(one, two)
+
+    expect(merged).to.deep.eq(result)
+  })
+
+  it('should allow nulling a value', () => {
+    const one = {
+      a: { c: 3 },
+    }
+    const two = {
+      a: { c: null },
+    }
+    const result = {
+      a: { c: null },
+    }
+    const merged = mergeDeep(one, two)
+
+    expect(merged).to.deep.eq(result)
+  })
+
   it('should merge deep', () => {
     const one = {
       a: 1,