The case inconsistencies with the moderate
endpoints come from them being inspired by Mbin rather than Lemmy.
For max consistency, it might be an idea to kill moderate/unban
and just send a Boolean tomoderate/ban
(e.g. with "banned": true/false)
For this particular issue, perhaps not.
But the fact that both me and you are using PieFed instances and are participating in a comment chain started by a hexbear user demonstrates that there isn't much 'hard-coding' against other sites.
The commit that @davel@lemmy.ml referenced is for initial database setup. It's not an unreasonable default list to populate the 'banned_instances' table, and is trivial for admins to change after setup is complete.