@3dsource/angular-unreal-module 0.0.143-dev.8 → 0.0.143

package/fesm2022/3dsource-angular-unreal-module.mjs

@@ -7191,20 +7191,21 @@ class UnrealSceneComponent {
         this.unrealConfig = inject(UNREAL_CONFIG);
         this.autoHighResolution = false;
         this.destroyRef.onDestroy(() => {
-            // Resource release MUST be synchronous and must NOT depend on the
-            // route-scoped UnrealEffects. provideUnreal() is registered per 3D route,
-            // so on navigation the route's environment injector (and its
-            // UnrealEffects) can be torn down before the
-            // `destroyUnrealScene → destroyRemoteConnections` effect chain — which
-            // closes the PeerConnection/WebSocket — gets a chance to run, leaking the
-            // stream session. The engine services are providedIn:'root' singletons,
-            // so we close them directly here. close()/closePC() are idempotent, so
-            // the effect chain re-running them (when it does run) is a harmless no-op.
-            this.webRtcPlayerService.closePC();
+            // Mirror of the `destroyRemoteConnections$` effect's teardown tap, run
+            // directly here as a safeguard. provideUnrealModule (effects) is
+            // registered per 3D route, so on navigation the route's environment
+            // injector — and its UnrealEffects — may be torn down before the
+            // `destroyUnrealScene → destroyRemoteConnections` effect chain closes the
+            // WebSocket/PeerConnection, which would leak the stream session. The
+            // engine services are providedIn:'root' singletons, so we close them
+            // directly. Keep the SAME calls, args and order as the effect's tap
+            // (close → closePC); both are idempotent, so whichever runs second is a
+            // harmless no-op.
             this.signallingService.close({
                 code: WSCloseCode_FORCE_CIRRUS_CLOSE,
                 reason: DisconnectReason.Destroy,
             });
+            this.webRtcPlayerService.closePC();
             // State cleanup is reducer-driven, not effect-driven: the
             // `destroyUnrealScene` reducer resets the whole UnrealState to
             // initialState (cirrusConnected/viewportReady/… → false). Reducers run in