@xh/hoist 75.0.0-SNAPSHOT.1753731626581 → 75.0.0-SNAPSHOT.1754335298677

package/CHANGELOG.md

@@ -23,6 +23,12 @@
   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 bugs where `Store.modifyRecords`, `Store.revertRecords` and `Store.revert` were not properly
+  handling changes to `SummaryRecords`.
 
 ### 🐞 Bug Fixes
 

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.
      *
@@ -396,4 +407,5 @@ export declare class Store extends HoistBase {
     private createFieldMap;
     private parseExperimental;
     private parseIdSpec;
+    private revertSummaryRecords;
 }

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

@@ -24,7 +24,8 @@ import {
     remove as lodashRemove,
     uniq,
     first,
-    some
+    some,
+    partition
 } from 'lodash';
 import {Field, FieldSpec} from './Field';
 import {parseFilter} from './filter/Utils';
@@ -153,6 +154,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 +360,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 +393,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 +438,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 +561,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 +577,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;
             }
@@ -573,24 +591,45 @@ export class Store extends HoistBase {
                 data: updatedData,
                 parent: currentRec.parent,
                 store: currentRec.store,
-                committedData: currentRec.committedData
+                committedData: currentRec.committedData,
+                isSummary: currentRec.isSummary
             });
 
             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;
     }
 
     /**
@@ -606,13 +645,20 @@ export class Store extends HoistBase {
         records = castArray(records);
         if (isEmpty(records)) return;
 
-        const recs = records.map(it => (it instanceof StoreRecord ? it : this.getOrThrow(it)));
+        const recs = records.map(it => (it instanceof StoreRecord ? it : this.getOrThrow(it))),
+            [summaryRecsToRevert, recsToRevert] = partition(recs, 'isSummary');
 
-        this._current = this._current
-            .withTransaction({update: recs.map(r => this.getCommittedOrThrow(r.id))})
-            .normalize(this._committed);
+        if (!isEmpty(summaryRecsToRevert)) {
+            this.revertSummaryRecords(summaryRecsToRevert);
+        }
 
-        this.rebuildFiltered();
+        if (!isEmpty(recsToRevert)) {
+            this._current = this._current
+                .withTransaction({update: recsToRevert.map(r => this.getCommittedOrThrow(r.id))})
+                .normalize(this._committed);
+
+            this.rebuildFiltered();
+        }
     }
 
     /**
@@ -625,6 +671,7 @@ export class Store extends HoistBase {
     @action
     revert() {
         this._current = this._committed;
+        if (this.summaryRecords) this.revertSummaryRecords(this.summaryRecords);
         this.rebuildFiltered();
     }
 
@@ -705,7 +752,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} */
@@ -1082,6 +1129,24 @@ export class Store extends HoistBase {
             'idSpec should be either a name of a field, or a function to generate an id.'
         );
     }
+
+    @action
+    private revertSummaryRecords(records: StoreRecord[]) {
+        this.summaryRecords = this.summaryRecords.map(summaryRec => {
+            const recToRevert = records.find(it => it.id === summaryRec.id);
+            if (!recToRevert) return summaryRec;
+
+            const ret = new StoreRecord({
+                id: recToRevert.id,
+                raw: recToRevert.raw,
+                data: {...recToRevert.committedData},
+                store: this,
+                isSummary: true
+            });
+            ret.finalize();
+            return ret;
+        });
+    }
 }
 
 //---------------------------------------------------------------------

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.1753731626581",
+  "version": "75.0.0-SNAPSHOT.1754335298677",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",