@xh/hoist 71.0.0-SNAPSHOT.1736263796812 → 71.0.0-SNAPSHOT.1736264155874

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

@@ -15,7 +15,7 @@ export declare class DataAccess<T> {
         state: ViewUserState;
     }>;
     /** Fetch the latest version of a view. */
-    fetchViewAsync(token: string): Promise<View<T>>;
+    fetchViewAsync(info: ViewInfo): Promise<View<T>>;
     /** Create a new view, owned by the current user.*/
     createViewAsync(spec: ViewCreateSpec): Promise<View<T>>;
     /** Update all aspects of a view's metadata.*/

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

@@ -183,7 +183,7 @@ export declare class ViewManagerModel<T = PlainObject> extends HoistModel {
      */
     private constructor();
     doLoadAsync(loadSpec: LoadSpec): Promise<void>;
-    selectViewAsync(view: string | ViewInfo, opts?: {
+    selectViewAsync(info: ViewInfo, opts?: {
         alertUnsavedChanges: boolean;
     }): Promise<void>;
     saveAsAsync(spec: ViewCreateSpec): Promise<void>;

package/build/types/desktop/cmp/viewmanager/ViewManagerLocalModel.d.ts

@@ -7,7 +7,5 @@ export declare class ViewManagerLocalModel extends HoistModel {
     readonly manageDialogModel: ManageDialogModel;
     readonly saveAsDialogModel: SaveAsDialogModel;
     isVisible: boolean;
-    saveAsync(): Promise<void>;
-    revertAsync(): Promise<void>;
     constructor(parent: ViewManagerModel);
 }

package/cmp/viewmanager/DataAccess.ts

@@ -48,14 +48,14 @@ export class DataAccess<T> {
     }
 
     /** Fetch the latest version of a view. */
-    async fetchViewAsync(token: string): Promise<View<T>> {
+    async fetchViewAsync(info: ViewInfo): Promise<View<T>> {
         const {model} = this;
-        if (!token) return View.createDefault(model);
+        if (!info) return View.createDefault(model);
         try {
-            const raw = await XH.fetchJson({url: 'xhView/get', params: {token}});
+            const raw = await XH.fetchJson({url: 'xhView/get', params: {token: info.token}});
             return View.fromBlob(raw, model);
         } catch (e) {
-            throw XH.exception({message: `Unable to fetch view with token ${token}`, cause: e});
+            throw XH.exception({message: `Unable to fetch ${info.typedName}`, cause: e});
         }
     }
 

package/cmp/viewmanager/ViewInfo.ts

@@ -5,7 +5,6 @@
  * Copyright © 2025 Extremely Heavy Industries Inc.
  */
 import {SECONDS} from '@xh/hoist/utils/datetime';
-import {throwIf} from '@xh/hoist/utils/js';
 import {ViewManagerModel} from './ViewManagerModel';
 import {JsonBlob} from '@xh/hoist/svc';
 import {PlainObject, XH} from '@xh/hoist/core';
@@ -58,8 +57,6 @@ export class ViewInfo {
     readonly model: ViewManagerModel;
 
     constructor(blob: JsonBlob, model: ViewManagerModel) {
-        throwIf(blob.type !== model.type, 'View type does not match ViewManager type.');
-
         this.token = blob.token;
         this.type = blob.type;
         this.name = blob.name;

package/cmp/viewmanager/ViewManagerModel.ts

@@ -21,7 +21,7 @@ import {fmtDateTime} from '@xh/hoist/format';
 import {action, bindable, makeObservable, observable, comparer, runInAction} from '@xh/hoist/mobx';
 import {olderThan, 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 {find, isEqual, isNil, isNull, isUndefined, lowerCase, uniqBy} from 'lodash';
 import {ReactNode} from 'react';
 import {ViewInfo} from './ViewInfo';
 import {View} from './View';
@@ -337,7 +337,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
             if (!view.isDefault) {
                 const latestInfo = find(views, {token: view.token});
                 if (latestInfo && latestInfo.lastUpdated > view.lastUpdated) {
-                    this.loadViewAsync(view.token, this.pendingValue);
+                    this.loadViewAsync(latestInfo, this.pendingValue);
                 }
             }
         } catch (e) {
@@ -346,12 +346,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
         }
     }
 
-    async selectViewAsync(
-        view: string | ViewInfo,
-        opts = {alertUnsavedChanges: true}
-    ): Promise<void> {
-        const token = isObject(view) ? view.token : view;
-
+    async selectViewAsync(info: ViewInfo, opts = {alertUnsavedChanges: true}): Promise<void> {
         // ensure any pending auto-save gets completed
         if (this.isValueDirty && this.isViewAutoSavable) {
             await this.maybeAutoSaveAsync();
@@ -367,7 +362,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
             return;
         }
 
-        return this.loadViewAsync(token);
+        await this.loadViewAsync(info).catch(e => this.handleException(e));
     }
 
     async saveAsAsync(spec: ViewCreateSpec): Promise<void> {
@@ -385,22 +380,26 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
             return;
         }
         const {pendingValue, view, dataAccess} = this;
+        try {
+            if (!(await this.maybeConfirmSaveAsync(view, pendingValue))) {
+                return;
+            }
+            const updated = await dataAccess
+                .updateViewValueAsync(view, pendingValue.value)
+                .linkTo(this.saveTask);
 
-        if (!(await this.maybeConfirmSaveAsync(view, pendingValue))) {
-            return;
+            this.setAsView(updated);
+            this.noteSuccess(`Saved ${view.typedName}`);
+        } catch (e) {
+            this.handleException(e, {
+                message: `Failed to save ${view.typedName}.  If this persists consider \`Save As...\`.`
+            });
         }
-        const updated = await dataAccess
-            .updateViewValueAsync(view, pendingValue.value)
-            .linkTo(this.saveTask);
-
-        this.setAsView(updated);
-        this.noteSuccess(`Saved ${view.typedName}`);
-
         this.refreshAsync();
     }
 
     async resetAsync(): Promise<void> {
-        return this.loadViewAsync(this.view.token);
+        await this.loadViewAsync(this.view.info).catch(e => this.handleException(e));
     }
 
     //--------------------------------
@@ -484,7 +483,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
         const {views} = this;
 
         if (toDelete.some(view => view.isCurrentView) && !views.some(view => view.isCurrentView)) {
-            await this.loadViewAsync(this.initialViewSpec?.(views)?.token);
+            await this.loadViewAsync(this.initialViewSpec?.(views));
         }
 
         if (exception) throw exception;
@@ -570,13 +569,13 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
     }
 
     private async loadViewAsync(
-        token: string,
+        info: ViewInfo,
         pendingValue: PendingValue<T> = null
     ): Promise<void> {
         return this.dataAccess
-            .fetchViewAsync(token)
+            .fetchViewAsync(info)
             .thenAction(latest => {
-                this.setAsView(latest, pendingValue?.token == token ? pendingValue : null);
+                this.setAsView(latest, pendingValue?.token == info?.token ? pendingValue : null);
                 this.providers.forEach(it => it.pushStateToTarget());
                 this.lastPushed = Date.now();
             })
@@ -650,7 +649,7 @@ export class ViewManagerModel<T = PlainObject> extends HoistModel {
 
     private async maybeConfirmSaveAsync(view: View, pendingValue: PendingValue<T>) {
         // Get latest from server for reference
-        const latest = await this.dataAccess.fetchViewAsync(view.token),
+        const latest = await this.dataAccess.fetchViewAsync(view.info),
             isGlobal = latest.isGlobal,
             isStale = latest.lastUpdated > pendingValue.baseUpdated;
         if (!isStale && !isGlobal) return true;

package/desktop/cmp/viewmanager/ViewManager.ts

@@ -118,7 +118,7 @@ const menuButton = hoistCmp.factory<ViewManagerLocalModel>({
 const saveButton = hoistCmp.factory<ViewManagerLocalModel>({
     render({model, mode, ...rest}) {
         if (hideStateButton(model, mode)) return null;
-        const {parent} = model,
+        const {parent, saveAsDialogModel} = model,
             {typeDisplayName, isLoading, isValueDirty} = parent;
         return button({
             className: 'xh-view-manager__save-button',
@@ -126,7 +126,9 @@ const saveButton = hoistCmp.factory<ViewManagerLocalModel>({
             tooltip: `Save changes to this ${typeDisplayName}`,
             intent: 'primary',
             disabled: !isValueDirty || isLoading,
-            onClick: () => model.saveAsync(),
+            onClick: () => {
+                parent.isViewSavable ? parent.saveAsync() : saveAsDialogModel.open();
+            },
             ...rest
         });
     }
@@ -142,7 +144,7 @@ const revertButton = hoistCmp.factory<ViewManagerLocalModel>({
             tooltip: `Revert changes to this ${typeDisplayName}`,
             intent: 'danger',
             disabled: !isValueDirty || isLoading,
-            onClick: () => model.revertAsync(),
+            onClick: () => model.parent.resetAsync(),
             ...rest
         });
     }

package/desktop/cmp/viewmanager/ViewManagerLocalModel.ts

@@ -23,24 +23,6 @@ export class ViewManagerLocalModel extends HoistModel {
     @bindable
     isVisible = true;
 
-    async saveAsync() {
-        const {parent} = this,
-            {view} = parent;
-
-        if (!parent.isViewSavable) {
-            this.saveAsDialogModel.open();
-            return;
-        }
-
-        return parent.saveAsync().catchDefault({
-            message: `Failed to save ${view.typedName}.  If this persists consider \`Save As...\`.`
-        });
-    }
-
-    async revertAsync() {
-        return this.parent.resetAsync().catchDefault();
-    }
-
     constructor(parent: ViewManagerModel) {
         super();
         makeObservable(this);

package/desktop/cmp/viewmanager/ViewMenu.ts

@@ -63,7 +63,7 @@ function getNavMenuItems(model: ViewManagerModel): ReactNode[] {
                 className: 'xh-view-manager__menu-item',
                 icon: view.isDefault ? Icon.check() : Icon.placeholder(),
                 text: `Default ${startCase(typeDisplayName)}`,
-                onClick: () => model.selectViewAsync(null).catchDefault()
+                onClick: () => model.selectViewAsync(null)
             })
         );
     }
@@ -89,7 +89,7 @@ function getOtherMenuItems(model: ViewManagerLocalModel): ReactNode[] {
             icon: Icon.save(),
             text: 'Save',
             disabled: !isViewSavable || !isValueDirty,
-            onClick: () => model.saveAsync()
+            onClick: () => parent.saveAsync()
         }),
         menuItem({
             icon: Icon.placeholder(),
@@ -100,7 +100,7 @@ function getOtherMenuItems(model: ViewManagerLocalModel): ReactNode[] {
             icon: Icon.reset(),
             text: `Revert`,
             disabled: !isValueDirty,
-            onClick: () => model.revertAsync()
+            onClick: () => parent.resetAsync()
         }),
         menuDivider({omit: !enableAutoSave}),
         menuItem({
@@ -168,6 +168,6 @@ function viewMenuItem(view: ViewInfo, model: ViewManagerModel): ReactNode {
         text: view.name,
         title: title.join(' | '),
         icon,
-        onClick: () => model.selectViewAsync(view).catchDefault()
+        onClick: () => model.selectViewAsync(view)
     });
 }

package/desktop/cmp/viewmanager/dialog/ManageDialogModel.ts

@@ -86,7 +86,7 @@ export class ManageDialogModel extends HoistModel {
     }
 
     activateSelectedViewAndClose() {
-        this.viewManagerModel.selectViewAsync(this.selectedView).catchDefault();
+        this.viewManagerModel.selectViewAsync(this.selectedView);
         this.close();
     }
 

package/package.json

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