corestore 6.4.1 → 6.4.3

package/index.js

@@ -60,6 +60,11 @@ module.exports = class Corestore extends EventEmitter {
     }
   }
 
+  _emitCore (name, core) {
+    this.emit(name, core)
+    if (this !== this._root) this._root.emit(name, core)
+  }
+
   _incFindingPeers () {
     if (++this._findingPeersCount !== 1) return
 
@@ -201,7 +206,7 @@ module.exports = class Corestore extends EventEmitter {
     this.cores.set(id, core)
     core.ready().then(() => {
       if (core.closing) return // extra safety here as ready is a tick after open
-      this.emit('core-open', core)
+      this._emitCore('core-open', core)
       for (const { stream } of this._replicationStreams) {
         core.replicate(stream, { session: true })
       }
@@ -209,7 +214,7 @@ module.exports = class Corestore extends EventEmitter {
       this.cores.delete(id)
     })
     core.once('close', () => {
-      this.emit('core-close', core)
+      this._emitCore('core-close', core)
       this.cores.delete(id)
     })
     core.on('conflict', (len, fork, proof) => {
@@ -258,12 +263,15 @@ module.exports = class Corestore extends EventEmitter {
       this._findingPeers.push(core.findingPeers())
     }
 
-    core.once('close', () => {
+    const gc = () => {
       // technically better to also clear _findingPeers if we added it,
       // but the lifecycle for those are pretty short so prob not worth the complexity
       // as _decFindingPeers clear them all.
       this._sessions.delete(core)
-    })
+    }
+
+    core.ready().catch(gc)
+    core.once('close', gc)
 
     return core
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "corestore",
-  "version": "6.4.1",
+  "version": "6.4.3",
   "description": "A Hypercore factory that simplifies managing collections of cores.",
   "main": "index.js",
   "scripts": {