This does not work for InstanceMethod because there is a branch check enabled at https://github.com/nodejs/node/blob/main/src/js_native_api_v8.cc#L1028.

Thank you for working on this!

It's not clear from the comments if this is working yet. It sounds like maybe not?

I would suggest adding a few more tests

• Make a C++ class that inherits from Event, then pass it to a native event target

  const myEvent = new CPPClassThatInheritsFromEvent('nameOfEvent');
  const target = new EventTarget();
  const e = await new Promise(resolve => {
   target.addEventListener('nameOfEvent', resolve, { once: true });
   target.dispatchEvent(myEvent);
  });
  assert(e === myEvent);

• Make a C++ class that inherits from EventTarget. Maybe sure you can use EventTarget's functionality

  const target = new CPPClassThatInheritsFromEventTarget();
  const event = new Event('foo');
  const e = await new Promise(resolve => {
   target.addEventListener('foo', resolve, { once: true });
   target.dispatchEvent(event);
  });
  assert(e === event);

This does not work for InstanceMethod because there is a branch check enabled at https://github.com/nodejs/node/blob/main/src/js_native_api_v8.cc#L1028.

Hi @legendecas , is there anything we can do to address this branch check? Is it absolutely needed?