From 09d17fec14347fc7885cab5f791c777f86e032c3 Mon Sep 17 00:00:00 2001
From: filtered <176114999+webfiltered@users.noreply.github.com>
Date: Mon, 12 May 2025 13:32:08 +1000
Subject: [PATCH] nit - prevent duplicate subgraph registrations

---
 src/services/subgraphService.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/services/subgraphService.ts b/src/services/subgraphService.ts
index 97d7f3cac..2c01351a1 100644
--- a/src/services/subgraphService.ts
+++ b/src/services/subgraphService.ts
@@ -13,7 +13,7 @@ import { useLitegraphService } from './litegraphService'
 
 export const useSubgraphService = () => {
   /** @todo Move to store */
-  const subgraphs: Subgraph[] = []
+  const subgraphs = new Map<string, Subgraph>()
 
   /** Loads a single subgraph definition and registers it with the node def store */
   const deserialiseSubgraph = (
@@ -63,7 +63,7 @@ export const useSubgraphService = () => {
 
     for (const subgraphData of graphData.definitions.subgraphs) {
       const subgraph =
-        subgraphs.find((x) => x.id === subgraphData.id) ??
+        subgraphs.get(subgraphData.id) ??
         comfyApp.graph.createSubgraph(subgraphData as ExportedSubgraph)
 
       // @ts-expect-error Zod
@@ -73,7 +73,7 @@ export const useSubgraphService = () => {
 
   /** Registers a new subgraph (e.g. user converted from nodes) */
   const registerNewSubgraph = (subgraph: Subgraph) => {
-    subgraphs.push(subgraph)
+    subgraphs.set(subgraph.id, subgraph)
 
     deserialiseSubgraph(subgraph, subgraph.asSerialisable())
   }