@xh/hoist 75.0.0-SNAPSHOT.1752765224740 → 75.0.0-SNAPSHOT.1752854198386

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## 75.0-SNAPSHOT - Unreleased
 
+### 🎁 New Features
+
+* Added new `GroupingChooserModel.sortDimensions` config - can be set to false to respect the order
+  in which dimensions are provided to the model.
+
 ### 🐞 Bug Fixes
 
 * DashCanvas: ensure `allowAdd=false` is not enforced if loadingState
@@ -10,7 +15,7 @@
   within the canvas, not the next parent container up that has `position: relative;`.
 
 * `useContextModel` is now reactive to a change of an (observable) resolved model when it is set.
-Previously this value was cached on first render.
+  Previously this value was cached on first render.
 
 * Fixes to framework components that bind to grids (e.g. `ColChooserButton`, `ColAutosizeButton`,
   `GridFindField`) to rebind to a new observable GridModel available via context.
@@ -26,9 +31,9 @@ Previously this value was cached on first render.
 ### 🎁 New Features
 
 * Further refinements to the `GroupingChooser` desktop UI.
-  * Added new props `favoritesSide` and `favoritesTitle`.
-  * Deprecated `popoverTitle` prop - use `editorTitle` instead.
-  * Moved "Save as Favorite" button to a new compact toolbar within the popover.
+    * Added new props `favoritesSide` and `favoritesTitle`.
+    * Deprecated `popoverTitle` prop - use `editorTitle` instead.
+    * Moved "Save as Favorite" button to a new compact toolbar within the popover.
 
 ### 🐞 Bug Fixes
 

package/build/types/cmp/grouping/GroupingChooserModel.d.ts

@@ -1,29 +1,34 @@
-import { HoistModel, PersistOptions } from '@xh/hoist/core';
+import { HoistModel, PersistOptions, SelectOption } from '@xh/hoist/core';
 export interface GroupingChooserConfig {
+    /** True to accept an empty list as a valid value. */
+    allowEmpty?: boolean;
+    /**
+     * False (default) waits for the user to dismiss the popover before updating the
+     * external/observable value.
+     */
+    commitOnChange?: boolean;
     /**
      * Dimensions available for selection. When using GroupingChooser to create Cube queries,
      * it is recommended to pass the `dimensions` from the related cube (or a subset thereof).
      * Note that {@link CubeField} meets the `DimensionSpec` interface.
      */
     dimensions?: (DimensionSpec | string)[];
-    /** Initial value as an array of dimension names, or a function to produce such an array. */
-    initialValue?: string[] | (() => string[]);
     /**
      * Initial favorites as an array of dim name arrays, or a function to produce such an array.
      * Ignored if `persistWith.persistFavorites: false`.
      */
     initialFavorites?: string[][] | (() => string[][]);
-    /** Options governing persistence. */
-    persistWith?: GroupingChooserPersistOptions;
-    /** True to accept an empty list as a valid value. */
-    allowEmpty?: boolean;
+    /** Initial value as an array of dimension names, or a function to produce such an array. */
+    initialValue?: string[] | (() => string[]);
     /** Maximum number of dimensions allowed in a single grouping. */
     maxDepth?: number;
+    /** Options governing persistence. */
+    persistWith?: GroupingChooserPersistOptions;
     /**
-     * False (default) waits for the user to dismiss the popover before updating the
-     * external/observable value.
+     * True (default) to auto-sort dimensions by label. Set to false to show them in the order
+     * provided in the `dimensions` config.
      */
-    commitOnChange?: boolean;
+    sortDimensions?: boolean;
 }
 /**
  * Metadata for dimensions that are available for selection via a GroupingChooser control.
@@ -45,9 +50,10 @@ export declare class GroupingChooserModel extends HoistModel {
     value: string[];
     favorites: string[][];
     allowEmpty: boolean;
-    maxDepth: number;
     commitOnChange: boolean;
+    maxDepth: number;
     persistFavorites: boolean;
+    sortDimensions: boolean;
     pendingValue: string[];
     editorIsOpen: boolean;
     popoverRef: import("react").RefObject<HTMLElement> & import("react").RefCallback<HTMLElement>;
@@ -58,11 +64,13 @@ export declare class GroupingChooserModel extends HoistModel {
     get isValid(): boolean;
     get isAddEnabled(): boolean;
     get isAddFavoriteEnabled(): boolean;
-    constructor({ dimensions, initialValue, initialFavorites, persistWith, allowEmpty, maxDepth, commitOnChange }: GroupingChooserConfig);
+    constructor({ allowEmpty, commitOnChange, dimensions, initialFavorites, initialValue, maxDepth, persistWith, sortDimensions }: GroupingChooserConfig);
     setDimensions(dimensions: Array<DimensionSpec | string>): void;
     setValue(value: string[]): void;
     toggleEditor(): void;
     closeEditor(): void;
+    /** Transform dimension names into SelectOptions, with displayName and optional sort. */
+    getDimSelectOpts(dims?: string[]): SelectOption[];
     addPendingDim(dimName: string): void;
     replacePendingDimAtIdx(dimName: string, idx: number): void;
     removePendingDimAtIdx(idx: number): void;

package/build/types/core/exception/ExceptionHandler.d.ts

@@ -1,4 +1,4 @@
-import { HoistException } from '../';
+import { HoistException, PlainObject } from '../';
 export interface ExceptionHandlerOptions {
     /** Text (ideally user-friendly) describing the error. */
     message?: string;
@@ -113,8 +113,13 @@ export declare class ExceptionHandler {
      */
     logOnServerAsync(options: ExceptionHandlerLoggingOptions): Promise<boolean>;
     /**
-     * Serialize an error object safely for submission to server, or user display.
-     * This method will avoid circular references and will trim the depth of the object.
+     * Sanitize an exception for submission to server. This method will avoid circular references
+     * trim the depth of the object, and redact sensitive info (e.g. passwords).
+     */
+    sanitizeException(exception: HoistException): PlainObject;
+    /**
+     * Serialize an error object safely for user display.  This method will avoid circular
+     * references, trim the depth of the object, and redact sensitive info (e.g. passwords).
      */
     stringifyErrorSafely(exception: HoistException): string;
     private parseArgs;
@@ -122,6 +127,5 @@ export declare class ExceptionHandler {
     private logException;
     private parseOptions;
     private sessionMismatch;
-    private cleanStack;
     private cloneAndTrim;
 }

package/cmp/grouping/GroupingChooserModel.ts

@@ -5,14 +5,39 @@
  * Copyright © 2025 Extremely Heavy Industries Inc.
  */
 
-import {HoistModel, PersistableState, PersistenceProvider, PersistOptions} from '@xh/hoist/core';
+import {
+    HoistModel,
+    PersistableState,
+    PersistenceProvider,
+    PersistOptions,
+    SelectOption
+} from '@xh/hoist/core';
 import {genDisplayName} from '@xh/hoist/data';
 import {action, computed, makeObservable, observable} from '@xh/hoist/mobx';
 import {executeIfFunction, throwIf} from '@xh/hoist/utils/js';
 import {createObservableRef} from '@xh/hoist/utils/react';
-import {difference, isArray, isEmpty, isEqual, isObject, isString, keys, sortBy} from 'lodash';
+import {
+    compact,
+    difference,
+    isArray,
+    isEmpty,
+    isEqual,
+    isObject,
+    isString,
+    keys,
+    sortBy
+} from 'lodash';
 
 export interface GroupingChooserConfig {
+    /** True to accept an empty list as a valid value. */
+    allowEmpty?: boolean;
+
+    /**
+     * False (default) waits for the user to dismiss the popover before updating the
+     * external/observable value.
+     */
+    commitOnChange?: boolean;
+
     /**
      * Dimensions available for selection. When using GroupingChooser to create Cube queries,
      * it is recommended to pass the `dimensions` from the related cube (or a subset thereof).
@@ -20,29 +45,26 @@ export interface GroupingChooserConfig {
      */
     dimensions?: (DimensionSpec | string)[];
 
-    /** Initial value as an array of dimension names, or a function to produce such an array. */
-    initialValue?: string[] | (() => string[]);
-
     /**
      * Initial favorites as an array of dim name arrays, or a function to produce such an array.
      * Ignored if `persistWith.persistFavorites: false`.
      */
     initialFavorites?: string[][] | (() => string[][]);
 
-    /** Options governing persistence. */
-    persistWith?: GroupingChooserPersistOptions;
-
-    /** True to accept an empty list as a valid value. */
-    allowEmpty?: boolean;
+    /** Initial value as an array of dimension names, or a function to produce such an array. */
+    initialValue?: string[] | (() => string[]);
 
     /** Maximum number of dimensions allowed in a single grouping. */
     maxDepth?: number;
 
+    /** Options governing persistence. */
+    persistWith?: GroupingChooserPersistOptions;
+
     /**
-     * False (default) waits for the user to dismiss the popover before updating the
-     * external/observable value.
+     * True (default) to auto-sort dimensions by label. Set to false to show them in the order
+     * provided in the `dimensions` config.
      */
-    commitOnChange?: boolean;
+    sortDimensions?: boolean;
 }
 
 /**
@@ -70,9 +92,10 @@ export class GroupingChooserModel extends HoistModel {
     @observable.ref favorites: string[][] = [];
 
     allowEmpty: boolean;
-    maxDepth: number;
     commitOnChange: boolean;
+    maxDepth: number;
     persistFavorites: boolean = false;
+    sortDimensions: boolean;
 
     // Implementation fields for Control
     @observable.ref pendingValue: string[] = [];
@@ -117,20 +140,22 @@ export class GroupingChooserModel extends HoistModel {
     }
 
     constructor({
+        allowEmpty = false,
+        commitOnChange = false,
         dimensions,
-        initialValue = [],
         initialFavorites = [],
-        persistWith = null,
-        allowEmpty = false,
+        initialValue = [],
         maxDepth = null,
-        commitOnChange = false
+        persistWith = null,
+        sortDimensions = true
     }: GroupingChooserConfig) {
         super();
         makeObservable(this);
 
         this.allowEmpty = allowEmpty;
-        this.maxDepth = maxDepth;
         this.commitOnChange = commitOnChange;
+        this.maxDepth = maxDepth;
+        this.sortDimensions = sortDimensions;
 
         this.setDimensions(dimensions);
 
@@ -187,6 +212,15 @@ export class GroupingChooserModel extends HoistModel {
         this.editorIsOpen = false;
     }
 
+    /** Transform dimension names into SelectOptions, with displayName and optional sort. */
+    getDimSelectOpts(dims: string[] = this.availableDims): SelectOption[] {
+        const ret = compact(dims).map(dimName => ({
+            value: dimName,
+            label: this.getDimDisplayName(dimName)
+        }));
+        return this.sortDimensions ? sortBy(ret, 'label') : ret;
+    }
+
     //-------------------------
     // Value handling
     //-------------------------

package/core/exception/Exception.ts

@@ -210,7 +210,7 @@ export class Exception {
 
     private static createInternal(attributes: PlainObject, baseError?: Error) {
         const {message, ...rest} = attributes;
-        return Object.assign(
+        const ret = Object.assign(
             baseError ?? new Error(message),
             {
                 isRoutine: false,
@@ -218,6 +218,11 @@ export class Exception {
             },
             rest
         ) as HoistException;
+
+        // statuses of 0, 4XX, 5XX are server errors, so stack irrelevant and potentially misleading
+        if (ret.stack == null || /^[045]/.test(ret.httpStatus)) delete ret.stack;
+
+        return ret;
     }
 }
 

package/core/exception/ExceptionHandler.ts

@@ -185,8 +185,7 @@ export class ExceptionHandler {
     async logOnServerAsync(options: ExceptionHandlerLoggingOptions): Promise<boolean> {
         const {exception, userAlerted, userMessage} = options;
         try {
-            const error = this.stringifyErrorSafely(exception),
-                username = XH.getUsername();
+            const username = XH.getUsername();
 
             if (!username) {
                 logWarn('Error report cannot be submitted to UI server - user unknown', this);
@@ -194,7 +193,7 @@ export class ExceptionHandler {
             }
 
             const data: PlainObject = {
-                error: JSON.parse(error),
+                error: this.sanitizeException(exception),
                 userAlerted
             };
             if (userMessage) data.userMessage = stripTags(userMessage);
@@ -216,8 +215,16 @@ export class ExceptionHandler {
     }
 
     /**
-     * Serialize an error object safely for submission to server, or user display.
-     * This method will avoid circular references and will trim the depth of the object.
+     * Sanitize an exception for submission to server. This method will avoid circular references
+     * trim the depth of the object, and redact sensitive info (e.g. passwords).
+     */
+    sanitizeException(exception: HoistException): PlainObject {
+        return JSON.parse(this.stringifyErrorSafely(exception));
+    }
+
+    /**
+     * Serialize an error object safely for user display.  This method will avoid circular
+     * references, trim the depth of the object, and redact sensitive info (e.g. passwords).
      */
     stringifyErrorSafely(exception: HoistException): string {
         try {
@@ -287,8 +294,6 @@ export class ExceptionHandler {
             this.hideParams(e, opts);
         }
 
-        this.cleanStack(e);
-
         return {e, opts};
     }
 
@@ -342,12 +347,6 @@ export class ExceptionHandler {
         return e.name === 'SessionMismatchException';
     }
 
-    private cleanStack(e: HoistException) {
-        // statuses of 0, 4XX, 5XX are server errors, so the javascript stack
-        // is irrelevant and potentially misleading
-        if (/^[045]/.test(e.httpStatus)) delete e.stack;
-    }
-
     private cloneAndTrim(obj, depth = 5) {
         // Create a depth-constrained, deep copy of an object for safe-use in stringify
         //   - Skip private _XXXX properties.

package/desktop/cmp/grouping/GroupingChooser.ts

@@ -29,7 +29,7 @@ import {dragDropContext, draggable, droppable} from '@xh/hoist/kit/react-beautif
 import {apiDeprecated, elemWithin, getTestId} from '@xh/hoist/utils/js';
 import {splitLayoutProps} from '@xh/hoist/utils/react';
 import classNames from 'classnames';
-import {compact, isEmpty, isNil, isUndefined, sortBy} from 'lodash';
+import {isEmpty, isNil, isUndefined} from 'lodash';
 import './GroupingChooser.scss';
 import {ReactNode} from 'react';
 
@@ -264,7 +264,7 @@ const dimensionList = hoistCmp.factory<GroupingChooserModel>({
 const dimensionRow = hoistCmp.factory<GroupingChooserModel>({
     render({model, dimension, idx}) {
         // The options for this select include its current value
-        const options = getDimOptions([...model.availableDims, dimension], model);
+        const options = model.getDimSelectOpts([...model.availableDims, dimension]);
 
         return draggable({
             key: dimension,
@@ -346,7 +346,8 @@ const dimensionRow = hoistCmp.factory<GroupingChooserModel>({
 const addDimensionControl = hoistCmp.factory<GroupingChooserModel>({
     render({model}) {
         if (!model.isAddEnabled) return null;
-        const options = getDimOptions(model.availableDims, model);
+
+        const options = model.getDimSelectOpts();
         return div({
             className: 'xh-grouping-chooser__add-control',
             items: [
@@ -372,9 +373,8 @@ const addDimensionControl = hoistCmp.factory<GroupingChooserModel>({
 });
 
 /**
- * Extract integer values from CSS transform string.
- * Works for both `translate` and `translate3d`
- * e.g. `translate3d(250px, 150px, 0px)` is equivalent to [250, 150, 0]
+ * Extract integer values from CSS transform string. Works for both `translate` and `translate3d`.
+ * e.g. `translate3d(250px, 150px, 0px) -> [250, 150, 0]`
  */
 function parseTransform(transformStr: string): number[] {
     return transformStr
@@ -383,16 +383,6 @@ function parseTransform(transformStr: string): number[] {
         ?.map(it => parseInt(it));
 }
 
-/**
- * Convert a list of dim names into select options
- */
-function getDimOptions(dims, model) {
-    const ret = compact(dims).map(dimName => {
-        return {value: dimName, label: model.getDimDisplayName(dimName)};
-    });
-    return sortBy(ret, 'label');
-}
-
 //------------------
 // Favorites
 //------------------

package/mobile/cmp/grouping/GroupingChooser.ts

@@ -15,7 +15,7 @@ import '@xh/hoist/mobile/register';
 import {dialogPanel, panel} from '@xh/hoist/mobile/cmp/panel';
 import {splitLayoutProps} from '@xh/hoist/utils/react';
 import classNames from 'classnames';
-import {compact, isEmpty, sortBy} from 'lodash';
+import {isEmpty} from 'lodash';
 
 import './GroupingChooser.scss';
 
@@ -134,7 +134,7 @@ const dimensionList = hoistCmp.factory<GroupingChooserModel>({
 const dimensionRow = hoistCmp.factory<GroupingChooserModel>({
     render({model, dimension, idx}) {
         // The options for this select include its current value
-        const options = getDimOptions([...model.availableDims, dimension], model);
+        const options = model.getDimSelectOpts([...model.availableDims, dimension]);
 
         return draggable({
             key: dimension,
@@ -182,7 +182,8 @@ const dimensionRow = hoistCmp.factory<GroupingChooserModel>({
 const addDimensionControl = hoistCmp.factory<GroupingChooserModel>({
     render({model}) {
         if (!model.isAddEnabled) return null;
-        const options = getDimOptions(model.availableDims, model);
+
+        const options = model.getDimSelectOpts();
         return div({
             className: 'xh-grouping-chooser__add-control',
             items: [
@@ -203,16 +204,6 @@ const addDimensionControl = hoistCmp.factory<GroupingChooserModel>({
     }
 });
 
-/**
- * Convert a list of dim names into select options
- */
-function getDimOptions(dims, model) {
-    const ret = compact(dims).map(dimName => {
-        return {value: dimName, label: model.getDimDisplayName(dimName)};
-    });
-    return sortBy(ret, 'label');
-}
-
 //------------------
 // Favorites
 //------------------

package/package.json

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

package/promise/Promise.ts

@@ -202,8 +202,10 @@ const enhancePromise = promisePrototype => {
             if (!options) return this;
 
             const startTime = Date.now(),
-                doTrack = (exception: unknown = null) => {
-                    if (exception && exception['isRoutine']) return;
+                doTrack = (rejectReason: unknown = null) => {
+                    const exception = rejectReason != null ? Exception.create(rejectReason) : null;
+
+                    if (exception?.isRoutine) return;
 
                     const endTime = Date.now(),
                         opts: TrackOptions = isString(options) ? {message: options} : {...options};
@@ -222,7 +224,14 @@ const enhancePromise = promisePrototype => {
                     ) {
                         opts.elapsed = null;
                     }
-                    if (exception) opts.severity = 'ERROR';
+                    if (exception) {
+                        opts.severity = 'ERROR';
+                        opts.data = {
+                            error: XH.exceptionHandler.sanitizeException(exception),
+                            data: opts.data
+                        };
+                        opts.correlationId = opts.correlationId ?? exception.correlationId;
+                    }
 
                     XH.track(opts);
                 };