copy-anything - npm Package Compare versions

Comparing version 1.3.0 to 1.4.0

dist/index.cjs.js

@@ -5,17 +5,2 @@ 'use strict';
 
-function assignProp(carry, key, newVal, originalObject) {
-    var propType = originalObject.propertyIsEnumerable(key)
-        ? 'enumerable'
-        : 'nonenumerable';
-    if (propType === 'enumerable')
-        carry[key] = newVal;
-    if (propType === 'nonenumerable') {
-        Object.defineProperty(carry, key, {
-            value: newVal,
-            enumerable: false,
-            writable: true,
-            configurable: true
-        });
-    }
-}
 /**
@@ -33,9 +18,6 @@ * Copy (clone) an object and all its props recursively to get rid of any prop referenced of the original object. Arrays are also cloned, however objects inside arrays are still linked.
         return target;
-    var props = Object.getOwnPropertyNames(target);
-    var symbols = Object.getOwnPropertySymbols(target);
-    return props.concat(symbols).reduce(function (carry, key) {
-        // @ts-ignore
+    return Object.keys(target)
+        .reduce(function (carry, key) {
         var val = target[key];
-        var newVal = copy(val);
-        assignProp(carry, key, newVal, target);
+        carry[key] = copy(val);
         return carry;
@@ -42,0 +24,0 @@ }, {});

dist/index.esm.js

 import { isArray, isPlainObject } from 'is-what';
 
-function assignProp(carry, key, newVal, originalObject) {
-    var propType = originalObject.propertyIsEnumerable(key)
-        ? 'enumerable'
-        : 'nonenumerable';
-    if (propType === 'enumerable')
-        carry[key] = newVal;
-    if (propType === 'nonenumerable') {
-        Object.defineProperty(carry, key, {
-            value: newVal,
-            enumerable: false,
-            writable: true,
-            configurable: true
-        });
-    }
-}
 /**
@@ -30,9 +15,6 @@ * Copy (clone) an object and all its props recursively to get rid of any prop referenced of the original object. Arrays are also cloned, however objects inside arrays are still linked.
         return target;
-    var props = Object.getOwnPropertyNames(target);
-    var symbols = Object.getOwnPropertySymbols(target);
-    return props.concat(symbols).reduce(function (carry, key) {
-        // @ts-ignore
+    return Object.keys(target)
+        .reduce(function (carry, key) {
         var val = target[key];
-        var newVal = copy(val);
-        assignProp(carry, key, newVal, target);
+        carry[key] = copy(val);
         return carry;
@@ -39,0 +21,0 @@ }, {});

package.json

 {
   "name": "copy-anything",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "An optimised way to copy'ing an object. A small and simple integration",
@@ -5,0 +5,0 @@ "main": "dist/index.cjs.js",

README.md

@@ -19,4 +19,2 @@ # Copy anything 🎭
 - works with arrays and objects in arrays!
-- supports symbols
-- supports enumerable & nonenumerable props
 - **does not break special class instances**　‼️
@@ -23,0 +21,0 @@

src/index.ts

 import { isPlainObject, isArray } from 'is-what'
 
-function assignProp (carry, key, newVal, originalObject) {
-  const propType = originalObject.propertyIsEnumerable(key)
-    ? 'enumerable'
-    : 'nonenumerable'
-  if (propType === 'enumerable') carry[key] = newVal
-  if (propType === 'nonenumerable') {
-    Object.defineProperty(carry, key, {
-      value: newVal,
-      enumerable: false,
-      writable: true,
-      configurable: true
-    })
-  }
-}
-
 /**
@@ -28,12 +13,8 @@ * Copy (clone) an object and all its props recursively to get rid of any prop referenced of the original object. Arrays are also cloned, however objects inside arrays are still linked.
   if (!isPlainObject(target)) return target
-  const props = Object.getOwnPropertyNames(target)
-  const symbols = Object.getOwnPropertySymbols(target)
-  return [...props, ...symbols]
+  return Object.keys(target)
     .reduce((carry, key) => {
-      // @ts-ignore
       const val = target[key]
-      const newVal = copy(val)
-      assignProp(carry, key, newVal, target)
+      carry[key] = copy(val)
       return carry
     }, {})
 }

test/index.js

@@ -115,47 +115,1 @@ import test from 'ava'
 })
-
-test('symbols as keys', t => {
-  let res, target
-  const mySymbol = Symbol('mySymbol')
-  target = { value: 42, [mySymbol]: 'hello' }
-  res = copy(target)
-  // change original
-  target.value = 1
-  target[mySymbol] = 2
-  t.is(res.value, 42)
-  t.is(res[mySymbol], 'hello')
-  t.is(target.value, 1)
-  t.is(target[mySymbol], 2)
-})
-
-test('nonenumerable keys', t => {
-  let target, res
-  const mySymbol = Symbol('mySymbol')
-  target = { value: 42 }
-  Object.defineProperty(target, 'id', {
-    value: 1,
-    writable: true,
-    enumerable: false,
-    configurable: true
-  })
-  Object.defineProperty(target, mySymbol, {
-    value: 'original',
-    writable: true,
-    enumerable: false,
-    configurable: true
-  })
-  res = copy(target)
-  // change original
-  target.id = 100
-  target[mySymbol] = 'new'
-  target.value = 300
-  t.is(res.value, 42)
-  t.is(res.id, 1)
-  t.is(res[mySymbol], 'original')
-  t.is(Object.keys(res).length, 1)
-  t.true(Object.keys(res).includes('value'))
-  t.is(target.id, 100)
-  t.is(target[mySymbol], 'new')
-  t.is(target.value, 300)
-  t.is(Object.keys(target).length, 1)
-})

No alert changes

Worsened metrics

Total package byte prevSize

14452

decreased by-17.24%

Lines of code

290

decreased by-25.26%

Number of lines in readme file

89

decreased by-2.2%

No dependency changes