fix: remove node globals

package.json

@@ -17,6 +17,9 @@
     "coverage-publish": "aegir coverage --provider codecov",
     "docs": "aegir docs"
   },
+  "browser": {
+    "./src/utils.js": "./src/utils.browser.js"
+  },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ipfs/interface-datastore.git"
@@ -38,9 +41,11 @@
     "dirty-chai": "^2.0.1"
   },
   "dependencies": {
+    "buffer": "^5.5.0",
     "class-is": "^1.1.0",
     "err-code": "^2.0.0",
-    "uuid": "^3.2.2"
+    "iso-random-stream": "^1.1.1",
+    "nanoid": "^2.1.11"
   },
   "engines": {
     "node": ">=8.0.0",

src/key.js

@@ -1,6 +1,7 @@
 'use strict'
 
-const uuid = require('uuid/v4')
+const { Buffer } = require('buffer')
+const uuid = require('nanoid')
 const withIs = require('class-is')
 
 const pathSepS = '/'

src/tests.js

@@ -2,10 +2,11 @@
 /* eslint max-nested-callbacks: ["error", 8] */
 'use strict'
 
+const { Buffer } = require('buffer')
+const randomBytes = require('iso-random-stream/src/random')
 const chai = require('chai')
 chai.use(require('dirty-chai'))
 const expect = chai.expect
-const crypto = require('crypto')
 
 const Key = require('../src').Key
 
@@ -145,9 +146,9 @@ module.exports = (test) => {
       const b = store.batch()
       const count = 400
       for (let i = 0; i < count; i++) {
-        b.put(new Key(`/a/hello${i}`), crypto.randomBytes(32))
-        b.put(new Key(`/q/hello${i}`), crypto.randomBytes(64))
-        b.put(new Key(`/z/hello${i}`), crypto.randomBytes(128))
+        b.put(new Key(`/a/hello${i}`), randomBytes(32))
+        b.put(new Key(`/q/hello${i}`), randomBytes(64))
+        b.put(new Key(`/z/hello${i}`), randomBytes(128))
       }
 
       await b.commit()

src/utils.browser.js

@@ -0,0 +1,53 @@
+'use strict'
+
+const uuid = require('nanoid')
+
+exports.filter = (iterable, filterer) => {
+  return (async function * () {
+    for await (const value of iterable) {
+      const keep = await filterer(value)
+      if (!keep) continue
+      yield value
+    }
+  })()
+}
+
+// Not just sort, because the sorter is given all the values and should return
+// them all sorted
+exports.sortAll = (iterable, sorter) => {
+  return (async function * () {
+    let values = []
+    for await (const value of iterable) values.push(value)
+    values = await sorter(values)
+    for (const value of values) yield value
+  })()
+}
+
+exports.take = (iterable, n) => {
+  return (async function * () {
+    if (n <= 0) return
+    let i = 0
+    for await (const value of iterable) {
+      yield value
+      i++
+      if (i >= n) return
+    }
+  })()
+}
+
+exports.map = (iterable, mapper) => {
+  return (async function * () {
+    for await (const value of iterable) {
+      yield mapper(value)
+    }
+  })()
+}
+
+exports.replaceStartWith = function (s, r) {
+  const matcher = new RegExp('^' + r)
+  return s.replace(matcher, '')
+}
+
+exports.tmpdir = () => {
+  return `datastore_${uuid()}`
+}

src/utils.js

@@ -2,7 +2,7 @@
 
 const path = require('path')
 const os = require('os')
-const uuid = require('uuid/v4')
+const uuid = require('nanoid')
 
 exports.filter = (iterable, filterer) => {
   return (async function * () {