astreatss / ultimateinvestigator Goto Github PK

The bot created this issue to inform you that pyup.io has been set up on this repo.
Once you have closed it, the bot will open pull requests for updates as soon as they are available.

tortoise-orm is... not dead, per se, but it's pretty evident that it's essentially on maintenance-only mode. Its typehinting is also lackluster, and its feature set is sometimes limiting.

Ideally, we would switch over to prisma (the Python variant), as the Realms Playerlist Bot has done. It produces amazing typehints and is well supported.

The only real issue with Prisma is that my current database provider for this bot, Supabase (yes, under its free tier, only the Playerlist Bot pays for a database right now), provides a URL for connections under IPv4 that Prisma cannot handle (mostly because the URL is technically invalidly formatted). The IPv6 URL can be handle by Prisma, but IPv6 has issues on the computers I use to test and run the bot, so some workaround has to be found. I have ideas, but who knows.

If I can't find a workaround, another library will have to be found, ideally one that can support psycopg as well as the current asyncpg as I've heard that psycopg can handle the string better. An alternative is switching to another (free) database provider, which is becoming more tempting by the day, admittedly.

These would, of course, not be triggered by a message, but probably by an admin noting how they've been found. This would allow admins to easily make posts for witness testimonies and perhaps add them into Truth Bullet calculations.

Currently, the bot loops through every Truth Bullet and alias that is from a specific channel to see if a message contains any Truth Bullet. This is... not good, so making one database query to do that would be better. Likely requires a raw query or a switch to Prisma.

Users can already disable giving out a role for the best Truth Bullet finders, but having the message in general might seem weird. It should be simple enough to add a toggle to entirely remove it.

Also, while I'm at it: it seems like a good time to make the phrasing of "Truth Bullet Finder"/"Detective" more consistent.