This is interesting, it did not act like this before. But also a bit dangerous since I have to go through inferfaces now and see whether we need something as concrete and not just type.

I'm assuming this just affects classes, since interface and type do not have runtime counterparts...

@cenk1cenk2 what do you think about merging only this change as-is to start? Or, if that's not ok, what would you like to see here before merging?

Hey @karlhorky,

Sorry for the delay, I do not get what is the difference between moving the type export exactly as a type export from a file to the index.

Does this solve your issue?

Yes, it resolves my issue - I built locally and tested this (also mentioned in the PR description above):

Proof (type keywords in dist/index.d.ts):

The previous version didn't work: #747 (comment)

I don't know what the difference is either, but I would assume that tsdown or some other part of the build turns exported type-only exports into type + runtime exports when using the export * from './interfaces' pattern.

Current (broken)

packages/listr2/src/index.ts

export * from './interfaces'

packages/listr2/src/interfaces/index.ts

export type { Task as ListrTaskObject, TaskWrapper as ListrTaskWrapper } from '@lib'

Output:

export { ... TaskWrapper as ListrTaskWrapper, ...

After this PR (fixed)

packages/listr2/src/index.ts

export { ... type Task as ListrTaskObject, type TaskWrapper as ListrTaskWrapper } from './lib'

Output (note the type keyword inside the curly brackets):

export { ... type TaskWrapper as ListrTaskWrapper, ...

Thank you for the explanation yet again.

Reported the export * flattening + type qualifier dropping problem in rolldown-plugin-dts and rollup-plugin-dts too:

• export * loses type modifiers when flattening re-exports sxzz/rolldown-plugin-dts#95
• export * loses type modifiers when flattening re-exports Swatinem/rollup-plugin-dts#354

@cenk1cenk2 if you would publish a new release with this change, I'll take a look to verify it's working.

Ah i see it did not match eoth the commits so did not publish, sorry missed that. I will trigger one when infront of computer.

Sorry for that.

🎉 This PR is included in version 9.0.4 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

🎉 This PR is included in version 3.0.4 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

🎉 This PR is included in version 3.0.4 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

🎉 This PR is included in version 3.0.4 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

Nice, thanks for the release!

I can confirm that [email protected] now displays errors with importing ListrTaskWrapper without a type keyword when verbatimModuleSyntax: true is present in tsconfig.json 🎉

'ListrTaskWrapper' resolves to a type-only declaration and must be imported using a type-only import when 'verbatimModuleSyntax' is enabled. ts(1485)