From ec58b6579dd18dc465fb729dcf0d02f102e572f4 Mon Sep 17 00:00:00 2001
From: Rebecca Stevens <rebecca.stevens@outlook.co.nz>
Date: Fri, 27 Nov 2020 15:15:32 +1300
Subject: [PATCH] refactor: extract out defaultClone

---
 src/options.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/options.ts b/src/options.ts
index 83bfad4..48e3f88 100644
--- a/src/options.ts
+++ b/src/options.ts
@@ -27,7 +27,7 @@ export type FullOptions<O extends Options = Options> = FlattenAlias<{
 	readonly arrayMerge: O[`arrayMerge`] extends undefined
 		? typeof defaultArrayMerge
 		: NonNullable<O[`arrayMerge`]>
-	readonly clone: O[`arrayMerge`] extends undefined ? true : NonNullable<O[`clone`]>
+	readonly clone: O[`arrayMerge`] extends undefined ? typeof defaultClone : NonNullable<O[`clone`]>
 	readonly customMerge?: O[`customMerge`]
 	readonly isMergeable: O[`arrayMerge`] extends undefined
 		? typeof defaultIsMergeable
@@ -52,6 +52,8 @@ export type ObjectMerge<K = any> = (
 	key: K
 ) => ((target: any, source: any, options: FullOptions) => any) | undefined
 
+const defaultClone = true as const
+
 function defaultIsMergeable(value: unknown): value is Record<Property, unknown> | Array<unknown> {
 	return Array.isArray(value) || isPlainObj(value)
 }
@@ -82,7 +84,7 @@ export function getFullOptions<O extends Options>(options?: O): FullOptions<O> {
 	return {
 		arrayMerge: defaultArrayMerge,
 		isMergeable: defaultIsMergeable,
-		clone: true,
+		clone: defaultClone,
 		...overrides,
 		cloneUnlessOtherwiseSpecified,
 	} as unknown as FullOptions<O>