fix: harden linked widget interactions

Author: jaeone94

browser_tests/tests/vueNodes/widgets/linked/linkedStandardWidgets.spec.ts

@@ -166,6 +166,7 @@ test.describe(
       await comfyPage.page.keyboard.type('ignored input')
       await expect(focusedLinkedContent).toHaveCount(0)
 
+      await comfyPage.nextFrame()
       await expect
         .poll(() => Promise.all(widgetRefs.map((widget) => widget.getValue())))
         .toStrictEqual(valuesBefore)

src/renderer/extensions/vueNodes/components/NodeWidgets.test.ts

@@ -1,7 +1,7 @@
 /* eslint-disable testing-library/no-container */
 /* eslint-disable testing-library/no-node-access */
 import { createTestingPinia } from '@pinia/testing'
-import { render, screen } from '@testing-library/vue'
+import { fireEvent, render, screen } from '@testing-library/vue'
 import { setActivePinia } from 'pinia'
 import { nextTick } from 'vue'
 import { describe, expect, it, vi } from 'vitest'
@@ -19,6 +19,11 @@ import { createNodeExecutionId } from '@/types/nodeIdentification'
 import { widgetId } from '@/types/widgetId'
 
 const GRAPH_ID = 'graph-test'
+const mockShowNodeOptions = vi.hoisted(() => vi.fn())
+
+vi.mock('@/composables/graph/useMoreOptionsMenu', () => ({
+  showNodeOptions: mockShowNodeOptions
+}))
 
 vi.mock('@/renderer/core/canvas/canvasStore', () => ({
   useCanvasStore: () => ({
@@ -382,4 +387,39 @@ describe('NodeWidgets', () => {
     expect(control).toHaveAttribute('data-linked-display', 'control')
     expect(screen.queryByTestId('linked-widget-placeholder')).toBeNull()
   })
+
+  it('dispatches context menu actions from a mounted linked widget', async () => {
+    const linkedWidgetId = widgetId(GRAPH_ID, toNodeId('test_node'), 'prompt')
+    const nodeData = createMockNodeData('TestNode', [
+      createMockWidget({
+        widgetId: linkedWidgetId,
+        name: 'prompt',
+        nodeId: toNodeId('test_node'),
+        type: 'text',
+        slotMetadata: {
+          index: 0,
+          linked: true,
+          type: 'STRING'
+        }
+      })
+    ])
+
+    const { container } = renderComponent(nodeData, () => {
+      useWidgetValueStore().registerWidget(linkedWidgetId, {
+        type: 'text',
+        value: 'stale local prompt',
+        options: {}
+      })
+    })
+
+    const control = container.querySelector('.widget-stub')
+    expect(control).not.toBeNull()
+    await fireEvent.contextMenu(control!)
+
+    expect(mockShowNodeOptions).toHaveBeenCalledWith(
+      expect.any(MouseEvent),
+      'prompt',
+      toNodeId('test_node')
+    )
+  })
 })

src/renderer/extensions/vueNodes/widgets/components/WidgetSelectDropdown.test.ts

@@ -13,6 +13,12 @@ import WidgetSelectDropdown from '@/renderer/extensions/vueNodes/widgets/compone
 import type { SimplifiedWidget } from '@/types/simplifiedWidget'
 import { createMockWidget } from './widgetTestUtils'
 
+vi.mock('@/composables/useFeatureFlags', () => ({
+  useFeatureFlags: () => ({
+    flags: { modelUploadButtonEnabled: false }
+  })
+}))
+
 const mockCheckState = vi.hoisted(() => vi.fn())
 const mockAssetsData = vi.hoisted(() => ({ items: [] as AssetItem[] }))
 

src/renderer/extensions/vueNodes/widgets/components/WidgetTextarea.test.ts

@@ -174,6 +174,17 @@ describe('WidgetTextarea Value Binding', () => {
       expect(textarea.value).toBe('initial content')
     })
 
+    it('keeps an unlinked disabled textarea readonly without disabling it', () => {
+      const widget = createTextareaWidget('readable content', {
+        disabled: true
+      })
+      renderComponent(widget, 'readable content')
+
+      const textarea = screen.getByRole('textbox')
+      expect(textarea).toHaveAttribute('readonly')
+      expect(textarea).toBeEnabled()
+    })
+
     it('restores the textarea after a link is removed', async () => {
       const widget = createTextareaWidget('stale multiline', {
         disabled: true

src/renderer/extensions/vueNodes/widgets/components/WidgetTextarea.vue

@@ -32,7 +32,7 @@
       "
       :placeholder
       :readonly="isReadOnly"
-      :disabled="Boolean(widget.options?.disabled)"
+      :disabled="isLinked || undefined"
       :aria-hidden="isLinked ? 'true' : undefined"
       :inert="isLinked ? true : undefined"
       :tabindex="isLinked ? -1 : undefined"