fix: address CodeRabbit review feedback

- Remove unused keysBySource from store public API
- Fix socketless required input validation (widget-only inputs)

Amp-Thread-ID: https://ampcode.com/threads/T-019c0d45-1155-70d9-830a-ed6e50e23ee8
Co-authored-by: Amp <amp@ampcode.com>

.github/workflows/ci-lint-format.yaml

@@ -21,7 +21,6 @@ jobs:
         uses: actions/checkout@v6
         with:
           ref: ${{ !github.event.pull_request.head.repo.fork && github.head_ref || github.ref }}
-          token: ${{ secrets.PR_GH_TOKEN }}
 
       - name: Setup frontend
         uses: ./.github/actions/setup-frontend

.github/workflows/ci-tests-e2e.yaml

@@ -5,8 +5,8 @@ on:
   push:
     branches: [main, master, core/*, desktop/*]
   pull_request:
-    branches-ignore: [wip/*, draft/*, temp/*]
-  workflow_dispatch:
+    branches-ignore:
+      [wip/*, draft/*, temp/*, vue-nodes-migration, sno-playwright-*]
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}

.github/workflows/i18n-update-core.yaml

@@ -17,8 +17,6 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v6
-        with:
-          token: ${{ secrets.PR_GH_TOKEN }}
 
       # Setup playwright environment
       - name: Setup ComfyUI Frontend

.github/workflows/pr-update-playwright-expectations.yaml

@@ -173,7 +173,6 @@ jobs:
         uses: actions/checkout@v6
         with:
           ref: ${{ needs.setup.outputs.branch }}
-          token: ${{ secrets.PR_GH_TOKEN }}
 
       # Download all changed snapshot files from shards
       - name: Download snapshot artifacts
@@ -181,7 +180,7 @@ jobs:
         with:
           pattern: snapshots-shard-*
           path: ./downloaded-snapshots
-          merge-multiple: true
+          merge-multiple: false
 
       - name: List downloaded files
         run: |
@@ -207,13 +206,13 @@ jobs:
           echo "MERGING CHANGED SNAPSHOTS"
           echo "=========================================="
 
-          # Check if any artifacts were downloaded (merge-multiple puts files directly in path)
+          # Check if any artifacts were downloaded
           if [ ! -d "./downloaded-snapshots" ]; then
             echo "No snapshot artifacts to merge"
             echo "=========================================="
             echo "MERGE COMPLETE"
             echo "=========================================="
-            echo "Files merged: 0"
+            echo "Shards merged: 0"
             exit 0
           fi
 
@@ -223,29 +222,37 @@ jobs:
             exit 1
           fi
 
-          # Count files to merge
-          file_count=$(find ./downloaded-snapshots -type f | wc -l)
+          merged_count=0
 
-          if [ "$file_count" -eq 0 ]; then
-            echo "No snapshot files found in downloaded artifacts"
-            echo "=========================================="
-            echo "MERGE COMPLETE"
-            echo "=========================================="
-            echo "Files merged: 0"
-            exit 0
-          fi
+          # For each shard's changed files, copy them directly
+          for shard_dir in ./downloaded-snapshots/snapshots-shard-*/; do
+            if [ ! -d "$shard_dir" ]; then
+              continue
+            fi
 
-          echo "Merging $file_count snapshot file(s)..."
+            shard_name=$(basename "$shard_dir")
+            file_count=$(find "$shard_dir" -type f | wc -l)
 
-          # Copy all files directly, preserving directory structure
-          # With merge-multiple: true, files are directly in ./downloaded-snapshots/ without shard subdirs
-          cp -v -r ./downloaded-snapshots/* browser_tests/ 2>&1 | sed 's/^/  /'
+            if [ "$file_count" -eq 0 ]; then
+              echo "  $shard_name: no files"
+              continue
+            fi
+
+            echo "Processing $shard_name ($file_count file(s))..."
+
+            # Copy files directly, preserving directory structure
+            # Since files are already in correct structure (no browser_tests/ prefix), just copy them all
+            cp -v -r "$shard_dir"* browser_tests/ 2>&1 | sed 's/^/  /'
+
+            merged_count=$((merged_count + 1))
+            echo "  ✓ Merged"
+            echo ""
+          done
 
-          echo ""
           echo "=========================================="
           echo "MERGE COMPLETE"
           echo "=========================================="
-          echo "Files merged: $file_count"
+          echo "Shards merged: $merged_count"
 
       - name: Show changes
         run: |

.github/workflows/release-draft-create.yaml

@@ -50,7 +50,6 @@ jobs:
           ALGOLIA_API_KEY: ${{ secrets.ALGOLIA_API_KEY }}
           ENABLE_MINIFY: 'true'
           USE_PROD_CONFIG: 'true'
-          IS_NIGHTLY: ${{  case(github.ref == 'refs/heads/main', 'true', 'false') }}
         run: |
           pnpm install --frozen-lockfile
           pnpm build

browser_tests/fixtures/ComfyPage.ts

@@ -5,7 +5,7 @@ import * as fs from 'fs'
 
 import type { LGraphNode, LGraph } from '../../src/lib/litegraph/src/litegraph'
 import type { NodeId } from '../../src/platform/workflow/validation/schemas/workflowSchema'
-import type { KeyCombo } from '../../src/platform/keybindings'
+import type { KeyCombo } from '../../src/schemas/keyBindingSchema'
 import type { useWorkspaceStore } from '../../src/stores/workspaceStore'
 import { NodeBadgeMode } from '../../src/types/nodeSource'
 import { ComfyActionbar } from '../helpers/actionbar'

browser_tests/tests/actionbar.spec.ts

@@ -7,7 +7,7 @@ import { webSocketFixture } from '../fixtures/ws.ts'
 
 const test = mergeTests(comfyPageFixture, webSocketFixture)
 
-test.describe('Actionbar', { tag: '@ui' }, () => {
+test.describe('Actionbar', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
   })

browser_tests/tests/bottomPanelShortcuts.spec.ts

@@ -2,7 +2,7 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Bottom Panel Shortcuts', { tag: '@ui' }, () => {
+test.describe('Bottom Panel Shortcuts', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
   })

browser_tests/tests/browserTabTitle.spec.ts

@@ -2,7 +2,7 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Browser tab title', { tag: '@smoke' }, () => {
+test.describe('Browser tab title', () => {
   test.describe('Beta Menu', () => {
     test.beforeEach(async ({ comfyPage }) => {
       await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')

browser_tests/tests/changeTracker.spec.ts

@@ -19,7 +19,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Change Tracker', { tag: '@workflow' }, () => {
+test.describe('Change Tracker', () => {
   test.describe('Undo/Redo', () => {
     test.beforeEach(async ({ comfyPage }) => {
       await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')

browser_tests/tests/colorPalette.spec.ts

@@ -151,7 +151,7 @@ const customColorPalettes: Record<string, Palette> = {
   }
 }
 
-test.describe('Color Palette', { tag: ['@screenshot', '@settings'] }, () => {
+test.describe('Color Palette', () => {
   test('Can show custom color palette', async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.CustomColorPalettes', customColorPalettes)
     // Reload to apply the new setting. Setting Comfy.CustomColorPalettes directly
@@ -194,110 +194,104 @@ test.describe('Color Palette', { tag: ['@screenshot', '@settings'] }, () => {
   })
 })
 
-test.describe(
-  'Node Color Adjustments',
-  { tag: ['@screenshot', '@settings'] },
-  () => {
+test.describe('Node Color Adjustments', () => {
+  test.beforeEach(async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('nodes/every_node_color')
+  })
+
+  test('should adjust opacity via node opacity setting', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting('Comfy.Node.Opacity', 0.5)
+
+    // Drag mouse to force canvas to redraw
+    await comfyPage.page.mouse.move(0, 0)
+
+    await expect(comfyPage.canvas).toHaveScreenshot('node-opacity-0.5.png')
+
+    await comfyPage.setSetting('Comfy.Node.Opacity', 1.0)
+
+    await comfyPage.page.mouse.move(8, 8)
+    await expect(comfyPage.canvas).toHaveScreenshot('node-opacity-1.png')
+  })
+
+  test('should persist color adjustments when changing themes', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting('Comfy.Node.Opacity', 0.2)
+    await comfyPage.setSetting('Comfy.ColorPalette', 'arc')
+    await comfyPage.nextFrame()
+    await comfyPage.page.mouse.move(0, 0)
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'node-opacity-0.2-arc-theme.png'
+    )
+  })
+
+  test('should not serialize color adjustments in workflow', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting('Comfy.Node.Opacity', 0.5)
+    await comfyPage.setSetting('Comfy.ColorPalette', 'light')
+    await comfyPage.nextFrame()
+    const parsed = await (
+      await comfyPage.page.waitForFunction(
+        () => {
+          const workflow = localStorage.getItem('workflow')
+          if (!workflow) return null
+          try {
+            const data = JSON.parse(workflow)
+            return Array.isArray(data?.nodes) ? data : null
+          } catch {
+            return null
+          }
+        },
+        { timeout: 3000 }
+      )
+    ).jsonValue()
+    expect(parsed.nodes).toBeDefined()
+    expect(Array.isArray(parsed.nodes)).toBe(true)
+    for (const node of parsed.nodes) {
+      if (node.bgcolor) expect(node.bgcolor).not.toMatch(/hsla/)
+      if (node.color) expect(node.color).not.toMatch(/hsla/)
+    }
+  })
+
+  test('should lighten node colors when switching to light theme', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting('Comfy.ColorPalette', 'light')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('node-lightened-colors.png')
+  })
+
+  test.describe('Context menu color adjustments', () => {
     test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('nodes/every_node_color')
+      await comfyPage.setSetting('Comfy.ColorPalette', 'light')
+      await comfyPage.setSetting('Comfy.Node.Opacity', 0.3)
+      const node = await comfyPage.getFirstNodeRef()
+      await node?.clickContextMenuOption('Colors')
     })
 
-    test('should adjust opacity via node opacity setting', async ({
+    test('should persist color adjustments when changing custom node colors', async ({
       comfyPage
     }) => {
-      await comfyPage.setSetting('Comfy.Node.Opacity', 0.5)
-
-      // Drag mouse to force canvas to redraw
-      await comfyPage.page.mouse.move(0, 0)
-
-      await expect(comfyPage.canvas).toHaveScreenshot('node-opacity-0.5.png')
-
-      await comfyPage.setSetting('Comfy.Node.Opacity', 1.0)
-
-      await comfyPage.page.mouse.move(8, 8)
-      await expect(comfyPage.canvas).toHaveScreenshot('node-opacity-1.png')
-    })
-
-    test('should persist color adjustments when changing themes', async ({
-      comfyPage
-    }) => {
-      await comfyPage.setSetting('Comfy.Node.Opacity', 0.2)
-      await comfyPage.setSetting('Comfy.ColorPalette', 'arc')
-      await comfyPage.nextFrame()
-      await comfyPage.page.mouse.move(0, 0)
+      await comfyPage.page
+        .locator('.litemenu-entry.submenu span:has-text("red")')
+        .click()
       await expect(comfyPage.canvas).toHaveScreenshot(
-        'node-opacity-0.2-arc-theme.png'
+        'node-opacity-0.3-color-changed.png'
       )
     })
 
-    test('should not serialize color adjustments in workflow', async ({
+    test('should persist color adjustments when removing custom node color', async ({
       comfyPage
     }) => {
-      await comfyPage.setSetting('Comfy.Node.Opacity', 0.5)
-      await comfyPage.setSetting('Comfy.ColorPalette', 'light')
-      await comfyPage.nextFrame()
-      const parsed = await (
-        await comfyPage.page.waitForFunction(
-          () => {
-            const workflow = localStorage.getItem('workflow')
-            if (!workflow) return null
-            try {
-              const data = JSON.parse(workflow)
-              return Array.isArray(data?.nodes) ? data : null
-            } catch {
-              return null
-            }
-          },
-          { timeout: 3000 }
-        )
-      ).jsonValue()
-      expect(parsed.nodes).toBeDefined()
-      expect(Array.isArray(parsed.nodes)).toBe(true)
-      for (const node of parsed.nodes) {
-        if (node.bgcolor) expect(node.bgcolor).not.toMatch(/hsla/)
-        if (node.color) expect(node.color).not.toMatch(/hsla/)
-      }
-    })
-
-    test('should lighten node colors when switching to light theme', async ({
-      comfyPage
-    }) => {
-      await comfyPage.setSetting('Comfy.ColorPalette', 'light')
-      await comfyPage.nextFrame()
+      await comfyPage.page
+        .locator('.litemenu-entry.submenu span:has-text("No color")')
+        .click()
       await expect(comfyPage.canvas).toHaveScreenshot(
-        'node-lightened-colors.png'
+        'node-opacity-0.3-color-removed.png'
       )
     })
-
-    test.describe('Context menu color adjustments', () => {
-      test.beforeEach(async ({ comfyPage }) => {
-        await comfyPage.setSetting('Comfy.ColorPalette', 'light')
-        await comfyPage.setSetting('Comfy.Node.Opacity', 0.3)
-        const node = await comfyPage.getFirstNodeRef()
-        await node?.clickContextMenuOption('Colors')
-      })
-
-      test('should persist color adjustments when changing custom node colors', async ({
-        comfyPage
-      }) => {
-        await comfyPage.page
-          .locator('.litemenu-entry.submenu span:has-text("red")')
-          .click()
-        await expect(comfyPage.canvas).toHaveScreenshot(
-          'node-opacity-0.3-color-changed.png'
-        )
-      })
-
-      test('should persist color adjustments when removing custom node color', async ({
-        comfyPage
-      }) => {
-        await comfyPage.page
-          .locator('.litemenu-entry.submenu span:has-text("No color")')
-          .click()
-        await expect(comfyPage.canvas).toHaveScreenshot(
-          'node-opacity-0.3-color-removed.png'
-        )
-      })
-    })
-  }
-)
+  })
+})

browser_tests/tests/commands.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Keybindings', { tag: '@keyboard' }, () => {
+test.describe('Keybindings', () => {
   test('Should execute command', async ({ comfyPage }) => {
     await comfyPage.registerCommand('TestCommand', () => {
       window['foo'] = true

browser_tests/tests/copyPaste.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Copy Paste', { tag: ['@screenshot', '@workflow'] }, () => {
+test.describe('Copy Paste', () => {
   test('Can copy and paste node', async ({ comfyPage }) => {
     await comfyPage.clickEmptyLatentNode()
     await comfyPage.page.mouse.move(10, 10)

browser_tests/tests/customIcons.spec.ts

@@ -22,7 +22,7 @@ async function verifyCustomIconSvg(iconElement: Locator) {
   expect(decodedSvg).toContain("<svg xmlns='http://www.w3.org/2000/svg'")
 }
 
-test.describe('Custom Icons', { tag: '@settings' }, () => {
+test.describe('Custom Icons', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
   })

browser_tests/tests/dialog.spec.ts

@@ -1,14 +1,14 @@
 import type { Locator } from '@playwright/test'
 import { expect } from '@playwright/test'
 
-import type { Keybinding } from '../../src/platform/keybindings'
+import type { Keybinding } from '../../src/schemas/keyBindingSchema'
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
 test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Load workflow warning', { tag: '@ui' }, () => {
+test.describe('Load workflow warning', () => {
   test('Should display a warning when loading a workflow with missing nodes', async ({
     comfyPage
   }) => {

browser_tests/tests/domWidget.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('DOM Widget', { tag: '@widget' }, () => {
+test.describe('DOM Widget', () => {
   test('Collapsed multiline textarea is not visible', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('widgets/collapsed_multiline')
     const textareaWidget = comfyPage.page.locator('.comfy-multiline-input')
@@ -29,16 +29,12 @@ test.describe('DOM Widget', { tag: '@widget' }, () => {
     await expect(lastMultiline).not.toBeVisible()
   })
 
-  test(
-    'Position update when entering focus mode',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
-      await comfyPage.executeCommand('Workspace.ToggleFocusMode')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot('focus-mode-on.png')
-    }
-  )
+  test('Position update when entering focus mode', async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
+    await comfyPage.executeCommand('Workspace.ToggleFocusMode')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('focus-mode-on.png')
+  })
 
   // No DOM widget should be created by creation of interim LGraphNode objects.
   test('Copy node with DOM widget by dragging + alt', async ({ comfyPage }) => {

browser_tests/tests/execution.spec.ts

@@ -6,48 +6,40 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Execution', { tag: ['@smoke', '@workflow'] }, () => {
-  test(
-    'Report error on unconnected slot',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.disconnectEdge()
-      await comfyPage.clickEmptySpace()
+test.describe('Execution', () => {
+  test('Report error on unconnected slot', async ({ comfyPage }) => {
+    await comfyPage.disconnectEdge()
+    await comfyPage.clickEmptySpace()
 
-      await comfyPage.executeCommand('Comfy.QueuePrompt')
-      await expect(comfyPage.page.locator('.comfy-error-report')).toBeVisible()
-      await comfyPage.page.locator('.p-dialog-close-button').click()
-      await comfyPage.page.locator('.comfy-error-report').waitFor({
-        state: 'hidden'
-      })
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'execution-error-unconnected-slot.png'
-      )
-    }
-  )
+    await comfyPage.executeCommand('Comfy.QueuePrompt')
+    await expect(comfyPage.page.locator('.comfy-error-report')).toBeVisible()
+    await comfyPage.page.locator('.p-dialog-close-button').click()
+    await comfyPage.page.locator('.comfy-error-report').waitFor({
+      state: 'hidden'
+    })
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'execution-error-unconnected-slot.png'
+    )
+  })
 })
 
-test.describe(
-  'Execute to selected output nodes',
-  { tag: ['@smoke', '@workflow'] },
-  () => {
-    test('Execute to selected output nodes', async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('execution/partial_execution')
-      const input = await comfyPage.getNodeRefById(3)
-      const output1 = await comfyPage.getNodeRefById(1)
-      const output2 = await comfyPage.getNodeRefById(4)
+test.describe('Execute to selected output nodes', () => {
+  test('Execute to selected output nodes', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('execution/partial_execution')
+    const input = await comfyPage.getNodeRefById(3)
+    const output1 = await comfyPage.getNodeRefById(1)
+    const output2 = await comfyPage.getNodeRefById(4)
+    expect(await (await input.getWidget(0)).getValue()).toBe('foo')
+    expect(await (await output1.getWidget(0)).getValue()).toBe('')
+    expect(await (await output2.getWidget(0)).getValue()).toBe('')
+
+    await output1.click('title')
+
+    await comfyPage.executeCommand('Comfy.QueueSelectedOutputNodes')
+    await expect(async () => {
       expect(await (await input.getWidget(0)).getValue()).toBe('foo')
-      expect(await (await output1.getWidget(0)).getValue()).toBe('')
+      expect(await (await output1.getWidget(0)).getValue()).toBe('foo')
       expect(await (await output2.getWidget(0)).getValue()).toBe('')
-
-      await output1.click('title')
-
-      await comfyPage.executeCommand('Comfy.QueueSelectedOutputNodes')
-      await expect(async () => {
-        expect(await (await input.getWidget(0)).getValue()).toBe('foo')
-        expect(await (await output1.getWidget(0)).getValue()).toBe('foo')
-        expect(await (await output2.getWidget(0)).getValue()).toBe('')
-      }).toPass({ timeout: 2_000 })
-    })
-  }
-)
+    }).toPass({ timeout: 2_000 })
+  })
+})

browser_tests/tests/featureFlags.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Feature Flags', { tag: ['@slow', '@settings'] }, () => {
+test.describe('Feature Flags', () => {
   test('Client and server exchange feature flags on connection', async ({
     comfyPage
   }) => {

browser_tests/tests/graph.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Graph', { tag: ['@smoke', '@canvas'] }, () => {
+test.describe('Graph', () => {
   // Should be able to fix link input slot index after swap the input order
   // Ref: https://github.com/Comfy-Org/ComfyUI_frontend/issues/3348
   test('Fix link input slots', async ({ comfyPage }) => {

browser_tests/tests/graphCanvasMenu.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Graph Canvas Menu', { tag: ['@screenshot', '@canvas'] }, () => {
+test.describe('Graph Canvas Menu', () => {
   test.beforeEach(async ({ comfyPage }) => {
     // Set link render mode to spline to make sure it's not affected by other tests'
     // side effects.
@@ -15,33 +15,29 @@ test.describe('Graph Canvas Menu', { tag: ['@screenshot', '@canvas'] }, () => {
     await comfyPage.setSetting('Comfy.Graph.CanvasMenu', true)
   })
 
-  test(
-    'Can toggle link visibility',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      const button = comfyPage.page.getByTestId('toggle-link-visibility-button')
-      await button.click()
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'canvas-with-hidden-links.png'
-      )
-      const hiddenLinkRenderMode = await comfyPage.page.evaluate(() => {
-        return window['LiteGraph'].HIDDEN_LINK
-      })
-      expect(await comfyPage.getSetting('Comfy.LinkRenderMode')).toBe(
-        hiddenLinkRenderMode
-      )
+  test('Can toggle link visibility', async ({ comfyPage }) => {
+    const button = comfyPage.page.getByTestId('toggle-link-visibility-button')
+    await button.click()
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'canvas-with-hidden-links.png'
+    )
+    const hiddenLinkRenderMode = await comfyPage.page.evaluate(() => {
+      return window['LiteGraph'].HIDDEN_LINK
+    })
+    expect(await comfyPage.getSetting('Comfy.LinkRenderMode')).toBe(
+      hiddenLinkRenderMode
+    )
 
-      await button.click()
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'canvas-with-visible-links.png'
-      )
-      expect(await comfyPage.getSetting('Comfy.LinkRenderMode')).not.toBe(
-        hiddenLinkRenderMode
-      )
-    }
-  )
+    await button.click()
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'canvas-with-visible-links.png'
+    )
+    expect(await comfyPage.getSetting('Comfy.LinkRenderMode')).not.toBe(
+      hiddenLinkRenderMode
+    )
+  })
 
   test('Toggle minimap button is clickable and has correct test id', async ({
     comfyPage

browser_tests/tests/groupNode.spec.ts

@@ -8,7 +8,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Group Node', { tag: '@node' }, () => {
+test.describe('Group Node', () => {
   test.describe('Node library sidebar', () => {
     const groupNodeName = 'DefautWorkflowGroupNode'
     const groupNodeCategory = 'group nodes>workflow'
@@ -89,20 +89,16 @@ test.describe('Group Node', { tag: '@node' }, () => {
   // does not have a v-model on the query, so we cannot observe the raw
   // query update, and thus cannot set the spinning state between the raw query
   // update and the debounced search update.
-  test.skip(
-    'Can be added to canvas using search',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      const groupNodeName = 'DefautWorkflowGroupNode'
-      await comfyPage.convertAllNodesToGroupNode(groupNodeName)
-      await comfyPage.doubleClickCanvas()
-      await comfyPage.nextFrame()
-      await comfyPage.searchBox.fillAndSelectFirstNode(groupNodeName)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'group-node-copy-added-from-search.png'
-      )
-    }
-  )
+  test.skip('Can be added to canvas using search', async ({ comfyPage }) => {
+    const groupNodeName = 'DefautWorkflowGroupNode'
+    await comfyPage.convertAllNodesToGroupNode(groupNodeName)
+    await comfyPage.doubleClickCanvas()
+    await comfyPage.nextFrame()
+    await comfyPage.searchBox.fillAndSelectFirstNode(groupNodeName)
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'group-node-copy-added-from-search.png'
+    )
+  })
 
   test('Displays tooltip on title hover', async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.EnableTooltips', true)

browser_tests/tests/interaction.spec.ts

@@ -13,7 +13,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Item Interaction', { tag: ['@screenshot', '@node'] }, () => {
+test.describe('Item Interaction', () => {
   test('Can select/delete all items', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('groups/mixed_graph_items')
     await comfyPage.canvas.press('Control+a')
@@ -60,17 +60,13 @@ test.describe('Node Interaction', () => {
       })
     })
 
-    test(
-      '@2x Can highlight selected',
-      { tag: '@screenshot' },
-      async ({ comfyPage }) => {
-        await expect(comfyPage.canvas).toHaveScreenshot('default.png')
-        await comfyPage.clickTextEncodeNode1()
-        await expect(comfyPage.canvas).toHaveScreenshot('selected-node1.png')
-        await comfyPage.clickTextEncodeNode2()
-        await expect(comfyPage.canvas).toHaveScreenshot('selected-node2.png')
-      }
-    )
+    test('@2x Can highlight selected', async ({ comfyPage }) => {
+      await expect(comfyPage.canvas).toHaveScreenshot('default.png')
+      await comfyPage.clickTextEncodeNode1()
+      await expect(comfyPage.canvas).toHaveScreenshot('selected-node1.png')
+      await comfyPage.clickTextEncodeNode2()
+      await expect(comfyPage.canvas).toHaveScreenshot('selected-node2.png')
+    })
 
     const dragSelectNodes = async (
       comfyPage: ComfyPage,
@@ -154,12 +150,12 @@ test.describe('Node Interaction', () => {
     })
   })
 
-  test('Can drag node', { tag: '@screenshot' }, async ({ comfyPage }) => {
+  test('Can drag node', async ({ comfyPage }) => {
     await comfyPage.dragNode2()
     await expect(comfyPage.canvas).toHaveScreenshot('dragged-node1.png')
   })
 
-  test.describe('Edge Interaction', { tag: '@screenshot' }, () => {
+  test.describe('Edge Interaction', () => {
     test.beforeEach(async ({ comfyPage }) => {
       await comfyPage.setSetting('Comfy.LinkRelease.Action', 'no action')
       await comfyPage.setSetting('Comfy.LinkRelease.ActionShift', 'no action')
@@ -226,18 +222,12 @@ test.describe('Node Interaction', () => {
     })
   })
 
-  test(
-    'Can adjust widget value',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.adjustWidgetValue()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'adjusted-widget-value.png'
-      )
-    }
-  )
+  test('Can adjust widget value', async ({ comfyPage }) => {
+    await comfyPage.adjustWidgetValue()
+    await expect(comfyPage.canvas).toHaveScreenshot('adjusted-widget-value.png')
+  })
 
-  test('Link snap to slot', { tag: '@screenshot' }, async ({ comfyPage }) => {
+  test('Link snap to slot', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('links/snap_to_slot')
     await expect(comfyPage.canvas).toHaveScreenshot('snap_to_slot.png')
 
@@ -254,67 +244,57 @@ test.describe('Node Interaction', () => {
     await expect(comfyPage.canvas).toHaveScreenshot('snap_to_slot_linked.png')
   })
 
-  test(
-    'Can batch move links by drag with shift',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('links/batch_move_links')
-      await expect(comfyPage.canvas).toHaveScreenshot('batch_move_links.png')
+  test('Can batch move links by drag with shift', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('links/batch_move_links')
+    await expect(comfyPage.canvas).toHaveScreenshot('batch_move_links.png')
 
-      const outputSlot1Pos = {
-        x: 304,
-        y: 127
-      }
-      const outputSlot2Pos = {
-        x: 307,
-        y: 310
-      }
-
-      await comfyPage.page.keyboard.down('Shift')
-      await comfyPage.dragAndDrop(outputSlot1Pos, outputSlot2Pos)
-      await comfyPage.page.keyboard.up('Shift')
-
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'batch_move_links_moved.png'
-      )
+    const outputSlot1Pos = {
+      x: 304,
+      y: 127
     }
-  )
-
-  test(
-    'Can batch disconnect links with ctrl+alt+click',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      const loadCheckpointClipSlotPos = {
-        x: 332,
-        y: 508
-      }
-      await comfyPage.canvas.click({
-        modifiers: ['Control', 'Alt'],
-        position: loadCheckpointClipSlotPos
-      })
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'batch-disconnect-links-disconnected.png'
-      )
+    const outputSlot2Pos = {
+      x: 307,
+      y: 310
     }
-  )
 
-  test(
-    'Can toggle dom widget node open/closed',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await expect(comfyPage.canvas).toHaveScreenshot('default.png')
-      await comfyPage.clickTextEncodeNodeToggler()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'text-encode-toggled-off.png'
-      )
-      await comfyPage.delay(1000)
-      await comfyPage.clickTextEncodeNodeToggler()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'text-encode-toggled-back-open.png'
-      )
+    await comfyPage.page.keyboard.down('Shift')
+    await comfyPage.dragAndDrop(outputSlot1Pos, outputSlot2Pos)
+    await comfyPage.page.keyboard.up('Shift')
+
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'batch_move_links_moved.png'
+    )
+  })
+
+  test('Can batch disconnect links with ctrl+alt+click', async ({
+    comfyPage
+  }) => {
+    const loadCheckpointClipSlotPos = {
+      x: 332,
+      y: 508
     }
-  )
+    await comfyPage.canvas.click({
+      modifiers: ['Control', 'Alt'],
+      position: loadCheckpointClipSlotPos
+    })
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'batch-disconnect-links-disconnected.png'
+    )
+  })
+
+  test('Can toggle dom widget node open/closed', async ({ comfyPage }) => {
+    await expect(comfyPage.canvas).toHaveScreenshot('default.png')
+    await comfyPage.clickTextEncodeNodeToggler()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'text-encode-toggled-off.png'
+    )
+    await comfyPage.delay(1000)
+    await comfyPage.clickTextEncodeNodeToggler()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'text-encode-toggled-back-open.png'
+    )
+  })
 
   test('Can close prompt dialog with canvas click (number widget)', async ({
     comfyPage
@@ -361,23 +341,19 @@ test.describe('Node Interaction', () => {
     await expect(legacyPrompt).toBeHidden()
   })
 
-  test(
-    'Can double click node title to edit',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('nodes/single_ksampler')
-      await comfyPage.canvas.dblclick({
-        position: {
-          x: 50,
-          y: 10
-        },
-        delay: 5
-      })
-      await comfyPage.page.keyboard.type('Hello World')
-      await comfyPage.page.keyboard.press('Enter')
-      await expect(comfyPage.canvas).toHaveScreenshot('node-title-edited.png')
-    }
-  )
+  test('Can double click node title to edit', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('nodes/single_ksampler')
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: 50,
+        y: 10
+      },
+      delay: 5
+    })
+    await comfyPage.page.keyboard.type('Hello World')
+    await comfyPage.page.keyboard.press('Enter')
+    await expect(comfyPage.canvas).toHaveScreenshot('node-title-edited.png')
+  })
 
   test('Double click node body does not trigger edit', async ({
     comfyPage
@@ -393,41 +369,29 @@ test.describe('Node Interaction', () => {
     expect(await comfyPage.page.locator('.node-title-editor').count()).toBe(0)
   })
 
-  test(
-    'Can group selected nodes',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.GroupSelectedNodes.Padding', 10)
-      await comfyPage.select2Nodes()
-      await comfyPage.page.keyboard.down('Control')
-      await comfyPage.page.keyboard.press('KeyG')
-      await comfyPage.page.keyboard.up('Control')
-      await comfyPage.nextFrame()
-      // Confirm group title
-      await comfyPage.page.keyboard.press('Enter')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'group-selected-nodes.png'
-      )
-    }
-  )
+  test('Can group selected nodes', async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.GroupSelectedNodes.Padding', 10)
+    await comfyPage.select2Nodes()
+    await comfyPage.page.keyboard.down('Control')
+    await comfyPage.page.keyboard.press('KeyG')
+    await comfyPage.page.keyboard.up('Control')
+    await comfyPage.nextFrame()
+    // Confirm group title
+    await comfyPage.page.keyboard.press('Enter')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('group-selected-nodes.png')
+  })
 
-  test(
-    'Can fit group to contents',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('groups/oversized_group')
-      await comfyPage.ctrlA()
-      await comfyPage.nextFrame()
-      await comfyPage.executeCommand('Comfy.Graph.FitGroupToContents')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'group-fit-to-contents.png'
-      )
-    }
-  )
+  test('Can fit group to contents', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('groups/oversized_group')
+    await comfyPage.ctrlA()
+    await comfyPage.nextFrame()
+    await comfyPage.executeCommand('Comfy.Graph.FitGroupToContents')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('group-fit-to-contents.png')
+  })
 
-  test('Can pin/unpin nodes', { tag: '@screenshot' }, async ({ comfyPage }) => {
+  test('Can pin/unpin nodes', async ({ comfyPage }) => {
     await comfyPage.select2Nodes()
     await comfyPage.executeCommand('Comfy.Canvas.ToggleSelectedNodes.Pin')
     await comfyPage.nextFrame()
@@ -437,22 +401,20 @@ test.describe('Node Interaction', () => {
     await expect(comfyPage.canvas).toHaveScreenshot('nodes-unpinned.png')
   })
 
-  test(
-    'Can bypass/unbypass nodes with keyboard shortcut',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.select2Nodes()
-      await comfyPage.canvas.press('Control+b')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot('nodes-bypassed.png')
-      await comfyPage.canvas.press('Control+b')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot('nodes-unbypassed.png')
-    }
-  )
+  test('Can bypass/unbypass nodes with keyboard shortcut', async ({
+    comfyPage
+  }) => {
+    await comfyPage.select2Nodes()
+    await comfyPage.canvas.press('Control+b')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('nodes-bypassed.png')
+    await comfyPage.canvas.press('Control+b')
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('nodes-unbypassed.png')
+  })
 })
 
-test.describe('Group Interaction', { tag: '@screenshot' }, () => {
+test.describe('Group Interaction', () => {
   test('Can double click group title to edit', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('groups/single_group')
     await comfyPage.canvas.dblclick({
@@ -468,7 +430,7 @@ test.describe('Group Interaction', { tag: '@screenshot' }, () => {
   })
 })
 
-test.describe('Canvas Interaction', { tag: '@screenshot' }, () => {
+test.describe('Canvas Interaction', () => {
   test('Can zoom in/out', async ({ comfyPage }) => {
     await comfyPage.zoom(-100)
     await expect(comfyPage.canvas).toHaveScreenshot('zoomed-in.png')
@@ -670,7 +632,7 @@ test.describe('Widget Interaction', () => {
   })
 })
 
-test.describe('Load workflow', { tag: '@screenshot' }, () => {
+test.describe('Load workflow', () => {
   test('Can load workflow with string node id', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('nodes/string_node_id')
     await expect(comfyPage.canvas).toHaveScreenshot('string_node_id.png')
@@ -862,7 +824,7 @@ test.describe('Viewport settings', () => {
   })
 })
 
-test.describe('Canvas Navigation', { tag: '@screenshot' }, () => {
+test.describe('Canvas Navigation', () => {
   test.describe('Legacy Mode', () => {
     test.beforeEach(async ({ comfyPage }) => {
       await comfyPage.setSetting('Comfy.Canvas.NavigationMode', 'legacy')

browser_tests/tests/keybindings.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Keybindings', { tag: '@keyboard' }, () => {
+test.describe('Keybindings', () => {
   test('Should not trigger non-modifier keybinding when typing in input fields', async ({
     comfyPage
   }) => {

browser_tests/tests/litegraphEvent.spec.ts

@@ -14,7 +14,7 @@ function listenForEvent(): Promise<Event> {
   })
 }
 
-test.describe('Canvas Event', { tag: '@canvas' }, () => {
+test.describe('Canvas Event', () => {
   test('Emit litegraph:canvas empty-release', async ({ comfyPage }) => {
     const eventPromise = comfyPage.page.evaluate(listenForEvent)
     const disconnectPromise = comfyPage.disconnectEdge()

browser_tests/tests/loadWorkflowInMedia.spec.ts

@@ -6,50 +6,46 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe(
-  'Load Workflow in Media',
-  { tag: ['@screenshot', '@workflow'] },
-  () => {
-    const fileNames = [
-      'workflow.webp',
-      'edited_workflow.webp',
-      'no_workflow.webp',
-      'large_workflow.webp',
-      'workflow_prompt_parameters.png',
-      'workflow.webm',
-      // Skipped due to 3d widget unstable visual result.
-      // 3d widget shows grid after fully loaded.
-      // 'workflow.glb',
-      'workflow.mp4',
-      'workflow.mov',
-      'workflow.m4v',
-      'workflow.svg'
-      // TODO: Re-enable after fixing test asset to use core nodes only
-      // Currently opens missing nodes dialog which is outside scope of AVIF loading functionality
-      // 'workflow.avif'
-    ]
-    fileNames.forEach(async (fileName) => {
-      test(`Load workflow in ${fileName} (drop from filesystem)`, async ({
-        comfyPage
-      }) => {
-        await comfyPage.dragAndDropFile(`workflowInMedia/${fileName}`)
-        await expect(comfyPage.canvas).toHaveScreenshot(`${fileName}.png`)
-      })
+test.describe('Load Workflow in Media', () => {
+  const fileNames = [
+    'workflow.webp',
+    'edited_workflow.webp',
+    'no_workflow.webp',
+    'large_workflow.webp',
+    'workflow_prompt_parameters.png',
+    'workflow.webm',
+    // Skipped due to 3d widget unstable visual result.
+    // 3d widget shows grid after fully loaded.
+    // 'workflow.glb',
+    'workflow.mp4',
+    'workflow.mov',
+    'workflow.m4v',
+    'workflow.svg'
+    // TODO: Re-enable after fixing test asset to use core nodes only
+    // Currently opens missing nodes dialog which is outside scope of AVIF loading functionality
+    // 'workflow.avif'
+  ]
+  fileNames.forEach(async (fileName) => {
+    test(`Load workflow in ${fileName} (drop from filesystem)`, async ({
+      comfyPage
+    }) => {
+      await comfyPage.dragAndDropFile(`workflowInMedia/${fileName}`)
+      await expect(comfyPage.canvas).toHaveScreenshot(`${fileName}.png`)
     })
+  })
 
-    const urls = [
-      'https://comfyanonymous.github.io/ComfyUI_examples/hidream/hidream_dev_example.png'
-    ]
-    urls.forEach(async (url) => {
-      test(`Load workflow from URL ${url} (drop from different browser tabs)`, async ({
-        comfyPage
-      }) => {
-        await comfyPage.dragAndDropURL(url)
-        const readableName = url.split('/').pop()
-        await expect(comfyPage.canvas).toHaveScreenshot(
-          `dropped_workflow_url_${readableName}.png`
-        )
-      })
+  const urls = [
+    'https://comfyanonymous.github.io/ComfyUI_examples/hidream/hidream_dev_example.png'
+  ]
+  urls.forEach(async (url) => {
+    test(`Load workflow from URL ${url} (drop from different browser tabs)`, async ({
+      comfyPage
+    }) => {
+      await comfyPage.dragAndDropURL(url)
+      const readableName = url.split('/').pop()
+      await expect(comfyPage.canvas).toHaveScreenshot(
+        `dropped_workflow_url_${readableName}.png`
+      )
     })
-  }
-)
+  })
+})

browser_tests/tests/lodThreshold.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('LOD Threshold', { tag: ['@screenshot', '@canvas'] }, () => {
+test.describe('LOD Threshold', () => {
   test('Should switch to low quality mode at correct zoom threshold', async ({
     comfyPage
   }) => {
@@ -149,55 +149,53 @@ test.describe('LOD Threshold', { tag: ['@screenshot', '@canvas'] }, () => {
     expect(state.scale).toBeLessThan(0.2) // Very zoomed out
   })
 
-  test(
-    'Should show visual difference between LOD on and off',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      // Load a workflow with text-heavy nodes for clear visual difference
-      await comfyPage.loadWorkflow('default')
+  test('Should show visual difference between LOD on and off', async ({
+    comfyPage
+  }) => {
+    // Load a workflow with text-heavy nodes for clear visual difference
+    await comfyPage.loadWorkflow('default')
 
-      // Set zoom level clearly below the threshold to ensure LOD activates
-      const targetZoom = 0.4 // Well below default threshold of ~0.571
+    // Set zoom level clearly below the threshold to ensure LOD activates
+    const targetZoom = 0.4 // Well below default threshold of ~0.571
 
-      // Zoom to target level
-      await comfyPage.page.evaluate((zoom) => {
-        window['app'].canvas.ds.scale = zoom
-        window['app'].canvas.setDirty(true, true)
-      }, targetZoom)
-      await comfyPage.nextFrame()
+    // Zoom to target level
+    await comfyPage.page.evaluate((zoom) => {
+      window['app'].canvas.ds.scale = zoom
+      window['app'].canvas.setDirty(true, true)
+    }, targetZoom)
+    await comfyPage.nextFrame()
 
-      // Take snapshot with LOD active (default 8px setting)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'lod-comparison-low-quality.png'
-      )
+    // Take snapshot with LOD active (default 8px setting)
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'lod-comparison-low-quality.png'
+    )
 
-      const lowQualityState = await comfyPage.page.evaluate(() => {
-        const canvas = window['app'].canvas
-        return {
-          lowQuality: canvas.low_quality,
-          scale: canvas.ds.scale
-        }
-      })
-      expect(lowQualityState.lowQuality).toBe(true)
+    const lowQualityState = await comfyPage.page.evaluate(() => {
+      const canvas = window['app'].canvas
+      return {
+        lowQuality: canvas.low_quality,
+        scale: canvas.ds.scale
+      }
+    })
+    expect(lowQualityState.lowQuality).toBe(true)
 
-      // Disable LOD to see high quality at same zoom
-      await comfyPage.setSetting('LiteGraph.Canvas.MinFontSizeForLOD', 0)
-      await comfyPage.nextFrame()
+    // Disable LOD to see high quality at same zoom
+    await comfyPage.setSetting('LiteGraph.Canvas.MinFontSizeForLOD', 0)
+    await comfyPage.nextFrame()
 
-      // Take snapshot with LOD disabled (full quality at same zoom)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'lod-comparison-high-quality.png'
-      )
+    // Take snapshot with LOD disabled (full quality at same zoom)
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'lod-comparison-high-quality.png'
+    )
 
-      const highQualityState = await comfyPage.page.evaluate(() => {
-        const canvas = window['app'].canvas
-        return {
-          lowQuality: canvas.low_quality,
-          scale: canvas.ds.scale
-        }
-      })
-      expect(highQualityState.lowQuality).toBe(false)
-      expect(highQualityState.scale).toBeCloseTo(targetZoom, 2)
-    }
-  )
+    const highQualityState = await comfyPage.page.evaluate(() => {
+      const canvas = window['app'].canvas
+      return {
+        lowQuality: canvas.low_quality,
+        scale: canvas.ds.scale
+      }
+    })
+    expect(highQualityState.lowQuality).toBe(false)
+    expect(highQualityState.scale).toBeCloseTo(targetZoom, 2)
+  })
 })

browser_tests/tests/menu.spec.ts

@@ -2,7 +2,7 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Menu', { tag: '@ui' }, () => {
+test.describe('Menu', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
   })

browser_tests/tests/minimap.spec.ts

@@ -2,7 +2,7 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Minimap', { tag: '@canvas' }, () => {
+test.describe('Minimap', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
     await comfyPage.setSetting('Comfy.Minimap.Visible', true)

browser_tests/tests/mobileBaseline.spec.ts

@@ -1,39 +1,35 @@
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 import { expect } from '@playwright/test'
 
-test.describe(
-  'Mobile Baseline Snapshots',
-  { tag: ['@mobile', '@screenshot'] },
-  () => {
-    test('@mobile empty canvas', async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.ConfirmClear', false)
-      await comfyPage.executeCommand('Comfy.ClearWorkflow')
-      await expect(async () => {
-        expect(await comfyPage.getGraphNodesCount()).toBe(0)
-      }).toPass({ timeout: 256 })
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot('mobile-empty-canvas.png')
-    })
+test.describe('Mobile Baseline Snapshots', () => {
+  test('@mobile empty canvas', async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.ConfirmClear', false)
+    await comfyPage.executeCommand('Comfy.ClearWorkflow')
+    await expect(async () => {
+      expect(await comfyPage.getGraphNodesCount()).toBe(0)
+    }).toPass({ timeout: 256 })
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('mobile-empty-canvas.png')
+  })
 
-    test('@mobile default workflow', async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('default')
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'mobile-default-workflow.png'
-      )
-    })
+  test('@mobile default workflow', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('default')
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'mobile-default-workflow.png'
+    )
+  })
 
-    test('@mobile settings dialog', async ({ comfyPage }) => {
-      await comfyPage.settingDialog.open()
-      await comfyPage.nextFrame()
+  test('@mobile settings dialog', async ({ comfyPage }) => {
+    await comfyPage.settingDialog.open()
+    await comfyPage.nextFrame()
 
-      await expect(comfyPage.settingDialog.root).toHaveScreenshot(
-        'mobile-settings-dialog.png',
-        {
-          mask: [
-            comfyPage.settingDialog.root.getByTestId('current-user-indicator')
-          ]
-        }
-      )
-    })
-  }
-)
+    await expect(comfyPage.settingDialog.root).toHaveScreenshot(
+      'mobile-settings-dialog.png',
+      {
+        mask: [
+          comfyPage.settingDialog.root.getByTestId('current-user-indicator')
+        ]
+      }
+    )
+  })
+})

browser_tests/tests/nodeBadge.spec.ts

@@ -8,7 +8,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Node Badge', { tag: ['@screenshot', '@smoke', '@node'] }, () => {
+test.describe('Node Badge', () => {
   test('Can add badge', async ({ comfyPage }) => {
     await comfyPage.page.evaluate(() => {
       const LGraphBadge = window['LGraphBadge']
@@ -66,60 +66,50 @@ test.describe('Node Badge', { tag: ['@screenshot', '@smoke', '@node'] }, () => {
   })
 })
 
-test.describe(
-  'Node source badge',
-  { tag: ['@screenshot', '@smoke', '@node'] },
-  () => {
-    Object.values(NodeBadgeMode).forEach(async (mode) => {
-      test(`Shows node badges (${mode})`, async ({ comfyPage }) => {
-        // Execution error workflow has both custom node and core node.
-        await comfyPage.loadWorkflow('nodes/execution_error')
-        await comfyPage.setSetting('Comfy.NodeBadge.NodeSourceBadgeMode', mode)
-        await comfyPage.setSetting('Comfy.NodeBadge.NodeIdBadgeMode', mode)
-        await comfyPage.nextFrame()
-        await comfyPage.resetView()
-        await expect(comfyPage.canvas).toHaveScreenshot(
-          `node-badge-${mode}.png`
-        )
-      })
-    })
-  }
-)
-
-test.describe(
-  'Node badge color',
-  { tag: ['@screenshot', '@smoke', '@node'] },
-  () => {
-    test('Can show node badge with unknown color palette', async ({
-      comfyPage
-    }) => {
-      await comfyPage.setSetting(
-        'Comfy.NodeBadge.NodeIdBadgeMode',
-        NodeBadgeMode.ShowAll
-      )
-      await comfyPage.setSetting('Comfy.ColorPalette', 'unknown')
+test.describe('Node source badge', () => {
+  Object.values(NodeBadgeMode).forEach(async (mode) => {
+    test(`Shows node badges (${mode})`, async ({ comfyPage }) => {
+      // Execution error workflow has both custom node and core node.
+      await comfyPage.loadWorkflow('nodes/execution_error')
+      await comfyPage.setSetting('Comfy.NodeBadge.NodeSourceBadgeMode', mode)
+      await comfyPage.setSetting('Comfy.NodeBadge.NodeIdBadgeMode', mode)
       await comfyPage.nextFrame()
-      // Click empty space to trigger canvas re-render.
-      await comfyPage.clickEmptySpace()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'node-badge-unknown-color-palette.png'
-      )
+      await comfyPage.resetView()
+      await expect(comfyPage.canvas).toHaveScreenshot(`node-badge-${mode}.png`)
     })
+  })
+})
 
-    test('Can show node badge with light color palette', async ({
-      comfyPage
-    }) => {
-      await comfyPage.setSetting(
-        'Comfy.NodeBadge.NodeIdBadgeMode',
-        NodeBadgeMode.ShowAll
-      )
-      await comfyPage.setSetting('Comfy.ColorPalette', 'light')
-      await comfyPage.nextFrame()
-      // Click empty space to trigger canvas re-render.
-      await comfyPage.clickEmptySpace()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'node-badge-light-color-palette.png'
-      )
-    })
-  }
-)
+test.describe('Node badge color', () => {
+  test('Can show node badge with unknown color palette', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting(
+      'Comfy.NodeBadge.NodeIdBadgeMode',
+      NodeBadgeMode.ShowAll
+    )
+    await comfyPage.setSetting('Comfy.ColorPalette', 'unknown')
+    await comfyPage.nextFrame()
+    // Click empty space to trigger canvas re-render.
+    await comfyPage.clickEmptySpace()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'node-badge-unknown-color-palette.png'
+    )
+  })
+
+  test('Can show node badge with light color palette', async ({
+    comfyPage
+  }) => {
+    await comfyPage.setSetting(
+      'Comfy.NodeBadge.NodeIdBadgeMode',
+      NodeBadgeMode.ShowAll
+    )
+    await comfyPage.setSetting('Comfy.ColorPalette', 'light')
+    await comfyPage.nextFrame()
+    // Click empty space to trigger canvas re-render.
+    await comfyPage.clickEmptySpace()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'node-badge-light-color-palette.png'
+    )
+  })
+})

browser_tests/tests/nodeDisplay.spec.ts

@@ -8,7 +8,7 @@ test.beforeEach(async ({ comfyPage }) => {
 
 // If an input is optional by node definition, it should be shown as
 // a hollow circle no matter what shape it was defined in the workflow JSON.
-test.describe('Optional input', { tag: ['@screenshot', '@node'] }, () => {
+test.describe('Optional input', () => {
   test('No shape specified', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('inputs/optional_input_no_shape')
     await expect(comfyPage.canvas).toHaveScreenshot('optional_input.png')

browser_tests/tests/nodeHelp.spec.ts

@@ -23,7 +23,7 @@ async function selectNodeWithPan(comfyPage: ComfyPage, nodeRef: NodeReference) {
   await nodeRef.click('title')
 }
 
-test.describe('Node Help', { tag: ['@slow', '@ui'] }, () => {
+test.describe('Node Help', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setup()
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')

browser_tests/tests/nodeSearchBox.spec.ts

@@ -7,7 +7,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Node search box', { tag: '@node' }, () => {
+test.describe('Node search box', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.LinkRelease.Action', 'search box')
     await comfyPage.setSetting('Comfy.LinkRelease.ActionShift', 'search box')
@@ -46,14 +46,14 @@ test.describe('Node search box', { tag: '@node' }, () => {
     await expect(comfyPage.searchBox.input).toBeVisible()
   })
 
-  test('Can add node', { tag: '@screenshot' }, async ({ comfyPage }) => {
+  test('Can add node', async ({ comfyPage }) => {
     await comfyPage.doubleClickCanvas()
     await expect(comfyPage.searchBox.input).toHaveCount(1)
     await comfyPage.searchBox.fillAndSelectFirstNode('KSampler')
     await expect(comfyPage.canvas).toHaveScreenshot('added-node.png')
   })
 
-  test('Can auto link node', { tag: '@screenshot' }, async ({ comfyPage }) => {
+  test('Can auto link node', async ({ comfyPage }) => {
     await comfyPage.disconnectEdge()
     // Select the second item as the first item is always reroute
     await comfyPage.searchBox.fillAndSelectFirstNode('CLIPTextEncode', {
@@ -62,47 +62,41 @@ test.describe('Node search box', { tag: '@node' }, () => {
     await expect(comfyPage.canvas).toHaveScreenshot('auto-linked-node.png')
   })
 
-  test(
-    'Can auto link batch moved node',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('links/batch_move_links')
+  test('Can auto link batch moved node', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('links/batch_move_links')
 
-      const outputSlot1Pos = {
-        x: 304,
-        y: 127
-      }
-      const emptySpacePos = {
-        x: 5,
-        y: 5
-      }
-      await comfyPage.page.keyboard.down('Shift')
-      await comfyPage.dragAndDrop(outputSlot1Pos, emptySpacePos)
-      await comfyPage.page.keyboard.up('Shift')
-
-      // Select the second item as the first item is always reroute
-      await comfyPage.searchBox.fillAndSelectFirstNode('Load Checkpoint', {
-        suggestionIndex: 0
-      })
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'auto-linked-node-batch.png'
-      )
+    const outputSlot1Pos = {
+      x: 304,
+      y: 127
     }
-  )
-
-  test(
-    'Link release connecting to node with no slots',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.disconnectEdge()
-      await expect(comfyPage.searchBox.input).toHaveCount(1)
-      await comfyPage.page.locator('.p-chip-remove-icon').click()
-      await comfyPage.searchBox.fillAndSelectFirstNode('KSampler')
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'added-node-no-connection.png'
-      )
+    const emptySpacePos = {
+      x: 5,
+      y: 5
     }
-  )
+    await comfyPage.page.keyboard.down('Shift')
+    await comfyPage.dragAndDrop(outputSlot1Pos, emptySpacePos)
+    await comfyPage.page.keyboard.up('Shift')
+
+    // Select the second item as the first item is always reroute
+    await comfyPage.searchBox.fillAndSelectFirstNode('Load Checkpoint', {
+      suggestionIndex: 0
+    })
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'auto-linked-node-batch.png'
+    )
+  })
+
+  test('Link release connecting to node with no slots', async ({
+    comfyPage
+  }) => {
+    await comfyPage.disconnectEdge()
+    await expect(comfyPage.searchBox.input).toHaveCount(1)
+    await comfyPage.page.locator('.p-chip-remove-icon').click()
+    await comfyPage.searchBox.fillAndSelectFirstNode('KSampler')
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'added-node-no-connection.png'
+    )
+  })
 
   test('Has correct aria-labels on search results', async ({ comfyPage }) => {
     const node = 'Load Checkpoint'
@@ -257,45 +251,40 @@ test.describe('Node search box', { tag: '@node' }, () => {
   })
 })
 
-test.describe('Release context menu', { tag: '@node' }, () => {
+test.describe('Release context menu', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.LinkRelease.Action', 'context menu')
     await comfyPage.setSetting('Comfy.LinkRelease.ActionShift', 'search box')
     await comfyPage.setSetting('Comfy.NodeSearchBoxImpl', 'default')
   })
 
-  test(
-    'Can trigger on link release',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.disconnectEdge()
-      const contextMenu = comfyPage.page.locator('.litecontextmenu')
-      // Wait for context menu with correct title (slot name | slot type)
-      // The title shows the output slot name and type from the disconnected link
-      await expect(contextMenu.locator('.litemenu-title')).toContainText(
-        'CLIP | CLIP'
-      )
-      await comfyPage.page.mouse.move(10, 10)
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'link-release-context-menu.png'
-      )
-    }
-  )
+  test('Can trigger on link release', async ({ comfyPage }) => {
+    await comfyPage.disconnectEdge()
+    const contextMenu = comfyPage.page.locator('.litecontextmenu')
+    // Wait for context menu with correct title (slot name | slot type)
+    // The title shows the output slot name and type from the disconnected link
+    await expect(contextMenu.locator('.litemenu-title')).toContainText(
+      'CLIP | CLIP'
+    )
+    await comfyPage.page.mouse.move(10, 10)
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'link-release-context-menu.png'
+    )
+  })
 
-  test(
-    'Can search and add node from context menu',
-    { tag: '@screenshot' },
-    async ({ comfyPage, comfyMouse }) => {
-      await comfyPage.disconnectEdge()
-      await comfyMouse.move({ x: 10, y: 10 })
-      await comfyPage.clickContextMenuItem('Search')
-      await comfyPage.searchBox.fillAndSelectFirstNode('CLIP Prompt')
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'link-context-menu-search.png'
-      )
-    }
-  )
+  test('Can search and add node from context menu', async ({
+    comfyPage,
+    comfyMouse
+  }) => {
+    await comfyPage.disconnectEdge()
+    await comfyMouse.move({ x: 10, y: 10 })
+    await comfyPage.clickContextMenuItem('Search')
+    await comfyPage.searchBox.fillAndSelectFirstNode('CLIP Prompt')
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'link-context-menu-search.png'
+    )
+  })
 
   test('Existing user (pre-1.24.1) gets context menu by default on link release', async ({
     comfyPage

browser_tests/tests/noteNode.spec.ts

@@ -6,8 +6,8 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Note Node', { tag: '@node' }, () => {
-  test('Can load node nodes', { tag: '@screenshot' }, async ({ comfyPage }) => {
+test.describe('Note Node', () => {
+  test('Can load node nodes', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('nodes/note_nodes')
     await expect(comfyPage.canvas).toHaveScreenshot('note_nodes.png')
   })

browser_tests/tests/primitiveNode.spec.ts

@@ -7,7 +7,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Primitive Node', { tag: ['@screenshot', '@node'] }, () => {
+test.describe('Primitive Node', () => {
   test('Can load with correct size', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('primitive/primitive_node')
     await expect(comfyPage.canvas).toHaveScreenshot('primitive_node.png')

browser_tests/tests/recordAudio.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Record Audio Node', { tag: '@screenshot' }, () => {
+test.describe('Record Audio Node', () => {
   test('should add a record audio node and take a screenshot', async ({
     comfyPage
   }) => {

browser_tests/tests/remoteWidgets.spec.ts

@@ -3,7 +3,7 @@ import { expect } from '@playwright/test'
 import type { ComfyPage } from '../fixtures/ComfyPage'
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Remote COMBO Widget', { tag: '@widget' }, () => {
+test.describe('Remote COMBO Widget', () => {
   const mockOptions = ['d', 'c', 'b', 'a']
 
   const addRemoteWidgetNode = async (

browser_tests/tests/rerouteNode.spec.ts

@@ -3,7 +3,7 @@ import { expect } from '@playwright/test'
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 import { getMiddlePoint } from '../fixtures/utils/litegraphUtils'
 
-test.describe('Reroute Node', { tag: ['@screenshot', '@node'] }, () => {
+test.describe('Reroute Node', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
   })
@@ -38,96 +38,92 @@ test.describe('Reroute Node', { tag: ['@screenshot', '@node'] }, () => {
   })
 })
 
-test.describe(
-  'LiteGraph Native Reroute Node',
-  { tag: ['@screenshot', '@node'] },
-  () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
-      await comfyPage.setSetting('LiteGraph.Reroute.SplineOffset', 80)
-    })
+test.describe('LiteGraph Native Reroute Node', () => {
+  test.beforeEach(async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
+    await comfyPage.setSetting('LiteGraph.Reroute.SplineOffset', 80)
+  })
 
-    test('loads from workflow', async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('reroute/native_reroute')
-      await expect(comfyPage.canvas).toHaveScreenshot('native_reroute.png')
-    })
+  test('loads from workflow', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('reroute/native_reroute')
+    await expect(comfyPage.canvas).toHaveScreenshot('native_reroute.png')
+  })
 
-    test('@2x @0.5x Can add reroute by alt clicking on link', async ({
-      comfyPage
-    }) => {
-      const loadCheckpointNode = (
-        await comfyPage.getNodeRefsByTitle('Load Checkpoint')
-      )[0]
-      const clipEncodeNode = (
-        await comfyPage.getNodeRefsByTitle('CLIP Text Encode (Prompt)')
-      )[0]
+  test('@2x @0.5x Can add reroute by alt clicking on link', async ({
+    comfyPage
+  }) => {
+    const loadCheckpointNode = (
+      await comfyPage.getNodeRefsByTitle('Load Checkpoint')
+    )[0]
+    const clipEncodeNode = (
+      await comfyPage.getNodeRefsByTitle('CLIP Text Encode (Prompt)')
+    )[0]
 
-      const slot1 = await loadCheckpointNode.getOutput(1)
-      const slot2 = await clipEncodeNode.getInput(0)
-      const middlePoint = getMiddlePoint(
-        await slot1.getPosition(),
-        await slot2.getPosition()
-      )
+    const slot1 = await loadCheckpointNode.getOutput(1)
+    const slot2 = await clipEncodeNode.getInput(0)
+    const middlePoint = getMiddlePoint(
+      await slot1.getPosition(),
+      await slot2.getPosition()
+    )
 
-      await comfyPage.page.keyboard.down('Alt')
-      await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
-      await comfyPage.page.keyboard.up('Alt')
+    await comfyPage.page.keyboard.down('Alt')
+    await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
+    await comfyPage.page.keyboard.up('Alt')
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'native_reroute_alt_click.png'
-      )
-    })
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'native_reroute_alt_click.png'
+    )
+  })
 
-    test('Can add reroute by clicking middle of link context menu', async ({
-      comfyPage
-    }) => {
-      const loadCheckpointNode = (
-        await comfyPage.getNodeRefsByTitle('Load Checkpoint')
-      )[0]
-      const clipEncodeNode = (
-        await comfyPage.getNodeRefsByTitle('CLIP Text Encode (Prompt)')
-      )[0]
+  test('Can add reroute by clicking middle of link context menu', async ({
+    comfyPage
+  }) => {
+    const loadCheckpointNode = (
+      await comfyPage.getNodeRefsByTitle('Load Checkpoint')
+    )[0]
+    const clipEncodeNode = (
+      await comfyPage.getNodeRefsByTitle('CLIP Text Encode (Prompt)')
+    )[0]
 
-      const slot1 = await loadCheckpointNode.getOutput(1)
-      const slot2 = await clipEncodeNode.getInput(0)
-      const middlePoint = getMiddlePoint(
-        await slot1.getPosition(),
-        await slot2.getPosition()
-      )
+    const slot1 = await loadCheckpointNode.getOutput(1)
+    const slot2 = await clipEncodeNode.getInput(0)
+    const middlePoint = getMiddlePoint(
+      await slot1.getPosition(),
+      await slot2.getPosition()
+    )
 
-      await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
-      await comfyPage.page
-        .locator('.litecontextmenu .litemenu-entry', { hasText: 'Add Reroute' })
-        .click()
+    await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
+    await comfyPage.page
+      .locator('.litecontextmenu .litemenu-entry', { hasText: 'Add Reroute' })
+      .click()
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'native_reroute_context_menu.png'
-      )
-    })
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'native_reroute_context_menu.png'
+    )
+  })
 
-    test('Can delete link that is connected to two reroutes', async ({
-      comfyPage
-    }) => {
-      // https://github.com/Comfy-Org/ComfyUI_frontend/issues/4695
-      await comfyPage.loadWorkflow(
-        'reroute/single-native-reroute-default-workflow'
-      )
+  test('Can delete link that is connected to two reroutes', async ({
+    comfyPage
+  }) => {
+    // https://github.com/Comfy-Org/ComfyUI_frontend/issues/4695
+    await comfyPage.loadWorkflow(
+      'reroute/single-native-reroute-default-workflow'
+    )
 
-      // To find the clickable midpoint button, we use the hardcoded value from the browser logs
-      // since the link is a bezier curve and not a straight line.
-      const middlePoint = { x: 359.4188232421875, y: 468.7716979980469 }
+    // To find the clickable midpoint button, we use the hardcoded value from the browser logs
+    // since the link is a bezier curve and not a straight line.
+    const middlePoint = { x: 359.4188232421875, y: 468.7716979980469 }
 
-      // Click the middle point of the link to open the context menu.
-      await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
+    // Click the middle point of the link to open the context menu.
+    await comfyPage.page.mouse.click(middlePoint.x, middlePoint.y)
 
-      // Click the "Delete" context menu option.
-      await comfyPage.page
-        .locator('.litecontextmenu .litemenu-entry', { hasText: 'Delete' })
-        .click()
+    // Click the "Delete" context menu option.
+    await comfyPage.page
+      .locator('.litecontextmenu .litemenu-entry', { hasText: 'Delete' })
+      .click()
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'native_reroute_delete_from_midpoint_context_menu.png'
-      )
-    })
-  }
-)
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'native_reroute_delete_from_midpoint_context_menu.png'
+    )
+  })
+})

browser_tests/tests/rightClickMenu.spec.ts

@@ -7,49 +7,43 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe(
-  'Canvas Right Click Menu',
-  { tag: ['@screenshot', '@ui'] },
-  () => {
-    test('Can add node', async ({ comfyPage }) => {
-      await comfyPage.rightClickCanvas()
-      await expect(comfyPage.canvas).toHaveScreenshot('right-click-menu.png')
-      await comfyPage.page.getByText('Add Node').click()
-      await comfyPage.nextFrame()
-      await comfyPage.page.getByText('loaders').click()
-      await comfyPage.nextFrame()
-      await comfyPage.page.getByText('Load VAE').click()
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot('add-node-node-added.png')
-    })
+test.describe('Canvas Right Click Menu', () => {
+  test('Can add node', async ({ comfyPage }) => {
+    await comfyPage.rightClickCanvas()
+    await expect(comfyPage.canvas).toHaveScreenshot('right-click-menu.png')
+    await comfyPage.page.getByText('Add Node').click()
+    await comfyPage.nextFrame()
+    await comfyPage.page.getByText('loaders').click()
+    await comfyPage.nextFrame()
+    await comfyPage.page.getByText('Load VAE').click()
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('add-node-node-added.png')
+  })
 
-    test('Can add group', async ({ comfyPage }) => {
-      await comfyPage.rightClickCanvas()
-      await expect(comfyPage.canvas).toHaveScreenshot('right-click-menu.png')
-      await comfyPage.page.getByText('Add Group', { exact: true }).click()
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'add-group-group-added.png'
-      )
-    })
+  test('Can add group', async ({ comfyPage }) => {
+    await comfyPage.rightClickCanvas()
+    await expect(comfyPage.canvas).toHaveScreenshot('right-click-menu.png')
+    await comfyPage.page.getByText('Add Group', { exact: true }).click()
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot('add-group-group-added.png')
+  })
 
-    test('Can convert to group node', async ({ comfyPage }) => {
-      await comfyPage.select2Nodes()
-      await expect(comfyPage.canvas).toHaveScreenshot('selected-2-nodes.png')
-      await comfyPage.rightClickCanvas()
-      await comfyPage.clickContextMenuItem('Convert to Group Node (Deprecated)')
-      await comfyPage.promptDialogInput.fill('GroupNode2CLIP')
-      await comfyPage.page.keyboard.press('Enter')
-      await comfyPage.promptDialogInput.waitFor({ state: 'hidden' })
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'right-click-node-group-node.png'
-      )
-    })
-  }
-)
+  test('Can convert to group node', async ({ comfyPage }) => {
+    await comfyPage.select2Nodes()
+    await expect(comfyPage.canvas).toHaveScreenshot('selected-2-nodes.png')
+    await comfyPage.rightClickCanvas()
+    await comfyPage.clickContextMenuItem('Convert to Group Node (Deprecated)')
+    await comfyPage.promptDialogInput.fill('GroupNode2CLIP')
+    await comfyPage.page.keyboard.press('Enter')
+    await comfyPage.promptDialogInput.waitFor({ state: 'hidden' })
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'right-click-node-group-node.png'
+    )
+  })
+})
 
-test.describe('Node Right Click Menu', { tag: ['@screenshot', '@ui'] }, () => {
+test.describe('Node Right Click Menu', () => {
   test('Can open properties panel', async ({ comfyPage }) => {
     await comfyPage.rightClickEmptyLatentNode()
     await expect(comfyPage.canvas).toHaveScreenshot('right-click-node.png')

browser_tests/tests/selectionToolbox.spec.ts

@@ -10,7 +10,7 @@ test.beforeEach(async ({ comfyPage }) => {
 const BLUE_COLOR = 'rgb(51, 51, 85)'
 const RED_COLOR = 'rgb(85, 51, 51)'
 
-test.describe('Selection Toolbox', { tag: ['@screenshot', '@ui'] }, () => {
+test.describe('Selection Toolbox', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.Canvas.SelectionToolbox', true)
   })

browser_tests/tests/selectionToolboxSubmenus.spec.ts

@@ -7,190 +7,178 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe(
-  'Selection Toolbox - More Options Submenus',
-  { tag: '@ui' },
-  () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.Canvas.SelectionToolbox', true)
-      await comfyPage.loadWorkflow('nodes/single_ksampler')
-      await comfyPage.nextFrame()
-      await comfyPage.selectNodes(['KSampler'])
-      await comfyPage.nextFrame()
-    })
+test.describe('Selection Toolbox - More Options Submenus', () => {
+  test.beforeEach(async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.Canvas.SelectionToolbox', true)
+    await comfyPage.loadWorkflow('nodes/single_ksampler')
+    await comfyPage.nextFrame()
+    await comfyPage.selectNodes(['KSampler'])
+    await comfyPage.nextFrame()
+  })
 
-    const openMoreOptions = async (comfyPage: ComfyPage) => {
-      const ksamplerNodes = await comfyPage.getNodeRefsByTitle('KSampler')
-      if (ksamplerNodes.length === 0) {
-        throw new Error('No KSampler nodes found')
-      }
-
-      // Drag the KSampler to the center of the screen
-      const nodePos = await ksamplerNodes[0].getPosition()
-      const viewportSize = comfyPage.page.viewportSize()
-      const centerX = viewportSize.width / 3
-      const centerY = viewportSize.height / 2
-      await comfyPage.dragAndDrop(
-        { x: nodePos.x, y: nodePos.y },
-        { x: centerX, y: centerY }
-      )
-      await comfyPage.nextFrame()
-
-      await ksamplerNodes[0].click('title')
-      await comfyPage.nextFrame()
-
-      await expect(comfyPage.page.locator('.selection-toolbox')).toBeVisible({
-        timeout: 5000
-      })
-
-      const moreOptionsBtn = comfyPage.page.locator(
-        '[data-testid="more-options-button"]'
-      )
-      await expect(moreOptionsBtn).toBeVisible({ timeout: 3000 })
-
-      await comfyPage.page.click('[data-testid="more-options-button"]')
-
-      await comfyPage.nextFrame()
-
-      const menuOptionsVisible = await comfyPage.page
-        .getByText('Rename')
-        .isVisible({ timeout: 2000 })
-        .catch(() => false)
-      if (menuOptionsVisible) {
-        return
-      }
-
-      await moreOptionsBtn.click({ force: true })
-      await comfyPage.nextFrame()
-
-      const menuOptionsVisibleAfterClick = await comfyPage.page
-        .getByText('Rename')
-        .isVisible({ timeout: 2000 })
-        .catch(() => false)
-      if (menuOptionsVisibleAfterClick) {
-        return
-      }
-
-      throw new Error('Could not open More Options menu - popover not showing')
+  const openMoreOptions = async (comfyPage: ComfyPage) => {
+    const ksamplerNodes = await comfyPage.getNodeRefsByTitle('KSampler')
+    if (ksamplerNodes.length === 0) {
+      throw new Error('No KSampler nodes found')
     }
 
-    test('opens Node Info from More Options menu', async ({ comfyPage }) => {
-      await openMoreOptions(comfyPage)
-      const nodeInfoButton = comfyPage.page.getByText('Node Info', {
-        exact: true
-      })
-      await expect(nodeInfoButton).toBeVisible()
-      await nodeInfoButton.click()
-      await comfyPage.nextFrame()
+    // Drag the KSampler to the center of the screen
+    const nodePos = await ksamplerNodes[0].getPosition()
+    const viewportSize = comfyPage.page.viewportSize()
+    const centerX = viewportSize.width / 3
+    const centerY = viewportSize.height / 2
+    await comfyPage.dragAndDrop(
+      { x: nodePos.x, y: nodePos.y },
+      { x: centerX, y: centerY }
+    )
+    await comfyPage.nextFrame()
+
+    await ksamplerNodes[0].click('title')
+    await comfyPage.nextFrame()
+
+    await expect(comfyPage.page.locator('.selection-toolbox')).toBeVisible({
+      timeout: 5000
     })
 
-    test('changes node shape via Shape submenu', async ({ comfyPage }) => {
-      const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
-      const initialShape = await nodeRef.getProperty<number>('shape')
+    const moreOptionsBtn = comfyPage.page.locator(
+      '[data-testid="more-options-button"]'
+    )
+    await expect(moreOptionsBtn).toBeVisible({ timeout: 3000 })
 
-      await openMoreOptions(comfyPage)
-      await comfyPage.page.getByText('Shape', { exact: true }).hover()
-      await expect(
-        comfyPage.page.getByText('Box', { exact: true })
-      ).toBeVisible({
-        timeout: 5000
-      })
-      await comfyPage.page.getByText('Box', { exact: true }).click()
-      await comfyPage.nextFrame()
+    await comfyPage.page.click('[data-testid="more-options-button"]')
 
-      const newShape = await nodeRef.getProperty<number>('shape')
-      expect(newShape).not.toBe(initialShape)
-      expect(newShape).toBe(1)
-    })
+    await comfyPage.nextFrame()
 
-    test('changes node color via Color submenu swatch', async ({
-      comfyPage
-    }) => {
-      const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
-      const initialColor = await nodeRef.getProperty<string | undefined>(
-        'color'
-      )
+    const menuOptionsVisible = await comfyPage.page
+      .getByText('Rename')
+      .isVisible({ timeout: 2000 })
+      .catch(() => false)
+    if (menuOptionsVisible) {
+      return
+    }
 
-      await openMoreOptions(comfyPage)
-      await comfyPage.page.getByText('Color', { exact: true }).click()
-      const blueSwatch = comfyPage.page.locator('[title="Blue"]')
-      await expect(blueSwatch.first()).toBeVisible({ timeout: 5000 })
-      await blueSwatch.first().click()
-      await comfyPage.nextFrame()
+    await moreOptionsBtn.click({ force: true })
+    await comfyPage.nextFrame()
 
-      const newColor = await nodeRef.getProperty<string | undefined>('color')
-      expect(newColor).toBe('#223')
-      if (initialColor) {
-        expect(newColor).not.toBe(initialColor)
-      }
-    })
+    const menuOptionsVisibleAfterClick = await comfyPage.page
+      .getByText('Rename')
+      .isVisible({ timeout: 2000 })
+      .catch(() => false)
+    if (menuOptionsVisibleAfterClick) {
+      return
+    }
 
-    test('renames a node using Rename action', async ({ comfyPage }) => {
-      const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
-      await openMoreOptions(comfyPage)
-      await comfyPage.page
-        .getByText('Rename', { exact: true })
-        .click({ force: true })
-      const input = comfyPage.page.locator(
-        '.group-title-editor.node-title-editor .editable-text input'
-      )
-      await expect(input).toBeVisible()
-      await input.fill('RenamedNode')
-      await input.press('Enter')
-      await comfyPage.nextFrame()
-      const newTitle = await nodeRef.getProperty<string>('title')
-      expect(newTitle).toBe('RenamedNode')
-    })
-
-    test('closes More Options menu when clicking outside', async ({
-      comfyPage
-    }) => {
-      await openMoreOptions(comfyPage)
-      const renameItem = comfyPage.page.getByText('Rename', { exact: true })
-      await expect(renameItem).toBeVisible({ timeout: 5000 })
-
-      // Wait for multiple frames to allow PrimeVue's outside click handler to initialize
-      for (let i = 0; i < 30; i++) {
-        await comfyPage.nextFrame()
-      }
-
-      await comfyPage.page
-        .locator('#graph-canvas')
-        .click({ position: { x: 0, y: 50 }, force: true })
-
-      await comfyPage.nextFrame()
-      await expect(
-        comfyPage.page.getByText('Rename', { exact: true })
-      ).not.toBeVisible()
-    })
-
-    test('closes More Options menu when clicking the button again (toggle)', async ({
-      comfyPage
-    }) => {
-      await openMoreOptions(comfyPage)
-      await expect(
-        comfyPage.page.getByText('Rename', { exact: true })
-      ).toBeVisible({ timeout: 5000 })
-
-      await comfyPage.page.evaluate(() => {
-        const btn = document.querySelector(
-          '[data-testid="more-options-button"]'
-        )
-        if (btn) {
-          const event = new MouseEvent('click', {
-            bubbles: true,
-            cancelable: true,
-            view: window,
-            detail: 1
-          })
-          btn.dispatchEvent(event)
-        }
-      })
-      await comfyPage.nextFrame()
-
-      await expect(
-        comfyPage.page.getByText('Rename', { exact: true })
-      ).not.toBeVisible()
-    })
+    throw new Error('Could not open More Options menu - popover not showing')
   }
-)
+
+  test('opens Node Info from More Options menu', async ({ comfyPage }) => {
+    await openMoreOptions(comfyPage)
+    const nodeInfoButton = comfyPage.page.getByText('Node Info', {
+      exact: true
+    })
+    await expect(nodeInfoButton).toBeVisible()
+    await nodeInfoButton.click()
+    await comfyPage.nextFrame()
+  })
+
+  test('changes node shape via Shape submenu', async ({ comfyPage }) => {
+    const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
+    const initialShape = await nodeRef.getProperty<number>('shape')
+
+    await openMoreOptions(comfyPage)
+    await comfyPage.page.getByText('Shape', { exact: true }).hover()
+    await expect(comfyPage.page.getByText('Box', { exact: true })).toBeVisible({
+      timeout: 5000
+    })
+    await comfyPage.page.getByText('Box', { exact: true }).click()
+    await comfyPage.nextFrame()
+
+    const newShape = await nodeRef.getProperty<number>('shape')
+    expect(newShape).not.toBe(initialShape)
+    expect(newShape).toBe(1)
+  })
+
+  test('changes node color via Color submenu swatch', async ({ comfyPage }) => {
+    const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
+    const initialColor = await nodeRef.getProperty<string | undefined>('color')
+
+    await openMoreOptions(comfyPage)
+    await comfyPage.page.getByText('Color', { exact: true }).click()
+    const blueSwatch = comfyPage.page.locator('[title="Blue"]')
+    await expect(blueSwatch.first()).toBeVisible({ timeout: 5000 })
+    await blueSwatch.first().click()
+    await comfyPage.nextFrame()
+
+    const newColor = await nodeRef.getProperty<string | undefined>('color')
+    expect(newColor).toBe('#223')
+    if (initialColor) {
+      expect(newColor).not.toBe(initialColor)
+    }
+  })
+
+  test('renames a node using Rename action', async ({ comfyPage }) => {
+    const nodeRef = (await comfyPage.getNodeRefsByTitle('KSampler'))[0]
+    await openMoreOptions(comfyPage)
+    await comfyPage.page
+      .getByText('Rename', { exact: true })
+      .click({ force: true })
+    const input = comfyPage.page.locator(
+      '.group-title-editor.node-title-editor .editable-text input'
+    )
+    await expect(input).toBeVisible()
+    await input.fill('RenamedNode')
+    await input.press('Enter')
+    await comfyPage.nextFrame()
+    const newTitle = await nodeRef.getProperty<string>('title')
+    expect(newTitle).toBe('RenamedNode')
+  })
+
+  test('closes More Options menu when clicking outside', async ({
+    comfyPage
+  }) => {
+    await openMoreOptions(comfyPage)
+    const renameItem = comfyPage.page.getByText('Rename', { exact: true })
+    await expect(renameItem).toBeVisible({ timeout: 5000 })
+
+    // Wait for multiple frames to allow PrimeVue's outside click handler to initialize
+    for (let i = 0; i < 30; i++) {
+      await comfyPage.nextFrame()
+    }
+
+    await comfyPage.page
+      .locator('#graph-canvas')
+      .click({ position: { x: 0, y: 50 }, force: true })
+
+    await comfyPage.nextFrame()
+    await expect(
+      comfyPage.page.getByText('Rename', { exact: true })
+    ).not.toBeVisible()
+  })
+
+  test('closes More Options menu when clicking the button again (toggle)', async ({
+    comfyPage
+  }) => {
+    await openMoreOptions(comfyPage)
+    await expect(
+      comfyPage.page.getByText('Rename', { exact: true })
+    ).toBeVisible({ timeout: 5000 })
+
+    await comfyPage.page.evaluate(() => {
+      const btn = document.querySelector('[data-testid="more-options-button"]')
+      if (btn) {
+        const event = new MouseEvent('click', {
+          bubbles: true,
+          cancelable: true,
+          view: window,
+          detail: 1
+        })
+        btn.dispatchEvent(event)
+      }
+    })
+    await comfyPage.nextFrame()
+
+    await expect(
+      comfyPage.page.getByText('Rename', { exact: true })
+    ).not.toBeVisible()
+  })
+})

browser_tests/tests/subgraph-rename-dialog.spec.ts

@@ -12,7 +12,7 @@ const SELECTORS = {
   promptDialog: '.graphdialog input'
 } as const
 
-test.describe('Subgraph Slot Rename Dialog', { tag: '@subgraph' }, () => {
+test.describe('Subgraph Slot Rename Dialog', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
   })

browser_tests/tests/subgraph.spec.ts

@@ -16,7 +16,7 @@ const SELECTORS = {
   domWidget: '.comfy-multiline-input'
 } as const
 
-test.describe('Subgraph Operations', { tag: ['@slow', '@subgraph'] }, () => {
+test.describe('Subgraph Operations', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
   })

browser_tests/tests/templates.spec.ts

@@ -13,7 +13,7 @@ async function checkTemplateFileExists(
   return response.ok()
 }
 
-test.describe('Templates', { tag: ['@slow', '@workflow'] }, () => {
+test.describe('Templates', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
     await comfyPage.setSetting('Comfy.Workflow.ShowMissingModelsWarning', false)
@@ -207,114 +207,109 @@ test.describe('Templates', { tag: ['@slow', '@workflow'] }, () => {
     await expect(nav).toBeVisible() // Nav should be visible at tablet size
   })
 
-  test(
-    'template cards descriptions adjust height dynamically',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      // Setup test by intercepting templates response to inject cards with varying description lengths
-      await comfyPage.page.route(
-        '**/templates/index.json',
-        async (route, _) => {
-          const response = [
+  test('template cards descriptions adjust height dynamically', async ({
+    comfyPage
+  }) => {
+    // Setup test by intercepting templates response to inject cards with varying description lengths
+    await comfyPage.page.route('**/templates/index.json', async (route, _) => {
+      const response = [
+        {
+          moduleName: 'default',
+          title: 'Test Templates',
+          type: 'image',
+          templates: [
             {
-              moduleName: 'default',
-              title: 'Test Templates',
-              type: 'image',
-              templates: [
-                {
-                  name: 'short-description',
-                  title: 'Short Description',
-                  mediaType: 'image',
-                  mediaSubtype: 'webp',
-                  description: 'This is a short description.'
-                },
-                {
-                  name: 'medium-description',
-                  title: 'Medium Description',
-                  mediaType: 'image',
-                  mediaSubtype: 'webp',
-                  description:
-                    'This is a medium length description that should take up two lines on most displays.'
-                },
-                {
-                  name: 'long-description',
-                  title: 'Long Description',
-                  mediaType: 'image',
-                  mediaSubtype: 'webp',
-                  description:
-                    'This is a much longer description that should definitely wrap to multiple lines. It contains enough text to demonstrate how the cards handle varying amounts of content while maintaining a consistent layout grid.'
-                }
-              ]
+              name: 'short-description',
+              title: 'Short Description',
+              mediaType: 'image',
+              mediaSubtype: 'webp',
+              description: 'This is a short description.'
+            },
+            {
+              name: 'medium-description',
+              title: 'Medium Description',
+              mediaType: 'image',
+              mediaSubtype: 'webp',
+              description:
+                'This is a medium length description that should take up two lines on most displays.'
+            },
+            {
+              name: 'long-description',
+              title: 'Long Description',
+              mediaType: 'image',
+              mediaSubtype: 'webp',
+              description:
+                'This is a much longer description that should definitely wrap to multiple lines. It contains enough text to demonstrate how the cards handle varying amounts of content while maintaining a consistent layout grid.'
             }
           ]
-          await route.fulfill({
-            status: 200,
-            body: JSON.stringify(response),
-            headers: {
-              'Content-Type': 'application/json',
-              'Cache-Control': 'no-store'
-            }
-          })
         }
-      )
-
-      // Mock the thumbnail images to avoid 404s
-      await comfyPage.page.route('**/templates/**.webp', async (route) => {
-        const headers = {
-          'Content-Type': 'image/webp',
+      ]
+      await route.fulfill({
+        status: 200,
+        body: JSON.stringify(response),
+        headers: {
+          'Content-Type': 'application/json',
           'Cache-Control': 'no-store'
         }
-        await route.fulfill({
-          status: 200,
-          path: 'browser_tests/assets/example.webp',
-          headers
-        })
       })
+    })
 
-      // Open templates dialog
-      await comfyPage.executeCommand('Comfy.BrowseTemplates')
-      await expect(comfyPage.templates.content).toBeVisible()
+    // Mock the thumbnail images to avoid 404s
+    await comfyPage.page.route('**/templates/**.webp', async (route) => {
+      const headers = {
+        'Content-Type': 'image/webp',
+        'Cache-Control': 'no-store'
+      }
+      await route.fulfill({
+        status: 200,
+        path: 'browser_tests/assets/example.webp',
+        headers
+      })
+    })
 
-      // Wait for cards to load
-      await expect(
-        comfyPage.page.locator(
-          '[data-testid="template-workflow-short-description"]'
-        )
-      ).toBeVisible({ timeout: 5000 })
+    // Open templates dialog
+    await comfyPage.executeCommand('Comfy.BrowseTemplates')
+    await expect(comfyPage.templates.content).toBeVisible()
 
-      // Verify all three cards with different descriptions are visible
-      const shortDescCard = comfyPage.page.locator(
+    // Wait for cards to load
+    await expect(
+      comfyPage.page.locator(
         '[data-testid="template-workflow-short-description"]'
       )
-      const mediumDescCard = comfyPage.page.locator(
-        '[data-testid="template-workflow-medium-description"]'
-      )
-      const longDescCard = comfyPage.page.locator(
-        '[data-testid="template-workflow-long-description"]'
-      )
+    ).toBeVisible({ timeout: 5000 })
 
-      await expect(shortDescCard).toBeVisible()
-      await expect(mediumDescCard).toBeVisible()
-      await expect(longDescCard).toBeVisible()
+    // Verify all three cards with different descriptions are visible
+    const shortDescCard = comfyPage.page.locator(
+      '[data-testid="template-workflow-short-description"]'
+    )
+    const mediumDescCard = comfyPage.page.locator(
+      '[data-testid="template-workflow-medium-description"]'
+    )
+    const longDescCard = comfyPage.page.locator(
+      '[data-testid="template-workflow-long-description"]'
+    )
 
-      // Verify descriptions are visible and have line-clamp class
-      // The description is in a p tag with text-muted class
-      const shortDesc = shortDescCard.locator('p.text-muted.line-clamp-2')
-      const mediumDesc = mediumDescCard.locator('p.text-muted.line-clamp-2')
-      const longDesc = longDescCard.locator('p.text-muted.line-clamp-2')
+    await expect(shortDescCard).toBeVisible()
+    await expect(mediumDescCard).toBeVisible()
+    await expect(longDescCard).toBeVisible()
 
-      await expect(shortDesc).toContainText('short description')
-      await expect(mediumDesc).toContainText('medium length description')
-      await expect(longDesc).toContainText('much longer description')
+    // Verify descriptions are visible and have line-clamp class
+    // The description is in a p tag with text-muted class
+    const shortDesc = shortDescCard.locator('p.text-muted.line-clamp-2')
+    const mediumDesc = mediumDescCard.locator('p.text-muted.line-clamp-2')
+    const longDesc = longDescCard.locator('p.text-muted.line-clamp-2')
 
-      // Verify grid layout maintains consistency
-      const templateGrid = comfyPage.page.locator(
-        '[data-testid="template-workflows-content"]'
-      )
-      await expect(templateGrid).toBeVisible()
-      await expect(templateGrid).toHaveScreenshot(
-        'template-grid-varying-content.png'
-      )
-    }
-  )
+    await expect(shortDesc).toContainText('short description')
+    await expect(mediumDesc).toContainText('medium length description')
+    await expect(longDesc).toContainText('much longer description')
+
+    // Verify grid layout maintains consistency
+    const templateGrid = comfyPage.page.locator(
+      '[data-testid="template-workflows-content"]'
+    )
+    await expect(templateGrid).toBeVisible()
+    await expect(templateGrid).toHaveScreenshot(
+      'template-grid-varying-content.png'
+    )
+  })
 })

browser_tests/tests/useSettingSearch.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Settings Search functionality', { tag: '@settings' }, () => {
+test.describe('Settings Search functionality', () => {
   test.beforeEach(async ({ comfyPage }) => {
     // Register test settings to verify hidden/deprecated filtering
     await comfyPage.page.evaluate(() => {

browser_tests/tests/userSelectView.spec.ts

@@ -5,7 +5,7 @@ import { userSelectPageFixture as test } from '../fixtures/UserSelectPage'
 /**
  * Expects ComfyUI backend to be launched with `--multi-user` flag.
  */
-test.describe('User Select View', { tag: '@settings' }, () => {
+test.describe('User Select View', () => {
   test.beforeEach(async ({ userSelectPage, page }) => {
     await page.goto(userSelectPage.url)
     await page.evaluate(() => {

browser_tests/tests/versionMismatchWarnings.spec.ts

@@ -3,7 +3,7 @@ import { expect } from '@playwright/test'
 import type { SystemStats } from '../../src/schemas/apiSchema'
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Version Mismatch Warnings', { tag: '@slow' }, () => {
+test.describe('Version Mismatch Warnings', () => {
   const ALWAYS_AHEAD_OF_INSTALLED_VERSION = '100.100.100'
   const ALWAYS_BEHIND_INSTALLED_VERSION = '0.0.0'
 

browser_tests/tests/viewport.spec.ts

@@ -2,7 +2,7 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Viewport', { tag: ['@screenshot', '@smoke', '@canvas'] }, () => {
+test.describe('Viewport', () => {
   test('Fits view to nodes when saved viewport position is offscreen', async ({
     comfyPage
   }) => {

browser_tests/tests/vueNodes/groups/groups.spec.ts

@@ -5,7 +5,7 @@ import {
 
 const CREATE_GROUP_HOTKEY = 'Control+g'
 
-test.describe('Vue Node Groups', { tag: '@screenshot' }, () => {
+test.describe('Vue Node Groups', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.VueNodes.Enabled', true)
     await comfyPage.setSetting('Comfy.Minimap.ShowGroups', true)

browser_tests/tests/vueNodes/interactions/canvas/pan.spec.ts

@@ -9,14 +9,10 @@ test.describe('Vue Nodes Canvas Pan', () => {
     await comfyPage.vueNodes.waitForNodes()
   })
 
-  test(
-    '@mobile Can pan with touch',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.panWithTouch({ x: 64, y: 64 }, { x: 256, y: 256 })
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'vue-nodes-paned-with-touch.png'
-      )
-    }
-  )
+  test('@mobile Can pan with touch', async ({ comfyPage }) => {
+    await comfyPage.panWithTouch({ x: 64, y: 64 }, { x: 256, y: 256 })
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'vue-nodes-paned-with-touch.png'
+    )
+  })
 })

browser_tests/tests/vueNodes/interactions/canvas/zoom.spec.ts

@@ -10,30 +10,25 @@ test.describe('Vue Nodes Zoom', () => {
     await comfyPage.vueNodes.waitForNodes()
   })
 
-  test(
-    'should not capture drag while zooming with ctrl+shift+drag',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      const checkpointNode =
-        comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
-      const nodeBoundingBox = await checkpointNode.boundingBox()
-      if (!nodeBoundingBox) throw new Error('Node bounding box not available')
+  test('should not capture drag while zooming with ctrl+shift+drag', async ({
+    comfyPage
+  }) => {
+    const checkpointNode = comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
+    const nodeBoundingBox = await checkpointNode.boundingBox()
+    if (!nodeBoundingBox) throw new Error('Node bounding box not available')
 
-      const nodeMidpointX = nodeBoundingBox.x + nodeBoundingBox.width / 2
-      const nodeMidpointY = nodeBoundingBox.y + nodeBoundingBox.height / 2
+    const nodeMidpointX = nodeBoundingBox.x + nodeBoundingBox.width / 2
+    const nodeMidpointY = nodeBoundingBox.y + nodeBoundingBox.height / 2
 
-      // Start the Ctrl+Shift drag-to-zoom on the canvas and continue dragging over
-      // the node. The node should not capture the drag while drag-zooming.
-      await comfyPage.page.keyboard.down('Control')
-      await comfyPage.page.keyboard.down('Shift')
-      await comfyPage.dragAndDrop(
-        { x: 200, y: 300 },
-        { x: nodeMidpointX, y: nodeMidpointY }
-      )
+    // Start the Ctrl+Shift drag-to-zoom on the canvas and continue dragging over
+    // the node. The node should not capture the drag while drag-zooming.
+    await comfyPage.page.keyboard.down('Control')
+    await comfyPage.page.keyboard.down('Shift')
+    await comfyPage.dragAndDrop(
+      { x: 200, y: 300 },
+      { x: nodeMidpointX, y: nodeMidpointY }
+    )
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'zoomed-in-ctrl-shift.png'
-      )
-    }
-  )
+    await expect(comfyPage.canvas).toHaveScreenshot('zoomed-in-ctrl-shift.png')
+  })
 })

browser_tests/tests/vueNodes/interactions/links/linkInteraction.spec.ts

@@ -98,7 +98,7 @@ async function connectSlots(
   await nextFrame()
 }
 
-test.describe('Vue Node Link Interaction', { tag: '@screenshot' }, () => {
+test.describe('Vue Node Link Interaction', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
     await comfyPage.setSetting('Comfy.VueNodes.Enabled', true)

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

@@ -5,7 +5,7 @@ import {
 import type { ComfyPage } from '../../../../fixtures/ComfyPage'
 import { fitToViewInstant } from '../../../../helpers/fitToView'
 
-test.describe('Vue Node Bring to Front', { tag: '@screenshot' }, () => {
+test.describe('Vue Node Bring to Front', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
     await comfyPage.setSetting('Comfy.VueNodes.Enabled', true)

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

@@ -1,8 +1,8 @@
 import {
+  type ComfyPage,
   comfyExpect as expect,
   comfyPageFixture as test
 } from '../../../../fixtures/ComfyPage'
-import type { ComfyPage } from '../../../../fixtures/ComfyPage'
 import type { Position } from '../../../../fixtures/types'
 
 test.describe('Vue Node Moving', () => {
@@ -29,47 +29,39 @@ test.describe('Vue Node Moving', () => {
     expect(diffY).toBeGreaterThan(0)
   }
 
-  test(
-    'should allow moving nodes by dragging',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      const loadCheckpointHeaderPos =
-        await getLoadCheckpointHeaderPos(comfyPage)
-      await comfyPage.dragAndDrop(loadCheckpointHeaderPos, {
-        x: 256,
-        y: 256
-      })
+  test('should allow moving nodes by dragging', async ({ comfyPage }) => {
+    const loadCheckpointHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
+    await comfyPage.dragAndDrop(loadCheckpointHeaderPos, {
+      x: 256,
+      y: 256
+    })
 
-      const newHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
-      await expectPosChanged(loadCheckpointHeaderPos, newHeaderPos)
+    const newHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
+    await expectPosChanged(loadCheckpointHeaderPos, newHeaderPos)
 
-      await expect(comfyPage.canvas).toHaveScreenshot('vue-node-moved-node.png')
-    }
-  )
+    await expect(comfyPage.canvas).toHaveScreenshot('vue-node-moved-node.png')
+  })
 
-  test(
-    '@mobile should allow moving nodes by dragging on touch devices',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      // Disable minimap (gets in way of the node on small screens)
-      await comfyPage.setSetting('Comfy.Minimap.Visible', false)
+  test('@mobile should allow moving nodes by dragging on touch devices', async ({
+    comfyPage
+  }) => {
+    // Disable minimap (gets in way of the node on small screens)
+    await comfyPage.setSetting('Comfy.Minimap.Visible', false)
 
-      const loadCheckpointHeaderPos =
-        await getLoadCheckpointHeaderPos(comfyPage)
-      await comfyPage.panWithTouch(
-        {
-          x: 64,
-          y: 64
-        },
-        loadCheckpointHeaderPos
-      )
+    const loadCheckpointHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
+    await comfyPage.panWithTouch(
+      {
+        x: 64,
+        y: 64
+      },
+      loadCheckpointHeaderPos
+    )
 
-      const newHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
-      await expectPosChanged(loadCheckpointHeaderPos, newHeaderPos)
+    const newHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
+    await expectPosChanged(loadCheckpointHeaderPos, newHeaderPos)
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'vue-node-moved-node-touch.png'
-      )
-    }
-  )
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'vue-node-moved-node-touch.png'
+    )
+  })
 })

browser_tests/tests/vueNodes/nodeStates/bypass.spec.ts

@@ -15,25 +15,22 @@ test.describe('Vue Node Bypass', () => {
     await comfyPage.vueNodes.waitForNodes()
   })
 
-  test(
-    'should allow toggling bypass on a selected node with hotkey',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.page.getByText('Load Checkpoint').click()
-      await comfyPage.page.keyboard.press(BYPASS_HOTKEY)
+  test('should allow toggling bypass on a selected node with hotkey', async ({
+    comfyPage
+  }) => {
+    await comfyPage.page.getByText('Load Checkpoint').click()
+    await comfyPage.page.keyboard.press(BYPASS_HOTKEY)
 
-      const checkpointNode =
-        comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
-      await expect(checkpointNode).toHaveClass(BYPASS_CLASS)
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'vue-node-bypassed-state.png'
-      )
+    const checkpointNode = comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
+    await expect(checkpointNode).toHaveClass(BYPASS_CLASS)
+    await comfyPage.nextFrame()
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'vue-node-bypassed-state.png'
+    )
 
-      await comfyPage.page.keyboard.press(BYPASS_HOTKEY)
-      await expect(checkpointNode).not.toHaveClass(BYPASS_CLASS)
-    }
-  )
+    await comfyPage.page.keyboard.press(BYPASS_HOTKEY)
+    await expect(checkpointNode).not.toHaveClass(BYPASS_CLASS)
+  })
 
   test('should allow toggling bypass on multiple selected nodes with hotkey', async ({
     comfyPage

browser_tests/tests/vueNodes/nodeStates/colors.spec.ts

@@ -3,7 +3,7 @@ import {
   comfyPageFixture as test
 } from '../../../fixtures/ComfyPage'
 
-test.describe('Vue Node Custom Colors', { tag: '@screenshot' }, () => {
+test.describe('Vue Node Custom Colors', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
     await comfyPage.setSetting('Comfy.Canvas.SelectionToolbox', true)

browser_tests/tests/vueNodes/nodeStates/mute.spec.ts

@@ -12,24 +12,19 @@ test.describe('Vue Node Mute', () => {
     await comfyPage.vueNodes.waitForNodes()
   })
 
-  test(
-    'should allow toggling mute on a selected node with hotkey',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.page.getByText('Load Checkpoint').click()
-      await comfyPage.page.keyboard.press(MUTE_HOTKEY)
+  test('should allow toggling mute on a selected node with hotkey', async ({
+    comfyPage
+  }) => {
+    await comfyPage.page.getByText('Load Checkpoint').click()
+    await comfyPage.page.keyboard.press(MUTE_HOTKEY)
 
-      const checkpointNode =
-        comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
-      await expect(checkpointNode).toHaveCSS('opacity', MUTE_OPACITY)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'vue-node-muted-state.png'
-      )
+    const checkpointNode = comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
+    await expect(checkpointNode).toHaveCSS('opacity', MUTE_OPACITY)
+    await expect(comfyPage.canvas).toHaveScreenshot('vue-node-muted-state.png')
 
-      await comfyPage.page.keyboard.press(MUTE_HOTKEY)
-      await expect(checkpointNode).not.toHaveCSS('opacity', MUTE_OPACITY)
-    }
-  )
+    await comfyPage.page.keyboard.press(MUTE_HOTKEY)
+    await expect(checkpointNode).not.toHaveCSS('opacity', MUTE_OPACITY)
+  })
 
   test('should allow toggling mute on multiple selected nodes with hotkey', async ({
     comfyPage

browser_tests/tests/vueNodes/widgets/load/uploadWidgets.spec.ts

@@ -9,17 +9,13 @@ test.describe('Vue Upload Widgets', () => {
     await comfyPage.vueNodes.waitForNodes()
   })
 
-  test(
-    'should hide canvas-only upload buttons',
-    { tag: '@screenshot' },
-    async ({ comfyPage }) => {
-      await comfyPage.setup()
-      await comfyPage.loadWorkflow('widgets/all_load_widgets')
-      await comfyPage.vueNodes.waitForNodes()
+  test('should hide canvas-only upload buttons', async ({ comfyPage }) => {
+    await comfyPage.setup()
+    await comfyPage.loadWorkflow('widgets/all_load_widgets')
+    await comfyPage.vueNodes.waitForNodes()
 
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'vue-nodes-upload-widgets.png'
-      )
-    }
-  )
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'vue-nodes-upload-widgets.png'
+    )
+  })
 })

browser_tests/tests/widget.spec.ts

@@ -6,7 +6,7 @@ test.beforeEach(async ({ comfyPage }) => {
   await comfyPage.setSetting('Comfy.UseNewMenu', 'Disabled')
 })
 
-test.describe('Combo text widget', { tag: ['@screenshot', '@widget'] }, () => {
+test.describe('Combo text widget', () => {
   test('Truncates text when resized', async ({ comfyPage }) => {
     await comfyPage.resizeLoadCheckpointNode(0.2, 1)
     await expect(comfyPage.canvas).toHaveScreenshot(
@@ -79,7 +79,7 @@ test.describe('Combo text widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe('Boolean widget', { tag: ['@screenshot', '@widget'] }, () => {
+test.describe('Boolean widget', () => {
   test('Can toggle', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('widgets/boolean_widget')
     await expect(comfyPage.canvas).toHaveScreenshot('boolean_widget.png')
@@ -92,7 +92,7 @@ test.describe('Boolean widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe('Slider widget', { tag: ['@screenshot', '@widget'] }, () => {
+test.describe('Slider widget', () => {
   test('Can drag adjust value', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('inputs/simple_slider')
     const node = (await comfyPage.getFirstNodeRef())!
@@ -113,7 +113,7 @@ test.describe('Slider widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe('Number widget', { tag: ['@screenshot', '@widget'] }, () => {
+test.describe('Number widget', () => {
   test('Can drag adjust value', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('widgets/seed_widget')
 
@@ -134,28 +134,22 @@ test.describe('Number widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe(
-  'Dynamic widget manipulation',
-  { tag: ['@screenshot', '@widget'] },
-  () => {
-    test('Auto expand node when widget is added dynamically', async ({
-      comfyPage
-    }) => {
-      await comfyPage.loadWorkflow('nodes/single_ksampler')
+test.describe('Dynamic widget manipulation', () => {
+  test('Auto expand node when widget is added dynamically', async ({
+    comfyPage
+  }) => {
+    await comfyPage.loadWorkflow('nodes/single_ksampler')
 
-      await comfyPage.page.evaluate(() => {
-        window['graph'].nodes[0].addWidget('number', 'new_widget', 10)
-        window['graph'].setDirtyCanvas(true, true)
-      })
-
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'ksampler_widget_added.png'
-      )
+    await comfyPage.page.evaluate(() => {
+      window['graph'].nodes[0].addWidget('number', 'new_widget', 10)
+      window['graph'].setDirtyCanvas(true, true)
     })
-  }
-)
 
-test.describe('Image widget', { tag: ['@screenshot', '@widget'] }, () => {
+    await expect(comfyPage.canvas).toHaveScreenshot('ksampler_widget_added.png')
+  })
+})
+
+test.describe('Image widget', () => {
   test('Can load image', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('widgets/load_image_widget')
     await expect(comfyPage.canvas).toHaveScreenshot('load_image_widget.png')
@@ -242,103 +236,99 @@ test.describe('Image widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe(
-  'Animated image widget',
-  { tag: ['@screenshot', '@widget'] },
-  () => {
-    // https://github.com/Comfy-Org/ComfyUI_frontend/issues/3718
-    test.skip('Shows preview of uploaded animated image', async ({
-      comfyPage
-    }) => {
-      await comfyPage.loadWorkflow('widgets/load_animated_webp')
+test.describe('Animated image widget', () => {
+  // https://github.com/Comfy-Org/ComfyUI_frontend/issues/3718
+  test.skip('Shows preview of uploaded animated image', async ({
+    comfyPage
+  }) => {
+    await comfyPage.loadWorkflow('widgets/load_animated_webp')
 
-      // Get position of the load animated webp node
-      const nodes = await comfyPage.getNodeRefsByType(
-        'DevToolsLoadAnimatedImageTest'
-      )
-      const loadAnimatedWebpNode = nodes[0]
-      const { x, y } = await loadAnimatedWebpNode.getPosition()
+    // Get position of the load animated webp node
+    const nodes = await comfyPage.getNodeRefsByType(
+      'DevToolsLoadAnimatedImageTest'
+    )
+    const loadAnimatedWebpNode = nodes[0]
+    const { x, y } = await loadAnimatedWebpNode.getPosition()
 
-      // Drag and drop image file onto the load animated webp node
-      await comfyPage.dragAndDropFile('animated_webp.webp', {
-        dropPosition: { x, y }
-      })
-
-      // Expect the image preview to change automatically
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'animated_image_preview_drag_and_dropped.png'
-      )
-
-      // Move mouse and click on canvas to trigger render
-      await comfyPage.page.mouse.click(64, 64)
-
-      // Expect the image preview to change to the next frame of the animation
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'animated_image_preview_drag_and_dropped_next_frame.png'
-      )
+    // Drag and drop image file onto the load animated webp node
+    await comfyPage.dragAndDropFile('animated_webp.webp', {
+      dropPosition: { x, y }
     })
 
-    test('Can drag-and-drop animated webp image', async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('widgets/load_animated_webp')
+    // Expect the image preview to change automatically
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'animated_image_preview_drag_and_dropped.png'
+    )
 
-      // Get position of the load animated webp node
-      const nodes = await comfyPage.getNodeRefsByType(
-        'DevToolsLoadAnimatedImageTest'
-      )
-      const loadAnimatedWebpNode = nodes[0]
-      const { x, y } = await loadAnimatedWebpNode.getPosition()
+    // Move mouse and click on canvas to trigger render
+    await comfyPage.page.mouse.click(64, 64)
 
-      // Drag and drop image file onto the load animated webp node
-      await comfyPage.dragAndDropFile('animated_webp.webp', {
-        dropPosition: { x, y },
-        waitForUpload: true
-      })
+    // Expect the image preview to change to the next frame of the animation
+    await expect(comfyPage.canvas).toHaveScreenshot(
+      'animated_image_preview_drag_and_dropped_next_frame.png'
+    )
+  })
 
-      // Expect the filename combo value to be updated
-      const fileComboWidget = await loadAnimatedWebpNode.getWidget(0)
-      const filename = await fileComboWidget.getValue()
-      expect(filename).toContain('animated_webp.webp')
+  test('Can drag-and-drop animated webp image', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('widgets/load_animated_webp')
+
+    // Get position of the load animated webp node
+    const nodes = await comfyPage.getNodeRefsByType(
+      'DevToolsLoadAnimatedImageTest'
+    )
+    const loadAnimatedWebpNode = nodes[0]
+    const { x, y } = await loadAnimatedWebpNode.getPosition()
+
+    // Drag and drop image file onto the load animated webp node
+    await comfyPage.dragAndDropFile('animated_webp.webp', {
+      dropPosition: { x, y },
+      waitForUpload: true
     })
 
-    test('Can preview saved animated webp image', async ({ comfyPage }) => {
-      await comfyPage.loadWorkflow('widgets/save_animated_webp')
+    // Expect the filename combo value to be updated
+    const fileComboWidget = await loadAnimatedWebpNode.getWidget(0)
+    const filename = await fileComboWidget.getValue()
+    expect(filename).toContain('animated_webp.webp')
+  })
 
-      // Get position of the load animated webp node
-      const loadNodes = await comfyPage.getNodeRefsByType(
-        'DevToolsLoadAnimatedImageTest'
-      )
-      const loadAnimatedWebpNode = loadNodes[0]
-      const { x, y } = await loadAnimatedWebpNode.getPosition()
+  test('Can preview saved animated webp image', async ({ comfyPage }) => {
+    await comfyPage.loadWorkflow('widgets/save_animated_webp')
 
-      // Drag and drop image file onto the load animated webp node
-      await comfyPage.dragAndDropFile('animated_webp.webp', {
-        dropPosition: { x, y }
-      })
-      await comfyPage.nextFrame()
+    // Get position of the load animated webp node
+    const loadNodes = await comfyPage.getNodeRefsByType(
+      'DevToolsLoadAnimatedImageTest'
+    )
+    const loadAnimatedWebpNode = loadNodes[0]
+    const { x, y } = await loadAnimatedWebpNode.getPosition()
 
-      // Get the SaveAnimatedWEBP node
-      const saveNodes = await comfyPage.getNodeRefsByType('SaveAnimatedWEBP')
-      const saveAnimatedWebpNode = saveNodes[0]
-      if (!saveAnimatedWebpNode)
-        throw new Error('SaveAnimatedWEBP node not found')
-
-      // Simulate the graph executing
-      await comfyPage.page.evaluate(
-        ([loadId, saveId]) => {
-          // Set the output of the SaveAnimatedWEBP node to equal the loader node's image
-          window['app'].nodeOutputs[saveId] = window['app'].nodeOutputs[loadId]
-          app.canvas.setDirty(true)
-        },
-        [loadAnimatedWebpNode.id, saveAnimatedWebpNode.id]
-      )
-      await expect(
-        comfyPage.page.locator('.dom-widget').locator('img')
-      ).toHaveCount(2)
+    // Drag and drop image file onto the load animated webp node
+    await comfyPage.dragAndDropFile('animated_webp.webp', {
+      dropPosition: { x, y }
     })
-  }
-)
+    await comfyPage.nextFrame()
 
-test.describe('Load audio widget', { tag: ['@screenshot', '@widget'] }, () => {
+    // Get the SaveAnimatedWEBP node
+    const saveNodes = await comfyPage.getNodeRefsByType('SaveAnimatedWEBP')
+    const saveAnimatedWebpNode = saveNodes[0]
+    if (!saveAnimatedWebpNode)
+      throw new Error('SaveAnimatedWEBP node not found')
+
+    // Simulate the graph executing
+    await comfyPage.page.evaluate(
+      ([loadId, saveId]) => {
+        // Set the output of the SaveAnimatedWEBP node to equal the loader node's image
+        window['app'].nodeOutputs[saveId] = window['app'].nodeOutputs[loadId]
+        app.canvas.setDirty(true)
+      },
+      [loadAnimatedWebpNode.id, saveAnimatedWebpNode.id]
+    )
+    await expect(
+      comfyPage.page.locator('.dom-widget').locator('img')
+    ).toHaveCount(2)
+  })
+})
+
+test.describe('Load audio widget', () => {
   test('Can load audio', async ({ comfyPage }) => {
     await comfyPage.loadWorkflow('widgets/load_audio_widget')
     // Wait for the audio widget to be rendered in the DOM
@@ -348,7 +338,7 @@ test.describe('Load audio widget', { tag: ['@screenshot', '@widget'] }, () => {
   })
 })
 
-test.describe('Unserialized widgets', { tag: '@widget' }, () => {
+test.describe('Unserialized widgets', () => {
   test('Unserialized widgets values do not mark graph as modified', async ({
     comfyPage
   }) => {

browser_tests/tests/workflowTabThumbnail.spec.ts

@@ -1,9 +1,8 @@
 import { expect } from '@playwright/test'
 
-import { comfyPageFixture as test } from '../fixtures/ComfyPage'
-import type { ComfyPage } from '../fixtures/ComfyPage'
+import { type ComfyPage, comfyPageFixture as test } from '../fixtures/ComfyPage'
 
-test.describe('Workflow Tab Thumbnails', { tag: '@workflow' }, () => {
+test.describe('Workflow Tab Thumbnails', () => {
   test.beforeEach(async ({ comfyPage }) => {
     await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
     await comfyPage.setSetting('Comfy.Workflow.WorkflowTabsPosition', 'Topbar')

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comfyorg/comfyui-frontend",
-  "version": "1.39.4",
+  "version": "1.39.0",
   "private": true,
   "description": "Official front-end implementation of ComfyUI",
   "homepage": "https://comfy.org",

src/App.vue

@@ -46,8 +46,6 @@ onMounted(() => {
     document.addEventListener('contextmenu', showContextMenu)
   }
 
-  // Handle preload errors that occur during dynamic imports (e.g., stale chunks after deployment)
-  // See: https://vite.dev/guide/build#load-error-handling
   window.addEventListener('vite:preloadError', (event) => {
     event.preventDefault()
     // eslint-disable-next-line no-undef

src/components/custom/widget/WorkflowTemplateSelectorDialog.vue

@@ -256,11 +256,6 @@
                         "
                       />
                     </template>
-                    <LogoOverlay
-                      v-if="template.logos?.length"
-                      :logos="template.logos"
-                      :get-logo-url="workflowTemplatesStore.getLogoUrl"
-                    />
                     <ProgressSpinner
                       v-if="loadingTemplate === template.name"
                       class="absolute inset-0 z-10 m-auto h-12 w-12"
@@ -402,7 +397,6 @@ import AudioThumbnail from '@/components/templates/thumbnails/AudioThumbnail.vue
 import CompareSliderThumbnail from '@/components/templates/thumbnails/CompareSliderThumbnail.vue'
 import DefaultThumbnail from '@/components/templates/thumbnails/DefaultThumbnail.vue'
 import HoverDissolveThumbnail from '@/components/templates/thumbnails/HoverDissolveThumbnail.vue'
-import LogoOverlay from '@/components/templates/thumbnails/LogoOverlay.vue'
 import Button from '@/components/ui/button/Button.vue'
 import BaseModalLayout from '@/components/widget/layout/BaseModalLayout.vue'
 import LeftSidePanel from '@/components/widget/panel/LeftSidePanel.vue'
@@ -778,7 +772,7 @@ useIntersectionObserver(loadTrigger, () => {
 // Reset pagination when filters change
 watch(
   [
-    filteredTemplates,
+    searchQuery,
     selectedNavItem,
     sortBy,
     selectedModels,

src/components/dialog/content/setting/KeybindingPanel.vue

@@ -150,11 +150,13 @@ import { useI18n } from 'vue-i18n'
 
 import SearchBox from '@/components/common/SearchBox.vue'
 import Button from '@/components/ui/button/Button.vue'
-import { KeyComboImpl } from '@/platform/keybindings/keyCombo'
-import { KeybindingImpl } from '@/platform/keybindings/keybinding'
-import { useKeybindingService } from '@/platform/keybindings/keybindingService'
-import { useKeybindingStore } from '@/platform/keybindings/keybindingStore'
+import { useKeybindingService } from '@/services/keybindingService'
 import { useCommandStore } from '@/stores/commandStore'
+import {
+  KeyComboImpl,
+  KeybindingImpl,
+  useKeybindingStore
+} from '@/stores/keybindingStore'
 import { normalizeI18nKey } from '@/utils/formatUtil'
 
 import PanelTemplate from './PanelTemplate.vue'
@@ -263,15 +265,18 @@ function cancelEdit() {
 }
 
 async function saveKeybinding() {
-  const commandId = currentEditingCommand.value?.id
-  const combo = newBindingKeyCombo.value
+  if (currentEditingCommand.value && newBindingKeyCombo.value) {
+    const updated = keybindingStore.updateKeybindingOnCommand(
+      new KeybindingImpl({
+        commandId: currentEditingCommand.value.id,
+        combo: newBindingKeyCombo.value
+      })
+    )
+    if (updated) {
+      await keybindingService.persistUserKeybindings()
+    }
+  }
   cancelEdit()
-  if (!combo || commandId == undefined) return
-
-  const updated = keybindingStore.updateKeybindingOnCommand(
-    new KeybindingImpl({ commandId, combo })
-  )
-  if (updated) await keybindingService.persistUserKeybindings()
 }
 
 async function resetKeybinding(commandData: ICommandData) {

src/components/dialog/content/setting/UserPanel.vue

@@ -63,18 +63,14 @@
             <i class="pi pi-sign-out" />
             {{ $t('auth.signOut.signOut') }}
           </Button>
-          <i18n-t
+          <Button
             v-if="!isApiKeyLogin"
-            keypath="auth.deleteAccount.contactSupport"
-            tag="p"
-            class="text-muted text-sm"
+            class="w-fit"
+            variant="destructive-textonly"
+            @click="handleDeleteAccount"
           >
-            <template #email>
-              <a href="mailto:support@comfy.org" class="underline"
-                >support@comfy.org</a
-              >
-            </template>
-          </i18n-t>
+            {{ $t('auth.deleteAccount.deleteAccount') }}
+          </Button>
         </div>
       </div>
 
@@ -120,6 +116,7 @@ const {
   providerName,
   providerIcon,
   handleSignOut,
-  handleSignIn
+  handleSignIn,
+  handleDeleteAccount
 } = useCurrentUser()
 </script>

src/components/dialog/content/setting/keybinding/KeyComboDisplay.vue

@@ -13,7 +13,7 @@
 import Tag from 'primevue/tag'
 import { computed } from 'vue'
 
-import type { KeyComboImpl } from '@/platform/keybindings/keyCombo'
+import type { KeyComboImpl } from '@/stores/keybindingStore'
 
 const { keyCombo, isModified = false } = defineProps<{
   keyCombo: KeyComboImpl

src/components/helpcenter/HelpCenterPopups.vue

@@ -1,6 +1,6 @@
 <template>
   <!-- Help Center Popup positioned within canvas area -->
-  <Teleport to="body">
+  <Teleport to="#graph-canvas-container">
     <div
       v-if="isHelpCenterVisible"
       class="help-center-popup"

src/components/node/NodePreview.test.ts

@@ -76,6 +76,14 @@ describe('NodePreview', () => {
     expect(wrapper.find('._sb_preview_badge').text()).toBe('Preview')
   })
 
+  it('applies text-ellipsis class to node header for text truncation', () => {
+    const wrapper = mountComponent()
+    const nodeHeader = wrapper.find('.node_header')
+
+    expect(nodeHeader.classes()).toContain('text-ellipsis')
+    expect(nodeHeader.classes()).toContain('mr-4')
+  })
+
   it('sets title attribute on node header with full display name', () => {
     const wrapper = mountComponent()
     const nodeHeader = wrapper.find('.node_header')

src/components/node/NodePreview.vue

@@ -10,7 +10,7 @@ https://github.com/Nuked88/ComfyUI-N-Sidebar/blob/7ae7da4a9761009fb6629bc04c6830
   <div v-else class="_sb_node_preview bg-component-node-background">
     <div class="_sb_table">
       <div
-        class="node_header text-ellipsis"
+        class="node_header mr-4 text-ellipsis"
         :title="nodeDef.display_name"
         :style="{
           backgroundColor: litegraphColors.NODE_DEFAULT_COLOR,

src/components/sidebar/SideToolbar.vue

@@ -41,7 +41,7 @@
           :is-small="isSmall"
         />
         <SidebarHelpCenterIcon v-if="!isIntegratedTabBar" :is-small="isSmall" />
-        <SidebarBottomPanelToggleButton v-if="!isCloud" :is-small="isSmall" />
+        <SidebarBottomPanelToggleButton :is-small="isSmall" />
         <SidebarShortcutsToggleButton :is-small="isSmall" />
         <SidebarSettingsButton :is-small="isSmall" />
         <ModeToggle
@@ -65,12 +65,11 @@ import SidebarBottomPanelToggleButton from '@/components/sidebar/SidebarBottomPa
 import SidebarSettingsButton from '@/components/sidebar/SidebarSettingsButton.vue'
 import SidebarShortcutsToggleButton from '@/components/sidebar/SidebarShortcutsToggleButton.vue'
 import { useFeatureFlags } from '@/composables/useFeatureFlags'
-import { isCloud } from '@/platform/distribution/types'
 import { useSettingStore } from '@/platform/settings/settingStore'
 import { useTelemetry } from '@/platform/telemetry'
 import { useCanvasStore } from '@/renderer/core/canvas/canvasStore'
 import { useCommandStore } from '@/stores/commandStore'
-import { useKeybindingStore } from '@/platform/keybindings/keybindingStore'
+import { useKeybindingStore } from '@/stores/keybindingStore'
 import { useMenuItemStore } from '@/stores/menuItemStore'
 import { useUserStore } from '@/stores/userStore'
 import { useWorkspaceStore } from '@/stores/workspaceStore'

src/components/sidebar/tabs/AssetsSidebarGridView.vue

@@ -2,7 +2,7 @@
   <div class="flex h-full flex-col">
     <!-- Active Jobs Grid -->
     <div
-      v-if="isQueuePanelV2Enabled && activeJobItems.length"
+      v-if="activeJobItems.length"
       class="grid max-h-[50%] scrollbar-custom overflow-y-auto"
       :style="gridStyle"
     >
@@ -65,7 +65,6 @@ import MediaAssetCard from '@/platform/assets/components/MediaAssetCard.vue'
 import type { AssetItem } from '@/platform/assets/schemas/assetSchema'
 import { isActiveJobState } from '@/utils/queueUtil'
 import { cn } from '@/utils/tailwindUtil'
-import { useSettingStore } from '@/platform/settings/settingStore'
 
 const {
   assets,
@@ -91,11 +90,6 @@ const emit = defineEmits<{
 
 const { t } = useI18n()
 const { jobItems } = useJobList()
-const settingStore = useSettingStore()
-
-const isQueuePanelV2Enabled = computed(() =>
-  settingStore.get('Comfy.Queue.QPOV2')
-)
 
 type AssetGridItem = { key: string; asset: AssetItem }
 

src/components/sidebar/tabs/AssetsSidebarListView.vue

@@ -1,7 +1,7 @@
 <template>
   <div class="flex h-full flex-col">
     <div
-      v-if="isQueuePanelV2Enabled && activeJobItems.length"
+      v-if="activeJobItems.length"
       class="flex max-h-[50%] scrollbar-custom flex-col gap-2 overflow-y-auto px-2"
     >
       <AssetsListItem
@@ -133,7 +133,6 @@ import {
 } from '@/utils/formatUtil'
 import { iconForJobState } from '@/utils/queueDisplay'
 import { cn } from '@/utils/tailwindUtil'
-import { useSettingStore } from '@/platform/settings/settingStore'
 
 const {
   assets,
@@ -155,11 +154,6 @@ const emit = defineEmits<{
 
 const { t } = useI18n()
 const { jobItems } = useJobList()
-const settingStore = useSettingStore()
-
-const isQueuePanelV2Enabled = computed(() =>
-  settingStore.get('Comfy.Queue.QPOV2')
-)
 const hoveredJobId = ref<string | null>(null)
 const hoveredAssetId = ref<string | null>(null)
 

src/components/sidebar/tabs/nodeLibrary/NodeTreeLeaf.vue

@@ -13,7 +13,10 @@
           severity="danger"
         />
       </template>
-      <template v-if="isUserBlueprint" #actions>
+      <template
+        v-if="nodeDef.name.startsWith(useSubgraphStore().typePrefix)"
+        #actions
+      >
         <Button
           variant="destructive"
           size="icon-sm"
@@ -125,18 +128,8 @@ const editBlueprint = async () => {
   await useSubgraphStore().editBlueprint(props.node.data.name)
 }
 const menu = ref<InstanceType<typeof ContextMenu> | null>(null)
-const subgraphStore = useSubgraphStore()
-const isUserBlueprint = computed(() => {
-  const name = nodeDef.value.name
-  if (!name.startsWith(subgraphStore.typePrefix)) return false
-  return !subgraphStore.isGlobalBlueprint(
-    name.slice(subgraphStore.typePrefix.length)
-  )
-})
 const menuItems = computed<MenuItem[]>(() => {
-  if (!isUserBlueprint.value) return []
-
-  return [
+  const items: MenuItem[] = [
     {
       label: t('g.delete'),
       icon: 'pi pi-trash',
@@ -144,14 +137,15 @@ const menuItems = computed<MenuItem[]>(() => {
       command: deleteBlueprint
     }
   ]
+  return items
 })
 function handleContextMenu(event: Event) {
-  if (!isUserBlueprint.value) return
+  if (!nodeDef.value.name.startsWith(useSubgraphStore().typePrefix)) return
   menu.value?.show(event)
 }
 function deleteBlueprint() {
   if (!props.node.data) return
-  void subgraphStore.deleteBlueprint(props.node.data.name)
+  void useSubgraphStore().deleteBlueprint(props.node.data.name)
 }
 
 const nodePreviewStyle = ref<CSSProperties>({

src/components/templates/thumbnails/LogoOverlay.test.ts

@@ -1,157 +0,0 @@
-import { mount } from '@vue/test-utils'
-import type { ComponentProps } from 'vue-component-type-helpers'
-import { nextTick, ref } from 'vue'
-import { describe, expect, it, vi } from 'vitest'
-
-import LogoOverlay from '@/components/templates/thumbnails/LogoOverlay.vue'
-import type { LogoInfo } from '@/platform/workflow/templates/types/template'
-
-vi.mock('vue-i18n', () => ({
-  useI18n: () => ({
-    t: (key: string) =>
-      key === 'templates.logoProviderSeparator' ? ' & ' : key,
-    locale: ref('en')
-  })
-}))
-
-type LogoOverlayProps = ComponentProps<typeof LogoOverlay>
-
-describe('LogoOverlay', () => {
-  function mockGetLogoUrl(provider: string) {
-    return `/logos/${provider}.png`
-  }
-
-  function mountOverlay(
-    logos: LogoInfo[],
-    props: Partial<LogoOverlayProps> = {}
-  ) {
-    return mount(LogoOverlay, {
-      props: {
-        logos,
-        getLogoUrl: mockGetLogoUrl,
-        ...props
-      }
-    })
-  }
-
-  it('renders nothing when logos array is empty', () => {
-    const wrapper = mountOverlay([])
-    expect(wrapper.findAll('img')).toHaveLength(0)
-  })
-
-  it('renders a single logo with correct src and alt', () => {
-    const wrapper = mountOverlay([{ provider: 'Google' }])
-    const img = wrapper.find('img')
-    expect(img.attributes('src')).toBe('/logos/Google.png')
-    expect(img.attributes('alt')).toBe('Google')
-  })
-
-  it('renders multiple separate logo entries', () => {
-    const wrapper = mountOverlay([
-      { provider: 'Google' },
-      { provider: 'OpenAI' },
-      { provider: 'Stability' }
-    ])
-    expect(wrapper.findAll('img')).toHaveLength(3)
-  })
-
-  it('displays provider name as label for single provider', () => {
-    const wrapper = mountOverlay([{ provider: 'Google' }])
-    const span = wrapper.find('span')
-    expect(span.text()).toBe('Google')
-  })
-
-  it('images are not draggable', () => {
-    const wrapper = mountOverlay([{ provider: 'Google' }])
-    const img = wrapper.find('img')
-    expect(img.attributes('draggable')).toBe('false')
-  })
-
-  it('filters out logos with empty URLs', () => {
-    function getLogoUrl(provider: string) {
-      return provider === 'Google' ? '/logos/Google.png' : ''
-    }
-    const wrapper = mount(LogoOverlay, {
-      props: {
-        logos: [{ provider: 'Google' }, { provider: 'Unknown' }],
-        getLogoUrl
-      }
-    })
-    expect(wrapper.findAll('img')).toHaveLength(1)
-  })
-
-  it('renders one logo per unique provider', () => {
-    const wrapper = mountOverlay([
-      { provider: 'Google' },
-      { provider: 'OpenAI' }
-    ])
-    expect(wrapper.findAll('img')).toHaveLength(2)
-  })
-
-  describe('stacked logos', () => {
-    it('renders multiple providers as stacked overlapping logos', () => {
-      const wrapper = mountOverlay([{ provider: ['WaveSpeed', 'Hunyuan'] }])
-      const images = wrapper.findAll('img')
-      expect(images).toHaveLength(2)
-      expect(images[0].attributes('alt')).toBe('WaveSpeed')
-      expect(images[1].attributes('alt')).toBe('Hunyuan')
-    })
-
-    it('joins provider names with locale-aware conjunction for default label', () => {
-      const wrapper = mountOverlay([{ provider: ['WaveSpeed', 'Hunyuan'] }])
-      const span = wrapper.find('span')
-      expect(span.text()).toBe('WaveSpeed and Hunyuan')
-    })
-
-    it('uses custom label when provided', () => {
-      const wrapper = mountOverlay([
-        { provider: ['WaveSpeed', 'Hunyuan'], label: 'Custom Label' }
-      ])
-      const span = wrapper.find('span')
-      expect(span.text()).toBe('Custom Label')
-    })
-
-    it('applies negative gap for overlap effect', () => {
-      const wrapper = mountOverlay([
-        { provider: ['WaveSpeed', 'Hunyuan'], gap: -8 }
-      ])
-      const images = wrapper.findAll('img')
-      expect(images[1].attributes('style')).toContain('margin-left: -8px')
-    })
-
-    it('applies default gap when not specified', () => {
-      const wrapper = mountOverlay([{ provider: ['WaveSpeed', 'Hunyuan'] }])
-      const images = wrapper.findAll('img')
-      expect(images[1].attributes('style')).toContain('margin-left: -6px')
-    })
-
-    it('filters out invalid providers from stacked logos', () => {
-      function getLogoUrl(provider: string) {
-        return provider === 'WaveSpeed' ? '/logos/WaveSpeed.png' : ''
-      }
-      const wrapper = mount(LogoOverlay, {
-        props: {
-          logos: [{ provider: ['WaveSpeed', 'Unknown'] }],
-          getLogoUrl
-        }
-      })
-      expect(wrapper.findAll('img')).toHaveLength(1)
-      expect(wrapper.find('span').text()).toBe('WaveSpeed')
-    })
-  })
-
-  describe('error handling', () => {
-    it('keeps showing remaining providers when one image fails in stacked logos', async () => {
-      const wrapper = mountOverlay([{ provider: ['Google', 'OpenAI'] }])
-      const images = wrapper.findAll('[data-testid="logo-img"]')
-      expect(images).toHaveLength(2)
-
-      await images[0].trigger('error')
-      await nextTick()
-
-      const remainingImages = wrapper.findAll('[data-testid="logo-img"]')
-      expect(remainingImages).toHaveLength(2)
-      expect(remainingImages[1].attributes('alt')).toBe('OpenAI')
-    })
-  })
-})

src/components/templates/thumbnails/LogoOverlay.vue

@@ -1,117 +0,0 @@
-<template>
-  <div
-    v-for="logo in validLogos"
-    :key="logo.key"
-    :class="
-      cn('pointer-events-none absolute z-10', logo.position ?? defaultPosition)
-    "
-  >
-    <div
-      v-show="!hasAllFailed(logo.providers)"
-      data-testid="logo-pill"
-      class="flex items-center gap-1.5 rounded-full bg-black/20 py-1 pr-2"
-      :style="{ opacity: logo.opacity ?? 0.85 }"
-    >
-      <div class="ml-0.5 flex items-center">
-        <img
-          v-for="(provider, providerIndex) in logo.providers"
-          :key="provider"
-          data-testid="logo-img"
-          :src="logo.urls[providerIndex]"
-          :alt="provider"
-          class="h-6 w-6 rounded-full border-2 border-white object-cover"
-          :class="{ relative: providerIndex > 0 }"
-          :style="
-            providerIndex > 0 ? { marginLeft: `${logo.gap ?? -6}px` } : {}
-          "
-          draggable="false"
-          @error="onImageError(provider)"
-        />
-      </div>
-      <span class="text-sm font-medium text-white">
-        {{ logo.label }}
-      </span>
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed, ref } from 'vue'
-import { useI18n } from 'vue-i18n'
-
-import type { LogoInfo } from '@/platform/workflow/templates/types/template'
-import { cn } from '@/utils/tailwindUtil'
-
-const { t, locale } = useI18n()
-
-function formatProviderList(providers: string[]): string {
-  const localeValue = String(locale.value)
-  try {
-    return new Intl.ListFormat(localeValue, {
-      style: 'long',
-      type: 'conjunction'
-    }).format(providers)
-  } catch {
-    return providers.join(t('templates.logoProviderSeparator'))
-  }
-}
-
-const {
-  logos,
-  getLogoUrl,
-  defaultPosition = 'top-2 left-2'
-} = defineProps<{
-  logos: LogoInfo[]
-  getLogoUrl: (provider: string) => string
-  defaultPosition?: string
-}>()
-
-const failedLogos = ref(new Set<string>())
-
-function onImageError(provider: string) {
-  failedLogos.value = new Set([...failedLogos.value, provider])
-}
-
-function hasAllFailed(providers: string[]): boolean {
-  return providers.every((p) => failedLogos.value.has(p))
-}
-
-interface ValidatedLogo {
-  key: string
-  providers: string[]
-  urls: string[]
-  label: string
-  position: string | undefined
-  opacity: number | undefined
-  gap: number | undefined
-}
-
-const validLogos = computed<ValidatedLogo[]>(() => {
-  const result: ValidatedLogo[] = []
-
-  logos.forEach((logo, index) => {
-    const providers = Array.isArray(logo.provider)
-      ? logo.provider
-      : [logo.provider]
-    const urls = providers.map((p) => getLogoUrl(p))
-    const validProviders = providers.filter((_, i) => urls[i])
-    const validUrls = urls.filter((url) => url)
-
-    if (validProviders.length === 0) return
-
-    const providerKey = validProviders.join('-')
-    const layoutKey = `${logo.position ?? ''}-${logo.opacity ?? ''}-${logo.gap ?? ''}`
-    result.push({
-      key: providerKey ? `${providerKey}-${layoutKey}` : `logo-${index}`,
-      providers: validProviders,
-      urls: validUrls,
-      label: logo.label ?? formatProviderList(validProviders),
-      position: logo.position,
-      opacity: logo.opacity,
-      gap: logo.gap
-    })
-  })
-
-  return result
-})
-</script>

src/composables/auth/useCurrentUser.ts

@@ -1,6 +1,9 @@
 import { whenever } from '@vueuse/core'
 import { computed, watch } from 'vue'
 
+import { useFirebaseAuthActions } from '@/composables/auth/useFirebaseAuthActions'
+import { t } from '@/i18n'
+import { useDialogService } from '@/services/dialogService'
 import { useApiKeyAuthStore } from '@/stores/apiKeyAuthStore'
 import { useCommandStore } from '@/stores/commandStore'
 import { useFirebaseAuthStore } from '@/stores/firebaseAuthStore'
@@ -10,6 +13,8 @@ export const useCurrentUser = () => {
   const authStore = useFirebaseAuthStore()
   const commandStore = useCommandStore()
   const apiKeyStore = useApiKeyAuthStore()
+  const dialogService = useDialogService()
+  const { deleteAccount } = useFirebaseAuthActions()
 
   const firebaseUser = computed(() => authStore.currentUser)
   const isApiKeyLogin = computed(() => apiKeyStore.isAuthenticated)
@@ -111,6 +116,18 @@ export const useCurrentUser = () => {
     await commandStore.execute('Comfy.User.OpenSignInDialog')
   }
 
+  const handleDeleteAccount = async () => {
+    const confirmed = await dialogService.confirm({
+      title: t('auth.deleteAccount.confirmTitle'),
+      message: t('auth.deleteAccount.confirmMessage'),
+      type: 'delete'
+    })
+
+    if (confirmed) {
+      await deleteAccount()
+    }
+  }
+
   return {
     loading: authStore.loading,
     isLoggedIn,
@@ -124,6 +141,7 @@ export const useCurrentUser = () => {
     resolvedUserInfo,
     handleSignOut,
     handleSignIn,
+    handleDeleteAccount,
     onUserResolved,
     onTokenRefreshed,
     onUserLogout

src/composables/auth/useFirebaseAuthActions.ts

@@ -206,6 +206,21 @@ export const useFirebaseAuthActions = () => {
     [createReauthenticationRecovery<[string], void>()]
   )
 
+  const deleteAccount = wrapWithErrorHandlingAsync(
+    async () => {
+      await authStore.deleteAccount()
+      toastStore.add({
+        severity: 'success',
+        summary: t('auth.deleteAccount.success'),
+        detail: t('auth.deleteAccount.successDetail'),
+        life: 5000
+      })
+    },
+    reportError,
+    undefined,
+    [createReauthenticationRecovery<[], void>()]
+  )
+
   return {
     logout,
     sendPasswordReset,
@@ -217,6 +232,7 @@ export const useFirebaseAuthActions = () => {
     signInWithEmail,
     signUpWithEmail,
     updatePassword,
+    deleteAccount,
     accessError,
     reportError
   }

src/composables/canvas/useSelectedLiteGraphItems.test.ts

@@ -276,7 +276,7 @@ describe('useSelectedLiteGraphItems', () => {
       expect(selectedNodes).toContainEqual(subNode2)
     })
 
-    it('toggleSelectedNodesMode should not apply state to subgraph children', () => {
+    it('toggleSelectedNodesMode should apply unified state to subgraph children', () => {
       const { toggleSelectedNodesMode } = useSelectedLiteGraphItems()
       const subNode1 = { id: 11, mode: LGraphEventMode.ALWAYS } as LGraphNode
       const subNode2 = { id: 12, mode: LGraphEventMode.NEVER } as LGraphNode
@@ -294,8 +294,9 @@ describe('useSelectedLiteGraphItems', () => {
       // regularNode: BYPASS -> NEVER (since BYPASS != NEVER)
       expect(regularNode.mode).toBe(LGraphEventMode.NEVER)
 
-      // Subgraph children do not change state
-      expect(subNode1.mode).toBe(LGraphEventMode.ALWAYS) // was ALWAYS, stays ALWAYS
+      // Subgraph children get unified state (same as their parent):
+      // Both children should now be NEVER, regardless of their previous states
+      expect(subNode1.mode).toBe(LGraphEventMode.NEVER) // was ALWAYS, now NEVER
       expect(subNode2.mode).toBe(LGraphEventMode.NEVER) // was NEVER, stays NEVER
     })
 
@@ -316,9 +317,9 @@ describe('useSelectedLiteGraphItems', () => {
       // Selected subgraph should toggle to ALWAYS (since it was already NEVER)
       expect(subgraphNode.mode).toBe(LGraphEventMode.ALWAYS)
 
-      // All children should be unchanged
+      // All children should also get ALWAYS (unified with parent's new state)
       expect(subNode1.mode).toBe(LGraphEventMode.ALWAYS)
-      expect(subNode2.mode).toBe(LGraphEventMode.BYPASS)
+      expect(subNode2.mode).toBe(LGraphEventMode.ALWAYS)
     })
   })
 

src/composables/canvas/useSelectedLiteGraphItems.ts

@@ -2,7 +2,10 @@ import type { LGraphNode, Positionable } from '@/lib/litegraph/src/litegraph'
 import { LGraphEventMode, Reroute } from '@/lib/litegraph/src/litegraph'
 import { useCanvasStore } from '@/renderer/core/canvas/canvasStore'
 import { app } from '@/scripts/app'
-import { collectFromNodes } from '@/utils/graphTraversalUtil'
+import {
+  collectFromNodes,
+  traverseNodesDepthFirst
+} from '@/utils/graphTraversalUtil'
 
 /**
  * Composable for handling selected LiteGraph items filtering and operations.
@@ -94,10 +97,16 @@ export function useSelectedLiteGraphItems() {
   }
 
   /**
-   * Toggle the execution mode of all selected nodes
+   * Toggle the execution mode of all selected nodes with unified subgraph behavior.
    *
-   * - If any nodes are not already the specified node mode → all are set to specified mode
-   * - Otherwise → set all nodes to ALWAYS
+   * Top-level behavior (selected nodes): Standard toggle logic
+   * - If the selected node is already in the specified mode → set to ALWAYS
+   * - Otherwise → set to the specified mode
+   *
+   * Subgraph behavior (children of selected subgraph nodes): Unified state application
+   * - All children inherit the same mode that their parent subgraph node was set to
+   * - This creates predictable behavior: if you toggle a subgraph to "mute",
+   *   ALL nodes inside become muted, regardless of their previous individual states
    *
    * @param mode - The LGraphEventMode to toggle to (e.g., NEVER for mute, BYPASS for bypass)
    */
@@ -115,8 +124,27 @@ export function useSelectedLiteGraphItems() {
     )
     const newModeForSelectedNode = allNodesMatch ? LGraphEventMode.ALWAYS : mode
 
-    for (const selectedNode of selectedNodeArray)
+    // Process each selected node independently to determine its target state and apply to children
+    selectedNodeArray.forEach((selectedNode) => {
+      // Apply standard toggle logic to the selected node itself
+
       selectedNode.mode = newModeForSelectedNode
+
+      // If this selected node is a subgraph, apply the same mode uniformly to all its children
+      // This ensures predictable behavior: all children get the same state as their parent
+      if (selectedNode.isSubgraphNode?.() && selectedNode.subgraph) {
+        traverseNodesDepthFirst([selectedNode], {
+          visitor: (node) => {
+            // Skip the parent node since we already handled it above
+            if (node === selectedNode) return undefined
+
+            // Apply the parent's new mode to all children uniformly
+            node.mode = newModeForSelectedNode
+            return undefined
+          }
+        })
+      }
+    })
   }
 
   return {

src/composables/graph/useGraphErrorState.ts

@@ -0,0 +1,76 @@
+import { watch } from 'vue'
+
+import type { LGraphNode, Subgraph } from '@/lib/litegraph/src/litegraph'
+import { app } from '@/scripts/app'
+import { useGraphErrorStateStore } from '@/stores/graphErrorStateStore'
+import { parseNodeLocatorId } from '@/types/nodeIdentification'
+import {
+  findSubgraphByUuid,
+  forEachNode,
+  forEachSubgraphNode
+} from '@/utils/graphTraversalUtil'
+
+export function useGraphErrorState(): void {
+  const store = useGraphErrorStateStore()
+
+  watch(
+    () => store.version,
+    () => {
+      const rootGraph = app.rootGraph
+      if (!rootGraph) return
+
+      forEachNode(rootGraph, (node) => {
+        node.has_errors = false
+        if (node.inputs) {
+          for (const slot of node.inputs) {
+            slot.hasErrors = false
+          }
+        }
+      })
+
+      for (const [nodeId, keys] of store.keysByNode) {
+        if (keys.size === 0) continue
+
+        const parsed = parseNodeLocatorId(nodeId)
+        if (!parsed) continue
+
+        const targetGraph = parsed.subgraphUuid
+          ? findSubgraphByUuid(rootGraph, parsed.subgraphUuid)
+          : rootGraph
+        if (!targetGraph) continue
+
+        const node = targetGraph.getNodeById(parsed.localNodeId)
+        if (!node) continue
+
+        node.has_errors = true
+
+        for (const key of keys) {
+          const error = store.errorsByKey.get(key)
+          if (error && error.target.kind === 'slot' && node.inputs) {
+            const slotName = error.target.slotName
+            const slot = node.inputs.find((s) => s.name === slotName)
+            if (slot) {
+              slot.hasErrors = true
+            }
+          }
+        }
+
+        propagateErrorToParents(node)
+      }
+    },
+    { immediate: true }
+  )
+}
+
+function propagateErrorToParents(node: LGraphNode): void {
+  const subgraph = node.graph as Subgraph | undefined
+  if (!subgraph || subgraph.isRootGraph) return
+
+  const subgraphId = subgraph.id
+  if (!subgraphId) return
+
+  forEachSubgraphNode(app.rootGraph, subgraphId, (subgraphNode) => {
+    subgraphNode.has_errors = true
+    propagateErrorToParents(subgraphNode)
+  })
+}

src/composables/graph/useMoreOptionsMenu.ts

@@ -1,9 +1,8 @@
 import { computed, ref } from 'vue'
 import type { Ref } from 'vue'
 
-import type { LGraphGroup, LGraphNode } from '@/lib/litegraph/src/litegraph'
+import type { LGraphGroup } from '@/lib/litegraph/src/litegraph'
 import { useCanvasStore } from '@/renderer/core/canvas/canvasStore'
-import { getExtraOptionsForWidget } from '@/services/litegraphService'
 import { isLGraphGroup } from '@/utils/litegraphUtil'
 
 import {
@@ -46,8 +45,6 @@ export enum BadgeVariant {
 // Global singleton for NodeOptions component reference
 let nodeOptionsInstance: null | NodeOptionsInstance = null
 
-const hoveredWidgetName = ref<string>()
-
 /**
  * Toggle the node options popover
  * @param event - The trigger event
@@ -64,13 +61,6 @@ export function toggleNodeOptions(event: Event) {
  * @param event - The trigger event (must be MouseEvent for position)
  */
 export function showNodeOptions(event: MouseEvent) {
-  hoveredWidgetName.value = undefined
-  const target = event.target
-  if (target instanceof HTMLElement) {
-    const widgetEl = target.closest('.lg-node-widget')
-    if (widgetEl instanceof HTMLElement)
-      hoveredWidgetName.value = widgetEl.dataset.widgetName
-  }
   if (nodeOptionsInstance?.show) {
     nodeOptionsInstance.show(event)
   }
@@ -143,8 +133,8 @@ export function useMoreOptionsMenu() {
   } = useGroupMenuOptions()
   const {
     getBasicSelectionOptions,
-    getMultipleNodesOptions,
-    getSubgraphOptions
+    getSubgraphOptions,
+    getMultipleNodesOptions
   } = useSelectionMenuOptions()
 
   const hasSubgraphs = hasSubgraphsComputed
@@ -174,13 +164,13 @@ export function useMoreOptionsMenu() {
 
     // For single node selection, also get LiteGraph menu items to merge
     const litegraphOptions: MenuOption[] = []
-    const node: LGraphNode | undefined = selectedNodes.value[0]
     if (
       selectedNodes.value.length === 1 &&
       !groupContext &&
       canvasStore.canvas
     ) {
       try {
+        const node = selectedNodes.value[0]
         const rawItems = canvasStore.canvas.getNodeMenuOptions(node)
         // Don't apply structuring yet - we'll do it after merging with Vue options
         litegraphOptions.push(
@@ -259,18 +249,6 @@ export function useMoreOptionsMenu() {
       options.push(...getImageMenuOptions(selectedNodes.value[0]))
       options.push({ type: 'divider' })
     }
-    const rawName = hoveredWidgetName.value
-    const widget = node?.widgets?.find((w) => w.name === rawName)
-    if (widget) {
-      const widgetOptions = convertContextMenuToOptions(
-        getExtraOptionsForWidget(node, widget)
-      )
-      if (widgetOptions) {
-        options.push(...widgetOptions)
-        options.push({ type: 'divider' })
-      }
-    }
-
     // Section 6 & 7: Extensions and Delete are handled by buildStructuredMenu
 
     // Mark all Vue options with source

src/composables/graph/useRequiredConnectionValidator.test.ts

@@ -0,0 +1,122 @@
+import { createPinia, setActivePinia } from 'pinia'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+
+import { useGraphErrorStateStore } from '@/stores/graphErrorStateStore'
+
+vi.mock('@/scripts/api', () => ({
+  api: {
+    addEventListener: vi.fn(),
+    removeEventListener: vi.fn()
+  }
+}))
+
+vi.mock('@/scripts/app', () => ({
+  app: {
+    rootGraph: null
+  }
+}))
+
+vi.mock('@/stores/nodeDefStore', () => ({
+  useNodeDefStore: vi.fn(() => ({
+    nodeDefsByName: {}
+  }))
+}))
+
+vi.mock('@/utils/graphTraversalUtil', () => ({
+  forEachNode: vi.fn()
+}))
+
+describe('useRequiredConnectionValidator', () => {
+  beforeEach(() => {
+    setActivePinia(createPinia())
+    vi.clearAllMocks()
+  })
+
+  describe('store integration', () => {
+    it('adds errors for missing required connections', () => {
+      const store = useGraphErrorStateStore()
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'frontend',
+        errors: [
+          {
+            key: 'frontend:missing:1:model',
+            source: 'frontend',
+            target: { kind: 'slot', nodeId: '1', slotName: 'model' },
+            code: 'MISSING_REQUIRED_INPUT'
+          }
+        ]
+      })
+
+      expect(store.hasSlotError('1', 'model')).toBe(true)
+      expect(store.hasErrorsForNode('1')).toBe(true)
+    })
+
+    it('clears errors when connections are made', () => {
+      const store = useGraphErrorStateStore()
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'frontend',
+        errors: [
+          {
+            key: 'frontend:missing:1:model',
+            source: 'frontend',
+            target: { kind: 'slot', nodeId: '1', slotName: 'model' }
+          }
+        ]
+      })
+
+      expect(store.hasSlotError('1', 'model')).toBe(true)
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'frontend',
+        errors: []
+      })
+
+      expect(store.hasSlotError('1', 'model')).toBe(false)
+    })
+
+    it('preserves backend errors when frontend errors change', () => {
+      const store = useGraphErrorStateStore()
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'backend',
+        errors: [
+          {
+            key: 'backend:node:2',
+            source: 'backend',
+            target: { kind: 'node', nodeId: '2' }
+          }
+        ]
+      })
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'frontend',
+        errors: [
+          {
+            key: 'frontend:missing:1:model',
+            source: 'frontend',
+            target: { kind: 'slot', nodeId: '1', slotName: 'model' }
+          }
+        ]
+      })
+
+      expect(store.hasErrorsForNode('1')).toBe(true)
+      expect(store.hasErrorsForNode('2')).toBe(true)
+
+      store.execute({
+        type: 'REPLACE_SOURCE',
+        source: 'frontend',
+        errors: []
+      })
+
+      expect(store.hasErrorsForNode('1')).toBe(false)
+      expect(store.hasErrorsForNode('2')).toBe(true)
+    })
+  })
+})

src/composables/graph/useRequiredConnectionValidator.ts

@@ -0,0 +1,74 @@
+import { useDebounceFn } from '@vueuse/core'
+import { onUnmounted } from 'vue'
+
+import type { LGraphNode } from '@/lib/litegraph/src/litegraph'
+import { api } from '@/scripts/api'
+import { app } from '@/scripts/app'
+import { useGraphErrorStateStore } from '@/stores/graphErrorStateStore'
+import type { GraphError } from '@/stores/graphErrorStateStore'
+import { useNodeDefStore } from '@/stores/nodeDefStore'
+import { createNodeLocatorId } from '@/types/nodeIdentification'
+import { forEachNode } from '@/utils/graphTraversalUtil'
+
+export function useRequiredConnectionValidator(): void {
+  const errorStore = useGraphErrorStateStore()
+  const nodeDefStore = useNodeDefStore()
+
+  function validate(): void {
+    const rootGraph = app.rootGraph
+    if (!rootGraph) return
+
+    const errors: GraphError[] = []
+
+    forEachNode(rootGraph, (node: LGraphNode) => {
+      const nodeDef = nodeDefStore.nodeDefsByName[node.type ?? '']
+      if (!nodeDef?.input?.required) return
+
+      const subgraphId =
+        node.graph && !node.graph.isRootGraph ? node.graph.id : null
+      const locatorId = subgraphId
+        ? createNodeLocatorId(subgraphId, node.id)
+        : String(node.id)
+
+      for (const inputName of Object.keys(nodeDef.input.required)) {
+        const slot = node.inputs?.find((s) => s.name === inputName)
+
+        const hasConnection = slot?.link !== null && slot?.link !== undefined
+        const hasWidgetValue = hasWidgetValueForInput(node, inputName)
+
+        if (!hasConnection && !hasWidgetValue) {
+          errors.push({
+            key: `frontend:missing:${locatorId}:${inputName}`,
+            source: 'frontend',
+            target: { kind: 'slot', nodeId: locatorId, slotName: inputName },
+            code: 'MISSING_REQUIRED_INPUT'
+          })
+        }
+      }
+    })
+
+    errorStore.execute({ type: 'REPLACE_SOURCE', source: 'frontend', errors })
+  }
+
+  function hasWidgetValueForInput(
+    node: LGraphNode,
+    inputName: string
+  ): boolean {
+    if (!node.widgets) return false
+    const widget = node.widgets.find((w) => w.name === inputName)
+    if (!widget) return false
+    return (
+      widget.value !== undefined && widget.value !== null && widget.value !== ''
+    )
+  }
+
+  const debouncedValidate = useDebounceFn(validate, 200)
+
+  api.addEventListener('graphChanged', debouncedValidate)
+
+  onUnmounted(() => {
+    api.removeEventListener('graphChanged', debouncedValidate)
+  })
+
+  validate()
+}

src/composables/maskeditor/useMaskEditorLoader.ts

@@ -59,7 +59,10 @@ function mkFileUrl(props: { ref: ImageRef; preview?: boolean }): string {
   }
 
   const pathPlusQueryParams = api.apiURL(
-    '/view?' + params.toString() + app.getPreviewFormatParam()
+    '/view?' +
+      params.toString() +
+      app.getPreviewFormatParam() +
+      app.getRandParam()
   )
   const imageElement = new Image()
   imageElement.crossOrigin = 'anonymous'

src/composables/queue/useCompletionSummary.test.ts

@@ -10,7 +10,7 @@ type MockTask = {
   executionEndTimestamp?: number
   previewOutput?: {
     isImage: boolean
-    url: string
+    urlWithTimestamp: string
   }
 }
 
@@ -72,7 +72,7 @@ describe('useCompletionSummary', () => {
     if (previewUrl) {
       task.previewOutput = {
         isImage,
-        url: previewUrl
+        urlWithTimestamp: previewUrl
       }
     }
 

src/composables/queue/useCompletionSummary.ts

@@ -78,7 +78,7 @@ export const useCompletionSummary = () => {
             completedCount++
             const preview = task.previewOutput
             if (preview?.isImage) {
-              imagePreviews.push(preview.url)
+              imagePreviews.push(preview.urlWithTimestamp)
             }
           } else if (state === 'failed') {
             failedCount++

src/composables/useCoreCommands.ts

@@ -862,7 +862,7 @@ export function useCoreCommands(): ComfyCommand[] {
           userEmail: userEmail.value,
           userId: resolvedUserInfo.value?.id
         })
-        window.open(supportUrl, '_blank', 'noopener,noreferrer')
+        window.open(supportUrl, '_blank')
       }
     },
     {

src/composables/useTemplateFiltering.ts

@@ -1,4 +1,4 @@
-import { refDebounced, watchDebounced } from '@vueuse/core'
+import { refThrottled, watchDebounced } from '@vueuse/core'
 import Fuse from 'fuse.js'
 import type { IFuseOptions } from 'fuse.js'
 import { computed, ref, watch } from 'vue'
@@ -119,7 +119,7 @@ export function useTemplateFiltering(
     )
   })
 
-  const debouncedSearchQuery = refDebounced(searchQuery, 150)
+  const debouncedSearchQuery = refThrottled(searchQuery, 50)
 
   const filteredBySearch = computed(() => {
     if (!debouncedSearchQuery.value.trim()) {

src/constants/coreKeybindings.ts

@@ -1,4 +1,4 @@
-import type { Keybinding } from './types'
+import type { Keybinding } from '@/schemas/keyBindingSchema'
 
 export const CORE_KEYBINDINGS: Keybinding[] = [
   {
@@ -76,6 +76,7 @@ export const CORE_KEYBINDINGS: Keybinding[] = [
     },
     commandId: 'Comfy.ShowSettingsDialog'
   },
+  // For '=' both holding shift and not holding shift
   {
     combo: {
       key: '=',
@@ -93,6 +94,7 @@ export const CORE_KEYBINDINGS: Keybinding[] = [
     commandId: 'Comfy.Canvas.ZoomIn',
     targetElementId: 'graph-canvas'
   },
+  // For number pad '+'
   {
     combo: {
       key: '+',

src/core/graph/subgraph/proxyWidgetUtils.ts

@@ -105,7 +105,6 @@ export function addWidgetPromotionOptions(
       content: `Promote Widget: ${widget.label ?? widget.name}`,
       callback: () => {
         promoteWidget(node, widget, promotableParents)
-        widget.callback?.(widget.value)
       }
     })
   else {
@@ -113,7 +112,6 @@ export function addWidgetPromotionOptions(
       content: `Un-Promote Widget: ${widget.label ?? widget.name}`,
       callback: () => {
         demoteWidget(node, widget, parents)
-        widget.callback?.(widget.value)
       }
     })
   }

src/core/graph/widgets/dynamicWidgets.ts

@@ -557,7 +557,7 @@ function withComfyAutogrow(node: LGraphNode): asserts node is AutogrowNode {
       if (!autogrowGroup) return
       if (app.configuringGraph && input.widget)
         ensureWidgetForInput(node, input)
-      if (iscon) {
+      if (iscon && linf) {
         if (swappingConnection || !linf) return
         autogrowInputConnected(slot, this)
       } else {

src/extensions/core/clipspace.ts

@@ -187,7 +187,7 @@ export class ClipspaceDialog extends ComfyDialog {
 
 app.registerExtension({
   name: 'Comfy.Clipspace',
-  init() {
+  init(app) {
     app.openClipspace = function () {
       if (!ClipspaceDialog.instance) {
         ClipspaceDialog.instance = new ClipspaceDialog()

src/extensions/core/groupNode.ts

@@ -292,10 +292,10 @@ export class GroupNodeConfig {
       this.processNode(node, seenInputs, seenOutputs)
     }
 
-    for (const p of this._convertedToProcess) {
+    for (const p of this.#convertedToProcess) {
       p()
     }
-    this._convertedToProcess = []
+    this.#convertedToProcess = []
     if (!this.nodeDef) return
     await app.registerNodeDef(`${PREFIX}${SEPARATOR}` + this.name, this.nodeDef)
     useNodeDefStore().addNodeDef(this.nodeDef)
@@ -773,7 +773,7 @@ export class GroupNodeConfig {
     }
   }
 
-  private _convertedToProcess: (() => void)[] = []
+  #convertedToProcess: (() => void)[] = []
   processNodeInputs(
     node: GroupNodeData,
     seenInputs: Record<string, number>,
@@ -804,7 +804,7 @@ export class GroupNodeConfig {
     )
 
     // Converted inputs have to be processed after all other nodes as they'll be at the end of the list
-    this._convertedToProcess.push(() =>
+    this.#convertedToProcess.push(() =>
       this.processConvertedWidgets(
         inputs,
         node,

src/extensions/core/imageCompare.ts

@@ -1,5 +1,6 @@
 import type { NodeOutputWith } from '@/schemas/apiSchema'
 import { api } from '@/scripts/api'
+import { app } from '@/scripts/app'
 import { useExtensionService } from '@/services/extensionService'
 
 type ImageCompareOutput = NodeOutputWith<{
@@ -22,14 +23,15 @@ useExtensionService().registerExtension({
       onExecuted?.call(this, output)
 
       const { a_images: aImages, b_images: bImages } = output
+      const rand = app.getRandParam()
 
       const beforeUrl =
         aImages && aImages.length > 0
-          ? api.apiURL(`/view?${new URLSearchParams(aImages[0])}`)
+          ? api.apiURL(`/view?${new URLSearchParams(aImages[0])}${rand}`)
           : ''
       const afterUrl =
         bImages && bImages.length > 0
-          ? api.apiURL(`/view?${new URLSearchParams(bImages[0])}`)
+          ? api.apiURL(`/view?${new URLSearchParams(bImages[0])}${rand}`)
           : ''
 
       const widget = node.widgets?.find((w) => w.type === 'imagecompare')

src/extensions/core/index.ts

@@ -13,9 +13,7 @@ import './imageCompare'
 import './imageCrop'
 import './load3d'
 import './maskeditor'
-if (!isCloud) {
-  await import('./nodeTemplates')
-}
+import './nodeTemplates'
 import './noteNode'
 import './previewAny'
 import './rerouteNode'

src/extensions/core/load3d/Load3dUtils.ts

@@ -2,6 +2,7 @@ import type Load3d from '@/extensions/core/load3d/Load3d'
 import { t } from '@/i18n'
 import { useToastStore } from '@/platform/updates/common/toastStore'
 import { api } from '@/scripts/api'
+import { app } from '@/scripts/app'
 
 class Load3dUtils {
   static async generateThumbnailIfNeeded(
@@ -132,7 +133,8 @@ class Load3dUtils {
     const params = [
       'filename=' + encodeURIComponent(filename),
       'type=' + type,
-      'subfolder=' + subfolder
+      'subfolder=' + subfolder,
+      app.getRandParam().substring(1)
     ].join('&')
 
     return `/view?${params}`

src/extensions/core/rerouteNode.ts

@@ -13,7 +13,7 @@ import { getWidgetConfig, mergeIfValid, setWidgetConfig } from './widgetInputs'
 
 app.registerExtension({
   name: 'Comfy.RerouteNode',
-  registerCustomNodes() {
+  registerCustomNodes(app) {
     interface RerouteNode extends LGraphNode {
       __outputType?: string | number
     }