bunshin-clone 1.7.1 → 1.8.0

package/README.md

@@ -36,6 +36,7 @@ bunshinClone<T>(value, options);
 interface BunshinCloneOptions {
   preserveBufferSharing: boolean; // default: false
   preserveDescriptors: boolean;   // default: false
+  preserveSymbolKeys: boolean;    // default: false
   strictDescriptors: boolean;     // default: false
 }
 ```
@@ -48,9 +49,13 @@ If `true`, preserves shared backing buffers between TypedArray and DataView inst
 
 If `true`, preserves property descriptors (getters/setters, etc.).
 
+### `preserveSymbolKeys`
+
+If `true`, preserves symbol keys (slower).
+
 ### `strictDescriptors`
 
-If `true`, throws if descriptor cannot be merged (e.g. non-configurable or non-writable)
+If `true`, throws if descriptor cannot be merged (e.g. non-configurable or non-writable).
 
 ## 📖 Details
 

package/dist/index.cjs

@@ -2,7 +2,7 @@
 
 // src/index.ts
 var EMPTY_OPTIONS = {};
-var { propertyIsEnumerable: IS_ENUMERABLE } = Object.prototype;
+var { hasOwnProperty: HAS_OWN, propertyIsEnumerable: IS_ENUMERABLE } = Object.prototype;
 function bunshinClone(value, options = EMPTY_OPTIONS, refs = /* @__PURE__ */ new WeakMap()) {
   return clone(value, resolveOptions(options), refs);
 }
@@ -18,7 +18,7 @@ function clone(value, settings, refs) {
     return cloneWithDescriptors(value, settings, refs);
   }
   if (Array.isArray(value)) {
-    if (isShallowClonableArray(value)) {
+    if (isShallowArray(value)) {
       const result2 = value.slice();
       refs.set(value, result2);
       return result2;
@@ -34,11 +34,20 @@ function clone(value, settings, refs) {
   if (isPlainObject(value)) {
     const result = Object.create(Object.getPrototypeOf(value));
     refs.set(value, result);
-    for (const key of Reflect.ownKeys(value)) {
-      if (isUnsafeKey(key) || !IS_ENUMERABLE.call(value, key)) {
-        continue;
+    if (!settings.preserveSymbolKeys) {
+      for (const key in value) {
+        if (isUnsafeKey(key) || !HAS_OWN.call(value, key)) {
+          continue;
+        }
+        result[key] = clone(value[key], settings, refs);
+      }
+    } else {
+      for (const key of Reflect.ownKeys(value)) {
+        if (isUnsafeKey(key) || !IS_ENUMERABLE.call(value, key)) {
+          continue;
+        }
+        result[key] = clone(value[key], settings, refs);
       }
-      result[key] = clone(value[key], settings, refs);
     }
     return result;
   }
@@ -209,14 +218,6 @@ function createErrorInstance(error, settings, refs) {
   result.name = name;
   return result;
 }
-function isShallowClonableArray(array) {
-  for (let i = 0, l = array.length; i < l; i++) {
-    if (isObject(array[i])) {
-      return false;
-    }
-  }
-  return true;
-}
 function forEachOwnKey(object, callback) {
   for (const key of Object.keys(object)) {
     callback(key);
@@ -235,6 +236,14 @@ function isPlainObject(value) {
   const proto = Object.getPrototypeOf(value);
   return proto === Object.prototype || proto === null;
 }
+function isShallowArray(array) {
+  for (let i = 0, l = array.length; i < l; i++) {
+    if (isObject(array[i])) {
+      return false;
+    }
+  }
+  return true;
+}
 function isUnsafeKey(key) {
   return typeof key === "string" && (key === "__proto__" || key === "prototype" || key === "constructor");
 }
@@ -242,8 +251,13 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = false,
-    strictDescriptors = false
+    strictDescriptors = false,
+    preserveSymbolKeys = false
   } = options;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
+  }
   if (typeof preserveBufferSharing !== "boolean") {
     console.warn("Invalid preserveBufferSharing option. Fallback: false.");
     preserveBufferSharing = false;
@@ -256,11 +270,17 @@ function resolveOptions(options) {
     console.warn("Invalid strictDescriptors option. Fallback: false.");
     strictDescriptors = false;
   }
-  return { preserveBufferSharing, preserveDescriptors, strictDescriptors };
+  return {
+    preserveBufferSharing,
+    preserveDescriptors,
+    preserveSymbolKeys,
+    strictDescriptors
+  };
 }
 
 exports.bunshinClone = bunshinClone;
 exports.forEachOwnKey = forEachOwnKey;
 exports.isObject = isObject;
 exports.isPlainObject = isPlainObject;
+exports.isShallowArray = isShallowArray;
 exports.isUnsafeKey = isUnsafeKey;

package/dist/index.d.cts

@@ -1,6 +1,7 @@
 interface BunshinCloneOptions {
     preserveBufferSharing: boolean;
     preserveDescriptors: boolean;
+    preserveSymbolKeys: boolean;
     strictDescriptors: boolean;
 }
 type PlainObject = Record<PropertyKey, unknown>;
@@ -9,6 +10,7 @@ declare function bunshinClone<T>(value: T, options?: Partial<BunshinCloneOptions
 declare function forEachOwnKey(object: PlainObject, callback: (key: string | symbol) => void): void;
 declare function isObject(value: unknown): value is object;
 declare function isPlainObject(value: unknown): value is PlainObject;
+declare function isShallowArray(array: unknown[]): boolean;
 declare function isUnsafeKey(key: PropertyKey): boolean;
 
-export { type BunshinCloneOptions, bunshinClone, forEachOwnKey, isObject, isPlainObject, isUnsafeKey };
+export { type BunshinCloneOptions, bunshinClone, forEachOwnKey, isObject, isPlainObject, isShallowArray, isUnsafeKey };

package/dist/index.d.ts

@@ -1,6 +1,7 @@
 interface BunshinCloneOptions {
     preserveBufferSharing: boolean;
     preserveDescriptors: boolean;
+    preserveSymbolKeys: boolean;
     strictDescriptors: boolean;
 }
 type PlainObject = Record<PropertyKey, unknown>;
@@ -9,6 +10,7 @@ declare function bunshinClone<T>(value: T, options?: Partial<BunshinCloneOptions
 declare function forEachOwnKey(object: PlainObject, callback: (key: string | symbol) => void): void;
 declare function isObject(value: unknown): value is object;
 declare function isPlainObject(value: unknown): value is PlainObject;
+declare function isShallowArray(array: unknown[]): boolean;
 declare function isUnsafeKey(key: PropertyKey): boolean;
 
-export { type BunshinCloneOptions, bunshinClone, forEachOwnKey, isObject, isPlainObject, isUnsafeKey };
+export { type BunshinCloneOptions, bunshinClone, forEachOwnKey, isObject, isPlainObject, isShallowArray, isUnsafeKey };

package/dist/index.js

@@ -1,6 +1,6 @@
 // src/index.ts
 var EMPTY_OPTIONS = {};
-var { propertyIsEnumerable: IS_ENUMERABLE } = Object.prototype;
+var { hasOwnProperty: HAS_OWN, propertyIsEnumerable: IS_ENUMERABLE } = Object.prototype;
 function bunshinClone(value, options = EMPTY_OPTIONS, refs = /* @__PURE__ */ new WeakMap()) {
   return clone(value, resolveOptions(options), refs);
 }
@@ -16,7 +16,7 @@ function clone(value, settings, refs) {
     return cloneWithDescriptors(value, settings, refs);
   }
   if (Array.isArray(value)) {
-    if (isShallowClonableArray(value)) {
+    if (isShallowArray(value)) {
       const result2 = value.slice();
       refs.set(value, result2);
       return result2;
@@ -32,11 +32,20 @@ function clone(value, settings, refs) {
   if (isPlainObject(value)) {
     const result = Object.create(Object.getPrototypeOf(value));
     refs.set(value, result);
-    for (const key of Reflect.ownKeys(value)) {
-      if (isUnsafeKey(key) || !IS_ENUMERABLE.call(value, key)) {
-        continue;
+    if (!settings.preserveSymbolKeys) {
+      for (const key in value) {
+        if (isUnsafeKey(key) || !HAS_OWN.call(value, key)) {
+          continue;
+        }
+        result[key] = clone(value[key], settings, refs);
+      }
+    } else {
+      for (const key of Reflect.ownKeys(value)) {
+        if (isUnsafeKey(key) || !IS_ENUMERABLE.call(value, key)) {
+          continue;
+        }
+        result[key] = clone(value[key], settings, refs);
       }
-      result[key] = clone(value[key], settings, refs);
     }
     return result;
   }
@@ -207,14 +216,6 @@ function createErrorInstance(error, settings, refs) {
   result.name = name;
   return result;
 }
-function isShallowClonableArray(array) {
-  for (let i = 0, l = array.length; i < l; i++) {
-    if (isObject(array[i])) {
-      return false;
-    }
-  }
-  return true;
-}
 function forEachOwnKey(object, callback) {
   for (const key of Object.keys(object)) {
     callback(key);
@@ -233,6 +234,14 @@ function isPlainObject(value) {
   const proto = Object.getPrototypeOf(value);
   return proto === Object.prototype || proto === null;
 }
+function isShallowArray(array) {
+  for (let i = 0, l = array.length; i < l; i++) {
+    if (isObject(array[i])) {
+      return false;
+    }
+  }
+  return true;
+}
 function isUnsafeKey(key) {
   return typeof key === "string" && (key === "__proto__" || key === "prototype" || key === "constructor");
 }
@@ -240,8 +249,13 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = false,
-    strictDescriptors = false
+    strictDescriptors = false,
+    preserveSymbolKeys = false
   } = options;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
+  }
   if (typeof preserveBufferSharing !== "boolean") {
     console.warn("Invalid preserveBufferSharing option. Fallback: false.");
     preserveBufferSharing = false;
@@ -254,7 +268,12 @@ function resolveOptions(options) {
     console.warn("Invalid strictDescriptors option. Fallback: false.");
     strictDescriptors = false;
   }
-  return { preserveBufferSharing, preserveDescriptors, strictDescriptors };
+  return {
+    preserveBufferSharing,
+    preserveDescriptors,
+    preserveSymbolKeys,
+    strictDescriptors
+  };
 }
 
-export { bunshinClone, forEachOwnKey, isObject, isPlainObject, isUnsafeKey };
+export { bunshinClone, forEachOwnKey, isObject, isPlainObject, isShallowArray, isUnsafeKey };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunshin-clone",
-  "version": "1.7.1",
+  "version": "1.8.0",
   "description": "High-performance deep clone utility with descriptor support",
   "keywords": [
     "array",