"const" declarations from .d.ts modules should be marked with "declare"

[aleclarson]

A 'declare' modifier is required for a top level declaration in a .d.ts file.

Repro: https://github.com/aleclarson/repro/tree/rollup-dts-1

I also tested with .ts modules, and that works fine. The problem is only with .d.ts modules.

Note: This also happens with function exports.

[Swatinem]

the plugin will use whatever code is provided.
If the source .d.ts files are missing declare modifiers (likely because they are hand-written), the plugin will not interfere with that.

[cspotcode]

@Swatinem shouldn't the plugin add the declare modifier? Is that too messy to implement? As far as I can tell, it is implied by export, so whenever the input has export, the output should have declare.

As I understand it, rollup-plugin-dts is moving the export modifier to a different statement, which means it is changing and breaking the input.

[Swatinem]

I think by now I have code that properly adds the missing attributes. Do you have a specific snippet that is failing?

[cspotcode]

@Swatinem sure, here's a minimal reproduction: https://github.com/cspotcode/repros/tree/Swatinem/rollup-plugin-dts/24

Thanks for taking a look.

[Swatinem]

Ah thanks for the repro case, this was indeed straight forward to fix ;-)