fix: lifecycle operation presedence

The async-mutex library works as highest value = higher priority. This change reverses the priority in the lifecycle manager to fix this.
ably · Nov 25, 2024 · 8136f0c · 8136f0c
1 parent 5ae54bd
commit 8136f0c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/core/room-lifecycle-manager.ts b/src/core/room-lifecycle-manager.ts
@@ -31,11 +31,13 @@ export interface ContributesToRoomLifecycle extends HandlesDiscontinuity {
 /**
  * The order of precedence for lifecycle operations, passed to the mutex which allows
  * us to ensure that internal operations take precedence over user-driven operations.
+ *
+ * The higher the number, the higher the priority.
  */
 enum LifecycleOperationPrecedence {
-  Internal = 0,
+  Internal = 2,
   Release = 1,
-  AttachOrDetach = 2,
+  AttachOrDetach = 0,
 }
 
 /**