fix: tsconfig conflict issue that occurs with ts-proto-descriptors

[protango]

When importing ts-proto-descriptors into a typescript project, it will try to import the typescript source code, and not the generated javascript. This is an issue when the dependent project's tsconfig file does not agree with the tsconfig for ts-proto.

For example, my project depends on ts-proto-descriptors has a tsconfig that has enabled esModuleInterop. But the ts-proto tsconfig does not have this option set, meaning the ts-proto-descriptors source is not compatible with my project and throws the following typescript errors:

node_modules/ts-proto-descriptors/google/protobuf/compiler/plugin.ts:609:5 - error TS2367: This condition will always return 'true' since the types 'Constructor<Long>' and '{ default: LongConstructor; prototype: Long; MAX_UNSIGNED_VALUE: Long; MAX_VALUE: Long; MIN_VALUE: Long; NEG_ONE: Long; ONE: Long; ... 11 more ...; fromValue(val: string | ... 2 more ... | { ...; }): Long; }' have no overlap.

609 if (util.Long !== Long) {
        ~~~~~~~~~~~~~~~~~~

node_modules/ts-proto-descriptors/google/protobuf/descriptor.ts:4526:5 - error TS2367: This condition will always return 'true' since the types 'Constructor<Long>' and '{ default: LongConstructor; prototype: Long; MAX_UNSIGNED_VALUE: Long; MAX_VALUE: Long; MIN_VALUE: Long; NEG_ONE: Long; ONE: Long; ... 11 more ...; fromValue(val: string | ... 2 more ... | { ...; }): Long; }' have no overlap.

4526 if (util.Long !== Long) {
         ~~~~~~~~~~~~~~~~~~


Found 2 errors.

This PR addresses the underlying issue by exporting only the generated javascript from ts-proto-descriptors (along with ts definition files for type safety). This means that it cannot be broken by the outer project's ts-config because it is already compiled.

From a developer perspective, there will be no change as the definition files will maintain type-safety, as if you were working with typescript directly. However if you really wanted to import the TS source, then you could also do that by just specifying the path explicitly (eg. import { FileDescriptorProto } from "ts-proto-descriptors/index").

NOTE: This change will require a release to NPM for https://www.npmjs.com/package/ts-proto-descriptors

[protango]

Also just realised this PR should resolve #217

[stephenh]

@protango thanks for the fix; I've pushed ts-proto-descriptors@1.3.0 up to npm, can you try it out and let me know if it works?

[stephenh]

🎉 This PR is included in version 1.81.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀