From f1eb89af7aa32fffaa87a54692cc9e7ce0150ce4 Mon Sep 17 00:00:00 2001
From: Will Miao <willmiao.dev@gmail.com>
Date: Mon, 23 Feb 2026 16:47:09 +0800
Subject: [PATCH] refactor: Extract isNodeEnabled helper to eliminate mode
 check duplication

Consolidate node enabled state checks into isNodeEnabled() helper function
to improve code clarity and maintainability. Follows DRY principle.
---
 static/js/utils/uiHelpers.js | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/static/js/utils/uiHelpers.js b/static/js/utils/uiHelpers.js
index b897ea70..84b6ea54 100644
--- a/static/js/utils/uiHelpers.js
+++ b/static/js/utils/uiHelpers.js
@@ -457,6 +457,14 @@ function getWidgetNames(node) {
   return [];
 }
 
+function isNodeEnabled(node) {
+  if (!node) {
+    return false;
+  }
+  // ComfyUI node mode: 0 = Normal/Enabled, others = Always/Never/OnEvent
+  return node.mode === undefined || node.mode === 0;
+}
+
 function isAbsolutePath(path) {
   if (typeof path !== 'string') {
     return false;
@@ -507,10 +515,7 @@ export async function sendLoraToWorkflow(loraSyntax, replaceMode = false, syntax
   }
 
   const loraNodes = filterRegistryNodes(registry.nodes, (node) => {
-    if (!node) {
-      return false;
-    }
-    if (node.mode !== undefined && node.mode !== 0) {
+    if (!isNodeEnabled(node)) {
       return false;
     }
     if (node.capabilities && typeof node.capabilities === 'object') {
@@ -572,10 +577,7 @@ export async function sendModelPathToWorkflow(modelPath, options) {
   }
 
   const targetNodes = filterRegistryNodes(registry.nodes, (node) => {
-    if (!node) {
-      return false;
-    }
-    if (node.mode !== undefined && node.mode !== 0) {
+    if (!isNodeEnabled(node)) {
       return false;
     }
     const widgetNames = getWidgetNames(node);