@univerjs/sheets-formula 0.21.0 → 0.21.1

package/lib/index.js

@@ -1,6 +1,6 @@
 import { AbsoluteRefType, BooleanNumber, BuildTextUtils, CellValueType, CommandType, DependentOn, Direction, Disposable, DisposableCollection, DrawingTypeEnum, ICommandService, IConfigService, ILogService, IUniverInstanceService, ImageSourceType, Inject, Injector, InterceptorEffectEnum, LocaleService, ObjectMatrix, ObjectRelativeFromH, ObjectRelativeFromV, Optional, Plugin, PositionedObjectLayoutType, RANGE_TYPE, Rectangle, Tools, UniverInstanceType, WrapTextType, cellToRange, createDocumentModelWithStyle, createIdentifier, generateRandomId, getIntersectRange, isFormulaId, isFormulaString, isNodeEnv, isRealNum, merge, moveRangeByOffset, sequenceExecuteAsync, toDisposable, touchDependencies } from "@univerjs/core";
 import { AUTO_FILL_APPLY_TYPE, AUTO_FILL_DATA_TYPE, ClearSelectionFormatCommand, DeleteRangeMoveLeftCommand, DeleteRangeMoveUpCommand, EffectRefRangId, IAutoFillService, INTERCEPTOR_POINT, InsertColCommand, InsertColMutation, InsertRangeMoveDownCommand, InsertRangeMoveRightCommand, InsertRowCommand, InsertRowMutation, InsertSheetMutation, InterceptCellContentPriority, MoveColsCommand, MoveColsMutation, MoveRangeCommand, MoveRangeMutation, MoveRowsCommand, MoveRowsMutation, RefRangeService, RemoveColCommand, RemoveColMutation, RemoveDefinedNameCommand, RemoveRowCommand, RemoveRowMutation, RemoveSheetCommand, RemoveSheetMutation, ReorderRangeMutation, SCOPE_WORKBOOK_VALUE_DEFINED_NAME, SetBorderCommand, SetDefinedNameCommand, SetRangeCustomMetadataCommand, SetRangeValuesCommand, SetRangeValuesMutation, SetRowHiddenMutation, SetRowVisibleMutation, SetSelectionsOperation, SetStyleCommand, SetWorksheetActiveOperation, SetWorksheetNameCommand, SheetInterceptorService, SheetsSelectionsService, UniverSheetsPlugin, alignToMergedCellsBorders, expandToContinuousRange, findFirstNonEmptyCell, getSeparateEffectedRangesOnCommand, getSheetCommandTarget, handleCommonDefaultRangeChangeWithEffectRefCommands, handleDefaultRangeChangeWithEffectRefCommands, handleDeleteRangeMoveLeft, handleDeleteRangeMoveUp, handleIRemoveCol, handleIRemoveRow, handleInsertCol, handleInsertRangeMoveDown, handleInsertRangeMoveRight, handleInsertRow, handleMoveCols, handleMoveRange, handleMoveRows, runRefRangeMutations } from "@univerjs/sheets";
-import { AstTreeBuilder, AsyncCustomFunction, CustomFunction, ENGINE_FORMULA_CYCLE_REFERENCE_COUNT, ENGINE_FORMULA_RETURN_DEPENDENCY_TREE, ErrorType, FUNCTION_NAMES_ARRAY, FUNCTION_NAMES_COMPATIBILITY, FUNCTION_NAMES_CUBE, FUNCTION_NAMES_DATABASE, FUNCTION_NAMES_DATE, FUNCTION_NAMES_ENGINEERING, FUNCTION_NAMES_FINANCIAL, FUNCTION_NAMES_INFORMATION, FUNCTION_NAMES_LOGICAL, FUNCTION_NAMES_LOOKUP, FUNCTION_NAMES_MATH, FUNCTION_NAMES_STATISTICAL, FUNCTION_NAMES_TEXT, FUNCTION_NAMES_WEB, FormulaDataModel, FormulaExecuteStageType, FormulaExecutedStateType, FunctionType, IActiveDirtyManagerService, IDefinedNamesService, IFormulaCurrentConfigService, IFunctionService, ISuperTableService, Interpreter, Lexer, LexerTreeBuilder, OtherFormulaMarkDirty, RegisterOtherFormulaService, RemoveDefinedNameMutation, RemoveSuperTableMutation, SetArrayFormulaDataMutation, SetDefinedNameMutation, SetFormulaCalculationNotificationMutation, SetFormulaCalculationResultMutation, SetFormulaCalculationStartMutation, SetFormulaCalculationStopMutation, SetFormulaDataMutation, SetFormulaStringBatchCalculationMutation, SetImageFormulaDataMutation, SetSuperTableMutation, SetTriggerFormulaCalculationStartMutation, UniverFormulaEnginePlugin, deserializeRangeWithSheetWithCache, functionArray, functionCompatibility, functionCube, functionDatabase, functionDate, functionEngineering, functionFinancial, functionInformation, functionLogical, functionLookup, functionMath, functionMeta, functionStatistical, functionText, functionUniver, functionWeb, generateExecuteAstNodeData, generateStringWithSequence, getObjectValue, initSheetFormulaData, isReferenceStrings, sequenceNodeType, serializeRange, serializeRangeToRefString, serializeRangeWithSheet, serializeRangeWithSpreadsheet, stripErrorMargin } from "@univerjs/engine-formula";
+import { ALL_IMPLEMENTED_FUNCTIONS_SET, AstTreeBuilder, AsyncCustomFunction, CustomFunction, ENGINE_FORMULA_CYCLE_REFERENCE_COUNT, ENGINE_FORMULA_RETURN_DEPENDENCY_TREE, ErrorType, FUNCTION_NAMES_ARRAY, FUNCTION_NAMES_COMPATIBILITY, FUNCTION_NAMES_CUBE, FUNCTION_NAMES_DATABASE, FUNCTION_NAMES_DATE, FUNCTION_NAMES_ENGINEERING, FUNCTION_NAMES_FINANCIAL, FUNCTION_NAMES_INFORMATION, FUNCTION_NAMES_LOGICAL, FUNCTION_NAMES_LOOKUP, FUNCTION_NAMES_MATH, FUNCTION_NAMES_STATISTICAL, FUNCTION_NAMES_TEXT, FUNCTION_NAMES_WEB, FormulaDataModel, FormulaExecuteStageType, FormulaExecutedStateType, FunctionType, IActiveDirtyManagerService, IDefinedNamesService, IFormulaCurrentConfigService, IFunctionService, ISuperTableService, Interpreter, Lexer, LexerTreeBuilder, OtherFormulaMarkDirty, RegisterOtherFormulaService, RemoveDefinedNameMutation, RemoveSuperTableMutation, SetArrayFormulaDataMutation, SetDefinedNameMutation, SetFormulaCalculationNotificationMutation, SetFormulaCalculationResultMutation, SetFormulaCalculationStartMutation, SetFormulaCalculationStopMutation, SetFormulaDataMutation, SetFormulaStringBatchCalculationMutation, SetImageFormulaDataMutation, SetSuperTableMutation, SetTriggerFormulaCalculationStartMutation, UniverFormulaEnginePlugin, deserializeRangeWithSheetWithCache, generateExecuteAstNodeData, generateStringWithSequence, getObjectValue, initSheetFormulaData, isReferenceStrings, sequenceNodeType, serializeRange, serializeRangeToRefString, serializeRangeWithSheet, serializeRangeWithSpreadsheet, stripErrorMargin } from "@univerjs/engine-formula";
 import { BehaviorSubject, map } from "rxjs";
 import { IRPCChannelService, fromModule, toModule } from "@univerjs/rpc";
 
@@ -2054,7 +2054,7 @@ let UpdateFormulaController = class UpdateFormulaController extends Disposable {
 				this._handleSetRangeValuesMutation(params);
 			}
 		}));
-		this.disposeWithMe(this._univerInstanceService.getTypeOfUnitAdded$(UniverInstanceType.UNIVER_SHEET).subscribe((unit) => this._handleWorkbookAdded(unit)));
+		this.disposeWithMe(this._univerInstanceService.getTypeOfUnitAdded$(UniverInstanceType.UNIVER_SHEET).subscribe((event) => this._handleWorkbookAdded(event.unit)));
 		this.disposeWithMe(this._univerInstanceService.getTypeOfUnitDisposed$(UniverInstanceType.UNIVER_SHEET).pipe(map((unit) => unit.getUnitId())).subscribe((unitId) => this._handleWorkbookDisposed(unitId)));
 	}
 	_handleSetRangeValuesMutation(params) {
@@ -2321,7 +2321,7 @@ UpdateFormulaController = __decorate([
 //#endregion
 //#region package.json
 var name = "@univerjs/sheets-formula";
-var version = "0.21.0";
+var version = "0.21.1";
 
 //#endregion
 //#region src/common/plugin-name.ts
@@ -15421,33 +15421,100 @@ function getFunctionName(item, localeService) {
 //#endregion
 //#region src/services/description.service.ts
 const IDescriptionService = createIdentifier("formula.description-service");
-let DescriptionService = class DescriptionService {
+let DescriptionService = class DescriptionService extends Disposable {
 	constructor(_functionService, _localeService, _configService) {
+		super();
 		this._functionService = _functionService;
 		this._localeService = _localeService;
 		this._configService = _configService;
-		_defineProperty(this, "_descriptions", []);
+		_defineProperty(this, "_descriptions", /* @__PURE__ */ new Map());
 		this._initialize();
 	}
+	_initialize() {
+		this.disposeWithMe(toDisposable(this._localeService.localeChanged$.subscribe(() => {
+			this._functionService.clearDescriptions();
+			const newDescriptions = /* @__PURE__ */ new Map();
+			this._descriptions.forEach((item) => {
+				const functionName = getFunctionName(item, this._localeService).toUpperCase();
+				newDescriptions.set(functionName, item);
+			});
+			this._descriptions = newDescriptions;
+			this._initRegisterDescriptions();
+		})));
+		this._initDescriptions();
+		this._initRegisterDescriptions();
+	}
+	_initDescriptions() {
+		var _config$description;
+		const localeService = this._localeService;
+		FUNCTION_LIST.forEach((item) => {
+			if (ALL_IMPLEMENTED_FUNCTIONS_SET.has(item.functionName)) {
+				const functionName = getFunctionName(item, localeService).toUpperCase();
+				this._descriptions.set(functionName, item);
+			}
+		});
+		const config = this._configService.getConfig(PLUGIN_CONFIG_KEY_BASE);
+		config === null || config === void 0 || (_config$description = config.description) === null || _config$description === void 0 || _config$description.forEach((item) => {
+			const functionName = getFunctionName(item, localeService).toUpperCase();
+			this._descriptions.set(functionName, item);
+		});
+	}
+	_initRegisterDescriptions() {
+		const localeService = this._localeService;
+		const functionListLocale = Array.from(this._descriptions.values()).map((functionInfo) => ({
+			functionName: getFunctionName(functionInfo, localeService),
+			functionType: functionInfo.functionType,
+			description: localeService.t(functionInfo.description),
+			abstract: localeService.t(functionInfo.abstract),
+			functionParameter: functionInfo.functionParameter.map((item) => ({
+				name: localeService.t(item.name),
+				detail: localeService.t(item.detail),
+				example: item.example,
+				require: item.require,
+				repeat: item.repeat
+			}))
+		}));
+		this._functionService.registerDescriptions(...functionListLocale);
+	}
+	_registerDescriptions(descriptions) {
+		const localeService = this._localeService;
+		const functionListLocale = descriptions.map((functionInfo) => ({
+			functionName: getFunctionName(functionInfo, localeService),
+			functionType: functionInfo.functionType,
+			description: localeService.t(functionInfo.description),
+			abstract: localeService.t(functionInfo.abstract),
+			functionParameter: functionInfo.functionParameter.map((item) => ({
+				name: localeService.t(item.name),
+				detail: localeService.t(item.detail),
+				example: item.example,
+				require: item.require,
+				repeat: item.repeat
+			}))
+		}));
+		this._functionService.registerDescriptions(...functionListLocale);
+	}
 	dispose() {
-		this._localeService.localeChanged$.complete();
+		super.dispose();
+		this._descriptions.clear();
 	}
 	getDescriptions() {
 		return this._functionService.getDescriptions();
 	}
 	hasFunction(searchText) {
-		return this._functionService.getDescriptions().get(searchText.toLocaleUpperCase()) !== void 0;
+		return this._descriptions.has(searchText.toUpperCase());
 	}
 	getFunctionInfo(searchText) {
-		return this._functionService.getDescriptions().get(searchText.toLocaleUpperCase());
+		const item = this._descriptions.get(searchText.toUpperCase());
+		if (!item) return;
+		return this._functionService.getDescription(getFunctionName(item, this._localeService));
 	}
 	getSearchListByName(searchText) {
-		const searchList = [];
 		const functionList = this._functionService.getDescriptions();
-		const _searchText = searchText.toLocaleUpperCase().trim();
+		const _searchText = searchText.toUpperCase().trim();
+		const searchList = [];
 		functionList.forEach((item) => {
 			const { functionName, abstract, functionType } = item;
-			if (functionName.toLocaleUpperCase().indexOf(_searchText) > -1 && functionType !== FunctionType.DefinedName) searchList.push({
+			if (functionName.toUpperCase().indexOf(_searchText) > -1 && functionType !== FunctionType.DefinedName) searchList.push({
 				name: functionName,
 				desc: abstract
 			});
@@ -15455,12 +15522,12 @@ let DescriptionService = class DescriptionService {
 		return searchList;
 	}
 	getSearchListByNameFirstLetter(searchText) {
-		const searchList = [];
 		const functionList = this._functionService.getDescriptions();
-		const _searchText = searchText.toLocaleUpperCase().trim();
+		const _searchText = searchText.toUpperCase().trim();
+		const searchList = [];
 		functionList.forEach((item) => {
 			const { functionName, abstract, functionType } = item;
-			if (functionName.toLocaleUpperCase().indexOf(_searchText) === 0) searchList.push({
+			if (functionName.toUpperCase().indexOf(_searchText) === 0) searchList.push({
 				name: functionName,
 				desc: abstract,
 				functionType
@@ -15469,8 +15536,9 @@ let DescriptionService = class DescriptionService {
 		return searchList;
 	}
 	getSearchListByType(type) {
+		const functionList = this._functionService.getDescriptions();
 		const searchList = [];
-		this._functionService.getDescriptions().forEach((item) => {
+		functionList.forEach((item) => {
 			const { functionName, functionType, abstract } = item;
 			if ((functionType === type || type === -1) && functionType !== FunctionType.DefinedName) searchList.push({
 				name: functionName,
@@ -15479,79 +15547,43 @@ let DescriptionService = class DescriptionService {
 		});
 		return searchList;
 	}
-	registerDescriptions(description) {
-		this._descriptions = this._descriptions.concat(description);
-		this._registerDescriptions();
+	registerDescriptions(descriptions) {
+		const localeService = this._localeService;
+		const functionNames = [];
+		descriptions.forEach((item) => {
+			const functionName = getFunctionName(item, localeService).toUpperCase();
+			functionNames.push(functionName);
+			this._descriptions.set(functionName, item);
+		});
+		this._registerDescriptions(descriptions);
 		return toDisposable(() => {
-			const functionNames = description.map((item) => item.functionName);
 			this.unregisterDescriptions(functionNames);
 		});
 	}
 	unregisterDescriptions(functionNames) {
-		this._descriptions = this._descriptions.filter((item) => !functionNames.includes(item.functionName));
-		this._functionService.unregisterDescriptions(...functionNames);
+		const removeFunctionNames = [];
+		functionNames.forEach((name) => {
+			const functionName = name.toUpperCase();
+			const item = this._descriptions.get(functionName);
+			if (!item) return;
+			removeFunctionNames.push(getFunctionName(item, this._localeService));
+			this._descriptions.delete(functionName);
+		});
+		this._functionService.unregisterDescriptions(...removeFunctionNames);
 	}
 	hasDescription(name) {
-		return this._descriptions.some((item) => item.functionName === name);
+		return this._descriptions.has(name.toUpperCase());
 	}
 	hasDefinedNameDescription(name) {
-		return this._descriptions.some((item) => item.functionName === name && item.functionType === FunctionType.DefinedName);
+		const item = this._descriptions.get(name.toUpperCase());
+		if (!item) return false;
+		return item.functionType === FunctionType.DefinedName;
 	}
 	isFormulaDefinedName(name) {
-		const items = this._descriptions.filter((item) => item.functionName === name && item.functionType === FunctionType.DefinedName);
-		if (items.length === 0) return false;
-		const token = items[0].description;
-		return !isReferenceStrings(token);
-	}
-	_initialize() {
-		this._localeService.localeChanged$.subscribe(() => {
-			this._registerDescriptions();
-		});
-		this._initDescription();
-		this._registerDescriptions();
-	}
-	_initDescription() {
-		var _config$description;
-		const functions = [
-			...functionArray,
-			...functionCompatibility,
-			...functionCube,
-			...functionDatabase,
-			...functionDate,
-			...functionEngineering,
-			...functionFinancial,
-			...functionInformation,
-			...functionLogical,
-			...functionLookup,
-			...functionMath,
-			...functionMeta,
-			...functionStatistical,
-			...functionText,
-			...functionUniver,
-			...functionWeb
-		].map((item) => item[1]);
-		const filterFunctionList = FUNCTION_LIST.filter((item) => {
-			return functions.includes(item.functionName);
-		});
-		const config = this._configService.getConfig(PLUGIN_CONFIG_KEY_BASE);
-		this._descriptions = filterFunctionList.concat((_config$description = config === null || config === void 0 ? void 0 : config.description) !== null && _config$description !== void 0 ? _config$description : []);
-	}
-	_registerDescriptions() {
-		const localeService = this._localeService;
-		const functionListLocale = this._descriptions.map((functionInfo) => ({
-			functionName: getFunctionName(functionInfo, localeService),
-			functionType: functionInfo.functionType,
-			description: localeService.t(functionInfo.description),
-			abstract: localeService.t(functionInfo.abstract),
-			functionParameter: functionInfo.functionParameter.map((item) => ({
-				name: localeService.t(item.name),
-				detail: localeService.t(item.detail),
-				example: item.example,
-				require: item.require,
-				repeat: item.repeat
-			}))
-		}));
-		this._functionService.registerDescriptions(...functionListLocale);
+		const item = this._descriptions.get(name.toUpperCase());
+		if (!item) return false;
+		if (item.functionType !== FunctionType.DefinedName) return false;
+		return !isReferenceStrings(item.description);
 	}
 };
 DescriptionService = __decorate([
@@ -15578,37 +15610,66 @@ let DefinedNameController = class DefinedNameController extends Disposable {
 		this._changeSheetListener();
 	}
 	_descriptionListener() {
-		toDisposable(this._definedNamesService.update$.subscribe(() => {
-			this._registerDescriptions();
-		}));
+		this.disposeWithMe(toDisposable(this._definedNamesService.update$.subscribe((event) => {
+			this._updateDescriptions(event);
+		})));
 	}
 	_changeUnitListener() {
-		toDisposable(this._univerInstanceService.getCurrentTypeOfUnit$(UniverInstanceType.UNIVER_SHEET).subscribe((workbook) => {
+		this.disposeWithMe(toDisposable(this._univerInstanceService.getCurrentTypeOfUnit$(UniverInstanceType.UNIVER_SHEET).subscribe((workbook) => {
 			this._unRegisterDescriptions();
-			if (workbook) this._registerDescriptions();
-		}));
+			if (workbook) this._initRegisterDescriptions(workbook.getUnitId());
+		})));
 	}
 	_changeSheetListener() {
 		this.disposeWithMe(this._commandService.onCommandExecuted((command, options) => {
 			if (options === null || options === void 0 ? void 0 : options.fromCollab) return;
 			if (command.id === SetWorksheetActiveOperation.id) {
-				this._unregisterDescriptionsForNotInSheetId();
-				this._registerDescriptions();
+				const params = command.params;
+				this._unregisterDescriptionsForNotInSheetId(params.unitId, params.subUnitId);
+				this._initRegisterDescriptions(params.unitId, params.subUnitId);
 			} else if (command.id === SetDefinedNameMutation.id) {
-				const param = command.params;
-				this._registerDescription(param);
+				const params = command.params;
+				this._registerDescription(params);
 			} else if (command.id === RemoveDefinedNameMutation.id) {
-				const param = command.params;
-				this._unregisterDescription(param);
+				const params = command.params;
+				this._unregisterDescription(params);
 			}
 		}));
 	}
-	_registerDescription(param) {
-		const target = this._getUnitIdAndSheetId(param);
+	_updateDescriptions(event) {
+		const target = getSheetCommandTarget(this._univerInstanceService);
 		if (!target) return;
-		const { sheetId } = target;
-		const { name, comment, formulaOrRefString, localSheetId } = param;
-		if (!this._descriptionService.hasDescription(name) && (localSheetId == null || localSheetId === SCOPE_WORKBOOK_VALUE_DEFINED_NAME || localSheetId === sheetId)) this._descriptionService.registerDescriptions([{
+		const { unitId, subUnitId } = target;
+		const { type, unitId: updateUnitId, definedNames } = event;
+		if (updateUnitId !== unitId) return;
+		if (type === "update") {
+			const functionList = [];
+			definedNames.forEach((definedName) => {
+				const { name, comment, formulaOrRefString, localSheetId } = definedName;
+				if (localSheetId == null || localSheetId === SCOPE_WORKBOOK_VALUE_DEFINED_NAME || localSheetId === subUnitId) functionList.push({
+					functionName: name,
+					description: formulaOrRefString + (comment || ""),
+					abstract: formulaOrRefString,
+					functionType: FunctionType.DefinedName,
+					functionParameter: []
+				});
+			});
+			this._descriptionService.registerDescriptions(functionList);
+		} else if (type === "remove") {
+			const functionList = [];
+			definedNames.forEach((definedName) => {
+				functionList.push(definedName.name);
+			});
+			this._descriptionService.unregisterDescriptions(functionList);
+		}
+	}
+	_registerDescription(params) {
+		const target = getSheetCommandTarget(this._univerInstanceService, params);
+		if (!target) return;
+		const { subUnitId } = target;
+		const { name, comment, formulaOrRefString, localSheetId } = params;
+		if (this._descriptionService.hasDescription(name)) return;
+		if (localSheetId == null || localSheetId === SCOPE_WORKBOOK_VALUE_DEFINED_NAME || localSheetId === subUnitId) this._descriptionService.registerDescriptions([{
 			functionName: name,
 			description: formulaOrRefString + (comment || ""),
 			abstract: formulaOrRefString,
@@ -15621,9 +15682,9 @@ let DefinedNameController = class DefinedNameController extends Disposable {
 		this._descriptionService.unregisterDescriptions([name]);
 	}
 	_unRegisterDescriptions() {
-		if (this._preUnitId == null) return;
+		if (this._preUnitId === null) return;
 		const definedNames = this._definedNamesService.getDefinedNameMap(this._preUnitId);
-		if (definedNames == null) return;
+		if (!definedNames) return;
 		const functionList = [];
 		Array.from(Object.values(definedNames)).forEach((value) => {
 			const { name } = value;
@@ -15632,28 +15693,21 @@ let DefinedNameController = class DefinedNameController extends Disposable {
 		this._descriptionService.unregisterDescriptions(functionList);
 		this._preUnitId = null;
 	}
-	_getUnitIdAndSheetId(params = {}) {
-		const { unitId, subUnitId } = params;
-		const workbook = unitId ? this._univerInstanceService.getUnit(unitId, UniverInstanceType.UNIVER_SHEET) : this._univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
-		if (!workbook) return null;
-		const worksheet = subUnitId ? workbook.getSheetBySheetId(subUnitId) : workbook.getActiveSheet(true);
-		if (!worksheet) return null;
-		return {
-			unitId: workbook.getUnitId(),
-			sheetId: worksheet.getSheetId()
-		};
-	}
-	_registerDescriptions() {
-		const target = this._getUnitIdAndSheetId();
+	_initRegisterDescriptions(unitId, subUnitId) {
+		const target = getSheetCommandTarget(this._univerInstanceService, {
+			unitId,
+			subUnitId
+		});
 		if (!target) return;
-		const { unitId, sheetId } = target;
-		const definedNames = this._definedNamesService.getDefinedNameMap(unitId);
+		const { unitId: _unitId, subUnitId: _subUnitId } = target;
+		const definedNames = this._definedNamesService.getDefinedNameMap(_unitId);
 		if (!definedNames) return;
 		const functionList = [];
-		this._preUnitId = unitId;
+		this._preUnitId = _unitId;
 		Array.from(Object.values(definedNames)).forEach((value) => {
 			const { name, comment, formulaOrRefString, localSheetId } = value;
-			if (!this._descriptionService.hasDescription(name) && (localSheetId == null || localSheetId === SCOPE_WORKBOOK_VALUE_DEFINED_NAME || localSheetId === sheetId)) functionList.push({
+			if (this._descriptionService.hasDescription(name)) return;
+			if (localSheetId == null || localSheetId === SCOPE_WORKBOOK_VALUE_DEFINED_NAME || localSheetId === _subUnitId) functionList.push({
 				functionName: name,
 				description: formulaOrRefString + (comment || ""),
 				abstract: formulaOrRefString,
@@ -15663,16 +15717,13 @@ let DefinedNameController = class DefinedNameController extends Disposable {
 		});
 		this._descriptionService.registerDescriptions(functionList);
 	}
-	_unregisterDescriptionsForNotInSheetId() {
-		const target = this._getUnitIdAndSheetId();
-		if (!target) return;
-		const { unitId, sheetId } = target;
+	_unregisterDescriptionsForNotInSheetId(unitId, subUnitId) {
 		const definedNames = this._definedNamesService.getDefinedNameMap(unitId);
 		if (!definedNames) return;
 		const functionList = [];
 		Array.from(Object.values(definedNames)).forEach((value) => {
 			const { name, localSheetId } = value;
-			if (localSheetId !== SCOPE_WORKBOOK_VALUE_DEFINED_NAME && localSheetId !== sheetId) functionList.push(name);
+			if (localSheetId !== SCOPE_WORKBOOK_VALUE_DEFINED_NAME && localSheetId !== subUnitId) functionList.push(name);
 		});
 		this._descriptionService.unregisterDescriptions(functionList);
 	}
@@ -16179,19 +16230,16 @@ let RegisterFunctionService = class RegisterFunctionService extends Disposable {
 		const { locales, description, calculate } = params;
 		if (locales) this._localeService.load(locales);
 		const disposables = new DisposableCollection();
-		if (description) disposables.add(this._descriptionService.registerDescriptions(description));
-		else {
-			const descriptionList = calculate.map(([_func, functionName, functionIntroduction]) => {
-				return {
-					functionName,
-					functionType: FunctionType.User,
-					description: "",
-					abstract: functionIntroduction || "",
-					functionParameter: []
-				};
-			});
-			disposables.add(this._functionService.registerDescriptions(...descriptionList));
-		}
+		const descriptions = description !== null && description !== void 0 ? description : calculate.map(([_func, functionName, functionIntroduction]) => {
+			return {
+				functionName,
+				functionType: FunctionType.User,
+				description: "",
+				abstract: functionIntroduction || "",
+				functionParameter: []
+			};
+		});
+		disposables.add(this._descriptionService.registerDescriptions(descriptions));
 		disposables.add(this._registerLocalExecutors(calculate));
 		if (this._remoteRegisterFunctionService) disposables.add(this._registerRemoteExecutors(calculate));
 		return disposables;

package/lib/types/controllers/defined-name.controller.d.ts

@@ -31,10 +31,10 @@ export declare class DefinedNameController extends Disposable {
     private _descriptionListener;
     private _changeUnitListener;
     private _changeSheetListener;
+    private _updateDescriptions;
     private _registerDescription;
     private _unregisterDescription;
     private _unRegisterDescriptions;
-    private _getUnitIdAndSheetId;
-    private _registerDescriptions;
+    private _initRegisterDescriptions;
     private _unregisterDescriptionsForNotInSheetId;
 }

package/lib/types/services/description.service.d.ts

@@ -15,7 +15,7 @@
  */
 import type { IDisposable } from '@univerjs/core';
 import type { IFunctionInfo, IFunctionNames } from '@univerjs/engine-formula';
-import { IConfigService, LocaleService } from '@univerjs/core';
+import { Disposable, IConfigService, LocaleService } from '@univerjs/core';
 import { FunctionType, IFunctionService } from '@univerjs/engine-formula';
 export interface ISearchItem {
     name: string;
@@ -32,60 +32,52 @@ export interface IDescriptionService {
     hasFunction(searchText: string): boolean;
     /**
      * get function info by name
-     * @param searchText
      */
     getFunctionInfo(searchText: string): IFunctionInfo | undefined;
     /**
      * get search list by name
-     * @param searchText
-     * @returns
      */
     getSearchListByName(searchText: string): ISearchItem[];
     /**
      * get search list by name, from first letter
-     * @param searchText
-     * @returns
      */
     getSearchListByNameFirstLetter(searchText: string): ISearchItemWithType[];
     /**
      * get search list by type, if type is -1, return all
-     * @param type
-     * @returns
      */
     getSearchListByType(type: number): ISearchItem[];
     /**
      * register descriptions
-     * @param functionList
      */
     registerDescriptions(functionList: IFunctionInfo[]): IDisposable;
     /**
      * unregister descriptions
-     * @param functionList
      */
     unregisterDescriptions(functionNames: string[]): void;
     /**
      * check if has description
-     * @param name
      */
     hasDescription(name: string): boolean;
     /**
      * check if has defined name description
-     * @param name
      */
     hasDefinedNameDescription(name: string): boolean;
     /**
      * check if is formula defined name
-     * @param name
      */
     isFormulaDefinedName(name: string): boolean;
 }
 export declare const IDescriptionService: import("@wendellhu/redi").IdentifierDecorator<IDescriptionService>;
-export declare class DescriptionService implements IDescriptionService, IDisposable {
+export declare class DescriptionService extends Disposable implements IDescriptionService {
     private readonly _functionService;
     private readonly _localeService;
     private readonly _configService;
     private _descriptions;
     constructor(_functionService: IFunctionService, _localeService: LocaleService, _configService: IConfigService);
+    private _initialize;
+    private _initDescriptions;
+    private _initRegisterDescriptions;
+    private _registerDescriptions;
     dispose(): void;
     getDescriptions(): Map<IFunctionNames, IFunctionInfo>;
     hasFunction(searchText: string): boolean;
@@ -93,12 +85,9 @@ export declare class DescriptionService implements IDescriptionService, IDisposa
     getSearchListByName(searchText: string): ISearchItem[];
     getSearchListByNameFirstLetter(searchText: string): ISearchItemWithType[];
     getSearchListByType(type: number): ISearchItem[];
-    registerDescriptions(description: IFunctionInfo[]): IDisposable;
+    registerDescriptions(descriptions: IFunctionInfo[]): IDisposable;
     unregisterDescriptions(functionNames: string[]): void;
     hasDescription(name: string): boolean;
     hasDefinedNameDescription(name: string): boolean;
     isFormulaDefinedName(name: string): boolean;
-    private _initialize;
-    private _initDescription;
-    private _registerDescriptions;
 }