Add TypeScript definitions

[botic]

Directly ship the type definitions with this package and add a type property to package.json.

[Findus23]

Hi,

Feel free to create a Pull Request for this. I am not so familiar with creating proper type definitions.

[botic]

Yep, but I never did that before and my TypeScript knowledge is very basic at the moment. That's why I thought opening an issue would be more appropriate.

[Findus23]

I rewrote the library in typescript, so this should also be fixed now.

It would be amazing if you could test it (see the typescript branch and the 3.0.0 beta releases) to see if everything still works and the typescript-features themselves also work (I am not completly sure I published it to npm correctly as it might not contain the index.ts file)

[botic]

Thanks for the effort! You should include a types property in the package.json as well. The type definition file must be included in the files list. I used this doc page for my own packages: https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html#including-declarations-in-your-npm-package

Otherwise I get this error:

[Findus23]

@botic
The thing is that now that the library is written in typescript there is no type declaration file, just the source. Do you know how this should be documented properly in the packages.json

[botic]

@botic
The thing is that now that the library is written in typescript there is no type declaration file, just the source. Do you know how this should be documented properly in the packages.json

But you need to include the type definition in the published package, since the NPM package will only ship as compiled JavaScript code and not "Typescript-native".

[Findus23]

@botic And how would I do this during the build process?

And why wouldn't I want to ship the typescript file too in the npm package? Wouldn't that allow the typescript compiler to use the full source during its compile run (instead of just the compiler output and the d.ts file)?

[botic]

If you don't add the type definitions to the package, users of the package will have no type information. Why re-writing in Typescript if not shipping the typings with the final package?

Add a "declaration": true to the tsconfig.json and the Typescript compiler will generate a corresponding type definition for each compiled JavaScript file. Since this Matomo tracker only has an index.js as output, the you just add it to the package.json and it will be shipped with the final NPM package.

"types:" "index.d.ts",
"files": [
    "index.js",
    "index.d.ts"
]

After publishing a stable version, a blue TS icon will show users that the packages includes type information:

[Findus23]

Many thanks, now I got it.

The thing I did wrong was that I forgot to export the main class which caused the type declarations to be empty. 95f60cc should have fixed this, it would be great if you could test it.

[Gustavhol]

@Findus23 you need to change index.js to index.ts in package.json row 18. Else there's nothing exported.

[Findus23]

@Gustavhol
Are you sure? I had it before (95f60cc#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L18-R21)
but @botic mentioned that I need to put the compiled js in the package.json.

Do you have any page that explains how the package.json should look like?

Also this package should ideally work both for Typescript and non-Typescript users.

[botic]

This packages should be useable without any TypeScript installed. So you ship the JavaScript as main + TypeScript type declarations. There is also an official guide for this: https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html#including-declarations-in-your-npm-package