Merge branch 'main' into feat/website-prefetch

Amp-Thread-ID: https://ampcode.com/threads/T-019dbdb1-a449-770b-acb2-58c0cc47ad32
Co-authored-by: Amp <amp@ampcode.com>

.claude/skills/perf-fix-with-proof/SKILL.md

@@ -171,7 +171,7 @@ test('canvas text rendering with many nodes', async ({ comfyPage }) => {
 | ----------------- | ----------------------------------------------------- |
 | Perf test file    | `browser_tests/tests/performance.spec.ts`             |
 | PerformanceHelper | `browser_tests/fixtures/helpers/PerformanceHelper.ts` |
-| Perf reporter     | `browser_tests/fixtures/utils/perfReporter.ts`        |
+| Perf reporter     | `browser_tests/helpers/perfReporter.ts`               |
 | CI workflow       | `.github/workflows/ci-perf-report.yaml`               |
 | Report generator  | `scripts/perf-report.ts`                              |
 | Stats utilities   | `scripts/perf-stats.ts`                               |

.claude/skills/reviewing-unit-tests/SKILL.md

@@ -1,156 +0,0 @@
----
-name: reviewing-unit-tests
-description: Use when reviewing Vitest unit-test diffs in ComfyUI_frontend, especially new mocks, store tests, component tests, or bugfix regression tests.
----
-
-# Reviewing Unit Tests for ComfyUI_frontend
-
-## Overview
-
-Review for behavior and current repo rules, not motion. Compare to authoritative rules, not prior diffs or legacy snippets.
-
-## Review Workflow
-
-1. Identify the test type: component, store, composable, util, or bugfix regression.
-2. Name the behavior the test proves. If you cannot say it in one sentence, request changes.
-3. Open the authoritative doc section before judging structure.
-4. Scan the red flags below.
-5. State the verdict first. Name the failure mode. Cite the doc or rule.
-
-## Source of Truth / Precedence
-
-When docs and examples conflict, use this order:
-
-1. Explicit repo rules, lint rules, and note blocks.
-2. [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md)
-3. Rule sections in [`docs/testing/unit-testing.md`](../../../docs/testing/unit-testing.md), [`docs/testing/store-testing.md`](../../../docs/testing/store-testing.md), and [`docs/testing/component-testing.md`](../../../docs/testing/component-testing.md)
-4. Example snippets
-5. Prior diffs
-
-Apply these repo-specific clarifications:
-
-- [`docs/testing/component-testing.md`](../../../docs/testing/component-testing.md) starts with the authoritative rule: new component tests use `@testing-library/vue` with `@testing-library/user-event`. The `@vue/test-utils` snippets below it are legacy examples.
-- [`docs/testing/store-testing.md`](../../../docs/testing/store-testing.md) still contains `as any` examples. Treat them as legacy snippets, not approval for new or edited test code.
-- If docs conflict, prefer the stricter newer rule and call out the doc ambiguity. Do not approve through it.
-- Motion != fix.
-
-## 30-Second Red Flags
-
-| If you see...                                                                             | Failure mode                    | Default action                                                |
-| ----------------------------------------------------------------------------------------- | ------------------------------- | ------------------------------------------------------------- |
-| New `@vue/test-utils` import in a new component test                                      | legacy test API                 | Request changes                                               |
-| `vi.mock('vue-i18n', ...)`                                                                | mocked i18n                     | Request changes                                               |
-| `as any`, `@ts-expect-error`, `as Mock`, `as ReturnType<typeof vi.fn>`, `as unknown as X` | unnecessary cast or type escape | Request changes unless the author proves no safer type exists |
-| `getXMock()`, renamed wrapper, or helper that only returns a mocked value                 | alias-by-renaming               | Request changes                                               |
-| `beforeEach` recreates the return object for a module-mocked composable or service        | shared mock setup drift         | Request changes                                               |
-| Assertions only check defaults, mock plumbing, or CSS hooks                               | non-behavioral test             | Request changes                                               |
-| Bugfix test has no proof it fails on pre-fix code                                         | unproven regression             | Request changes                                               |
-
-## Rationalization Table
-
-| Excuse                              | Reality                                                                                                |
-| ----------------------------------- | ------------------------------------------------------------------------------------------------------ |
-| "I restructured the mocks"          | If the indirection stayed, nothing improved. Flag `alias-by-renaming`.                                 |
-| "The docs do it"                    | Rule, note, and lint beat legacy snippet. Compare to the current rule, not the nearest example.        |
-| "TypeScript required the cast"      | `vi.mocked()` usually narrows mock methods. Assertion-only references need no cast.                    |
-| "Putting it in `beforeEach` is DRY" | Recreating module mock state in hooks hides singleton behavior and drifts from the documented pattern. |
-| "It is only a nit"                  | Explicit repo-rule violations are never nits.                                                          |
-| "No behavior changed, just cleanup" | Motion != fix. Ask what behavior got stronger.                                                         |
-| "Mental revert is enough"           | For bugfix tests, establish red on pre-fix code or ask the author to show it.                          |
-
-## Mocking Rules
-
-- Fail helpers that do not remove repeated setup, encode domain meaning, or simplify assertions. Barely earning the abstraction is not enough.
-- For composables with reactive or singleton state, define stable mock state inside the `vi.mock()` factory. Access it per test via the composable itself. See [`docs/testing/unit-testing.md`](../../../docs/testing/unit-testing.md) "Mocking Composables with Reactive State".
-- This does not ban local test data builders or per-test `vi.spyOn(...)`.
-- Mock seams, not the project-owned module you are trying to exercise. For store tests, prefer real Pinia plus `createTestingPinia({ stubActions: false })` per [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md) and [`docs/testing/store-testing.md`](../../../docs/testing/store-testing.md).
-
-### Alias-by-Renaming
-
-```ts
-// Before
-const mockAdd = vi.hoisted(() => vi.fn())
-
-// After: same indirection, new name
-function getToastAddMock() {
-  return useToast().add
-}
-```
-
-If the wrapper only renames or relays a mocked value, fail it. Inline the lookup at the call site or fetch the singleton mock via the documented pattern.
-
-### `vi.mocked()` Scope
-
-| Use case                                                        | `vi.mocked()` required? |
-| --------------------------------------------------------------- | ----------------------- |
-| `.mockReturnValue`, `.mockResolvedValue`, `.mockImplementation` | Yes                     |
-| `.mock.calls`, `.mock.results`                                  | Yes                     |
-| `expect(fn).toHaveBeenCalled()`                                 | No                      |
-| `expect(fn).toHaveBeenCalledWith(...)`                          | No                      |
-
-- Flag casts whenever `vi.mocked()` would narrow correctly.
-- Do not add `vi.mocked()` around assertion-only references just for style.
-
-### Reset Hygiene
-
-- Flag per-mock `mockClear()` or `mockReset()` when `vi.clearAllMocks()` or `vi.resetAllMocks()` already runs in the relevant hook chain.
-- Review for redundancy or broken state management. Do not bikeshed `clearAllMocks` vs `resetAllMocks` unless behavior depends on it.
-
-### Third-Party Seams
-
-- Distinguish trivial hooks from behavior-rich APIs.
-- Mocking single-method third-party hooks like `primevue/usetoast` is usually acceptable.
-- That exception does not justify mocking behavior-rich third-party modules.
-
-### `vue-i18n`
-
-- Never mock `vue-i18n` in component tests.
-- Use real `createI18n` per [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md) and the shared [`testI18n`](../../../src/components/searchbox/v2/__test__/testUtils.ts) setup.
-
-## Test-Body Rules
-
-| Smell                                                             | Review bar                                                                                                                                    |
-| ----------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------- |
-| Change-detector test                                              | Reject. Default values alone prove nothing.                                                                                                   |
-| Mock-only assertion                                               | Accept collaborator-call assertions only when the call is the meaningful external effect and the test also exercises the triggering behavior. |
-| Non-behavioral assertion                                          | Reject tests that only check classes, utility hooks, or styling internals.                                                                    |
-| New component test using `@vue/test-utils`                        | Request changes. Use `@testing-library/vue` plus `@testing-library/user-event`.                                                               |
-| `any`, `as any`, or `@ts-expect-error` in new or edited test code | Request changes unless the author proves no safer type exists. Legacy doc snippets do not authorize it.                                       |
-
-## Bugfix Regression Proof
-
-For `fix:` PRs or bugfix diffs:
-
-1. Identify the production change that fixes the bug.
-2. Verify the new test fails on pre-fix code, or ask the author to show it.
-3. If the test passes on broken code, request changes.
-
-A regression test that never proves red does not pin the bug.
-
-## Review Output Rules
-
-- State verdict before procedural questions.
-- Do not lead with approval language like `LGTM, just one nit` or `approve and move on?`.
-- Name the failure mode directly: `alias-by-renaming`, `unnecessary cast`, `mocked i18n`, `mock-only assertion`, `unproven regression`.
-- Link the authoritative doc section in the review comment.
-- If an explicit repo rule, lint rule, or authoritative doc note is violated, do not downgrade it to "minor deviation" or "nit".
-
-## Quick Reference
-
-| When you see...                     | Read this                                                                                                                                                                                         |
-| ----------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| New `vi.mock(...)` for a composable | [`docs/testing/unit-testing.md`](../../../docs/testing/unit-testing.md) -> "Mocking Composables with Reactive State"                                                                              |
-| New store test or store mock        | [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md) setup + [`docs/testing/store-testing.md`](../../../docs/testing/store-testing.md)                                   |
-| New component test                  | Top note in [`docs/testing/component-testing.md`](../../../docs/testing/component-testing.md)                                                                                                     |
-| `vue-i18n` in a component test      | [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md) + [`src/components/searchbox/v2/__test__/testUtils.ts`](../../../src/components/searchbox/v2/__test__/testUtils.ts) |
-| Cast around a mock                  | [`docs/guidance/typescript.md`](../../../docs/guidance/typescript.md) -> "Type Assertion Hierarchy"                                                                                               |
-
-## Key Files to Read
-
-| Purpose                              | Path                                                                                                              |
-| ------------------------------------ | ----------------------------------------------------------------------------------------------------------------- |
-| Composable mocking patterns          | [`docs/testing/unit-testing.md`](../../../docs/testing/unit-testing.md)                                           |
-| Store testing patterns               | [`docs/testing/store-testing.md`](../../../docs/testing/store-testing.md)                                         |
-| Repo-wide Vitest setup defaults      | [`docs/testing/vitest-patterns.md`](../../../docs/testing/vitest-patterns.md)                                     |
-| Component testing rule for new tests | [`docs/testing/component-testing.md`](../../../docs/testing/component-testing.md)                                 |
-| Real i18n setup                      | [`src/components/searchbox/v2/__test__/testUtils.ts`](../../../src/components/searchbox/v2/__test__/testUtils.ts) |

.env_example

@@ -46,9 +46,3 @@ ALGOLIA_API_KEY=684d998c36b67a9a9fce8fc2d8860579
 # SENTRY_ORG=comfy-org
 # SENTRY_PROJECT=cloud-frontend-staging
 # SENTRY_PROJECT_PROD= # prod project slug for sourcemap uploads
-
-# Ashby (apps/website careers page build).
-# Server-only; read inside the Astro build context. Do NOT prefix with PUBLIC_.
-# When unset, the committed snapshot at apps/website/src/data/ashby-roles.snapshot.json is used.
-# WEBSITE_ASHBY_API_KEY=
-# WEBSITE_ASHBY_JOB_BOARD_NAME=comfy-org

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

@@ -21,7 +21,7 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 30
     container:
-      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.17
+      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.12
       credentials:
         username: ${{ github.actor }}
         password: ${{ secrets.GITHUB_TOKEN }}

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

@@ -82,7 +82,7 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 60
     container:
-      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.17
+      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.16
       credentials:
         username: ${{ github.actor }}
         password: ${{ secrets.GITHUB_TOKEN }}
@@ -140,7 +140,7 @@ jobs:
     needs: setup
     runs-on: ubuntu-latest
     container:
-      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.17
+      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.16
       credentials:
         username: ${{ github.actor }}
         password: ${{ secrets.GITHUB_TOKEN }}

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

@@ -2,7 +2,7 @@ name: 'CI: Website E2E'
 
 on:
   push:
-    branches: [main]
+    branches: [main, website/*]
     paths:
       - 'apps/website/**'
       - 'packages/design-system/**'
@@ -17,7 +17,7 @@ on:
       - 'pnpm-lock.yaml'
 
 concurrency:
-  group: ${{ github.workflow }}-${{ github.repository }}-${{ github.head_ref || github.ref }}
+  group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
 
 jobs:
@@ -45,8 +45,6 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Build website
-        env:
-          WEBSITE_GITHUB_STARS_OVERRIDE: 110000
         run: pnpm --filter @comfyorg/website build
 
       - name: Run Playwright tests

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

@@ -77,7 +77,7 @@ jobs:
     needs: setup
     runs-on: ubuntu-latest
     container:
-      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.17
+      image: ghcr.io/comfy-org/comfyui-ci-container:0.0.16
       credentials:
         username: ${{ github.actor }}
         password: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/pr-update-website-screenshots.yaml

@@ -18,7 +18,6 @@ jobs:
     timeout-minutes: 15
     permissions:
       contents: write
-      issues: write
       pull-requests: read
     # Trigger: (1) label, (2) /slash-command, or (3) checkbox in E2E status comment
     # ⚠️ This condition is duplicated on `post-starting-comment` — keep them in sync.
@@ -87,8 +86,6 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Build website
-        env:
-          WEBSITE_GITHUB_STARS_OVERRIDE: 110000
         run: pnpm --filter @comfyorg/website build
 
       - name: Update screenshots
@@ -140,10 +137,7 @@ jobs:
                 name: 'Update Website Screenshots'
               })
             } catch (e) {
-              if (e.status !== 404) {
-                throw e
-              }
-              core.info('Label "Update Website Screenshots" was already removed')
+              // Label may already be removed
             }
 
   post-starting-comment:

.husky/pre-push

@@ -1,13 +1,5 @@
 #!/usr/bin/env bash
 
-# Skip in CI: the canonical knip check runs in ci-lint-format on every
-# PR, and bot workflows (e.g. i18n-update-core) populate ComfyUI/ via
-# setup-comfyui-server, which contaminates knip's project glob with the
-# devtools copy under custom_nodes and produces false-positive failures.
-if [ -n "${CI-}" ]; then
-  exit 0
-fi
-
 # Run Knip with cache via package script
 pnpm knip 1>&2
 

apps/desktop-ui/src/assets/css/style.css

@@ -20,15 +20,15 @@
 }
 
 .p-button-danger {
-  background-color: var(--color-coral-700);
+  background-color: var(--color-coral-red-600);
 }
 
 .p-button-danger:hover {
-  background-color: var(--color-coral-600);
+  background-color: var(--color-coral-red-500);
 }
 
 .p-button-danger:active {
-  background-color: var(--color-coral-500);
+  background-color: var(--color-coral-red-400);
 }
 
 .task-div .p-card {

apps/website/.gitignore

@@ -2,7 +2,6 @@ dist/
 .astro/
 test-results/
 playwright-report/
-results.json
 
 # Platform-specific Playwright snapshots (CI runs Linux)
 *-win32.png

apps/website/README.md

@@ -1,148 +0,0 @@
-# @comfyorg/website
-
-Marketing/brand website built with Astro + Vue.
-
-## Ashby careers integration
-
-`/careers` and `/zh-CN/careers` are rendered from Ashby's public job board
-API at build time. Data flow:
-
-1. `src/pages/careers.astro` awaits `fetchRolesForBuild()` during the
-   Astro build.
-2. `src/utils/ashby.ts` calls
-   `GET https://api.ashbyhq.com/posting-api/job-board/{board}?includeCompensation=false`,
-   validates the envelope and each posting with Zod
-   (`src/utils/ashby.schema.ts`), and maps to the domain type in
-   `src/data/roles.ts`.
-3. On any failure (network, HTTP 4xx/5xx, envelope schema drift),
-   the fetcher falls back to the committed JSON snapshot at
-   `src/data/ashby-roles.snapshot.json`.
-4. `src/utils/ashby.ci.ts` emits GitHub Actions annotations and a
-   `$GITHUB_STEP_SUMMARY` block so stale fetches are visible on green
-   builds.
-
-### Required environment variables
-
-Both are build-time only. Never prefix with `PUBLIC_` (Astro would
-inline that into the client bundle).
-
-| Name                           | Purpose                     | Default (when unset)              |
-| ------------------------------ | --------------------------- | --------------------------------- |
-| `WEBSITE_ASHBY_API_KEY`        | Ashby API key (Basic auth)  | Build uses the committed snapshot |
-| `WEBSITE_ASHBY_JOB_BOARD_NAME` | Ashby public job board slug | Build uses the committed snapshot |
-
-### CI wiring (manual step — required)
-
-This repo's `.github/workflows/*.yaml` changes cannot be pushed by a
-GitHub App. A maintainer must apply the following edits **once**:
-
-**`.github/workflows/ci-website-build.yaml`** — pass the env into the
-build step and run the unit tests before it:
-
-```yaml
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v6
-      - name: Setup frontend
-        uses: ./.github/actions/setup-frontend
-
-      - name: Run website unit tests
-        run: pnpm --filter @comfyorg/website test:unit
-
-      - name: Build website
-        env:
-          WEBSITE_ASHBY_API_KEY: ${{ secrets.WEBSITE_ASHBY_API_KEY }}
-          WEBSITE_ASHBY_JOB_BOARD_NAME: ${{ vars.WEBSITE_ASHBY_JOB_BOARD_NAME || 'comfy-org' }}
-        run: pnpm --filter @comfyorg/website build
-
-      - name: Verify API key is not leaked into build output
-        env:
-          WEBSITE_ASHBY_API_KEY: ${{ secrets.WEBSITE_ASHBY_API_KEY }}
-        run: |
-          set +x
-          if [ -z "${WEBSITE_ASHBY_API_KEY:-}" ]; then
-            echo "Secret not available in this run; skipping leak check."
-            exit 0
-          fi
-          # grep -rlF prints only file paths (never match content).
-          MATCHES=$(grep -rlF --exclude-dir=node_modules --null \
-            -e "$WEBSITE_ASHBY_API_KEY" apps/website/dist/ 2>/dev/null \
-            | tr '\0' '\n' || true)
-          if [ -n "$MATCHES" ]; then
-            echo "::error title=Ashby API key leaked into build output::$MATCHES"
-            exit 1
-          fi
-```
-
-**`.github/workflows/ci-vercel-website-preview.yaml`** — add the
-two env vars to the top-level `env:` block so `vercel build` (both
-`deploy-preview` and `deploy-production` jobs) sees them:
-
-```yaml
-env:
-  VERCEL_ORG_ID: ${{ secrets.VERCEL_WEBSITE_ORG_ID }}
-  VERCEL_PROJECT_ID: ${{ secrets.VERCEL_WEBSITE_PROJECT_ID }}
-  VERCEL_TOKEN: ${{ secrets.VERCEL_WEBSITE_TOKEN }}
-  VERCEL_SCOPE: comfyui
-  WEBSITE_ASHBY_API_KEY: ${{ secrets.WEBSITE_ASHBY_API_KEY }}
-  WEBSITE_ASHBY_JOB_BOARD_NAME: ${{ vars.WEBSITE_ASHBY_JOB_BOARD_NAME || 'comfy-org' }}
-```
-
-The secret must also be added to the Vercel project environment
-(`vercel env add WEBSITE_ASHBY_API_KEY …` or via the Vercel UI) so
-that `vercel build` in the preview job has access to it.
-
-Fork PRs do not exercise this path: `ci-vercel-website-preview.yaml`
-receives an empty `VERCEL_TOKEN` for forks and fails at `vercel pull`
-before the build runs. Fork-safe PR interactions (the preview-URL
-comment) are handled by `pr-vercel-website-preview.yaml`.
-
-### Refreshing the snapshot
-
-When a maintainer wants to update the committed snapshot (e.g. after
-onboarding/offboarding roles):
-
-```bash
-WEBSITE_ASHBY_API_KEY=… WEBSITE_ASHBY_JOB_BOARD_NAME=comfy-org \
-  pnpm --filter @comfyorg/website ashby:refresh-snapshot
-git commit apps/website/src/data/ashby-roles.snapshot.json
-```
-
-The script exits non-zero on any non-fresh outcome so stale/empty
-snapshots can't be accidentally committed.
-
-## HubSpot contact form
-
-The contact page uses HubSpot's hosted form embed for the interest form:
-
-```html
-<script
-  src="https://js-na2.hsforms.net/forms/embed/developer/244637579.js"
-  defer
-></script>
-<div
-  class="hs-form-html"
-  data-region="na2"
-  data-form-id="94e05eab-1373-47f7-ab5e-d84f9e6aa262"
-  data-portal-id="244637579"
-></div>
-```
-
-The localized `/zh-CN/contact` page uses the same portal and script with form
-ID `6885750c-02ef-4aa2-ba0d-213be9cccf93`.
-
-This keeps submission handling, validation, anti-spam updates, and field
-configuration in HubSpot. The local implementation in
-`src/components/contact/HubspotFormEmbed.vue` only loads the hosted script and
-renders the documented embed container.
-
-## Scripts
-
-- `pnpm dev` — Astro dev server
-- `pnpm build` — production build to `dist/`
-- `pnpm typecheck` — `astro check`
-- `pnpm test:unit` — Vitest unit tests
-- `pnpm test:e2e` — Playwright E2E tests (requires `pnpm build` first)
-- `pnpm ashby:refresh-snapshot` — refresh the committed careers snapshot

apps/website/astro.config.ts

@@ -7,12 +7,6 @@ export default defineConfig({
   site: 'https://comfy.org',
   output: 'static',
   prefetch: { prefetchAll: true },
-  redirects: {
-    '/cloud/enterprise-case-studies/comfyui-at-architectural-scale-how-moment-factory-reimagined-3d-projection-mapping':
-      '/customers/moment-factory/',
-    '/cloud/enterprise-case-studies/how-series-entertainment-rebuilt-game-and-video-production-with-comfyui':
-      '/customers/series-entertainment/'
-  },
   build: {
     assets: '_website'
   },

apps/website/e2e/careers.spec.ts

@@ -1,57 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { test } from './fixtures/blockExternalMedia'
-
-test.describe('Careers page @smoke', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto('/careers')
-  })
-
-  test('has correct title', async ({ page }) => {
-    await expect(page).toHaveTitle('Careers — Comfy')
-  })
-
-  test('Roles section heading is visible', async ({ page }) => {
-    await expect(
-      page.getByRole('heading', { name: 'Roles', level: 2 })
-    ).toBeVisible()
-  })
-
-  test('renders at least one role from the snapshot', async ({ page }) => {
-    const roles = page.getByTestId('careers-role-link')
-    await expect(roles.first()).toBeVisible()
-    expect(await roles.count()).toBeGreaterThan(0)
-  })
-
-  test('each role links to jobs.ashbyhq.com', async ({ page }) => {
-    const roles = page.getByTestId('careers-role-link')
-    const count = await roles.count()
-    for (let i = 0; i < count; i++) {
-      const href = await roles.nth(i).getAttribute('href')
-      expect(href).toMatch(/^https:\/\/jobs\.ashbyhq\.com\//)
-    }
-  })
-
-  test('ENGINEERING category filter narrows the role list', async ({
-    page
-  }) => {
-    const allCount = await page.getByTestId('careers-role-link').count()
-    await page.getByRole('button', { name: 'ENGINEERING', exact: true }).click()
-    const engineeringLocator = page.getByTestId('careers-role-link')
-    await expect(engineeringLocator.first()).toBeVisible()
-    const engineeringCount = await engineeringLocator.count()
-    expect(engineeringCount).toBeLessThanOrEqual(allCount)
-    expect(engineeringCount).toBeGreaterThan(0)
-  })
-})
-
-test.describe('Careers page (zh-CN) @smoke', () => {
-  test('renders localized heading and roles', async ({ page }) => {
-    await page.goto('/zh-CN/careers')
-    await expect(page).toHaveTitle('招聘 — Comfy')
-    await expect(
-      page.getByRole('heading', { name: '职位', level: 2 })
-    ).toBeVisible()
-    await expect(page.getByTestId('careers-role-link').first()).toBeVisible()
-  })
-})

apps/website/package.json

@@ -9,13 +9,10 @@
     "build": "astro build",
     "preview": "astro preview",
     "typecheck": "astro check",
-    "test:unit": "vitest run",
-    "test:coverage": "vitest run --coverage",
     "test:e2e": "playwright test",
     "test:e2e:local": "cross-env PLAYWRIGHT_LOCAL=1 playwright test",
     "test:visual": "playwright test --project visual",
-    "test:visual:update": "playwright test --project visual --update-snapshots",
-    "ashby:refresh-snapshot": "tsx ./scripts/refresh-ashby-snapshot.ts"
+    "test:visual:update": "playwright test --project visual --update-snapshots"
   },
   "dependencies": {
     "@astrojs/sitemap": "catalog:",
@@ -26,9 +23,7 @@
     "cva": "catalog:",
     "gsap": "catalog:",
     "lenis": "catalog:",
-    "posthog-js": "catalog:",
-    "vue": "catalog:",
-    "zod": "catalog:"
+    "vue": "catalog:"
   },
   "devDependencies": {
     "@astrojs/check": "catalog:",
@@ -37,9 +32,7 @@
     "@tailwindcss/vite": "catalog:",
     "astro": "catalog:",
     "tailwindcss": "catalog:",
-    "tsx": "catalog:",
-    "typescript": "catalog:",
-    "vitest": "catalog:"
+    "typescript": "catalog:"
   },
   "nx": {
     "tags": [
@@ -96,22 +89,6 @@
           "command": "astro check"
         }
       },
-      "test:unit": {
-        "executor": "nx:run-commands",
-        "cache": true,
-        "options": {
-          "cwd": "apps/website",
-          "command": "vitest run"
-        }
-      },
-      "test:coverage": {
-        "executor": "nx:run-commands",
-        "cache": true,
-        "options": {
-          "cwd": "apps/website",
-          "command": "vitest run --coverage"
-        }
-      },
       "test:e2e": {
         "executor": "nx:run-commands",
         "dependsOn": [

apps/website/scripts/refresh-ashby-snapshot.ts

@@ -1,33 +0,0 @@
-import { renameSync, writeFileSync } from 'node:fs'
-import { fileURLToPath } from 'node:url'
-
-import { fetchRolesForBuild } from '../src/utils/ashby'
-
-const snapshotPath = fileURLToPath(
-  new URL('../src/data/ashby-roles.snapshot.json', import.meta.url)
-)
-const tempPath = `${snapshotPath}.tmp`
-
-const outcome = await fetchRolesForBuild()
-
-if (outcome.status !== 'fresh') {
-  const reason = 'reason' in outcome ? outcome.reason : '(none)'
-  console.error(
-    `Snapshot refresh aborted. Outcome: ${outcome.status}; reason: ${reason}`
-  )
-  process.exit(1)
-}
-
-writeFileSync(
-  tempPath,
-  JSON.stringify(outcome.snapshot, null, 2) + '\n',
-  'utf8'
-)
-renameSync(tempPath, snapshotPath)
-const totalRoles = outcome.snapshot.departments.reduce(
-  (n, d) => n + d.roles.length,
-  0
-)
-process.stdout.write(
-  `Wrote snapshot with ${totalRoles} role(s) to ${snapshotPath}\n`
-)

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

@@ -1,42 +1,121 @@
 <script setup lang="ts">
 import { computed, ref } from 'vue'
 
-import type { Department } from '../../data/roles'
 import type { Locale } from '../../i18n/translations'
 
 import { t } from '../../i18n/translations'
 import CategoryNav from '../common/CategoryNav.vue'
 import SectionLabel from '../common/SectionLabel.vue'
 
-const { locale = 'en', departments = [] } = defineProps<{
-  locale?: Locale
-  departments?: readonly Department[]
-}>()
+const { locale = 'en' } = defineProps<{ locale?: Locale }>()
 
 const activeCategory = ref('all')
 
-const visibleDepartments = computed(() =>
-  departments.filter((d) => d.roles.length > 0)
-)
+interface Role {
+  title: string
+  department: string
+  location: string
+  id: string
+}
+
+interface Department {
+  name: string
+  key: string
+  roles: Role[]
+}
+
+const departments: Department[] = [
+  {
+    name: 'ENGINEERING',
+    key: 'engineering',
+    roles: [
+      {
+        title: 'Design Engineer',
+        department: 'Engineering',
+        location: 'San Francisco',
+        id: 'abc787b9-ad85-421c-8218-debd23bea096'
+      },
+      {
+        title: 'Software Engineer',
+        department: 'Engineering',
+        location: 'San Francisco',
+        id: '99dc26c7-51ca-43cd-a1ba-7d475a0f4a40'
+      },
+      {
+        title: 'Product Manager',
+        department: 'Engineering',
+        location: 'London, UK',
+        id: '12dbc26e-9f6d-49bf-83c6-130f7566d03c'
+      },
+      {
+        title: 'Tech Lead Manager, Frontend',
+        department: 'Engineering',
+        location: 'San Francisco',
+        id: 'a0665088-3314-457a-aa7b-12ca5c3eb261'
+      }
+    ]
+  },
+  {
+    name: 'DESIGN',
+    key: 'design',
+    roles: [
+      {
+        title: 'Creative Director',
+        department: 'Design',
+        location: 'San Francisco',
+        id: '49fa0b07-3fa1-4a3a-b2c6-d2cc684ad63f'
+      },
+      {
+        title: 'Graphic Designer',
+        department: 'Design',
+        location: 'London, UK',
+        id: '19ba10aa-4961-45e8-8473-66a8a7a8079d'
+      },
+      {
+        title: 'Freelance Motion Designer',
+        department: 'Design',
+        location: 'Remote',
+        id: 'a7ccc2b4-4d9d-4e04-b39c-28a711995b5b'
+      }
+    ]
+  },
+  {
+    name: 'MARKETING',
+    key: 'marketing',
+    roles: [
+      {
+        title: 'Lifecycle Growth Marketer',
+        department: 'Marketing',
+        location: 'San Francisco',
+        id: 'be74d210-3b50-408c-9f61-8fee8833ce64'
+      },
+      {
+        title: 'Graphic Designer',
+        department: 'Marketing',
+        location: 'London, UK',
+        id: '28dea965-662b-4786-b024-c9a1b6bc1f23'
+      }
+    ]
+  }
+]
 
 const categories = computed(() => [
   { label: 'ALL', value: 'all' },
-  ...visibleDepartments.value.map((d) => ({ label: d.name, value: d.key }))
+  ...departments.map((d) => ({ label: d.name, value: d.key }))
 ])
 
 const filteredDepartments = computed(() =>
   activeCategory.value === 'all'
-    ? visibleDepartments.value
-    : visibleDepartments.value.filter((d) => d.key === activeCategory.value)
+    ? departments
+    : departments.filter((d) => d.key === activeCategory.value)
 )
-
-const hasRoles = computed(() => visibleDepartments.value.length > 0)
 </script>
 
 <template>
-  <section class="px-6 py-20 md:px-20 md:py-32" data-testid="careers-roles">
+  <section class="px-6 py-20 md:px-20 md:py-32">
     <div class="mx-auto max-w-6xl">
       <div class="flex flex-col gap-12 md:flex-row md:gap-20">
+        <!-- Left sidebar -->
         <div class="shrink-0 md:w-48">
           <div
             class="bg-primary-comfy-ink sticky top-20 z-10 py-4 md:top-28 md:py-0"
@@ -47,7 +126,6 @@ const hasRoles = computed(() => visibleDepartments.value.length > 0)
               {{ t('careers.roles.heading', locale) }}
             </h2>
             <CategoryNav
-              v-if="hasRoles"
               v-model="activeCategory"
               :categories="categories"
               class="mt-4"
@@ -55,15 +133,8 @@ const hasRoles = computed(() => visibleDepartments.value.length > 0)
           </div>
         </div>
 
+        <!-- Role listings -->
         <div class="min-w-0 flex-1">
-          <p
-            v-if="!hasRoles"
-            class="text-primary-warm-gray text-base md:text-lg"
-            data-testid="careers-roles-empty"
-          >
-            {{ t('careers.roles.empty', locale) }}
-          </p>
-
           <div
             v-for="dept in filteredDepartments"
             :key="dept.key"
@@ -76,11 +147,10 @@ const hasRoles = computed(() => visibleDepartments.value.length > 0)
             <a
               v-for="role in dept.roles"
               :key="role.id"
-              :href="role.applyUrl"
+              :href="`https://jobs.ashbyhq.com/comfy-org/${role.id}`"
               target="_blank"
               rel="noopener noreferrer"
               class="border-primary-warm-gray/20 group flex items-center justify-between border-b py-5"
-              data-testid="careers-role-link"
             >
               <div class="min-w-0">
                 <span

apps/website/src/components/contact/FormSection.vue

@@ -1,12 +1,13 @@
 <script setup lang="ts">
+import { cn } from '@comfyorg/tailwind-utils'
 import { ref } from 'vue'
 
 import type { Locale, TranslationKey } from '../../i18n/translations'
 
 import { useHeroAnimation } from '../../composables/useHeroAnimation'
 import { t } from '../../i18n/translations'
+import BrandButton from '../common/BrandButton.vue'
 import SectionLabel from '../common/SectionLabel.vue'
-import HubspotFormEmbed from './HubspotFormEmbed.vue'
 
 const { locale = 'en' } = defineProps<{
   locale?: Locale
@@ -16,6 +17,30 @@ function tk(suffix: string): TranslationKey {
   return `contact.form.${suffix}` as TranslationKey
 }
 
+const firstName = ref('')
+const lastName = ref('')
+const company = ref('')
+const phone = ref('')
+const selectedPackage = ref('')
+const comfyUsage = ref('')
+const lookingFor = ref('')
+
+const packageOptions = [
+  'packageIndividual',
+  'packageTeams',
+  'packageEnterprise'
+] as const
+
+const usageOptions = [
+  'usingYesProduction',
+  'usingYesTesting',
+  'usingNotYet',
+  'usingOtherTools'
+] as const
+
+const inputClass =
+  'text-primary-comfy-canvas placeholder:text-primary-comfy-canvas/30 border-primary-warm-gray/20 focus:border-primary-comfy-yellow mt-2 w-full rounded-2xl border bg-transparency-white-t4 p-4 text-sm transition-colors outline-none'
+
 const sectionRef = ref<HTMLElement>()
 const badgeRef = ref<HTMLElement>()
 const headingRef = ref<HTMLElement>()
@@ -30,6 +55,10 @@ useHeroAnimation({
   video: formRef,
   parallax: false
 })
+
+function handleSubmit() {
+  // TODO: implement form submission
+}
 </script>
 
 <template>
@@ -76,7 +105,160 @@ useHeroAnimation({
 
     <!-- Right column: form -->
     <div ref="formRef" class="mt-12 lg:mt-0 lg:w-1/2">
-      <HubspotFormEmbed :locale />
+      <form class="space-y-6" @submit.prevent="handleSubmit">
+        <!-- First Name + Last Name -->
+        <div class="lg:grid lg:grid-cols-2 lg:gap-4">
+          <div>
+            <label class="text-primary-comfy-canvas text-xs">
+              {{ t(tk('firstName'), locale) }}*
+            </label>
+            <input
+              v-model="firstName"
+              type="text"
+              required
+              :placeholder="t(tk('firstNamePlaceholder'), locale)"
+              :class="inputClass"
+            />
+          </div>
+          <div class="mt-6 lg:mt-0">
+            <label class="text-primary-comfy-canvas text-xs">
+              {{ t(tk('lastName'), locale) }}*
+            </label>
+            <input
+              v-model="lastName"
+              type="text"
+              required
+              :placeholder="t(tk('lastNamePlaceholder'), locale)"
+              :class="inputClass"
+            />
+          </div>
+        </div>
+
+        <!-- Company + Phone -->
+        <div class="lg:grid lg:grid-cols-2 lg:gap-4">
+          <div>
+            <label class="text-primary-comfy-canvas text-xs">
+              {{ t(tk('company'), locale) }}*
+            </label>
+            <input
+              v-model="company"
+              type="text"
+              required
+              :placeholder="t(tk('companyPlaceholder'), locale)"
+              :class="inputClass"
+            />
+          </div>
+          <div class="mt-6 lg:mt-0">
+            <label class="text-primary-comfy-canvas text-xs">
+              {{ t(tk('phone'), locale) }}
+            </label>
+            <input v-model="phone" type="tel" :class="inputClass" />
+          </div>
+        </div>
+
+        <!-- Package selection -->
+        <div>
+          <p class="text-primary-comfy-canvas text-xs">
+            {{ t(tk('packageQuestion'), locale) }}
+          </p>
+          <div class="mt-3 flex flex-wrap gap-3">
+            <label
+              v-for="opt in packageOptions"
+              :key="opt"
+              :class="
+                cn(
+                  'bg-transparency-white-t4 flex cursor-pointer items-center gap-2 rounded-lg border px-6 py-2 text-xs font-bold tracking-wider transition-colors',
+                  selectedPackage === opt
+                    ? 'border-primary-comfy-yellow text-primary-comfy-yellow'
+                    : 'text-primary-comfy-canvas border-(--site-border-subtle)'
+                )
+              "
+            >
+              <input
+                v-model="selectedPackage"
+                type="radio"
+                name="package"
+                :value="opt"
+                class="sr-only"
+              />
+              <span
+                :class="
+                  cn(
+                    'flex size-4 shrink-0 items-center justify-center rounded-full border',
+                    selectedPackage === opt
+                      ? 'border-primary-comfy-yellow'
+                      : 'border-primary-warm-gray/40'
+                  )
+                "
+              >
+                <span
+                  v-if="selectedPackage === opt"
+                  class="bg-primary-comfy-yellow size-2 rounded-full"
+                />
+              </span>
+              {{ t(tk(opt), locale) }}
+            </label>
+          </div>
+        </div>
+
+        <!-- Comfy usage -->
+        <div>
+          <p class="text-primary-comfy-canvas text-xs">
+            {{ t(tk('usingComfy'), locale) }}
+          </p>
+          <div class="mt-3 space-y-3">
+            <label
+              v-for="opt in usageOptions"
+              :key="opt"
+              class="flex cursor-pointer items-center gap-3"
+            >
+              <span
+                :class="
+                  cn(
+                    'flex size-4 shrink-0 items-center justify-center rounded-full border',
+                    comfyUsage === opt
+                      ? 'border-primary-comfy-yellow'
+                      : 'border-(--site-border-subtle)'
+                  )
+                "
+              >
+                <span
+                  v-if="comfyUsage === opt"
+                  class="bg-primary-comfy-yellow size-2 rounded-full"
+                />
+              </span>
+              <input
+                v-model="comfyUsage"
+                type="radio"
+                :value="opt"
+                class="sr-only"
+              />
+              <span class="text-primary-comfy-canvas text-sm">
+                {{ t(tk(opt), locale) }}
+              </span>
+            </label>
+          </div>
+        </div>
+
+        <!-- Looking for -->
+        <div>
+          <label class="text-primary-comfy-canvas text-xs">
+            {{ t(tk('lookingFor'), locale) }}
+          </label>
+          <textarea
+            v-model="lookingFor"
+            :placeholder="t(tk('lookingForPlaceholder'), locale)"
+            :class="cn(inputClass, 'min-h-24 resize-y')"
+          />
+        </div>
+
+        <!-- Submit -->
+        <div>
+          <BrandButton type="submit" variant="outline" size="sm">
+            {{ t(tk('submit'), locale) }}
+          </BrandButton>
+        </div>
+      </form>
     </div>
   </section>
 </template>

apps/website/src/components/contact/HubspotFormEmbed.vue

@@ -1,126 +0,0 @@
-<script setup lang="ts">
-import { computed, onMounted, ref } from 'vue'
-
-import type { Locale } from '../../i18n/translations'
-
-import { t } from '../../i18n/translations'
-
-const { locale = 'en' } = defineProps<{
-  locale?: Locale
-}>()
-
-const HUBSPOT_CONTACT_PORTAL_ID = '244637579'
-const HUBSPOT_CONTACT_REGION = 'na2'
-const HUBSPOT_CONTACT_SCRIPT_ID = 'hubspot-contact-form-embed'
-const HUBSPOT_CONTACT_SCRIPT_SRC = `https://js-${HUBSPOT_CONTACT_REGION}.hsforms.net/forms/embed/developer/${HUBSPOT_CONTACT_PORTAL_ID}.js`
-
-const hubspotContactFormIds: Record<Locale, string> = {
-  en: '94e05eab-1373-47f7-ab5e-d84f9e6aa262',
-  'zh-CN': '6885750c-02ef-4aa2-ba0d-213be9cccf93'
-}
-
-const hasEmbedLoadError = ref(false)
-const hubspotContactFormId = computed(() => hubspotContactFormIds[locale])
-
-const hubspotFormStyles: Record<`--${string}`, string> = {
-  '--hsf-global__font-family': "'PP Formula', sans-serif",
-  '--hsf-global__color': '#c2bfb9',
-  '--hsf-background__background-color': '#211927',
-  '--hsf-background__border-width': '0',
-  '--hsf-background__padding': '0',
-  '--hsf-button__font-family': "'PP Formula', sans-serif",
-  '--hsf-button__font-size': '14px',
-  '--hsf-button__color': '#211927',
-  '--hsf-button__background-color': '#f2ff59',
-  '--hsf-button__border-radius': '16px',
-  '--hsf-button__padding': '10px 24px',
-  '--hsf-richtext__font-family': "'PP Formula', sans-serif",
-  '--hsf-richtext__color': '#c2bfb9',
-  '--hsf-heading__font-family': "'PP Formula', sans-serif",
-  '--hsf-heading__color': '#c2bfb9',
-  '--hsf-field-label__font-family': "'PP Formula', sans-serif",
-  '--hsf-field-label__font-size': '12px',
-  '--hsf-field-label__color': '#c2bfb9',
-  '--hsf-field-description__font-family': "'PP Formula', sans-serif",
-  '--hsf-field-description__color': '#c2bfb9',
-  '--hsf-field-footer__font-family': "'PP Formula', sans-serif",
-  '--hsf-field-footer__color': '#c2bfb9',
-  '--hsf-field-input__font-family': "'PP Formula', sans-serif",
-  '--hsf-field-input__color': '#c2bfb9',
-  '--hsf-field-input__background-color': '#2a2230',
-  '--hsf-field-input__placeholder-color': '#585159',
-  '--hsf-field-input__border-color': '#3b3539',
-  '--hsf-field-input__border-width': '1px',
-  '--hsf-field-input__border-style': 'solid',
-  '--hsf-field-input__border-radius': '16px',
-  '--hsf-field-input__padding': '16px',
-  '--hsf-field-textarea__font-family': "'PP Formula', sans-serif",
-  '--hsf-field-textarea__color': '#c2bfb9',
-  '--hsf-field-textarea__background-color': '#2a2230',
-  '--hsf-field-textarea__placeholder-color': '#585159',
-  '--hsf-field-textarea__border-color': '#3b3539',
-  '--hsf-field-textarea__border-width': '1px',
-  '--hsf-field-textarea__border-style': 'solid',
-  '--hsf-field-textarea__border-radius': '16px',
-  '--hsf-field-textarea__padding': '16px',
-  '--hsf-field-checkbox__color': '#c2bfb9',
-  '--hsf-field-checkbox__background-color': '#2a2230',
-  '--hsf-field-checkbox__border-color': '#464147',
-  '--hsf-field-checkbox__border-width': '1px',
-  '--hsf-field-checkbox__border-style': 'solid',
-  '--hsf-field-radio__color': '#c2bfb9',
-  '--hsf-field-radio__background-color': '#2a2230',
-  '--hsf-field-radio__border-color': '#464147',
-  '--hsf-field-radio__border-width': '1px',
-  '--hsf-field-radio__border-style': 'solid',
-  '--hsf-erroralert__font-family': "'PP Formula', sans-serif",
-  '--hsf-infoalert__font-family': "'PP Formula', sans-serif"
-}
-
-onMounted(() => {
-  if (document.getElementById(HUBSPOT_CONTACT_SCRIPT_ID)) return
-
-  const script = document.createElement('script')
-  script.id = HUBSPOT_CONTACT_SCRIPT_ID
-  script.src = HUBSPOT_CONTACT_SCRIPT_SRC
-  script.defer = true
-  script.addEventListener(
-    'error',
-    () => {
-      hasEmbedLoadError.value = true
-      script.remove()
-    },
-    { once: true }
-  )
-
-  document.head.append(script)
-})
-</script>
-
-<template>
-  <div class="min-h-[640px] w-full">
-    <p
-      v-if="hasEmbedLoadError"
-      class="text-primary-comfy-canvas text-sm/6"
-      role="status"
-    >
-      {{ t('contact.form.embedLoadErrorPrefix', locale) }}
-      <a
-        class="text-primary-comfy-yellow underline"
-        href="mailto:hello@comfy.org"
-      >
-        hello@comfy.org
-      </a>
-      {{ t('contact.form.embedLoadErrorSuffix', locale) }}
-    </p>
-    <div
-      v-else
-      :key="hubspotContactFormId"
-      class="hs-form-html"
-      :style="hubspotFormStyles"
-      :data-region="HUBSPOT_CONTACT_REGION"
-      :data-form-id="hubspotContactFormId"
-      :data-portal-id="HUBSPOT_CONTACT_PORTAL_ID"
-    />
-  </div>
-</template>

apps/website/src/components/product/api/StepsSection.vue

@@ -13,13 +13,13 @@ const steps = [
     number: '01',
     titleKey: 'api.steps.step1.title' as const,
     descriptionKey: 'api.steps.step1.description' as const,
-    image: 'https://media.comfy.org/website/enterprise/enterprise_node_1.webp'
+    image: 'https://media.comfy.org/website/api/logo-purple.webp'
   },
   {
     number: '02',
     titleKey: 'api.steps.step2.title' as const,
     descriptionKey: 'api.steps.step2.description' as const,
-    image: 'https://media.comfy.org/website/enterprise/enterprise_node_2.webp'
+    image: 'https://media.comfy.org/website/api/logo-yellow.webp'
   },
   {
     number: '03',

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

@@ -10,12 +10,12 @@ const cards = [
   {
     titleKey: 'enterprise.byoKey.card1.title' as const,
     descriptionKey: 'enterprise.byoKey.card1.description' as const,
-    image: 'https://media.comfy.org/website/enterprise/enterprise_node_1.webp'
+    image: 'https://media.comfy.org/website/api/logo-purple.webp'
   },
   {
     titleKey: 'enterprise.byoKey.card2.title' as const,
     descriptionKey: 'enterprise.byoKey.card2.description' as const,
-    image: 'https://media.comfy.org/website/enterprise/enterprise_node_2.webp'
+    image: 'https://media.comfy.org/website/api/logo-yellow.webp'
   }
 ]
 </script>

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

@@ -16,11 +16,9 @@ const midRightRef = ref<HTMLElement>()
 const bottomLeftRef = ref<HTMLElement>()
 const bottomRightRef = ref<HTMLElement>()
 
-const parallaxOpts = { trigger: sectionRef, mediaQuery: '(min-width: 1024px)' }
-
-useParallax([topLeftRef, topRightRef], { ...parallaxOpts, y: 200 })
-useParallax([midLeftRef, midRightRef], { ...parallaxOpts, y: 300 })
-useParallax([bottomLeftRef, bottomRightRef], { ...parallaxOpts, y: 400 })
+useParallax([topLeftRef, topRightRef], { trigger: sectionRef, y: 200 })
+useParallax([midLeftRef, midRightRef], { trigger: sectionRef, y: 300 })
+useParallax([bottomLeftRef, bottomRightRef], { trigger: sectionRef, y: 400 })
 </script>
 
 <template>

apps/website/src/composables/useParallax.ts

@@ -19,8 +19,6 @@ interface ParallaxOptions {
   start?: string
   /** ScrollTrigger end value (default: 'bottom top') */
   end?: string
-  /** Media query string — animation only runs when matched (responsive) */
-  mediaQuery?: string
 }
 
 export function useParallax(
@@ -28,27 +26,24 @@ export function useParallax(
   options: ParallaxOptions = {}
 ) {
   const { fromY = 0, y = 200 } = options
-  let ctx: gsap.Context | gsap.MatchMedia | undefined
+  let ctx: gsap.Context | undefined
 
   onMounted(() => {
-    if (prefersReducedMotion()) return
-
     const triggerEl = options.trigger?.value
+    const els = elements
+      .map((r) => r.value)
+      .filter((el): el is HTMLElement => !!el && el.offsetParent !== null)
+    if (!els.length || prefersReducedMotion()) return
 
-    const createAnimations = () => {
-      const els = elements
-        .map((r) => r.value)
-        .filter((el): el is HTMLElement => !!el && el.offsetParent !== null)
-      if (!els.length) return
-
-      const trigger = triggerEl ?? els[0]
-      const scrollTrigger = {
-        trigger,
-        start: options.start ?? 'top bottom',
-        end: options.end ?? 'bottom top',
-        scrub: 1
-      }
+    const trigger = triggerEl ?? els[0]
+    const scrollTrigger = {
+      trigger,
+      start: options.start ?? 'top bottom',
+      end: options.end ?? 'bottom top',
+      scrub: 1
+    }
 
+    ctx = gsap.context(() => {
       els.forEach((el) => {
         gsap.fromTo(
           el,
@@ -56,15 +51,7 @@ export function useParallax(
           { y: resolve(y, el, trigger), ease: 'none', scrollTrigger }
         )
       })
-    }
-
-    if (options.mediaQuery) {
-      const mm = gsap.matchMedia()
-      mm.add(options.mediaQuery, createAnimations)
-      ctx = mm
-    } else {
-      ctx = gsap.context(createAnimations)
-    }
+    })
   })
 
   onUnmounted(() => {

apps/website/src/data/ashby-roles.snapshot.json

@@ -1,169 +0,0 @@
-{
-  "fetchedAt": "2026-04-24T18:59:03.989Z",
-  "departments": [
-    {
-      "name": "DESIGN",
-      "key": "design",
-      "roles": [
-        {
-          "id": "4c5d6afb78652df7",
-          "title": "Freelance Motion Designer",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/a7ccc2b4-4d9d-4e04-b39c-28a711995b5b/application"
-        },
-        {
-          "id": "0f5256cf302e552b",
-          "title": "Creative Artist",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/19ba10aa-4961-45e8-8473-66a8a7a8079d/application"
-        },
-        {
-          "id": "e915f2c78b17f93b",
-          "title": "Senior Product Designer",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/b2e864c6-4754-4e04-8f46-1022baa103c3/application"
-        },
-        {
-          "id": "b9f9a23219be7cd4",
-          "title": "Design Engineer",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/abc787b9-ad85-421c-8218-debd23bea096/application"
-        },
-        {
-          "id": "5746486d87874937",
-          "title": "Graphic Designer",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/49fa0b07-3fa1-4a3a-b2c6-d2cc684ad63f/application"
-        },
-        {
-          "id": "547b6ba622c800a5",
-          "title": "Senior Product Designer - Craft",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/a32c6769-b791-41f4-9225-50bbd8a1cf0f/application"
-        },
-        {
-          "id": "7bb02634a24763bc",
-          "title": "Staff Product Designer - Systems",
-          "department": "Design",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/0bc8356b-615e-4f40-b632-fd3b2691be34/application"
-        }
-      ]
-    },
-    {
-      "name": "ENGINEERING",
-      "key": "engineering",
-      "roles": [
-        {
-          "id": "102d58e35a8a9817",
-          "title": "Senior Software Engineer, Frontend",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/c3e0584d-5490-491f-aae4-b5922ef63fd2/application"
-        },
-        {
-          "id": "d01d69fba7743905",
-          "title": "Senior Software Engineer, Backend Generalist",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/732f8b39-076d-4847-afe3-f54d4451607e/application"
-        },
-        {
-          "id": "f36f60cfd5bb5910",
-          "title": "Senior/Staff Applied Machine Learning Engineer",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/5cc4d0bc-97b0-463b-8466-3ec1d07f6ac0/application"
-        },
-        {
-          "id": "9d8ec4c65e20b19e",
-          "title": "Software Engineer, Frontend",
-          "department": "Engineering",
-          "location": "Remote",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/99dc26c7-51ca-43cd-a1ba-7d475a0f4a40/application"
-        },
-        {
-          "id": "be94b193d1f4d482",
-          "title": "Tech Lead Manager, Frontend",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/a0665088-3314-457a-aa7b-12ca5c3eb261/application"
-        },
-        {
-          "id": "ab48f5db6bd1783c",
-          "title": "Software Engineer, Core ComfyUI Contributor",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/7d4062d6-d500-445a-9a5f-014971af259f/application"
-        },
-        {
-          "id": "c5dff4ee628bdcd1",
-          "title": "Software Engineer, ComfyUI Desktop",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/ad2f76cb-a787-47d8-81c5-7e7f917747c0/application"
-        },
-        {
-          "id": "4302a7aaa87e16e3",
-          "title": "Product Manager, ComfyUI",
-          "department": "Engineering",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/9e4b9029-c3e9-436b-82c4-a1a9f1b8c16e/application"
-        }
-      ]
-    },
-    {
-      "name": "MARKETING",
-      "key": "marketing",
-      "roles": [
-        {
-          "id": "b5803a0d4785d406",
-          "title": "Lifecycle Growth Marketer",
-          "department": "Marketing",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/be74d210-3b50-408c-9f61-8fee8833ce64/application"
-        },
-        {
-          "id": "130d7218d7895bdb",
-          "title": "Partnership & Events Marketing Manager",
-          "department": "Marketing",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/89d3ff75-2055-4e92-9c69-81feff55627c/application"
-        }
-      ]
-    },
-    {
-      "name": "OPERATIONS",
-      "key": "operations",
-      "roles": [
-        {
-          "id": "ec68ae44dd5943c9",
-          "title": "Senior Technical Recruiter",
-          "department": "Operations",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/d5008532-c45d-46e6-ba2c-20489d364362/application"
-        },
-        {
-          "id": "16f556001ce1cef4",
-          "title": "BizOps Strategist",
-          "department": "Operations",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/145b8558-0ab4-43e8-8fac-b59059cf2537/application"
-        },
-        {
-          "id": "8e773a72c1b8e099",
-          "title": "Founding Customer Success Manager",
-          "department": "Operations",
-          "location": "San Francisco",
-          "applyUrl": "https://jobs.ashbyhq.com/comfy-org/a1c5c5ed-62ac-4767-af57-a3ba4e0bf5e4/application"
-        }
-      ]
-    }
-  ]
-}

apps/website/src/data/roles.ts

@@ -1,18 +0,0 @@
-export interface Role {
-  id: string
-  title: string
-  department: string
-  location: string
-  applyUrl: string
-}
-
-export interface Department {
-  name: string
-  key: string
-  roles: Role[]
-}
-
-export interface RolesSnapshot {
-  fetchedAt: string
-  departments: Department[]
-}

apps/website/src/i18n/translations.ts

@@ -4,12 +4,12 @@ const translations = {
   // HeroSection
   'hero.title': {
     en: 'Professional Control\nof Visual AI',
-    'zh-CN': '视觉 AI 的\n最强可控性'
+    'zh-CN': '视觉 AI 的\n专业控制'
   },
   'hero.subtitle': {
     en: 'Comfy is the AI creation engine for visual professionals who demand control over every model, every parameter, and every output.',
     'zh-CN':
-      'Comfy 是面向专业视觉人士的 AI 创作引擎。您可以精确掌控每个模型、每个参数和每个输出。'
+      'Comfy 是面向视觉专业人士的 AI 创作引擎，让您掌控每个模型、每个参数和每个输出。'
   },
 
   // ProductShowcaseSection
@@ -20,11 +20,11 @@ const translations = {
   },
   'showcase.subtitle2': {
     en: 'Start from a community template or build from scratch.',
-    'zh-CN': '从工作流模板开始，或从零构建。'
+    'zh-CN': '从社区模板开始，或从零构建。'
   },
   'showcase.feature1.title': {
     en: 'Full Control with Nodes',
-    'zh-CN': '节点带来的可控性'
+    'zh-CN': '节点式完全控制'
   },
   'showcase.feature1.description': {
     en: 'Build powerful AI pipelines by connecting nodes on an infinite canvas. Every model, parameter, and processing step is visible and adjustable.',
@@ -49,8 +49,8 @@ const translations = {
     'zh-CN':
       '浏览和混搭数千个社区共享的工作流。从经过验证的模板开始，按需自定义。'
   },
-  'showcase.badgeHow': { en: 'HOW', 'zh-CN': '了解' },
-  'showcase.badgeWorks': { en: 'WORKS', 'zh-CN': '运行方式' },
+  'showcase.badgeHow': { en: 'HOW', 'zh-CN': '如何' },
+  'showcase.badgeWorks': { en: 'WORKS', 'zh-CN': '运作' },
 
   // UseCaseSection
   'useCase.label': {
@@ -83,7 +83,8 @@ const translations = {
   },
   'useCase.body': {
     en: 'Powered by 60,000+ nodes, thousands of workflows,\nand a community that builds faster than any one company could.',
-    'zh-CN': '60,000+ 节点，数千条工作流，\n一个比任何公司速度都更快的社区。'
+    'zh-CN':
+      '由 60,000+ 节点、数千个工作流\n和一个比任何公司都更快构建的社区驱动。'
   },
   'useCase.cta': {
     en: 'EXPLORE WORKFLOWS',
@@ -163,7 +164,7 @@ const translations = {
   },
   'products.local.cta': {
     en: 'SEE LOCAL FEATURES',
-    'zh-CN': '查看本地版属性'
+    'zh-CN': '查看本地版特性'
   },
   'products.cloud.title': {
     en: 'Comfy\nCloud',
@@ -175,7 +176,7 @@ const translations = {
   },
   'products.cloud.cta': {
     en: 'SEE CLOUD FEATURES',
-    'zh-CN': '查看云端属性'
+    'zh-CN': '查看云端特性'
   },
   'products.api.title': {
     en: 'Comfy\nAPI',
@@ -187,7 +188,7 @@ const translations = {
   },
   'products.api.cta': {
     en: 'SEE API FEATURES',
-    'zh-CN': '查看 API 属性'
+    'zh-CN': '查看 API 特性'
   },
   'products.enterprise.title': {
     en: 'Comfy\nEnterprise',
@@ -199,7 +200,7 @@ const translations = {
   },
   'products.enterprise.cta': {
     en: 'SEE ENTERPRISE FEATURES',
-    'zh-CN': '查看企业版属性'
+    'zh-CN': '查看企业版特性'
   },
 
   // CaseStudySpotlightSection
@@ -1214,7 +1215,7 @@ const translations = {
   'pricing.included.feature4.description': {
     en: 'All plans will include a monthly pool of credits that are spent on active workflow runtime and <a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">Partner Nodes</a> like Nano Banana Pro.',
     'zh-CN':
-      '所有计划均包含每月积分池，可用于工作流运行和<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">合作伙伴节点</a>（如 Nano Banana Pro）。'
+      '所有计划均包含每月积分池，可用于工作流运行和<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">合作节点</a>（如 Nano Banana Pro）。'
   },
   'pricing.included.feature5.title': {
     en: 'Add more credits anytime',
@@ -1244,12 +1245,12 @@ const translations = {
   },
   'pricing.included.feature8.title': {
     en: 'Partner Nodes',
-    'zh-CN': '合作伙伴节点'
+    'zh-CN': '合作节点'
   },
   'pricing.included.feature8.description': {
     en: 'Run <strong>proprietary models</strong> through Comfy\'s <a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">Partner Nodes</a>, such as Nano Banana. The amount of credits each node uses depends on the model and parameters you set in the node, but these credits are the same ones that your monthly subscription comes with. These credits can also be used across Comfy Cloud and local ComfyUI. Read more about Partner nodes <a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">here</a>.',
     'zh-CN':
-      '通过 Comfy 的<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">合作伙伴节点</a>运行<strong>专有模型</strong>，如 Nano Banana。每个节点消耗的积分取决于所用模型和参数设置，且与月度订阅积分通用。积分可在 Comfy Cloud 和本地 ComfyUI 间通用。了解更多关于合作伙伴节点的信息请点击<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">此处</a>。'
+      '通过 Comfy 的<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">合作节点</a>运行<strong>专有模型</strong>，如 Nano Banana。每个节点消耗的积分取决于所用模型和参数设置，且与月度订阅积分通用。积分可在 Comfy Cloud 和本地 ComfyUI 间通用。了解更多关于合作节点的信息请点击<a href="https://docs.comfy.org/tutorials/partner-nodes/overview" class="text-primary-comfy-yellow underline">此处</a>。'
   },
   'pricing.included.feature9.title': {
     en: 'Job queue',
@@ -1504,10 +1505,6 @@ const translations = {
 
   // CareersRolesSection
   'careers.roles.heading': { en: 'Roles', 'zh-CN': '职位' },
-  'careers.roles.empty': {
-    en: 'No open roles right now. Check back soon.',
-    'zh-CN': '目前暂无开放职位，请稍后再来查看。'
-  },
 
   // CareersFAQSection
   'careers.faq.heading': { en: 'Q&A', 'zh-CN': 'Q&A' },
@@ -3298,13 +3295,82 @@ const translations = {
     en: 'Find your answer here',
     'zh-CN': '在这里找到答案'
   },
-  'contact.form.embedLoadErrorPrefix': {
-    en: 'Unable to load the contact form. Email us at',
-    'zh-CN': '联系表单无法加载。请发送邮件至'
+  'contact.form.firstName': {
+    en: 'First name',
+    'zh-CN': '名'
   },
-  'contact.form.embedLoadErrorSuffix': {
-    en: "and we'll route your request.",
-    'zh-CN': '我们会为您处理请求。'
+  'contact.form.lastName': {
+    en: 'Last Name',
+    'zh-CN': '姓'
+  },
+  'contact.form.company': {
+    en: 'Company',
+    'zh-CN': '公司'
+  },
+  'contact.form.phone': {
+    en: 'Phone Number (optional)',
+    'zh-CN': '电话号码（可选）'
+  },
+  'contact.form.packageQuestion': {
+    en: 'Are you interested in learning more about our Enterprise Services, which start at $100K annually, our individual packages, or our team packages?',
+    'zh-CN':
+      '您是否有兴趣了解更多关于我们的企业服务（年费起价 $100K）、个人套餐或团队套餐？'
+  },
+  'contact.form.packageIndividual': {
+    en: 'INDIVIDUAL',
+    'zh-CN': '个人'
+  },
+  'contact.form.packageTeams': {
+    en: 'TEAMS',
+    'zh-CN': '团队'
+  },
+  'contact.form.packageEnterprise': {
+    en: 'ENTERPRISE',
+    'zh-CN': '企业'
+  },
+  'contact.form.usingComfy': {
+    en: 'Are you /your team currently using Comfy?',
+    'zh-CN': '您/您的团队目前是否在使用 Comfy？'
+  },
+  'contact.form.usingYesProduction': {
+    en: 'Yes, in production',
+    'zh-CN': '是，在生产环境中'
+  },
+  'contact.form.usingYesTesting': {
+    en: 'Yes, testing / experimenting',
+    'zh-CN': '是，测试/实验中'
+  },
+  'contact.form.usingNotYet': {
+    en: 'Not yet, evaluating',
+    'zh-CN': '尚未使用，评估中'
+  },
+  'contact.form.usingOtherTools': {
+    en: 'Not using Comfy yet, but using other GenAI tools',
+    'zh-CN': '尚未使用 Comfy，但在使用其他 GenAI 工具'
+  },
+  'contact.form.lookingFor': {
+    en: 'What are you looking for?',
+    'zh-CN': '您在寻找什么？'
+  },
+  'contact.form.lookingForPlaceholder': {
+    en: 'Tell us about your team needs, expected usage, or other specific requirements.',
+    'zh-CN': '请告诉我们您的团队需求、预期使用情况或其他具体要求。'
+  },
+  'contact.form.submit': {
+    en: 'SUBMIT',
+    'zh-CN': '提交'
+  },
+  'contact.form.firstNamePlaceholder': {
+    en: 'Jane',
+    'zh-CN': 'Jane'
+  },
+  'contact.form.lastNamePlaceholder': {
+    en: 'Smith',
+    'zh-CN': 'Smith'
+  },
+  'contact.form.companyPlaceholder': {
+    en: 'jane@acme.org',
+    'zh-CN': 'jane@acme.org'
   },
 
   'customers.story.whatsNext': {

apps/website/src/layouts/BaseLayout.astro

@@ -133,15 +133,9 @@ const websiteJsonLd = {
     <script>
       import { initSmoothScroll, cancelScroll } from '../scripts/smoothScroll'
       import { ScrollTrigger } from '../scripts/gsapSetup'
-      import { initPostHog, capturePageview } from '../scripts/posthog'
 
       initSmoothScroll()
 
-      if (import.meta.env.PROD) {
-        initPostHog()
-        document.addEventListener('astro:page-load', capturePageview)
-      }
-
       document.addEventListener('astro:page-load', () => {
         ScrollTrigger.refresh()
       })

apps/website/src/pages/careers.astro

@@ -5,20 +5,6 @@ import RolesSection from '../components/careers/RolesSection.vue'
 import WhyJoinSection from '../components/careers/WhyJoinSection.vue'
 import TeamPhotosSection from '../components/careers/TeamPhotosSection.vue'
 import FAQSection from '../components/common/FAQSection.vue'
-import { fetchRolesForBuild } from '../utils/ashby'
-import { reportAshbyOutcome } from '../utils/ashby.ci'
-
-const outcome = await fetchRolesForBuild()
-reportAshbyOutcome(outcome)
-
-if (outcome.status === 'failed') {
-  throw new Error(
-    `Ashby fetch failed and no snapshot is available. Reason: ${outcome.reason}. ` +
-      'Run `pnpm --filter @comfyorg/website ashby:refresh-snapshot` locally and commit the snapshot.'
-  )
-}
-
-const departments = outcome.snapshot.departments
 ---
 
 <BaseLayout
@@ -26,7 +12,7 @@ const departments = outcome.snapshot.departments
   description="Join the team building the operating system for generative AI. Open roles in engineering, design, marketing, and more."
 >
   <HeroSection />
-  <RolesSection departments={departments} client:visible />
+  <RolesSection client:visible />
   <WhyJoinSection client:visible />
   <TeamPhotosSection client:visible />
   <FAQSection

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

@@ -5,20 +5,6 @@ import RolesSection from '../../components/careers/RolesSection.vue'
 import WhyJoinSection from '../../components/careers/WhyJoinSection.vue'
 import TeamPhotosSection from '../../components/careers/TeamPhotosSection.vue'
 import FAQSection from '../../components/common/FAQSection.vue'
-import { fetchRolesForBuild } from '../../utils/ashby'
-import { reportAshbyOutcome } from '../../utils/ashby.ci'
-
-const outcome = await fetchRolesForBuild()
-reportAshbyOutcome(outcome)
-
-if (outcome.status === 'failed') {
-  throw new Error(
-    `Ashby fetch failed and no snapshot is available. Reason: ${outcome.reason}. ` +
-      'Run `pnpm --filter @comfyorg/website ashby:refresh-snapshot` locally and commit the snapshot.'
-  )
-}
-
-const departments = outcome.snapshot.departments
 ---
 
 <BaseLayout
@@ -26,7 +12,7 @@ const departments = outcome.snapshot.departments
   description="加入构建生成式 AI 操作系统的团队。工程、设计、市场营销等岗位开放招聘中。"
 >
   <HeroSection locale="zh-CN" />
-  <RolesSection locale="zh-CN" departments={departments} client:visible />
+  <RolesSection locale="zh-CN" client:visible />
   <WhyJoinSection locale="zh-CN" client:visible />
   <TeamPhotosSection client:visible />
   <FAQSection

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

@@ -10,7 +10,7 @@ import GetStartedSection from '../../components/home/GetStartedSection.vue'
 import BuildWhatSection from '../../components/home/BuildWhatSection.vue'
 ---
 
-<BaseLayout title="Comfy — 视觉 AI 的最强可控性">
+<BaseLayout title="Comfy — 视觉 AI 的专业控制">
   <HeroSection locale="zh-CN" client:load />
   <SocialProofBarSection />
   <ProductShowcaseSection locale="zh-CN" client:load />

apps/website/src/scripts/posthog.ts

@@ -1,36 +0,0 @@
-import posthog from 'posthog-js'
-
-const POSTHOG_KEY =
-  import.meta.env.PUBLIC_POSTHOG_KEY ??
-  'phc_iKfK86id4xVYws9LybMje0h44eGtfwFgRPIBehmy8rO'
-const POSTHOG_API_HOST =
-  import.meta.env.PUBLIC_POSTHOG_API_HOST ?? 'https://t.comfy.org'
-const POSTHOG_UI_HOST =
-  import.meta.env.PUBLIC_POSTHOG_UI_HOST ?? 'https://us.posthog.com'
-
-let initialized = false
-
-export function initPostHog() {
-  if (initialized || typeof window === 'undefined' || !POSTHOG_KEY) return
-  try {
-    posthog.init(POSTHOG_KEY, {
-      api_host: POSTHOG_API_HOST,
-      ui_host: POSTHOG_UI_HOST,
-      capture_pageview: false,
-      capture_pageleave: true,
-      person_profiles: 'identified_only'
-    })
-    initialized = true
-  } catch (error) {
-    console.error('PostHog init failed', error)
-  }
-}
-
-export function capturePageview() {
-  if (!initialized) return
-  try {
-    posthog.capture('$pageview')
-  } catch (error) {
-    console.error('PostHog pageview capture failed', error)
-  }
-}

apps/website/src/utils/ashby.ci.test.ts

@@ -1,130 +0,0 @@
-import { mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
-import { tmpdir } from 'node:os'
-import { join } from 'node:path'
-
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-
-import type { FetchOutcome } from './ashby'
-import type { RolesSnapshot } from '../data/roles'
-
-import { reportAshbyOutcome, resetAshbyReporterForTests } from './ashby.ci'
-
-function baseSnapshot(): RolesSnapshot {
-  return {
-    fetchedAt: new Date().toISOString(),
-    departments: [
-      {
-        name: 'ENGINEERING',
-        key: 'engineering',
-        roles: [
-          {
-            id: 'x',
-            title: 'Design Engineer',
-            department: 'Engineering',
-            location: 'San Francisco',
-            applyUrl: 'https://jobs.ashbyhq.com/comfy-org/x'
-          }
-        ]
-      }
-    ]
-  }
-}
-
-function freshOutcome(droppedCount = 0): FetchOutcome {
-  return {
-    status: 'fresh',
-    droppedCount,
-    droppedRoles:
-      droppedCount === 0
-        ? []
-        : [{ title: 'Bad Role', reason: 'jobUrl: Invalid url' }],
-    snapshot: {
-      fetchedAt: new Date().toISOString(),
-      departments: [
-        {
-          name: 'ENGINEERING',
-          key: 'engineering',
-          roles: [
-            {
-              id: 'x',
-              title: 'Design Engineer',
-              department: 'Engineering',
-              location: 'San Francisco',
-              applyUrl: 'https://jobs.ashbyhq.com/comfy-org/x'
-            }
-          ]
-        }
-      ]
-    }
-  }
-}
-
-describe('reportAshbyOutcome', () => {
-  let writeSpy: ReturnType<typeof vi.spyOn>
-  let summaryDir: string
-  let summaryPath: string
-  const originalSummary = process.env.GITHUB_STEP_SUMMARY
-
-  beforeEach(() => {
-    resetAshbyReporterForTests()
-    writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true)
-    summaryDir = mkdtempSync(join(tmpdir(), 'ashby-summary-'))
-    summaryPath = join(summaryDir, 'summary.md')
-    writeFileSync(summaryPath, '')
-    process.env.GITHUB_STEP_SUMMARY = summaryPath
-  })
-
-  afterEach(() => {
-    writeSpy.mockRestore()
-    rmSync(summaryDir, { recursive: true, force: true })
-    if (originalSummary === undefined) delete process.env.GITHUB_STEP_SUMMARY
-    else process.env.GITHUB_STEP_SUMMARY = originalSummary
-  })
-
-  it('emits nothing on a clean fresh outcome', () => {
-    reportAshbyOutcome(freshOutcome(0))
-    expect(writeSpy).not.toHaveBeenCalled()
-    expect(readFileSync(summaryPath, 'utf8')).toContain('Fresh')
-  })
-
-  it('emits exactly one set of annotations across repeated calls', () => {
-    reportAshbyOutcome(freshOutcome(1))
-    reportAshbyOutcome(freshOutcome(1))
-    expect(writeSpy).toHaveBeenCalledTimes(1)
-    const annotation = writeSpy.mock.calls[0]![0] as string
-    expect(annotation).toContain('::warning title=Ashby: dropped 1 invalid')
-    expect(readFileSync(summaryPath, 'utf8')).toContain('Dropped')
-  })
-
-  it('emits ::error for auth failures in a stale outcome', () => {
-    reportAshbyOutcome({
-      status: 'stale',
-      reason: 'HTTP 401 Unauthorized',
-      snapshot: baseSnapshot()
-    })
-    const annotation = writeSpy.mock.calls[0]![0] as string
-    expect(annotation).toContain('::error title=Ashby authentication failed')
-  })
-
-  it('emits ::warning for missing-env stale outcomes', () => {
-    reportAshbyOutcome({
-      status: 'stale',
-      reason: 'missing WEBSITE_ASHBY_API_KEY or WEBSITE_ASHBY_JOB_BOARD_NAME',
-      snapshot: baseSnapshot()
-    })
-    const annotation = writeSpy.mock.calls[0]![0] as string
-    expect(annotation).toContain('::warning title=Ashby integration')
-  })
-
-  it('emits ::error for a failed outcome and writes no fresh-only sections', () => {
-    reportAshbyOutcome({ status: 'failed', reason: 'HTTP 500 Server Error' })
-    const annotation = writeSpy.mock.calls[0]![0] as string
-    expect(annotation).toContain('::error title=Ashby fetch failed')
-    expect(readFileSync(summaryPath, 'utf8')).toContain('Failed')
-  })
-
-  it('does not throw when GITHUB_STEP_SUMMARY is not set', () => {
-    delete process.env.GITHUB_STEP_SUMMARY
-    expect(() => reportAshbyOutcome(freshOutcome(0))).not.toThrow()
-  })
-})

apps/website/src/utils/ashby.ci.ts

@@ -1,113 +0,0 @@
-import { appendFileSync } from 'node:fs'
-
-import type { FetchOutcome } from './ashby'
-
-let hasReported = false
-
-export function resetAshbyReporterForTests(): void {
-  hasReported = false
-}
-
-export function reportAshbyOutcome(outcome: FetchOutcome): void {
-  if (hasReported) return
-  hasReported = true
-
-  const lines = buildAnnotations(outcome)
-  for (const line of lines) {
-    process.stdout.write(`${line}\n`)
-  }
-
-  const summaryPath = process.env.GITHUB_STEP_SUMMARY
-  if (summaryPath) {
-    try {
-      appendFileSync(summaryPath, buildStepSummary(outcome))
-    } catch {
-      // Writing the summary is best-effort; do not fail the build if the
-      // runner's summary file is unavailable (e.g. local dev).
-    }
-  }
-}
-
-function buildAnnotations(outcome: FetchOutcome): string[] {
-  if (outcome.status === 'fresh') {
-    if (outcome.droppedCount === 0) return []
-    const roleCount = outcome.droppedCount === 1 ? 'role' : 'roles'
-    const drops = outcome.droppedRoles
-      .map((d) => `  - ${d.title ? `"${d.title}"` : '(untitled)'}: ${d.reason}`)
-      .join('%0A')
-    return [
-      `::warning title=Ashby: dropped ${outcome.droppedCount} invalid ${roleCount}::Dropped roles:%0A${drops}%0A%0AAction items:%0A  1. Fix the posting in Ashby admin (e.g. assign a department, fix the URL).%0A  2. If the v1 schema is too strict for a legitimate case, relax the field in apps/website/src/utils/ashby.schema.ts and add a test.%0A  3. These roles will not appear on the careers page until fixed.`
-    ]
-  }
-
-  if (outcome.status === 'stale') {
-    return [staleAnnotation(outcome.reason)]
-  }
-
-  return [
-    `::error title=Ashby fetch failed and no snapshot is available::Cannot build careers page without data.%0A%0AReason: ${escapeAnnotation(outcome.reason)}%0A%0AAction items:%0A  1. Run \`pnpm --filter @comfyorg/website ashby:refresh-snapshot\` locally with a valid WEBSITE_ASHBY_API_KEY.%0A  2. Commit apps/website/src/data/ashby-roles.snapshot.json.%0A  3. Push and re-run CI.`
-  ]
-}
-
-function staleAnnotation(reason: string): string {
-  const escaped = escapeAnnotation(reason)
-  if (reason.startsWith('missing ')) {
-    return `::warning title=Ashby integration::${escaped}. Falling back to committed snapshot.%0A%0AAction items:%0A  1. If you're a contributor without key access, this is expected. The snapshot will be used.%0A  2. If this is CI, check that the \`WEBSITE_ASHBY_API_KEY\` secret exists in the repo and is referenced in .github/workflows/ci-website-build.yaml.`
-  }
-  if (reason.startsWith('HTTP 401') || reason.startsWith('HTTP 403')) {
-    return `::error title=Ashby authentication failed::${escaped}. The WEBSITE_ASHBY_API_KEY is missing, invalid, or revoked. Build continues with the last-known-good snapshot.%0A%0AAction items:%0A  1. Open Ashby → Settings → API Keys and confirm the key is active.%0A  2. Update the \`WEBSITE_ASHBY_API_KEY\` secret in GitHub Actions and Vercel.%0A  3. Re-run this workflow.`
-  }
-  if (reason.startsWith('envelope')) {
-    return `::error title=Ashby schema mismatch::${escaped}. The Ashby API contract has likely changed. Build continues with the snapshot, but future updates will fail until the schema is fixed.%0A%0AAction items:%0A  1. Check https://developers.ashbyhq.com/reference for API changelog.%0A  2. Update apps/website/src/utils/ashby.schema.ts to match the new shape.`
-  }
-  return `::warning title=Ashby API unavailable::${escaped}. Using last-known-good snapshot.%0A%0AAction items:%0A  1. Check https://status.ashbyhq.com%0A  2. Re-run this workflow once Ashby is healthy.`
-}
-
-function escapeAnnotation(value: string): string {
-  return value.replace(/\r?\n/g, '%0A').replace(/\r/g, '%0D')
-}
-
-function buildStepSummary(outcome: FetchOutcome): string {
-  const header = '## 💼 Careers (Ashby)\n'
-  const rows: Array<[string, string]> = []
-
-  if (outcome.status === 'fresh') {
-    rows.push(['Status', '✅ Fresh (fetched from Ashby)'])
-    rows.push([
-      'Roles',
-      String(
-        outcome.snapshot.departments.reduce((n, d) => n + d.roles.length, 0)
-      )
-    ])
-    rows.push(['Dropped', String(outcome.droppedCount)])
-  } else if (outcome.status === 'stale') {
-    rows.push(['Status', '⚠️ Stale (using snapshot — Ashby fetch failed)'])
-    rows.push([
-      'Roles',
-      String(
-        outcome.snapshot.departments.reduce((n, d) => n + d.roles.length, 0)
-      )
-    ])
-    rows.push(['Reason', outcome.reason])
-    rows.push(['Snapshot age', describeSnapshotAge(outcome.snapshot.fetchedAt)])
-  } else {
-    rows.push(['Status', '❌ Failed (no snapshot available)'])
-    rows.push(['Reason', outcome.reason])
-  }
-
-  const table =
-    '| | |\n|---|---|\n' +
-    rows.map(([k, v]) => `| **${k}** | ${v} |`).join('\n') +
-    '\n'
-
-  return `${header}${table}\n`
-}
-
-function describeSnapshotAge(fetchedAt: string): string {
-  const fetched = new Date(fetchedAt).getTime()
-  if (Number.isNaN(fetched)) return 'unknown'
-  const days = Math.floor((Date.now() - fetched) / 86_400_000)
-  if (days <= 0) return 'today'
-  if (days === 1) return '1 day'
-  return `${days} days`
-}

apps/website/src/utils/ashby.schema.ts

@@ -1,17 +0,0 @@
-import { z } from 'zod'
-
-export const AshbyJobPostingSchema = z.object({
-  title: z.string().min(1),
-  department: z.string().optional(),
-  location: z.string().optional(),
-  isListed: z.boolean(),
-  jobUrl: z.string().url(),
-  applyUrl: z.string().url().optional()
-})
-
-export const AshbyJobBoardResponseSchema = z.object({
-  apiVersion: z.literal('1'),
-  jobs: z.array(z.unknown())
-})
-
-export type AshbyJobPosting = z.infer<typeof AshbyJobPostingSchema>

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

@@ -1,328 +0,0 @@
-import { mkdtempSync, rmSync, writeFileSync } from 'node:fs'
-import { tmpdir } from 'node:os'
-import { join } from 'node:path'
-import { pathToFileURL } from 'node:url'
-
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-
-import type { AshbyJobPosting } from './ashby.schema'
-import type { RolesSnapshot } from '../data/roles'
-
-import { fetchRolesForBuild, resetAshbyFetcherForTests } from './ashby'
-
-const BASE_URL = 'https://ashby.test'
-const BOARD = 'comfy-org'
-const KEY = 'abc-123-secret'
-
-function validJob(overrides: Partial<AshbyJobPosting> = {}): unknown {
-  return {
-    title: 'Design Engineer',
-    department: 'Engineering',
-    location: 'San Francisco',
-    isListed: true,
-    jobUrl: 'https://jobs.ashbyhq.com/comfy-org/design-engineer',
-    applyUrl: 'https://jobs.ashbyhq.com/comfy-org/design-engineer/apply',
-    ...overrides
-  }
-}
-
-function response(body: unknown, init: Partial<ResponseInit> = {}): Response {
-  const base: ResponseInit = {
-    status: 200,
-    headers: { 'content-type': 'application/json' }
-  }
-  return new Response(JSON.stringify(body), { ...base, ...init })
-}
-
-function makeSnapshot(roleCount = 2): RolesSnapshot {
-  const roles = Array.from({ length: roleCount }, (_, i) => ({
-    id: `snapshot-role-${i}`,
-    title: `Snapshot Role ${i}`,
-    department: 'Engineering',
-    location: 'San Francisco',
-    applyUrl: `https://jobs.ashbyhq.com/comfy-org/snapshot-${i}`
-  }))
-  return {
-    fetchedAt: '2026-04-01T00:00:00.000Z',
-    departments: [{ name: 'ENGINEERING', key: 'engineering', roles }]
-  }
-}
-
-function withSnapshotDir(snapshot: RolesSnapshot | null): URL {
-  const dir = mkdtempSync(join(tmpdir(), 'ashby-test-'))
-  const file = join(dir, 'ashby-roles.snapshot.json')
-  if (snapshot) writeFileSync(file, JSON.stringify(snapshot))
-  return pathToFileURL(file)
-}
-
-describe('fetchRolesForBuild', () => {
-  const savedApiKey = process.env.WEBSITE_ASHBY_API_KEY
-  const savedBoardName = process.env.WEBSITE_ASHBY_JOB_BOARD_NAME
-
-  beforeEach(() => {
-    resetAshbyFetcherForTests()
-    delete process.env.WEBSITE_ASHBY_API_KEY
-    delete process.env.WEBSITE_ASHBY_JOB_BOARD_NAME
-  })
-
-  afterEach(() => {
-    vi.restoreAllMocks()
-    process.env.WEBSITE_ASHBY_API_KEY = savedApiKey
-    process.env.WEBSITE_ASHBY_JOB_BOARD_NAME = savedBoardName
-  })
-
-  it('returns fresh when the API succeeds', async () => {
-    const fetchImpl = vi.fn(async () =>
-      response({ apiVersion: '1', jobs: [validJob()] })
-    )
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    expect(outcome.droppedCount).toBe(0)
-    expect(outcome.snapshot.departments).toHaveLength(1)
-    expect(outcome.snapshot.departments[0]!.roles[0]!.applyUrl).toMatch(
-      /design-engineer\/apply$/
-    )
-  })
-
-  it('falls back to jobUrl when applyUrl is missing and keeps the role', async () => {
-    const job = validJob()
-    delete (job as Record<string, unknown>).applyUrl
-    const fetchImpl = vi.fn(async () =>
-      response({ apiVersion: '1', jobs: [job] })
-    )
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    expect(outcome.snapshot.departments[0]!.roles[0]!.applyUrl).toBe(
-      'https://jobs.ashbyhq.com/comfy-org/design-engineer'
-    )
-  })
-
-  it('drops invalid roles individually and keeps the valid ones', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () =>
-      response({
-        apiVersion: '1',
-        jobs: [validJob(), validJob({ title: 'Bad Role', jobUrl: 'not-a-url' })]
-      })
-    )
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    expect(outcome.droppedCount).toBe(1)
-    expect(outcome.droppedRoles[0]!.title).toBe('Bad Role')
-    expect(outcome.snapshot.departments[0]!.roles).toHaveLength(1)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('renders an empty-but-fresh outcome when hiring is paused', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () => response({ apiVersion: '1', jobs: [] }))
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    expect(outcome.snapshot.departments).toEqual([])
-    expect(outcome.droppedCount).toBe(0)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('normalizes missing department and location to safe defaults', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const job = validJob()
-    delete (job as Record<string, unknown>).department
-    delete (job as Record<string, unknown>).location
-    const fetchImpl = vi.fn(async () =>
-      response({ apiVersion: '1', jobs: [job] })
-    )
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    const [department] = outcome.snapshot.departments
-    expect(department?.name).toBe('OTHER')
-    expect(department?.roles[0]?.location).toBe('Remote')
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('filters out roles with isListed=false', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () =>
-      response({
-        apiVersion: '1',
-        jobs: [validJob(), validJob({ title: 'Hidden', isListed: false })]
-      })
-    )
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('fresh')
-    if (outcome.status !== 'fresh') return
-    const titles = outcome.snapshot.departments.flatMap((d) =>
-      d.roles.map((r) => r.title)
-    )
-    expect(titles).not.toContain('Hidden')
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('returns stale with missing env when the snapshot is present', async () => {
-    const snapshot = makeSnapshot()
-    const snapshotUrl = withSnapshotDir(snapshot)
-    const fetchImpl = vi.fn()
-    const outcome = await fetchRolesForBuild({
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('stale')
-    if (outcome.status !== 'stale') return
-    expect(outcome.reason).toMatch(/^missing /)
-    expect(fetchImpl).not.toHaveBeenCalled()
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('returns failed when both env and snapshot are missing', async () => {
-    const snapshotUrl = withSnapshotDir(null)
-    const outcome = await fetchRolesForBuild({
-      snapshotUrl,
-      fetchImpl: vi.fn() as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('failed')
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('does not retry on HTTP 401', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () => response({}, { status: 401 }))
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('stale')
-    if (outcome.status !== 'stale') return
-    expect(outcome.reason).toMatch(/^HTTP 401/)
-    expect(fetchImpl).toHaveBeenCalledTimes(1)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('retries 5xx up to the configured limit then falls back to snapshot', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () => response({}, { status: 503 }))
-    const sleep = vi.fn(async () => undefined)
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      retryDelaysMs: [1, 1, 1],
-      sleep,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('stale')
-    expect(fetchImpl).toHaveBeenCalledTimes(4)
-    expect(sleep).toHaveBeenCalledTimes(3)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('falls back to snapshot on envelope schema mismatch', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () => response({ apiVersion: '2', jobs: [] }))
-    const outcome = await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(outcome.status).toBe('stale')
-    if (outcome.status !== 'stale') return
-    expect(outcome.reason).toMatch(/^envelope schema/)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('memoizes within a single process', async () => {
-    const fetchImpl = vi.fn(async () =>
-      response({ apiVersion: '1', jobs: [validJob()] })
-    )
-    const opts = {
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    }
-    const [a, b] = await Promise.all([
-      fetchRolesForBuild(opts),
-      fetchRolesForBuild(opts)
-    ])
-    expect(a).toBe(b)
-    expect(fetchImpl).toHaveBeenCalledTimes(1)
-  })
-
-  it('never writes to the snapshot file on success', async () => {
-    const snapshot = makeSnapshot()
-    const snapshotUrl = withSnapshotDir(snapshot)
-    const before = new URL(snapshotUrl.href)
-    const fs = await import('node:fs')
-    const initial = fs.readFileSync(before).toString()
-    const fetchImpl = vi.fn(async () =>
-      response({ apiVersion: '1', jobs: [validJob()] })
-    )
-    await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    const after = fs.readFileSync(before).toString()
-    expect(after).toBe(initial)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-
-  it('does not retry on 4xx auth failures for 403', async () => {
-    const snapshotUrl = withSnapshotDir(makeSnapshot())
-    const fetchImpl = vi.fn(async () => response({}, { status: 403 }))
-    await fetchRolesForBuild({
-      apiKey: KEY,
-      boardName: BOARD,
-      baseUrl: BASE_URL,
-      snapshotUrl,
-      fetchImpl: fetchImpl as unknown as typeof fetch
-    })
-    expect(fetchImpl).toHaveBeenCalledTimes(1)
-    rmSync(new URL('.', snapshotUrl), { recursive: true, force: true })
-  })
-})

apps/website/src/utils/ashby.ts

@@ -1,299 +0,0 @@
-import { createHash } from 'node:crypto'
-import { readFile } from 'node:fs/promises'
-
-import type { AshbyJobPosting } from './ashby.schema'
-import type { Department, Role, RolesSnapshot } from '../data/roles'
-
-import bundledSnapshot from '../data/ashby-roles.snapshot.json' with { type: 'json' }
-import {
-  AshbyJobBoardResponseSchema,
-  AshbyJobPostingSchema
-} from './ashby.schema'
-
-const DEFAULT_BASE_URL = 'https://api.ashbyhq.com'
-const DEFAULT_TIMEOUT_MS = 10_000
-const RETRY_DELAYS_MS = [1_000, 2_000, 4_000]
-
-export interface DroppedRole {
-  title: string
-  reason: string
-}
-
-export type FetchOutcome =
-  | {
-      status: 'fresh'
-      snapshot: RolesSnapshot
-      droppedCount: number
-      droppedRoles: DroppedRole[]
-    }
-  | { status: 'stale'; snapshot: RolesSnapshot; reason: string }
-  | { status: 'failed'; reason: string }
-
-interface FetchRolesOptions {
-  apiKey?: string
-  boardName?: string
-  baseUrl?: string
-  timeoutMs?: number
-  retryDelaysMs?: readonly number[]
-  fetchImpl?: typeof fetch
-  snapshotUrl?: URL
-  sleep?: (ms: number) => Promise<void>
-}
-
-let inflight: Promise<FetchOutcome> | undefined
-
-export function resetAshbyFetcherForTests(): void {
-  inflight = undefined
-}
-
-export function fetchRolesForBuild(
-  options: FetchRolesOptions = {}
-): Promise<FetchOutcome> {
-  inflight ??= doFetchRolesForBuild(options)
-  return inflight
-}
-
-async function doFetchRolesForBuild(
-  options: FetchRolesOptions
-): Promise<FetchOutcome> {
-  const apiKey = options.apiKey ?? process.env.WEBSITE_ASHBY_API_KEY
-  const boardName =
-    options.boardName ?? process.env.WEBSITE_ASHBY_JOB_BOARD_NAME
-
-  if (!apiKey || !boardName) {
-    return fallback(
-      'missing WEBSITE_ASHBY_API_KEY or WEBSITE_ASHBY_JOB_BOARD_NAME',
-      options.snapshotUrl
-    )
-  }
-
-  const result = await tryFetchAndParse(apiKey, boardName, options)
-  if (result.kind === 'ok') {
-    return {
-      status: 'fresh',
-      snapshot: {
-        fetchedAt: new Date().toISOString(),
-        departments: result.departments
-      },
-      droppedCount: result.droppedRoles.length,
-      droppedRoles: result.droppedRoles
-    }
-  }
-
-  return fallback(result.reason, options.snapshotUrl)
-}
-
-async function fallback(
-  reason: string,
-  snapshotUrl: URL | undefined
-): Promise<FetchOutcome> {
-  const snapshot = await readSnapshot(snapshotUrl)
-  if (snapshot) return { status: 'stale', snapshot, reason }
-  return { status: 'failed', reason }
-}
-
-interface FetchOk {
-  kind: 'ok'
-  departments: Department[]
-  droppedRoles: DroppedRole[]
-}
-
-interface FetchErr {
-  kind: 'err'
-  reason: string
-}
-
-async function tryFetchAndParse(
-  apiKey: string,
-  boardName: string,
-  options: FetchRolesOptions
-): Promise<FetchOk | FetchErr> {
-  const baseUrl = options.baseUrl ?? DEFAULT_BASE_URL
-  const timeoutMs = options.timeoutMs ?? DEFAULT_TIMEOUT_MS
-  const retryDelaysMs = options.retryDelaysMs ?? RETRY_DELAYS_MS
-  const fetchImpl = options.fetchImpl ?? fetch
-  const sleep = options.sleep ?? defaultSleep
-
-  const url = `${baseUrl}/posting-api/job-board/${encodeURIComponent(
-    boardName
-  )}?includeCompensation=false`
-  const authHeader = `Basic ${Buffer.from(`${apiKey}:`).toString('base64')}`
-
-  let lastReason = 'unknown error'
-  for (let attempt = 0; attempt <= retryDelaysMs.length; attempt++) {
-    if (attempt > 0) await sleep(retryDelaysMs[attempt - 1])
-
-    const response = await callOnce(fetchImpl, url, authHeader, timeoutMs)
-    if (response.kind === 'err') {
-      lastReason = response.reason
-      if (!response.retryable) return response
-      continue
-    }
-
-    const envelope = AshbyJobBoardResponseSchema.safeParse(response.body)
-    if (!envelope.success) {
-      return {
-        kind: 'err',
-        reason: `envelope schema validation failed: ${envelope.error.issues
-          .map((i) => `${i.path.join('.') || '<root>'}: ${i.message}`)
-          .join('; ')}`
-      }
-    }
-
-    return parseRoles(envelope.data.jobs)
-  }
-
-  return { kind: 'err', reason: lastReason }
-}
-
-type CallResponse =
-  | { kind: 'ok'; body: unknown }
-  | { kind: 'err'; reason: string; retryable: boolean }
-
-async function callOnce(
-  fetchImpl: typeof fetch,
-  url: string,
-  authHeader: string,
-  timeoutMs: number
-): Promise<CallResponse> {
-  const controller = new AbortController()
-  const timer = setTimeout(() => controller.abort(), timeoutMs)
-  try {
-    const res = await fetchImpl(url, {
-      method: 'GET',
-      headers: {
-        Authorization: authHeader,
-        Accept: 'application/json; version=1'
-      },
-      signal: controller.signal
-    })
-    if (res.ok) {
-      return { kind: 'ok', body: await res.json() }
-    }
-    const retryable =
-      res.status === 429 || (res.status >= 500 && res.status < 600)
-    return {
-      kind: 'err',
-      reason: `HTTP ${res.status} ${res.statusText || ''}`.trim(),
-      retryable
-    }
-  } catch (error) {
-    const reason =
-      error instanceof Error
-        ? `network error: ${error.message}`
-        : 'network error'
-    return { kind: 'err', reason, retryable: true }
-  } finally {
-    clearTimeout(timer)
-  }
-}
-
-function parseRoles(jobs: readonly unknown[]): FetchOk {
-  const valid: AshbyJobPosting[] = []
-  const droppedRoles: DroppedRole[] = []
-
-  for (const raw of jobs) {
-    const parsed = AshbyJobPostingSchema.safeParse(raw)
-    if (!parsed.success) {
-      droppedRoles.push({
-        title: extractTitle(raw),
-        reason: parsed.error.issues
-          .map((i) => `${i.path.join('.') || '<root>'}: ${i.message}`)
-          .join('; ')
-      })
-      continue
-    }
-    if (!parsed.data.isListed) continue
-    valid.push(parsed.data)
-  }
-
-  return { kind: 'ok', departments: groupByDepartment(valid), droppedRoles }
-}
-
-function extractTitle(raw: unknown): string {
-  if (
-    raw !== null &&
-    typeof raw === 'object' &&
-    'title' in raw &&
-    typeof (raw as { title: unknown }).title === 'string'
-  ) {
-    return (raw as { title: string }).title
-  }
-  return ''
-}
-
-const DEFAULT_DEPARTMENT = 'Other'
-const DEFAULT_LOCATION = 'Remote'
-
-function groupByDepartment(jobs: readonly AshbyJobPosting[]): Department[] {
-  const byKey = new Map<string, Department>()
-  for (const job of jobs) {
-    const displayDepartment = normalizeDepartment(job.department)
-    const name = displayDepartment.toUpperCase()
-    const key = slugify(name)
-    const existing = byKey.get(key)
-    const role = toDomainRole(job, displayDepartment)
-    if (existing) {
-      existing.roles.push(role)
-    } else {
-      byKey.set(key, { name, key, roles: [role] })
-    }
-  }
-  return [...byKey.values()].sort((a, b) => a.name.localeCompare(b.name))
-}
-
-function toDomainRole(job: AshbyJobPosting, department: string): Role {
-  const applyUrl = job.applyUrl ?? job.jobUrl
-  return {
-    id: createHash('sha1').update(applyUrl).digest('hex').slice(0, 16),
-    title: job.title,
-    department: capitalize(department),
-    location: (job.location ?? '').trim() || DEFAULT_LOCATION,
-    applyUrl
-  }
-}
-
-function normalizeDepartment(raw: string | undefined): string {
-  const trimmed = (raw ?? '').trim()
-  return trimmed.length > 0 ? trimmed : DEFAULT_DEPARTMENT
-}
-
-function slugify(value: string): string {
-  return value
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/^-+|-+$/g, '')
-}
-
-function capitalize(value: string): string {
-  return value.charAt(0).toUpperCase() + value.slice(1).toLowerCase()
-}
-
-async function readSnapshot(
-  snapshotUrl: URL | undefined
-): Promise<RolesSnapshot | null> {
-  if (!snapshotUrl) {
-    return isRolesSnapshot(bundledSnapshot) ? bundledSnapshot : null
-  }
-  try {
-    const text = await readFile(snapshotUrl, 'utf8')
-    const parsed: unknown = JSON.parse(text)
-    if (isRolesSnapshot(parsed)) return parsed
-    return null
-  } catch {
-    return null
-  }
-}
-
-function isRolesSnapshot(value: unknown): value is RolesSnapshot {
-  if (value === null || typeof value !== 'object') return false
-  const candidate = value as { fetchedAt?: unknown; departments?: unknown }
-  return (
-    typeof candidate.fetchedAt === 'string' &&
-    Array.isArray(candidate.departments)
-  )
-}
-
-function defaultSleep(ms: number): Promise<void> {
-  return new Promise((resolve) => setTimeout(resolve, ms))
-}

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

@@ -1,36 +0,0 @@
-import { afterEach, describe, expect, it, vi } from 'vitest'
-
-import { fetchGitHubStars, formatStarCount } from './github'
-
-describe('fetchGitHubStars', () => {
-  const savedOverride = process.env.WEBSITE_GITHUB_STARS_OVERRIDE
-
-  afterEach(() => {
-    vi.restoreAllMocks()
-    if (savedOverride === undefined)
-      delete process.env.WEBSITE_GITHUB_STARS_OVERRIDE
-    else process.env.WEBSITE_GITHUB_STARS_OVERRIDE = savedOverride
-  })
-
-  it('uses the build-time override without calling GitHub', async () => {
-    process.env.WEBSITE_GITHUB_STARS_OVERRIDE = '110000'
-    const fetchMock = vi.spyOn(globalThis, 'fetch')
-
-    await expect(fetchGitHubStars('Comfy-Org', 'ComfyUI')).resolves.toBe(110000)
-    expect(fetchMock).not.toHaveBeenCalled()
-  })
-
-  it('fails fast when the build-time override is malformed', async () => {
-    process.env.WEBSITE_GITHUB_STARS_OVERRIDE = '110K'
-
-    await expect(fetchGitHubStars('Comfy-Org', 'ComfyUI')).rejects.toThrow(
-      'WEBSITE_GITHUB_STARS_OVERRIDE must be a non-negative integer'
-    )
-  })
-})
-
-describe('formatStarCount', () => {
-  it('formats the visual-test override to match committed snapshots', () => {
-    expect(formatStarCount(110000)).toBe('110K')
-  })
-})

apps/website/src/utils/github.ts

@@ -2,9 +2,6 @@ export async function fetchGitHubStars(
   owner: string,
   repo: string
 ): Promise<number | null> {
-  const override = readGitHubStarsOverride()
-  if (override !== undefined) return override
-
   try {
     const res = await fetch(`https://api.github.com/repos/${owner}/${repo}`, {
       headers: { Accept: 'application/vnd.github.v3+json' }
@@ -28,17 +25,3 @@ export function formatStarCount(count: number): string {
   }
   return count.toString()
 }
-
-function readGitHubStarsOverride(): number | undefined {
-  const rawCount = process.env.WEBSITE_GITHUB_STARS_OVERRIDE
-  if (rawCount === undefined || rawCount === '') return undefined
-
-  const count = Number(rawCount)
-  if (!Number.isSafeInteger(count) || count < 0) {
-    throw new Error(
-      'WEBSITE_GITHUB_STARS_OVERRIDE must be a non-negative integer'
-    )
-  }
-
-  return count
-}

apps/website/tsconfig.json

@@ -8,10 +8,8 @@
   "include": [
     "src/**/*",
     "e2e/**/*",
-    "scripts/**/*",
     "astro.config.ts",
-    "playwright.config.ts",
-    "vitest.config.ts"
+    "playwright.config.ts"
   ],
   "exclude": ["src/**/*.stories.ts"],
   "references": [{ "path": "./tsconfig.stories.json" }]

apps/website/vitest.config.ts

@@ -1,9 +0,0 @@
-import { defineConfig } from 'vitest/config'
-
-export default defineConfig({
-  test: {
-    environment: 'node',
-    include: ['src/**/*.{test,spec}.ts'],
-    globals: false
-  }
-})

browser_tests/AGENTS.md

@@ -15,15 +15,11 @@ browser_tests/
 │   ├── VueNodeHelpers.ts - Vue Nodes 2.0 helpers
 │   ├── selectors.ts      - Centralized TestIds
 │   ├── data/             - Static test data (mock API responses, workflow JSONs, node definitions)
-│   ├── components/       - Page object classes (locators, user interactions)
-│   │   ├── Actionbar.ts
+│   ├── components/       - Page object components (locators, user interactions)
 │   │   ├── ContextMenu.ts
-│   │   ├── ManageGroupNode.ts
 │   │   ├── SettingDialog.ts
 │   │   ├── SidebarTab.ts
-│   │   ├── Templates.ts
-│   │   ├── Topbar.ts
-│   │   └── ...
+│   │   └── Topbar.ts
 │   ├── helpers/          - Focused helper classes (domain-specific actions)
 │   │   ├── CanvasHelper.ts
 │   │   ├── CommandHelper.ts
@@ -32,36 +28,17 @@ browser_tests/
 │   │   ├── SettingsHelper.ts
 │   │   ├── WorkflowHelper.ts
 │   │   └── ...
-│   └── utils/            - Standalone utility functions (used by tests or fixtures)
-│       ├── builderTestUtils.ts
-│       ├── clipboardSpy.ts
-│       ├── fitToView.ts
-│       ├── perfReporter.ts
-│       └── ...
+│   └── utils/            - Pure utility functions (no page dependency)
+├── helpers/          - Test-specific utilities
 └── tests/            - Test files (*.spec.ts)
 ```
 
 ### Architectural Separation
 
 - **`fixtures/data/`** — Static test data only. Mock API responses, workflow JSONs, node definitions. No code, no imports from Playwright.
-- **`fixtures/components/`** — Page object components. Classes that own locators for a specific UI region (e.g. `Actionbar`, `ContextMenu`, `ManageGroupNode`).
-- **`fixtures/helpers/`** — Helper classes that coordinate actions across multiple regions without owning a locator surface of their own (e.g. `CanvasHelper`, `WorkflowHelper`, `NodeOperationsHelper`).
-- **`fixtures/utils/`** — Standalone utility functions. Exported functions (not classes) used by tests or fixtures (e.g. `fitToView`, `clipboardSpy`, `builderTestUtils`).
-
-### Placement Rule
-
-When adding a new file, use this decision tree:
-
-```mermaid
-flowchart TD
-    A[New file in browser_tests/fixtures/] --> B{Has any code?}
-    B -- No, JSON/data only --> D[fixtures/data/]
-    B -- Yes --> C{Is it a class?}
-    C -- No, exported functions --> U[fixtures/utils/]
-    C -- Yes --> E{Owns locators for a<br/>specific UI region?}
-    E -- Yes --> P[fixtures/components/]
-    E -- No, coordinates actions<br/>across the app --> H[fixtures/helpers/]
-```
+- **`fixtures/components/`** — Page object components. Encapsulate locators and user interactions for a specific UI area.
+- **`fixtures/helpers/`** — Focused helper classes. Domain-specific actions that coordinate multiple page objects (e.g. canvas operations, workflow loading).
+- **`fixtures/utils/`** — Pure utility functions. No `Page` dependency; stateless helpers that can be used anywhere.
 
 ## Page Object Locator Style
 

browser_tests/README.md

@@ -51,7 +51,6 @@ DISABLE_VUE_PLUGINS=true
 # Test against dev server (recommended) or backend directly
 PLAYWRIGHT_TEST_URL=http://localhost:5173  # Dev server
 # PLAYWRIGHT_TEST_URL=http://localhost:8188  # Direct backend
-PLAYWRIGHT_SETUP_API_URL=http://localhost:8188  # Setup/auth API when using the dev server URL above
 
 # Path to ComfyUI for backing up user data/settings before tests
 TEST_COMFYUI_DIR=/path/to/your/ComfyUI
@@ -140,9 +139,12 @@ Always check for existing helpers and fixtures before implementing new ones:
 
 - **ComfyPage**: Main fixture with methods for canvas interaction and node management
 - **ComfyMouse**: Helper for precise mouse operations on the canvas
-- **Component Fixtures**: Check `browser_tests/fixtures/components/` for UI component page objects (e.g. `Actionbar.ts`, `Templates.ts`, `ContextMenu.ts`)
-- **Helper Classes**: Check `browser_tests/fixtures/helpers/` for domain-specific helper classes wired into ComfyPage (e.g. `CanvasHelper.ts`, `WorkflowHelper.ts`)
-- **Utility Functions**: Check `browser_tests/fixtures/utils/` for standalone utilities (e.g. `fitToView.ts`, `clipboardSpy.ts`, `builderTestUtils.ts`)
+- **Helpers**: Check `browser_tests/helpers/` for specialized helpers like:
+  - `actionbar.ts`: Interact with the action bar
+  - `manageGroupNode.ts`: Group node management operations
+  - `templates.ts`: Template workflows operations
+- **Component Fixtures**: Check `browser_tests/fixtures/components/` for UI component helpers
+- **Utility Functions**: Check `browser_tests/utils/` and `browser_tests/fixtures/utils/` for shared utilities
 
 Most common testing needs are already addressed by these helpers, which will make your tests more consistent and reliable.
 

browser_tests/assets/subgraphs/subgraph-multi-instance-promoted-text-values.json

@@ -0,0 +1,284 @@
+{
+  "id": "2f54e2f0-6db4-4bdf-84a8-9c3ea3ec0123",
+  "revision": 0,
+  "last_node_id": 13,
+  "last_link_id": 9,
+  "nodes": [
+    {
+      "id": 11,
+      "type": "422723e8-4bf6-438c-823f-881ca81acead",
+      "pos": [120, 180],
+      "size": [210, 168],
+      "flags": {},
+      "order": 0,
+      "mode": 0,
+      "inputs": [
+        { "name": "clip", "type": "CLIP", "link": null },
+        { "name": "model", "type": "MODEL", "link": null },
+        { "name": "positive", "type": "CONDITIONING", "link": null },
+        { "name": "negative", "type": "CONDITIONING", "link": null },
+        { "name": "latent_image", "type": "LATENT", "link": null }
+      ],
+      "outputs": [],
+      "properties": {},
+      "widgets_values": ["Alpha\n"]
+    },
+    {
+      "id": 12,
+      "type": "422723e8-4bf6-438c-823f-881ca81acead",
+      "pos": [420, 180],
+      "size": [210, 168],
+      "flags": {},
+      "order": 1,
+      "mode": 0,
+      "inputs": [
+        { "name": "clip", "type": "CLIP", "link": null },
+        { "name": "model", "type": "MODEL", "link": null },
+        { "name": "positive", "type": "CONDITIONING", "link": null },
+        { "name": "negative", "type": "CONDITIONING", "link": null },
+        { "name": "latent_image", "type": "LATENT", "link": null }
+      ],
+      "outputs": [],
+      "properties": {},
+      "widgets_values": ["Beta\n"]
+    },
+    {
+      "id": 13,
+      "type": "422723e8-4bf6-438c-823f-881ca81acead",
+      "pos": [720, 180],
+      "size": [210, 168],
+      "flags": {},
+      "order": 2,
+      "mode": 0,
+      "inputs": [
+        { "name": "clip", "type": "CLIP", "link": null },
+        { "name": "model", "type": "MODEL", "link": null },
+        { "name": "positive", "type": "CONDITIONING", "link": null },
+        { "name": "negative", "type": "CONDITIONING", "link": null },
+        { "name": "latent_image", "type": "LATENT", "link": null }
+      ],
+      "outputs": [],
+      "properties": {},
+      "widgets_values": ["Gamma\n"]
+    }
+  ],
+  "links": [],
+  "groups": [],
+  "definitions": {
+    "subgraphs": [
+      {
+        "id": "422723e8-4bf6-438c-823f-881ca81acead",
+        "version": 1,
+        "state": {
+          "lastGroupId": 0,
+          "lastNodeId": 11,
+          "lastLinkId": 15,
+          "lastRerouteId": 0
+        },
+        "revision": 0,
+        "config": {},
+        "name": "New Subgraph",
+        "inputNode": {
+          "id": -10,
+          "bounding": [481.59912109375, 379.13336181640625, 120, 160]
+        },
+        "outputNode": {
+          "id": -20,
+          "bounding": [1121.59912109375, 379.13336181640625, 120, 40]
+        },
+        "inputs": [
+          {
+            "id": "0f07c10e-5705-4764-9b24-b69606c6dbcc",
+            "name": "text",
+            "type": "STRING",
+            "linkIds": [10],
+            "pos": { "0": 581.59912109375, "1": 399.13336181640625 }
+          },
+          {
+            "id": "214a5060-24dd-4299-ab78-8027dc5b9c59",
+            "name": "clip",
+            "type": "CLIP",
+            "linkIds": [11],
+            "pos": { "0": 581.59912109375, "1": 419.13336181640625 }
+          },
+          {
+            "id": "8ab94c5d-e7df-433c-9177-482a32340552",
+            "name": "model",
+            "type": "MODEL",
+            "linkIds": [12],
+            "pos": { "0": 581.59912109375, "1": 439.13336181640625 }
+          },
+          {
+            "id": "8a4cd719-8c67-473b-9b44-ac0582d02641",
+            "name": "positive",
+            "type": "CONDITIONING",
+            "linkIds": [13],
+            "pos": { "0": 581.59912109375, "1": 459.13336181640625 }
+          },
+          {
+            "id": "a78d6b3a-ad40-4300-b0a5-2cdbdb8dc135",
+            "name": "negative",
+            "type": "CONDITIONING",
+            "linkIds": [14],
+            "pos": { "0": 581.59912109375, "1": 479.13336181640625 }
+          },
+          {
+            "id": "4c7abe0c-902d-49ef-a5b0-cbf02b50b693",
+            "name": "latent_image",
+            "type": "LATENT",
+            "linkIds": [15],
+            "pos": { "0": 581.59912109375, "1": 499.13336181640625 }
+          }
+        ],
+        "outputs": [],
+        "widgets": [],
+        "nodes": [
+          {
+            "id": 10,
+            "type": "CLIPTextEncode",
+            "pos": [661.59912109375, 314.13336181640625],
+            "size": [400, 200],
+            "flags": {},
+            "order": 1,
+            "mode": 0,
+            "inputs": [
+              {
+                "localized_name": "clip",
+                "name": "clip",
+                "type": "CLIP",
+                "link": 11
+              },
+              {
+                "localized_name": "text",
+                "name": "text",
+                "type": "STRING",
+                "widget": { "name": "text" },
+                "link": 10
+              }
+            ],
+            "outputs": [
+              {
+                "localized_name": "CONDITIONING",
+                "name": "CONDITIONING",
+                "type": "CONDITIONING",
+                "links": null
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "CLIPTextEncode"
+            },
+            "widgets_values": [""]
+          },
+          {
+            "id": 11,
+            "type": "KSampler",
+            "pos": [674.1234741210938, 570.5839233398438],
+            "size": [270, 262],
+            "flags": {},
+            "order": 0,
+            "mode": 0,
+            "inputs": [
+              {
+                "localized_name": "model",
+                "name": "model",
+                "type": "MODEL",
+                "link": 12
+              },
+              {
+                "localized_name": "positive",
+                "name": "positive",
+                "type": "CONDITIONING",
+                "link": 13
+              },
+              {
+                "localized_name": "negative",
+                "name": "negative",
+                "type": "CONDITIONING",
+                "link": 14
+              },
+              {
+                "localized_name": "latent_image",
+                "name": "latent_image",
+                "type": "LATENT",
+                "link": 15
+              }
+            ],
+            "outputs": [
+              {
+                "localized_name": "LATENT",
+                "name": "LATENT",
+                "type": "LATENT",
+                "links": null
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "KSampler"
+            },
+            "widgets_values": [0, "randomize", 20, 8, "euler", "simple", 1]
+          }
+        ],
+        "groups": [],
+        "links": [
+          {
+            "id": 10,
+            "origin_id": -10,
+            "origin_slot": 0,
+            "target_id": 10,
+            "target_slot": 1,
+            "type": "STRING"
+          },
+          {
+            "id": 11,
+            "origin_id": -10,
+            "origin_slot": 1,
+            "target_id": 10,
+            "target_slot": 0,
+            "type": "CLIP"
+          },
+          {
+            "id": 12,
+            "origin_id": -10,
+            "origin_slot": 2,
+            "target_id": 11,
+            "target_slot": 0,
+            "type": "MODEL"
+          },
+          {
+            "id": 13,
+            "origin_id": -10,
+            "origin_slot": 3,
+            "target_id": 11,
+            "target_slot": 1,
+            "type": "CONDITIONING"
+          },
+          {
+            "id": 14,
+            "origin_id": -10,
+            "origin_slot": 4,
+            "target_id": 11,
+            "target_slot": 2,
+            "type": "CONDITIONING"
+          },
+          {
+            "id": 15,
+            "origin_id": -10,
+            "origin_slot": 5,
+            "target_id": 11,
+            "target_slot": 3,
+            "type": "LATENT"
+          }
+        ],
+        "extra": {}
+      }
+    ]
+  },
+  "config": {},
+  "extra": {
+    "ds": {
+      "scale": 1,
+      "offset": [0, 0]
+    },
+    "frontendVersion": "1.24.1"
+  },
+  "version": 0.4
+}

browser_tests/assets/widgets/painter_with_input.json

@@ -1,74 +0,0 @@
-{
-  "last_node_id": 2,
-  "last_link_id": 1,
-  "nodes": [
-    {
-      "id": 2,
-      "type": "LoadImage",
-      "pos": [50, 50],
-      "size": [400, 314],
-      "flags": {},
-      "order": 0,
-      "mode": 0,
-      "inputs": [],
-      "outputs": [
-        {
-          "name": "IMAGE",
-          "type": "IMAGE",
-          "links": [1]
-        },
-        {
-          "name": "MASK",
-          "type": "MASK",
-          "links": null
-        }
-      ],
-      "properties": {
-        "Node name for S&R": "LoadImage"
-      },
-      "widgets_values": ["example.png", "image"]
-    },
-    {
-      "id": 1,
-      "type": "Painter",
-      "pos": [450, 50],
-      "size": [450, 550],
-      "flags": {},
-      "order": 1,
-      "mode": 0,
-      "inputs": [
-        {
-          "name": "image",
-          "type": "IMAGE",
-          "link": 1
-        }
-      ],
-      "outputs": [
-        {
-          "name": "IMAGE",
-          "type": "IMAGE",
-          "links": null
-        },
-        {
-          "name": "MASK",
-          "type": "MASK",
-          "links": null
-        }
-      ],
-      "properties": {
-        "Node name for S&R": "Painter"
-      },
-      "widgets_values": ["", 512, 512, "#000000"]
-    }
-  ],
-  "links": [[1, 2, 0, 1, 0, "IMAGE"]],
-  "groups": [],
-  "config": {},
-  "extra": {
-    "ds": {
-      "offset": [0, 0],
-      "scale": 1
-    }
-  },
-  "version": 0.4
-}

browser_tests/fixtures/ComfyMouse.ts

@@ -1,4 +1,4 @@
-import type { Locator, Mouse } from '@playwright/test'
+import type { Mouse } from '@playwright/test'
 
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import type { Position } from '@e2e/fixtures/types'
@@ -72,22 +72,6 @@ export class ComfyMouse implements Omit<Mouse, 'move'> {
     await this.nextFrame()
   }
 
-  async resizeByDragging(
-    element: Locator,
-    { x, y }: { x?: number; y?: number }
-  ) {
-    const elementBox = await element.boundingBox()
-    if (!elementBox) throw new Error('element should have layout')
-
-    const cx = elementBox.x + elementBox.width / 2
-    const cy = elementBox.y + elementBox.height / 2
-
-    await this.dragAndDrop(
-      { x: cx, y: cy },
-      { x: cx + (x ?? 0), y: cy + (y ?? 0) }
-    )
-  }
-
   //#region Pass-through
   async click(...args: Parameters<Mouse['click']>) {
     return await this.mouse.click(...args)

browser_tests/fixtures/ComfyPage.ts

@@ -5,8 +5,8 @@ import MCR from 'monocart-coverage-reports'
 
 import { COVERAGE_OUTPUT_DIR } from '@e2e/coverageConfig'
 import { NodeBadgeMode } from '@/types/nodeSource'
-import { ComfyActionbar } from '@e2e/fixtures/components/Actionbar'
-import { ComfyTemplates } from '@e2e/fixtures/components/Templates'
+import { ComfyActionbar } from '@e2e/helpers/actionbar'
+import { ComfyTemplates } from '@e2e/helpers/templates'
 import { ComfyMouse } from '@e2e/fixtures/ComfyMouse'
 import { TestIds } from '@e2e/fixtures/selectors'
 import { comfyExpect } from '@e2e/fixtures/utils/customMatchers'
@@ -22,7 +22,6 @@ import { MediaLightbox } from '@e2e/fixtures/components/MediaLightbox'
 import { QueuePanel } from '@e2e/fixtures/components/QueuePanel'
 import { SettingDialog } from '@e2e/fixtures/components/SettingDialog'
 import { TemplatesDialog } from '@e2e/fixtures/components/TemplatesDialog'
-import { TitleEditor } from '@e2e/fixtures/components/TitleEditor'
 import {
   AssetsSidebarTab,
   ModelLibrarySidebarTab,
@@ -55,13 +54,11 @@ class ComfyPropertiesPanel {
   readonly root: Locator
   readonly panelTitle: Locator
   readonly searchBox: Locator
-  readonly titleEditor: TitleEditor
 
   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)
   }
 }
 
@@ -140,7 +137,6 @@ class ComfyMenu {
 
 export class ComfyPage {
   public readonly url: string
-  public readonly apiUrl: string
   // All canvas position operations are based on default view of canvas.
   public readonly canvas: Locator
   public readonly selectionToolbox: Locator
@@ -163,7 +159,6 @@ export class ComfyPage {
   public readonly settingDialog: SettingDialog
   public readonly confirmDialog: ConfirmDialog
   public readonly templatesDialog: TemplatesDialog
-  public readonly titleEditor: TitleEditor
   public readonly mediaLightbox: MediaLightbox
   public readonly vueNodes: VueNodeHelpers
   public readonly appMode: AppModeHelper
@@ -200,7 +195,6 @@ export class ComfyPage {
     public readonly request: APIRequestContext
   ) {
     this.url = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-    this.apiUrl = process.env.PLAYWRIGHT_SETUP_API_URL || this.url
     this.canvas = page.locator('#graph-canvas')
     this.selectionToolbox = page.getByTestId(TestIds.selectionToolbox.root)
     this.widgetTextBox = page.getByPlaceholder('text').nth(1)
@@ -210,14 +204,13 @@ export class ComfyPage {
     this.workflowUploadInput = page.locator('#comfy-file-input')
 
     this.searchBox = new ComfyNodeSearchBox(page)
-    this.searchBoxV2 = new ComfyNodeSearchBoxV2(this)
+    this.searchBoxV2 = new ComfyNodeSearchBoxV2(page)
     this.menu = new ComfyMenu(page)
     this.actionbar = new ComfyActionbar(page)
     this.templates = new ComfyTemplates(page)
     this.settingDialog = new SettingDialog(page, this)
     this.confirmDialog = new ConfirmDialog(page)
     this.templatesDialog = new TemplatesDialog(page)
-    this.titleEditor = new TitleEditor(page)
     this.mediaLightbox = new MediaLightbox(page)
     this.vueNodes = new VueNodeHelpers(page)
     this.appMode = new AppModeHelper(this)
@@ -243,7 +236,7 @@ export class ComfyPage {
   }
 
   async setupUser(username: string) {
-    const res = await this.request.get(`${this.apiUrl}/api/users`)
+    const res = await this.request.get(`${this.url}/api/users`)
     if (res.status() !== 200)
       throw new Error(`Failed to retrieve users: ${await res.text()}`)
 
@@ -257,7 +250,7 @@ export class ComfyPage {
   }
 
   async createUser(username: string) {
-    const resp = await this.request.post(`${this.apiUrl}/api/users`, {
+    const resp = await this.request.post(`${this.url}/api/users`, {
       data: { username }
     })
 
@@ -269,7 +262,7 @@ export class ComfyPage {
 
   async setupSettings(settings: Record<string, unknown>) {
     const resp = await this.request.post(
-      `${this.apiUrl}/api/devtools/set_settings`,
+      `${this.url}/api/devtools/set_settings`,
       {
         data: settings
       }

browser_tests/fixtures/components/ComfyNodeSearchBoxV2.ts

@@ -1,112 +1,29 @@
-import type { Locator } from '@playwright/test'
+import type { Locator, Page } from '@playwright/test'
 
-import type { RootCategoryId } from '@/components/searchbox/v2/rootCategories'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { TestIds } from '@e2e/fixtures/selectors'
-
-const { searchBoxV2 } = TestIds
-
-export type { RootCategoryId }
 
 export class ComfyNodeSearchBoxV2 {
   readonly dialog: Locator
   readonly input: Locator
-  readonly filterSearch: Locator
   readonly results: Locator
   readonly filterOptions: Locator
-  readonly filterChips: Locator
-  readonly noResults: Locator
-  readonly nodeIdBadge: Locator
-  readonly sidebarToggle: Locator
-  readonly sidebarBackdrop: Locator
-  readonly filterChipsScroll: Locator
 
-  constructor(private comfyPage: ComfyPage) {
-    const page = comfyPage.page
+  constructor(readonly page: Page) {
     this.dialog = page.getByRole('search')
-    this.input = this.dialog.getByRole('combobox')
-    this.filterSearch = this.dialog.getByRole('textbox', { name: 'Search' })
-    this.results = this.dialog.getByTestId(searchBoxV2.resultItem)
-    this.filterOptions = this.dialog.getByTestId(searchBoxV2.filterOption)
-    this.filterChips = this.dialog.getByTestId(searchBoxV2.filterChip)
-    this.noResults = this.dialog.getByTestId(searchBoxV2.noResults)
-    this.nodeIdBadge = this.dialog.getByTestId(searchBoxV2.nodeIdBadge)
-    this.sidebarToggle = this.dialog.getByTestId(searchBoxV2.sidebarToggle)
-    this.sidebarBackdrop = this.dialog.getByTestId(searchBoxV2.sidebarBackdrop)
-    this.filterChipsScroll = this.dialog.getByTestId(
-      searchBoxV2.filterChipsScroll
-    )
+    this.input = this.dialog.locator('input[type="text"]')
+    this.results = this.dialog.getByTestId('result-item')
+    this.filterOptions = this.dialog.getByTestId('filter-option')
   }
 
-  /** Sidebar category tree button (e.g. `sampling`, `sampling/custom_sampling`). */
   categoryButton(categoryId: string): Locator {
-    return this.dialog.getByTestId(searchBoxV2.category(categoryId))
+    return this.dialog.getByTestId(`category-${categoryId}`)
   }
 
-  /** Top filter-bar root category chip (e.g. `comfy`, `essentials`). */
-  rootCategoryButton(id: RootCategoryId): Locator {
-    return this.dialog.getByTestId(searchBoxV2.rootCategory(id))
+  filterBarButton(name: string): Locator {
+    return this.dialog.getByRole('button', { name })
   }
 
-  /** Top filter-bar input/output type popover trigger. */
-  typeFilterButton(key: 'input' | 'output'): Locator {
-    return this.dialog.getByTestId(searchBoxV2.typeFilter(key))
-  }
-
-  async applyTypeFilter(
-    key: 'input' | 'output',
-    typeName: string
-  ): Promise<void> {
-    const trigger = this.typeFilterButton(key)
-    await trigger.click()
-    await this.filterOptions.first().waitFor({ state: 'visible' })
-    await this.filterSearch.fill(typeName)
-    await this.filterOptions.filter({ hasText: typeName }).first().click()
-    // The popover does not auto-close on selection — toggle the trigger.
-    await trigger.click()
-    await this.filterOptions.first().waitFor({ state: 'hidden' })
-  }
-
-  async removeFilterChip(index = 0): Promise<void> {
-    await this.filterChips
-      .nth(index)
-      .getByTestId(searchBoxV2.chipDelete)
-      .click()
-  }
-
-  async toggle(): Promise<void> {
-    await this.comfyPage.command.executeCommand('Workspace.SearchBox.Toggle')
-  }
-
-  async open(): Promise<void> {
-    if (await this.input.isVisible()) return
-    await this.toggle()
-    await this.input.waitFor({ state: 'visible' })
-  }
-
-  async openByDoubleClickCanvas(): Promise<void> {
-    // Use page.mouse.dblclick (not canvas.dblclick) so the z-999 Vue overlay
-    // does not intercept; coords target a viewport spot that is on the canvas
-    // and clear of both the side toolbar and any default-graph nodes.
-    await this.comfyPage.page.mouse.dblclick(200, 200, { delay: 5 })
-  }
-
-  async ensureV2Search(): Promise<void> {
-    await this.comfyPage.settings.setSetting(
-      'Comfy.NodeSearchBoxImpl',
-      'default'
-    )
-  }
-
-  async setup(): Promise<void> {
-    await this.ensureV2Search()
-    await this.comfyPage.settings.setSetting(
-      'Comfy.LinkRelease.Action',
-      'search box'
-    )
-    await this.comfyPage.settings.setSetting(
-      'Comfy.LinkRelease.ActionShift',
-      'search box'
-    )
+  async reload(comfyPage: ComfyPage) {
+    await comfyPage.settings.setSetting('Comfy.NodeSearchBoxImpl', 'default')
   }
 }

browser_tests/fixtures/components/ContextMenu.ts

@@ -4,13 +4,11 @@ import type { Locator, Page } from '@playwright/test'
 export class ContextMenu {
   public readonly primeVueMenu: Locator
   public readonly litegraphMenu: Locator
-  public readonly litegraphContextMenu: Locator
   public readonly menuItems: Locator
 
   constructor(public readonly page: Page) {
     this.primeVueMenu = page.locator('.p-contextmenu, .p-menu')
     this.litegraphMenu = page.locator('.litemenu')
-    this.litegraphContextMenu = page.locator('.litecontextmenu')
     this.menuItems = page.locator('.p-menuitem, .litemenu-entry')
   }
 
@@ -41,10 +39,7 @@ export class ContextMenu {
     const litegraphVisible = await this.litegraphMenu
       .isVisible()
       .catch(() => false)
-    const litegraphContextVisible = await this.litegraphContextMenu
-      .isVisible()
-      .catch(() => false)
-    return primeVueVisible || litegraphVisible || litegraphContextVisible
+    return primeVueVisible || litegraphVisible
   }
 
   async assertHasItems(items: string[]): Promise<void> {
@@ -76,8 +71,7 @@ export class ContextMenu {
   async waitForHidden(): Promise<void> {
     await Promise.all([
       this.primeVueMenu.waitFor({ state: 'hidden' }),
-      this.litegraphMenu.waitFor({ state: 'hidden' }),
-      this.litegraphContextMenu.waitFor({ state: 'hidden' })
+      this.litegraphMenu.waitFor({ state: 'hidden' })
     ])
   }
 }

browser_tests/fixtures/components/PublishDialog.ts

@@ -1,72 +0,0 @@
-import type { Locator, Page } from '@playwright/test'
-
-import { BaseDialog } from '@e2e/fixtures/components/BaseDialog'
-import { TestIds } from '@e2e/fixtures/selectors'
-
-export class PublishDialog extends BaseDialog {
-  readonly nav: Locator
-  readonly footer: Locator
-  readonly savePrompt: Locator
-  readonly describeStep: Locator
-  readonly finishStep: Locator
-  readonly profilePrompt: Locator
-  readonly gateFlow: Locator
-  readonly nameInput: Locator
-  readonly descriptionTextarea: Locator
-  readonly tagsInput: Locator
-  readonly backButton: Locator
-  readonly nextButton: Locator
-  readonly publishButton: Locator
-
-  constructor(page: Page) {
-    super(page, TestIds.publish.dialog)
-    this.nav = this.root.getByTestId(TestIds.publish.nav)
-    this.footer = this.root.getByTestId(TestIds.publish.footer)
-    this.savePrompt = this.root.getByTestId(TestIds.publish.savePrompt)
-    this.describeStep = this.root.getByTestId(TestIds.publish.describeStep)
-    this.finishStep = this.root.getByTestId(TestIds.publish.finishStep)
-    this.profilePrompt = this.root.getByTestId(TestIds.publish.profilePrompt)
-    this.gateFlow = this.root.getByTestId(TestIds.publish.gateFlow)
-    this.nameInput = this.root.getByTestId(TestIds.publish.nameInput)
-    this.descriptionTextarea = this.describeStep.locator('textarea')
-    this.tagsInput = this.root.getByTestId(TestIds.publish.tagsInput)
-    this.backButton = this.footer.getByRole('button', { name: 'Back' })
-    this.nextButton = this.footer.getByRole('button', { name: 'Next' })
-    this.publishButton = this.footer.getByRole('button', {
-      name: 'Publish to ComfyHub'
-    })
-  }
-
-  // Uses showPublishDialog() via Vite-bundled lazy imports that work in both
-  // dev and production, rather than clicking through the UI.
-  async open(): Promise<void> {
-    await this.page.evaluate(async () => {
-      await window.app!.extensionManager.dialog.showPublishDialog()
-    })
-    await this.waitForVisible()
-  }
-
-  tagSuggestion(name: string): Locator {
-    return this.describeStep.getByText(name, { exact: true })
-  }
-
-  navStep(label: string): Locator {
-    return this.nav.getByRole('button', { name: label })
-  }
-
-  currentNavStep(): Locator {
-    return this.nav.locator('[aria-current="step"]')
-  }
-
-  async goNext(): Promise<void> {
-    await this.nextButton.click()
-  }
-
-  async goBack(): Promise<void> {
-    await this.backButton.click()
-  }
-
-  async goToStep(label: string): Promise<void> {
-    await this.navStep(label).click()
-  }
-}

browser_tests/fixtures/components/SidebarTab.ts

@@ -250,26 +250,6 @@ export class ModelLibrarySidebarTab extends SidebarTab {
   }
 }
 
-type MediaFilterKind = 'image' | 'video' | 'audio' | '3d'
-type MediaFilterLabel = 'Image' | 'Video' | 'Audio' | '3D'
-
-function getMediaFilterLabel(
-  filter: MediaFilterKind | MediaFilterLabel
-): MediaFilterLabel {
-  switch (filter) {
-    case 'image':
-      return 'Image'
-    case 'video':
-      return 'Video'
-    case 'audio':
-      return 'Audio'
-    case '3d':
-      return '3D'
-    default:
-      return filter
-  }
-}
-
 export class AssetsSidebarTab extends SidebarTab {
   // --- Tab navigation ---
   public readonly generatedTab: Locator
@@ -281,13 +261,6 @@ export class AssetsSidebarTab extends SidebarTab {
   // --- Search & filter ---
   public readonly searchInput: Locator
   public readonly settingsButton: Locator
-  public readonly filterButton: Locator
-
-  // --- Filter menu checkboxes (cloud-only, shown inside filter popover) ---
-  public readonly filterImageCheckbox: Locator
-  public readonly filterVideoCheckbox: Locator
-  public readonly filterAudioCheckbox: Locator
-  public readonly filter3DCheckbox: Locator
 
   // --- View mode ---
   public readonly listViewOption: Locator
@@ -296,8 +269,6 @@ export class AssetsSidebarTab extends SidebarTab {
   // --- Sort options (cloud-only, shown inside settings popover) ---
   public readonly sortNewestFirst: Locator
   public readonly sortOldestFirst: Locator
-  public readonly sortLongestFirst: Locator
-  public readonly sortFastestFirst: Locator
 
   // --- Asset cards ---
   public readonly assetCards: Locator
@@ -328,17 +299,10 @@ export class AssetsSidebarTab extends SidebarTab {
     )
     this.searchInput = page.getByPlaceholder('Search Assets...')
     this.settingsButton = page.getByRole('button', { name: 'View settings' })
-    this.filterButton = page.getByRole('button', { name: 'Filter by' })
-    this.filterImageCheckbox = page.getByRole('checkbox', { name: 'Image' })
-    this.filterVideoCheckbox = page.getByRole('checkbox', { name: 'Video' })
-    this.filterAudioCheckbox = page.getByRole('checkbox', { name: 'Audio' })
-    this.filter3DCheckbox = page.getByRole('checkbox', { name: '3D' })
     this.listViewOption = page.getByText('List view')
     this.gridViewOption = page.getByText('Grid view')
     this.sortNewestFirst = page.getByText('Newest first')
     this.sortOldestFirst = page.getByText('Oldest first')
-    this.sortLongestFirst = page.getByText('Generation time (longest first)')
-    this.sortFastestFirst = page.getByText('Generation time (fastest first)')
     this.assetCards = page
       .getByRole('button')
       .and(page.locator('[data-selected]'))
@@ -370,12 +334,6 @@ export class AssetsSidebarTab extends SidebarTab {
     return this.page.getByText(title)
   }
 
-  filterCheckbox(filter: MediaFilterKind | MediaFilterLabel) {
-    return this.page.getByRole('checkbox', {
-      name: getMediaFilterLabel(filter)
-    })
-  }
-
   getAssetCardByName(name: string) {
     return this.assetCards.filter({ hasText: name })
   }
@@ -425,29 +383,6 @@ export class AssetsSidebarTab extends SidebarTab {
       .waitFor({ state: 'visible', timeout: 3000 })
   }
 
-  async openFilterMenu() {
-    await this.dismissToasts()
-    await this.filterButton.click()
-    await this.filterCheckbox('Image').waitFor({
-      state: 'visible',
-      timeout: 3000
-    })
-  }
-
-  async toggleMediaTypeFilter(
-    filter: MediaFilterKind | MediaFilterLabel
-  ): Promise<void> {
-    const checkbox = this.filterCheckbox(filter)
-    const before = await checkbox.getAttribute('aria-checked')
-    await checkbox.click()
-    const expected = before === 'true' ? 'false' : 'true'
-    await expect(checkbox).toHaveAttribute('aria-checked', expected)
-  }
-
-  async getAssetCardOrder(): Promise<string[]> {
-    return await this.assetCards.allInnerTexts()
-  }
-
   async rightClickAsset(name: string) {
     const card = this.getAssetCardByName(name)
     await card.click({ button: 'right' })

browser_tests/fixtures/components/SubgraphBreadcrumbPanel.ts

@@ -10,8 +10,6 @@ export class SubgraphBreadcrumbPanel {
   readonly activeItem: Locator
   readonly missingNodesIcon: Locator
   readonly blueprintTag: Locator
-  readonly rootItem: Locator
-  readonly rootBlueprintTag: Locator
 
   constructor(public readonly page: Page) {
     this.root = page.getByTestId(TestIds.breadcrumb.subgraph)
@@ -25,10 +23,10 @@ export class SubgraphBreadcrumbPanel {
       TestIds.breadcrumb.missingNodesIcon
     )
     this.blueprintTag = this.root.getByTestId(TestIds.breadcrumb.blueprintTag)
-    this.rootItem = page.getByTestId(TestIds.breadcrumb.item('root'))
-    this.rootBlueprintTag = this.rootItem.getByTestId(
-      TestIds.breadcrumb.blueprintTag
-    )
+  }
+
+  rootItem(): Locator {
+    return this.page.getByTestId(TestIds.breadcrumb.item('root'))
   }
 
   subgraphItem(subgraphId: string): Locator {

browser_tests/fixtures/components/TitleEditor.ts

@@ -1,33 +0,0 @@
-import type { Locator, Page } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import { TestIds } from '@e2e/fixtures/selectors'
-
-/**
- * The node/group title-editing input. Rendered in three scopes: the canvas
- * overlay (page-wide), the properties panel, and the Vue node itself.
- */
-export class TitleEditor {
-  public readonly input: Locator
-
-  constructor(scope: Page | Locator) {
-    this.input = scope.getByTestId(TestIds.node.titleInput)
-  }
-
-  async setTitle(title: string): Promise<void> {
-    await this.input.fill(title)
-    await this.input.press('Enter')
-  }
-
-  async cancel(): Promise<void> {
-    await this.input.press('Escape')
-  }
-
-  async expectVisible(): Promise<void> {
-    await expect(this.input).toBeVisible()
-  }
-
-  async expectHidden(): Promise<void> {
-    await expect(this.input).toBeHidden()
-  }
-}

browser_tests/fixtures/components/WidgetBoundingBox.ts

@@ -1,54 +0,0 @@
-import type { Locator } from '@playwright/test'
-
-import { TestIds } from '@e2e/fixtures/selectors'
-
-class BoundingBoxCoordinate {
-  public readonly root: Locator
-  public readonly input: Locator
-  public readonly incrementButton: Locator
-  public readonly decrementButton: Locator
-
-  constructor(root: Locator) {
-    this.root = root
-    this.input = root.locator('input')
-    this.incrementButton = root.getByTestId(TestIds.widgets.increment)
-    this.decrementButton = root.getByTestId(TestIds.widgets.decrement)
-  }
-
-  async type(value: string | number): Promise<void> {
-    await this.input.fill(String(value))
-    await this.input.press('Enter')
-  }
-
-  async focus(): Promise<void> {
-    await this.input.focus()
-  }
-
-  async increment(): Promise<void> {
-    await this.incrementButton.click()
-  }
-
-  async decrement(): Promise<void> {
-    await this.decrementButton.click()
-  }
-}
-
-export class WidgetBoundingBoxFixture {
-  public readonly root: Locator
-  public readonly x: BoundingBoxCoordinate
-  public readonly y: BoundingBoxCoordinate
-  public readonly width: BoundingBoxCoordinate
-  public readonly height: BoundingBoxCoordinate
-
-  constructor(parent: Locator) {
-    this.root = parent.getByTestId('bounding-box')
-    this.x = new BoundingBoxCoordinate(this.root.getByTestId('bounding-box-x'))
-    this.y = new BoundingBoxCoordinate(this.root.getByTestId('bounding-box-y'))
-    this.width = new BoundingBoxCoordinate(
-      this.root.getByTestId('bounding-box-width')
-    )
-    this.height = new BoundingBoxCoordinate(
-      this.root.getByTestId('bounding-box-height')
-    )
-  }
-}

browser_tests/fixtures/helpers/AssetsHelper.ts

@@ -1,72 +1,32 @@
 import type { Page, Route } from '@playwright/test'
-import type {
-  CreateAssetExportData,
-  CreateAssetExportResponse,
-  JobsListResponse,
-  ListAssetsResponse
-} from '@comfyorg/ingest-types'
+import type { JobsListResponse } from '@comfyorg/ingest-types'
 
-import type {
-  JobDetail,
-  RawJobListItem
-} from '@/platform/remote/comfyui/jobs/jobTypes'
+import type { RawJobListItem } from '@/platform/remote/comfyui/jobs/jobTypes'
 
-const jobsListRoutePattern = '**/api/jobs?*'
-const assetsListRoutePattern = /\/api\/assets(?:\?.*)?$/
-const assetExportRoutePattern = '**/api/assets/export'
+const jobsListRoutePattern = /\/api\/jobs(?:\?.*)?$/
 const inputFilesRoutePattern = /\/internal\/files\/input(?:\?.*)?$/
 const historyRoutePattern = /\/api\/history$/
 
-/**
- * Media kinds supported by the assets sidebar filter UI. The string values
- * match what the backend stores on `preview_output.mediaType` (`images` is
- * intentionally plural to match existing API conventions; the others are
- * singular as emitted by `useMediaAssetGalleryStore`).
- *
- * The sidebar filter ultimately matches on the filename extension, so the
- * fixture also picks an extension-appropriate filename for each kind.
- */
-export type MediaKindFixture = 'images' | 'video' | 'audio' | '3D'
-
-const DEFAULT_EXTENSION: Record<MediaKindFixture, string> = {
-  images: 'png',
-  video: 'mp4',
-  audio: 'wav',
-  '3D': 'glb'
-}
-
 /** Factory to create a mock completed job with preview output. */
 export function createMockJob(
-  overrides: Partial<RawJobListItem> & {
-    id: string
-    /**
-     * Optional shorthand to set both `preview_output.mediaType` and an
-     * extension-appropriate filename. Ignored when `preview_output` is also
-     * supplied via `overrides`.
-     */
-    mediaKind?: MediaKindFixture
-  }
+  overrides: Partial<RawJobListItem> & { id: string }
 ): RawJobListItem {
-  const { mediaKind, ...rest } = overrides
   const now = Date.now()
-  const extension = mediaKind ? DEFAULT_EXTENSION[mediaKind] : 'png'
-  const mediaType = mediaKind ?? 'images'
-
   return {
     status: 'completed',
     create_time: now,
     execution_start_time: now,
     execution_end_time: now + 5000,
     preview_output: {
-      filename: `output_${rest.id}.${extension}`,
+      filename: `output_${overrides.id}.png`,
       subfolder: '',
       type: 'output',
       nodeId: '1',
-      mediaType
+      mediaType: 'images'
     },
     outputs_count: 1,
     priority: 0,
-    ...rest
+    ...overrides
   }
 }
 
@@ -94,46 +54,6 @@ export function createMockJobs(
   )
 }
 
-/**
- * Create one job per requested media kind, in the order supplied. Jobs share
- * a stable timestamp ordering (newer first) so callers can rely on the result
- * order when mediaType filters are inactive.
- */
-export function createMixedMediaJobs(
-  kinds: MediaKindFixture[]
-): RawJobListItem[] {
-  const now = Date.now()
-  return kinds.map((kind, i) =>
-    createMockJob({
-      id: `${kind}-${String(i + 1).padStart(3, '0')}`,
-      mediaKind: kind,
-      create_time: now - i * 60_000,
-      execution_start_time: now - i * 60_000,
-      execution_end_time: now - i * 60_000 + 5000
-    })
-  )
-}
-
-/**
- * Create jobs with explicit `(create_time, execution duration)` pairs so that
- * sort assertions for newest/oldest and longest/fastest are unambiguous.
- *
- * Each spec entry yields a job whose `execution_end_time - execution_start_time`
- * equals `durationMs`. The first spec becomes id `job-001`, etc.
- */
-export function createJobsWithExecutionTimes(
-  specs: ReadonlyArray<{ createTime: number; durationMs: number }>
-): RawJobListItem[] {
-  return specs.map((spec, i) =>
-    createMockJob({
-      id: `job-${String(i + 1).padStart(3, '0')}`,
-      create_time: spec.createTime,
-      execution_start_time: spec.createTime,
-      execution_end_time: spec.createTime + spec.durationMs
-    })
-  )
-}
-
 /** Create mock imported file names with various media types. */
 export function createMockImportedFiles(count: number): string[] {
   const extensions = ['png', 'jpg', 'mp4', 'wav', 'glb', 'txt']
@@ -168,23 +88,12 @@ function getExecutionDuration(job: RawJobListItem): number {
 
 export class AssetsHelper {
   private jobsRouteHandler: ((route: Route) => Promise<void>) | null = null
-  private cloudAssetsRouteHandler: ((route: Route) => Promise<void>) | null =
-    null
-  private assetExportRouteHandler: ((route: Route) => Promise<void>) | null =
-    null
   private inputFilesRouteHandler: ((route: Route) => Promise<void>) | null =
     null
   private deleteHistoryRouteHandler: ((route: Route) => Promise<void>) | null =
     null
   private generatedJobs: RawJobListItem[] = []
-  private cloudAssetsResponse: ListAssetsResponse | null = null
-  private assetExportRequests: CreateAssetExportData['body'][] = []
-  private assetExportResponse: CreateAssetExportResponse | null = null
   private importedFiles: string[] = []
-  private readonly jobDetailRouteHandlers = new Map<
-    string,
-    (route: Route) => Promise<void>
-  >()
 
   constructor(private readonly page: Page) {}
 
@@ -261,82 +170,6 @@ export class AssetsHelper {
     await this.page.route(jobsListRoutePattern, this.jobsRouteHandler)
   }
 
-  async mockCloudAssets(response: ListAssetsResponse): Promise<void> {
-    this.cloudAssetsResponse = response
-
-    if (this.cloudAssetsRouteHandler) {
-      return
-    }
-
-    this.cloudAssetsRouteHandler = async (route: Route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(this.cloudAssetsResponse)
-      })
-    }
-
-    await this.page.route(assetsListRoutePattern, this.cloudAssetsRouteHandler)
-  }
-
-  async mockEmptyCloudAssets(): Promise<void> {
-    await this.mockCloudAssets({
-      assets: [],
-      total: 0,
-      has_more: false
-    })
-  }
-
-  async captureAssetExportRequests(
-    response: CreateAssetExportResponse = {
-      task_id: 'asset-export-task',
-      status: 'created'
-    }
-  ): Promise<CreateAssetExportData['body'][]> {
-    this.assetExportRequests = []
-    this.assetExportResponse = response
-
-    if (this.assetExportRouteHandler) {
-      return this.assetExportRequests
-    }
-
-    this.assetExportRouteHandler = async (route: Route) => {
-      this.assetExportRequests.push(
-        route.request().postDataJSON() as CreateAssetExportData['body']
-      )
-
-      await route.fulfill({
-        status: 202,
-        contentType: 'application/json',
-        body: JSON.stringify(this.assetExportResponse)
-      })
-    }
-
-    await this.page.route(assetExportRoutePattern, this.assetExportRouteHandler)
-
-    return this.assetExportRequests
-  }
-
-  async mockJobDetail(jobId: string, detail: JobDetail): Promise<void> {
-    const pattern = `**/api/jobs/${encodeURIComponent(jobId)}`
-    const existingHandler = this.jobDetailRouteHandlers.get(pattern)
-
-    if (existingHandler) {
-      await this.page.unroute(pattern, existingHandler)
-    }
-
-    const handler = async (route: Route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(detail)
-      })
-    }
-
-    this.jobDetailRouteHandlers.set(pattern, handler)
-    await this.page.route(pattern, handler)
-  }
-
   async mockInputFiles(files: string[]): Promise<void> {
     this.importedFiles = [...files]
 
@@ -392,9 +225,6 @@ export class AssetsHelper {
 
   async clearMocks(): Promise<void> {
     this.generatedJobs = []
-    this.cloudAssetsResponse = null
-    this.assetExportRequests = []
-    this.assetExportResponse = null
     this.importedFiles = []
 
     if (this.jobsRouteHandler) {
@@ -402,22 +232,6 @@ export class AssetsHelper {
       this.jobsRouteHandler = null
     }
 
-    if (this.cloudAssetsRouteHandler) {
-      await this.page.unroute(
-        assetsListRoutePattern,
-        this.cloudAssetsRouteHandler
-      )
-      this.cloudAssetsRouteHandler = null
-    }
-
-    if (this.assetExportRouteHandler) {
-      await this.page.unroute(
-        assetExportRoutePattern,
-        this.assetExportRouteHandler
-      )
-      this.assetExportRouteHandler = null
-    }
-
     if (this.inputFilesRouteHandler) {
       await this.page.unroute(
         inputFilesRoutePattern,
@@ -433,10 +247,5 @@ export class AssetsHelper {
       )
       this.deleteHistoryRouteHandler = null
     }
-
-    for (const [pattern, handler] of this.jobDetailRouteHandlers) {
-      await this.page.unroute(pattern, handler)
-    }
-    this.jobDetailRouteHandlers.clear()
   }
 }

browser_tests/fixtures/helpers/CanvasHelper.ts

@@ -74,7 +74,7 @@ export class CanvasHelper {
    * Use with `page.mouse` APIs when Vue DOM overlays above the canvas would
    * cause Playwright's actionability check to fail on the canvas locator.
    */
-  async toAbsolute(position: Position): Promise<Position> {
+  private async toAbsolute(position: Position): Promise<Position> {
     const box = await this.canvas.boundingBox()
     if (!box) throw new Error('Canvas bounding box not available')
     return { x: box.x + position.x, y: box.y + position.y }
@@ -150,28 +150,6 @@ export class CanvasHelper {
     await nextFrame(this.page)
   }
 
-  async getOffset(): Promise<[number, number]> {
-    return this.page.evaluate(
-      () => [...window.app!.canvas.ds.offset] as [number, number]
-    )
-  }
-
-  async getNodeTitleHeight(): Promise<number> {
-    return this.page.evaluate(() => window.LiteGraph!.NODE_TITLE_HEIGHT)
-  }
-
-  /**
-   * Hold `Control+Shift` and drag from `from` to `to` using page-absolute
-   * coordinates.
-   */
-  async ctrlShiftDrag(from: Position, to: Position): Promise<void> {
-    await this.page.keyboard.down('Control')
-    await this.page.keyboard.down('Shift')
-    await this.dragAndDrop(from, to)
-    await this.page.keyboard.up('Shift')
-    await this.page.keyboard.up('Control')
-  }
-
   async convertOffsetToCanvas(
     pos: [number, number]
   ): Promise<[number, number]> {
@@ -264,39 +242,11 @@ export class CanvasHelper {
     await this.page.mouse.up({ button: 'middle' })
   }
 
-  async disconnectEdge(
-    options: { modifiers?: ('Shift' | 'Control' | 'Alt' | 'Meta')[] } = {}
-  ): Promise<void> {
-    const { modifiers = [] } = options
-    for (const mod of modifiers) await this.page.keyboard.down(mod)
-    try {
-      await this.dragAndDrop(
-        DefaultGraphPositions.clipTextEncodeNode1InputSlot,
-        DefaultGraphPositions.emptySpace
-      )
-    } finally {
-      for (const mod of modifiers) await this.page.keyboard.up(mod)
-    }
-  }
-
-  async middleClick(position: Position): Promise<void> {
-    await this.mouseClickAt(position, { button: 'middle' })
-  }
-
-  async dblclickGroupTitle(title: string): Promise<void> {
-    const clientPos = await this.page.evaluate((targetTitle) => {
-      const groups = window.app!.canvas.graph?.groups ?? []
-      const group = groups.find(
-        (g: { title: string }) => g.title === targetTitle
-      )
-      if (!group) return null
-      const cx = group.pos[0] + group.size[0] / 2
-      const cy = group.pos[1] + group.titleHeight / 2
-      return window.app!.canvasPosToClientPos([cx, cy])
-    }, title)
-    if (!clientPos) throw new Error(`Group "${title}" not found`)
-    await this.page.mouse.dblclick(clientPos[0], clientPos[1], { delay: 5 })
-    await nextFrame(this.page)
+  async disconnectEdge(): Promise<void> {
+    await this.dragAndDrop(
+      DefaultGraphPositions.clipTextEncodeNode1InputSlot,
+      DefaultGraphPositions.emptySpace
+    )
   }
 
   async connectEdge(options: { reverse?: boolean } = {}): Promise<void> {

browser_tests/fixtures/helpers/ClipboardHelper.ts

@@ -4,7 +4,7 @@ import { basename } from 'path'
 import type { Locator, Page } from '@playwright/test'
 
 import type { KeyboardHelper } from '@e2e/fixtures/helpers/KeyboardHelper'
-import { getMimeType } from '@e2e/fixtures/utils/mimeTypeUtil'
+import { getMimeType } from '@e2e/fixtures/helpers/mimeTypeUtil'
 
 export class ClipboardHelper {
   constructor(

browser_tests/fixtures/helpers/CloudAuthHelper.ts

@@ -13,11 +13,7 @@ import type { Page } from '@playwright/test'
  * so the SDK believes a user is signed in. Must be called before navigation.
  */
 export class CloudAuthHelper {
-  private readonly appUrl: string
-
-  constructor(private readonly page: Page) {
-    this.appUrl = process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
-  }
+  constructor(private readonly page: Page) {}
 
   /**
    * Set up all auth mocks. Must be called before `comfyPage.setup()`.
@@ -38,7 +34,7 @@ export class CloudAuthHelper {
    */
   private async seedFirebaseIndexedDB(): Promise<void> {
     // Navigate to a lightweight endpoint to get a same-origin context
-    await this.page.goto(`${this.appUrl}/api/users`)
+    await this.page.goto('http://localhost:8188/api/users')
 
     await this.page.evaluate(() => {
       const MOCK_USER_DATA = {

browser_tests/fixtures/helpers/DragDropHelper.ts

@@ -3,7 +3,7 @@ import { readFileSync } from 'fs'
 import type { Page } from '@playwright/test'
 
 import type { Position } from '@e2e/fixtures/types'
-import { getMimeType } from '@e2e/fixtures/utils/mimeTypeUtil'
+import { getMimeType } from '@e2e/fixtures/helpers/mimeTypeUtil'
 import { assetPath } from '@e2e/fixtures/utils/paths'
 import { nextFrame } from '@e2e/fixtures/utils/timing'
 

browser_tests/fixtures/helpers/NodeOperationsHelper.ts

@@ -1,10 +1,6 @@
 import type { Locator } from '@playwright/test'
 
-import type {
-  GraphAddOptions,
-  LGraph,
-  LGraphNode
-} from '@/lib/litegraph/src/litegraph'
+import type { LGraph, LGraphNode } from '@/lib/litegraph/src/litegraph'
 import type {
   ComfyWorkflowJSON,
   NodeId
@@ -43,48 +39,6 @@ export class NodeOperationsHelper {
     })
   }
 
-  async getSelectedNodeIds(): Promise<NodeId[]> {
-    return await this.page.evaluate(() => {
-      const selected = window.app?.canvas?.selected_nodes
-      if (!selected) return []
-      return Object.keys(selected).map(Number)
-    })
-  }
-
-  /**
-   * Add a node to the graph by type.
-   * @param type - The node type (e.g. 'KSampler', 'VAEDecode')
-   * @param options - GraphAddOptions (ghost, skipComputeOrder). When ghost is
-   *   true and position is provided, a synthetic MouseEvent is created as the
-   *   dragEvent.
-   * @param position - When ghost is true, client coordinates for the ghost
-   *   placement dragEvent. Otherwise, world coordinates assigned to node.pos.
-   */
-  async addNode(
-    type: string,
-    options?: Omit<GraphAddOptions, 'dragEvent'>,
-    position?: Position
-  ): Promise<NodeReference> {
-    const id = await this.page.evaluate(
-      ([nodeType, opts, pos]) => {
-        const node = window.LiteGraph!.createNode(nodeType)!
-        const addOpts: Record<string, unknown> = { ...opts }
-        if (opts?.ghost && pos) {
-          addOpts.dragEvent = new MouseEvent('click', {
-            clientX: pos.x,
-            clientY: pos.y
-          })
-        } else if (pos) {
-          node.pos = [pos.x, pos.y]
-        }
-        window.app!.graph.add(node, addOpts as GraphAddOptions)
-        return node.id
-      },
-      [type, options ?? {}, position ?? null] as const
-    )
-    return new NodeReference(id, this.comfyPage)
-  }
-
   /** Remove all nodes from the graph and clean. */
   async clearGraph() {
     await this.comfyPage.settings.setSetting('Comfy.ConfirmClear', false)

browser_tests/fixtures/helpers/PublishApiHelper.ts

@@ -1,231 +0,0 @@
-import type { Page, Route } from '@playwright/test'
-
-import type {
-  AssetInfo,
-  HubAssetUploadUrlResponse,
-  HubLabelInfo,
-  HubLabelListResponse,
-  HubProfile,
-  WorkflowPublishInfo
-} from '@comfyorg/ingest-types'
-
-import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
-import { PublishDialog } from '@e2e/fixtures/components/PublishDialog'
-
-import type { ShareableAssetsResponse } from '@/schemas/apiSchema'
-
-const DEFAULT_PROFILE: HubProfile = {
-  username: 'testuser',
-  display_name: 'Test User',
-  description: 'A test creator',
-  avatar_url: undefined
-}
-
-const DEFAULT_TAG_LABELS: HubLabelInfo[] = [
-  { name: 'anime', display_name: 'anime', type: 'tag' },
-  { name: 'upscale', display_name: 'upscale', type: 'tag' },
-  { name: 'faceswap', display_name: 'faceswap', type: 'tag' },
-  { name: 'img2img', display_name: 'img2img', type: 'tag' },
-  { name: 'controlnet', display_name: 'controlnet', type: 'tag' }
-]
-
-const DEFAULT_PUBLISH_RESPONSE: WorkflowPublishInfo = {
-  workflow_id: 'test-workflow-id-456',
-  share_id: 'test-share-id-123',
-  publish_time: new Date().toISOString(),
-  listed: true,
-  assets: []
-}
-
-const DEFAULT_UPLOAD_URL_RESPONSE: HubAssetUploadUrlResponse = {
-  upload_url: 'https://mock-s3.example.com/upload',
-  public_url: 'https://mock-s3.example.com/asset.png',
-  token: 'mock-upload-token'
-}
-
-export class PublishApiHelper {
-  private routeHandlers: Array<{
-    pattern: string
-    handler: (route: Route) => Promise<void>
-  }> = []
-
-  constructor(private readonly page: Page) {}
-
-  async mockProfile(profile: HubProfile | null): Promise<void> {
-    await this.addRoute('**/hub/profiles/me', async (route) => {
-      if (route.request().method() !== 'GET') {
-        await route.continue()
-        return
-      }
-      if (profile === null) {
-        await route.fulfill({ status: 404, body: 'Not found' })
-      } else {
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify(profile)
-        })
-      }
-    })
-  }
-
-  async mockTagLabels(
-    labels: HubLabelInfo[] = DEFAULT_TAG_LABELS
-  ): Promise<void> {
-    const response: HubLabelListResponse = { labels }
-    await this.addRoute('**/hub/labels**', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(response)
-      })
-    })
-  }
-
-  async mockPublishStatus(
-    status: 'unpublished' | WorkflowPublishInfo
-  ): Promise<void> {
-    await this.addRoute('**/userdata/*/publish', async (route) => {
-      if (route.request().method() !== 'GET') {
-        await route.continue()
-        return
-      }
-      if (status === 'unpublished') {
-        await route.fulfill({ status: 404, body: 'Not found' })
-      } else {
-        await route.fulfill({
-          status: 200,
-          contentType: 'application/json',
-          body: JSON.stringify(status)
-        })
-      }
-    })
-  }
-
-  async mockShareableAssets(assets: AssetInfo[] = []): Promise<void> {
-    const response: ShareableAssetsResponse = { assets }
-    await this.addRoute('**/assets/from-workflow', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(response)
-      })
-    })
-  }
-
-  async mockPublishWorkflow(
-    response: WorkflowPublishInfo = DEFAULT_PUBLISH_RESPONSE
-  ): Promise<void> {
-    await this.removeRoutes('**/hub/workflows')
-    await this.addRoute('**/hub/workflows', async (route) => {
-      if (route.request().method() !== 'POST') {
-        await route.continue()
-        return
-      }
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(response)
-      })
-    })
-  }
-
-  async mockPublishWorkflowError(
-    statusCode = 500,
-    message = 'Failed to publish workflow'
-  ): Promise<void> {
-    await this.removeRoutes('**/hub/workflows')
-    await this.addRoute('**/hub/workflows', async (route) => {
-      if (route.request().method() !== 'POST') {
-        await route.continue()
-        return
-      }
-      await route.fulfill({
-        status: statusCode,
-        contentType: 'application/json',
-        body: JSON.stringify({ message })
-      })
-    })
-  }
-
-  async mockUploadUrl(
-    response: HubAssetUploadUrlResponse = DEFAULT_UPLOAD_URL_RESPONSE
-  ): Promise<void> {
-    await this.addRoute('**/hub/assets/upload-url', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(response)
-      })
-    })
-  }
-
-  async setupDefaultMocks(options?: {
-    hasProfile?: boolean
-    hasPrivateAssets?: boolean
-  }): Promise<void> {
-    const { hasProfile = true, hasPrivateAssets = false } = options ?? {}
-
-    await this.mockProfile(hasProfile ? DEFAULT_PROFILE : null)
-    await this.mockTagLabels()
-    await this.mockPublishStatus('unpublished')
-    await this.mockShareableAssets(
-      hasPrivateAssets
-        ? [
-            {
-              id: 'asset-1',
-              name: 'my_model.safetensors',
-              preview_url: '',
-              storage_url: '',
-              model: true,
-              public: false,
-              in_library: true
-            }
-          ]
-        : []
-    )
-    await this.mockPublishWorkflow()
-    await this.mockUploadUrl()
-  }
-
-  async cleanup(): Promise<void> {
-    for (const { pattern, handler } of this.routeHandlers) {
-      await this.page.unroute(pattern, handler)
-    }
-    this.routeHandlers = []
-  }
-
-  private async addRoute(
-    pattern: string,
-    handler: (route: Route) => Promise<void>
-  ): Promise<void> {
-    this.routeHandlers.push({ pattern, handler })
-    await this.page.route(pattern, handler)
-  }
-
-  private async removeRoutes(pattern: string): Promise<void> {
-    const handlers = this.routeHandlers.filter(
-      (route) => route.pattern === pattern
-    )
-    for (const { handler } of handlers) {
-      await this.page.unroute(pattern, handler)
-    }
-    this.routeHandlers = this.routeHandlers.filter(
-      (route) => route.pattern !== pattern
-    )
-  }
-}
-
-export const publishFixture = comfyPageFixture.extend<{
-  publishApi: PublishApiHelper
-  publishDialog: PublishDialog
-}>({
-  publishApi: async ({ comfyPage }, use) => {
-    const helper = new PublishApiHelper(comfyPage.page)
-    await use(helper)
-    await helper.cleanup()
-  },
-  publishDialog: async ({ comfyPage }, use) => {
-    await use(new PublishDialog(comfyPage.page))
-  }
-})

browser_tests/fixtures/selectors.ts

@@ -59,9 +59,6 @@ export const TestIds = {
     missingModelCopyName: 'missing-model-copy-name',
     missingModelCopyUrl: 'missing-model-copy-url',
     missingModelDownload: 'missing-model-download',
-    missingModelActions: 'missing-model-actions',
-    missingModelDownloadAll: 'missing-model-download-all',
-    missingModelRefresh: 'missing-model-refresh',
     missingModelImportUnsupported: 'missing-model-import-unsupported',
     missingMediaGroup: 'error-group-missing-media',
     missingMediaRow: 'missing-media-row',
@@ -86,11 +83,7 @@ export const TestIds = {
     queueButton: 'queue-button',
     queueModeMenuTrigger: 'queue-mode-menu-trigger',
     saveButton: 'save-workflow-button',
-    subscribeButton: 'topbar-subscribe-button',
-    loginButton: 'login-button',
-    loginButtonPopover: 'login-button-popover',
-    loginButtonPopoverLearnMore: 'login-button-popover-learn-more',
-    actionBarButtons: 'action-bar-buttons'
+    subscribeButton: 'topbar-subscribe-button'
   },
   nodeLibrary: {
     bookmarksSection: 'node-library-bookmarks-section'
@@ -210,25 +203,12 @@ export const TestIds = {
   },
   queue: {
     overlayToggle: 'queue-overlay-toggle',
-    clearHistoryAction: 'clear-history-action',
-    jobAssetsList: 'job-assets-list'
+    clearHistoryAction: 'clear-history-action'
   },
   errors: {
     imageLoadError: 'error-loading-image',
     videoLoadError: 'error-loading-video'
   },
-  publish: {
-    dialog: 'publish-dialog',
-    savePrompt: 'publish-save-prompt',
-    describeStep: 'publish-describe-step',
-    finishStep: 'publish-finish-step',
-    footer: 'publish-footer',
-    profilePrompt: 'publish-profile-prompt',
-    nav: 'publish-nav',
-    gateFlow: 'publish-gate-flow',
-    nameInput: 'publish-name-input',
-    tagsInput: 'publish-tags-input'
-  },
   loading: {
     overlay: 'loading-overlay'
   },
@@ -254,20 +234,6 @@ export const TestIds = {
     batchCounter: 'batch-counter',
     batchNext: 'batch-next',
     batchPrev: 'batch-prev'
-  },
-  searchBoxV2: {
-    resultItem: 'result-item',
-    filterOption: 'filter-option',
-    filterChip: 'filter-chip',
-    chipDelete: 'chip-delete',
-    noResults: 'no-results',
-    nodeIdBadge: 'node-id-badge',
-    sidebarToggle: 'toggle-category-sidebar',
-    sidebarBackdrop: 'sidebar-backdrop',
-    filterChipsScroll: 'filter-chips-scroll',
-    category: (id: string) => `category-${id}`,
-    rootCategory: (id: string) => `search-category-${id}`,
-    typeFilter: (key: 'input' | 'output') => `search-filter-${key}`
   }
 } as const
 

browser_tests/fixtures/utils/litegraphUtils.ts

@@ -1,8 +1,7 @@
 import { expect } from '@playwright/test'
 
-import type { SerialisableLLink } from '@/lib/litegraph/src/types/serialisation'
 import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema'
-import { ManageGroupNode } from '@e2e/fixtures/components/ManageGroupNode'
+import { ManageGroupNode } from '@e2e/helpers/manageGroupNode'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 import type { Position, Size } from '@e2e/fixtures/types'
 import { VueNodeFixture } from '@e2e/fixtures/utils/vueNodeFixtures'
@@ -170,36 +169,6 @@ class NodeSlotReference {
       [this.type, this.node.id, this.index] as const
     )
   }
-
-  async getLink(): Promise<SerialisableLLink | null> {
-    return await this.node.comfyPage.page.evaluate(
-      ([type, id, index]) => {
-        const graph = window.app!.canvas.graph!
-        const node = graph.getNodeById(id)
-        if (!node) throw new Error(`Node ${id} not found.`)
-        const linkId =
-          type === 'input'
-            ? node.inputs[index].link
-            : (node.outputs[index].links ?? [])[0]
-        if (linkId == null) return null
-        const link =
-          graph.links instanceof Map
-            ? graph.links.get(linkId)
-            : graph.links[linkId]
-        if (!link) return null
-        return {
-          id: link.id,
-          origin_id: link.origin_id,
-          origin_slot: link.origin_slot,
-          target_id: link.target_id,
-          target_slot: link.target_slot,
-          type: link.type,
-          parentId: link.parentId
-        }
-      },
-      [this.type, this.node.id, this.index] as const
-    )
-  }
 }
 
 export class NodeWidgetReference {
@@ -357,23 +326,6 @@ export class NodeReference {
     const nodeSize = await this.getSize()
     return { x: nodePos.x + nodeSize.width / 2, y: nodePos.y - 15 }
   }
-  async dragBy(
-    delta: Position,
-    options?: {
-      modifiers?: ('Shift' | 'Control' | 'Alt' | 'Meta')[]
-    }
-  ): Promise<void> {
-    const titlePos = await this.getTitlePosition()
-    const target = { x: titlePos.x + delta.x, y: titlePos.y + delta.y }
-    const modifiers = options?.modifiers ?? []
-    const keyboard = this.comfyPage.page.keyboard
-    for (const mod of modifiers) await keyboard.down(mod)
-    try {
-      await this.comfyPage.canvasOps.dragAndDrop(titlePos, target)
-    } finally {
-      for (const mod of modifiers) await keyboard.up(mod)
-    }
-  }
   async isPinned() {
     return !!(await this.getFlags()).pinned
   }

browser_tests/fixtures/utils/vueNodeFixtures.ts

@@ -1,13 +1,12 @@
 import type { Locator } from '@playwright/test'
 
-import { TitleEditor } from '@e2e/fixtures/components/TitleEditor'
 import { TestIds } from '@e2e/fixtures/selectors'
 
 /** DOM-centric helper for a single Vue-rendered node on the canvas. */
 export class VueNodeFixture {
   public readonly header: Locator
   public readonly title: Locator
-  public readonly titleEditor: TitleEditor
+  public readonly titleInput: Locator
   public readonly body: Locator
   public readonly pinIndicator: Locator
   public readonly collapseButton: Locator
@@ -17,7 +16,7 @@ export class VueNodeFixture {
   constructor(private readonly locator: Locator) {
     this.header = locator.locator('[data-testid^="node-header-"]')
     this.title = locator.getByTestId('node-title')
-    this.titleEditor = new TitleEditor(locator)
+    this.titleInput = locator.getByTestId('node-title-input')
     this.body = locator.locator('[data-testid^="node-body-"]')
     this.pinIndicator = locator.getByTestId(TestIds.node.pinIndicator)
     this.collapseButton = locator.getByTestId('node-collapse-button')
@@ -31,8 +30,17 @@ export class VueNodeFixture {
 
   async setTitle(value: string): Promise<void> {
     await this.header.dblclick()
-    await this.titleEditor.expectVisible()
-    await this.titleEditor.setTitle(value)
+    const input = this.titleInput
+    await input.waitFor({ state: 'visible' })
+    await input.fill(value)
+    await input.press('Enter')
+  }
+
+  async cancelTitleEdit(): Promise<void> {
+    await this.header.dblclick()
+    const input = this.titleInput
+    await input.waitFor({ state: 'visible' })
+    await input.press('Escape')
   }
 
   async toggleCollapse(): Promise<void> {

browser_tests/globalTeardown.ts

@@ -2,7 +2,7 @@ import { config as dotenvConfig } from 'dotenv'
 import MCR from 'monocart-coverage-reports'
 
 import { COVERAGE_OUTPUT_DIR, coverageSourceFilter } from '@e2e/coverageConfig'
-import { writePerfReport } from '@e2e/fixtures/utils/perfReporter'
+import { writePerfReport } from '@e2e/helpers/perfReporter'
 import { restorePath } from '@e2e/utils/backupUtils'
 
 dotenvConfig()

browser_tests/helpers/builderTestUtils.ts

@@ -5,7 +5,7 @@ import type { AppModeHelper } from '@e2e/fixtures/helpers/AppModeHelper'
 import type { NodeReference } from '@e2e/fixtures/utils/litegraphUtils'
 
 import { comfyExpect } from '@e2e/fixtures/ComfyPage'
-import { fitToViewInstant } from '@e2e/fixtures/utils/fitToView'
+import { fitToViewInstant } from '@e2e/helpers/fitToView'
 
 interface BuilderSetupResult {
   inputNodeTitle: string

browser_tests/helpers/perfReporter.ts

@@ -56,13 +56,7 @@ export function writePerfReport(
   gitSha = process.env.GITHUB_SHA ?? 'local',
   branch = process.env.GITHUB_HEAD_REF ?? 'local'
 ) {
-  let entries
-  try {
-    entries = readdirSync('test-results', { withFileTypes: true })
-  } catch {
-    return
-  }
-  if (!entries.length) return
+  if (!readdirSync('test-results', { withFileTypes: true }).length) return
 
   let tempFiles: string[]
   try {

browser_tests/helpers/promotedWidgets.ts

@@ -1,6 +1,6 @@
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
 
-export type PromotedWidgetEntry = [string, string]
+type PromotedWidgetEntry = [string, string]
 
 function isPromotedWidgetEntry(entry: unknown): entry is PromotedWidgetEntry {
   return (

browser_tests/tests/actionBarButtons.spec.ts

@@ -1,140 +0,0 @@
-import type { Page } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { TestIds } from '@e2e/fixtures/selectors'
-
-const ICON_CLASS = 'icon-[lucide--star]'
-const BUTTON_LABEL = 'Test Action'
-const BUTTON_TOOLTIP = 'Test action tooltip'
-
-async function registerTestButton(
-  page: Page,
-  opts: {
-    name?: string
-    icon?: string
-    label?: string
-    tooltip?: string
-  } = {}
-): Promise<void> {
-  await page.evaluate(
-    ({ name, icon, label, tooltip }) => {
-      window.app!.registerExtension({
-        name,
-        actionBarButtons: [{ icon, label, tooltip, onClick: () => {} }]
-      })
-    },
-    {
-      name: opts.name ?? 'TestActionBarButton',
-      icon: opts.icon ?? ICON_CLASS,
-      label: opts.label ?? BUTTON_LABEL,
-      tooltip: opts.tooltip ?? BUTTON_TOOLTIP
-    }
-  )
-}
-
-test.describe('ActionBar Buttons', { tag: ['@ui'] }, () => {
-  test.describe('Empty state', () => {
-    test('container is hidden when no extension registers buttons', async ({
-      comfyPage
-    }) => {
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.actionBarButtons)
-      ).toBeHidden()
-    })
-  })
-
-  test.describe('Button rendering', () => {
-    test('registered button is visible with correct label', async ({
-      comfyPage
-    }) => {
-      await registerTestButton(comfyPage.page)
-      const container = comfyPage.page.getByTestId(
-        TestIds.topbar.actionBarButtons
-      )
-      await expect(container).toBeVisible()
-      await expect(
-        container.getByRole('button', { name: BUTTON_TOOLTIP })
-      ).toBeVisible()
-      await expect(container.getByText(BUTTON_LABEL)).toBeVisible()
-    })
-
-    test('button icon is rendered', async ({ comfyPage }) => {
-      await registerTestButton(comfyPage.page)
-      const icon = comfyPage.page
-        .getByTestId(TestIds.topbar.actionBarButtons)
-        .getByRole('button', { name: BUTTON_TOOLTIP })
-        .locator('i')
-      await expect(icon).toHaveClass(ICON_CLASS)
-    })
-
-    test('multiple registered buttons all appear', async ({ comfyPage }) => {
-      await comfyPage.page.evaluate(() => {
-        window.app!.registerExtension({
-          name: 'TestActionBarButtons',
-          actionBarButtons: [
-            {
-              icon: 'icon-[lucide--star]',
-              label: 'First',
-              tooltip: 'First action',
-              onClick: () => {}
-            },
-            {
-              icon: 'icon-[lucide--heart]',
-              label: 'Second',
-              tooltip: 'Second action',
-              onClick: () => {}
-            }
-          ]
-        })
-      })
-
-      const container = comfyPage.page.getByTestId(
-        TestIds.topbar.actionBarButtons
-      )
-      await expect(
-        container.getByRole('button', { name: 'First action' })
-      ).toBeVisible()
-      await expect(
-        container.getByRole('button', { name: 'Second action' })
-      ).toBeVisible()
-    })
-  })
-
-  test.describe('Click handler', () => {
-    test('clicking a button fires its onClick handler', async ({
-      comfyPage
-    }) => {
-      const onClickFired = comfyPage.page.evaluate(
-        ({ icon, label, tooltip }) =>
-          new Promise<boolean>((resolve) => {
-            window.app!.registerExtension({
-              name: 'TestActionBarButton',
-              actionBarButtons: [
-                { icon, label, tooltip, onClick: () => resolve(true) }
-              ]
-            })
-          }),
-        { icon: ICON_CLASS, label: BUTTON_LABEL, tooltip: BUTTON_TOOLTIP }
-      )
-
-      const button = comfyPage.page
-        .getByTestId(TestIds.topbar.actionBarButtons)
-        .getByRole('button', { name: BUTTON_TOOLTIP })
-      await button.click()
-
-      await expect(onClickFired).resolves.toBe(true)
-    })
-  })
-
-  test.describe('Mobile layout', { tag: ['@mobile'] }, () => {
-    test('button label is hidden on mobile viewport', async ({ comfyPage }) => {
-      await registerTestButton(comfyPage.page)
-      const container = comfyPage.page.getByTestId(
-        TestIds.topbar.actionBarButtons
-      )
-      await expect(container).toBeVisible()
-      await expect(container.getByText(BUTTON_LABEL)).toBeHidden()
-    })
-  })
-})

browser_tests/tests/appModeArrange.spec.ts

@@ -2,7 +2,7 @@ import {
   comfyPageFixture as test,
   comfyExpect as expect
 } from '@e2e/fixtures/ComfyPage'
-import { setupBuilder } from '@e2e/fixtures/utils/builderTestUtils'
+import { setupBuilder } from '@e2e/helpers/builderTestUtils'
 
 test.describe('App mode arrange step', { tag: '@ui' }, () => {
   test.beforeEach(async ({ comfyPage }) => {

browser_tests/tests/appModePruning.spec.ts

@@ -3,8 +3,8 @@ import {
   comfyPageFixture as test,
   comfyExpect as expect
 } from '@e2e/fixtures/ComfyPage'
-import { setupBuilder } from '@e2e/fixtures/utils/builderTestUtils'
-import { fitToViewInstant } from '@e2e/fixtures/utils/fitToView'
+import { setupBuilder } from '@e2e/helpers/builderTestUtils'
+import { fitToViewInstant } from '@e2e/helpers/fitToView'
 
 const RESIZE_NODE_TITLE = 'Resize Image/Mask'
 const RESIZE_NODE_ID = '1'

browser_tests/tests/appModeWidgetRename.spec.ts

@@ -5,7 +5,7 @@ import {
 import {
   saveAndReopenInAppMode,
   setupSubgraphBuilder
-} from '@e2e/fixtures/utils/builderTestUtils'
+} from '@e2e/helpers/builderTestUtils'
 
 test.describe('App mode widget rename', { tag: ['@ui', '@subgraph'] }, () => {
   test.beforeEach(async ({ comfyPage }) => {

browser_tests/tests/bottomPanelLogs.spec.ts

@@ -12,7 +12,7 @@ import { webSocketFixture } from '@e2e/fixtures/ws'
 import {
   getClipboardText,
   interceptClipboardWrite
-} from '@e2e/fixtures/utils/clipboardSpy'
+} from '@e2e/helpers/clipboardSpy'
 
 const test = mergeTests(comfyPageFixture, logsTerminalFixture, webSocketFixture)
 

browser_tests/tests/builderReorder.spec.ts

@@ -8,7 +8,7 @@ import {
   builderSaveAs,
   openWorkflowFromSidebar,
   setupBuilder
-} from '@e2e/fixtures/utils/builderTestUtils'
+} from '@e2e/helpers/builderTestUtils'
 
 const WIDGETS = ['seed', 'steps', 'cfg']
 

browser_tests/tests/builderSaveFlow.spec.ts

@@ -8,8 +8,8 @@ import {
   builderSaveAs,
   openWorkflowFromSidebar,
   setupBuilder
-} from '@e2e/fixtures/utils/builderTestUtils'
-import { fitToViewInstant } from '@e2e/fixtures/utils/fitToView'
+} from '@e2e/helpers/builderTestUtils'
+import { fitToViewInstant } from '@e2e/helpers/fitToView'
 
 /**
  * After a first save, open save-as again from the chevron,

browser_tests/tests/canvasLayoutSettings.spec.ts

@@ -1,175 +0,0 @@
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '@e2e/fixtures/ComfyPage'
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import type { Size } from '@e2e/fixtures/types'
-
-const expectedGroupSize = (
-  nodeBounds: Size,
-  padding: number,
-  titleHeight: number
-): Size => ({
-  width: nodeBounds.width + padding * 2,
-  // Group height adds one title row above the contained node bounds (which
-  // themselves already include the node's own title), independent of padding.
-  height: nodeBounds.height + padding * 2 + titleHeight
-})
-
-test.describe('Canvas layout settings', { tag: '@canvas' }, () => {
-  test.describe('Comfy.SnapToGrid.GridSize', () => {
-    const DRAG_DELTA = { x: 550, y: 330 } as const
-
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.nodeOps.clearGraph()
-    })
-
-    const createNode = async (comfyPage: ComfyPage) => {
-      const note = await comfyPage.nodeOps.addNode('Note', undefined, {
-        x: 0,
-        y: 0
-      })
-      await note.centerOnNode()
-      return note
-    }
-
-    test('shift+drag rounds final node position to multiples of grid size', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.SnapToGrid.GridSize', 100)
-      const note = await createNode(comfyPage)
-
-      await note.dragBy(DRAG_DELTA, { modifiers: ['Shift'] })
-
-      // raw final world pos = (550, 330); rounded to nearest 100 = (600, 300)
-      const after = await note.getProperty<[number, number]>('pos')
-      expect(after[0]).toBe(600)
-      expect(after[1]).toBe(300)
-    })
-
-    test('grid size determines the snap multiple', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.SnapToGrid.GridSize', 50)
-      const note = await createNode(comfyPage)
-
-      await note.dragBy(DRAG_DELTA, { modifiers: ['Shift'] })
-
-      // raw final world pos = (550, 330); rounded to nearest 50 = (550, 350)
-      const after = await note.getProperty<[number, number]>('pos')
-      expect(after[0]).toBe(550)
-      expect(after[1]).toBe(350)
-    })
-
-    test('drag without shift bypasses snap regardless of grid size', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.SnapToGrid.GridSize', 100)
-      const note = await createNode(comfyPage)
-      const before = await note.getProperty<[number, number]>('pos')
-
-      await note.dragBy(DRAG_DELTA)
-
-      const after = await note.getProperty<[number, number]>('pos')
-      expect(after[0]).toBe(before[0] + DRAG_DELTA.x)
-      expect(after[1]).toBe(before[1] + DRAG_DELTA.y)
-    })
-  })
-
-  test.describe('Comfy.GroupSelectedNodes.Padding', () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.workflow.loadWorkflow('nodes/single_ksampler')
-    })
-
-    const groupAroundAllNodesWithPadding = async (
-      comfyPage: ComfyPage,
-      padding: number
-    ): Promise<Size> => {
-      await comfyPage.settings.setSetting(
-        'Comfy.GroupSelectedNodes.Padding',
-        padding
-      )
-      await comfyPage.command.executeCommand('Comfy.Canvas.SelectAll')
-      await comfyPage.command.executeCommand('Comfy.Graph.GroupSelectedNodes')
-      return comfyPage.page.evaluate(() => {
-        const group = window.app!.graph.groups[0]
-        return { width: group.size[0], height: group.size[1] }
-      })
-    }
-
-    test('padding=0 makes the group exactly enclose the selection', async ({
-      comfyPage
-    }) => {
-      const ksampler = (
-        await comfyPage.nodeOps.getNodeRefsByType('KSampler')
-      )[0]
-      const nodeBounds = await ksampler.getBounding()
-      const titleHeight = await comfyPage.canvasOps.getNodeTitleHeight()
-
-      const group = await groupAroundAllNodesWithPadding(comfyPage, 0)
-
-      expect(group).toEqual(expectedGroupSize(nodeBounds, 0, titleHeight))
-    })
-
-    test('padding=50 grows the group by 100 around the selection', async ({
-      comfyPage
-    }) => {
-      const ksampler = (
-        await comfyPage.nodeOps.getNodeRefsByType('KSampler')
-      )[0]
-      const nodeBounds = await ksampler.getBounding()
-      const titleHeight = await comfyPage.canvasOps.getNodeTitleHeight()
-
-      const group = await groupAroundAllNodesWithPadding(comfyPage, 50)
-
-      expect(group).toEqual(expectedGroupSize(nodeBounds, 50, titleHeight))
-    })
-  })
-
-  test.describe('LiteGraph.ContextMenu.Scaling', () => {
-    const ZOOM_SCALE = 2
-    const litegraphContextMenu = (comfyPage: ComfyPage) =>
-      comfyPage.page.locator('.litecontextmenu')
-
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.workflow.loadWorkflow('widgets/load_image_widget')
-      await comfyPage.canvasOps.setScale(ZOOM_SCALE)
-    })
-
-    const openComboMenu = async (comfyPage: ComfyPage) => {
-      const loadImage = (
-        await comfyPage.nodeOps.getNodeRefsByType('LoadImage')
-      )[0]
-      const fileCombo = await loadImage.getWidget(0)
-      await fileCombo.click()
-    }
-
-    test('combo widget popup is scaled when setting is enabled', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('LiteGraph.ContextMenu.Scaling', true)
-
-      await openComboMenu(comfyPage)
-
-      const menu = litegraphContextMenu(comfyPage)
-      await expect(menu).toBeVisible()
-      await expect(menu).toHaveCSS(
-        'transform',
-        `matrix(${ZOOM_SCALE}, 0, 0, ${ZOOM_SCALE}, 0, 0)`
-      )
-    })
-
-    test('combo widget popup is not scaled when setting is disabled', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'LiteGraph.ContextMenu.Scaling',
-        false
-      )
-
-      await openComboMenu(comfyPage)
-
-      const menu = litegraphContextMenu(comfyPage)
-      await expect(menu).toBeVisible()
-      await expect(menu).toHaveCSS('transform', 'none')
-    })
-  })
-})

browser_tests/tests/canvasSettings.spec.ts

@@ -1,400 +0,0 @@
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '@e2e/fixtures/ComfyPage'
-import { sleep } from '@e2e/fixtures/utils/timing'
-
-const CLIP_NODE_COUNT = 2
-
-const getClipNodesDragBox = async (comfyPage: ComfyPage) => {
-  const clipNodes = await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-  expect(
-    clipNodes,
-    'Default workflow is expected to contain exactly two CLIPTextEncode nodes'
-  ).toHaveLength(CLIP_NODE_COUNT)
-  const p1 = await clipNodes[0].getPosition()
-  const p2 = await clipNodes[1].getPosition()
-  const margin = 64
-  const from = await comfyPage.canvasOps.toAbsolute({
-    x: Math.min(p1.x, p2.x) - margin,
-    y: Math.min(p1.y, p2.y) - margin
-  })
-  const to = await comfyPage.canvasOps.toAbsolute({
-    x: Math.max(p1.x, p2.x) + margin,
-    y: Math.max(p1.y, p2.y) + margin
-  })
-  return { from, to }
-}
-
-test.describe('Canvas settings', { tag: '@canvas' }, () => {
-  test.describe('Comfy.Graph.CanvasInfo', () => {
-    test(
-      'toggles the bottom-left HUD',
-      { tag: '@screenshot' },
-      async ({ comfyPage }) => {
-        const box = await comfyPage.canvas.boundingBox()
-        expect(box, 'Canvas bounding box must be available').not.toBeNull()
-        // HUD is drawn ~80px tall along the bottom edge of the canvas; grab a
-        // comfortable 180px × 160px strip to catch it across viewports.
-        const HUD_WIDTH = 180
-        const HUD_HEIGHT = 160
-        const hudClip = {
-          x: box!.x,
-          y: box!.y + box!.height - HUD_HEIGHT,
-          width: HUD_WIDTH,
-          height: HUD_HEIGHT
-        }
-
-        await test.step('Capture HUD region with setting off', async () => {
-          await comfyPage.settings.setSetting('Comfy.Graph.CanvasInfo', false)
-          await comfyPage.canvasOps.resetView()
-          await comfyPage.canvasOps.moveMouseToEmptyArea()
-          await expect(comfyPage.page).toHaveScreenshot(
-            'canvas-info-hud-off.png',
-            { clip: hudClip, maxDiffPixels: 50 }
-          )
-        })
-
-        await test.step('Capture HUD region with setting on', async () => {
-          await comfyPage.settings.setSetting('Comfy.Graph.CanvasInfo', true)
-          await comfyPage.canvasOps.moveMouseToEmptyArea()
-          await expect(comfyPage.page).toHaveScreenshot(
-            'canvas-info-hud-on.png',
-            { clip: hudClip, maxDiffPixels: 50 }
-          )
-        })
-      }
-    )
-  })
-
-  test.describe('Comfy.Graph.CtrlShiftZoom', () => {
-    const CTRL_SHIFT_DRAG_FROM = { x: 100, y: 100 }
-    const CTRL_SHIFT_DRAG_TO = { x: 400, y: 400 }
-
-    test('Ctrl+Shift+drag zooms canvas when enabled', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.Graph.CtrlShiftZoom', true)
-      await comfyPage.canvasOps.resetView()
-      const initialScale = await comfyPage.canvasOps.getScale()
-
-      await comfyPage.canvasOps.ctrlShiftDrag(
-        CTRL_SHIFT_DRAG_FROM,
-        CTRL_SHIFT_DRAG_TO
-      )
-
-      await expect
-        .poll(() => comfyPage.canvasOps.getScale())
-        .not.toBeCloseTo(initialScale, 2)
-    })
-
-    test('Ctrl+Shift+drag does not zoom when disabled', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Graph.CtrlShiftZoom', false)
-      await comfyPage.canvasOps.resetView()
-      const initialScale = await comfyPage.canvasOps.getScale()
-
-      await comfyPage.canvasOps.ctrlShiftDrag(
-        CTRL_SHIFT_DRAG_FROM,
-        CTRL_SHIFT_DRAG_TO
-      )
-
-      expect(await comfyPage.canvasOps.getScale()).toBeCloseTo(initialScale, 2)
-    })
-  })
-
-  test.describe('Comfy.Graph.LiveSelection', () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Canvas.NavigationMode',
-        'standard'
-      )
-    })
-
-    test('selects nodes mid-drag when enabled', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('Comfy.Graph.LiveSelection', true)
-      const { from, to } = await getClipNodesDragBox(comfyPage)
-
-      await comfyPage.page.mouse.move(from.x, from.y)
-      await comfyPage.page.mouse.down()
-      await comfyPage.page.mouse.move(to.x, to.y, { steps: 10 })
-
-      await expect
-        .poll(() => comfyPage.nodeOps.getSelectedGraphNodesCount())
-        .toBe(CLIP_NODE_COUNT)
-
-      await comfyPage.page.mouse.up()
-      await comfyPage.nextFrame()
-    })
-
-    test('defers selection to drag end when disabled', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting('Comfy.Graph.LiveSelection', false)
-      const { from, to } = await getClipNodesDragBox(comfyPage)
-
-      await comfyPage.page.mouse.move(from.x, from.y)
-      await comfyPage.page.mouse.down()
-      await comfyPage.page.mouse.move(to.x, to.y, { steps: 10 })
-      expect(await comfyPage.nodeOps.getSelectedGraphNodesCount()).toBe(0)
-
-      await comfyPage.page.mouse.up()
-      await expect
-        .poll(() => comfyPage.nodeOps.getSelectedGraphNodesCount())
-        .toBe(CLIP_NODE_COUNT)
-    })
-  })
-
-  test.describe('Comfy.Canvas.MouseWheelScroll', () => {
-    const WHEEL_POS = { x: 400, y: 400 }
-
-    test('wheel zooms when set to zoom', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Canvas.MouseWheelScroll',
-        'zoom'
-      )
-      const initialScale = await comfyPage.canvasOps.getScale()
-
-      await comfyPage.page.mouse.move(WHEEL_POS.x, WHEEL_POS.y)
-      await comfyPage.page.mouse.wheel(0, -120)
-      await comfyPage.page.mouse.wheel(0, -120)
-      await comfyPage.nextFrame()
-
-      expect(await comfyPage.canvasOps.getScale()).not.toBeCloseTo(
-        initialScale,
-        3
-      )
-    })
-
-    test('wheel pans when set to panning', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Canvas.MouseWheelScroll',
-        'panning'
-      )
-      const initialScale = await comfyPage.canvasOps.getScale()
-      const initialOffset = await comfyPage.canvasOps.getOffset()
-
-      await comfyPage.page.mouse.move(WHEEL_POS.x, WHEEL_POS.y)
-      await comfyPage.page.mouse.wheel(0, 120)
-      await comfyPage.page.mouse.wheel(0, 120)
-      await comfyPage.nextFrame()
-
-      expect(await comfyPage.canvasOps.getScale()).toBeCloseTo(initialScale, 3)
-      const offset = await comfyPage.canvasOps.getOffset()
-      expect(
-        Math.abs(offset[0] - initialOffset[0]) +
-          Math.abs(offset[1] - initialOffset[1])
-      ).toBeGreaterThan(1)
-    })
-  })
-
-  test.describe('Comfy.Canvas.LeftMouseClickBehavior', () => {
-    test('override to panning makes empty left-drag pan the canvas', async ({
-      comfyPage
-    }) => {
-      await test.step("Flip to 'select' then back to 'panning' (NavigationMode→custom)", async () => {
-        await comfyPage.settings.setSetting(
-          'Comfy.Canvas.LeftMouseClickBehavior',
-          'select'
-        )
-        await comfyPage.settings.setSetting(
-          'Comfy.Canvas.LeftMouseClickBehavior',
-          'panning'
-        )
-      })
-
-      await comfyPage.canvasOps.resetView()
-
-      const initialOffset = await comfyPage.canvasOps.getOffset()
-      await comfyPage.canvasOps.dragAndDrop(
-        { x: 200, y: 300 },
-        { x: 400, y: 500 }
-      )
-      const offset = await comfyPage.canvasOps.getOffset()
-
-      expect(
-        Math.abs(offset[0] - initialOffset[0]) +
-          Math.abs(offset[1] - initialOffset[1])
-      ).toBeGreaterThan(50)
-      expect(await comfyPage.nodeOps.getSelectedGraphNodesCount()).toBe(0)
-    })
-
-    test('override to select turns empty left-drag into a selection rectangle', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Canvas.LeftMouseClickBehavior',
-        'select'
-      )
-      const { from, to } = await getClipNodesDragBox(comfyPage)
-
-      await comfyPage.canvasOps.dragAndDrop(from, to)
-
-      await expect
-        .poll(() => comfyPage.nodeOps.getSelectedGraphNodesCount())
-        .toBe(CLIP_NODE_COUNT)
-    })
-  })
-
-  test.describe('Pointer settings', () => {
-    /**
-     * Press left-mouse at canvas-relative `pos`, hold for `holdMs` (0 = no
-     * hold), nudge by `(dx, dy)` absolute pixels, then release. Spec-local
-     * because it exists only to probe the CanvasPointer timing thresholds.
-     */
-    const holdDragAt = async (
-      comfyPage: ComfyPage,
-      pos: { x: number; y: number },
-      opts: { dx: number; dy: number; holdMs: number }
-    ) => {
-      const abs = await comfyPage.canvasOps.toAbsolute(pos)
-      await comfyPage.page.mouse.move(abs.x, abs.y)
-      await comfyPage.page.mouse.down()
-      await sleep(opts.holdMs)
-      await comfyPage.page.mouse.move(abs.x + opts.dx, abs.y + opts.dy)
-      await comfyPage.page.mouse.up()
-      await comfyPage.nextFrame()
-    }
-
-    test('DoubleClickTime controls whether two clicks open the title editor', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.DoubleClickTitleToEdit',
-        true
-      )
-      const clipNodes =
-        await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-      expect(
-        clipNodes,
-        'Default workflow must have CLIPTextEncode nodes'
-      ).toHaveLength(CLIP_NODE_COUNT)
-      const titlePos = await clipNodes[0].getTitlePosition()
-      const CLICK_GAP_MS = 200
-
-      await test.step(`Gap (${CLICK_GAP_MS}ms) exceeds DoubleClickTime → editor stays hidden`, async () => {
-        await comfyPage.settings.setSetting(
-          'Comfy.Pointer.DoubleClickTime',
-          100
-        )
-        await comfyPage.canvasOps.mouseClickAt(titlePos)
-        await sleep(CLICK_GAP_MS)
-        await comfyPage.canvasOps.mouseClickAt(titlePos)
-        await comfyPage.titleEditor.expectHidden()
-      })
-
-      await test.step(`Gap (${CLICK_GAP_MS}ms) within DoubleClickTime → editor opens`, async () => {
-        await comfyPage.settings.setSetting(
-          'Comfy.Pointer.DoubleClickTime',
-          1000
-        )
-        await comfyPage.canvasOps.mouseClickAt(titlePos)
-        await sleep(CLICK_GAP_MS)
-        await comfyPage.canvasOps.mouseClickAt(titlePos)
-        await comfyPage.titleEditor.expectVisible()
-      })
-    })
-
-    test('ClickBufferTime governs the click-vs-drag time threshold', async ({
-      comfyPage
-    }) => {
-      // Keep drift generous so only elapsed time distinguishes click vs drag.
-      await comfyPage.settings.setSetting('Comfy.Pointer.ClickDrift', 20)
-      const node = (
-        await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-      )[0]
-      const titlePos = await node.getTitlePosition()
-      const NUDGE = 2
-      const HOLD_MS = 250
-
-      await test.step(`Buffer=2000ms (hold=${HOLD_MS}ms within buffer) → click, node stays put`, async () => {
-        await comfyPage.settings.setSetting(
-          'Comfy.Pointer.ClickBufferTime',
-          2000
-        )
-        const before = await node.getPosition()
-        await holdDragAt(comfyPage, titlePos, {
-          dx: NUDGE,
-          dy: NUDGE,
-          holdMs: HOLD_MS
-        })
-        const after = await node.getPosition()
-        expect(after.x).toBeCloseTo(before.x, 0)
-        expect(after.y).toBeCloseTo(before.y, 0)
-      })
-
-      await test.step(`Buffer=50ms (hold=${HOLD_MS}ms exceeds buffer) → drag, node moves`, async () => {
-        await comfyPage.settings.setSetting('Comfy.Pointer.ClickBufferTime', 50)
-        const before = await node.getPosition()
-        await holdDragAt(comfyPage, titlePos, {
-          dx: NUDGE,
-          dy: NUDGE,
-          holdMs: HOLD_MS
-        })
-        const after = await node.getPosition()
-        expect(
-          Math.abs(after.x - before.x) + Math.abs(after.y - before.y)
-        ).toBeGreaterThan(0)
-      })
-    })
-
-    test('ClickDrift governs the click-vs-drag distance threshold', async ({
-      comfyPage
-    }) => {
-      // Keep buffer generous so only drift distance matters.
-      await comfyPage.settings.setSetting('Comfy.Pointer.ClickBufferTime', 2000)
-      const node = (
-        await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-      )[0]
-      const titlePos = await node.getTitlePosition()
-      const NUDGE = 8
-
-      await test.step(`Drift=20px (nudge=${NUDGE}px within tolerance) → click, node stays put`, async () => {
-        await comfyPage.settings.setSetting('Comfy.Pointer.ClickDrift', 20)
-        const before = await node.getPosition()
-        await holdDragAt(comfyPage, titlePos, {
-          dx: NUDGE,
-          dy: NUDGE,
-          holdMs: 0
-        })
-        const after = await node.getPosition()
-        expect(after.x).toBeCloseTo(before.x, 0)
-        expect(after.y).toBeCloseTo(before.y, 0)
-      })
-
-      await test.step(`Drift=1px (nudge=${NUDGE}px exceeds tolerance) → drag, node moves`, async () => {
-        await comfyPage.settings.setSetting('Comfy.Pointer.ClickDrift', 1)
-        const before = await node.getPosition()
-        await holdDragAt(comfyPage, titlePos, {
-          dx: NUDGE,
-          dy: NUDGE,
-          holdMs: 0
-        })
-        const after = await node.getPosition()
-        expect(
-          Math.abs(after.x - before.x) + Math.abs(after.y - before.y)
-        ).toBeGreaterThan(0)
-      })
-    })
-  })
-
-  test.describe('LiteGraph.Canvas.MaximumFps', () => {
-    // Behavioural FPS counting via rAF is not reliable under Playwright
-    // (CI jitter, background throttling, canvas-idle behaviour). Assert the
-    // render-loop throttle value instead — that is what actually governs
-    // frame cadence.
-    const getFrameGap = (comfyPage: ComfyPage) =>
-      comfyPage.page.evaluate(() => window.app!.canvas.maximumFps * 1000)
-
-    test('caps the render loop frame gap', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting('LiteGraph.Canvas.MaximumFps', 30)
-      await expect.poll(() => getFrameGap(comfyPage)).toBeCloseTo(1000 / 30, 1)
-
-      await comfyPage.settings.setSetting('LiteGraph.Canvas.MaximumFps', 60)
-      await expect.poll(() => getFrameGap(comfyPage)).toBeCloseTo(1000 / 60, 1)
-
-      await comfyPage.settings.setSetting('LiteGraph.Canvas.MaximumFps', 0)
-      await expect.poll(() => getFrameGap(comfyPage)).toBe(0)
-    })
-  })
-})

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

@@ -1,5 +1,4 @@
 import { expect } from '@playwright/test'
-import type { Route } from '@playwright/test'
 
 import type { Asset, ListAssetsResponse } from '@comfyorg/ingest-types'
 import { comfyPageFixture } from '@e2e/fixtures/ComfyPage'
@@ -13,35 +12,25 @@ function makeAssetsResponse(assets: Asset[]): ListAssetsResponse {
 }
 
 const CLOUD_ASSETS: Asset[] = [STABLE_CHECKPOINT, STABLE_CHECKPOINT_2]
-const WAITING_FOR_WIDGET_TYPE = 'waiting:type'
-const WAITING_FOR_WIDGET_VALUE = 'waiting:value'
 
 // Stub /api/assets before the app loads. The local ComfyUI backend has no
 // /api/assets endpoint (returns 503), which poisons the assets store on
 // first load. Narrow pattern avoids intercepting static /assets/*.js bundles.
 //
 // TODO: Consider moving this stub into ComfyPage fixture for all @cloud tests.
-const test = comfyPageFixture.extend<{
-  cloudAssetRequests: string[]
-  stubCloudAssets: void
-}>({
-  cloudAssetRequests: async ({ page: _page }, use) => {
-    await use([])
-  },
+const test = comfyPageFixture.extend<{ stubCloudAssets: void }>({
   stubCloudAssets: [
-    async ({ cloudAssetRequests, page }, use) => {
-      const pattern = /\/api\/assets(?:\?.*)?$/
-      const assetsRouteHandler = (route: Route) => {
-        cloudAssetRequests.push(route.request().url())
-        return route.fulfill({
+    async ({ page }, use) => {
+      const pattern = '**/api/assets?*'
+      await page.route(pattern, (route) =>
+        route.fulfill({
           status: 200,
           contentType: 'application/json',
           body: JSON.stringify(makeAssetsResponse(CLOUD_ASSETS))
         })
-      }
-      await page.route(pattern, assetsRouteHandler)
+      )
       await use()
-      await page.unroute(pattern, assetsRouteHandler)
+      await page.unroute(pattern)
     },
     { auto: true }
   ]
@@ -53,36 +42,23 @@ test.describe('Asset-supported node default value', { tag: '@cloud' }, () => {
   })
 
   test('should use first cloud asset when server default is not in assets', async ({
-    cloudAssetRequests,
     comfyPage
   }) => {
-    // Wait for the checkpoint asset query to complete and the existing widget
-    // to upgrade into asset mode before creating a fresh node. The current
-    // default node may keep a previously resolved value; what matters is that
-    // new nodes resolve against the cloud asset list after the fetch.
-    await expect
-      .poll(() =>
-        cloudAssetRequests.some((url) => {
-          const includeTags =
-            new URL(url).searchParams.get('include_tags') ?? ''
-          return includeTags.split(',').includes('checkpoints')
-        })
-      )
-      .toBe(true)
-
+    // The default workflow contains a CheckpointLoaderSimple node whose
+    // server default (from object_info) is a local file not in cloud assets.
+    // Wait for the existing node's asset widget to mount, confirming the
+    // assets store has been populated from the stub before adding a new node.
     await expect
       .poll(
         () =>
-          comfyPage.page.evaluate((waitingForWidgetType) => {
+          comfyPage.page.evaluate(() => {
             const node = window.app!.graph.nodes.find(
               (n: { type: string }) => n.type === 'CheckpointLoaderSimple'
             )
-            return (
-              node?.widgets?.find(
-                (w: { name: string }) => w.name === 'ckpt_name'
-              )?.type ?? waitingForWidgetType
-            )
-          }, WAITING_FOR_WIDGET_TYPE),
+            return node?.widgets?.find(
+              (w: { name: string }) => w.name === 'ckpt_name'
+            )?.type
+          }),
         { timeout: 10_000 }
       )
       .toBe('asset')
@@ -105,22 +81,15 @@ test.describe('Asset-supported node default value', { tag: '@cloud' }, () => {
     await expect
       .poll(
         () =>
-          comfyPage.page.evaluate(
-            ({ id, waitingForWidgetType, waitingForWidgetValue }) => {
-              const node = window.app!.graph.getNodeById(id)
-              const widget = node?.widgets?.find(
-                (w: { name: string }) => w.name === 'ckpt_name'
-              )
-              if (widget?.type !== 'asset') return waitingForWidgetType
-              const val = String(widget?.value ?? '')
-              return val === 'Select model' ? waitingForWidgetValue : val
-            },
-            {
-              id: nodeId,
-              waitingForWidgetType: WAITING_FOR_WIDGET_TYPE,
-              waitingForWidgetValue: WAITING_FOR_WIDGET_VALUE
-            }
-          ),
+          comfyPage.page.evaluate((id) => {
+            const node = window.app!.graph.getNodeById(id)
+            const widget = node?.widgets?.find(
+              (w: { name: string }) => w.name === 'ckpt_name'
+            )
+            if (widget?.type !== 'asset') return 'waiting:type'
+            const val = String(widget?.value ?? '')
+            return val === 'Select model' ? 'waiting:value' : val
+          }, nodeId),
         { timeout: 15_000 }
       )
       .toBe(CLOUD_ASSETS[0].name)

browser_tests/tests/dialogs/publishDialog.spec.ts

@@ -1,306 +0,0 @@
-import { expect } from '@playwright/test'
-
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import type { PublishDialog } from '@e2e/fixtures/components/PublishDialog'
-
-import { publishFixture as test } from '@e2e/fixtures/helpers/PublishApiHelper'
-
-const PUBLISH_FEATURE_FLAGS = {
-  comfyhub_upload_enabled: true,
-  comfyhub_profile_gate_enabled: true
-} as const
-
-async function saveAndOpenPublishDialog(
-  comfyPage: ComfyPage,
-  dialog: PublishDialog,
-  workflowName: string
-): Promise<void> {
-  await comfyPage.menu.topbar.saveWorkflow(workflowName)
-  const overwriteDialog = comfyPage.page.locator(
-    '.p-dialog:has-text("Overwrite")'
-  )
-  // Bounded wait: point-in-time isVisible() can miss dialogs that open
-  // slightly after saveWorkflow() resolves.
-  try {
-    await overwriteDialog.waitFor({ state: 'visible', timeout: 500 })
-    await comfyPage.confirmDialog.click('overwrite')
-  } catch {
-    // No overwrite dialog — workflow name was unique.
-  }
-
-  await dialog.open()
-}
-
-test.describe('Publish dialog - wizard navigation', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks()
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-publish-wf')
-  })
-
-  test('opens on the Describe step by default', async ({ publishDialog }) => {
-    await expect(publishDialog.describeStep).toBeVisible()
-    await expect(publishDialog.nameInput).toBeVisible()
-    await expect(publishDialog.descriptionTextarea).toBeVisible()
-  })
-
-  test('pre-fills workflow name from active workflow', async ({
-    publishDialog
-  }) => {
-    await expect(publishDialog.nameInput).toHaveValue(/test-publish-wf/)
-  })
-
-  test('Next button navigates to Examples step', async ({ publishDialog }) => {
-    await publishDialog.goNext()
-    await expect(publishDialog.describeStep).toBeHidden()
-    // Examples step should show thumbnail toggle and upload area
-    await expect(
-      publishDialog.root.getByText('Select a thumbnail')
-    ).toBeVisible()
-  })
-
-  test('Back button returns to Describe step from Examples', async ({
-    publishDialog
-  }) => {
-    await publishDialog.goNext()
-    await expect(publishDialog.describeStep).toBeHidden()
-
-    await publishDialog.goBack()
-    await expect(publishDialog.describeStep).toBeVisible()
-  })
-
-  test('navigates through all steps to Finish', async ({ publishDialog }) => {
-    await publishDialog.goNext() // → Examples
-    await publishDialog.goNext() // → Finish
-    await expect(publishDialog.finishStep).toBeVisible()
-    await expect(publishDialog.publishButton).toBeVisible()
-  })
-
-  test('clicking nav step navigates directly', async ({ publishDialog }) => {
-    await publishDialog.goToStep('Finish publishing')
-    await expect(publishDialog.finishStep).toBeVisible()
-
-    await publishDialog.goToStep('Describe your workflow')
-    await expect(publishDialog.describeStep).toBeVisible()
-  })
-
-  test('closes dialog via Escape key', async ({ comfyPage, publishDialog }) => {
-    await comfyPage.page.keyboard.press('Escape')
-    await expect(publishDialog.root).toBeHidden()
-  })
-})
-
-test.describe('Publish dialog - Describe step', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks()
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-describe-wf')
-  })
-
-  test('allows editing the workflow name', async ({ publishDialog }) => {
-    await publishDialog.nameInput.clear()
-    await publishDialog.nameInput.fill('My Custom Workflow')
-    await expect(publishDialog.nameInput).toHaveValue('My Custom Workflow')
-  })
-
-  test('allows editing the description', async ({ publishDialog }) => {
-    await publishDialog.descriptionTextarea.fill(
-      'A great workflow for anime art'
-    )
-    await expect(publishDialog.descriptionTextarea).toHaveValue(
-      'A great workflow for anime art'
-    )
-  })
-
-  test('displays tag suggestions from mocked API', async ({
-    publishDialog
-  }) => {
-    await expect(publishDialog.root.getByText('anime')).toBeVisible()
-    await expect(publishDialog.root.getByText('upscale')).toBeVisible()
-  })
-
-  // TODO(#11548): Tag click emits update:tags but the tag does not appear in
-  // the active list during E2E. Needs investigation of the parent state
-  // binding.
-  test.fixme('clicking a tag suggestion adds it', async ({ publishDialog }) => {
-    await publishDialog.root.getByText('anime').click()
-
-    await expect(publishDialog.tagsInput.getByText('anime')).toBeVisible()
-  })
-})
-
-test.describe('Publish dialog - Examples step', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks()
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-examples-wf')
-    await publishDialog.goNext() // Navigate to Examples step
-  })
-
-  test('shows thumbnail type toggle options', async ({ publishDialog }) => {
-    await expect(
-      publishDialog.root.getByText('Image', { exact: true })
-    ).toBeVisible()
-    await expect(
-      publishDialog.root.getByText('Video', { exact: true })
-    ).toBeVisible()
-    await expect(
-      publishDialog.root.getByText('Image comparison', { exact: true })
-    ).toBeVisible()
-  })
-
-  test('shows example image upload tile', async ({ publishDialog }) => {
-    await expect(
-      publishDialog.root.getByRole('button', { name: 'Upload example image' })
-    ).toBeVisible()
-  })
-})
-
-test.describe('Publish dialog - Finish step with profile', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks({ hasProfile: true })
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-finish-wf')
-    await publishDialog.goToStep('Finish publishing')
-  })
-
-  test('shows profile card with username', async ({ publishDialog }) => {
-    await expect(publishDialog.finishStep).toBeVisible()
-    await expect(publishDialog.root.getByText('@testuser')).toBeVisible()
-    await expect(publishDialog.root.getByText('Test User')).toBeVisible()
-  })
-
-  test('publish button is enabled when no private assets', async ({
-    publishDialog
-  }) => {
-    await expect(publishDialog.publishButton).toBeEnabled()
-  })
-})
-
-test.describe('Publish dialog - Finish step with private assets', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks({
-      hasProfile: true,
-      hasPrivateAssets: true
-    })
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-assets-wf')
-    await publishDialog.goToStep('Finish publishing')
-  })
-
-  test('publish button is disabled until assets acknowledged', async ({
-    publishDialog
-  }) => {
-    await expect(publishDialog.finishStep).toBeVisible()
-    await expect(publishDialog.publishButton).toBeDisabled()
-
-    const checkbox = publishDialog.finishStep.getByRole('checkbox')
-    await checkbox.check()
-
-    await expect(publishDialog.publishButton).toBeEnabled()
-  })
-})
-
-test.describe('Publish dialog - no profile', () => {
-  test.beforeEach(async ({ comfyPage, publishApi, publishDialog }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks({ hasProfile: false })
-    await saveAndOpenPublishDialog(
-      comfyPage,
-      publishDialog,
-      'test-noprofile-wf'
-    )
-    await publishDialog.goToStep('Finish publishing')
-  })
-
-  test('shows profile creation prompt when user has no profile', async ({
-    publishDialog
-  }) => {
-    await expect(publishDialog.profilePrompt).toBeVisible()
-    await expect(
-      publishDialog.root.getByText('Create a profile to publish to ComfyHub')
-    ).toBeVisible()
-  })
-
-  test('clicking create profile CTA shows profile creation form', async ({
-    publishDialog
-  }) => {
-    await publishDialog.root
-      .getByRole('button', { name: 'Create a profile' })
-      .click()
-    await expect(publishDialog.gateFlow).toBeVisible()
-  })
-})
-
-test.describe('Publish dialog - unsaved workflow', () => {
-  test.beforeEach(async ({ comfyPage, publishApi }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-    await publishApi.setupDefaultMocks()
-    // Don't save workflow — open dialog on the default temporary workflow
-  })
-
-  test('shows save prompt for temporary workflow', async ({
-    comfyPage,
-    publishDialog
-  }) => {
-    // Create a new workflow to ensure it's temporary
-    await comfyPage.menu.topbar.triggerTopbarCommand(['New'])
-    await publishDialog.open()
-
-    await expect(publishDialog.savePrompt).toBeVisible()
-    await expect(
-      publishDialog.root.getByText(
-        'You must save your workflow before publishing'
-      )
-    ).toBeVisible()
-    // Nav should be hidden when save is required
-    await expect(publishDialog.nav).toBeHidden()
-  })
-})
-
-test.describe('Publish dialog - submission', () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.featureFlags.setFlags(PUBLISH_FEATURE_FLAGS)
-  })
-
-  test('successful publish closes dialog', async ({
-    comfyPage,
-    publishApi,
-    publishDialog
-  }) => {
-    await publishApi.setupDefaultMocks({ hasProfile: true })
-    await saveAndOpenPublishDialog(comfyPage, publishDialog, 'test-submit-wf')
-    await publishDialog.goToStep('Finish publishing')
-    await expect(publishDialog.finishStep).toBeVisible()
-
-    await publishDialog.publishButton.click()
-    await expect(publishDialog.root).toBeHidden({ timeout: 10_000 })
-  })
-
-  test('failed publish shows error toast', async ({
-    comfyPage,
-    publishApi,
-    publishDialog
-  }) => {
-    await publishApi.setupDefaultMocks({ hasProfile: true })
-    // Override publish mock with error response
-    await publishApi.mockPublishWorkflowError(500, 'Internal error')
-
-    await saveAndOpenPublishDialog(
-      comfyPage,
-      publishDialog,
-      'test-submit-fail-wf'
-    )
-    await publishDialog.goToStep('Finish publishing')
-    await expect(publishDialog.finishStep).toBeVisible()
-
-    await publishDialog.publishButton.click()
-
-    // Error toast should appear
-    await expect(comfyPage.toast.visibleToasts.first()).toBeVisible({
-      timeout: 10_000
-    })
-    // Dialog should remain open
-    await expect(publishDialog.root).toBeVisible()
-  })
-})

browser_tests/tests/errorDialog.spec.ts

@@ -8,7 +8,7 @@ import { TestIds } from '@e2e/fixtures/selectors'
 import {
   interceptClipboardWrite,
   getClipboardText
-} from '@e2e/fixtures/utils/clipboardSpy'
+} from '@e2e/helpers/clipboardSpy'
 
 async function triggerConfigureError(
   comfyPage: ComfyPage,

browser_tests/tests/linkNodeInteractionSettings.spec.ts

@@ -1,208 +0,0 @@
-import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '@e2e/fixtures/ComfyPage'
-import { DefaultGraphPositions } from '@e2e/fixtures/constants/defaultGraphPositions'
-
-const VAE_DECODE_SAMPLES_INPUT_SLOT = 0
-const DEFAULT_GROUP_TITLE = 'Group'
-
-test.describe('Link & node interaction settings', { tag: '@canvas' }, () => {
-  test.describe('Comfy.LinkRelease.Action', () => {
-    test('"search box" opens node search on link release', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.LinkRelease.Action',
-        'search box'
-      )
-      await comfyPage.canvasOps.disconnectEdge()
-      await expect(comfyPage.searchBoxV2.input).toBeVisible()
-    })
-
-    test('"context menu" opens litegraph connection menu on link release', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.LinkRelease.Action',
-        'context menu'
-      )
-      await comfyPage.canvasOps.disconnectEdge()
-      await expect(comfyPage.contextMenu.litegraphContextMenu).toBeVisible()
-    })
-
-    test('"no action" suppresses both search box and context menu', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.LinkRelease.Action',
-        'no action'
-      )
-      await comfyPage.canvasOps.disconnectEdge()
-      await expect(comfyPage.searchBoxV2.input).toBeHidden()
-      await expect(comfyPage.contextMenu.litegraphContextMenu).toBeHidden()
-    })
-  })
-
-  test.describe('Comfy.LinkRelease.ActionShift', () => {
-    test('shift+drag dispatches to ActionShift (not Action)', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.LinkRelease.Action',
-        'no action'
-      )
-      await comfyPage.settings.setSetting(
-        'Comfy.LinkRelease.ActionShift',
-        'search box'
-      )
-
-      await comfyPage.canvasOps.disconnectEdge({ modifiers: ['Shift'] })
-
-      await expect(comfyPage.searchBoxV2.input).toBeVisible()
-    })
-  })
-
-  test.describe('Comfy.Node.DoubleClickTitleToEdit', () => {
-    test('enabled → double-click on node title opens editor', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.DoubleClickTitleToEdit',
-        true
-      )
-      const [node] = await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-      await comfyPage.canvasOps.mouseDblclickAt(await node.getTitlePosition())
-      await comfyPage.titleEditor.expectVisible()
-    })
-
-    test('disabled → double-click on node title stays hidden', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.DoubleClickTitleToEdit',
-        false
-      )
-      const [node] = await comfyPage.nodeOps.getNodeRefsByType('CLIPTextEncode')
-      await comfyPage.canvasOps.mouseDblclickAt(await node.getTitlePosition())
-      await comfyPage.titleEditor.expectHidden()
-    })
-  })
-
-  test.describe('Comfy.Group.DoubleClickTitleToEdit', () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.workflow.loadWorkflow('groups/single_group_only')
-    })
-
-    test('enabled → double-click on group title opens editor', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Group.DoubleClickTitleToEdit',
-        true
-      )
-      await comfyPage.canvasOps.dblclickGroupTitle(DEFAULT_GROUP_TITLE)
-      await comfyPage.titleEditor.expectVisible()
-    })
-
-    test('disabled → double-click on group title stays hidden', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Group.DoubleClickTitleToEdit',
-        false
-      )
-      await comfyPage.canvasOps.dblclickGroupTitle(DEFAULT_GROUP_TITLE)
-      await comfyPage.titleEditor.expectHidden()
-    })
-  })
-
-  test.describe('Comfy.Node.BypassAllLinksOnDelete', () => {
-    test('enabled → deleting KSampler bridges EmptyLatentImage → VAEDecode.samples', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.BypassAllLinksOnDelete',
-        true
-      )
-      const [kSampler] = await comfyPage.nodeOps.getNodeRefsByType('KSampler')
-      const [emptyLatent] =
-        await comfyPage.nodeOps.getNodeRefsByType('EmptyLatentImage')
-      const [vaeDecode] = await comfyPage.nodeOps.getNodeRefsByType('VAEDecode')
-      const vaeSamplesInput = await vaeDecode.getInput(
-        VAE_DECODE_SAMPLES_INPUT_SLOT
-      )
-
-      await test.step('precondition: KSampler feeds VAEDecode.samples', async () => {
-        expect(
-          (await vaeSamplesInput.getLink())?.origin_id,
-          'VAEDecode.samples should originate from KSampler before delete'
-        ).toBe(kSampler.id)
-      })
-
-      await kSampler.delete()
-
-      await expect
-        .poll(async () => (await vaeSamplesInput.getLink())?.origin_id ?? null)
-        .toBe(emptyLatent.id)
-    })
-
-    test('disabled → deleting KSampler drops VAEDecode.samples', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.BypassAllLinksOnDelete',
-        false
-      )
-      const [kSampler] = await comfyPage.nodeOps.getNodeRefsByType('KSampler')
-      const [vaeDecode] = await comfyPage.nodeOps.getNodeRefsByType('VAEDecode')
-      const vaeSamplesInput = await vaeDecode.getInput(
-        VAE_DECODE_SAMPLES_INPUT_SLOT
-      )
-
-      await kSampler.delete()
-
-      await expect.poll(() => vaeSamplesInput.getLink()).toBeNull()
-    })
-  })
-
-  test.describe('Comfy.Node.MiddleClickRerouteNode', () => {
-    async function countReroutes(comfyPage: ComfyPage): Promise<number> {
-      return (await comfyPage.nodeOps.getNodeRefsByType('Reroute')).length
-    }
-
-    test('enabled → middle-click on an output slot creates a Reroute', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.MiddleClickRerouteNode',
-        true
-      )
-      const before = await countReroutes(comfyPage)
-
-      await comfyPage.canvasOps.middleClick(
-        DefaultGraphPositions.loadCheckpointNodeClipOutputSlot
-      )
-
-      await expect.poll(() => countReroutes(comfyPage)).toBe(before + 1)
-    })
-
-    test('disabled → middle-click on an output slot does nothing', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.Node.MiddleClickRerouteNode',
-        false
-      )
-      const before = await countReroutes(comfyPage)
-
-      await comfyPage.canvasOps.middleClick(
-        DefaultGraphPositions.loadCheckpointNodeClipOutputSlot
-      )
-      await comfyPage.nextFrame()
-
-      expect(await countReroutes(comfyPage)).toBe(before)
-    })
-  })
-})

browser_tests/tests/loginButton.spec.ts

@@ -1,106 +0,0 @@
-import type { Page } from '@playwright/test'
-import { expect } from '@playwright/test'
-
-import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import { SignInDialog } from '@e2e/fixtures/components/SignInDialog'
-import { TestIds } from '@e2e/fixtures/selectors'
-
-/**
- * Enable the show_signin_button server feature flag so LoginButton renders
- * in WorkflowTabs (which uses `flags.showSignInButton ?? isDesktop`).
- * The flag is reset automatically on each fresh page load in beforeEach.
- */
-async function enableLoginButtonFlag(page: Page): Promise<void> {
-  await page.evaluate(() => {
-    window.app!.api.serverFeatureFlags.value = {
-      ...window.app!.api.serverFeatureFlags.value,
-      show_signin_button: true
-    }
-  })
-}
-
-test.describe('Login Button', { tag: ['@ui'] }, () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.setup()
-  })
-
-  test.describe('Visibility', () => {
-    test('button is hidden when show_signin_button flag is off', async ({
-      comfyPage
-    }) => {
-      await comfyPage.page.evaluate(() => {
-        window.app!.api.serverFeatureFlags.value = {
-          ...window.app!.api.serverFeatureFlags.value,
-          show_signin_button: false
-        }
-      })
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.loginButton)
-      ).toBeHidden()
-    })
-
-    test('button is visible when show_signin_button flag is enabled', async ({
-      comfyPage
-    }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.loginButton)
-      ).toBeVisible()
-    })
-  })
-
-  test.describe('ARIA', () => {
-    test('button has correct aria-label', async ({ comfyPage }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      const button = comfyPage.page.getByTestId(TestIds.topbar.loginButton)
-      await expect(button).toHaveAttribute('aria-label', /.+/)
-    })
-  })
-
-  test.describe('Click behaviour', () => {
-    test('clicking the button opens the sign-in dialog', async ({
-      comfyPage
-    }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      const dialog = new SignInDialog(comfyPage.page)
-      await comfyPage.page.getByTestId(TestIds.topbar.loginButton).click()
-      await expect(dialog.root).toBeVisible()
-    })
-  })
-
-  test.describe('Hover popover', () => {
-    test('hovering shows an informational popover', async ({ comfyPage }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      await comfyPage.page.getByTestId(TestIds.topbar.loginButton).hover()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.loginButtonPopover)
-      ).toBeVisible()
-    })
-
-    test('popover contains a Learn more link', async ({ comfyPage }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      await comfyPage.page.getByTestId(TestIds.topbar.loginButton).hover()
-      const learnMoreLink = comfyPage.page.getByTestId(
-        TestIds.topbar.loginButtonPopoverLearnMore
-      )
-      await expect(learnMoreLink).toBeVisible()
-      await expect(learnMoreLink).toHaveAttribute('href', /api-nodes/)
-    })
-
-    test('popover hides after mouse leaves the button area', async ({
-      comfyPage
-    }) => {
-      await enableLoginButtonFlag(comfyPage.page)
-      const button = comfyPage.page.getByTestId(TestIds.topbar.loginButton)
-      await button.hover()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.loginButtonPopover)
-      ).toBeVisible()
-
-      await comfyPage.canvas.hover()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.topbar.loginButtonPopover)
-      ).toBeHidden()
-    })
-  })
-})

browser_tests/tests/nodeGhostPlacement.spec.ts

@@ -22,14 +22,18 @@ async function addGhostAtCenter(comfyPage: ComfyPage) {
   await comfyPage.page.mouse.move(centerX, centerY)
   await comfyPage.nextFrame()
 
-  const nodeRef = await comfyPage.nodeOps.addNode(
-    'VAEDecode',
-    { ghost: true },
-    { x: centerX, y: centerY }
+  const nodeId = await comfyPage.page.evaluate(
+    ([clientX, clientY]) => {
+      const node = window.LiteGraph!.createNode('VAEDecode')!
+      const event = new MouseEvent('click', { clientX, clientY })
+      window.app!.graph.add(node, { ghost: true, dragEvent: event })
+      return node.id
+    },
+    [centerX, centerY] as const
   )
   await comfyPage.nextFrame()
 
-  return { nodeId: nodeRef.id, centerX, centerY }
+  return { nodeId, centerX, centerY }
 }
 
 function getNodeById(comfyPage: ComfyPage, nodeId: number | string) {
@@ -76,6 +80,7 @@ for (const mode of ['litegraph', 'vue'] as const) {
         },
         [centerX, centerY] as const
       )
+      await comfyPage.nextFrame()
 
       expect(Math.abs(result.diffX)).toBeLessThan(5)
       expect(Math.abs(result.diffY)).toBeLessThan(5)
@@ -148,184 +153,5 @@ for (const mode of ['litegraph', 'vue'] as const) {
       const after = await getNodeById(comfyPage, nodeId)
       expect(after).toBeNull()
     })
-
-    test('moving ghost onto existing node and clicking places correctly', async ({
-      comfyPage
-    }) => {
-      // Get existing KSampler node from the default workflow
-      const [ksamplerRef] =
-        await comfyPage.nodeOps.getNodeRefsByTitle('KSampler')
-      const ksamplerPos = await ksamplerRef.getPosition()
-      const ksamplerSize = await ksamplerRef.getSize()
-      const targetX = Math.round(ksamplerPos.x + ksamplerSize.width / 2)
-      const targetY = Math.round(ksamplerPos.y + ksamplerSize.height / 2)
-
-      // Start ghost placement away from the existing node
-      const startX = 50
-      const startY = 50
-      await comfyPage.page.mouse.move(startX, startY, { steps: 20 })
-      await comfyPage.nextFrame()
-
-      const ghostRef = await comfyPage.nodeOps.addNode(
-        'VAEDecode',
-        { ghost: true },
-        { x: startX, y: startY }
-      )
-      await comfyPage.nextFrame()
-
-      // Move ghost onto the existing node
-      await comfyPage.page.mouse.move(targetX, targetY, { steps: 20 })
-      await comfyPage.nextFrame()
-
-      // Click to finalize — on top of the existing node
-      await comfyPage.page.mouse.click(targetX, targetY)
-      await comfyPage.nextFrame()
-
-      // Ghost should be placed (no longer ghost)
-      const ghostResult = await getNodeById(comfyPage, ghostRef.id)
-      expect(ghostResult).not.toBeNull()
-      expect(ghostResult!.ghost).toBe(false)
-
-      // Ghost node should have moved from its start position toward where we clicked
-      const ghostPos = await ghostRef.getPosition()
-      expect(
-        Math.abs(ghostPos.x - startX) > 20 || Math.abs(ghostPos.y - startY) > 20
-      ).toBe(true)
-
-      // Existing node should NOT be selected
-      const selectedIds = await comfyPage.nodeOps.getSelectedNodeIds()
-      expect(selectedIds).not.toContain(ksamplerRef.id)
-    })
-
-    test(
-      'subgraph blueprint added from search box enters ghost mode',
-      { tag: ['@subgraph'] },
-      async ({ comfyPage }) => {
-        await comfyPage.settings.setSetting(
-          'Comfy.NodeSearchBoxImpl',
-          'default'
-        )
-
-        // Convert a node to a subgraph and publish it as a blueprint
-        const nodeRef = await comfyPage.nodeOps.getNodeRefById('3')
-        await nodeRef.click('title')
-        await comfyPage.nextFrame()
-        await comfyPage.command.executeCommand('Comfy.Graph.ConvertToSubgraph')
-        await comfyPage.nextFrame()
-        await comfyPage.nextFrame()
-        const subgraphNodes =
-          await comfyPage.nodeOps.getNodeRefsByTitle('New Subgraph')
-        expect(subgraphNodes).toHaveLength(1)
-        const subgraphNode = subgraphNodes[0]
-
-        const blueprintName = `ghost-test-${Date.now()}`
-        await subgraphNode.click('title')
-        await comfyPage.command.executeCommand('Comfy.PublishSubgraph', {
-          name: blueprintName
-        })
-        await expect(comfyPage.visibleToasts).toHaveCount(1, { timeout: 5000 })
-        await comfyPage.toast.closeToasts(1)
-
-        const nodeCountBefore = await comfyPage.nodeOps.getGraphNodesCount()
-
-        // Open v2 search box and search for the published blueprint
-        const { searchBoxV2 } = comfyPage
-        await searchBoxV2.open()
-
-        await searchBoxV2.input.fill(blueprintName)
-        await expect(searchBoxV2.results.first()).toBeVisible()
-
-        // Click the result to add the node (v2 search box uses ghost mode)
-        await searchBoxV2.results.first().click()
-        await comfyPage.nextFrame()
-
-        // A new node should exist on the graph in ghost mode
-        const nodeCountAfter = await comfyPage.nodeOps.getGraphNodesCount()
-        expect(nodeCountAfter).toBe(nodeCountBefore + 1)
-
-        const ghostNodeId = await comfyPage.page.evaluate(() => {
-          return window.app!.canvas.state.ghostNodeId
-        })
-        expect(ghostNodeId).not.toBeNull()
-
-        const ghostState = await getNodeById(comfyPage, ghostNodeId!)
-        expect(ghostState).not.toBeNull()
-        expect(ghostState!.ghost).toBe(true)
-
-        // Wait for search box to close, then click to confirm placement
-        await expect(searchBoxV2.input).toBeHidden()
-        await comfyPage.nextFrame()
-        const viewport = comfyPage.page.viewportSize()!
-        await comfyPage.page.mouse.click(
-          Math.round(viewport.width / 2),
-          Math.round(viewport.height / 2)
-        )
-        await comfyPage.nextFrame()
-
-        const afterPlace = await getNodeById(comfyPage, ghostNodeId!)
-        expect(afterPlace).not.toBeNull()
-        expect(afterPlace!.ghost).toBe(false)
-      }
-    )
-
-    test(
-      'Escape during ghost placement inside a subgraph cancels the ghost without exiting the subgraph',
-      { tag: ['@subgraph'] },
-      async ({ comfyPage }) => {
-        await comfyPage.searchBoxV2.setup()
-        await comfyPage.settings.setSetting(
-          'Comfy.NodeSearchBoxImpl.FollowCursor',
-          true
-        )
-        await comfyPage.workflow.loadWorkflow('subgraphs/basic-subgraph')
-
-        if (mode === 'vue') {
-          await comfyPage.vueNodes.waitForNodes()
-          await comfyPage.vueNodes.enterSubgraph('2')
-        } else {
-          const subgraphNode = await comfyPage.nodeOps.getNodeRefById('2')
-          await subgraphNode.navigateIntoSubgraph()
-        }
-        await expect.poll(() => comfyPage.subgraph.isInSubgraph()).toBe(true)
-
-        const subgraphId = await comfyPage.subgraph.getActiveGraphId()
-        const initialNodeCount = await comfyPage.subgraph.getNodeCount()
-
-        const { searchBoxV2 } = comfyPage
-        await searchBoxV2.open()
-        await searchBoxV2.input.fill('KSampler')
-        await expect(searchBoxV2.results.first()).toBeVisible()
-        await comfyPage.page.keyboard.press('Enter')
-        await expect(searchBoxV2.input).toBeHidden()
-
-        await expect
-          .poll(() =>
-            comfyPage.page.evaluate(
-              () => window.app!.canvas.state.ghostNodeId != null
-            )
-          )
-          .toBe(true)
-
-        await comfyPage.keyboard.press('Escape')
-
-        await expect
-          .poll(() => comfyPage.subgraph.isInSubgraph(), {
-            message:
-              'Escape during ghost placement should cancel the ghost, not exit the subgraph'
-          })
-          .toBe(true)
-        await expect
-          .poll(() => comfyPage.subgraph.getActiveGraphId())
-          .toBe(subgraphId)
-        await expect
-          .poll(() =>
-            comfyPage.page.evaluate(() => window.app!.canvas.state.ghostNodeId)
-          )
-          .toBeNull()
-        await expect
-          .poll(() => comfyPage.subgraph.getNodeCount())
-          .toBe(initialNodeCount)
-      }
-    )
   })
 }

browser_tests/tests/nodeHelp.spec.ts

@@ -3,7 +3,7 @@ import {
   comfyPageFixture as test
 } from '@e2e/fixtures/ComfyPage'
 import type { ComfyPage } from '@e2e/fixtures/ComfyPage'
-import { fitToViewInstant } from '@e2e/fixtures/utils/fitToView'
+import { fitToViewInstant } from '@e2e/helpers/fitToView'
 import type { WorkspaceStore } from '@e2e/types/globals'
 import type { NodeReference } from '@e2e/fixtures/utils/litegraphUtils'
 

browser_tests/tests/nodeSearchBoxV2.spec.ts

@@ -5,19 +5,32 @@ import {
 
 test.describe('Node search box V2', { tag: '@node' }, () => {
   test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.searchBoxV2.setup()
+    await comfyPage.settings.setSetting('Comfy.UseNewMenu', 'Disabled')
+    await comfyPage.settings.setSetting('Comfy.NodeSearchBoxImpl', 'default')
+    await comfyPage.settings.setSetting(
+      'Comfy.LinkRelease.Action',
+      'search box'
+    )
+    await comfyPage.settings.setSetting(
+      'Comfy.LinkRelease.ActionShift',
+      'search box'
+    )
+    await comfyPage.searchBoxV2.reload(comfyPage)
   })
 
   test('Can open search and add node', async ({ comfyPage }) => {
     const { searchBoxV2 } = comfyPage
     const initialCount = await comfyPage.nodeOps.getGraphNodesCount()
 
-    await searchBoxV2.open()
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
+
     await searchBoxV2.input.fill('KSampler')
     await expect(searchBoxV2.results.first()).toBeVisible()
 
     await comfyPage.page.keyboard.press('Enter')
     await expect(searchBoxV2.input).toBeHidden()
+
     await expect
       .poll(() => comfyPage.nodeOps.getGraphNodesCount())
       .toBe(initialCount + 1)
@@ -27,28 +40,33 @@ test.describe('Node search box V2', { tag: '@node' }, () => {
     const { searchBoxV2 } = comfyPage
     const initialCount = await comfyPage.nodeOps.getGraphNodesCount()
 
-    await searchBoxV2.open()
-    // Default results should be visible without typing.
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
+
+    // Default results should be visible without typing
     await expect(searchBoxV2.results.first()).toBeVisible()
 
+    // Enter should add the first (selected) result
     await comfyPage.page.keyboard.press('Enter')
     await expect(searchBoxV2.input).toBeHidden()
+
     await expect
       .poll(() => comfyPage.nodeOps.getGraphNodesCount())
       .toBe(initialCount + 1)
   })
 
   test.describe('Category navigation', () => {
-    test('Bookmarked filter shows only bookmarked nodes', async ({
-      comfyPage
-    }) => {
+    test('Favorites shows only bookmarked nodes', async ({ comfyPage }) => {
       const { searchBoxV2 } = comfyPage
       await comfyPage.settings.setSetting('Comfy.NodeLibrary.Bookmarks.V2', [
         'KSampler'
       ])
+      await searchBoxV2.reload(comfyPage)
 
-      await searchBoxV2.open()
-      await searchBoxV2.rootCategoryButton('favorites').click()
+      await comfyPage.canvasOps.doubleClick()
+      await expect(searchBoxV2.input).toBeVisible()
+
+      await searchBoxV2.categoryButton('favorites').click()
 
       await expect(searchBoxV2.results).toHaveCount(1)
       await expect(searchBoxV2.results.first()).toContainText('KSampler')
@@ -59,10 +77,13 @@ test.describe('Node search box V2', { tag: '@node' }, () => {
     }) => {
       const { searchBoxV2 } = comfyPage
 
-      await searchBoxV2.open()
+      await comfyPage.canvasOps.doubleClick()
+      await expect(searchBoxV2.input).toBeVisible()
+
       await searchBoxV2.categoryButton('sampling').click()
 
       await expect(searchBoxV2.results.first()).toBeVisible()
+      await expect.poll(() => searchBoxV2.results.count()).toBeGreaterThan(0)
     })
   })
 
@@ -70,23 +91,26 @@ test.describe('Node search box V2', { tag: '@node' }, () => {
     test('Can filter by input type via filter bar', async ({ comfyPage }) => {
       const { searchBoxV2 } = comfyPage
 
-      await searchBoxV2.open()
+      await comfyPage.canvasOps.doubleClick()
+      await expect(searchBoxV2.input).toBeVisible()
 
-      await test.step('Open Input filter popover', async () => {
-        await searchBoxV2.typeFilterButton('input').click()
-        await expect(searchBoxV2.filterOptions.first()).toBeVisible()
-      })
+      // Click "Input" filter chip in the filter bar
+      await searchBoxV2.filterBarButton('Input').click()
 
-      await test.step('Select MODEL type', async () => {
-        await searchBoxV2.filterSearch.fill('MODEL')
-        await searchBoxV2.filterOptions
-          .filter({ hasText: 'MODEL' })
-          .first()
-          .click()
-      })
+      // Filter options should appear
+      await expect(searchBoxV2.filterOptions.first()).toBeVisible()
 
-      await expect(searchBoxV2.filterChips).toHaveCount(1)
-      await expect(searchBoxV2.filterChips.first()).toContainText('MODEL')
+      // Type to narrow and select MODEL
+      await searchBoxV2.input.fill('MODEL')
+      await searchBoxV2.filterOptions
+        .filter({ hasText: 'MODEL' })
+        .first()
+        .click()
+
+      // Filter chip should appear and results should be filtered
+      await expect(
+        searchBoxV2.dialog.getByText('Input:', { exact: false }).locator('..')
+      ).toContainText('MODEL')
       await expect(searchBoxV2.results.first()).toBeVisible()
     })
   })
@@ -96,180 +120,32 @@ test.describe('Node search box V2', { tag: '@node' }, () => {
       const { searchBoxV2 } = comfyPage
       const initialCount = await comfyPage.nodeOps.getGraphNodesCount()
 
-      await searchBoxV2.open()
+      await comfyPage.canvasOps.doubleClick()
+      await expect(searchBoxV2.input).toBeVisible()
+
       await searchBoxV2.input.fill('KSampler')
       const results = searchBoxV2.results
       await expect(results.first()).toBeVisible()
 
-      await test.step('First result is selected by default', async () => {
-        await expect(results.first()).toHaveAttribute('aria-selected', 'true')
-      })
+      // First result selected by default
+      await expect(results.first()).toHaveAttribute('aria-selected', 'true')
 
-      await test.step('ArrowDown moves selection to next result', async () => {
-        await comfyPage.page.keyboard.press('ArrowDown')
-        await expect(results.nth(1)).toHaveAttribute('aria-selected', 'true')
-        await expect(results.first()).toHaveAttribute('aria-selected', 'false')
-      })
+      // ArrowDown moves selection
+      await comfyPage.page.keyboard.press('ArrowDown')
+      await expect(results.nth(1)).toHaveAttribute('aria-selected', 'true')
+      await expect(results.first()).toHaveAttribute('aria-selected', 'false')
 
-      await test.step('ArrowUp moves selection back', async () => {
-        await comfyPage.page.keyboard.press('ArrowUp')
-        await expect(results.first()).toHaveAttribute('aria-selected', 'true')
-      })
+      // ArrowUp moves back
+      await comfyPage.page.keyboard.press('ArrowUp')
+      await expect(results.first()).toHaveAttribute('aria-selected', 'true')
 
-      await test.step('Enter selects and adds the node', async () => {
-        await comfyPage.page.keyboard.press('Enter')
-        await expect(searchBoxV2.input).toBeHidden()
-        await expect
-          .poll(() => comfyPage.nodeOps.getGraphNodesCount())
-          .toBe(initialCount + 1)
-      })
-    })
-  })
+      // Enter selects and adds node
+      await comfyPage.page.keyboard.press('Enter')
+      await expect(searchBoxV2.input).toBeHidden()
 
-  test.describe('Category sidebar', () => {
-    test('Sidebar toggle hides and shows the category sidebar', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      await searchBoxV2.open()
-
-      const samplingCategory = searchBoxV2.categoryButton('sampling')
-      await expect(samplingCategory).toBeVisible()
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'true'
-      )
-
-      await searchBoxV2.sidebarToggle.click()
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'false'
-      )
-      await expect(samplingCategory).toBeHidden()
-
-      await searchBoxV2.sidebarToggle.click()
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'true'
-      )
-      await expect(samplingCategory).toBeVisible()
-    })
-
-    test('Filter bar scrolls horizontally while the sidebar toggle stays pinned', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      // Narrow viewport so the chips overflow the filter bar
-      await comfyPage.page.setViewportSize({ width: 360, height: 800 })
-      await searchBoxV2.open()
-
-      const scrollEl = searchBoxV2.filterChipsScroll
-      const dims = await scrollEl.evaluate((el) => ({
-        scrollWidth: el.scrollWidth,
-        clientWidth: el.clientWidth
-      }))
-      expect(dims.scrollWidth).toBeGreaterThan(dims.clientWidth)
-
-      await scrollEl.evaluate((el) => {
-        el.scrollLeft = el.scrollWidth
-      })
-
-      // The toggle lives outside the scroll container, so even when the
-      // chips scroll hundreds of px it must remain visible in the viewport.
-      await expect(searchBoxV2.sidebarToggle).toBeInViewport()
-    })
-
-    test('@mobile Sidebar is collapsed by default on mobile', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      await searchBoxV2.open()
-
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'false'
-      )
-      await expect(searchBoxV2.categoryButton('sampling')).toBeHidden()
-    })
-
-    test('@mobile Clicking outside the sidebar closes it', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      await searchBoxV2.open()
-
-      await searchBoxV2.sidebarToggle.click()
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'true'
-      )
-      await expect(searchBoxV2.categoryButton('sampling')).toBeVisible()
-      await expect(searchBoxV2.sidebarBackdrop).toBeVisible()
-
-      // The backdrop spans the full content area, but the sidebar (z-20)
-      // covers its left ~208px (w-52). Click past that to land on the
-      // backdrop rather than the sidebar.
-      await searchBoxV2.sidebarBackdrop.click({ position: { x: 240, y: 40 } })
-
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'false'
-      )
-      await expect(searchBoxV2.categoryButton('sampling')).toBeHidden()
-      await expect(searchBoxV2.sidebarBackdrop).toBeHidden()
-    })
-
-    test('@mobile Focusing the search input closes the sidebar', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      await searchBoxV2.open()
-
-      await searchBoxV2.sidebarToggle.click()
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'true'
-      )
-
-      await searchBoxV2.input.focus()
-
-      await expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-        'aria-expanded',
-        'false'
-      )
-    })
-
-    test('Sidebar state across mobile/desktop resizes', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-      const switchToDesktop = () =>
-        comfyPage.page.setViewportSize({ width: 1280, height: 800 })
-      const switchToMobile = () =>
-        comfyPage.page.setViewportSize({ width: 360, height: 800 })
-      const expectExpanded = (value: 'true' | 'false') =>
-        expect(searchBoxV2.sidebarToggle).toHaveAttribute(
-          'aria-expanded',
-          value
-        )
-
-      await switchToDesktop()
-      await searchBoxV2.open()
-      await expectExpanded('true')
-
-      await switchToMobile()
-      await expectExpanded('false')
-
-      await searchBoxV2.sidebarToggle.click()
-      await switchToDesktop()
-      await expectExpanded('true')
-
-      await searchBoxV2.sidebarToggle.click()
-      await switchToMobile()
-      await expectExpanded('false')
-
-      await switchToDesktop()
-      await expectExpanded('false')
+      await expect
+        .poll(() => comfyPage.nodeOps.getGraphNodesCount())
+        .toBe(initialCount + 1)
     })
   })
 })

browser_tests/tests/nodeSearchBoxV2Extended.spec.ts

@@ -2,17 +2,27 @@ import {
   comfyExpect as expect,
   comfyPageFixture as test
 } from '@e2e/fixtures/ComfyPage'
-import { RootCategory } from '@/components/searchbox/v2/rootCategories'
 
 test.describe('Node search box V2 extended', { tag: '@node' }, () => {
   test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.searchBoxV2.setup()
+    await comfyPage.settings.setSetting('Comfy.UseNewMenu', 'Disabled')
+    await comfyPage.settings.setSetting('Comfy.NodeSearchBoxImpl', 'default')
+    await comfyPage.settings.setSetting(
+      'Comfy.LinkRelease.Action',
+      'search box'
+    )
+    await comfyPage.settings.setSetting(
+      'Comfy.LinkRelease.ActionShift',
+      'search box'
+    )
+    await comfyPage.searchBoxV2.reload(comfyPage)
   })
 
   test('Double-click on empty canvas opens search', async ({ comfyPage }) => {
     const { searchBoxV2 } = comfyPage
 
-    await searchBoxV2.openByDoubleClickCanvas()
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
     await expect(searchBoxV2.dialog).toBeVisible()
   })
 
@@ -22,40 +32,43 @@ test.describe('Node search box V2 extended', { tag: '@node' }, () => {
     const { searchBoxV2 } = comfyPage
     const initialCount = await comfyPage.nodeOps.getGraphNodesCount()
 
-    await searchBoxV2.open()
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
+
     await searchBoxV2.input.fill('KSampler')
     await expect(searchBoxV2.results.first()).toBeVisible()
 
     await comfyPage.page.keyboard.press('Escape')
     await expect(searchBoxV2.input).toBeHidden()
+
     await expect
       .poll(() => comfyPage.nodeOps.getGraphNodesCount())
       .toBe(initialCount)
   })
 
-  for (const closeKey of ['Enter', 'Escape'] as const) {
-    test(`Reopening search after ${closeKey} has no persisted state`, async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
+  test('Search clears when reopening', async ({ comfyPage }) => {
+    const { searchBoxV2 } = comfyPage
 
-      await searchBoxV2.open()
-      await searchBoxV2.input.fill('KSampler')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      await comfyPage.page.keyboard.press(closeKey)
-      await expect(searchBoxV2.input).toBeHidden()
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
 
-      await searchBoxV2.open()
-      await expect(searchBoxV2.input).toHaveValue('')
-      await expect(searchBoxV2.filterChips).toHaveCount(0)
-    })
-  }
+    await searchBoxV2.input.fill('KSampler')
+    await expect(searchBoxV2.results.first()).toBeVisible()
+
+    await comfyPage.page.keyboard.press('Escape')
+    await expect(searchBoxV2.input).toBeHidden()
+
+    await comfyPage.canvasOps.doubleClick()
+    await expect(searchBoxV2.input).toBeVisible()
+    await expect(searchBoxV2.input).toHaveValue('')
+  })
 
   test.describe('Category navigation', () => {
     test('Category navigation updates results', async ({ comfyPage }) => {
       const { searchBoxV2 } = comfyPage
 
-      await searchBoxV2.open()
+      await comfyPage.canvasOps.doubleClick()
+      await expect(searchBoxV2.input).toBeVisible()
 
       await searchBoxV2.categoryButton('sampling').click()
       await expect(searchBoxV2.results.first()).toBeVisible()
@@ -63,6 +76,7 @@ test.describe('Node search box V2 extended', { tag: '@node' }, () => {
 
       await searchBoxV2.categoryButton('loaders').click()
       await expect(searchBoxV2.results.first()).toBeVisible()
+
       await expect
         .poll(() => searchBoxV2.results.allTextContents())
         .not.toEqual(samplingResults)
@@ -73,328 +87,58 @@ test.describe('Node search box V2 extended', { tag: '@node' }, () => {
     test('Filter chip removal restores results', async ({ comfyPage }) => {
       const { searchBoxV2 } = comfyPage
 
-      await searchBoxV2.open()
-
-      // Search first to keep the result set under the 64-item cap.
-      await searchBoxV2.input.fill('Load')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const unfilteredCount = await searchBoxV2.results.count()
-
-      await test.step('Apply Input/MODEL filter', async () => {
-        await searchBoxV2.applyTypeFilter('input', 'MODEL')
-        await expect(searchBoxV2.filterChips).toHaveCount(1)
-        await expect
-          .poll(() => searchBoxV2.results.count())
-          .not.toBe(unfilteredCount)
-      })
-
-      await test.step('Remove the filter chip', async () => {
-        await searchBoxV2.removeFilterChip()
-        await expect(searchBoxV2.filterChips).toHaveCount(0)
-        await expect(searchBoxV2.results).toHaveCount(unfilteredCount)
-      })
-    })
-  })
-
-  test.describe('Link release', () => {
-    test('Link release opens search with pre-applied type filter', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await comfyPage.canvasOps.disconnectEdge()
+      await comfyPage.canvasOps.doubleClick()
       await expect(searchBoxV2.input).toBeVisible()
 
-      // disconnectEdge pulls a CLIP link → expect a single CLIP filter chip.
-      await expect(searchBoxV2.filterChips).toHaveCount(1)
-      await expect(searchBoxV2.filterChips.first()).toContainText('CLIP')
+      // Record initial result text for comparison
+      await expect(searchBoxV2.results.first()).toBeVisible()
+      const unfilteredResults = await searchBoxV2.results.allTextContents()
+
+      // Apply Input filter with MODEL type
+      await searchBoxV2.filterBarButton('Input').click()
+      await expect(searchBoxV2.filterOptions.first()).toBeVisible()
+      await searchBoxV2.input.fill('MODEL')
+      await searchBoxV2.filterOptions
+        .filter({ hasText: 'MODEL' })
+        .first()
+        .click()
+
+      // Verify filter chip appeared and results changed
+      const filterChip = searchBoxV2.dialog.getByTestId('filter-chip')
+      await expect(filterChip).toBeVisible()
+      await expect(searchBoxV2.results.first()).toBeVisible()
+      await expect
+        .poll(() => searchBoxV2.results.allTextContents())
+        .not.toEqual(unfilteredResults)
+
+      // Remove filter by clicking the chip delete button
+      await filterChip.getByTestId('chip-delete').click()
+
+      // Filter chip should be removed
+      await expect(filterChip).toBeHidden()
+      await expect(searchBoxV2.results.first()).toBeVisible()
     })
+  })
 
-    test('Link release auto-connects added node', async ({ comfyPage }) => {
+  test.describe('Keyboard navigation', () => {
+    test('ArrowUp on first item keeps first selected', async ({
+      comfyPage
+    }) => {
       const { searchBoxV2 } = comfyPage
-      const NODE_TYPE = 'CLIPTextEncode'
-      const refsBefore = await comfyPage.nodeOps.getNodeRefsByType(NODE_TYPE)
-      const idsBefore = new Set(refsBefore.map((n) => n.id))
 
-      await comfyPage.canvasOps.disconnectEdge()
+      await comfyPage.canvasOps.doubleClick()
       await expect(searchBoxV2.input).toBeVisible()
 
-      await searchBoxV2.input.fill('CLIP Text Encode')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      await comfyPage.page.keyboard.press('Enter')
-      await expect(searchBoxV2.input).toBeHidden()
-
-      // A new CLIPTextEncode node should have been added.
-      await expect
-        .poll(() =>
-          comfyPage.nodeOps
-            .getNodeRefsByType(NODE_TYPE)
-            .then((refs) => refs.length)
-        )
-        .toBe(refsBefore.length + 1)
-
-      // Verify the auto-connect: the newly-added node's CLIP input must be
-      // connected (proves the release wasn't just dropped).
-      const refsAfter = await comfyPage.nodeOps.getNodeRefsByType(NODE_TYPE)
-      const newNode = refsAfter.find((n) => !idsBefore.has(n.id))
-      expect(newNode, 'expected a new CLIPTextEncode node').toBeDefined()
-      const clipInput = await newNode!.getInput(0)
-      await expect.poll(() => clipInput.getLinkCount()).toBe(1)
-    })
-  })
-
-  test.describe('Filter combinations', () => {
-    test('Output type filter filters results', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-
-      await searchBoxV2.input.fill('Load')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const unfilteredCount = await searchBoxV2.results.count()
-
-      await searchBoxV2.applyTypeFilter('output', 'IMAGE')
-      await expect(searchBoxV2.filterChips).toHaveCount(1)
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .not.toBe(unfilteredCount)
-    })
-
-    test('Multiple type filters (Input + Output) narrows results', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-
-      await searchBoxV2.applyTypeFilter('input', 'MODEL')
-      await expect(searchBoxV2.filterChips).toHaveCount(1)
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const singleFilterCount = await searchBoxV2.results.count()
-
-      await searchBoxV2.applyTypeFilter('output', 'LATENT')
-      await expect(searchBoxV2.filterChips).toHaveCount(2)
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .toBeLessThan(singleFilterCount)
-    })
-
-    test('Root filter + search query narrows results', async ({
-      comfyPage
-    }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-      await searchBoxV2.input.fill('Sampler')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const unfilteredCount = await searchBoxV2.results.count()
-
-      await searchBoxV2.rootCategoryButton('comfy').click()
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .toBeLessThan(unfilteredCount)
-      await expect.poll(() => searchBoxV2.results.count()).toBeGreaterThan(0)
-    })
-
-    test('Root filter + category selection', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-
-      await searchBoxV2.rootCategoryButton('comfy').click()
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const comfyCount = await searchBoxV2.results.count()
-
-      // Under root filter, categories are prefixed (e.g. comfy/sampling).
-      await searchBoxV2.categoryButton('comfy/sampling').click()
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .toBeLessThan(comfyCount)
-    })
-  })
-
-  test.describe('Category sidebar', () => {
-    test('Category tree expand and collapse', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-
-      const samplingBtn = searchBoxV2.categoryButton('sampling')
-      const subcategory = searchBoxV2.categoryButton('sampling/custom_sampling')
-
-      await test.step('Expanding sampling reveals its subcategories', async () => {
-        await samplingBtn.click()
-        await expect(subcategory).toBeVisible()
-      })
-
-      await test.step('Collapsing sampling hides its subcategories', async () => {
-        await samplingBtn.click()
-        await expect(subcategory).toBeHidden()
-      })
-    })
-
-    test('Subcategory narrows results to subset', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-
-      await searchBoxV2.categoryButton('sampling').click()
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const parentCount = await searchBoxV2.results.count()
-
-      const subcategory = searchBoxV2.categoryButton('sampling/custom_sampling')
-      await expect(subcategory).toBeVisible()
-      await subcategory.click()
-
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .toBeLessThan(parentCount)
-    })
-
-    test('Most relevant resets category filter', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const defaultCount = await searchBoxV2.results.count()
-
-      await searchBoxV2.categoryButton('sampling').click()
-      await expect
-        .poll(() => searchBoxV2.results.count())
-        .not.toBe(defaultCount)
-
-      await searchBoxV2.categoryButton('most-relevant').click()
-      await expect(searchBoxV2.results).toHaveCount(defaultCount)
-    })
-
-    test(
-      'Blueprint root chip filters to published blueprints',
-      { tag: ['@subgraph'] },
-      async ({ comfyPage }) => {
-        const blueprintName = `chip-test-${crypto.randomUUID().slice(0, 8)}`
-        const nodeRef = await comfyPage.nodeOps.getNodeRefById('3')
-        await nodeRef.click('title')
-        await comfyPage.command.executeCommand('Comfy.Graph.ConvertToSubgraph')
-        await expect
-          .poll(() =>
-            comfyPage.nodeOps
-              .getNodeRefsByTitle('New Subgraph')
-              .then((refs) => refs.length)
-          )
-          .toBe(1)
-        const subgraphNodes =
-          await comfyPage.nodeOps.getNodeRefsByTitle('New Subgraph')
-        await subgraphNodes[0].click('title')
-        await comfyPage.command.executeCommand('Comfy.PublishSubgraph', {
-          name: blueprintName
-        })
-        await expect(comfyPage.visibleToasts).toHaveCount(1, { timeout: 5000 })
-        await comfyPage.toast.closeToasts(1)
-
-        const { searchBoxV2 } = comfyPage
-        await searchBoxV2.open()
-
-        const blueprintsChip = searchBoxV2.rootCategoryButton(
-          RootCategory.Blueprint
-        )
-        await expect(blueprintsChip).toBeVisible()
-        await blueprintsChip.click()
-
-        // Blueprints persist across tests on the same worker; filter by the
-        // unique name we just published rather than asserting the full list.
-        await expect(
-          searchBoxV2.results.filter({ hasText: blueprintName })
-        ).toHaveCount(1)
-      }
-    )
-  })
-
-  test.describe('Search behavior', () => {
-    test('Search narrows results progressively', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-      const getCount = () => searchBoxV2.results.count()
-
-      await searchBoxV2.open()
-
-      await searchBoxV2.input.fill('S')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-      const count1 = await getCount()
-
-      await searchBoxV2.input.fill('Sa')
-      await expect.poll(getCount).toBeLessThan(count1)
-      const count2 = await getCount()
-
-      await searchBoxV2.input.fill('Sampler')
-      await expect.poll(getCount).toBeLessThan(count2)
-    })
-
-    test('No results shown for nonsensical query', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-      await searchBoxV2.input.fill('zzzxxxyyy_nonexistent_node')
-
-      await expect(searchBoxV2.noResults).toBeVisible()
-      await expect(searchBoxV2.results).toHaveCount(0)
-    })
-  })
-
-  test.describe('Filter chip interaction', () => {
-    test('Multiple filter chips displayed', async ({ comfyPage }) => {
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-      await searchBoxV2.applyTypeFilter('input', 'MODEL')
-      await searchBoxV2.applyTypeFilter('output', 'LATENT')
-
-      await expect(searchBoxV2.filterChips).toHaveCount(2)
-      const chipTexts = await searchBoxV2.filterChips.allTextContents()
-      expect(chipTexts.some((t) => t.includes('MODEL'))).toBe(true)
-      expect(chipTexts.some((t) => t.includes('LATENT'))).toBe(true)
-    })
-  })
-
-  test.describe('Settings-driven behavior', () => {
-    test('Node ID name shown when setting enabled', async ({ comfyPage }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.NodeSearchBoxImpl.ShowIdName',
-        true
-      )
-      const { searchBoxV2 } = comfyPage
-
-      await searchBoxV2.open()
-      await searchBoxV2.input.fill('VAE Decode')
-      await expect(searchBoxV2.results.first()).toBeVisible()
-
-      await expect(searchBoxV2.nodeIdBadge.first()).toBeVisible()
-      await expect(searchBoxV2.nodeIdBadge.first()).toContainText('VAEDecode')
-    })
-
-    test('Follow-cursor disabled places node without ghost mode', async ({
-      comfyPage
-    }) => {
-      await comfyPage.settings.setSetting(
-        'Comfy.NodeSearchBoxImpl.FollowCursor',
-        false
-      )
-      const { searchBoxV2 } = comfyPage
-      const initialCount = await comfyPage.nodeOps.getGraphNodesCount()
-
-      await searchBoxV2.open()
-
       await searchBoxV2.input.fill('KSampler')
-      await expect(searchBoxV2.results.first()).toBeVisible()
+      const results = searchBoxV2.results
+      await expect(results.first()).toBeVisible()
 
-      await searchBoxV2.results.first().click()
-      await expect(searchBoxV2.input).toBeHidden()
+      // First result should be selected by default
+      await expect(results.first()).toHaveAttribute('aria-selected', 'true')
 
-      await expect
-        .poll(() => comfyPage.nodeOps.getGraphNodesCount())
-        .toBe(initialCount + 1)
-
-      await expect(
-        comfyPage.page.locator('[data-node-id][data-ghost]')
-      ).toHaveCount(0)
+      // ArrowUp on first item should keep first selected
+      await comfyPage.page.keyboard.press('ArrowUp')
+      await expect(results.first()).toHaveAttribute('aria-selected', 'true')
     })
   })
 })

browser_tests/tests/painter.spec.ts

@@ -7,7 +7,7 @@ import {
   drawStroke,
   hasCanvasContent,
   triggerSerialization
-} from '@e2e/fixtures/utils/painter'
+} from '@e2e/helpers/painter'
 import type { TestGraphAccess } from '@e2e/types/globals'
 
 test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
@@ -163,7 +163,7 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
         .poll(() => cursor.evaluate((el: HTMLElement) => el.style.transform))
         .not.toBe(transform1)
 
-      await comfyPage.page.mouse.move(box.x + box.width + 50, box.y)
+      await comfyPage.page.mouse.move(0, 0)
       await expect(cursor).toBeHidden()
     })
 
@@ -187,10 +187,7 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
         box.y + box.height * 0.5,
         { steps: 10 }
       )
-      await comfyPage.page.mouse.move(
-        box.x + box.width + 20,
-        box.y + box.height * 0.5
-      )
+      await comfyPage.page.mouse.move(box.x - 20, box.y + box.height * 0.5)
       await comfyPage.page.mouse.up()
 
       await comfyPage.nextFrame()
@@ -411,7 +408,6 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
 
       await expect
         .poll(() => canvas.evaluate((el: HTMLCanvasElement) => el.width))
-        // default 512 + slider step 64 = 576
         .toBe(576)
     })
 
@@ -497,29 +493,6 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
           .toBe(false)
       }
     )
-
-    test('Clear on empty canvas is harmless', async ({ comfyPage }) => {
-      const painterWidget = comfyPage.vueNodes
-        .getNodeLocator('1')
-        .locator('.widget-expands')
-      const canvas = painterWidget.locator('canvas')
-
-      await expect
-        .poll(() => hasCanvasContent(canvas), {
-          message: 'canvas should start empty'
-        })
-        .toBe(false)
-
-      await painterWidget
-        .getByTestId('painter-clear-button')
-        .dispatchEvent('click')
-
-      await expect
-        .poll(() => hasCanvasContent(canvas), {
-          message: 'canvas should still be empty after clearing empty canvas'
-        })
-        .toBe(false)
-    })
   })
 
   test.describe('Serialization', () => {
@@ -587,6 +560,36 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
   })
 
   test.describe('Eraser', () => {
+    test('Eraser removes previously drawn content', async ({ comfyPage }) => {
+      const node = comfyPage.vueNodes.getNodeLocator('1')
+      const painterWidget = node.locator('.widget-expands')
+      const canvas = painterWidget.locator('canvas')
+      await expect(canvas).toBeVisible()
+
+      await drawStroke(comfyPage.page, canvas)
+      await comfyPage.nextFrame()
+      await expect.poll(() => hasCanvasContent(canvas)).toBe(true)
+
+      await painterWidget.getByRole('button', { name: 'Eraser' }).click()
+      await drawStroke(comfyPage.page, canvas)
+      await comfyPage.nextFrame()
+
+      await expect
+        .poll(
+          () =>
+            canvas.evaluate((el: HTMLCanvasElement) => {
+              const ctx = el.getContext('2d')
+              if (!ctx) return false
+              const cx = Math.floor(el.width / 2)
+              const cy = Math.floor(el.height / 2)
+              const { data } = ctx.getImageData(cx - 5, cy - 5, 10, 10)
+              return data.every((v, i) => i % 4 !== 3 || v === 0)
+            }),
+          { message: 'erased area should be transparent' }
+        )
+        .toBe(true)
+    })
+
     test('Eraser on empty canvas adds no content', async ({ comfyPage }) => {
       const node = comfyPage.vueNodes.getNodeLocator('1')
       const painterWidget = node.locator('.widget-expands')
@@ -601,318 +604,18 @@ test.describe('Painter', { tag: ['@widget', '@vue-nodes'] }, () => {
     })
   })
 
-  test.describe('Serialization — unchanged canvas', () => {
-    test(
-      'Unchanged canvas does not re-upload on second serialization',
-      { tag: '@slow' },
-      async ({ comfyPage }) => {
-        let uploadCount = 0
-
-        await comfyPage.page.route('**/upload/image', async (route) => {
-          uploadCount++
-          const mockResponse: UploadImageResponse = { name: 'painter-test.png' }
-          await route.fulfill({
-            status: 200,
-            contentType: 'application/json',
-            body: JSON.stringify(mockResponse)
-          })
-        })
-
-        const canvas = comfyPage.vueNodes
-          .getNodeLocator('1')
-          .locator('.widget-expands canvas')
-
-        await drawStroke(comfyPage.page, canvas)
-        await triggerSerialization(comfyPage.page)
-        expect(uploadCount, 'first serialization should upload once').toBe(1)
-
-        await triggerSerialization(comfyPage.page)
-        expect(
-          uploadCount,
-          'second serialization without new drawing should not re-upload'
-        ).toBe(1)
-      }
-    )
-  })
-
-  test.describe('Settings persistence', () => {
-    test('Tool selection is saved to node properties', async ({
-      comfyPage
-    }) => {
-      const painterWidget = comfyPage.vueNodes
-        .getNodeLocator('1')
-        .locator('.widget-expands')
-
-      await painterWidget.getByRole('button', { name: 'Eraser' }).click()
-
-      await expect
-        .poll(
-          () =>
-            comfyPage.page.evaluate(() => {
-              const graph = window.graph as TestGraphAccess | undefined
-              return graph?._nodes_by_id?.['1']?.properties?.painterTool as
-                | string
-                | undefined
-            }),
-          { message: 'painterTool property should update to eraser' }
-        )
-        .toBe('eraser')
-    })
-
-    test('Brush size change is saved to node properties', async ({
-      comfyPage
-    }) => {
-      const sizeRow = comfyPage.vueNodes
-        .getNodeLocator('1')
-        .locator('.widget-expands')
-        .getByTestId('painter-size-row')
-      const sizeSlider = sizeRow.getByRole('slider')
-
-      await expect(
-        sizeRow.getByTestId('painter-size-value'),
-        'brush size should start at default 20'
-      ).toHaveText('20')
-
-      await sizeSlider.focus()
-      for (let i = 0; i < 10; i++) {
-        await sizeSlider.press('ArrowRight')
-      }
-
-      await expect
-        .poll(
-          () =>
-            comfyPage.page.evaluate(() => {
-              const graph = window.graph as TestGraphAccess | undefined
-              return graph?._nodes_by_id?.['1']?.properties
-                ?.painterBrushSize as number | undefined
-            }),
-          { message: 'painterBrushSize property should update to 30' }
-        )
-        .toBe(30)
-    })
-  })
-
-  test('Controls collapse to single column in compact mode', async ({
-    comfyPage
-  }) => {
-    const painterWidget = comfyPage.vueNodes
-      .getNodeLocator('1')
-      .locator('.widget-expands')
-    const toolLabel = painterWidget.getByText('Tool', { exact: true })
-
-    await expect(
-      toolLabel,
-      'tool label should be visible in two-column layout'
-    ).toBeVisible()
-
-    await comfyPage.page.evaluate(() => {
-      const graph = window.graph as TestGraphAccess | undefined
-      const node = graph?._nodes_by_id?.['1']
-      if (node) {
-        node.size = [200, 400]
-        window.app!.canvas.setDirty(true, true)
-      }
-    })
-
-    await expect(
-      toolLabel,
-      'tool label should hide in compact single-column layout'
-    ).toBeHidden()
-  })
-
-  test('Multiple sequential strokes at different positions all accumulate', async ({
-    comfyPage
-  }) => {
+  test('Multiple strokes accumulate on the canvas', async ({ comfyPage }) => {
     const canvas = comfyPage.vueNodes
       .getNodeLocator('1')
       .locator('.widget-expands canvas')
     await expect(canvas).toBeVisible()
 
-    await drawStroke(comfyPage.page, canvas, { yPct: 0.25 })
-    await drawStroke(comfyPage.page, canvas, { yPct: 0.5 })
-    await drawStroke(comfyPage.page, canvas, { yPct: 0.75 })
+    await drawStroke(comfyPage.page, canvas, { yPct: 0.3 })
     await comfyPage.nextFrame()
+    await expect.poll(() => hasCanvasContent(canvas)).toBe(true)
 
-    const hasContentAtRow = (yFraction: number) =>
-      canvas.evaluate((el: HTMLCanvasElement, y: number) => {
-        const ctx = el.getContext('2d')
-        if (!ctx) return false
-        const cy = Math.floor(el.height * y)
-        const { data } = ctx.getImageData(0, cy - 5, el.width, 10)
-        for (let i = 3; i < data.length; i += 4) {
-          if (data[i] > 0) return true
-        }
-        return false
-      }, yFraction)
-
-    await expect
-      .poll(() => hasContentAtRow(0.25), {
-        message: 'top stroke should be present'
-      })
-      .toBe(true)
-    await expect
-      .poll(() => hasContentAtRow(0.5), {
-        message: 'middle stroke should be present'
-      })
-      .toBe(true)
-    await expect
-      .poll(() => hasContentAtRow(0.75), {
-        message: 'bottom stroke should be present'
-      })
-      .toBe(true)
+    await drawStroke(comfyPage.page, canvas, { yPct: 0.7 })
+    await comfyPage.nextFrame()
+    await expect.poll(() => hasCanvasContent(canvas)).toBe(true)
   })
 })
-
-test.describe(
-  'Painter — input image connection',
-  { tag: ['@widget', '@vue-nodes', '@slow'] },
-  () => {
-    test.setTimeout(60_000)
-
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.page.evaluate(() => window.app?.graph?.clear())
-      await comfyPage.workflow.loadWorkflow('widgets/painter_with_input')
-    })
-
-    test('Width, height, and bg_color controls hide when input is connected', async ({
-      comfyPage
-    }) => {
-      const painterWidget = comfyPage.vueNodes
-        .getNodeLocator('1')
-        .locator('.widget-expands')
-
-      await expect(
-        painterWidget.getByTestId('painter-width-row'),
-        'width row should be hidden when input is connected'
-      ).toBeHidden()
-      await expect(
-        painterWidget.getByTestId('painter-height-row'),
-        'height row should be hidden when input is connected'
-      ).toBeHidden()
-      await expect(
-        painterWidget.getByTestId('painter-bg-color-row'),
-        'background color row should be hidden when input is connected'
-      ).toBeHidden()
-      await expect(
-        painterWidget.getByTestId('painter-dimension-text'),
-        'dimension text should be visible when input is connected'
-      ).toBeVisible()
-    })
-
-    test('Canvas resizes to match input image dimensions after execution', async ({
-      comfyPage
-    }) => {
-      await comfyPage.runButton.click()
-
-      const node = comfyPage.vueNodes.getNodeLocator('1')
-      const img = node.locator('.widget-expands img')
-      await expect(
-        img,
-        'input image should appear after execution'
-      ).toBeVisible({
-        timeout: 30_000
-      })
-
-      await expect
-        .poll(
-          () =>
-            img.evaluate(
-              (el: HTMLImageElement) => el.complete && el.naturalWidth > 0
-            ),
-          {
-            message: 'input image should be fully decoded',
-            timeout: 30_000
-          }
-        )
-        .toBe(true)
-
-      const { nw, nh } = await img.evaluate((el: HTMLImageElement) => ({
-        nw: el.naturalWidth,
-        nh: el.naturalHeight
-      }))
-
-      const canvas = node.locator('.widget-expands canvas')
-      await expect
-        .poll(() => canvas.evaluate((el: HTMLCanvasElement) => el.width), {
-          message: 'canvas width should match input image natural width'
-        })
-        .toBe(nw)
-      await expect
-        .poll(() => canvas.evaluate((el: HTMLCanvasElement) => el.height), {
-          message: 'canvas height should match input image natural height'
-        })
-        .toBe(nh)
-    })
-
-    test('Drawing over input image produces content on canvas', async ({
-      comfyPage
-    }) => {
-      await comfyPage.runButton.click()
-
-      const node = comfyPage.vueNodes.getNodeLocator('1')
-      const img = node.locator('.widget-expands img')
-      await expect(
-        img,
-        'input image should appear after execution'
-      ).toBeVisible({
-        timeout: 30_000
-      })
-      await expect
-        .poll(
-          () =>
-            img.evaluate(
-              (el: HTMLImageElement) => el.complete && el.naturalWidth > 0
-            ),
-          { message: 'input image should be fully decoded', timeout: 30_000 }
-        )
-        .toBe(true)
-
-      const nw = await img.evaluate((el: HTMLImageElement) => el.naturalWidth)
-      const canvas = node.locator('.widget-expands canvas')
-      await expect
-        .poll(() => canvas.evaluate((el: HTMLCanvasElement) => el.width), {
-          message: 'canvas should resize to match input image width',
-          timeout: 15_000
-        })
-        .toBe(nw)
-
-      // Use dispatchEvent to bypass the LiteGraph canvas z-index overlay that
-      // intercepts coordinate-based hit testing from page.mouse
-      const box = await canvas.boundingBox()
-      if (!box) throw new Error('Canvas bounding box not found')
-      const startX = box.x + box.width * 0.3
-      const endX = box.x + box.width * 0.7
-      const midY = box.y + box.height * 0.5
-      const pointerOpts = {
-        bubbles: true,
-        cancelable: true,
-        pointerId: 1,
-        button: 0,
-        isPrimary: true
-      }
-      await canvas.dispatchEvent('pointerdown', {
-        ...pointerOpts,
-        clientX: startX,
-        clientY: midY
-      })
-      for (let i = 1; i <= 10; i++) {
-        await canvas.dispatchEvent('pointermove', {
-          ...pointerOpts,
-          clientX: startX + (endX - startX) * (i / 10),
-          clientY: midY
-        })
-      }
-      await canvas.dispatchEvent('pointerup', {
-        ...pointerOpts,
-        clientX: endX,
-        clientY: midY
-      })
-
-      await expect
-        .poll(() => hasCanvasContent(canvas), {
-          message: 'drawing over input image should produce canvas content'
-        })
-        .toBe(true)
-    })
-  }
-)

browser_tests/tests/performance.spec.ts

@@ -1,10 +1,7 @@
 import { expect } from '@playwright/test'
 
 import { comfyPageFixture as test } from '@e2e/fixtures/ComfyPage'
-import {
-  logMeasurement,
-  recordMeasurement
-} from '@e2e/fixtures/utils/perfReporter'
+import { logMeasurement, recordMeasurement } from '@e2e/helpers/perfReporter'
 
 test.describe('Performance', { tag: ['@perf'] }, () => {
   test('canvas idle style recalculations', async ({ comfyPage }) => {

browser_tests/tests/propertiesPanel/PropertiesPanelHelper.ts

@@ -1,7 +1,6 @@
 import type { Locator, Page } from '@playwright/test'
 import { expect } from '@playwright/test'
 
-import { TitleEditor } from '@e2e/fixtures/components/TitleEditor'
 import { TestIds } from '@e2e/fixtures/selectors'
 
 export class PropertiesPanelHelper {
@@ -9,14 +8,12 @@ export class PropertiesPanelHelper {
   readonly panelTitle: Locator
   readonly searchBox: Locator
   readonly closeButton: Locator
-  readonly titleEditor: TitleEditor
 
   constructor(readonly page: Page) {
     this.root = page.getByTestId(TestIds.propertiesPanel.root)
     this.panelTitle = this.root.locator('h3')
     this.searchBox = this.root.getByPlaceholder(/^Search/)
     this.closeButton = this.root.locator('button[aria-pressed]')
-    this.titleEditor = new TitleEditor(this.root)
   }
 
   get tabs(): Locator {
@@ -31,6 +28,10 @@ export class PropertiesPanelHelper {
     return this.panelTitle.locator('i[class*="lucide--pencil"]')
   }
 
+  get titleInput(): Locator {
+    return this.root.getByTestId(TestIds.node.titleInput)
+  }
+
   getNodeStateButton(state: 'Normal' | 'Bypass' | 'Mute'): Locator {
     return this.root.locator('button', { hasText: state })
   }
@@ -85,8 +86,8 @@ export class PropertiesPanelHelper {
 
   async editTitle(newTitle: string): Promise<void> {
     await this.titleEditIcon.click()
-    await this.titleEditor.expectVisible()
-    await this.titleEditor.setTitle(newTitle)
+    await this.titleInput.fill(newTitle)
+    await this.titleInput.press('Enter')
   }
 
   async searchWidgets(query: string): Promise<void> {

browser_tests/tests/propertiesPanel/errorsTabMissingModels.spec.ts

@@ -5,7 +5,7 @@ import { TestIds } from '@e2e/fixtures/selectors'
 import {
   interceptClipboardWrite,
   getClipboardText
-} from '@e2e/fixtures/utils/clipboardSpy'
+} from '@e2e/helpers/clipboardSpy'
 import {
   cleanupFakeModel,
   loadWorkflowAndOpenErrorsTab
@@ -99,58 +99,5 @@ test.describe('Errors tab - Missing models', { tag: '@ui' }, () => {
       )
       await expect(downloadButton.first()).toBeVisible()
     })
-
-    test('Should render Download all and Refresh actions for one downloadable model', async ({
-      comfyPage
-    }) => {
-      await loadWorkflowAndOpenErrorsTab(comfyPage, 'missing/missing_models')
-
-      await expect(
-        comfyPage.page.getByTestId(TestIds.dialogs.missingModelActions)
-      ).toBeVisible()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.dialogs.missingModelDownloadAll)
-      ).toBeVisible()
-      await expect(
-        comfyPage.page.getByTestId(TestIds.dialogs.missingModelRefresh)
-      ).toBeVisible()
-    })
-
-    test('Should clear resolved missing model when Refresh is clicked', async ({
-      comfyPage
-    }) => {
-      await loadWorkflowAndOpenErrorsTab(comfyPage, 'missing/missing_models')
-      await comfyPage.page.route(/\/object_info$/, 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 })
-      })
-
-      const objectInfoResponse = comfyPage.page.waitForResponse((response) => {
-        const url = new URL(response.url())
-        return url.pathname.endsWith('/object_info') && response.ok()
-      })
-      const modelFoldersResponse = comfyPage.page.waitForResponse(
-        (response) => {
-          const url = new URL(response.url())
-          return url.pathname.endsWith('/experiment/models') && response.ok()
-        }
-      )
-      const refreshButton = comfyPage.page.getByTestId(
-        TestIds.dialogs.missingModelRefresh
-      )
-
-      await Promise.all([
-        objectInfoResponse,
-        modelFoldersResponse,
-        refreshButton.click()
-      ])
-      await expect(
-        comfyPage.page.getByTestId(TestIds.dialogs.missingModelsGroup)
-      ).toBeHidden()
-    })
   })
 })