index.d.ts is not bundled with pixi-tilemap 2.1.0 about tilemap

[ShukantPal]

@NintendoBen This will be fixed in the next release of pixi.js (pixijs/pixi.js). Each scoped @pixi/ package will ship with its own typings.

[ShukantPal]

@NintendoBen Here's the PR btw: pixijs/pixijs#6849

[ShukantPal]

Wait, the index.d.ts file isn't in the shipped npm package?

[ShukantPal]

@NintendoBen 2.1.3 was just released, can you try again?

[ben-zabloski]

I think the problem I'm having is just the nature of TypeScript not working well with the existing pixi plugin architecture which benefits from JavaScript's flexibility. I'll probably just have to find a way to make TypeScript happy 😄

[ben-zabloski]

As a temporary workaround, I created a copy of the pixi-tilemap.d.ts type definitions in my own source/types and retargeted the @pixi module imports to point at "pixi.js" instead and smoothed out a couple warnings here and there like changing "String" to "string", etc... So far things seem to be working ok!

[ben-zabloski]

Thanks for the info @SukantPal ! 😁

That will definitely solve the pixi type definition issue(s), but I think that the pixi-tilemap definitions will still be missing until they get bundled within the pixi-tilemap npm package root? Unless I’m missing something - I don’t consider myself a TypeScript expert!

[ben-zabloski]

That’s right I believe 😅

[ShukantPal]

https://github.com/pixijs/pixi-tilemap/blob/master/index.d.ts

How could that be? @ivanpopelyshev ^^

[ShukantPal]

@NintendoBen Dod you figure out why the index.d.ts is missing?

[ben-zabloski]

I’m not sure to be honest - just running npm install, it seems like the index.d.ts file doesn’t exist in the node_modules/pixi-tilemap folder. I’m not that familiar with the npm package publish process, so I’m not sure what could be going wrong.

[ben-zabloski]

Will do - I’ll try it shortly 😊

[ben-zabloski]

@SukantPal - looks like the index.d.ts file is included now! When I swap out my custom pixi-tilemap.d.ts definitions with the bundled pixi-tilemap/index.d.ts definitions, they're now picked up automatically by Visual Studio Code. The next trick will be to have the @pixi package typings generated and included 😄

Thanks for responding so quickly to this 😁