[Fix/Refactor] Make all logs async

[arielj]

This PR is a continuation of #3384.

In that PR I moved the lastPlay logs to be async, and this PR moves the rest of the logs to behave the same.

This has 2 main benefits:

• we write logs in batch every 1 second when there are enqueued message instead of writing once for every message (avoid writes when they happen too quickly which is an expensive operation)
• if, for any reason, writing to a log fails, the main action that is logging something is not impacted and can continue normally ( should fix Games don't launch a second time #3289 for example)

I included one fix: there was an error in the logs when stopping a sideloaded app

---

Use the following Checklist if you have changed something on the Backend or Frontend:

• Tested the feature and it's working on a current and clean install.
• Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
• Created / Updated Tests (If necessary)
• Created / Updated documentation (If necessary)

[flavioislima]

Maybe run it if the ENV for CI is true.
Something like:
if (process.env.GITHUB_ACTIONS) {

[arielj]

one thing I'm seeing now is that this is testing a case that will never happen

because from the getPath method docs, when it returns a string it's a valid path and when it's invalid it throws an error (that we are not catching), it should never return an invalid path https://www.electronjs.org/docs/latest/api/app#appgetpathname

we seem to be handling an invalid path incorrectly

I'd rather remove this test to avoid a false sense of correctness, we should actually do a try/catch in createNewLogFileAndClearOldOnes and return if the path is invalid

[arielj]

I updated the function to try/catch and re-implemented the test to reflect the Error thrown.

[flavioislima]

LGTM