@ngxs/store 3.7.4-dev.master-2826e05 → 3.7.4-dev.master-a6cc926

package/fesm5/ngxs-store.js

@@ -1,9 +1,9 @@
 import { __assign, __spread, __extends, __values } from 'tslib';
 import { NgZone, Injectable, Inject, PLATFORM_ID, defineInjectable, inject, InjectionToken, INJECTOR, ɵglobal, Optional, SkipSelf, ErrorHandler, Injector, ɵivyEnabled, NgModule, APP_BOOTSTRAP_LISTENER } from '@angular/core';
-import { isAngularInTestMode, memoize, INITIAL_STATE_TOKEN, NgxsBootstrapper, NGXS_STATE_CONTEXT_FACTORY, NGXS_STATE_FACTORY, InitialState } from '@ngxs/store/internals';
+import { memoize, INITIAL_STATE_TOKEN, NgxsBootstrapper, NGXS_STATE_CONTEXT_FACTORY, NGXS_STATE_FACTORY, InitialState } from '@ngxs/store/internals';
 import { isPlatformServer } from '@angular/common';
 import { Observable, Subject, BehaviorSubject, of, forkJoin, throwError, EMPTY, from } from 'rxjs';
-import { filter, map, shareReplay, take, exhaustMap, mergeMap, defaultIfEmpty, catchError, takeUntil, tap, publishReplay, refCount, distinctUntilChanged } from 'rxjs/operators';
+import { filter, map, shareReplay, take, exhaustMap, mergeMap, defaultIfEmpty, catchError, takeUntil, tap, distinctUntilChanged } from 'rxjs/operators';
 
 /**
  * @fileoverview added by tsickle
@@ -256,8 +256,9 @@ if (false) {
      * - Object.freeze on the state and actions to guarantee immutability
      * (default: false)
      *
-     * @deprecated This property is no longer necessary when the Ivy compiler is used.
-     * We'll determine the development mode through the `ngDevMode`. It's still essential with View Engine.
+     * Note: this property will be accounted only in development mode when using the Ivy compiler.
+     * It makes sense to use it only during development to ensure there're no state mutations.
+     * When building for production, the Object.freeze will be tree-shaken away.
      * @type {?}
      */
     NgxsConfig.prototype.developmentMode;
@@ -2128,21 +2129,13 @@ var InternalStateOperations = /** @class */ (function () {
              */
             function (actionOrActions) { return _this._dispatcher.dispatch(actionOrActions); })
         };
-        // We have to have that duplication since this will allow us to tree-shake `ensureStateAndActionsAreImmutable`
-        // and `deepFreeze` in Ivy production build.
-        // The below `if` condition checks 2 things:
-        // 1) if we're in View Engine (`ngDevMode` is `undefined`)
-        // 2) if we're running tests, we should fallback to `config.developmentMode` to be backwards-compatible
-        if (typeof ngDevMode === 'undefined' || (ngDevMode && isAngularInTestMode())) {
+        if (typeof ngDevMode === 'undefined' || ngDevMode) {
             return this._config.developmentMode
                 ? ensureStateAndActionsAreImmutable(rootStateOperations)
                 : rootStateOperations;
         }
         else {
-            // If we're in Ivy and not running tests, then tree-shake `ensureStateAndActionsAreImmutable` and `deepFreeze`.
-            return ngDevMode
-                ? ensureStateAndActionsAreImmutable(rootStateOperations)
-                : rootStateOperations;
+            return rootStateOperations;
         }
     };
     /**
@@ -3472,7 +3465,7 @@ var Store = /** @class */ (function () {
          * because state is being changed actually within the `<root>` zone, see `InternalDispatcher#dispatchSingle`.
          * All selects would use this stream, and it would call leave only once for any state change across all active selectors.
          */
-        this._selectableStateStream = this._stateStream.pipe(leaveNgxs(this._internalExecutionStrategy), publishReplay(1), refCount());
+        this._selectableStateStream = this._stateStream.pipe(leaveNgxs(this._internalExecutionStrategy), shareReplay({ bufferSize: 1, refCount: true }));
         this.initStateStream(initialStateValue);
     }
     /**