bunshin-clone 1.8.13 → 1.8.15

package/README.md

@@ -47,7 +47,7 @@ If `true`, preserves shared backing buffers between TypedArray and DataView inst
 
 ### `preserveDescriptors`
 
-If `true`, preserves property descriptors (getters/setters, etc.).
+If `true`, preserves property descriptors, including non-enumerable properties.
 
 ### `preserveSymbolKeys`
 

package/dist/index.cjs

@@ -81,15 +81,18 @@ function clone(value, settings, refs) {
   refs.set(value, value);
   return value;
 }
-var IS_ENUMERABLE = Object.prototype.propertyIsEnumerable;
 var OWN_DESCS = Object.getOwnPropertyDescriptors;
-var OWN_NAMES = Object.getOwnPropertyNames;
-var OWN_SYMBOLS = Object.getOwnPropertySymbols;
+var OWN_ENUM_STRING_KEYS = Object.keys;
+var OWN_ENUM_SYMBOL_KEYS = (object) => OWN_SYMBOL_KEYS(object).filter(
+  (k) => Object.prototype.propertyIsEnumerable.call(object, k)
+);
+var OWN_STRING_KEYS = Object.getOwnPropertyNames;
+var OWN_SYMBOL_KEYS = Object.getOwnPropertySymbols;
 function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = OWN_DESCS(object);
-  for (const key of OWN_NAMES(object)) {
+  for (const key of OWN_STRING_KEYS(object)) {
     if (isUnsafeKey(key)) {
       continue;
     }
@@ -106,13 +109,13 @@ function cloneWithDescriptors(object, settings, refs) {
     }
   }
   if (settings.preserveSymbolKeys) {
-    for (const symbol of OWN_SYMBOLS(object)) {
-      const desc = { ...descs[symbol] };
+    for (const key of OWN_SYMBOL_KEYS(object)) {
+      const desc = { ...descs[key] };
       if ("value" in desc) {
         desc.value = clone(desc.value, settings, refs);
       }
       try {
-        Object.defineProperty(result, symbol, desc);
+        Object.defineProperty(result, key, desc);
       } catch (error) {
         if (settings.strictDescriptors) {
           throw error;
@@ -130,16 +133,14 @@ function cloneArray(array, settings, refs) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  for (const key of Object.keys(object)) {
+  for (const key of OWN_ENUM_STRING_KEYS(object)) {
     if (!isUnsafeKey(key)) {
       result[key] = clone(object[key], settings, refs);
     }
   }
   if (settings.preserveSymbolKeys) {
-    for (const symbol of OWN_SYMBOLS(object)) {
-      if (IS_ENUMERABLE.call(object, symbol)) {
-        result[symbol] = clone(object[symbol], settings, refs);
-      }
+    for (const key of OWN_ENUM_SYMBOL_KEYS(object)) {
+      result[key] = clone(object[key], settings, refs);
     }
   }
   return result;
@@ -212,7 +213,7 @@ function cloneError(error, settings, refs) {
   if ("cause" in error && cause !== void 0) {
     result.cause = clone(cause, settings, refs);
   }
-  for (const key of Object.keys(error)) {
+  for (const key of OWN_ENUM_STRING_KEYS(error)) {
     Reflect.set(result, key, clone(Reflect.get(error, key), settings, refs));
   }
   return result;

package/dist/index.js

@@ -79,15 +79,18 @@ function clone(value, settings, refs) {
   refs.set(value, value);
   return value;
 }
-var IS_ENUMERABLE = Object.prototype.propertyIsEnumerable;
 var OWN_DESCS = Object.getOwnPropertyDescriptors;
-var OWN_NAMES = Object.getOwnPropertyNames;
-var OWN_SYMBOLS = Object.getOwnPropertySymbols;
+var OWN_ENUM_STRING_KEYS = Object.keys;
+var OWN_ENUM_SYMBOL_KEYS = (object) => OWN_SYMBOL_KEYS(object).filter(
+  (k) => Object.prototype.propertyIsEnumerable.call(object, k)
+);
+var OWN_STRING_KEYS = Object.getOwnPropertyNames;
+var OWN_SYMBOL_KEYS = Object.getOwnPropertySymbols;
 function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = OWN_DESCS(object);
-  for (const key of OWN_NAMES(object)) {
+  for (const key of OWN_STRING_KEYS(object)) {
     if (isUnsafeKey(key)) {
       continue;
     }
@@ -104,13 +107,13 @@ function cloneWithDescriptors(object, settings, refs) {
     }
   }
   if (settings.preserveSymbolKeys) {
-    for (const symbol of OWN_SYMBOLS(object)) {
-      const desc = { ...descs[symbol] };
+    for (const key of OWN_SYMBOL_KEYS(object)) {
+      const desc = { ...descs[key] };
       if ("value" in desc) {
         desc.value = clone(desc.value, settings, refs);
       }
       try {
-        Object.defineProperty(result, symbol, desc);
+        Object.defineProperty(result, key, desc);
       } catch (error) {
         if (settings.strictDescriptors) {
           throw error;
@@ -128,16 +131,14 @@ function cloneArray(array, settings, refs) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  for (const key of Object.keys(object)) {
+  for (const key of OWN_ENUM_STRING_KEYS(object)) {
     if (!isUnsafeKey(key)) {
       result[key] = clone(object[key], settings, refs);
     }
   }
   if (settings.preserveSymbolKeys) {
-    for (const symbol of OWN_SYMBOLS(object)) {
-      if (IS_ENUMERABLE.call(object, symbol)) {
-        result[symbol] = clone(object[symbol], settings, refs);
-      }
+    for (const key of OWN_ENUM_SYMBOL_KEYS(object)) {
+      result[key] = clone(object[key], settings, refs);
     }
   }
   return result;
@@ -210,7 +211,7 @@ function cloneError(error, settings, refs) {
   if ("cause" in error && cause !== void 0) {
     result.cause = clone(cause, settings, refs);
   }
-  for (const key of Object.keys(error)) {
+  for (const key of OWN_ENUM_STRING_KEYS(error)) {
     Reflect.set(result, key, clone(Reflect.get(error, key), settings, refs));
   }
   return result;

package/package.json

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