@openfin/remote-adapter 44.100.25 → 44.100.26

package/out/remote-adapter.js

@@ -20,35 +20,6 @@ var InteropBroker$1 = {};
 
 var base = {};
 
-var promises = {};
-
-Object.defineProperty(promises, "__esModule", { value: true });
-promises.promiseMapSerial = promises.serial = promises.promiseMap = promises.promisify = void 0;
-function promisify(func) {
-    return (...args) => new Promise((resolve, reject) => {
-        func(...args, (err, val) => (err ? reject(err) : resolve(val)));
-    });
-}
-promises.promisify = promisify;
-async function promiseMap(arr, asyncF) {
-    return Promise.all(arr.map(asyncF));
-}
-promises.promiseMap = promiseMap;
-async function serial(arr) {
-    const ret = [];
-    for (const func of arr) {
-        // eslint-disable-next-line no-await-in-loop
-        const next = await func();
-        ret.push(next);
-    }
-    return ret;
-}
-promises.serial = serial;
-async function promiseMapSerial(arr, func) {
-    return serial(arr.map((value, index, array) => () => func(value, index, array)));
-}
-promises.promiseMapSerial = promiseMapSerial;
-
 var __classPrivateFieldSet$h = (commonjsGlobal && commonjsGlobal.__classPrivateFieldSet) || function (receiver, state, value, kind, f) {
     if (kind === "m") throw new TypeError("Private method is not writable");
     if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a setter");
@@ -63,7 +34,6 @@ var __classPrivateFieldGet$i = (commonjsGlobal && commonjsGlobal.__classPrivateF
 var _EmitterBase_emitterAccessor, _EmitterBase_deregisterOnceListeners;
 Object.defineProperty(base, "__esModule", { value: true });
 base.Reply = base.EmitterBase = base.Base = void 0;
-const promises_1 = promises;
 class Base {
     /**
      * @internal
@@ -141,27 +111,8 @@ class EmitterBase extends Base {
             return this.hasEmitter() ? this.getOrCreateEmitter().emit(eventType, payload, ...args) : false;
         };
         this.hasEmitter = () => this.wire.eventAggregator.has(__classPrivateFieldGet$i(this, _EmitterBase_emitterAccessor, "f"));
-        /**
-         * Cleans up after removal of a listener, e.g. deleting any lingering deregistration handlers for a
-         * `once` subscription.
-         *
-         * @remarks Implementing this as a `removeListener` handler ensures that direct removal of a listener
-         * on the base emitter will not leak additional core handlers.  We could do this in the forwarding method,
-         * which would involve less "magic," but would be more-vulnerable to accidental re-introduction of a leak.
-         */
-        this.cleanUpRemovedListener = (eventType, listener) => {
-            const deregister = __classPrivateFieldGet$i(this, _EmitterBase_deregisterOnceListeners, "f").get(listener);
-            if (deregister) {
-                const emitter = this.wire.eventAggregator.getOrCreate(__classPrivateFieldGet$i(this, _EmitterBase_emitterAccessor, "f"));
-                emitter.removeListener(eventType, deregister);
-            }
-        };
         this.getOrCreateEmitter = () => {
-            const emitter = this.wire.eventAggregator.getOrCreate(__classPrivateFieldGet$i(this, _EmitterBase_emitterAccessor, "f"));
-            if (!emitter.listeners('removeListener').includes(this.cleanUpRemovedListener)) {
-                emitter.on('removeListener', this.cleanUpRemovedListener);
-            }
-            return emitter;
+            return this.wire.eventAggregator.getOrCreate(__classPrivateFieldGet$i(this, _EmitterBase_emitterAccessor, "f"));
         };
         this.listeners = (type) => this.hasEmitter() ? this.getOrCreateEmitter().listeners(type) : [];
         this.listenerCount = (type) => this.hasEmitter() ? this.getOrCreateEmitter().listenerCount(type) : 0;
@@ -201,7 +152,6 @@ class EmitterBase extends Base {
         };
         __classPrivateFieldSet$h(this, _EmitterBase_emitterAccessor, [topic, ...additionalAccessors], "f");
         __classPrivateFieldSet$h(this, _EmitterBase_deregisterOnceListeners, new WeakMap(), "f");
-        this.listeners = (event) => this.hasEmitter() ? this.getOrCreateEmitter().listeners(event) : [];
     }
     /**
      * Adds a listener to the end of the listeners array for the specified event.
@@ -279,6 +229,10 @@ class EmitterBase extends Base {
         const emitter = await this.deregisterEventListener(eventType, options);
         if (emitter) {
             emitter.removeListener(eventType, listener);
+            const deregister = __classPrivateFieldGet$i(this, _EmitterBase_deregisterOnceListeners, "f").get(listener);
+            if (deregister) {
+                emitter.removeListener(eventType, deregister);
+            }
             this.deleteEmitterIfNothingRegistered(emitter);
         }
         return this;
@@ -314,12 +268,13 @@ class EmitterBase extends Base {
         }
         else if (this.hasEmitter()) {
             const events = this.getOrCreateEmitter().eventNames();
-            await (0, promises_1.promiseMap)(events, removeByEvent);
+            await Promise.all(events.map(removeByEvent));
         }
         return this;
     }
     deleteEmitterIfNothingRegistered(emitter) {
-        if (emitter.eventNames().every((type) => type === 'removeListener')) {
+        // TODO: maybe emitterMap should clean up itself..
+        if (emitter.eventNames().length === 0) {
             this.wire.eventAggregator.delete(__classPrivateFieldGet$i(this, _EmitterBase_emitterAccessor, "f"));
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@openfin/remote-adapter",
-    "version": "44.100.25",
+    "version": "44.100.26",
     "description": "Establish intermachine runtime connections using webRTC.",
     "license": "SEE LICENSE IN LICENSE.md",
     "private": false,
@@ -20,6 +20,6 @@
     "author": "OpenFin",
     "dependencies": {
         "lodash": "^4.17.21",
-        "@openfin/core": "44.100.25"
+        "@openfin/core": "44.100.26"
     }
 }