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

package/CHANGELOG.md

@@ -2,9 +2,19 @@
 
 ## 68.0.0-SNAPSHOT - unreleased
 
+### 💥 Breaking Changes (upgrade difficulty: 🟢 LOW - Hoist Core update only)
+
+* Requires `hoist-core >= 21.1` for consolidated polling of Alert Banner updates (see below).
+
 ### ⚙️ Technical
 
 * Updated Admin Console's Cluster tab to refresh more frequently.
+* Consolidated the polling check for Alert Banner updates into existing `EnvironmentService`
+  polling, avoiding an extra request and improving alert banner responsiveness.
+
+### ⚙️ Typescript API Adjustments
+
+* Corrected types of enhanced `Promise` methods.
 
 ## 67.0.0 - 2024-09-03
 

package/admin/columns/Tracking.ts

@@ -119,8 +119,7 @@ export const correlationId: ColumnSpec = {
 export const error: ColumnSpec = {
     field: {
         name: 'error',
-        type: 'string',
-        displayName: 'Error Details'
+        type: 'string'
     },
     flex: true,
     minWidth: 150,

package/admin/tabs/cluster/BaseInstanceModel.ts

@@ -54,7 +54,10 @@ export class BaseInstanceModel extends HoistModel {
         forOwn(stats, (v, k) => {
             // Convert numbers that look like recent timestamps to date values.
             if (
-                (k.endsWith('Time') || k.endsWith('Date') || k == 'timestamp') &&
+                (k.endsWith('Time') ||
+                    k.endsWith('Date') ||
+                    k.endsWith('Timestamp') ||
+                    k == 'timestamp') &&
                 isNumber(v) &&
                 v > Date.now() - 365 * DAYS
             ) {

package/admin/tabs/cluster/services/DetailsModel.ts

@@ -48,14 +48,6 @@ export class DetailsModel extends HoistModel {
             loadSpec
         });
         if (loadSpec.isStale) return;
-        this.preprocessRawData(resp);
         this.stats = resp;
     }
-
-    private preprocessRawData(resp) {
-        // Format distributed objects for readability
-        resp.distributedObjects?.forEach(obj => {
-            obj.name = obj.name.substring(obj.name.indexOf('_') + 1);
-        });
-    }
 }

package/admin/tabs/general/alertBanner/AlertBannerModel.ts

@@ -10,13 +10,13 @@ import {FormModel} from '@xh/hoist/cmp/form';
 import {fragment, p} from '@xh/hoist/cmp/layout';
 import {HoistModel, Intent, LoadSpec, managed, PlainObject, XH} from '@xh/hoist/core';
 import {dateIs, required} from '@xh/hoist/data';
-import {action, computed, makeObservable, observable} from '@xh/hoist/mobx';
+import {action, bindable, computed, makeObservable, observable} from '@xh/hoist/mobx';
 import {AlertBannerSpec} from '@xh/hoist/svc';
 import {isEqual, isMatch, sortBy, without} from 'lodash';
 
 export class AlertBannerModel extends HoistModel {
-    savedValue;
-    @observable.ref savedPresets: PlainObject[] = [];
+    savedValue: AlertBannerSpec;
+    @bindable.ref savedPresets: PlainObject[] = [];
 
     @managed
     formModel = new FormModel({
@@ -118,7 +118,6 @@ export class AlertBannerModel extends HoistModel {
         this.formModel.setValues({...preset, expires: null});
     }
 
-    @action
     addPreset() {
         const {message, intent, iconName, enableClose, clientApps} = this.formModel.values,
             dateCreated = Date.now(),
@@ -188,24 +187,6 @@ export class AlertBannerModel extends HoistModel {
         }
     }
 
-    async saveBannerSpecAsync(spec: AlertBannerSpec) {
-        const {active, message, intent, iconName, enableClose, clientApps} = spec;
-        try {
-            await XH.fetchService.postJson({
-                url: 'alertBannerAdmin/setAlertSpec',
-                body: spec,
-                track: {
-                    category: 'Audit',
-                    message: 'Updated Alert Banner',
-                    data: {active, message, intent, iconName, enableClose, clientApps},
-                    logData: ['active']
-                }
-            });
-        } catch (e) {
-            XH.handleException(e);
-        }
-    }
-
     //----------------
     // Implementation
     //----------------
@@ -232,7 +213,7 @@ export class AlertBannerModel extends HoistModel {
         let preservedPublishDate = null;
 
         // Ask some questions if we are dealing with live stuff
-        if (XH.alertBannerService.enabled && (active || savedValue?.active)) {
+        if (active || savedValue?.active) {
             // Question 1. Reshow when modifying an active && already active, closable banner?
             if (
                 active &&
@@ -299,7 +280,25 @@ export class AlertBannerModel extends HoistModel {
             };
 
         await this.saveBannerSpecAsync(value);
-        await XH.alertBannerService.checkForBannerAsync();
+        await XH.environmentService.pollServerAsync();
         await this.refreshAsync();
     }
+
+    private async saveBannerSpecAsync(spec: AlertBannerSpec) {
+        const {active, message, intent, iconName, enableClose, clientApps} = spec;
+        try {
+            await XH.fetchService.postJson({
+                url: 'alertBannerAdmin/setAlertSpec',
+                body: spec,
+                track: {
+                    category: 'Audit',
+                    message: 'Updated Alert Banner',
+                    data: {active, message, intent, iconName, enableClose, clientApps},
+                    logData: ['active']
+                }
+            });
+        } catch (e) {
+            XH.handleException(e);
+        }
+    }
 }

package/admin/tabs/general/alertBanner/AlertBannerPanel.ts

@@ -37,7 +37,7 @@ import {toolbar} from '@xh/hoist/desktop/cmp/toolbar';
 import {dateTimeRenderer} from '@xh/hoist/format';
 import {Icon} from '@xh/hoist/icon';
 import {menu, menuItem, popover} from '@xh/hoist/kit/blueprint';
-import {LocalDate, SECONDS} from '@xh/hoist/utils/datetime';
+import {LocalDate} from '@xh/hoist/utils/datetime';
 import {isEmpty} from 'lodash';
 import {ReactNode} from 'react';
 import {AlertBannerModel} from './AlertBannerModel';
@@ -72,15 +72,13 @@ const formPanel = hoistCmp.factory<AlertBannerModel>(({model}) => {
                 labelWidth: 100
             },
             items: [
-                XH.alertBannerService.enabled
+                XH.getConf('xhAlertBannerConfig', {}).enabled
                     ? div({
                           className: `${baseClassName}__intro`,
                           items: [
                               p(`Show an alert banner to all ${XH.appName} users.`),
                               p(
-                                  `Configure and preview below. Presets can be saved and loaded via bottom bar menu. Banner will appear to all users within ${
-                                      XH.alertBannerService.interval / SECONDS
-                                  }s once marked Active and saved.`
+                                  'Configure and preview below. Presets can be saved and loaded via bottom bar menu. Banner will appear to all users once marked Active and saved.'
                               )
                           ]
                       })

package/build/types/admin/tabs/cluster/services/DetailsModel.d.ts

@@ -8,5 +8,4 @@ export declare class DetailsModel extends HoistModel {
     get selectedRecord(): StoreRecord;
     onLinked(): void;
     doLoadAsync(loadSpec: LoadSpec): Promise<void>;
-    private preprocessRawData;
 }

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "68.0.0-SNAPSHOT.1726258981470",
+  "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;