Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

declarationMap support #113

Closed
kacpak opened this issue Sep 11, 2020 · 5 comments
Closed

declarationMap support #113

kacpak opened this issue Sep 11, 2020 · 5 comments
Labels
help wanted Extra attention is needed

Comments

@kacpak
Copy link

kacpak commented Sep 11, 2020

Hi, I haven't found information about it and I cannot test it out to verify myself this week, but I'm wondering if this plugin supports TypeScript's declaration map support?
If no - would you consider adding it in?

@Swatinem
Copy link
Owner

So rollup on its own has excellent sourceMap support. But we do some pre- and post-processing of whatever rollup does, apart from the synthetic AST that I generate. And I really don’t want to manually maintain a sourceMap for all that.

I suspect doing that is a huge amount of work, and I certainly don’t want to do that on my own :-D

@segevfiner
Copy link

@Swatinem
Copy link
Owner

Yes, magic-string would help here, and I converted the preprocessor code to it some time ago. There is also postprocessor code that fixes up some namespace declarations which would also need to be adapted to magic-string. I never got around to that.

PRs welcome though.

@aleclarson
Copy link
Contributor

Should keep this open in case someone wants to tackle it. But tag it with ”help wanted”.

@NWYLZW
Copy link
Contributor

NWYLZW commented Dec 20, 2024

Hi, I haven't found information about it and I cannot test it out to verify myself this week, but I'm wondering if this plugin supports TypeScript's declaration map support? If no - would you consider adding it in?

Now I have initially solved this problem, but it has not been merged with the upstream. Perhaps you can try @jiek/rollup-plugin-dts and provide relevant feedback. I hope it can help you.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants