emittery 1.0.0 → 1.0.2

package/index.js

@@ -10,8 +10,10 @@ const listenerRemoved = Symbol('listenerRemoved');
 let canEmitMetaEvents = false;
 let isGlobalDebugEnabled = false;
 
+const isEventKeyType = key => typeof key === 'string' || typeof key === 'symbol' || typeof key === 'number';
+
 function assertEventName(eventName) {
-	if (typeof eventName !== 'string' && typeof eventName !== 'symbol' && typeof eventName !== 'number') {
+	if (!isEventKeyType(eventName)) {
 		throw new TypeError('`eventName` must be a string, symbol, or number');
 	}
 }
@@ -32,7 +34,7 @@ function getListeners(instance, eventName) {
 }
 
 function getEventProducers(instance, eventName) {
-	const key = typeof eventName === 'string' || typeof eventName === 'symbol' || typeof eventName === 'number' ? eventName : anyProducer;
+	const key = isEventKeyType(eventName) ? eventName : anyProducer;
 	const producers = producersMap.get(instance);
 	if (!producers.has(key)) {
 		return;
@@ -216,13 +218,15 @@ export default class Emittery {
 	}
 
 	static get isDebugEnabled() {
+		// In a browser environment, `globalThis.process` can potentially reference a DOM Element with a `#process` ID,
+		// so instead of just type checking `globalThis.process`, we need to make sure that `globalThis.process.env` exists.
 		// eslint-disable-next-line n/prefer-global/process
-		if (typeof globalThis.process !== 'object') {
+		if (typeof globalThis.process?.env !== 'object') {
 			return isGlobalDebugEnabled;
 		}
 
 		// eslint-disable-next-line n/prefer-global/process
-		const {env} = globalThis.process || {env: {}};
+		const {env} = globalThis.process ?? {env: {}};
 		return env.DEBUG === 'emittery' || env.DEBUG === '*' || isGlobalDebugEnabled;
 	}
 
@@ -237,7 +241,7 @@ export default class Emittery {
 
 		producersMap.get(this).set(anyProducer, new Set());
 
-		this.debug = options.debug || {};
+		this.debug = options.debug ?? {};
 
 		if (this.debug.enabled === undefined) {
 			this.debug.enabled = false;
@@ -351,7 +355,7 @@ export default class Emittery {
 
 		enqueueProducers(this, eventName, eventData);
 
-		const listeners = getListeners(this, eventName) || new Set();
+		const listeners = getListeners(this, eventName) ?? new Set();
 		const anyListeners = anyMap.get(this);
 		const staticListeners = [...listeners];
 		const staticAnyListeners = isMetaEvent(eventName) ? [] : [...anyListeners];
@@ -380,7 +384,7 @@ export default class Emittery {
 
 		this.logIfDebugEnabled('emitSerial', eventName, eventData);
 
-		const listeners = getListeners(this, eventName) || new Set();
+		const listeners = getListeners(this, eventName) ?? new Set();
 		const anyListeners = anyMap.get(this);
 		const staticListeners = [...listeners];
 		const staticAnyListeners = [...anyListeners];
@@ -430,7 +434,7 @@ export default class Emittery {
 		for (const eventName of eventNames) {
 			this.logIfDebugEnabled('clear', eventName, undefined);
 
-			if (typeof eventName === 'string' || typeof eventName === 'symbol' || typeof eventName === 'number') {
+			if (isEventKeyType(eventName)) {
 				const set = getListeners(this, eventName);
 				if (set) {
 					set.clear();
@@ -469,13 +473,16 @@ export default class Emittery {
 		let count = 0;
 
 		for (const eventName of eventNames) {
-			if (typeof eventName === 'string') {
-				count += anyMap.get(this).size + (getListeners(this, eventName) || new Set()).size
-					+ (getEventProducers(this, eventName) || new Set()).size + (getEventProducers(this) || new Set()).size;
+			if (isEventKeyType(eventName)) {
+				count += anyMap.get(this).size
+					+ (getListeners(this, eventName)?.size ?? 0)
+					+ (getEventProducers(this, eventName)?.size ?? 0)
+					+ (getEventProducers(this)?.size ?? 0);
+
 				continue;
 			}
 
-			if (typeof eventName !== 'undefined') {
+			if (eventName !== undefined) {
 				assertEventName(eventName);
 			}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "emittery",
-	"version": "1.0.0",
+	"version": "1.0.2",
 	"description": "Simple and modern async event emitter",
 	"license": "MIT",
 	"repository": "sindresorhus/emittery",
@@ -13,6 +13,7 @@
 	"type": "module",
 	"exports": "./index.js",
 	"types": "./index.d.ts",
+	"sideEffects": false,
 	"engines": {
 		"node": ">=14.16"
 	},
@@ -58,7 +59,7 @@
 		"nyc": "^15.1.0",
 		"p-event": "^5.0.1",
 		"tsd": "^0.23.0",
-		"xo": "^0.52.3"
+		"xo": "^0.55.0"
 	},
 	"nyc": {
 		"reporter": [