@xh/hoist 77.0.0-SNAPSHOT.1761157373322 → 77.0.0-SNAPSHOT.1761257771095

package/CHANGELOG.md

@@ -2,14 +2,6 @@
 
 ## 77.0.0-SNAPSHOT - unreleased
 
-## 76.2.0 - 2025-10-22
-
-### ⚙️ Technical
-
-* Performance improvements to Store for large data sets.
-* New property `cubeRowType` on `ViewRowData` supports identifying bucketed rows.
-* `waitFor` can accept a null value for a timeout.
-
 ## 76.1.0 - 2025-10-17
 
 ### 🎁 New Features

package/admin/jsonsearch/impl/JsonSearchImplModel.ts

@@ -73,7 +73,7 @@ export class JsonSearchImplModel extends HoistModel {
     override onLinked() {
         this.gridModel = new GridModel({
             ...this.gridModelConfig,
-            emptyText: 'No matches found.',
+            emptyText: 'No matches found...',
             selModel: 'single'
         });
 

package/admin/tabs/activity/tracking/ActivityTrackingModel.ts

@@ -395,7 +395,7 @@ export class ActivityTrackingModel extends HoistModel implements ActivityDetailP
             treeStyle: TreeStyle.HIGHLIGHTS_AND_BORDERS,
             autosizeOptions: {mode: 'managed', includeCollapsedChildren: true},
             exportOptions: {filename: exportFilename('activity-summary')},
-            emptyText: 'No activity reported.',
+            emptyText: 'No activity reported...',
             sortBy: ['cubeLabel'],
             expandLevel: 1,
             levelLabels: () => ['Total', ...this.groupingChooserModel.valueDisplayNames],

package/admin/tabs/cluster/instances/logs/LogDisplayModel.ts

@@ -136,7 +136,7 @@ export class LogDisplayModel extends HoistModel {
             hideHeaders: true,
             rowBorders: false,
             sizingMode: 'tiny',
-            emptyText: 'No log entries found.',
+            emptyText: 'No log entries found...',
             sortBy: 'rowNum|asc',
             autosizeOptions: {mode: 'disabled'},
             store: {

package/admin/tabs/userData/users/UserModel.ts

@@ -25,7 +25,6 @@ export class UserModel extends HoistModel {
         makeObservable(this);
 
         this.gridModel = new GridModel({
-            emptyText: 'No users found.',
             persistWith: this.persistWith,
             colChooserModel: true,
             enableExport: true,

package/build/types/data/Store.d.ts

@@ -401,7 +401,6 @@ export declare class Store extends HoistBase {
     private rebuildFiltered;
     private createRecord;
     private createRecords;
-    private get summaryRecordIds();
     private parseRaw;
     private parseUpdate;
     private createDataDefaults;

package/build/types/data/cube/ViewRowData.d.ts

@@ -7,8 +7,6 @@ export declare class ViewRowData {
     constructor(id: string);
     /** Unique id. */
     id: string;
-    /** Denotes a type for the row */
-    cubeRowType: 'leaf' | 'aggregate' | 'bucket';
     /**
      * Label of the row. The dimension value or, for leaf rows. the underlying cubeId.
      * Suitable for display, although apps will typically wish to customize leaf row rendering.

package/cmp/chart/impl/copyToClipboard.ts

@@ -24,14 +24,14 @@ export function installCopyToClipboard(Highcharts) {
             try {
                 const blobPromise = convertChartToPngAsync(this),
                     clipboardItemInput = new window.ClipboardItem({
-                        // Safari requires an unresolved promise.  See https://bugs.webkit.org/show_bug.cgi?id=222262 for discussion
+                        // Safari requires an unresolved promise. See https://bugs.webkit.org/show_bug.cgi?id=222262 for discussion
                         'image/png': Highcharts.isSafari ? blobPromise : await blobPromise
                     });
                 await window.navigator.clipboard.write([clipboardItemInput]);
                 XH.successToast('Chart copied to clipboard');
             } catch (e) {
                 XH.handleException(e, {showAlert: false, logOnServer: true});
-                XH.dangerToast('Error: Chart could not be copied.  This error has been logged.');
+                XH.dangerToast('Error: Chart could not be copied. This error has been logged.');
             }
         }
     });
@@ -41,14 +41,8 @@ export function installCopyToClipboard(Highcharts) {
 // Implementation
 //------------------
 async function convertChartToPngAsync(chart) {
-    const svg = await new Promise((resolve, reject) =>
-            chart.getSVGForLocalExport(
-                chart.options.exporting,
-                {},
-                () => reject('Cannot fallback to export server'),
-                svg => resolve(svg)
-            )
-        ),
+    // v12 replacement for getSVGForLocalExport
+    const svg = chart.getSVG(),
         svgUrl = svgToDataUrl(svg),
         pngDataUrl = await svgUrlToPngDataUrlAsync(svgUrl),
         ret = await loadBlob(pngDataUrl);
@@ -65,7 +59,7 @@ function memoryCleanup(svgUrl) {
 }
 
 /**
- * Convert dataUri converted to blob
+ * Convert dataUri to blob
  */
 async function loadBlob(dataUrl) {
     const fetched = await fetch(dataUrl);
@@ -84,7 +78,7 @@ function svgToDataUrl(svg) {
     try {
         // Safari requires data URI since it doesn't allow navigation to blob
         // URLs.
-        // foreignObjects dont work well in Blobs in Chrome (#14780).
+        // foreignObjects don't work well in Blobs in Chrome (#14780).
         if (!isWebKitButNotChrome && svg.indexOf('<foreignObject') === -1) {
             return domurl.createObjectURL(
                 new window.Blob([svg], {
@@ -94,12 +88,12 @@ function svgToDataUrl(svg) {
         }
     } catch (e) {}
 
-    // safari, firefox, or svgs with foreignObect returns this
+    // Safari, Firefox, or SVGs with foreignObject fallback
     return 'data:image/svg+xml;charset=UTF-8,' + encodeURIComponent(svg);
 }
 
 /**
- * Get PNG data:URL from image URL. Pass in callbacks to handle results.
+ * Get PNG data:URL from image URL.
  */
 async function svgUrlToPngDataUrlAsync(imageURL, scale = 1) {
     const img = new window.Image(),

package/cmp/treemap/TreeMap.ts

@@ -172,8 +172,6 @@ class TreeMapLocalModel extends HoistModel {
             this.prevConfig = cloneDeep(chartCfg);
             this.createChart(config);
         }
-
-        this.updateLabelVisibility();
     }
 
     createChart(config) {
@@ -195,13 +193,25 @@ class TreeMapLocalModel extends HoistModel {
 
         assign(config.chart, parentDims, {renderTo: chartElem});
         this.withDebug(['Creating new TreeMap', `${newData.length} records`], () => {
-            this.chart = Highcharts.chart(config);
+            this.chart = Highcharts.chart(config, () => {
+                this.updateLabelVisibility();
+            });
         });
     }
 
     @logWithDebug
     reloadSeriesData(newData) {
-        this.chart?.series[0].setData(newData, true, false);
+        if (!this.chart) return;
+
+        this.chart.series[0].setData(newData, true, false);
+
+        // Use an event handler to trigger label updates
+        // This approach was required when `cluster` series option is enabled
+        const onRedraw = () => {
+            this.updateLabelVisibility();
+            Highcharts.removeEvent(this.chart, 'redraw', onRedraw);
+        };
+        Highcharts.addEvent(this.chart, 'redraw', onRedraw);
     }
 
     startResize = ({width, height}) => {

package/cmp/treemap/TreeMapModel.ts

@@ -465,7 +465,7 @@ export class TreeMapModel extends HoistModel {
     //----------------------
     defaultOnClick = (record, e) => {
         const {gridModel} = this;
-        if (!gridModel) return;
+        if (!gridModel || !record) return;
 
         // Select nodes in grid
         const {selModel} = gridModel;
@@ -477,7 +477,7 @@ export class TreeMapModel extends HoistModel {
     };
 
     defaultOnDoubleClick = record => {
-        if (!this.gridModel?.treeMode || isEmpty(record.children)) return;
+        if (!this.gridModel?.treeMode || isEmpty(record?.children)) return;
         this.toggleNodeExpanded(record.treePath);
     };
 }

package/data/Store.ts

@@ -1041,36 +1041,26 @@ export class Store extends HoistBase {
         return ret;
     }
 
-    private createRecords(
-        rawData: PlainObject[],
-        parent: StoreRecord,
-        recordMap: Map<StoreRecordId, StoreRecord> = new Map(),
-        summaryRecordIds: Set<StoreRecordId> = this.summaryRecordIds
-    ) {
+    private createRecords(rawData: PlainObject[], parent: StoreRecord, recordMap = new Map()) {
         const {loadTreeData, loadTreeDataFrom} = this;
-
         rawData.forEach(raw => {
             const rec = this.createRecord(raw, parent),
                 {id} = rec;
 
             throwIf(
-                recordMap.has(id) || summaryRecordIds.has(id),
+                recordMap.has(id) || this.summaryRecords?.some(it => it.id === id),
                 `ID ${id} is not unique. Use the 'Store.idSpec' config to resolve a unique ID for each record.`
             );
 
             recordMap.set(id, rec);
 
             if (loadTreeData && raw[loadTreeDataFrom]) {
-                this.createRecords(raw[loadTreeDataFrom], rec, recordMap, summaryRecordIds);
+                this.createRecords(raw[loadTreeDataFrom], rec, recordMap);
             }
         });
         return recordMap;
     }
 
-    private get summaryRecordIds(): Set<StoreRecordId> {
-        return new Set(this.summaryRecords?.map(it => it.id) ?? []);
-    }
-
     private parseRaw(data: PlainObject): PlainObject {
         // a) create/prepare the data object
         const ret = Object.create(this._dataDefaults);

package/data/cube/ViewRowData.ts

@@ -19,9 +19,6 @@ export class ViewRowData {
     /** Unique id. */
     id: string;
 
-    /** Denotes a type for the row */
-    cubeRowType: 'leaf' | 'aggregate' | 'bucket';
-
     /**
      * Label of the row. The dimension value or, for leaf rows. the underlying cubeId.
      * Suitable for display, although apps will typically wish to customize leaf row rendering.

package/data/cube/row/AggregateRow.ts

@@ -38,7 +38,6 @@ export class AggregateRow extends BaseRow {
 
         this.dim = dim;
         this.dimName = dimName;
-        this.data.cubeRowType = 'aggregate';
         this.data.cubeLabel = strVal;
         this.data.cubeDimension = dimName;
 

package/data/cube/row/BucketRow.ts

@@ -35,7 +35,6 @@ export class BucketRow extends BaseRow {
         super(view, id);
 
         this.bucketSpec = bucketSpec;
-        this.data.cubeRowType = 'bucket';
         this.data.cubeLabel = bucketSpec.labelFn(bucketVal);
         this.data.cubeDimension = bucketSpec.name;
 

package/data/cube/row/LeafRow.ts

@@ -33,8 +33,8 @@ export class LeafRow extends BaseRow {
 
     constructor(view: View, id: string, rawRecord: StoreRecord) {
         super(view, id);
+
         this.cubeRecordId = rawRecord.id;
-        this.data.cubeRowType = 'leaf';
         this.data.cubeLabel = rawRecord.id.toString();
         this.data.cubeDimension = null;
 

package/kit/highcharts/index.ts

@@ -9,8 +9,8 @@ import {checkVersion, logError} from '@xh/hoist/utils/js';
 
 export let Highcharts = null;
 
-const MIN_VERSION = '11.1.0';
-const MAX_VERSION = '11.*.*';
+const MIN_VERSION = '12.4.0';
+const MAX_VERSION = '12.*.*';
 
 /**
  * Expose application versions of Highcharts to Hoist.

package/package.json

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

package/promise/Promise.ts

@@ -132,15 +132,15 @@ export function waitFor(
     condition: () => boolean,
     {interval = 50, timeout = 5 * SECONDS}: {interval?: number; timeout?: number} = {}
 ): Promise<void> {
-    if (interval <= 0) throw new Error('Invalid interval');
-    if (timeout != null && timeout <= 0) throw new Error('Invalid timeout');
+    if (!isNumber(interval) || interval <= 0) throw new Error('Invalid interval');
+    if (!isNumber(timeout) || timeout <= 0) throw new Error('Invalid timeout');
 
     const startTime = Date.now();
     return new Promise((resolve, reject) => {
         const resolveOnMet = () => {
             if (condition()) {
                 resolve();
-            } else if (timeout != null && olderThan(startTime, timeout)) {
+            } else if (olderThan(startTime, timeout)) {
                 reject(Exception.timeout({interval: Date.now() - startTime}));
             } else {
                 setTimeout(resolveOnMet, interval);