A new cleanup pass has just been completed on mediabot_v3, with a focus on reliability and consistency rather than new features.
This round of work mainly addressed command management and schema coherence:
active flag in PUBLIC_COMMANDSshowcmd so it now displays the command status (active or on hold)DBCommands.pmThe goal was simple: make command handling more predictable and avoid inconsistencies between the code and the database schema.
Before these fixes, the code expected an active field that was not yet present in the SQL schema. That kind of mismatch is exactly the sort of thing that leads to annoying bugs, confusing behavior, and wasted debugging time.
Now the behavior is clearer:
showcmd reflects the real state of the commandThis cleanup was intentionally limited to the core command/database logic.
The radio-related parts were left aside on purpose, so the review could stay focused on general Mediabot stability and maintainability.
Nothing flashy here, just useful work:
Sometimes the best magic is simply removing the hidden curses from the old code.
You must be logged in to reply.