@xh/hoist 76.0.0-SNAPSHOT.1755640263068 → 76.0.0-SNAPSHOT.1755652707785

package/CHANGELOG.md

@@ -4,13 +4,20 @@
 
 ### 🎁 New Features
 
-* Added new `extraConfirmText`, `extraConfirmLabel` properties to `MessageOptions`.  Use this option
-  to require the specified text to be re-typed by a user when confirming a potentially destructive or disruptive action.
+* Added new `extraConfirmText`, `extraConfirmLabel` properties to `MessageOptions`. Use this option
+  to require the specified text to be re-typed by a user when confirming a potentially destructive
+  or disruptive action.
+
+### 🐞 Bug Fixes
+
+* Handled an edge-case `ViewManager` bug where `enableDefault` changed to `false` after some user
+  state had already been persisted w/users pointed at in-code default view. The manager now calls
+  its configured `initialViewSpec` function as expected in this case.
 
 ### ⚙️ Technical
-* Provide support for triggering browser GC during development. Useful for troubleshooting memory
-  consumption issues.  Requires running chromium-based browser with flag, e.g.
-  "start chrome --js-flags="--expose-gc"
+
+* Added control to trigger browser GC from app footer. Useful for troubleshooting memory issues.
+  Requires running chromium-based browser via e.g. `start chrome --js-flags="--expose-gc`.
 
 ## 75.0.1 - 2025-08-11
 
@@ -25,11 +32,11 @@
       Provides easier discoverability on desktop and supports this feature on mobile, where we
       don't have context menus.
 * Enhanced `FilterChooser` to better handle filters with different `op`s on the same field.
-  * Multiple "inclusive" ops (e.g. `=`, `like`) will be OR'ed together.
-  * Multiple "exclusive" ops (e.g. `!=`, `not like`) will be AND'ed together.
-  * Range ops (e.g. `<`, `>` ) use a heuristic to avoid creating a filter that could never match.
-  * This behavior is consistent with current behavior and user intuition and should maximize the
-    ability to create useful queries using this component.
+    * Multiple "inclusive" ops (e.g. `=`, `like`) will be OR'ed together.
+    * Multiple "exclusive" ops (e.g. `!=`, `not like`) will be AND'ed together.
+    * Range ops (e.g. `<`, `>` ) use a heuristic to avoid creating a filter that could never match.
+    * This behavior is consistent with current behavior and user intuition and should maximize the
+      ability to create useful queries using this component.
 * Deprecated the `RelativeTimestamp.options` prop - all the same options are now top-level props.
 * Added new `GroupingChooserModel.sortDimensions` config. Set to `false` to respect the order in
   which `dimensions` are provided to the model.
@@ -63,8 +70,9 @@
 * Hoist now sets a reference to an app's singleton `AuthModel` on a static `instance` property of
   the app-specified class. App developers can declare a typed static `instance` property on their
   model class and use it to access the singleton with its proper type, vs. `XH.authModel`.
-  * The `XH.authModel` property is still set and available - this is a non-breaking change.
-  * This approach was already (and continues to be) used for services and the `AppModel` singleton.
+    * The `XH.authModel` property is still set and available - this is a non-breaking change.
+    * This approach was already (and continues to be) used for services and the `AppModel`
+      singleton.
 
 ### 💥 Breaking Changes (upgrade difficulty: 🟢 LOW - removing deprecations)
 

package/build/types/cmp/viewmanager/DataAccess.d.ts

@@ -14,7 +14,7 @@ export declare class DataAccess<T> {
         views: ViewInfo[];
         state: ViewUserState;
     }>;
-    /** Fetch the latest version of a view. */
+    /** Fetch the latest version of a view, or the in-code default if token null/undefined/empty. */
     fetchViewAsync(token: string): Promise<View<T>>;
     /** Create a new view, owned by the current user.*/
     createViewAsync(spec: ViewCreateSpec): Promise<View<T>>;

package/cmp/viewmanager/DataAccess.ts

@@ -47,7 +47,7 @@ export class DataAccess<T> {
         }
     }
 
-    /** Fetch the latest version of a view. */
+    /** Fetch the latest version of a view, or the in-code default if token null/undefined/empty. */
     async fetchViewAsync(token: string): Promise<View<T>> {
         const {model} = this;
         if (!token) return View.createDefault(model);

package/cmp/viewmanager/ViewManagerModel.ts

@@ -19,7 +19,7 @@ import type {ViewManagerProvider, ReactionSpec} from '@xh/hoist/core';
 import {genDisplayName} from '@xh/hoist/data';
 import {fmtDateTime} from '@xh/hoist/format';
 import {action, bindable, makeObservable, observable, comparer, runInAction} from '@xh/hoist/mobx';
-import {SECONDS} from '@xh/hoist/utils/datetime';
+import {ONE_SECOND, SECONDS} from '@xh/hoist/utils/datetime';
 import {executeIfFunction, pluralize, throwIf} from '@xh/hoist/utils/js';
 import {find, isEqual, isNil, isNull, isObject, isUndefined, lowerCase, uniqBy} from 'lodash';
 import {ReactNode} from 'react';
@@ -531,8 +531,8 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
     // Implementation
     //------------------
     private async initAsync() {
-        let {dataAccess, pendingValueStorageKey} = this,
-            initialState;
+        let {dataAccess, pendingValueStorageKey, enableDefault} = this,
+            initialState: ViewUserState;
 
         try {
             // 1) Initialize views and related state
@@ -547,17 +547,23 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
                 }
             });
 
-            // 2) Initialize/choose initial view.  Null is ok, and will yield default.
+            // 2) Select the initial view.
             let initialView: ViewInfo,
                 initialTkn: string = initialState.currentView;
-            if (isUndefined(initialTkn)) {
+            if (isUndefined(initialTkn) || (isNull(initialTkn) && !enableDefault)) {
+                // Token undefined (no prior view) or null (in-code default *had* been loaded) but
+                // default no longer enabled - call initialViewSpec.
                 initialView = this.initialViewSpec?.(views);
             } else if (!isNull(initialTkn)) {
+                // Token provided - find the view, falling back to initialViewSpec if not found.
                 initialView = find(views, {token: initialTkn}) ?? this.initialViewSpec?.(views);
             } else {
+                // Token null - active signal to load in-code default.
                 initialView = null;
             }
 
+            // Note that the above routine failed to resolve a view, we will pass undefined here
+            // and load the in-code default, even if not enabled. We have no other choice!
             await this.loadViewAsync(initialView?.token, this.pendingValue);
         } catch (e) {
             // Always ensure at least default view is installed (other state defaults are fine)
@@ -594,7 +600,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
                 track: () => this.userPinned,
                 run: userPinned => dataAccess.updateStateAsync({userPinned}),
                 equals: comparer.structural,
-                debounce: 1 * SECONDS
+                debounce: ONE_SECOND
             },
             {
                 track: () => this.autoSave,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "76.0.0-SNAPSHOT.1755640263068",
+  "version": "76.0.0-SNAPSHOT.1755652707785",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",