Antti Pihlaja

Electron support?

40

[package.json](https://github.com/apihlaja/node-irsdk/blob/master/package.json#L26) has only node.js version defined at the moment. But Electron binaries are available too. Should figure out which versions of Electron actually work.

Port to N-API?

2

It seems Node.js have new API for native addons, should take a look if it makes sense to port: https://github.com/nodejs/node-addon-api

Missing command: FFBCommand

1

v1.1.1 had all other commands, except FFBCommand. The implementation actually exist: https://github.com/apihlaja/node-irsdk/compare/feature/ffb-cmd?expand=1 The problem is, I have no idea what the command is supposed to do. If I cant come...

I'd assume this is issue still: https://github.com/patrickmoore/Mu/issues/3#issuecomment-249925901 I guess it's yet another unescaped string, should take a look.

`package.json` should have `os: win32` and `arch: x64` defined. Then it's possible to define this as optional dependency in your project. And maybe `npm` gives more meaning full error messages...