Merge pull request #263 from liam-hq/refactor-remove-set-edges

refactor: remove unused convertElkEdgesToEdges function

frontend/.changeset/popular-actors-smoke.md

@@ -0,0 +1,5 @@
+---
+"@liam-hq/erd-core": patch
+---
+
+refactor: remove unused convertElkEdgesToEdges function

frontend/packages/erd-core/src/components/ERDRenderer/ERDContent/useAutoLayout/getElkLayout.ts

@@ -1,7 +1,6 @@
 import type { Edge, Node } from '@xyflow/react'
 import type { ElkNode, LayoutOptions } from 'elkjs'
 import ELK from 'elkjs/lib/elk.bundled.js'
-import { convertElkEdgesToEdges } from './convertElkEdgesToEdges'
 import { convertElkNodesToNodes } from './convertElkNodesToNodes'
 import { convertNodesToElkNodes } from './convertNodesToElkNodes'
 
@@ -24,7 +23,7 @@ type Params = {
   edges: Edge[]
 }
 
-export async function getElkLayout({ nodes, edges }: Params): Promise<Params> {
+export async function getElkLayout({ nodes, edges }: Params): Promise<Node[]> {
   const graph: ElkNode = {
     id: 'root',
     layoutOptions,
@@ -38,14 +37,8 @@ export async function getElkLayout({ nodes, edges }: Params): Promise<Params> {
 
   const layout = await elk.layout(graph)
   if (!layout.children) {
-    return {
-      nodes,
-      edges,
-    }
+    return nodes
   }
 
-  return {
-    nodes: convertElkNodesToNodes(layout.children, nodes),
-    edges: convertElkEdgesToEdges(layout.edges ?? [], edges),
-  }
+  return convertElkNodesToNodes(layout.children, nodes)
 }

frontend/packages/erd-core/src/components/ERDRenderer/ERDContent/useAutoLayout/useAutoLayout.ts

@@ -4,21 +4,20 @@ import { getElkLayout } from './getElkLayout'
 
 export const useAutoLayout = () => {
   const nodesInitialized = useNodesInitialized()
-  const { getNodes, setNodes, getEdges, setEdges, fitView } = useReactFlow()
+  const { getNodes, setNodes, getEdges, fitView } = useReactFlow()
 
   const handleLayout = useCallback(async () => {
     const nodes = getNodes()
     const edges = getEdges()
 
-    const { nodes: newNodes, edges: newEdges } = await getElkLayout({
+    const newNodes = await getElkLayout({
       nodes,
       edges,
     })
 
     setNodes(newNodes)
-    setEdges(newEdges)
     setTimeout(() => fitView(), 0)
-  }, [getNodes, setNodes, getEdges, setEdges, fitView])
+  }, [getNodes, setNodes, getEdges, fitView])
 
   useEffect(() => {
     if (nodesInitialized) {

frontend/packages/erd-core/src/components/ERDRenderer/convertDBStructureToNodes.ts

@@ -46,9 +46,6 @@ export const convertDBStructureToNodes = ({
         ? null
         : `${rel.foreignTableName}-${rel.foreignColumnName}`,
     data: { relationship: rel },
-    style: {
-      opacity: 0,
-    },
   }))
 
   return { nodes, edges }