refactor(graphviz): simplify diagram editor and naming #438

Update provider names, remove redundant methods, and streamline editor property change handling for improved clarity and maintainability.

Author: phodal

exts/ext-diagram/src/main/kotlin/cc/unitmesh/diagram/graphviz/GraphvizElementManager.kt

@@ -1,21 +1,15 @@
 package cc.unitmesh.diagram.graphviz
 
+import cc.unitmesh.diagram.graphviz.model.*
 import com.intellij.diagram.AbstractDiagramElementManager
 import com.intellij.diagram.DiagramBuilder
 import com.intellij.openapi.actionSystem.CommonDataKeys
 import com.intellij.openapi.actionSystem.DataContext
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.vfs.VirtualFile
 import com.intellij.openapi.vfs.pointers.VirtualFilePointerManager
-import com.intellij.psi.PsiManager
 import com.intellij.ui.SimpleColoredText
 import com.intellij.ui.SimpleTextAttributes
-import cc.unitmesh.diagram.graphviz.model.GraphvizNodeData
-import cc.unitmesh.diagram.graphviz.model.GraphvizDiagramRootData
-import cc.unitmesh.diagram.graphviz.model.GraphvizSimpleNodeData
-import cc.unitmesh.diagram.graphviz.model.GraphvizEntityNodeData
-import cc.unitmesh.diagram.graphviz.model.GraphvizNodeField
-import cc.unitmesh.diagram.graphviz.model.GraphvizAttributeItem
 import com.intellij.util.ArrayUtil
 import javax.swing.Icon
 
@@ -42,7 +36,6 @@ class GraphvizElementManager : AbstractDiagramElementManager<GraphvizNodeData>()
         return when (nodeElement) {
             is GraphvizEntityNodeData -> nodeElement.getFields().toTypedArray()
             is GraphvizSimpleNodeData -> {
-                // Convert attributes to GraphvizAttributeItem objects
                 nodeElement.getAttributes().map { (key, value) ->
                     GraphvizAttributeItem(key, value)
                 }.toTypedArray()
@@ -87,7 +80,11 @@ class GraphvizElementManager : AbstractDiagramElementManager<GraphvizNodeData>()
             else -> element.getName()
         }
     }
-    
+
+    override fun canBeBuiltFrom(element: Any?): Boolean {
+        return element is GraphvizDiagramRootData || super.canBeBuiltFrom(element)
+    }
+
     override fun getItemName(
         nodeElement: GraphvizNodeData?,
         nodeItem: Any?,

exts/ext-diagram/src/main/kotlin/cc/unitmesh/diagram/graphviz/GraphvizUmlProvider.kt

@@ -21,12 +21,11 @@ class GraphvizUmlProvider : DiagramProvider<GraphvizNodeData>() {
 
     @Pattern("[a-zA-Z0-9_-]*")
     override fun getID(): String {
-
-        return "GraphvizDOT"
+        return "AutoDev GraphvizDOT"
     }
 
     override fun getPresentableName(): String {
-        return "Graphviz DOT Diagram"
+        return "AutoDev Graphviz DOT Diagram"
     }
 
     override fun createDataModel(

exts/ext-diagram/src/main/kotlin/cc/unitmesh/diagram/graphviz/editor/GraphvizPreviewFileEditor.kt

@@ -104,13 +104,9 @@ class GraphvizPreviewFileEditor(private val project: Project, private val file:
 
     override fun isValid(): Boolean = !isDisposed
 
-    override fun addPropertyChangeListener(listener: PropertyChangeListener) {
-        // No properties to listen to
-    }
+    override fun addPropertyChangeListener(listener: PropertyChangeListener) {}
 
-    override fun removePropertyChangeListener(listener: PropertyChangeListener) {
-        // No properties to listen to
-    }
+    override fun removePropertyChangeListener(listener: PropertyChangeListener) {}
 
     override fun getCurrentLocation(): FileEditorLocation? = null
 
@@ -136,7 +132,7 @@ class GraphvizPreviewFileEditor(private val project: Project, private val file:
                     if (isDisposed) return@addRequest
 
                     try {
-                        renderUml()
+                        myPanel!!.draw()
                     } catch (e: Exception) {
                         // Handle rendering errors gracefully
                         e.printStackTrace()
@@ -145,23 +141,4 @@ class GraphvizPreviewFileEditor(private val project: Project, private val file:
             }
         })
     }
-
-    private fun renderUml() {
-        if (isDisposed) return
-
-        // Dispose old panel
-        myPanel?.let { Disposer.dispose(it) }
-
-        // Create new panel
-        myPanel = GraphvizDiagramPanel(this)
-
-        umlPanelWrapper.removeAll()
-        umlPanelWrapper.add(myPanel!!.getComponent())
-
-        // Draw the diagram
-        myPanel!!.draw()
-
-        umlPanelWrapper.revalidate()
-        umlPanelWrapper.repaint()
-    }
 }

exts/ext-diagram/src/main/kotlin/cc/unitmesh/diagram/graphviz/model/GraphvizDiagramRootData.kt

@@ -18,28 +18,6 @@ class GraphvizDiagramRootData(
     override fun getName(): String = name
 
     override fun getIcon(): Icon = PlatformIcons.FILE_ICON
-    
-    /**
-     * Get the virtual file associated with this root data
-     */
+
     fun getVirtualFile(): VirtualFile? = virtualFilePointer.file
-    
-    /**
-     * Get the file pointer
-     */
-    fun getFilePointer(): VirtualFilePointer = virtualFilePointer
-    
-    override fun equals(other: Any?): Boolean {
-        if (this === other) return true
-        if (other !is GraphvizDiagramRootData) return false
-        return virtualFilePointer == other.virtualFilePointer
-    }
-    
-    override fun hashCode(): Int {
-        return virtualFilePointer.hashCode()
-    }
-    
-    override fun toString(): String {
-        return "GraphvizDiagramRootData(name='$name')"
-    }
 }