[RoomLifeCycleManager] `_operationInProgress` flag incorrectly set

[sacOO7]

• Currently, operationInProgress flag is set to false only when attach operation finishes.
• It should be set false at the end of other operations like Detach, Release, and Internal operations.
• Also, not sure if we really need this code snippet in place https://github.com/ably/ably-chat-js/blob/533c4d0314bcce942a30b4d7d6a012c3b081420d/src/core/room-lifecycle-manager.ts#L130-L134
• This sets operationInProgress to true when the room has just initialized and not performing any operation.

[sacOO7]

Also, I feel operationInProgress should be set to false at the end of mtx promise, not in between right? Like in case of Attach op, it's set in between, https://github.com/ably/ably-chat-js/blob/533c4d0314bcce942a30b4d7d6a012c3b081420d/src/core/room-lifecycle-manager.ts#L592-L600 While we are still emitting discontinuity events as a part of mutually exclusive/atomic attach op.

[sacOO7]

I mean, the flag can be simply set to true before starting mtx and set to false after mtx op ends.

[sacOO7]

Btw, if operationInProgress is only meant for Attach op, we should update doc + variable name for the same.