fix: harden manager survey resize, late config, and timeout states

Address PR review:
- Recover from the error state when the survey URL arrives after the
  dialog opens (reactive load state instead of a one-time seed).
- Tolerate a trailing slash in the configured URL when extracting the
  survey id for the resize message match.
- Unmount the iframe in the error/timeout state so the fallback UI no
  longer stacks beneath a blank frame.

.agents/checks/adr-compliance.md

@@ -33,15 +33,15 @@ Flag:
 - **New circular entity dependencies** — New circular imports between `LGraph` ↔ `Subgraph`, `LGraphNode` ↔ `LGraphCanvas`, or similar entity classes.
 - **Direct `graph._version++`** — Mutating the private version counter directly instead of through a public API. Extensions already depend on this side-channel; it must become a proper API.
 
-### Dedicated Stores and Data/Behavior Separation
+### Centralized Registries and ECS-Style Access
 
-Entity data lives in dedicated Pinia stores keyed by string IDs (`widgetValueStore`, `domWidgetStore`, `layoutStore`, `nodeOutputStore`, `subgraphNavigationStore`, `previewExposureStore`), not on entity instances.
+All entity data access should move toward centralized query patterns, not instance property access.
 
 Flag:
 
-- **New instance method/property patterns** — Adding `node.someProperty` or `node.someMethod()` for data that belongs in a dedicated store (e.g. widget values → `widgetValueStore` keyed by `WidgetId`).
+- **New instance method/property patterns** — Adding `node.someProperty` or `node.someMethod()` for data that should be a component in the World, queried via `world.getComponent(entityId, ComponentType)`.
 - **OOP inheritance for entity modeling** — Extending entity classes with new subclasses instead of composing behavior through components and systems.
-- **Duplicated authority** — Storing the same entity state in both a class property and a store, or across two stores, so ownership becomes ambiguous. Each piece of state should have one owning store.
+- **Scattered state** — New entity state stored in multiple locations (class properties, stores, local variables) instead of being consolidated in the World or in a single store.
 
 ### Extension Ecosystem Impact
 

.coderabbit.yaml

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

.github/workflows/ci-dist-telemetry-scan.yaml

@@ -133,24 +133,3 @@ jobs:
             exit 1
           fi
           echo '✅ No Customer.io references found'
-
-      - name: Scan dist for Cloudflare Turnstile sitekey references
-        run: |
-          set -euo pipefail
-          echo '🔍 Scanning for Cloudflare Turnstile sitekeys...'
-          if rg --no-ignore -n \
-            -g '*.html' \
-            -g '*.js' \
-            -e '0x4AAAAAADnYZPVOpFCL_zeo' \
-            -e '0x4AAAAAADnYY4_Q0qxHZ5a7' \
-            -e '1x00000000000000000000AA' \
-            dist; then
-            echo '❌ ERROR: Cloudflare Turnstile sitekey found in dist assets!'
-            echo 'The per-env Turnstile sitekeys are cloud-only and must be tree-shaken from OSS builds.'
-            echo ''
-            echo 'To fix this:'
-            echo '1. Gate sitekey selection on the __DISTRIBUTION__ build define, not the runtime isCloud const'
-            echo '2. See getTurnstileSiteKey() in src/config/turnstile.ts'
-            exit 1
-          fi
-          echo '✅ No Turnstile sitekey references found'

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

@@ -88,9 +88,9 @@ jobs:
       - name: Strip non-source entries from coverage
         if: steps.coverage-shards.outputs.has-coverage == 'true'
         run: |
+          # Drop served bundle scripts (localhost-8188/assets/*.js) that V8 records but have no source file on disk, which would abort genhtml.
           lcov --remove coverage/playwright/coverage.lcov \
             '*localhost-8188*' \
-            'assets/images/*' \
             -o coverage/playwright/coverage.lcov \
             --ignore-errors unused
           wc -l coverage/playwright/coverage.lcov
@@ -121,8 +121,7 @@ jobs:
             --title "ComfyUI E2E Coverage" \
             --no-function-coverage \
             --precision 1 \
-            --ignore-errors source,unmapped,range \
-            --synthesize-missing
+            --ignore-errors source,unmapped
 
       - name: Upload HTML report artifact
         if: steps.coverage-shards.outputs.has-coverage == 'true'

.github/workflows/cla.yml

@@ -1,63 +0,0 @@
-name: CLA Assistant
-
-on:
-  issue_comment:
-    types: [created]
-  pull_request_target:
-    types: [opened, synchronize, closed]
-  merge_group:
-
-permissions:
-  actions: write
-  contents: read # 'read' is enough because signatures live in a REMOTE repo
-  pull-requests: write
-  statuses: write
-
-jobs:
-  cla-assistant:
-    runs-on: ubuntu-latest
-    steps:
-      - name: CLA Assistant
-        # Run on PR events, on "recheck" comment, or when someone posts the exact signing phrase.
-        # IMPORTANT: this phrase must match `custom-pr-sign-comment` below.
-        if: >
-          github.event_name == 'pull_request_target' ||
-          github.event.comment.body == 'recheck' ||
-          github.event.comment.body == 'I have read and agree to the Contributor License Agreement'
-        uses: contributor-assistant/github-action@ca4a40a7d1004f18d9960b404b97e5f30a505a08 # v2.6.1
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          # PAT required to write to the centralized signatures repo.
-          PERSONAL_ACCESS_TOKEN: ${{ secrets.PERSONAL_ACCESS_TOKEN }}
-        with:
-          # Where the CLA document lives (shown to contributors)
-          path-to-document: https://github.com/Comfy-Org/comfy-cla/blob/main/comfyui_icla.md
-
-          # Centralized signature storage
-          remote-organization-name: comfy-org
-          remote-repository-name: comfy-cla
-          path-to-signatures: signatures/cla.json
-          branch: main
-
-          # Allowlist bots so they don't need to sign (optional, comma-separated).
-          # *[bot] is a catch-all for any GitHub App bot account.
-          allowlist: action@github.com,actions-user,ampagent,claude,comfy-pr-bot,GitHub Action,github-actions,Glary Bot,Glary-Bot,*[bot]
-
-          # Custom PR comment messages
-          custom-notsigned-prcomment: |
-            🎉 Thank you for your contribution, we really appreciate it! 🎉
-
-            Like many open source projects, we require contributors to sign our [Contributor License Agreement (CLA)](https://github.com/Comfy-Org/comfy-cla/blob/main/comfyui_icla.md). A CLA makes the ownership of contributions explicit, so contributors and the project share a clear understanding of how the code can be used. By signing, you:
-
-            - Confirm that you own your contribution.
-            - Keep the right to reuse your own code.
-            - Grant us a copyright license to include and share it within our projects.
-
-            CLAs are standard practice across major open source projects including those under the Apache Software Foundation and the Linux Foundation. Ours is based on the Apache Software Foundation's CLA. Most importantly, it would enable us to relicense the project under a more permissive license in the future, giving the project and its community greater flexibility.
-
-            ✍ **To sign, please post a new comment on this PR with exactly the following text:** ✍
-
-          custom-pr-sign-comment: I have read and agree to the Contributor License Agreement
-
-          custom-allsigned-prcomment: |
-            ✅ All contributors have signed the CLA. Thank you! This PR is ready to be merged.

.github/workflows/pr-backport.yaml

@@ -67,11 +67,6 @@ jobs:
         uses: actions/checkout@v6
         with:
           fetch-depth: 0
-          # Persist a token with `workflow` scope so the backport push can
-          # include changes to .github/workflows/**. The default GITHUB_TOKEN
-          # is refused by GitHub when a push creates/updates workflow files,
-          # which silently aborted the whole job (see PR #12804 backport).
-          token: ${{ secrets.PR_GH_TOKEN }}
 
       - name: Configure git
         run: |

.i18nrc.cjs

@@ -21,8 +21,7 @@ module.exports = defineConfig({
     'ar',
     'tr',
     'pt-BR',
-    'fa',
-    'he'
+    'fa'
   ],
   reference: `Special names to keep untranslated: flux, photomaker, clip, vae, cfg, stable audio, stable cascade, stable zero, controlnet, lora, HiDream, Civitai, Hugging Face.
   'latent' is the short form of 'latent space'.
@@ -38,11 +37,5 @@ module.exports = defineConfig({
   - Keep commonly used technical terms in English when they are standard in Persian software (e.g., node, workflow).
   - Use Arabic-Indic numerals (۰-۹) for numbers where appropriate.
   - Maintain consistency with terminology used in Persian software and design applications.
-
-  IMPORTANT Hebrew Translation Guidelines:
-  - For 'he' locale: Use modern, formal Hebrew (עברית תקנית) for a professional tone throughout the UI.
-  - Hebrew is a right-to-left (RTL) language. Keep all interpolation placeholders ({name}, {count}), pipe-separated plural forms, and English technical terms intact and in their original positions.
-  - Preferred glossary: node = צומת (plural צמתים), workflow = תהליך עבודה, queue = תור, canvas = קנבס, widget = פקד, subgraph = תת-גרף, prompt = פרומפט/הנחיה (per context), bypass = עקיפה, mute = השתקה.
-  - Keep widely-recognized technical terms in English (Latin script): API, GPU, CUDA, VAE, CLIP, LoRA, ControlNet, Civitai, Hugging Face, Nodes 2.0, etc.
   `
 })

.storybook/main.ts

@@ -83,16 +83,6 @@ const config: StorybookConfig = {
             replacement:
               process.cwd() + '/src/storybook/mocks/useBillingContext.ts'
           },
-          {
-            find: '@/composables/useFeatureFlags',
-            replacement:
-              process.cwd() + '/src/storybook/mocks/useFeatureFlags.ts'
-          },
-          {
-            find: '@/platform/workspace/stores/teamWorkspaceStore',
-            replacement:
-              process.cwd() + '/src/storybook/mocks/teamWorkspaceStore.ts'
-          },
           {
             find: '@/utils/formatUtil',
             replacement:

AGENTS.md

@@ -179,9 +179,6 @@ This project uses **pnpm**. Always prefer scripts defined in `package.json` (e.g
 23. Favor pure functions (especially testable ones)
 24. Do not use function expressions if it's possible to use function declarations instead
 25. Watch out for [Code Smells](https://wiki.c2.com/?CodeSmell) and refactor to avoid them
-26. Do not add alias helpers whose implementation is just a single-line call to another function
-    - Bad: `function id(value) { return nodeId(value) }`
-    - Use the real function directly, or introduce a named helper only when it adds validation, branching, domain meaning, or shared behavior beyond renaming
 
 ## Design Standards
 
@@ -249,7 +246,7 @@ All architectural decisions are documented in `docs/adr/`. Code changes must be
 ### Entity Architecture Constraints (ADR 0003 + ADR 0008)
 
 1. **Command pattern for all mutations**: Every entity state change must be a serializable, idempotent, deterministic command — replayable, undoable, and transmittable over CRDT. No imperative fire-and-forget mutation APIs. Systems produce command batches, not direct side effects.
-2. **Dedicated stores over instance state**: Entity data lives in dedicated Pinia stores keyed by string IDs — widget values in `widgetValueStore` keyed by `WidgetId` (`graphId:nodeId:name`, see `src/types/widgetId.ts`), plus `domWidgetStore`, `layoutStore`, `nodeOutputStore`, `subgraphNavigationStore`, and `previewExposureStore`. Prefer a focused store to a single unified registry. Do not add new instance properties/methods to entity classes for data that belongs in a store. Do not use OOP inheritance for entity modeling.
+2. **Centralized registries and ECS-style access**: Entity data lives in the World (centralized registry), queried via `world.getComponent(entityId, ComponentType)`. Do not add new instance properties/methods to entity classes. Do not use OOP inheritance for entity modeling.
 3. **No god-object growth**: Do not add methods to `LGraphNode`, `LGraphCanvas`, `LGraph`, or `Subgraph`. Extract to systems, stores, or composables.
 4. **Plain data components**: ECS components are plain data objects — no methods, no back-references to parent entities. Behavior belongs in systems (pure functions).
 5. **Extension ecosystem impact**: Changes to entity callbacks (`onConnectionsChange`, `onRemoved`, `onAdded`, `onConnectInput/Output`, `onConfigure`, `onWidgetChanged`), `node.widgets` access, `node.serialize`, or `graph._version++` affect 40+ custom node repos and require migration guidance.

apps/website/e2e/download.spec.ts

@@ -47,11 +47,6 @@ test.describe('Download page @smoke', () => {
     const downloadBtn = hero.getByRole('link', { name: /DOWNLOAD DESKTOP/i })
     await expect(downloadBtn).toBeVisible()
     await expect(downloadBtn).toHaveAttribute('target', '_blank')
-    await expect(downloadBtn).toHaveAttribute(
-      'href',
-      'https://comfy.org/download/windows/nsis/x64'
-    )
-    await expect(downloadBtn).toHaveAttribute('data-astro-prefetch', 'false')
 
     const githubBtn = hero.getByRole('link', { name: /INSTALL FROM GITHUB/i })
     await expect(githubBtn).toBeVisible()
@@ -78,7 +73,7 @@ test.describe('Download page @smoke', () => {
     })
 
     const windowsBtn = hero.locator(
-      'a[href="https://comfy.org/download/windows/nsis/x64"]'
+      'a[href="https://download.comfy.org/windows/nsis/x64"]'
     )
     await expect(windowsBtn).toBeVisible()
     await expect(windowsBtn).toHaveText(/DOWNLOAD DESKTOP/i)

apps/website/e2e/launches.spec.ts

@@ -1,231 +0,0 @@
-import type { Page } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import { externalLinks } from '../src/config/routes'
-import { drops } from '../src/data/drops'
-import type { Locale } from '../src/i18n/translations'
-import { t } from '../src/i18n/translations'
-import { test } from './fixtures/blockExternalMedia'
-
-const PATH_EN = '/launches'
-const PATH_ZH = '/zh-CN/launches'
-const CLOUD_URL = 'https://cloud.comfy.org'
-
-const LOCALES: ReadonlyArray<readonly [string, Locale]> = [
-  [PATH_EN, 'en'],
-  [PATH_ZH, 'zh-CN']
-]
-
-function heroSection(page: Page, locale: Locale) {
-  return page.locator('section').filter({
-    has: page.getByRole('heading', {
-      level: 1,
-      name: t('launches.hero.title', locale)
-    })
-  })
-}
-
-function ctaSection(page: Page, locale: Locale) {
-  return page.locator('section').filter({
-    has: page.getByRole('heading', {
-      level: 2,
-      name: t('launches.cta.heading', locale)
-    })
-  })
-}
-
-function dropsSection(page: Page, locale: Locale) {
-  return page.locator('section').filter({
-    has: page.getByRole('heading', {
-      level: 2,
-      name: t('launches.section.title', locale)
-    })
-  })
-}
-
-test.describe('Launches landing — desktop @smoke', () => {
-  test('renders the configured title at /launches', async ({ page }) => {
-    await page.goto(PATH_EN)
-    await expect(page).toHaveTitle(t('launches.page.title', 'en'))
-  })
-
-  test('renders the localized title at /zh-CN/launches', async ({ page }) => {
-    await page.goto(PATH_ZH)
-    await expect(page).toHaveTitle(t('launches.page.title', 'zh-CN'))
-  })
-
-  test('is indexable at both locales', async ({ page }) => {
-    await page.goto(PATH_EN)
-    await expect(page.locator('meta[name="robots"]')).toHaveCount(0)
-
-    await page.goto(PATH_ZH)
-    await expect(page.locator('meta[name="robots"]')).toHaveCount(0)
-  })
-
-  test('hero h1 renders the localized title in both locales', async ({
-    page
-  }) => {
-    await page.goto(PATH_EN)
-    await expect(
-      page.getByRole('heading', {
-        level: 1,
-        name: t('launches.hero.title', 'en')
-      })
-    ).toBeVisible()
-
-    await page.goto(PATH_ZH)
-    await expect(
-      page.getByRole('heading', {
-        level: 1,
-        name: t('launches.hero.title', 'zh-CN')
-      })
-    ).toBeVisible()
-  })
-
-  test('hero primary CTA links to /download per locale', async ({ page }) => {
-    for (const [path, locale, expectedHref] of [
-      [PATH_EN, 'en', '/download'],
-      [PATH_ZH, 'zh-CN', '/zh-CN/download']
-    ] as const) {
-      await page.goto(path)
-      const primary = heroSection(page, locale).getByRole('link', {
-        name: t('launches.hero.primary', locale)
-      })
-      await expect(primary).toBeVisible()
-      await expect(primary).toHaveAttribute('href', expectedHref)
-    }
-  })
-
-  test('hero secondary CTA opens external Cloud in a new tab on both locales', async ({
-    page
-  }) => {
-    for (const [path, locale] of LOCALES) {
-      await page.goto(path)
-      const secondary = heroSection(page, locale).getByRole('link', {
-        name: t('launches.hero.secondary', locale)
-      })
-      await expect(secondary).toBeVisible()
-      await expect(secondary).toHaveAttribute('href', CLOUD_URL)
-      await expect(secondary).toHaveAttribute('target', '_blank')
-      await expect(secondary).toHaveAttribute('rel', 'noopener noreferrer')
-    }
-  })
-
-  test('closing CTA shows heading and both action buttons in both locales', async ({
-    page
-  }) => {
-    for (const [path, locale] of LOCALES) {
-      await page.goto(path)
-      const section = ctaSection(page, locale)
-      await expect(
-        section.getByRole('heading', {
-          level: 2,
-          name: t('launches.cta.heading', locale)
-        })
-      ).toBeVisible()
-
-      const primary = section.getByRole('link', {
-        name: t('launches.cta.primary', locale)
-      })
-      await expect(primary).toBeVisible()
-      await expect(primary).toHaveAttribute('href', externalLinks.cloud)
-      await expect(primary).toHaveAttribute('target', '_blank')
-      await expect(primary).toHaveAttribute('rel', 'noopener noreferrer')
-
-      const secondary = section.getByRole('link', {
-        name: t('launches.cta.secondary', locale)
-      })
-      await expect(secondary).toBeVisible()
-      await expect(secondary).toHaveAttribute('href', externalLinks.workflows)
-      await expect(secondary).toHaveAttribute('target', '_blank')
-      await expect(secondary).toHaveAttribute('rel', 'noopener noreferrer')
-    }
-  })
-
-  test('drops section renders one card per data entry with the correct localized href in both locales', async ({
-    page
-  }) => {
-    for (const [path, locale] of LOCALES) {
-      await page.goto(path)
-      const section = dropsSection(page, locale)
-
-      await expect(
-        section.getByRole('heading', {
-          level: 2,
-          name: t('launches.section.title', locale)
-        })
-      ).toBeVisible()
-
-      const cards = section.locator('[data-slot="card"]')
-      await expect(cards).toHaveCount(drops.length)
-
-      for (const [i, drop] of drops.entries()) {
-        const card = cards.nth(i)
-        await expect(card).toContainText(drop.title[locale])
-        const explore = card.getByRole('link', {
-          name: drop.cta.label[locale]
-        })
-        await expect(explore).toBeVisible()
-        await expect(explore).toHaveAttribute('href', drop.cta.href[locale])
-      }
-    }
-  })
-
-  test('desktop: first 4 drop cards are wider than cards 5+', async ({
-    page
-  }) => {
-    await page.goto(PATH_EN)
-    const cards = dropsSection(page, 'en').locator('[data-slot="card"]')
-    await expect(cards).toHaveCount(drops.length)
-
-    await expect
-      .poll(async () => {
-        const firstWidth = (await cards.nth(0).boundingBox())?.width ?? 0
-        const fifthWidth = (await cards.nth(4).boundingBox())?.width ?? 0
-        return firstWidth - fifthWidth
-      })
-      .toBeGreaterThan(0)
-  })
-})
-
-test.describe('Launches landing — mobile @mobile', () => {
-  test('drops grid stacks in a single column at mobile width', async ({
-    page
-  }) => {
-    await page.goto(PATH_EN)
-    const cards = dropsSection(page, 'en').locator('[data-slot="card"]')
-    await expect(cards).toHaveCount(drops.length)
-
-    const viewport = page.viewportSize()
-    expect(viewport, 'viewport size').not.toBeNull()
-
-    await expect
-      .poll(async () => (await cards.nth(0).boundingBox())?.width ?? 0)
-      .toBeGreaterThanOrEqual(viewport!.width * 0.7)
-
-    await expect
-      .poll(async () => {
-        const firstBox = await cards.nth(0).boundingBox()
-        const secondBox = await cards.nth(1).boundingBox()
-        if (!firstBox || !secondBox) return false
-        return secondBox.y >= firstBox.y + firstBox.height
-      })
-      .toBe(true)
-  })
-
-  test('closing CTA heading stays within viewport width', async ({ page }) => {
-    await page.goto(PATH_EN)
-    const heading = page.getByRole('heading', {
-      level: 2,
-      name: t('launches.cta.heading', 'en')
-    })
-    await heading.scrollIntoViewIfNeeded()
-    await expect(heading).toBeVisible()
-
-    const box = await heading.boundingBox()
-    expect(box, 'CTA heading bounding box').not.toBeNull()
-    const viewport = page.viewportSize()
-    expect(viewport, 'viewport size').not.toBeNull()
-    expect(box!.x + box!.width).toBeLessThanOrEqual(viewport!.width + 1)
-  })
-})

apps/website/public/robots.txt

@@ -29,5 +29,6 @@ Allow: /
 Disallow: /_astro/
 Disallow: /_website/
 Disallow: /_vercel/
+Disallow: /payment/
 
 Sitemap: https://comfy.org/sitemap-index.xml

apps/website/src/components/blocks/CtaCenter01.vue

@@ -1,14 +1,10 @@
 <script setup lang="ts">
-import type { AnchorHTMLAttributes } from 'vue'
-
-import Button from '../ui/button/Button.vue'
-import { resolveRel } from '../../utils/cta'
+import BrandButton from '../common/BrandButton.vue'
 
 type Cta = {
   label: string
   href: string
-  target?: AnchorHTMLAttributes['target']
-  rel?: AnchorHTMLAttributes['rel']
+  target?: '_blank' | '_self' | '_parent' | '_top'
 }
 
 type TermsLink = {
@@ -16,11 +12,10 @@ type TermsLink = {
   href: string
 }
 
-const { heading, primaryCta, secondaryCta, termsLink } = defineProps<{
+defineProps<{
   heading: string
   primaryCta: Cta
-  secondaryCta?: Cta
-  termsLink?: TermsLink
+  termsLink: TermsLink
 }>()
 </script>
 
@@ -29,37 +24,23 @@ const { heading, primaryCta, secondaryCta, termsLink } = defineProps<{
     class="max-w-9xl mx-auto flex flex-col items-center px-6 py-16 text-center lg:py-24"
   >
     <h2
-      class="max-w-3xl text-4xl/snug font-light tracking-tight text-pretty text-primary-comfy-canvas lg:text-6xl/snug"
+      class="text-4xl font-light tracking-tight text-primary-comfy-canvas lg:text-6xl"
     >
       {{ heading }}
     </h2>
 
-    <div class="mt-10 flex flex-col gap-4 sm:flex-row lg:mt-12">
-      <Button
-        as="a"
-        :href="primaryCta.href"
-        :target="primaryCta.target"
-        :rel="resolveRel(primaryCta)"
-        variant="default"
-        size="lg"
-      >
-        {{ primaryCta.label }}
-      </Button>
-      <Button
-        v-if="secondaryCta"
-        as="a"
-        :href="secondaryCta.href"
-        :target="secondaryCta.target"
-        :rel="resolveRel(secondaryCta)"
-        variant="outline"
-        size="lg"
-      >
-        {{ secondaryCta.label }}
-      </Button>
-    </div>
+    <BrandButton
+      :href="primaryCta.href"
+      :target="primaryCta.target"
+      :rel="primaryCta.target === '_blank' ? 'noopener noreferrer' : undefined"
+      variant="outline"
+      size="lg"
+      class="mt-10 px-20 py-4 text-base uppercase lg:mt-12"
+    >
+      {{ primaryCta.label }}
+    </BrandButton>
 
     <a
-      v-if="termsLink"
       :href="termsLink.href"
       class="mt-8 text-sm text-primary-comfy-canvas/70 underline underline-offset-4 transition-colors hover:text-primary-comfy-canvas"
     >

apps/website/src/components/blocks/HeroLivestream01.vue

@@ -1,166 +0,0 @@
-<script setup lang="ts">
-import type { AnchorHTMLAttributes } from 'vue'
-import { computed, onMounted, ref } from 'vue'
-import { useNow } from '@vueuse/core'
-
-import Button from '../ui/button/Button.vue'
-import { resolveRel } from '../../utils/cta'
-
-type Cta = {
-  label: string
-  href: string
-  target?: AnchorHTMLAttributes['target']
-  rel?: AnchorHTMLAttributes['rel']
-}
-
-type Visual =
-  | {
-      type: 'image'
-      src: string
-      alt: string
-      width?: number
-      height?: number
-    }
-  | {
-      type: 'video'
-      src: string
-      alt: string
-      poster?: string
-      width?: number
-      height?: number
-    }
-
-const {
-  visual,
-  eyebrow,
-  title,
-  subtitle,
-  primaryCta,
-  secondaryCta,
-  youtubeVideoId,
-  startDateTime,
-  endDateTime
-} = defineProps<{
-  visual?: Visual
-  eyebrow?: string
-  title: string
-  subtitle?: string
-  primaryCta: Cta
-  secondaryCta?: Cta
-  youtubeVideoId: string
-  startDateTime: string
-  endDateTime: string
-}>()
-
-const embedUrl = computed(
-  () =>
-    `https://www.youtube-nocookie.com/embed/${youtubeVideoId}?autoplay=1&mute=1&rel=0`
-)
-
-// Keep SSR/initial paint deterministic on the logo and only flip to the embed
-// after client hydration — avoids a build-time `now` leaking into the markup.
-const mounted = ref(false)
-onMounted(() => {
-  mounted.value = true
-})
-
-const now = useNow({ interval: 30_000 })
-const startMs = computed(() => new Date(startDateTime).getTime())
-const endMs = computed(() => new Date(endDateTime).getTime())
-
-const isLive = computed(
-  () =>
-    mounted.value &&
-    now.value.getTime() >= startMs.value &&
-    now.value.getTime() < endMs.value
-)
-</script>
-
-<template>
-  <section
-    class="max-w-9xl mx-auto flex flex-col items-center px-6 py-16 text-center lg:py-24"
-  >
-    <div
-      v-if="isLive"
-      class="mb-10 aspect-video w-full overflow-hidden rounded-2xl lg:mb-12"
-    >
-      <iframe
-        :src="embedUrl"
-        :title="title"
-        class="size-full"
-        loading="lazy"
-        allow="autoplay; encrypted-media; picture-in-picture"
-        allowfullscreen
-      />
-    </div>
-    <slot v-else name="visual">
-      <img
-        v-if="visual?.type === 'image'"
-        :src="visual.src"
-        :alt="visual.alt"
-        :width="visual.width"
-        :height="visual.height"
-        fetchpriority="high"
-        decoding="async"
-        class="mb-10 h-auto w-full max-w-md lg:mb-12 lg:max-w-lg"
-      />
-      <video
-        v-else-if="visual?.type === 'video'"
-        :src="visual.src"
-        :poster="visual.poster"
-        :aria-label="visual.alt"
-        :width="visual.width"
-        :height="visual.height"
-        autoplay
-        loop
-        muted
-        playsinline
-        preload="metadata"
-        class="mb-10 h-auto w-full max-w-md lg:mb-12 lg:max-w-2xl"
-      />
-    </slot>
-
-    <p
-      v-if="eyebrow"
-      class="mb-4 text-sm font-medium tracking-wide text-primary-comfy-canvas/70 uppercase"
-    >
-      {{ eyebrow }}
-    </p>
-
-    <h1
-      class="max-w-3xl text-4xl/snug font-light tracking-tight text-pretty text-primary-comfy-canvas lg:text-6xl/snug"
-    >
-      {{ title }}
-    </h1>
-
-    <p
-      v-if="subtitle"
-      class="mt-6 max-w-2xl text-base text-primary-comfy-canvas/70 lg:text-lg"
-    >
-      {{ subtitle }}
-    </p>
-
-    <div class="mt-10 flex flex-col gap-4 sm:flex-row lg:mt-12">
-      <Button
-        as="a"
-        :href="primaryCta.href"
-        :target="primaryCta.target"
-        :rel="resolveRel(primaryCta)"
-        size="lg"
-      >
-        {{ primaryCta.label }}
-      </Button>
-      <Button
-        v-if="secondaryCta"
-        as="a"
-        :href="secondaryCta.href"
-        :target="secondaryCta.target"
-        :rel="resolveRel(secondaryCta)"
-        variant="outline"
-        size="lg"
-      >
-        {{ secondaryCta.label }}
-      </Button>
-    </div>
-  </section>
-</template>

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

@@ -6,7 +6,6 @@ import type { HTMLAttributes } from 'vue'
 
 import type { BrandButtonVariants } from './brandButton.variants'
 import { brandButtonVariants } from './brandButton.variants'
-import { resolveRel } from '../../utils/cta'
 
 const props = defineProps<{
   href?: string
@@ -17,8 +16,9 @@ const props = defineProps<{
   class?: HTMLAttributes['class']
 }>()
 
-const resolvedRel = computed(() =>
-  resolveRel({ rel: props.rel, target: props.target })
+const resolvedRel = computed(
+  () =>
+    props.rel ?? (props.target === '_blank' ? 'noopener noreferrer' : undefined)
 )
 </script>
 

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

@@ -44,7 +44,6 @@ const topColumns: { title: string; links: FooterLink[] }[] = [
     title: t('footer.resources', locale),
     links: [
       { label: t('nav.learning', locale), href: routes.learning },
-      { label: t('nav.launches', locale), href: routes.launches },
       {
         label: t('footer.blog', locale),
         href: externalLinks.blog,

apps/website/src/components/product/local/DownloadLocalButton.vue

@@ -72,7 +72,6 @@ const buttons = computed<ButtonSpec[]>(() => {
     size="lg"
     :class="customClass"
     :aria-label="btn.ariaLabel"
-    :data-astro-prefetch="btn.key === 'windows' ? 'false' : undefined"
     @click="captureDownloadClick(btn.key)"
   >
     <span class="inline-flex items-center gap-2">

apps/website/src/components/ui/badge/Badge.vue

@@ -25,7 +25,7 @@ const {
     data-slot="badge"
     :data-variant="variant"
     :data-size="size"
-    :class="cn(badgeVariants({ size, variant }), className)"
+    :class="cn(badgeVariants({ variant, size }), className)"
   >
     <slot name="prepend">
       <component :is="prependIcon" v-if="prependIcon" />

apps/website/src/components/ui/badge/index.ts

@@ -4,16 +4,15 @@ import { cva } from 'cva'
 export const badgeVariants = cva({
   base: 'text-primary-warm-gray font-formula leading-none focus-visible:border-ring focus-visible:ring-ring/50 aria-invalid:border-destructive aria-invalid:ring-destructive/20 inline-flex w-fit shrink-0 items-center justify-center gap-1 overflow-hidden rounded-full border border-transparent font-medium whitespace-nowrap transition-[color,box-shadow] focus-visible:ring-[3px] [&>svg]:pointer-events-none [&>svg]:size-3',
   variants: {
-    size: {
-      md: 'px-4 py-1 text-xs',
-      xs: 'px-2 py-0.5 text-[9px]'
-    },
     variant: {
       default: 'bg-transparency-ink-t80',
       subtle: 'bg-transparency-white-t4 text-primary-comfy-canvas',
-      category: 'text-primary-comfy-yellow px-0 font-semibold uppercase',
       accent:
         'before:bg-primary-comfy-yellow relative isolate overflow-visible rounded-none bg-transparent px-2 py-0.5 text-[9px] font-bold tracking-wide text-primary-comfy-ink uppercase before:absolute before:inset-0 before:-z-10 before:-skew-x-12 before:rounded-sm'
+    },
+    size: {
+      md: 'px-4 py-1 text-xs',
+      xs: 'px-2 py-0.5 text-[9px]'
     }
   },
   defaultVariants: {

apps/website/src/components/ui/button/index.ts

@@ -8,8 +8,7 @@ export const buttonVariants = cva(
   {
     variants: {
       size: {
-        sm: 'h-8 px-4 py-2 text-xs md:text-sm',
-        default: 'h-10 px-6 py-2.5 text-xs md:text-sm',
+        default: 'h-10 px-6 py-2.5',
         lg: 'h-14 px-8 py-4 text-base'
       },
       variant: {
@@ -18,8 +17,6 @@ export const buttonVariants = cva(
         outline:
           'text-primary-comfy-yellow hover:bg-primary-comfy-yellow border uppercase hover:text-primary-comfy-ink',
         link: "text-primary-comfy-yellow h-auto justify-start px-0 py-1 text-base uppercase hover:opacity-90 [&_svg:not([class*='size-'])]:size-6",
-        underlineLink:
-          "text-primary-comfy-yellow relative h-auto justify-start px-0 py-1 uppercase after:absolute after:bottom-0 after:left-0 after:h-0.5 after:w-full after:origin-left after:scale-x-0 after:bg-current after:transition-transform after:duration-200 hover:opacity-90 hover:after:scale-x-100 [&_svg:not([class*='size-'])]:size-6",
         nav: 'text-primary-warm-white hover:text-primary-comfy-yellow h-auto justify-between px-0 py-1 text-start text-2xl font-medium',
         navMuted:
           'hover:text-primary-comfy-yellow h-auto w-full justify-between px-0 py-1 text-start text-2xl font-medium text-primary-comfy-canvas uppercase'

apps/website/src/components/ui/card/Card.vue

@@ -1,22 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div
-    data-slot="card"
-    :class="
-      cn(
-        'bg-transparency-white-t4 text-primary-warm-white rounded-4.5xl flex flex-col gap-6 shadow-sm',
-        className
-      )
-    "
-  >
-    <slot />
-  </div>
-</template>

apps/website/src/components/ui/card/CardContent.vue

@@ -1,14 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div data-slot="card-content" :class="cn('px-6', className)">
-    <slot />
-  </div>
-</template>

apps/website/src/components/ui/card/CardDescription.vue

@@ -1,19 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div
-    data-slot="card-description"
-    :class="
-      cn('line-clamp-3 text-base text-primary-comfy-canvas/70', className)
-    "
-  >
-    <slot />
-  </div>
-</template>

apps/website/src/components/ui/card/CardFooter.vue

@@ -1,14 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div data-slot="card-footer" :class="cn('flex items-center', className)">
-    <slot />
-  </div>
-</template>

apps/website/src/components/ui/card/CardHeader.vue

@@ -1,14 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div data-slot="card-header" :class="cn('flex flex-col gap-1.5', className)">
-    <slot />
-  </div>
-</template>

apps/website/src/components/ui/card/CardTitle.vue

@@ -1,22 +0,0 @@
-<script setup lang="ts">
-import type { HTMLAttributes } from 'vue'
-import { cn } from '@comfyorg/tailwind-utils'
-
-const { class: className } = defineProps<{
-  class?: HTMLAttributes['class']
-}>()
-</script>
-
-<template>
-  <div
-    data-slot="card-title"
-    :class="
-      cn(
-        'text-xl leading-none font-medium text-primary-comfy-canvas md:text-2xl',
-        className
-      )
-    "
-  >
-    <slot />
-  </div>
-</template>

apps/website/src/composables/useDownloadUrl.ts

@@ -3,7 +3,7 @@ import { computed, onMounted, ref } from 'vue'
 import { externalLinks } from '@/config/routes'
 
 export const downloadUrls = {
-  windows: 'https://comfy.org/download/windows/nsis/x64',
+  windows: 'https://download.comfy.org/windows/nsis/x64',
   macArm: 'https://download.comfy.org/mac/dmg/arm64'
 } as const
 

apps/website/src/composables/useHeroLogo.ts

@@ -22,12 +22,6 @@ interface HeroLogoConfig {
   cursorTiltStrength: number
   bgScale: number
   slideDuration: number
-  svgMarkup: string
-  fitAxis: 'width' | 'height'
-  targetSize: number
-  respectReducedMotion: boolean
-  baseUrl: string
-  fadeInDurationMs: number
 }
 
 const DEFAULTS: HeroLogoConfig = {
@@ -40,25 +34,19 @@ const DEFAULTS: HeroLogoConfig = {
   extrudeDepth: 200,
   cursorTiltStrength: 0.5,
   bgScale: 0.8,
-  slideDuration: 0.4,
-  svgMarkup: SVG_MARKUP,
-  fitAxis: 'height',
-  targetSize: 3,
-  respectReducedMotion: true,
-  baseUrl: BASE_URL,
-  fadeInDurationMs: 0
+  slideDuration: 0.4
 }
 
-function buildImageUrls(baseUrl: string): string[] {
+function buildImageUrls(): string[] {
   return Array.from({ length: IMAGE_COUNT }, (_, i) => {
     const index = String(i).padStart(5, '0')
-    return `${baseUrl}/image_sequence_${index}.webp`
+    return `${BASE_URL}/image_sequence_${index}.webp`
   })
 }
 
-function parseShapes(markup: string): THREE.Shape[] {
+function parseShapes(): THREE.Shape[] {
   const loader = new SVGLoader()
-  const svgData = loader.parse(markup)
+  const svgData = loader.parse(SVG_MARKUP)
   const shapes: THREE.Shape[] = []
   svgData.paths.forEach((path) => {
     shapes.push(...SVGLoader.createShapes(path))
@@ -97,8 +85,7 @@ export function useHeroLogo(
   onMounted(async () => {
     try {
       const container = containerRef.value
-      if (!container || (cfg.respectReducedMotion && prefersReducedMotion()))
-        return
+      if (!container || prefersReducedMotion()) return
 
       const { width, height } = container.getBoundingClientRect()
 
@@ -115,9 +102,6 @@ export function useHeroLogo(
       renderer.domElement.style.width = '100%'
       renderer.domElement.style.height = '100%'
       renderer.domElement.style.opacity = '0'
-      if (cfg.fadeInDurationMs > 0) {
-        renderer.domElement.style.transition = `opacity ${cfg.fadeInDurationMs}ms ease`
-      }
       renderer.domElement.setAttribute('aria-hidden', 'true')
       container.appendChild(renderer.domElement)
 
@@ -142,36 +126,24 @@ export function useHeroLogo(
       camera.position.z = cfg.zoom
 
       // SVG shape
-      const shapes = parseShapes(cfg.svgMarkup)
+      const shapes = parseShapes()
       const tempGeo = new THREE.ShapeGeometry(shapes)
       tempGeo.computeBoundingBox()
-      const bb = tempGeo.boundingBox
-      if (!bb) {
-        tempGeo.dispose()
-        cleanup?.()
-        return
-      }
+      const bb = tempGeo.boundingBox!
       const cx = (bb.max.x + bb.min.x) / 2
       const cy = (bb.max.y + bb.min.y) / 2
-      const fitExtent =
-        cfg.fitAxis === 'width' ? bb.max.x - bb.min.x : bb.max.y - bb.min.y
-      if (fitExtent <= 0) {
-        tempGeo.dispose()
-        cleanup?.()
-        return
-      }
-      const scaleFactor = cfg.targetSize / fitExtent
+      const scaleFactor = 3 / (bb.max.y - bb.min.y)
       tempGeo.dispose()
 
       // Image sequence textures — load first frame eagerly, rest lazily
-      const urls = buildImageUrls(cfg.baseUrl)
+      const urls = buildImageUrls()
       const textures = await loadTextures(urls.slice(0, 1))
       if (disposed) return
 
       renderer.domElement.style.opacity = '1'
       loaded.value = true
 
-      void loadTextures(urls.slice(1)).then((rest) => {
+      loadTextures(urls.slice(1)).then((rest) => {
         if (!disposed) textures.push(...rest)
       })
 

apps/website/src/config/routes.ts

@@ -8,7 +8,6 @@ const baseRoutes = {
   cloudEnterprise: '/cloud/enterprise',
   api: '/api',
   gallery: '/gallery',
-  launches: '/launches',
   about: '/about',
   careers: '/careers',
   customers: '/customers',
@@ -60,7 +59,6 @@ export const externalLinks = {
   discord: 'https://discord.com/invite/comfyorg',
   docs: 'https://docs.comfy.org/',
   docsApi: 'https://docs.comfy.org/api-reference/cloud',
-  docsMcp: 'https://docs.comfy.org/agent-tools/cloud',
   docsSubscription: 'https://docs.comfy.org/support/subscription/subscribing',
   github: 'https://github.com/Comfy-Org/ComfyUI',
   githubInstall: 'https://github.com/Comfy-Org/ComfyUI#installing',

apps/website/src/data/drops.ts

@@ -1,248 +0,0 @@
-// Image URLs are placeholders at media.comfy.org/website/drops/<id>.png —
-// asset uploads and native zh-CN review are pending follow-ups (see
-// apps/website/.scratch/drops-page/PRD.md).
-import { externalLinks } from '../config/routes'
-import type { LocalizedText } from '../i18n/translations'
-
-type DropMedia =
-  | { type: 'image'; src: string; alt: LocalizedText }
-  | { type: 'video'; src: string; alt: LocalizedText; poster?: string }
-
-export type Drop = {
-  id: string
-  badge?: LocalizedText
-  category: LocalizedText
-  media: DropMedia
-  title: LocalizedText
-  description: LocalizedText
-  cta: { label: LocalizedText; href: LocalizedText }
-}
-
-const EXPLORE: LocalizedText = { en: 'EXPLORE', 'zh-CN': '探索' }
-const PLATFORM: LocalizedText = { en: 'Platform', 'zh-CN': '平台' }
-const CLOUD: LocalizedText = { en: 'Cloud', 'zh-CN': '云端' }
-const COMMUNITY: LocalizedText = { en: 'Community', 'zh-CN': '社区' }
-const DEVELOPER: LocalizedText = { en: 'Developer', 'zh-CN': '开发者' }
-const MODELS_AND_NODES: LocalizedText = {
-  en: 'Models & Nodes',
-  'zh-CN': '模型与节点'
-}
-const NEW_BADGE: LocalizedText = { en: 'NEW', 'zh-CN': '新' }
-const FEATURED_BADGE: LocalizedText = { en: 'FEATURED', 'zh-CN': '精选' }
-
-function imageFor(fileName: string, alt: LocalizedText): DropMedia {
-  return {
-    type: 'image',
-    src: `https://media.comfy.org/website/drops/${fileName}`,
-    alt
-  }
-}
-
-function videoFor(
-  fileName: string,
-  alt: LocalizedText,
-  poster?: string
-): DropMedia {
-  return {
-    type: 'video',
-    src: `https://media.comfy.org/website/drops/${fileName}`,
-    alt,
-    ...(poster && {
-      poster: `https://media.comfy.org/website/drops/${poster}`
-    })
-  }
-}
-
-export const drops: readonly Drop[] = [
-  {
-    id: 'desktop-client',
-    badge: NEW_BADGE,
-    category: PLATFORM,
-    media: imageFor('Drops_2x2card_Desktop.jpg', {
-      en: 'New Desktop Client',
-      'zh-CN': '新桌面客户端'
-    }),
-    title: { en: 'New Desktop Client', 'zh-CN': '新桌面客户端' },
-    description: {
-      en: 'A faster, redesigned desktop app for ComfyUI — one-click install and managed updates.',
-      'zh-CN': '更快、重新设计的 ComfyUI 桌面应用程序 — 一键安装与受管更新。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: '/download', 'zh-CN': '/zh-CN/download' }
-    }
-  },
-  {
-    id: 'app-mode',
-    badge: NEW_BADGE,
-    category: PLATFORM,
-    media: videoFor('Drops_2x2card_APP.mp4', {
-      en: 'App Mode',
-      'zh-CN': 'App 模式'
-    }),
-    title: { en: 'App Mode', 'zh-CN': 'App 模式' },
-    description: {
-      en: 'A simplified view of your workflows. Flip back to the node graph anytime to go deeper.',
-      'zh-CN': '工作流的简化视图。随时切换回节点图视图以深入了解。'
-    },
-    // TODO: no destination page yet — link out when App Mode lands.
-    cta: {
-      label: EXPLORE,
-      href: {
-        en: 'https://docs.comfy.org/interface/app-mode',
-        'zh-CN': 'https://docs.comfy.org/zh/interface/app-mode'
-      }
-    }
-  },
-  {
-    id: 'comfy-api',
-    badge: NEW_BADGE,
-    category: DEVELOPER,
-    media: imageFor('Drops_2x2card_API.jpg', {
-      en: 'Comfy API',
-      'zh-CN': 'Comfy API'
-    }),
-    title: { en: 'Comfy API', 'zh-CN': 'Comfy API' },
-    description: {
-      en: 'Turn any workflow into a production endpoint. Automate generation and scale to thousands of outputs.',
-      'zh-CN': '将任意工作流变成生产端点。自动化生成并扩展到数千个输出。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: '/api', 'zh-CN': '/zh-CN/api' }
-    }
-  },
-  {
-    id: 'comfy-mcp',
-    badge: NEW_BADGE,
-    category: CLOUD,
-    media: imageFor('Drops_2x2card_MCP.jpg', {
-      en: 'Comfy MCP',
-      'zh-CN': 'Comfy MCP'
-    }),
-    title: { en: 'Comfy MCP', 'zh-CN': 'Comfy MCP' },
-    description: {
-      en: 'The full power of ComfyUI from anywhere — no setup, no GPU required.',
-      'zh-CN': '随时随地体验 ComfyUI 的全部能力 — 无需配置，无需 GPU。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: externalLinks.docsMcp, 'zh-CN': externalLinks.docsMcp }
-    }
-  },
-  {
-    id: 'community-workflows',
-    category: COMMUNITY,
-    media: imageFor('Drops_3x3card_Comm Workflows.jpg', {
-      en: 'Community Workflows',
-      'zh-CN': '社区工作流'
-    }),
-    title: {
-      en: 'Community Workflows',
-      'zh-CN': '社区工作流'
-    },
-    description: {
-      en: 'Browse and remix thousands of community-shared workflows. Start from a proven template.',
-      'zh-CN': '浏览和混搭数千个社区共享的工作流。从经过验证的模板开始。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: externalLinks.workflows, 'zh-CN': externalLinks.workflows }
-    }
-  },
-  {
-    id: 'supported-models',
-    category: MODELS_AND_NODES,
-    media: imageFor('Drops_Supported models.jpg', {
-      en: 'Supported Models',
-      'zh-CN': '支持的模型'
-    }),
-    title: { en: 'Supported Models', 'zh-CN': '支持的模型' },
-    description: {
-      en: 'Run the latest open and partner models — every checkpoint, LoRA, and ControlNet, ready to use in your graph.',
-      'zh-CN':
-        '运行最新的开源和合作伙伴模型 — 每个 checkpoint、LoRA 和 ControlNet 都可直接在工作流中使用。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: '/p/supported-models', 'zh-CN': '/zh-CN/p/supported-models' }
-    }
-  },
-  {
-    id: 'supported-nodes',
-    category: MODELS_AND_NODES,
-    media: videoFor('Drops_3x3card_supported nodes.mp4', {
-      en: 'Supported Nodes',
-      'zh-CN': '支持的节点'
-    }),
-    title: { en: 'Supported Nodes', 'zh-CN': '支持的节点' },
-    description: {
-      en: 'Thousands of community and partner nodes, curated and verified to run on Comfy Cloud.',
-      'zh-CN':
-        '数千个社区与合作伙伴节点，经过精选与验证，可在 Comfy Cloud 上运行。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: {
-        en: '/cloud/supported-nodes',
-        'zh-CN': '/zh-CN/cloud/supported-nodes'
-      }
-    }
-  },
-  {
-    id: 'comfy-enterprise',
-    category: CLOUD,
-    media: imageFor('Drops_3x3card_enterprise.png', {
-      en: 'Comfy Enterprise',
-      'zh-CN': 'Comfy 企业版'
-    }),
-    title: { en: 'Comfy Enterprise', 'zh-CN': 'Comfy 企业版' },
-    description: {
-      en: 'Enterprise-grade infrastructure for the creative engine inside your organization.',
-      'zh-CN': '为您组织内创意引擎提供的企业级基础设施。'
-    },
-    cta: {
-      label: EXPLORE,
-      href: { en: '/cloud/enterprise', 'zh-CN': '/zh-CN/cloud/enterprise' }
-    }
-  },
-  {
-    id: 'learning-hub',
-    category: COMMUNITY,
-    media: imageFor('Drops_3x3_Learninghub.jpg', {
-      en: 'Learning Hub',
-      'zh-CN': '学习中心'
-    }),
-    title: { en: 'Learning Hub', 'zh-CN': '学习中心' },
-    description: {
-      en: 'Walkthroughs and ready-to-run workflows to take you from first render to production pipeline.',
-      'zh-CN': '配套教程与开箱即用的工作流，带您从第一次渲染走向生产管线。'
-    },
-    cta: {
-      label: { en: 'START LEARNING', 'zh-CN': '开始学习' },
-      href: { en: '/learning', 'zh-CN': '/zh-CN/learning' }
-    }
-  },
-  {
-    id: 'share-comfy',
-    badge: NEW_BADGE,
-    category: COMMUNITY,
-    media: videoFor('Drops_3x3card_Affilliate.mp4', {
-      en: 'Comfy Affiliate',
-      'zh-CN': 'Comfy Affiliate'
-    }),
-    title: {
-      en: 'Comfy Affiliate',
-      'zh-CN': 'Comfy Affiliate'
-    },
-    description: {
-      en: 'Share Comfy with your audience and earn for every creator you bring on board.',
-      'zh-CN': '与您的受众分享 Comfy，为您带来的每一位创作者获得回报。'
-    },
-    // /affiliates is locale-invariant: same URL in both locales.
-    cta: {
-      label: { en: 'LEARN MORE', 'zh-CN': '了解更多' },
-      href: { en: '/affiliates', 'zh-CN': '/affiliates' }
-    }
-  }
-]

apps/website/src/data/mainNavigation.ts

@@ -180,11 +180,6 @@ export function getMainNavigation(locale: Locale): NavItem[] {
             },
             // TODO: no /brand page yet
             // { label: t('nav.brand', locale), href: '#' },
-            {
-              label: t('nav.launches', locale),
-              href: routes.launches,
-              badge: 'new'
-            },
             {
               label: t('nav.blogs', locale),
               href: externalLinks.blog,

apps/website/src/i18n/translations.ts

@@ -1849,7 +1849,6 @@ const translations = {
   'nav.aboutUs': { en: 'About Us', 'zh-CN': '关于我们' },
   'nav.careers': { en: 'Careers', 'zh-CN': '招聘' },
   'nav.customerStories': { en: 'Customer Stories', 'zh-CN': '客户故事' },
-  'nav.launches': { en: 'Launches', 'zh-CN': '发布' },
   'nav.downloadLocal': { en: 'DOWNLOAD DESKTOP', 'zh-CN': '下载桌面版' },
   'nav.launchCloud': { en: 'LAUNCH CLOUD', 'zh-CN': '启动云端' },
   'nav.ctaDesktopPrefix': { en: 'DOWNLOAD', 'zh-CN': '下载' },
@@ -4929,70 +4928,6 @@ const translations = {
   'affiliate.cta.termsLabel': {
     en: 'Read the affiliate program terms',
     'zh-CN': '阅读联盟计划条款'
-  },
-
-  // Launches page (/launches) — head metadata
-  // zh-CN strings pending native review (see apps/website/.scratch/drops-page/PRD.md)
-  'launches.page.title': {
-    en: 'ComfyUI Live Demo & Q&A — June 29 Launch Livestream',
-    'zh-CN': 'ComfyUI 直播演示与问答 — 6 月 29 日发布直播'
-  },
-  'launches.page.description': {
-    en: 'Join the ComfyUI livestream on June 29 for a hands-on product demo and live Q&A. See what’s new across desktop, cloud, and community, and get your questions answered.',
-    'zh-CN':
-      '6 月 29 日加入 ComfyUI 直播，观看实操产品演示并参与实时问答。了解桌面、云端和社区的最新内容，并获得解答。'
-  },
-
-  // Launches page (/launches) — hero section
-  // zh-CN strings pending native review (see apps/website/.scratch/drops-page/PRD.md)
-  'launches.hero.title': {
-    en: 'Everything new in ComfyUI',
-    'zh-CN': 'ComfyUI 全新内容'
-  },
-  'launches.hero.primary': {
-    en: 'Download Desktop',
-    'zh-CN': '下载桌面版'
-  },
-  'launches.hero.secondary': {
-    en: 'Launch Cloud',
-    'zh-CN': '启动云端'
-  },
-  'launches.hero.visualAlt': {
-    en: 'Comfy',
-    'zh-CN': 'Comfy'
-  },
-
-  // Launches page (/launches) — subscribe banner
-  // zh-CN strings pending native review (see apps/website/.scratch/drops-page/PRD.md)
-  'launches.banner.text': {
-    en: 'Join the live stream. Get answers in real time.',
-    'zh-CN': '加入直播，实时获得解答。'
-  },
-  'launches.banner.cta': {
-    en: 'Join livestream',
-    'zh-CN': '加入直播'
-  },
-
-  // Launches page (/launches) — closing CTA
-  // zh-CN strings pending native review (see apps/website/.scratch/drops-page/PRD.md)
-  'launches.cta.heading': {
-    en: 'Everything Comfy ships. All in one place.',
-    'zh-CN': 'Comfy 的全部内容，一处尽享。'
-  },
-  'launches.cta.primary': {
-    en: 'Open Comfy Cloud',
-    'zh-CN': '打开 Comfy Cloud'
-  },
-  'launches.cta.secondary': {
-    en: 'Try Workflow',
-    'zh-CN': '试用工作流'
-  },
-
-  // Launches page (/launches) — launches grid
-  // zh-CN strings pending native review (see apps/website/.scratch/drops-page/PRD.md)
-  'launches.section.title': {
-    en: 'Latest Launches',
-    'zh-CN': '最新发布'
   }
 } as const satisfies Record<string, Record<Locale, string>>
 

apps/website/src/pages/launches.astro

@@ -1,20 +0,0 @@
----
-import BaseLayout from '../layouts/BaseLayout.astro'
-import CtaSection from '../templates/drops/CtaSection.vue'
-import DropsSection from '../templates/drops/DropsSection.vue'
-import HeroSection from '../templates/drops/HeroSection.vue'
-import SubscribeBanner from '../templates/drops/SubscribeBanner.vue'
-import { t } from '../i18n/translations'
-
-const locale = 'en' as const
----
-
-<BaseLayout
-  title={t('launches.page.title', locale)}
-  description={t('launches.page.description', locale)}
->
-  <SubscribeBanner locale={locale} client:load />
-  <HeroSection locale={locale} client:load />
-  <DropsSection locale={locale} />
-  <CtaSection locale={locale} />
-</BaseLayout>

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

@@ -1,20 +0,0 @@
----
-import BaseLayout from '../../layouts/BaseLayout.astro'
-import CtaSection from '../../templates/drops/CtaSection.vue'
-import DropsSection from '../../templates/drops/DropsSection.vue'
-import HeroSection from '../../templates/drops/HeroSection.vue'
-import SubscribeBanner from '../../templates/drops/SubscribeBanner.vue'
-import { t } from '../../i18n/translations'
-
-const locale = 'zh-CN' as const
----
-
-<BaseLayout
-  title={t('launches.page.title', locale)}
-  description={t('launches.page.description', locale)}
->
-  <SubscribeBanner locale={locale} client:load />
-  <HeroSection locale={locale} client:load />
-  <DropsSection locale={locale} />
-  <CtaSection locale={locale} />
-</BaseLayout>

apps/website/src/templates/drops/CtaSection.vue

@@ -1,25 +0,0 @@
-<script setup lang="ts">
-import type { Locale } from '../../i18n/translations'
-
-import CtaCenter01 from '../../components/blocks/CtaCenter01.vue'
-import { externalLinks } from '../../config/routes'
-import { t } from '../../i18n/translations'
-
-const { locale = 'en' } = defineProps<{ locale?: Locale }>()
-</script>
-
-<template>
-  <CtaCenter01
-    :heading="t('launches.cta.heading', locale)"
-    :primary-cta="{
-      label: t('launches.cta.primary', locale),
-      href: externalLinks.cloud,
-      target: '_blank'
-    }"
-    :secondary-cta="{
-      label: t('launches.cta.secondary', locale),
-      href: externalLinks.workflows,
-      target: '_blank'
-    }"
-  />
-</template>

apps/website/src/templates/drops/DropCard.vue

@@ -1,76 +0,0 @@
-<script setup lang="ts">
-import type { Drop } from '../../data/drops'
-import type { Locale } from '../../i18n/translations'
-import Badge from '../../components/ui/badge/Badge.vue'
-
-import ButtonPill from '../../components/ui/button-pill/ButtonPill.vue'
-import Card from '../../components/ui/card/Card.vue'
-import CardContent from '../../components/ui/card/CardContent.vue'
-import CardDescription from '../../components/ui/card/CardDescription.vue'
-import CardFooter from '../../components/ui/card/CardFooter.vue'
-import CardHeader from '../../components/ui/card/CardHeader.vue'
-import CardTitle from '../../components/ui/card/CardTitle.vue'
-
-const { drop, locale } = defineProps<{
-  drop: Drop
-  locale: Locale
-}>()
-</script>
-
-<template>
-  <Card class="group/pill-trigger relative h-full overflow-hidden">
-    <a
-      :href="drop.cta.href[locale]"
-      :aria-label="`${drop.title[locale]} — ${drop.cta.label[locale]}`"
-      class="rounded-4.5xl focus-visible:ring-primary-comfy-yellow absolute inset-0 z-10 focus-visible:ring-2 focus-visible:ring-offset-2 focus-visible:outline-none"
-    />
-
-    <div class="flex flex-col-reverse">
-      <CardHeader class="gap-2 px-6">
-        <Badge variant="category">
-          {{ drop.category[locale] }}
-        </Badge>
-        <CardTitle class="pt-4">
-          {{ drop.title[locale] }}
-        </CardTitle>
-        <CardDescription>
-          {{ drop.description[locale] }}
-        </CardDescription>
-      </CardHeader>
-
-      <CardContent class="relative p-2">
-        <div class="aspect-video w-full overflow-hidden rounded-4xl">
-          <img
-            v-if="drop.media.type === 'image'"
-            :src="drop.media.src"
-            :alt="drop.media.alt[locale]"
-            loading="lazy"
-            decoding="async"
-            class="size-full object-cover object-center transition-transform duration-500 ease-out group-hover/pill-trigger:scale-105"
-          />
-          <video
-            v-else
-            :src="drop.media.src"
-            :poster="drop.media.poster"
-            :aria-label="drop.media.alt[locale]"
-            autoplay
-            loop
-            muted
-            playsinline
-            preload="metadata"
-            class="size-full object-cover object-center transition-transform duration-500 ease-out group-hover/pill-trigger:scale-105"
-          />
-        </div>
-        <Badge v-if="drop.badge" variant="accent" class="absolute top-6 left-8">
-          {{ drop.badge[locale] }}
-        </Badge>
-      </CardContent>
-    </div>
-
-    <CardFooter class="mt-auto px-6 pb-6">
-      <ButtonPill as="span" variant="ghost" icon-position="left">
-        {{ drop.cta.label[locale] }}
-      </ButtonPill>
-    </CardFooter>
-  </Card>
-</template>

apps/website/src/templates/drops/DropsSection.vue

@@ -1,29 +0,0 @@
-<script setup lang="ts">
-import type { Locale } from '../../i18n/translations'
-
-import DropCard from './DropCard.vue'
-import { drops } from '../../data/drops'
-import { t } from '../../i18n/translations'
-
-const { locale = 'en' } = defineProps<{ locale?: Locale }>()
-</script>
-
-<template>
-  <section class="max-w-9xl mx-auto px-6 py-16 lg:py-24">
-    <h2
-      class="text-primary-warm-white text-3xl font-light tracking-tight lg:text-5xl"
-    >
-      {{ t('launches.section.title', locale) }}
-    </h2>
-
-    <div class="mt-10 grid grid-cols-1 gap-6 md:grid-cols-6 lg:mt-12">
-      <div
-        v-for="(drop, index) in drops"
-        :key="drop.id"
-        :class="index < 4 ? 'md:col-span-3' : 'md:col-span-2'"
-      >
-        <DropCard :drop :locale />
-      </div>
-    </div>
-  </section>
-</template>

apps/website/src/templates/drops/HeroSection.vue

@@ -1,35 +0,0 @@
-<script setup lang="ts">
-import type { Locale } from '../../i18n/translations'
-
-import HeroLivestream01 from '../../components/blocks/HeroLivestream01.vue'
-import LaunchesHeroLogo from './LaunchesHeroLogo.vue'
-import { externalLinks, getRoutes } from '../../config/routes'
-import { t } from '../../i18n/translations'
-import { livestream } from './livestream'
-
-const { locale = 'en' } = defineProps<{ locale?: Locale }>()
-
-const routes = getRoutes(locale)
-</script>
-
-<template>
-  <HeroLivestream01
-    :title="t('launches.hero.title', locale)"
-    :primary-cta="{
-      label: t('launches.hero.primary', locale),
-      href: routes.download
-    }"
-    :secondary-cta="{
-      label: t('launches.hero.secondary', locale),
-      href: externalLinks.cloud,
-      target: '_blank'
-    }"
-    :youtube-video-id="livestream.youtubeVideoId"
-    :start-date-time="livestream.startDateTime"
-    :end-date-time="livestream.endDateTime"
-  >
-    <template #visual>
-      <LaunchesHeroLogo :label="t('launches.hero.visualAlt', locale)" />
-    </template>
-  </HeroLivestream01>
-</template>

apps/website/src/templates/drops/SubscribeBanner.vue

@@ -1,61 +0,0 @@
-<script setup lang="ts">
-import { useTimeoutFn } from '@vueuse/core'
-import { onMounted, ref } from 'vue'
-
-import type { Locale } from '../../i18n/translations'
-
-import { t } from '../../i18n/translations'
-import Button from '@/components/ui/button/Button.vue'
-import { resolveRel } from '../../utils/cta'
-import { livestream } from './livestream'
-
-const { locale = 'en' } = defineProps<{ locale?: Locale }>()
-
-const signUpHref = `https://www.youtube.com/watch?v=${livestream.youtubeVideoId}`
-const signUpRel = resolveRel({ target: '_blank' })
-
-// Hide once the livestream window closes — both for visitors arriving after
-// the event and for visitors whose tab is open when it ends.
-const endMs = new Date(livestream.endDateTime).getTime()
-const visible = ref(true)
-
-// useTimeoutFn auto-clears on unmount. Arm it client-side only so SSR never
-// schedules a long-lived server timer.
-const { start } = useTimeoutFn(
-  () => {
-    visible.value = false
-  },
-  () => Math.max(0, endMs - Date.now()),
-  { immediate: false }
-)
-
-onMounted(() => {
-  if (endMs - Date.now() <= 0) {
-    visible.value = false
-  } else {
-    start()
-  }
-})
-</script>
-
-<template>
-  <div v-if="visible" class="px-4">
-    <div
-      class="bg-primary-comfy-plum max-w-8xl rounded-5xl text-primary-warm-white mx-auto flex w-full flex-col items-center justify-center gap-2 px-6 py-5 text-center text-sm sm:flex-row sm:gap-4"
-    >
-      <p class="ppformula-text-center">
-        {{ t('launches.banner.text', locale) }}
-      </p>
-      <Button
-        :href="signUpHref"
-        as="a"
-        variant="underlineLink"
-        size="sm"
-        target="_blank"
-        :rel="signUpRel"
-      >
-        {{ t('launches.banner.cta', locale) }}
-      </Button>
-    </div>
-  </div>
-</template>

apps/website/src/templates/drops/livestream.ts

@@ -1,5 +0,0 @@
-export const livestream = {
-  youtubeVideoId: 'yo7b_zHd20g',
-  startDateTime: '2026-06-29T15:00:00Z',
-  endDateTime: '2026-07-02T17:15:00Z'
-} as const

apps/website/src/utils/cta.test.ts

@@ -1,18 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { resolveRel } from './cta'
-
-describe('resolveRel', () => {
-  it('prefers an explicit rel over the target-derived default', () => {
-    expect(resolveRel({ rel: 'nofollow', target: '_blank' })).toBe('nofollow')
-  })
-
-  it('adds noopener noreferrer for _blank targets', () => {
-    expect(resolveRel({ target: '_blank' })).toBe('noopener noreferrer')
-  })
-
-  it('returns undefined for non-blank targets', () => {
-    expect(resolveRel({ target: '_self' })).toBeUndefined()
-    expect(resolveRel({})).toBeUndefined()
-  })
-})

apps/website/src/utils/cta.ts

@@ -1,10 +0,0 @@
-import type { AnchorHTMLAttributes } from 'vue'
-
-export function resolveRel(cta: {
-  rel?: AnchorHTMLAttributes['rel']
-  target?: AnchorHTMLAttributes['target']
-}): AnchorHTMLAttributes['rel'] {
-  return (
-    cta.rel ?? (cta.target === '_blank' ? 'noopener noreferrer' : undefined)
-  )
-}

apps/website/src/utils/video.ts

@@ -1,7 +1,7 @@
 /** @knipIgnoreUsedByStackedPR */
 export type VideoFormat = 'webm' | 'mp4'
 
-type VideoSource = {
+export type VideoSource = {
   src: string
   type: `video/${VideoFormat}`
   format: VideoFormat

apps/website/vercel.json

@@ -14,24 +14,6 @@
         { "type": "host", "value": "website-frontend-comfyui.vercel.app" }
       ],
       "headers": [{ "key": "X-Robots-Tag", "value": "index, follow" }]
-    },
-    {
-      "source": "/payment/(.*)",
-      "headers": [
-        {
-          "key": "X-Robots-Tag",
-          "value": "noindex"
-        }
-      ]
-    },
-    {
-      "source": "/:locale/payment/(.*)",
-      "headers": [
-        {
-          "key": "X-Robots-Tag",
-          "value": "noindex"
-        }
-      ]
     }
   ],
   "redirects": [

browser_tests/assets/missing/missing_model_nested_promoted_widget.json

@@ -1,197 +0,0 @@
-{
-  "id": "test-missing-model-nested-promoted-widget",
-  "revision": 0,
-  "last_node_id": 4,
-  "last_link_id": 0,
-  "nodes": [
-    {
-      "id": 3,
-      "type": "outer-subgraph-with-promoted-missing-model",
-      "pos": [10, 250],
-      "size": [400, 200],
-      "flags": {},
-      "order": 0,
-      "mode": 0,
-      "title": "Resolved Shared Outer Subgraph",
-      "inputs": [],
-      "outputs": [],
-      "properties": {},
-      "widgets_values": ["resolved_model.safetensors"]
-    },
-    {
-      "id": 4,
-      "type": "outer-subgraph-with-promoted-missing-model",
-      "pos": [450, 250],
-      "size": [400, 200],
-      "flags": {},
-      "order": 1,
-      "mode": 0,
-      "title": "Outer Subgraph with Promoted Missing Model",
-      "inputs": [],
-      "outputs": [],
-      "properties": {},
-      "widgets_values": ["fake_model.safetensors"]
-    }
-  ],
-  "links": [],
-  "groups": [],
-  "definitions": {
-    "subgraphs": [
-      {
-        "id": "outer-subgraph-with-promoted-missing-model",
-        "version": 1,
-        "state": {
-          "lastGroupId": 0,
-          "lastNodeId": 2,
-          "lastLinkId": 2,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "Outer Subgraph with Promoted Missing Model",
-        "inputNode": {
-          "id": -10,
-          "bounding": [100, 200, 120, 60]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [600, 200, 120, 60]
-        },
-        "inputs": [
-          {
-            "id": "outer-ckpt-name-input-id",
-            "name": "ckpt_name",
-            "type": "COMBO",
-            "linkIds": [2],
-            "pos": { "0": 150, "1": 220 }
-          }
-        ],
-        "outputs": [],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 2,
-            "type": "inner-subgraph-with-promoted-missing-model",
-            "pos": [250, 180],
-            "size": [400, 200],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "localized_name": "ckpt_name",
-                "name": "ckpt_name",
-                "type": "COMBO",
-                "widget": {
-                  "name": "ckpt_name"
-                },
-                "link": 2
-              }
-            ],
-            "outputs": [],
-            "properties": {},
-            "widgets_values": ["fake_model.safetensors"]
-          }
-        ],
-        "links": [
-          {
-            "id": 2,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 2,
-            "target_slot": 0,
-            "type": "COMBO"
-          }
-        ]
-      },
-      {
-        "id": "inner-subgraph-with-promoted-missing-model",
-        "version": 1,
-        "state": {
-          "lastGroupId": 0,
-          "lastNodeId": 1,
-          "lastLinkId": 1,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "Inner Subgraph with Promoted Missing Model",
-        "inputNode": {
-          "id": -10,
-          "bounding": [100, 200, 120, 60]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [500, 200, 120, 60]
-        },
-        "inputs": [
-          {
-            "id": "inner-ckpt-name-input-id",
-            "name": "ckpt_name",
-            "type": "COMBO",
-            "linkIds": [1],
-            "pos": { "0": 150, "1": 220 }
-          }
-        ],
-        "outputs": [],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 1,
-            "type": "CheckpointLoaderSimple",
-            "pos": [250, 180],
-            "size": [315, 98],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "localized_name": "ckpt_name",
-                "name": "ckpt_name",
-                "type": "COMBO",
-                "widget": {
-                  "name": "ckpt_name"
-                },
-                "link": 1
-              }
-            ],
-            "outputs": [
-              { "name": "MODEL", "type": "MODEL", "links": null },
-              { "name": "CLIP", "type": "CLIP", "links": null },
-              { "name": "VAE", "type": "VAE", "links": null }
-            ],
-            "properties": {
-              "Node name for S&R": "CheckpointLoaderSimple"
-            },
-            "widgets_values": ["fake_model.safetensors"]
-          }
-        ],
-        "links": [
-          {
-            "id": 1,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 1,
-            "target_slot": 0,
-            "type": "COMBO"
-          }
-        ]
-      }
-    ]
-  },
-  "config": {},
-  "extra": {
-    "ds": {
-      "scale": 1,
-      "offset": [0, 0]
-    }
-  },
-  "models": [
-    {
-      "name": "fake_model.safetensors",
-      "url": "http://localhost:8188/api/devtools/fake_model.safetensors",
-      "directory": "checkpoints"
-    }
-  ],
-  "version": 0.4
-}

browser_tests/fixtures/ComfyPage.ts

@@ -56,16 +56,12 @@ class ComfyPropertiesPanel {
   readonly panelTitle: Locator
   readonly searchBox: Locator
   readonly titleEditor: TitleEditor
-  readonly toggleButton: Locator
 
   constructor(readonly page: Page) {
     this.root = page.getByTestId(TestIds.propertiesPanel.root)
     this.panelTitle = this.root.locator('h3')
     this.searchBox = this.root.getByPlaceholder(/^Search/)
     this.titleEditor = new TitleEditor(this.root)
-    this.toggleButton = page.getByRole('button', {
-      name: 'Toggle properties panel'
-    })
   }
 }
 

browser_tests/fixtures/VueNodeHelpers.ts

@@ -4,9 +4,8 @@
 import type { Locator, Page } from '@playwright/test'
 
 import { TestIds } from '@e2e/fixtures/selectors'
-import { getSlotKey } from '@/renderer/core/layout/slots/slotIdentifier'
-import { toNodeId } from '@/types/nodeId'
 import { VueNodeFixture } from '@e2e/fixtures/utils/vueNodeFixtures'
+import { getSlotKey } from '@/renderer/core/layout/slots/slotIdentifier'
 
 export class VueNodeHelpers {
   /**
@@ -44,7 +43,7 @@ export class VueNodeHelpers {
       .locator('.lg-slot--input')
       .filter({
         has: this.page.locator(
-          `[data-slot-key="${getSlotKey(toNodeId(nodeId), slotIndex, true)}"]`
+          `[data-slot-key="${getSlotKey(nodeId, slotIndex, true)}"]`
         )
       })
   }
@@ -252,18 +251,14 @@ export class VueNodeHelpers {
     const key = await slot.getByTestId('slot-dot').getAttribute('data-slot-key')
     if (!key) return false
 
-    const [rawNodeId, type, slotId] = key.split('-')
-    const nodeId = toNodeId(rawNodeId)
-    return await this.page.evaluate(
-      ([nodeId, type, slotId]) => {
-        const node = app?.canvas?.graph?.getNodeById(nodeId)
-        if (!node) return false
+    return await this.page.evaluate((key) => {
+      const [nodeId, type, slotId] = key.split('-')
+      const node = app?.canvas?.graph?.getNodeById(nodeId)
+      if (!node) return false
 
-        return type === 'in'
-          ? node.inputs[Number(slotId)]?.link !== null
-          : !!node.outputs[Number(slotId)]?.links?.length
-      },
-      [nodeId, type, slotId] as const
-    )
+      return type === 'in'
+        ? node.inputs[Number(slotId)]?.link !== null
+        : !!node.outputs[Number(slotId)].links?.length
+    }, key)
   }
 }

browser_tests/fixtures/components/SidebarTab.ts

@@ -93,7 +93,7 @@ export class NodeLibrarySidebarTabV2 extends SidebarTab {
   public readonly searchInput: Locator
   public readonly sidebarContent: Locator
   public readonly allTab: Locator
-  public readonly essentialsTab: Locator
+  public readonly blueprintsTab: Locator
   public readonly sortButton: Locator
   public readonly nodePreview: Locator
 
@@ -101,8 +101,8 @@ export class NodeLibrarySidebarTabV2 extends SidebarTab {
     super(page, 'node-library')
     this.searchInput = page.getByPlaceholder('Search...')
     this.sidebarContent = page.locator('.sidebar-content-container')
-    this.allTab = this.getTab('All nodes')
-    this.essentialsTab = this.getTab('Essentials')
+    this.allTab = this.getTab('All')
+    this.blueprintsTab = this.getTab('Blueprints')
     this.sortButton = this.sidebarContent.getByRole('button', { name: 'Sort' })
     this.nodePreview = page.getByTestId(TestIds.sidebar.nodePreviewCard)
   }

browser_tests/fixtures/data/cloudWorkspace.ts

@@ -1,96 +0,0 @@
-import type {
-  BillingStatusResponse,
-  Member,
-  Plan,
-  WorkspaceWithRole
-} from '@/platform/workspace/api/workspaceApi'
-import type { RemoteConfig } from '@/platform/remoteConfig/types'
-
-// `/api/features` is the remote-config source: production builds resolve the
-// workspaces flag from it (the `ff:` localStorage override is dev-only).
-export const WORKSPACE_FEATURE_FLAG: RemoteConfig = {
-  team_workspaces_enabled: true
-}
-
-export const TEAM_WORKSPACE: WorkspaceWithRole = {
-  id: 'ws-team',
-  name: 'Team Comfy',
-  type: 'team',
-  created_at: '2025-01-01T00:00:00Z',
-  joined_at: '2025-01-02T00:00:00Z',
-  role: 'owner',
-  subscription_tier: 'PRO'
-}
-
-export const CREATOR: Member = {
-  id: 'u-liz',
-  name: 'Liz',
-  email: 'liz@test.comfy.org',
-  joined_at: '2025-01-01T00:00:00Z',
-  role: 'owner',
-  is_original_owner: true
-}
-
-// Identity must match the CloudAuthHelper mock user so this row counts as
-// "(You)".
-export const VIEWER: Member = {
-  id: 'u-me',
-  name: 'E2E Test User',
-  email: 'e2e@test.comfy.org',
-  joined_at: '2025-01-02T00:00:00Z',
-  role: 'owner',
-  is_original_owner: false
-}
-
-export const MEMBER_JANE: Member = {
-  id: 'u-jane',
-  name: 'Jane',
-  email: 'jane@test.comfy.org',
-  joined_at: '2025-01-03T00:00:00Z',
-  role: 'member',
-  is_original_owner: false
-}
-
-export const MEMBER_JOHN: Member = {
-  id: 'u-john',
-  name: 'John',
-  email: 'john@test.comfy.org',
-  joined_at: '2025-01-04T00:00:00Z',
-  role: 'member',
-  is_original_owner: false
-}
-
-export const DEFAULT_TEAM_MEMBERS: Member[] = [
-  CREATOR,
-  VIEWER,
-  MEMBER_JANE,
-  MEMBER_JOHN
-]
-
-export const TEAM_BILLING_STATUS: BillingStatusResponse = {
-  is_active: true,
-  subscription_status: 'active',
-  subscription_tier: 'PRO',
-  subscription_duration: 'MONTHLY',
-  plan_slug: 'pro-monthly',
-  billing_status: 'paid',
-  has_funds: true,
-  renewal_date: '2099-02-20T00:00:00Z'
-}
-
-// `max_seats > 1` on the current plan is what flips `isOnTeamPlan`, which gates
-// the whole role-management UI.
-export const TEAM_PRO_PLAN: Plan = {
-  slug: 'pro-monthly',
-  tier: 'PRO',
-  duration: 'MONTHLY',
-  price_cents: 10000,
-  credits_cents: 21100,
-  max_seats: 30,
-  availability: { available: true },
-  seat_summary: {
-    seat_count: 4,
-    total_cost_cents: 40000,
-    total_credits_cents: 0
-  }
-}

browser_tests/fixtures/helpers/CloudWorkspaceMockHelper.ts

@@ -1,150 +0,0 @@
-import type { Page, Route } from '@playwright/test'
-
-import type { Member } from '@/platform/workspace/api/workspaceApi'
-
-import { mockSystemStats } from '@e2e/fixtures/data/systemStats'
-import {
-  DEFAULT_TEAM_MEMBERS,
-  TEAM_BILLING_STATUS,
-  TEAM_PRO_PLAN,
-  TEAM_WORKSPACE,
-  WORKSPACE_FEATURE_FLAG
-} from '@e2e/fixtures/data/cloudWorkspace'
-import { CloudAuthHelper } from '@e2e/fixtures/helpers/CloudAuthHelper'
-
-interface RoleChangeRequest {
-  url: string
-  role: string
-}
-
-interface MemberMockState {
-  members: Member[]
-  patches: RoleChangeRequest[]
-}
-
-const jsonRoute = (body: unknown) => ({
-  status: 200,
-  contentType: 'application/json',
-  body: JSON.stringify(body)
-})
-
-/**
- * Boots the cloud app against fully mocked workspace + billing endpoints so
- * member/role specs can drive a raw `page` (the `comfyPage` fixture would try
- * to reach the OSS devtools backend during setup).
- *
- * Returns the mutable mock state: `members` reflects PATCH-applied roles and
- * `patches` records every role-change request for assertion.
- */
-export class CloudWorkspaceMockHelper {
-  constructor(private readonly page: Page) {}
-
-  async setup(
-    members: Member[] = DEFAULT_TEAM_MEMBERS
-  ): Promise<MemberMockState> {
-    const state = await this.mockBoot(members)
-    await new CloudAuthHelper(this.page).mockAuth()
-    await this.page.addInitScript(() => {
-      localStorage.setItem('Comfy.userId', 'test-user-e2e')
-      localStorage.setItem('Comfy.Workspace.LastWorkspaceId', 'ws-team')
-    })
-    return state
-  }
-
-  private async mockBoot(members: Member[]): Promise<MemberMockState> {
-    const state: MemberMockState = {
-      members: members.map((m) => ({ ...m })),
-      patches: []
-    }
-    const { page } = this
-
-    await page.route('**/api/features', (r) =>
-      r.fulfill(jsonRoute(WORKSPACE_FEATURE_FLAG))
-    )
-    await page.route('**/api/system_stats', (r) =>
-      r.fulfill(jsonRoute(mockSystemStats))
-    )
-    await page.route('**/api/users', (r) =>
-      r.fulfill(
-        jsonRoute({
-          storage: 'server',
-          migrated: true,
-          users: { 'test-user-e2e': 'E2E Test User' }
-        })
-      )
-    )
-    // A non-empty settings payload with TutorialCompleted marks the user as
-    // returning, so the new-user Templates dialog never auto-opens to block the
-    // Settings button. Errors tab off suppresses the model-folder 401 toast.
-    await page.route('**/api/settings', (r) =>
-      r.fulfill(
-        jsonRoute({
-          'Comfy.TutorialCompleted': true,
-          'Comfy.RightSidePanel.ShowErrorsTab': false
-        })
-      )
-    )
-    await page.route('**/api/userdata**', (r) => r.fulfill(jsonRoute([])))
-    await page.route('**/api/extensions', (r) => r.fulfill(jsonRoute([])))
-    await page.route('**/api/object_info', (r) => r.fulfill(jsonRoute({})))
-    await page.route('**/api/global_subgraphs', (r) => r.fulfill(jsonRoute({})))
-    await page.route('**/api/i18n', (r) => r.fulfill(jsonRoute({})))
-    await page.route('**/api/auth/session', (r) =>
-      r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
-    )
-    await page.route('**/api/auth/token', (r) =>
-      r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
-    )
-    await page.route('**/releases**', (r) => r.fulfill(jsonRoute([])))
-
-    await page.route('**/api/workspaces', (r) =>
-      r.fulfill(jsonRoute({ workspaces: [TEAM_WORKSPACE] }))
-    )
-
-    await page.route('**/api/workspace/members**', (route: Route) => {
-      const request = route.request()
-      if (request.method() === 'PATCH') {
-        const url = request.url()
-        const id = url.match(/\/api\/workspace\/members\/([^/?]+)/)?.[1]
-        const { role } = request.postDataJSON() as { role: Member['role'] }
-        state.patches.push({ url, role })
-        const member = state.members.find((m) => m.id === id)
-        if (member) member.role = role
-        // Echo the updated row like the real BE; the store merges only the role
-        // locally, so the response body shape is not load-bearing.
-        return route.fulfill(jsonRoute(member))
-      }
-      return route.fulfill(
-        jsonRoute({
-          members: state.members,
-          pagination: { offset: 0, limit: 50, total: state.members.length }
-        })
-      )
-    })
-    await page.route('**/api/workspace/invites', (r) =>
-      r.fulfill(jsonRoute({ invites: [] }))
-    )
-
-    await page.route('**/api/billing/status', (r) =>
-      r.fulfill(jsonRoute(TEAM_BILLING_STATUS))
-    )
-    await page.route('**/api/billing/balance', (r) =>
-      r.fulfill(
-        jsonRoute({
-          amount_micros: 6000,
-          currency: 'usd',
-          effective_balance_micros: 6000,
-          cloud_credit_balance_micros: 5000,
-          prepaid_balance_micros: 1000
-        })
-      )
-    )
-    await page.route('**/api/billing/plans', (r) =>
-      r.fulfill(
-        jsonRoute({ current_plan_slug: 'pro-monthly', plans: [TEAM_PRO_PLAN] })
-      )
-    )
-
-    return state
-  }
-}

browser_tests/fixtures/helpers/NodeOperationsHelper.ts

@@ -5,9 +5,10 @@ import type {
   LGraph,
   LGraphNode
 } from '@/lib/litegraph/src/litegraph'
-import type { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/workflowSchema'
-import { toNodeId } from '@/types/nodeId'
-import type { NodeId, SerializedNodeId } from '@/types/nodeId'
+import type {
+  ComfyWorkflowJSON,
+  NodeId
+} from '@/platform/workflow/validation/schemas/workflowSchema'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { DefaultGraphPositions } from '@e2e/fixtures/constants/defaultGraphPositions'
 import type { Position, Size } from '@e2e/fixtures/types'
@@ -41,12 +42,11 @@ export class NodeOperationsHelper {
   }
 
   async getSelectedNodeIds(): Promise<NodeId[]> {
-    const selectedNodeIds = await this.page.evaluate(() => {
+    return await this.page.evaluate(() => {
       const selected = window.app?.canvas?.selected_nodes
       if (!selected) return []
-      return Object.keys(selected)
+      return Object.keys(selected).map(Number)
     })
-    return selectedNodeIds.map(toNodeId)
   }
 
   /**
@@ -114,8 +114,8 @@ export class NodeOperationsHelper {
     return this.getNodeRefById(id)
   }
 
-  async getNodeRefById(id: SerializedNodeId): Promise<NodeReference> {
-    return new NodeReference(toNodeId(id), this.comfyPage)
+  async getNodeRefById(id: NodeId): Promise<NodeReference> {
+    return new NodeReference(id, this.comfyPage)
   }
 
   async getNodeRefsByType(
@@ -136,7 +136,7 @@ export class NodeOperationsHelper {
           },
           { type, includeSubgraph }
         )
-      ).map((id: SerializedNodeId) => this.getNodeRefById(id))
+      ).map((id: NodeId) => this.getNodeRefById(id))
     )
   }
 
@@ -148,7 +148,7 @@ export class NodeOperationsHelper {
             .app!.graph.nodes.filter((n: LGraphNode) => n.title === title)
             .map((n: LGraphNode) => n.id)
         }, title)
-      ).map((id: SerializedNodeId) => this.getNodeRefById(id))
+      ).map((id: NodeId) => this.getNodeRefById(id))
     )
   }
 

browser_tests/fixtures/helpers/Preview3DPipelineFixture.ts

@@ -1,9 +1,6 @@
 import { expect } from '@playwright/test'
 import type { Locator, Page } from '@playwright/test'
 
-import { toNodeId } from '@/types/nodeId'
-import type { NodeId } from '@/types/nodeId'
-
 import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { assetPath } from '@e2e/fixtures/utils/paths'
@@ -48,9 +45,9 @@ async function orbitDragFromCanvasCenter(
 
 export class Preview3DPipelineContext {
   /** Matches node ids in `browser_tests/assets/3d/preview3d_pipeline.json`. */
-  static readonly loadNodeId = toNodeId(1)
+  static readonly loadNodeId = '1'
   /** Matches node ids in `browser_tests/assets/3d/preview3d_pipeline.json`. */
-  static readonly previewNodeId = toNodeId(2)
+  static readonly previewNodeId = '2'
 
   readonly load3d: Load3DHelper
   readonly preview3d: Load3DHelper
@@ -64,9 +61,9 @@ export class Preview3DPipelineContext {
     )
   }
 
-  async getModelFileWidgetValue(nodeId: NodeId): Promise<string> {
+  async getModelFileWidgetValue(nodeId: string): Promise<string> {
     return this.comfyPage.page.evaluate((id) => {
-      const node = window.app!.graph.getNodeById(id)
+      const node = window.app!.graph.getNodeById(Number(id))
       if (!node?.widgets) return ''
       const w = node.widgets.find((x) => x.name === 'model_file')
       const v = w?.value
@@ -74,9 +71,9 @@ export class Preview3DPipelineContext {
     }, nodeId)
   }
 
-  async getLastTimeModelFile(nodeId: NodeId): Promise<string> {
+  async getLastTimeModelFile(nodeId: string): Promise<string> {
     return this.comfyPage.page.evaluate((id) => {
-      const node = window.app!.graph.getNodeById(id)
+      const node = window.app!.graph.getNodeById(Number(id))
       if (!node?.properties) return ''
       const v = (node.properties as Record<string, unknown>)[
         'Last Time Model File'
@@ -85,9 +82,9 @@ export class Preview3DPipelineContext {
     }, nodeId)
   }
 
-  async getCameraStateFromProperties(nodeId: NodeId): Promise<unknown> {
+  async getCameraStateFromProperties(nodeId: string): Promise<unknown> {
     return this.comfyPage.page.evaluate((id) => {
-      const node = window.app!.graph.getNodeById(id)
+      const node = window.app!.graph.getNodeById(Number(id))
       if (!node?.properties) return null
       const cfg = (node.properties as Record<string, unknown>)['Camera Config']
       if (cfg === null || typeof cfg !== 'object') return null

browser_tests/fixtures/helpers/SubgraphHelper.ts

@@ -7,7 +7,6 @@ import type {
 } from '@/lib/litegraph/src/litegraph'
 import type { ComfyWorkflow } from '@/platform/workflow/management/stores/comfyWorkflow'
 import type { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/workflowSchema'
-import { toNodeId } from '@/types/nodeId'
 
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { SubgraphEditor } from '@e2e/fixtures/components/SubgraphEditor'
@@ -550,7 +549,6 @@ export class SubgraphHelper {
   }
 
   static getTextSlotPosition(page: Page, nodeId: string) {
-    const localNodeId = toNodeId(nodeId)
     return page.evaluate((id) => {
       const node = window.app!.canvas.graph!.getNodeById(id)
       if (!node) return null
@@ -567,7 +565,7 @@ export class SubgraphHelper {
         }
       }
       return null
-    }, localNodeId)
+    }, nodeId)
   }
 
   static async expectWidgetBelowHeader(

browser_tests/fixtures/selectors.ts

@@ -112,10 +112,6 @@ export const TestIds = {
     root: 'properties-panel',
     errorsTab: 'panel-tab-errors'
   },
-  assets: {
-    browserModal: 'asset-browser-modal',
-    card: 'asset-card'
-  },
   subgraphEditor: {
     hiddenSection: 'subgraph-editor-hidden-section',
     iconEye: 'icon-eye',

browser_tests/fixtures/utils/cloudBillingMocks.ts

@@ -1,34 +0,0 @@
-import type { Page } from '@playwright/test'
-
-import { jsonRoute } from '@e2e/fixtures/utils/jsonRoute'
-
-/**
- * Minimal valid billing shapes so the billing facade resolves while a
- * subscription dialog mounts. Active personal sub with zero balance.
- */
-export async function mockBilling(page: Page) {
-  await page.route('**/api/billing/status', (r) =>
-    r.fulfill(
-      jsonRoute({
-        is_active: true,
-        has_funds: true,
-        subscription_status: 'active',
-        subscription_tier: 'pro',
-        subscription_duration: 'MONTHLY',
-        billing_status: 'paid'
-      })
-    )
-  )
-  await page.route('**/api/billing/balance', (r) =>
-    r.fulfill(jsonRoute({ amount_micros: 0, currency: 'usd' }))
-  )
-  await page.route('**/api/billing/plans', (r) =>
-    r.fulfill(jsonRoute({ plans: [] }))
-  )
-  await page.route('**/customers/cloud-subscription-status', (r) =>
-    r.fulfill(jsonRoute({ is_active: false }))
-  )
-  await page.route('**/customers/balance', (r) =>
-    r.fulfill(jsonRoute({ amount_micros: 0, currency: 'usd' }))
-  )
-}

browser_tests/fixtures/utils/cloudBootMocks.ts

@@ -1,64 +0,0 @@
-import type { Page } from '@playwright/test'
-
-import type { RemoteConfig } from '@/platform/remoteConfig/types'
-
-import { mockSystemStats } from '@e2e/fixtures/data/systemStats'
-import { CloudAuthHelper } from '@e2e/fixtures/helpers/CloudAuthHelper'
-import { jsonRoute } from '@e2e/fixtures/utils/jsonRoute'
-
-interface CloudBootOptions {
-  /** Remote-config payload for `/api/features` (enables the flags under test). */
-  features: RemoteConfig
-  /** Body for `/api/settings` (defaults to `{}`). */
-  settings?: unknown
-}
-
-/**
- * Stub the core endpoints the cloud app hits on boot so a raw `page` reaches the
- * working app without falling through to the OSS devtools backend. Specs layer
- * their own feature- or flow-specific routes on top.
- */
-export async function mockCloudBoot(
-  page: Page,
-  { features, settings = {} }: CloudBootOptions
-) {
-  await page.route('**/api/features', (r) => r.fulfill(jsonRoute(features)))
-  await page.route('**/api/system_stats', (r) =>
-    r.fulfill(jsonRoute(mockSystemStats))
-  )
-  await page.route('**/api/users', (r) =>
-    r.fulfill(
-      jsonRoute({
-        storage: 'server',
-        migrated: true,
-        users: { 'test-user-e2e': 'E2E Test User' }
-      })
-    )
-  )
-  await page.route('**/api/user', (r) =>
-    r.fulfill(jsonRoute({ status: 'active' }))
-  )
-  await page.route('**/api/settings', (r) => r.fulfill(jsonRoute(settings)))
-  await page.route('**/api/userdata**', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/extensions', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/object_info', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/global_subgraphs', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/i18n', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/auth/session', (r) =>
-    r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
-  )
-  await page.route('**/releases**', (r) => r.fulfill(jsonRoute([])))
-}
-
-/**
- * Mock Firebase auth and pre-select the e2e user so the cloud app boots
- * signed-in. The signed-in email (`e2e@test.comfy.org`) is what the
- * original-owner gate matches against the members self-row.
- */
-export async function bootCloud(page: Page) {
-  const auth = new CloudAuthHelper(page)
-  await auth.mockAuth()
-  await page.addInitScript(() => {
-    localStorage.setItem('Comfy.userId', 'test-user-e2e')
-  })
-}

browser_tests/fixtures/utils/jsonRoute.ts

@@ -1,12 +0,0 @@
-/**
- * Build a 200 JSON body for `route.fulfill()`. Generic so callers can type the
- * payload (e.g. `jsonRoute({ ... } satisfies RemoteConfig)`) and catch contract
- * drift against the real API shape.
- */
-export function jsonRoute<T>(body: T) {
-  return {
-    status: 200,
-    contentType: 'application/json',
-    body: JSON.stringify(body)
-  }
-}

browser_tests/fixtures/utils/litegraphUtils.ts

@@ -1,7 +1,7 @@
 import { expect } from '@playwright/test'
 
 import type { SerialisableLLink } from '@/lib/litegraph/src/types/serialisation'
-import type { NodeId } from '@/types/nodeId'
+import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import type { Position, Size } from '@e2e/fixtures/types'
 import { VueNodeFixture } from '@e2e/fixtures/utils/vueNodeFixtures'

browser_tests/fixtures/utils/objectInfo.ts

@@ -1,136 +0,0 @@
-import type { Page, Route } from '@playwright/test'
-
-import {
-  getComboSpecComboOptions,
-  isComboInputSpec,
-  isComboInputSpecV1
-} from '@/schemas/nodeDefSchema'
-import type {
-  ComboInputSpec,
-  ComboInputSpecV2,
-  ComfyNodeDef,
-  InputSpec
-} from '@/schemas/nodeDefSchema'
-
-type ObjectInfoResponse = Record<string, ComfyNodeDef>
-
-type ComboInput = ComboInputSpec | ComboInputSpecV2
-
-const OBJECT_INFO_ROUTE = '**/object_info'
-
-function getRequiredInput(
-  objectInfo: ObjectInfoResponse,
-  nodeType: string,
-  inputName: string
-): InputSpec {
-  const nodeInfo = objectInfo[nodeType]
-  if (!nodeInfo) {
-    throw new Error(`Missing object_info entry for ${nodeType}`)
-  }
-
-  const requiredInputs = nodeInfo.input?.required
-  if (!requiredInputs) {
-    throw new Error(`Missing required inputs for ${nodeType}`)
-  }
-
-  const input = requiredInputs[inputName]
-  if (!input) {
-    throw new Error(`Missing input ${nodeType}.${inputName}`)
-  }
-
-  return input
-}
-
-function getComboInput(
-  objectInfo: ObjectInfoResponse,
-  nodeType: string,
-  inputName: string
-): ComboInput {
-  const input = getRequiredInput(objectInfo, nodeType, inputName)
-  if (isComboInputSpec(input)) {
-    return input
-  }
-
-  throw new Error(`Expected ${nodeType}.${inputName} to be a combo input`)
-}
-
-export function setComboInputOptions(
-  objectInfo: ObjectInfoResponse,
-  nodeType: string,
-  inputName: string,
-  values: ReadonlyArray<string | number>
-): void {
-  const input = getComboInput(objectInfo, nodeType, inputName)
-  const nextValues = [...values]
-
-  if (isComboInputSpecV1(input)) {
-    input[0] = nextValues
-    return
-  }
-
-  input[1] = { ...input[1], options: nextValues }
-}
-
-export function appendComboInputOptions(
-  objectInfo: ObjectInfoResponse,
-  nodeType: string,
-  inputName: string,
-  values: ReadonlyArray<string | number>
-): void {
-  const input = getComboInput(objectInfo, nodeType, inputName)
-  setComboInputOptions(objectInfo, nodeType, inputName, [
-    ...getComboSpecComboOptions(input),
-    ...values
-  ])
-}
-
-export async function routeObjectInfoFromSetupApi(
-  page: Page,
-  customize?: (objectInfo: ObjectInfoResponse) => void | Promise<void>
-): Promise<() => Promise<void>> {
-  const setupApiUrl =
-    process.env.PLAYWRIGHT_SETUP_API_URL ?? 'http://127.0.0.1:8188'
-  const objectInfoUrl = new URL('/object_info', setupApiUrl).toString()
-
-  const objectInfoRouteHandler = async (route: Route) => {
-    let objectInfo: ObjectInfoResponse
-    try {
-      const response = await fetch(objectInfoUrl, {
-        signal: AbortSignal.timeout(5_000)
-      })
-      if (!response.ok) {
-        await route.fulfill({
-          status: response.status,
-          contentType: response.headers.get('content-type') ?? 'text/plain',
-          body: await response.text()
-        })
-        return
-      }
-
-      objectInfo = (await response.json()) as ObjectInfoResponse
-    } catch (error) {
-      const message = error instanceof Error ? error.message : String(error)
-      await route.fulfill({
-        status: 502,
-        contentType: 'application/json',
-        body: JSON.stringify({
-          error: `Failed to fetch setup object_info from ${objectInfoUrl}: ${message}`
-        })
-      })
-      return
-    }
-
-    await customize?.(objectInfo)
-    await route.fulfill({
-      status: 200,
-      contentType: 'application/json',
-      body: JSON.stringify(objectInfo)
-    })
-  }
-
-  await page.route(OBJECT_INFO_ROUTE, objectInfoRouteHandler)
-  return async () => {
-    if (page.isClosed()) return
-    await page.unroute(OBJECT_INFO_ROUTE, objectInfoRouteHandler)
-  }
-}

browser_tests/fixtures/utils/promotedMissingModel.ts

@@ -1,425 +0,0 @@
-import { readFileSync } from 'fs'
-
-import { expect } from '@playwright/test'
-import type { Locator } from '@playwright/test'
-
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { TestIds } from '@e2e/fixtures/selectors'
-import { loadWorkflowAndOpenErrorsTab } from '@e2e/fixtures/helpers/ErrorsTabHelper'
-import { assetPath } from '@e2e/fixtures/utils/paths'
-import { PropertiesPanelHelper } from '@e2e/tests/propertiesPanel/PropertiesPanelHelper'
-
-import type { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/workflowSchema'
-import { toNodeId } from '@/types/nodeId'
-
-const PROMOTED_MODEL_WIDGET_NAME = 'ckpt_name'
-
-interface PromotedMissingModelWorkflow {
-  workflowName: string
-  hostNodeId: number
-  hostNodeTitle: string
-  sharedDefinitionSiblingHostNodeId?: number
-  sharedDefinitionSiblingHostNodeTitle?: string
-}
-
-type RootWorkflowNode = {
-  id: number | string
-  widgets_values?: unknown[] | Record<string, unknown>
-}
-
-type RootWorkflowData = ComfyWorkflowJSON & {
-  nodes?: RootWorkflowNode[]
-}
-
-export const NESTED_PROMOTED_MISSING_MODEL_WORKFLOW: PromotedMissingModelWorkflow =
-  {
-    workflowName: 'missing/missing_model_nested_promoted_widget',
-    hostNodeId: 4,
-    hostNodeTitle: 'Outer Subgraph with Promoted Missing Model',
-    sharedDefinitionSiblingHostNodeId: 3,
-    sharedDefinitionSiblingHostNodeTitle: 'Resolved Shared Outer Subgraph'
-  }
-
-export function getMissingModelLabel(group: Locator, modelName: string) {
-  return group.getByRole('button', { name: modelName, exact: true })
-}
-
-export async function expectSingleMissingModelReference(
-  group: Locator,
-  modelName: string
-) {
-  await expectMissingModelReferenceCount(group, modelName, 1)
-}
-
-export async function expectMissingModelReferenceCount(
-  group: Locator,
-  modelName: string,
-  count: number
-) {
-  await expect(getMissingModelLabel(group, modelName)).toHaveCount(1)
-  const badge = group.getByTestId(TestIds.dialogs.missingModelReferenceCount)
-  if (count === 1) {
-    await expect(badge).toBeHidden()
-    return
-  }
-  await expect(badge).toBeVisible()
-  await expect(badge).toHaveText(String(count))
-}
-
-export async function loadPromotedMissingModelAndOpenErrorsTab(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  modelName: string
-): Promise<Locator> {
-  await loadWorkflowAndOpenErrorsTab(comfyPage, workflow.workflowName)
-
-  const missingModelGroup = comfyPage.page.getByTestId(
-    TestIds.dialogs.missingModelsGroup
-  )
-  await expectSingleMissingModelReference(missingModelGroup, modelName)
-  return missingModelGroup
-}
-
-export async function loadPromotedMissingModelWithHostValuesAndOpenErrorsTab(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  hostValues: Record<number, string>,
-  modelName: string,
-  expectedReferenceCount: number
-): Promise<Locator> {
-  await loadPromotedMissingModelWithHostValues(comfyPage, workflow, hostValues)
-
-  const errorOverlay = comfyPage.page.getByTestId(TestIds.dialogs.errorOverlay)
-  await expect(errorOverlay).toBeVisible()
-  await errorOverlay.getByTestId(TestIds.dialogs.errorOverlaySeeErrors).click()
-  await expect(errorOverlay).toBeHidden()
-
-  const missingModelGroup = comfyPage.page.getByTestId(
-    TestIds.dialogs.missingModelsGroup
-  )
-  await expectMissingModelReferenceCount(
-    missingModelGroup,
-    modelName,
-    expectedReferenceCount
-  )
-  return missingModelGroup
-}
-
-export async function expectNoMissingModelUi(comfyPage: ComfyPage) {
-  const panel = new PropertiesPanelHelper(comfyPage.page)
-  await expect(
-    comfyPage.page.getByTestId(TestIds.dialogs.errorOverlay)
-  ).toBeHidden()
-  await panel.open(comfyPage.actionbar.propertiesButton)
-  await expect(
-    panel.root.getByTestId(TestIds.propertiesPanel.errorsTab)
-  ).toBeHidden()
-  await expect(
-    comfyPage.page.getByTestId(TestIds.dialogs.missingModelsGroup)
-  ).toBeHidden()
-}
-
-export async function selectVueComboPromotedModelByTitle(
-  comfyPage: ComfyPage,
-  nodeTitle: string,
-  modelName: string
-) {
-  await comfyPage.vueNodes.selectComboOption(
-    nodeTitle,
-    PROMOTED_MODEL_WIDGET_NAME,
-    modelName
-  )
-}
-
-export async function selectVueAssetPromotedModel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  currentModelName: string,
-  modelName: string
-) {
-  await selectModelFromFormDropdown(
-    comfyPage,
-    comfyPage.vueNodes.getNodeByTitle(workflow.hostNodeTitle),
-    currentModelName,
-    modelName
-  )
-}
-
-export async function selectSectionComboPromotedModel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  modelName: string
-) {
-  const panel = await openHostNodeParametersPanel(comfyPage, workflow)
-  const combo = panel.contentArea.getByRole('combobox', {
-    name: PROMOTED_MODEL_WIDGET_NAME,
-    exact: true
-  })
-  await combo.click()
-  await comfyPage.page
-    .getByRole('option', { name: modelName, exact: true })
-    .click()
-}
-
-export async function selectSectionAssetPromotedModel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  currentModelName: string,
-  modelName: string
-) {
-  const panel = await openHostNodeParametersPanel(comfyPage, workflow)
-  await selectModelFromFormDropdown(
-    comfyPage,
-    panel.contentArea,
-    currentModelName,
-    modelName
-  )
-}
-
-export async function setLegacyPromotedComboModel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  modelName: string
-) {
-  await comfyPage.page.evaluate(
-    ({ hostNodeId, widgetName, value }) => {
-      type LegacyPromotedWidget = {
-        name?: string
-        value?: unknown
-        callback?: (value: string) => void
-        setValue?: (
-          value: string,
-          options: {
-            e: PointerEvent
-            node: unknown
-            canvas: unknown
-          }
-        ) => void
-      }
-      type LegacyPromotedNode = {
-        onWidgetChanged?: (
-          name: string,
-          newValue: string,
-          oldValue: unknown,
-          widget: LegacyPromotedWidget
-        ) => void
-        widgets?: LegacyPromotedWidget[]
-      }
-      type LegacyPromotedGraph = {
-        getNodeById: (nodeId: number) => LegacyPromotedNode | undefined
-      }
-
-      const currentGraph = window.app?.graph as LegacyPromotedGraph | undefined
-      const hostNode: LegacyPromotedNode | undefined =
-        currentGraph?.getNodeById(hostNodeId)
-      if (!hostNode) {
-        throw new Error(`Expected subgraph host node ${hostNodeId}`)
-      }
-
-      const widget = hostNode.widgets?.find(
-        (entry) => entry.name === widgetName
-      ) as LegacyPromotedWidget | undefined
-      if (!widget) {
-        throw new Error(`Expected host ${widgetName} widget`)
-      }
-
-      const oldValue = widget.value
-      if (widget.setValue) {
-        widget.setValue(value, {
-          e: new PointerEvent('pointerup'),
-          node: hostNode,
-          canvas: window.app!.canvas
-        })
-        return
-      }
-
-      widget.value = value
-      widget.callback?.(value)
-      hostNode.onWidgetChanged?.(
-        widget.name ?? widgetName,
-        value,
-        oldValue,
-        widget
-      )
-    },
-    {
-      hostNodeId: workflow.hostNodeId,
-      widgetName: PROMOTED_MODEL_WIDGET_NAME,
-      value: modelName
-    }
-  )
-}
-
-export async function selectLegacyPromotedAssetModel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  assetId: string
-) {
-  await clickLegacyHostPromotedWidget(comfyPage, workflow)
-
-  const modal = comfyPage.page.locator(
-    '[data-component-id="AssetBrowserModal"]'
-  )
-  await expect(modal).toBeVisible()
-  const assetCard = modal.locator(`[data-asset-id="${assetId}"]`)
-  await expect(assetCard).toBeVisible()
-  await assetCard.getByRole('button', { name: 'Use', exact: true }).click()
-  await expect(modal).toBeHidden()
-}
-
-export async function expectResolvedPromotedModelSuppressesStaleInteriorErrors(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  expectedStaleInteriorWidgets: Array<{
-    subgraphNodeIdToEnter: string
-    nodeTitle: string
-  }>,
-  resolvedModelName: string,
-  staleModelName: string
-) {
-  await loadPromotedMissingModelWithHostValues(comfyPage, workflow, {
-    [workflow.hostNodeId]: resolvedModelName
-  })
-
-  const promotedModelCombo = comfyPage.vueNodes
-    .getNodeByTitle(workflow.hostNodeTitle)
-    .getByRole('combobox', { name: PROMOTED_MODEL_WIDGET_NAME, exact: true })
-  await expect(promotedModelCombo).toContainText(resolvedModelName)
-  await expectNoMissingModelUi(comfyPage)
-
-  for (const step of expectedStaleInteriorWidgets) {
-    await enterSubgraphForStaleInteriorCheck(
-      comfyPage,
-      step.subgraphNodeIdToEnter
-    )
-    await expect.poll(() => comfyPage.subgraph.isInSubgraph()).toBe(true)
-    await comfyPage.nextFrame()
-
-    const node = comfyPage.vueNodes.getNodeByTitle(step.nodeTitle)
-    await expect(node).toBeVisible()
-
-    const staleCombo = node.getByRole('combobox', {
-      name: PROMOTED_MODEL_WIDGET_NAME,
-      exact: true
-    })
-    await expect(
-      staleCombo,
-      `${step.nodeTitle} should expose the stale linked interior widget`
-    ).toBeDisabled()
-    await expect(
-      staleCombo,
-      `${step.nodeTitle} should keep the stale interior value`
-    ).toContainText(staleModelName)
-    await expectNoMissingModelUi(comfyPage)
-  }
-}
-
-async function openHostNodeParametersPanel(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow
-): Promise<PropertiesPanelHelper> {
-  await comfyPage.vueNodes.selectNode(String(workflow.hostNodeId))
-  const panel = new PropertiesPanelHelper(comfyPage.page)
-  await panel.open(comfyPage.actionbar.propertiesButton)
-  await expect(panel.getTab('Parameters')).toBeVisible()
-  await panel.switchToTab('Parameters')
-  return panel
-}
-
-async function loadPromotedMissingModelWithHostValues(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow,
-  hostValues: Record<number, string>
-) {
-  const graphData = readPromotedMissingModelWorkflow(workflow.workflowName)
-  for (const [hostNodeId, value] of Object.entries(hostValues)) {
-    setRootHostWidgetValue(graphData, Number(hostNodeId), value)
-  }
-
-  await comfyPage.workflow.loadGraphData(graphData)
-  await comfyPage.vueNodes.waitForNodes()
-}
-
-function readPromotedMissingModelWorkflow(workflowName: string) {
-  return JSON.parse(
-    readFileSync(assetPath(`${workflowName}.json`), 'utf-8')
-  ) as RootWorkflowData
-}
-
-function setRootHostWidgetValue(
-  graphData: RootWorkflowData,
-  hostNodeId: number,
-  value: string
-) {
-  const hostNode = graphData.nodes?.find(
-    (node) => Number(node.id) === hostNodeId
-  )
-  if (!hostNode) throw new Error(`Expected host node ${hostNodeId}`)
-
-  if (Array.isArray(hostNode.widgets_values)) {
-    hostNode.widgets_values[0] = value
-    return
-  }
-
-  hostNode.widgets_values = {
-    ...(hostNode.widgets_values ?? {}),
-    [PROMOTED_MODEL_WIDGET_NAME]: value
-  }
-}
-
-async function selectModelFromFormDropdown(
-  comfyPage: ComfyPage,
-  root: Locator,
-  currentModelName: string,
-  nextModelName: string
-) {
-  const trigger = root
-    .getByRole('button', { name: currentModelName, exact: true })
-    .first()
-  await expect(trigger).toBeVisible()
-  await trigger.click()
-
-  const menu = comfyPage.page.getByTestId('form-dropdown-menu')
-  await expect(menu).toBeVisible()
-  await menu.getByText(nextModelName, { exact: true }).click()
-  await expect(menu).toBeHidden()
-}
-
-async function clickLegacyHostPromotedWidget(
-  comfyPage: ComfyPage,
-  workflow: PromotedMissingModelWorkflow
-) {
-  const hostNode = await comfyPage.nodeOps.getNodeRefById(workflow.hostNodeId)
-  await hostNode.centerOnNode()
-  const widget = await hostNode.getWidgetByName(PROMOTED_MODEL_WIDGET_NAME)
-  await widget.click()
-}
-
-async function enterSubgraphForStaleInteriorCheck(
-  comfyPage: ComfyPage,
-  nodeId: string
-) {
-  const numericNodeId = Number(nodeId)
-  if (Number.isNaN(numericNodeId)) {
-    throw new Error(`Expected numeric subgraph node id, got ${nodeId}`)
-  }
-
-  const normalizedNodeId = String(numericNodeId)
-  const enterButton =
-    comfyPage.vueNodes.getSubgraphEnterButton(normalizedNodeId)
-  if ((await enterButton.count()) > 0) {
-    await comfyPage.vueNodes.enterSubgraph(normalizedNodeId)
-    return
-  }
-
-  await comfyPage.page.evaluate((targetNodeId) => {
-    const graph = window.app?.canvas.graph
-    const node = graph?.getNodeById(targetNodeId)
-    if (!node?.isSubgraphNode()) {
-      throw new Error(`Expected visible subgraph node ${targetNodeId}`)
-    }
-    window.app!.canvas.setGraph(node.subgraph)
-  }, toNodeId(normalizedNodeId))
-  await comfyPage.nextFrame()
-  await comfyPage.vueNodes.waitForNodes()
-}

browser_tests/fixtures/utils/promotedWidgets.ts

@@ -1,5 +1,4 @@
 import type { NodeProperty } from '@/lib/litegraph/src/LGraphNode'
-import { toNodeId } from '@/types/nodeId'
 
 import { parsePreviewExposures } from '@/core/schemas/previewExposureSchema'
 import type { PreviewExposure } from '@/core/schemas/previewExposureSchema'
@@ -45,7 +44,6 @@ export async function getPromotedWidgets(
   comfyPage: ComfyPage,
   nodeId: string
 ): Promise<PromotedWidgetEntry[]> {
-  const localNodeId = toNodeId(nodeId)
   const { widgetSources, previewExposures } = await comfyPage.page.evaluate(
     (id) => {
       const node = window.app!.canvas.graph!.getNodeById(id)
@@ -93,7 +91,7 @@ export async function getPromotedWidgets(
       })
       return { widgetSources, previewExposures }
     },
-    localNodeId
+    nodeId
   )
 
   const exposures = isNodeProperty(previewExposures)

browser_tests/fixtures/utils/workspaceMocks.ts

@@ -1,68 +0,0 @@
-import type { Page } from '@playwright/test'
-
-import type {
-  Member,
-  WorkspaceWithRole
-} from '@/platform/workspace/api/workspaceApi'
-
-import { jsonRoute } from '@e2e/fixtures/utils/jsonRoute'
-
-export function workspace(
-  type: 'personal' | 'team',
-  role: 'owner' | 'member'
-): WorkspaceWithRole {
-  return {
-    id: `ws-${type}`,
-    name: type === 'team' ? 'My Team' : 'Personal Workspace',
-    type,
-    role,
-    created_at: '2026-01-01T00:00:00Z',
-    joined_at: '2026-01-01T00:00:00Z'
-  }
-}
-
-export function member(
-  overrides: Partial<Member> & Pick<Member, 'email' | 'role'>
-): Member {
-  return {
-    id: `user-${overrides.email}`,
-    name: overrides.email,
-    joined_at: '2026-01-01T00:00:00Z',
-    is_original_owner: false,
-    ...overrides
-  }
-}
-
-/**
- * Stub the workspace resolution + members list so the cloud app boots into the
- * given workspace with the given roster (drives the original-owner gate).
- */
-export async function mockWorkspace(
-  page: Page,
-  ws: WorkspaceWithRole,
-  members: Member[]
-) {
-  await page.route('**/api/workspaces', async (route) => {
-    if (route.request().method() !== 'GET') return route.fallback()
-    await route.fulfill(jsonRoute({ workspaces: [ws] }))
-  })
-  await page.route('**/api/auth/token', (r) =>
-    r.fulfill(
-      jsonRoute({
-        token: 'mock-workspace-token',
-        expires_at: new Date(Date.now() + 60 * 60 * 1000).toISOString(),
-        workspace: { id: ws.id, name: ws.name, type: ws.type },
-        role: ws.role,
-        permissions: []
-      })
-    )
-  )
-  await page.route('**/api/workspace/members**', (r) =>
-    r.fulfill(
-      jsonRoute({
-        members,
-        pagination: { offset: 0, limit: 50, total: members.length }
-      })
-    )
-  )
-}

browser_tests/tests/billingFacadeConsumers.spec.ts

@@ -1,194 +0,0 @@
-import { expect } from '@playwright/test'
-import type { Page } from '@playwright/test'
-
-import type { CloudSubscriptionStatusResponse } from '@/platform/cloud/subscription/composables/useSubscription'
-import type { RemoteConfig } from '@/platform/remoteConfig/types'
-import type {
-  BillingBalanceResponse,
-  BillingStatusResponse
-} from '@/platform/workspace/api/workspaceApi'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { mockSystemStats } from '@e2e/fixtures/data/systemStats'
-import { CloudAuthHelper } from '@e2e/fixtures/helpers/CloudAuthHelper'
-
-/**
- * Billing facade consumers — FE-933 (B3) regression.
- *
- * The repointed surfaces (avatar popover tier badge / balance, free-tier
- * dialog renewal date) must keep rendering from `useBillingContext`. The facade
- * selects its backend by flag: `team_workspaces_enabled: false` routes through
- * the legacy `/customers/*` endpoints, while `true` routes a personal workspace
- * through the workspace `/api/billing/*` endpoints. Both shapes are mocked here.
- * Drives a raw `page` (not the `comfyPage` fixture) so the cloud app boots
- * against fully mocked endpoints — same pattern as creditsTile.spec.ts.
- */
-const APP_URL = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-
-const jsonRoute = (body: unknown) => ({
-  status: 200,
-  contentType: 'application/json',
-  body: JSON.stringify(body)
-})
-
-// The workspace `/api/billing/status` shape mirrors the legacy subscription
-// status; map the fields so a single test fixture drives both backends.
-const toWorkspaceStatus = (
-  s: CloudSubscriptionStatusResponse
-): BillingStatusResponse => ({
-  is_active: s.is_active ?? false,
-  subscription_tier: s.subscription_tier ?? undefined,
-  subscription_duration: s.subscription_duration ?? undefined,
-  renewal_date: s.renewal_date ?? undefined,
-  cancel_at: s.end_date ?? undefined,
-  has_funds: s.has_fund ?? true
-})
-
-const mockBalance: BillingBalanceResponse = {
-  amount_micros: 6000, // -> 12,660 credits
-  currency: 'usd',
-  effective_balance_micros: 6000
-}
-
-async function mockCloudBoot(
-  page: Page,
-  subscriptionStatus: CloudSubscriptionStatusResponse,
-  remoteConfig: RemoteConfig = { team_workspaces_enabled: false }
-) {
-  await page.route('**/api/features', (r) => r.fulfill(jsonRoute(remoteConfig)))
-  await page.route('**/api/system_stats', (r) =>
-    r.fulfill(jsonRoute(mockSystemStats))
-  )
-  await page.route('**/api/users', (r) =>
-    r.fulfill(
-      jsonRoute({
-        storage: 'server',
-        migrated: true,
-        users: { 'test-user-e2e': 'E2E Test User' }
-      })
-    )
-  )
-  // TutorialCompleted suppresses the new-user template browser, whose modal
-  // overlay would otherwise intercept clicks on the topbar.
-  await page.route('**/api/settings', (r) =>
-    r.fulfill(jsonRoute({ 'Comfy.TutorialCompleted': true }))
-  )
-  await page.route('**/api/userdata**', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/extensions', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/object_info', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/global_subgraphs', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/i18n', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/auth/session', (r) =>
-    r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
-  )
-  await page.route('**/releases**', (r) => r.fulfill(jsonRoute([])))
-
-  // Single personal workspace.
-  await page.route('**/api/workspaces', (r) =>
-    r.fulfill(
-      jsonRoute({
-        workspaces: [
-          {
-            id: 'ws-personal',
-            name: 'Personal Workspace',
-            type: 'personal',
-            role: 'owner'
-          }
-        ]
-      })
-    )
-  )
-
-  // Legacy backend (team_workspaces_enabled: false).
-  await page.route('**/customers/cloud-subscription-status', (r) =>
-    r.fulfill(jsonRoute(subscriptionStatus))
-  )
-  await page.route('**/customers/balance', (r) =>
-    r.fulfill(jsonRoute(mockBalance))
-  )
-
-  // Workspace backend (team_workspaces_enabled: true) — a personal workspace
-  // now routes through `/api/billing/*`.
-  await page.route('**/api/billing/status', (r) =>
-    r.fulfill(jsonRoute(toWorkspaceStatus(subscriptionStatus)))
-  )
-  await page.route('**/api/billing/balance', (r) =>
-    r.fulfill(jsonRoute(mockBalance))
-  )
-  await page.route('**/api/billing/plans', (r) =>
-    r.fulfill(jsonRoute({ plans: [] }))
-  )
-}
-
-async function bootApp(page: Page) {
-  const auth = new CloudAuthHelper(page)
-  await auth.mockAuth()
-
-  await page.addInitScript(() => {
-    localStorage.setItem('Comfy.userId', 'test-user-e2e')
-  })
-
-  await page.goto(APP_URL)
-  await page.waitForFunction(() => !!window.app?.extensionManager, null, {
-    timeout: 45_000
-  })
-}
-
-test.describe('Billing facade consumers (FE-933)', { tag: '@cloud' }, () => {
-  test('avatar popover renders tier badge and balance from the facade', async ({
-    page
-  }) => {
-    test.setTimeout(60_000)
-
-    await mockCloudBoot(page, {
-      is_active: true,
-      subscription_tier: 'PRO',
-      subscription_duration: 'MONTHLY',
-      renewal_date: '2099-02-20T10:00:00Z',
-      end_date: null
-    })
-    await bootApp(page)
-
-    await page.getByRole('button', { name: 'Current user' }).click()
-    const popover = page.locator('.current-user-popover')
-    await expect(popover).toBeVisible()
-
-    await expect(popover.getByText('Pro', { exact: true })).toBeVisible()
-    await expect(popover.getByText('12,660')).toBeVisible()
-    await expect(popover.getByTestId('add-credits-button')).toBeVisible()
-  })
-
-  test('free-tier dialog shows the renewal date from the facade', async ({
-    page
-  }) => {
-    test.setTimeout(60_000)
-
-    // Boots with team workspaces enabled (production shape); the facade routes a
-    // personal workspace through the workspace `/api/billing/*` endpoints. With
-    // subscription gating on, an inactive FREE user gets the "Subscribe to run"
-    // button, which opens the free-tier dialog on click. (refreshRemoteConfig
-    // overwrites window.__CONFIG__ from /api/features, so the flags must come
-    // from the features mock, not an init script.)
-    await mockCloudBoot(
-      page,
-      {
-        is_active: false,
-        subscription_tier: 'FREE',
-        subscription_duration: 'MONTHLY',
-        // 10:00Z keeps the en-US calendar date stable across CI timezones.
-        renewal_date: '2099-02-20T10:00:00Z',
-        end_date: null
-      },
-      { team_workspaces_enabled: true, subscription_required: true }
-    )
-    await bootApp(page)
-
-    await page.getByTestId('subscribe-to-run-button').click()
-
-    // T5: the dialog must source the date from facade renewalDate — when this
-    // line read the legacy store it silently vanished for team users.
-    await expect(
-      page.getByText('Your credits refresh on Feb 20, 2099.')
-    ).toBeVisible()
-  })
-})

browser_tests/tests/bottomPanelShortcuts.spec.ts

@@ -223,23 +223,4 @@ test.describe('Bottom Panel Shortcuts', { tag: '@ui' }, () => {
     await expect(comfyPage.settingDialog.root).toBeVisible()
     await expect(comfyPage.settingDialog.category('Keybinding')).toBeVisible()
   })
-
-  test('should focus keybindings search when opening manage shortcuts', async ({
-    comfyPage
-  }) => {
-    const { bottomPanel } = comfyPage
-
-    await bottomPanel.keyboardShortcutsButton.click()
-    await bottomPanel.shortcuts.manageButton.click()
-
-    await expect(comfyPage.settingDialog.root).toBeVisible()
-    await expect(comfyPage.settingDialog.category('Keybinding')).toBeVisible()
-
-    await expect(
-      comfyPage.page.getByPlaceholder('Search Keybindings...')
-    ).toBeFocused()
-    await expect(
-      comfyPage.page.getByPlaceholder('Search Settings...')
-    ).not.toBeFocused()
-  })
 })

browser_tests/tests/cloud-asset-promoted-widget.spec.ts

@@ -1,100 +0,0 @@
-import { expect } from '@playwright/test'
-import type { Page } from '@playwright/test'
-
-import {
-  assetRequestIncludesTag,
-  createCloudAssetsFixture
-} from '@e2e/fixtures/assetApiFixture'
-import {
-  STABLE_CHECKPOINT,
-  STABLE_CHECKPOINT_2
-} from '@e2e/fixtures/data/assetFixtures'
-import { TestIds } from '@e2e/fixtures/selectors'
-import { toNodeId } from '@/types/nodeId'
-
-const WORKFLOW = 'missing/missing_model_promoted_widget'
-const HOST_NODE_ID = toNodeId(2)
-const WIDGET_NAME = 'ckpt_name'
-const SELECTED_MODEL = STABLE_CHECKPOINT_2.name
-
-const test = createCloudAssetsFixture([STABLE_CHECKPOINT, STABLE_CHECKPOINT_2])
-
-interface WidgetSnapshot {
-  type: string
-  value: string
-  hasLayout: boolean
-}
-
-async function getHostWidgetSnapshot(page: Page): Promise<WidgetSnapshot> {
-  return await page.evaluate(
-    ({ nodeId, widgetName }) => {
-      const node = window.app!.graph.getNodeById(nodeId)
-      const widget = node?.widgets?.find((widget) => widget.name === widgetName)
-
-      return {
-        type: widget?.type ?? '',
-        value: String(widget?.value ?? ''),
-        hasLayout: widget?.last_y != null
-      }
-    },
-    { nodeId: HOST_NODE_ID, widgetName: WIDGET_NAME }
-  )
-}
-
-test.describe(
-  'Promoted subgraph asset widgets',
-  { tag: ['@cloud', '@canvas', '@widget'] },
-  () => {
-    test.afterEach(async ({ comfyPage }) => {
-      await comfyPage.nodeOps.clearGraph()
-    })
-
-    test('legacy asset browser selection updates the promoted host widget value', async ({
-      cloudAssetRequests,
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Assets.UseAssetAPI', true)
-      await comfyPage.workflow.loadWorkflow(WORKFLOW)
-
-      await expect
-        .poll(
-          () =>
-            cloudAssetRequests.some((url) =>
-              assetRequestIncludesTag(url, 'checkpoints')
-            ),
-          { timeout: 10_000 }
-        )
-        .toBe(true)
-      await expect
-        .poll(() => getHostWidgetSnapshot(comfyPage.page))
-        .toMatchObject({
-          type: 'asset',
-          hasLayout: true
-        })
-      const initialWidget = await getHostWidgetSnapshot(comfyPage.page)
-      expect(initialWidget.value).not.toBe(SELECTED_MODEL)
-
-      const hostNode = await comfyPage.nodeOps.getNodeRefById(HOST_NODE_ID)
-      await hostNode.centerOnNode()
-      const promotedWidget = await hostNode.getWidgetByName(WIDGET_NAME)
-      await promotedWidget.click()
-
-      const modal = comfyPage.page.getByTestId(TestIds.assets.browserModal)
-      await expect(modal).toBeVisible()
-
-      const assetCard = modal
-        .getByTestId(TestIds.assets.card)
-        .filter({ hasText: SELECTED_MODEL })
-        .first()
-      await expect(assetCard).toBeVisible()
-      await assetCard.getByRole('button', { name: 'Use' }).click()
-
-      await expect(modal).toBeHidden()
-      await expect
-        .poll(() =>
-          getHostWidgetSnapshot(comfyPage.page).then((widget) => widget.value)
-        )
-        .toBe(SELECTED_MODEL)
-    })
-  }
-)

browser_tests/tests/cloudSurveyGate.spec.ts

@@ -4,7 +4,8 @@ import type { Page } from '@playwright/test'
 import type { RemoteConfig } from '@/platform/remoteConfig/types'
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { bootCloud, mockCloudBoot } from '@e2e/fixtures/utils/cloudBootMocks'
+import { mockSystemStats } from '@e2e/fixtures/data/systemStats'
+import { CloudAuthHelper } from '@e2e/fixtures/helpers/CloudAuthHelper'
 
 /**
  * getSurveyCompletedStatus fails safe: a transient 401 on `/` must not bounce a
@@ -15,12 +16,51 @@ import { bootCloud, mockCloudBoot } from '@e2e/fixtures/utils/cloudBootMocks'
  */
 const APP_URL = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
 
-// `/api/features` is the remote-config source: production builds resolve
-// `onboardingSurveyEnabled` from it (the `ff:` localStorage override is
-// dev-only). Enable the survey so the gate is actually live.
-const BOOT_FEATURES = {
-  onboarding_survey_enabled: true
-} satisfies RemoteConfig
+function jsonRoute(body: unknown) {
+  return {
+    status: 200,
+    contentType: 'application/json',
+    body: JSON.stringify(body)
+  }
+}
+
+async function mockCloudBoot(page: Page) {
+  // `/api/features` is the remote-config source: production builds resolve
+  // `onboardingSurveyEnabled` from it (the `ff:` localStorage override is
+  // dev-only). Enable the survey so the gate is actually live.
+  await page.route('**/api/features', (r) =>
+    r.fulfill(
+      jsonRoute({ onboarding_survey_enabled: true } satisfies RemoteConfig)
+    )
+  )
+  await page.route('**/api/system_stats', (r) =>
+    r.fulfill(jsonRoute(mockSystemStats))
+  )
+  await page.route('**/api/users', (r) =>
+    r.fulfill(
+      jsonRoute({
+        storage: 'server',
+        migrated: true,
+        users: { 'test-user-e2e': 'E2E Test User' }
+      })
+    )
+  )
+  // Cloud user status (getUserCloudStatus) — an active account so the gate
+  // proceeds to the survey check instead of bouncing back to login.
+  await page.route('**/api/user', (r) =>
+    r.fulfill(jsonRoute({ status: 'active' }))
+  )
+  await page.route('**/api/settings', (r) => r.fulfill(jsonRoute({})))
+  await page.route('**/api/userdata**', (r) => r.fulfill(jsonRoute([])))
+  await page.route('**/api/extensions', (r) => r.fulfill(jsonRoute([])))
+  await page.route('**/api/object_info', (r) => r.fulfill(jsonRoute({})))
+  await page.route('**/api/global_subgraphs', (r) => r.fulfill(jsonRoute({})))
+  await page.route('**/api/i18n', (r) => r.fulfill(jsonRoute({})))
+  await page.route('**/api/auth/session', (r) =>
+    r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
+  )
+  await page.route('**/releases**', (r) => r.fulfill(jsonRoute([])))
+}
 
 // Genuine "not completed": the cloud backend returns 404 for a survey key that
 // was never stored. This is the response that must still route to the survey.
@@ -49,13 +89,22 @@ async function mockSurveyTransient401(page: Page) {
   )
 }
 
+async function bootCloud(page: Page) {
+  const auth = new CloudAuthHelper(page)
+  await auth.mockAuth()
+  // Pre-select the mock user to skip the user-select screen.
+  await page.addInitScript(() => {
+    localStorage.setItem('Comfy.userId', 'test-user-e2e')
+  })
+}
+
 test.describe('Cloud onboarding survey gate', { tag: '@cloud' }, () => {
   test('a transient 401 on the survey check does not bounce a working user to the survey', async ({
     page
   }) => {
-    test.slow()
+    test.setTimeout(60_000)
 
-    await mockCloudBoot(page, { features: BOOT_FEATURES })
+    await mockCloudBoot(page)
     await mockSurveyTransient401(page)
     await bootCloud(page)
 
@@ -73,9 +122,9 @@ test.describe('Cloud onboarding survey gate', { tag: '@cloud' }, () => {
   test('a not-completed (404) user landing on / is routed to the survey', async ({
     page
   }) => {
-    test.slow()
+    test.setTimeout(60_000)
 
-    await mockCloudBoot(page, { features: BOOT_FEATURES })
+    await mockCloudBoot(page)
     await mockSurveyNotCompleted(page)
     await bootCloud(page)
 

browser_tests/tests/currentUserPopoverCredits.spec.ts

@@ -2,10 +2,7 @@ import { expect } from '@playwright/test'
 
 import type { CloudSubscriptionStatusResponse } from '@/platform/cloud/subscription/composables/useSubscription'
 import type { RemoteConfig } from '@/platform/remoteConfig/types'
-import type {
-  BillingStatusResponse,
-  WorkspaceWithRole
-} from '@/platform/workspace/api/workspaceApi'
+import type { WorkspaceWithRole } from '@/platform/workspace/api/workspaceApi'
 import type { WorkspaceTokenResponse } from '@/platform/workspace/stores/workspaceAuthStore'
 import type { operations } from '@/types/comfyRegistryTypes'
 import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
@@ -54,20 +51,6 @@ const mockSubscriptionStatus: CloudSubscriptionStatusResponse = {
   end_date: FUTURE_DATE
 }
 
-// With team workspaces enabled, the facade routes a personal workspace through
-// `/api/billing/*`. The cancelled-but-active state maps to `is_active: true`
-// with `subscription_status: 'canceled'`; a paid tier keeps "Add credits"
-// visible (free tier would swap it for "Upgrade to add credits").
-const mockBillingStatus: BillingStatusResponse = {
-  is_active: true,
-  subscription_status: 'canceled',
-  subscription_tier: 'PRO',
-  subscription_duration: 'MONTHLY',
-  has_funds: true,
-  cancel_at: FUTURE_DATE,
-  renewal_date: FUTURE_DATE
-}
-
 // ~6.3M credits — a 7-digit balance is what pushes the second action button out
 // of the popover before the fix.
 const mockBalance: CustomerBalanceResponse = {
@@ -122,32 +105,6 @@ const test = comfyPageFixture.extend({
       })
     )
 
-    // Flag-on (team workspaces enabled) routes a personal workspace through the
-    // workspace billing endpoints, so the popover sources its data from here.
-    await page.route('**/api/billing/status', (route) =>
-      route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(mockBillingStatus)
-      })
-    )
-
-    await page.route('**/api/billing/balance', (route) =>
-      route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(mockBalance)
-      })
-    )
-
-    await page.route('**/api/billing/plans', (route) =>
-      route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify({ plans: [] })
-      })
-    )
-
     await use(page)
   }
 })

browser_tests/tests/dialogs/creditsTile.spec.ts

@@ -1,264 +0,0 @@
-import { expect } from '@playwright/test'
-import type { Page } from '@playwright/test'
-
-import type { RemoteConfig } from '@/platform/remoteConfig/types'
-import type { BillingStatusResponse } from '@/platform/workspace/api/workspaceApi'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { mockSystemStats } from '@e2e/fixtures/data/systemStats'
-import { CloudAuthHelper } from '@e2e/fixtures/helpers/CloudAuthHelper'
-
-// Drives a raw `page` (not the `comfyPage` fixture) so the cloud app boots
-// against fully mocked endpoints; `comfyPage` would try to reach the OSS
-// devtools backend during setup.
-
-/**
- * Credits tile (Settings ▸ Workspace ▸ Plan & Credits) — DES-247 / FE-964.
- *
- * The credits tile only lives inside the authenticated cloud app, which the
- * shared `comfyPage` fixture can't boot (it expects the OSS devtools backend).
- * Instead this drives a raw page: mock Firebase auth + every boot endpoint so
- * the cloud app initializes against fully stubbed data. With team workspaces
- * enabled the facade routes a personal workspace through the workspace
- * `/api/billing/*` endpoints (mocked with an active Pro subscription); the
- * legacy `/customers/*` shapes are mocked too for the flag-off path. The tile
- * should then render its total / progress bar / monthly+additional breakdown /
- * add-credits.
- */
-const APP_URL = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-
-const jsonRoute = (body: unknown) => ({
-  status: 200,
-  contentType: 'application/json',
-  body: JSON.stringify(body)
-})
-
-// Legacy `/customers/balance` and workspace `/api/billing/balance` share the
-// same response shape, so one body fulfills both endpoints.
-const balanceRoute = (balance: {
-  amount: number
-  monthly: number
-  prepaid: number
-}) =>
-  jsonRoute({
-    amount_micros: balance.amount,
-    currency: 'usd',
-    effective_balance_micros: balance.amount,
-    cloud_credit_balance_micros: balance.monthly,
-    prepaid_balance_micros: balance.prepaid
-  })
-
-// 6000 -> 12,660 total; 5000 -> 10,550 monthly remaining; 1000 -> 2,110 extra.
-const DEFAULT_BALANCE = { amount: 6000, monthly: 5000, prepaid: 1000 }
-
-const mockBillingStatus: BillingStatusResponse = {
-  is_active: true,
-  subscription_tier: 'PRO',
-  subscription_duration: 'MONTHLY',
-  renewal_date: '2099-02-20T12:00:00Z',
-  has_funds: true
-}
-
-async function mockCloudBoot(page: Page) {
-  // Frontend-origin boot endpoints (proxied to the backend in production).
-  // `/api/features` is the remote-config source: production builds resolve
-  // `teamWorkspacesEnabled` from it (the `ff:` localStorage override is
-  // dev-only), and the flag gates the Workspace settings panel.
-  await page.route('**/api/features', (r) =>
-    r.fulfill(
-      jsonRoute({ team_workspaces_enabled: true } satisfies RemoteConfig)
-    )
-  )
-  await page.route('**/api/system_stats', (r) =>
-    r.fulfill(jsonRoute(mockSystemStats))
-  )
-  // Include the mock user so the multi-user select screen auto-selects it
-  // (paired with the `Comfy.userId` localStorage seed below).
-  await page.route('**/api/users', (r) =>
-    r.fulfill(
-      jsonRoute({
-        storage: 'server',
-        migrated: true,
-        users: { 'test-user-e2e': 'E2E Test User' }
-      })
-    )
-  )
-  // Non-empty settings with a completed tutorial keep the cloud app from
-  // booting as a new user, whose Workflow Templates dialog would otherwise
-  // auto-open and intercept the Settings click behind its modal backdrop.
-  await page.route('**/api/settings', (r) =>
-    r.fulfill(jsonRoute({ 'Comfy.TutorialCompleted': true }))
-  )
-  await page.route('**/api/userdata**', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/extensions', (r) => r.fulfill(jsonRoute([])))
-  await page.route('**/api/object_info', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/global_subgraphs', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/i18n', (r) => r.fulfill(jsonRoute({})))
-  await page.route('**/api/auth/session', (r) =>
-    r.fulfill(jsonRoute({ token: 'mock-workspace-token' }))
-  )
-  await page.route('**/releases**', (r) => r.fulfill(jsonRoute([])))
-
-  // Single personal workspace.
-  await page.route('**/api/workspaces', (r) =>
-    r.fulfill(
-      jsonRoute({
-        workspaces: [
-          {
-            id: 'ws-personal',
-            name: 'Personal Workspace',
-            type: 'personal',
-            role: 'owner'
-          }
-        ]
-      })
-    )
-  )
-
-  // Legacy billing (flag-off path, api.comfy.org/customers/*).
-  await page.route('**/customers/cloud-subscription-status', (r) =>
-    r.fulfill(
-      jsonRoute({
-        is_active: true,
-        subscription_tier: 'PRO',
-        subscription_duration: 'MONTHLY',
-        renewal_date: '2099-02-20T12:00:00Z',
-        end_date: null
-      })
-    )
-  )
-  await page.route('**/customers/balance', (r) =>
-    r.fulfill(balanceRoute(DEFAULT_BALANCE))
-  )
-
-  // Workspace billing (flag-on path) — a personal workspace now routes through
-  // `/api/billing/*`.
-  await page.route('**/api/billing/status', (r) =>
-    r.fulfill(jsonRoute(mockBillingStatus))
-  )
-  await page.route('**/api/billing/balance', (r) =>
-    r.fulfill(balanceRoute(DEFAULT_BALANCE))
-  )
-  await page.route('**/api/billing/plans', (r) =>
-    r.fulfill(jsonRoute({ plans: [] }))
-  )
-}
-
-async function mockBalance(
-  page: Page,
-  balance: { amount: number; monthly: number; prepaid: number }
-) {
-  await page.unroute('**/customers/balance')
-  await page.unroute('**/api/billing/balance')
-  await page.route('**/customers/balance', (r) =>
-    r.fulfill(balanceRoute(balance))
-  )
-  await page.route('**/api/billing/balance', (r) =>
-    r.fulfill(balanceRoute(balance))
-  )
-}
-
-/** Boots the mocked cloud app and opens Settings ▸ Workspace ▸ Plan & Credits. */
-async function openPlanAndCredits(page: Page) {
-  const auth = new CloudAuthHelper(page)
-  await auth.mockAuth()
-
-  // Pre-select the mock user to skip the user-select screen.
-  await page.addInitScript(() => {
-    localStorage.setItem('Comfy.userId', 'test-user-e2e')
-  })
-
-  await page.goto(APP_URL)
-  await page.waitForFunction(() => !!window.app?.extensionManager, null, {
-    timeout: 45_000
-  })
-
-  // Open Settings ▸ Workspace.
-  await page
-    .getByRole('button', { name: /^Settings/ })
-    .first()
-    .click()
-  const dialog = page.getByTestId('settings-dialog')
-  await expect(dialog).toBeVisible()
-  await dialog.locator('nav').getByRole('button', { name: 'Workspace' }).click()
-
-  return dialog.getByRole('main')
-}
-
-test.describe('Credits tile (Plan & Credits)', { tag: '@cloud' }, () => {
-  test('renders the unified tile with breakdown and add-credits', async ({
-    page
-  }) => {
-    test.setTimeout(60_000)
-
-    await mockCloudBoot(page)
-
-    const content = await openPlanAndCredits(page)
-
-    // Total + remaining suffix (Pro monthly allowance = 21,100; remaining
-    // 10,550 -> used 10,550).
-    await expect(content.getByText('Total credits')).toBeVisible()
-    await expect(content.getByText('12,660')).toBeVisible()
-
-    // Monthly usage bar header + used / left-of-total labels.
-    await expect(content.getByText('Monthly', { exact: true })).toBeVisible()
-    await expect(content.getByText(/Refills Feb/)).toBeVisible()
-    await expect(content.getByText('10,550 used')).toBeVisible()
-    await expect(content.getByText('10,550 left of 21,100')).toBeVisible()
-
-    // Additional credits row + subtitle.
-    await expect(content.getByText('Additional credits')).toBeVisible()
-    await expect(content.getByText('2,110')).toBeVisible()
-    await expect(content.getByText('Used after monthly runs out')).toBeVisible()
-
-    // Permission-gated add-credits action (personal owner can top up).
-    await expect(
-      content.getByRole('button', { name: 'Add credits' })
-    ).toBeVisible()
-
-    // Narrow container (DES-247 responsive variants): drop the used/remaining
-    // labels and the breakdown subtitle, compact the monthly summary numbers.
-    await page.setViewportSize({ width: 360, height: 800 })
-    await expect(content.getByText('10,550 used')).toBeHidden()
-    await expect(content.getByText('remaining', { exact: true })).toBeHidden()
-    await expect(content.getByText('Used after monthly runs out')).toBeHidden()
-    await expect(content.getByText('10,550 left of 21,100')).toBeHidden()
-    await expect(content.getByText('11K left of 21K')).toBeVisible()
-  })
-
-  test('renders the depleted-credit empty states', async ({ page }) => {
-    test.setTimeout(60_000)
-
-    await mockCloudBoot(page)
-    // Monthly allowance fully spent; additional credits keep generation going.
-    await mockBalance(page, { amount: 1000, monthly: 0, prepaid: 1000 })
-
-    const content = await openPlanAndCredits(page)
-
-    // 0-monthly state: depletion notice + IN USE badge on additional credits.
-    await expect(
-      content.getByText('Monthly credits are used up. Refills Feb 20')
-    ).toBeVisible()
-    await expect(
-      content.getByText("You're now spending additional credits.")
-    ).toBeVisible()
-    await expect(content.getByText('In use')).toBeVisible()
-    await expect(content.getByText('0 left of 21,100')).toBeVisible()
-
-    // Drain the remaining additional credits and refresh the tile: the
-    // out-of-credits notice takes over and the badge drops.
-    await mockBalance(page, { amount: 0, monthly: 0, prepaid: 0 })
-    await content.getByRole('button', { name: 'Refresh credits' }).click()
-
-    await expect(
-      content.getByText("You're out of credits. Credits refill Feb 20")
-    ).toBeVisible()
-    await expect(
-      content.getByText('Add more credits to continue generating.')
-    ).toBeVisible()
-    await expect(content.getByText('In use')).toBeHidden()
-    await expect(
-      content.getByRole('button', { name: 'Add credits' })
-    ).toBeVisible()
-  })
-})

browser_tests/tests/dialogs/memberRoleChange.spec.ts

@@ -1,264 +0,0 @@
-import { expect } from '@playwright/test'
-import type { Locator, Page } from '@playwright/test'
-
-import type { Member } from '@/platform/workspace/api/workspaceApi'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import {
-  CREATOR,
-  MEMBER_JANE,
-  MEMBER_JOHN,
-  VIEWER
-} from '@e2e/fixtures/data/cloudWorkspace'
-import { CloudWorkspaceMockHelper } from '@e2e/fixtures/helpers/CloudWorkspaceMockHelper'
-
-// Drives a raw `page` (not the `comfyPage` fixture) so the cloud app boots
-// against fully mocked endpoints; `comfyPage` would try to reach the OSS
-// devtools backend during setup.
-
-/**
- * Member role change (Settings ▸ Workspace ▸ Members) — Figma 2993-15512.
- *
- * The viewer is a promoted owner (not the workspace creator), so the spec can
- * distinguish the creator guard from the self guard: the creator row and the
- * viewer's own row hide the row menu, every other row exposes
- * "Change role ›" (Owner / Member) plus "Remove member". Promoting a member
- * sends PATCH /api/workspace/members/:id {role}, flips the Role column,
- * re-sorts the row under the creator, and the promoted owner stays demotable.
- */
-const APP_URL = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-
-async function openMembersTab(page: Page): Promise<Locator> {
-  await page.goto(APP_URL)
-  await page.waitForFunction(() => !!window.app?.extensionManager, null, {
-    timeout: 45_000
-  })
-
-  await page
-    .getByRole('button', { name: /^Settings/ })
-    .first()
-    .click()
-  const dialog = page.getByTestId('settings-dialog')
-  await expect(dialog).toBeVisible()
-  await dialog.locator('nav').getByRole('button', { name: 'Workspace' }).click()
-
-  const content = dialog.getByRole('main')
-  await content.getByRole('tab', { name: /Members/ }).click()
-  await expect(content.getByText('4 of 30 members')).toBeVisible()
-  return content
-}
-
-function memberRow(content: Locator, email: string): Locator {
-  return content
-    .locator('div.grid')
-    .filter({ has: content.page().getByText(email, { exact: true }) })
-}
-
-function menuButton(row: Locator): Locator {
-  return row.getByRole('button', { name: 'More Options' })
-}
-
-// Reka submenus open on real pointer travel or keyboard; Playwright's
-// synthetic hover doesn't trigger the pointermove handler, so drive the
-// subtrigger with ArrowRight instead.
-async function openChangeRoleSubmenu(page: Page) {
-  const trigger = page.getByRole('menuitem', { name: 'Change role' })
-  await expect(trigger).toBeVisible()
-  await trigger.press('ArrowRight')
-  await expect(
-    page.getByRole('menuitemradio', { name: 'Owner', exact: true })
-  ).toBeVisible()
-}
-
-test.describe('Member role change (Members tab)', { tag: '@cloud' }, () => {
-  test.describe.configure({ timeout: 60_000 })
-
-  test('row menus respect creator and self guards', async ({ page }) => {
-    await new CloudWorkspaceMockHelper(page).setup()
-    const content = await openMembersTab(page)
-
-    // US8/US9 — no row actions on the creator row (Liz) nor on the viewer's
-    // own row; the two plain members each expose a menu.
-    await expect(
-      menuButton(memberRow(content, MEMBER_JOHN.email))
-    ).toBeVisible()
-    await expect(
-      menuButton(memberRow(content, MEMBER_JANE.email))
-    ).toBeVisible()
-    await expect(menuButton(memberRow(content, CREATOR.email))).toHaveCount(0)
-    await expect(menuButton(memberRow(content, VIEWER.email))).toHaveCount(0)
-
-    // US1/US12 — the row menu exposes Change role and the FE-768 remove flow.
-    await menuButton(memberRow(content, MEMBER_JANE.email)).click()
-    await expect(
-      page.getByRole('menuitem', { name: 'Change role' })
-    ).toBeVisible()
-    await page.getByRole('menuitem', { name: 'Remove member' }).click()
-    await expect(page.getByText('Remove this member?')).toBeVisible()
-  })
-
-  test('selecting the current role is a no-op', async ({ page }) => {
-    const state = await new CloudWorkspaceMockHelper(page).setup()
-    const content = await openMembersTab(page)
-
-    const janeRow = memberRow(content, MEMBER_JANE.email)
-    await menuButton(janeRow).click()
-    await openChangeRoleSubmenu(page)
-
-    // The current role is a checked radio item so assistive tech can announce
-    // which role is active.
-    await expect(
-      page.getByRole('menuitemradio', { name: 'Member', exact: true })
-    ).toHaveAttribute('aria-checked', 'true')
-    await expect(
-      page.getByRole('menuitemradio', { name: 'Owner', exact: true })
-    ).toHaveAttribute('aria-checked', 'false')
-
-    await page
-      .getByRole('menuitemradio', { name: 'Member', exact: true })
-      .click()
-
-    await expect(page.getByRole('heading', { name: /an owner\?/ })).toHaveCount(
-      0
-    )
-    expect(state.patches).toHaveLength(0)
-  })
-
-  test('promote dialog shows the Figma copy and cancelling keeps the role', async ({
-    page
-  }) => {
-    const state = await new CloudWorkspaceMockHelper(page).setup()
-    const content = await openMembersTab(page)
-
-    const janeRow = memberRow(content, MEMBER_JANE.email)
-    await menuButton(janeRow).click()
-    await openChangeRoleSubmenu(page)
-    await page
-      .getByRole('menuitemradio', { name: 'Owner', exact: true })
-      .click()
-
-    await expect(
-      page.getByRole('heading', { name: 'Make Jane an owner?' })
-    ).toBeVisible()
-    await expect(page.getByText("They'll be able to:")).toBeVisible()
-    await expect(page.getByText('Add additional credits')).toBeVisible()
-    await expect(
-      page.getByText('Manage members, payment methods, and workspace settings')
-    ).toBeVisible()
-    await expect(
-      page.getByText(
-        'Promote and demote other owners (except the workspace creator).'
-      )
-    ).toBeVisible()
-
-    await page.getByRole('button', { name: 'Cancel', exact: true }).click()
-    await expect(
-      page.getByRole('heading', { name: 'Make Jane an owner?' })
-    ).toHaveCount(0)
-    await expect(janeRow.getByText('Member', { exact: true })).toBeVisible()
-    expect(state.patches).toHaveLength(0)
-  })
-
-  test('promoting a member re-sorts the row under the creator and stays demotable', async ({
-    page
-  }) => {
-    const state = await new CloudWorkspaceMockHelper(page).setup()
-    const content = await openMembersTab(page)
-
-    const emails = content.getByText(/@test\.comfy\.org/)
-    await expect(emails).toHaveText([
-      CREATOR.email,
-      VIEWER.email,
-      MEMBER_JOHN.email,
-      MEMBER_JANE.email
-    ])
-
-    const janeRow = memberRow(content, MEMBER_JANE.email)
-    await menuButton(janeRow).click()
-    await openChangeRoleSubmenu(page)
-    await page
-      .getByRole('menuitemradio', { name: 'Owner', exact: true })
-      .click()
-    await page.getByRole('button', { name: 'Make owner' }).click()
-
-    await expect(page.getByText('Role updated')).toBeVisible()
-    await expect(janeRow.getByText('Owner', { exact: true })).toBeVisible()
-    await expect(emails).toHaveText([
-      CREATOR.email,
-      VIEWER.email,
-      MEMBER_JANE.email,
-      MEMBER_JOHN.email
-    ])
-    expect(state.patches).toEqual([
-      {
-        url: expect.stringContaining('/api/workspace/members/u-jane'),
-        role: 'owner'
-      }
-    ])
-
-    // The promoted owner keeps its row menu (still demotable).
-    await expect(menuButton(janeRow)).toBeVisible()
-  })
-
-  test('demoting an owner returns them to member', async ({ page }) => {
-    const ownerJane: Member = { ...MEMBER_JANE, role: 'owner' }
-    const state = await new CloudWorkspaceMockHelper(page).setup([
-      CREATOR,
-      VIEWER,
-      ownerJane,
-      MEMBER_JOHN
-    ])
-    const content = await openMembersTab(page)
-
-    const janeRow = memberRow(content, MEMBER_JANE.email)
-    await expect(janeRow.getByText('Owner', { exact: true })).toBeVisible()
-
-    await menuButton(janeRow).click()
-    await openChangeRoleSubmenu(page)
-    await page
-      .getByRole('menuitemradio', { name: 'Member', exact: true })
-      .click()
-    await expect(
-      page.getByRole('heading', { name: 'Demote Jane to member?' })
-    ).toBeVisible()
-    await expect(page.getByText("They'll lose admin access.")).toBeVisible()
-    await page.getByRole('button', { name: 'Demote to member' }).click()
-
-    await expect(janeRow.getByText('Member', { exact: true })).toBeVisible()
-    expect(state.patches).toEqual([
-      {
-        url: expect.stringContaining('/api/workspace/members/u-jane'),
-        role: 'member'
-      }
-    ])
-  })
-
-  test('failed role change keeps the dialog open with an error toast', async ({
-    page
-  }) => {
-    await new CloudWorkspaceMockHelper(page).setup()
-    // Override the member route so PATCH fails after boot succeeds.
-    await page.route('**/api/workspace/members/**', (route) =>
-      route.request().method() === 'PATCH'
-        ? route.fulfill({ status: 500, body: '{}' })
-        : route.fallback()
-    )
-    const content = await openMembersTab(page)
-
-    const janeRow = memberRow(content, MEMBER_JANE.email)
-    await menuButton(janeRow).click()
-    await openChangeRoleSubmenu(page)
-    await page
-      .getByRole('menuitemradio', { name: 'Owner', exact: true })
-      .click()
-    await page.getByRole('button', { name: 'Make owner' }).click()
-
-    // US10 — error toast, dialog stays open, role unchanged.
-    await expect(page.getByText('Failed to update role')).toBeVisible()
-    await expect(
-      page.getByRole('heading', { name: 'Make Jane an owner?' })
-    ).toBeVisible()
-    await page.getByRole('button', { name: 'Cancel', exact: true }).click()
-    await expect(janeRow.getByText('Member', { exact: true })).toBeVisible()
-  })
-})

browser_tests/tests/dialogs/pricingTableDeepLink.spec.ts

@@ -1,128 +0,0 @@
-import { expect } from '@playwright/test'
-import type { Page } from '@playwright/test'
-
-import type { RemoteConfig } from '@/platform/remoteConfig/types'
-import type {
-  Member,
-  WorkspaceWithRole
-} from '@/platform/workspace/api/workspaceApi'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { mockBilling } from '@e2e/fixtures/utils/cloudBillingMocks'
-import { bootCloud, mockCloudBoot } from '@e2e/fixtures/utils/cloudBootMocks'
-import { jsonRoute } from '@e2e/fixtures/utils/jsonRoute'
-import {
-  member,
-  mockWorkspace,
-  workspace
-} from '@e2e/fixtures/utils/workspaceMocks'
-
-/**
- * The `?pricing=` deep link opens the pricing table on app load, gated to the
- * original owner (canManageSubscriptionLifecycle). Drives a raw `page` so the
- * cloud app boots against fully mocked endpoints, like the survey-gate spec.
- */
-const APP_URL = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-
-// CloudAuthHelper.mockAuth() signs in as this email; the original-owner gate
-// matches it against the members self-row.
-const SELF_EMAIL = 'e2e@test.comfy.org'
-
-const BOOT_FEATURES = { team_workspaces_enabled: true } satisfies RemoteConfig
-// Disable the experimental Asset API: with it on (cloud default) the unmocked
-// asset endpoints 403 and workflow restore throws uncaught, aborting the
-// GraphCanvas onMounted chain before the deep-link loader.
-const BOOT_SETTINGS = { 'Comfy.Assets.UseAssetAPI': false }
-
-// The deep-link loader runs at the tail of GraphCanvas onMounted, so the boot
-// chain must not throw before it: a missing settings subpath, prompt exec_info,
-// or queue status each abort that chain.
-async function mockGraphBootExtras(page: Page) {
-  // Boot only reads these; fall back on any write so an unexpected POST/PUT
-  // surfaces instead of being masked by a blanket 200.
-  await page.route('**/api/settings/**', (route) => {
-    if (route.request().method() !== 'GET') return route.fallback()
-    return route.fulfill(jsonRoute({}))
-  })
-  await page.route('**/api/prompt', (route) => {
-    if (route.request().method() !== 'GET') return route.fallback()
-    return route.fulfill(jsonRoute({ exec_info: { queue_remaining: 0 } }))
-  })
-  await page.route('**/api/queue', (route) => {
-    if (route.request().method() !== 'GET') return route.fallback()
-    return route.fulfill(jsonRoute({ queue_running: [], queue_pending: [] }))
-  })
-}
-
-async function setupCloudApp(
-  page: Page,
-  ws: WorkspaceWithRole,
-  members: Member[]
-) {
-  await mockCloudBoot(page, {
-    features: BOOT_FEATURES,
-    settings: BOOT_SETTINGS
-  })
-  await mockGraphBootExtras(page)
-  await mockBilling(page)
-  await mockWorkspace(page, ws, members)
-  await bootCloud(page)
-}
-
-const pricingHeading = (page: Page) =>
-  page.getByRole('heading', { name: 'Choose a Plan' })
-
-test.describe('Pricing table deep link', { tag: '@cloud' }, () => {
-  test('opens the pricing table for a personal owner', async ({ page }) => {
-    test.slow()
-    await setupCloudApp(page, workspace('personal', 'owner'), [])
-
-    await page.goto(`${APP_URL}/?pricing=1`)
-
-    await expect(pricingHeading(page)).toBeVisible({ timeout: 45_000 })
-    await expect(page).not.toHaveURL(/[?&]pricing=/)
-  })
-
-  test('opens on the Team tab for ?pricing=team', async ({ page }) => {
-    test.slow()
-    await setupCloudApp(page, workspace('personal', 'owner'), [])
-
-    await page.goto(`${APP_URL}/?pricing=team`)
-
-    await expect(pricingHeading(page)).toBeVisible({ timeout: 45_000 })
-    await expect(
-      page.getByRole('button', { name: 'For Teams' })
-    ).toHaveAttribute('aria-pressed', 'true')
-  })
-
-  test('opens for a team original owner', async ({ page }) => {
-    test.slow()
-    await setupCloudApp(page, workspace('team', 'owner'), [
-      member({ email: SELF_EMAIL, role: 'owner', is_original_owner: true })
-    ])
-
-    await page.goto(`${APP_URL}/?pricing=1`)
-
-    await expect(pricingHeading(page)).toBeVisible({ timeout: 45_000 })
-  })
-
-  test('is a silent no-op for a team member', async ({ page }) => {
-    test.slow()
-    await setupCloudApp(page, workspace('team', 'member'), [
-      member({
-        email: 'creator@test.comfy.org',
-        role: 'owner',
-        is_original_owner: true
-      }),
-      member({ email: SELF_EMAIL, role: 'member' })
-    ])
-
-    await page.goto(`${APP_URL}/?pricing=1`)
-
-    await page.waitForFunction(() => !!window.app?.extensionManager, null, {
-      timeout: 45_000
-    })
-    await expect(page).not.toHaveURL(/[?&]pricing=/)
-    await expect(pricingHeading(page)).toBeHidden()
-  })
-})

browser_tests/tests/graph.spec.ts

@@ -1,7 +1,5 @@
 import { expect } from '@playwright/test'
 
-import { toNodeId } from '@/types/nodeId'
-
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 
 test.beforeEach(async ({ comfyPage }) => {
@@ -41,22 +39,16 @@ test.describe('Graph', { tag: ['@smoke', '@canvas'] }, () => {
     await comfyPage.workflow.loadWorkflow('links/duplicate_links_slot_drift')
 
     function evaluateGraph() {
-      const nodeIds = {
-        switchCfg: toNodeId(120),
-        ksampler85: toNodeId(85),
-        ksampler86: toNodeId(86)
-      }
-
-      return comfyPage.page.evaluate((nodeIds) => {
+      return comfyPage.page.evaluate(() => {
         const graph = window.app!.graph!
 
         const subgraph = graph.subgraphs.values().next().value
         if (!subgraph) return { error: 'No subgraph found' }
 
         // Node 120 = Switch (CFG), connects to both KSamplerAdvanced 85 and 86
-        const switchCfg = subgraph.getNodeById(nodeIds.switchCfg)
-        const ksampler85 = subgraph.getNodeById(nodeIds.ksampler85)
-        const ksampler86 = subgraph.getNodeById(nodeIds.ksampler86)
+        const switchCfg = subgraph.getNodeById(120)
+        const ksampler85 = subgraph.getNodeById(85)
+        const ksampler86 = subgraph.getNodeById(86)
         if (!switchCfg || !ksampler85 || !ksampler86)
           return { error: 'Required nodes not found' }
 
@@ -82,10 +74,7 @@ test.describe('Graph', { tag: ['@smoke', '@canvas'] }, () => {
         // Count links from Switch(CFG) to node 85 cfg (should be 1, not 2)
         let cfgLinkToNode85Count = 0
         for (const link of subgraph.links.values()) {
-          if (
-            String(link.origin_id) === '120' &&
-            String(link.target_id) === '85'
-          )
+          if (link.origin_id === 120 && link.target_id === 85)
             cfgLinkToNode85Count++
         }
 
@@ -100,7 +89,7 @@ test.describe('Graph', { tag: ['@smoke', '@canvas'] }, () => {
           switchOutputLinkCount,
           cfgLinkToNode85Count
         }
-      }, nodeIds)
+      })
     }
 
     // Poll graph state once, then assert all properties

browser_tests/tests/imageCompare.spec.ts

@@ -4,9 +4,6 @@ import { expect } from '@playwright/test'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 import { TestIds } from '@e2e/fixtures/selectors'
-import { toNodeId } from '@/types/nodeId'
-
-const IMAGE_COMPARE_NODE_ID = toNodeId(1)
 
 test.describe('Image Compare', { tag: ['@widget', '@vue-nodes'] }, () => {
   test.beforeEach(async ({ comfyPage }) => {
@@ -32,15 +29,15 @@ test.describe('Image Compare', { tag: ['@widget', '@vue-nodes'] }, () => {
     }
   ) {
     await comfyPage.page.evaluate(
-      ({ nodeId, value }) => {
-        const node = window.app!.graph.getNodeById(nodeId)
+      ({ value }) => {
+        const node = window.app!.graph.getNodeById(1)
         const widget = node?.widgets?.find((w) => w.type === 'imagecompare')
         if (widget) {
           widget.value = value
           widget.callback?.(value)
         }
       },
-      { nodeId: IMAGE_COMPARE_NODE_ID, value }
+      { value }
     )
     await comfyPage.nextFrame()
   }
@@ -453,11 +450,11 @@ test.describe('Image Compare', { tag: ['@widget', '@vue-nodes'] }, () => {
   test('ImageCompare node enforces minimum size', async ({ comfyPage }) => {
     const minWidth = 400
     const minHeight = 350
-    const size = await comfyPage.page.evaluate((nodeId) => {
-      const graphNode = window.app!.graph.getNodeById(nodeId)
+    const size = await comfyPage.page.evaluate(() => {
+      const graphNode = window.app!.graph.getNodeById(1)
       if (!graphNode?.size) return null
       return { width: graphNode.size[0], height: graphNode.size[1] }
-    }, IMAGE_COMPARE_NODE_ID)
+    })
     expect(
       size,
       'ImageCompare node id 1 must exist in loaded workflow graph'
@@ -603,15 +600,15 @@ test.describe('Image Compare', { tag: ['@widget', '@vue-nodes'] }, () => {
   }) => {
     const url = createTestImageDataUrl('Legacy', '#c00')
     await comfyPage.page.evaluate(
-      ({ nodeId, url }) => {
-        const node = window.app!.graph.getNodeById(nodeId)
+      ({ url }) => {
+        const node = window.app!.graph.getNodeById(1)
         const widget = node?.widgets?.find((w) => w.type === 'imagecompare')
         if (widget) {
           widget.value = url
           widget.callback?.(url)
         }
       },
-      { nodeId: IMAGE_COMPARE_NODE_ID, url }
+      { url }
     )
     await comfyPage.nextFrame()
 

browser_tests/tests/imageCrop.spec.ts

@@ -1,7 +1,6 @@
 import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { toNodeId } from '@/types/nodeId'
 
 test.describe('Image Crop', () => {
   test.beforeEach(async ({ comfyPage }) => {
@@ -96,15 +95,15 @@ test.describe('Image Crop', () => {
       const newBounds = { x: 50, y: 100, width: 200, height: 300 }
 
       await comfyPage.page.evaluate(
-        ({ nodeId, bounds }) => {
-          const node = window.app!.graph.getNodeById(nodeId)
+        ({ bounds }) => {
+          const node = window.app!.graph.getNodeById(1)
           const widget = node?.widgets?.find((w) => w.type === 'imagecrop')
           if (widget) {
             widget.value = bounds
             widget.callback?.(bounds)
           }
         },
-        { nodeId: toNodeId(1), bounds: newBounds }
+        { bounds: newBounds }
       )
       await comfyPage.nextFrame()
 

browser_tests/tests/load3d/gizmoControls.spec.ts

@@ -2,16 +2,15 @@ import { expect } from '@playwright/test'
 import type { Page } from '@playwright/test'
 
 import { load3dTest as test } from '@e2e/fixtures/helpers/Load3DFixtures'
-import { toNodeId } from '@/types/nodeId'
 
 const getGizmoConfig = (page: Page) =>
-  page.evaluate((nodeId) => {
-    const n = window.app!.graph.getNodeById(nodeId)
+  page.evaluate(() => {
+    const n = window.app!.graph.getNodeById(1)
     const modelConfig = n?.properties?.['Model Config'] as
       | { gizmo?: { enabled: boolean; mode: string } }
       | undefined
     return modelConfig?.gizmo
-  }, toNodeId(1))
+  })
 
 test.describe('Load3D Gizmo Controls', () => {
   test(

browser_tests/tests/load3d/load3d.spec.ts

@@ -2,7 +2,6 @@ import { expect } from '@playwright/test'
 
 import { assetPath } from '@e2e/fixtures/utils/paths'
 import { load3dTest as test } from '@e2e/fixtures/helpers/Load3DFixtures'
-import { toNodeId } from '@/types/nodeId'
 
 test.describe('Load3D', () => {
   test(
@@ -68,13 +67,13 @@ test.describe('Load3D', () => {
 
       await expect
         .poll(() =>
-          comfyPage.page.evaluate((nodeId) => {
-            const n = window.app!.graph.getNodeById(nodeId)
+          comfyPage.page.evaluate(() => {
+            const n = window.app!.graph.getNodeById(1)
             const config = n?.properties?.['Scene Config'] as
               | Record<string, string>
               | undefined
             return config?.backgroundColor
-          }, toNodeId(1))
+          })
         )
         .toBe('#cc3333')
 

browser_tests/tests/maskEditor.spec.ts

@@ -3,7 +3,6 @@ import { expect, mergeTests } from '@playwright/test'
 import { ExecutionHelper } from '@e2e/fixtures/helpers/ExecutionHelper'
 import { maskEditorTest as test } from '@e2e/fixtures/helpers/MaskEditorHelper'
 import { webSocketFixture } from '@e2e/fixtures/ws'
-import { toNodeId } from '@/types/nodeId'
 
 const wstest = mergeTests(test, webSocketFixture)
 
@@ -332,8 +331,7 @@ wstest(
 
     async function getNodeOutput() {
       return await comfyPage.page.evaluate(
-        (nodeId) => graph!.getNodeById(nodeId)!.images?.[0]?.filename,
-        toNodeId(1)
+        () => graph!.getNodeById('1')!.images?.[0]?.filename
       )
     }
 

browser_tests/tests/nodeGhostPlacement.spec.ts

@@ -2,8 +2,6 @@ import {
   comfyExpect as expect,
   comfyPageFixture as test
 } from '@e2e/fixtures/ComfyPage'
-import { toNodeId } from '@/types/nodeId'
-import type { SerializedNodeId } from '@/types/nodeId'
 
 type ComfyPage = Parameters<Parameters<typeof test>[2]>[0]['comfyPage']
 
@@ -34,13 +32,12 @@ async function addGhostAtCenter(comfyPage: ComfyPage) {
   return { nodeId: nodeRef.id, centerX, centerY }
 }
 
-function getNodeById(comfyPage: ComfyPage, nodeId: SerializedNodeId) {
-  const localNodeId = toNodeId(nodeId)
+function getNodeById(comfyPage: ComfyPage, nodeId: number | string) {
   return comfyPage.page.evaluate((id) => {
     const node = window.app!.graph.getNodeById(id)
     if (!node) return null
     return { ghost: !!node.flags.ghost }
-  }, localNodeId)
+  }, nodeId)
 }
 
 for (const mode of ['litegraph', 'vue'] as const) {

browser_tests/tests/nodeLibraryEssentials.spec.ts

@@ -33,11 +33,49 @@ test.describe('Node Library Essentials Tab', { tag: '@ui' }, () => {
     })
   })
 
+  test('Node library opens via sidebar', async ({ comfyPage }) => {
+    const tabButton = comfyPage.page.locator('.node-library-tab-button')
+    await tabButton.click()
+
+    const sidebarContent = comfyPage.page.locator(
+      '.comfy-vue-side-bar-container'
+    )
+    await expect(sidebarContent).toBeVisible()
+  })
+
+  test('Essentials tab is visible in node library', async ({ comfyPage }) => {
+    const tabButton = comfyPage.page.locator('.node-library-tab-button')
+    await tabButton.click()
+
+    const essentialsTab = comfyPage.page.getByRole('tab', {
+      name: /essentials/i
+    })
+    await expect(essentialsTab).toBeVisible()
+  })
+
+  test('Clicking essentials tab shows essential node cards', async ({
+    comfyPage
+  }) => {
+    const tabButton = comfyPage.page.locator('.node-library-tab-button')
+    await tabButton.click()
+
+    const essentialsTab = comfyPage.page.getByRole('tab', {
+      name: /essentials/i
+    })
+    await essentialsTab.click()
+
+    const essentialCards = comfyPage.page.locator('[data-node-name]')
+    await expect(essentialCards.first()).toBeVisible()
+  })
+
   test('Essential node cards have node names', async ({ comfyPage }) => {
-    await comfyPage.settings.setSetting('Comfy.NodeLibrary.NewDesign', true)
-    const tab = comfyPage.menu.nodeLibraryTabV2
-    await tab.open()
-    await tab.essentialsTab.click()
+    const tabButton = comfyPage.page.locator('.node-library-tab-button')
+    await tabButton.click()
+
+    const essentialsTab = comfyPage.page.getByRole('tab', {
+      name: /essentials/i
+    })
+    await essentialsTab.click()
 
     const firstCard = comfyPage.page.locator('[data-node-name]').first()
     await expect(firstCard).toBeVisible()
@@ -48,18 +86,21 @@ test.describe('Node Library Essentials Tab', { tag: '@ui' }, () => {
   test('Node library can switch between all and essentials tabs', async ({
     comfyPage
   }) => {
-    await comfyPage.settings.setSetting('Comfy.NodeLibrary.NewDesign', true)
-    const tab = comfyPage.menu.nodeLibraryTabV2
-    await tab.open()
-    await tab.allTab.click()
+    const tabButton = comfyPage.page.locator('.node-library-tab-button')
+    await tabButton.click()
 
-    await tab.essentialsTab.click()
-    await expect(tab.essentialsTab).toHaveAttribute('aria-selected', 'true')
+    const essentialsTab = comfyPage.page.getByRole('tab', {
+      name: /essentials/i
+    })
+    const allNodesTab = comfyPage.page.getByRole('tab', { name: /^all$/i })
+
+    await essentialsTab.click()
+    await expect(essentialsTab).toHaveAttribute('aria-selected', 'true')
     const essentialCards = comfyPage.page.locator('[data-node-name]')
     await expect(essentialCards.first()).toBeVisible()
 
-    await tab.allTab.click()
-    await expect(tab.allTab).toHaveAttribute('aria-selected', 'true')
-    await expect(tab.essentialsTab).toHaveAttribute('aria-selected', 'false')
+    await allNodesTab.click()
+    await expect(allNodesTab).toHaveAttribute('aria-selected', 'true')
+    await expect(essentialsTab).toHaveAttribute('aria-selected', 'false')
   })
 })

browser_tests/tests/nodeReplacement.spec.ts

@@ -12,7 +12,6 @@ import {
   setupNodeReplacement
 } from '@e2e/fixtures/helpers/NodeReplacementHelper'
 import { TestIds } from '@e2e/fixtures/selectors'
-import { toNodeId } from '@/types/nodeId'
 
 const renderModes = [
   { name: 'vue nodes', vueNodesEnabled: true },
@@ -246,10 +245,8 @@ test.describe('Node replacement', { tag: ['@node', '@ui'] }, () => {
             .click()
 
           const replacedNodeOutputLinkCount = await comfyPage.page.evaluate(
-            (nodeId) =>
-              window.app!.graph!.getNodeById(nodeId)?.outputs[0]?.links
-                ?.length ?? 0,
-            toNodeId(2)
+            () =>
+              window.app!.graph!.getNodeById(2)?.outputs[0]?.links?.length ?? 0
           )
           expect(
             replacedNodeOutputLinkCount,

browser_tests/tests/propertiesPanel/errorsTabCloudMissingModels.spec.ts

@@ -10,20 +10,8 @@ import {
   countAssetRequestsByTag,
   createCloudAssetsFixture
 } from '@e2e/fixtures/assetApiFixture'
-import {
-  cleanupFakeModel,
-  loadWorkflowAndOpenErrorsTab
-} from '@e2e/fixtures/helpers/ErrorsTabHelper'
-import {
-  NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-  expectNoMissingModelUi,
-  loadPromotedMissingModelAndOpenErrorsTab,
-  selectLegacyPromotedAssetModel,
-  selectSectionAssetPromotedModel,
-  selectVueAssetPromotedModel
-} from '@e2e/fixtures/utils/promotedMissingModel'
+import { loadWorkflowAndOpenErrorsTab } from '@e2e/fixtures/helpers/ErrorsTabHelper'
 import { TestIds } from '@e2e/fixtures/selectors'
-import { toNodeId } from '@/types/nodeId'
 import { PropertiesPanelHelper } from '@e2e/tests/propertiesPanel/PropertiesPanelHelper'
 
 import type { AssetMetadata } from '@/platform/assets/schemas/assetSchema'
@@ -32,8 +20,6 @@ const WORKFLOW = 'missing/nested_subgraph_installed_model'
 const IMPORT_SECTIONS_WORKFLOW = 'missing/cloud_missing_model_import_sections'
 const OUTER_SUBGRAPH_NODE_ID = '205'
 const LOTUS_MODEL_NAME = 'lotus-depth-d-v1-1.safetensors'
-const FAKE_MODEL_NAME = 'fake_model.safetensors'
-const RESOLVED_PROMOTED_MODEL_NAME = 'resolved_model.safetensors'
 const CLOUD_IMPORTABLE_MODEL_NAME = 'cloud_importable_model.safetensors'
 const CLOUD_UNKNOWN_MODEL_NAME = 'cloud_unknown_model.safetensors'
 const CLOUD_IMPORTED_CANONICAL_MODEL_NAME =
@@ -69,25 +55,7 @@ const EXISTING_CLOUD_IMPORTABLE_MODEL: Asset & { hash?: string } = {
   }
 }
 
-const RESOLVED_PROMOTED_MODEL_ASSET: Asset & { hash?: string } = {
-  id: 'test-resolved-promoted-model',
-  name: RESOLVED_PROMOTED_MODEL_NAME,
-  hash: 'blake3:0000000000000000000000000000000000000000000000000000000000000205',
-  size: 1_024,
-  mime_type: 'application/octet-stream',
-  tags: ['models', 'checkpoints'],
-  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: RESOLVED_PROMOTED_MODEL_NAME
-  }
-}
-
 const test = createCloudAssetsFixture([LOTUS_DIFFUSION_MODEL])
-const promotedModelTest = createCloudAssetsFixture([
-  RESOLVED_PROMOTED_MODEL_ASSET
-])
 
 function getRequestedIncludeTags(requestUrl: string): string[] {
   return (
@@ -385,94 +353,13 @@ test.describe(
 
       await expect
         .poll(() =>
-          comfyPage.page.evaluate((nodeId) => {
-            const node = window.app!.graph.getNodeById(nodeId)
+          comfyPage.page.evaluate(() => {
+            const node = window.app!.graph.getNodeById(1)
             return node?.widgets?.find((widget) => widget.name === 'ckpt_name')
               ?.value
-          }, toNodeId(1))
+          })
         )
         .toBe(CLOUD_IMPORTED_CANONICAL_MODEL_NAME)
     })
   }
 )
-
-promotedModelTest.describe(
-  'Errors tab - Cloud promoted subgraph missing models',
-  { tag: '@cloud' },
-  () => {
-    promotedModelTest.beforeEach(async ({ comfyPage }) => {
-      await cleanupFakeModel(comfyPage)
-      await comfyPage.settings.setSetting('Comfy.Assets.UseAssetAPI', true)
-      await comfyPage.settings.setSetting(
-        'Comfy.RightSidePanel.ShowErrorsTab',
-        true
-      )
-    })
-
-    promotedModelTest.afterEach(async ({ comfyPage }) => {
-      await cleanupFakeModel(comfyPage)
-    })
-
-    promotedModelTest(
-      'Changing a Cloud Vue promoted asset widget clears a nested subgraph error',
-      { tag: ['@vue-nodes', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        await loadPromotedMissingModelAndOpenErrorsTab(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME
-        )
-
-        await selectVueAssetPromotedModel(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME,
-          RESOLVED_PROMOTED_MODEL_NAME
-        )
-
-        await expectNoMissingModelUi(comfyPage)
-      }
-    )
-
-    promotedModelTest(
-      'Changing a Cloud Vue promoted asset from the Parameters tab clears a nested subgraph error',
-      { tag: ['@vue-nodes', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        await loadPromotedMissingModelAndOpenErrorsTab(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME
-        )
-
-        await selectSectionAssetPromotedModel(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME,
-          RESOLVED_PROMOTED_MODEL_NAME
-        )
-
-        await expectNoMissingModelUi(comfyPage)
-      }
-    )
-
-    promotedModelTest(
-      'Changing a Cloud legacy promoted asset clears a nested subgraph error',
-      { tag: ['@canvas', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        await loadPromotedMissingModelAndOpenErrorsTab(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME
-        )
-
-        await selectLegacyPromotedAssetModel(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          RESOLVED_PROMOTED_MODEL_ASSET.id
-        )
-
-        await expectNoMissingModelUi(comfyPage)
-      }
-    )
-  }
-)

browser_tests/tests/propertiesPanel/errorsTabMissingMediaRuntime.spec.ts

@@ -8,10 +8,6 @@ import {
 } from '@e2e/fixtures/assetApiFixture'
 import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import {
-  routeObjectInfoFromSetupApi,
-  setComboInputOptions
-} from '@e2e/fixtures/utils/objectInfo'
 import {
   createRouteMockJob,
   jobsRouteFixture
@@ -90,6 +86,50 @@ interface CloudUploadAssetState {
   isUploadedAssetAvailable: boolean
 }
 
+type ObjectInfoResponse = Record<
+  string,
+  { input?: { required?: Record<string, unknown> } }
+>
+
+function setComboInputOptions(
+  objectInfo: ObjectInfoResponse,
+  nodeType: string,
+  inputName: string,
+  values: string[]
+) {
+  const nodeInfo = objectInfo[nodeType]
+  if (!nodeInfo) {
+    throw new Error(`Missing object_info entry for ${nodeType}`)
+  }
+
+  const requiredInputs = nodeInfo.input?.required
+  if (!requiredInputs) {
+    throw new Error(`Missing required inputs for ${nodeType}`)
+  }
+
+  const input = requiredInputs[inputName]
+  if (!Array.isArray(input)) {
+    throw new Error(`Expected ${nodeType}.${inputName} to be a combo input`)
+  }
+
+  const [valuesOrType, options] = input
+  const optionsObject =
+    options && typeof options === 'object' && !Array.isArray(options)
+  if (Array.isArray(valuesOrType)) {
+    input[0] = values
+  } else if (valuesOrType !== 'COMBO') {
+    throw new Error(`Expected ${nodeType}.${inputName} to have combo options`)
+  }
+
+  if (optionsObject) {
+    Object.assign(options, { options: values })
+  } else if (!Array.isArray(valuesOrType)) {
+    throw new Error(
+      `Expected ${nodeType}.${inputName} to have options metadata`
+    )
+  }
+}
+
 async function routeCloudBootstrapApis(page: Page) {
   await page.route('**/api/settings**', async (route) => {
     await route.fulfill({
@@ -121,10 +161,57 @@ async function routeCloudBootstrapApis(page: Page) {
   })
 }
 
+async function routeSetupObjectInfo(
+  page: Page,
+  customize?: (objectInfo: ObjectInfoResponse) => void
+) {
+  const setupApiUrl =
+    process.env.PLAYWRIGHT_SETUP_API_URL ?? 'http://127.0.0.1:8188'
+  const objectInfoUrl = new URL('/object_info', setupApiUrl).toString()
+
+  const objectInfoRouteHandler = async (route: Route) => {
+    try {
+      const response = await fetch(objectInfoUrl, {
+        signal: AbortSignal.timeout(5_000)
+      })
+      if (!response.ok) {
+        await route.fulfill({
+          status: response.status,
+          contentType: response.headers.get('content-type') ?? 'text/plain',
+          body: await response.text()
+        })
+        return
+      }
+
+      const objectInfo = (await response.json()) as ObjectInfoResponse
+      customize?.(objectInfo)
+
+      await route.fulfill({
+        status: response.status,
+        contentType: 'application/json',
+        body: JSON.stringify(objectInfo)
+      })
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error)
+      await route.fulfill({
+        status: 502,
+        contentType: 'application/json',
+        body: JSON.stringify({
+          error: `Failed to fetch setup object_info from ${objectInfoUrl}: ${message}`
+        })
+      })
+    }
+  }
+
+  await page.route('**/object_info', objectInfoRouteHandler)
+  return async () =>
+    await page.unroute('**/object_info', objectInfoRouteHandler)
+}
+
 const cloudOutputTest = createCloudAssetsFixture([cloudOutputAsset]).extend({
   page: async ({ page }, use) => {
     await routeCloudBootstrapApis(page)
-    const unrouteObjectInfo = await routeObjectInfoFromSetupApi(page)
+    const unrouteObjectInfo = await routeSetupObjectInfo(page)
 
     try {
       await use(page)
@@ -138,16 +225,13 @@ const cloudEmptyMediaInputsTest = createCloudAssetsFixture([]).extend({
   page: async ({ page }, use) => {
     await routeCloudBootstrapApis(page)
 
-    const unrouteObjectInfo = await routeObjectInfoFromSetupApi(
-      page,
-      (objectInfo) => {
-        for (const node of emptyMediaLoaderNodes) {
-          setComboInputOptions(objectInfo, node.nodeType, node.widgetName, [
-            node.serverOnlyOption
-          ])
-        }
+    const unrouteObjectInfo = await routeSetupObjectInfo(page, (objectInfo) => {
+      for (const node of emptyMediaLoaderNodes) {
+        setComboInputOptions(objectInfo, node.nodeType, node.widgetName, [
+          node.serverOnlyOption
+        ])
       }
-    )
+    })
 
     try {
       await use(page)
@@ -162,7 +246,7 @@ const cloudUploadRaceTest = comfyPageFixture.extend<{
 }>({
   page: async ({ page }, use) => {
     await routeCloudBootstrapApis(page)
-    const unrouteObjectInfo = await routeObjectInfoFromSetupApi(page)
+    const unrouteObjectInfo = await routeSetupObjectInfo(page)
 
     const state: CloudUploadAssetState = {
       isUploadedAssetAvailable: false

browser_tests/tests/propertiesPanel/errorsTabModeAware.spec.ts

@@ -8,46 +8,12 @@ import {
   openErrorsTab,
   loadWorkflowAndOpenErrorsTab
 } from '@e2e/fixtures/helpers/ErrorsTabHelper'
-import {
-  appendComboInputOptions,
-  routeObjectInfoFromSetupApi
-} from '@e2e/fixtures/utils/objectInfo'
-import {
-  NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-  expectMissingModelReferenceCount,
-  expectNoMissingModelUi,
-  expectResolvedPromotedModelSuppressesStaleInteriorErrors,
-  expectSingleMissingModelReference,
-  getMissingModelLabel,
-  loadPromotedMissingModelAndOpenErrorsTab,
-  loadPromotedMissingModelWithHostValuesAndOpenErrorsTab,
-  selectSectionComboPromotedModel,
-  selectVueComboPromotedModelByTitle,
-  setLegacyPromotedComboModel
-} from '@e2e/fixtures/utils/promotedMissingModel'
 
 const FAKE_MODEL_NAME = 'fake_model.safetensors'
-const RESOLVED_PROMOTED_MODEL_NAME = 'resolved_model.safetensors'
 
-const promotedModelTest = test.extend({
-  page: async ({ page }, use) => {
-    const unrouteObjectInfo = await routeObjectInfoFromSetupApi(
-      page,
-      (objectInfo) =>
-        appendComboInputOptions(
-          objectInfo,
-          'CheckpointLoaderSimple',
-          'ckpt_name',
-          [RESOLVED_PROMOTED_MODEL_NAME]
-        )
-    )
-    try {
-      await use(page)
-    } finally {
-      await unrouteObjectInfo()
-    }
-  }
-})
+function getModelLabel(group: Locator, modelName: string = FAKE_MODEL_NAME) {
+  return group.getByRole('button', { name: modelName, exact: true })
+}
 
 async function expectReferenceBadge(group: Locator, count: number) {
   await expect(
@@ -203,9 +169,7 @@ test.describe('Errors tab - Mode-aware errors', { tag: '@ui' }, () => {
         TestIds.dialogs.missingModelsGroup
       )
       await expect(missingModelGroup).toBeVisible()
-      await expect(
-        getMissingModelLabel(missingModelGroup, FAKE_MODEL_NAME)
-      ).toBeVisible()
+      await expect(getModelLabel(missingModelGroup)).toBeVisible()
 
       const node = await comfyPage.nodeOps.getNodeRefById('1')
       await node.click('title')
@@ -301,9 +265,7 @@ test.describe('Errors tab - Mode-aware errors', { tag: '@ui' }, () => {
 
       const node1 = await comfyPage.nodeOps.getNodeRefById('1')
       await node1.click('title')
-      await expect(
-        getMissingModelLabel(missingModelGroup, FAKE_MODEL_NAME)
-      ).toBeVisible()
+      await expect(getModelLabel(missingModelGroup)).toBeVisible()
       await expect(
         missingModelGroup.getByTestId(TestIds.dialogs.missingModelLocate)
       ).toHaveCount(1)
@@ -419,184 +381,92 @@ test.describe('Errors tab - Mode-aware errors', { tag: '@ui' }, () => {
       await cleanupFakeModel(comfyPage)
     })
 
-    promotedModelTest(
-      'Changing an OSS Vue promoted model clears a nested subgraph error',
-      { tag: ['@vue-nodes', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        let missingModelGroup: Locator
-
-        await test.step('A: shared-definition active host reports the missing model', async () => {
-          missingModelGroup = await loadPromotedMissingModelAndOpenErrorsTab(
-            comfyPage,
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-            FAKE_MODEL_NAME
-          )
-        })
-
-        await test.step('B: bypassing the resolved sibling host keeps the active host error visible', async () => {
-          const siblingHostNodeId =
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.sharedDefinitionSiblingHostNodeId
-          if (siblingHostNodeId === undefined) {
-            throw new Error('Expected a shared-definition sibling host')
-          }
-
-          const siblingHost =
-            await comfyPage.nodeOps.getNodeRefById(siblingHostNodeId)
-          await siblingHost.centerOnNode()
-          await siblingHost.click('title')
-          await comfyPage.keyboard.bypass()
-          await expect.poll(() => siblingHost.isBypassed()).toBeTruthy()
-          await comfyPage.canvas.click({ position: { x: 700, y: 650 } })
-          await openErrorsTab(comfyPage)
-          await expectSingleMissingModelReference(
-            missingModelGroup,
-            FAKE_MODEL_NAME
-          )
-        })
-
-        await test.step('C: changing the active host promoted widget resolves the model', async () => {
-          const activeHost = await comfyPage.nodeOps.getNodeRefById(
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.hostNodeId
-          )
-          await activeHost.centerOnNode()
-          await selectVueComboPromotedModelByTitle(
-            comfyPage,
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.hostNodeTitle,
-            RESOLVED_PROMOTED_MODEL_NAME
-          )
-        })
-
-        await test.step('D: the missing model UI clears', async () => {
-          await expectNoMissingModelUi(comfyPage)
-        })
-
-        await test.step('E: two missing shared-definition hosts report two references', async () => {
-          const siblingHostNodeId =
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.sharedDefinitionSiblingHostNodeId
-          if (siblingHostNodeId === undefined) {
-            throw new Error('Expected a shared-definition sibling host')
-          }
-
-          missingModelGroup =
-            await loadPromotedMissingModelWithHostValuesAndOpenErrorsTab(
-              comfyPage,
-              NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-              {
-                [siblingHostNodeId]: FAKE_MODEL_NAME,
-                [NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.hostNodeId]:
-                  FAKE_MODEL_NAME
-              },
-              FAKE_MODEL_NAME,
-              2
-            )
-        })
-
-        await test.step('F: changing one missing host leaves the other missing reference', async () => {
-          await selectVueComboPromotedModelByTitle(
-            comfyPage,
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.hostNodeTitle,
-            RESOLVED_PROMOTED_MODEL_NAME
-          )
-          await expectMissingModelReferenceCount(
-            missingModelGroup,
-            FAKE_MODEL_NAME,
-            1
-          )
-        })
-
-        await test.step('G: changing the remaining missing host clears the model error', async () => {
-          const siblingHostTitle =
-            NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.sharedDefinitionSiblingHostNodeTitle
-          if (siblingHostTitle === undefined) {
-            throw new Error('Expected a shared-definition sibling host title')
-          }
-
-          await selectVueComboPromotedModelByTitle(
-            comfyPage,
-            siblingHostTitle,
-            RESOLVED_PROMOTED_MODEL_NAME
-          )
-          await expectNoMissingModelUi(comfyPage)
-        })
-      }
-    )
-
-    promotedModelTest(
-      'Changing an OSS Vue promoted model from the Parameters tab clears a nested subgraph error',
-      { tag: ['@vue-nodes', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        await loadPromotedMissingModelAndOpenErrorsTab(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME
-        )
-
-        await selectSectionComboPromotedModel(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          RESOLVED_PROMOTED_MODEL_NAME
-        )
-
-        await expectNoMissingModelUi(comfyPage)
-      }
-    )
-
-    promotedModelTest(
-      'Changing an OSS legacy promoted model clears a nested subgraph error',
+    test(
+      'Resolving a promoted missing model widget through the legacy canvas path clears its error',
       { tag: ['@canvas', '@widget', '@subgraph'] },
       async ({ comfyPage }) => {
+        const resolvedModelName = 'v1-5-pruned-emaonly-fp16.safetensors'
+
         await comfyPage.settings.setSetting('Comfy.VueNodes.Enabled', false)
-        await loadPromotedMissingModelAndOpenErrorsTab(
+        await loadWorkflowAndOpenErrorsTab(
           comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          FAKE_MODEL_NAME
+          'missing/missing_model_promoted_widget'
         )
 
-        await setLegacyPromotedComboModel(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          RESOLVED_PROMOTED_MODEL_NAME
+        const missingModelGroup = comfyPage.page.getByTestId(
+          TestIds.dialogs.missingModelsGroup
         )
+        await expect(getModelLabel(missingModelGroup)).toBeVisible()
 
-        await expectNoMissingModelUi(comfyPage)
+        await comfyPage.page.evaluate((value) => {
+          const hostNode = window.app!.graph!.getNodeById(2)
+          if (!hostNode?.isSubgraphNode()) {
+            throw new Error('Expected subgraph host node')
+          }
+
+          const interiorNode = hostNode.subgraph.getNodeById(1)
+          const widget = interiorNode?.widgets?.find(
+            (entry) => entry.name === 'ckpt_name'
+          )
+          type SettableWidget = typeof widget & {
+            setValue?: (
+              value: string,
+              options: {
+                e: PointerEvent
+                node: unknown
+                canvas: unknown
+              }
+            ) => void
+          }
+          const settableWidget = widget as SettableWidget | undefined
+
+          if (!settableWidget?.setValue) {
+            throw new Error('Expected concrete ckpt_name widget')
+          }
+
+          settableWidget.setValue(value, {
+            e: new PointerEvent('pointerup'),
+            node: hostNode,
+            canvas: window.app!.canvas
+          })
+        }, resolvedModelName)
+
+        await expect(missingModelGroup).toBeHidden()
       }
     )
 
-    promotedModelTest(
+    test(
       'Refreshing a resolved promoted missing model clears the combo invalid state',
       { tag: ['@widget', '@subgraph'] },
       async ({ comfyPage }) => {
         await comfyPage.settings.setSetting('Comfy.VueNodes.Enabled', true)
         await loadWorkflowAndOpenErrorsTab(
           comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.workflowName
+          'missing/missing_model_promoted_widget'
         )
         await comfyPage.vueNodes.waitForNodes()
 
         const missingModelGroup = comfyPage.page.getByTestId(
           TestIds.dialogs.missingModelsGroup
         )
-        await expect(
-          getMissingModelLabel(missingModelGroup, FAKE_MODEL_NAME)
-        ).toBeVisible()
+        await expect(getModelLabel(missingModelGroup)).toBeVisible()
 
         const promotedModelCombo = comfyPage.vueNodes
-          .getNodeByTitle(NESTED_PROMOTED_MISSING_MODEL_WORKFLOW.hostNodeTitle)
+          .getNodeByTitle('Subgraph with Promoted Missing Model')
           .getByRole('combobox', { name: 'ckpt_name', exact: true })
         await expect(promotedModelCombo).toHaveAttribute('aria-invalid', 'true')
 
-        const unrouteObjectInfo = await routeObjectInfoFromSetupApi(
-          comfyPage.page,
-          (objectInfo) =>
-            appendComboInputOptions(
-              objectInfo,
-              'CheckpointLoaderSimple',
-              'ckpt_name',
-              [FAKE_MODEL_NAME, RESOLVED_PROMOTED_MODEL_NAME]
-            )
-        )
-
+        const objectInfoRoute = /\/object_info$/
         try {
+          await comfyPage.page.route(objectInfoRoute, async (route) => {
+            const response = await route.fetch()
+            const objectInfo = await response.json()
+            const ckptName =
+              objectInfo.CheckpointLoaderSimple.input.required.ckpt_name
+            ckptName[0] = [...ckptName[0], 'fake_model.safetensors']
+            await route.fulfill({ response, json: objectInfo })
+          })
+
           await comfyPage.page
             .getByTestId(TestIds.dialogs.missingModelRefresh)
             .click()
@@ -608,31 +478,11 @@ test.describe('Errors tab - Mode-aware errors', { tag: '@ui' }, () => {
             'true'
           )
         } finally {
-          await unrouteObjectInfo()
+          await comfyPage.page.unroute(objectInfoRoute)
         }
       }
     )
 
-    promotedModelTest(
-      'Reloading a resolved nested promoted model ignores stale interior values',
-      { tag: ['@vue-nodes', '@widget', '@subgraph'] },
-      async ({ comfyPage }) => {
-        await expectResolvedPromotedModelSuppressesStaleInteriorErrors(
-          comfyPage,
-          NESTED_PROMOTED_MISSING_MODEL_WORKFLOW,
-          [
-            {
-              subgraphNodeIdToEnter: '4',
-              nodeTitle: 'Inner Subgraph with Promoted Missing Model'
-            },
-            { subgraphNodeIdToEnter: '2', nodeTitle: 'Load Checkpoint' }
-          ],
-          RESOLVED_PROMOTED_MODEL_NAME,
-          FAKE_MODEL_NAME
-        )
-      }
-    )
-
     test('Bypassing a subgraph hides interior errors, un-bypassing restores them', async ({
       comfyPage
     }) => {

browser_tests/tests/rendererToggleStability.spec.ts

@@ -4,16 +4,15 @@ import {
 } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import type { Position } from '@e2e/fixtures/types'
-import type { NodeId } from '@/types/nodeId'
 
-type NodeSnapshot = { id: NodeId } & Position
+type NodeSnapshot = { id: number } & Position
 
 async function getAllNodePositions(
   comfyPage: ComfyPage
 ): Promise<NodeSnapshot[]> {
   return comfyPage.page.evaluate(() =>
     window.app!.graph.nodes.map((n) => ({
-      id: n.id,
+      id: n.id as number,
       x: n.pos[0],
       y: n.pos[1]
     }))
@@ -22,7 +21,7 @@ async function getAllNodePositions(
 
 async function getNodePosition(
   comfyPage: ComfyPage,
-  nodeId: NodeId
+  nodeId: number
 ): Promise<Position | undefined> {
   return comfyPage.page.evaluate((targetNodeId) => {
     const node = window.app!.graph.nodes.find((n) => n.id === targetNodeId)

browser_tests/tests/sidebar/assets.spec.ts

@@ -1095,33 +1095,6 @@ test.describe('Assets sidebar - drag and drop', () => {
     const fileComboWidget = await nodes[0].getWidget(0)
     await expect.poll(() => fileComboWidget.getValue()).toBe('test.png [temp]')
   })
-
-  test('Loading as workflow reuses asset name', async ({ comfyPage }) => {
-    await comfyPage.assets.mockOutputHistory([
-      createMockJob({
-        id: 'job',
-        preview_output: {
-          filename: `testimage.png`,
-          type: 'temp',
-          nodeId: '1',
-          mediaType: 'images'
-        }
-      })
-    ])
-    const path = comfyPage.assetPath('workflowInMedia/workflow.webp')
-    await comfyPage.page.route('**/view?**', (route) => route.fulfill({ path }))
-
-    const { assetsTab } = comfyPage.menu
-    await assetsTab.open()
-    await assetsTab.waitForAssets()
-    await expect(assetsTab.assetCards).toHaveCount(1)
-
-    const targetPosition = { x: 400, y: 100 }
-    await assetsTab.assetCards.dragTo(comfyPage.canvas, { targetPosition })
-
-    const getTabName = () => comfyPage.menu.topbar.getActiveTabName()
-    await expect.poll(getTabName).toContain('testimage')
-  })
 })
 
 test('Insert as node', { tag: '@vue-nodes' }, async ({ comfyPage }) => {

browser_tests/tests/sidebar/nodeLibraryV2.spec.ts

@@ -10,6 +10,20 @@ test.describe('Node library sidebar V2', () => {
     await tab.open()
   })
 
+  test('Can switch between tabs', async ({ comfyPage }) => {
+    const tab = comfyPage.menu.nodeLibraryTabV2
+
+    await expect(tab.allTab).toHaveAttribute('aria-selected', 'true')
+
+    await tab.blueprintsTab.click()
+    await expect(tab.blueprintsTab).toHaveAttribute('aria-selected', 'true')
+    await expect(tab.allTab).toHaveAttribute('aria-selected', 'false')
+
+    await tab.allTab.click()
+    await expect(tab.allTab).toHaveAttribute('aria-selected', 'true')
+    await expect(tab.blueprintsTab).toHaveAttribute('aria-selected', 'false')
+  })
+
   test('All tab displays node tree with folders', async ({ comfyPage }) => {
     const tab = comfyPage.menu.nodeLibraryTabV2
 
@@ -109,9 +123,8 @@ test.describe('Node library sidebar V2', () => {
 
   test('Blueprint previews include description', async ({ comfyPage }) => {
     const tab = comfyPage.menu.nodeLibraryTabV2
-    await tab.allTab.click()
+    await tab.blueprintsTab.click()
 
-    await tab.expandFolder('Comfy Blueprints')
     await tab.getNode('test blueprint').hover()
     await expect(tab.nodePreview, 'Preview displays on hover').toBeVisible()
     await expect(tab.nodePreview).toContainText('Inverts the image')

browser_tests/tests/subgraph/subgraphBreadcrumb.spec.ts

@@ -3,7 +3,6 @@ import { expect, mergeTests } from '@playwright/test'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
 import { subgraphBreadcrumbFixture } from '@e2e/fixtures/helpers/SubgraphBreadcrumbHelper'
-import { toNodeId } from '@/types/nodeId'
 
 const test = mergeTests(comfyPageFixture, subgraphBreadcrumbFixture)
 
@@ -199,7 +198,7 @@ test.describe('Subgraph Breadcrumb', { tag: ['@subgraph'] }, () => {
 
       const rootNodeTitle = await comfyPage.page.evaluate(
         (nodeId) => window.app!.graph!.getNodeById(nodeId)?.title ?? null,
-        toNodeId(OUTER_SUBGRAPH_NODE_ID_IN_NESTED)
+        OUTER_SUBGRAPH_NODE_ID_IN_NESTED
       )
       expect(rootNodeTitle).toBe(newName)
     })

browser_tests/tests/subgraph/subgraphDraftPositions.spec.ts

@@ -1,6 +1,6 @@
 import { expect } from '@playwright/test'
 
-import type { NodeId } from '@/types/nodeId'
+import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema'
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 
 interface SubgraphNodePosition {

browser_tests/tests/subgraph/subgraphLifecycle.spec.ts

@@ -1,11 +1,7 @@
-import type { ConsoleMessage } from '@playwright/test'
 import { expect } from '@playwright/test'
 
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { TestIds } from '@e2e/fixtures/selectors'
 import { getPseudoPreviewWidgets } from '@e2e/fixtures/utils/promotedWidgets'
-import { toNodeId } from '@/types/nodeId'
 
 const domPreviewSelector = '.image-preview'
 
@@ -58,12 +54,12 @@ test.describe('Subgraph Lifecycle', { tag: ['@subgraph'] }, () => {
         })
         .toBeGreaterThan(0)
 
-      await comfyPage.page.evaluate((nodeId) => {
+      await comfyPage.page.evaluate(() => {
         const graph = window.app!.graph!
-        const subgraphNode = graph.getNodeById(nodeId)
+        const subgraphNode = graph.getNodeById('5')
         if (!subgraphNode || !subgraphNode.isSubgraphNode()) return
         graph.unpackSubgraph(subgraphNode)
-      }, toNodeId(5))
+      })
       await comfyPage.nextFrame()
 
       await expect
@@ -99,225 +95,4 @@ test.describe('Subgraph Lifecycle', { tag: ['@subgraph'] }, () => {
       await expect(comfyPage.page.locator(domPreviewSelector)).toHaveCount(0)
     })
   })
-
-  test.describe('Detach Race Repro', { tag: ['@vue-nodes'] }, () => {
-    const SUBGRAPH_NODE_TITLE = 'New Subgraph'
-
-    // Queues legacy onNodeRemoved/onSelectionChange so unpack completes first,
-    // widening the race window so a guard regression deterministically surfaces.
-    async function deferLegacyHandlers(comfyPage: ComfyPage) {
-      return await comfyPage.page.evaluateHandle(() => {
-        const graph = window.app!.graph!
-        const canvas = window.app!.canvas!
-        const queue: Array<() => void> = []
-        const originalNodeRemoved = graph.onNodeRemoved
-        const originalSelectionChange = canvas.onSelectionChange
-        graph.onNodeRemoved = function (node) {
-          queue.push(() => originalNodeRemoved?.call(this, node))
-        }
-        canvas.onSelectionChange = function (selected) {
-          queue.push(() => originalSelectionChange?.call(this, selected))
-        }
-        return {
-          drain: () => {
-            for (const fn of queue.splice(0)) fn()
-          },
-          restore: () => {
-            graph.onNodeRemoved = originalNodeRemoved
-            canvas.onSelectionChange = originalSelectionChange
-          }
-        }
-      })
-    }
-
-    type DeferredHandlers = Awaited<ReturnType<typeof deferLegacyHandlers>>
-
-    // Defers only the legacy selection-change callback, so the detached host
-    // node lingers in the reactive selection while onNodeRemoved still runs
-    // normally and clears it from the canvas. This isolates the panel render
-    // path: a panel mounted during this window reads the stale selection.
-    async function deferSelectionChange(
-      comfyPage: ComfyPage
-    ): Promise<DeferredHandlers> {
-      return await comfyPage.page.evaluateHandle(() => {
-        const canvas = window.app!.canvas!
-        const queue: Array<() => void> = []
-        const original = canvas.onSelectionChange
-        canvas.onSelectionChange = function (selected) {
-          queue.push(() => original?.call(this, selected))
-        }
-        return {
-          drain: () => {
-            for (const fn of queue.splice(0)) fn()
-          },
-          restore: () => {
-            canvas.onSelectionChange = original
-          }
-        }
-      })
-    }
-
-    function isNullGraphErrorText(text: string): boolean {
-      return text.includes('NullGraphError') || text.endsWith('has no graph')
-    }
-
-    // Vue's default errorHandler routes render throws to console.error,
-    // not pageerror - listen to both.
-    function captureNullGraphErrors(comfyPage: ComfyPage) {
-      const captured: string[] = []
-      const onPageError = (err: Error) => {
-        if (
-          err.name === 'NullGraphError' ||
-          isNullGraphErrorText(err.message ?? '')
-        ) {
-          captured.push(`pageerror ${err.name}: ${err.message}`)
-        }
-      }
-      const onConsoleMessage = (msg: ConsoleMessage) => {
-        if (msg.type() !== 'error') return
-        const text = msg.text()
-        if (isNullGraphErrorText(text)) {
-          captured.push(`console.error: ${text}`)
-        }
-      }
-      comfyPage.page.on('pageerror', onPageError)
-      comfyPage.page.on('console', onConsoleMessage)
-      return {
-        getErrors: () => [...captured],
-        stop: () => {
-          comfyPage.page.off('pageerror', onPageError)
-          comfyPage.page.off('console', onConsoleMessage)
-        }
-      }
-    }
-
-    async function unpackViaContextMenu(comfyPage: ComfyPage, title: string) {
-      const fixture = await comfyPage.vueNodes.getFixtureByTitle(title)
-      await comfyPage.contextMenu.openForVueNode(fixture.header)
-      await comfyPage.contextMenu.clickMenuItemExact('Unpack Subgraph')
-    }
-
-    async function reopenRightSidePanel(comfyPage: ComfyPage) {
-      const { propertiesPanel } = comfyPage.menu
-      await propertiesPanel.toggleButton.click()
-      await expect(propertiesPanel.root).toBeHidden()
-      await propertiesPanel.toggleButton.click()
-      await comfyPage.nextFrame()
-    }
-
-    // Unpacks the subgraph behind deferred teardown, runs an optional
-    // interaction while the node is detached but not yet cleaned up, then
-    // drains the deferred handlers and reports any NullGraphErrors seen.
-    async function unpackAndCaptureNullGraphErrors(
-      comfyPage: ComfyPage,
-      options: {
-        defer: (comfyPage: ComfyPage) => Promise<DeferredHandlers>
-        duringWindow?: (comfyPage: ComfyPage) => Promise<void>
-      }
-    ): Promise<string[]> {
-      const subgraphNode =
-        comfyPage.vueNodes.getNodeByTitle(SUBGRAPH_NODE_TITLE)
-      const errors = captureNullGraphErrors(comfyPage)
-      const deferred = await options.defer(comfyPage)
-      try {
-        await unpackViaContextMenu(comfyPage, SUBGRAPH_NODE_TITLE)
-        await expect(subgraphNode).toHaveCount(0)
-        await options.duringWindow?.(comfyPage)
-        await deferred.evaluate((handlers) => handlers.drain())
-        // Let drained-handler reactive flushes settle before stop().
-        await comfyPage.nextFrame()
-        return errors.getErrors()
-      } finally {
-        await deferred.evaluate((handlers) => handlers.restore())
-        await deferred.dispose()
-        errors.stop()
-      }
-    }
-
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.RightSidePanel.IsOpen', true)
-      await comfyPage.workflow.loadWorkflow(
-        'subgraphs/subgraph-with-promoted-text-widget'
-      )
-      const subgraphNode =
-        comfyPage.vueNodes.getNodeByTitle(SUBGRAPH_NODE_TITLE)
-      await expect(subgraphNode).toBeVisible()
-
-      const fixture =
-        await comfyPage.vueNodes.getFixtureByTitle(SUBGRAPH_NODE_TITLE)
-      await fixture.header.click()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.propertiesPanel.root)
-      ).toBeVisible()
-      await comfyPage.nextFrame()
-    })
-
-    test('unpack does not surface NullGraphError on the LGraphNode render path', async ({
-      comfyPage
-    }) => {
-      const nullGraphErrors = await unpackAndCaptureNullGraphErrors(comfyPage, {
-        defer: deferLegacyHandlers
-      })
-      expect(
-        nullGraphErrors,
-        'LGraphNode render path: detach race must not surface NullGraphError'
-      ).toEqual([])
-    })
-
-    test('unpack does not surface NullGraphError from the TabSubgraphInputs panel', async ({
-      comfyPage
-    }) => {
-      const nullGraphErrors = await unpackAndCaptureNullGraphErrors(comfyPage, {
-        defer: deferLegacyHandlers
-      })
-      expect(
-        nullGraphErrors,
-        'TabSubgraphInputs panel: detach race must not surface NullGraphError'
-      ).toEqual([])
-    })
-
-    test('unpack with subgraph editor open does not surface NullGraphError from the SubgraphEditor panel', async ({
-      comfyPage
-    }) => {
-      await comfyPage.page.getByTestId(TestIds.subgraphEditor.toggle).click()
-      await comfyPage.nextFrame()
-
-      const nullGraphErrors = await unpackAndCaptureNullGraphErrors(comfyPage, {
-        defer: deferLegacyHandlers
-      })
-      expect(
-        nullGraphErrors,
-        'SubgraphEditor panel: detach race must not surface NullGraphError'
-      ).toEqual([])
-    })
-
-    test('reopening the right side panel after unpack does not surface NullGraphError', async ({
-      comfyPage
-    }) => {
-      const nullGraphErrors = await unpackAndCaptureNullGraphErrors(comfyPage, {
-        defer: deferSelectionChange,
-        duringWindow: reopenRightSidePanel
-      })
-      expect(
-        nullGraphErrors,
-        'TabSubgraphInputs remount: stale selection must not surface NullGraphError'
-      ).toEqual([])
-    })
-
-    test('reopening the right side panel with the subgraph editor open does not surface NullGraphError', async ({
-      comfyPage
-    }) => {
-      await comfyPage.page.getByTestId(TestIds.subgraphEditor.toggle).click()
-      await comfyPage.nextFrame()
-
-      const nullGraphErrors = await unpackAndCaptureNullGraphErrors(comfyPage, {
-        defer: deferSelectionChange,
-        duringWindow: reopenRightSidePanel
-      })
-      expect(
-        nullGraphErrors,
-        'SubgraphEditor remount: stale selection must not surface NullGraphError'
-      ).toEqual([])
-    })
-  })
 })

browser_tests/tests/subgraph/subgraphNavigation.spec.ts

@@ -2,7 +2,6 @@ import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 import { TestIds } from '@e2e/fixtures/selectors'
-import { toNodeId } from '@/types/nodeId'
 
 const UPDATED_SUBGRAPH_TITLE = 'Updated Subgraph Title'
 
@@ -261,18 +260,17 @@ test.describe('Subgraph Navigation', { tag: ['@slow', '@subgraph'] }, () => {
       await comfyPage.workflow.loadWorkflow('subgraphs/basic-subgraph')
 
       const subgraphNodeId = await comfyPage.subgraph.findSubgraphNodeId()
-      const localSubgraphNodeId = toNodeId(subgraphNodeId)
 
       await comfyPage.page.evaluate((nodeId) => {
         const node = window.app!.canvas.graph!.getNodeById(nodeId)!
         node.progress = 0.5
-      }, localSubgraphNodeId)
+      }, subgraphNodeId)
 
       await expect
         .poll(() =>
           comfyPage.page.evaluate(
             (nodeId) => window.app!.canvas.graph!.getNodeById(nodeId)!.progress,
-            localSubgraphNodeId
+            subgraphNodeId
           )
         )
         .toBe(0.5)
@@ -289,7 +287,7 @@ test.describe('Subgraph Navigation', { tag: ['@slow', '@subgraph'] }, () => {
         .poll(() =>
           comfyPage.page.evaluate((nodeId) => {
             return window.app!.canvas.graph!.getNodeById(nodeId)!.progress
-          }, localSubgraphNodeId)
+          }, subgraphNodeId)
         )
         .toBeUndefined()
     })
@@ -300,12 +298,11 @@ test.describe('Subgraph Navigation', { tag: ['@slow', '@subgraph'] }, () => {
       await comfyPage.workflow.loadWorkflow('subgraphs/basic-subgraph')
 
       const subgraphNodeId = await comfyPage.subgraph.findSubgraphNodeId()
-      const localSubgraphNodeId = toNodeId(subgraphNodeId)
 
       await comfyPage.page.evaluate((nodeId) => {
         const node = window.app!.canvas.graph!.getNodeById(nodeId)!
         node.progress = 0.7
-      }, localSubgraphNodeId)
+      }, subgraphNodeId)
 
       const subgraphNode =
         await comfyPage.nodeOps.getNodeRefById(subgraphNodeId)

browser_tests/tests/subgraph/subgraphPromotion.spec.ts

@@ -471,10 +471,11 @@ test.describe(
           'subgraphs/subgraph-with-promoted-text-widget'
         )
 
+        let initialWidgetCount = 0
         await expect
           .poll(() => getPromotedWidgetCount(comfyPage, '11'))
           .toBeGreaterThan(0)
-        const initialWidgetCount = await getPromotedWidgetCount(comfyPage, '11')
+        initialWidgetCount = await getPromotedWidgetCount(comfyPage, '11')
 
         const subgraphNode = await comfyPage.nodeOps.getNodeRefById('11')
         await subgraphNode.navigateIntoSubgraph()

browser_tests/tests/subgraph/subgraphSerialization.spec.ts

@@ -1,8 +1,5 @@
 import { expect } from '@playwright/test'
 
-import { toNodeId } from '@/types/nodeId'
-import type { NodeId } from '@/types/nodeId'
-
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import { comfyExpect, comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 import { SubgraphHelper } from '@e2e/fixtures/helpers/SubgraphHelper'
@@ -42,10 +39,9 @@ async function getPrimitiveFanoutSnapshot(
   comfyPage: ComfyPage,
   hostNodeId: string
 ): Promise<PrimitiveFanoutSnapshot> {
-  const localHostNodeId = toNodeId(hostNodeId)
   return comfyPage.page.evaluate((id) => {
     const graph = window.app!.canvas.graph!
-    const hostNode = graph.getNodeById(id)
+    const hostNode = graph.getNodeById(Number(id))
     if (!hostNode?.isSubgraphNode?.()) {
       throw new Error(`Host node ${id} is not a SubgraphNode`)
     }
@@ -84,7 +80,7 @@ async function getPrimitiveFanoutSnapshot(
       primitiveOriginLinkCount,
       serializedProperties: serializedNode?.properties ?? {}
     }
-  }, localHostNodeId)
+  }, hostNodeId)
 }
 
 async function getSerializedSubgraphNodeProperties(
@@ -107,20 +103,19 @@ async function expectPromotedWidgetsToResolveToInteriorNodes(
 ) {
   expect(widgets.length).toBeGreaterThan(0)
 
-  const hostNodeId = toNodeId(hostSubgraphNodeId)
-  const interiorNodeIds = widgets.map(([id]) => toNodeId(id))
+  const interiorNodeIds = widgets.map(([id]) => id)
   const results = await comfyPage.page.evaluate(
     ([hostId, ids]) => {
       const graph = window.app!.graph!
-      const hostNode = graph.getNodeById(hostId)
+      const hostNode = graph.getNodeById(Number(hostId))
       if (!hostNode?.isSubgraphNode()) return ids.map(() => false)
 
       return ids.map((id) => {
-        const interiorNode = hostNode.subgraph.getNodeById(id)
+        const interiorNode = hostNode.subgraph.getNodeById(Number(id))
         return interiorNode !== null && interiorNode !== undefined
       })
     },
-    [hostNodeId, interiorNodeIds] as const
+    [hostSubgraphNodeId, interiorNodeIds] as const
   )
 
   expect(results).toEqual(widgets.map(() => true))
@@ -575,7 +570,8 @@ test.describe('Subgraph Serialization', { tag: ['@subgraph'] }, () => {
         const allGraphs = [graph, ...graph.subgraphs.values()]
         const allIds = allGraphs
           .flatMap((g) => g._nodes)
-          .map((n) => String(n.id))
+          .map((n) => n.id)
+          .filter((id): id is number => typeof id === 'number')
 
         return { allIds, uniqueCount: new Set(allIds).size }
       })
@@ -591,7 +587,10 @@ test.describe('Subgraph Serialization', { tag: ['@subgraph'] }, () => {
 
       const rootIds = await comfyPage.page.evaluate(() => {
         const graph = window.app!.canvas.graph!
-        return graph._nodes.map((n) => Number(n.id)).sort((a, b) => a - b)
+        return graph._nodes
+          .map((n) => n.id)
+          .filter((id): id is number => typeof id === 'number')
+          .sort((a, b) => a - b)
       })
 
       expect(rootIds).toEqual([1, 2, 5])
@@ -634,18 +633,18 @@ test.describe('Subgraph Serialization', { tag: ['@subgraph'] }, () => {
           )
         ]
 
-        const SENTINEL_IDS = new Set(['-1', '-10', '-20'])
-        const isSentinelNodeId = (id: number | string) =>
-          SENTINEL_IDS.has(String(id))
+        const SENTINEL_IDS = new Set([-1, -10, -20])
+        const isSentinelNodeId = (id: number | string): id is number =>
+          typeof id === 'number' && SENTINEL_IDS.has(id)
 
         const checkEndpoint = (
           label: string,
           kind: 'origin_id' | 'target_id',
-          id: NodeId,
+          id: number | string,
           g: typeof graph
         ): string | null => {
           if (isSentinelNodeId(id)) return null
-          if (!g.getNodeById(id)) {
+          if (typeof id !== 'number' || !g._nodes_by_id[id]) {
             return `${label}: ${kind} ${id} invalid or not found`
           }
           return null

browser_tests/tests/subgraph/subgraphSlots.spec.ts

@@ -6,7 +6,6 @@ import type { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/w
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
 import { SubgraphHelper } from '@e2e/fixtures/helpers/SubgraphHelper'
-import { toNodeId } from '@/types/nodeId'
 import {
   expectSlotsWithinBounds,
   measureNodeSlotOffsets
@@ -461,17 +460,16 @@ test.describe('Subgraph Slots', { tag: ['@slow', '@subgraph'] }, () => {
       const subgraphNodeAfter = comfyPage.vueNodes.getNodeLocator('19')
       await expect(subgraphNodeAfter).toBeVisible()
 
-      const subgraphNodeId = toNodeId(19)
       await expect
         .poll(() =>
-          comfyPage.page.evaluate((nodeId) => {
-            const node = window.app!.canvas.graph!.getNodeById(nodeId)
+          comfyPage.page.evaluate(() => {
+            const node = window.app!.canvas.graph!.getNodeById('19')
             if (!node) return null
             const widget = node.widgets?.find((entry: { name: string }) =>
               entry.name.includes('seed')
             )
             return widget?.label || widget?.name || null
-          }, subgraphNodeId)
+          })
         )
         .toBe(RENAMED_LABEL)
 

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

@@ -1,6 +1,6 @@
 import type { Locator, Page } from '@playwright/test'
 
-import type { NodeId } from '@/types/nodeId'
+import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema'
 import { getSlotKey } from '@/renderer/core/layout/slots/slotIdentifier'
 import {
   comfyExpect as expect,
@@ -67,7 +67,7 @@ function slotLocator(
   slotIndex: number,
   isInput: boolean
 ) {
-  const key = getSlotKey(nodeId, slotIndex, isInput)
+  const key = getSlotKey(String(nodeId), slotIndex, isInput)
   return page.locator(`[data-slot-key="${key}"]`)
 }
 

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

@@ -6,7 +6,6 @@ import {
 } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import type { Position } from '@e2e/fixtures/types'
-import { VueNodeFixture } from '@e2e/fixtures/utils/vueNodeFixtures'
 
 test.describe('Vue Node Moving', { tag: '@vue-nodes' }, () => {
   const getHeaderPos = async (
@@ -336,79 +335,6 @@ test.describe('Vue Node Moving', { tag: '@vue-nodes' }, () => {
     await comfyPage.canvasOps.moveMouseToEmptyArea()
   })
 
-  test('pointerCancel stops autopan', async ({ comfyPage }) => {
-    const ksampler = await comfyPage.vueNodes.getFixtureByTitle('KSampler')
-    await ksampler.header.click({ trial: true })
-    await comfyPage.page.mouse.down()
-
-    const getOffset = () => comfyPage.canvasOps.getOffset()
-    const initialOffset = await getOffset()
-    await comfyPage.page.mouse.move(10, 10, { steps: 20 })
-    await expect.poll(getOffset, 'drag with autopan').not.toEqual(initialOffset)
-
-    await test.step('move outside pan range and cancel drag', async () => {
-      await comfyPage.page.mouse.move(400, 400, { steps: 20 })
-      await ksampler.header.evaluate((node) =>
-        node.dispatchEvent(new PointerEvent('pointercancel', { bubbles: true }))
-      )
-    })
-
-    const secondaryOffset = await getOffset()
-
-    await comfyPage.page.mouse.move(10, 10, { steps: 20 })
-    await comfyPage.nextFrame()
-    expect(await getOffset(), 'drag canceled').toEqual(secondaryOffset)
-  })
-
-  test('dragging a node moves all selected items', async ({
-    comfyPage,
-    comfyMouse
-  }) => {
-    const samplerLocator = comfyPage.vueNodes.getNodeByTitle('KSampler')
-    const ksampler = new VueNodeFixture(samplerLocator)
-    const loaderLocator = comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')
-    const loader = new VueNodeFixture(loaderLocator)
-
-    await test.step('create graph with group and reroute', async () => {
-      await comfyPage.nodeOps.clearGraph()
-      await comfyPage.searchBoxV2.addNode('Load Checkpoint')
-      const samplerOptions = { position: { x: 800, y: 200 } }
-      await comfyPage.searchBoxV2.addNode('KSampler', samplerOptions)
-      await ksampler.getSlot('model').dragTo(loader.getSlot('MODEL'))
-
-      await test.step('add reroute', async () => {
-        const b1 = await ksampler.getSlot('model').boundingBox()
-        const b2 = await loader.getSlot('MODEL').boundingBox()
-        if (!b1 || !b2) throw new Error('Failed to get bounds')
-
-        const x = (b1.x + b2.x + (b1.width + b2.width) / 2) / 2
-        const y = (b1.y + b2.y + (b1.height + b2.height) / 2) / 2
-        await comfyPage.page.keyboard.down('Alt')
-        await comfyPage.page.mouse.click(x, y)
-        await comfyPage.page.keyboard.up('Alt')
-
-        const rerouteCount = () =>
-          comfyPage.page.evaluate(() => graph!.reroutes.size)
-        await expect.poll(rerouteCount).toBe(1)
-      })
-
-      await comfyPage.keyboard.selectAll()
-      await comfyPage.page.keyboard.press('Control+G')
-      await comfyPage.keyboard.selectAll()
-    })
-
-    const getReroutePos = () =>
-      comfyPage.page.evaluate(() => [...graph!.reroutes.values()][0])
-    const getGroupPos = () =>
-      comfyPage.page.evaluate(() => graph!.groups[0].pos)
-    const initialReroutePos = await getReroutePos()
-    const initialGroupPos = await getGroupPos()
-    await comfyMouse.dragElementBy(ksampler.title, { x: 100 })
-
-    await expect.poll(getReroutePos).not.toEqual(initialReroutePos)
-    await expect.poll(getGroupPos).not.toEqual(initialGroupPos)
-  })
-
   test(
     '@mobile should allow moving nodes by dragging on touch devices',
     { tag: '@screenshot' },

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

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

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

@@ -5,7 +5,6 @@ import {
   comfyPageFixture
 } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { toNodeId } from '@/types/nodeId'
 import {
   cleanupFakeModel,
   dismissErrorOverlay,
@@ -60,13 +59,12 @@ async function selectLoadImageNodeForPaste(
   comfyPage: ComfyPage,
   loadImageId: string
 ): Promise<void> {
-  const localLoadImageId = toNodeId(loadImageId)
   await comfyPage.page.evaluate((nodeId) => {
-    const node = window.app!.graph.getNodeById(nodeId)
+    const node = window.app!.graph.getNodeById(Number(nodeId))
     if (!node) throw new Error(`Load Image node ${nodeId} not found`)
     window.app!.canvas.selectNode(node)
     window.app!.canvas.current_node = node
-  }, localLoadImageId)
+  }, loadImageId)
 }
 
 async function setupLoadImageErrorScenario(comfyPage: ComfyPage) {
@@ -149,7 +147,7 @@ test.describe('Vue Node Error', { tag: '@vue-nodes' }, () => {
             }
             return index
           },
-          { nodeId: toNodeId(ksamplerId), inputName: KSAMPLER_MODEL_INPUT_NAME }
+          { nodeId: ksamplerId, inputName: KSAMPLER_MODEL_INPUT_NAME }
         )
         const modelInputSlotRow = comfyPage.vueNodes.getInputSlotRow(
           ksamplerId,