bunshin-clone 2.0.1 → 2.0.3

package/README.md

@@ -37,7 +37,6 @@ interface BunshinCloneOptions {
   preserveBufferSharing: boolean; // default: false
   preserveDescriptors: boolean;   // default: false
   preserveSymbolKeys: boolean;    // default: false
-  strictDescriptors: boolean;     // default: false
 }
 ```
 
@@ -52,155 +51,3 @@ If `true`, preserves property descriptors, including non-enumerable properties.
 ### `preserveSymbolKeys`
 
 If `true`, preserves symbol keys (slower).
-
-### `strictDescriptors`
-
-If `true`, throws if property descriptor cannot be copied (e.g. non-configurable or non-writable).
-
-## 📖 Details
-
-<details>
-<summary>Read more</summary>
-
-### Example
-
-```ts
-const source = { foo: 1, nested: { x: 1 } };
-
-const result = bunshinClone(source);
-
-console.log(result);
-// { foo: 1, nested: { x: 1 } }
-
-console.log(result === source); // false
-console.log(result.nested === source.nested); // false
-```
-
-### Supported Types
-
-bunshin-clone correctly handles:
-
-- Object (plain + prototype preserved)
-- Array
-- Map
-- Set
-- Date
-- RegExp
-- ArrayBuffer
-- DataView
-- TypedArray (Uint8Array, etc.)
-- DOMException / Error
-- Blob
-- ImageData
-- URL
-- URLSearchParams
-
-### Circular ref
-
-```ts
-const a: any = { x: 1 };
-a.self = a;
-
-const result = bunshinClone(a);
-
-result.self === result; // true
-```
-
-### Descriptor Behavior
-
-#### Default (fast path)
-
-```ts
-const source = {
-  get x() {
-    return 42;
-  },
-};
-
-const result = bunshinClone(source);
-
-result.x; // 42
-// getter is NOT preserved
-```
-
-#### preserveDescriptors: true
-
-```ts
-const source = {};
-Object.defineProperty(source, 'x', {
-  get: () => 42,
-  enumerable: true,
-});
-
-const result = bunshinClone(source, {
-  preserveDescriptors: true,
-});
-
-Object.getOwnPropertyDescriptor(result, 'x')?.get;
-// => preserved
-```
-
-#### Unsupported / Pass-through Types
-
-Some values are returned as-is:
-
-- Function
-- WeakMap / WeakSet
-- Proxy (not cloned)
-- Other non-cloneable host objects
-
-```ts
-const fn = () => {};
-
-bunshinClone(fn) === fn; // true
-```
-
-### Design Notes
-
-#### Deep clone (no structural sharing)
-
-Unlike merge utilities, bunshin-clone always produces a new structure:
-
-```ts
-const source = { a: { b: 1 } };
-
-const result = bunshinClone(source);
-
-result !== source; // true
-result.a !== source.a; // true
-```
-
-#### Getter / Setter behavior
-
-- Default: evaluated and converted to value
-- preserveDescriptors: preserved as-is
-
-#### Descriptor safety
-
-When `preserveDescriptors` is enabled:
-
-- descriptors are cloned safely
-- original object is never mutated
-- errors are controlled via `strictDescriptors`
-
-### Performance
-
-- No proxy / no diffing
-- Minimal branching
-- Fast path for plain objects and arrays
-- Competitive with structuredClone in many cases
-
-### Comparison
-
-| Feature             | Bunshin Clone | structuredClone | lodash.clonedeep |
-| ------------------- | ------------- | --------------- | ---------------- |
-| Circular refs       | ✅            | ✅              | ✅               |
-| Map / Set           | ✅            | ✅              | ⚠️ (partial)     |
-| TypedArray          | ✅            | ✅              | ⚠️ (shallow)     |
-| Descriptor support  | ✅            | ❌              | ❌               |
-| Functions           | pass-through  | ❌ (throws)     | pass-through     |
-| Prototype preserved | ✅            | ❌              | ⚠️               |
-| Custom control      | ✅            | ❌              | ❌               |
-| Performance         | ⚡ fast       | ⚡ fast         | 🐢 slower        |
-
-</details>

package/dist/index.bundle.cjs

@@ -147,6 +147,12 @@ function cloneMap(map, settings, refs) {
   return result;
 }
 var OWN_DESCS = Object.getOwnPropertyDescriptors;
+function OWN_ENUM_KEYS(object, symbol) {
+  return typeof symbol !== "boolean" || !symbol ? OWN_ENUM_STRING_KEYS(object) : [
+    ...OWN_ENUM_STRING_KEYS(object),
+    ...OWN_ENUM_SYMBOL_KEYS(object)
+  ];
+}
 function OWN_ENUM_STRING_KEYS(object) {
   const keys = Object.keys(object);
   for (const key of keys) {
@@ -165,6 +171,12 @@ function OWN_ENUM_SYMBOL_KEYS(object) {
   }
   return keys;
 }
+function OWN_KEYS(object, symbol) {
+  return typeof symbol !== "boolean" || !symbol ? OWN_STRING_KEYS(object) : [
+    ...OWN_STRING_KEYS(object),
+    ...OWN_SYMBOL_KEYS(object)
+  ];
+}
 function OWN_STRING_KEYS(object) {
   const keys = Object.getOwnPropertyNames(object);
   for (const key of keys) {
@@ -183,17 +195,9 @@ function isUnsafeKey(key) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  function copy(key) {
+  for (const key of OWN_ENUM_KEYS(object, settings.preserveSymbolKeys)) {
     result[key] = clone(object[key], settings, refs);
   }
-  for (const key of OWN_ENUM_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_ENUM_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
-  }
   return result;
 }
 
@@ -248,26 +252,12 @@ function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = OWN_DESCS(object);
-  function copy(key) {
+  for (const key of OWN_KEYS(object, settings.preserveSymbolKeys)) {
     const desc = { ...descs[key] };
     if ("value" in desc) {
       desc.value = clone(desc.value, settings, refs);
     }
-    try {
-      Object.defineProperty(result, key, desc);
-    } catch (error) {
-      if (settings.strictDescriptors) {
-        throw error;
-      }
-    }
-  }
-  for (const key of OWN_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
+    Object.defineProperty(result, key, desc);
   }
   return result;
 }
@@ -337,13 +327,8 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = 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;
@@ -352,15 +337,14 @@ function resolveOptions(options) {
     console.warn("Invalid preserveDescriptors option. Fallback: false.");
     preserveDescriptors = false;
   }
-  if (typeof strictDescriptors !== "boolean") {
-    console.warn("Invalid strictDescriptors option. Fallback: false.");
-    strictDescriptors = false;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
   }
   return {
     preserveBufferSharing,
     preserveDescriptors,
-    preserveSymbolKeys,
-    strictDescriptors
+    preserveSymbolKeys
   };
 }
 

package/dist/index.bundle.js

@@ -145,6 +145,12 @@ function cloneMap(map, settings, refs) {
   return result;
 }
 var OWN_DESCS = Object.getOwnPropertyDescriptors;
+function OWN_ENUM_KEYS(object, symbol) {
+  return typeof symbol !== "boolean" || !symbol ? OWN_ENUM_STRING_KEYS(object) : [
+    ...OWN_ENUM_STRING_KEYS(object),
+    ...OWN_ENUM_SYMBOL_KEYS(object)
+  ];
+}
 function OWN_ENUM_STRING_KEYS(object) {
   const keys = Object.keys(object);
   for (const key of keys) {
@@ -163,6 +169,12 @@ function OWN_ENUM_SYMBOL_KEYS(object) {
   }
   return keys;
 }
+function OWN_KEYS(object, symbol) {
+  return typeof symbol !== "boolean" || !symbol ? OWN_STRING_KEYS(object) : [
+    ...OWN_STRING_KEYS(object),
+    ...OWN_SYMBOL_KEYS(object)
+  ];
+}
 function OWN_STRING_KEYS(object) {
   const keys = Object.getOwnPropertyNames(object);
   for (const key of keys) {
@@ -181,17 +193,9 @@ function isUnsafeKey(key) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  function copy(key) {
+  for (const key of OWN_ENUM_KEYS(object, settings.preserveSymbolKeys)) {
     result[key] = clone(object[key], settings, refs);
   }
-  for (const key of OWN_ENUM_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_ENUM_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
-  }
   return result;
 }
 
@@ -246,26 +250,12 @@ function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = OWN_DESCS(object);
-  function copy(key) {
+  for (const key of OWN_KEYS(object, settings.preserveSymbolKeys)) {
     const desc = { ...descs[key] };
     if ("value" in desc) {
       desc.value = clone(desc.value, settings, refs);
     }
-    try {
-      Object.defineProperty(result, key, desc);
-    } catch (error) {
-      if (settings.strictDescriptors) {
-        throw error;
-      }
-    }
-  }
-  for (const key of OWN_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
+    Object.defineProperty(result, key, desc);
   }
   return result;
 }
@@ -335,13 +325,8 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = 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;
@@ -350,15 +335,14 @@ function resolveOptions(options) {
     console.warn("Invalid preserveDescriptors option. Fallback: false.");
     preserveDescriptors = false;
   }
-  if (typeof strictDescriptors !== "boolean") {
-    console.warn("Invalid strictDescriptors option. Fallback: false.");
-    strictDescriptors = false;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
   }
   return {
     preserveBufferSharing,
     preserveDescriptors,
-    preserveSymbolKeys,
-    strictDescriptors
+    preserveSymbolKeys
   };
 }
 

package/dist/index.cjs

@@ -151,17 +151,9 @@ function cloneMap(map, settings, refs) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  function copy(key) {
+  for (const key of own.OWN_ENUM_KEYS(object, settings.preserveSymbolKeys)) {
     result[key] = clone(object[key], settings, refs);
   }
-  for (const key of own.OWN_ENUM_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of own.OWN_ENUM_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
-  }
   return result;
 }
 
@@ -214,26 +206,12 @@ function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = own.OWN_DESCS(object);
-  function copy(key) {
+  for (const key of own.OWN_KEYS(object, settings.preserveSymbolKeys)) {
     const desc = { ...descs[key] };
     if ("value" in desc) {
       desc.value = clone(desc.value, settings, refs);
     }
-    try {
-      Object.defineProperty(result, key, desc);
-    } catch (error) {
-      if (settings.strictDescriptors) {
-        throw error;
-      }
-    }
-  }
-  for (const key of own.OWN_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of own.OWN_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
+    Object.defineProperty(result, key, desc);
   }
   return result;
 }
@@ -303,13 +281,8 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = 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;
@@ -318,15 +291,14 @@ function resolveOptions(options) {
     console.warn("Invalid preserveDescriptors option. Fallback: false.");
     preserveDescriptors = false;
   }
-  if (typeof strictDescriptors !== "boolean") {
-    console.warn("Invalid strictDescriptors option. Fallback: false.");
-    strictDescriptors = false;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
   }
   return {
     preserveBufferSharing,
     preserveDescriptors,
-    preserveSymbolKeys,
-    strictDescriptors
+    preserveSymbolKeys
   };
 }
 

package/dist/index.d.cts

@@ -4,7 +4,6 @@ interface BunshinCloneOptions {
     preserveBufferSharing: boolean;
     preserveDescriptors: boolean;
     preserveSymbolKeys: boolean;
-    strictDescriptors: boolean;
 }
 type PlainObject = Record<PropertyKey, unknown>;
 

package/dist/index.d.ts

@@ -4,7 +4,6 @@ interface BunshinCloneOptions {
     preserveBufferSharing: boolean;
     preserveDescriptors: boolean;
     preserveSymbolKeys: boolean;
-    strictDescriptors: boolean;
 }
 type PlainObject = Record<PropertyKey, unknown>;
 

package/dist/index.js

@@ -1,4 +1,4 @@
-import { OWN_DESCS, OWN_STRING_KEYS, OWN_SYMBOL_KEYS, OWN_ENUM_STRING_KEYS, OWN_ENUM_SYMBOL_KEYS } from '@y14e/own';
+import { OWN_DESCS, OWN_KEYS, OWN_ENUM_KEYS } from '@y14e/own';
 
 // src/utils.ts
 function isObject(value) {
@@ -149,17 +149,9 @@ function cloneMap(map, settings, refs) {
 function clonePlainObject(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
-  function copy(key) {
+  for (const key of OWN_ENUM_KEYS(object, settings.preserveSymbolKeys)) {
     result[key] = clone(object[key], settings, refs);
   }
-  for (const key of OWN_ENUM_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_ENUM_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
-  }
   return result;
 }
 
@@ -212,26 +204,12 @@ function cloneWithDescriptors(object, settings, refs) {
   const result = Object.create(Object.getPrototypeOf(object));
   refs.set(object, result);
   const descs = OWN_DESCS(object);
-  function copy(key) {
+  for (const key of OWN_KEYS(object, settings.preserveSymbolKeys)) {
     const desc = { ...descs[key] };
     if ("value" in desc) {
       desc.value = clone(desc.value, settings, refs);
     }
-    try {
-      Object.defineProperty(result, key, desc);
-    } catch (error) {
-      if (settings.strictDescriptors) {
-        throw error;
-      }
-    }
-  }
-  for (const key of OWN_STRING_KEYS(object)) {
-    copy(key);
-  }
-  if (settings.preserveSymbolKeys) {
-    for (const key of OWN_SYMBOL_KEYS(object)) {
-      copy(key);
-    }
+    Object.defineProperty(result, key, desc);
   }
   return result;
 }
@@ -301,13 +279,8 @@ function resolveOptions(options) {
   let {
     preserveBufferSharing = false,
     preserveDescriptors = 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;
@@ -316,15 +289,14 @@ function resolveOptions(options) {
     console.warn("Invalid preserveDescriptors option. Fallback: false.");
     preserveDescriptors = false;
   }
-  if (typeof strictDescriptors !== "boolean") {
-    console.warn("Invalid strictDescriptors option. Fallback: false.");
-    strictDescriptors = false;
+  if (typeof preserveSymbolKeys !== "boolean") {
+    console.warn("Invalid preserveSymbolKeys option. Fallback: false.");
+    preserveSymbolKeys = false;
   }
   return {
     preserveBufferSharing,
     preserveDescriptors,
-    preserveSymbolKeys,
-    strictDescriptors
+    preserveSymbolKeys
   };
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunshin-clone",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "description": "High-performance deep clone utility with descriptor support",
   "keywords": [
     "array",
@@ -68,9 +68,6 @@
     "prepublishOnly": "npm run build",
     "test": "bun test"
   },
-  "dependencies": {
-    "@y14e/own": "^1.0.1"
-  },
   "devDependencies": {
     "bun-types": "latest",
     "tsup": "^8.0.0",
@@ -82,5 +79,8 @@
   },
   "allowScripts": {
     "esbuild": true
+  },
+  "dependencies": {
+    "@y14e/own": "^1.0.2"
   }
 }