jackonelli / aoc-bot Goto Github PK

A daily leaderboard update push to Discord
Prints the leaderboard, including indications of position changes from last day.

For portability, like building on rbpi.
And sanity, frankly.

Testing is hard because actual AoC updates are hard rare.

We ought to artificially construct two jsons:
test_prev.json
test_later.json

where test_later.json has a lot of corner case events registered, e.g.:

• New players
• Old players leaving the leaderboard
• New star 1 only
• New star 2 only
• Both starts new.

Maximally efificient snail requests

Good in terminal,
bad in discord.
Pad with blankspaces work?

Do not trust the calculated local score in json.
Calculate based on day completion instead.

Found what I was looking for:

https://serenity-rs.github.io/serenity/current/serenity/http/client/struct.Http.html#method.send_message

Instead of starting a stupid loop in the message fn (which works surprisingly well), the link above seems to be exactly what I want.

It should rather be:
new_players and lost_players
And the diff check should be more thorough. Number of players is suff. but not nec. cond for diff.

We all ought to face our (async) fears

reasonable to have at least two crates:

• aoc-data
• aoc-bot

I.e. non-panicking error handling.
I.e. actual error handling.
The bot should never exit on error

What happens if you stars from diff days?

Given the plan (#14) to split into two crates:

• aoc-bot
• aoc-data

The aoc-data should not depend on serenity. Curently, it does so through the AocError, which is easily fixed.

Someone may want it