@warlock.js/cascade 4.9.0 → 4.9.1

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to `@warlock.js/cascade` are documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). `@warlock.js/*` packages are released in lockstep — every package shares the same version number, so a version below may list only the changes that affected this package.
 
+## 4.9.1
+
+### Fixed
+
+- `save({ merge })` silently dropped a `Date` written over a column that already held a `Date` — the dirty tracker's merge treated anything `typeof "object"` as mergeable and recursed into the `Date`, which has no own enumerable properties, so nothing was copied and the old value survived. The column never went dirty and `save()` returned `{ success: true, modifiedCount: 0 }` without issuing an `UPDATE`. Only plain objects deep-merge now; `Date`, `Map`, `Set`, `RegExp` and every other class instance replace, matching what `model.data` already did. Writing into an empty column always worked, so only overwrites were affected
+
 ## 4.9.0 - 2026-08-06
 
 ### Fixed

package/cjs/index.cjs

@@ -691,8 +691,17 @@ var DatabaseDirtyTracker = class {
 	/**
 	* Recursively merges source object into target object, performing a deep merge.
 	*
-	* For nested objects, the merge is recursive. For arrays and primitives, the source
-	* value replaces the target value. All values are cloned to prevent reference sharing.
+	* Only **plain** objects are merged recursively. Everything else — arrays,
+	* primitives, and class instances such as `Date` / `Map` / `Set` / `RegExp` —
+	* replaces the target value. All values are cloned to prevent reference sharing.
+	*
+	* The plain-object guard is load-bearing, not tidiness. A bare
+	* `typeof value === "object"` also matches a `Date`, and `Object.entries(date)`
+	* is `[]` — so merging a `Date` over a column that already held a `Date`
+	* recursed into it, copied nothing, and left the old value in the snapshot.
+	* The column then never went dirty and `save()` returned
+	* `{ success: true, modifiedCount: 0 }` without issuing an `UPDATE`. This is
+	* the same lesson `canBeFlatten` above already encodes.
 	*
 	* @param target - The object to merge into
 	* @param source - The object to merge from
@@ -700,7 +709,7 @@ var DatabaseDirtyTracker = class {
 	*/
 	mergeIntoRaw(target, source) {
 		for (const [key, value] of Object.entries(source)) {
-			if (value && typeof value === "object" && !Array.isArray(value) && target[key] && typeof target[key] === "object" && !Array.isArray(target[key])) {
+			if ((0, _mongez_supportive_is.isPlainObject)(value) && (0, _mongez_supportive_is.isPlainObject)(target[key])) {
 				this.mergeIntoRaw(target[key], value);
 				continue;
 			}