Merge branch 'main' into DrJKL-patch-1

.coderabbit.yaml

@@ -2,6 +2,7 @@ issue_enrichment:
   auto_enrich:
     enabled: true
 reviews:
+  profile: assertive
   high_level_summary: false
   request_changes_workflow: true
   auto_review:

.github/workflows/cla.yml

@@ -41,7 +41,7 @@ jobs:
 
           # Allowlist bots so they don't need to sign (optional, comma-separated).
           # *[bot] is a catch-all for any GitHub App bot account.
-          allowlist: actions-user,ampagent,claude,comfy-pr-bot,github-actions,*[bot]
+          allowlist: actions-user,ampagent,claude,coderabbitai[bot],comfy-pr-bot,dependabot[bot],github-actions[bot],copilot-swe-agent[bot],devin-ai-integration[bot],*[bot]
 
           # Custom PR comment messages
           custom-notsigned-prcomment: |

browser_tests/tests/vueNodes/interactions/node/tooltips.spec.ts

@@ -1,26 +0,0 @@
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '@e2e/fixtures/ComfyPage'
-
-test.describe('tooltips', { tag: '@vue-nodes' }, async () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.settings.setSetting('Comfy.EnableTooltips', true)
-    await comfyPage.settings.setSetting('LiteGraph.Node.TooltipDelay', 0)
-  })
-
-  test('widget value tooltips', async ({ comfyPage }) => {
-    const tooltip = comfyPage.page.locator('.p-tooltip-text')
-    await comfyPage.vueNodes.getWidgetByName('load check', 'ckpt_name').hover()
-    await expect(tooltip, 'displays for combos').toContainText('v1-5-pruned')
-
-    await comfyPage.vueNodes.getWidgetByName('ksampler', 'seed').hover()
-    await expect(tooltip, 'displays for numbers').toContainText('15668')
-
-    await comfyPage.vueNodes.getNodeLocator('6').getByLabel('text').hover()
-    await expect(tooltip).toBeVisible()
-    await expect(tooltip, "doesn't display for prompts").not.toContainText(
-      'purple galaxy bottle'
-    )
-  })
-})

src/renderer/extensions/vueNodes/composables/useProcessedWidgets.ts

@@ -46,12 +46,6 @@ import {
   getLocatorIdFromNodeData
 } from '@/utils/graphTraversalUtil'
 
-const TOOLTIP_VALUE_TYPES = ['asset', 'combo', 'number', 'text'] as const
-type TooltipValueType = (typeof TOOLTIP_VALUE_TYPES)[number]
-function isTooltipValueType(val: unknown): val is TooltipValueType {
-  return TOOLTIP_VALUE_TYPES.includes(val as TooltipValueType)
-}
-
 interface ProcessedWidget {
   advanced: boolean
   handleContextMenu: (e: PointerEvent) => void
@@ -73,7 +67,7 @@ interface ProcessedWidget {
 }
 
 interface WidgetUiCallbacks {
-  getTooltipConfig: (widget: SafeWidgetData, fullVal?: string) => TooltipOptions
+  getTooltipConfig: (widget: SafeWidgetData) => TooltipOptions
   handleNodeRightClick: (e: PointerEvent, nodeId: string) => void
 }
 
@@ -325,11 +319,7 @@ export function computeProcessedWidgets({
       executionErrorStore
     )
 
-    const valueTooltip =
-      isTooltipValueType(widget.type) && String(value).length > 10
-        ? String(value)
-        : undefined
-    const tooltipConfig = ui.getTooltipConfig(widget, valueTooltip)
+    const tooltipConfig = ui.getTooltipConfig(widget)
     const handleContextMenu = (e: PointerEvent) => {
       e.preventDefault()
       e.stopPropagation()
@@ -385,10 +375,7 @@ export function useProcessedWidgets(
   const { getWidgetTooltip, createTooltipConfig } = useNodeTooltips(nodeType)
 
   const ui: WidgetUiCallbacks = {
-    getTooltipConfig: (widget, fullValue = '') =>
-      createTooltipConfig(
-        [getWidgetTooltip(widget), fullValue].join('\n\n').trim()
-      ),
+    getTooltipConfig: (widget) => createTooltipConfig(getWidgetTooltip(widget)),
     handleNodeRightClick
   }