Turned on type script definitions

[dolanmiu]

You guys are using Typescript already, so you just need to turn it on and allow it to be published

If this does not work, then I will think of other solutions :)

[chenguoguo]

Awesome! We will take a look!

[chenguoguo]

I'm trying this on my Mac and I got the following error, ideas?

npm ERR! Darwin 16.4.0
npm ERR! argv "/Users/guoguo/.nvm/versions/node/v4.0.0/bin/node" "/Users/guoguo/.nvm/versions/node/v4.0.0/bin/npm" "install"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! code ELIFECYCLE
npm ERR! [email protected] prepublish: `tsc --listFiles`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the [email protected] prepublish script 'tsc --listFiles'.
npm ERR! This is most likely a problem with the snowboy package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tsc --listFiles
npm ERR! You can get their info via:
npm ERR!     npm owner ls snowboy
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/guoguo/snowboy-release/npm-debug.log

[dolanmiu]

Is there more logs? That log doesn't explain anything apart from:

tsc --listFiles is bad

[chenguoguo]

Here was the log before the error (I was simply doing npm install under the root directory):

> [email protected] preinstall /Users/guoguo/snowboy-release
> npm install node-pre-gyp

[email protected] node_modules/node-pre-gyp
├── [email protected]
├── [email protected] ([email protected])
├── [email protected] ([email protected], [email protected], [email protected], [email protected])
├── [email protected] ([email protected], [email protected])
├── [email protected] ([email protected])
├── [email protected] ([email protected], [email protected], [email protected], [email protected], [email protected], [email protected])
├── [email protected] ([email protected], [email protected], [email protected], [email protected])
├── [email protected] ([email protected], [email protected], [email protected])
└── [email protected] ([email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected])
npm WARN engine [email protected]: wanted: {"node":">=4.2.0"} (current: {"node":"4.0.0","npm":"2.14.2"})
\
> [email protected] install /Users/guoguo/snowboy-release
> node-pre-gyp install --fallback-to-build

[snowboy] Success: "/Users/guoguo/snowboy-release/lib/node/binding/Release/node-v46-darwin-x64/snowboy.node" already installed
Pass --update-binary to reinstall or --build-from-source to recompile

> [email protected] prepublish /Users/guoguo/snowboy-release
> tsc --listFiles

lib/node/index.ts(55,14): error TS4073: Parameter 'model' of public method from exported class has or is using private name 'HotwordModel'.
lib/node/index.ts(100,63): error TS4019: Implements clause of exported class 'SnowboyDetect' has or is using private name 'SnowboyDetectInterface'.
lib/node/index.ts(104,24): error TS4063: Parameter 'options' of constructor from exported class has or is using private name 'DetectorOptions'.
/Users/guoguo/snowboy-release/node_modules/typescript/lib/lib.es6.d.ts
/Users/guoguo/snowboy-release/lib/node/index.ts
/Users/guoguo/snowboy-release/lib/node/node-pre-gyp.d.ts
/Users/guoguo/snowboy-release/lib/node/SnowboyDetectNative.d.ts
/Users/guoguo/snowboy-release/node_modules/@types/node/index.d.ts

[dolanmiu]

And there is no error on the master branch?

Damn, I have no idea then :(

[chenguoguo]

Correct, the master branch is able to build...

Were you able to reproduce the error on your side?

[dolanmiu]

Ok, I am able to reproduce it, and found a solution here:

microsoft/TypeScript#6307

To fix, HotwordModel, SnowboyDetectInterface and DetectorOptions need to be exported

[Pajn]

Yeah, just add an export keyword for them. You need for TypeScript users to be able to import them otherwise the compiler have no possibility to know what the type names refer to.

[chenguoguo]

@dolanmiu is exporting it enough? if yes could you update this pull request?

[dolanmiu]

I have not updated it to export yet, I can do that in a sec

[dolanmiu]

Done, can you check?

I ran npm run prepublish, and it gave no errors

[chenguoguo]

Merging it, thanks!