fix: don't save and display empty controls

admyral/compiler/workflow_compiler.py

@@ -157,13 +157,18 @@ def _into_dag(self, workflow: "Workflow") -> WorkflowDAG:
         return WorkflowDAG(
             name=workflow.name,
             description=workflow.description,
-            controls=workflow.controls,
+            controls=self._filter_empty_controls(workflow.controls),
             start=WorkflowStart(
                 triggers=self._handle_triggers(workflow),
             ),
             dag=self.nodes,
         )
 
+    def _filter_empty_controls(self, controls: list[str] | None) -> list[str] | None:
+        if not controls:
+            return None
+        return [control for control in controls if control]
+
     def _compute_unique_node_id(self, base: str) -> str:
         if base not in self.nodes:
             return base

web/src/components/save-workflow-button/save-workflow-button.tsx

@@ -62,6 +62,12 @@ export default function SaveWorkflowButton() {
 			return;
 		}
 
+		if (workflow.controls) {
+			workflow.controls = workflow.controls.filter(
+				(control) => control.trim() !== "",
+			);
+		}
+
 		// Make sure that we only save args which are present in the current
 		// action definition. If an action is updated and an argument is
 		// removed, we want to clean it up here.