Support Electron v13-14 and Node v16

[ykhwong]

Support Electron v13-14 and Node v16

[quuu]

bump please ^

[ykhwong]

If we don't use AppVeyor anymore, please consider deleting AppVeyor's webhook from the repo setting.

[ash0x0]

@ykhwong did you test using this? platform, env?

[ykhwong]

Yes, I tested with electron abi-89 and win10 x86_64 and confirmed it to work. I don't have a testing environment for the other platforms for now.

[sonukapoor]

@ykhwong Would you mind providing a fix for 15 as well? I am sure a lot of people will appreciate that.

[ykhwong]

@ykhwong Would you mind providing a fix for 15 as well? I am sure a lot of people will appreciate that.

You can find it here: #363

[sonukapoor]

Thanks @ykhwong. Does this also fix the non-context-aware error mentioned here?

#387

[ykhwong]

Thanks @ykhwong. Does this also fix the non-context-aware error mentioned here?

#387

I don't think so.

[sonukapoor]

@ykhwong Any idea how to fix this?

[ykhwong]

@sonukapoor I haven't encountered the 'non-context-aware' error with the pull request branch ykhwong:update-electron-v15, so I have no idea how to fix it.

[sonukapoor]

@ykhwong If you try to load the module in the renderer you should see an error. The solution is to use NAN_MODULE_WORKER_ENABLED as mentioned in the linked PR. @marcelblum mentioned though that the clean up is missing, so the PR is sitting there for an C++ expert to come and help.

[marcelblum]

@sonukapoor sad to say but unfortunately you're better off just using the actively maintained uiohook-napi which works well in latest Electron. iohook seems to be dead.

[sonukapoor]

@marcelblum Do you know how to use this within the Electron renderer?

[marcelblum]

@sonukapoor uiohook-napi works just like iohook, almost a drop-in replacement. I found a few keycodes were different in certain platform-specific situations but that was pretty much the only difference. uiohook-napi binds to the same uiohook library.

[sonukapoor]

@marcelblum I was trying to load this using __non_webpack_require and the prebuilt version for windows, but it doesnt seem to work. I get the node.napi.node is not a valid Win32 application. I will try to import it directly.