@elementor/editor-variables 4.0.0-532 → 4.0.0-533

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elementor/editor-variables",
-  "version": "4.0.0-532",
+  "version": "4.0.0-533",
   "private": false,
   "author": "Elementor Team",
   "homepage": "https://elementor.com/",
@@ -39,22 +39,22 @@
     "dev": "tsup --config=../../tsup.dev.ts"
   },
   "dependencies": {
-    "@elementor/editor": "4.0.0-532",
-    "@elementor/editor-canvas": "4.0.0-532",
-    "@elementor/editor-controls": "4.0.0-532",
-    "@elementor/editor-current-user": "4.0.0-532",
-    "@elementor/editor-mcp": "4.0.0-532",
-    "@elementor/editor-panels": "4.0.0-532",
-    "@elementor/editor-props": "4.0.0-532",
-    "@elementor/editor-ui": "4.0.0-532",
-    "@elementor/editor-v1-adapters": "4.0.0-532",
-    "@elementor/menus": "4.0.0-532",
-    "@elementor/http-client": "4.0.0-532",
+    "@elementor/editor": "4.0.0-533",
+    "@elementor/editor-canvas": "4.0.0-533",
+    "@elementor/editor-controls": "4.0.0-533",
+    "@elementor/editor-current-user": "4.0.0-533",
+    "@elementor/editor-mcp": "4.0.0-533",
+    "@elementor/editor-panels": "4.0.0-533",
+    "@elementor/editor-props": "4.0.0-533",
+    "@elementor/editor-ui": "4.0.0-533",
+    "@elementor/editor-v1-adapters": "4.0.0-533",
+    "@elementor/menus": "4.0.0-533",
+    "@elementor/http-client": "4.0.0-533",
     "@elementor/icons": "^1.63.0",
-    "@elementor/mixpanel": "4.0.0-532",
-    "@elementor/schema": "4.0.0-532",
+    "@elementor/mixpanel": "4.0.0-533",
+    "@elementor/schema": "4.0.0-533",
     "@elementor/ui": "1.37.2",
-    "@elementor/utils": "4.0.0-532",
+    "@elementor/utils": "4.0.0-533",
     "@wordpress/i18n": "^5.13.0"
   },
   "peerDependencies": {

package/src/batch-operations.ts

@@ -22,7 +22,8 @@ export const isTempId = ( id: string ): boolean => {
 
 export const buildOperationsArray = (
 	originalVariables: TVariablesList,
-	currentVariables: TVariablesList
+	currentVariables: TVariablesList,
+	deletedVariables: string[]
 ): BatchOperation[] => {
 	const operations: BatchOperation[] = [];
 
@@ -51,6 +52,7 @@ export const buildOperationsArray = (
 				( original.label !== variable.label ||
 					original.value !== variable.value ||
 					original.order !== variable.order ||
+					original.type !== variable.type ||
 					syncChanged )
 			) {
 				operations.push( {
@@ -60,6 +62,7 @@ export const buildOperationsArray = (
 						...( original.label !== variable.label && { label: variable.label } ),
 						...( original.value !== variable.value && { value: variable.value } ),
 						...( original.order !== variable.order && { order: variable.order } ),
+						...( original.type !== variable.type && { type: variable.type } ),
 						...( syncChanged && { sync_to_v3: variable.sync_to_v3 } ),
 					},
 				} );
@@ -67,19 +70,11 @@ export const buildOperationsArray = (
 		}
 	} );
 
-	Object.entries( originalVariables ).forEach( ( [ id, variable ] ) => {
-		if ( isTempId( id ) || variable.deleted ) {
-			return;
-		}
-
-		const currentVariable = currentVariables[ id ];
-
-		if ( ! currentVariable || currentVariable.deleted ) {
-			operations.push( {
-				type: 'delete',
-				id,
-			} );
-		}
+	deletedVariables.forEach( ( id: string ) => {
+		operations.push( {
+			type: 'delete',
+			id,
+		} );
 	} );
 
 	return operations.filter( ( op ) => {

package/src/components/variables-manager/hooks/use-variables-manager-state.ts

@@ -68,7 +68,7 @@ export const useVariablesManagerState = () => {
 	const handleSave = useCallback( async (): Promise< { success: boolean } > => {
 		const originalVariables = getVariables( false );
 		setIsSaving( true );
-		const result = await service.batchSave( originalVariables, variables );
+		const result = await service.batchSave( originalVariables, variables, deletedVariables );
 
 		if ( result.success ) {
 			await service.load();
@@ -80,7 +80,7 @@ export const useVariablesManagerState = () => {
 		}
 
 		return { success: result.success };
-	}, [ variables ] );
+	}, [ variables, deletedVariables ] );
 
 	const filteredVariables = useCallback( () => {
 		const list = variablesToList( variables ).filter( ( v ) => ! v.deleted );

package/src/mcp/index.ts

@@ -1,7 +1,12 @@
+import { isAngieAvailable } from '@elementor/editor-mcp';
+
 import { initManageVariableTool } from './manage-variable-tool';
 import { initVariablesResource } from './variables-resource';
 
 export function initMcp() {
+	if ( ! isAngieAvailable() ) {
+		return;
+	}
 	initManageVariableTool();
 	initVariablesResource();
 }

package/src/service.ts

@@ -186,8 +186,8 @@ export const service = {
 			} );
 	},
 
-	batchSave: ( originalVariables: TVariablesList, currentVariables: TVariablesList ) => {
-		const operations = buildOperationsArray( originalVariables, currentVariables );
+	batchSave: ( originalVariables: TVariablesList, currentVariables: TVariablesList, deletedVariables: string[] ) => {
+		const operations = buildOperationsArray( originalVariables, currentVariables, deletedVariables );
 		const batchPayload = { operations, watermark: storage.state.watermark };
 
 		if ( operations.length === 0 ) {