Guard resize observer updates

.github/workflows/ci-perf-report.yaml

@@ -54,14 +54,10 @@ jobs:
       - name: Start ComfyUI server
         uses: ./.github/actions/start-comfyui-server
 
-      # PRs run each test once to keep wall time bounded; main runs 3× so the
-      # baseline saved to perf-data has enough samples to median over noise.
       - name: Run performance tests
         id: perf
         continue-on-error: true
-        env:
-          PERF_REPEAT: ${{ github.event_name == 'push' && github.ref == 'refs/heads/main' && '3' || '2' }}
-        run: pnpm exec playwright test --project=performance --workers=1 --repeat-each=$PERF_REPEAT
+        run: pnpm exec playwright test --project=performance --workers=1 --repeat-each=3
 
       - name: Upload perf metrics
         if: always()

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

@@ -20,8 +20,6 @@ jobs:
       github.event.workflow_run.conclusion == 'success'
     runs-on: ubuntu-latest
     timeout-minutes: 10
-    outputs:
-      has-coverage: ${{ steps.coverage-shards.outputs.has-coverage }}
 
     steps:
       - name: Checkout repository
@@ -39,33 +37,31 @@ jobs:
           path: temp/coverage-shards
           if_no_artifact_found: warn
 
-      - name: Detect shard coverage data
-        id: coverage-shards
-        run: |
-          if [ -d temp/coverage-shards ] && find temp/coverage-shards -name 'coverage.lcov' -type f | grep -q .; then
-            echo "has-coverage=true" >> "$GITHUB_OUTPUT"
-          else
-            echo "has-coverage=false" >> "$GITHUB_OUTPUT"
-            echo "No E2E coverage shard artifacts found; treating this run as skipped." >> "$GITHUB_STEP_SUMMARY"
-          fi
-
       - name: Install lcov
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
         run: sudo apt-get install -y -qq lcov
 
       - name: Merge shard coverage into single LCOV
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
         run: |
           mkdir -p coverage/playwright
           LCOV_FILES=$(find temp/coverage-shards -name 'coverage.lcov' -type f)
+          if [ -z "$LCOV_FILES" ]; then
+            echo "No coverage.lcov files found"
+            touch coverage/playwright/coverage.lcov
+            exit 0
+          fi
           ADD_ARGS=""
           for f in $LCOV_FILES; do ADD_ARGS="$ADD_ARGS -a $f"; done
           lcov $ADD_ARGS -o coverage/playwright/coverage.lcov
           wc -l coverage/playwright/coverage.lcov
 
       - name: Validate merged coverage
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
         run: |
+          SHARD_COUNT=$(find temp/coverage-shards -name 'coverage.lcov' -type f | wc -l | tr -d ' ')
+          if [ "$SHARD_COUNT" -eq 0 ]; then
+            echo "::notice::No shard coverage files; upstream E2E was likely skipped."
+            exit 0
+          fi
+
           MERGED_SF=$(grep -c '^SF:' coverage/playwright/coverage.lcov || echo 0)
           MERGED_LH=$(awk -F: '/^LH:/{s+=$2}END{print s+0}' coverage/playwright/coverage.lcov)
           MERGED_LF=$(awk -F: '/^LF:/{s+=$2}END{print s+0}' coverage/playwright/coverage.lcov)
@@ -86,7 +82,7 @@ jobs:
           done
 
       - name: Upload merged coverage data
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
+        if: always()
         uses: actions/upload-artifact@v6
         with:
           name: e2e-coverage
@@ -95,7 +91,7 @@ jobs:
           if-no-files-found: warn
 
       - name: Upload E2E coverage to Codecov
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
+        if: always()
         uses: codecov/codecov-action@1af58845a975a7985b0beb0cbe6fbbb71a41dbad # v5.5.3
         with:
           files: coverage/playwright/coverage.lcov
@@ -104,7 +100,6 @@ jobs:
           fail_ci_if_error: false
 
       - name: Generate HTML coverage report
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
         run: |
           if [ ! -s coverage/playwright/coverage.lcov ]; then
             echo "No coverage data; generating placeholder report."
@@ -119,7 +114,6 @@ jobs:
             --precision 1
 
       - name: Upload HTML report artifact
-        if: steps.coverage-shards.outputs.has-coverage == 'true'
         uses: actions/upload-artifact@v6
         with:
           name: e2e-coverage-html
@@ -128,9 +122,7 @@ jobs:
 
   deploy:
     needs: merge
-    if: >
-      github.event.workflow_run.head_branch == 'main' &&
-      needs.merge.outputs.has-coverage == 'true'
+    if: github.event.workflow_run.head_branch == 'main'
     runs-on: ubuntu-latest
     permissions:
       pages: write

.github/workflows/ci-tests-extension-api.yaml

@@ -1,88 +0,0 @@
-# Description: Extension API test suite (I-TF) + compat-floor gate (I-TF.7)
-#
-# Runs on any PR touching extension-api declaration files, extension-api-v2
-# implementation/tests, or the touch-point DB/rollup (blast-radius changes).
-#
-# Two jobs:
-#   test          — vitest run against src/extension-api-v2/__tests__/
-#   compat-floor  — python scripts/check-compat-floor.py (exits 1 if any
-#                   blast_radius ≥ 2.0 category is missing a stub triple)
-#
-# The compat-floor job is the CI enforcement of PLAN.md §Compat-floor:
-#   "Every blast_radius ≥ 2.0 pattern MUST pass v1 + v2 + migration before v2 ships."
-name: 'CI: Tests Extension API'
-
-on:
-  push:
-    branches: [main, master, dev*, core/*, extension-v2*]
-    paths:
-      - 'src/extension-api/**'
-      - 'src/extension-api-v2/**'
-      - 'packages/extension-api/**'
-      - 'vitest.extension-api.config.mts'
-      - 'research/touch-points/rollup.yaml'
-      - 'research/touch-points/behavior-categories.yaml'
-      - 'scripts/check-compat-floor.py'
-      - 'pnpm-lock.yaml'
-  pull_request:
-    branches-ignore: [wip/*, draft/*, temp/*]
-    paths:
-      - 'src/extension-api/**'
-      - 'src/extension-api-v2/**'
-      - 'packages/extension-api/**'
-      - 'vitest.extension-api.config.mts'
-      - 'research/touch-points/rollup.yaml'
-      - 'research/touch-points/behavior-categories.yaml'
-      - 'scripts/check-compat-floor.py'
-      - 'pnpm-lock.yaml'
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
-jobs:
-  test:
-    name: Extension API tests (vitest)
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Setup frontend
-        uses: ./.github/actions/setup-frontend
-
-      - name: Run extension-api test suite
-        run: pnpm test:extension-api
-
-      - name: Run with coverage (push only)
-        if: github.event_name == 'push'
-        run: pnpm test:extension-api:coverage
-
-      - name: Upload coverage to Codecov
-        if: github.event_name == 'push'
-        uses: codecov/codecov-action@1af58845a975a7985b0beb0cbe6fbbb71a41dbad # v5.5.3
-        with:
-          files: coverage/lcov.info
-          flags: extension-api
-          token: ${{ secrets.CODECOV_TOKEN }}
-          fail_ci_if_error: false
-
-  compat-floor:
-    name: Compat-floor gate (blast_radius ≥ 2.0)
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Set up Python
-        uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
-        with:
-          python-version: '3.11'
-
-      - name: Install PyYAML
-        run: pip install pyyaml
-
-      - name: Check compat floor
-        run: python3 scripts/check-compat-floor.py
-        # Exits 1 if any blast_radius ≥ 2.0 behavior category is missing
-        # any of its three stub files (v1/v2/migration). Enforces PLAN.md §Compat-floor.

.github/workflows/ci-vercel-website-preview.yaml

@@ -4,9 +4,6 @@ name: 'CI: Vercel Website Preview'
 on:
   pull_request:
     types: [opened, synchronize, reopened]
-    branches-ignore:
-      - 'core/**'
-      - 'cloud/**'
     paths:
       - 'apps/website/**'
       - 'packages/design-system/**'

.github/workflows/extension-api-publish.yml

@@ -1,97 +0,0 @@
-# Description: Publish @comfyorg/extension-api to npm with provenance attestation.
-#
-# Triggered by a tag push matching 'extension-api-v*' (e.g. extension-api-v0.1.0).
-# Also supports workflow_dispatch for a manual dry-run (set dry_run: true).
-#
-# Prerequisites (one-time human setup):
-#   - NPM_TOKEN secret must be set in the repo/org settings with publish
-#     access to the @comfyorg scope on npmjs.com.
-#   - The @comfyorg npm scope already exists (used by @comfyorg/comfyui-frontend).
-#
-# PKG4.D4 (MIG1 / Phase A — surface-only shim)
-name: 'Extension API: Publish'
-
-on:
-  push:
-    tags:
-      - 'extension-api-v*'
-  workflow_dispatch:
-    inputs:
-      dry_run:
-        description: 'Dry run — build and verify without publishing'
-        required: false
-        default: 'true'
-        type: boolean
-
-permissions:
-  contents: write   # needed to create GitHub Release
-  id-token: write   # needed for npm provenance via OIDC
-
-jobs:
-  publish:
-    name: Publish @comfyorg/extension-api
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v6
-        with:
-          fetch-depth: 0  # full history for release notes
-
-      - name: Setup frontend
-        uses: ./.github/actions/setup-frontend
-
-      - name: Setup npm registry
-        uses: actions/setup-node@v4
-        with:
-          registry-url: 'https://registry.npmjs.org/'
-
-      - name: Build package
-        run: pnpm --filter @comfyorg/extension-api build
-
-      - name: Typecheck package
-        run: pnpm --filter @comfyorg/extension-api typecheck
-
-      - name: Verify package version matches tag
-        if: github.event_name == 'push'
-        run: |
-          TAG="${GITHUB_REF_NAME}"                           # e.g. extension-api-v0.1.0
-          PKG_VERSION=$(node -p "require('./packages/extension-api/package.json').version")
-          TAG_VERSION="${TAG#extension-api-v}"               # strip prefix → 0.1.0
-          if [ "$PKG_VERSION" != "$TAG_VERSION" ]; then
-            echo "::error::Tag '$TAG' implies version '$TAG_VERSION' but packages/extension-api/package.json has '$PKG_VERSION'. Update the package.json before tagging."
-            exit 1
-          fi
-          echo "Version check passed: $PKG_VERSION"
-
-      - name: Publish to npm (with provenance)
-        if: github.event_name == 'push' || inputs.dry_run == 'false'
-        run: |
-          cd packages/extension-api
-          npm publish --provenance --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-
-      - name: Dry-run report
-        if: inputs.dry_run == 'true'
-        run: |
-          echo "=== DRY RUN — would publish ==="
-          cd packages/extension-api
-          npm pack --dry-run
-          echo "=== End dry run ==="
-
-      - name: Create GitHub Release
-        if: github.event_name == 'push'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const tag = context.ref.replace('refs/tags/', '')
-            const { data: release } = await github.rest.repos.createRelease({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              tag_name: tag,
-              name: tag,
-              generate_release_notes: true,
-              draft: false,
-              prerelease: context.ref.includes('-alpha') || context.ref.includes('-beta') || context.ref.includes('-rc')
-            })
-            console.log(`Release created: ${release.html_url}`)

.github/workflows/extension-api-typecheck.yml

@@ -1,65 +0,0 @@
-# Description: Typecheck and build the @comfyorg/extension-api package.
-# Runs on PRs and pushes touching the public type surface, the core .v2.ts
-# implementations, or the package scaffold — so regressions in the published
-# contract are caught before merge.
-#
-# PKG4.D3 (MIG1 / Phase A — surface-only shim)
-name: 'Extension API: Typecheck'
-
-on:
-  push:
-    branches: [main, master, dev*, core/*, extension-v2*]
-    paths:
-      - 'src/extension-api/**'
-      - 'src/extensions/core/*.v2.ts'
-      - 'src/services/extension-api-service.ts'
-      - 'packages/extension-api/**'
-      - '.github/workflows/extension-api-*.yml'
-      - 'pnpm-lock.yaml'
-      - 'pnpm-workspace.yaml'
-  pull_request:
-    branches-ignore: [wip/*, draft/*, temp/*]
-    paths:
-      - 'src/extension-api/**'
-      - 'src/extensions/core/*.v2.ts'
-      - 'src/services/extension-api-service.ts'
-      - 'packages/extension-api/**'
-      - '.github/workflows/extension-api-*.yml'
-      - 'pnpm-lock.yaml'
-      - 'pnpm-workspace.yaml'
-  merge_group:
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
-jobs:
-  typecheck:
-    name: Build + typecheck @comfyorg/extension-api
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Setup frontend
-        uses: ./.github/actions/setup-frontend
-
-      - name: Build package (emit declarations)
-        run: pnpm --filter @comfyorg/extension-api build
-
-      - name: Typecheck package
-        run: pnpm --filter @comfyorg/extension-api typecheck
-
-      - name: Smoke-test consumer (tsc --noEmit on minimal extension)
-        # Verifies the published types are consumable from an external module
-        # that imports from '@comfyorg/extension-api'. Uses a minimal fixture
-        # checked in to packages/extension-api/test/smoke/.
-        run: |
-          cd packages/extension-api
-          if [ -d test/smoke ]; then
-            pnpm exec tsc --noEmit --project test/smoke/tsconfig.json
-          else
-            echo "No smoke test found — skipping (add packages/extension-api/test/smoke/ to enable)"
-          fi

AGENTS.md

@@ -172,7 +172,7 @@ This project uses **pnpm**. Always prefer scripts defined in `package.json` (e.g
 16. Whenever a new piece of code is written, the author should ask themselves 'is there a simpler way to introduce the same functionality?'. If the answer is yes, the simpler course should be chosen
 17. [Refactoring](https://refactoring.com/catalog/) should be used to make complex code simpler
 18. Try to minimize the surface area (exported values) of each module and composable
-19. Don't use barrel files, e.g. `/some/package/index.ts` to re-export within `/src`. **Exception**: `src/extension-api/index.ts` is the published npm package entry point (`@comfyorg/extension-api`) and is explicitly exempt from this rule.
+19. Don't use barrel files, e.g. `/some/package/index.ts` to re-export within `/src`
 20. Keep functions short and functional
 21. Minimize [nesting](https://wiki.c2.com/?ArrowAntiPattern), e.g. `if () { ... }` or `for () { ... }`
 22. Avoid mutable state, prefer immutability and assignment at point of declaration

apps/website/e2e/customers.spec.ts

@@ -1,33 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { test } from './fixtures/blockExternalMedia'
-
-test.describe('Customers @smoke', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto('/customers')
-  })
-
-  test('hero image declares intrinsic dimensions so layout reserves space before load', async ({
-    page
-  }) => {
-    const heroImage = page.locator('img[alt="Comfy 3D logo"]')
-    await expect(heroImage).toBeVisible()
-    await expect(heroImage).toHaveAttribute('width', /^\d+$/)
-    await expect(heroImage).toHaveAttribute('height', /^\d+$/)
-
-    // Regression guard: an unloaded <img> without intrinsic dimensions
-    // collapses to ~0px, then jumps to its natural size on load and pushes
-    // the video below it. Reserved space must persist before bytes arrive.
-    const heightWhileUnloaded = await page.evaluate(() => {
-      const img = document.querySelector<HTMLImageElement>(
-        'img[alt="Comfy 3D logo"]'
-      )
-      if (!img) return null
-      img.removeAttribute('src')
-      return img.getBoundingClientRect().height
-    })
-
-    expect(heightWhileUnloaded).not.toBeNull()
-    expect(heightWhileUnloaded!).toBeGreaterThan(100)
-  })
-})

apps/website/e2e/payment.spec.ts

@@ -61,7 +61,7 @@ test.describe('Payment failed page @smoke', () => {
   test('shows failure heading and subtitle', async ({ page }) => {
     await expect(
       page.getByRole('heading', {
-        name: /Unable to complete payment/i,
+        name: /Payment was not completed/i,
         level: 1
       })
     ).toBeVisible()
@@ -102,7 +102,7 @@ test.describe('Payment pages zh-CN @smoke', () => {
     await expect(page).toHaveTitle('支付失败 — Comfy')
     await expectNoIndex(page)
     await expect(
-      page.getByRole('heading', { name: '无法完成支付', level: 1 })
+      page.getByRole('heading', { name: '支付未完成', level: 1 })
     ).toBeVisible()
     await expect(page.getByRole('link', { name: '联系支持' })).toHaveAttribute(
       'href',

apps/website/e2e/visual-responsive.spec.ts

@@ -26,8 +26,8 @@ async function assertNoOverflow(page: Page) {
 }
 
 async function navigateAndSettle(page: Page, url: string) {
-  await page.goto(url, { waitUntil: 'domcontentloaded' })
-  await page.waitForLoadState('load')
+  await page.goto(url)
+  await page.waitForLoadState('networkidle')
 }
 
 test.describe('Home', { tag: '@visual' }, () => {
@@ -126,7 +126,6 @@ test.describe('Overflow guards', { tag: '@visual' }, () => {
   const pages = [
     '/',
     '/cloud',
-    '/cloud/enterprise',
     '/cloud/pricing',
     '/contact',
     '/download',

apps/website/package.json

@@ -27,7 +27,6 @@
     "gsap": "catalog:",
     "lenis": "catalog:",
     "posthog-js": "catalog:",
-    "three": "catalog:",
     "vue": "catalog:",
     "zod": "catalog:"
   },

apps/website/playwright.config.ts

@@ -28,7 +28,7 @@ export default defineConfig({
     ? [['html'], ['json', { outputFile: 'results.json' }]]
     : 'html',
   expect: {
-    toHaveScreenshot: { maxDiffPixels: 100 }
+    toHaveScreenshot: { maxDiffPixels: 50 }
   },
   ...maybeLocalOptions,
   webServer: {

apps/website/src/components/careers/TeamPhotosSection.vue

@@ -1,4 +1,6 @@
 <script setup lang="ts">
+import { onMounted, ref } from 'vue'
+
 const photos = [
   {
     src: 'https://media.comfy.org/website/careers/team0.webp',
@@ -15,34 +17,45 @@ const photos = [
   {
     src: 'https://media.comfy.org/website/careers/team3.webp',
     alt: 'Team on a boat'
-  },
-  {
-    src: 'https://media.comfy.org/website/careers/team4.webp',
-    alt: 'Teammates posing at a restaurant'
-  },
-  {
-    src: 'https://media.comfy.org/website/careers/team5.webp',
-    alt: 'Teammates at a social gathering'
-  },
-  {
-    src: 'https://media.comfy.org/website/careers/team6.webp',
-    alt: 'Team sailing at golden hour'
-  },
-  {
-    src: 'https://media.comfy.org/website/careers/team7.webp',
-    alt: 'Team on a sailboat at sunset'
   }
 ]
+
+const loopedPhotos = [...photos, ...photos, ...photos]
+
+const scrollRef = ref<HTMLElement>()
+
+function onScroll() {
+  const el = scrollRef.value
+  if (!el) return
+
+  const third = el.scrollWidth / 3
+  const maxScroll = el.scrollWidth - el.clientWidth
+
+  if (el.scrollLeft >= maxScroll - 1) {
+    el.scrollLeft -= third
+  } else if (el.scrollLeft <= 1) {
+    el.scrollLeft += third
+  }
+}
+
+onMounted(() => {
+  const el = scrollRef.value
+  if (el) {
+    el.scrollLeft = el.scrollWidth / 3
+  }
+})
 </script>
 
 <template>
   <section class="py-12 md:py-24">
     <div
+      ref="scrollRef"
       class="flex gap-4 overflow-x-auto px-6 md:gap-6 md:px-20"
       style="scrollbar-width: none"
+      @scroll="onScroll"
     >
       <div
-        v-for="(photo, i) in photos"
+        v-for="(photo, i) in loopedPhotos"
         :key="i"
         class="aspect-3/4 h-64 shrink-0 md:h-96"
       >

apps/website/src/components/common/GitHubStarBadge.vue

@@ -13,7 +13,7 @@ const { stars } = defineProps<{
     target="_blank"
     rel="noopener noreferrer"
     :aria-label="`ComfyUI on GitHub — ${stars} stars`"
-    class="hidden shrink-0 items-center gap-1 lg:flex"
+    class="hidden shrink-0 items-center gap-2 lg:flex"
   >
     <NodeBadge
       :segments="[{ text: stars }]"
@@ -22,7 +22,7 @@ const { stars } = defineProps<{
       size-class="h-5 sm:h-5"
     />
     <span
-      class="bg-primary-comfy-yellow block size-6 shrink-0"
+      class="bg-primary-comfy-yellow block size-7"
       aria-hidden="true"
       style="mask: url('/icons/social/github.svg') center / contain no-repeat"
     />

apps/website/src/components/customers/FeedbackSection.vue

@@ -75,7 +75,7 @@ const progressPercent = computed(() => `${progress.value * 100}%`)
       <!-- Progress bar -->
       <div class="h-1 flex-1 rounded-full bg-white/20">
         <div
-          class="bg-primary-comfy-yellow h-full rounded-full"
+          class="bg-primary-comfy-yellow h-full rounded-full transition-all duration-200"
           :style="{ width: progressPercent }"
         />
       </div>

apps/website/src/components/customers/HeroSection.vue

@@ -5,7 +5,6 @@ import { useHeroAnimation } from '../../composables/useHeroAnimation'
 import SectionLabel from '../common/SectionLabel.vue'
 import type { Locale } from '../../i18n/translations'
 import { t } from '../../i18n/translations'
-import { ScrollTrigger } from '../../scripts/gsapSetup'
 import VideoPlayer from '../common/VideoPlayer.vue'
 
 const { locale = 'en' } = defineProps<{ locale?: Locale }>()
@@ -23,10 +22,6 @@ useHeroAnimation({
   logo: logoRef,
   video: videoRef
 })
-
-function handleLogoLoad() {
-  ScrollTrigger.refresh(true)
-}
 </script>
 
 <template>
@@ -42,10 +37,7 @@ function handleLogoLoad() {
         <img
           src="https://media.comfy.org/website/customers/c-projection.webp"
           alt="Comfy 3D logo"
-          width="1568"
-          height="1763"
-          class="mx-auto h-auto w-full max-w-md lg:max-w-none"
-          @load="handleLogoLoad"
+          class="mx-auto w-full max-w-md lg:max-w-none"
         />
       </div>
 

apps/website/src/components/home/HeroSection.vue

@@ -1,31 +1,24 @@
 <script setup lang="ts">
-import { ref } from 'vue'
-
 import type { Locale } from '../../i18n/translations'
 import { externalLinks } from '../../config/routes'
-import { useHeroLogo } from '../../composables/useHeroLogo'
 import { t } from '../../i18n/translations'
 import BrandButton from '../common/BrandButton.vue'
 
 const { locale = 'en' } = defineProps<{ locale?: Locale }>()
-
-const logoContainer = ref<HTMLElement>()
-const { loaded: logoLoaded } = useHeroLogo(logoContainer)
 </script>
 
 <template>
   <section
     class="relative flex min-h-auto flex-col lg:flex-row lg:items-center"
   >
-    <div
-      ref="logoContainer"
-      class="relative flex aspect-square w-full flex-1 items-center justify-center"
-    >
-      <img
-        v-show="!logoLoaded"
-        src="https://media.comfy.org/website/homepage/hero-logo-seq/Logo00.webp"
-        alt="Comfy logo"
-        class="w-3/5"
+    <div class="relative flex-1">
+      <video
+        src="https://media.comfy.org/website/homepage/hero-logo-seq.webm"
+        autoplay
+        loop
+        muted
+        playsinline
+        class="w-full"
       />
     </div>
 

apps/website/src/components/home/UseCaseSection.vue

@@ -25,7 +25,7 @@ const categories: Category[] = [
   {
     label: t('useCase.vfx', locale),
     leftSrc: 'https://media.comfy.org/website/homepage/use-case/left1.webm',
-    rightSrc: 'https://media.comfy.org/website/homepage/use-case/right1.webm'
+    rightSrc: 'https://media.comfy.org/website/homepage/use-case/right1.webp'
   },
   {
     label: t('useCase.advertising', locale),

apps/website/src/components/pricing/PriceSection.vue

@@ -77,10 +77,7 @@ const plans: PricingPlan[] = [
     ctaKey: 'pricing.plan.creator.cta',
     ctaHref: subscribeUrl('creator'),
     featureIntroKey: 'pricing.plan.creator.featureIntro',
-    features: [
-      { text: 'pricing.plan.creator.feature1' },
-      { text: 'pricing.plan.creator.feature2' }
-    ],
+    features: [{ text: 'pricing.plan.creator.feature1' }],
     isPopular: true
   },
   {
@@ -93,10 +90,7 @@ const plans: PricingPlan[] = [
     ctaKey: 'pricing.plan.pro.cta',
     ctaHref: subscribeUrl('pro'),
     featureIntroKey: 'pricing.plan.pro.featureIntro',
-    features: [
-      { text: 'pricing.plan.pro.feature1' },
-      { text: 'pricing.plan.pro.feature2' }
-    ]
+    features: [{ text: 'pricing.plan.pro.feature1' }]
   },
   {
     id: 'enterprise',

apps/website/src/components/product/enterprise/HeroSection.vue

@@ -35,20 +35,20 @@ onMounted(() => {
 
 <template>
   <section
-    class="max-w-9xl relative mx-auto flex flex-col items-center overflow-hidden pt-16 lg:flex-row-reverse lg:items-center lg:overflow-x-visible lg:overflow-y-clip lg:pt-[min(8vw,10rem)] lg:pb-[min(8vw,10rem)]"
+    class="max-w-9xl relative mx-auto flex flex-col items-center overflow-hidden lg:flex-row-reverse lg:items-center lg:overflow-x-visible lg:overflow-y-clip lg:pb-[min(8vw,10rem)]"
   >
     <!-- Illustration (overlaps text slightly; stacks above on mobile, right on lg) -->
     <div
-      class="aspect-square w-4/5 max-w-md scale-150 self-center md:max-w-2xl lg:pointer-events-none lg:z-1 lg:-ml-12 lg:-translate-x-[10%] lg:self-center xl:size-[clamp(32rem,max(40vh,32vw),36rem)] xl:min-h-[min(32vw,24rem)] xl:min-w-[min(24vw,20rem)]"
+      class="aspect-square w-4/5 max-w-md scale-150 self-center md:max-w-2xl lg:pointer-events-none lg:z-1 lg:-ml-12 lg:-translate-x-[10%] lg:translate-y-[40px] lg:self-center xl:size-[clamp(32rem,max(40vh,32vw),36rem)] xl:min-h-[min(32vw,24rem)] xl:min-w-[min(24vw,20rem)]"
     >
       <svg
         ref="svgRef"
-        class="block size-full overflow-visible"
+        class="block size-full"
         viewBox="0 0 1600 1046"
         fill="none"
         aria-hidden="true"
       >
-        <g>
+        <g clip-path="url(#enterpriseHeroClip)">
           <rect width="1600" height="1046" fill="#211927" />
           <rect
             width="800"
@@ -84,7 +84,7 @@ onMounted(() => {
           />
 
           <!-- Exploding block cluster -->
-          <g class="block-cluster" clip-path="url(#enterpriseHeroBlockClip)">
+          <g class="block-cluster">
             <path
               class="block-piece"
               d="M1018.44 635.715L1018.45 581.73C1018.46 574.554 1013.42 565.829 1007.21 562.242L960.479 535.262C956.544 532.99 949.469 533.096 945.535 535.368L898.79 562.373C892.576 565.963 887.537 574.691 887.535 581.867L887.52 635.852C887.519 640.395 890.967 646.574 894.902 648.845L941.632 675.825C947.845 679.412 957.918 679.409 964.132 675.819L1010.88 648.815C1014.82 646.538 1018.44 640.267 1018.44 635.715Z"
@@ -353,7 +353,7 @@ onMounted(() => {
             <stop stop-color="#211927" stop-opacity="0" />
             <stop offset="1" stop-color="#211927" />
           </linearGradient>
-          <clipPath id="enterpriseHeroBlockClip">
+          <clipPath id="enterpriseHeroClip">
             <rect width="1600" height="1046" fill="white" />
           </clipPath>
         </defs>

apps/website/src/components/product/shared/CloudBannerSection.vue

@@ -8,9 +8,7 @@ const { locale = 'en' } = defineProps<{ locale?: Locale }>()
 </script>
 
 <template>
-  <section
-    class="bg-transparency-white-t4 relative z-20 p-4 text-center lg:px-20 lg:py-8"
-  >
+  <section class="bg-transparency-white-t4 p-4 text-center lg:px-20 lg:py-8">
     <p
       class="text-primary-comfy-canvas relative z-10 text-lg font-semibold lg:text-sm lg:font-normal"
     >

apps/website/src/composables/useHeroLogo.ts

@@ -1,328 +0,0 @@
-import type { Ref } from 'vue'
-import { onMounted, onUnmounted, ref } from 'vue'
-
-import * as THREE from 'three'
-import { SVGLoader } from 'three/addons/loaders/SVGLoader.js'
-
-import { prefersReducedMotion } from './useReducedMotion'
-
-const IMAGE_COUNT = 16
-const BASE_URL = 'https://media.comfy.org/website/homepage/hero-logo-seq'
-
-const SVG_MARKUP = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 375 404"><path fill="#000000" d="M296.597 302.576C297.299 300.205 297.682 297.705 297.682 295.078C297.682 280.529 285.938 268.736 271.45 268.736H153.883C147.564 268.8 142.395 263.673 142.395 257.328C142.395 256.174 142.586 255.084 142.841 254.059L174.499 143.309C175.839 138.438 180.307 134.849 185.541 134.849L303.554 134.72C328.446 134.72 349.444 117.864 355.763 94.8555L373.506 33.1353C374.081 30.9562 374.4 28.5848 374.4 26.2134C374.4 11.7288 362.72 0 348.295 0H205.518C180.754 0 159.819 16.7279 153.373 39.4804L141.373 81.5886C139.969 86.3954 135.565 89.9205 130.332 89.9205H96.0573C71.4845 89.9205 50.7412 106.328 44.1034 128.824L0.957382 280.144C0.319127 282.387 0 284.823 0 287.258C0 301.807 11.7439 313.6 26.2323 313.6H59.9321C66.2508 313.6 71.4207 318.727 71.4207 325.137C71.4207 326.226 71.293 327.316 70.9739 328.341L59.0385 370.065C58.4641 372.308 58.0811 374.615 58.0811 376.987C58.0811 391.471 69.7612 403.2 84.1857 403.2L227.027 403.072C251.855 403.072 272.79 386.28 279.172 363.399L296.533 302.64L296.597 302.576Z"/></svg>`
-
-interface HeroLogoConfig {
-  speed: number
-  tiltX: number
-  tiltZ: number
-  zoom: number
-  fov: number
-  logoColor: string
-  extrudeDepth: number
-  cursorTiltStrength: number
-  bgScale: number
-  slideDuration: number
-}
-
-const DEFAULTS: HeroLogoConfig = {
-  speed: 1,
-  tiltX: -0.1,
-  tiltZ: -0.1,
-  zoom: 7,
-  fov: 50,
-  logoColor: '#F2FF59',
-  extrudeDepth: 200,
-  cursorTiltStrength: 0.5,
-  bgScale: 0.8,
-  slideDuration: 0.4
-}
-
-function buildImageUrls(): string[] {
-  return Array.from({ length: IMAGE_COUNT }, (_, i) => {
-    const index = String(i).padStart(5, '0')
-    return `${BASE_URL}/image_sequence_${index}.webp`
-  })
-}
-
-function parseShapes(): THREE.Shape[] {
-  const loader = new SVGLoader()
-  const svgData = loader.parse(SVG_MARKUP)
-  const shapes: THREE.Shape[] = []
-  svgData.paths.forEach((path) => {
-    shapes.push(...SVGLoader.createShapes(path))
-  })
-  return shapes
-}
-
-function loadTextures(urls: string[]): Promise<THREE.Texture[]> {
-  return Promise.all(
-    urls.map(
-      (url) =>
-        new Promise<THREE.Texture | null>((resolve) => {
-          const img = new Image()
-          img.crossOrigin = 'anonymous'
-          img.onload = () => {
-            const tex = new THREE.Texture(img)
-            tex.needsUpdate = true
-            tex.colorSpace = THREE.SRGBColorSpace
-            resolve(tex)
-          }
-          img.onerror = () => resolve(null)
-          img.src = url
-        })
-    )
-  ).then((results) => results.filter((t): t is THREE.Texture => t !== null))
-}
-
-export function useHeroLogo(
-  containerRef: Ref<HTMLElement | undefined>,
-  config: Partial<HeroLogoConfig> = {}
-) {
-  const cfg = { ...DEFAULTS, ...config }
-  const loaded = ref(false)
-  let cleanup: (() => void) | undefined
-
-  onMounted(async () => {
-    try {
-      const container = containerRef.value
-      if (!container || prefersReducedMotion()) return
-
-      const { width, height } = container.getBoundingClientRect()
-
-      const renderer = new THREE.WebGLRenderer({
-        antialias: true,
-        stencil: true,
-        alpha: true
-      })
-      renderer.setSize(width, height)
-      renderer.setPixelRatio(Math.min(window.devicePixelRatio, 2))
-      renderer.outputColorSpace = THREE.SRGBColorSpace
-      renderer.domElement.style.position = 'absolute'
-      renderer.domElement.style.inset = '0'
-      renderer.domElement.style.width = '100%'
-      renderer.domElement.style.height = '100%'
-      renderer.domElement.style.opacity = '0'
-      renderer.domElement.setAttribute('aria-hidden', 'true')
-      container.appendChild(renderer.domElement)
-
-      let disposed = false
-      const teardowns: Array<() => void> = []
-      cleanup = () => {
-        disposed = true
-        teardowns.forEach((fn) => fn())
-      }
-      teardowns.push(() => {
-        renderer.dispose()
-        renderer.domElement.remove()
-      })
-
-      const scene = new THREE.Scene()
-      const camera = new THREE.PerspectiveCamera(
-        cfg.fov,
-        width / height,
-        0.1,
-        1000
-      )
-      camera.position.z = cfg.zoom
-
-      // SVG shape
-      const shapes = parseShapes()
-      const tempGeo = new THREE.ShapeGeometry(shapes)
-      tempGeo.computeBoundingBox()
-      const bb = tempGeo.boundingBox!
-      const cx = (bb.max.x + bb.min.x) / 2
-      const cy = (bb.max.y + bb.min.y) / 2
-      const scaleFactor = 3 / (bb.max.y - bb.min.y)
-      tempGeo.dispose()
-
-      // Image sequence textures — load first frame eagerly, rest lazily
-      const urls = buildImageUrls()
-      const textures = await loadTextures(urls.slice(0, 1))
-      if (disposed) return
-
-      renderer.domElement.style.opacity = '1'
-      loaded.value = true
-
-      loadTextures(urls.slice(1)).then((rest) => {
-        if (!disposed) textures.push(...rest)
-      })
-
-      // Background plane (stencil read)
-      const bgPlaneGeo = new THREE.PlaneGeometry(14, 14)
-      const bgPlaneMat = new THREE.MeshBasicMaterial({
-        transparent: true,
-        opacity: 1,
-        map: textures[0] ?? null,
-        depthTest: false,
-        depthWrite: false,
-        stencilWrite: true,
-        stencilFunc: THREE.EqualStencilFunc,
-        stencilRef: 1,
-        stencilFail: THREE.KeepStencilOp,
-        stencilZFail: THREE.KeepStencilOp,
-        stencilZPass: THREE.KeepStencilOp
-      })
-      const bgPlane = new THREE.Mesh(bgPlaneGeo, bgPlaneMat)
-      bgPlane.renderOrder = 1
-      bgPlane.scale.set(cfg.bgScale, cfg.bgScale, 1)
-      scene.add(bgPlane)
-
-      // Logo group
-      const group = new THREE.Group()
-      scene.add(group)
-
-      const s = scaleFactor
-      const depth = cfg.extrudeDepth
-
-      // Front face
-      const shapeGeo = new THREE.ShapeGeometry(shapes)
-      shapeGeo.translate(-cx, -cy, 0)
-      shapeGeo.scale(s, -s, s)
-      const shapeMat = new THREE.MeshBasicMaterial({
-        color: cfg.logoColor,
-        side: THREE.DoubleSide,
-        depthTest: false,
-        depthWrite: false,
-        transparent: true
-      })
-      const logoMesh = new THREE.Mesh(shapeGeo, shapeMat)
-      logoMesh.renderOrder = 2
-      group.add(logoMesh)
-
-      // Extrusion stencil mask
-      const extrudeGeo = new THREE.ExtrudeGeometry(shapes, {
-        depth,
-        bevelEnabled: false
-      })
-      extrudeGeo.translate(-cx, -cy, -depth)
-      extrudeGeo.scale(s, -s, s)
-      const extrudeMat = new THREE.MeshBasicMaterial({
-        colorWrite: false,
-        depthWrite: true,
-        depthTest: true,
-        stencilWrite: true,
-        stencilRef: 1,
-        stencilFunc: THREE.AlwaysStencilFunc,
-        stencilZPass: THREE.ReplaceStencilOp,
-        stencilFail: THREE.KeepStencilOp,
-        stencilZFail: THREE.KeepStencilOp,
-        side: THREE.DoubleSide
-      })
-      const extrudeMesh = new THREE.Mesh(extrudeGeo, extrudeMat)
-      extrudeMesh.renderOrder = 0
-      group.add(extrudeMesh)
-
-      // Interaction
-      let isDragging = false
-      let previousX = 0
-      let dragVelocity = 0
-      let currentTiltX = 0
-      let currentTiltY = 0
-      let pointerX = 0
-      let pointerY = 0
-      let rotationT = 0
-      let currentSlide = 0
-      let slideTimer = 0
-      let animationId = 0
-
-      function onMouseMove(e: MouseEvent) {
-        pointerX = (e.clientX / window.innerWidth) * 2 - 1
-        pointerY = (e.clientY / window.innerHeight) * 2 - 1
-      }
-
-      function onPointerDown(e: PointerEvent) {
-        isDragging = true
-        dragVelocity = 0
-        previousX = e.clientX
-      }
-
-      function onPointerMove(e: PointerEvent) {
-        if (!isDragging) return
-        dragVelocity = (e.clientX - previousX) * 0.005
-        rotationT += dragVelocity
-        previousX = e.clientX
-      }
-
-      function onPointerUp() {
-        isDragging = false
-      }
-
-      function onResize() {
-        const rect = container!.getBoundingClientRect()
-        camera.aspect = rect.width / rect.height
-        camera.updateProjectionMatrix()
-        renderer.setSize(rect.width, rect.height)
-      }
-
-      window.addEventListener('mousemove', onMouseMove)
-      renderer.domElement.addEventListener('pointerdown', onPointerDown)
-      window.addEventListener('pointermove', onPointerMove)
-      window.addEventListener('pointerup', onPointerUp)
-      window.addEventListener('resize', onResize)
-
-      const clock = new THREE.Clock()
-
-      function animate() {
-        if (disposed) return
-        animationId = requestAnimationFrame(animate)
-        const dt = clock.getDelta()
-
-        if (!isDragging && Math.abs(dragVelocity) > 0.0001) {
-          dragVelocity *= 0.95
-          rotationT += dragVelocity
-        } else if (!isDragging) {
-          dragVelocity = 0
-        }
-
-        rotationT += cfg.speed * dt
-
-        currentTiltX += (pointerY - currentTiltX) * 0.08
-        currentTiltY += (pointerX - currentTiltY) * 0.08
-
-        group.rotation.y = rotationT % (Math.PI * 2)
-        group.rotation.x = cfg.tiltX - currentTiltX * cfg.cursorTiltStrength
-        group.rotation.z = cfg.tiltZ
-
-        if (textures.length > 1) {
-          slideTimer += dt
-          if (slideTimer >= cfg.slideDuration) {
-            slideTimer = 0
-            currentSlide = (currentSlide + 1) % textures.length
-            bgPlaneMat.map = textures[currentSlide]
-            bgPlaneMat.needsUpdate = true
-          }
-        }
-
-        renderer.render(scene, camera)
-      }
-
-      animate()
-
-      teardowns.push(
-        () => cancelAnimationFrame(animationId),
-        () => window.removeEventListener('mousemove', onMouseMove),
-        () =>
-          renderer.domElement.removeEventListener('pointerdown', onPointerDown),
-        () => window.removeEventListener('pointermove', onPointerMove),
-        () => window.removeEventListener('pointerup', onPointerUp),
-        () => window.removeEventListener('resize', onResize),
-        () => bgPlaneGeo.dispose(),
-        () => bgPlaneMat.dispose(),
-        () => shapeGeo.dispose(),
-        () => shapeMat.dispose(),
-        () => extrudeGeo.dispose(),
-        () => extrudeMat.dispose(),
-        () => textures.forEach((tex) => tex.dispose())
-      )
-    } catch (err) {
-      console.error('[useHeroLogo] initialization failed:', err)
-      cleanup?.()
-    }
-  })
-
-  onUnmounted(() => {
-    cleanup?.()
-  })
-
-  return { loaded }
-}

apps/website/src/i18n/translations.ts

@@ -1119,10 +1119,6 @@ const translations = {
     en: 'Import your own LoRAs',
     'zh-CN': '导入你自己的 LoRA'
   },
-  'pricing.plan.creator.feature2': {
-    en: '3 concurrent API jobs',
-    'zh-CN': '3 个并发 API 任务'
-  },
 
   'pricing.plan.pro.label': { en: 'PRO', 'zh-CN': '专业版' },
   'pricing.plan.pro.summary': {
@@ -1147,10 +1143,6 @@ const translations = {
     en: 'Longer workflow runtime (up to 1 hour)',
     'zh-CN': '更长工作流运行时长（最长 1 小时）'
   },
-  'pricing.plan.pro.feature2': {
-    en: '5 concurrent API jobs',
-    'zh-CN': '5 个并发 API 任务'
-  },
 
   'pricing.enterprise.label': { en: 'ENTERPRISE', 'zh-CN': '企业版' },
   'pricing.enterprise.heading': {
@@ -1607,7 +1599,7 @@ const translations = {
   },
   'nav.comfyHub': { en: 'Comfy Hub', 'zh-CN': 'Comfy Hub' },
   'nav.gallery': { en: 'Gallery', 'zh-CN': '画廊' },
-  'nav.blogs': { en: 'Blog', 'zh-CN': '博客' },
+  'nav.blogs': { en: 'Blogs', 'zh-CN': '博客' },
   'nav.github': { en: 'GitHub', 'zh-CN': 'GitHub' },
   'nav.discord': { en: 'Discord', 'zh-CN': 'Discord' },
   'nav.docs': { en: 'Docs', 'zh-CN': '文档' },
@@ -3506,6 +3498,18 @@ const translations = {
     en: 'Dale Carman | Co-founder @ Groove Jones',
     'zh-CN': 'Dale Carman | Groove Jones 联合创始人'
   },
+  'customers.detail.groove-jones.topic-10.block.2.label': {
+    en: 'GROOVE JONES CONTRIBUTORS',
+    'zh-CN': 'GROOVE JONES 贡献者'
+  },
+  'customers.detail.groove-jones.topic-10.block.2.name': {
+    en: 'TBD',
+    'zh-CN': '待补充'
+  },
+  'customers.detail.groove-jones.topic-10.block.2.role': {
+    en: 'TBD',
+    'zh-CN': '待补充'
+  },
 
   // Contact – FormSection
   'contact.form.badge': {
@@ -3690,8 +3694,8 @@ const translations = {
     'zh-CN': '支付'
   },
   'payment.failed.title': {
-    en: 'Unable to complete payment',
-    'zh-CN': '无法完成支付'
+    en: 'Payment was not completed',
+    'zh-CN': '支付未完成'
   },
   'payment.failed.subtitle': {
     en: "Your payment didn't go through and you have not been charged. Reach out to support or read the subscription docs if you need help.",

apps/website/src/layouts/BaseLayout.astro

@@ -10,7 +10,6 @@ import { fetchGitHubStars, formatStarCount } from '../utils/github'
 interface Props {
   title: string
   description?: string
-  keywords?: string[]
   ogImage?: string
   noindex?: boolean
 }
@@ -18,13 +17,10 @@ interface Props {
 const {
   title,
   description = 'Comfy is the AI creation engine for visual professionals who demand control.',
-  keywords,
   ogImage = 'https://media.comfy.org/website/comfy.webp',
   noindex = false,
 } = Astro.props
 
-const keywordsContent = keywords && keywords.length > 0 ? keywords.join(', ') : undefined
-
 const siteBase = Astro.site ?? 'https://comfy.org'
 const canonicalURL = new URL(Astro.url.pathname, siteBase)
 const ogImageURL = new URL(ogImage, siteBase)
@@ -66,7 +62,6 @@ const websiteJsonLd = {
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1" />
     <meta name="description" content={description} />
-    {keywordsContent && <meta name="keywords" content={keywordsContent} />}
     {noindex && <meta name="robots" content="noindex, nofollow" />}
     <title>{title}</title>
 

apps/website/src/pages/cloud/index.astro

@@ -7,14 +7,9 @@ import AudienceSection from '../../components/product/cloud/AudienceSection.vue'
 import PricingSection from '../../components/product/cloud/PricingSection.vue'
 import ProductCardsSection from '../../components/product/cloud/ProductCardsSection.vue'
 import FAQSection from '../../components/product/cloud/FAQSection.vue'
-import { t } from '../../i18n/translations'
 ---
 
-<BaseLayout
-  title="Comfy Cloud — AI in the Cloud"
-  description={t('cloud.hero.subtitle', 'en')}
-  keywords={['comfyui web app', 'comfyui app', 'comfyui online', 'comfyui cloud', 'comfy cloud', 'comfy ui application', 'comfyui browser', 'cloud comfyui', 'managed comfyui']}
->
+<BaseLayout title="Comfy Cloud — AI in the Cloud">
   <HeroSection />
   <ReasonSection />
   <AIModelsSection />

apps/website/src/pages/download.astro

@@ -7,14 +7,9 @@ import ReasonSection from '../components/product/local/ReasonSection.vue'
 import EcoSystemSection from '../components/product/local/EcoSystemSection.vue'
 import ProductCardsSection from '../components/product/local/ProductCardsSection.vue'
 import FAQSection from '../components/product/local/FAQSection.vue'
-import { t } from '../i18n/translations'
 ---
 
-<BaseLayout
-  title="Download Comfy — Run AI Locally"
-  description={t('download.hero.subtitle', 'en')}
-  keywords={['comfyui app', 'comfyui desktop app', 'comfy ui application', 'comfyui download', 'download comfyui', 'comfyui windows', 'comfyui mac', 'comfyui linux', 'comfyui local']}
->
+<BaseLayout title="Download Comfy — Run AI Locally">
   <CloudBannerSection />
   <HeroSection client:load />
   <ReasonSection />

apps/website/src/pages/index.astro

@@ -8,14 +8,9 @@ import UseCaseSection from '../components/home/UseCaseSection.vue'
 import CaseStudySpotlightSection from '../components/home/CaseStudySpotlightSection.vue'
 import GetStartedSection from '../components/home/GetStartedSection.vue'
 import BuildWhatSection from '../components/home/BuildWhatSection.vue'
-import { t } from '../i18n/translations'
 ---
 
-<BaseLayout
-  title="Comfy — Professional Control of Visual AI"
-  description={t('hero.subtitle', 'en')}
-  keywords={['comfyui app', 'comfyui web app', 'comfy ui application', 'comfyui application', 'comfy app', 'comfyui', 'visual ai app', 'node-based ai', 'generative ai workflows']}
->
+<BaseLayout title="Comfy — Professional Control of Visual AI">
   <HeroSection client:load />
   <SocialProofBarSection />
   <ProductShowcaseSection client:load />

apps/website/src/pages/zh-CN/cloud/index.astro

@@ -7,14 +7,9 @@ import AudienceSection from '../../../components/product/cloud/AudienceSection.v
 import PricingSection from '../../../components/product/cloud/PricingSection.vue'
 import ProductCardsSection from '../../../components/product/cloud/ProductCardsSection.vue'
 import FAQSection from '../../../components/product/cloud/FAQSection.vue'
-import { t } from '../../../i18n/translations'
 ---
 
-<BaseLayout
-  title="Comfy Cloud — 云端 AI"
-  description={t('cloud.hero.subtitle', 'zh-CN')}
-  keywords={['comfyui web app', 'comfyui app', 'comfyui online', 'comfyui cloud', 'ComfyUI 网页版', 'ComfyUI 云端', 'ComfyUI 应用', 'Comfy Cloud', '云端 ComfyUI']}
->
+<BaseLayout title="Comfy Cloud — 云端 AI">
   <HeroSection locale="zh-CN" />
   <ReasonSection locale="zh-CN" />
   <AIModelsSection locale="zh-CN" />

apps/website/src/pages/zh-CN/download.astro

@@ -7,14 +7,9 @@ import ReasonSection from '../../components/product/local/ReasonSection.vue'
 import EcoSystemSection from '../../components/product/local/EcoSystemSection.vue'
 import ProductCardsSection from '../../components/product/local/ProductCardsSection.vue'
 import FAQSection from '../../components/product/local/FAQSection.vue'
-import { t } from '../../i18n/translations'
 ---
 
-<BaseLayout
-  title="下载 — Comfy"
-  description={t('download.hero.subtitle', 'zh-CN')}
-  keywords={['comfyui app', 'comfyui desktop app', 'comfyui download', 'ComfyUI 下载', 'ComfyUI 桌面应用', 'ComfyUI 应用', 'ComfyUI Windows', 'ComfyUI macOS', 'ComfyUI Linux']}
->
+<BaseLayout title="下载 — Comfy">
   <CloudBannerSection locale="zh-CN" />
   <HeroSection locale="zh-CN" client:load />
   <ReasonSection locale="zh-CN" />

apps/website/src/pages/zh-CN/index.astro

@@ -8,14 +8,9 @@ import UseCaseSection from '../../components/home/UseCaseSection.vue'
 import CaseStudySpotlightSection from '../../components/home/CaseStudySpotlightSection.vue'
 import GetStartedSection from '../../components/home/GetStartedSection.vue'
 import BuildWhatSection from '../../components/home/BuildWhatSection.vue'
-import { t } from '../../i18n/translations'
 ---
 
-<BaseLayout
-  title="Comfy — 视觉 AI 的最强可控性"
-  description={t('hero.subtitle', 'zh-CN')}
-  keywords={['comfyui app', 'comfyui web app', 'comfyui application', 'ComfyUI 应用', 'ComfyUI 网页版', 'ComfyUI 桌面应用', 'ComfyUI 下载', '可视化 AI', '节点式 AI', '生成式 AI 工作流']}
->
+<BaseLayout title="Comfy — 视觉 AI 的最强可控性">
   <HeroSection locale="zh-CN" client:load />
   <SocialProofBarSection />
   <ProductShowcaseSection locale="zh-CN" client:load />

browser_tests/assets/missing/nested_subgraph_installed_model.json

@@ -1,232 +0,0 @@
-{
-  "id": "14af6003-d4ee-4dee-8e3d-cbff2e5519b3",
-  "revision": 0,
-  "last_node_id": 205,
-  "last_link_id": 383,
-  "nodes": [
-    {
-      "id": 205,
-      "type": "821645cc-a5d2-468f-990c-17d9de2e0d1b",
-      "pos": [4720, 5820],
-      "size": [400, 470],
-      "flags": {},
-      "order": 0,
-      "mode": 0,
-      "inputs": [
-        {
-          "label": "lotus_model",
-          "name": "unet_name_1",
-          "type": "COMBO",
-          "widget": {
-            "name": "unet_name_1"
-          },
-          "link": null
-        }
-      ],
-      "outputs": [
-        {
-          "name": "IMAGE",
-          "type": "IMAGE",
-          "links": null
-        }
-      ],
-      "properties": {
-        "proxyWidgets": [["76", "unet_name"]]
-      },
-      "widgets_values": []
-    }
-  ],
-  "links": [],
-  "groups": [],
-  "definitions": {
-    "subgraphs": [
-      {
-        "id": "821645cc-a5d2-468f-990c-17d9de2e0d1b",
-        "version": 1,
-        "state": {
-          "lastGroupId": 8,
-          "lastNodeId": 205,
-          "lastLinkId": 383,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "Depth to Image (Z-Image-Turbo)",
-        "inputNode": {
-          "id": -10,
-          "bounding": [28, 4936, 128, 68]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [1599, 4936, 128, 68]
-        },
-        "inputs": [
-          {
-            "id": "80e6915f-5d59-4d6b-a197-d8c565ad2922",
-            "name": "unet_name_1",
-            "type": "COMBO",
-            "linkIds": [258],
-            "pos": [132, 4960]
-          }
-        ],
-        "outputs": [
-          {
-            "id": "47f9a22d-6619-4917-9447-a7d5d08dceb5",
-            "name": "IMAGE",
-            "type": "IMAGE",
-            "linkIds": [],
-            "pos": [1623, 4960]
-          }
-        ],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 76,
-            "type": "a1134394-29e4-48dc-9b1e-e601a14d6fb8",
-            "pos": [250, 4910],
-            "size": [400, 210],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "name": "unet_name",
-                "type": "COMBO",
-                "widget": {
-                  "name": "unet_name"
-                },
-                "link": 258
-              }
-            ],
-            "outputs": [
-              {
-                "name": "IMAGE",
-                "type": "IMAGE",
-                "links": []
-              }
-            ],
-            "properties": {
-              "proxyWidgets": [["203", "unet_name"]]
-            },
-            "widgets_values": []
-          }
-        ],
-        "groups": [],
-        "links": [
-          {
-            "id": 258,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 76,
-            "target_slot": 0,
-            "type": "COMBO"
-          }
-        ],
-        "extra": {
-          "workflowRendererVersion": "LG",
-          "ds": {
-            "scale": 1,
-            "offset": [-30, -4760]
-          }
-        }
-      },
-      {
-        "id": "a1134394-29e4-48dc-9b1e-e601a14d6fb8",
-        "version": 1,
-        "state": {
-          "lastGroupId": 8,
-          "lastNodeId": 205,
-          "lastLinkId": 383,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "Image to Depth Map (Lotus)",
-        "inputNode": {
-          "id": -10,
-          "bounding": [-60, -173, 128, 68]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [1650, -173, 128, 68]
-        },
-        "inputs": [
-          {
-            "id": "d721b249-fd2a-441b-9a78-2805f04e2644",
-            "name": "unet_name",
-            "type": "COMBO",
-            "linkIds": [256],
-            "pos": [44, -149]
-          }
-        ],
-        "outputs": [
-          {
-            "id": "2ec278bd-0b66-4b30-9c5b-994d5f638214",
-            "name": "IMAGE",
-            "type": "IMAGE",
-            "linkIds": [],
-            "pos": [1674, -149]
-          }
-        ],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 203,
-            "type": "UNETLoader",
-            "pos": [180, -200],
-            "size": [400, 200],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "name": "unet_name",
-                "type": "COMBO",
-                "widget": {
-                  "name": "unet_name"
-                },
-                "link": 256
-              }
-            ],
-            "outputs": [
-              {
-                "name": "MODEL",
-                "type": "MODEL",
-                "links": []
-              }
-            ],
-            "properties": {},
-            "widgets_values": ["lotus-depth-d-v1-1.safetensors", "default"]
-          }
-        ],
-        "groups": [],
-        "links": [
-          {
-            "id": 256,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 203,
-            "target_slot": 0,
-            "type": "COMBO"
-          }
-        ],
-        "extra": {
-          "workflowRendererVersion": "LG",
-          "ds": {
-            "scale": 1,
-            "offset": [40, 350]
-          }
-        }
-      }
-    ]
-  },
-  "config": {},
-  "extra": {
-    "workflowRendererVersion": "LG",
-    "ds": {
-      "scale": 1,
-      "offset": [-4500, -5670]
-    }
-  },
-  "version": 0.4
-}

browser_tests/assets/subgraphs/subgraph-with-link-and-proxied-primitive.json

@@ -1,179 +0,0 @@
-{
-  "id": "5c4a1450-26b8-4b34-b5ea-e3465273441e",
-  "revision": 0,
-  "last_node_id": 4,
-  "last_link_id": 2,
-  "nodes": [
-    {
-      "id": 2,
-      "type": "16aadaf6-aa66-4041-843e-589a6572a3ac",
-      "pos": [602, 409],
-      "size": [225, 144],
-      "flags": {},
-      "order": 0,
-      "mode": 0,
-      "inputs": [],
-      "outputs": [],
-      "properties": {
-        "proxyWidgets": [
-          ["1", "value"],
-          ["4", "value"]
-        ]
-      },
-      "widgets_values": []
-    }
-  ],
-  "links": [],
-  "groups": [],
-  "definitions": {
-    "subgraphs": [
-      {
-        "id": "16aadaf6-aa66-4041-843e-589a6572a3ac",
-        "version": 1,
-        "state": {
-          "lastGroupId": 0,
-          "lastNodeId": 4,
-          "lastLinkId": 2,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "New Subgraph",
-        "inputNode": {
-          "id": -10,
-          "bounding": [349, 383, 128, 68]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [867, 383, 128, 48]
-        },
-        "inputs": [
-          {
-            "id": "50fd1af4-4f20-434f-9828-6971210be4e9",
-            "name": "value",
-            "type": "STRING",
-            "linkIds": [1],
-            "pos": [453, 407]
-          }
-        ],
-        "outputs": [],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 1,
-            "type": "PrimitiveString",
-            "pos": [537, 368],
-            "size": [270, 108],
-            "flags": {},
-            "order": 2,
-            "mode": 0,
-            "inputs": [
-              {
-                "localized_name": "value",
-                "name": "value",
-                "type": "STRING",
-                "widget": {
-                  "name": "value"
-                },
-                "link": 1
-              }
-            ],
-            "outputs": [
-              {
-                "localized_name": "STRING",
-                "name": "STRING",
-                "type": "STRING",
-                "links": null
-              }
-            ],
-            "properties": {
-              "Node name for S&R": "PrimitiveString"
-            },
-            "widgets_values": [""]
-          },
-          {
-            "id": 3,
-            "type": "PrimitiveInt",
-            "pos": [534.9899497487436, 515.4924623115581],
-            "size": [270, 104],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "localized_name": "value",
-                "name": "value",
-                "type": "INT",
-                "widget": {
-                  "name": "value"
-                },
-                "link": 2
-              }
-            ],
-            "outputs": [
-              {
-                "localized_name": "INT",
-                "name": "INT",
-                "type": "INT",
-                "links": null
-              }
-            ],
-            "properties": {
-              "Node name for S&R": "PrimitiveInt"
-            },
-            "widgets_values": [0, "randomize"]
-          },
-          {
-            "id": 4,
-            "type": "PrimitiveNode",
-            "pos": [258.4381232333541, 549.1608040200999],
-            "size": [225, 104],
-            "flags": {},
-            "order": 1,
-            "mode": 0,
-            "inputs": [],
-            "outputs": [
-              {
-                "name": "INT",
-                "type": "INT",
-                "widget": {
-                  "name": "value"
-                },
-                "links": [2]
-              }
-            ],
-            "properties": {
-              "Run widget replace on values": false
-            },
-            "widgets_values": [0, "randomize"]
-          }
-        ],
-        "groups": [],
-        "links": [
-          {
-            "id": 1,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 1,
-            "target_slot": 0,
-            "type": "STRING"
-          },
-          {
-            "id": 2,
-            "origin_id": 4,
-            "origin_slot": 0,
-            "target_id": 3,
-            "target_slot": 0,
-            "type": "INT"
-          }
-        ],
-        "extra": {}
-      }
-    ]
-  },
-  "config": {},
-  "extra": {
-    "frontendVersion": "1.44.17"
-  },
-  "version": 0.4
-}

browser_tests/fixtures/ComfyPage.ts

@@ -190,9 +190,6 @@ export class ComfyPage {
   /** Worker index to test user ID */
   public readonly userIds: string[] = []
 
-  /** Whether the current test runs in Vue Nodes mode (initialized from `@vue-nodes` tag). */
-  public isVueNodes = false
-
   /** Test user ID for the current context */
   get id() {
     return this.userIds[comfyPageFixture.info().parallelIndex]
@@ -355,12 +352,6 @@ export class ComfyPage {
     await nextFrame(this.page)
   }
 
-  async idleFrames(count: number) {
-    for (let i = 0; i < count; i++) {
-      await this.nextFrame()
-    }
-  }
-
   async delay(ms: number) {
     return sleep(ms)
   }
@@ -469,15 +460,10 @@ export const testComfySnapToGridGridSize = 50
 const COLLECT_COVERAGE = process.env.COLLECT_COVERAGE === 'true'
 
 export const comfyPageFixture = base.extend<{
-  initialFeatureFlags: Record<string, unknown>
   comfyPage: ComfyPage
   comfyMouse: ComfyMouse
   comfyFiles: ComfyFiles
 }>({
-  // Allows configuring feature flags for tests with before initial setup:
-  // `test.use({ initialFeatureFlags: { my_flag: true } })`.
-  initialFeatureFlags: [{}, { option: true }],
-
   page: async ({ page, browserName }, use) => {
     if (browserName !== 'chromium' || !COLLECT_COVERAGE) {
       return use(page)
@@ -494,7 +480,7 @@ export const comfyPageFixture = base.extend<{
     await mcr.add(coverage)
   },
 
-  comfyPage: async ({ page, request, initialFeatureFlags }, use, testInfo) => {
+  comfyPage: async ({ page, request }, use, testInfo) => {
     const comfyPage = new ComfyPage(page, request)
 
     const { parallelIndex } = testInfo
@@ -503,7 +489,6 @@ export const comfyPageFixture = base.extend<{
     comfyPage.userIds[parallelIndex] = userId
 
     const isVueNodes = testInfo.tags.includes('@vue-nodes')
-    comfyPage.isVueNodes = isVueNodes
 
     try {
       await comfyPage.setupSettings({
@@ -539,10 +524,6 @@ export const comfyPageFixture = base.extend<{
       await comfyPage.cloudAuth.mockAuth()
     }
 
-    if (Object.keys(initialFeatureFlags).length > 0) {
-      await comfyPage.featureFlags.seedFlags(initialFeatureFlags)
-    }
-
     await comfyPage.setup()
 
     if (isVueNodes) {

browser_tests/fixtures/VueNodeHelpers.ts

@@ -5,7 +5,6 @@ import type { Locator, Page } from '@playwright/test'
 
 import { TestIds } from '@e2e/fixtures/selectors'
 import { VueNodeFixture } from '@e2e/fixtures/utils/vueNodeFixtures'
-import { getSlotKey } from '@/renderer/core/layout/slots/slotIdentifier'
 
 export class VueNodeHelpers {
   /**
@@ -38,22 +37,6 @@ export class VueNodeHelpers {
     return this.getNodeLocator(nodeId).getByTestId(TestIds.node.innerWrapper)
   }
 
-  getInputSlotRow(nodeId: string, slotIndex: number): Locator {
-    return this.getNodeLocator(nodeId)
-      .locator('.lg-slot--input')
-      .filter({
-        has: this.page.locator(
-          `[data-slot-key="${getSlotKey(nodeId, slotIndex, true)}"]`
-        )
-      })
-  }
-
-  getInputSlotConnectionDot(nodeId: string, slotIndex: number): Locator {
-    return this.getInputSlotRow(nodeId, slotIndex).getByTestId(
-      TestIds.node.slotConnectionDot
-    )
-  }
-
   /**
    * Get locator for Vue nodes by the node's title (displayed name in the header).
    * Matches against the actual title element, not the full node body.
@@ -217,20 +200,13 @@ export class VueNodeHelpers {
     }
   }
 
-  /**
-   * Locator for the Enter Subgraph footer button.
-   */
-  getSubgraphEnterButton(nodeId?: string): Locator {
-    const root = nodeId ? this.getNodeLocator(nodeId) : this.page
-    return root.getByTestId(TestIds.widgets.subgraphEnterButton).first()
-  }
-
   /**
    * Enter the subgraph of a node.
    * @param nodeId - The ID of the node to enter the subgraph of. If not provided, the first matched subgraph will be entered.
    */
   async enterSubgraph(nodeId?: string): Promise<void> {
-    const editButton = this.getSubgraphEnterButton(nodeId)
+    const locator = nodeId ? this.getNodeLocator(nodeId) : this.page
+    const editButton = locator.getByTestId(TestIds.widgets.subgraphEnterButton)
 
     // The footer tab button extends below the node body (visible area),
     // but its bounding box center overlaps the node body div.

browser_tests/fixtures/assetApiFixture.ts

@@ -1,34 +1,8 @@
 import { test as base } from '@playwright/test'
-import type { Page, Route } from '@playwright/test'
 
-import type { Asset, ListAssetsResponse } from '@comfyorg/ingest-types'
-import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
 import type { AssetHelper } from '@e2e/fixtures/helpers/AssetHelper'
 import { createAssetHelper } from '@e2e/fixtures/helpers/AssetHelper'
 
-const ASSETS_ROUTE_PATTERN = /\/api\/assets(?:\?.*)?$/
-const cloudAssetRequestsByPage = new WeakMap<Page, string[]>()
-
-function makeAssetsResponse(assets: ReadonlyArray<Asset>): ListAssetsResponse {
-  return { assets: [...assets], total: assets.length, has_more: false }
-}
-
-export function assetRequestIncludesTag(url: string, tag: string): boolean {
-  const includeTags = new URL(url).searchParams.get('include_tags') ?? ''
-  return includeTags
-    .split(',')
-    .map((value) => value.trim())
-    .filter(Boolean)
-    .includes(tag)
-}
-
-export function countAssetRequestsByTag(
-  requests: string[],
-  tag: string
-): number {
-  return requests.filter((url) => assetRequestIncludesTag(url, tag)).length
-}
-
 export const assetApiFixture = base.extend<{
   assetApi: AssetHelper
 }>({
@@ -40,31 +14,3 @@ export const assetApiFixture = base.extend<{
     await assetApi.clearMocks()
   }
 })
-
-export function createCloudAssetsFixture(assets: ReadonlyArray<Asset>) {
-  return comfyPageFixture.extend<{
-    cloudAssetRequests: string[]
-  }>({
-    page: async ({ page }, use) => {
-      const cloudAssetRequests: string[] = []
-      cloudAssetRequestsByPage.set(page, cloudAssetRequests)
-
-      async function assetsRouteHandler(route: Route) {
-        cloudAssetRequests.push(route.request().url())
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify(makeAssetsResponse(assets))
-        })
-      }
-
-      await page.route(ASSETS_ROUTE_PATTERN, assetsRouteHandler)
-      await use(page)
-      await page.unroute(ASSETS_ROUTE_PATTERN, assetsRouteHandler)
-      cloudAssetRequestsByPage.delete(page)
-    },
-    cloudAssetRequests: async ({ page }, use) => {
-      await use(cloudAssetRequestsByPage.get(page) ?? [])
-    }
-  })
-}

browser_tests/fixtures/components/Actionbar.ts

@@ -39,32 +39,10 @@ class ComfyQueueButton {
     await this.dropdownButton.click()
     return new ComfyQueueButtonOptions(this.actionbar.page)
   }
-
-  public async openOptions() {
-    const options = new ComfyQueueButtonOptions(this.actionbar.page)
-    if (!(await options.menu.isVisible())) {
-      await this.dropdownButton.click()
-    }
-    return options
-  }
 }
 
 class ComfyQueueButtonOptions {
-  public readonly menu: Locator
-  public readonly modeItems: Locator
-
-  constructor(public readonly page: Page) {
-    this.menu = page.getByRole('menu')
-    this.modeItems = this.menu.getByRole('menuitem')
-  }
-
-  public modeItem(name: string) {
-    return this.menu.getByRole('menuitem', { name, exact: true })
-  }
-
-  public async selectMode(name: string) {
-    await this.modeItem(name).click()
-  }
+  constructor(public readonly page: Page) {}
 
   public async setMode(mode: AutoQueueMode) {
     await this.page.evaluate((mode) => {

browser_tests/fixtures/components/ContextMenu.ts

@@ -20,7 +20,6 @@ export class ContextMenu {
 
   async clickMenuItemExact(name: string): Promise<void> {
     await this.page.getByRole('menuitem', { name, exact: true }).click()
-    await this.waitForHidden()
   }
 
   /**

browser_tests/fixtures/components/Topbar.ts

@@ -1,23 +1,16 @@
 import type { Locator, Page } from '@playwright/test'
 
 import type { WorkspaceStore } from '@e2e/types/globals'
-import { TestIds } from '@e2e/fixtures/selectors'
 
 export class Topbar {
   private readonly menuLocator: Locator
   private readonly menuTrigger: Locator
   readonly newWorkflowButton: Locator
-  readonly workflowTabs: Locator
-  readonly integratedTabBarActions: Locator
 
   constructor(public readonly page: Page) {
     this.menuLocator = page.locator('.comfy-command-menu')
     this.menuTrigger = page.locator('.comfy-menu-button-wrapper')
     this.newWorkflowButton = page.locator('.new-blank-workflow-button')
-    this.workflowTabs = page.getByTestId(TestIds.topbar.workflowTabs)
-    this.integratedTabBarActions = this.workflowTabs.getByTestId(
-      TestIds.topbar.integratedTabBarActions
-    )
   }
 
   async getTabNames(): Promise<string[]> {
@@ -82,7 +75,7 @@ export class Topbar {
   }
 
   getSaveDialog(): Locator {
-    return this.page.getByRole('dialog').getByRole('textbox')
+    return this.page.locator('.p-dialog-content input')
   }
 
   saveWorkflow(workflowName: string): Promise<void> {
@@ -116,9 +109,9 @@ export class Topbar {
 
     // Check if a confirmation dialog appeared (e.g., "Overwrite existing file?")
     // If so, return early to let the test handle the confirmation
-    const confirmationDialog = this.page
-      .getByRole('dialog')
-      .filter({ hasText: 'Overwrite' })
+    const confirmationDialog = this.page.locator(
+      '.p-dialog:has-text("Overwrite")'
+    )
     if (await confirmationDialog.isVisible()) {
       return
     }

browser_tests/fixtures/components/WidgetSelectDropdown.ts

@@ -1,12 +0,0 @@
-import type { Locator } from '@playwright/test'
-
-export class WidgetSelectDropdownFixture {
-  public readonly selection: Locator
-
-  constructor(public readonly root: Locator) {
-    this.selection = root.locator('button span span')
-  }
-  async selectedItem(): Promise<string> {
-    return await this.selection.innerText()
-  }
-}

browser_tests/fixtures/helpers/AppModeHelper.ts

@@ -9,15 +9,13 @@ import { BuilderFooterHelper } from '@e2e/fixtures/helpers/BuilderFooterHelper'
 import { BuilderSaveAsHelper } from '@e2e/fixtures/helpers/BuilderSaveAsHelper'
 import { BuilderSelectHelper } from '@e2e/fixtures/helpers/BuilderSelectHelper'
 import { BuilderStepsHelper } from '@e2e/fixtures/helpers/BuilderStepsHelper'
-import { MobileAppHelper } from '@e2e/fixtures/helpers/MobileAppHelper'
 
 export class AppModeHelper {
+  readonly steps: BuilderStepsHelper
   readonly footer: BuilderFooterHelper
-  readonly mobile: MobileAppHelper
   readonly saveAs: BuilderSaveAsHelper
   readonly select: BuilderSelectHelper
   readonly outputHistory: OutputHistoryComponent
-  readonly steps: BuilderStepsHelper
   readonly widgets: AppModeWidgetHelper
 
   /** The "Connect an output" popover shown when saving without outputs. */
@@ -62,16 +60,13 @@ export class AppModeHelper {
   public readonly vueNodeSwitchDismissButton: Locator
   /** The "Don't show again" checkbox inside the Vue Node switch popup. */
   public readonly vueNodeSwitchDontShowAgainCheckbox: Locator
-  /** The main content area where outputs are displayed*/
-  public readonly centerPanel: Locator
 
   constructor(private readonly comfyPage: ComfyPage) {
-    this.mobile = new MobileAppHelper(comfyPage)
+    this.steps = new BuilderStepsHelper(comfyPage)
     this.footer = new BuilderFooterHelper(comfyPage)
     this.saveAs = new BuilderSaveAsHelper(comfyPage)
     this.select = new BuilderSelectHelper(comfyPage)
     this.outputHistory = new OutputHistoryComponent(comfyPage.page)
-    this.steps = new BuilderStepsHelper(comfyPage)
     this.widgets = new AppModeWidgetHelper(comfyPage)
 
     this.connectOutputPopover = this.page.getByTestId(
@@ -130,7 +125,6 @@ export class AppModeHelper {
     this.vueNodeSwitchDontShowAgainCheckbox = this.page.getByTestId(
       TestIds.appMode.vueNodeSwitchDontShowAgain
     )
-    this.centerPanel = this.page.getByTestId(TestIds.linear.centerPanel)
   }
 
   private get page(): Page {

browser_tests/fixtures/helpers/AssetHelper.ts

@@ -215,12 +215,11 @@ export class AssetHelper {
     return this.store.size
   }
   private handleListAssets(route: Route, url: URL) {
-    const includeTags = parseAssetTagParam(url.searchParams.get('include_tags'))
-    const excludeTags = parseAssetTagParam(url.searchParams.get('exclude_tags'))
+    const includeTags = url.searchParams.get('include_tags')?.split(',') ?? []
     const limit = parseInt(url.searchParams.get('limit') ?? '0', 10)
     const offset = parseInt(url.searchParams.get('offset') ?? '0', 10)
 
-    let filtered = this.getFilteredAssets(includeTags, excludeTags)
+    let filtered = this.getFilteredAssets(includeTags)
     if (limit > 0) {
       filtered = filtered.slice(offset, offset + limit)
     }
@@ -297,29 +296,15 @@ export class AssetHelper {
     this.paginationOptions = null
     this.uploadResponse = null
   }
-  private getFilteredAssets(
-    includeTags: string[],
-    excludeTags: string[]
-  ): Asset[] {
+  private getFilteredAssets(tags: string[]): Asset[] {
     const assets = [...this.store.values()]
+    if (tags.length === 0) return assets
 
-    return assets.filter(
-      (asset) =>
-        includeTags.every((tag) => (asset.tags ?? []).includes(tag)) &&
-        excludeTags.every((tag) => !(asset.tags ?? []).includes(tag))
+    return assets.filter((asset) =>
+      tags.every((tag) => (asset.tags ?? []).includes(tag))
     )
   }
 }
-
-function parseAssetTagParam(value: string | null): string[] {
-  return (
-    value
-      ?.split(',')
-      .map((tag) => tag.trim())
-      .filter(Boolean) ?? []
-  )
-}
-
 export function createAssetHelper(
   page: Page,
   ...operators: AssetOperator[]

browser_tests/fixtures/helpers/BuilderSelectHelper.ts

@@ -127,7 +127,9 @@ export class BuilderSelectHelper {
     await popoverTrigger.click()
     await this.page.getByText('Rename', { exact: true }).click()
 
-    const dialogInput = this.page.getByRole('dialog').getByRole('textbox')
+    const dialogInput = this.page.locator(
+      '.p-dialog-content input[type="text"]'
+    )
     await dialogInput.fill(newName)
     await this.page.keyboard.press('Enter')
     await dialogInput.waitFor({ state: 'hidden' })

browser_tests/fixtures/helpers/DragDropHelper.ts

@@ -1,5 +1,4 @@
 import { readFileSync } from 'fs'
-import { basename } from 'path'
 
 import type { Page } from '@playwright/test'
 
@@ -14,7 +13,6 @@ export class DragDropHelper {
   async dragAndDropExternalResource(
     options: {
       fileName?: string
-      filePath?: string
       url?: string
       dropPosition?: Position
       waitForUpload?: boolean
@@ -24,14 +22,13 @@ export class DragDropHelper {
     const {
       dropPosition = { x: 100, y: 100 },
       fileName,
-      filePath,
       url,
       waitForUpload = false,
       preserveNativePropagation = false
     } = options
 
-    if (!fileName && !filePath && !url)
-      throw new Error('Must provide fileName, filePath, or url')
+    if (!fileName && !url)
+      throw new Error('Must provide either fileName or url')
 
     const evaluateParams: {
       dropPosition: Position
@@ -42,22 +39,12 @@ export class DragDropHelper {
       preserveNativePropagation: boolean
     } = { dropPosition, preserveNativePropagation }
 
-    if (fileName || filePath) {
-      const resolvedPath = filePath ?? assetPath(fileName!)
-      const displayName = fileName ?? basename(resolvedPath)
-      let buffer: Buffer
-      try {
-        buffer = readFileSync(resolvedPath)
-      } catch (error) {
-        const reason = error instanceof Error ? error.message : String(error)
-        throw new Error(
-          `Failed to read drag-and-drop fixture at "${resolvedPath}": ${reason}`,
-          { cause: error }
-        )
-      }
+    if (fileName) {
+      const filePath = assetPath(fileName)
+      const buffer = readFileSync(filePath)
 
-      evaluateParams.fileName = displayName
-      evaluateParams.fileType = getMimeType(displayName)
+      evaluateParams.fileName = fileName
+      evaluateParams.fileType = getMimeType(fileName)
       evaluateParams.buffer = [...new Uint8Array(buffer)]
     }
 
@@ -161,13 +148,6 @@ export class DragDropHelper {
     return this.dragAndDropExternalResource({ fileName, ...options })
   }
 
-  async dragAndDropFilePath(
-    filePath: string,
-    options: { dropPosition?: Position; waitForUpload?: boolean } = {}
-  ): Promise<void> {
-    return this.dragAndDropExternalResource({ filePath, ...options })
-  }
-
   async dragAndDropURL(
     url: string,
     options: {

browser_tests/fixtures/helpers/JobsApiMock.ts

@@ -1,176 +0,0 @@
-import type { Page, Route } from '@playwright/test'
-import type {
-  JobDetailResponse,
-  JobEntry,
-  JobsListResponse
-} from '@comfyorg/ingest-types'
-
-const jobsListRoutePattern = /\/api\/jobs(?:\?.*)?$/
-const jobDetailRoutePattern = /\/api\/jobs\/[^/?#]+(?:\?.*)?$/
-const historyRoutePattern = /\/api\/history(?:\?.*)?$/
-const defaultJobsListLimit = 100
-
-export type MockJobRecord = {
-  listItem: JobEntry
-  detail: JobDetailResponse
-}
-
-function parsePositiveIntegerParam(url: URL, name: string): number | undefined {
-  const value = Number(url.searchParams.get(name))
-
-  return Number.isInteger(value) && value > 0 ? value : undefined
-}
-
-function getJobIdFromRequest(route: Route): string | null {
-  const url = new URL(route.request().url())
-  const jobId = url.pathname.split('/').at(-1)
-
-  return jobId ? decodeURIComponent(jobId) : null
-}
-
-export class JobsApiMock {
-  private listRouteHandler: ((route: Route) => Promise<void>) | null = null
-  private detailRouteHandler: ((route: Route) => Promise<void>) | null = null
-  private historyRouteHandler: ((route: Route) => Promise<void>) | null = null
-  private jobsById = new Map<string, MockJobRecord>()
-
-  constructor(private readonly page: Page) {}
-
-  async mockJobs(jobs: MockJobRecord[]): Promise<void> {
-    this.jobsById = new Map(
-      jobs.map(
-        (job) => [job.listItem.id, job] satisfies [string, MockJobRecord]
-      )
-    )
-    await this.ensureRoutesRegistered()
-  }
-
-  async clear(): Promise<void> {
-    this.jobsById.clear()
-
-    if (this.listRouteHandler) {
-      await this.page.unroute(jobsListRoutePattern, this.listRouteHandler)
-      this.listRouteHandler = null
-    }
-
-    if (this.detailRouteHandler) {
-      await this.page.unroute(jobDetailRoutePattern, this.detailRouteHandler)
-      this.detailRouteHandler = null
-    }
-
-    if (this.historyRouteHandler) {
-      await this.page.unroute(historyRoutePattern, this.historyRouteHandler)
-      this.historyRouteHandler = null
-    }
-  }
-
-  private async ensureRoutesRegistered(): Promise<void> {
-    if (!this.listRouteHandler) {
-      this.listRouteHandler = async (route: Route) => {
-        const url = new URL(route.request().url())
-        const statuses = url.searchParams
-          .get('status')
-          ?.split(',')
-          .map((status) => status.trim())
-          .filter(Boolean)
-
-        let filteredJobs = Array.from(
-          this.jobsById.values(),
-          ({ listItem }) => listItem
-        )
-
-        if (statuses?.length) {
-          filteredJobs = filteredJobs.filter((job) =>
-            statuses.includes(job.status)
-          )
-        }
-
-        const offset = parsePositiveIntegerParam(url, 'offset') ?? 0
-        const limit =
-          parsePositiveIntegerParam(url, 'limit') ?? defaultJobsListLimit
-        const total = filteredJobs.length
-        const visibleJobs = filteredJobs.slice(offset, offset + limit)
-
-        const response = {
-          jobs: visibleJobs,
-          pagination: {
-            offset,
-            limit,
-            total,
-            has_more: offset + visibleJobs.length < total
-          }
-        } satisfies JobsListResponse
-
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify(response)
-        })
-      }
-
-      await this.page.route(jobsListRoutePattern, this.listRouteHandler)
-    }
-
-    if (!this.detailRouteHandler) {
-      this.detailRouteHandler = async (route: Route) => {
-        const jobId = getJobIdFromRequest(route)
-        const job = jobId ? this.jobsById.get(jobId) : undefined
-
-        if (!job) {
-          await route.fulfill({
-            status: 404,
-            contentType: 'application/json',
-            body: JSON.stringify({ error: 'Job not found' })
-          })
-          return
-        }
-
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify(job.detail)
-        })
-      }
-
-      await this.page.route(jobDetailRoutePattern, this.detailRouteHandler)
-    }
-
-    if (!this.historyRouteHandler) {
-      this.historyRouteHandler = async (route: Route) => {
-        const request = route.request()
-        if (request.method() !== 'POST') {
-          await route.continue()
-          return
-        }
-
-        const requestBody = request.postDataJSON() as
-          | { delete?: string[]; clear?: boolean }
-          | undefined
-
-        if (requestBody?.clear) {
-          this.jobsById = new Map(
-            Array.from(this.jobsById).filter(([, job]) => {
-              const status = job.listItem.status
-
-              return status === 'pending' || status === 'in_progress'
-            })
-          )
-        }
-
-        if (requestBody?.delete?.length) {
-          for (const jobId of requestBody.delete) {
-            this.jobsById.delete(jobId)
-          }
-        }
-
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify({})
-        })
-      }
-
-      await this.page.route(historyRoutePattern, this.historyRouteHandler)
-    }
-  }
-}

browser_tests/fixtures/helpers/MaskEditorHelper.ts

@@ -1,136 +0,0 @@
-import type { Locator } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-
-const MASK_CANVAS_INDEX = 2
-const RGB_CANVAS_INDEX = 1
-
-export type BrushSliderLabel = 'thickness'
-
-export class MaskEditorHelper {
-  constructor(private comfyPage: ComfyPage) {}
-
-  private get page() {
-    return this.comfyPage.page
-  }
-
-  async loadImageOnNode() {
-    await this.comfyPage.workflow.loadWorkflow('widgets/load_image_widget')
-
-    const loadImageNode = (
-      await this.comfyPage.nodeOps.getNodeRefsByType('LoadImage')
-    )[0]
-    const { x, y } = await loadImageNode.getPosition()
-
-    await this.comfyPage.dragDrop.dragAndDropFile('image64x64.webp', {
-      dropPosition: { x, y }
-    })
-
-    const imagePreview = this.page.locator('.image-preview')
-    await expect(imagePreview).toBeVisible()
-    await expect(imagePreview.locator('img')).toBeVisible()
-    await expect(imagePreview).toContainText('x')
-
-    return {
-      imagePreview,
-      nodeId: String(loadImageNode.id)
-    }
-  }
-
-  async openDialog(): Promise<Locator> {
-    const { imagePreview } = await this.loadImageOnNode()
-
-    await imagePreview.getByRole('region').hover()
-    await this.page.getByLabel('Edit or mask image').click()
-
-    const dialog = this.page.locator('.mask-editor-dialog')
-    await expect(dialog).toBeVisible()
-    await expect(
-      dialog.getByRole('heading', { name: 'Mask Editor' })
-    ).toBeVisible()
-
-    const canvasContainer = dialog.locator('#maskEditorCanvasContainer')
-    await expect(canvasContainer).toBeVisible()
-    await expect(canvasContainer.locator('canvas')).toHaveCount(4)
-
-    return dialog
-  }
-
-  async drawStrokeOnPointerZone(dialog: Locator) {
-    const pointerZone = dialog.getByTestId('pointer-zone')
-    await expect(pointerZone).toBeVisible()
-
-    const box = await pointerZone.boundingBox()
-    if (!box) throw new Error('Pointer zone bounding box not found')
-
-    const startX = box.x + box.width * 0.3
-    const startY = box.y + box.height * 0.5
-    const endX = box.x + box.width * 0.7
-    const endY = box.y + box.height * 0.5
-
-    await this.page.mouse.move(startX, startY)
-    await this.page.mouse.down()
-    await this.page.mouse.move(endX, endY, { steps: 10 })
-    await this.page.mouse.up()
-
-    return { startX, startY, endX, endY, box }
-  }
-
-  async drawStrokeAndExpectPixels(dialog: Locator) {
-    await this.drawStrokeOnPointerZone(dialog)
-    await expect.poll(() => this.pollMaskPixelCount()).toBeGreaterThan(0)
-  }
-
-  getCanvasPixelData(canvasIndex: number) {
-    return this.page.evaluate((idx) => {
-      const canvases = document.querySelectorAll(
-        '#maskEditorCanvasContainer canvas'
-      )
-      const canvas = canvases[idx] as HTMLCanvasElement | undefined
-      if (!canvas) return null
-      const ctx = canvas.getContext('2d')
-      if (!ctx) return null
-      const data = ctx.getImageData(0, 0, canvas.width, canvas.height)
-      let nonTransparentPixels = 0
-      for (let i = 3; i < data.data.length; i += 4) {
-        if (data.data[i] > 0) nonTransparentPixels++
-      }
-      return { nonTransparentPixels, totalPixels: data.data.length / 4 }
-    }, canvasIndex)
-  }
-
-  pollMaskPixelCount(): Promise<number> {
-    return this.getCanvasPixelData(MASK_CANVAS_INDEX).then(
-      (d) => d?.nonTransparentPixels ?? 0
-    )
-  }
-
-  pollRgbPixelCount(): Promise<number> {
-    return this.getCanvasPixelData(RGB_CANVAS_INDEX).then(
-      (d) => d?.nonTransparentPixels ?? 0
-    )
-  }
-
-  getCanvasSnapshot(canvasIndex: number): Promise<string> {
-    return this.page.evaluate((idx) => {
-      const canvas = document.querySelectorAll(
-        '#maskEditorCanvasContainer canvas'
-      )[idx] as HTMLCanvasElement | undefined
-      return canvas?.toDataURL() ?? ''
-    }, canvasIndex)
-  }
-
-  brushInput(dialog: Locator, label: BrushSliderLabel): Locator {
-    return dialog.getByTestId(`brush-${label}-input`)
-  }
-}
-
-export const maskEditorTest = comfyPageFixture.extend<{
-  maskEditor: MaskEditorHelper
-}>({
-  maskEditor: async ({ comfyPage }, use) => {
-    await use(new MaskEditorHelper(comfyPage))
-  }
-})

browser_tests/fixtures/helpers/MobileAppHelper.ts

@@ -1,33 +0,0 @@
-import type { Locator, Page } from '@playwright/test'
-
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { TestIds } from '@e2e/fixtures/selectors'
-
-export class MobileAppHelper {
-  private readonly page: Page
-  readonly contentPanel: Locator
-  readonly navigation: Locator
-  readonly navigationTabs: Locator
-  readonly view: Locator
-  readonly workflows: Locator
-
-  constructor(comfyPage: ComfyPage) {
-    this.page = comfyPage.page
-    this.view = this.page.getByTestId(TestIds.linear.mobile)
-    this.contentPanel = this.page.getByRole('tabpanel')
-    this.navigation = this.page.getByRole('tablist').filter({ hasText: 'Run' })
-    this.navigationTabs = this.navigation.getByRole('tab')
-    this.workflows = this.view.getByTestId(TestIds.linear.mobileWorkflows)
-  }
-
-  async switchWorkflow(workflowName: string) {
-    await this.workflows.click()
-    await this.page.getByRole('menu').getByText(workflowName).click()
-  }
-  async navigateTab(name: 'run' | 'outputs' | 'assets') {
-    await this.navigation.getByRole('tab', { name }).click()
-  }
-  async tap(locator: Locator, { count = 1 }: { count?: number } = {}) {
-    for (let i = 0; i < count; i++) await locator.tap()
-  }
-}

browser_tests/fixtures/helpers/NodeOperationsHelper.ts

@@ -18,7 +18,9 @@ export class NodeOperationsHelper {
   public readonly promptDialogInput: Locator
 
   constructor(private comfyPage: ComfyPage) {
-    this.promptDialogInput = this.page.getByRole('dialog').getByRole('textbox')
+    this.promptDialogInput = this.page.locator(
+      '.p-dialog-content input[type="text"]'
+    )
   }
 
   private get page() {

browser_tests/fixtures/helpers/SubgraphHelper.ts

@@ -362,9 +362,6 @@ export class SubgraphHelper {
 
     await this.comfyPage.nextFrame()
     await expect.poll(async () => this.isInSubgraph()).toBe(false)
-    if (this.comfyPage.isVueNodes) {
-      await this.comfyPage.vueNodes.waitForNodes()
-    }
   }
 
   async countGraphPseudoPreviewEntries(): Promise<number> {

browser_tests/fixtures/jobsApiMockFixture.ts

@@ -1,15 +0,0 @@
-import { test as base } from '@playwright/test'
-
-import { JobsApiMock } from '@e2e/fixtures/helpers/JobsApiMock'
-
-export const jobsApiMockFixture = base.extend<{
-  jobsApi: JobsApiMock
-}>({
-  jobsApi: async ({ page }, use) => {
-    const jobsApi = new JobsApiMock(page)
-
-    await use(jobsApi)
-
-    await jobsApi.clear()
-  }
-})

browser_tests/fixtures/selectors.ts

@@ -91,8 +91,6 @@ export const TestIds = {
     loginButton: 'login-button',
     loginButtonPopover: 'login-button-popover',
     loginButtonPopoverLearnMore: 'login-button-popover-learn-more',
-    workflowTabs: 'topbar-workflow-tabs',
-    integratedTabBarActions: 'integrated-tab-bar-actions',
     actionBarButtons: 'action-bar-buttons'
   },
   nodeLibrary: {
@@ -116,8 +114,7 @@ export const TestIds = {
     titleInput: 'node-title-input',
     pinIndicator: 'node-pin-indicator',
     innerWrapper: 'node-inner-wrapper',
-    mainImage: 'main-image',
-    slotConnectionDot: 'slot-connection-dot'
+    mainImage: 'main-image'
   },
   selectionToolbox: {
     root: 'selection-toolbox',
@@ -144,14 +141,6 @@ export const TestIds = {
     domWidgetTextarea: 'dom-widget-textarea',
     subgraphEnterButton: 'subgraph-enter-button'
   },
-  linear: {
-    centerPanel: 'linear-center-panel',
-    mobile: 'linear-mobile',
-    mobileNavigation: 'linear-mobile-navigation',
-    mobileWorkflows: 'linear-mobile-workflows',
-    outputInfo: 'linear-output-info',
-    widgetContainer: 'linear-widgets'
-  },
   builder: {
     footerNav: 'builder-footer-nav',
     saveButton: 'builder-save-button',
@@ -222,7 +211,6 @@ export const TestIds = {
   },
   queue: {
     overlayToggle: 'queue-overlay-toggle',
-    jobDetailsPopover: 'queue-job-details-popover',
     clearHistoryAction: 'clear-history-action',
     jobAssetsList: 'job-assets-list',
     notificationBanner: 'queue-notification-banner'

browser_tests/fixtures/utils/jobFixtures.ts

@@ -1,52 +0,0 @@
-import type { JobDetailResponse, JobEntry } from '@comfyorg/ingest-types'
-
-import type { MockJobRecord } from '@e2e/fixtures/helpers/JobsApiMock'
-
-export function createMockJob(
-  overrides: Partial<JobEntry> & { id: string }
-): JobEntry {
-  const now = Date.now()
-
-  return {
-    status: 'completed',
-    create_time: now,
-    execution_start_time: now,
-    execution_end_time: now + 5_000,
-    preview_output: {
-      filename: `output_${overrides.id}.png`,
-      subfolder: '',
-      type: 'output',
-      nodeId: '1',
-      mediaType: 'images'
-    },
-    outputs_count: 1,
-    ...overrides
-  }
-}
-
-function isTerminalStatus(status: JobEntry['status']) {
-  return status === 'completed' || status === 'failed' || status === 'cancelled'
-}
-
-function createMockJobRecord(listItem: JobEntry): MockJobRecord {
-  const updateTime =
-    listItem.execution_end_time ??
-    listItem.execution_start_time ??
-    listItem.create_time
-  const detail: JobDetailResponse = {
-    ...listItem,
-    update_time: updateTime,
-    ...(isTerminalStatus(listItem.status) ? { outputs: {} } : {})
-  }
-
-  return {
-    listItem,
-    detail
-  }
-}
-
-export function createMockJobRecords(
-  listItems: readonly JobEntry[]
-): MockJobRecord[] {
-  return listItems.map(createMockJobRecord)
-}

browser_tests/fixtures/utils/mimeTypeUtil.ts

@@ -7,9 +7,6 @@ export function getMimeType(fileName: string): string {
   if (name.endsWith('.avif')) return 'image/avif'
   if (name.endsWith('.webm')) return 'video/webm'
   if (name.endsWith('.mp4')) return 'video/mp4'
-  if (name.endsWith('.mp3')) return 'audio/mpeg'
-  if (name.endsWith('.flac')) return 'audio/flac'
-  if (name.endsWith('.ogg') || name.endsWith('.opus')) return 'audio/ogg'
   if (name.endsWith('.json')) return 'application/json'
   if (name.endsWith('.glb')) return 'model/gltf-binary'
   return 'application/octet-stream'

browser_tests/fixtures/utils/paths.ts

@@ -1,7 +1,3 @@
 export function assetPath(fileName: string): string {
   return `./browser_tests/assets/${fileName}`
 }
-
-export function metadataFixturePath(fileName: string): string {
-  return `./src/scripts/metadata/__fixtures__/${fileName}`
-}

browser_tests/fixtures/utils/vueNodeFixtures.ts

@@ -13,7 +13,6 @@ export class VueNodeFixture {
   public readonly collapseButton: Locator
   public readonly collapseIcon: Locator
   public readonly root: Locator
-  public readonly widgets: Locator
 
   constructor(private readonly locator: Locator) {
     this.header = locator.locator('[data-testid^="node-header-"]')
@@ -24,7 +23,6 @@ export class VueNodeFixture {
     this.collapseButton = locator.getByTestId('node-collapse-button')
     this.collapseIcon = this.collapseButton.locator('i')
     this.root = locator
-    this.widgets = this.locator.locator('.lg-node-widget')
   }
 
   async getTitle(): Promise<string> {
@@ -41,16 +39,6 @@ export class VueNodeFixture {
     await this.collapseButton.click()
   }
 
-  /**
-   * Select this node and delete it via the Delete key, waiting for the node
-   * element to leave the DOM before resolving.
-   */
-  async delete(): Promise<void> {
-    await this.header.click()
-    await this.header.press('Delete')
-    await this.locator.waitFor({ state: 'hidden' })
-  }
-
   async getCollapseIconClass(): Promise<string> {
     return (await this.collapseIcon.getAttribute('class')) ?? ''
   }

browser_tests/tests/appMode.spec.ts

@@ -1,154 +0,0 @@
-import {
-  comfyPageFixture as test,
-  comfyExpect as expect
-} from '@e2e/fixtures/ComfyPage'
-import { WidgetSelectDropdownFixture } from '@e2e/fixtures/components/WidgetSelectDropdown'
-
-test.describe('App mode usage', () => {
-  test('Drag and Drop', async ({ comfyPage, comfyFiles }) => {
-    await comfyPage.settings.setSetting('Comfy.VueNodes.Enabled', true)
-    await comfyPage.settings.setSetting(
-      'Comfy.NodeSearchBoxImpl',
-      'v1 (legacy)'
-    )
-    const { centerPanel } = comfyPage.appMode
-    await comfyPage.appMode.enterAppModeWithInputs([['3', 'seed']])
-    await expect(centerPanel, 'Enter app mode').toBeVisible()
-
-    //an app without an image input will load the workflow
-    await test.step('App without an image input loads workflow', async () => {
-      await comfyPage.dragDrop.dragAndDropFile('workflowInMedia/workflow.webp')
-      await expect(centerPanel).toBeHidden()
-    })
-
-    //prep a load image
-    await test.step('Add a load image node', async () => {
-      await comfyPage.workflow.loadWorkflow('default')
-      await comfyPage.page.mouse.dblclick(200, 200, { delay: 5 })
-      await comfyPage.searchBox.fillAndSelectFirstNode('Load Image')
-      const loadImage = await comfyPage.vueNodes.getNodeLocator('10')
-      await expect(loadImage).toBeVisible()
-    })
-
-    const imageInput = new WidgetSelectDropdownFixture(
-      comfyPage.appMode.linearWidgets.locator('.lg-node-widget')
-    )
-
-    await test.step('Enter app mode with image input', async () => {
-      await comfyPage.appMode.enterAppModeWithInputs([['10', 'image']])
-      await expect(centerPanel).toBeVisible()
-
-      await expect(imageInput.root).toBeVisible()
-    })
-
-    await test.step('Dragging an image redirects to image input', async () => {
-      const initialImage = await imageInput.selectedItem()
-
-      await comfyPage.dragDrop.dragAndDropExternalResource({
-        fileName: 'workflow.webp',
-        filePath: './browser_tests/assets/workflowInMedia/workflow.webp',
-        preserveNativePropagation: true
-      })
-      comfyFiles.deleteAfterTest({ filename: 'workflow.webp', type: 'input' })
-
-      await expect(imageInput.selection).not.toHaveText(initialImage)
-      await expect(
-        centerPanel,
-        'A file with workflow should not open a new workflow'
-      ).toBeVisible()
-    })
-
-    await test.step('Dragging a url redirects to image input', async () => {
-      const secondImage = await imageInput.selectedItem()
-      await comfyPage.dragDrop.dragAndDropURL('/assets/images/og-image.png', {
-        preserveNativePropagation: true
-      })
-      comfyFiles.deleteAfterTest({
-        filename: 'og-image.png',
-        type: 'input'
-      })
-      await expect(imageInput.selection).not.toHaveText(secondImage)
-    })
-  })
-
-  test('Widget Interaction', async ({ comfyPage }) => {
-    await comfyPage.appMode.enterAppModeWithInputs([
-      ['3', 'seed'],
-      ['3', 'sampler_name'],
-      ['6', 'text']
-    ])
-    const seed = comfyPage.appMode.linearWidgets.getByLabel('seed', {
-      exact: true
-    })
-    const { input, incrementButton, decrementButton } =
-      comfyPage.vueNodes.getInputNumberControls(seed)
-    const initialValue = Number(await input.inputValue())
-
-    await seed.dragTo(incrementButton, { steps: 5 })
-    const intermediateValue = Number(await input.inputValue())
-    expect(intermediateValue).toBeGreaterThan(initialValue)
-
-    await seed.dragTo(decrementButton, { steps: 5 })
-    const endValue = Number(await input.inputValue())
-    expect(endValue).toBeLessThan(intermediateValue)
-
-    const sampler = comfyPage.appMode.linearWidgets.getByLabel('sampler_name', {
-      exact: true
-    })
-    await sampler.click()
-
-    await comfyPage.page.getByRole('searchbox').fill('uni')
-    await comfyPage.page.keyboard.press('ArrowDown')
-    await comfyPage.page.keyboard.press('Enter')
-    await expect(sampler).toHaveText('uni_pc')
-
-    //verify values are consistent with litegraph
-  })
-
-  test.describe('Mobile', { tag: ['@mobile'] }, () => {
-    test('panel navigation', async ({ comfyPage }) => {
-      const { mobile } = comfyPage.appMode
-      await comfyPage.appMode.enterAppModeWithInputs([['3', 'steps']])
-      await expect(mobile.view).toBeVisible()
-      await expect(mobile.navigation).toBeVisible()
-
-      await mobile.navigateTab('assets')
-      await expect(mobile.contentPanel).toHaveAccessibleName('Assets')
-
-      const buttons = await mobile.navigationTabs.all()
-      await buttons[0].dragTo(buttons[2], { steps: 5 })
-      await expect(mobile.contentPanel).toHaveAccessibleName('Outputs')
-
-      await mobile.navigateTab('run')
-      await expect(comfyPage.appMode.linearWidgets).toBeInViewport({ ratio: 1 })
-
-      const steps = comfyPage.page.getByRole('spinbutton')
-      const initialValue = Number(await steps.inputValue())
-      await mobile.tap(
-        comfyPage.page.getByRole('button', { name: 'increment' }),
-        { count: 5 }
-      )
-      await expect(steps).toHaveValue(String(initialValue + 5))
-      await mobile.tap(
-        comfyPage.page.getByRole('button', { name: 'decrement' }),
-        { count: 3 }
-      )
-
-      await expect(steps).toHaveValue(String(initialValue + 2))
-    })
-
-    test('workflow selection', async ({ comfyPage }) => {
-      const widgetNames = ['seed', 'steps', 'denoise', 'cfg']
-      for (const name of widgetNames)
-        await comfyPage.appMode.enterAppModeWithInputs([['3', name]])
-      await expect(comfyPage.appMode.mobile.workflows).toBeVisible()
-
-      const widgets = comfyPage.appMode.linearWidgets
-      await comfyPage.appMode.mobile.navigateTab('run')
-      for (let i = 0; i < widgetNames.length; i++) {
-        await comfyPage.appMode.mobile.switchWorkflow(`(${i + 2})`)
-        await expect(widgets.getByText(widgetNames[i])).toBeVisible()
-      }
-    })
-  })
-})

browser_tests/tests/appModeBuilder.spec.ts

@@ -1,121 +0,0 @@
-import {
-  comfyPageFixture as test,
-  comfyExpect as expect
-} from '@e2e/fixtures/ComfyPage'
-
-test.describe('App mode builder selection', () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.appMode.enableLinearMode()
-  })
-
-  test('Can independently select inputs of same name', async ({
-    comfyPage
-  }) => {
-    await comfyPage.settings.setSetting('Comfy.VueNodes.Enabled', true)
-    const items = comfyPage.appMode.select.inputItems
-
-    await comfyPage.vueNodes.selectNodes(['6', '7'])
-    await comfyPage.command.executeCommand('Comfy.Graph.ConvertToSubgraph')
-
-    await comfyPage.appMode.enterBuilder()
-    await comfyPage.appMode.steps.goToInputs()
-    await expect(items).toHaveCount(0)
-
-    const prompts = comfyPage.vueNodes
-      .getNodeByTitle('New Subgraph')
-      .locator('.lg-node-widget')
-    const count = await prompts.count()
-    for (let i = 0; i < count; i++) {
-      await expect(prompts.nth(i)).toBeVisible()
-      await prompts.nth(i).click()
-      await expect(items).toHaveCount(i + 1)
-    }
-  })
-
-  test('Can select outputs', async ({ comfyPage }) => {
-    await comfyPage.appMode.enterBuilder()
-    await comfyPage.appMode.steps.goToOutputs()
-
-    await comfyPage.nodeOps
-      .getNodeRefById('9')
-      .then((ref) => ref.centerOnNode())
-    const saveImage = await comfyPage.vueNodes.getNodeLocator('9')
-    await saveImage.click()
-
-    const items = comfyPage.appMode.select.inputItems
-    await expect(items).toHaveCount(1)
-  })
-
-  test('Can not select nodes with errors or notes', async ({ comfyPage }) => {
-    //Manually set error state on checkpoint loader
-    //Shouldn't be needed on ci, but has spotty reliability
-    await comfyPage.page.evaluate(() => (graph!.nodes[6].has_errors = true))
-    await comfyPage.settings.setSetting('Comfy.VueNodes.Enabled', true)
-
-    const items = comfyPage.appMode.select.inputItems
-    await comfyPage.appMode.enterBuilder()
-    await comfyPage.appMode.steps.goToInputs()
-    await expect(items).toHaveCount(0)
-
-    await comfyPage.appMode.select.selectInputWidget(
-      'Load Checkpoint',
-      'ckpt_name'
-    )
-    await expect(items).toHaveCount(0)
-
-    await comfyPage.workflow.loadWorkflow('nodes/note_nodes')
-    await comfyPage.appMode.enterBuilder()
-    await comfyPage.appMode.steps.goToInputs()
-    await expect(items).toHaveCount(0)
-
-    await comfyPage.appMode.select.selectInputWidget('Note', 'text')
-    await comfyPage.appMode.select.selectInputWidget('Markdown Note', 'text')
-
-    await expect(items).toHaveCount(0)
-  })
-
-  test('Marks canvas readOnly', async ({ comfyPage }) => {
-    await comfyPage.settings.setSetting(
-      'Comfy.NodeSearchBoxImpl',
-      'v1 (legacy)'
-    )
-
-    await comfyPage.page.mouse.dblclick(100, 100, { delay: 5 })
-    await expect(
-      comfyPage.searchBox.input,
-      'Canvas is initially editable'
-    ).toHaveCount(1)
-    await comfyPage.page.keyboard.press('Escape')
-
-    await comfyPage.appMode.enterBuilder()
-    await comfyPage.appMode.steps.goToInputs()
-
-    await comfyPage.page.mouse.dblclick(100, 100, { delay: 5 })
-    await expect(
-      comfyPage.searchBox.input,
-      'Entering builder makes the canvas readonly'
-    ).toHaveCount(0)
-
-    await comfyPage.page.keyboard.press('Space')
-    await comfyPage.page.mouse.dblclick(100, 100, { delay: 5 })
-    await expect(
-      comfyPage.searchBox.input,
-      'Canvas remains readonly after pressing space'
-    ).toHaveCount(0)
-
-    const ksampler = await comfyPage.vueNodes.getFixtureByTitle('KSampler')
-    // oxlint-disable-next-line playwright/no-force-option -- Node container has conditional pointer-events:none that blocks actionability
-    await ksampler.header.dblclick({ force: true })
-    await expect(
-      ksampler.titleEditor.input,
-      'Double clicking node titles will not initiate a rename'
-    ).toBeHidden()
-
-    await comfyPage.page.keyboard.press('Escape')
-    await comfyPage.page.mouse.dblclick(100, 100, { delay: 5 })
-    await expect(
-      comfyPage.searchBox.input,
-      'Canvas is no longer readonly after exiting'
-    ).toHaveCount(1)
-  })
-})

browser_tests/tests/assetHelper.spec.ts

@@ -133,29 +133,6 @@ test.describe('AssetHelper', () => {
       expect(data.assets[0].id).toBe(STABLE_CHECKPOINT.id)
     })
 
-    test('GET /assets filters by exclude_tags', async ({
-      comfyPage,
-      assetApi
-    }) => {
-      assetApi.configure(
-        withAsset(STABLE_INPUT_IMAGE),
-        withAsset({
-          ...STABLE_INPUT_IMAGE,
-          id: 'missing-input',
-          tags: ['input', 'missing']
-        })
-      )
-      await assetApi.mock()
-
-      const { body } = await assetApi.fetch(
-        `${comfyPage.url}/api/assets?include_tags=input,&exclude_tags= missing,`
-      )
-      const data = body as { assets: Array<{ id: string }> }
-      expect(data.assets.map((asset) => asset.id)).toEqual([
-        STABLE_INPUT_IMAGE.id
-      ])
-    })
-
     test('GET /assets/:id returns single asset or 404', async ({
       comfyPage,
       assetApi

browser_tests/tests/cloud-asset-default.spec.ts

@@ -1,20 +1,51 @@
 import { expect } from '@playwright/test'
+import type { Route } from '@playwright/test'
 
-import type { Asset } from '@comfyorg/ingest-types'
-import {
-  assetRequestIncludesTag,
-  createCloudAssetsFixture
-} from '@e2e/fixtures/assetApiFixture'
+import type { Asset, ListAssetsResponse } from '@comfyorg/ingest-types'
+import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
 import {
   STABLE_CHECKPOINT,
   STABLE_CHECKPOINT_2
 } from '@e2e/fixtures/data/assetFixtures'
 
+function makeAssetsResponse(assets: Asset[]): ListAssetsResponse {
+  return { assets, total: assets.length, has_more: false }
+}
+
 const CLOUD_ASSETS: Asset[] = [STABLE_CHECKPOINT, STABLE_CHECKPOINT_2]
 const WAITING_FOR_WIDGET_TYPE = 'waiting:type'
 const WAITING_FOR_WIDGET_VALUE = 'waiting:value'
 
-const test = createCloudAssetsFixture(CLOUD_ASSETS)
+// Stub /api/assets before the app loads. The local ComfyUI backend has no
+// /api/assets endpoint (returns 503), which poisons the assets store on
+// first load. Narrow pattern avoids intercepting static /assets/*.js bundles.
+//
+// TODO: Consider moving this stub into ComfyPage fixture for all @cloud tests.
+const test = comfyPageFixture.extend<{
+  cloudAssetRequests: string[]
+  stubCloudAssets: void
+}>({
+  cloudAssetRequests: async ({ page: _page }, use) => {
+    await use([])
+  },
+  stubCloudAssets: [
+    async ({ cloudAssetRequests, page }, use) => {
+      const pattern = /\/api\/assets(?:\?.*)?$/
+      const assetsRouteHandler = (route: Route) => {
+        cloudAssetRequests.push(route.request().url())
+        return route.fulfill({
+          status: 200,
+          contentType: 'application/json',
+          body: JSON.stringify(makeAssetsResponse(CLOUD_ASSETS))
+        })
+      }
+      await page.route(pattern, assetsRouteHandler)
+      await use()
+      await page.unroute(pattern, assetsRouteHandler)
+    },
+    { auto: true }
+  ]
+})
 
 test.describe('Asset-supported node default value', { tag: '@cloud' }, () => {
   test.afterEach(async ({ comfyPage }) => {
@@ -31,9 +62,11 @@ test.describe('Asset-supported node default value', { tag: '@cloud' }, () => {
     // new nodes resolve against the cloud asset list after the fetch.
     await expect
       .poll(() =>
-        cloudAssetRequests.some((url) =>
-          assetRequestIncludesTag(url, 'checkpoints')
-        )
+        cloudAssetRequests.some((url) => {
+          const includeTags =
+            new URL(url).searchParams.get('include_tags') ?? ''
+          return includeTags.split(',').includes('checkpoints')
+        })
       )
       .toBe(true)
 

browser_tests/tests/defaultKeybindings.spec.ts

@@ -229,9 +229,9 @@ test.describe('Default Keybindings', { tag: '@keyboard' }, () => {
       // The dialog appearing proves the keybinding was intercepted by the app.
       await comfyPage.keyboard.press('Control+s')
 
-      // The Save As dialog should appear
-      const saveDialog = comfyPage.page.getByRole('dialog')
-      await expect(saveDialog).toBeVisible()
+      // The Save As dialog should appear (p-dialog overlay)
+      const dialogOverlay = comfyPage.page.locator('.p-dialog-mask')
+      await expect(dialogOverlay).toBeVisible()
 
       // Dismiss the dialog
       await comfyPage.keyboard.press('Escape')

browser_tests/tests/dialogs/publishDialog.spec.ts

@@ -16,9 +16,9 @@ async function saveAndOpenPublishDialog(
   workflowName: string
 ): Promise<void> {
   await comfyPage.menu.topbar.saveWorkflow(workflowName)
-  const overwriteDialog = comfyPage.page
-    .getByRole('dialog')
-    .filter({ hasText: 'Overwrite' })
+  const overwriteDialog = comfyPage.page.locator(
+    '.p-dialog:has-text("Overwrite")'
+  )
   // Bounded wait: point-in-time isVisible() can miss dialogs that open
   // slightly after saveWorkflow() resolves.
   try {

browser_tests/tests/keybindingPanel.spec.ts

@@ -1,548 +0,0 @@
-import type { Locator, Page } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-
-const MULTI_BINDING_COMMAND = 'Comfy.Canvas.DeleteSelectedItems'
-const SINGLE_BINDING_COMMAND = 'Comfy.SaveWorkflow'
-const NO_BINDING_COMMAND = 'TestCommand.KeybindingPanelE2E.NoBinding'
-
-async function searchKeybindings(page: Page, query: string) {
-  await getKeybindingSearchInput(page).fill(query)
-}
-
-async function clearSearch(page: Page) {
-  await getKeybindingSearchInput(page).clear()
-}
-
-function getKeybindingSearchInput(page: Page): Locator {
-  return page.getByPlaceholder('Search Keybindings...')
-}
-
-function getCommandRow(page: Page, commandId: string): Locator {
-  return page
-    .locator('.keybinding-panel tr')
-    .filter({ has: page.locator(`[title="${commandId}"]`) })
-}
-
-function getExpansionContent(page: Page, commandId: string): Locator {
-  // PrimeVue renders the expansion row as the next sibling <tr> of the
-  // expanded row. Scoping by sibling avoids matching unrelated expanded rows.
-  return getCommandRow(page, commandId)
-    .locator('xpath=following-sibling::tr[1]')
-    .getByTestId('keybinding-expansion-content')
-}
-
-async function openContextMenu(page: Page, commandId: string) {
-  const row = getCommandRow(page, commandId)
-  await row.locator(`[title="${commandId}"]`).click({ button: 'right' })
-  await expect(
-    page.getByRole('menuitem', { name: /Change keybinding/i })
-  ).toBeVisible()
-}
-
-function getKeybindingInput(page: Page): Locator {
-  return getEditKeybindingDialog(page).locator('input[autofocus]')
-}
-
-function getEditKeybindingDialog(page: Page): Locator {
-  return page.getByRole('dialog', { name: /Modify keybinding/i })
-}
-
-function getRemoveAllKeybindingsDialog(page: Page): Locator {
-  return page.getByRole('dialog', { name: /Remove all keybindings/i })
-}
-
-function getResetAllKeybindingsDialog(page: Page): Locator {
-  return page.getByRole('dialog', { name: /Reset all keybindings/i })
-}
-
-async function pressComboOnInput(page: Page, combo: string) {
-  const input = getKeybindingInput(page)
-  await expect(input).toBeFocused()
-  await input.press(combo)
-}
-
-async function saveAndCloseKeybindingDialog(page: Page) {
-  const dialog = getEditKeybindingDialog(page)
-  await dialog.getByRole('button', { name: /Save/i }).click()
-  await expect(dialog).toBeHidden()
-}
-
-async function cancelAndCloseDialog(page: Page) {
-  const dialog = getEditKeybindingDialog(page)
-  await dialog.getByRole('button', { name: /Cancel/i }).click()
-  await expect(dialog).toBeHidden()
-}
-
-async function addKeybindingToRow(page: Page, row: Locator, combo: string) {
-  await row.getByRole('button', { name: /Add new keybinding/i }).click()
-  await pressComboOnInput(page, combo)
-  await saveAndCloseKeybindingDialog(page)
-}
-
-test.beforeEach(async ({ comfyPage }) => {
-  await registerNoBindingCommand(comfyPage)
-  await comfyPage.settingDialog.open()
-  await comfyPage.settingDialog.category('Keybinding').click()
-})
-
-test.afterEach(async ({ comfyPage }) => {
-  await comfyPage.settings.setSetting('Comfy.Keybinding.NewBindings', [])
-  await comfyPage.settings.setSetting('Comfy.Keybinding.UnsetBindings', [])
-})
-
-async function registerNoBindingCommand(comfyPage: ComfyPage) {
-  await comfyPage.page.evaluate((commandId) => {
-    const app = window.app!
-    app.registerExtension({
-      name: 'TestExtension.KeybindingPanelE2E',
-      commands: [{ id: commandId, function: () => {} }]
-    })
-  }, NO_BINDING_COMMAND)
-}
-
-test.describe('Keybinding Panel', { tag: '@keyboard' }, () => {
-  test.describe('Row Expansion', () => {
-    test('Click on row with 2+ keybindings toggles expansion', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-      const row = getCommandRow(page, MULTI_BINDING_COMMAND)
-      await expect(row).toBeVisible()
-
-      await row.locator(`[title="${MULTI_BINDING_COMMAND}"]`).click()
-
-      const expansionContent = getExpansionContent(page, MULTI_BINDING_COMMAND)
-      await expect(expansionContent).toBeVisible()
-
-      await row.locator(`[title="${MULTI_BINDING_COMMAND}"]`).click()
-      await expect(expansionContent).toBeHidden()
-    })
-
-    test('Click on row with 1 keybinding does not expand', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-      await expect(row).toBeVisible()
-
-      await row.locator(`[title="${SINGLE_BINDING_COMMAND}"]`).click()
-
-      const expansionContent = getExpansionContent(page, SINGLE_BINDING_COMMAND)
-      await expect(expansionContent).toBeHidden()
-    })
-  })
-
-  test.describe('Double-Click', () => {
-    test('Double-click row with 0 keybindings opens Add dialog', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, NO_BINDING_COMMAND)
-      const row = getCommandRow(page, NO_BINDING_COMMAND)
-      await expect(row).toBeVisible()
-
-      await row.locator(`[title="${NO_BINDING_COMMAND}"]`).dblclick()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test('Double-click row with 1 keybinding opens Edit dialog', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-      await expect(row).toBeVisible()
-
-      await row.locator(`[title="${SINGLE_BINDING_COMMAND}"]`).dblclick()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-  })
-
-  test.describe('Context Menu', () => {
-    test('Right-click row shows context menu with correct items', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      await openContextMenu(page, SINGLE_BINDING_COMMAND)
-
-      const changeItem = page.getByRole('menuitem', {
-        name: /Change keybinding/i
-      })
-      const addItem = page.getByRole('menuitem', {
-        name: /Add new keybinding/i
-      })
-      const resetItem = page.getByRole('menuitem', {
-        name: /Reset to default/i
-      })
-      const removeItem = page.getByRole('menuitem', {
-        name: /Remove keybinding/i
-      })
-
-      await expect(changeItem).toBeVisible()
-      await expect(addItem).toBeVisible()
-      await expect(resetItem).toBeVisible()
-      await expect(removeItem).toBeVisible()
-
-      await page.keyboard.press('Escape')
-    })
-
-    test("Context menu 'Add new keybinding' opens add dialog", async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      await openContextMenu(page, SINGLE_BINDING_COMMAND)
-
-      await page.getByRole('menuitem', { name: /Add new keybinding/i }).click()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test("Context menu 'Change keybinding' on single-binding command opens edit dialog", async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      await openContextMenu(page, SINGLE_BINDING_COMMAND)
-
-      await page.getByRole('menuitem', { name: /Change keybinding/i }).click()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test("Context menu 'Change keybinding' on multi-binding command expands row", async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-
-      const expansionContent = getExpansionContent(page, MULTI_BINDING_COMMAND)
-      await expect(expansionContent).toBeHidden()
-
-      await openContextMenu(page, MULTI_BINDING_COMMAND)
-
-      await page.getByRole('menuitem', { name: /Change keybinding/i }).click()
-
-      await expect(expansionContent).toBeVisible()
-    })
-
-    test("Context menu 'Remove keybinding' after adding second binding shows confirm dialog", async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      await addKeybindingToRow(page, row, 'Control+Shift+F9')
-
-      await openContextMenu(page, SINGLE_BINDING_COMMAND)
-      await page.getByRole('menuitem', { name: /Remove keybinding/i }).click()
-
-      const confirmDialog = getRemoveAllKeybindingsDialog(page)
-      await expect(confirmDialog).toBeVisible()
-      await confirmDialog.getByRole('button', { name: /Remove all/i }).click()
-
-      await expect(row.locator('td').nth(1)).toContainText('-')
-    })
-
-    test("Context menu 'Reset to default' resets modified command", async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      await addKeybindingToRow(page, row, 'Control+Shift+F10')
-
-      await openContextMenu(page, SINGLE_BINDING_COMMAND)
-      await page.getByRole('menuitem', { name: /Reset to default/i }).click()
-
-      await expect(row.getByRole('button', { name: /Reset/i })).toBeDisabled()
-    })
-
-    test('Context menu items disabled when no keybindings', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, NO_BINDING_COMMAND)
-      await openContextMenu(page, NO_BINDING_COMMAND)
-
-      const changeItem = page.getByRole('menuitem', {
-        name: /Change keybinding/i
-      })
-      const removeItem = page.getByRole('menuitem', {
-        name: /Remove keybinding/i
-      })
-
-      await expect(changeItem).toHaveAttribute('data-disabled', '')
-      await expect(removeItem).toHaveAttribute('data-disabled', '')
-
-      await page.keyboard.press('Escape')
-    })
-  })
-
-  test.describe('Action Buttons', () => {
-    test('Edit button opens edit dialog for single-binding command', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      const editButton = row.getByRole('button', { name: /^Edit$/i })
-      await expect(editButton).toBeVisible()
-      await editButton.click()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test('Add button opens add dialog', async ({ comfyPage }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      await row.getByRole('button', { name: /Add new keybinding/i }).click()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test('Reset button is disabled for unmodified commands', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      const resetButton = row.getByRole('button', { name: /Reset/i })
-      await expect(resetButton).toBeDisabled()
-    })
-
-    test('Reset button resets modified keybinding', async ({ comfyPage }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-
-      await addKeybindingToRow(page, row, 'Control+Shift+F11')
-
-      const resetButton = row.getByRole('button', { name: /Reset/i })
-      await expect(resetButton).toBeEnabled()
-
-      await resetButton.click()
-
-      await expect(resetButton).toBeDisabled()
-    })
-
-    test('Delete button is disabled for commands with 0 keybindings', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, NO_BINDING_COMMAND)
-      const row = getCommandRow(page, NO_BINDING_COMMAND)
-
-      const deleteButton = row.getByRole('button', { name: /Delete/i })
-      await expect(deleteButton).toBeDisabled()
-    })
-
-    test('Delete button removes single keybinding directly', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, NO_BINDING_COMMAND)
-      const row = getCommandRow(page, NO_BINDING_COMMAND)
-
-      await addKeybindingToRow(page, row, 'Control+Shift+F12')
-
-      const deleteButton = row.getByRole('button', { name: /Delete/i })
-      await expect(deleteButton).toBeEnabled()
-      await deleteButton.click()
-
-      await expect(row.locator('td').nth(1)).toContainText('-')
-    })
-
-    test('Delete button on command with 2+ keybindings shows confirm dialog', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-      const row = getCommandRow(page, MULTI_BINDING_COMMAND)
-
-      const deleteButton = row.getByRole('button', { name: /Delete/i })
-      await deleteButton.click()
-
-      const confirmDialog = getRemoveAllKeybindingsDialog(page)
-      await expect(confirmDialog).toBeVisible()
-
-      await confirmDialog.getByRole('button', { name: /Cancel/i }).click()
-      await expect(confirmDialog).toBeHidden()
-      await expect(row.locator('td').nth(1)).not.toContainText('-')
-    })
-  })
-
-  test.describe('Expanded Row Actions', () => {
-    test('Edit button in expanded row opens edit dialog for that binding', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-
-      await page.locator(`[title="${MULTI_BINDING_COMMAND}"]`).click()
-      const expansionContent = getExpansionContent(page, MULTI_BINDING_COMMAND)
-      await expect(expansionContent).toBeVisible()
-
-      const firstBindingRow = expansionContent
-        .getByTestId('keybinding-expansion-binding')
-        .first()
-      await firstBindingRow.getByRole('button', { name: /^Edit$/i }).click()
-
-      const input = getKeybindingInput(page)
-      await expect(input).toBeVisible()
-
-      await cancelAndCloseDialog(page)
-    })
-
-    test('Delete button in expanded row removes that binding and collapses', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-
-      await page.locator(`[title="${MULTI_BINDING_COMMAND}"]`).click()
-      const expansionContent = getExpansionContent(page, MULTI_BINDING_COMMAND)
-      await expect(expansionContent).toBeVisible()
-
-      const bindingRows = expansionContent.getByTestId(
-        'keybinding-expansion-binding'
-      )
-      await expect
-        .poll(() => bindingRows.count(), {
-          message: 'Expected at least 2 bindings'
-        })
-        .toBeGreaterThanOrEqual(2)
-      const initialBindingCount = await bindingRows.count()
-
-      await bindingRows
-        .first()
-        .getByRole('button', { name: /Remove keybinding/i })
-        .click()
-
-      if (initialBindingCount === 2) {
-        // Expansion auto-collapses when bindings drop below 2
-        await expect(expansionContent).toBeHidden()
-      } else {
-        await expect(bindingRows).toHaveCount(initialBindingCount - 1)
-      }
-    })
-  })
-
-  test.describe('Reset All', () => {
-    test('Reset All button shows confirmation and resets on confirm', async ({
-      comfyPage
-    }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const row = getCommandRow(page, SINGLE_BINDING_COMMAND)
-      await addKeybindingToRow(page, row, 'Control+Shift+F8')
-
-      await expect(row.getByRole('button', { name: /Reset/i })).toBeEnabled()
-
-      await clearSearch(page)
-
-      const resetAllButton = page
-        .locator('.keybinding-panel')
-        .getByRole('button', { name: /Reset All/i })
-      await resetAllButton.click()
-
-      const confirmDialog = getResetAllKeybindingsDialog(page)
-      await expect(confirmDialog).toBeVisible()
-      await expect(confirmDialog).toContainText(/Reset all keybindings/i)
-
-      await confirmDialog.getByRole('button', { name: /Reset All/i }).click()
-
-      await expect(comfyPage.toast.visibleToasts).toHaveCount(1)
-
-      await searchKeybindings(page, SINGLE_BINDING_COMMAND)
-      const rowAfterReset = getCommandRow(page, SINGLE_BINDING_COMMAND)
-      await expect(
-        rowAfterReset.getByRole('button', { name: /Reset/i })
-      ).toBeDisabled()
-    })
-
-    test('Reset All confirmation can be cancelled', async ({ comfyPage }) => {
-      const { page } = comfyPage
-
-      const resetAllButton = page
-        .locator('.keybinding-panel')
-        .getByRole('button', { name: /Reset All/i })
-      await resetAllButton.click()
-
-      const confirmDialog = getResetAllKeybindingsDialog(page)
-      await expect(confirmDialog).toBeVisible()
-      await confirmDialog.getByRole('button', { name: /Cancel/i }).click()
-
-      await expect(confirmDialog).toBeHidden()
-    })
-  })
-
-  test.describe('Search Filter', () => {
-    test('Typing in search clears expanded rows', async ({ comfyPage }) => {
-      const { page } = comfyPage
-
-      await searchKeybindings(page, MULTI_BINDING_COMMAND)
-
-      await page.locator(`[title="${MULTI_BINDING_COMMAND}"]`).click()
-      const expansionContent = getExpansionContent(page, MULTI_BINDING_COMMAND)
-      await expect(expansionContent).toBeVisible()
-
-      // Changing the filter triggers watch(filters, ...) which clears expansion
-      await searchKeybindings(page, MULTI_BINDING_COMMAND + ' ')
-      await expect(expansionContent).toBeHidden()
-    })
-  })
-})

browser_tests/tests/layoutSidebarSettings.spec.ts

@@ -1,127 +0,0 @@
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '@e2e/fixtures/ComfyPage'
-
-test.describe('Layout & sidebar settings', { tag: ['@settings'] }, () => {
-  test.describe('Comfy.Sidebar.Size', () => {
-    test('"small" applies small-sidebar class', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Size', 'small')
-      await expect(comfyPage.menu.sideToolbar).toContainClass('small-sidebar')
-    })
-
-    test('"normal" does not apply small-sidebar class', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Size', 'normal')
-      await expect(comfyPage.menu.sideToolbar).not.toContainClass(
-        'small-sidebar'
-      )
-    })
-  })
-
-  test.describe('Comfy.Sidebar.Style', () => {
-    // `isConnected` overrides the Style setting when the toolbar overflows;
-    // small (48px) items keep content under the default viewport so Style
-    // actually drives rendering.
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Size', 'small')
-    })
-
-    test('"connected" applies connected-sidebar class', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Style', 'connected')
-      await expect(comfyPage.menu.sideToolbar).toContainClass(
-        'connected-sidebar'
-      )
-      await expect(comfyPage.menu.sideToolbar).not.toContainClass(
-        'floating-sidebar'
-      )
-    })
-
-    test('"floating" applies floating-sidebar class', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Style', 'floating')
-      await expect(comfyPage.menu.sideToolbar).toContainClass(
-        'floating-sidebar'
-      )
-      await expect(comfyPage.menu.sideToolbar).not.toContainClass(
-        'connected-sidebar'
-      )
-    })
-
-    test('"floating" + Size "normal" is overridden to connected by overflow', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Size', 'normal')
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Style', 'floating')
-      await expect(comfyPage.menu.sideToolbar).toContainClass(
-        'connected-sidebar'
-      )
-      await expect(comfyPage.menu.sideToolbar).toContainClass(
-        'overflowing-sidebar'
-      )
-    })
-
-    test('"floating" + Size "normal" renders floating in a viewport tall enough to fit', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Size', 'normal')
-      await comfyPage.settings.setSetting('Comfy.Sidebar.Style', 'floating')
-      await comfyPage.page.setViewportSize({ width: 1280, height: 1500 })
-      await expect(comfyPage.menu.sideToolbar).toContainClass(
-        'floating-sidebar'
-      )
-      await expect(comfyPage.menu.sideToolbar).not.toContainClass(
-        'overflowing-sidebar'
-      )
-    })
-  })
-
-  test.describe('Comfy.UI.TabBarLayout', () => {
-    test('"Default" renders integrated tab bar actions container', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.UI.TabBarLayout', 'Default')
-      await expect(comfyPage.menu.topbar.integratedTabBarActions).toBeAttached()
-    })
-
-    test('"Legacy" does not render integrated tab bar actions container', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.UI.TabBarLayout', 'Legacy')
-      await expect(comfyPage.menu.topbar.integratedTabBarActions).toHaveCount(0)
-    })
-  })
-
-  test.describe('Comfy.TreeExplorer.ItemPadding', () => {
-    // The setting writes a CSS var consumed by .p-tree-node-content,
-    // which only renders in the legacy PrimeVue Tree.
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.NodeLibrary.NewDesign', false)
-      await comfyPage.menu.nodeLibraryTab.open()
-    })
-
-    test('low padding (0px) is applied to tree node content', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.TreeExplorer.ItemPadding', 0)
-      await expect(
-        comfyPage.menu.nodeLibraryTab.nodeLibraryTree
-          .locator('.p-tree-node-content')
-          .first()
-      ).toHaveCSS('padding', '0px')
-    })
-
-    test('high padding (8px) is applied to tree node content', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.TreeExplorer.ItemPadding', 8)
-      await expect(
-        comfyPage.menu.nodeLibraryTab.nodeLibraryTree
-          .locator('.p-tree-node-content')
-          .first()
-      ).toHaveCSS('padding', '8px')
-    })
-  })
-})

browser_tests/tests/load3d/load3dLod.spec.ts

@@ -1,32 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { load3dTest as test } from '@e2e/fixtures/helpers/Load3DFixtures'
-
-test.describe('Load3D LOD', () => {
-  test(
-    'canvas pixel dimensions scale with ComfyUI canvas zoom level',
-    { tag: '@smoke' },
-    async ({ comfyPage, load3d }) => {
-      await expect(load3d.canvas).toBeVisible()
-
-      await expect
-        .poll(() => load3d.canvas.evaluate((el: HTMLCanvasElement) => el.width))
-        .toBeGreaterThan(0)
-
-      const initialWidth = await load3d.canvas.evaluate(
-        (el: HTMLCanvasElement) => el.width
-      )
-
-      await comfyPage.page.evaluate(() => {
-        const node = window.app!.graph!.nodes[0]
-        window.app!.canvas.ds.scale = 2.0
-        node.onResize?.(node.size)
-      })
-      await comfyPage.nextFrame()
-
-      await expect
-        .poll(() => load3d.canvas.evaluate((el: HTMLCanvasElement) => el.width))
-        .toBeGreaterThan(initialWidth)
-    }
-  )
-})

browser_tests/tests/maskEditor.spec.ts

@@ -1,13 +1,117 @@
+import type { Page } from '@playwright/test'
 import { expect } from '@playwright/test'
 
-import { maskEditorTest as test } from '@e2e/fixtures/helpers/MaskEditorHelper'
+import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
+import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 
 test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
+  async function loadImageOnNode(comfyPage: ComfyPage) {
+    await comfyPage.workflow.loadWorkflow('widgets/load_image_widget')
+
+    const loadImageNode = (
+      await comfyPage.nodeOps.getNodeRefsByType('LoadImage')
+    )[0]
+    const { x, y } = await loadImageNode.getPosition()
+
+    await comfyPage.dragDrop.dragAndDropFile('image64x64.webp', {
+      dropPosition: { x, y }
+    })
+
+    const imagePreview = comfyPage.page.locator('.image-preview')
+    await expect(imagePreview).toBeVisible()
+    await expect(imagePreview.locator('img')).toBeVisible()
+    await expect(imagePreview).toContainText('x')
+
+    return {
+      imagePreview,
+      nodeId: String(loadImageNode.id)
+    }
+  }
+
+  async function openMaskEditorDialog(comfyPage: ComfyPage) {
+    const { imagePreview } = await loadImageOnNode(comfyPage)
+
+    await imagePreview.getByRole('region').hover()
+    await comfyPage.page.getByLabel('Edit or mask image').click()
+
+    const dialog = comfyPage.page.locator('.mask-editor-dialog')
+    await expect(dialog).toBeVisible()
+    await expect(
+      dialog.getByRole('heading', { name: 'Mask Editor' })
+    ).toBeVisible()
+
+    const canvasContainer = dialog.locator('#maskEditorCanvasContainer')
+    await expect(canvasContainer).toBeVisible()
+    await expect(canvasContainer.locator('canvas')).toHaveCount(4)
+
+    return dialog
+  }
+
+  async function getMaskCanvasPixelData(page: Page) {
+    return page.evaluate(() => {
+      const canvases = document.querySelectorAll(
+        '#maskEditorCanvasContainer canvas'
+      )
+      // The mask canvas is the 3rd canvas (index 2, z-30)
+      const maskCanvas = canvases[2] as HTMLCanvasElement
+      if (!maskCanvas) return null
+      const ctx = maskCanvas.getContext('2d')
+      if (!ctx) return null
+      const data = ctx.getImageData(0, 0, maskCanvas.width, maskCanvas.height)
+      let nonTransparentPixels = 0
+      for (let i = 3; i < data.data.length; i += 4) {
+        if (data.data[i] > 0) nonTransparentPixels++
+      }
+      return { nonTransparentPixels, totalPixels: data.data.length / 4 }
+    })
+  }
+
+  function pollMaskPixelCount(page: Page): Promise<number> {
+    return getMaskCanvasPixelData(page).then(
+      (d) => d?.nonTransparentPixels ?? 0
+    )
+  }
+
+  async function drawStrokeOnPointerZone(
+    page: Page,
+    dialog: ReturnType<typeof page.locator>
+  ) {
+    const pointerZone = dialog.locator(
+      '.maskEditor-ui-container [class*="w-[calc"]'
+    )
+    await expect(pointerZone).toBeVisible()
+
+    const box = await pointerZone.boundingBox()
+    if (!box) throw new Error('Pointer zone bounding box not found')
+
+    const startX = box.x + box.width * 0.3
+    const startY = box.y + box.height * 0.5
+    const endX = box.x + box.width * 0.7
+    const endY = box.y + box.height * 0.5
+
+    await page.mouse.move(startX, startY)
+    await page.mouse.down()
+    await page.mouse.move(endX, endY, { steps: 10 })
+    await page.mouse.up()
+
+    return { startX, startY, endX, endY, box }
+  }
+
+  async function drawStrokeAndExpectPixels(
+    comfyPage: ComfyPage,
+    dialog: ReturnType<typeof comfyPage.page.locator>
+  ) {
+    await drawStrokeOnPointerZone(comfyPage.page, dialog)
+    await expect
+      .poll(() => pollMaskPixelCount(comfyPage.page))
+      .toBeGreaterThan(0)
+  }
+
   test(
     'opens mask editor from image preview button',
     { tag: ['@smoke', '@screenshot'] },
-    async ({ comfyPage, maskEditor }) => {
-      const { imagePreview } = await maskEditor.loadImageOnNode()
+    async ({ comfyPage }) => {
+      const { imagePreview } = await loadImageOnNode(comfyPage)
 
       // Hover over the image panel to reveal action buttons
       await imagePreview.getByRole('region').hover()
@@ -35,8 +139,8 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
   test(
     'opens mask editor from context menu',
     { tag: ['@smoke', '@screenshot'] },
-    async ({ comfyPage, maskEditor }) => {
-      const { nodeId } = await maskEditor.loadImageOnNode()
+    async ({ comfyPage }) => {
+      const { nodeId } = await loadImageOnNode(comfyPage)
 
       const nodeHeader = comfyPage.vueNodes
         .getNodeLocator(nodeId)
@@ -62,61 +166,63 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
     }
   )
 
-  test('draws a brush stroke on the mask canvas', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('draws a brush stroke on the mask canvas', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    const dataBefore = await maskEditor.getCanvasPixelData(2)
+    const dataBefore = await getMaskCanvasPixelData(comfyPage.page)
     expect(dataBefore).not.toBeNull()
     expect(dataBefore!.nonTransparentPixels).toBe(0)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
   })
 
-  test('undo reverts a brush stroke', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('undo reverts a brush stroke', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
 
     const undoButton = dialog.locator('button[title="Undo"]')
     await expect(undoButton).toBeVisible()
     await undoButton.click()
 
-    await expect.poll(() => maskEditor.pollMaskPixelCount()).toBe(0)
+    await expect.poll(() => pollMaskPixelCount(comfyPage.page)).toBe(0)
   })
 
-  test('redo restores an undone stroke', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('redo restores an undone stroke', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
 
     const undoButton = dialog.locator('button[title="Undo"]')
     await undoButton.click()
 
-    await expect.poll(() => maskEditor.pollMaskPixelCount()).toBe(0)
+    await expect.poll(() => pollMaskPixelCount(comfyPage.page)).toBe(0)
 
     const redoButton = dialog.locator('button[title="Redo"]')
     await expect(redoButton).toBeVisible()
     await redoButton.click()
 
-    await expect.poll(() => maskEditor.pollMaskPixelCount()).toBeGreaterThan(0)
+    await expect
+      .poll(() => pollMaskPixelCount(comfyPage.page))
+      .toBeGreaterThan(0)
   })
 
-  test('clear button removes all mask content', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('clear button removes all mask content', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
 
     const clearButton = dialog.getByRole('button', { name: 'Clear' })
     await expect(clearButton).toBeVisible()
     await clearButton.click()
 
-    await expect.poll(() => maskEditor.pollMaskPixelCount()).toBe(0)
+    await expect.poll(() => pollMaskPixelCount(comfyPage.page)).toBe(0)
   })
 
-  test('cancel closes the dialog without saving', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('cancel closes the dialog without saving', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
 
     const cancelButton = dialog.getByRole('button', { name: 'Cancel' })
     await cancelButton.click()
@@ -124,10 +230,10 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
     await expect(dialog).toBeHidden()
   })
 
-  test('invert button inverts the mask', async ({ maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('invert button inverts the mask', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    const dataBefore = await maskEditor.getCanvasPixelData(2)
+    const dataBefore = await getMaskCanvasPixelData(comfyPage.page)
     expect(dataBefore).not.toBeNull()
     const pixelsBefore = dataBefore!.nonTransparentPixels
 
@@ -136,29 +242,26 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
     await invertButton.click()
 
     await expect
-      .poll(() => maskEditor.pollMaskPixelCount())
+      .poll(() => pollMaskPixelCount(comfyPage.page))
       .toBeGreaterThan(pixelsBefore)
   })
 
-  test('keyboard shortcut Ctrl+Z triggers undo', async ({
-    comfyPage,
-    maskEditor
-  }) => {
-    const dialog = await maskEditor.openDialog()
+  test('keyboard shortcut Ctrl+Z triggers undo', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
-    await maskEditor.drawStrokeAndExpectPixels(dialog)
+    await drawStrokeAndExpectPixels(comfyPage, dialog)
 
     const modifier = process.platform === 'darwin' ? 'Meta+z' : 'Control+z'
     await comfyPage.page.keyboard.press(modifier)
 
-    await expect.poll(() => maskEditor.pollMaskPixelCount()).toBe(0)
+    await expect.poll(() => pollMaskPixelCount(comfyPage.page)).toBe(0)
   })
 
   test(
     'tool panel shows all five tools',
     { tag: ['@smoke'] },
-    async ({ maskEditor }) => {
-      const dialog = await maskEditor.openDialog()
+    async ({ comfyPage }) => {
+      const dialog = await openMaskEditorDialog(comfyPage)
 
       const toolPanel = dialog.locator('.maskEditor-ui-container')
       await expect(toolPanel).toBeVisible()
@@ -176,9 +279,9 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
   )
 
   test('switching tools updates the selected indicator', async ({
-    maskEditor
+    comfyPage
   }) => {
-    const dialog = await maskEditor.openDialog()
+    const dialog = await openMaskEditorDialog(comfyPage)
 
     const toolEntries = dialog.locator('.maskEditor_toolPanelContainer')
     await expect(toolEntries).toHaveCount(5)
@@ -197,9 +300,9 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
   })
 
   test('brush settings panel is visible with thickness controls', async ({
-    maskEditor
+    comfyPage
   }) => {
-    const dialog = await maskEditor.openDialog()
+    const dialog = await openMaskEditorDialog(comfyPage)
 
     // The side panel should show brush settings by default
     const thicknessLabel = dialog.getByText('Thickness')
@@ -212,11 +315,8 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
     await expect(hardnessLabel).toBeVisible()
   })
 
-  test('save uploads all layers and closes dialog', async ({
-    comfyPage,
-    maskEditor
-  }) => {
-    const dialog = await maskEditor.openDialog()
+  test('save uploads all layers and closes dialog', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
     let maskUploadCount = 0
     let imageUploadCount = 0
@@ -259,8 +359,8 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
     ).toBeGreaterThan(0)
   })
 
-  test('save failure keeps dialog open', async ({ comfyPage, maskEditor }) => {
-    const dialog = await maskEditor.openDialog()
+  test('save failure keeps dialog open', async ({ comfyPage }) => {
+    const dialog = await openMaskEditorDialog(comfyPage)
 
     // Fail all upload routes
     await comfyPage.page.route('**/upload/mask', (route) =>
@@ -280,23 +380,23 @@ test.describe('Mask Editor', { tag: '@vue-nodes' }, () => {
   test(
     'eraser tool removes mask content',
     { tag: ['@screenshot'] },
-    async ({ maskEditor }) => {
-      const dialog = await maskEditor.openDialog()
+    async ({ comfyPage }) => {
+      const dialog = await openMaskEditorDialog(comfyPage)
 
       // Draw a stroke with the mask pen (default tool)
-      await maskEditor.drawStrokeAndExpectPixels(dialog)
+      await drawStrokeAndExpectPixels(comfyPage, dialog)
 
-      const pixelsAfterDraw = await maskEditor.getCanvasPixelData(2)
+      const pixelsAfterDraw = await getMaskCanvasPixelData(comfyPage.page)
 
       // Switch to eraser tool (3rd tool, index 2)
       const toolEntries = dialog.locator('.maskEditor_toolPanelContainer')
       await toolEntries.nth(2).click()
 
       // Draw over the same area with the eraser
-      await maskEditor.drawStrokeOnPointerZone(dialog)
+      await drawStrokeOnPointerZone(comfyPage.page, dialog)
 
       await expect
-        .poll(() => maskEditor.pollMaskPixelCount())
+        .poll(() => pollMaskPixelCount(comfyPage.page))
         .toBeLessThan(pixelsAfterDraw!.nonTransparentPixels)
     }
   )

browser_tests/tests/maskEditorBrushLayers.spec.ts

@@ -1,100 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { maskEditorTest as test } from '@e2e/fixtures/helpers/MaskEditorHelper'
-
-const RGB_PAINT_TOOL_INDEX = 1 // RGB / color paint tool
-const ERASER_TOOL_INDEX = 2 // Eraser tool
-
-test.describe(
-  'Mask Editor brush adjustment and layer management',
-  { tag: '@vue-nodes' },
-  () => {
-    test.describe('Brush settings interaction', () => {
-      test('Adjusting brush thickness slider changes stroke output', async ({
-        comfyPage,
-        maskEditor
-      }) => {
-        const dialog = await maskEditor.openDialog()
-        const thicknessInput = maskEditor.brushInput(dialog, 'thickness')
-
-        // Thin brush
-        await thicknessInput.fill('2')
-        await expect(thicknessInput).toHaveValue('2')
-
-        await maskEditor.drawStrokeOnPointerZone(dialog)
-        await expect
-          .poll(() => maskEditor.pollMaskPixelCount())
-          .toBeGreaterThan(0)
-        const thinPixels = await maskEditor.pollMaskPixelCount()
-
-        await comfyPage.page.keyboard.press('Control+z')
-        await expect.poll(() => maskEditor.pollMaskPixelCount()).toBe(0)
-
-        // Thick brush
-        await thicknessInput.fill('200')
-        await expect(thicknessInput).toHaveValue('200')
-
-        await maskEditor.drawStrokeOnPointerZone(dialog)
-        await expect
-          .poll(() => maskEditor.pollMaskPixelCount())
-          .toBeGreaterThan(thinPixels)
-      })
-    })
-
-    test.describe('Layer management', () => {
-      test('Drawing on different tools produces independent mask data', async ({
-        maskEditor
-      }) => {
-        const dialog = await maskEditor.openDialog()
-
-        await maskEditor.drawStrokeOnPointerZone(dialog)
-        await expect
-          .poll(() => maskEditor.pollMaskPixelCount())
-          .toBeGreaterThan(0)
-        const maskSnapshotAfterPen = await maskEditor.getCanvasSnapshot(2)
-
-        const toolEntries = dialog.locator('.maskEditor_toolPanelContainer')
-        await expect(toolEntries).toHaveCount(5)
-        await toolEntries.nth(RGB_PAINT_TOOL_INDEX).click()
-        await expect(toolEntries.nth(RGB_PAINT_TOOL_INDEX)).toHaveClass(
-          /Selected/
-        )
-
-        await maskEditor.drawStrokeOnPointerZone(dialog)
-        await expect
-          .poll(() => maskEditor.pollRgbPixelCount())
-          .toBeGreaterThan(0)
-
-        await expect
-          .poll(() => maskEditor.getCanvasSnapshot(2))
-          .toBe(maskSnapshotAfterPen)
-      })
-
-      test("Switching between tools preserves previous tool's mask data", async ({
-        maskEditor
-      }) => {
-        const dialog = await maskEditor.openDialog()
-
-        await maskEditor.drawStrokeOnPointerZone(dialog)
-        await expect
-          .poll(() => maskEditor.pollMaskPixelCount())
-          .toBeGreaterThan(0)
-
-        const maskSnapshot = await maskEditor.getCanvasSnapshot(2)
-
-        const toolEntries = dialog.locator('.maskEditor_toolPanelContainer')
-        await expect(toolEntries).toHaveCount(5)
-
-        await toolEntries.nth(ERASER_TOOL_INDEX).click()
-        await expect(toolEntries.nth(ERASER_TOOL_INDEX)).toHaveClass(/Selected/)
-
-        await toolEntries.nth(0).click()
-        await expect(toolEntries.nth(0)).toHaveClass(/Selected/)
-
-        await expect
-          .poll(() => maskEditor.getCanvasSnapshot(2))
-          .toBe(maskSnapshot)
-      })
-    })
-  }
-)

browser_tests/tests/metadataWorkflowImport.spec.ts

@@ -1,62 +0,0 @@
-import {
-  comfyPageFixture as test,
-  comfyExpect as expect
-} from '@e2e/fixtures/ComfyPage'
-import { metadataFixturePath } from '@e2e/fixtures/utils/paths'
-
-type MetadataFixture = {
-  fileName: string
-  parser: string
-}
-
-// Each fixture embeds the same single-KSampler workflow (see
-// scripts/generate-embedded-metadata-test-files.py), exercising a different
-// parser in src/scripts/metadata/. Dropping the file should import that
-// workflow.
-const FIXTURES: readonly MetadataFixture[] = [
-  { fileName: 'with_metadata.png', parser: 'png' },
-  { fileName: 'with_metadata.avif', parser: 'avif' },
-  { fileName: 'with_metadata.webp', parser: 'webp' },
-  { fileName: 'with_metadata_exif_prefix.webp', parser: 'webp (exif prefix)' },
-  { fileName: 'with_metadata.flac', parser: 'flac' },
-  { fileName: 'with_metadata.mp3', parser: 'mp3' },
-  { fileName: 'with_metadata.opus', parser: 'ogg' },
-  { fileName: 'with_metadata.mp4', parser: 'isobmff' },
-  { fileName: 'with_metadata.webm', parser: 'ebml (webm)' }
-] as const
-
-test.describe(
-  'Metadata drop-to-load workflow import',
-  { tag: ['@workflow'] },
-  () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.nodeOps.clearGraph()
-      await expect.poll(() => comfyPage.nodeOps.getGraphNodesCount()).toBe(0)
-    })
-
-    for (const { fileName, parser } of FIXTURES) {
-      test(`loads embedded workflow from ${fileName} (${parser})`, async ({
-        comfyPage
-      }) => {
-        await test.step(`drop ${fileName} on canvas`, async () => {
-          await comfyPage.dragDrop.dragAndDropFilePath(
-            metadataFixturePath(fileName)
-          )
-        })
-
-        await test.step('graph contains only the embedded KSampler', async () => {
-          await expect
-            .poll(() => comfyPage.nodeOps.getGraphNodesCount())
-            .toBe(1)
-
-          const ksamplers =
-            await comfyPage.nodeOps.getNodeRefsByType('KSampler')
-          expect(
-            ksamplers,
-            'exactly one KSampler should have been loaded from the fixture'
-          ).toHaveLength(1)
-        })
-      })
-    }
-  }
-)

browser_tests/tests/painter.spec.ts

@@ -692,27 +692,19 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
     })
   })
 
-  test('Controls stack label above widget in compact mode', async ({
+  test('Controls collapse to single column in compact mode', async ({
     comfyPage
   }) => {
     const painterWidget = comfyPage.vueNodes
       .getNodeLocator('1')
       .locator('.widget-expands')
     const toolLabel = painterWidget.getByText('Tool', { exact: true })
-    const brushButton = painterWidget.getByText('Brush', { exact: true })
 
     await expect(
       toolLabel,
-      'tool label should be visible in wide layout'
+      'tool label should be visible in two-column layout'
     ).toBeVisible()
 
-    const wideLabelBox = await toolLabel.boundingBox()
-    const wideBrushBox = await brushButton.boundingBox()
-    expect(
-      wideLabelBox && wideBrushBox && wideLabelBox.x < wideBrushBox.x,
-      'label should sit to the left of the brush button in wide layout'
-    ).toBe(true)
-
     await comfyPage.page.evaluate(() => {
       const graph = window.graph as TestGraphAccess | undefined
       const node = graph?._nodes_by_id?.['1']
@@ -724,22 +716,8 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
 
     await expect(
       toolLabel,
-      'tool label should remain visible in compact layout'
-    ).toBeVisible()
-
-    await expect
-      .poll(
-        async () => {
-          const labelBox = await toolLabel.boundingBox()
-          const brushBox = await brushButton.boundingBox()
-          if (!labelBox || !brushBox) return false
-          return labelBox.y + labelBox.height <= brushBox.y
-        },
-        {
-          message: 'label should stack above the brush button in compact layout'
-        }
-      )
-      .toBe(true)
+      'tool label should hide in compact single-column layout'
+    ).toBeHidden()
   })
 
   test('Multiple sequential strokes at different positions all accumulate', async ({

browser_tests/tests/performance.spec.ts

@@ -351,45 +351,6 @@ test.describe('Performance', { tag: ['@perf'] }, () => {
     })
   })
 
-  test(
-    'subgraph transition (enter and exit)',
-    { tag: ['@vue-nodes'] },
-    async ({ comfyPage }, testInfo) => {
-      // Heaviest perf test: loads an 80-node subgraph and pays ~30s/repeat.
-      // The signal is dominated by N=80 mount cost, so a single sample per
-      // CI invocation is sufficient — early-return on subsequent repeats.
-      if (testInfo.repeatEachIndex > 0) return
-
-      // Load workflow with a subgraph containing 80 interior nodes.
-      // Entering the subgraph unmounts root nodes and mounts all 80 interior
-      // nodes synchronously — this is the bottleneck we're measuring.
-      await comfyPage.workflow.loadWorkflow('subgraphs/large-subgraph-80-nodes')
-
-      await comfyPage.idleFrames(30)
-
-      await comfyPage.vueNodes.enterSubgraph()
-      await comfyPage.vueNodes.waitForNodes(80)
-      await comfyPage.idleFrames(30)
-
-      // Exit back to root graph before measuring a fresh enter/exit cycle
-      await comfyPage.subgraph.exitViaBreadcrumb()
-      await comfyPage.idleFrames(10)
-
-      // Start measuring the enter transition
-      await comfyPage.perf.startMeasuring()
-
-      await comfyPage.vueNodes.enterSubgraph()
-      await comfyPage.vueNodes.waitForNodes(80)
-      await comfyPage.idleFrames(30)
-
-      const m = await comfyPage.perf.stopMeasuring('subgraph-transition-enter')
-      recordMeasurement(m)
-      console.log(
-        `Subgraph enter (80 nodes): ${m.taskDurationMs.toFixed(0)}ms task, ${m.layouts} layouts, TBT=${m.totalBlockingTimeMs.toFixed(0)}ms`
-      )
-    }
-  )
-
   test('workflow execution', async ({ comfyPage }) => {
     // Uses lightweight PrimitiveString → PreviewAny workflow (no GPU needed)
     await comfyPage.workflow.loadWorkflow('execution/partial_execution')

browser_tests/tests/previewAsText.spec.ts

@@ -1,42 +0,0 @@
-import {
-  comfyPageFixture as test,
-  comfyExpect as expect
-} from '../fixtures/ComfyPage'
-
-test.describe('Preview as Text node', () => {
-  test('does not include preview widget values in the API prompt', async ({
-    comfyPage
-  }) => {
-    await comfyPage.page.evaluate(() => {
-      const node = window.LiteGraph!.createNode('PreviewAny')!
-      node.pos = [500, 200]
-      window.app!.graph.add(node)
-    })
-
-    // Simulate a previous execution: backend returned text and the frontend
-    // populated the preview widget values. The next prompt submission must
-    // NOT echo those values back as inputs (which would change the cache
-    // signature and trigger a redundant re-execution).
-    await comfyPage.page.evaluate(() => {
-      const node = window.app!.graph.nodes.find((n) => n.type === 'PreviewAny')!
-      for (const widget of node.widgets ?? []) {
-        if (widget.name?.startsWith('preview_')) {
-          widget.value = 'rendered preview content from previous execution'
-        }
-      }
-    })
-
-    const apiWorkflow = await comfyPage.workflow.getExportedWorkflow({
-      api: true
-    })
-
-    const previewEntry = Object.values(apiWorkflow).find(
-      (n) => n.class_type === 'PreviewAny'
-    )
-    expect(previewEntry).toBeDefined()
-
-    expect(previewEntry!.inputs).not.toHaveProperty('preview_markdown')
-    expect(previewEntry!.inputs).not.toHaveProperty('preview_text')
-    expect(previewEntry!.inputs).not.toHaveProperty('previewMode')
-  })
-})

browser_tests/tests/propertiesPanel/errorsTabCloudMissingModels.spec.ts

@@ -1,93 +0,0 @@
-import { expect } from '@playwright/test'
-
-import type { Asset } from '@comfyorg/ingest-types'
-import {
-  countAssetRequestsByTag,
-  createCloudAssetsFixture
-} from '@e2e/fixtures/assetApiFixture'
-import { TestIds } from '@e2e/fixtures/selectors'
-import { PropertiesPanelHelper } from '@e2e/tests/propertiesPanel/PropertiesPanelHelper'
-
-const WORKFLOW = 'missing/nested_subgraph_installed_model'
-const OUTER_SUBGRAPH_NODE_ID = '205'
-const LOTUS_MODEL_NAME = 'lotus-depth-d-v1-1.safetensors'
-
-const LOTUS_DIFFUSION_MODEL: Asset = {
-  id: 'test-lotus-depth-d-v1-1',
-  name: LOTUS_MODEL_NAME,
-  asset_hash:
-    'blake3:0000000000000000000000000000000000000000000000000000000000000203',
-  size: 1_024,
-  mime_type: 'application/octet-stream',
-  tags: ['models', 'diffusion_models'],
-  created_at: '2026-05-05T00:00:00Z',
-  updated_at: '2026-05-05T00:00:00Z',
-  last_access_time: '2026-05-05T00:00:00Z',
-  user_metadata: {
-    filename: LOTUS_MODEL_NAME
-  }
-}
-
-const test = createCloudAssetsFixture([LOTUS_DIFFUSION_MODEL])
-
-test.describe(
-  'Errors tab - Cloud missing models',
-  { tag: ['@cloud', '@vue-nodes'] },
-  () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.RightSidePanel.ShowErrorsTab',
-        true
-      )
-    })
-
-    test('keeps installed models resolved after returning from a nested subgraph', async ({
-      cloudAssetRequests,
-      comfyPage
-    }) => {
-      await comfyPage.workflow.loadWorkflow(WORKFLOW)
-
-      const panel = new PropertiesPanelHelper(comfyPage.page)
-      const errorOverlay = comfyPage.page.getByTestId(
-        TestIds.dialogs.errorOverlay
-      )
-      const errorsTab = panel.root.getByTestId(
-        TestIds.propertiesPanel.errorsTab
-      )
-
-      await expect
-        .poll(
-          () => countAssetRequestsByTag(cloudAssetRequests, 'diffusion_models'),
-          { timeout: 10_000 }
-        )
-        .toBeGreaterThan(0)
-      await expect(errorOverlay).toBeHidden()
-      await panel.open(comfyPage.actionbar.propertiesButton)
-      await expect(errorsTab).toBeHidden()
-      await panel.close()
-
-      await comfyPage.vueNodes.enterSubgraph(OUTER_SUBGRAPH_NODE_ID)
-      await expect.poll(() => comfyPage.subgraph.isInSubgraph()).toBe(true)
-      await expect(errorOverlay).toBeHidden()
-
-      const requestCountBeforeRootReturn = countAssetRequestsByTag(
-        cloudAssetRequests,
-        'diffusion_models'
-      )
-
-      await comfyPage.subgraph.exitViaBreadcrumb()
-      await panel.open(comfyPage.actionbar.propertiesButton)
-
-      await expect
-        .poll(
-          () =>
-            countAssetRequestsByTag(cloudAssetRequests, 'diffusion_models') >
-            requestCountBeforeRootReturn,
-          { timeout: 10_000 }
-        )
-        .toBe(true)
-
-      await expect(errorsTab).toBeHidden()
-    })
-  }
-)

browser_tests/tests/queue/queueOverlay.spec.ts

@@ -1,19 +1,13 @@
-import { expect, mergeTests } from '@playwright/test'
-import type { JobEntry } from '@comfyorg/ingest-types'
+import { expect } from '@playwright/test'
 
-import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
-import { jobsApiMockFixture } from '@e2e/fixtures/jobsApiMockFixture'
-import {
-  createMockJob,
-  createMockJobRecords
-} from '@e2e/fixtures/utils/jobFixtures'
+import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
+import { createMockJob } from '@e2e/fixtures/helpers/AssetsHelper'
 import { TestIds } from '@e2e/fixtures/selectors'
-
-const test = mergeTests(comfyPageFixture, jobsApiMockFixture)
+import type { RawJobListItem } from '@/platform/remote/comfyui/jobs/jobTypes'
 
 const now = Date.now()
 
-const MOCK_JOBS: JobEntry[] = [
+const MOCK_JOBS: RawJobListItem[] = [
   createMockJob({
     id: 'job-completed-1',
     status: 'completed',
@@ -37,25 +31,20 @@ const MOCK_JOBS: JobEntry[] = [
     execution_start_time: now - 30_000,
     execution_end_time: now - 28_000,
     outputs_count: 0
-  }),
-  createMockJob({
-    id: 'job-failed-bottom',
-    status: 'failed',
-    create_time: now - 180_000,
-    execution_start_time: now - 180_000,
-    execution_end_time: now - 178_000,
-    outputs_count: 0
   })
 ]
 
 test.describe('Queue overlay', () => {
-  test.beforeEach(async ({ comfyPage, jobsApi }) => {
-    await jobsApi.mockJobs(createMockJobRecords(MOCK_JOBS))
-    await comfyPage.settings.setSetting('Comfy.Minimap.Visible', false)
+  test.beforeEach(async ({ comfyPage }) => {
+    await comfyPage.assets.mockOutputHistory(MOCK_JOBS)
     await comfyPage.settings.setSetting('Comfy.Queue.QPOV2', false)
     await comfyPage.setup()
   })
 
+  test.afterEach(async ({ comfyPage }) => {
+    await comfyPage.assets.clearMocks()
+  })
+
   test('Toggle button opens expanded queue overlay', async ({ comfyPage }) => {
     const toggle = comfyPage.page.getByTestId(TestIds.queue.overlayToggle)
     await toggle.click()
@@ -117,64 +106,4 @@ test.describe('Queue overlay', () => {
 
     await expect(comfyPage.page.locator('[data-job-id]').first()).toBeHidden()
   })
-
-  test('Job details popover stays inside the viewport for bottom rows', async ({
-    comfyPage
-  }) => {
-    await comfyPage.page.setViewportSize({ width: 1280, height: 420 })
-
-    const toggle = comfyPage.page.getByTestId(TestIds.queue.overlayToggle)
-    await toggle.click()
-
-    const bottomJob = comfyPage.page.locator(
-      '[data-job-id="job-failed-bottom"]'
-    )
-    await expect(bottomJob).toBeVisible()
-    await bottomJob.scrollIntoViewIfNeeded()
-    await expect(bottomJob).toBeVisible()
-
-    const viewportSize = comfyPage.page.viewportSize()
-    if (!viewportSize) throw new Error('Viewport must be available')
-
-    const rowBox = await bottomJob.boundingBox()
-    if (!rowBox) throw new Error('Bottom job row should be measurable')
-    expect(
-      rowBox.y + rowBox.height,
-      'Test row should be low enough to exercise bottom-edge collision handling'
-    ).toBeGreaterThan(viewportSize.height * 0.55)
-    await expect
-      .poll(async () =>
-        bottomJob.evaluate((element) => {
-          const rect = element.getBoundingClientRect()
-          const hitTarget = document.elementFromPoint(
-            rect.x + rect.width / 2,
-            rect.y + rect.height / 2
-          )
-          return hitTarget ? element.contains(hitTarget) : false
-        })
-      )
-      .toBe(true)
-
-    await comfyPage.page.mouse.move(0, 0)
-    await comfyPage.page.mouse.move(
-      rowBox.x + rowBox.width / 2,
-      rowBox.y + rowBox.height / 2,
-      { steps: 5 }
-    )
-
-    const popover = comfyPage.page.getByTestId(TestIds.queue.jobDetailsPopover)
-    await expect(popover).toBeVisible()
-
-    await expect
-      .poll(async () => {
-        const popoverBox = await popover.boundingBox()
-        if (!popoverBox) return false
-
-        return (
-          popoverBox.y >= 0 &&
-          popoverBox.y + popoverBox.height <= viewportSize.height
-        )
-      })
-      .toBe(true)
-  })
 })

browser_tests/tests/queueButtonModes.spec.ts

@@ -1,63 +0,0 @@
-import { expect } from '@playwright/test'
-
-import type { PromptResponse } from '@/schemas/apiSchema'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-
-const queueModeLabels = ['Run', 'Run (On Change)', 'Run (Instant)']
-const runOnChangeLabel = queueModeLabels[1]
-
-test.describe('Queue button modes', { tag: '@ui' }, () => {
-  test('Run button is visible in topbar', async ({ comfyPage }) => {
-    await expect(comfyPage.actionbar.queueButton.primaryButton).toBeVisible()
-  })
-
-  test('Queue mode trigger menu is visible', async ({ comfyPage }) => {
-    await expect(comfyPage.actionbar.queueButton.dropdownButton).toBeVisible()
-  })
-
-  test('Clicking queue mode trigger opens mode menu', async ({ comfyPage }) => {
-    const options = await comfyPage.actionbar.queueButton.openOptions()
-
-    await expect(options.menu).toBeVisible()
-  })
-
-  test('Queue mode menu shows available modes', async ({ comfyPage }) => {
-    const options = await comfyPage.actionbar.queueButton.openOptions()
-
-    await expect(options.menu).toBeVisible()
-    await expect(options.modeItems).toHaveText(queueModeLabels)
-  })
-
-  test('Selecting a non-default mode updates the Run button label', async ({
-    comfyPage
-  }) => {
-    const queueButton = comfyPage.actionbar.queueButton
-    const options = await queueButton.openOptions()
-
-    await expect(options.menu).toBeVisible()
-    await options.selectMode(runOnChangeLabel)
-
-    await expect(queueButton.primaryButton).toContainText(runOnChangeLabel)
-  })
-
-  test('Run button sends prompt when clicked', async ({ comfyPage }) => {
-    let promptQueued = false
-    const mockResponse: PromptResponse = {
-      prompt_id: 'test-id',
-      node_errors: {},
-      error: ''
-    }
-    await comfyPage.page.route('**/api/prompt', async (route) => {
-      promptQueued = true
-      await route.fulfill({
-        status: 200,
-        body: JSON.stringify(mockResponse)
-      })
-    })
-
-    await comfyPage.actionbar.queueButton.primaryButton.click()
-
-    await expect.poll(() => promptQueued).toBe(true)
-  })
-})

browser_tests/tests/saveImageAndWebp.spec.ts

@@ -16,7 +16,7 @@ test.describe(
       await comfyPage.runButton.click()
 
       const saveImageNode = comfyPage.vueNodes.getNodeByTitle('Save Image')
-      const saveWebmNode = comfyPage.vueNodes.getNodeByTitle('Save WEBM')
+      const saveWebmNode = comfyPage.vueNodes.getNodeByTitle('SaveWEBM')
 
       // Wait for SaveImage to render an img inside .image-preview
       await expect(saveImageNode.locator('.image-preview img')).toBeVisible({

browser_tests/tests/subgraph/subgraphPromotion.spec.ts

@@ -535,75 +535,6 @@ test.describe(
           .poll(() => getPromotedWidgetCount(comfyPage, '11'))
           .toBeLessThan(initialWidgetCount)
       })
-
-      test('Does not cleanup unconfigured Primitive', async ({ comfyPage }) => {
-        await comfyPage.workflow.loadWorkflow(
-          'subgraphs/subgraph-with-link-and-proxied-primitive'
-        )
-        await expect
-          .poll(
-            () => getPromotedWidgetCount(comfyPage, '2'),
-            'Primitive widget is restored on load'
-          )
-          .toBe(2)
-
-        await comfyPage.page.evaluate(() => app!.canvas.setDirty(true))
-        const subgraphNode = await comfyPage.nodeOps.getFirstNodeRef()
-        const promotedPrimitive = await subgraphNode!.getWidget(1)
-        await expect
-          .poll(
-            () => promotedPrimitive.getValue(),
-            'Primitive widget is not in a disconnected state'
-          )
-          .toBe(0)
-      })
     })
-
-    test.fail(
-      'Promoted text widget is removed when source node is deleted inside the subgraph',
-      { tag: '@vue-nodes' },
-      async ({ comfyPage }) => {
-        await comfyPage.settings.setSetting('Comfy.UseNewMenu', 'Top')
-
-        const clipFixture = await comfyPage.vueNodes.getFixtureByTitle(
-          'CLIP Text Encode (Prompt)'
-        )
-        await comfyPage.contextMenu.openForVueNode(clipFixture.header)
-        await comfyPage.contextMenu.clickMenuItemExact('Convert to Subgraph')
-
-        const subgraphNode = comfyPage.vueNodes
-          .getNodeByTitle('New Subgraph')
-          .first()
-        await expect(subgraphNode).toBeVisible()
-
-        const subgraphNodeId =
-          await comfyPage.vueNodes.getNodeIdByTitle('New Subgraph')
-
-        await expect
-          .poll(() => getPromotedWidgetNames(comfyPage, subgraphNodeId))
-          .toContain('text')
-        await expect(
-          subgraphNode.getByTestId(TestIds.widgets.domWidgetTextarea)
-        ).toBeVisible()
-
-        await comfyPage.vueNodes.enterSubgraph(subgraphNodeId)
-        await expect.poll(() => comfyPage.subgraph.isInSubgraph()).toBe(true)
-        await comfyPage.vueNodes.waitForNodes()
-
-        const interiorClip = await comfyPage.vueNodes.getFixtureByTitle(
-          'CLIP Text Encode (Prompt)'
-        )
-        await interiorClip.delete()
-
-        await comfyPage.subgraph.exitViaBreadcrumb()
-
-        const subgraphNodeAfter =
-          comfyPage.vueNodes.getNodeLocator(subgraphNodeId)
-        await expect(subgraphNodeAfter).toBeVisible()
-        await expect(
-          subgraphNodeAfter.getByTestId(TestIds.widgets.domWidgetTextarea)
-        ).toBeHidden()
-      }
-    )
   }
 )

browser_tests/tests/topbar/workflowTabs.spec.ts

@@ -1,5 +1,4 @@
 import { expect } from '@playwright/test'
-import type { Locator, Page } from '@playwright/test'
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 
@@ -126,48 +125,6 @@ test.describe('Workflow tabs', () => {
     await expect(activeTab.locator('text=•')).toBeVisible()
   })
 
-  test('Can drag tab to end', async ({ comfyPage }) => {
-    const topbar = comfyPage.menu.topbar
-
-    await topbar.newWorkflowButton.click()
-    await topbar.newWorkflowButton.click()
-    await expect.poll(() => topbar.getTabNames()).toHaveLength(3)
-    const [a, b, c] = await topbar.getTabNames()
-
-    await topbar.getTab(0).dragTo(topbar.getTab(2))
-
-    await expect.poll(() => topbar.getTabNames()).toEqual([b, c, a])
-  })
-
-  test('Can drag tab to start', async ({ comfyPage }) => {
-    const topbar = comfyPage.menu.topbar
-
-    await topbar.newWorkflowButton.click()
-    await topbar.newWorkflowButton.click()
-    await expect.poll(() => topbar.getTabNames()).toHaveLength(3)
-    const [a, b, c] = await topbar.getTabNames()
-
-    await topbar.getTab(2).dragTo(topbar.getTab(0))
-
-    await expect.poll(() => topbar.getTabNames()).toEqual([c, a, b])
-  })
-
-  test('Drag preserves active tab', async ({ comfyPage }) => {
-    const topbar = comfyPage.menu.topbar
-
-    await topbar.newWorkflowButton.click()
-    await topbar.newWorkflowButton.click()
-    await expect.poll(() => topbar.getTabNames()).toHaveLength(3)
-
-    const [, b] = await topbar.getTabNames()
-    await topbar.getTab(1).click()
-    await expect.poll(() => topbar.getActiveTabName()).toContain(b)
-
-    await topbar.getTab(0).dragTo(topbar.getTab(2))
-
-    await expect.poll(() => topbar.getActiveTabName()).toContain(b)
-  })
-
   test('Multiple tabs can be created, switched, and closed', async ({
     comfyPage
   }) => {
@@ -189,79 +146,4 @@ test.describe('Workflow tabs', () => {
     await topbar.closeWorkflowTab('Unsaved Workflow (2)')
     await expect.poll(() => topbar.getTabNames()).toHaveLength(2)
   })
-
-  test.describe('Closing a modified workflow tab (FE-419)', () => {
-    async function modifyActiveWorkflow(page: Page, activeTab: Locator) {
-      await page.evaluate(() => {
-        const graph = window.app?.graph
-        const node = window.LiteGraph?.createNode('Note')
-        if (graph && node) graph.add(node)
-      })
-      await expect(
-        activeTab.getByTestId('workflow-dirty-indicator')
-      ).toHaveCount(1)
-    }
-
-    test('shows "Close anyway" label and no Cancel button on dirtyClose dialog', async ({
-      comfyPage
-    }) => {
-      const topbar = comfyPage.menu.topbar
-
-      await topbar.newWorkflowButton.click()
-      await expect.poll(() => topbar.getTabNames()).toHaveLength(2)
-
-      await modifyActiveWorkflow(comfyPage.page, topbar.getActiveTab())
-      await topbar.closeWorkflowTab('Unsaved Workflow (2)')
-
-      const dialog = comfyPage.page.getByRole('dialog')
-      await expect(dialog).toBeVisible()
-      await expect(
-        dialog.getByRole('button', { name: 'Close anyway' })
-      ).toBeVisible()
-      await expect(dialog.getByRole('button', { name: 'Save' })).toBeVisible()
-      await expect(dialog.getByRole('button', { name: 'Cancel' })).toHaveCount(
-        0
-      )
-    })
-
-    test('clicking "Close anyway" closes the tab without saving', async ({
-      comfyPage
-    }) => {
-      const topbar = comfyPage.menu.topbar
-
-      await topbar.newWorkflowButton.click()
-      await expect.poll(() => topbar.getTabNames()).toHaveLength(2)
-
-      await modifyActiveWorkflow(comfyPage.page, topbar.getActiveTab())
-      await topbar.closeWorkflowTab('Unsaved Workflow (2)')
-
-      await comfyPage.page
-        .getByRole('dialog')
-        .getByRole('button', { name: 'Close anyway' })
-        .click()
-
-      await expect.poll(() => topbar.getTabNames()).toHaveLength(1)
-      await expect
-        .poll(() => topbar.getActiveTabName())
-        .toContain('Unsaved Workflow')
-    })
-
-    test('dismissing the dialog keeps the modified tab open', async ({
-      comfyPage
-    }) => {
-      const topbar = comfyPage.menu.topbar
-
-      await topbar.newWorkflowButton.click()
-      await expect.poll(() => topbar.getTabNames()).toHaveLength(2)
-
-      await modifyActiveWorkflow(comfyPage.page, topbar.getActiveTab())
-      await topbar.closeWorkflowTab('Unsaved Workflow (2)')
-
-      await expect(comfyPage.page.getByRole('dialog')).toBeVisible()
-      await comfyPage.page.keyboard.press('Escape')
-      await expect(comfyPage.page.getByRole('dialog')).toBeHidden()
-
-      await expect.poll(() => topbar.getTabNames()).toHaveLength(2)
-    })
-  })
 })

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

@@ -1,5 +1,3 @@
-import type { Locator } from '@playwright/test'
-
 import {
   comfyExpect as expect,
   comfyPageFixture as test
@@ -41,19 +39,6 @@ test.describe('Vue Node Moving', { tag: '@vue-nodes' }, () => {
     expect(Math.abs(a.y - b.y)).toBeLessThanOrEqual(tol)
   }
 
-  const dragFromTabButton = async (comfyPage: ComfyPage, button: Locator) => {
-    const box = await button.boundingBox()
-    if (!box) throw new Error('Tab button has no bounding box')
-    const start = {
-      x: box.x + box.width / 2,
-      y: box.y + box.height * 0.75
-    }
-    await comfyPage.canvasOps.dragAndDrop(start, {
-      x: start.x + 120,
-      y: start.y + 80
-    })
-  }
-
   test('should allow moving nodes by dragging', async ({ comfyPage }) => {
     const loadCheckpointHeaderPos = await getLoadCheckpointHeaderPos(comfyPage)
     await comfyPage.canvasOps.dragAndDrop(loadCheckpointHeaderPos, {
@@ -105,63 +90,6 @@ test.describe('Vue Node Moving', { tag: '@vue-nodes' }, () => {
     await expectPosChanged(headerPos, afterPos)
   })
 
-  test('should not toggle advanced inputs when dragging by the Advanced button', async ({
-    comfyPage
-  }) => {
-    await comfyPage.settings.setSetting(
-      'Comfy.Node.AlwaysShowAdvancedWidgets',
-      false
-    )
-    await comfyPage.nodeOps.addNode(
-      'ModelSamplingFlux',
-      {},
-      {
-        x: 500,
-        y: 200
-      }
-    )
-    await comfyPage.vueNodes.waitForNodes()
-
-    const node = comfyPage.vueNodes.getNodeByTitle('ModelSamplingFlux')
-    const showButton = node.getByText('Show advanced inputs')
-    const widgets = node.locator('.lg-node-widget')
-
-    await expect(showButton).toBeVisible()
-    await expect(widgets).toHaveCount(2)
-
-    const beforePos = await node.boundingBox()
-    if (!beforePos) throw new Error('Node has no bounding box')
-
-    await dragFromTabButton(comfyPage, showButton)
-
-    await expect(showButton).toBeVisible()
-    await expect(node.getByText('Hide advanced inputs')).toBeHidden()
-    await expect(widgets).toHaveCount(2)
-
-    const afterPos = await node.boundingBox()
-    if (!afterPos) throw new Error('Node missing after drag')
-    await expectPosChanged(beforePos, afterPos)
-  })
-
-  test('should not enter subgraph when dragging by the Enter Subgraph button', async ({
-    comfyPage
-  }) => {
-    await comfyPage.workflow.loadWorkflow('subgraphs/basic-subgraph')
-
-    const subgraphNode = await comfyPage.nodeOps.getNodeRefById('2')
-    const beforePos = await subgraphNode.getPosition()
-
-    await dragFromTabButton(
-      comfyPage,
-      comfyPage.vueNodes.getSubgraphEnterButton('2')
-    )
-
-    expect(await comfyPage.subgraph.isInSubgraph()).toBe(false)
-
-    const afterPos = await subgraphNode.getPosition()
-    await expectPosChanged(beforePos, afterPos)
-  })
-
   test('should move all selected nodes together when dragging one with Meta held', async ({
     comfyPage
   }) => {

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

@@ -13,7 +13,6 @@ import {
   ExecutionHelper,
   buildKSamplerError
 } from '@e2e/fixtures/helpers/ExecutionHelper'
-import { fitToViewInstant } from '@e2e/fixtures/utils/fitToView'
 import { webSocketFixture } from '@e2e/fixtures/ws'
 
 const test = mergeTests(comfyPageFixture, webSocketFixture)
@@ -21,7 +20,6 @@ const test = mergeTests(comfyPageFixture, webSocketFixture)
 const ERROR_CLASS = /ring-destructive-background/
 const UNKNOWN_NODE_ID = '1'
 const INNER_EXECUTION_ID = '2:1'
-const KSAMPLER_MODEL_INPUT_NAME = 'model'
 
 test.describe('Vue Node Error', { tag: '@vue-nodes' }, () => {
   test('should display error state when node is missing (node from workflow is not installed)', async ({
@@ -73,59 +71,6 @@ test.describe('Vue Node Error', { tag: '@vue-nodes' }, () => {
       ).toHaveClass(ERROR_CLASS)
     })
 
-    test(
-      'highlights the missing required input slot',
-      { tag: ['@screenshot', '@node'] },
-      async ({ comfyPage }) => {
-        const ksamplerId = await comfyPage.vueNodes.getNodeIdByTitle('KSampler')
-        const ksamplerNode = comfyPage.vueNodes.getNodeLocator(ksamplerId)
-        const modelInputIndex = await comfyPage.page.evaluate(
-          ({ nodeId, inputName }) => {
-            const node = window.app!.graph.getNodeById(nodeId)
-            const index =
-              node?.inputs?.findIndex((input) => input.name === inputName) ?? -1
-            if (index < 0) {
-              throw new Error(`Input slot "${inputName}" not found`)
-            }
-            return index
-          },
-          { nodeId: ksamplerId, inputName: KSAMPLER_MODEL_INPUT_NAME }
-        )
-        const modelInputSlotRow = comfyPage.vueNodes.getInputSlotRow(
-          ksamplerId,
-          modelInputIndex
-        )
-        const modelInputSlotHighlight =
-          comfyPage.vueNodes.getInputSlotConnectionDot(
-            ksamplerId,
-            modelInputIndex
-          )
-        const exec = new ExecutionHelper(comfyPage)
-        await exec.mockValidationFailure({
-          [ksamplerId]: buildKSamplerError(
-            'required_input_missing',
-            KSAMPLER_MODEL_INPUT_NAME,
-            `Required input is missing: ${KSAMPLER_MODEL_INPUT_NAME}`
-          )
-        })
-
-        await comfyPage.runButton.click()
-        await dismissErrorOverlay(comfyPage)
-        await fitToViewInstant(comfyPage)
-
-        await expect(modelInputSlotRow).toBeVisible()
-        await expect(modelInputSlotRow).toBeInViewport()
-        await expect(modelInputSlotHighlight).toHaveClass(/before:ring-error/)
-        await expect(
-          comfyPage.vueNodes.getNodeInnerWrapper(ksamplerId)
-        ).toHaveClass(ERROR_CLASS)
-        await comfyPage.expectScreenshot(
-          ksamplerNode,
-          'vue-node-required-input-missing-slot-error.png'
-        )
-      }
-    )
-
     test('clears error ring when user edits an out-of-range number widget back into range', async ({
       comfyPage
     }) => {

docs/adr/0008-entity-component-system.md

@@ -249,7 +249,6 @@ Companion architecture documents that expand on the design in this ADR:
 | [ECS Lifecycle Scenarios](../architecture/ecs-lifecycle-scenarios.md)                            | Before/after walkthroughs of lifecycle operations (node removal, link creation, etc.)       |
 | [World API and Command Layer](../architecture/ecs-world-command-api.md)                          | How each lifecycle scenario maps to a command in the World API                              |
 | [Subgraph Boundaries and Widget Promotion](../architecture/subgraph-boundaries-and-promotion.md) | Design rationale for modeling subgraphs as node components, not separate entities           |
-| [ADR 0009: Subgraph promoted widgets](0009-subgraph-promoted-widgets-use-linked-inputs.md)       | Follow-up decision for promoted widget identity and value ownership at subgraph boundaries  |
 | [Appendix: Critical Analysis](../architecture/appendix-critical-analysis.md)                     | Independent verification of the accuracy of the architecture documents                      |
 | [Change Tracker](../architecture/change-tracker.md)                                              | Documents the current undo/redo system that ECS cross-cutting concerns will replace         |
 

docs/adr/0009-subgraph-promoted-widgets-use-linked-inputs.md

@@ -1,328 +0,0 @@
-# 9. Subgraph promoted widgets use linked inputs
-
-Date: 2026-05-05
-
-Appendices:
-
-- [Before/after flow diagrams](./0009-subgraph-promoted-widgets-use-linked-inputs/before-after-flows.md)
-- [System comparison](./0009-subgraph-promoted-widgets-use-linked-inputs/system-comparison.md)
-- [Removing `disambiguatingSourceNodeId`](./0009-subgraph-promoted-widgets-use-linked-inputs/disambiguating-source-node-id.md)
-
-## Status
-
-Proposed
-
-## Context
-
-Subgraph widget promotion historically had two overlapping representations:
-
-1. `properties.proxyWidgets`, a serialized list of source node/widget tuples;
-2. linked subgraph inputs, where an interior widget-bearing input is exposed
-   through the subgraph boundary.
-
-This created ambiguous ownership. Runtime value reads could collapse to an
-interior source widget, while host `widgets_values` could also carry an
-exterior value. Multiple host instances of the same subgraph could therefore
-stomp one another, and serialization could mutate interior widgets as a
-persistence carrier for exterior values.
-
-The ECS widget migration makes that ambiguity more expensive: widgets are
-becoming entities with component state keyed by stable entity identity, and
-subgraphs are modeled as graph boundary structure rather than a separate
-promotion-specific entity kind.
-
-## Decision
-
-Promoted widgets are represented only as standard linked `SubgraphInput`
-widgets. A promoted widget is a host-scoped widget entity owned by a subgraph
-input on a host `SubgraphNode`. The interior source widget supplies schema,
-type, options, tooltip, and default metadata, but it is not the owner of the
-host value.
-
-Display-only preview surfacing, such as `$$canvas-image-preview`, is not a
-promoted widget. It is a separate preview-exposure system because it has no
-host-owned widget value, does not feed prompt serialization, and often points at
-virtual `serialize: false` pseudo-widgets that may not exist on the source node.
-
-`properties.proxyWidgets` becomes a legacy load-time input only. Successful
-repair consumes entries from `proxyWidgets`; canonical saves do not re-emit
-those entries. The standard serialized representation is the existing subgraph
-interface/input form plus host-node `widgets_values`.
-
-Display-only preview exposures use their own host-node-scoped serialized entry,
-`properties.previewExposures`, instead of `properties.proxyWidgets` and instead
-of linked `SubgraphInput` widgets. Canonical preview-exposure JSON uses preview
-language, not widget language:
-
-```ts
-type PreviewExposure = {
-  name: string
-  sourceNodeId: string
-  sourcePreviewName: string
-}
-```
-
-Host-node scope preserves current behavior where different instances of the
-same subgraph can choose different exposed previews.
-
-The entry intentionally stores only host preview identity and source locator
-identity. `name` is the host-scoped stable identity for this preview exposure,
-analogous to `SubgraphInput.name`; it is not a display label. It is generated
-with existing collision behavior, such as `nextUniqueName(...)`, when an
-exposure is created. Media type, display labels, titles, image/video/audio URLs,
-and other runtime preview details are derived from the current graph and output
-state. Array order is the canonical display order. Preview exposures do not get
-a separate persisted `label` in this slice; if a future rename UX needs one, it
-should follow the same rule as subgraph inputs: `name` is identity and `label`
-is display-only.
-
-Preview exposures are persisted user choices after creation. Packing nodes into
-a subgraph may auto-add recommended preview exposures for supported output
-nodes, and users may explicitly add or remove additional preview exposures
-afterward. Normal load/save does not re-derive previews from node type alone,
-because that would make old workflows change when support for new preview node
-types is added. Unresolved preview exposures remain persisted and inert;
-automatic cleanup does not prune them. They are removed only by explicit user
-action or by destruction/unpacking of the owning host.
-
-Preview exposures compose through nested subgraph hosts by chaining immediate
-boundaries. If an outer subgraph wants to show a preview exposed by an inner
-subgraph host, the outer `previewExposures` entry points at the immediate inner
-`SubgraphNode`, and `sourcePreviewName` names the inner host's preview-exposure
-identity, not the deepest interior preview name. Runtime preview resolution may
-then follow the inner host's own preview exposures to find media. Canonical JSON
-does not persist flattened deep paths, because deep paths would couple host UI
-state to private nested graph internals.
-
-## Identity and value ownership
-
-- UI/value identity is host-scoped: host node locator plus
-  `SubgraphInput.name`.
-- Host-scoped identity means the host `SubgraphNode` instance within its
-  containing `graphScope`; the interior source node is not the state or
-  persistence owner.
-- `SubgraphInput.name` is the stable internal identity.
-- `SubgraphInput.label` / `localized_name` are display-only.
-- `SubgraphInput.id` may be used for slot-instance reconciliation, not as the
-  persisted widget value key.
-- Source node/widget identity remains metadata for diagnostics, missing-model
-  lookup, schema projection, and migration only.
-- The host/exterior value wins over the interior/source value during repair,
-  persistence, and prompt serialization.
-
-This follows the existing widget/slot convention: `name` is identity, `label`
-is display.
-
-Promoted-widget value state is a host-scoped sparse overlay over source-widget
-metadata and defaults. The source widget remains the schema/default provider;
-host value state is materialized only when the exterior value differs from the
-effective source default or when restored from persisted host state. Canonical
-save/load must not eagerly mirror source defaults or use interior widgets as
-persistence carriers.
-
-## Forward migration
-
-Loading a workflow with legacy `proxyWidgets` runs a one-way repair:
-
-1. Parse `properties.proxyWidgets` with the existing Zod-inferred tuple type.
-2. Invalid raw `proxyWidgets` data logs `console.error`, does not throw, and is
-   not quarantined.
-3. Build a multi-pass association map before mutation:
-   - normalized legacy proxy entry;
-   - projected legacy promoted-widget order;
-   - host `widgets_values` value, preserving sparse holes;
-   - repair strategy or failure reason;
-   - whether the entry is a value widget or display-only preview exposure.
-4. Defer mutations until node IDs/entity IDs are stable and the subgraph graph
-   is configured.
-5. On flush, re-resolve against current graph state, because clone/paste/load
-   flows may have remapped or created nodes and links.
-6. If already represented by a linked `SubgraphInput`, consider the legacy
-   entry resolved and consume it.
-7. Otherwise repair through existing subgraph input/link systems.
-8. If the entry is display-only preview surfacing, migrate it into the separate
-   preview-exposure representation instead of creating a linked `SubgraphInput`.
-9. If value-widget repair fails, write inert quarantine metadata and warn.
-
-The repair is idempotent. Pending plans store tuple/value data and re-check the
-current graph before applying mutations.
-
-Legacy entries are classified as preview exposures when either:
-
-- the legacy source name starts with `$$`; or
-- the source node resolves to a matching pseudo-preview widget, such as a
-  `serialize: false` preview/video/audio UI widget.
-
-Everything else is treated as a value-widget promotion candidate. An unresolved
-preview-shaped entry remains inert at runtime and is still persisted, because
-preview-capable pseudo-widgets and output media can be removed and re-added
-dynamically. It is not quarantined because it has no user value to preserve. A
-non-`$$` entry that cannot resolve to a source widget is a value-widget repair
-failure and follows the quarantine path unless it can resolve to a
-pseudo-preview widget.
-
-## Proxy widget error quarantine
-
-Valid legacy entries that cannot be repaired are persisted in
-`properties.proxyWidgetErrorQuarantine`. Quarantined entries are inert: they do
-not hydrate runtime promoted widgets, do not participate in execution, and are
-not used for app-mode/favorites identity.
-
-Quarantine entries preserve enough information to avoid data loss and support
-future tooling:
-
-```ts
-type ProxyWidgetErrorQuarantineEntry = {
-  originalEntry: ProxyWidgetTuple
-  reason:
-    | 'missingSourceNode'
-    | 'missingSourceWidget'
-    | 'missingSubgraphInput'
-    | 'ambiguousSubgraphInput'
-    | 'unlinkedSourceWidget'
-    | 'primitiveBypassFailed'
-  hostValue?: TWidgetValue
-  attemptedAtVersion: 1
-}
-```
-
-Unresolved legacy UI selections/favorites are dropped with `console.warn`.
-Workflow-level promotion/value intent is preserved by
-`proxyWidgetErrorQuarantine`, not by a second UI quarantine format.
-
-## Primitive-node repair
-
-Legacy `proxyWidgets` may point at `PrimitiveNode` outputs. Primitive nodes
-serve nearly the same purpose as subgraph inputs: they provide a widget value to
-one or more target widget inputs. The migration repairs this expected legacy
-shape in the first migration rather than quarantining it by default.
-
-Primitive repair:
-
-- coalesces exact duplicate legacy entries during planning;
-- uses the primitive node's user title as the base input name when the node was
-  renamed, otherwise the primitive output widget name;
-- applies existing naming behavior and `nextUniqueName(...)` for collisions;
-- uses the existing primitive merge/config compatibility logic;
-- creates one `SubgraphInput` for the primitive fanout;
-- reconnects every former primitive output target to that input in target
-  order, using standard connect/disconnect APIs;
-- applies the host value when one exists, otherwise seeds from the source
-  primitive value;
-- leaves the primitive node and its widget value in place, but disconnected and
-  inert.
-
-Primitive repair is all-or-quarantine. If any target cannot be validated or
-reconnected, the migration does not leave a partial rewrite; it quarantines the
-entry with `hostValue` and logs the reason.
-
-## Serialization
-
-After repair/quarantine:
-
-- `properties.proxyWidgets` is omitted for repaired entries;
-- display-only preview entries are omitted from `properties.proxyWidgets` and
-  emitted through `properties.previewExposures`;
-- `properties.proxyWidgetErrorQuarantine` carries unrepaired valid entries;
-- preview exposures do not carry quarantine values because they do not own user
-  values; unresolved preview exposures remain inert in `previewExposures`;
-- host `widgets_values` contains host-owned values only for canonical host
-  widgets, not source-owned defaults or interior persistence copies;
-- quarantined legacy values live in `proxyWidgetErrorQuarantine.hostValue`;
-- array-form `widgets_values` remains for now.
-
-Preview exposures are display-only UI metadata. They drive host canvas/app-mode
-preview rendering, but they do not create prompt inputs, do not create
-`widgets_values`, do not alter node execution order, do not become executable
-graph edges, and do not participate in prompt serialization. Runtime mapping
-from backend `display_node`/output messages to a host preview exposure is a UI
-projection only.
-
-The old `SubgraphNode.serialize()` behavior that copied exterior promoted
-values into connected interior widgets is removed. A temporary TODO should mark
-that removal point until the migration is proven stable. Host values are
-serialized through standard subgraph-input widgets instead.
-
-Longer term, `widgets_values` should move from array order to an object/map
-keyed by stable widget name, but that migration is out of scope for this
-decision.
-
-## App mode, builder, and favorites
-
-The runtime migration and UI identity migration ship in the same slice. The UI
-must not persist promoted selections by source node/widget identity after this
-change.
-
-Canonical UI identity is:
-
-```ts
-type PromotedWidgetUiIdentity = {
-  hostNodeLocator: string
-  subgraphInputName: string
-}
-```
-
-Legacy source-identity selections are migrated when they resolve through the
-standard input created or confirmed by the migration. Unresolved selections are
-dropped with a warning.
-
-Preview exposure output selections are also host-scoped and must not persist
-interior source node identity. Canonical preview/output identity is:
-
-```ts
-type PreviewExposureUiIdentity = {
-  hostNodeLocator: string
-  previewName: string
-}
-```
-
-The UI references the explicit preview exposure itself. This keeps subgraphs
-opaque: consumers select the host boundary contract, not the interior node that
-currently supplies media. Legacy output selections that refer to interior
-preview source nodes may migrate if they resolve to a preview-exposure chain;
-otherwise they are dropped with `console.warn`. There is no separate preview UI
-quarantine.
-
-## PromotionStore
-
-`PromotionStore` becomes vestigial. It may remain temporarily as a derived
-runtime compatibility/index layer for existing consumers, but it is not
-serialized authority, must not create promotions without linked
-`SubgraphInput`s, and should be removed once consumers query the standard graph
-interface directly.
-
-## Considered options
-
-### Keep `proxyWidgets` as canonical serialized topology
-
-Rejected. This preserves two representations for the same concept and keeps
-source-widget identity in the value-ownership path.
-
-### Preserve bare promoted widgets as degraded runtime state
-
-Rejected. This would avoid some migration complexity, but it perpetuates the
-ambiguity that caused host/source value bugs and makes ECS identity less clear.
-
-### Quarantine primitive-node promotions by default
-
-Rejected. Primitive-node proxy promotions are expected legacy workflows, and
-quarantining them would break users unnecessarily. They are repaired by bypassing
-the primitive node when the repair can be validated all-or-nothing.
-
-### Migrate `widgets_values` to object/map form now
-
-Rejected for this slice. Name-keyed object form is the desired long-term
-direction, but combining it with the promotion migration increases blast radius
-for existing workflow consumers that still assume array order.
-
-## Consequences
-
-- Promoted widget values become host-instance-owned and ECS-compatible.
-- Source widgets remain metadata/default providers, not persistence carriers.
-- Legacy workflows are repaired toward one standard representation.
-- Quarantine preserves unrepaired valid legacy data without reintroducing bare
-  runtime promotion.
-- Primitive fanout repair is more complex, but avoids breaking common existing
-  workflows.
-- UI code must migrate with the runtime migration to avoid mixed identity states.
-- `PromotionStore` has a clear removal path.

docs/adr/0009-subgraph-promoted-widgets-use-linked-inputs/before-after-flows.md

@@ -1,210 +0,0 @@
-# Appendix: Before and after flows
-
-This appendix visualizes the ownership and migration flows described in
-[ADR 0009](../0009-subgraph-promoted-widgets-use-linked-inputs.md).
-
-## Before: proxy widgets and linked inputs overlap
-
-Historically, promoted widgets could be represented both as serialized
-`properties.proxyWidgets` entries and as linked subgraph inputs. Runtime value
-reads could collapse back to the interior source widget, while host
-`widgets_values` could also carry an exterior value for the same promoted UI.
-
-```mermaid
-flowchart TD
-  workflow[Workflow JSON] --> proxyWidgets[properties.proxyWidgets]
-  workflow --> hostValues[host widgets_values]
-  proxyWidgets --> promotionStore[PromotionStore / promotion runtime]
-  promotionStore --> sourceWidget[Interior source widget]
-  linkedInput[Linked SubgraphInput] --> hostWidget[Host promoted widget]
-  sourceWidget --> hostWidget
-  hostValues --> hostWidget
-  hostWidget --> prompt[Prompt serialization]
-  hostWidget -. may copy value back .-> sourceWidget
-  sourceWidget -. shared by host instances .-> otherHost[Another host instance]
-
-  classDef legacy fill:#fff3cd,stroke:#a66f00,color:#332200
-  classDef ambiguous fill:#f8d7da,stroke:#842029,color:#330000
-  classDef canonical fill:#d1e7dd,stroke:#0f5132,color:#052e16
-
-  class proxyWidgets,promotionStore legacy
-  class sourceWidget,hostValues ambiguous
-  class linkedInput,hostWidget canonical
-```
-
-Key problems in the old flow:
-
-- `properties.proxyWidgets` and linked `SubgraphInput` widgets could describe
-  the same promotion.
-- Interior source widgets supplied both schema metadata and, in some flows,
-  persisted host values.
-- Multiple host instances of the same subgraph could stomp one another through
-  the shared interior widget value.
-- Display-only previews were mixed into widget-promotion language even though
-  they do not own values or feed prompt serialization.
-
-## After: linked inputs are the promoted-widget boundary
-
-Promoted value widgets are now represented only as standard linked
-`SubgraphInput` widgets. The source widget remains the schema/default provider,
-but the host `SubgraphNode` owns the promoted value.
-
-```mermaid
-flowchart TD
-  workflow[Workflow JSON] --> subgraphInterface[Subgraph interface / inputs]
-  workflow --> hostValues[host widgets_values]
-  subgraphInterface --> subgraphInput[SubgraphInput.name]
-  subgraphInput --> hostWidget[Host-scoped widget entity]
-  hostValues --> hostWidget
-  sourceWidget[Interior source widget] --> schema[Schema, type, options, tooltip, default]
-  schema --> hostWidget
-  hostWidget --> prompt[Prompt serialization]
-
-  hostIdentity[Host node locator + SubgraphInput.name] --> hostWidget
-  sourceWidget -. metadata only .-> diagnostics[Diagnostics / lookup / migration]
-  sourceWidget -. no host value ownership .-> schema
-
-  classDef owner fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef metadata fill:#cff4fc,stroke:#055160,color:#032830
-  classDef persisted fill:#e2e3e5,stroke:#41464b,color:#212529
-
-  class subgraphInterface,subgraphInput,hostWidget,hostIdentity owner
-  class sourceWidget,schema,diagnostics metadata
-  class workflow,hostValues persisted
-```
-
-Canonical ownership after the migration:
-
-- UI/value identity is host-scoped: host node locator plus
-  `SubgraphInput.name`.
-- `SubgraphInput.name` is stable identity; labels and localized names are
-  display-only.
-- Host values win during repair, persistence, and prompt serialization.
-- Source widgets provide metadata and defaults only.
-- Canonical saves omit repaired `properties.proxyWidgets` entries.
-
-## Legacy load migration
-
-Loading a workflow with legacy `proxyWidgets` performs an idempotent repair. The
-repair builds a plan before mutating graph state, then re-resolves against the
-current graph when node IDs and links are stable.
-
-```mermaid
-flowchart TD
-  start[Load workflow] --> parse{Parse properties.proxyWidgets}
-  parse -->|invalid raw data| invalid[console.error and ignore]
-  parse -->|valid tuples| plan[Build repair plan]
-  plan --> classify{Classify entry}
-
-  classify -->|value widget| valueRepair{Already linked SubgraphInput?}
-  valueRepair -->|yes| consume[Consume legacy proxy entry]
-  valueRepair -->|no| repair[Repair through subgraph input/link systems]
-  repair --> repairResult{Repair succeeded?}
-  repairResult -->|yes| consume
-  repairResult -->|no| quarantine[Persist proxyWidgetErrorQuarantine]
-
-  classify -->|primitive fanout| primitive[Validate all primitive targets]
-  primitive --> primitiveResult{All targets reconnectable?}
-  primitiveResult -->|yes| primitiveRepair[Create one SubgraphInput and reconnect fanout]
-  primitiveRepair --> consume
-  primitiveResult -->|no| quarantine
-
-  classify -->|display-only preview| preview[Create / keep previewExposures entry]
-  preview --> consume
-
-  consume --> save[Canonical save]
-  quarantine --> save
-  save --> omit[Omit repaired entries from proxyWidgets]
-  save --> keepQuarantine[Persist unrepaired value intent in quarantine]
-  save --> keepPreview[Persist previews in previewExposures]
-
-  classDef ok fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef warn fill:#fff3cd,stroke:#a66f00,color:#332200
-  classDef error fill:#f8d7da,stroke:#842029,color:#330000
-  classDef neutral fill:#e2e3e5,stroke:#41464b,color:#212529
-
-  class consume,repair,primitiveRepair,preview,save,omit,keepPreview ok
-  class plan,classify,valueRepair,primitive,primitiveResult,repairResult neutral
-  class quarantine,keepQuarantine warn
-  class invalid error
-```
-
-## Preview exposures are separate from value widgets
-
-Display-only previews, such as `$$canvas-image-preview`, are not promoted
-widgets. They have host-scoped serialized identity, but they do not create
-prompt inputs, do not create `widgets_values`, and do not own user values.
-
-```mermaid
-flowchart TD
-  hostNode[Host SubgraphNode] --> previewExposures[properties.previewExposures]
-  previewExposures --> exposure[PreviewExposure.name]
-  exposure --> sourceLocator[sourceNodeId + sourcePreviewName]
-  sourceLocator --> runtimePreview[Runtime preview/output state]
-  runtimePreview --> hostCanvas[Host canvas / app-mode preview]
-
-  exposure --> uiIdentity[hostNodeLocator + previewName]
-  runtimePreview -. UI projection only .-> hostCanvas
-  previewExposures -. no prompt input .-> noPrompt[No prompt serialization]
-  previewExposures -. no value widget .-> noValue[No widgets_values entry]
-  previewExposures -. no graph edge .-> noEdge[No executable graph edge]
-
-  classDef preview fill:#cff4fc,stroke:#055160,color:#032830
-  classDef noValue fill:#f8d7da,stroke:#842029,color:#330000
-  classDef persisted fill:#e2e3e5,stroke:#41464b,color:#212529
-
-  class previewExposures,exposure,sourceLocator,runtimePreview,hostCanvas,uiIdentity preview
-  class noPrompt,noValue,noEdge noValue
-  class hostNode persisted
-```
-
-For nested subgraphs, preview exposures chain across immediate host boundaries
-instead of persisting flattened deep paths.
-
-```mermaid
-flowchart LR
-  outerHost[Outer SubgraphNode] --> outerExposure[Outer previewExposures entry]
-  outerExposure --> innerHost[Immediate inner SubgraphNode]
-  innerHost --> innerExposure[Inner previewExposures entry]
-  innerExposure --> deepestPreview[Interior preview source]
-  deepestPreview --> media[Resolved media]
-
-  outerExposure -. sourcePreviewName names inner preview identity .-> innerExposure
-  outerExposure -. does not persist deep private path .-> opaque[Subgraph internals remain opaque]
-
-  classDef boundary fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef preview fill:#cff4fc,stroke:#055160,color:#032830
-  classDef note fill:#fff3cd,stroke:#a66f00,color:#332200
-
-  class outerHost,innerHost boundary
-  class outerExposure,innerExposure,deepestPreview,media preview
-  class opaque note
-```
-
-## Serialization summary
-
-```mermaid
-flowchart TD
-  canonical[Canonical serialized SubgraphNode] --> inputs[Subgraph interface / inputs]
-  canonical --> values[widgets_values for host-owned values]
-  canonical --> previews[properties.previewExposures]
-  canonical --> quarantine[properties.proxyWidgetErrorQuarantine]
-  canonical -. omits repaired entries .-> noProxy[No canonical proxyWidgets]
-
-  inputs --> valueWidgets[Promoted value widgets]
-  values --> valueWidgets
-  previews --> previewUi[Display-only preview UI]
-  quarantine --> futureTooling[Future recovery tooling]
-
-  valueWidgets --> prompt[Prompt serialization]
-  previewUi -. not serialized into prompt .-> prompt
-  quarantine -. inert .-> prompt
-
-  classDef canonical fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef inert fill:#fff3cd,stroke:#a66f00,color:#332200
-  classDef removed fill:#f8d7da,stroke:#842029,color:#330000
-
-  class inputs,values,valueWidgets,prompt,canonical canonical
-  class previews,previewUi,quarantine,futureTooling inert
-  class noProxy removed
-```

docs/adr/0009-subgraph-promoted-widgets-use-linked-inputs/disambiguating-source-node-id.md

@@ -1,147 +0,0 @@
-# Appendix: Removing `disambiguatingSourceNodeId`
-
-This appendix explains where the existing promotion system needs
-`disambiguatingSourceNodeId`, why that need appears, and how the canonical form
-chosen by [ADR 0009](../0009-subgraph-promoted-widgets-use-linked-inputs.md)
-removes the pattern from promoted-widget identity.
-
-## Why the disambiguator exists
-
-The legacy promotion model identifies a promoted widget by source location:
-
-```ts
-type PromotedWidgetSource = {
-  sourceNodeId: string
-  sourceWidgetName: string
-  disambiguatingSourceNodeId?: string
-}
-```
-
-`sourceNodeId` is the immediate interior node visible from the host subgraph.
-That is not always the original widget owner. When promotions pass through
-nested subgraphs, two promoted widgets can have the same immediate
-`sourceNodeId` and `sourceWidgetName` while pointing at different leaf widgets.
-`disambiguatingSourceNodeId` carries the deepest source node ID so the runtime
-can choose the right promoted view.
-
-```mermaid
-flowchart TD
-  outerHost[Outer host SubgraphNode] --> middleNode[Interior middle SubgraphNode]
-  middleNode --> middleWidgetA[Promoted widget view: text]
-  middleNode --> middleWidgetB[Promoted widget view: text]
-  middleWidgetA --> leafA[Leaf source node 17 / widget text]
-  middleWidgetB --> leafB[Leaf source node 42 / widget text]
-
-  oldKeyA[Old key: middleNodeId + text + disambiguatingSourceNodeId 17]
-  oldKeyB[Old key: middleNodeId + text + disambiguatingSourceNodeId 42]
-  middleWidgetA -. requires .-> oldKeyA
-  middleWidgetB -. requires .-> oldKeyB
-
-  classDef host fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef ambiguous fill:#fff3cd,stroke:#a66f00,color:#332200
-  classDef leaf fill:#cff4fc,stroke:#055160,color:#032830
-
-  class outerHost host
-  class middleNode,middleWidgetA,middleWidgetB,oldKeyA,oldKeyB ambiguous
-  class leafA,leafB leaf
-```
-
-The disambiguator is therefore not a domain concept. It is compensating for an
-identity model that asks host UI state to identify private nested internals.
-
-## Existing places that need it
-
-| Area                         | Current use of `disambiguatingSourceNodeId`                                                                                                   | Ambiguity being patched                                                                             |
-| ---------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------- |
-| Promotion source types       | `PromotedWidgetSource` and `PromotedWidgetView` carry the optional field.                                                                     | Source identity needs more than immediate node ID plus widget name for nested promoted views.       |
-| Concrete widget resolution   | `findWidgetByIdentity(...)` matches promoted views by `(disambiguatingSourceNodeId ?? sourceNodeId)` when a source node ID is supplied.       | Multiple promoted views under the same intermediate node can share a widget name.                   |
-| Legacy proxy normalization   | Prefixed legacy names such as `123:widget_name` are converted into structured source identity and tested with candidate disambiguators.       | Old serialized names encode leaf identity inside the widget name string.                            |
-| Promotion store keys         | `makePromotionEntryKey(...)`, `isPromoted(...)`, and `demote(...)` include the field in equality.                                             | Store-level uniqueness would collapse distinct nested promotions without the leaf ID.               |
-| Linked promotion propagation | `SubgraphNode._resolveLinkedPromotionBySubgraphInput(...)` preserves the leaf ID when a linked input targets an inner subgraph promoted view. | The outer host otherwise sees only the immediate inner `SubgraphNode` and the promoted widget name. |
-| Subgraph editor UI           | The editor uses the field when resolving active widgets and when writing reordered/toggled promotions back to the store.                      | UI list operations must not merge same-name promoted views from different leaves.                   |
-
-## New promoted-widget identity
-
-ADR 0009 moves promoted value identity to the host boundary:
-
-```ts
-type PromotedWidgetUiIdentity = {
-  hostNodeLocator: string
-  subgraphInputName: string
-}
-```
-
-The canonical widget is owned by a `SubgraphInput` on the host
-`SubgraphNode`. The host widget no longer needs to identify the deepest source
-node to preserve value identity. The source widget is consulted for schema,
-defaults, diagnostics, and migration, but it is not the value owner.
-
-```mermaid
-flowchart TD
-  host[Host SubgraphNode] --> inputA[SubgraphInput.name: prompt]
-  host --> inputB[SubgraphInput.name: negative_prompt]
-  inputA --> hostWidgetA[Host-owned widget entity]
-  inputB --> hostWidgetB[Host-owned widget entity]
-
-  hostWidgetA -. schema/default metadata .-> sourceA[Interior source widget text]
-  hostWidgetB -. schema/default metadata .-> sourceB[Interior source widget text]
-
-  identityA[Identity: hostNodeLocator + prompt] --> hostWidgetA
-  identityB[Identity: hostNodeLocator + negative_prompt] --> hostWidgetB
-  sourceA -. not part of host value key .-> identityA
-  sourceB -. not part of host value key .-> identityB
-
-  classDef owner fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef metadata fill:#cff4fc,stroke:#055160,color:#032830
-  classDef removed fill:#f8d7da,stroke:#842029,color:#330000
-
-  class host,inputA,inputB,hostWidgetA,hostWidgetB,identityA,identityB owner
-  class sourceA,sourceB metadata
-```
-
-This is the same rule the subgraph interface already uses: `name` is stable
-identity, and `label` / `localized_name` are display-only.
-
-## How the new form removes each need
-
-| Previous disambiguation site                      | New canonical replacement                                                                                                         |
-| ------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------- |
-| `PromotedWidgetSource.disambiguatingSourceNodeId` | Host value identity is `hostNodeLocator + SubgraphInput.name`; source locator fields become migration/diagnostic metadata only.   |
-| `PromotedWidgetView.disambiguatingSourceNodeId`   | Host-scoped widget entities are derived from subgraph inputs, not from promoted views chained through nested source widgets.      |
-| `findWidgetByIdentity(...)` leaf matching         | Runtime value lookup starts from the host input identity; source traversal is metadata resolution, not value identity resolution. |
-| Legacy prefixed widget-name normalization         | Load migration consumes legacy source-shaped entries and writes standard subgraph input state or quarantine metadata.             |
-| PromotionStore source-key equality                | `PromotionStore` becomes a temporary derived index; canonical consumers query subgraph inputs directly.                           |
-| Linked promotion propagation across nested hosts  | Nested value composition is represented boundary-by-boundary by linked subgraph inputs with stable names.                         |
-| Subgraph editor active widget matching            | Editor state can operate on host boundary entries instead of matching leaf source widgets through same-name promoted views.       |
-
-## Boundary-by-boundary nested flow
-
-The new form avoids flattened deep source paths. Each host boundary exposes its
-own named input, and the next outer host links to that immediate boundary
-contract.
-
-```mermaid
-flowchart LR
-  leaf[Leaf node widget] --> innerInput[Inner SubgraphInput.name: text]
-  innerInput --> innerHostWidget[Inner host-owned widget]
-  innerHostWidget --> outerInput[Outer SubgraphInput.name: prompt]
-  outerInput --> outerHostWidget[Outer host-owned widget]
-
-  innerIdentity[Inner value key: innerHost + text] --> innerHostWidget
-  outerIdentity[Outer value key: outerHost + prompt] --> outerHostWidget
-  leaf -. schema/default source .-> innerHostWidget
-  leaf -. not persisted as outer value key .-> outerIdentity
-
-  classDef boundary fill:#d1e7dd,stroke:#0f5132,color:#052e16
-  classDef source fill:#cff4fc,stroke:#055160,color:#032830
-  classDef note fill:#fff3cd,stroke:#a66f00,color:#332200
-
-  class innerInput,innerHostWidget,outerInput,outerHostWidget,innerIdentity,outerIdentity boundary
-  class leaf source
-```
-
-Because each layer has its own stable `SubgraphInput.name`, two same-name leaf
-widgets no longer require a persisted leaf-node disambiguator at the outer host.
-If the user exposes both, the collision is resolved when the host inputs are
-created by assigning distinct input names with the existing unique-name
-behavior.

docs/adr/0009-subgraph-promoted-widgets-use-linked-inputs/system-comparison.md

@@ -1,37 +0,0 @@
-# Appendix: System comparison
-
-This appendix compares the legacy promoted-widget systems with the canonical
-linked-input model chosen by
-[ADR 0009](../0009-subgraph-promoted-widgets-use-linked-inputs.md).
-
-| Concern                    | Legacy `properties.proxyWidgets` promotions                                                              | Linked `SubgraphInput` promotions before migration                                               | New canonical linked-input system                                                                           |
-| -------------------------- | -------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------ | ----------------------------------------------------------------------------------------------------------- |
-| Serialized authority       | `properties.proxyWidgets` stores source node/widget tuples as promotion topology.                        | Subgraph interface/input links can also represent the same exposed widget.                       | Subgraph interface/input links are the only canonical topology for promoted value widgets.                  |
-| Load-time role             | Hydrates promoted widgets directly from legacy tuples.                                                   | May already describe the promoted widget, creating overlap with `proxyWidgets`.                  | Existing linked inputs are accepted as resolved; legacy tuples are consumed by repair or quarantined.       |
-| Save-time role             | Could be re-emitted as promotion state.                                                                  | Serialized as normal subgraph interface data.                                                    | Repaired `proxyWidgets` entries are omitted; standard subgraph inputs plus host `widgets_values` are saved. |
-| Value owner                | Ambiguous: host `widgets_values` and the interior source widget could both carry the value.              | Closer to the desired boundary model, but still coexisted with source/proxy ownership paths.     | Host `SubgraphNode` owns value state through host-scoped widget identity.                                   |
-| Schema/default provider    | Interior source widget provides schema and may also become persistence carrier.                          | Interior source widget provides source metadata through the link.                                | Interior source widget provides schema, type, options, tooltip, and defaults only.                          |
-| UI identity                | Often persisted by source node/widget identity.                                                          | Can use subgraph input identity, but mixed states still exist while proxy identity remains.      | Host node locator plus `SubgraphInput.name`.                                                                |
-| Display label handling     | Source widget identity and display concerns can blur.                                                    | Uses existing subgraph input naming conventions.                                                 | `SubgraphInput.name` is stable identity; `label` / `localized_name` are display-only.                       |
-| Multiple host instances    | Risk of host instances stomping one another through shared interior values.                              | Better host boundary shape, but overlap with proxy/source value paths can reintroduce ambiguity. | Host-instance-owned sparse overlay prevents shared interior widget value stomping.                          |
-| Prompt serialization       | May read values through promoted runtime state that can collapse to source widgets.                      | Can serialize through standard subgraph input widgets when used consistently.                    | Promoted values serialize only through standard host-owned subgraph-input widgets.                          |
-| Interior mutation on save  | Existing `SubgraphNode.serialize()` behavior could copy exterior values into connected interior widgets. | Could still be affected by legacy copy-back behavior.                                            | Copy-back is removed; source widgets are not persistence carriers.                                          |
-| Primitive-node promotions  | Legacy tuples may point at `PrimitiveNode` outputs.                                                      | Not the canonical primitive fanout representation by itself.                                     | Repaired all-or-nothing into one `SubgraphInput` that reconnects validated fanout targets.                  |
-| Invalid or unresolved data | Invalid data could sit in legacy promotion state or fail repair paths.                                   | Missing linked inputs can be ambiguous when proxy data exists.                                   | Invalid raw data logs and is ignored; unrepaired valid value entries go to `proxyWidgetErrorQuarantine`.    |
-| Display-only previews      | Often mixed into `proxyWidgets` despite not being value widgets.                                         | Linked inputs are inappropriate because previews do not own values or prompt inputs.             | Separate host-scoped `properties.previewExposures` entries model preview UI only.                           |
-| Preview persistence        | Preview selections can depend on source preview/widget-like identity.                                    | No clean distinction from promoted widget inputs.                                                | Preview identity is host node locator plus `previewName`; unresolved previews stay inert and persisted.     |
-| Nested preview behavior    | Deep source identity can leak through host UI state.                                                     | Linked value inputs do not model display-only preview composition.                               | Preview exposures chain across immediate subgraph host boundaries; deep private paths are not persisted.    |
-| ECS compatibility          | Weak: value identity can depend on source widget tuples and mutable interior widgets.                    | Partial: linked inputs fit boundary modeling, but duplicate authority remains.                   | Strong: host-scoped widget entity identity maps cleanly to ECS component state.                             |
-| Long-term status           | Legacy load-time input only.                                                                             | Becomes the standard representation once overlap is removed.                                     | Canonical system; `PromotionStore` becomes a temporary derived compatibility/index layer.                   |
-
-## Practical migration summary
-
-| Legacy shape                                                               | New result                                                                                                 |
-| -------------------------------------------------------------------------- | ---------------------------------------------------------------------------------------------------------- |
-| Valid `proxyWidgets` entry already represented by a linked `SubgraphInput` | Entry is consumed; the existing linked input remains canonical.                                            |
-| Valid value-widget `proxyWidgets` entry without a linked input             | Repair creates or reconnects standard subgraph input/link state.                                           |
-| Valid primitive fanout entry                                               | Repair creates one `SubgraphInput`, reconnects all validated targets, and leaves the primitive node inert. |
-| Valid value-widget entry that cannot be repaired                           | Entry is persisted in `properties.proxyWidgetErrorQuarantine` with the host value when available.          |
-| Preview-shaped legacy entry                                                | Entry is migrated into `properties.previewExposures`, not a linked input.                                  |
-| Unresolved preview exposure                                                | Entry remains inert in `previewExposures`; it is not quarantined because it owns no user value.            |
-| Invalid raw `proxyWidgets` data                                            | Logs `console.error`, does not throw, and is not quarantined.                                              |

docs/architecture/ecs-migration-plan.md

@@ -231,11 +231,6 @@ assigning synthetic widget IDs (via `lastWidgetId` counter on LGraphState).
 the ID mapping — widgets currently lack independent IDs, so the bridge must
 maintain a `(nodeId, widgetName) -> WidgetEntityId` lookup.
 
-**Promoted-widget caveat:** ADR 0009 assigns promoted value widgets a
-host-boundary identity (`host node locator + SubgraphInput.name`). Interior
-source node/widget identity is preserved only as migration and diagnostic
-metadata.
-
 ### 2c. Read-only bridge for Node metadata
 
 Populate `NodeType`, `NodeVisual`, `Properties`, `Execution` components by
@@ -668,10 +663,6 @@ The 6 proto-ECS stores use 6 different keying strategies:
 | NodeOutputStore         | `"${subgraphId}:${nodeId}"`       |
 | SubgraphNavigationStore | subgraphId or `'root'`            |
 
-ADR 0009 refines the promoted-widget target: promoted value widgets should use
-host boundary identity (`host node locator + SubgraphInput.name`), not interior
-source node/widget identity.
-
 The World unifies these under branded entity IDs. But stores that use
 composite keys (e.g., `nodeId:widgetName`) reflect a genuine structural
 reality — a widget is identified by its relationship to a node. Synthetic

docs/architecture/extension-api-v2/names-appendix.md

@@ -1,187 +0,0 @@
-# Names That Must Agree Across Layers
-
-**Task:** DOC1.E6  
-**Date:** 2026-05-08  
-**Patterns cross-walked:** S2.N16 (widget array access), S13.SC1 (ComfyNodeDef inspection), S15.OS1 (dynamic output mutation)
-
-This appendix enumerates the terms that span at least two of the four layers — Python backend, v1 frontend (`ComfyExtension`/LiteGraph), v2 extension API (`NodeHandle`/`WidgetHandle`), and ECS World components — and calls out real inconsistencies where the same concept is named differently or the semantics diverge. Future contributors who rename or refactor any of these terms must propagate the change across all layers listed.
-
----
-
-## Layers
-
-| Layer | Owner | Primary source |
-|-------|-------|---------------|
-| **Python backend** | ComfyUI server | `NODE_CLASS_MAPPINGS`, `INPUT_TYPES`, `RETURN_TYPES` |
-| **v1 frontend** | LiteGraph / ComfyExtension | `src/types/comfy.ts`, `src/schemas/nodeDefSchema.ts`, `LGraphNode.ts` |
-| **v2 extension API** | This project | `src/extension-api/node.ts`, `src/extension-api/widget.ts` |
-| **ECS World** | Alex's branch (PR #11939) | `src/services/extension-api-service.ts` (stubs), `@/world/entityIds` |
-
----
-
-## Term 1 — Node class identifier (`class_type` / `type` / `comfyClass`)
-
-**What it is:** The string that identifies which Python class backs a node. Used to look up `object_info`, serialize the prompt, and match `INPUT_TYPES` definitions.
-
-| Layer | Name | Value example | Notes |
-|-------|------|--------------|-------|
-| Python backend | Python class name | `'KSampler'` | The class registered in `NODE_CLASS_MAPPINGS` |
-| Execution prompt JSON (API format) | `class_type` | `"class_type": "KSampler"` | Key in the flat prompt dict |
-| UWF backend spec | `class_type` | `"class_type": "KSampler"` | Unchanged from API format (per `uwf-backend-data-model.md`) |
-| `ComfyNodeDef` (v1 schema) | `name` | `nodeData.name === 'KSampler'` | The `name` field in the server's `/object_info` response |
-| `LGraphNode` (v1) | `node.type` | `node.type === 'KSampler'` | LiteGraph's class string; set at registration |
-| v2 `NodeHandle` | `handle.type` | `handle.type === 'KSampler'` | `readonly type: string` in `src/extension-api/node.ts:260` |
-| v2 `NodeHandle` | `handle.comfyClass` | `handle.comfyClass === 'KSampler'` | `readonly comfyClass: string` in `src/extension-api/node.ts:269` |
-| ECS `NodeTypeData` component | `type` + `comfyClass` | both fields | Stub in `extension-api-service.ts:61–63` |
-
-**⚠ Inconsistency — `type` vs `comfyClass`:** v2 `NodeHandle` exposes **both** `type` and `comfyClass` because for most nodes they are equal, but for virtual/reroute nodes `type` is the LiteGraph registration string while `comfyClass` is the actual Python class backing the node. Extensions that compare against ComfyUI node names should always use `comfyClass`. Extensions that filter by LiteGraph registration (for `nodeTypes:` filter in `defineNodeExtension`) should use `type`. The distinction must be preserved — collapsing them back to one field would break reroute/virtual-node detection.
-
-**Rule:** `class_type` (wire format, Python, UWF) = `comfyClass` (v2 API) = `name` in `ComfyNodeDef` = `node.type` in LiteGraph for non-virtual nodes.
-
----
-
-## Term 2 — Node display name (`display_name` / `title`)
-
-**What it is:** The human-readable label shown in the node header and search results. Distinct from the class identifier.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend | `NODE_DISPLAY_NAME_MAPPINGS[class]` (optional) | If absent, falls back to the class name |
-| `ComfyNodeDef` (v1 schema) | `display_name` | `nodeData.display_name` — always a string per zod schema (`src/schemas/nodeDefSchema.ts:279`) |
-| `LGraphNode` (v1) | `node.title` | Set during `nodeCreated`; extensions mutate `node.title` directly to rename nodes |
-| v2 `NodeHandle` | `handle.title` (getter) + `handle.setTitle(s)` | `src/extension-api/node.ts:304–315`; accessor pair per D3.3/D6 hybrid rule |
-| ECS `NodeVisualData` component | `title` | Field name consistent with LiteGraph |
-
-**No inconsistency.** `display_name` lives only in the schema/backend layer; `title` is the runtime/frontend term. The two layers don't overlap. Extension authors should use `handle.setTitle()` in v2; `node.title =` is the v1 equivalent. Both are stable.
-
----
-
-## Term 3 — Widget name (`name` / `input_name` / `widget.name`)
-
-**What it is:** The stable per-node-instance identifier for a widget slot. Used as the key in `widgets_values_named`, `promoted_inputs`, and for `WidgetHandle` lookup.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend `INPUT_TYPES` | Dict key (e.g. `'seed'`, `'steps'`) | The name as declared in the Python class |
-| UWF `promoted_inputs` | `input_name` | `{ node_id, input_name, display_name }` — snake_case to match Python origin |
-| UWF `spec.nodes.{id}.inputs` | Named key in the inputs object | e.g. `"seed": { "value": 123, "link": null }` |
-| v1 `LGraphNode.widgets` | `widget.name` | `widget.name === 'seed'` — used in `node.widgets.find(w => w.name === ...)` (S2.N16 pattern) |
-| v1 `node.widgets` iteration | position index (implicit) | `widgets_values` positional array — the root cause of widget shift bugs (S17.WV1) |
-| v2 `NodeHandle.widget(name)` | `name` argument | Lookup by name, not position (`src/extension-api/node.ts:383`) |
-| v2 `NodeHandle.addWidget(type, name, ...)` | `name` parameter | `src/extension-api/node.ts:396–403`; stable key for the widget's lifetime |
-| v2 `WidgetHandle.name` | `readonly name: string` | `src/extension-api/widget.ts:277` — "stable within the node's lifetime" |
-| ECS `WidgetComponentSchema` | `name` field (inferred) | Widget schema component expected to carry `name` per `widgetComponents` import |
-
-**⚠ Inconsistency — positional array vs. named:** v1 serialization stores widget values as a positional array (`widgets_values: [123, 20, 7.5]`). v2 API and UWF both use `name` as the stable key. The bridge is Austin's PR #10392 (`widgets_values_named`). Until that merges, any code that reads `node.widgets[i]` by index is fragile; code that reads `widget.name` is UWF-safe. The v2 `WidgetHandle` **must** be looked up by name, never by index — this is enforced by the API shape.
-
-**Rule:** Always use the Python-declared input name as the canonical widget identifier. Never use position. `widget.name` (v1) = `name` parameter (v2 addWidget) = `input_name` (UWF wire format).
-
----
-
-## Term 4 — Widget type string (`type` / `widgetType` / widget constructor key)
-
-**What it is:** The string describing what kind of widget a slot is (e.g. `'INT'`, `'STRING'`, `'COMBO'`, `'IMAGE'`). Controls which widget constructor is used and which validation rules apply.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend | Return value of `INPUT_TYPES()` tuple: first element | e.g. `("INT", {"default": 42})` — the type string |
-| `ComfyNodeDef` / `InputSpec` | Type string as zod-inferred from the schema | `INPUT_TYPES['required']['steps'] = ['INT', {...}]` |
-| `zBaseInputOptions` | `widgetType` field (optional override) | `src/schemas/nodeDefSchema.ts:33` — overrides the slot type for widget selection; rare |
-| v1 `getCustomWidgets` return | Record key | `{ MY_WIDGET: constructor }` — extension-registered type strings |
-| v1 `node.addWidget(type, ...)` | `type` first arg | The LiteGraph widget constructor key |
-| v2 `NodeHandle.addWidget(type, name, ...)` | `type` first arg | `src/extension-api/node.ts:395, 403` |
-| v2 `WidgetHandle.type` | `readonly type: string` | `src/extension-api/widget.ts` line ~280 |
-| ECS `WidgetComponentSchema` | `type` field | Expected to match the Python-declared type string |
-
-**⚠ Inconsistency — `widgetType` override:** The `widgetType` field in `zBaseInputOptions` is an override that makes the frontend render a different widget than the Python type implies. Extensions that inspect `nodeData.input.required[name][1].widgetType` to determine rendering (S13.SC1 pattern) must check this field **before** using the slot's primary type. The v2 `ComfyNodeDef`-inspection helper (`ctx.inspectNodeDef`) must resolve this correctly — it cannot just return `InputSpec[0]` (the Python type) as `widgetType`.
-
-**Rule:** Python type string = v1 `widget.type` = v2 `WidgetHandle.type` = `widgetType` override if present (takes precedence).
-
----
-
-## Term 5 — Slot type string (connection type / `'IMAGE'`, `'LATENT'`, etc.)
-
-**What it is:** The type label on a node's input/output slot that governs which connections are valid. Distinct from widget type (a slot may be a pure connection point with no widget).
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend | `RETURN_TYPES` tuple element | e.g. `RETURN_TYPES = ('IMAGE', 'MASK')` |
-| Python backend `INPUT_TYPES` | First element of required/optional tuple | `'IMAGE'` means "must receive an IMAGE connection" |
-| UWF `spec.nodes.{id}.outputs` | `{ "name": "IMAGE" }` per output | Output type declarations (new in UWF — not in old API format) |
-| v1 `LGraphNode` slot | `slot.type` | String on the slot object; extensions read/mutate via S15.OS1 |
-| v1 `node.addInput(name, type)` | `type` second arg | e.g. `node.addInput('mask', 'MASK')` |
-| v1 `node.addOutput(name, type)` | `type` second arg | S15.OS1 pattern |
-| v2 `SlotInfo` | `readonly type: string` | `src/extension-api/node.ts:82–83` |
-| v2 slot events | `event.slot.type` | Available in `onSlotConnected`, `onSlotDisconnected` |
-
-**⚠ Inconsistency — dynamic mutation (S15.OS1):** v1 allows `slot.type = 'IMAGE'` and `node.outputs[i].type = newType` at runtime. v2 restricts this: output types must be declared in `INPUT_TYPES` / schema; runtime mutation is only via `node.declareOutputs(spec)` (proposed, not yet implemented). This is an intentional breaking change. The UWF spec formalizes this by requiring `spec.nodes.{id}.outputs` to be declared at save time, not derived from runtime state.
-
-**Rule:** Slot type strings are uppercase by convention (matching Python `RETURN_TYPES`). v2 enforces schema-declaration; mutation-at-runtime is deprecated.
-
----
-
-## Term 6 — Node output name (`output_name` / `RETURN_NAMES`)
-
-**What it is:** Optional human-readable names for a node's output slots. Not the type — the label shown on the output connector.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend | `RETURN_NAMES` class attribute | e.g. `RETURN_NAMES = ('upscaled_image', 'mask')` — optional |
-| `ComfyNodeDef` schema | `output_name` | `z.array(z.string()).optional()` in `src/schemas/nodeDefSchema.ts:275` |
-| v1 `LGraphNode` | `output.name` | String on the slot object; `node.outputs[i].name` |
-| v2 `SlotInfo` | `readonly name: string` | `src/extension-api/node.ts:80–81` — same field name |
-
-**No inconsistency.** `RETURN_NAMES` → `output_name` in the schema → `slot.name` at runtime. All three refer to the same string. Field name shifts from snake_case (`output_name`) in the schema to camelCase-neutral (`name`) on the slot object — consistent with the rest of the frontend.
-
----
-
-## Term 7 — Extension name (`ComfyExtension.name` / `ExtensionOptions.name`)
-
-**What it is:** The unique identifier for an extension used for hook ordering (D10b), scope registry keys, deprecation telemetry, and conflict detection.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| v1 `ComfyExtension` | `name: string` | Required field; `src/types/comfy.ts:108`; typically a dotted namespace like `'Comfy.Sidebar'` |
-| v2 `ExtensionOptions` | `name: string` | Required field; `src/extension-api/lifecycle.ts`; same semantic |
-| ECS scope registry | `extensionName` | Key component of `NodeInstanceScope`; used in `${extensionName}:${nodeEntityId}` scope key |
-| D6 telemetry | `apiVersion` | Separate field added by I-EXT.3 to `ExtensionOptions` for version tracking — not a replacement for `name` |
-
-**No inconsistency.** Same field name and semantics across v1 and v2. The scope registry key format is `${extensionName}:${nodeEntityId}` — both components must be stable.
-
-**Rule:** Extension names should follow the dotted-namespace convention (e.g. `'MyPublisher.MyExtension'`) to avoid collisions. This is currently advisory, not enforced.
-
----
-
-## Term 8 — Node input display name (`display_name` / widget label)
-
-**What it is:** The label shown next to the widget in the UI. Distinct from the internal `name` (key) used for serialization.
-
-| Layer | Name | Notes |
-|-------|------|-------|
-| Python backend | `display_name` in input options dict | `INPUT_TYPES()['required']['steps'] = ['INT', {'display_name': 'Steps'}]` |
-| `zBaseInputOptions` schema | `display_name: z.string().optional()` | `src/schemas/nodeDefSchema.ts:27` |
-| UWF `promoted_inputs` | `display_name` field | `{ node_id, input_name, display_name }` — the UI label for app-mode promoted inputs |
-| v1 `widget` | `widget.label` | Optional; falls back to `widget.name` if absent. Inspected in S2.N16 patterns |
-| v2 `WidgetHandle` | `label` getter | `src/extension-api/widget.ts:355` — "Defaults to the widget name" |
-
-**No inconsistency** in naming — all layers call it `display_name` (schema/wire) or `label` (runtime). The two forms are consistent: `display_name` is the static schema-declared label; `label` is the runtime-settable display string. v2 exposes `label` as a settable accessor; the Python-declared `display_name` becomes its initial value.
-
----
-
-## Summary: real inconsistencies to track
-
-| # | Inconsistency | Risk | Resolution |
-|---|--------------|------|-----------|
-| 1 | `type` vs `comfyClass` on `NodeHandle` — two fields, must not collapse | Medium | Document: use `comfyClass` for Python identity, `type` for LiteGraph registration. Enforced by distinct fields. |
-| 2 | Widget identity: positional index (v1 `widgets_values`) vs name key (v2 / UWF) | **HIGH** | Bridge: Austin's PR #10392 (`widgets_values_named`). v2 `WidgetHandle` is name-only. Never look up by position in v2 code. |
-| 3 | `widgetType` override in `InputSpec` takes precedence over slot type for rendering | Medium | `ctx.inspectNodeDef` must resolve `widgetType` before returning slot type. Do not skip this field. |
-| 4 | Slot type mutation (S15.OS1): `slot.type = X` is valid v1, banned in v2/UWF | Medium | v2 must not expose a `setType()` mutator on `SlotInfo`. Schema-declare outputs; UWF enforces at save time. |
-
----
-
-## Cross-references
-
-- **S2.N16** — widget array iteration/mutation (`node.widgets[i]`, `node.widgets.find(w => w.name === ...)`): the `name` field is the stable key; position is not. v2 forces name-based lookup.
-- **S13.SC1** — `ComfyNodeDef` inspection: callers must resolve `widgetType` override (Term 4) and understand `display_name` vs runtime `label` (Term 8). The `ctx.inspectNodeDef` typed helper (D4 G1 BLOCKER) wraps this correctly.
-- **S15.OS1** — dynamic output mutation: slot `type` strings are the agreed layer (Term 5), but v1 allows mutation that v2/UWF forbids. Track in I-PG.B2 as `strangler-bridge` until UWF Phase 3 covers output schema declaration.
-- **UWF backend data model** — `class_type`, `input_name`, `display_name` snake_case keys mirror Python origin. v2 API uses camelCase (`comfyClass`, widget `name`, widget `label`) per JS convention. No semantic difference; only case convention changes at the API boundary.

docs/architecture/proto-ecs-stores.md

@@ -17,10 +17,6 @@ Six stores extract entity state out of class instances into centralized, queryab
 | NodeOutputStore         | Execution results   | `nodeLocatorId`               | `"${subgraphId}:${nodeId}"`       | Output data, preview URLs     |
 | SubgraphNavigationStore | Canvas viewport     | `subgraphId`                  | `subgraphId` or `'root'`          | LRU viewport cache            |
 
-ADR 0009 refines promoted-widget identity: promoted value widgets are keyed by
-the host boundary (`host node locator + SubgraphInput.name`), while interior
-source node/widget identity is migration and diagnostic metadata only.
-
 ## 2. WidgetValueStore
 
 **File:** `src/stores/widgetValueStore.ts`
@@ -258,9 +254,6 @@ Each store invents its own identity scheme:
 | NodeOutputStore  | `"${subgraphId}:${nodeId}"`       | Composite string        | No         |
 
 In the ECS target, all of these would use branded entity IDs (`WidgetEntityId`, `NodeEntityId`, etc.) with compile-time cross-kind protection.
-For promoted value widgets, ADR 0009 narrows the target key to host boundary
-identity (`host node locator + SubgraphInput.name`) instead of interior source
-identity.
 
 ## 6. Extraction Map