I've fixed part of it
here. OnDisconnect isn't called for sourcetv spectators during mapchanges, but OnClientPutInServer is after they loaded the new map. Should I mimic the core forwards and always call the disconnect forwards on mapchange and the onconnect ones after mapchange? There is no "player_disconnect" event to catch actual disconnects for sourcetv clients though.