@univerjs/sheets 1.0.0-alpha.8 → 1.0.0-beta.1

package/lib/cjs/facade.js

@@ -191,9 +191,9 @@ let FDefinedNameBuilder = class FDefinedNameBuilder {
 	setRefByRange(row, column, numRows, numColumns) {
 		this._definedNameParam.formulaOrRefString = (0, _univerjs_engine_formula.serializeRange)({
 			startRow: row,
-			endRow: row + (numRows !== null && numRows !== void 0 ? numRows : 1) - 1,
+			endRow: row + (numRows ?? 1) - 1,
 			startColumn: column,
-			endColumn: column + (numColumns !== null && numColumns !== void 0 ? numColumns : 1) - 1
+			endColumn: column + (numColumns ?? 1) - 1
 		});
 		return this;
 	}
@@ -433,9 +433,9 @@ let FDefinedName = class FDefinedName extends _univerjs_core_facade.FBase {
 	setRefByRange(row, column, numRows, numColumns) {
 		this._definedNameParam.formulaOrRefString = (0, _univerjs_engine_formula.serializeRange)({
 			startRow: row,
-			endRow: row + (numRows !== null && numRows !== void 0 ? numRows : 1) - 1,
+			endRow: row + (numRows ?? 1) - 1,
 			startColumn: column,
-			endColumn: column + (numColumns !== null && numColumns !== void 0 ? numColumns : 1) - 1
+			endColumn: column + (numColumns ?? 1) - 1
 		});
 		this._apply();
 	}
@@ -1619,10 +1619,9 @@ let FWorksheetPermission = class FWorksheetPermission extends _univerjs_core_fac
 		if (!this.canEdit()) return false;
 		const rules = this._rangeProtectionRuleModel.getSubunitRuleList(this._unitId, this._subUnitId);
 		for (const rule of rules) for (const range of rule.ranges) if (row >= range.startRow && row <= range.endRow && col >= range.startColumn && col <= range.endColumn) {
-			var _permission$value;
 			const permissionPoint = new _univerjs_sheets.RangeProtectionPermissionEditPoint(this._unitId, this._subUnitId, rule.permissionId);
 			const permission = this._permissionService.getPermissionPoint(permissionPoint.id);
-			return (_permission$value = permission === null || permission === void 0 ? void 0 : permission.value) !== null && _permission$value !== void 0 ? _permission$value : false;
+			return (permission === null || permission === void 0 ? void 0 : permission.value) ?? false;
 		}
 		return true;
 	}
@@ -1661,10 +1660,9 @@ let FWorksheetPermission = class FWorksheetPermission extends _univerjs_core_fac
 		if (!this.canView()) return false;
 		const rules = this._rangeProtectionRuleModel.getSubunitRuleList(this._unitId, this._subUnitId);
 		for (const rule of rules) for (const range of rule.ranges) if (row >= range.startRow && row <= range.endRow && col >= range.startColumn && col <= range.endColumn) {
-			var _permission$value2;
 			const permissionPoint = new _univerjs_sheets.RangeProtectionPermissionViewPoint(this._unitId, this._subUnitId, rule.permissionId);
 			const permission = this._permissionService.getPermissionPoint(permissionPoint.id);
-			return (_permission$value2 = permission === null || permission === void 0 ? void 0 : permission.value) !== null && _permission$value2 !== void 0 ? _permission$value2 : false;
+			return (permission === null || permission === void 0 ? void 0 : permission.value) ?? false;
 		}
 		return true;
 	}
@@ -1683,12 +1681,11 @@ let FWorksheetPermission = class FWorksheetPermission extends _univerjs_core_fac
 	* ```
 	*/
 	getPoint(point) {
-		var _permissionPoint$valu;
 		const PointClass = WORKSHEET_PERMISSION_POINT_MAP[point];
 		if (!PointClass) throw new Error(`Unknown worksheet permission point: ${point}`);
 		const instance = new PointClass(this._unitId, this._subUnitId);
 		const permissionPoint = this._permissionService.getPermissionPoint(instance.id);
-		return (_permissionPoint$valu = permissionPoint === null || permissionPoint === void 0 ? void 0 : permissionPoint.value) !== null && _permissionPoint$valu !== void 0 ? _permissionPoint$valu : true;
+		return (permissionPoint === null || permissionPoint === void 0 ? void 0 : permissionPoint.value) ?? true;
 	}
 	/**
 	* Get a snapshot of all permission points.
@@ -2020,6 +2017,11 @@ function covertToColRange(range, worksheet) {
 //#endregion
 //#region src/facade/f-worksheet.ts
 var _FWorksheet;
+function assertValidRowInsertion(methodName, rowIndex, rowCount, howMany) {
+	if (!Number.isInteger(howMany) || howMany <= 0) throw new RangeError(`${methodName}(): row count ${howMany} must be a positive integer.`);
+	const lastRowIndex = rowCount - 1;
+	if (!Number.isInteger(rowIndex) || rowIndex < 0 || rowIndex > lastRowIndex) throw new RangeError(`${methodName}(): row index ${rowIndex} is out of bounds. The worksheet has ${rowCount} rows; expected an integer from 0 to ${lastRowIndex}. To insert ${howMany} row(s) at the bottom, use insertRowsAfter(${lastRowIndex}, ${howMany}). To resize the worksheet directly, use setRowCount(${rowCount + howMany}).`);
+}
 let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FBaseInitialable {
 	/**
 	* Creates a new worksheet facade instance
@@ -2300,9 +2302,9 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 			sheet = this._worksheet;
 			range = {
 				startRow: rowOrA1Notation,
-				endRow: rowOrA1Notation + (numRows !== null && numRows !== void 0 ? numRows : 1) - 1,
+				endRow: rowOrA1Notation + (numRows ?? 1) - 1,
 				startColumn: column,
-				endColumn: column + (numColumns !== null && numColumns !== void 0 ? numColumns : 1) - 1,
+				endColumn: column + (numColumns ?? 1) - 1,
 				unitId: this._workbook.getUnitId(),
 				sheetId: this._worksheet.getSheetId()
 			};
@@ -2339,7 +2341,7 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	}
 	/**
 	* Inserts a row after the given row position.
-	* @param {number} afterPosition - The row after which the new row should be added, starting at 0 for the first row.
+	* @param {number} afterPosition - The existing row after which the new row should be added. The index is zero-based and must be between 0 and `getMaxRows() - 1`.
 	* @returns {FWorksheet} This sheet, for chaining.
 	* @example
 	* ```typescript
@@ -2356,7 +2358,7 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	}
 	/**
 	* Inserts a row before the given row position.
-	* @param {number} beforePosition - The row before which the new row should be added, starting at 0 for the first row.
+	* @param {number} beforePosition - The existing row before which the new row should be added. The index is zero-based and must be between 0 and `getMaxRows() - 1`.
 	* @returns {FWorksheet} This sheet, for chaining.
 	* @example
 	* ```typescript
@@ -2373,8 +2375,8 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	}
 	/**
 	* Inserts one or more consecutive blank rows in a sheet starting at the specified location.
-	* @param {number} rowIndex - The index indicating where to insert a row, starting at 0 for the first row.
-	* @param {number} numRows - The number of rows to insert.
+	* @param {number} rowIndex - The existing row before which rows are inserted. The index is zero-based and must be between 0 and `getMaxRows() - 1`.
+	* @param {number} numRows - The positive number of rows to insert.
 	* @returns {FWorksheet} This sheet, for chaining.
 	* @example
 	* ```typescript
@@ -2387,12 +2389,13 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	* ```
 	*/
 	insertRows(rowIndex, numRows = 1) {
+		assertValidRowInsertion("insertRows", rowIndex, this.getMaxRows(), numRows);
 		return this.insertRowsBefore(rowIndex, numRows);
 	}
 	/**
 	* Inserts a number of rows after the given row position.
-	* @param {number} afterPosition - The row after which the new rows should be added, starting at 0 for the first row.
-	* @param {number} howMany - The number of rows to insert.
+	* @param {number} afterPosition - The existing row after which the new rows should be added. The index is zero-based and must be between 0 and `getMaxRows() - 1`.
+	* @param {number} howMany - The positive number of rows to insert.
 	* @returns {FWorksheet} This sheet, for chaining.
 	* @example
 	* ```typescript
@@ -2405,6 +2408,7 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	* ```
 	*/
 	insertRowsAfter(afterPosition, howMany) {
+		assertValidRowInsertion("insertRowsAfter", afterPosition, this.getMaxRows(), howMany);
 		const unitId = this._workbook.getUnitId();
 		const subUnitId = this._worksheet.getSheetId();
 		const direction = _univerjs_core.Direction.DOWN;
@@ -2429,8 +2433,8 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	}
 	/**
 	* Inserts a number of rows before the given row position.
-	* @param {number} beforePosition - The row before which the new rows should be added, starting at 0 for the first row.
-	* @param {number} howMany - The number of rows to insert.
+	* @param {number} beforePosition - The existing row before which the new rows should be added. The index is zero-based and must be between 0 and `getMaxRows() - 1`.
+	* @param {number} howMany - The positive number of rows to insert.
 	* @returns {FWorksheet} This sheet, for chaining.
 	* @example
 	* ```typescript
@@ -2443,6 +2447,7 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 	* ```
 	*/
 	insertRowsBefore(beforePosition, howMany) {
+		assertValidRowInsertion("insertRowsBefore", beforePosition, this.getMaxRows(), howMany);
 		const unitId = this._workbook.getUnitId();
 		const subUnitId = this._worksheet.getSheetId();
 		const direction = _univerjs_core.Direction.UP;
@@ -4146,7 +4151,8 @@ let FWorksheet = _FWorksheet = class FWorksheet extends _univerjs_core_facade.FB
 		return this._worksheet.getColumnManager().getCustomMetadata(index);
 	}
 	/**
-	* Appends a row to the bottom of the current data region in the sheet. If a cell's content begins with =, it's interpreted as a formula.
+	* Appends a row to the bottom of the current data region in the sheet. If the destination row is already within the worksheet capacity,
+	* this method writes into that row without increasing `getMaxRows()`. If a cell's content begins with =, it's interpreted as a formula.
 	* @param {CellValue[]} rowContents - An array of values for the new row.
 	* @returns {FWorksheet} Returns the current worksheet instance for method chaining.
 	* @example
@@ -4651,8 +4657,8 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getFontFamily(type = "row") {
-		var _this$getCellStyleDat, _this$getCellStyleDat2;
-		return (_this$getCellStyleDat = (_this$getCellStyleDat2 = this.getCellStyleData(type)) === null || _this$getCellStyleDat2 === void 0 ? void 0 : _this$getCellStyleDat2.ff) !== null && _this$getCellStyleDat !== void 0 ? _this$getCellStyleDat : null;
+		var _this$getCellStyleDat;
+		return ((_this$getCellStyleDat = this.getCellStyleData(type)) === null || _this$getCellStyleDat === void 0 ? void 0 : _this$getCellStyleDat.ff) ?? null;
 	}
 	/**
 	* Get the font size of the cell.
@@ -4673,8 +4679,8 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getFontSize(type = "row") {
-		var _this$getCellStyleDat3, _this$getCellStyleDat4;
-		return (_this$getCellStyleDat3 = (_this$getCellStyleDat4 = this.getCellStyleData(type)) === null || _this$getCellStyleDat4 === void 0 ? void 0 : _this$getCellStyleDat4.fs) !== null && _this$getCellStyleDat3 !== void 0 ? _this$getCellStyleDat3 : null;
+		var _this$getCellStyleDat2;
+		return ((_this$getCellStyleDat2 = this.getCellStyleData(type)) === null || _this$getCellStyleDat2 === void 0 ? void 0 : _this$getCellStyleDat2.fs) ?? null;
 	}
 	/**
 	* Return first cell style in this range.
@@ -4724,9 +4730,9 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 		}));
 	}
 	getValue(includeRichText) {
-		var _this$_worksheet$getC, _this$_worksheet$getC2;
+		var _this$_worksheet$getC;
 		if (includeRichText) return this.getValueAndRichTextValue();
-		return (_this$_worksheet$getC = (_this$_worksheet$getC2 = this._worksheet.getCell(this._range.startRow, this._range.startColumn)) === null || _this$_worksheet$getC2 === void 0 ? void 0 : _this$_worksheet$getC2.v) !== null && _this$_worksheet$getC !== void 0 ? _this$_worksheet$getC : null;
+		return ((_this$_worksheet$getC = this._worksheet.getCell(this._range.startRow, this._range.startColumn)) === null || _this$_worksheet$getC === void 0 ? void 0 : _this$_worksheet$getC.v) ?? null;
 	}
 	/**
 	* Returns the raw value of the top-left cell in the range. Empty cells return `null`.
@@ -4781,8 +4787,8 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 		for (let r = startRow; r <= endRow; r++) {
 			const row = [];
 			for (let c = startColumn; c <= endColumn; c++) {
-				var _this$_worksheet$getC3, _this$_worksheet$getC4;
-				row.push((_this$_worksheet$getC3 = (_this$_worksheet$getC4 = this._worksheet.getCell(r, c)) === null || _this$_worksheet$getC4 === void 0 ? void 0 : _this$_worksheet$getC4.v) !== null && _this$_worksheet$getC3 !== void 0 ? _this$_worksheet$getC3 : null);
+				var _this$_worksheet$getC2;
+				row.push(((_this$_worksheet$getC2 = this._worksheet.getCell(r, c)) === null || _this$_worksheet$getC2 === void 0 ? void 0 : _this$_worksheet$getC2.v) ?? null);
 			}
 			range.push(row);
 		}
@@ -4914,8 +4920,7 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getCellData() {
-		var _this$_worksheet$getC5;
-		return (_this$_worksheet$getC5 = this._worksheet.getCell(this._range.startRow, this._range.startColumn)) !== null && _this$_worksheet$getC5 !== void 0 ? _this$_worksheet$getC5 : null;
+		return this._worksheet.getCell(this._range.startRow, this._range.startColumn) ?? null;
 	}
 	/**
 	* Alias for getCellDataGrid.
@@ -5036,8 +5041,7 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getFormula() {
-		var _this$_formulaDataMod;
-		return (_this$_formulaDataMod = this._formulaDataModel.getFormulaStringByCell(this._range.startRow, this._range.startColumn, this._worksheet.getSheetId(), this._workbook.getUnitId())) !== null && _this$_formulaDataMod !== void 0 ? _this$_formulaDataMod : "";
+		return this._formulaDataModel.getFormulaStringByCell(this._range.startRow, this._range.startColumn, this._worksheet.getSheetId(), this._workbook.getUnitId()) ?? "";
 	}
 	/**
 	* Returns the formulas (A1 notation) for the cells in the range. Entries in the 2D array are empty strings for cells with no formula.
@@ -5083,9 +5087,9 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	}
 	/** Gets whether the top-left cell shrinks its font size to fit the cell width. */
 	getShrinkToFit() {
-		var _this$_worksheet$getC6;
+		var _this$_worksheet$getC3;
 		const { startRow, startColumn } = this._range;
-		return ((_this$_worksheet$getC6 = this._worksheet.getComposedCellStyle(startRow, startColumn)) === null || _this$_worksheet$getC6 === void 0 ? void 0 : _this$_worksheet$getC6.stf) === _univerjs_core.BooleanNumber.TRUE;
+		return ((_this$_worksheet$getC3 = this._worksheet.getComposedCellStyle(startRow, startColumn)) === null || _this$_worksheet$getC3 === void 0 ? void 0 : _this$_worksheet$getC3.stf) === _univerjs_core.BooleanNumber.TRUE;
 	}
 	/**
 	* Gets whether text wrapping is enabled for cells in the range.
@@ -5245,10 +5249,9 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getCustomMetaData() {
-		var _cell$custom;
 		this._logService.warn(SHEETS_CUSTOM_FIELD_WARNING_MESSAGE);
 		const cell = this.getCellData();
-		return (_cell$custom = cell === null || cell === void 0 ? void 0 : cell.custom) !== null && _cell$custom !== void 0 ? _cell$custom : null;
+		return (cell === null || cell === void 0 ? void 0 : cell.custom) ?? null;
 	}
 	/**
 	* Returns the custom meta data for the cells in the range.
@@ -5264,10 +5267,7 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	*/
 	getCustomMetaDatas() {
 		this._logService.warn(SHEETS_CUSTOM_FIELD_WARNING_MESSAGE);
-		return this.getCellDataGrid().map((row) => row.map((data) => {
-			var _data$custom;
-			return (_data$custom = data === null || data === void 0 ? void 0 : data.custom) !== null && _data$custom !== void 0 ? _data$custom : null;
-		}));
+		return this.getCellDataGrid().map((row) => row.map((data) => (data === null || data === void 0 ? void 0 : data.custom) ?? null));
 	}
 	/**
 	* Sets basic border properties for the current range.
@@ -5310,9 +5310,9 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	* ```
 	*/
 	getBackground() {
-		var _style$background$rgb, _style$background;
+		var _style$background;
 		const style = this.getCellStyle();
-		return (_style$background$rgb = style === null || style === void 0 || (_style$background = style.background) === null || _style$background === void 0 ? void 0 : _style$background.rgb) !== null && _style$background$rgb !== void 0 ? _style$background$rgb : _univerjs_core.DEFAULT_STYLES.bg.rgb;
+		return (style === null || style === void 0 || (_style$background = style.background) === null || _style$background === void 0 ? void 0 : _style$background.rgb) ?? _univerjs_core.DEFAULT_STYLES.bg.rgb;
 	}
 	/**
 	* Returns the background colors of the cells in the range.
@@ -5328,8 +5328,8 @@ let FRange = _FRange = class FRange extends _univerjs_core_facade.FBaseInitialab
 	*/
 	getBackgrounds() {
 		return this.getCellStyles().map((row) => row.map((style) => {
-			var _style$background$rgb2, _style$background2;
-			return (_style$background$rgb2 = style === null || style === void 0 || (_style$background2 = style.background) === null || _style$background2 === void 0 ? void 0 : _style$background2.rgb) !== null && _style$background$rgb2 !== void 0 ? _style$background$rgb2 : _univerjs_core.DEFAULT_STYLES.bg.rgb;
+			var _style$background2;
+			return (style === null || style === void 0 || (_style$background2 = style.background) === null || _style$background2 === void 0 ? void 0 : _style$background2.rgb) ?? _univerjs_core.DEFAULT_STYLES.bg.rgb;
 		}));
 	}
 	/**
@@ -6871,12 +6871,11 @@ let FWorkbookPermission = class FWorkbookPermission extends _univerjs_core_facad
 	* ```
 	*/
 	getPoint(point) {
-		var _permissionPoint$valu;
 		const PointClass = WORKBOOK_PERMISSION_POINT_MAP[point];
 		if (!PointClass) throw new Error(`Unknown workbook permission point: ${point}`);
 		const instance = new PointClass(this._unitId);
 		const permissionPoint = this._permissionService.getPermissionPoint(instance.id);
-		return (_permissionPoint$valu = permissionPoint === null || permissionPoint === void 0 ? void 0 : permissionPoint.value) !== null && _permissionPoint$valu !== void 0 ? _permissionPoint$valu : true;
+		return (permissionPoint === null || permissionPoint === void 0 ? void 0 : permissionPoint.value) ?? true;
 	}
 	/**
 	* Get a snapshot of all permission points.
@@ -7210,13 +7209,13 @@ let FWorkbook = class FWorkbook extends _univerjs_core_facade.FBaseInitialable {
 	* ```
 	*/
 	create(name, rows, columns, options) {
-		var _options$sheet, _options$sheet2, _options$index;
-		const newSheet = (0, _univerjs_core.mergeWorksheetSnapshotWithDefault)(_univerjs_core.Tools.deepClone((_options$sheet = options === null || options === void 0 ? void 0 : options.sheet) !== null && _options$sheet !== void 0 ? _options$sheet : {}));
+		var _options$sheet;
+		const newSheet = (0, _univerjs_core.mergeWorksheetSnapshotWithDefault)(_univerjs_core.Tools.deepClone((options === null || options === void 0 ? void 0 : options.sheet) ?? {}));
 		newSheet.name = name;
 		newSheet.rowCount = rows;
 		newSheet.columnCount = columns;
-		newSheet.id = options === null || options === void 0 || (_options$sheet2 = options.sheet) === null || _options$sheet2 === void 0 ? void 0 : _options$sheet2.id;
-		const newSheetIndex = (_options$index = options === null || options === void 0 ? void 0 : options.index) !== null && _options$index !== void 0 ? _options$index : this._workbook.getSheets().length;
+		newSheet.id = options === null || options === void 0 || (_options$sheet = options.sheet) === null || _options$sheet === void 0 ? void 0 : _options$sheet.id;
+		const newSheetIndex = (options === null || options === void 0 ? void 0 : options.index) ?? this._workbook.getSheets().length;
 		this._commandService.syncExecuteCommand(_univerjs_sheets.InsertSheetCommand.id, {
 			unitId: this.id,
 			index: newSheetIndex,
@@ -7323,12 +7322,12 @@ let FWorkbook = class FWorkbook extends _univerjs_core_facade.FBaseInitialable {
 	* ```
 	*/
 	insertSheet(sheetName, options) {
-		var _options$sheet3, _options$sheet4, _options$index2;
-		const newSheet = (0, _univerjs_core.mergeWorksheetSnapshotWithDefault)(_univerjs_core.Tools.deepClone((_options$sheet3 = options === null || options === void 0 ? void 0 : options.sheet) !== null && _options$sheet3 !== void 0 ? _options$sheet3 : {}));
+		var _options$sheet2;
+		const newSheet = (0, _univerjs_core.mergeWorksheetSnapshotWithDefault)(_univerjs_core.Tools.deepClone((options === null || options === void 0 ? void 0 : options.sheet) ?? {}));
 		if (sheetName !== void 0) newSheet.name = sheetName;
 		else delete newSheet.name;
-		newSheet.id = options === null || options === void 0 || (_options$sheet4 = options.sheet) === null || _options$sheet4 === void 0 ? void 0 : _options$sheet4.id;
-		const newSheetIndex = (_options$index2 = options === null || options === void 0 ? void 0 : options.index) !== null && _options$index2 !== void 0 ? _options$index2 : this._workbook.getSheets().length;
+		newSheet.id = options === null || options === void 0 || (_options$sheet2 = options.sheet) === null || _options$sheet2 === void 0 ? void 0 : _options$sheet2.id;
+		const newSheetIndex = (options === null || options === void 0 ? void 0 : options.index) ?? this._workbook.getSheets().length;
 		this._commandService.syncExecuteCommand(_univerjs_sheets.InsertSheetCommand.id, {
 			unitId: this.id,
 			index: newSheetIndex,
@@ -8205,12 +8204,11 @@ var FUniverSheetsMixin = class extends _univerjs_core_facade.FUniver {
 		})));
 		this.disposeWithMe(this.registerEventHandler(this.Event.BeforeGridlineEnableChange, () => commandService.beforeCommandExecuted((commandInfo) => {
 			if (commandInfo.id === _univerjs_sheets.ToggleGridlinesCommand.id) {
-				var _params$showGridlines;
 				const params = commandInfo.params;
 				const target = this.getSheetCommandTarget(params);
 				if (!target) return;
 				const { workbook, worksheet } = target;
-				const showGridlines = (_params$showGridlines = params.showGridlines) !== null && _params$showGridlines !== void 0 ? _params$showGridlines : !worksheet.hasHiddenGridLines();
+				const showGridlines = params.showGridlines ?? !worksheet.hasHiddenGridLines();
 				const eventParams = {
 					workbook,
 					worksheet,
@@ -8237,12 +8235,12 @@ var FUniverSheetsMixin = class extends _univerjs_core_facade.FUniver {
 		})));
 		this.disposeWithMe(this.registerEventHandler(this.Event.SheetCreated, () => commandService.onCommandExecuted((commandInfo) => {
 			if (commandInfo.id === _univerjs_sheets.InsertSheetCommand.id) {
-				var _params$sheet, _params$index;
+				var _params$sheet;
 				const params = commandInfo.params;
 				const workbook = (params === null || params === void 0 ? void 0 : params.unitId) ? this.getWorkbook(params.unitId) : this.getActiveWorkbook();
 				if (!workbook) return;
 				const sheets = workbook.getSheets();
-				const worksheet = (params === null || params === void 0 || (_params$sheet = params.sheet) === null || _params$sheet === void 0 ? void 0 : _params$sheet.id) ? workbook.getSheetBySheetId(params.sheet.id) : sheets[(_params$index = params === null || params === void 0 ? void 0 : params.index) !== null && _params$index !== void 0 ? _params$index : sheets.length - 1];
+				const worksheet = (params === null || params === void 0 || (_params$sheet = params.sheet) === null || _params$sheet === void 0 ? void 0 : _params$sheet.id) ? workbook.getSheetBySheetId(params.sheet.id) : sheets[(params === null || params === void 0 ? void 0 : params.index) ?? sheets.length - 1];
 				if (!worksheet) return;
 				const eventParams = {
 					workbook,