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

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

@@ -4732,15 +4732,6 @@ const RESIZE_DEBOUNCE_MS = 300;
 const FUNCTION_KEY_MIN = 112;
 const FUNCTION_KEY_MAX = 123;
 const TAB_KEY_CODE = 9;
-// Browser-destructive / navigation keys (Backspace can navigate back, Enter and
-// Delete leak into the stream). These are ignored in the viewport regardless of
-// dev mode — unless an HTML input is focused — to match non-dev-mode behavior.
-const BROWSER_DESTRUCTIVE_CODES = new Set([
-    'Backspace',
-    'Delete',
-    'Enter',
-    'NumpadEnter',
-]);
 class InputService extends SubService {
     constructor() {
         super();
@@ -5166,15 +5157,9 @@ class InputService extends SubService {
         };
         this.startListenKeys(keyBlock);
     }
-    /**
-     * Keys that must never reach the Unreal stream from the viewport — Backspace
-     * (browser back navigation), Enter and Delete — regardless of dev mode. An
-     * HTML input keeps them so text entry still works; only the viewport ignores
-     * them. Applied independently of the configurable key filter.
-     */
     isIgnoredViewportKey(event) {
         const isInputFieldFocused = event.target?.localName === 'input';
-        return !isInputFieldFocused && BROWSER_DESTRUCTIVE_CODES.has(event.code);
+        return !isInputFieldFocused;
     }
     startListenKeys(_filter) {
         const filter = _filter ?? (() => false);
@@ -7191,20 +7176,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