@xh/hoist 75.0.0-SNAPSHOT.1753727619217 → 75.0.0-SNAPSHOT.1754329446115

package/CHANGELOG.md

@@ -23,6 +23,11 @@
   which `dimensions` are provided to the model.
 * Added new `ClipboardButton.errorMessage` prop to customize or suppress a toast alert if the copy
   operation fails. Set to `false` to fail silently (the behavior prior to this change).
+* Added new `Cube.modifyRecordsAsync` for modifying individual field values in a local uncommitted
+  state. Additionally enhanced `Store.modifyRecords` to return a `StoreChangeLog` of updates.
+
+### 🐞 Bug Fixes
+* Fixed bug where `Store.modifyRecords` was not properly handling changes to `SummaryRecords`.
 
 ### 🐞 Bug Fixes
 
@@ -55,11 +60,20 @@
 * Removed deprecated `FetchService.setDefaultTimeout`
 * Removed deprecated `IdentityService.logoutAsync`
 
+### ✨ Styles
+
+* Upgraded the version of Hoist's default Inter UI font to a new major version, now v4.1. Note
+  that this brings slight differences to the font's appearance, including tweaks to internal
+  spacing and letterforms for tabular numbers. The name of the font face has also changed, from
+  `Inter Var` to `InterVariable`. The default value of the `--xh-font-family` CSS variable has been
+  updated to match, making this change transparent for most applications.
+
 ### 📚 Libraries
 
 * @auth0/auth0-spa-js `2.1 → 2.3`
 * @azure/msal-browser `4.12 → 4.16`
 * filesize `6.4 → 11.0`
+* inter-ui `3.19 → 4.1`
 * mobx-react-lite `4.0 → 4.1`
 * qs `6.13 → 6.14`
 * react-markdown `9.0 → 10.1`

package/build/types/cmp/dataview/DataViewModel.d.ts

@@ -110,7 +110,7 @@ export declare class DataViewModel extends HoistModel {
     ensureSelectionVisibleAsync(): Promise<void>;
     doLoadAsync(loadSpec: LoadSpec): Promise<any>;
     loadData(rawData: any[], rawSummaryData?: PlainObject): void;
-    updateData(rawData: PlainObject[] | StoreTransaction): PlainObject;
+    updateData(rawData: PlainObject[] | StoreTransaction): import("@xh/hoist/data").StoreChangeLog;
     clear(): void;
     setGroupBy(colIds: Some<string>): void;
     setSortBy(sorters: Some<GridSorterLike>): void;

package/build/types/cmp/grid/GridModel.d.ts

@@ -450,7 +450,7 @@ export declare class GridModel extends HoistModel {
     /** Load the underlying store. */
     loadData(rawData: any[], rawSummaryData?: Some<PlainObject>): void;
     /** Update the underlying store. */
-    updateData(rawData: PlainObject[] | StoreTransaction): PlainObject;
+    updateData(rawData: PlainObject[] | StoreTransaction): import("@xh/hoist/data").StoreChangeLog;
     /** Clear the underlying store, removing all rows. */
     clear(): void;
     /** @param colConfigs - {@link Column} or {@link ColumnGroup} configs. */

package/build/types/cmp/zoneGrid/ZoneGridModel.d.ts

@@ -243,7 +243,7 @@ export declare class ZoneGridModel extends HoistModel {
     ensureSelectionVisibleAsync(): Promise<void>;
     doLoadAsync(loadSpec: LoadSpec): Promise<any>;
     loadData(rawData: any[], rawSummaryData?: Some<PlainObject>): void;
-    updateData(rawData: PlainObject[] | StoreTransaction): PlainObject;
+    updateData(rawData: PlainObject[] | StoreTransaction): import("@xh/hoist/data").StoreChangeLog;
     clear(): void;
     setGroupBy(colIds: Some<string>): void;
     private createGridModel;

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

@@ -104,6 +104,16 @@ export interface StoreTransaction {
      */
     rawSummaryData?: Some<PlainObject>;
 }
+/**
+ * Collection of changes made to a Store's RecordSet. Unlike `StoreTransaction` which is used to
+ * specify changes, this object is used to report the actual changes made in a single transaction.
+ */
+export interface StoreChangeLog {
+    update?: StoreRecord[];
+    add?: StoreRecord[];
+    remove?: StoreRecordId[];
+    summaryRecords?: StoreRecord[];
+}
 export interface ChildRawData {
     /** ID of the pre-existing parent record. */
     parentId: string;
@@ -197,7 +207,7 @@ export declare class Store extends HoistBase {
      *      into adds and updates, with updates determined by matching existing records by ID.
      * @returns changes applied, or null if no record changes were made.
      */
-    updateData(rawData: PlainObject[] | StoreTransaction): PlainObject;
+    updateData(rawData: PlainObject[] | StoreTransaction): StoreChangeLog;
     /**
      * Re-runs the Filter on the current data. Applications only need to call this method if
      * the state underlying the filter, other than the record data itself, has changed. Store will
@@ -242,8 +252,9 @@ export declare class Store extends HoistBase {
      *      Records in this Store. Each object in the list must have an `id` property identifying
      *      the StoreRecord to modify, plus any other properties with updated field values to apply,
      *      e.g. `{id: 4, quantity: 100}, {id: 5, quantity: 99, customer: 'bob'}`.
+     * @returns changes applied, or null if no record changes were made.
      */
-    modifyRecords(modifications: Some<PlainObject>): void;
+    modifyRecords(modifications: Some<PlainObject>): StoreChangeLog;
     /**
      * Revert all changes made to the specified Records since they were last committed.
      *

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

@@ -1,4 +1,4 @@
-import { HoistBase, PlainObject } from '@xh/hoist/core';
+import { HoistBase, PlainObject, Some } from '@xh/hoist/core';
 import { CubeField, CubeFieldSpec } from './CubeField';
 import { QueryConfig } from './Query';
 import { View } from './View';
@@ -126,6 +126,19 @@ export declare class Cube extends HoistBase {
      * @param infoUpdates - new key-value pairs to be applied to existing info on this cube.
      */
     updateDataAsync(rawData: PlainObject[] | StoreTransaction, infoUpdates?: PlainObject): Promise<void>;
+    /**
+     * Similar to `updateDataAsync`, but intended for modifying individual field values in a local
+     * uncommitted state - i.e. when updating via an inline grid editor or similar control. Like
+     * `updateDataAsync`, this method will update its views asynchronously.
+     *
+     * This method largely delegates to {@link Store.modifyRecords} - see that method for more info.
+     *
+     * @param modifications - field-level modifications to apply to existing
+     *      Records in this Cube. Each object in the list must have an `id` property identifying
+     *      the StoreRecord to modify, plus any other properties with updated field values to apply,
+     *      e.g. `{id: 4, quantity: 100}, {id: 5, quantity: 99, customer: 'bob'}`.
+     */
+    modifyRecordsAsync(modifications: Some<PlainObject>): Promise<void>;
     /** Clear any/all data and info from this Cube. */
     clearAsync(): Promise<void>;
     /**

package/data/Store.ts

@@ -153,6 +153,17 @@ export interface StoreTransaction {
     rawSummaryData?: Some<PlainObject>;
 }
 
+/**
+ * Collection of changes made to a Store's RecordSet. Unlike `StoreTransaction` which is used to
+ * specify changes, this object is used to report the actual changes made in a single transaction.
+ */
+export interface StoreChangeLog {
+    update?: StoreRecord[];
+    add?: StoreRecord[];
+    remove?: StoreRecordId[];
+    summaryRecords?: StoreRecord[];
+}
+
 export interface ChildRawData {
     /** ID of the pre-existing parent record. */
     parentId: string;
@@ -348,13 +359,13 @@ export class Store extends HoistBase {
      */
     @action
     @logWithDebug
-    updateData(rawData: PlainObject[] | StoreTransaction): PlainObject {
+    updateData(rawData: PlainObject[] | StoreTransaction): StoreChangeLog {
         if (isEmpty(rawData)) return null;
 
-        const changeLog: PlainObject = {};
+        const changeLog: StoreChangeLog = {};
 
         // Build a transaction object out of a flat list of adds and updates
-        let rawTransaction;
+        let rawTransaction: StoreTransaction;
         if (isArray(rawData)) {
             const update = [],
                 add = [];
@@ -381,7 +392,7 @@ export class Store extends HoistBase {
         throwIf(!isEmpty(other), 'Unknown argument(s) passed to updateData().');
 
         // 1) Pre-process updates and adds into Records
-        let updateRecs, addRecs;
+        let updateRecs: StoreRecord[], addRecs: Map<StoreRecordId, StoreRecord>;
         if (update) {
             updateRecs = update.map(it => {
                 const recId = this.idSpec(it),
@@ -426,7 +437,11 @@ export class Store extends HoistBase {
         }
 
         // 3) Apply changes
-        let rsTransaction: any = {};
+        let rsTransaction: {
+            update?: StoreRecord[];
+            add?: StoreRecord[];
+            remove?: StoreRecordId[];
+        } = {};
         if (!isEmpty(updateRecs)) rsTransaction.update = updateRecs;
         if (!isEmpty(addRecs)) rsTransaction.add = Array.from(addRecs.values());
         if (!isEmpty(remove)) rsTransaction.remove = remove;
@@ -545,13 +560,15 @@ export class Store extends HoistBase {
      *      Records in this Store. Each object in the list must have an `id` property identifying
      *      the StoreRecord to modify, plus any other properties with updated field values to apply,
      *      e.g. `{id: 4, quantity: 100}, {id: 5, quantity: 99, customer: 'bob'}`.
+     * @returns changes applied, or null if no record changes were made.
      */
     @action
-    modifyRecords(modifications: Some<PlainObject>) {
+    modifyRecords(modifications: Some<PlainObject>): StoreChangeLog {
         modifications = castArray(modifications);
         if (isEmpty(modifications)) return;
 
-        const updateRecs = new Map();
+        // 1) Pre-process modifications into Records
+        const updateMap = new Map<StoreRecordId, StoreRecord>();
         let hadDupes = false;
         modifications.forEach(mod => {
             let {id} = mod;
@@ -559,7 +576,7 @@ export class Store extends HoistBase {
             // Ignore multiple updates for the same record - we are updating this Store in a
             // transaction after processing all modifications, so this method is not currently setup
             // to process more than one update for a given rec at a time.
-            if (updateRecs.has(id)) {
+            if (updateMap.has(id)) {
                 hadDupes = true;
                 return;
             }
@@ -577,20 +594,40 @@ export class Store extends HoistBase {
             });
 
             if (!equal(currentRec.data, updatedRec.data)) {
-                updateRecs.set(id, updatedRec);
+                updateMap.set(id, updatedRec);
             }
         });
 
-        if (isEmpty(updateRecs)) return;
+        if (isEmpty(updateMap)) return null;
 
         warnIf(
             hadDupes,
             'Store.modifyRecords() called with multiple updates for the same Records. Only the first modification for each StoreRecord was processed.'
         );
 
-        this._current = this._current.withTransaction({update: Array.from(updateRecs.values())});
+        const updateRecs = Array.from(updateMap.values()),
+            changeLog: StoreChangeLog = {};
 
-        this.rebuildFiltered();
+        // 2) Pre-process summary records, peeling them out of updates if needed
+        const {summaryRecords} = this;
+        let summaryUpdateRecs: StoreRecord[];
+        if (!isEmpty(summaryRecords)) {
+            summaryUpdateRecs = lodashRemove(updateRecs, ({id}) => some(summaryRecords, {id}));
+        }
+
+        if (!isEmpty(summaryUpdateRecs)) {
+            this.summaryRecords = summaryUpdateRecs;
+            changeLog.summaryRecords = this.summaryRecords;
+        }
+
+        // 3) Apply changes
+        if (!isEmpty(updateRecs)) {
+            this._current = this._current.withTransaction({update: updateRecs});
+            changeLog.update = updateRecs;
+            this.rebuildFiltered();
+        }
+
+        return changeLog;
     }
 
     /**
@@ -705,7 +742,7 @@ export class Store extends HoistBase {
     /** True if the store has changes which need to be committed. */
     @computed
     get isDirty(): boolean {
-        return this._current !== this._committed;
+        return this._current !== this._committed || this.summaryRecords?.some(it => it.isModified);
     }
 
     /** Alias for {@link Store.isDirty} */

package/data/cube/Cube.ts

@@ -5,7 +5,7 @@
  * Copyright © 2025 Extremely Heavy Industries Inc.
  */
 
-import {HoistBase, managed, PlainObject} from '@xh/hoist/core';
+import {HoistBase, managed, PlainObject, Some} from '@xh/hoist/core';
 import {action, makeObservable, observable} from '@xh/hoist/mobx';
 import {forEachAsync} from '@xh/hoist/utils/async';
 import {CubeField, CubeFieldSpec} from './CubeField';
@@ -244,6 +244,26 @@ export class Cube extends HoistBase {
         }
     }
 
+    /**
+     * Similar to `updateDataAsync`, but intended for modifying individual field values in a local
+     * uncommitted state - i.e. when updating via an inline grid editor or similar control. Like
+     * `updateDataAsync`, this method will update its views asynchronously.
+     *
+     * This method largely delegates to {@link Store.modifyRecords} - see that method for more info.
+     *
+     * @param modifications - field-level modifications to apply to existing
+     *      Records in this Cube. Each object in the list must have an `id` property identifying
+     *      the StoreRecord to modify, plus any other properties with updated field values to apply,
+     *      e.g. `{id: 4, quantity: 100}, {id: 5, quantity: 99, customer: 'bob'}`.
+     */
+    async modifyRecordsAsync(modifications: Some<PlainObject>): Promise<void> {
+        const changeLog = this.store.modifyRecords(modifications);
+
+        if (changeLog) {
+            await forEachAsync(this._connectedViews, v => v.noteCubeUpdated(changeLog));
+        }
+    }
+
     /** Clear any/all data and info from this Cube. */
     async clearAsync() {
         await this.loadDataAsync([]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "75.0.0-SNAPSHOT.1753727619217",
+  "version": "75.0.0-SNAPSHOT.1754329446115",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",
@@ -54,7 +54,7 @@
     "filesize": "~11.0.2",
     "golden-layout": "~1.5.9",
     "http-status-codes": "~2.3.0",
-    "inter-ui": "~3.19.3",
+    "inter-ui": "~4.1.1",
     "jwt-decode": "~4.0.0",
     "lodash": "~4.17.21",
     "lodash-inflection": "~1.5.0",

package/styles/XH.scss

@@ -9,7 +9,7 @@
 @use 'helpers';
 @use '~inter-ui/variable' with (
   $inter-font-display: block,
-  $inter-font-path: '~inter-ui/Inter (web)'
+  $inter-font-path: '~inter-ui/variable'
 );
 
 @include variable.all;

package/styles/vars.scss

@@ -374,7 +374,7 @@ body {
   //------------------------
   // Fonts
   //------------------------
-  --xh-font-family: var(--font-family, #{("Inter Var", system-ui, sans-serif)});
+  --xh-font-family: var(--font-family, #{("InterVariable", system-ui, sans-serif)});
   --xh-font-family-headings: var(--font-family-headings, var(--xh-font-family));
   --xh-font-family-mono: var(--font-family-mono, #{(Monaco, Consolas, monospace)});