@xh/hoist 86.2.0 → 86.4.0

package/mcp/formatters/typescript.ts

@@ -11,7 +11,7 @@
  */
 import {z} from 'zod';
 import type {MemberInfo, MemberIndexEntry, SymbolEntry, SymbolDetail} from '../data/ts-registry.js';
-import {resolveRepoRoot} from '../util/paths.js';
+import {resolveRepoRootPosix, toPosixPath} from '../util/paths.js';
 
 /** Remove blank lines from a JSDoc string to produce more compact output. */
 function collapseJsDoc(jsDoc: string): string {
@@ -31,8 +31,12 @@ export function truncateType(typeStr: string): string {
 
 /** Convert an absolute file path to a repo-relative path. */
 export function toRelativePath(filePath: string): string {
-    const root = resolveRepoRoot();
-    return filePath.startsWith(root) ? filePath.slice(root.length + 1) : filePath;
+    // Symbol filePaths originate from ts-morph (forward slashes on all platforms);
+    // compare in POSIX form so the repo-root prefix strips correctly on Windows,
+    // where `resolveRepoRoot()` would otherwise yield a backslash path.
+    const root = resolveRepoRootPosix();
+    const posix = toPosixPath(filePath);
+    return posix.startsWith(root) ? posix.slice(root.length + 1) : posix;
 }
 
 /**

package/mcp/util/paths.ts

@@ -39,6 +39,27 @@ export function resolveRepoRoot(): string {
     return repoRoot;
 }
 
+/** Convert a filesystem path to POSIX (forward-slash) form. */
+export function toPosixPath(p: string): string {
+    return p.replace(/\\/g, '/');
+}
+
+/**
+ * Repo root in POSIX (forward-slash) form, matching ts-morph's `getFilePath()`
+ * convention, which always reports forward slashes on every platform.
+ *
+ * On Windows, {@link resolveRepoRoot} returns a backslash path (e.g.
+ * `D:\hoist-react`) while ts-morph reports `D:/hoist-react/...`; comparing or
+ * slicing the two directly fails, silently filtering out every source file and
+ * yielding an empty symbol index. Use this whenever comparing against or
+ * slicing a ts-morph source-file path (see `data/ts-registry.ts` and
+ * `formatters/typescript.ts`). Filesystem access that stays within Node's
+ * `path` APIs should keep using {@link resolveRepoRoot}.
+ */
+export function resolveRepoRootPosix(): string {
+    return toPosixPath(resolveRepoRoot());
+}
+
 /**
  * Resolve the `@xh/hoist` library version from the repo root `package.json`.
  *

package/mobile/cmp/grid/impl/ColChooserModel.ts

@@ -4,7 +4,7 @@
  *
  * Copyright © 2026 Extremely Heavy Industries Inc.
  */
-import {GridModel} from '@xh/hoist/cmp/grid';
+import {GridModel, IColChooserModel} from '@xh/hoist/cmp/grid';
 import {HoistModel, HSide, XH} from '@xh/hoist/core';
 import '@xh/hoist/mobile/register';
 import {bindable, action, makeObservable, observable} from '@xh/hoist/mobx';
@@ -17,7 +17,7 @@ import {clone, find, sortBy} from 'lodash';
  * It is not necessary to manually create instances of this class within an application.
  * @internal
  */
-export class ColChooserModel extends HoistModel {
+export class ColChooserModel extends HoistModel implements IColChooserModel {
     override xhImpl = true;
 
     gridModel: GridModel;

package/mobile/cmp/input/SegmentedControl.ts

@@ -85,6 +85,7 @@ export const [SegmentedControl, segmentedControl] = hoistCmp.withFactory<Segment
 interface NormalizedOption extends SegmentedControlOption {
     label: string;
     intent?: Intent;
+    testId?: string;
     _key: string;
 }
 
@@ -97,13 +98,14 @@ class SegmentedControlModel extends HoistInputModel {
         return options.map((o: any, idx: number) => {
             const key = String(idx);
             if (isObject(o)) {
-                const {label, value, icon, disabled, intent} = o as SegmentedControlOption;
+                const {label, value, icon, disabled, intent, testId} = o as SegmentedControlOption;
                 return {
                     value: this.toInternal(value),
                     label: label ?? (icon ? '' : String(value)),
                     icon,
                     disabled,
                     intent,
+                    testId,
                     _key: key
                 };
             } else {
@@ -168,7 +170,8 @@ const cmp = hoistCmp.factory<SegmentedControlModel>(({model, className, ...props
 
     const buttons = model.normalizedOptions.map(opt => {
         const optIntent = opt.intent ?? defaultIntent,
-            selected = opt._key === selectedKey;
+            selected = opt._key === selectedKey,
+            optTestId = opt.testId ?? (testId ? `${testId}-${String(opt.value)}` : null);
         // Wrap the label so it can truncate with an ellipsis when the segment is too narrow,
         // rather than hard-clipping mid-character. Pass null for icon-only options so the Button
         // renders the icon alone (an empty span would suppress that).
@@ -187,7 +190,8 @@ const cmp = hoistCmp.factory<SegmentedControlModel>(({model, className, ...props
                 selected && 'xh-segmented-control-option--selected',
                 optIntent && `xh-segmented-control-option--${optIntent}`
             ),
-            onClick: () => model.onValueChange(opt._key)
+            onClick: () => model.onValueChange(opt._key),
+            testId: optTestId
         });
     });
 

package/mobile/cmp/input/Select.ts

@@ -21,7 +21,7 @@ import {action, bindable, makeObservable, observable, override} from '@xh/hoist/
 import {debouncePromise, wait} from '@xh/hoist/promise';
 import {throwIf, withDefault, mergeDeep} from '@xh/hoist/utils/js';
 import {createObservableRef, getLayoutProps} from '@xh/hoist/utils/react';
-import {escapeRegExp, isEqual, isNil, isPlainObject, keyBy} from 'lodash';
+import {escapeRegExp, isEqual, isNil, isPlainObject, unionWith} from 'lodash';
 import {Children, ReactNode, ReactPortal} from 'react';
 import ReactDom from 'react-dom';
 import './Select.scss';
@@ -157,6 +157,15 @@ export interface SelectProps extends HoistProps, HoistInputProps, LayoutProps {
 
     /** Field on provided options for sourcing each option's value (default `value`). */
     valueField?: string;
+
+    /**
+     * Function to generate a `SelectOption` for a (non-null) selected value not present in the
+     * current options list. Return null to fall back to the default value-as-label behavior.
+     *
+     * Useful with queryFn-based selects, readonly forms, or any case where options may not be
+     * loaded when a value is set, ensuring the value renders with its proper label.
+     */
+    generateOptionFn?: (value: any) => SelectOption;
 }
 
 /**
@@ -377,7 +386,8 @@ class SelectInputModel extends HoistInputModel {
 
     // Convert external value into option object(s). Options created if missing - this takes the
     // external value from the model, and we will respect that even if we don't know about it.
-    // (Exception for a null value, which we will only accept if explicitly present in options.)
+    // (Exception for a null value, which is never synthesized - accepted only if provided via
+    // options.)
     override toInternal(external) {
         return this.findOption(external, !isNil(external));
     }
@@ -393,7 +403,10 @@ class SelectInputModel extends HoistInputModel {
             }
         }
 
-        return createIfNotFound ? this.valueToOption(value) : null;
+        if (!createIfNotFound) return null;
+
+        // Value not among options - let the app generate an option for it, else synthesize one.
+        return this.componentProps.generateOptionFn?.(value) ?? this.valueToOption(value);
     }
 
     override toExternal(internal) {
@@ -453,15 +466,9 @@ class SelectInputModel extends HoistInputModel {
 
                 // Carry forward and add to any existing internalOpts to allow our value
                 // converters to continue all selected values in multiMode.
-                const matchesByVal = keyBy(matchOpts, 'value'),
-                    newOpts = [...matchOpts];
-
-                this.internalOptions.forEach(currOpt => {
-                    const matchOpt = matchesByVal[currOpt.value];
-                    if (!matchOpt) newOpts.push(currOpt); // avoiding dupes
-                });
-
-                this.internalOptions = newOpts;
+                this.internalOptions = unionWith(matchOpts, this.internalOptions, (a, b) =>
+                    isEqual(a.value, b.value)
+                );
 
                 // But only return the matching options back to the combo.
                 return matchOpts;
@@ -484,8 +491,7 @@ class SelectInputModel extends HoistInputModel {
     // Option Rendering
     //----------------------
     formatOptionLabel = (opt, params) => {
-        // Always display the standard label string in the value container (context == 'value').
-        // If we need to expose customization here, we could consider a dedicated prop.
+        // Display the standard label string in the value container (context == 'value').
         if (params.context !== 'menu') {
             return opt.label;
         }
@@ -501,7 +507,7 @@ class SelectInputModel extends HoistInputModel {
             return div(opt.label);
         }
 
-        return this.externalValue === opt.value
+        return isEqual(this.externalValue, opt.value)
             ? hbox({
                   items: [
                       div({

package/mobx/README.md

@@ -45,7 +45,8 @@ The following are re-exported from MobX and mobx-react-lite:
 
 The package configures MobX with `enforceActions: 'observed'`, meaning any modification to
 an observable property that is currently being observed must occur inside an `@action` method,
-`runInAction()` block, or `@bindable` setter. MobX logs a warning if this rule is violated.
+`runInAction()` block, `@bindable` setter, or a `reaction`/`when` run callback (MobX
+auto-wraps these in an action). MobX logs a warning if this rule is violated.
 
 This enforcement prevents accidental state mutations and makes data flow predictable.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "86.2.0",
+  "version": "86.4.0",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": {
     "type": "git",
@@ -38,8 +38,8 @@
     ]
   },
   "dependencies": {
-    "@auth0/auth0-spa-js": "~2.21.1",
-    "@azure/msal-browser": "~5.14.0",
+    "@auth0/auth0-spa-js": "~2.23.0",
+    "@azure/msal-browser": "~5.17.0",
     "@blueprintjs/core": "^6.3.2",
     "@blueprintjs/datetime": "^6.0.6",
     "@codemirror/commands": "^6.10.3",
@@ -90,7 +90,7 @@
     "semver": "~7.8.0",
     "short-unique-id": "~5.3.2",
     "store2": "~2.14.3",
-    "swiper": "^12.1.2",
+    "swiper": "^14.0.0",
     "ts-morph": "^27.0.2",
     "tsx": "^4.21.0",
     "ua-parser-js": "~2.0.4",
@@ -118,7 +118,7 @@
     "react-dom": "~18.2.0",
     "stylelint": "17.x",
     "stylelint-config-standard-scss": "17.x",
-    "type-fest": "4.x",
+    "type-fest": "5.x",
     "typescript": "~5.9.3"
   },
   "resolutions": {

package/svc/PrefService.ts

@@ -30,9 +30,8 @@ export class PrefService extends HoistService {
     override telemetryPrefix = 'xh.client.prefs';
 
     static instance: PrefService;
-
-    private _data = {};
-    private _updates = {};
+    private _data: Record<string, PrefEntry> = {};
+    private _updates: Record<string, any> = {}; // undefined indicates unset
 
     override async initAsync(ctx: InitContext) {
         // Flush on page teardown while the page is still alive.
@@ -52,6 +51,17 @@ export class PrefService extends HoistService {
         return this._data.hasOwnProperty(key);
     }
 
+    /**
+     * Check whether the current user has an explicit value on file for the given preference, vs.
+     * receiving the preference's server-side default value.
+     *
+     * @param key - unique key used to identify the pref.
+     */
+    isSet(key: string): boolean {
+        this.ensureKeyExists(key);
+        return !!this._data[key].isSet;
+    }
+
     /**
      * Get the value for a given key, either the user-specific value (if set) or the default.
      * Typically accessed via the convenience alias {@link XH.getPref}.
@@ -91,7 +101,9 @@ export class PrefService extends HoistService {
 
         // Change local value to sanitized copy and fire.
         value = deepFreeze(cloneDeep(value));
-        this._data[key].value = value;
+        const pref = this._data[key];
+        pref.value = value;
+        pref.isSet = true;
 
         // Schedule serialization to storage
         this._updates[key] = value;
@@ -99,11 +111,22 @@ export class PrefService extends HoistService {
     }
 
     /**
-     * Restore a preference to its default value.
+     * Restore a preference to its default value, clearing the user's explicit value on the server.
+     *
+     * Unlike `set()`, this clears the user's explicit value rather than persisting the default as
+     * one - so {@link isSet} will report `false` afterwards. Saved asynchronously (see `set()`).
      */
     unset(key: string) {
-        // TODO: round-trip this to the server as a proper unset?
-        this.set(key, this._data[key]?.defaultValue);
+        this.ensureKeyExists(key);
+        const pref = this._data[key];
+        if (!pref.isSet && isEqual(pref.value, pref.defaultValue)) return;
+
+        pref.value = pref.defaultValue;
+        pref.isSet = false;
+
+        // Schedule serialization to storage
+        this._updates[key] = undefined;
+        this.pushPendingBuffered();
     }
 
     /**
@@ -131,18 +154,44 @@ export class PrefService extends HoistService {
         // Clear synchronously with the capture, so overlapping flushes cannot post twice.
         this._updates = {};
 
+        // Partition into value updates and unsets.
+        // On a core that predates unset support, fall back to persisting default
+        const setPrefs = {},
+            unsetKeys = [];
+        forEach(updates, (value, key) => {
+            const pref = this._data[key];
+            if (value !== undefined) {
+                setPrefs[key] = value;
+            } else if (pref.hasOwnProperty('isSet')) {
+                unsetKeys.push(key);
+            } else {
+                setPrefs[key] = pref.defaultValue;
+            }
+        });
+
         await this.runner()
-            .span('set')
-            .run(ctx =>
-                terminationSafePostJson(
-                    {
-                        url: 'xh/setPrefs',
-                        body: updates,
-                        params: {clientUsername: XH.getUsername()}
-                    },
-                    ctx
-                )
-            );
+            .span('update')
+            .run(async ctx => {
+                const clientUsername = XH.getUsername(),
+                    tasks = [];
+                if (!isEmpty(setPrefs)) {
+                    tasks.push(
+                        terminationSafePostJson(
+                            {url: 'xh/setPrefs', body: setPrefs, params: {clientUsername}},
+                            ctx
+                        )
+                    );
+                }
+                if (!isEmpty(unsetKeys)) {
+                    tasks.push(
+                        terminationSafePostJson(
+                            {url: 'xh/unsetPrefs', body: unsetKeys, params: {clientUsername}},
+                            ctx
+                        )
+                    );
+                }
+                await Promise.all(tasks);
+            });
     }
 
     //-------------------
@@ -172,9 +221,13 @@ export class PrefService extends HoistService {
             });
     }
 
-    private validateBeforeSet(key, value) {
+    private ensureKeyExists(key: string) {
+        throwIf(!this.hasKey(key), `Preference key not found: '${key}'`);
+    }
+
+    private validateBeforeSet(key: string, value: any) {
+        this.ensureKeyExists(key);
         const pref = this._data[key];
-        throwIf(!pref, `Cannot set preference ${key}: not found`);
         throwIf(value === undefined, `Cannot set preference ${key}: value not defined`);
         throwIf(
             !this.valueIsOfType(value, pref.type),
@@ -201,3 +254,10 @@ export class PrefService extends HoistService {
         }
     }
 }
+
+interface PrefEntry {
+    type: string;
+    value: any;
+    defaultValue: any;
+    isSet: boolean;
+}

package/svc/README.md

@@ -186,6 +186,12 @@ XH.setPref('gridPageSize', 100);
 
 // Immediate save - no alias, access service directly
 await XH.prefService.pushAsync('criticalPref', value);
+
+// Distinguish an explicit user value from the server-side default
+if (XH.prefService.isSet('gridPageSize')) { /* user has customized this */ }
+
+// Clear the user's value, reverting to the default (real server-side unset)
+XH.prefService.unset('gridPageSize');
 ```
 
 Preferences are type-validated against server-defined types: `string`, `int`, `long`, `double`,

package/utils/datetime/LocalDate.ts

@@ -18,6 +18,15 @@ import moment, {Moment, MomentInput} from 'moment';
  * For efficiency and to enable strict equality checks, instances of this class are memoized:
  * only a single version of the object will be created and returned for each calendar day,
  * as long as the caller uses one of the *public factory methods*, which they always should!
+ *
+ * Instances serialize directly to their ISO date string (e.g. '2024-01-15') via built-in
+ * `toString()`, `valueOf()`, and `toJSON()` overrides. This means a LocalDate can be passed
+ * as-is within the params or body of a `FetchService` request (or any `JSON.stringify()` call)
+ * and will serialize as expected - prefer this over calling `toString()` or `format()` yourself.
+ *
+ * Instances also support natural comparison: because they are memoized, `===` tests whether two
+ * references are the same calendar day, while `valueOf()` returning the (lexically sortable) ISO
+ * string means the relational operators `<`, `>`, `<=`, `>=` order instances chronologically.
  */
 export class LocalDate {
     static readonly VALID_UNITS: Set<LocalDateUnit> = new Set([
@@ -77,7 +86,11 @@ export class LocalDate {
         return this.get(m.format('YYYY-MM-DD'));
     }
 
-    /** LocalDate representing the current day. */
+    /**
+     * LocalDate representing the current day in the browser's local time zone.
+     * See `currentAppDay()` / `currentServerDay()` to resolve "today" in the app or server zone,
+     * which can differ from the browser for users in another region.
+     */
     static today(): LocalDate {
         return this.from(moment());
     }
@@ -120,18 +133,26 @@ export class LocalDate {
         return this._isoString;
     }
 
+    /** JS `Date` for this day at midnight in the browser's local time zone. Fresh instance per call. */
     get date(): Date {
         return new Date(this.timestamp);
     }
 
+    /** A mutable moment.js clone - safe to modify without affecting this (immutable) instance. */
     get moment(): Moment {
         return this._moment.clone();
     }
 
+    /** Epoch millis for this day at midnight in the browser's local time zone. */
     get timestamp(): number {
         return this._date.getTime();
     }
 
+    /**
+     * Format this date using moment.js format tokens, primarily for display.
+     * Note: to send a LocalDate to the server, pass the instance directly rather than a formatted
+     * string - it serializes to an ISO date on its own (see class-level docs).
+     */
     format(...args): string {
         return this._moment.format(...args);
     }
@@ -186,6 +207,7 @@ export class LocalDate {
         return this._isoString;
     }
 
+    // Returns the ISO string (not a number) so the relational operators sort instances by date.
     valueOf(): string {
         return this._isoString;
     }
@@ -319,6 +341,7 @@ export class LocalDate {
         return this.isWeekday ? this : this.previousWeekday();
     }
 
+    /** Difference between this date and `other` in the given unit; positive when this is later. */
     diff(other: LocalDate, unit: LocalDateUnit = 'days'): number {
         this.ensureUnitValid(unit);
         return this._moment.diff(other._moment, unit);

package/desktop/cmp/filter/PopoverFilterChooser.scss

@@ -1,42 +0,0 @@
-/*
- * This file belongs to Hoist, an application development toolkit
- * developed by Extremely Heavy Industries (www.xh.io | info@xh.io)
- *
- * Copyright © 2026 Extremely Heavy Industries Inc.
- */
-.xh-popover-filter-chooser {
-  & > .bp6-popover-target {
-    display: flex;
-    flex: 1;
-  }
-
-  // Extra class names required to override the default styles of the popover
-  &__popover.bp6-popover.bp6-minimal {
-    margin-top: -15px !important;
-    box-shadow: none;
-
-    .bp6-popover-content {
-      background: transparent;
-    }
-
-    .xh-select__value-container--is-multi {
-      height: unset;
-      line-height: unset;
-    }
-  }
-
-  &__filter-chooser {
-    .xh-select__value-container--is-multi {
-      overflow-y: hidden !important;
-    }
-
-    .xh-select {
-      .xh-select__control--is-disabled {
-        background: var(--xh-input-bg);
-      }
-      .xh-select__multi-value--is-disabled .xh-select__multi-value__label {
-        color: unset;
-      }
-    }
-  }
-}