ProgramData version detection + Semver Version sorting

[QbDesu]

Fix proposal for #149
It's built on top of #139 so it's also a fix for #138. Which however still didn't explain why a lower version folder was sometimes detected, as there seemed to be a higher version folder detected.
This is also a bigger refactoring in general that also changes the validate functions which the original PR doesn't.

Tested to be working on Windows, but I did have to set it up the folders manually because discord installed itself in the normal appdata directory.

[QbDesu]

Interesting thing that the current implementation in this PR does not account for:
a 1.0.9002 folder in %PROGRAMDATA%\%USERNAME%\Discord, so without the app- prefix, reported to contain nothing but a .first-run file.
Edit: actually it does account for it, but still an interesting observation.

[QbDesu]

I reworked the complete PR. Needs some testing on Windows but seems to work correctly on Linux.

[QbDesu]

Btw, I wonder when the first person called snap complains that they aren't supported.

[dav1312]

Why is package-lock.json there? Isn't it generated already using npm i?

[QbDesu]

Yes, but the purpose of the package-lock.json is to record the versions of all installed dependencies, and the dependencies' dependencies, and so on. This is so that it can be reproduced. The package-lock.json will be used when running the npm ci which will try to repoduce the exact same package versions that are recorded in the file.

This is so that when a package gets an update that would break things or make them behave weirdly it won't mess up your build automation because it will use the same versions you used during development and for testing.

[zerebos]

Technically package-lock.json should not be there as this project is using yarn and generates a yarn.lock file https://github.com/BetterDiscord/Installer/blob/main/yarn.lock

[QbDesu]

replaced the package-lock.json and updated yarn lockfile instead

[QbDesu]

had to fix it because I accendentially commit some changes that were essentially undoing previous commits