@univerjs/sheets-thread-comment 0.24.0 → 0.25.0-insiders.20260608-e4336f7

package/lib/facade.js

@@ -6,7 +6,7 @@ import { deserializeRangeWithSheet } from "@univerjs/engine-formula";
 import { filter } from "rxjs";
 import { FEventName, FUniver } from "@univerjs/core/facade";
 
-//#region \0@oxc-project+runtime@0.129.0/helpers/typeof.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -17,7 +17,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/toPrimitive.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -30,14 +30,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.134.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.129.0/helpers/defineProperty.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -48,7 +48,7 @@ function _defineProperty(e, r, t) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/decorateParam.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/decorateParam.js
 function __decorateParam(paramIndex, decorator) {
 	return function(target, key) {
 		decorator(target, key, paramIndex);
@@ -56,7 +56,7 @@ function __decorateParam(paramIndex, decorator) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/decorate.js
+//#region \0@oxc-project+runtime@0.134.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);
@@ -194,7 +194,8 @@ var FTheadCommentBuilder = class FTheadCommentBuilder extends FTheadCommentItem
 	*
 	* // Add the comment to the cell A1
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const cell = fWorksheet.getRange('A1');
 	* const result = await cell.addCommentAsync(commentBuilder);
 	* console.log(result);
@@ -220,7 +221,8 @@ var FTheadCommentBuilder = class FTheadCommentBuilder extends FTheadCommentItem
 	*
 	* // Add the comment to the cell A1
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const cell = fWorksheet.getRange('A1');
 	* const result = await cell.addCommentAsync(commentBuilder);
 	* console.log(result);
@@ -245,7 +247,8 @@ var FTheadCommentBuilder = class FTheadCommentBuilder extends FTheadCommentItem
 	*
 	* // Add the comment to the cell A1
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const cell = fWorksheet.getRange('A1');
 	* const result = await cell.addCommentAsync(commentBuilder);
 	* console.log(result);
@@ -270,7 +273,8 @@ var FTheadCommentBuilder = class FTheadCommentBuilder extends FTheadCommentItem
 	*
 	* // Add the comment to the cell A1
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const cell = fWorksheet.getRange('A1');
 	* const result = await cell.addCommentAsync(commentBuilder);
 	* console.log(result);
@@ -295,7 +299,8 @@ var FTheadCommentBuilder = class FTheadCommentBuilder extends FTheadCommentItem
 	*
 	* // Add the comment to the cell A1
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const cell = fWorksheet.getRange('A1');
 	* const result = await cell.addCommentAsync(commentBuilder);
 	* console.log(result);
@@ -348,7 +353,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	* comments.forEach((comment) => {
 	*   console.log(comment.getIsRoot());
@@ -364,7 +370,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	* comments.forEach((comment) => {
 	*   console.log(comment.getCommentData());
@@ -381,7 +388,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	* comments.forEach((comment) => {
 	*   if (comment.getIsRoot()) {
@@ -405,7 +413,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	* comments.forEach((comment) => {
 	*   console.log(comment.getRange().getA1Notation());
@@ -432,7 +441,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	* comments.forEach((comment) => {
 	*   console.log(comment.getRichText());
@@ -453,7 +463,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	* const comments = fWorksheet.getComments();
 	*
 	* // Delete the first comment
@@ -487,7 +498,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	*
 	* // Create a new comment
 	* const richText = univerAPI.newRichText().insertText('hello univer');
@@ -533,7 +545,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	*
 	* // Create a new comment
 	* const richText = univerAPI.newRichText().insertText('hello univer');
@@ -566,7 +579,8 @@ let FThreadComment = _FThreadComment = class FThreadComment {
 	* @example
 	* ```ts
 	* const fWorkbook = univerAPI.getActiveWorkbook();
-	* const fWorksheet = fWorkbook.getActiveSheet();
+	* const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+	* if (!fWorksheet) return;
 	*
 	* // Create a new comment
 	* const richText = univerAPI.newRichText().insertText('hello univer');
@@ -843,26 +857,16 @@ FEventName.extend(FSheetsThreadCommentEventNameMixin);
 * @ignore
 */
 var FUniverSheetsThreadCommentMixin = class extends FUniver {
-	_getTargetSheet(params = {}) {
-		var _this$getActiveWorkbo;
-		const workbook = params.unitId ? this.getUniverSheet(params.unitId) : (_this$getActiveWorkbo = this.getActiveWorkbook) === null || _this$getActiveWorkbo === void 0 ? void 0 : _this$getActiveWorkbo.call(this);
-		if (!workbook) return null;
-		const worksheet = params.subUnitId ? workbook.getSheetBySheetId(params.subUnitId) : workbook.getActiveSheet();
-		if (!worksheet) return null;
-		return {
-			workbook,
-			worksheet
-		};
-	}
 	_initialize(injector) {
 		const commandService = injector.get(ICommandService);
 		this.disposeWithMe(this.registerEventHandler(this.Event.CommentAdded, () => commandService.onCommandExecuted((commandInfo) => {
 			var _threadComment$getRan, _threadComment$getRan2, _threadComment$getRan3, _threadComment$getRan4;
 			if (commandInfo.id !== AddCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { comment } = commandInfo.params;
+			const { comment } = params;
 			const threadComment = worksheet.getCommentById(comment.id);
 			if (!threadComment) return;
 			const eventParams = {
@@ -877,10 +881,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		this.disposeWithMe(this.registerEventHandler(this.Event.CommentUpdated, () => commandService.onCommandExecuted((commandInfo) => {
 			var _threadComment$getRan5, _threadComment$getRan6, _threadComment$getRan7, _threadComment$getRan8;
 			if (commandInfo.id !== UpdateCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { payload } = commandInfo.params;
+			const { payload } = params;
 			const threadComment = worksheet.getCommentById(payload.commentId);
 			if (!threadComment) return;
 			const eventParams = {
@@ -894,10 +899,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		})));
 		this.disposeWithMe(this.registerEventHandler(this.Event.CommentDeleted, () => commandService.onCommandExecuted((commandInfo) => {
 			if (commandInfo.id !== DeleteCommentCommand.id && commandInfo.id !== DeleteCommentTreeCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { commentId } = commandInfo.params;
+			const { commentId } = params;
 			const eventParams = {
 				workbook,
 				worksheet,
@@ -908,10 +914,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		this.disposeWithMe(this.registerEventHandler(this.Event.CommentResolved, () => commandService.onCommandExecuted((commandInfo) => {
 			var _threadComment$getRan9, _threadComment$getRan10, _threadComment$getRan11, _threadComment$getRan12;
 			if (commandInfo.id !== ResolveCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { commentId, resolved } = commandInfo.params;
+			const { commentId, resolved } = params;
 			const threadComment = worksheet.getCommentById(commentId);
 			if (!threadComment) return;
 			const eventParams = {
@@ -926,10 +933,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		})));
 		this.disposeWithMe(this.registerEventHandler(this.Event.BeforeCommentAdd, () => commandService.beforeCommandExecuted((commandInfo) => {
 			if (commandInfo.id !== AddCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { comment } = commandInfo.params;
+			const { comment } = params;
 			const { range } = deserializeRangeWithSheet(comment.ref);
 			const eventParams = {
 				workbook,
@@ -944,10 +952,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		this.disposeWithMe(this.registerEventHandler(this.Event.BeforeCommentUpdate, () => commandService.beforeCommandExecuted((commandInfo) => {
 			var _threadComment$getRan13, _threadComment$getRan14, _threadComment$getRan15, _threadComment$getRan16;
 			if (commandInfo.id !== UpdateCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { payload } = commandInfo.params;
+			const { payload } = params;
 			const threadComment = worksheet.getCommentById(payload.commentId);
 			if (!threadComment) return;
 			const eventParams = {
@@ -964,10 +973,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		this.disposeWithMe(this.registerEventHandler(this.Event.BeforeCommentDelete, () => commandService.beforeCommandExecuted((commandInfo) => {
 			var _threadComment$getRan17, _threadComment$getRan18, _threadComment$getRan19, _threadComment$getRan20;
 			if (commandInfo.id !== DeleteCommentCommand.id && commandInfo.id !== DeleteCommentTreeCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { commentId } = commandInfo.params;
+			const { commentId } = params;
 			const threadComment = worksheet.getCommentById(commentId);
 			if (!threadComment) return;
 			const eventParams = {
@@ -983,10 +993,11 @@ var FUniverSheetsThreadCommentMixin = class extends FUniver {
 		this.disposeWithMe(this.registerEventHandler(this.Event.BeforeCommentResolve, () => commandService.beforeCommandExecuted((commandInfo) => {
 			var _getRow, _getColumn;
 			if (commandInfo.id !== ResolveCommentCommand.id) return;
-			const target = this._getTargetSheet(commandInfo.params);
+			const params = commandInfo.params;
+			const target = this.getSheetCommandTarget(params);
 			if (!target) return;
 			const { workbook, worksheet } = target;
-			const { commentId, resolved } = commandInfo.params;
+			const { commentId, resolved } = params;
 			const threadComment = worksheet.getCommentById(commentId);
 			if (!threadComment) return;
 			const eventParams = {

package/lib/index.js

@@ -4,7 +4,7 @@ import { CopySheetCommand, RefRangeService, RemoveSheetCommand, SheetInterceptor
 import { AddCommentMutation, DeleteCommentMutation, IThreadCommentDataSourceService, ThreadCommentModel, UniverThreadCommentPlugin, UpdateCommentRefMutation } from "@univerjs/thread-comment";
 import { Subject } from "rxjs";
 
-//#region \0@oxc-project+runtime@0.129.0/helpers/typeof.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/typeof.js
 function _typeof(o) {
 	"@babel/helpers - typeof";
 	return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function(o) {
@@ -15,7 +15,7 @@ function _typeof(o) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/toPrimitive.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/toPrimitive.js
 function toPrimitive(t, r) {
 	if ("object" != _typeof(t) || !t) return t;
 	var e = t[Symbol.toPrimitive];
@@ -28,14 +28,14 @@ function toPrimitive(t, r) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/toPropertyKey.js
+//#region \0@oxc-project+runtime@0.134.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.129.0/helpers/defineProperty.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/defineProperty.js
 function _defineProperty(e, r, t) {
 	return (r = toPropertyKey(r)) in e ? Object.defineProperty(e, r, {
 		value: t,
@@ -46,7 +46,7 @@ function _defineProperty(e, r, t) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/decorateParam.js
+//#region \0@oxc-project+runtime@0.134.0/helpers/esm/decorateParam.js
 function __decorateParam(paramIndex, decorator) {
 	return function(target, key) {
 		decorator(target, key, paramIndex);
@@ -54,7 +54,7 @@ function __decorateParam(paramIndex, decorator) {
 }
 
 //#endregion
-//#region \0@oxc-project+runtime@0.129.0/helpers/decorate.js
+//#region \0@oxc-project+runtime@0.134.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);
@@ -449,7 +449,7 @@ SheetsThreadCommentRefRangeController = __decorate([
 //#endregion
 //#region package.json
 var name = "@univerjs/sheets-thread-comment";
-var version = "0.24.0";
+var version = "0.25.0-insiders.20260608-e4336f7";
 
 //#endregion
 //#region src/config/config.ts

package/lib/types/facade/f-range.d.ts

@@ -25,9 +25,9 @@ export interface IFRangeSheetsThreadCommentMixin {
      * @returns {FThreadComment | null} The comment of the start cell in the current range. If the cell does not have a comment, return `null`.
      * @example
      * ```ts
-     * const range = univerAPI.getActiveWorkbook()
-     *  .getActiveSheet()
-     *  .getActiveRange();
+     * const fWorksheet = univerAPI.getActiveWorkbook().getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
+     * const range = fWorksheet.getActiveRange();
      * const comment = range.getComment();
      * ```
      */
@@ -37,9 +37,9 @@ export interface IFRangeSheetsThreadCommentMixin {
      * @returns {FThreadComment[]} The comments in the current range.
      * @example
      * ```ts
-     * const range = univerAPI.getActiveWorkbook()
-     *  .getActiveSheet()
-     *  .getActiveRange();
+     * const fWorksheet = univerAPI.getActiveWorkbook().getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
+     * const range = fWorksheet.getActiveRange();
      * const comments = range.getComments();
      * comments.forEach((comment) => {
      *   console.log(comment.getContent());
@@ -65,7 +65,8 @@ export interface IFRangeSheetsThreadCommentMixin {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -90,9 +91,9 @@ export interface IFRangeSheetsThreadCommentMixin {
      * @returns Whether the comments are cleared successfully.
      * @example
      * ```ts
-     * const range = univerAPI.getActiveWorkbook()
-     *  .getActiveSheet()
-     *  .getActiveRange();
+     * const fWorksheet = univerAPI.getActiveWorkbook().getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
+     * const range = fWorksheet.getActiveRange();
      * const success = await range.clearCommentsAsync();
      * ```
      */

package/lib/types/facade/f-thread-comment.d.ts

@@ -117,7 +117,8 @@ export declare class FTheadCommentBuilder extends FTheadCommentItem {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -139,7 +140,8 @@ export declare class FTheadCommentBuilder extends FTheadCommentItem {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -161,7 +163,8 @@ export declare class FTheadCommentBuilder extends FTheadCommentItem {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -183,7 +186,8 @@ export declare class FTheadCommentBuilder extends FTheadCommentItem {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -205,7 +209,8 @@ export declare class FTheadCommentBuilder extends FTheadCommentItem {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -252,7 +257,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   console.log(comment.getIsRoot());
@@ -266,7 +272,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   console.log(comment.getCommentData());
@@ -280,7 +287,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   if (comment.getIsRoot()) {
@@ -299,7 +307,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   console.log(comment.getRange().getA1Notation());
@@ -317,7 +326,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   console.log(comment.getRichText());
@@ -331,7 +341,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      *
      * // Delete the first comment
@@ -355,7 +366,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      *
      * // Create a new comment
      * const richText = univerAPI.newRichText().insertText('hello univer');
@@ -386,7 +398,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      *
      * // Create a new comment
      * const richText = univerAPI.newRichText().insertText('hello univer');
@@ -412,7 +425,8 @@ export declare class FThreadComment {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      *
      * // Create a new comment
      * const richText = univerAPI.newRichText().insertText('hello univer');

package/lib/types/facade/f-univer.d.ts

@@ -37,7 +37,8 @@ export interface IFUniverSheetsThreadCommentMixin {
      *
      * // Add the comment to the cell A1
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const cell = fWorksheet.getRange('A1');
      * const result = await cell.addCommentAsync(commentBuilder);
      * console.log(result);
@@ -49,7 +50,6 @@ export interface IFUniverSheetsThreadCommentMixin {
  * @ignore
  */
 export declare class FUniverSheetsThreadCommentMixin extends FUniver implements IFUniverSheetsThreadCommentMixin {
-    private _getTargetSheet;
     _initialize(injector: Injector): void;
     /**
      * @ignore

package/lib/types/facade/f-workbook.d.ts

@@ -14,11 +14,10 @@
  * limitations under the License.
  */
 import type { IDisposable, IExecutionOptions } from '@univerjs/core';
-import type { CommentUpdate, IAddCommentCommandParams, IDeleteCommentCommandParams } from '@univerjs/thread-comment';
+import type { CommentUpdate, IAddCommentCommandParams, IDeleteCommentCommandParams, IUpdateCommentCommandParams } from '@univerjs/thread-comment';
 import { FWorkbook } from '@univerjs/sheets/facade';
 import { ThreadCommentModel } from '@univerjs/thread-comment';
 import { FThreadComment } from './f-thread-comment';
-type IUpdateCommandParams = any;
 /**
  * @ignore
  */
@@ -67,7 +66,7 @@ export interface IFWorkbookSheetsThreadCommentMixin {
     /**
      * @deprecated use `univerAPI.addEvent(univerAPI.Event.BeforeCommentUpdate, (params) => {})` as instead
      */
-    onBeforeUpdateThreadComment(this: FWorkbook, callback: (params: IUpdateCommandParams, options: IExecutionOptions | undefined) => void | false): IDisposable;
+    onBeforeUpdateThreadComment(this: FWorkbook, callback: (params: IUpdateCommentCommandParams, options: IExecutionOptions | undefined) => void | false): IDisposable;
     /**
      * @deprecated use `univerAPI.addEvent(univerAPI.Event.BeforeCommentDelete, (params) => {})` as instead
      */
@@ -98,7 +97,7 @@ export declare class FWorkbookSheetsThreadCommentMixin extends FWorkbook impleme
      * @param callback
      * @deprecated
      */
-    onBeforeUpdateThreadComment(callback: (params: IUpdateCommandParams, options: IExecutionOptions | undefined) => void | false): IDisposable;
+    onBeforeUpdateThreadComment(callback: (params: IUpdateCommentCommandParams, options: IExecutionOptions | undefined) => void | false): IDisposable;
     /**
      * @param callback
      * @deprecated
@@ -109,4 +108,3 @@ declare module '@univerjs/sheets/facade' {
     interface FWorkbook extends IFWorkbookSheetsThreadCommentMixin {
     }
 }
-export {};

package/lib/types/facade/f-worksheet.d.ts

@@ -27,7 +27,8 @@ export interface IFWorksheetCommentMixin {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const comments = fWorksheet.getComments();
      * comments.forEach((comment) => {
      *   const isRoot = comment.getIsRoot();
@@ -50,7 +51,8 @@ export interface IFWorksheetCommentMixin {
      * @example
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      * const result = await fWorksheet.clearComments();
      * console.log(result);
      * ```
@@ -61,7 +63,8 @@ export interface IFWorksheetCommentMixin {
      * @param {string} commentId comment id
      * ```ts
      * const fWorkbook = univerAPI.getActiveWorkbook();
-     * const fWorksheet = fWorkbook.getActiveSheet();
+     * const fWorksheet = fWorkbook.getSheetByName('Sheet1');
+     * if (!fWorksheet) return;
      *
      * // Create a new comment
      * const richText = univerAPI.newRichText().insertText('hello univer');