From d0f9f4966b107178c8d554c358e774b45b336ea0 Mon Sep 17 00:00:00 2001
From: Shu Yao <springrainyszxr@gmail.com>
Date: Fri, 6 Feb 2026 11:53:09 +0800
Subject: [PATCH] wip: fix front-end lag issue

---
 frontend/src/components/FormGenerator.vue |  8 ++-
 frontend/src/pages/WorkflowView.vue       | 80 +++++++++++++----------
 2 files changed, 52 insertions(+), 36 deletions(-)

diff --git a/frontend/src/components/FormGenerator.vue b/frontend/src/components/FormGenerator.vue
index 061b6674..cff85537 100755
--- a/frontend/src/components/FormGenerator.vue
+++ b/frontend/src/components/FormGenerator.vue
@@ -297,7 +297,9 @@ const setBaseYamlFromSource = (source) => {
   }
   try {
     let parsed = source
+    let yamlString = ''
     if (typeof source === 'string') {
+      yamlString = source
       parsed = yaml.load(source) || {}
     } else if (typeof source === 'object') {
       parsed = JSON.parse(JSON.stringify(source))
@@ -305,7 +307,9 @@ const setBaseYamlFromSource = (source) => {
       parsed = {}
     }
     baseYamlObject.value = parsed
-    baseYamlString.value = yaml.dump(parsed ?? null, yamlDumpOptions)
+    // Avoid expensive dump on modal open for object sources.
+    // Save path will generate YAML from current object when needed.
+    baseYamlString.value = yamlString
   } catch (error) {
     console.error('Failed to set base YAML from provided source:', error)
     baseYamlObject.value = null
@@ -1046,7 +1050,7 @@ watch(() => props.initialYaml, (newValue) => {
     baseYamlObject.value = null
     baseYamlString.value = ''
   }
-}, { immediate: true, deep: true })
+}, { immediate: true })
 
 watch(() => props.workflowName, async (newName, oldName) => {
   if (!newName) {
diff --git a/frontend/src/pages/WorkflowView.vue b/frontend/src/pages/WorkflowView.vue
index 78ba5d72..4c80ffd3 100755
--- a/frontend/src/pages/WorkflowView.vue
+++ b/frontend/src/pages/WorkflowView.vue
@@ -1294,67 +1294,81 @@ const updateVueFlowNodeId = (oldId, newId) => {
 // FormGenerator integration
 const snapshotYamlContent = () => cloneDeep(yamlContent.value ?? null)
 
-// Build YAML without specific node
+// Build YAML without specific node (shallow clone path to avoid full deep-clone on editor open)
 const buildYamlWithoutNode = (nodeId) => {
-  const snapshot = snapshotYamlContent()
-  if (!snapshot?.graph?.nodes || !Array.isArray(snapshot.graph.nodes)) {
-    return snapshot
+  const source = yamlContent.value
+  if (!source?.graph?.nodes || !Array.isArray(source.graph.nodes)) {
+    return source
+  }
+  return {
+    ...source,
+    graph: {
+      ...source.graph,
+      nodes: source.graph.nodes.filter(node => node?.id !== nodeId)
+    }
   }
-  snapshot.graph.nodes = snapshot.graph.nodes.filter(node => node?.id !== nodeId)
-  return snapshot
 }
 
 const buildYamlWithoutEdge = (fromId, toId) => {
-  const snapshot = snapshotYamlContent()
-  if (!snapshot?.graph?.edges || !Array.isArray(snapshot.graph.edges)) {
-    return snapshot
+  const source = yamlContent.value
+  if (!source?.graph?.edges || !Array.isArray(source.graph.edges)) {
+    return source
   }
   let removed = false
-  snapshot.graph.edges = snapshot.graph.edges.filter(edge => {
+  const filteredEdges = source.graph.edges.filter(edge => {
     if (!removed && edge?.from === fromId && edge?.to === toId) {
       removed = true
       return false
     }
     return true
   })
-  return snapshot
+  return {
+    ...source,
+    graph: {
+      ...source.graph,
+      edges: filteredEdges
+    }
+  }
 }
 
 const buildYamlWithoutVars = () => {
-  const snapshot = snapshotYamlContent()
-  if (!snapshot || typeof snapshot !== 'object') {
-    return snapshot
+  const source = yamlContent.value
+  if (!source || typeof source !== 'object') {
+    return source
   }
-  if (!Object.prototype.hasOwnProperty.call(snapshot, 'vars')) {
-    return snapshot
+  if (!Object.prototype.hasOwnProperty.call(source, 'vars')) {
+    return source
   }
-  const sanitized = { ...snapshot }
+  const sanitized = { ...source }
   delete sanitized.vars
   return sanitized
 }
 
 const buildYamlWithoutMemory = () => {
-  const snapshot = snapshotYamlContent()
-  if (!snapshot?.graph) {
-    return snapshot
+  const source = yamlContent.value
+  if (!source?.graph) {
+    return source
   }
-  if (Object.prototype.hasOwnProperty.call(snapshot.graph, 'memory')) {
-    const newGraph = { ...snapshot.graph }
+  if (Object.prototype.hasOwnProperty.call(source.graph, 'memory')) {
+    const newGraph = { ...source.graph }
     delete newGraph.memory
-    snapshot.graph = newGraph
+    return {
+      ...source,
+      graph: newGraph
+    }
   }
-  return snapshot
+  return source
 }
 
 const buildYamlWithoutGraph = () => {
-  const snapshot = snapshotYamlContent()
-  if (!snapshot || typeof snapshot !== 'object') {
-    return snapshot
+  const source = yamlContent.value
+  if (!source || typeof source !== 'object') {
+    return source
   }
-  if (!Object.prototype.hasOwnProperty.call(snapshot, 'graph')) {
-    return snapshot
+  if (!Object.prototype.hasOwnProperty.call(source, 'graph')) {
+    return source
   }
-  const sanitized = { ...snapshot }
+  const sanitized = { ...source }
   delete sanitized.graph
   return sanitized
 }
@@ -1398,12 +1412,10 @@ const openDynamicFormGenerator = (type, options = {}) => {
 
   const hasCustomYaml = Object.prototype.hasOwnProperty.call(options, 'initialYaml')
   const yamlSource = hasCustomYaml ? options.initialYaml : yamlContent.value
-  formGeneratorInitialYaml.value = yamlSource ? cloneDeep(yamlSource) : null
+  formGeneratorInitialYaml.value = yamlSource || null
 
   if (Object.prototype.hasOwnProperty.call(options, 'initialFormData')) {
-    formGeneratorInitialFormData.value = options.initialFormData
-      ? cloneDeep(options.initialFormData)
-      : null
+    formGeneratorInitialFormData.value = options.initialFormData || null
   } else {
     formGeneratorInitialFormData.value = null
   }