[refactor] use LGraphNodeProperties for color property tracking - addresses review feedback

Moves color and bgcolor property change tracking to the standard LGraphNodeProperties
system instead of manual event firing in setColorOption. This reduces custom code
and leverages the existing property instrumentation infrastructure.

Co-authored-by: DrJKL <[email protected]>

Author: christian-byrne

src/lib/litegraph/src/LGraphNode.ts

@@ -332,39 +332,14 @@ export class LGraphNode
 
   /** @inheritdoc {@link IColorable.setColorOption} */
   setColorOption(colorOption: ColorOption | null): void {
-    const oldColor = this.color
-    const oldBgcolor = this.bgcolor
-
     if (colorOption == null) {
       delete this.color
       delete this.bgcolor
     } else {
       this.color = colorOption.color
       this.bgcolor = colorOption.bgcolor
     }
-
-    // Trigger property change events for Vue node synchronization
-    if (this.graph) {
-      const newColor = this.color // undefined if deleted
-      const newBgcolor = this.bgcolor // undefined if deleted
-
-      if (oldColor !== newColor) {
-        this.graph.trigger('node:property:changed', {
-          nodeId: this.id,
-          property: 'color',
-          oldValue: oldColor,
-          newValue: newColor
-        })
-      }
-      if (oldBgcolor !== newBgcolor) {
-        this.graph.trigger('node:property:changed', {
-          nodeId: this.id,
-          property: 'bgcolor',
-          oldValue: oldBgcolor,
-          newValue: newBgcolor
-        })
-      }
-    }
+    // Property change events are now handled automatically by LGraphNodeProperties instrumentation
   }
 
   /** @inheritdoc {@link IColorable.getColorOption} */

src/lib/litegraph/src/LGraphNodeProperties.ts

@@ -6,7 +6,9 @@ import type { LGraphNode } from './LGraphNode'
 const DEFAULT_TRACKED_PROPERTIES: string[] = [
   'title',
   'flags.collapsed',
-  'mode'
+  'mode',
+  'color',
+  'bgcolor'
 ]
 
 /**