@elementor/editor-canvas 4.3.0-1004 → 4.3.0-1005

package/dist/index.js

@@ -4725,6 +4725,20 @@ var LOCAL_STYLE_META = {
   breakpoint: "desktop",
   state: null
 };
+var UnsupportedPropertyError = class extends Error {
+  elementType;
+  propertyName;
+  constructor(elementType, propertyName, availableProperties) {
+    super(
+      `Property "${propertyName}" does not exist on element type "${elementType}". Available properties are: ${availableProperties.join(
+        ", "
+      )}`
+    );
+    this.name = "UnsupportedPropertyError";
+    this.elementType = elementType;
+    this.propertyName = propertyName;
+  }
+};
 function resolvePropValue(value, forceKey) {
   const Utils = window.elementorV2.editorVariables.Utils;
   return import_editor_props6.Schema.adjustLlmPropValueSchema(value, {
@@ -4830,12 +4844,7 @@ var doUpdateElementProperty = (params) => {
     throw new Error(`No prop schema found for element type: ${elementType}`);
   }
   if (!elementPropSchema[propertyName]) {
-    const propertyNames = Object.keys(elementPropSchema);
-    throw new Error(
-      `Property "${propertyName}" does not exist on element type "${elementType}". Available properties are: ${propertyNames.join(
-        ", "
-      )}`
-    );
+    throw new UnsupportedPropertyError(elementType, propertyName, Object.keys(elementPropSchema));
   }
   const propKey = elementPropSchema[propertyName].key;
   const value = resolvePropValue(propertyValue, propKey);
@@ -4965,12 +4974,12 @@ Do NOT send "group" (it is resolved automatically). Use { "settings": {} } only
 V4 only: If MCP fails, give manual steps using V4 UI.
 
 V4 Editor structure:
-Panel tabs: General (\u2192 Settings section: ID, Tag, Link), Style, Interactions.
+Panel tabs: General (\u2192 Settings section: ID, Tag, and Link where the widget supports it), Style, Interactions.
 NO Advanced tab. Never mention Advanced tab.
+Note: \`link\` is valid only when the element's PropType schema (which you must already have) includes a \`link\` property. Sending \`link\` to a widget whose schema lacks it is skipped and reported in the response \`warnings\` (other changes still apply) and the link is lost.
 `);
   return configureElementToolPrompt.prompt();
 };
-var CONFIGURE_ELEMENT_GUIDE_TEXT = generatePrompt();
 
 // src/mcp/tools/configure-element/schema.ts
 var import_schema2 = require("@elementor/schema");
@@ -4994,7 +5003,10 @@ var inputSchema = {
 var outputSchema = {
   success: import_schema2.z.boolean().describe(
     "Whether the configuration change was successful, only if propertyName and propertyValue are provided"
-  )
+  ),
+  warnings: import_schema2.z.string().describe(
+    'Non-fatal notices. Present when some props were skipped because they are not in the element schema (e.g. a "link" on a widget with no link prop). Other changes were still applied.'
+  ).optional()
 };
 
 // src/mcp/tools/configure-element/tool.ts
@@ -5044,6 +5056,7 @@ var initConfigureElementTool = (reg) => {
       }
       const propertiesToUpdate = resolveCanonicalPropKeys(elementType, propertiesToChange);
       const toUpdate = Object.entries(propertiesToUpdate);
+      const skippedProps = [];
       for (const [propertyName, propertyValue] of toUpdate) {
         if (!import_editor_props7.Schema.isPropKeyConfigurable(propertyName)) {
           throw new Error(`Not allowed to update ${propertyName}`);
@@ -5056,6 +5069,10 @@ var initConfigureElementTool = (reg) => {
             propertyValue
           });
         } catch (error) {
+          if (error instanceof UnsupportedPropertyError) {
+            skippedProps.push(error.propertyName);
+            continue;
+          }
           const errorMessage = createUpdateErrorMessage({
             propertyName,
             elementId,
@@ -5068,7 +5085,10 @@ var initConfigureElementTool = (reg) => {
       }
       await applyStyleFromCss({ elementId, elementType, style });
       return {
-        success: true
+        success: true,
+        warnings: skippedProps.length ? `Skipped unsupported props (not in the "${elementType}" schema; other changes were applied): ${skippedProps.join(
+          ", "
+        )}.` : void 0
       };
     }
   });

package/dist/index.mjs

@@ -4697,6 +4697,20 @@ var LOCAL_STYLE_META = {
   breakpoint: "desktop",
   state: null
 };
+var UnsupportedPropertyError = class extends Error {
+  elementType;
+  propertyName;
+  constructor(elementType, propertyName, availableProperties) {
+    super(
+      `Property "${propertyName}" does not exist on element type "${elementType}". Available properties are: ${availableProperties.join(
+        ", "
+      )}`
+    );
+    this.name = "UnsupportedPropertyError";
+    this.elementType = elementType;
+    this.propertyName = propertyName;
+  }
+};
 function resolvePropValue(value, forceKey) {
   const Utils = window.elementorV2.editorVariables.Utils;
   return Schema.adjustLlmPropValueSchema(value, {
@@ -4802,12 +4816,7 @@ var doUpdateElementProperty = (params) => {
     throw new Error(`No prop schema found for element type: ${elementType}`);
   }
   if (!elementPropSchema[propertyName]) {
-    const propertyNames = Object.keys(elementPropSchema);
-    throw new Error(
-      `Property "${propertyName}" does not exist on element type "${elementType}". Available properties are: ${propertyNames.join(
-        ", "
-      )}`
-    );
+    throw new UnsupportedPropertyError(elementType, propertyName, Object.keys(elementPropSchema));
   }
   const propKey = elementPropSchema[propertyName].key;
   const value = resolvePropValue(propertyValue, propKey);
@@ -4937,12 +4946,12 @@ Do NOT send "group" (it is resolved automatically). Use { "settings": {} } only
 V4 only: If MCP fails, give manual steps using V4 UI.
 
 V4 Editor structure:
-Panel tabs: General (\u2192 Settings section: ID, Tag, Link), Style, Interactions.
+Panel tabs: General (\u2192 Settings section: ID, Tag, and Link where the widget supports it), Style, Interactions.
 NO Advanced tab. Never mention Advanced tab.
+Note: \`link\` is valid only when the element's PropType schema (which you must already have) includes a \`link\` property. Sending \`link\` to a widget whose schema lacks it is skipped and reported in the response \`warnings\` (other changes still apply) and the link is lost.
 `);
   return configureElementToolPrompt.prompt();
 };
-var CONFIGURE_ELEMENT_GUIDE_TEXT = generatePrompt();
 
 // src/mcp/tools/configure-element/schema.ts
 import { z as z2 } from "@elementor/schema";
@@ -4966,7 +4975,10 @@ var inputSchema = {
 var outputSchema = {
   success: z2.boolean().describe(
     "Whether the configuration change was successful, only if propertyName and propertyValue are provided"
-  )
+  ),
+  warnings: z2.string().describe(
+    'Non-fatal notices. Present when some props were skipped because they are not in the element schema (e.g. a "link" on a widget with no link prop). Other changes were still applied.'
+  ).optional()
 };
 
 // src/mcp/tools/configure-element/tool.ts
@@ -5016,6 +5028,7 @@ var initConfigureElementTool = (reg) => {
       }
       const propertiesToUpdate = resolveCanonicalPropKeys(elementType, propertiesToChange);
       const toUpdate = Object.entries(propertiesToUpdate);
+      const skippedProps = [];
       for (const [propertyName, propertyValue] of toUpdate) {
         if (!Schema2.isPropKeyConfigurable(propertyName)) {
           throw new Error(`Not allowed to update ${propertyName}`);
@@ -5028,6 +5041,10 @@ var initConfigureElementTool = (reg) => {
             propertyValue
           });
         } catch (error) {
+          if (error instanceof UnsupportedPropertyError) {
+            skippedProps.push(error.propertyName);
+            continue;
+          }
           const errorMessage = createUpdateErrorMessage({
             propertyName,
             elementId,
@@ -5040,7 +5057,10 @@ var initConfigureElementTool = (reg) => {
       }
       await applyStyleFromCss({ elementId, elementType, style });
       return {
-        success: true
+        success: true,
+        warnings: skippedProps.length ? `Skipped unsupported props (not in the "${elementType}" schema; other changes were applied): ${skippedProps.join(
+          ", "
+        )}.` : void 0
       };
     }
   });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@elementor/editor-canvas",
   "description": "Elementor Editor Canvas",
-  "version": "4.3.0-1004",
+  "version": "4.3.0-1005",
   "private": false,
   "author": "Elementor Team",
   "homepage": "https://elementor.com/",
@@ -37,26 +37,26 @@
     "react-dom": "^18.3.1"
   },
   "dependencies": {
-    "@elementor/editor": "4.3.0-1004",
-    "@elementor/editor-controls": "4.3.0-1004",
-    "@elementor/editor-documents": "4.3.0-1004",
-    "@elementor/editor-elements": "4.3.0-1004",
-    "@elementor/editor-interactions": "4.3.0-1004",
-    "@elementor/editor-mcp": "4.3.0-1004",
-    "@elementor/editor-notifications": "4.3.0-1004",
-    "@elementor/editor-props": "4.3.0-1004",
-    "@elementor/editor-responsive": "4.3.0-1004",
-    "@elementor/editor-styles": "4.3.0-1004",
-    "@elementor/editor-styles-repository": "4.3.0-1004",
-    "@elementor/editor-ui": "4.3.0-1004",
-    "@elementor/editor-v1-adapters": "4.3.0-1004",
-    "@elementor/events": "4.3.0-1004",
-    "@elementor/http-client": "4.3.0-1004",
-    "@elementor/schema": "4.3.0-1004",
-    "@elementor/twing": "4.3.0-1004",
+    "@elementor/editor": "4.3.0-1005",
+    "@elementor/editor-controls": "4.3.0-1005",
+    "@elementor/editor-documents": "4.3.0-1005",
+    "@elementor/editor-elements": "4.3.0-1005",
+    "@elementor/editor-interactions": "4.3.0-1005",
+    "@elementor/editor-mcp": "4.3.0-1005",
+    "@elementor/editor-notifications": "4.3.0-1005",
+    "@elementor/editor-props": "4.3.0-1005",
+    "@elementor/editor-responsive": "4.3.0-1005",
+    "@elementor/editor-styles": "4.3.0-1005",
+    "@elementor/editor-styles-repository": "4.3.0-1005",
+    "@elementor/editor-ui": "4.3.0-1005",
+    "@elementor/editor-v1-adapters": "4.3.0-1005",
+    "@elementor/events": "4.3.0-1005",
+    "@elementor/http-client": "4.3.0-1005",
+    "@elementor/schema": "4.3.0-1005",
+    "@elementor/twing": "4.3.0-1005",
     "@elementor/ui": "1.37.5",
-    "@elementor/utils": "4.3.0-1004",
-    "@elementor/wp-media": "4.3.0-1004",
+    "@elementor/utils": "4.3.0-1005",
+    "@elementor/wp-media": "4.3.0-1005",
     "@floating-ui/react": "^0.27.5",
     "@wordpress/i18n": "^5.13.0",
     "dompurify": "^3.2.6"

package/src/mcp/tools/configure-element/__tests__/tool.test.ts

@@ -0,0 +1,130 @@
+import { getContainer, getWidgetsCache } from '@elementor/editor-elements';
+
+import { doUpdateElementProperty, UnsupportedPropertyError } from '../../../utils/do-update-element-property';
+import { initConfigureElementTool } from '../tool';
+
+jest.mock( '@elementor/editor-elements', () => ( {
+	getContainer: jest.fn(),
+	getWidgetsCache: jest.fn(),
+} ) );
+
+jest.mock( '@elementor/editor-mcp', () => ( {
+	dispatchMcpStylesAppliedEvent: jest.fn(),
+} ) );
+
+jest.mock( '@elementor/editor-props', () => ( {
+	Schema: { isPropKeyConfigurable: () => true },
+} ) );
+
+jest.mock( '../prompt', () => ( {
+	CONFIGURE_ELEMENT_GUIDE_URI: 'configure-element-guide-uri',
+	generatePrompt: () => '',
+} ) );
+
+jest.mock( '../../../resources/widgets-schema-resource', () => ( {
+	WIDGET_SCHEMA_URI: 'elementor://widgets/schema/{widgetType}',
+} ) );
+
+jest.mock( '../../../resources/dynamic-tags-resource', () => ( {
+	DYNAMIC_TAGS_URI: 'elementor://dynamic-tags',
+} ) );
+
+jest.mock( '../../../utils/resolve-canonical-prop-name', () => ( {
+	resolveCanonicalPropKeys: ( _elementType: string, props: Record< string, unknown > ) => props,
+} ) );
+
+jest.mock( '../../../utils/convert-css-to-atomic', () => ( {
+	convertCssToAtomic: jest.fn().mockResolvedValue( { props: {}, customCss: '' } ),
+} ) );
+
+jest.mock( '../../../utils/do-update-element-property', () => ( {
+	...jest.requireActual( '../../../utils/do-update-element-property' ),
+	doUpdateElementProperty: jest.fn(),
+} ) );
+
+const ELEMENT_ID = 'el-1';
+const ELEMENT_TYPE = 'e-youtube';
+
+type Handler = ( params: {
+	elementId: string;
+	elementType: string;
+	propertiesToChange: Record< string, unknown >;
+	style: Record< string, string | null >;
+} ) => Promise< { success: boolean; warnings?: string } >;
+
+const getHandler = (): Handler => {
+	const addTool = jest.fn();
+	initConfigureElementTool( { addTool, resource: jest.fn() } as never );
+	return addTool.mock.calls[ 0 ][ 0 ].handler;
+};
+
+describe( 'configure-element handler unsupported props', () => {
+	beforeEach( () => {
+		jest.mocked( getWidgetsCache ).mockReturnValue( {
+			[ ELEMENT_TYPE ]: { atomic_props_schema: { title: { key: 'title' } } },
+		} as never );
+		jest.mocked( getContainer ).mockReturnValue( {
+			settings: { get: () => ELEMENT_TYPE },
+		} as never );
+		jest.mocked( doUpdateElementProperty ).mockReset();
+	} );
+
+	it( 'skips an unsupported prop, reports it in warnings, and still applies supported props', async () => {
+		// Arrange
+		jest.mocked( doUpdateElementProperty ).mockImplementation( ( { propertyName } ) => {
+			if ( 'link' === propertyName ) {
+				throw new UnsupportedPropertyError( ELEMENT_TYPE, 'link', [ 'title' ] );
+			}
+		} );
+		const handler = getHandler();
+
+		// Act
+		const result = await handler( {
+			elementId: ELEMENT_ID,
+			elementType: ELEMENT_TYPE,
+			propertiesToChange: { title: { $$type: 'string', value: 'Hi' }, link: { $$type: 'link', value: {} } },
+			style: {},
+		} );
+
+		// Assert
+		expect( result.success ).toBe( true );
+		expect( result.warnings ).toContain( 'link' );
+		expect( doUpdateElementProperty ).toHaveBeenCalledTimes( 2 );
+	} );
+
+	it( 'returns no warnings when every prop is supported', async () => {
+		// Arrange
+		jest.mocked( doUpdateElementProperty ).mockImplementation( () => undefined );
+		const handler = getHandler();
+
+		// Act
+		const result = await handler( {
+			elementId: ELEMENT_ID,
+			elementType: ELEMENT_TYPE,
+			propertiesToChange: { title: { $$type: 'string', value: 'Hi' } },
+			style: {},
+		} );
+
+		// Assert
+		expect( result.success ).toBe( true );
+		expect( result.warnings ).toBeUndefined();
+	} );
+
+	it( 'rethrows non-UnsupportedPropertyError failures', async () => {
+		// Arrange
+		jest.mocked( doUpdateElementProperty ).mockImplementation( () => {
+			throw new Error( 'Invalid PropValue' );
+		} );
+		const handler = getHandler();
+
+		// Act & Assert
+		await expect(
+			handler( {
+				elementId: ELEMENT_ID,
+				elementType: ELEMENT_TYPE,
+				propertiesToChange: { title: { $$type: 'string', value: 'Hi' } },
+				style: {},
+			} )
+		).rejects.toThrow( /Invalid PropValue/ );
+	} );
+} );

package/src/mcp/tools/configure-element/prompt.ts

@@ -117,11 +117,10 @@ Do NOT send "group" (it is resolved automatically). Use { "settings": {} } only
 V4 only: If MCP fails, give manual steps using V4 UI.
 
 V4 Editor structure:
-Panel tabs: General (→ Settings section: ID, Tag, Link), Style, Interactions.
+Panel tabs: General (→ Settings section: ID, Tag, and Link where the widget supports it), Style, Interactions.
 NO Advanced tab. Never mention Advanced tab.
+Note: \`link\` is valid only when the element's PropType schema (which you must already have) includes a \`link\` property. Sending \`link\` to a widget whose schema lacks it is skipped and reported in the response \`warnings\` (other changes still apply) and the link is lost.
 ` );
 
 	return configureElementToolPrompt.prompt();
 };
-
-export const CONFIGURE_ELEMENT_GUIDE_TEXT = generatePrompt();

package/src/mcp/tools/configure-element/schema.ts

@@ -36,4 +36,10 @@ export const outputSchema = {
 		.describe(
 			'Whether the configuration change was successful, only if propertyName and propertyValue are provided'
 		),
+	warnings: z
+		.string()
+		.describe(
+			'Non-fatal notices. Present when some props were skipped because they are not in the element schema (e.g. a "link" on a widget with no link prop). Other changes were still applied.'
+		)
+		.optional(),
 };

package/src/mcp/tools/configure-element/tool.ts

@@ -6,7 +6,7 @@ import { type PropValue, Schema } from '@elementor/editor-props';
 import { DYNAMIC_TAGS_URI } from '../../resources/dynamic-tags-resource';
 import { WIDGET_SCHEMA_URI } from '../../resources/widgets-schema-resource';
 import { convertCssToAtomic } from '../../utils/convert-css-to-atomic';
-import { doUpdateElementProperty } from '../../utils/do-update-element-property';
+import { doUpdateElementProperty, UnsupportedPropertyError } from '../../utils/do-update-element-property';
 import { resolveCanonicalPropKeys } from '../../utils/resolve-canonical-prop-name';
 import { CONFIGURE_ELEMENT_GUIDE_URI, generatePrompt } from './prompt';
 import { inputSchema as schema, outputSchema } from './schema';
@@ -61,6 +61,7 @@ export const initConfigureElementTool = ( reg: MCPRegistryEntry ) => {
 			}
 			const propertiesToUpdate = resolveCanonicalPropKeys( elementType, propertiesToChange );
 			const toUpdate = Object.entries( propertiesToUpdate );
+			const skippedProps: string[] = [];
 			for ( const [ propertyName, propertyValue ] of toUpdate as [ string, PropValue ][] ) {
 				if ( ! Schema.isPropKeyConfigurable( propertyName ) ) {
 					throw new Error( `Not allowed to update ${ propertyName }` );
@@ -73,6 +74,10 @@ export const initConfigureElementTool = ( reg: MCPRegistryEntry ) => {
 						propertyValue,
 					} );
 				} catch ( error ) {
+					if ( error instanceof UnsupportedPropertyError ) {
+						skippedProps.push( error.propertyName );
+						continue;
+					}
 					const errorMessage = createUpdateErrorMessage( {
 						propertyName,
 						elementId,
@@ -86,6 +91,11 @@ export const initConfigureElementTool = ( reg: MCPRegistryEntry ) => {
 			await applyStyleFromCss( { elementId, elementType, style } );
 			return {
 				success: true,
+				warnings: skippedProps.length
+					? `Skipped unsupported props (not in the "${ elementType }" schema; other changes were applied): ${ skippedProps.join(
+							', '
+					  ) }.`
+					: undefined,
 			};
 		},
 	} );

package/src/mcp/utils/__tests__/do-update-element-property.test.ts

@@ -8,7 +8,7 @@ import { Schema } from '@elementor/editor-props';
 import { getStylesSchema, getVariantByMeta } from '@elementor/editor-styles';
 import { __privateRunCommandSync } from '@elementor/editor-v1-adapters';
 
-import { doUpdateElementProperty } from '../do-update-element-property';
+import { doUpdateElementProperty, UnsupportedPropertyError } from '../do-update-element-property';
 
 jest.mock( '@elementor/editor-elements', () => ( {
 	createElementStyle: jest.fn(),
@@ -108,6 +108,32 @@ describe( 'doUpdateElementProperty', () => {
 		expect( __privateRunCommandSync ).not.toHaveBeenCalled();
 	} );
 
+	it( 'throws UnsupportedPropertyError when the property is not in the element schema', () => {
+		// Arrange
+		const propertyName = 'link';
+
+		// Act
+		let thrown: unknown;
+		try {
+			doUpdateElementProperty( {
+				elementId: ELEMENT_ID,
+				elementType: ELEMENT_TYPE,
+				propertyName,
+				propertyValue: { $$type: 'link', value: {} },
+			} );
+		} catch ( error ) {
+			thrown = error;
+		}
+
+		// Assert
+		expect( thrown ).toBeInstanceOf( UnsupportedPropertyError );
+		expect( ( thrown as UnsupportedPropertyError ).propertyName ).toBe( propertyName );
+		expect( ( thrown as UnsupportedPropertyError ).elementType ).toBe( ELEMENT_TYPE );
+		expect( ( thrown as Error ).message ).toContain( 'does not exist on element type' );
+		expect( Schema.validatePropValue ).not.toHaveBeenCalled();
+		expect( updateElementSettings ).not.toHaveBeenCalled();
+	} );
+
 	it( 'updates settings when Schema.validatePropValue reports valid PropValue', () => {
 		// Arrange
 		const propertyValue = 'resolved-title';

package/src/mcp/utils/do-update-element-property.ts

@@ -21,6 +21,23 @@ const LOCAL_STYLE_META = {
 	breakpoint: 'desktop',
 	state: null,
 } as const;
+
+export class UnsupportedPropertyError extends Error {
+	public readonly elementType: string;
+	public readonly propertyName: string;
+
+	constructor( elementType: string, propertyName: string, availableProperties: string[] ) {
+		super(
+			`Property "${ propertyName }" does not exist on element type "${ elementType }". Available properties are: ${ availableProperties.join(
+				', '
+			) }`
+		);
+		this.name = 'UnsupportedPropertyError';
+		this.elementType = elementType;
+		this.propertyName = propertyName;
+	}
+}
+
 type CustomCssWriteMode = 'replace' | 'merge-with-stored';
 type OwnParams = {
 	elementId: string;
@@ -151,12 +168,7 @@ export const doUpdateElementProperty = ( params: OwnParams ) => {
 		throw new Error( `No prop schema found for element type: ${ elementType }` );
 	}
 	if ( ! elementPropSchema[ propertyName ] ) {
-		const propertyNames = Object.keys( elementPropSchema );
-		throw new Error(
-			`Property "${ propertyName }" does not exist on element type "${ elementType }". Available properties are: ${ propertyNames.join(
-				', '
-			) }`
-		);
+		throw new UnsupportedPropertyError( elementType, propertyName, Object.keys( elementPropSchema ) );
 	}
 	const propKey = elementPropSchema[ propertyName ].key;
 	const value = resolvePropValue( propertyValue, propKey );