feat(editor): Easy $fromAI Button for AI Tools

cypress/composables/ndv.ts

@@ -225,7 +225,7 @@ export function populateMapperFields(fields: ReadonlyArray<[string, string]>) {
 		getParameterInputByName(name).type(value);
 
 		// Click on a parent to dismiss the pop up which hides the field below.
-		getParameterInputByName(name).parent().parent().parent().click('topLeft');
+		getParameterInputByName(name).parent().parent().parent().parent().click('topLeft');
 	}
 }
 

cypress/e2e/16-form-trigger-node.cy.ts

@@ -62,12 +62,14 @@ describe('n8n Form Trigger', () => {
 		getVisibleSelect().contains('Dropdown').click();
 		cy.contains('button', 'Add Field Option').click();
 		cy.contains('label', 'Field Options')
+			.parent()
 			.parent()
 			.nextAll()
 			.find('[data-test-id="parameter-input-field"]')
 			.eq(0)
 			.type('Option 1');
 		cy.contains('label', 'Field Options')
+			.parent()
 			.parent()
 			.nextAll()
 			.find('[data-test-id="parameter-input-field"]')

cypress/e2e/48-subworkflow-inputs.cy.ts

@@ -90,8 +90,8 @@ describe('Sub-workflow creation and typed usage', () => {
 		clickExecuteNode();
 
 		const expected = [
-			['-1', 'A String', '0:11:true2:3', 'aKey:-1', '[empty object]', 'false'],
-			['-1', 'Another String', '[empty array]', 'aDifferentKey:-1', '[empty array]', 'false'],
+			['-1', 'A String', '0:11:true2:3', 'aKey:-1', '[empty object]', 'true'],
+			['-1', 'Another String', '[empty array]', 'aDifferentKey:-1', '[empty array]', 'true'],
 		];
 		assertOutputTableContent(expected);
 
@@ -110,8 +110,8 @@ describe('Sub-workflow creation and typed usage', () => {
 		clickExecuteNode();
 
 		const expected2 = [
-			['-1', '5', '0:11:true2:3', 'aKey:-1', '[empty object]', 'false'],
-			['-1', '5', '[empty array]', 'aDifferentKey:-1', '[empty array]', 'false'],
+			['-1', '5', '0:11:true2:3', 'aKey:-1', '[empty object]', 'true'],
+			['-1', '5', '[empty array]', 'aDifferentKey:-1', '[empty array]', 'true'],
 		];
 
 		assertOutputTableContent(expected2);
@@ -167,8 +167,8 @@ describe('Sub-workflow creation and typed usage', () => {
 				);
 
 				assertOutputTableContent([
-					['[null]', '[null]', '[null]', '[null]', '[null]', 'false'],
-					['[null]', '[null]', '[null]', '[null]', '[null]', 'false'],
+					['[null]', '[null]', '[null]', '[null]', '[null]', 'true'],
+					['[null]', '[null]', '[null]', '[null]', '[null]', 'true'],
 				]);
 
 				clickExecuteNode();

packages/cli/src/__tests__/load-nodes-and-credentials.test.ts

@@ -107,7 +107,7 @@ describe('LoadNodesAndCredentials', () => {
 			};
 			fullNodeWrapper.description.properties = [existingProp];
 			const result = instance.convertNodeToAiTool(fullNodeWrapper);
-			expect(result.description.properties).toHaveLength(3); // Existing prop + toolDescription + notice
+			expect(result.description.properties).toHaveLength(2); // Existing prop + toolDescription
 			expect(result.description.properties).toContainEqual(existingProp);
 		});
 
@@ -121,9 +121,9 @@ describe('LoadNodesAndCredentials', () => {
 			};
 			fullNodeWrapper.description.properties = [resourceProp];
 			const result = instance.convertNodeToAiTool(fullNodeWrapper);
-			expect(result.description.properties[1].name).toBe('descriptionType');
-			expect(result.description.properties[2].name).toBe('toolDescription');
-			expect(result.description.properties[3]).toEqual(resourceProp);
+			expect(result.description.properties[0].name).toBe('descriptionType');
+			expect(result.description.properties[1].name).toBe('toolDescription');
+			expect(result.description.properties[2]).toEqual(resourceProp);
 		});
 
 		it('should handle nodes with operation property', () => {
@@ -136,9 +136,9 @@ describe('LoadNodesAndCredentials', () => {
 			};
 			fullNodeWrapper.description.properties = [operationProp];
 			const result = instance.convertNodeToAiTool(fullNodeWrapper);
-			expect(result.description.properties[1].name).toBe('descriptionType');
-			expect(result.description.properties[2].name).toBe('toolDescription');
-			expect(result.description.properties[3]).toEqual(operationProp);
+			expect(result.description.properties[0].name).toBe('descriptionType');
+			expect(result.description.properties[1].name).toBe('toolDescription');
+			expect(result.description.properties[2]).toEqual(operationProp);
 		});
 
 		it('should handle nodes with both resource and operation properties', () => {
@@ -158,17 +158,17 @@ describe('LoadNodesAndCredentials', () => {
 			};
 			fullNodeWrapper.description.properties = [resourceProp, operationProp];
 			const result = instance.convertNodeToAiTool(fullNodeWrapper);
-			expect(result.description.properties[1].name).toBe('descriptionType');
-			expect(result.description.properties[2].name).toBe('toolDescription');
-			expect(result.description.properties[3]).toEqual(resourceProp);
-			expect(result.description.properties[4]).toEqual(operationProp);
+			expect(result.description.properties[0].name).toBe('descriptionType');
+			expect(result.description.properties[1].name).toBe('toolDescription');
+			expect(result.description.properties[2]).toEqual(resourceProp);
+			expect(result.description.properties[3]).toEqual(operationProp);
 		});
 
 		it('should handle nodes with empty properties', () => {
 			fullNodeWrapper.description.properties = [];
 			const result = instance.convertNodeToAiTool(fullNodeWrapper);
-			expect(result.description.properties).toHaveLength(2);
-			expect(result.description.properties[1].name).toBe('toolDescription');
+			expect(result.description.properties).toHaveLength(1);
+			expect(result.description.properties[0].name).toBe('toolDescription');
 		});
 
 		it('should handle nodes with existing codex property', () => {

packages/cli/src/events/relays/telemetry.event-relay.ts

@@ -701,6 +701,7 @@ export class TelemetryEventRelay extends EventRelay {
 					sharing_role: userRole,
 					credential_type: null,
 					is_managed: false,
+					...TelemetryHelpers.resolveAIMetrics(workflow.nodes, this.nodeTypes),
 				};
 
 				if (!manualExecEventProperties.node_graph_string) {

packages/cli/src/load-nodes-and-credentials.ts

@@ -484,14 +484,6 @@ export class LoadNodesAndCredentials {
 					placeholder: `e.g. ${item.description.description}`,
 				};
 
-				const noticeProp: INodeProperties = {
-					displayName:
-						"Use the expression {{ $fromAI('placeholder_name') }} for any data to be filled by the model",
-					name: 'notice',
-					type: 'notice',
-					default: '',
-				};
-
 				item.description.properties.unshift(descProp);
 
 				// If node has resource or operation we can determine pre-populate tool description based on it
@@ -505,8 +497,6 @@ export class LoadNodesAndCredentials {
 						},
 					};
 				}
-
-				item.description.properties.unshift(noticeProp);
 			}
 		}
 

packages/design-system/src/components/N8nCheckbox/__snapshots__/Checkbox.test.ts.snap

@@ -31,34 +31,39 @@ exports[`components > N8nCheckbox > should render with both child and label 1`]
         class="container"
         data-test-id="input-label"
       >
-        <label
-          class="n8n-input-label inputLabel heading medium"
+        <div
+          class="labelRow"
         >
-          <div
-            class="main-content"
+          <label
+            class="n8n-input-label inputLabel heading medium"
           >
             <div
-              class="title"
+              class="main-content"
             >
-              <span
-                class="n8n-text size-medium regular"
+              <div
+                class="title"
               >
-
-                Checkbox 
-                <!--v-if-->
-
-              </span>
+                <span
+                  class="n8n-text size-medium regular"
+                >
+
+                  Checkbox 
+                  <!--v-if-->
+
+                </span>
+              </div>
+              <!--v-if-->
             </div>
-            <!--v-if-->
-          </div>
-          <div
-            class="trailing-content"
-          >
-            <!--v-if-->
-            <!--v-if-->
-            <!--v-if-->
-          </div>
-        </label>
+            <div
+              class="trailing-content"
+            >
+              <!--v-if-->
+              <!--v-if-->
+              <!--v-if-->
+            </div>
+          </label>
+          <!--v-if-->
+        </div>
 
 
       </div>
@@ -131,34 +136,39 @@ exports[`components > N8nCheckbox > should render with label 1`] = `
         class="container"
         data-test-id="input-label"
       >
-        <label
-          class="n8n-input-label inputLabel heading medium"
+        <div
+          class="labelRow"
         >
-          <div
-            class="main-content"
+          <label
+            class="n8n-input-label inputLabel heading medium"
           >
             <div
-              class="title"
+              class="main-content"
             >
-              <span
-                class="n8n-text size-medium regular"
+              <div
+                class="title"
               >
-
-                Checkbox 
-                <!--v-if-->
-
-              </span>
+                <span
+                  class="n8n-text size-medium regular"
+                >
+
+                  Checkbox 
+                  <!--v-if-->
+
+                </span>
+              </div>
+              <!--v-if-->
             </div>
-            <!--v-if-->
-          </div>
-          <div
-            class="trailing-content"
-          >
-            <!--v-if-->
-            <!--v-if-->
-            <!--v-if-->
-          </div>
-        </label>
+            <div
+              class="trailing-content"
+            >
+              <!--v-if-->
+              <!--v-if-->
+              <!--v-if-->
+            </div>
+          </label>
+          <!--v-if-->
+        </div>
 
 
       </div>