@open-mercato/core 0.4.2-canary-ec4978dbb3 → 0.4.2-canary-9074fd41ce

package/dist/modules/workflows/backend/definitions/visual-editor/page.js

@@ -33,7 +33,6 @@ import { apiCall } from "@open-mercato/ui/backend/utils/apiCall";
 import { flash } from "@open-mercato/ui/backend/FlashMessages";
 import { CircleQuestionMark, Info, PanelTopClose, PanelTopOpen, Play, Save, Trash2 } from "lucide-react";
 import { NODE_TYPE_ICONS, NODE_TYPE_COLORS, NODE_TYPE_LABELS } from "../../../lib/node-type-icons.js";
-import { DefinitionTriggersEditor } from "../../../components/DefinitionTriggersEditor.js";
 function VisualEditorPage() {
   const t = useT();
   const router = useRouter();
@@ -59,7 +58,6 @@ function VisualEditorPage() {
   const [icon, setIcon] = useState("");
   const [effectiveFrom, setEffectiveFrom] = useState("");
   const [effectiveTo, setEffectiveTo] = useState("");
-  const [triggers, setTriggers] = useState([]);
   useEffect(() => {
     const loadDefinition = async () => {
       if (!definitionId) {
@@ -87,7 +85,6 @@ function VisualEditorPage() {
         const graph = definitionToGraph(definition.definition);
         setNodes(graph.nodes);
         setEdges(graph.edges);
-        setTriggers(definition.definition?.triggers || []);
         flash("Workflow loaded successfully", "success");
       } catch (error) {
         console.error("Error loading workflow definition:", error);
@@ -212,11 +209,7 @@ function VisualEditorPage() {
       flash(`Cannot save: ${criticalErrors.length} validation error(s) found. Please fix them first.`, "error");
       return;
     }
-    const graphDefinition = graphToDefinition(nodes, edges, { includePositions: true });
-    const definitionData = {
-      ...graphDefinition,
-      triggers: triggers.length > 0 ? triggers : void 0
-    };
+    const definitionData = graphToDefinition(nodes, edges, { includePositions: true });
     const schemaResult = workflowDefinitionDataSchema.safeParse(definitionData);
     if (!schemaResult.success) {
       const firstIssue = schemaResult.error.issues[0];
@@ -272,7 +265,7 @@ function VisualEditorPage() {
     } finally {
       setIsSaving(false);
     }
-  }, [nodes, edges, workflowId, workflowName, description, version, enabled, category, tags, triggers, definitionId, router]);
+  }, [nodes, edges, workflowId, workflowName, description, version, enabled, category, tags, definitionId, router]);
   const handleTest = useCallback(() => {
     const errors = validateWorkflowGraph(nodes, edges);
     const criticalErrors = errors.filter((e) => e.type === "error");
@@ -375,7 +368,6 @@ function VisualEditorPage() {
     setIcon("");
     setEffectiveFrom("");
     setEffectiveTo("");
-    setTriggers([]);
     setShowClearConfirm(false);
     flash("Canvas cleared", "success");
   }, []);
@@ -474,152 +466,142 @@ function VisualEditorPage() {
         )
       ] })
     ] }) }),
-    showMetadata && /* @__PURE__ */ jsxs("div", { className: "bg-white border-b border-gray-200 px-6 py-4", children: [
-      /* @__PURE__ */ jsxs("div", { className: "rounded-lg border bg-card p-4", children: [
-        /* @__PURE__ */ jsx("h2", { className: "text-sm font-semibold uppercase text-muted-foreground mb-4", children: "Workflow Metadata" }),
-        /* @__PURE__ */ jsxs("div", { className: "grid grid-cols-3 gap-4", children: [
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "workflowId", className: "text-xs", children: "Workflow ID *" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "workflowId",
-                value: workflowId,
-                onChange: (e) => setWorkflowId(e.target.value),
-                placeholder: "checkout_workflow",
-                disabled: !!definitionId
-              }
-            ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: definitionId ? "Cannot be changed when editing" : "Lowercase, numbers, hyphens, underscores" })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "workflowName", className: "text-xs", children: "Workflow Name *" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "workflowName",
-                value: workflowName,
-                onChange: (e) => setWorkflowName(e.target.value),
-                placeholder: "Checkout Process"
-              }
-            )
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "category", className: "text-xs", children: "Category" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "category",
-                value: category,
-                onChange: (e) => setCategory(e.target.value),
-                placeholder: "E-Commerce"
-              }
-            )
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "col-span-3 space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "description", className: "text-xs", children: "Description" }),
-            /* @__PURE__ */ jsx(
-              Textarea,
-              {
-                id: "description",
-                value: description,
-                onChange: (e) => setDescription(e.target.value),
-                placeholder: "Describe the purpose of this workflow...",
-                rows: 2
-              }
-            )
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "version", className: "text-xs", children: "Version *" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "version",
-                type: "number",
-                value: version,
-                onChange: (e) => setVersion(parseInt(e.target.value) || 1),
-                min: 1,
-                disabled: !!definitionId
-              }
-            ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Version number (increment for major changes)" })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { className: "text-xs", children: "Enabled" }),
-            /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2 mt-2", children: [
-              /* @__PURE__ */ jsx(
-                Switch,
-                {
-                  id: "enabled",
-                  checked: enabled,
-                  onCheckedChange: setEnabled
-                }
-              ),
-              /* @__PURE__ */ jsx(Label, { htmlFor: "enabled", className: "text-sm font-normal cursor-pointer", children: "Only enabled workflows can be started" })
-            ] })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { className: "text-xs", children: "Tags" }),
-            /* @__PURE__ */ jsx(
-              TagsInput,
-              {
-                value: tags,
-                onChange: setTags,
-                placeholder: t("workflows.form.placeholders.tags")
-              }
-            ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: t("workflows.form.descriptions.tags") })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "icon", className: "text-xs", children: "Icon" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "icon",
-                value: icon,
-                onChange: (e) => setIcon(e.target.value),
-                placeholder: "ShoppingCart"
-              }
-            ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Icon name for visual identification" })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "effectiveFrom", className: "text-xs", children: "Effective From" }),
-            /* @__PURE__ */ jsx(
-              Input,
-              {
-                id: "effectiveFrom",
-                type: "date",
-                value: effectiveFrom,
-                onChange: (e) => setEffectiveFrom(e.target.value)
-              }
-            ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Workflow becomes active from this date" })
-          ] }),
-          /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
-            /* @__PURE__ */ jsx(Label, { htmlFor: "effectiveTo", className: "text-xs", children: "Effective To" }),
+    showMetadata && /* @__PURE__ */ jsx("div", { className: "bg-white border-b border-gray-200 px-6 py-4", children: /* @__PURE__ */ jsxs("div", { className: "rounded-lg border bg-card p-4", children: [
+      /* @__PURE__ */ jsx("h2", { className: "text-sm font-semibold uppercase text-muted-foreground mb-4", children: "Workflow Metadata" }),
+      /* @__PURE__ */ jsxs("div", { className: "grid grid-cols-3 gap-4", children: [
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "workflowId", className: "text-xs", children: "Workflow ID *" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "workflowId",
+              value: workflowId,
+              onChange: (e) => setWorkflowId(e.target.value),
+              placeholder: "checkout_workflow",
+              disabled: !!definitionId
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: definitionId ? "Cannot be changed when editing" : "Lowercase, numbers, hyphens, underscores" })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "workflowName", className: "text-xs", children: "Workflow Name *" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "workflowName",
+              value: workflowName,
+              onChange: (e) => setWorkflowName(e.target.value),
+              placeholder: "Checkout Process"
+            }
+          )
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "category", className: "text-xs", children: "Category" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "category",
+              value: category,
+              onChange: (e) => setCategory(e.target.value),
+              placeholder: "E-Commerce"
+            }
+          )
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "col-span-3 space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "description", className: "text-xs", children: "Description" }),
+          /* @__PURE__ */ jsx(
+            Textarea,
+            {
+              id: "description",
+              value: description,
+              onChange: (e) => setDescription(e.target.value),
+              placeholder: "Describe the purpose of this workflow...",
+              rows: 2
+            }
+          )
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "version", className: "text-xs", children: "Version *" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "version",
+              type: "number",
+              value: version,
+              onChange: (e) => setVersion(parseInt(e.target.value) || 1),
+              min: 1,
+              disabled: !!definitionId
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Version number (increment for major changes)" })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { className: "text-xs", children: "Enabled" }),
+          /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2 mt-2", children: [
             /* @__PURE__ */ jsx(
-              Input,
+              Switch,
               {
-                id: "effectiveTo",
-                type: "date",
-                value: effectiveTo,
-                onChange: (e) => setEffectiveTo(e.target.value)
+                id: "enabled",
+                checked: enabled,
+                onCheckedChange: setEnabled
               }
             ),
-            /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Workflow deactivates after this date" })
+            /* @__PURE__ */ jsx(Label, { htmlFor: "enabled", className: "text-sm font-normal cursor-pointer", children: "Only enabled workflows can be started" })
           ] })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { className: "text-xs", children: "Tags" }),
+          /* @__PURE__ */ jsx(
+            TagsInput,
+            {
+              value: tags,
+              onChange: setTags,
+              placeholder: t("workflows.form.placeholders.tags")
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: t("workflows.form.descriptions.tags") })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "icon", className: "text-xs", children: "Icon" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "icon",
+              value: icon,
+              onChange: (e) => setIcon(e.target.value),
+              placeholder: "ShoppingCart"
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Icon name for visual identification" })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "effectiveFrom", className: "text-xs", children: "Effective From" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "effectiveFrom",
+              type: "date",
+              value: effectiveFrom,
+              onChange: (e) => setEffectiveFrom(e.target.value)
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Workflow becomes active from this date" })
+        ] }),
+        /* @__PURE__ */ jsxs("div", { className: "space-y-1", children: [
+          /* @__PURE__ */ jsx(Label, { htmlFor: "effectiveTo", className: "text-xs", children: "Effective To" }),
+          /* @__PURE__ */ jsx(
+            Input,
+            {
+              id: "effectiveTo",
+              type: "date",
+              value: effectiveTo,
+              onChange: (e) => setEffectiveTo(e.target.value)
+            }
+          ),
+          /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: "Workflow deactivates after this date" })
         ] })
-      ] }),
-      /* @__PURE__ */ jsx(
-        DefinitionTriggersEditor,
-        {
-          value: triggers,
-          onChange: setTriggers,
-          className: "mt-4"
-        }
-      )
-    ] }),
+      ] })
+    ] }) }),
     /* @__PURE__ */ jsxs(PageBody, { className: "flex-1 flex overflow-hidden", children: [
       /* @__PURE__ */ jsx("div", { className: "w-88 bg-white border-r border-gray-200 p-6 overflow-y-auto", children: /* @__PURE__ */ jsxs("div", { className: "rounded-lg border bg-card p-4", children: [
         /* @__PURE__ */ jsx("h2", { className: "text-sm font-semibold uppercase text-muted-foreground mb-2", children: "Step Palette" }),