Refactor: restore use of spread operator (#106)

Because the Chromium bug has been fixed.

Ref: https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
Category: fix

Author: vweevers

abstract-chained-batch.js

@@ -80,14 +80,14 @@ class AbstractChainedBatch {
     if (keyError != null) throw keyError
     if (valueError != null) throw valueError
 
-    // Avoid spread operator because of https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
-    const op = Object.assign({}, options, {
+    const op = {
+      ...options,
       type: 'put',
       key,
       value,
       keyEncoding: db.keyEncoding(options.keyEncoding),
       valueEncoding: db.valueEncoding(options.valueEncoding)
-    })
+    }
 
     if (this.#prewriteRun !== null) {
       try {
@@ -135,7 +135,7 @@ class AbstractChainedBatch {
     // If the sublevel is not a descendant then we shouldn't emit events
     if (this.#publicOperations !== null && !siblings) {
       // Clone op before we mutate it for the private API
-      const publicOperation = Object.assign({}, op)
+      const publicOperation = { ...op }
       publicOperation.encodedKey = encodedKey
       publicOperation.encodedValue = encodedValue
 
@@ -149,7 +149,7 @@ class AbstractChainedBatch {
 
       this.#publicOperations.push(publicOperation)
     } else if (this.#legacyOperations !== null && !siblings) {
-      const legacyOperation = Object.assign({}, original)
+      const legacyOperation = { ...original }
 
       legacyOperation.type = 'put'
       legacyOperation.key = key
@@ -189,12 +189,12 @@ class AbstractChainedBatch {
 
     if (keyError != null) throw keyError
 
-    // Avoid spread operator because of https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
-    const op = Object.assign({}, options, {
+    const op = {
+      ...options,
       type: 'del',
       key,
       keyEncoding: db.keyEncoding(options.keyEncoding)
-    })
+    }
 
     if (this.#prewriteRun !== null) {
       try {
@@ -221,7 +221,7 @@ class AbstractChainedBatch {
 
     if (this.#publicOperations !== null) {
       // Clone op before we mutate it for the private API
-      const publicOperation = Object.assign({}, op)
+      const publicOperation = { ...op }
       publicOperation.encodedKey = encodedKey
 
       if (delegated) {
@@ -232,7 +232,7 @@ class AbstractChainedBatch {
 
       this.#publicOperations.push(publicOperation)
     } else if (this.#legacyOperations !== null) {
-      const legacyOperation = Object.assign({}, original)
+      const legacyOperation = { ...original }
 
       legacyOperation.type = 'del'
       legacyOperation.key = key

abstract-level.js

@@ -58,7 +58,8 @@ class AbstractLevel extends EventEmitter {
       permanence: manifest.permanence !== false,
 
       encodings: manifest.encodings || {},
-      events: Object.assign({}, manifest.events, {
+      events: {
+        ...manifest.events,
         opening: true,
         open: true,
         closing: true,
@@ -68,7 +69,7 @@ class AbstractLevel extends EventEmitter {
         del: true,
         batch: true,
         clear: true
-      })
+      }
     })
 
     // Monitor event listeners
@@ -332,8 +333,7 @@ class AbstractLevel extends EventEmitter {
 
     // Forward encoding options to the underlying store
     if (options.keyEncoding !== keyFormat || options.valueEncoding !== valueFormat) {
-      // Avoid spread operator because of https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
-      options = Object.assign({}, options, { keyEncoding: keyFormat, valueEncoding: valueFormat })
+      options = { ...options, keyEncoding: keyFormat, valueEncoding: valueFormat }
     }
 
     const encodedKey = keyEncoding.encode(key)
@@ -390,7 +390,7 @@ class AbstractLevel extends EventEmitter {
 
     // Forward encoding options
     if (options.keyEncoding !== keyFormat || options.valueEncoding !== valueFormat) {
-      options = Object.assign({}, options, { keyEncoding: keyFormat, valueEncoding: valueFormat })
+      options = { ...options, keyEncoding: keyFormat, valueEncoding: valueFormat }
     }
 
     const mappedKeys = new Array(keys.length)
@@ -454,11 +454,10 @@ class AbstractLevel extends EventEmitter {
 
     // Forward encoding options to the underlying store
     if (options === this.#defaultOptions.key) {
-      // Avoid Object.assign() for default options
+      // Avoid cloning for default options
       options = this.#defaultOptions.keyFormat
     } else if (options.keyEncoding !== keyFormat) {
-      // Avoid spread operator because of https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
-      options = Object.assign({}, options, { keyEncoding: keyFormat })
+      options = { ...options, keyEncoding: keyFormat }
     }
 
     const encodedKey = keyEncoding.encode(key)
@@ -504,11 +503,10 @@ class AbstractLevel extends EventEmitter {
 
     // Forward encoding options to the underlying store
     if (options === this.#defaultOptions.key) {
-      // Avoid Object.assign() for default options
+      // Avoid cloning for default options
       options = this.#defaultOptions.keyFormat
     } else if (options.keyEncoding !== keyFormat) {
-      // Avoid spread operator because of https://bugs.chromium.org/p/chromium/issues/detail?id=1204540
-      options = Object.assign({}, options, { keyEncoding: keyFormat })
+      options = { ...options, keyEncoding: keyFormat }
     }
 
     const mappedKeys = new Array(keys.length)
@@ -565,12 +563,12 @@ class AbstractLevel extends EventEmitter {
     const enableWriteEvent = this.#eventMonitor.write
     const original = options
 
-    // Avoid Object.assign() for default options
+    // Avoid cloning for default options
     // TODO: also apply this tweak to get() and getMany()
     if (options === this.#defaultOptions.entry) {
       options = this.#defaultOptions.entryFormat
     } else if (options.keyEncoding !== keyFormat || options.valueEncoding !== valueFormat) {
-      options = Object.assign({}, options, { keyEncoding: keyFormat, valueEncoding: valueFormat })
+      options = { ...options, keyEncoding: keyFormat, valueEncoding: valueFormat }
     }
 
     const encodedKey = keyEncoding.encode(key)
@@ -580,15 +578,16 @@ class AbstractLevel extends EventEmitter {
     await this._put(prefixedKey, encodedValue, options)
 
     if (enableWriteEvent) {
-      const op = Object.assign({}, original, {
+      const op = {
+        ...original,
         type: 'put',
         key,
         value,
         keyEncoding,
         valueEncoding,
         encodedKey,
         encodedValue
-      })
+      }
 
       this.emit('write', [op])
     } else {
@@ -622,11 +621,11 @@ class AbstractLevel extends EventEmitter {
     const enableWriteEvent = this.#eventMonitor.write
     const original = options
 
-    // Avoid Object.assign() for default options
+    // Avoid cloning for default options
     if (options === this.#defaultOptions.key) {
       options = this.#defaultOptions.keyFormat
     } else if (options.keyEncoding !== keyFormat) {
-      options = Object.assign({}, options, { keyEncoding: keyFormat })
+      options = { ...options, keyEncoding: keyFormat }
     }
 
     const encodedKey = keyEncoding.encode(key)
@@ -635,12 +634,13 @@ class AbstractLevel extends EventEmitter {
     await this._del(prefixedKey, options)
 
     if (enableWriteEvent) {
-      const op = Object.assign({}, original, {
+      const op = {
+        ...original,
         type: 'del',
         key,
         keyEncoding,
         encodedKey
-      })
+      }
 
       this.emit('write', [op])
     } else {
@@ -694,7 +694,7 @@ class AbstractLevel extends EventEmitter {
       // Clone the op so that we can freely mutate it. We can't use a class because the
       // op can have userland properties that we'd have to copy, negating the performance
       // benefits of a class. So use a plain object.
-      const op = Object.assign({}, options, operations[i])
+      const op = { ...options, ...operations[i] }
 
       // Hook functions can modify op but not its type or sublevel, so cache those
       const isPut = op.type === 'put'
@@ -753,7 +753,7 @@ class AbstractLevel extends EventEmitter {
       if (enableWriteEvent && !siblings) {
         // Clone op before we mutate it for the private API
         // TODO (future semver-major): consider sending this shape to private API too
-        publicOperation = Object.assign({}, op)
+        publicOperation = { ...op }
         publicOperation.encodedKey = encodedKey
 
         if (delegated) {

lib/prewrite-batch.js

@@ -58,7 +58,7 @@ class PrewriteBatch {
     // If the sublevel is not a descendant then we shouldn't emit events
     if (this.#publicOperations !== null && !siblings) {
       // Clone op before we mutate it for the private API
-      publicOperation = Object.assign({}, op)
+      publicOperation = { ...op }
       publicOperation.encodedKey = encodedKey
 
       if (delegated) {

test/iterator-range-test.js

@@ -38,7 +38,7 @@ exports.range = function (test, testCommon) {
     // Test the documented promise that in reverse mode,
     // "the returned entries are the same, but in reverse".
     if (!opts.reverse && !('limit' in opts)) {
-      const reverseOpts = Object.assign({}, opts, { reverse: true })
+      const reverseOpts = { ...opts, reverse: true }
 
       rangeTest(
         name + ' (flipped)',