-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Error: [remix] dest already exists. #8813
Comments
Same error from this issue reported too 👉 #8814 I'm using CSS URLs too |
Closed my issue as a dupe. For reference, this broke in |
Can confirm that I am experiencing the same issue but my build fails with different fatal error when I try to remove vite v5.1.3 building for production...
✓ 95 modules transformed.
x Build failed in 392ms
Error [RollupError]: "default" is not exported by "app/tailwind.css", imported by "app/root.tsx".
file: /WebstormProjects/project/app/root.tsx:7:7
5: import robotoFlex from "@fontsource-variable/roboto-flex/index.css?url";
6:
7: import styles from "./tailwind.css"; |
Thanks for raising this. I've found the problem, I should have a fix ready soon. |
@pajasevi Your issue is expected when removing |
@markdalgleish this issue also occurs if you add extra entry points that are css files to the vite pipeline via plugins: e.g.:
|
@robertjpayne Ah, that's interesting. If the build passes, how would that extra CSS file be part of your Remix app? |
@markdalgleish it just gets processed as it's own entry point so it gets put into the assets folder and added to the vite manifest. It was working in 2.6.0 and earlier. Attached a really really minimal example. Though I did see there is a PR (#8818) for direct support without having to use a plugin to do so. For what its worth we have an unusual integration as we are using Vite to pre-process/bundle assets for other parts of our existing web stack and using Remix in SPA mode to power specific paths that require more interaction. |
This just bit me too. Excited about the vite support! |
Yeah, I'm experiencing same issue. It happens with any css file that is imported using
|
@robertjpayne Thanks for sharing that repro. I've just confirmed that the PR fixing this issue also fixes your issue: #8829 |
Hi! Please don't mind me cause I am new to this, but how do you find out that the error is caused by the css import with |
@Lenghak Because if you remove the |
Right, sir. So it does require some experiences in order to come up with some hypothesizes or even have a sense to suspect where the error could be. I, personally, wouldn't think that the it comes from the CSS cause the error shows entirely different message. Thank you for your explanation sir, I really appreciate it. |
This is fixed by #8829, it'll be out in the next patch release. |
🤖 Hello there, We just published version Thanks! |
@danestves @robertjpayne Can you please confirm that v2.7.2 fixes your issue? |
@markdalgleish It fixed it, thank you. ❤️ |
I can confirm this is fixed for me as well. Thank you guys. |
Fixed thanks! |
Reproduction
Using this template 👉 https://github.com/kiliman/remix-vite-template that doesn't differ very much from the official one 👉 https://github.com/remix-run/remix/tree/main/templates/vite-express
It gives the following error when running
remix vite:build
:This only happens to me after upgrading to
2.7.0
of RemixSystem Info
Used Package Manager
npm
Expected Behavior
Build normally like the previous version (2.6.0)
Actual Behavior
Breaking the builds
The text was updated successfully, but these errors were encountered: