Npm install doesn't include type definitions

[jdforsythe]

protobuf.js version: 5.0.3

After npm install there should be type definitions available.

In master, package.json appears to be configured for type definitions:

However the process to release must be rewriting the file to remove the reference to index.d.ts and the type definition files are also excluded from the npm release completely.

This is perhaps the cause of grpc/grpc-node#572

[JustinBeckwith]

Greetings! This library is currently on version 6.8.8. I just tested that version with TypeScript and all appears to be good. Please try the latest version, and let us know if you're still running into trouble?

[nicolasnoble]

@JustinBeckwith: grpc still has a direct dependency on protobufjs 5, for API compatibility reasons.

[JustinBeckwith]

Apologies... I had no idea!

[JustinBeckwith]

@nicolasnoble regardless though... I wouldn't say this is a relevant bug on this repository, is it? Seems to me it was fixed, we just can't upgrade to the version with the fix?

[nicolasnoble]

No, grpc can't upgrade to 6 without causing a breaking change to grpc itself. Now technically, the method used that gets people to use protobuf 5 with grpc is deprecated, but is also still around.