... actually makes an HTTP request. We have a local room dict, which is where the Already in rooms: line above comes from, so I'd say we should probably return without doing anything (except the _cb_join_room_success() call) if the requested room is already in there.
Or, I'll consider a change in sibylbot to check if we're in a room before attempting to join it.
It appears the matrix SDK isn't super smart about this. For example:
That is to say, the SDK call here: https://github.com/TheSchwa/sibyl/blob/23863ad4f2e0d91c6e177ea93e14c5b8da49821a/protocols/sibyl_matrix.py#L357
... actually makes an HTTP request. We have a local room dict, which is where the
Already in rooms:line above comes from, so I'd say we should probably return without doing anything (except the_cb_join_room_success()call) if the requested room is already in there.Or, I'll consider a change in sibylbot to check if we're in a room before attempting to join it.