Just passing along my thoughts from first time fiddling with addons.
For Marson's rest Addons, it's needing to keep spotHostiles (Player.lua) duplicated on their end, since it is a local function of Player.lua. If this was a member function, it'd be possible to just use the base implementation if no changes are needed.
Rest and Run could use a way to just add additional logic into them without overriding the function. Because the entire function is being superloaded over, fixes that are made (especially subtle ones) don't get brought in without the addon writer adding things. Additionally, you can't do multiple things adding to them. If I was to make an addon working with either rest or runcheck, it would collide directly with the existing one.
Rest Addons
Moderator: Moderator
Rest Addons
ToME Tips - auto-generated spoilers for ToME. - someone else made. I find super awesome, so spreading as well.