feat: refine SuggestedEventName type for better type inference and remove redundant type checks in tests

Author: ido-pluto

src/utils/EventEmitter.ts

@@ -19,7 +19,7 @@ type KnownEventName<Events extends object> = Extract<{
 }[keyof Events], EventName>;
 type EventListener<Events extends object, Key extends EventName> = Key extends keyof Events ? Events[Key] extends AnyListener ? Events[Key] : never : AnyListener;
 type EventListenerArgs<Events extends object, Key extends EventName> = ListenerArgs<EventListener<Events, Key>>;
-type SuggestedEventName<Events extends object, ExtraEvents extends EventName> = KnownEventName<Events> | (ExtraEvents & {});
+type SuggestedEventName<Events extends object, ExtraEvents extends EventName> = KnownEventName<Events> | (ExtraEvents & object);
 type HasBroadEventKeys<Events extends object> = string extends keyof Events ? true : symbol extends keyof Events ? true : false;
 type DefaultExtraEvents<Events extends object> = HasBroadEventKeys<Events> extends true ? EventName : never;
 

test/eventEmitter.test.ts

@@ -2,7 +2,6 @@ import {describe, expect, test, vi} from "vitest";
 import {EventEmitter} from "../src/utils/EventEmitter.js";
 
 type Assert<T extends true> = T;
-type IsAny<T> = 0 extends (1 & T) ? true : false;
 type IsExact<T, Expected> = [T] extends [Expected] ? ([Expected] extends [T] ? true : false) : false;
 
 type TestEvents = {
@@ -48,10 +47,6 @@ describe("EventEmitter", () => {
         const emitter = new EventEmitter<TestEvents>();
 
         emitter.on("data", (value, label) => {
-            type ValueIsNotAny = Assert<IsAny<typeof value> extends false ? true : false>;
-            type LabelIsNotAny = Assert<IsAny<typeof label> extends false ? true : false>;
-            type ValueIsNumber = Assert<IsExact<typeof value, number>>;
-            type LabelIsString = Assert<IsExact<typeof label, string>>;
             const typedValue: number = value;
             const typedLabel: string = label;