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

fix: readd vue-tempate-compilation plugin #831

Merged
merged 2 commits into from
Dec 27, 2024

Conversation

chakAs3
Copy link
Collaborator

@chakAs3 chakAs3 commented Dec 27, 2024

🔗 Linked issue

Fixes #808 and fixes #805.

📚 Description

This PR reverts #740 that removes of the vue-templation-template plugin, which was initially extended from the vue3-vite project vite config. However, removing it only masked the memory issue rather than addressing its root cause—the resolution of the vue ESM bundler alias.

The vue-templation-template plugin is essential for rendering stories with custom string templates, so we need to retain it.

To work around the memory issue and ensure template compilation works correctly, i have added vue as a dependency in the root project. ( playground )

The underlying problem appears to be related to how the vue alias is resolved. When it isn’t found in the root project, some unusual string replacement occurs, leading to recursive paths and the memory issue. Further investigation may be needed to fully understand why vue cannot be resolved from sub-packages.

… playgound, to solve out-of-memory issues, while resolving vue alias.
Copy link

netlify bot commented Dec 27, 2024

Deploy Preview for nuxt-storybook ready!

Name Link
🔨 Latest commit f3ba991
🔍 Latest deploy log https://app.netlify.com/sites/nuxt-storybook/deploys/676ec4f9f1388300084bf4f4
😎 Deploy Preview https://deploy-preview-831--nuxt-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tobiasdiez tobiasdiez changed the title Revert removing vue-tempate-compilation plugin, add vue dependency to playgound, to solve out-of-memory issues, while resolving vue alias. fix: readd vue-tempate-compilation plugin Dec 27, 2024
Copy link
Collaborator

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Thanks a lot for finding a solution to this annoying alias resolution issue! 🚀

@tobiasdiez tobiasdiez merged commit 2c5456f into main Dec 27, 2024
23 checks passed
@tobiasdiez tobiasdiez deleted the chaks/resolve-alias-vue-esm-bundler branch December 27, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decorators do not work Doen't work with compatibilityVersion: 4 and /app folder
2 participants