@rowsncolumns/spreadsheet-state 29.0.0 → 29.1.1

package/dist/cjs/interface/create-table.js

@@ -1,27 +1,101 @@
 import { __assign, __read } from "tslib";
-import { areaIntersects, isAreasEqual, isCellWithinBounds, selectionFromActiveCell, } from "@rowsncolumns/utils";
+import { areaIntersects, isAreasEqual, isCellWithinBounds, selectionFromActiveCell, validateTableName, } from "@rowsncolumns/utils";
 import { uuid } from "@rowsncolumns/utils";
 import { produceWithPatches } from "immer";
-import { getGeneratedTableName } from "../generated-names";
-import { getNextTableColumnName } from "../helpers";
-import { addTableColumn, tableWouldOverlap } from "./table-utils";
+import { getTableRangeRefusal, shiftBand } from "../api/structural-guards";
+import { getNextTableTitle } from "../helpers";
+import { insertCellsShiftDown } from "./insert-cells-shift-down";
+import { addTableColumn, getTableHeaderNames } from "./table-utils";
+/**
+ * The data range a create-table targets: the spec's range, else the last selection, else the data
+ * region around the active cell (Ctrl+A's region), else the active cell. The hook's refusal guards
+ * and the create itself must agree on it, so both resolve it here.
+ */
+export var resolveCreateTableRange = function (_a) {
+    var _b;
+    var sheetId = _a.sheetId, activeCell = _a.activeCell, selections = _a.selections, tableSpec = _a.tableSpec, getContentfulRangeAroundCell = _a.getContentfulRangeAroundCell;
+    var lastSelection = selections === null || selections === void 0 ? void 0 : selections[selections.length - 1];
+    return (tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.range)
+        ? tableSpec.range
+        : lastSelection
+            ? lastSelection.range
+            : (_b = getContentfulRangeAroundCell === null || getContentfulRangeAroundCell === void 0 ? void 0 : getContentfulRangeAroundCell(sheetId, activeCell)) !== null && _b !== void 0 ? _b : selectionFromActiveCell(activeCell)[0].range;
+};
 export var createTable = function (_a) {
-    var locale = _a.locale, rowCount = _a.rowCount, columnCount = _a.columnCount, tables = _a.tables, basicFilter = _a.basicFilter, spreadsheetTheme = _a.theme, idCreationStrategy = _a.idCreationStrategy, enqueueCalculation = _a.enqueueCalculation, createHistory = _a.createHistory, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeSheetData = _a.onChangeSheetData, getFormattedValue = _a.getFormattedValue, getEffectiveValue = _a.getEffectiveValue, onRequestCalculate = _a.onRequestCalculate, getDependents = _a.getDependents, enqueueGraphOperation = _a.enqueueGraphOperation, getContentfulRangeAroundCell = _a.getContentfulRangeAroundCell, generatedNames = _a.generatedNames;
-    return function (sheetId, activeCell, selections, tableSpec, theme, bandedRange) {
-        var _a;
+    var locale = _a.locale, rowCount = _a.rowCount, columnCount = _a.columnCount, tables = _a.tables, namedRanges = _a.namedRanges, basicFilter = _a.basicFilter, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, spreadsheetTheme = _a.theme, idCreationStrategy = _a.idCreationStrategy, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, createHistory = _a.createHistory, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeSheetData = _a.onChangeSheetData, onChangeCitations = _a.onChangeCitations, onChangeCharts = _a.onChangeCharts, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getFormattedValue = _a.getFormattedValue, getEffectiveValue = _a.getEffectiveValue, onRequestCalculate = _a.onRequestCalculate, getDependents = _a.getDependents, getSheetName = _a.getSheetName, enqueueGraphOperation = _a.enqueueGraphOperation, getContentfulRangeAroundCell = _a.getContentfulRangeAroundCell, generatedNames = _a.generatedNames;
+    return function (sheetId, activeCell, selections, tableSpec, theme, bandedRange, options) {
         // Validate
         var lastSelection = selections === null || selections === void 0 ? void 0 : selections[selections.length - 1];
-        var saveHistory = createHistory();
-        var range = (tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.range)
-            ? tableSpec.range
-            : lastSelection
-                ? lastSelection.range
-                : (_a = getContentfulRangeAroundCell === null || getContentfulRangeAroundCell === void 0 ? void 0 : getContentfulRangeAroundCell(sheetId, activeCell)) !== null && _a !== void 0 ? _a : selectionFromActiveCell(activeCell)[0].range;
+        var dataRange = resolveCreateTableRange({
+            sheetId: sheetId,
+            activeCell: activeCell,
+            selections: selections,
+            tableSpec: tableSpec,
+            getContentfulRangeAroundCell: getContentfulRangeAroundCell,
+        });
+        // Excel's "My table has headers" unchecked: a `Column1..N` header row is inserted ABOVE the
+        // data — the range's columns shift down one row (Insert Cells → Shift cells down) and the
+        // table covers header + data. The caller has already run the shift's refusal guard.
+        var hasHeaders = (options === null || options === void 0 ? void 0 : options.hasHeaders) !== false;
+        var range = hasHeaders
+            ? dataRange
+            : __assign(__assign({}, dataRange), { endRowIndex: dataRange.endRowIndex + 1 });
         // Refuse to create a table whose rectangle overlaps an existing one on the
-        // same sheet — Excel rejects such workbooks on open.
-        if (tableWouldOverlap({ sheetId: sheetId, range: range }, tables)) {
+        // same sheet — Excel rejects such workbooks on open. With an inserted header
+        // row, tables wholly inside the shifted columns sit one row lower by then.
+        if (getTableRangeRefusal({
+            sheetId: sheetId,
+            range: range,
+            tables: tables,
+            insertedRows: hasHeaders
+                ? undefined
+                : {
+                    band: shiftBand("down", __assign(__assign({}, dataRange), { endRowIndex: dataRange.startRowIndex })),
+                    count: 1,
+                },
+        })) {
+            return;
+        }
+        // A caller-supplied id must be free: pushing a second table under an existing id would leave
+        // two tables answering to it (the hook shows the reason; a generated id is fresh by construction).
+        if ((tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.id) !== undefined &&
+            (tables === null || tables === void 0 ? void 0 : tables.some(function (table) { return table.id === tableSpec.id; }))) {
             return;
         }
+        // A caller-supplied title must be a valid Excel table name no table or defined name holds
+        // (the hook shows the reason; the generated default is valid by construction).
+        if ((tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.title) !== undefined &&
+            validateTableName(tableSpec.title, tables === null || tables === void 0 ? void 0 : tables.map(function (table) { return table.title; }), namedRanges === null || namedRanges === void 0 ? void 0 : namedRanges.map(function (namedRange) { return namedRange.name; }))) {
+            return;
+        }
+        if (!hasHeaders) {
+            insertCellsShiftDown({
+                rowCount: rowCount,
+                columnCount: columnCount,
+                locale: locale,
+                cellXfsRegistry: cellXfsRegistry,
+                sharedStringRegistry: sharedStringRegistry,
+                getActiveCell: getActiveCell !== null && getActiveCell !== void 0 ? getActiveCell : (function () { return activeCell; }),
+                getSelections: getSelections !== null && getSelections !== void 0 ? getSelections : (function () { return selections; }),
+                getDependents: getDependents,
+                getSheetName: getSheetName,
+                createHistory: createHistory,
+                enqueueGraphOperation: enqueueGraphOperation,
+                enqueueCalculation: enqueueCalculation,
+                enqueueStructureChange: enqueueStructureChange,
+                onChangeSheetData: onChangeSheetData,
+                onChangeSheets: onChangeSheets,
+                onChangeCitations: onChangeCitations,
+                onChangeTables: onChangeTables,
+                onChangeCharts: onChangeCharts,
+            })(sheetId, activeCell, [
+                { range: __assign(__assign({}, dataRange), { endRowIndex: dataRange.startRowIndex }) },
+            ]);
+        }
+        var saveHistory = createHistory();
+        // The header names: the header cells' text (falling back to `ColumnN`), or `Column1..N` for
+        // an inserted header row — its cells are blank until `addTableColumn` writes them.
+        var headerCellText = hasHeaders ? getFormattedValue : function () { return undefined; };
         // Crate a table
         onChangeTables === null || onChangeTables === void 0 ? void 0 : onChangeTables(function (prevTables) {
             var _a = __read(produceWithPatches(prevTables, function (draft) {
@@ -32,15 +106,16 @@ export var createTable = function (_a) {
                         table.sheetId === sheetId;
                 });
                 if (tableIndex === -1) {
-                    var columns = [];
-                    for (var columnIndex = range.startColumnIndex; columnIndex <= range.endColumnIndex; columnIndex++) {
-                        var formattedValue = getFormattedValue(sheetId, range.startRowIndex, columnIndex);
-                        var columnName = getNextTableColumnName(columns, generatedNames);
-                        columns.push({
-                            name: formattedValue || columnName,
-                        });
-                    }
-                    draft.push(__assign({ id: (_b = (_a = tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.id) !== null && _a !== void 0 ? _a : idCreationStrategy === null || idCreationStrategy === void 0 ? void 0 : idCreationStrategy("table")) !== null && _b !== void 0 ? _b : uuid(), range: range, sheetId: sheetId, title: (_c = tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.title) !== null && _c !== void 0 ? _c : getGeneratedTableName(draft.length + 1, generatedNames), columns: columns, filterSpecs: [], headerRow: true, filterButton: true, sortSpecs: [], showRowStripes: true, bandedRange: bandedRange, theme: theme !== null && theme !== void 0 ? theme : "TableStyleLight1" }, tableSpec));
+                    // Excel: header names are unique (`Name`, `Name2`, …) and blanks take `ColumnN`.
+                    var columns = getTableHeaderNames({
+                        sheetId: sheetId,
+                        range: range,
+                        getFormattedValue: headerCellText,
+                        generatedNames: generatedNames,
+                    }).map(function (name) { return ({ name: name }); });
+                    draft.push(__assign(__assign(__assign({ id: (_b = (_a = tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.id) !== null && _a !== void 0 ? _a : idCreationStrategy === null || idCreationStrategy === void 0 ? void 0 : idCreationStrategy("table")) !== null && _b !== void 0 ? _b : uuid(), sheetId: sheetId, title: (_c = tableSpec === null || tableSpec === void 0 ? void 0 : tableSpec.title) !== null && _c !== void 0 ? _c : getNextTableTitle(draft, generatedNames), columns: columns, filterSpecs: [], headerRow: true, filterButton: true, sortSpecs: [], showRowStripes: true, bandedRange: bandedRange, theme: theme !== null && theme !== void 0 ? theme : "TableStyleLight1" }, tableSpec), { 
+                        // The spec's range is the DATA range; the table also covers an inserted header row.
+                        range: range }), (hasHeaders ? {} : { headerRow: true, columns: columns })));
                 }
             }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
             // Mark table cells as dirty
@@ -61,7 +136,7 @@ export var createTable = function (_a) {
             var _a = __read(produceWithPatches(prevSheetData, function (draft) {
                 addTableColumn({
                     draft: draft,
-                    getFormattedValue: getFormattedValue,
+                    getFormattedValue: headerCellText,
                     sheetId: sheetId,
                     range: range,
                     locale: locale,

package/dist/cjs/interface/delete-cells-shift-left.d.ts

@@ -8,5 +8,5 @@ import type { UseOnDeleteCellsShiftLeftProps } from "../hooks/use-on-delete-cell
  * follows it; a range entirely inside the block's rows that the block partially covers contracts.
  * Refusals (torn merge/table/array) are decided by the hook before this runs.
  */
-export declare const deleteCellsShiftLeft: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getActiveCell, getSelections, getDependents, getSheetName, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, }: UseOnDeleteCellsShiftLeftProps<T, S>) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[]) => void;
+export declare const deleteCellsShiftLeft: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getActiveCell, getSelections, getDependents, getSheetName, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, onChangeCharts, }: UseOnDeleteCellsShiftLeftProps<T, S>) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[]) => void;
 //# sourceMappingURL=delete-cells-shift-left.d.ts.map

package/dist/cjs/interface/delete-cells-shift-left.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-cells-shift-left.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells-shift-left.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,QAAQ,EACR,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAgBnC,OAAO,KAAK,EAAE,8BAA8B,EAAE,MAAM,yCAAyC,CAAC;AAI9F;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,8RAkBvE,8BAA8B,CAAC,CAAC,EAAE,CAAC,CAAC,MAEnC,SAAS,MAAM,EACf,YAAY,aAAa,EACzB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,SAqRnD,CAAC"}
+{"version":3,"file":"delete-cells-shift-left.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells-shift-left.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,QAAQ,EACR,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAiBnC,OAAO,KAAK,EAAE,8BAA8B,EAAE,MAAM,yCAAyC,CAAC;AAI9F;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,8SAmBvE,8BAA8B,CAAC,CAAC,EAAE,CAAC,CAAC,MAEnC,SAAS,MAAM,EACf,YAAY,aAAa,EACzB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,SA+SnD,CAAC"}

package/dist/cjs/interface/delete-cells-shift-left.js

@@ -1,5 +1,6 @@
 import { __assign, __read, __spreadArray, __values } from "tslib";
 import { produceWithPatches } from "immer";
+import { refreshCitationAnchor, shiftChartsOnCellShift } from "../api/anchored-objects";
 import { collectUniqueDependentsForRange, moveFormulaAtPosition, } from "../api/dependency";
 import { contractRangesInFormula } from "../api/formula";
 import { createSheetDraft } from "../api/immer-utils";
@@ -14,7 +15,7 @@ import { SheetCell } from "../sheet-cell";
  * Refusals (torn merge/table/array) are decided by the hook before this runs.
  */
 export var deleteCellsShiftLeft = function (_a) {
-    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getSheetName = _a.getSheetName, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations;
+    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getSheetName = _a.getSheetName, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations, onChangeCharts = _a.onChangeCharts;
     return function (sheetId, activeCell, selections) {
         var _a;
         var saveHistory = createHistory();
@@ -70,6 +71,7 @@ export var deleteCellsShiftLeft = function (_a) {
         });
         // Update citations - delete if overlapping, shift back if after
         onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
+            var updatedAt = new Date().toISOString();
             var _a = __read(produceWithPatches(prevCitations, function (draft) {
                 for (var idx = draft.length - 1; idx >= 0; idx--) {
                     var citation = draft[idx];
@@ -81,11 +83,16 @@ export var deleteCellsShiftLeft = function (_a) {
                         var colsOverlap = !(citation.range.endColumnIndex < c1 ||
                             citation.range.startColumnIndex > c2);
                         if (colsOverlap) {
-                            draft.splice(idx, 1);
+                            // Soft-delete: keep the record for provenance / undo, hide it from the UI
+                            if (citation.active !== false) {
+                                citation.active = false;
+                                citation.updated_at = updatedAt;
+                            }
                         }
                         else if (citation.range.startColumnIndex > c2) {
                             citation.range.startColumnIndex -= count;
                             citation.range.endColumnIndex -= count;
+                            refreshCitationAnchor(citation);
                         }
                     }
                 }
@@ -93,6 +100,14 @@ export var deleteCellsShiftLeft = function (_a) {
             saveHistory({ citations: { patches: patches, inversePatches: inversePatches } });
             return nextState;
         });
+        // Chart ranges wholly inside the shifted rows contract/shift like references; one the block covers entirely is dropped (`#REF!`). The frame stays put.
+        onChangeCharts === null || onChangeCharts === void 0 ? void 0 : onChangeCharts(function (prevCharts) {
+            var _a = __read(produceWithPatches(prevCharts, function (draft) {
+                shiftChartsOnCellShift(draft, sheetId, "column", "delete", c1, count, { startIndex: r1, endIndex: r2 });
+            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+            saveHistory({ charts: { patches: patches, inversePatches: inversePatches } });
+            return nextState;
+        });
         // Update sheet data
         onChangeSheetData === null || onChangeSheetData === void 0 ? void 0 : onChangeSheetData(function (prevSheetData) {
             var _a = __read(produceWithPatches(prevSheetData, function (draft) {

package/dist/cjs/interface/delete-cells-shift-up.d.ts

@@ -9,5 +9,5 @@ import type { UseOnDeleteCellsShiftUpProps } from "../hooks/use-on-delete-cells-
  * partially covers contracts by the overlap. Refusals (torn merge/table/array) are decided by the
  * hook via `getShiftCellsRefusal` before this runs.
  */
-export declare const deleteCellsShiftUp: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getActiveCell, getSelections, getDependents, getSheetName, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, }: UseOnDeleteCellsShiftUpProps<T, S>) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[]) => void;
+export declare const deleteCellsShiftUp: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getActiveCell, getSelections, getDependents, getSheetName, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, onChangeCharts, }: UseOnDeleteCellsShiftUpProps<T, S>) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[]) => void;
 //# sourceMappingURL=delete-cells-shift-up.d.ts.map

package/dist/cjs/interface/delete-cells-shift-up.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-cells-shift-up.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells-shift-up.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,QAAQ,EACR,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAgBnC,OAAO,KAAK,EAAE,4BAA4B,EAAE,MAAM,uCAAuC,CAAC;AAI1F;;;;;;;GAOG;AACH,eAAO,MAAM,kBAAkB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,8RAkBrE,4BAA4B,CAAC,CAAC,EAAE,CAAC,CAAC,MAEjC,SAAS,MAAM,EACf,YAAY,aAAa,EACzB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,SAmQnD,CAAC"}
+{"version":3,"file":"delete-cells-shift-up.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells-shift-up.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,QAAQ,EACR,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAiBnC,OAAO,KAAK,EAAE,4BAA4B,EAAE,MAAM,uCAAuC,CAAC;AAI1F;;;;;;;GAOG;AACH,eAAO,MAAM,kBAAkB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,8SAmBrE,4BAA4B,CAAC,CAAC,EAAE,CAAC,CAAC,MAEjC,SAAS,MAAM,EACf,YAAY,aAAa,EACzB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,SA6RnD,CAAC"}

package/dist/cjs/interface/delete-cells-shift-up.js

@@ -1,5 +1,6 @@
 import { __assign, __read, __spreadArray, __values } from "tslib";
 import { produceWithPatches } from "immer";
+import { refreshCitationAnchor, shiftChartsOnCellShift } from "../api/anchored-objects";
 import { collectUniqueDependentsForRange, moveFormulaAtPosition, } from "../api/dependency";
 import { contractRangesInFormula } from "../api/formula";
 import { createSheetDraft } from "../api/immer-utils";
@@ -15,7 +16,7 @@ import { SheetCell } from "../sheet-cell";
  * hook via `getShiftCellsRefusal` before this runs.
  */
 export var deleteCellsShiftUp = function (_a) {
-    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getSheetName = _a.getSheetName, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations;
+    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getSheetName = _a.getSheetName, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations, onChangeCharts = _a.onChangeCharts;
     return function (sheetId, activeCell, selections) {
         var _a;
         var saveHistory = createHistory();
@@ -71,6 +72,7 @@ export var deleteCellsShiftUp = function (_a) {
         });
         // Update citations - delete if overlapping, shift back if after
         onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
+            var updatedAt = new Date().toISOString();
             var _a = __read(produceWithPatches(prevCitations, function (draft) {
                 for (var idx = draft.length - 1; idx >= 0; idx--) {
                     var citation = draft[idx];
@@ -82,11 +84,16 @@ export var deleteCellsShiftUp = function (_a) {
                         var rowsOverlap = !(citation.range.endRowIndex < r1 ||
                             citation.range.startRowIndex > r2);
                         if (rowsOverlap) {
-                            draft.splice(idx, 1);
+                            // Soft-delete: keep the record for provenance / undo, hide it from the UI
+                            if (citation.active !== false) {
+                                citation.active = false;
+                                citation.updated_at = updatedAt;
+                            }
                         }
                         else if (citation.range.startRowIndex > r2) {
                             citation.range.startRowIndex -= count;
                             citation.range.endRowIndex -= count;
+                            refreshCitationAnchor(citation);
                         }
                     }
                 }
@@ -94,6 +101,14 @@ export var deleteCellsShiftUp = function (_a) {
             saveHistory({ citations: { patches: patches, inversePatches: inversePatches } });
             return nextState;
         });
+        // Chart ranges wholly inside the shifted columns contract/shift like references; one the block covers entirely is dropped (`#REF!`). The frame stays put.
+        onChangeCharts === null || onChangeCharts === void 0 ? void 0 : onChangeCharts(function (prevCharts) {
+            var _a = __read(produceWithPatches(prevCharts, function (draft) {
+                shiftChartsOnCellShift(draft, sheetId, "row", "delete", r1, count, { startIndex: c1, endIndex: c2 });
+            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+            saveHistory({ charts: { patches: patches, inversePatches: inversePatches } });
+            return nextState;
+        });
         // Update sheet data
         onChangeSheetData === null || onChangeSheetData === void 0 ? void 0 : onChangeSheetData(function (prevSheetData) {
             var _a = __read(produceWithPatches(prevSheetData, function (draft) {

package/dist/cjs/interface/delete-cells.d.ts

@@ -2,5 +2,5 @@ import type { CellInterface, SelectionArea } from "@rowsncolumns/common-types";
 import type { CellData, SelectionAttributes, Sheet } from "@rowsncolumns/spreadsheet";
 import type { UseOnDeleteProps } from "../hooks/use-on-delete";
 export type ClearType = "all" | "contents" | "formats";
-export declare const deleteCells: <T extends CellData, S extends Sheet>({ locale, rowCount, cellXfsRegistry, sharedStringRegistry, getDataValidationById, getDependents, getDataRowCount, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, onChangeActiveCell, enqueueCalculation, enqueueStructureChange, createHistory, isArrayFormula, getRowHeight, }: UseOnDeleteProps<T, S>) => (sheetId: number, cell: CellInterface, selections: SelectionArea<SelectionAttributes>[], clearType?: ClearType) => void;
+export declare const deleteCells: <T extends CellData, S extends Sheet>({ locale, rowCount, cellXfsRegistry, sharedStringRegistry, getDataValidationById, getDependents, getDataRowCount, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, onChangeActiveCell, enqueueCalculation, enqueueStructureChange, createHistory, isArrayFormula, getRowHeight, tables, generatedNames, }: UseOnDeleteProps<T, S>) => (sheetId: number, cell: CellInterface, selections: SelectionArea<SelectionAttributes>[], clearType?: ClearType) => void;
 //# sourceMappingURL=delete-cells.d.ts.map

package/dist/cjs/interface/delete-cells.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAI/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,wSAkB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SAwPpC,CAAC"}
+{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAK/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,gUAoB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SAgSpC,CAAC"}

package/dist/cjs/interface/delete-cells.js

@@ -5,8 +5,9 @@ import { produceWithPatches } from "immer";
 import { createSheetCellIdentifier } from "../api/formula";
 import { createSheetDraft } from "../api/immer-utils";
 import { SheetCell } from "../sheet-cell";
+import { getTableHeaderRefills } from "./table-utils";
 export var deleteCells = function (_a) {
-    var locale = _a.locale, rowCount = _a.rowCount, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getDataValidationById = _a.getDataValidationById, getDependents = _a.getDependents, getDataRowCount = _a.getDataRowCount, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations, onChangeActiveCell = _a.onChangeActiveCell, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, createHistory = _a.createHistory, isArrayFormula = _a.isArrayFormula, getRowHeight = _a.getRowHeight;
+    var locale = _a.locale, rowCount = _a.rowCount, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getDataValidationById = _a.getDataValidationById, getDependents = _a.getDependents, getDataRowCount = _a.getDataRowCount, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations, onChangeActiveCell = _a.onChangeActiveCell, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, createHistory = _a.createHistory, isArrayFormula = _a.isArrayFormula, getRowHeight = _a.getRowHeight, tables = _a.tables, generatedNames = _a.generatedNames;
     return function (sheetId, cell, selections, clearType) {
         var e_1, _a;
         if (clearType === void 0) { clearType = "contents"; }
@@ -17,6 +18,17 @@ export var deleteCells = function (_a) {
             : selectionFromActiveCell(cell);
         var saveHistory = createHistory();
         var maxRowCount = getDataRowCount();
+        // A table survives Delete / Clear (Excel): only its cells empty out, and
+        // cleared header cells are re-named `ColumnN` so the header row and
+        // `columns[]` stay in step.
+        var headerRefills = shouldClearContents
+            ? getTableHeaderRefills({
+                sheetId: sheetId,
+                tables: tables,
+                selections: finalSelections,
+                generatedNames: generatedNames,
+            })
+            : [];
         // Bulk writes go through `onChangeSheetData` directly, so the
         // calculator's per-cell `invalidateCacheForCell` (which only fires on
         // recompute) never runs for the cleared positions. Invalidate the
@@ -51,8 +63,8 @@ export var deleteCells = function (_a) {
         // Update sheet data
         onChangeSheetData === null || onChangeSheetData === void 0 ? void 0 : onChangeSheetData(function (prevSheetData) {
             var _a = __read(produceWithPatches(prevSheetData, function (draft) {
-                var e_2, _a;
-                var _b;
+                var e_2, _a, e_3, _b;
+                var _c, _d;
                 try {
                     for (var finalSelections_2 = __values(finalSelections), finalSelections_2_1 = finalSelections_2.next(); !finalSelections_2_1.done; finalSelections_2_1 = finalSelections_2.next()) {
                         var selection = finalSelections_2_1.value;
@@ -62,7 +74,7 @@ export var deleteCells = function (_a) {
                             var columnCount = currentSheet[rowIndex].values.length;
                             for (var columnIndex = selection.range.startColumnIndex; columnIndex <=
                                 Math.min(columnCount, selection.range.endColumnIndex); columnIndex++) {
-                                var cellData = (_b = currentSheet[rowIndex].values) === null || _b === void 0 ? void 0 : _b[columnIndex];
+                                var cellData = (_c = currentSheet[rowIndex].values) === null || _c === void 0 ? void 0 : _c[columnIndex];
                                 // Create a cell
                                 sheetCell.assign(sheetId, cell, cellData, {
                                     locale: locale,
@@ -123,6 +135,27 @@ export var deleteCells = function (_a) {
                     }
                     finally { if (e_2) throw e_2.error; }
                 }
+                try {
+                    for (var headerRefills_1 = __values(headerRefills), headerRefills_1_1 = headerRefills_1.next(); !headerRefills_1_1.done; headerRefills_1_1 = headerRefills_1.next()) {
+                        var _e = headerRefills_1_1.value, rowIndex = _e.rowIndex, columnIndex = _e.columnIndex, name_1 = _e.name;
+                        var currentSheet = createSheetDraft(draft, sheetId, rowIndex);
+                        sheetCell.assign(sheetId, { rowIndex: rowIndex, columnIndex: columnIndex }, (_d = currentSheet[rowIndex].values) === null || _d === void 0 ? void 0 : _d[columnIndex], {
+                            locale: locale,
+                            getDataValidationById: getDataValidationById,
+                            cellXfsRegistry: cellXfsRegistry,
+                            sharedStringRegistry: sharedStringRegistry,
+                        });
+                        sheetCell.setUserEnteredValue(name_1);
+                        currentSheet[rowIndex].values[columnIndex] = sheetCell.getCellData();
+                    }
+                }
+                catch (e_3_1) { e_3 = { error: e_3_1 }; }
+                finally {
+                    try {
+                        if (headerRefills_1_1 && !headerRefills_1_1.done && (_b = headerRefills_1.return)) _b.call(headerRefills_1);
+                    }
+                    finally { if (e_3) throw e_3.error; }
+                }
             }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
             // Save history
             saveHistory({
@@ -137,53 +170,82 @@ export var deleteCells = function (_a) {
             });
             return nextState;
         });
-        // Update tables
-        onChangeTables === null || onChangeTables === void 0 ? void 0 : onChangeTables(function (prevTables) {
-            var _a = __read(produceWithPatches(prevTables, function (draft) {
-                var tableIndex = draft.findIndex(function (table) {
-                    return table.sheetId === sheetId &&
-                        selections.some(function (sel) { return areaInsideArea(table.range, sel.range); });
+        // Rename the columns whose header cells were cleared
+        if (headerRefills.length) {
+            onChangeTables === null || onChangeTables === void 0 ? void 0 : onChangeTables(function (prevTables) {
+                var _a = __read(produceWithPatches(prevTables, function (draft) {
+                    var e_4, _a;
+                    var _b;
+                    var _loop_1 = function (tableId, dimensionIndex, name_2) {
+                        var table = draft.find(function (table) { return table.sheetId === sheetId && table.id === tableId; });
+                        if (!table)
+                            return "continue";
+                        (_b = table.columns) !== null && _b !== void 0 ? _b : (table.columns = []);
+                        var column = table.columns[dimensionIndex];
+                        if (column) {
+                            column.name = name_2;
+                        }
+                        else {
+                            table.columns[dimensionIndex] = { name: name_2 };
+                        }
+                    };
+                    try {
+                        for (var headerRefills_2 = __values(headerRefills), headerRefills_2_1 = headerRefills_2.next(); !headerRefills_2_1.done; headerRefills_2_1 = headerRefills_2.next()) {
+                            var _c = headerRefills_2_1.value, tableId = _c.tableId, dimensionIndex = _c.dimensionIndex, name_2 = _c.name;
+                            _loop_1(tableId, dimensionIndex, name_2);
+                        }
+                    }
+                    catch (e_4_1) { e_4 = { error: e_4_1 }; }
+                    finally {
+                        try {
+                            if (headerRefills_2_1 && !headerRefills_2_1.done && (_a = headerRefills_2.return)) _a.call(headerRefills_2);
+                        }
+                        finally { if (e_4) throw e_4.error; }
+                    }
+                }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+                saveHistory({
+                    tables: { patches: patches, inversePatches: inversePatches },
                 });
-                if (tableIndex !== -1) {
-                    draft.splice(tableIndex, 1);
-                }
-            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
-            saveHistory({
-                tables: { patches: patches, inversePatches: inversePatches },
+                return nextState;
             });
-            return nextState;
-        });
-        // Update citations - remove citations that overlap with deleted cells
-        onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
-            var _a = __read(produceWithPatches(prevCitations, function (draft) {
-                var _loop_1 = function (idx) {
-                    var citation = draft[idx];
-                    // Check if citation is on the same sheet and overlaps with any deleted selection
-                    if (citation.range.sheetId === sheetId &&
-                        finalSelections.some(function (sel) {
-                            return areaInsideArea(citation.range, sel.range);
-                        })) {
-                        // Remove citation since its referenced cells are being deleted
-                        draft.splice(idx, 1);
+        }
+        // Update citations - retire citations whose cited cells lose their contents.
+        // Clearing formats only leaves the cited values in place, so the citations stay.
+        if (shouldClearContents) {
+            onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
+                var updatedAt = new Date().toISOString();
+                var _a = __read(produceWithPatches(prevCitations, function (draft) {
+                    var _loop_2 = function (idx) {
+                        var citation = draft[idx];
+                        // Check if citation is active, on the same sheet and inside a cleared selection
+                        if (citation.active !== false &&
+                            citation.range.sheetId === sheetId &&
+                            finalSelections.some(function (sel) {
+                                return areaInsideArea(citation.range, sel.range);
+                            })) {
+                            // Soft-delete: keep the record for provenance / undo, hide it from the UI
+                            citation.active = false;
+                            citation.updated_at = updatedAt;
+                        }
+                    };
+                    for (var idx = draft.length - 1; idx >= 0; idx--) {
+                        _loop_2(idx);
                     }
-                };
-                for (var idx = draft.length - 1; idx >= 0; idx--) {
-                    _loop_1(idx);
-                }
-            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
-            saveHistory({
-                citations: { patches: patches, inversePatches: inversePatches },
+                }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+                saveHistory({
+                    citations: { patches: patches, inversePatches: inversePatches },
+                });
+                return nextState;
             });
-            return nextState;
-        });
+        }
         // Update banded cells
         onChangeSheets === null || onChangeSheets === void 0 ? void 0 : onChangeSheets(function (prevSheets) {
             var _a = __read(produceWithPatches(prevSheets, function (draft) {
-                var e_3, _a;
+                var e_5, _a;
                 var _b, _c, _d, _e, _f;
                 var sheet = draft.find(function (sheet) { return sheet.sheetId === sheetId; });
                 if (sheet) {
-                    var _loop_2 = function (selection) {
+                    var _loop_3 = function (selection) {
                         for (var rowIndex = selection.range.startRowIndex; rowIndex <=
                             Math.min(maxRowCount !== null && maxRowCount !== void 0 ? maxRowCount : rowCount, selection.range.endRowIndex); rowIndex++) {
                             var isRowResizedByUser = !!((_c = (_b = sheet === null || sheet === void 0 ? void 0 : sheet.rowMetadata) === null || _b === void 0 ? void 0 : _b[rowIndex]) === null || _c === void 0 ? void 0 : _c.resizedByUser);
@@ -208,15 +270,15 @@ export var deleteCells = function (_a) {
                         // Adjust row height
                         for (var finalSelections_3 = __values(finalSelections), finalSelections_3_1 = finalSelections_3.next(); !finalSelections_3_1.done; finalSelections_3_1 = finalSelections_3.next()) {
                             var selection = finalSelections_3_1.value;
-                            _loop_2(selection);
+                            _loop_3(selection);
                         }
                     }
-                    catch (e_3_1) { e_3 = { error: e_3_1 }; }
+                    catch (e_5_1) { e_5 = { error: e_5_1 }; }
                     finally {
                         try {
                             if (finalSelections_3_1 && !finalSelections_3_1.done && (_a = finalSelections_3.return)) _a.call(finalSelections_3);
                         }
-                        finally { if (e_3) throw e_3.error; }
+                        finally { if (e_5) throw e_5.error; }
                     }
                 }
             }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];

package/dist/cjs/interface/delete-column.d.ts

@@ -1,4 +1,4 @@
 import type { CellData, Sheet } from "@rowsncolumns/spreadsheet";
 import type { UseOnDeleteColumnProps } from "../hooks/use-on-delete-column";
-export declare const deleteColumn: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getSheetName, getActiveCell, getSelections, getDependents, getGraphNodes, getGraphNodesInSheet, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeActiveCell, onChangeSelections, onChangeConditionalFormats, onChangeDataValidations, onChangeCitations, onChangePivotTables, tables, getArrayRange, onChangeNamedRanges, onChangeEmbeds, onChangeCharts, onChangeProtectedRanges, }: UseOnDeleteColumnProps<T, S>) => (sheetId: number, columnIndexes: number[]) => void;
+export declare const deleteColumn: <T extends CellData, S extends Sheet>({ rowCount, columnCount, locale, cellXfsRegistry, sharedStringRegistry, getSheetName, getActiveCell, getSelections, getDependents, getGraphNodes, getGraphNodesInSheet, createHistory, enqueueGraphOperation, enqueueCalculation, enqueueStructureChange, onChangeSheetData, onChangeSheets, onChangeTables, onChangeActiveCell, onChangeSelections, onChangeConditionalFormats, onChangeDataValidations, onChangeCitations, onChangePivotTables, tables, getArrayRange, onChangeNamedRanges, onChangeEmbeds, onChangeSlicers, onChangeCharts, onChangeProtectedRanges, }: UseOnDeleteColumnProps<T, S>) => (sheetId: number, columnIndexes: number[]) => void;
 //# sourceMappingURL=delete-column.d.ts.map

package/dist/cjs/interface/delete-column.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-column.d.ts","sourceRoot":"","sources":["../../../interface/delete-column.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAsBjE,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,+BAA+B,CAAC;AAG5E,eAAO,MAAM,YAAY,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,0hBA+B/D,sBAAsB,CAAC,CAAC,EAAE,CAAC,CAAC,MACrB,SAAS,MAAM,EAAE,eAAe,MAAM,EAAE,SA8bjD,CAAC"}
+{"version":3,"file":"delete-column.d.ts","sourceRoot":"","sources":["../../../interface/delete-column.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAwBjE,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,+BAA+B,CAAC;AAG5E,eAAO,MAAM,YAAY,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,2iBAgC/D,sBAAsB,CAAC,CAAC,EAAE,CAAC,CAAC,MACrB,SAAS,MAAM,EAAE,eAAe,MAAM,EAAE,SAsdjD,CAAC"}

package/dist/cjs/interface/delete-column.js

@@ -1,7 +1,7 @@
 import { __assign, __read, __spreadArray, __values } from "tslib";
 import { Direction } from "@rowsncolumns/common-types";
 import { produceWithPatches } from "immer";
-import { shiftChartsOnStructuralChange, shiftEmbedsOnStructuralChange, shiftProtectedRangesOnStructuralChange, } from "../api/anchored-objects";
+import { shiftChartsOnStructuralChange, shiftEmbedsOnStructuralChange, shiftSlicersOnStructuralChange, shiftProtectedRangesOnStructuralChange, refreshCitationAnchor, } from "../api/anchored-objects";
 import { shrinkMergesOnColumnDelete } from "../api/merges";
 import { updateNamedRangesOnStructuralChange, updatePrintSettingsOnStructuralChange, } from "../api/named-range-structural";
 import { shiftPivotsOnDelete } from "../api/pivot-range";
@@ -10,7 +10,7 @@ import { moveFilterView } from "../api/selection";
 import { updateFormulasForStructuralChange } from "../api/structural-formula-update";
 import { breakStructuredRefsForRemovedTableParts, collectRemovedTableParts, removeSpillsOfDeletedAnchors, } from "../api/structural-table-refs";
 export var deleteColumn = function (_a) {
-    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getSheetName = _a.getSheetName, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getGraphNodes = _a.getGraphNodes, getGraphNodesInSheet = _a.getGraphNodesInSheet, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeActiveCell = _a.onChangeActiveCell, onChangeSelections = _a.onChangeSelections, onChangeConditionalFormats = _a.onChangeConditionalFormats, onChangeDataValidations = _a.onChangeDataValidations, onChangeCitations = _a.onChangeCitations, onChangePivotTables = _a.onChangePivotTables, tables = _a.tables, getArrayRange = _a.getArrayRange, onChangeNamedRanges = _a.onChangeNamedRanges, onChangeEmbeds = _a.onChangeEmbeds, onChangeCharts = _a.onChangeCharts, onChangeProtectedRanges = _a.onChangeProtectedRanges;
+    var rowCount = _a.rowCount, columnCount = _a.columnCount, locale = _a.locale, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getSheetName = _a.getSheetName, getActiveCell = _a.getActiveCell, getSelections = _a.getSelections, getDependents = _a.getDependents, getGraphNodes = _a.getGraphNodes, getGraphNodesInSheet = _a.getGraphNodesInSheet, createHistory = _a.createHistory, enqueueGraphOperation = _a.enqueueGraphOperation, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeActiveCell = _a.onChangeActiveCell, onChangeSelections = _a.onChangeSelections, onChangeConditionalFormats = _a.onChangeConditionalFormats, onChangeDataValidations = _a.onChangeDataValidations, onChangeCitations = _a.onChangeCitations, onChangePivotTables = _a.onChangePivotTables, tables = _a.tables, getArrayRange = _a.getArrayRange, onChangeNamedRanges = _a.onChangeNamedRanges, onChangeEmbeds = _a.onChangeEmbeds, onChangeSlicers = _a.onChangeSlicers, onChangeCharts = _a.onChangeCharts, onChangeProtectedRanges = _a.onChangeProtectedRanges;
     return function (sheetId, columnIndexes) {
         if (!columnIndexes.length)
             return;
@@ -163,6 +163,7 @@ export var deleteColumn = function (_a) {
         });
         // Citations
         onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
+            var updatedAt = new Date().toISOString();
             var _a = __read(produceWithPatches(prevCitations, function (draft) {
                 var deleteEnd = minColumnIndex + count - 1;
                 for (var idx = draft.length - 1; idx >= 0; idx--) {
@@ -172,11 +173,16 @@ export var deleteColumn = function (_a) {
                     var citationStart = citation.range.startColumnIndex;
                     var citationEnd = citation.range.endColumnIndex;
                     if (citationStart <= deleteEnd && citationEnd >= minColumnIndex) {
-                        draft.splice(idx, 1);
+                        // Soft-delete: keep the record for provenance / undo, hide it from the UI
+                        if (citation.active !== false) {
+                            citation.active = false;
+                            citation.updated_at = updatedAt;
+                        }
                     }
                     else if (citationStart > deleteEnd) {
                         citation.range.startColumnIndex -= count;
                         citation.range.endColumnIndex -= count;
+                        refreshCitationAnchor(citation);
                     }
                 }
             }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
@@ -194,6 +200,13 @@ export var deleteColumn = function (_a) {
             saveHistory({ embeds: { patches: patches, inversePatches: inversePatches } });
             return nextState;
         });
+        onChangeSlicers === null || onChangeSlicers === void 0 ? void 0 : onChangeSlicers(function (prevSlicers) {
+            var _a = __read(produceWithPatches(prevSlicers, function (draft) {
+                shiftSlicersOnStructuralChange(draft, sheetId, "column", "delete", minColumnIndex, count);
+            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+            saveHistory({ slicers: { patches: patches, inversePatches: inversePatches } });
+            return nextState;
+        });
         onChangeCharts === null || onChangeCharts === void 0 ? void 0 : onChangeCharts(function (prevCharts) {
             var _a = __read(produceWithPatches(prevCharts, function (draft) {
                 shiftChartsOnStructuralChange(draft, sheetId, "column", "delete", minColumnIndex, count);