refactor: improves proxy event handling

src/events/ProxyEvent.ts

@@ -1,14 +1,15 @@
 import Nexo from "../types/Nexo.js";
 import { map } from "../utils/index.js";
 
-class ProxyEvent {
-  name: string;
-  proxy: Nexo.Proxy;
-  data: Record<string, unknown> = {};
+class ProxyEvent<Data = unknown> {
+  #name: string;
+  #proxy: Nexo.Proxy;
   #defaultPrevented: boolean = false;
   #returnValue: unknown;
 
-  constructor(name: string, proxy: Nexo.Proxy, data?: Record<string, unknown>) {
+  data: Data;
+
+  constructor(name: string, proxy: Nexo.Proxy, data?: Data) {
     if (!name.length) {
       throw Error(`ProxyEvent: event name cannot be empty`);
     }
@@ -17,12 +18,9 @@ class ProxyEvent {
       throw Error(`ProxyEvent ${name} proxy not found`);
     }
 
-    this.name = name;
-    this.proxy = proxy;
-
-    if (data) {
-      Object.assign(this.data, data);
-    }
+    this.#name = name;
+    this.#proxy = proxy;
+    this.data = data;
   }
 
   preventDefault(): void {
@@ -33,6 +31,14 @@ class ProxyEvent {
     this.#returnValue = value;
   }
 
+  get name(): string {
+    return this.#name;
+  }
+
+  get proxy(): Nexo.Proxy {
+    return this.#proxy;
+  }
+
   get defaultPrevented(): boolean {
     return this.#defaultPrevented;
   }

src/events/ProxyHandlerEvent.ts

@@ -1,12 +1,10 @@
 import Nexo from "../types/Nexo.js";
 import ProxyEvent from "./ProxyEvent.js";
 
-class ProxyHandlerEvent extends ProxyEvent {
-  constructor(
-    name: Nexo.proxy.handler,
-    proxy: Nexo.Proxy,
-    data?: Record<string, unknown>,
-  ) {
+class ProxyHandlerEvent<Data = unknown> extends ProxyEvent {
+  declare data: Data;
+
+  constructor(name: Nexo.proxy.handlerName, proxy: Nexo.Proxy, data?: Data) {
     super(`handler.${name}`, proxy, data);
   }
 }

src/events/handlers/apply.ts → src/handlers/apply.ts

@@ -1,8 +1,8 @@
-import Nexo from "../../types/Nexo.js";
-import { getTarget, getProxy, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getTarget, getProxy, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
-const apply = (mock: Nexo.Mock, that?: unknown, args?: unknown[]): unknown => {
+const apply = (mock: Nexo.Mock, that: unknown, args: unknown[]): unknown => {
   const proxy = map.tracables.get(mock);
   const data = map.proxies.get(proxy);
   const target = getTarget(data.target);

src/events/handlers/construct.ts → src/handlers/construct.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getProxy, getTarget, isTraceable, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getProxy, getTarget, isTraceable, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const construct = (mock: Nexo.Mock, args: unknown[]): object => {
   const proxy = map.tracables.get(mock);

src/events/handlers/defineProperty.ts → src/handlers/defineProperty.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getTarget, isTraceable, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getTarget, isTraceable, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const defineProperty = (
   mock: Nexo.Mock,

src/events/handlers/deleteProperty.ts → src/handlers/deleteProperty.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const deleteProperty = (mock: Nexo.Mock, key: Nexo.objectKey): boolean => {
   const proxy = map.tracables.get(mock);

src/events/handlers/get.ts → src/handlers/get.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getProxy, getTarget, isTraceable, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getProxy, getTarget, isTraceable, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const get = (mock: Nexo.Mock, key: Nexo.objectKey): unknown => {
   const proxy = map.tracables.get(mock);

src/events/handlers/getOwnPropertyDescriptor.ts → src/handlers/getOwnPropertyDescriptor.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getTarget, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getTarget, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const getOwnPropertyDescriptor = (
   mock: Nexo.Mock,

src/events/handlers/getPrototypeOf.ts → src/handlers/getPrototypeOf.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getTarget, isTraceable, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getTarget, isTraceable, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const getPrototypeOf = (mock: Nexo.Mock): object => {
   const proxy = map.tracables.get(mock);

src/events/handlers/has.ts → src/handlers/has.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const has = (mock: Nexo.Mock, key: Nexo.objectKey): boolean => {
   const proxy = map.tracables.get(mock);

src/events/handlers/set.ts → src/handlers/set.ts

@@ -1,6 +1,6 @@
-import Nexo from "../../types/Nexo.js";
-import { getTarget, isTraceable, map } from "../../utils/index.js";
-import ProxyHandlerEvent from "../ProxyHandlerEvent.js";
+import Nexo from "../types/Nexo.js";
+import { getTarget, isTraceable, map } from "../utils/index.js";
+import ProxyHandlerEvent from "../events/ProxyHandlerEvent.js";
 
 const set = (mock: Nexo.Mock, key: Nexo.objectKey, value: unknown): boolean => {
   const proxy = map.tracables.get(mock);

src/types/Nexo.ts

@@ -1,11 +1,9 @@
 import Nexo from "../Nexo.js";
-import PxEvent from "../events/ProxyEvent.js";
 
 declare namespace Nexo {
   type functionLike = (...args: unknown[]) => unknown;
   type traceable = object | functionLike;
   type objectKey = string | symbol;
-  type ProxyEvent = PxEvent;
 
   interface options {}
 
@@ -30,24 +28,15 @@ declare namespace Nexo {
       sandbox: Map<objectKey, unknown>;
     };
 
-    type handler =
-      // getters
+    type handlerName =
       | "get"
       | "has"
       | "deleteProperty"
       | "getOwnPropertyDescriptor"
-      | "get"
-      | "has"
-      | "deleteProperty"
-      | "getOwnPropertyDescriptor"
-      // setters
       | "set"
       | "defineProperty"
-      // function calls
       | "apply"
-      // class instances
       | "construct"
-      // objects
       | "getPrototypeOf";
   }
 }

src/utils/getProxy.ts

@@ -2,7 +2,7 @@ import Nexo from "../types/Nexo.js";
 import map from "./map.js";
 import findProxy from "./findProxy.js";
 import isTraceable from "./isTraceable.js";
-import handlers from "../events/handlers/index.js";
+import handlers from "../handlers/index.js";
 
 const getProxy = (scope: Nexo, target: Nexo.traceable | void): Nexo.Proxy => {
   // find proxy by target