@xh/hoist 68.0.0-SNAPSHOT.1726237236739 → 68.0.0-SNAPSHOT.1726599173889

package/build/types/admin/tabs/general/alertBanner/AlertBannerModel.d.ts

@@ -2,7 +2,7 @@ import { FormModel } from '@xh/hoist/cmp/form';
 import { HoistModel, Intent, LoadSpec, PlainObject } from '@xh/hoist/core';
 import { AlertBannerSpec } from '@xh/hoist/svc';
 export declare class AlertBannerModel extends HoistModel {
-    savedValue: any;
+    savedValue: AlertBannerSpec;
     savedPresets: PlainObject[];
     formModel: FormModel;
     bannerModel: any;
@@ -10,7 +10,7 @@ export declare class AlertBannerModel extends HoistModel {
     get iconOptions(): string[];
     constructor();
     doLoadAsync(loadSpec: LoadSpec): Promise<void>;
-    saveAsync(): Promise<any>;
+    saveAsync(): Promise<void>;
     resetForm(): void;
     loadPreset(preset: PlainObject): void;
     addPreset(): void;
@@ -19,7 +19,7 @@ export declare class AlertBannerModel extends HoistModel {
     get shouldDisableAddPreset(): boolean;
     loadPresetsAsync(): Promise<void>;
     savePresetsAsync(): Promise<void>;
-    saveBannerSpecAsync(spec: AlertBannerSpec): Promise<void>;
     private syncPreview;
     private saveInternalAsync;
+    private saveBannerSpecAsync;
 }

package/build/types/admin/tabs/general/config/ConfigPanelModel.d.ts

@@ -10,7 +10,7 @@ export declare class ConfigPanelModel extends HoistModel {
     gridModel: RestGridModel;
     differModel: DifferModel;
     constructor();
-    doLoadAsync(loadSpec: LoadSpec): Promise<any>;
+    doLoadAsync(loadSpec: LoadSpec): Promise<void>;
     openDiffer(): void;
     closeDiffer(): void;
     private valueRenderer;

package/build/types/admin/tabs/userData/jsonblob/JsonBlobModel.d.ts

@@ -8,7 +8,7 @@ export declare class JsonBlobModel extends HoistModel {
     gridModel: RestGridModel;
     differModel: DifferModel;
     constructor();
-    doLoadAsync(loadSpec: LoadSpec): Promise<any>;
+    doLoadAsync(loadSpec: LoadSpec): Promise<void>;
     openDiffer(): void;
     closeDiffer(): void;
 }

package/build/types/admin/tabs/userData/prefs/editor/PrefEditorModel.d.ts

@@ -10,7 +10,7 @@ export declare class PrefEditorModel extends HoistModel {
     gridModel: RestGridModel;
     differModel: DifferModel;
     constructor();
-    doLoadAsync(loadSpec: LoadSpec): Promise<any>;
+    doLoadAsync(loadSpec: LoadSpec): Promise<void>;
     openDiffer(): void;
     closeDiffer(): void;
 }

package/build/types/admin/tabs/userData/users/UserModel.d.ts

@@ -8,5 +8,5 @@ export declare class UserModel extends HoistModel {
     withRolesOnly: boolean;
     gridModel: GridModel;
     constructor();
-    doLoadAsync(loadSpec: LoadSpec): Promise<any>;
+    doLoadAsync(loadSpec: LoadSpec): Promise<void>;
 }

package/build/types/desktop/cmp/rest/RestGridModel.d.ts

@@ -90,8 +90,8 @@ export declare class RestGridModel extends HoistModel {
     loadData(rawData: PlainObject[], rawSummaryData?: PlainObject): void;
     addRecord(): void;
     cloneRecord(record: StoreRecord): void;
-    deleteRecordAsync(record: StoreRecord): Promise<any>;
-    bulkDeleteRecordsAsync(records: StoreRecord[]): Promise<any>;
+    deleteRecordAsync(record: StoreRecord): Promise<void>;
+    bulkDeleteRecordsAsync(records: StoreRecord[]): Promise<void>;
     editRecord(record: StoreRecord): void;
     viewRecord(record: StoreRecord): void;
     confirmDeleteRecords(): void;

package/build/types/desktop/cmp/rest/impl/RestFormModel.d.ts

@@ -34,7 +34,7 @@ export declare class RestFormModel extends HoistModel {
     openEdit(rec: any): void;
     openClone(data: any): void;
     openView(rec: any): void;
-    validateAndSaveAsync(): Promise<any>;
+    validateAndSaveAsync(): Promise<void>;
     private initForm;
     private saveRecordAsync;
     fieldModelConfig(editor: RestGridEditor): {

package/build/types/promise/Promise.d.ts

@@ -1,4 +1,4 @@
-import { Thunkable, ExceptionHandlerOptions, TaskObserver, TrackOptions } from '@xh/hoist/core';
+import { Thunkable, ExceptionHandlerOptions, TaskObserver, TrackOptions, Some, Awaitable } from '@xh/hoist/core';
 /**
  * Enhancements to the Global Promise object.
  * Merged in to definition here and implemented on the prototype below.
@@ -9,7 +9,7 @@ declare global {
          * Version of `then()` that wraps the callback in a MobX action, for use in a Promise chain
          * that modifies MobX observables.
          */
-        thenAction(onFulfilled?: (value: T) => any, onRejected?: (reason: any) => any): Promise<any>;
+        thenAction<TResult>(onFulfilled: (value: T) => Awaitable<TResult>): Promise<TResult>;
         /**
          * Version of `catch()` that will only catch certain exceptions.
          *
@@ -18,16 +18,16 @@ declare global {
          *      selector will be handled by this method.
          * @param fn - catch handler
          */
-        catchWhen(selector: ((e: any) => boolean) | string | string[], fn?: (reason: any) => any): Promise<any>;
+        catchWhen<TResult = undefined>(selector: ((e: any) => boolean) | Some<string>, fn?: (reason: any) => Awaitable<TResult>): Promise<T | TResult>;
         /**
          * Version of `catch()` that passes the error onto Hoist's default exception handler for
          * convention-driven logging and alerting. Typically called last in a Promise chain.
          */
-        catchDefault(options?: ExceptionHandlerOptions): Promise<any>;
+        catchDefault(options?: ExceptionHandlerOptions): Promise<T | undefined>;
         /**
          * Version of `catchDefault()` that will only catch certain exceptions.
          */
-        catchDefaultWhen(selector: ((e: any) => boolean) | string | string[], options: ExceptionHandlerOptions): Promise<any>;
+        catchDefaultWhen(selector: ((e: any) => boolean) | Some<string>, options: ExceptionHandlerOptions): Promise<T | undefined>;
         /**
          * Wait on a potentially async function before passing on the original value.
          * Useful when we want to block and do something on the promise chain, but do not want to

package/build/types/svc/AlertBannerService.d.ts

@@ -2,18 +2,14 @@ import { BannerSpec, HoistService, Intent } from '@xh/hoist/core';
 /**
  * Service to display an app-wide alert banner, as configured via the Hoist Admin console.
  *
- * For this service to be active, a client-visible `xhAlertBannerConfig` config must be specified
- * as `{enabled:true, interval: x}`, where `x` sets this service's polling frequency in seconds.
+ * Note that the client is provided with updated banner data from the server via
+ * EnvironmentService, and its regular polling.  See 'xhEnvPollConfig' for more information.
  */
 export declare class AlertBannerService extends HoistService {
     xhImpl: boolean;
     static instance: AlertBannerService;
-    private timer;
-    get interval(): number;
-    get enabled(): boolean;
     get lastDismissed(): number;
-    initAsync(): Promise<void>;
-    checkForBannerAsync(): Promise<void>;
+    updateBanner(spec: AlertBannerSpec): Promise<void>;
     genBannerSpec(message: string, intent: Intent, iconName: string, enableClose: boolean): BannerSpec;
     private onClose;
     private isTargetedApp;

package/build/types/svc/EnvironmentService.d.ts

@@ -1,7 +1,7 @@
 import { HoistService } from '@xh/hoist/core';
 /**
  * Load and report on the client and server environment, including software versions, timezones, and
- * and other technical information.
+ * other technical information.
  */
 export declare class EnvironmentService extends HoistService {
     static instance: EnvironmentService;
@@ -30,9 +30,13 @@ export declare class EnvironmentService extends HoistService {
     isTest(): boolean;
     isMinHoistCoreVersion(version: string): boolean;
     isMaxHoistCoreVersion(version: string): boolean;
+    /**
+     * Update critical environment information from server.
+     * @internal - not for app use. Called by `pollTimer` and as needed by Hoist code.
+     */
+    pollServerAsync(): Promise<void>;
     constructor();
     private startPolling;
-    private pollServerAsync;
     private setServerInfo;
     private get pollIntervalMs();
 }

package/build/types/svc/IdentityService.d.ts

@@ -54,7 +54,7 @@ export declare class IdentityService extends HoistService {
      */
     impersonateAsync(username: string): Promise<void>;
     /** Exit any active impersonation, reloading the app to resume accessing it as yourself. */
-    endImpersonateAsync(): Promise<any>;
+    endImpersonateAsync(): Promise<void>;
     private createUser;
     private hasGate;
     private canUserImpersonate;

package/core/exception/ExceptionHandler.ts

@@ -127,7 +127,7 @@ export class ExceptionHandler {
                     ),
                     actionButtonProps: {
                         icon: Icon.search(),
-                        onClick: () => exceptionDialogModel.show(e, opts)
+                        onClick: () => exceptionDialogModel.showDetails(e, opts)
                     },
                     intent: showAsError ? 'danger' : 'primary',
                     ...ExceptionHandler.TOAST_PROPS

package/package.json

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

package/promise/Promise.ts

@@ -10,7 +10,9 @@ import {
     ExceptionHandlerOptions,
     TaskObserver,
     TrackOptions,
-    XH
+    XH,
+    Some,
+    Awaitable
 } from '@xh/hoist/core';
 import {action} from '@xh/hoist/mobx';
 import {olderThan, SECONDS} from '@xh/hoist/utils/datetime';
@@ -26,10 +28,7 @@ declare global {
          * Version of `then()` that wraps the callback in a MobX action, for use in a Promise chain
          * that modifies MobX observables.
          */
-        thenAction(
-            onFulfilled?: (value: T) => any,
-            onRejected?: (reason: any) => any
-        ): Promise<any>;
+        thenAction<TResult>(onFulfilled: (value: T) => Awaitable<TResult>): Promise<TResult>;
 
         /**
          * Version of `catch()` that will only catch certain exceptions.
@@ -39,24 +38,24 @@ declare global {
          *      selector will be handled by this method.
          * @param fn - catch handler
          */
-        catchWhen(
-            selector: ((e: any) => boolean) | string | string[],
-            fn?: (reason: any) => any
-        ): Promise<any>;
+        catchWhen<TResult = undefined>(
+            selector: ((e: any) => boolean) | Some<string>,
+            fn?: (reason: any) => Awaitable<TResult>
+        ): Promise<T | TResult>;
 
         /**
          * Version of `catch()` that passes the error onto Hoist's default exception handler for
          * convention-driven logging and alerting. Typically called last in a Promise chain.
          */
-        catchDefault(options?: ExceptionHandlerOptions): Promise<any>;
+        catchDefault(options?: ExceptionHandlerOptions): Promise<T | undefined>;
 
         /**
          * Version of `catchDefault()` that will only catch certain exceptions.
          */
         catchDefaultWhen(
-            selector: ((e: any) => boolean) | string | string[],
+            selector: ((e: any) => boolean) | Some<string>,
             options: ExceptionHandlerOptions
-        ): Promise<any>;
+        ): Promise<T | undefined>;
 
         /**
          * Wait on a potentially async function before passing on the original value.

package/svc/AlertBannerService.ts

@@ -6,52 +6,28 @@
  */
 import {BannerModel} from '@xh/hoist/appcontainer/BannerModel';
 import {markdown} from '@xh/hoist/cmp/markdown';
-import {BannerSpec, HoistService, Intent, managed, XH} from '@xh/hoist/core';
+import {BannerSpec, HoistService, Intent, XH} from '@xh/hoist/core';
 import {Icon} from '@xh/hoist/icon';
-import {Timer} from '@xh/hoist/utils/async';
-import {SECONDS} from '@xh/hoist/utils/datetime';
 import {compact, isEmpty, map, trim} from 'lodash';
 
 /**
  * Service to display an app-wide alert banner, as configured via the Hoist Admin console.
  *
- * For this service to be active, a client-visible `xhAlertBannerConfig` config must be specified
- * as `{enabled:true, interval: x}`, where `x` sets this service's polling frequency in seconds.
+ * Note that the client is provided with updated banner data from the server via
+ * EnvironmentService, and its regular polling.  See 'xhEnvPollConfig' for more information.
  */
 export class AlertBannerService extends HoistService {
     override xhImpl = true;
 
     static instance: AlertBannerService;
 
-    @managed
-    private timer: Timer;
-
-    get interval(): number {
-        const conf = XH.getConf('xhAlertBannerConfig', {});
-        return conf.enabled && conf.interval ? conf.interval * SECONDS : -1;
-    }
-
-    get enabled(): boolean {
-        return this.interval > 0;
-    }
-
     get lastDismissed(): number {
         return XH.localStorageService.get('xhAlertBanner.lastDismissed');
     }
 
-    override async initAsync() {
-        this.timer = Timer.create({
-            runFn: () => this.checkForBannerAsync(),
-            interval: this.interval
-        });
-    }
-
-    async checkForBannerAsync() {
-        if (!this.enabled) return;
-
-        const data: AlertBannerSpec = await XH.fetchJson({url: 'xh/alertBanner'}),
-            {active, expires, publishDate, message, intent, iconName, enableClose, clientApps} =
-                data,
+    async updateBanner(spec: AlertBannerSpec) {
+        const {active, expires, publishDate, message, intent, iconName, enableClose, clientApps} =
+                spec,
             {lastDismissed, onClose} = this;
 
         if (

package/svc/EnvironmentService.ts

@@ -18,7 +18,7 @@ import {version as reactVersion} from 'react';
 
 /**
  * Load and report on the client and server environment, including software versions, timezones, and
- * and other technical information.
+ * other technical information.
  */
 export class EnvironmentService extends HoistService {
     static instance: EnvironmentService;
@@ -49,7 +49,7 @@ export class EnvironmentService extends HoistService {
     private pollTimer: Timer;
 
     override async initAsync() {
-        const {pollConfig, instanceName, ...serverEnv} = await XH.fetchJson({
+        const {pollConfig, instanceName, alertBanner, ...serverEnv} = await XH.fetchJson({
                 url: 'xh/environment'
             }),
             clientTimeZone = Intl.DateTimeFormat().resolvedOptions().timeZone ?? 'Unknown',
@@ -81,7 +81,10 @@ export class EnvironmentService extends HoistService {
         this.pollConfig = pollConfig;
         this.addReaction({
             when: () => XH.appIsRunning,
-            run: this.startPolling
+            run: () => {
+                XH.alertBannerService.updateBanner(alertBanner);
+                this.startPolling();
+            }
         });
     }
 
@@ -109,23 +112,11 @@ export class EnvironmentService extends HoistService {
         return checkMaxVersion(this.get('hoistCoreVersion'), version);
     }
 
-    //------------------------------
-    // Implementation
-    //------------------------------
-    constructor() {
-        super();
-        makeObservable(this);
-    }
-
-    private startPolling() {
-        this.pollTimer = Timer.create({
-            runFn: () => this.pollServerAsync(),
-            interval: this.pollIntervalMs,
-            delay: true
-        });
-    }
-
-    private async pollServerAsync() {
+    /**
+     * Update critical environment information from server.
+     * @internal - not for app use. Called by `pollTimer` and as needed by Hoist code.
+     */
+    async pollServerAsync() {
         let data;
         try {
             data = await XH.fetchJson({url: 'xh/environmentPoll'});
@@ -135,10 +126,11 @@ export class EnvironmentService extends HoistService {
         }
 
         // Update config/interval, and server info
-        const {pollConfig, instanceName, appVersion, appBuild} = data;
+        const {pollConfig, instanceName, alertBanner, appVersion, appBuild} = data;
         this.pollConfig = pollConfig;
         this.pollTimer.setInterval(this.pollIntervalMs);
         this.setServerInfo(instanceName, appVersion, appBuild);
+        XH.alertBannerService.updateBanner(alertBanner);
 
         // Handle version change
         if (appVersion != XH.getEnv('appVersion') || appBuild != XH.getEnv('appBuild')) {
@@ -163,6 +155,22 @@ export class EnvironmentService extends HoistService {
         }
     }
 
+    //------------------------------
+    // Implementation
+    //------------------------------
+    constructor() {
+        super();
+        makeObservable(this);
+    }
+
+    private startPolling() {
+        this.pollTimer = Timer.create({
+            runFn: () => this.pollServerAsync(),
+            interval: this.pollIntervalMs,
+            delay: true
+        });
+    }
+
     @action
     private setServerInfo(serverInstance: string, serverVersion: string, serverBuild: string) {
         this.serverInstance = serverInstance;