From 5223f27de682c1d8c9002cf21158255d477ef7b0 Mon Sep 17 00:00:00 2001 From: Subagent 5 Date: Thu, 29 Jan 2026 16:39:40 -0800 Subject: [PATCH] fix: address CodeRabbit review feedback - Add Comfy.Assets.UseAssetAPI toggle check (matches useComboWidget behavior) - Sync existing target widget value to asset widget (fixes placeholder issue) Amp-Thread-ID: https://ampcode.com/threads/T-019c0839-bbdc-754a-9d3b-151417058ded Co-authored-by: Amp --- src/extensions/core/widgetInputs.ts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/extensions/core/widgetInputs.ts b/src/extensions/core/widgetInputs.ts index e440d145ba..69636b608f 100644 --- a/src/extensions/core/widgetInputs.ts +++ b/src/extensions/core/widgetInputs.ts @@ -15,6 +15,7 @@ import type { import { assetService } from '@/platform/assets/services/assetService' import { createAssetWidget } from '@/platform/assets/utils/createAssetWidget' import { isCloud } from '@/platform/distribution/types' +import { useSettingStore } from '@/platform/settings/settingStore' import type { InputSpec } from '@/schemas/nodeDefSchema' import { app } from '@/scripts/app' import { @@ -232,14 +233,18 @@ export class PrimitiveNode extends LGraphNode { const [oldWidth, oldHeight] = this.size let widget: IBaseWidget - // Cloud: Use asset widget for model-eligible inputs + // Cloud: Use asset widget for model-eligible inputs when asset API is enabled if (isCloud && type === 'COMBO') { + const settingStore = useSettingStore() + const isUsingAssetAPI = settingStore.get('Comfy.Assets.UseAssetAPI') const isEligible = assetService.isAssetBrowserEligible( node.comfyClass, widgetName ) - if (isEligible) { + if (isUsingAssetAPI && isEligible) { widget = this._createAssetWidget(node, widgetName, inputData) + const theirWidget = node.widgets?.find((w) => w.name === widgetName) + if (theirWidget) widget.value = theirWidget.value this._finalizeWidget(widget, oldWidth, oldHeight, recreating) return }