`CacheResult` should in theory have either a `file != null` a `future !=
null` or be `disposed`. Apparently this invariant of `CacheResult` is
either being violated somewhere, or the `synchronized` blocks arent as
synchronized as id hoped they were. In fact, `dispose()` does not even
delete the file, so i can really only see this happening because the
first `synchronized` block that writes the file and the second
`synchronized` block that reads from the file hold the same lock.

I have no idea how this would happen, but hopefully this fixes it (since
the dispose didn't have a threading issue reported so far, i feel more
confident leaving the .deleteOnExit in there, but I'm also wrapping any
potential IOExceptions during access, because I am just so confused how
the internal state was broken.

* ApiUtil: Add cache with per request timeout and per class histogram

* MinionHelper: Only load minion helper data when needed

* Api: Make api response processing more async.

* Lower cache for /pv to 30 seconds and rename cacheDuration to max age

* Disk cache for the API

* Print api errors in console.

* No reason not to just append.

* Censor API Key

---------

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: nea <nea@nea.moe>

* Added old SkyBlock Menu.

* Execution!

* Add cache.

* Bingo has already enough to do.

* Typo.

* Revert "Typo."

This reverts commit b4a1c385e0c410b1e111797b8d39e7ff64b09ef5.

* Revert "Bingo has already enough to do."

This reverts commit 6e004d2d65dff47ea3bee5c5789cb725724df6ed.

* I am lazy.

* The map is lazy too.

* Hypixel moving features behind paywall.

* Add red text to the setting too.

* SEALED

* Fixed Booster Cookie checks. Reworked CookieWarning, kept same logic but accidentally added profile switch support.

* /trades does not require booster cookie.

* Allowing middle clicks (and any other mouse click combination)

---------

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>