@odoo/o-spreadsheet 18.0.16 → 18.0.17

package/dist/o-spreadsheet.cjs.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 18.0.16
- * @date 2025-02-14T08:44:19.475Z
- * @hash 39979ab
+ * @version 18.0.17
+ * @date 2025-02-25T05:58:39.632Z
+ * @hash 2ee4347
  */
 
 'use strict';
@@ -2066,17 +2066,7 @@ function toZoneWithoutBoundaryChanges(xc) {
  */
 function toUnboundedZone(xc) {
     const zone = toZoneWithoutBoundaryChanges(xc);
-    if (zone.right !== undefined && zone.right < zone.left) {
-        const tmp = zone.left;
-        zone.left = zone.right;
-        zone.right = tmp;
-    }
-    if (zone.bottom !== undefined && zone.bottom < zone.top) {
-        const tmp = zone.top;
-        zone.top = zone.bottom;
-        zone.bottom = tmp;
-    }
-    return zone;
+    return reorderZone(zone);
 }
 /**
  * Convert from a cartesian reference to a Zone.
@@ -2350,11 +2340,11 @@ function positions(zone) {
     return positions;
 }
 function reorderZone(zone) {
-    if (zone.left > zone.right) {
-        zone = { left: zone.right, right: zone.left, top: zone.top, bottom: zone.bottom };
+    if (zone.right !== undefined && zone.left > zone.right) {
+        zone = { ...zone, left: zone.right, right: zone.left };
     }
-    if (zone.top > zone.bottom) {
-        zone = { left: zone.left, right: zone.right, top: zone.bottom, bottom: zone.top };
+    if (zone.bottom !== undefined && zone.top > zone.bottom) {
+        zone = { ...zone, top: zone.bottom, bottom: zone.top };
     }
     return zone;
 }
@@ -3248,12 +3238,12 @@ function isTargetDependent(cmd) {
 function isRangeDependant(cmd) {
     return "ranges" in cmd;
 }
-function isZoneDependent(cmd) {
-    return "zone" in cmd;
-}
 function isPositionDependent(cmd) {
     return "col" in cmd && "row" in cmd && "sheetId" in cmd;
 }
+function isZoneDependent(cmd) {
+    return "sheetId" in cmd && "zone" in cmd;
+}
 const invalidateEvaluationCommands = new Set([
     "RENAME_SHEET",
     "DELETE_SHEET",
@@ -3265,6 +3255,7 @@ const invalidateEvaluationCommands = new Set([
     "REDO",
     "ADD_MERGE",
     "REMOVE_MERGE",
+    "DUPLICATE_SHEET",
     "UPDATE_LOCALE",
     "ADD_PIVOT",
     "UPDATE_PIVOT",
@@ -3293,7 +3284,6 @@ const invalidateChartEvaluationCommands = new Set([
 ]);
 const invalidateDependenciesCommands = new Set(["MOVE_RANGES"]);
 const invalidateCFEvaluationCommands = new Set([
-    "DUPLICATE_SHEET",
     "EVALUATE_CELLS",
     "ADD_CONDITIONAL_FORMAT",
     "REMOVE_CONDITIONAL_FORMAT",
@@ -3462,6 +3452,7 @@ exports.CommandResult = void 0;
     CommandResult["InvalidRange"] = "InvalidRange";
     CommandResult["InvalidZones"] = "InvalidZones";
     CommandResult["InvalidSheetId"] = "InvalidSheetId";
+    CommandResult["InvalidCellId"] = "InvalidCellId";
     CommandResult["InvalidFigureId"] = "InvalidFigureId";
     CommandResult["InputAlreadyFocused"] = "InputAlreadyFocused";
     CommandResult["MaximumRangesReached"] = "MaximumRangesReached";
@@ -6360,11 +6351,12 @@ class UuidGenerator {
         if (this.isFastIdStrategy) {
             this.fastIdStart++;
             return String(this.fastIdStart);
-            //@ts-ignore
         }
-        else if (window.crypto && window.crypto.getRandomValues) {
-            //@ts-ignore
-            return ([1e7] + -1e3).replace(/[018]/g, (c) => (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16));
+        else if (window.crypto) {
+            return "10000000-1000".replace(/[01]/g, (c) => {
+                const n = Number(c);
+                return (n ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (n / 4)))).toString(16);
+            });
         }
         else {
             // mainly for jest and other browsers that do not have the crypto functionality
@@ -6382,11 +6374,12 @@ class UuidGenerator {
         if (this.isFastIdStrategy) {
             this.fastIdStart++;
             return String(this.fastIdStart);
-            //@ts-ignore
         }
-        else if (window.crypto && window.crypto.getRandomValues) {
-            //@ts-ignore
-            return ([1e7] + -1e3 + -4e3 + -8e3 + -1e11).replace(/[018]/g, (c) => (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16));
+        else if (window.crypto) {
+            return "10000000-1000-4000-8000-100000000000".replace(/[018]/g, (c) => {
+                const n = Number(c);
+                return (n ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (n / 4)))).toString(16);
+            });
         }
         else {
             // mainly for jest and other browsers that do not have the crypto functionality
@@ -9670,6 +9663,9 @@ function getTrendDatasetForBarChart(config, dataset) {
     const filteredValues = [];
     const filteredLabels = [];
     const labels = [];
+    if (dataset.hidden) {
+        return;
+    }
     for (let i = 0; i < dataset.data.length; i++) {
         if (typeof dataset.data[i] === "number") {
             filteredValues.push(dataset.data[i]);
@@ -12005,6 +12001,25 @@ const LN = {
     isExported: true,
 };
 // -----------------------------------------------------------------------------
+// LOG
+// -----------------------------------------------------------------------------
+const LOG = {
+    description: _t("The logarithm of a number, for a given base."),
+    args: [
+        arg("value (number)", _t("The value for which to calculate the logarithm.")),
+        arg("base (number, default=10)", _t("The base of the logarithm.")),
+    ],
+    compute: function (value, base = { value: 10 }) {
+        const _value = toNumber(value, this.locale);
+        const _base = toNumber(base, this.locale);
+        assert(() => _value > 0, _t("The value (%s) must be strictly positive.", _value.toString()));
+        assert(() => _base > 0, _t("The base (%s) must be strictly positive.", _base.toString()));
+        assert(() => _base !== 1, _t("The base must be different from 1."));
+        return Math.log10(_value) / Math.log10(_base);
+    },
+    isExported: true,
+};
+// -----------------------------------------------------------------------------
 // MOD
 // -----------------------------------------------------------------------------
 function mod(dividend, divisor) {
@@ -12544,6 +12559,7 @@ var math = /*#__PURE__*/Object.freeze({
   ISODD: ISODD,
   ISO_CEILING: ISO_CEILING,
   LN: LN,
+  LOG: LOG,
   MOD: MOD,
   MUNIT: MUNIT,
   ODD: ODD,
@@ -28610,10 +28626,8 @@ function getChartDatasetFormat(getters, dataSets) {
 function getChartDatasetValues(getters, dataSets) {
     const datasetValues = [];
     for (const [dsIndex, ds] of Object.entries(dataSets)) {
-        if (getters.isColHidden(ds.dataRange.sheetId, ds.dataRange.zone.left)) {
-            continue;
-        }
         let label;
+        let hidden = getters.isColHidden(ds.dataRange.sheetId, ds.dataRange.zone.left);
         if (ds.labelCell) {
             const labelRange = ds.labelCell;
             const cell = labelRange
@@ -28640,9 +28654,9 @@ function getChartDatasetValues(getters, dataSets) {
             data.fill(1);
         }
         else if (data.every((cell) => cell === undefined || cell === null || !isNumber(cell.toString(), DEFAULT_LOCALE))) {
-            continue;
+            hidden = true;
         }
-        datasetValues.push({ data, label });
+        datasetValues.push({ data, label, hidden });
     }
     return datasetValues;
 }
@@ -28709,6 +28723,20 @@ const backgroundColorChartJSPlugin = {
         ctx.restore();
     },
 };
+function getChartJsLegend(fontColor, legend = {}) {
+    return {
+        ...legend,
+        labels: {
+            color: fontColor,
+            filter: (legendItem, data) => {
+                return "datasetIndex" in legendItem
+                    ? !data.datasets[legendItem.datasetIndex].hidden
+                    : true;
+            },
+            ...legend.labels,
+        },
+    };
+}
 
 class BarChart extends AbstractChart {
     dataSets;
@@ -28858,9 +28886,7 @@ function createBarChartRuntime(chart, getters) {
         ...localeFormat,
         horizontalChart: chart.horizontal,
     });
-    const legend = {
-        labels: { color: fontColor },
-    };
+    const legend = getChartJsLegend(fontColor);
     if (chart.legendPosition === "none") {
         legend.display = false;
     }
@@ -28924,11 +28950,12 @@ function createBarChartRuntime(chart, getters) {
     const colors = getChartColorsGenerator(definition, dataSetsValues.length);
     const trendDatasets = [];
     for (const index in dataSetsValues) {
-        const { label, data } = dataSetsValues[index];
+        const { label, data, hidden } = dataSetsValues[index];
         const color = colors.next();
         const dataset = {
             label,
             data,
+            hidden,
             borderColor: definition.background || BACKGROUND_CHART_COLOR,
             borderWidth: definition.stacked ? 1 : 0,
             backgroundColor: color,
@@ -29172,6 +29199,9 @@ function getTrendDatasetForLineChart(config, dataset, axisType, locale) {
     const filteredLabels = [];
     const labels = [];
     const datasetLength = dataset.data.length;
+    if (dataset.hidden) {
+        return;
+    }
     if (datasetLength < 2) {
         return;
     }
@@ -29245,9 +29275,8 @@ function createLineOrScatterChartRuntime(chart, getters) {
     const options = { format: dataSetFormat, locale, truncateLabels };
     const fontColor = chartFontColor(chart.background);
     const config = getDefaultChartJsRuntime(chart, labels, fontColor, options);
-    const legend = {
+    const legend = getChartJsLegend(fontColor, {
         labels: {
-            color: fontColor,
             generateLabels(chart) {
                 // color the legend labels with the dataset color, without any transparency
                 const { data } = chart;
@@ -29258,7 +29287,7 @@ function createLineOrScatterChartRuntime(chart, getters) {
                 return labels;
             },
         },
-    };
+    });
     if (chart.legendPosition === "none") {
         legend.display = false;
     }
@@ -29358,7 +29387,7 @@ function createLineOrScatterChartRuntime(chart, getters) {
     const cumulative = "cumulative" in chart ? chart.cumulative : false;
     const definition = chart.getDefinition();
     const colors = getChartColorsGenerator(definition, dataSetsValues.length);
-    for (let [index, { label, data }] of dataSetsValues.entries()) {
+    for (let [index, { label, data, hidden }] of dataSetsValues.entries()) {
         const color = colors.next();
         let backgroundRGBA = colorToRGBA(color);
         if (areaChart) {
@@ -29382,6 +29411,7 @@ function createLineOrScatterChartRuntime(chart, getters) {
         const dataset = {
             label,
             data,
+            hidden,
             tension: 0, // 0 -> render straight lines, which is much faster
             borderColor: color,
             backgroundColor,
@@ -29587,9 +29617,7 @@ function createComboChartRuntime(chart, getters) {
     const localeFormat = { format: mainDataSetFormat, locale };
     const fontColor = chartFontColor(chart.background);
     const config = getDefaultChartJsRuntime(chart, labels, fontColor, localeFormat);
-    const legend = {
-        labels: { color: fontColor },
-    };
+    const legend = getChartJsLegend(fontColor);
     if (chart.legendPosition === "none") {
         legend.display = false;
     }
@@ -29653,13 +29681,14 @@ function createComboChartRuntime(chart, getters) {
     const colors = getChartColorsGenerator(definition, dataSetsValues.length);
     let maxLength = 0;
     const trendDatasets = [];
-    for (let [index, { label, data }] of dataSetsValues.entries()) {
+    for (let [index, { label, data, hidden }] of dataSetsValues.entries()) {
         const design = definition.dataSets[index];
         const color = colors.next();
         const type = design?.type ?? "line";
         const dataset = {
             label: design?.label ?? label,
             data,
+            hidden,
             borderColor: color,
             backgroundColor: color,
             yAxisID: design?.yAxisId ?? "y",
@@ -30267,7 +30296,7 @@ function filterNegativeValues(labels, datasets) {
 function createPieChartRuntime(chart, getters) {
     const labelValues = getChartLabelValues(getters, chart.dataSets, chart.labelRange);
     let labels = labelValues.formattedValues;
-    let dataSetsValues = getChartDatasetValues(getters, chart.dataSets);
+    let dataSetsValues = getChartDatasetValues(getters, chart.dataSets).filter((dataSet) => !dataSet.hidden);
     if (shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle)) {
         labels.shift();
     }
@@ -30285,7 +30314,7 @@ function createPieChartRuntime(chart, getters) {
         const dataset = {
             label,
             data,
-            borderColor: BACKGROUND_CHART_COLOR,
+            borderColor: chart.background || "#FFFFFF",
             backgroundColor,
             hoverOffset: 30,
         };
@@ -30414,8 +30443,9 @@ function createPyramidChartRuntime(chart, getters) {
     const barDef = { ...chart.getDefinition(), type: "bar" };
     const barChart = new BarChart(barDef, chart.sheetId, getters);
     const barRuntime = createBarChartRuntime(barChart, getters);
+    // align design with filtered datasets
     const config = barRuntime.chartJsConfig;
-    let datasets = config.data?.datasets;
+    let datasets = config.data?.datasets.filter((dataSet) => !dataSet.hidden);
     if (datasets && datasets[0]) {
         datasets[0].data = datasets[0].data.map((value) => (value > 0 ? value : 0));
     }
@@ -30812,7 +30842,7 @@ function getWaterfallConfiguration(chart, labels, dataSeriesLabels, localeFormat
 function createWaterfallChartRuntime(chart, getters) {
     const labelValues = getChartLabelValues(getters, chart.dataSets, chart.labelRange);
     let labels = labelValues.formattedValues;
-    let dataSetsValues = getChartDatasetValues(getters, chart.dataSets);
+    let dataSetsValues = getChartDatasetValues(getters, chart.dataSets).filter((ds) => !ds.hidden);
     if (shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle)) {
         labels.shift();
     }
@@ -51168,6 +51198,10 @@ class CellPlugin extends CorePlugin {
                 return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessUpdateCell);
             case "CLEAR_CELL":
                 return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessClearCell);
+            case "UPDATE_CELL_POSITION":
+                return !cmd.cellId || this.cells[cmd.sheetId]?.[cmd.cellId]
+                    ? "Success" /* CommandResult.Success */
+                    : "InvalidCellId" /* CommandResult.InvalidCellId */;
             default:
                 return "Success" /* CommandResult.Success */;
         }
@@ -51212,6 +51246,9 @@ class CellPlugin extends CorePlugin {
             case "DELETE_CONTENT":
                 this.clearZones(cmd.sheetId, cmd.target);
                 break;
+            case "DELETE_SHEET": {
+                this.history.update("cells", cmd.sheetId, undefined);
+            }
         }
     }
     clearZones(sheetId, zones) {
@@ -52433,8 +52470,14 @@ class DataValidationPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "ADD_DATA_VALIDATION_RULE":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 return this.checkValidations(cmd, this.chainValidations(this.checkEmptyRange, this.checkCriterionTypeIsValid, this.checkCriterionHasValidNumberOfValues, this.checkCriterionValuesAreValid));
             case "REMOVE_DATA_VALIDATION_RULE":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.rules[cmd.sheetId].find((rule) => rule.id === cmd.id)) {
                     return "UnknownDataValidationRule" /* CommandResult.UnknownDataValidationRule */;
                 }
@@ -52654,6 +52697,7 @@ class DataValidationPlugin extends CorePlugin {
 class FigurePlugin extends CorePlugin {
     static getters = ["getFigures", "getFigure", "getFigureSheetId"];
     figures = {};
+    insertionOrders = []; // TODO use a list in master
     // ---------------------------------------------------------------------------
     // Command Handling
     // ---------------------------------------------------------------------------
@@ -52756,11 +52800,14 @@ class FigurePlugin extends CorePlugin {
     }
     addFigure(figure, sheetId) {
         this.history.update("figures", sheetId, figure.id, figure);
+        this.history.update("insertionOrders", this.insertionOrders.length, figure.id);
     }
     deleteSheet(sheetId) {
+        this.history.update("insertionOrders", this.insertionOrders.filter((id) => !this.figures[sheetId]?.[id]));
         this.history.update("figures", sheetId, undefined);
     }
     removeFigure(id, sheetId) {
+        this.history.update("insertionOrders", this.insertionOrders.filter((figureId) => figureId !== id));
         this.history.update("figures", sheetId, id, undefined);
     }
     checkFigureExists(sheetId, figureId) {
@@ -52779,7 +52826,14 @@ class FigurePlugin extends CorePlugin {
     // Getters
     // ---------------------------------------------------------------------------
     getFigures(sheetId) {
-        return Object.values(this.figures[sheetId] || {}).filter(isDefined);
+        const figures = [];
+        for (const figureId of this.insertionOrders) {
+            const figure = this.figures[sheetId]?.[figureId];
+            if (figure) {
+                figures.push(figure);
+            }
+        }
+        return figures;
     }
     getFigure(sheetId, figureId) {
         return this.figures[sheetId]?.[figureId];
@@ -52792,11 +52846,9 @@ class FigurePlugin extends CorePlugin {
     // ---------------------------------------------------------------------------
     import(data) {
         for (let sheet of data.sheets) {
-            const figures = {};
-            sheet.figures.forEach((figure) => {
-                figures[figure.id] = figure;
-            });
-            this.figures[sheet.id] = figures;
+            for (const figure of sheet.figures) {
+                this.addFigure(figure, sheet.id);
+            }
         }
     }
     export(data) {
@@ -54222,6 +54274,9 @@ class SheetPlugin extends CorePlugin {
             case "CREATE_SHEET": {
                 return this.checkValidations(cmd, this.checkSheetName, this.checkSheetPosition);
             }
+            case "DUPLICATE_SHEET": {
+                return this.sheets[cmd.sheetIdTo] ? "DuplicatedSheetId" /* CommandResult.DuplicatedSheetId */ : "Success" /* CommandResult.Success */;
+            }
             case "MOVE_SHEET":
                 try {
                     const currentIndex = this.orderedSheetIds.findIndex((id) => id === cmd.sheetId);
@@ -55033,6 +55088,10 @@ class SheetPlugin extends CorePlugin {
     checkZonesAreInSheet(cmd) {
         if (!("sheetId" in cmd))
             return "Success" /* CommandResult.Success */;
+        if ("ranges" in cmd &&
+            cmd.ranges.some((rangeData) => !this.getters.tryGetSheet(rangeData._sheetId))) {
+            return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+        }
         return this.checkZonesExistInSheet(cmd.sheetId, this.getCommandZones(cmd));
     }
 }
@@ -55040,6 +55099,7 @@ class SheetPlugin extends CorePlugin {
 class TablePlugin extends CorePlugin {
     static getters = ["getCoreTable", "getCoreTables", "getCoreTableMatchingTopLeft"];
     tables = {};
+    insertionOrders = {};
     adaptRanges(applyChange, sheetId) {
         const sheetIds = sheetId ? [sheetId] : this.getters.getSheetIds();
         for (const sheetId of sheetIds) {
@@ -55051,6 +55111,9 @@ class TablePlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "CREATE_TABLE":
+                if (cmd.ranges.some((rangeData) => !this.getters.tryGetSheet(rangeData._sheetId) || rangeData._sheetId !== cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 const zones = cmd.ranges.map((rangeData) => this.getters.getRangeFromRangeData(rangeData).zone);
                 if (!areZonesContinuous(zones)) {
                     return "NonContinuousTargets" /* CommandResult.NonContinuousTargets */;
@@ -55081,11 +55144,13 @@ class TablePlugin extends CorePlugin {
         switch (cmd.type) {
             case "CREATE_SHEET":
                 this.history.update("tables", cmd.sheetId, {});
+                this.history.update("insertionOrders", cmd.sheetId, []);
                 break;
             case "DELETE_SHEET": {
                 const tables = { ...this.tables };
                 delete tables[cmd.sheetId];
                 this.history.update("tables", tables);
+                this.history.update("insertionOrders", cmd.sheetId, undefined);
                 break;
             }
             case "DUPLICATE_SHEET": {
@@ -55097,6 +55162,9 @@ class TablePlugin extends CorePlugin {
                             : this.copyStaticTableForSheet(cmd.sheetIdTo, table);
                 }
                 this.history.update("tables", cmd.sheetIdTo, newTables);
+                this.history.update("insertionOrders", cmd.sheetIdTo, [
+                    ...(this.insertionOrders[cmd.sheetId] ?? []),
+                ]);
                 break;
             }
             case "CREATE_TABLE": {
@@ -55110,6 +55178,10 @@ class TablePlugin extends CorePlugin {
                     ? this.createDynamicTable(id, union, config)
                     : this.createStaticTable(id, cmd.tableType, union, config);
                 this.history.update("tables", cmd.sheetId, newTable.id, newTable);
+                this.history.update("insertionOrders", cmd.sheetId, [
+                    ...(this.insertionOrders[cmd.sheetId] ?? []),
+                    newTable.id,
+                ]);
                 break;
             }
             case "REMOVE_TABLE": {
@@ -55120,6 +55192,7 @@ class TablePlugin extends CorePlugin {
                     }
                 }
                 this.history.update("tables", cmd.sheetId, tables);
+                this.history.update("insertionOrders", cmd.sheetId, this.insertionOrders[cmd.sheetId]?.filter((id) => id in tables));
                 break;
             }
             case "UPDATE_TABLE": {
@@ -55155,7 +55228,14 @@ class TablePlugin extends CorePlugin {
         }
     }
     getCoreTables(sheetId) {
-        return this.tables[sheetId] ? Object.values(this.tables[sheetId]).filter(isDefined) : [];
+        const tables = [];
+        for (const tableId of this.insertionOrders[sheetId] || []) {
+            const table = this.tables[sheetId][tableId];
+            if (table) {
+                tables.push(table);
+            }
+        }
+        return tables;
     }
     getCoreTable({ sheetId, col, row }) {
         return this.getCoreTables(sheetId).find((table) => isInside(col, row, table.range.zone));
@@ -55438,6 +55518,7 @@ class TablePlugin extends CorePlugin {
     // ---------------------------------------------------------------------------
     import(data) {
         for (const sheet of data.sheets) {
+            const tableIds = [];
             for (const tableData of sheet.tables || []) {
                 const uuid = this.uuidGenerator.uuidv4();
                 const tableConfig = tableData.config || DEFAULT_TABLE_CONFIG;
@@ -55447,7 +55528,9 @@ class TablePlugin extends CorePlugin {
                     ? this.createDynamicTable(uuid, range, tableConfig)
                     : this.createStaticTable(uuid, tableType, range, tableConfig);
                 this.history.update("tables", sheet.id, table.id, table);
+                tableIds.push(table.id);
             }
+            this.history.update("insertionOrders", sheet.id, tableIds);
         }
     }
     export(data) {
@@ -55487,7 +55570,10 @@ class HeaderGroupingPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "GROUP_HEADERS": {
-                const { start, end } = cmd;
+                const { start, end, sheetId } = cmd;
+                if (!this.getters.tryGetSheet(sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.getters.doesHeadersExist(cmd.sheetId, cmd.dimension, [start, end])) {
                     return "InvalidHeaderGroupStartEnd" /* CommandResult.InvalidHeaderGroupStartEnd */;
                 }
@@ -55500,7 +55586,10 @@ class HeaderGroupingPlugin extends CorePlugin {
                 break;
             }
             case "UNGROUP_HEADERS": {
-                const { start, end } = cmd;
+                const { start, end, sheetId } = cmd;
+                if (!this.getters.tryGetSheet(sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.getters.doesHeadersExist(cmd.sheetId, cmd.dimension, [start, end])) {
                     return "InvalidHeaderGroupStartEnd" /* CommandResult.InvalidHeaderGroupStartEnd */;
                 }
@@ -55511,6 +55600,9 @@ class HeaderGroupingPlugin extends CorePlugin {
             }
             case "UNFOLD_HEADER_GROUP":
             case "FOLD_HEADER_GROUP":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 const group = this.findGroupWithStartEnd(cmd.sheetId, cmd.dimension, cmd.start, cmd.end);
                 if (!group) {
                     return "UnknownHeaderGroup" /* CommandResult.UnknownHeaderGroup */;
@@ -55911,6 +56003,9 @@ class PivotCorePlugin extends CorePlugin {
                 return this.checkDuplicatedMeasureIds(cmd.pivot);
             }
             case "UPDATE_PIVOT": {
+                if (!(cmd.pivotId in this.pivots)) {
+                    return "PivotIdNotFound" /* CommandResult.PivotIdNotFound */;
+                }
                 if (deepEquals(cmd.pivot, this.pivots[cmd.pivotId]?.definition)) {
                     return "NoChanges" /* CommandResult.NoChanges */;
                 }
@@ -55927,6 +56022,8 @@ class PivotCorePlugin extends CorePlugin {
                     return "EmptyName" /* CommandResult.EmptyName */;
                 }
                 break;
+            case "REMOVE_PIVOT":
+            case "DUPLICATE_PIVOT":
             case "INSERT_PIVOT": {
                 if (!(cmd.pivotId in this.pivots)) {
                     return "PivotIdNotFound" /* CommandResult.PivotIdNotFound */;
@@ -55976,7 +56073,7 @@ class PivotCorePlugin extends CorePlugin {
                 break;
             }
             case "UPDATE_PIVOT": {
-                this.history.update("pivots", cmd.pivotId, "definition", cmd.pivot);
+                this.history.update("pivots", cmd.pivotId, "definition", deepCopy(cmd.pivot));
                 this.compileCalculatedMeasures(cmd.pivot.measures);
                 break;
             }
@@ -56047,7 +56144,7 @@ class PivotCorePlugin extends CorePlugin {
     // Private
     // -------------------------------------------------------------------------
     addPivot(pivotId, pivot, formulaId = this.nextFormulaId.toString()) {
-        this.history.update("pivots", pivotId, { definition: pivot, formulaId });
+        this.history.update("pivots", pivotId, { definition: deepCopy(pivot), formulaId });
         this.compileCalculatedMeasures(pivot.measures);
         this.history.update("formulaIds", formulaId, pivotId);
         this.history.update("nextFormulaId", this.nextFormulaId + 1);
@@ -60979,6 +61076,9 @@ function updateChartRangesTransformation(toTransform, executed) {
     };
 }
 function createSheetTransformation(toTransform, executed) {
+    if (toTransform.sheetId === executed.sheetId) {
+        toTransform = { ...toTransform, sheetId: `${toTransform.sheetId}~` };
+    }
     if (toTransform.name === executed.name) {
         return {
             ...toTransform,
@@ -61623,15 +61723,6 @@ class Session extends EventBus {
         this.waitingAck = true;
         this.sendPendingMessage();
     }
-    dropPendingRevision(revisionId) {
-        this.revisions.drop(revisionId);
-        const revisionIds = this.pendingMessages
-            .filter((message) => message.type === "REMOTE_REVISION")
-            .map((message) => message.nextRevisionId);
-        this.trigger("pending-revisions-dropped", { revisionIds });
-        this.waitingAck = false;
-        this.waitingUndoRedoAck = false;
-    }
     /**
      * Send the next pending message
      */
@@ -61640,15 +61731,14 @@ class Session extends EventBus {
         if (!message)
             return;
         if (message.type === "REMOTE_REVISION") {
-            const revision = this.revisions.get(message.nextRevisionId);
+            let revision = this.revisions.get(message.nextRevisionId);
             if (revision.commands.length === 0) {
                 /**
-                 * The command is empty, we have to drop all the next local revisions
+                 * The command is empty, we have to rebase all the next local revisions
                  * to avoid issues with undo/redo
                  */
-                this.dropPendingRevision(revision.id);
-                this.pendingMessages = [];
-                return;
+                this.revisions.rebase(revision.id);
+                revision = this.revisions.get(message.nextRevisionId);
             }
             message = {
                 ...message,
@@ -61683,18 +61773,16 @@ class Session extends EventBus {
             case "REVISION_UNDONE": {
                 this.waitingAck = false;
                 this.pendingMessages = this.pendingMessages.filter((msg) => msg.nextRevisionId !== message.nextRevisionId);
-                const pendingRemoteRevisions = this.pendingMessages.filter((message) => message.type === "REMOTE_REVISION");
-                const firstTransformedRevisionIndex = pendingRemoteRevisions.findIndex((message) => !deepEquals(message.commands, this.revisions.get(message.nextRevisionId).commands));
-                if (firstTransformedRevisionIndex !== -1) {
+                const firstPendingRevisionId = this.pendingMessages.findIndex((message) => message.type === "REMOTE_REVISION");
+                if (firstPendingRevisionId !== -1) {
                     /**
                      * Some revisions undergo transformations that may cause issues with
                      * undo/redo if the transformation is destructive (we don't get back
                      * the original command by transforming it with the inverse).
-                     * To prevent these problems, we must discard all subsequent local
+                     * To prevent these problems, we must rebase all subsequent local
                      * revisions.
                      */
-                    this.dropPendingRevision(this.pendingMessages[firstTransformedRevisionIndex].nextRevisionId);
-                    this.pendingMessages = this.pendingMessages.slice(0, firstTransformedRevisionIndex);
+                    this.revisions.rebase(this.pendingMessages[firstPendingRevisionId].nextRevisionId);
                 }
                 this.serverRevisionId = message.nextRevisionId;
                 this.processedRevisions.add(message.nextRevisionId);
@@ -62783,6 +62871,10 @@ class SheetUIPlugin extends UIPlugin {
      */
     checkZonesAreInSheet(cmd) {
         const sheetId = "sheetId" in cmd ? cmd.sheetId : this.getters.tryGetActiveSheetId();
+        if ("ranges" in cmd &&
+            cmd.ranges.some((rangeData) => !this.getters.tryGetSheet(rangeData._sheetId))) {
+            return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+        }
         const zones = this.getters.getCommandZones(cmd);
         if (!sheetId && zones.length > 0) {
             return "NoActiveSheet" /* CommandResult.NoActiveSheet */;
@@ -63337,7 +63429,6 @@ class HistoryPlugin extends UIPlugin {
         super(config);
         this.session = config.session;
         this.session.on("new-local-state-update", this, this.onNewLocalStateUpdate);
-        this.session.on("pending-revisions-dropped", this, ({ revisionIds }) => this.drop(revisionIds));
         this.session.on("snapshot", this, () => {
             this.undoStack = [];
             this.redoStack = [];
@@ -63407,10 +63498,6 @@ class HistoryPlugin extends UIPlugin {
         const lastNonRedoRevision = this.getPossibleRevisionToRepeat();
         return canRepeatRevision(lastNonRedoRevision);
     }
-    drop(revisionIds) {
-        this.undoStack = this.undoStack.filter((id) => !revisionIds.includes(id));
-        this.redoStack = [];
-    }
     onNewLocalStateUpdate({ id }) {
         this.undoStack.push(id);
         this.redoStack = [];
@@ -66110,7 +66197,9 @@ class HeaderPositionsUIPlugin extends UIPlugin {
             case "UNGROUP_HEADERS":
             case "GROUP_HEADERS":
             case "CREATE_SHEET":
-                this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
+                if (this.getters.tryGetSheet(cmd.sheetId)) {
+                    this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
+                }
                 break;
             case "DUPLICATE_SHEET":
                 this.headerPositions[cmd.sheetIdTo] = deepCopy(this.headerPositions[cmd.sheetId]);
@@ -66118,12 +66207,14 @@ class HeaderPositionsUIPlugin extends UIPlugin {
         }
     }
     finalize() {
-        if (this.isDirty) {
-            for (const sheetId of this.getters.getSheetIds()) {
+        for (const sheetId of this.getters.getSheetIds()) {
+            // sheets can be created without this plugin being aware of it
+            // in concurrent situations.
+            if (this.isDirty || !this.headerPositions[sheetId]) {
                 this.headerPositions[sheetId] = this.computeHeaderPositionsOfSheet(sheetId);
             }
-            this.isDirty = false;
         }
+        this.isDirty = false;
     }
     /**
      * Returns the size, start and end coordinates of a column on an unfolded sheet
@@ -69626,9 +69717,16 @@ class SelectiveHistory {
         this.fastForward();
         this.insert(redoId, this.buildEmpty(redoId), insertAfter);
     }
-    drop(operationId) {
+    rebase(operationId) {
+        const operation = this.get(operationId);
+        const execution = [...this.tree.execution(this.HEAD_BRANCH).startAfter(operationId)];
         this.revertBefore(operationId);
+        const baseId = this.HEAD_OPERATION.id;
         this.tree.drop(operationId);
+        this.insert(operationId, operation, baseId);
+        for (const { operation } of execution) {
+            this.insert(operation.id, operation.data, this.HEAD_OPERATION.id);
+        }
     }
     /**
      * Revert the state as it was *before* the given operation was executed.
@@ -72670,6 +72768,11 @@ class Model extends EventBus {
             dispatch: (command) => {
                 const result = this.checkDispatchAllowed(command);
                 if (!result.isSuccessful) {
+                    // core views plugins need to be invalidated
+                    this.dispatchToHandlers(this.coreHandlers, {
+                        type: "UNDO",
+                        commands: [command],
+                    });
                     return;
                 }
                 this.isReplayingCommand = true;
@@ -73228,6 +73331,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.0.16";
-__info__.date = "2025-02-14T08:44:19.475Z";
-__info__.hash = "39979ab";
+__info__.version = "18.0.17";
+__info__.date = "2025-02-25T05:58:39.632Z";
+__info__.hash = "2ee4347";