@univerjs/drawing 1.0.0-rc.0 → 1.0.1

package/lib/cjs/index.js

@@ -1,34 +1,5 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
-//#region \0rolldown/runtime.js
-var __create = Object.create;
-var __defProp = Object.defineProperty;
-var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __copyProps = (to, from, except, desc) => {
-	if (from && typeof from === "object" || typeof from === "function") {
-		for (var keys = __getOwnPropNames(from), i = 0, n = keys.length, key; i < n; i++) {
-			key = keys[i];
-			if (!__hasOwnProp.call(to, key) && key !== except) {
-				__defProp(to, key, {
-					get: ((k) => from[k]).bind(null, key),
-					enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable
-				});
-			}
-		}
-	}
-	return to;
-};
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", {
-	value: mod,
-	enumerable: true
-}) : target, mod));
-
-//#endregion
 let _univerjs_core = require("@univerjs/core");
-let ot_json1 = require("ot-json1");
-ot_json1 = __toESM(ot_json1);
 let rxjs = require("rxjs");
 
 //#region src/basics/config.ts
@@ -50,7 +21,7 @@ let rxjs = require("rxjs");
 const DRAWING_IMAGE_WIDTH_LIMIT = 500;
 const DRAWING_IMAGE_HEIGHT_LIMIT = 500;
 const DRAWING_IMAGE_COUNT_LIMIT = 10;
-let DRAWING_IMAGE_ALLOW_SIZE = 5 * 1024 * 1024;
+let DRAWING_IMAGE_ALLOW_SIZE = 5242880;
 function setDrawingImageAllowSize(size) {
 	DRAWING_IMAGE_ALLOW_SIZE = size;
 }
@@ -85,7 +56,7 @@ const SetDrawingSelectedOperation = {
 //#endregion
 //#region package.json
 var name = "@univerjs/drawing";
-var version = "1.0.0-rc.0";
+var version = "1.0.1";
 
 //#endregion
 //#region src/config/config.ts
@@ -94,7 +65,7 @@ const configSymbol = Symbol(DRAWING_PLUGIN_CONFIG_KEY);
 const defaultPluginConfig = {};
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/typeof.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -105,7 +76,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPrimitive.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -118,14 +89,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/toPropertyKey.js
 function toPropertyKey(t) {
 	var i = toPrimitive(t, "string");
 	return "symbol" == _typeof(i) ? i : i + "";
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/defineProperty.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -159,7 +130,7 @@ function hasOnlyEnumerableOwnProperties(value) {
 	return Object.getOwnPropertyNames(value).every((key) => Object.prototype.propertyIsEnumerable.call(value, key));
 }
 function createJsonRemoveOp(path, value) {
-	return ot_json1.type.writeCursor().writeAtPath(path, "r", value).get();
+	return _univerjs_core.JSON1.type.writeCursor().writeAtPath(path, "r", value).get();
 }
 function normalizeJsonDocument(value) {
 	if (value === void 0) return;
@@ -179,12 +150,26 @@ function appendJsonUpdateOps(ops, path, newValue, oldValue, hasOldValue) {
 	const hasNormalizedOldValue = hasOldValue && normalizedOldValue !== void 0;
 	if (hasNormalizedOldValue && isJsonValueEqual(normalizedOldValue, normalizedNewValue)) return;
 	if (hasNormalizedOldValue && isPlainObject(oldValue) && isPlainObject(newValue) && hasOnlyEnumerableOwnProperties(oldValue) && hasOnlyEnumerableOwnProperties(newValue)) {
-		(/* @__PURE__ */ new Set([...Object.keys(oldValue), ...Object.keys(newValue)])).forEach((key) => appendJsonUpdateOps(ops, [...path, key], newValue[key], oldValue[key], Object.prototype.hasOwnProperty.call(oldValue, key) && oldValue[key] !== void 0));
+		for (const key of /* @__PURE__ */ new Set([...Object.keys(oldValue), ...Object.keys(newValue)])) appendJsonUpdateOps(ops, [...path, key], newValue[key], oldValue[key], Object.prototype.hasOwnProperty.call(oldValue, key) && oldValue[key] !== void 0);
 		return;
 	}
-	const op = !hasNormalizedOldValue ? normalizedNewValue === void 0 ? null : ot_json1.insertOp(path, normalizedNewValue) : normalizedNewValue === void 0 ? createJsonRemoveOp(path, normalizedOldValue) : ot_json1.replaceOp(path, normalizedOldValue, normalizedNewValue);
+	const op = !hasNormalizedOldValue ? normalizedNewValue === void 0 ? null : _univerjs_core.JSON1.insertOp(path, normalizedNewValue) : normalizedNewValue === void 0 ? createJsonRemoveOp(path, normalizedOldValue) : _univerjs_core.JSON1.replaceOp(path, normalizedOldValue, normalizedNewValue);
 	if (op && isNonEmptyOp(op)) ops.push(op);
 }
+function assertSafeDrawingKeys(...keys) {
+	for (const key of keys) if (key === "__proto__" || key === "constructor" || key === "prototype") throw new Error("Invalid drawing key");
+}
+function getOwn(record, key) {
+	return record != null && Object.prototype.hasOwnProperty.call(record, key) ? record[key] : void 0;
+}
+function validateDrawingData(data) {
+	for (const subUnitId of Object.keys(data)) {
+		assertSafeDrawingKeys(subUnitId);
+		const subUnit = data[subUnitId];
+		if (subUnit === null || subUnit === void 0 ? void 0 : subUnit.data) Object.keys(subUnit.data).forEach((key) => assertSafeDrawingKeys(key));
+		if (subUnit === null || subUnit === void 0 ? void 0 : subUnit.order) subUnit.order.forEach((key) => assertSafeDrawingKeys(key));
+	}
+}
 /**
 * unitId -> subUnitId -> drawingId -> drawingParam
 */
@@ -255,10 +240,10 @@ var UnitDrawingService = class {
 		this.refreshTransformNotification(updateParams);
 	}
 	getDrawingDataForUnit(unitId) {
-		return this.drawingManagerData[unitId] || {};
+		return getOwn(this.drawingManagerData, unitId) || {};
 	}
 	removeDrawingDataForUnit(unitId) {
-		const subUnits = this.drawingManagerData[unitId];
+		const subUnits = getOwn(this.drawingManagerData, unitId);
 		if (subUnits == null) return;
 		delete this.drawingManagerData[unitId];
 		const drawings = [];
@@ -276,11 +261,13 @@ var UnitDrawingService = class {
 		if (drawings.length > 0) this.removeNotification(drawings);
 	}
 	registerDrawingData(unitId, data) {
+		assertSafeDrawingKeys(unitId);
+		validateDrawingData(data);
 		this.drawingManagerData[unitId] = data;
 	}
 	initializeNotification(unitId) {
 		const drawings = [];
-		const data = this.drawingManagerData[unitId];
+		const data = getOwn(this.drawingManagerData, unitId);
 		if (data == null) return;
 		Object.keys(data).forEach((subUnitId) => {
 			this._establishDrawingMap(unitId, subUnitId);
@@ -299,6 +286,9 @@ var UnitDrawingService = class {
 		return this._getDrawingData(unitId, subUnitId);
 	}
 	setDrawingData(unitId, subUnitId, data) {
+		assertSafeDrawingKeys(unitId, subUnitId);
+		Object.keys(data).forEach((key) => assertSafeDrawingKeys(key));
+		this._establishDrawingMap(unitId, subUnitId);
 		const unitData = this.drawingManagerData[unitId];
 		const subUnitData = unitData[subUnitId];
 		this._oldDrawingManagerData = {
@@ -324,8 +314,8 @@ var UnitDrawingService = class {
 			ops.push(op);
 			invertOps.push(invertOp);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
-		const invertOp = invertOps.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
+		const invertOp = invertOps.reduce(_univerjs_core.JSON1.type.compose, null);
 		const { unitId, subUnitId } = insertParams[0];
 		return {
 			undo: invertOp,
@@ -420,9 +410,9 @@ var UnitDrawingService = class {
 			ops.unshift(op);
 			invertOps.push(invertOp);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: invertOps.reduce(ot_json1.type.compose, null),
+			undo: invertOps.reduce(_univerjs_core.JSON1.type.compose, null),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -454,8 +444,8 @@ var UnitDrawingService = class {
 				objects
 			};
 		}
-		const op = ops.reduce(ot_json1.type.compose, null);
-		const invertOp = invertOps.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
+		const invertOp = invertOps.reduce(_univerjs_core.JSON1.type.compose, null);
 		const { unitId, subUnitId } = updateParams[0];
 		return {
 			undo: invertOp,
@@ -492,9 +482,9 @@ var UnitDrawingService = class {
 		groupParams.forEach((groupParam) => {
 			ops.push(this._getGroupDrawingOp(groupParam));
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -507,9 +497,9 @@ var UnitDrawingService = class {
 		groupParams.forEach((groupParam) => {
 			ops.push(this._getUngroupDrawingOp(groupParam));
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -574,7 +564,7 @@ var UnitDrawingService = class {
 		const { parent, children } = groupParam;
 		const { unitId: groupUnitId, subUnitId: groupSubUnitId, drawingId: groupDrawingId } = parent;
 		const ops = [];
-		ops.push(ot_json1.insertOp([
+		ops.push(_univerjs_core.JSON1.insertOp([
 			groupUnitId,
 			groupSubUnitId,
 			"data",
@@ -596,13 +586,13 @@ var UnitDrawingService = class {
 			})));
 		});
 		if (maxChildIndex === Number.NEGATIVE_INFINITY) maxChildIndex = this._getDrawingOrder(groupUnitId, groupSubUnitId).length;
-		ops.push(ot_json1.insertOp([
+		ops.push(_univerjs_core.JSON1.insertOp([
 			groupUnitId,
 			groupSubUnitId,
 			"order",
 			maxChildIndex
 		], groupDrawingId));
-		return ops.reduce(ot_json1.type.compose, null);
+		return ops.reduce(_univerjs_core.JSON1.type.compose, null);
 	}
 	_getUngroupDrawingOp(groupParam) {
 		const { parent, children } = groupParam;
@@ -616,24 +606,41 @@ var UnitDrawingService = class {
 				drawingId
 			})));
 		});
-		ops.push(ot_json1.removeOp([
+		ops.push(_univerjs_core.JSON1.removeOp([
 			groupUnitId,
 			groupSubUnitId,
 			"data",
 			groupDrawingId
 		], true));
-		ops.push(ot_json1.removeOp([
+		ops.push(_univerjs_core.JSON1.removeOp([
 			groupUnitId,
 			groupSubUnitId,
 			"order",
 			this._getDrawingOrder(groupUnitId, groupSubUnitId).indexOf(groupDrawingId)
 		], true));
-		return ops.reduce(ot_json1.type.compose, null);
+		return ops.reduce(_univerjs_core.JSON1.type.compose, null);
 	}
 	applyJson1(unitId, subUnitId, jsonOp) {
-		this._establishDrawingMap(unitId, subUnitId);
-		this._oldDrawingManagerData = { ...this.drawingManagerData };
-		this.drawingManagerData = ot_json1.type.apply(this.drawingManagerData, jsonOp);
+		assertSafeDrawingKeys(unitId, subUnitId);
+		_univerjs_core.JSON1.type.checkValidOp(jsonOp);
+		const unitData = getOwn(this.drawingManagerData, unitId);
+		const data = {
+			...this.drawingManagerData,
+			[unitId]: {
+				...unitData,
+				[subUnitId]: getOwn(unitData, subUnitId) ?? {
+					data: {},
+					order: []
+				}
+			}
+		};
+		const nextData = _univerjs_core.JSON1.type.apply(data, jsonOp);
+		for (const id of Object.keys(nextData)) {
+			assertSafeDrawingKeys(id);
+			validateDrawingData(nextData[id]);
+		}
+		this._oldDrawingManagerData = this.drawingManagerData;
+		this.drawingManagerData = nextData;
 	}
 	featurePluginUpdateNotification(updateParams) {
 		this._featurePluginUpdate$.next(updateParams);
@@ -699,6 +706,9 @@ var UnitDrawingService = class {
 		return this._getDrawingOrder(unitId, subUnitId);
 	}
 	setDrawingOrder(unitId, subUnitId, order) {
+		assertSafeDrawingKeys(unitId, subUnitId);
+		order.forEach((key) => assertSafeDrawingKeys(key));
+		this._establishDrawingMap(unitId, subUnitId);
 		this.drawingManagerData[unitId][subUnitId].order = order;
 	}
 	orderUpdateNotification(orderParams) {
@@ -716,7 +726,7 @@ var UnitDrawingService = class {
 				drawingId
 			});
 			if (index === -1 || index === orders.length - 1) return;
-			const op = ot_json1.moveOp([
+			const op = _univerjs_core.JSON1.moveOp([
 				unitId,
 				subUnitId,
 				"order",
@@ -730,9 +740,9 @@ var UnitDrawingService = class {
 			ops.push(op);
 			if (!newIds.includes(orders[index + 1])) newIds.push(orders[index + 1]);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -754,7 +764,7 @@ var UnitDrawingService = class {
 				drawingId
 			});
 			if (index === -1 || index === 0) return;
-			const op = ot_json1.moveOp([
+			const op = _univerjs_core.JSON1.moveOp([
 				unitId,
 				subUnitId,
 				"order",
@@ -768,9 +778,9 @@ var UnitDrawingService = class {
 			ops.push(op);
 			if (!newIds.includes(orders[index - 1])) newIds.push(orders[index - 1]);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -789,7 +799,7 @@ var UnitDrawingService = class {
 		orderDrawingIds.forEach((orderDrawingId) => {
 			const { drawingId } = orderDrawingId;
 			const index = this._getDrawingCount(unitId, subUnitId) - 1;
-			const op = ot_json1.moveOp([
+			const op = _univerjs_core.JSON1.moveOp([
 				unitId,
 				subUnitId,
 				"order",
@@ -803,9 +813,9 @@ var UnitDrawingService = class {
 			ops.push(op);
 			if (!newIds.includes(orders[index])) newIds.push(orders[index]);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -823,7 +833,7 @@ var UnitDrawingService = class {
 		const ops = [];
 		orderSearchParams.forEach((orderSearchParam) => {
 			const { drawingId } = orderSearchParam;
-			const op = ot_json1.moveOp([
+			const op = _univerjs_core.JSON1.moveOp([
 				unitId,
 				subUnitId,
 				"order",
@@ -837,9 +847,9 @@ var UnitDrawingService = class {
 			ops.push(op);
 			if (!newIds.includes(orders[0])) newIds.push(orders[0]);
 		});
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
-			undo: ot_json1.type.invertWithDoc(op, this.drawingManagerData),
+			undo: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData),
 			redo: op,
 			unitId,
 			subUnitId,
@@ -857,7 +867,7 @@ var UnitDrawingService = class {
 		const currentIndex = orders.indexOf(drawingId);
 		const targetIndex = (0, _univerjs_core.normalizeDrawingOrderIndex)(zOrder, orders.length);
 		if (currentIndex < 0 || currentIndex === targetIndex) return null;
-		const op = ot_json1.moveOp([
+		const op = _univerjs_core.JSON1.moveOp([
 			unitId,
 			subUnitId,
 			"order",
@@ -868,7 +878,7 @@ var UnitDrawingService = class {
 			"order",
 			targetIndex
 		]);
-		const invertOp = ot_json1.type.invertWithDoc(op, this.drawingManagerData);
+		const invertOp = _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData);
 		const affectedDrawingIds = orders.slice(Math.min(currentIndex, targetIndex), Math.max(currentIndex, targetIndex) + 1);
 		return {
 			undo: invertOp,
@@ -903,44 +913,45 @@ var UnitDrawingService = class {
 		return this._getDrawingOrder(unitId, subUnitId).indexOf(drawingId);
 	}
 	_getCurrentBySearch(searchParam) {
-		var _this$drawingManagerD;
+		var _getOwn;
 		if (searchParam == null) return;
 		const { unitId, subUnitId, drawingId } = searchParam;
-		return (_this$drawingManagerD = this.drawingManagerData[unitId]) === null || _this$drawingManagerD === void 0 || (_this$drawingManagerD = _this$drawingManagerD[subUnitId]) === null || _this$drawingManagerD === void 0 || (_this$drawingManagerD = _this$drawingManagerD.data) === null || _this$drawingManagerD === void 0 ? void 0 : _this$drawingManagerD[drawingId];
+		return getOwn((_getOwn = getOwn(getOwn(this.drawingManagerData, unitId), subUnitId)) === null || _getOwn === void 0 ? void 0 : _getOwn.data, drawingId);
 	}
 	_getOldBySearch(searchParam) {
-		var _this$_oldDrawingMana;
+		var _getOwn2;
 		if (searchParam == null) return;
 		const { unitId, subUnitId, drawingId } = searchParam;
-		return (_this$_oldDrawingMana = this._oldDrawingManagerData[unitId]) === null || _this$_oldDrawingMana === void 0 || (_this$_oldDrawingMana = _this$_oldDrawingMana[subUnitId]) === null || _this$_oldDrawingMana === void 0 || (_this$_oldDrawingMana = _this$_oldDrawingMana.data) === null || _this$_oldDrawingMana === void 0 ? void 0 : _this$_oldDrawingMana[drawingId];
+		return getOwn((_getOwn2 = getOwn(getOwn(this._oldDrawingManagerData, unitId), subUnitId)) === null || _getOwn2 === void 0 ? void 0 : _getOwn2.data, drawingId);
 	}
 	_establishDrawingMap(unitId, subUnitId, drawingId) {
-		var _this$drawingManagerD2;
-		if (!this.drawingManagerData[unitId]) this.drawingManagerData[unitId] = {};
-		if (!this.drawingManagerData[unitId][subUnitId]) this.drawingManagerData[unitId][subUnitId] = {
+		assertSafeDrawingKeys(unitId, subUnitId);
+		if (drawingId != null) assertSafeDrawingKeys(drawingId);
+		if (!getOwn(this.drawingManagerData, unitId)) this.drawingManagerData[unitId] = {};
+		if (!getOwn(this.drawingManagerData[unitId], subUnitId)) this.drawingManagerData[unitId][subUnitId] = {
 			data: {},
 			order: []
 		};
 		if (drawingId == null) return null;
-		return (_this$drawingManagerD2 = this.drawingManagerData[unitId][subUnitId].data) === null || _this$drawingManagerD2 === void 0 ? void 0 : _this$drawingManagerD2[drawingId];
+		return getOwn(this.drawingManagerData[unitId][subUnitId].data, drawingId);
 	}
 	_addByParam(insertParam) {
 		const { unitId, subUnitId, drawingId } = insertParam;
 		this._establishDrawingMap(unitId, subUnitId, drawingId);
-		const op = [ot_json1.insertOp([
+		const op = [_univerjs_core.JSON1.insertOp([
 			unitId,
 			subUnitId,
 			"data",
 			drawingId
-		], insertParam), ot_json1.insertOp([
+		], insertParam), _univerjs_core.JSON1.insertOp([
 			unitId,
 			subUnitId,
 			"order",
 			this._getDrawingOrder(unitId, subUnitId).length
-		], drawingId)].reduce(ot_json1.type.compose, null);
+		], drawingId)].reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
 			op,
-			invertOp: ot_json1.type.invertWithDoc(op, this.drawingManagerData)
+			invertOp: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData)
 		};
 	}
 	_removeByParam(searchParam) {
@@ -953,20 +964,20 @@ var UnitDrawingService = class {
 			op: [],
 			invertOp: []
 		};
-		const op = [ot_json1.removeOp([
+		const op = [_univerjs_core.JSON1.removeOp([
 			unitId,
 			subUnitId,
 			"data",
 			drawingId
-		], true), ot_json1.removeOp([
+		], true), _univerjs_core.JSON1.removeOp([
 			unitId,
 			subUnitId,
 			"order",
 			this._getDrawingOrder(unitId, subUnitId).indexOf(drawingId)
-		], true)].reduce(ot_json1.type.compose, null);
+		], true)].reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
 			op,
-			invertOp: ot_json1.type.invertWithDoc(op, this.drawingManagerData)
+			invertOp: _univerjs_core.JSON1.type.invertWithDoc(op, this.drawingManagerData)
 		};
 	}
 	_updateByParam(updateParam) {
@@ -981,10 +992,10 @@ var UnitDrawingService = class {
 			op: [],
 			invertOp: []
 		};
-		const op = ops.reduce(ot_json1.type.compose, null);
+		const op = ops.reduce(_univerjs_core.JSON1.type.compose, null);
 		return {
 			op,
-			invertOp: ot_json1.type.invert(op)
+			invertOp: _univerjs_core.JSON1.type.invert(op)
 		};
 	}
 	_getUpdateParamCompareOp(newParam, oldParam) {
@@ -1005,12 +1016,12 @@ var UnitDrawingService = class {
 		return ops;
 	}
 	_getDrawingData(unitId, subUnitId) {
-		var _this$drawingManagerD3;
-		return ((_this$drawingManagerD3 = this.drawingManagerData[unitId]) === null || _this$drawingManagerD3 === void 0 || (_this$drawingManagerD3 = _this$drawingManagerD3[subUnitId]) === null || _this$drawingManagerD3 === void 0 ? void 0 : _this$drawingManagerD3.data) || {};
+		var _getOwn3;
+		return ((_getOwn3 = getOwn(getOwn(this.drawingManagerData, unitId), subUnitId)) === null || _getOwn3 === void 0 ? void 0 : _getOwn3.data) || {};
 	}
 	_getDrawingOrder(unitId, subUnitId) {
-		var _this$drawingManagerD4;
-		return ((_this$drawingManagerD4 = this.drawingManagerData[unitId]) === null || _this$drawingManagerD4 === void 0 || (_this$drawingManagerD4 = _this$drawingManagerD4[subUnitId]) === null || _this$drawingManagerD4 === void 0 ? void 0 : _this$drawingManagerD4.order) || [];
+		var _getOwn4;
+		return ((_getOwn4 = getOwn(getOwn(this.drawingManagerData, unitId), subUnitId)) === null || _getOwn4 === void 0 ? void 0 : _getOwn4.order) || [];
 	}
 	getDrawingVisible() {
 		return this._visible;
@@ -1131,7 +1142,7 @@ var URLImageService = class extends _univerjs_core.Disposable {
 };
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/decorateParam.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorateParam.js
 function __decorateParam(paramIndex, decorator) {
 	return function(target, key) {
 		decorator(target, key, paramIndex);
@@ -1139,7 +1150,7 @@ function __decorateParam(paramIndex, decorator) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.140.0/helpers/esm/decorate.js
+//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorate.js
 function __decorate(decorators, target, key, desc) {
 	var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
 	if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
@@ -1249,6 +1260,8 @@ const DRAWING_GROUPABLE_TYPES = [
 	_univerjs_core.DrawingTypeEnum.DRAWING_IMAGE,
 	_univerjs_core.DrawingTypeEnum.DRAWING_SHAPE,
 	_univerjs_core.DrawingTypeEnum.DRAWING_CHART,
+	_univerjs_core.DrawingTypeEnum.DRAWING_SLICER,
+	_univerjs_core.DrawingTypeEnum.DRAWING_TIMELINE,
 	_univerjs_core.DrawingTypeEnum.DRAWING_GROUP
 ];
 function isGroupableDrawingType(type) {