fix: do not destructure node callback results

Author: JonKrone

Diff for: src/core/components/dag.js

@@ -39,7 +39,7 @@ module.exports = function dag (self) {
         try {
           cid = new CID(cid)
         } catch (err) {
-          callback(err)
+          return callback(err)
         }
       }
 

Diff for: src/core/components/pin-set.js

@@ -74,9 +74,9 @@ exports = module.exports = function (dag) {
 
           seen[bs58Link] = true
 
-          dag.get(multihash, (err, { value }) => {
+          dag.get(multihash, (err, res) => {
             if (err) { return someCb(err) }
-            searchChildren(value, someCb)
+            searchChildren(res.value, someCb)
           })
         }, cb)
       }

Diff for: src/core/components/pin.js

@@ -157,7 +157,7 @@ module.exports = function pin (self) {
           // persist updated pin sets to datastore
           flushPins((err, root) => {
             if (err) { return callback(err) }
-            return callback(null, results.map(key => ({hash: key})))
+            return callback(null, results.map(hash => ({ hash })))
           })
         })
       })
@@ -215,7 +215,7 @@ module.exports = function pin (self) {
           flushPins((err, root) => {
             if (err) { return callback(err) }
             self.log(`Removed pins: ${results}`)
-            return callback(null, results.map(key => ({hash: key})))
+            return callback(null, results.map(hash => ({ hash })))
           })
         })
       })
@@ -375,8 +375,9 @@ module.exports = function pin (self) {
         async.parallel([
           cb => pinset.loadSet(pinRoot.value, types.recursive, cb),
           cb => pinset.loadSet(pinRoot.value, types.direct, cb)
-        ], (err, [rKeys, dKeys]) => {
+        ], (err, keys) => {
           if (err) { return callback(err) }
+          const [ rKeys, dKeys ] = keys
 
           directPins = new Set(dKeys.map(toB58String))
           recursivePins = new Set(rKeys.map(toB58String))