[FIX] Memory issue (WIP)

[Lissy93]

This is a draft, experimenting with going back to the Vue CLI to handle the rebuilding.

Previously this approach introduced a bug which caused a rebuild loop, which was why the new watch-for-changes function was written, but that function is memory intensive. But there have been some architectural changes since, so I'm giving this a second shot.

This will not be merged until I can verify 100% that it does not re-introduce the old infinite rebuild loop bug.

Category: Bugfix

Overview

Briefly outline your new changes...

Issue Number #1500

New Vars N/A

Screenshot N/A

Code Quality Checklist (Please complete)

• All changes are backwards compatible
• All lint checks and tests are passing
• There are no (new) build warnings or errors
• (If a new config option is added) Attribute is outlined in the schema and documented
• (If a new dependency is added) Package is essential, and has been checked out for security or performance
• Bumps version, if new feature added

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
dashy	✅ Ready (Inspect)	Visit Preview	Mar 14, 2024 6:20am

[netlify]

✅ Deploy Preview for dashy-dev ready!

Name	Link
🔨 Latest commit	6268584
🔍 Latest deploy log	https://app.netlify.com/sites/dashy-dev/deploys/65f29716f1ea220008ced5d9
😎 Deploy Preview	https://deploy-preview-1510--dashy-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[CrazyWolf13]

So I did a lot of testing and could't replicate the very high consuming (1.5GB RAM usage) anymore.

I could also not replicate the rebuild loop even with the new command as stated above.

However there are still benefits when going back to vue cli ->

• RAM usage slighly lower possibly makes it running on less than 1GB RAM
• Faster
• Slightly less CPU intensive

I will test for some more time and keep you updated, if I notice anything.

Thanks!

[CrazyWolf13]

I have tested with that command for a long time now and did not experience a single loop, so I think this can be merged and pushed to docker images etc.