refactor: use the `fetch` API

[lishaduck]

Status

READY

Description

Rewrote it all! :)

In more seriousness: 1) Uses the fetch API uniformly. 2) Adds a new use of await and removes a usage of async. Neither should actually affect the behavior. 3) Moved some code's scope to make it cleaner.

Probably review one-commit-at-a-time?

Closes: #32

Type of Change

• [ ] ✨ New feature (non-breaking change which adds functionality)
• [ ] 🛠️ Bug fix (non-breaking change which fixes an issue)
• [ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
• [x] 🧹 Code refactor
• [ ] ✅ Build configuration change
• [ ] 📝 Documentation
• [ ] 🗑️ Chore

[lishaduck]

Oh, another thing is that this is in sloppy mode, probably worth adding a "use strict" directive. I don't particularly want to file another PR, but if y'all want to keep this small(er), I don't mind if y'all add it yourselves.

[wolfenrain]

Oh, another thing is that this is in sloppy mode, probably worth adding a "use strict" directive. I don't particularly want to file another PR, but if y'all want to keep this small(er), I don't mind if y'all add it yourselves.

I am not against adding that here directly, afaik it is just a one-liner right?