-
Notifications
You must be signed in to change notification settings - Fork 809
bug: Rollup Fails to Process .mts Files After Latest StencilJS Update #6193
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
Comments
Hi @a7medm7med - thanks for raising. Do you mind putting together a minimum reproduction 🙏? |
@johnjenkins Sorry, I meant module TypeScript files (.mts). Here's a reproduction of the issue: Important: This issue occurs when "moduleResolution" is set to "bundler" but does not happen when it's set to "node". With "bundler", TypeScript resolves .d.mts files, whereas with "node", it resolves .d.ts files. |
@johnjenkins @christian-bromann Great work! However, in our main project, the build process gets stuck at "generate lazy started ..." without any warnings or errors. This issue does not occur in the reproduction repo, and I can't figure out why it happens in our main project. For now, we've downgraded StencilJS back to v27 until we can determine the cause. Once we understand the issue, I'll open a new issue. Any insights from you guys would be greatly appreciated in helping us figure out this! |
@a7medm7med - sorry 😔 no real insights Could you run with the —debug flag; perhaps it shows something |
UPDATE: We've been checking each component to identify the issue and found that it's caused by our date-time-field component, which uses The problem occurs due to how we import
When we try to exit the process, we get an error, but we still can't determine whether the issue is caused by
|
@a7medm7med same problem here, it seems to be related to dayjs |
@mynamespace / @a7medm7med could you make a minimum repro with dayjs? 🙏 |
Here's the repo: https://github.com/a7medm7med/stencils-dayjs-issue/upload Then, try saving any file other than the component file ( In our main project, running:
doesn't even start the dev server and the compiler is stuck at "generate lazy started ..." |
why did this was closed? the problem is still present in 4.31.0, I'm stuck at 4.27.0. |
@mynamespace can you raise a new issue with a reproduction case? Thank you! |
Uh oh!
There was an error while loading. Please reload this page.
Prerequisites
Stencil Version
4.28.0
Current Behavior
After updating to the latest StencilJS version, which includes an upgrade to the latest Rollup version, Rollup no longer processes .mts files correctly. This issue affects our setup using @floating-ui and i18next. The same configuration worked fine in previous StencilJS versions.
Expected Behavior
To support .mts files like previous versions
System Info
No response
Steps to Reproduce
Code Reproduction URL
https://github.com/a7medm7med/stencils-rollup-issue
Additional Information
No response
The text was updated successfully, but these errors were encountered: