bjohnso5 / hue-hacking-npm Goto Github PK

Hi there,
I'm using "hue-hacking-npm" as a dependency in my project.
Everything worked fine in the past.
However, with the latest updates of some of my other frameworks (Electron, etc.) I now get the following error on startup:

Cannot find module ... node_modules/hue-hacking-node/dist/index.js'. Please verify that the package.json has a valid "main" entry.

I guess that's because "hue-hacking-npm" doesn't distribute the dist-folder itself, but the main-file being referenced in package.json lies in that folder.

...
  "description": "Utility to control Philips Hue light bulbs",
  "main": "dist/index.js",
...

Is there a chance that could be fixed with an upcoming release? 😃
Thanks a lot!

Here's the getLamps() code:

In the last line, the returned data is converted from an object into an array. The object I see in my tests looks like:

{
"1": Lamp,
"2": Lamp,
"3": Lamp
}

But the resulting array is zero indexed and so to be able to call setColor I need a 1 based index. So I end up adding 1 to the index. I'm wondering if this is deliberate. I assume that the Hue API will never skip number, nor return indexes out of sequence, but I can't see any way to actually get the index out of the getLamps() response. Would you be open to a PR which added a field called index to the Lamp type?

The npm script that is run on Travis CI to automatically generate coverage reports and ship them to coveralls.io is broken, creating empty reports.

Investigation into what change broke the toolchain and how to fix it is required.

Gets do not return state as expected using needle. Investigate.

Hi there,

it would be awesome if we could get an updated version of this npm-module which contains the latest fixes/additions from @benknight.

Especially this one:
bjohnso5/hue-hacking@6739545

I need this new function ...
CIE1931ToHex : function (x, y, bri) {
... which is not available in the current version of this project/npm.

Thanks!