Fix: Move chromium deployment info generation to merge-reports job

The sharded chromium tests were generating deployment info only from shard 1, which could miss failures from other shards. This change moves the deployment info generation to the merge-reports job where we have visibility into the overall result of all shards.

Fixes #5338

.claude/commands/create-hotfix-release.md

@@ -1,85 +1,30 @@
 # Create Hotfix Release
 
-This command creates patch/hotfix releases for ComfyUI Frontend by backporting fixes to stable core branches. It handles both automated backports (preferred) and manual cherry-picking (fallback).
-
-**Process Overview:**
-1. **Check automated backports first** (via labels)
-2. **Skip to version bump** if backports already merged
-3. **Manual cherry-picking** if automation failed
-4. **Create patch release** with version bump  
-5. **Publish GitHub release** (manually uncheck "latest")
-6. **Update ComfyUI requirements.txt** via PR
+This command guides you through creating a patch/hotfix release for ComfyUI Frontend with comprehensive safety checks and human confirmations at each step.
 
 <task>
-Create a hotfix release by backporting commits/PRs from main to a core branch: $ARGUMENTS
+Create a hotfix release by cherry-picking commits or PR commits from main to a core branch: $ARGUMENTS
 
 Expected format: Comma-separated list of commits or PR numbers
 Examples: 
-- `#1234,#5678` (PRs - preferred)
-- `abc123,def456` (commit hashes)
-- `#1234,abc123` (mixed)
+- `abc123,def456,ghi789` (commits)
+- `#1234,#5678` (PRs)
+- `abc123,#1234,def456` (mixed)
 
-If no arguments provided, the command will guide you through identifying commits/PRs to backport.
+If no arguments provided, the command will help identify the correct core branch and guide you through selecting commits/PRs.
 </task>
 
 ## Prerequisites
 
-- Push access to repository
-- GitHub CLI (`gh`) authenticated  
-- Clean working tree
-- Understanding of what fixes need backporting
+Before starting, ensure:
+- You have push access to the repository
+- GitHub CLI (`gh`) is authenticated
+- You're on a clean working tree
+- You understand the commits/PRs you're cherry-picking
 
 ## Hotfix Release Process
 
-### Step 1: Try Automated Backports First
-
-**Check if automated backports were attempted:**
-
-1. **For each PR, check existing backport labels:**
-   ```bash
-   gh pr view #1234 --json labels | jq -r '.labels[].name'
-   ```
-
-2. **If no backport labels exist, add them now:**
-   ```bash
-   # Add backport labels (this triggers automated backports)
-   gh pr edit #1234 --add-label "needs-backport"
-   gh pr edit #1234 --add-label "1.24"  # Replace with target version
-   ```
-
-3. **Check for existing backport PRs:**
-   ```bash
-   # Check for backport PRs created by automation
-   PR_NUMBER=${ARGUMENTS%%,*}  # Extract first PR number from arguments
-   PR_NUMBER=${PR_NUMBER#\#}   # Remove # prefix
-   gh pr list --search "backport-${PR_NUMBER}-to" --json number,title,state,baseRefName
-   ```
-
-4. **Handle existing backport scenarios:**
-
-   **Scenario A: Automated backports already merged**
-   ```bash
-   # Check if backport PRs were merged to core branches
-   gh pr list --search "backport-${PR_NUMBER}-to" --state merged
-   ```
-   - If backport PRs are merged → Skip to Step 10 (Version Bump)
-   - **CONFIRMATION**: Automated backports completed, proceeding to version bump?
-
-   **Scenario B: Automated backport PRs exist but not merged**
-   ```bash
-   # Show open backport PRs that need merging
-   gh pr list --search "backport-${PR_NUMBER}-to" --state open
-   ```
-   - **ACTION REQUIRED**: Merge the existing backport PRs first
-   - Use: `gh pr merge [PR_NUMBER] --merge` for each backport PR
-   - After merging, return to this command and skip to Step 10 (Version Bump)
-   - **CONFIRMATION**: Have you merged all backport PRs? Ready to proceed to version bump?
-
-   **Scenario C: No automated backports or they failed**
-   - Continue to Step 2 for manual cherry-picking
-   - **CONFIRMATION**: Proceeding with manual cherry-picking because automation failed?
-
-### Step 2: Identify Target Core Branch
+### Step 1: Identify Target Core Branch
 
 1. Fetch the current ComfyUI requirements.txt from master branch:
    ```bash
@@ -91,7 +36,7 @@ If no arguments provided, the command will guide you through identifying commits
 5. Verify the core branch exists: `git ls-remote origin refs/heads/core/*`
 6. **CONFIRMATION REQUIRED**: Is `core/X.Y` the correct target branch?
 
-### Step 3: Parse and Validate Arguments
+### Step 2: Parse and Validate Arguments
 
 1. Parse the comma-separated list of commits/PRs
 2. For each item:
@@ -104,7 +49,7 @@ If no arguments provided, the command will guide you through identifying commits
    - **CONFIRMATION REQUIRED**: Use merge commit or cherry-pick individual commits?
 4. Validate all commit hashes exist in the repository
 
-### Step 4: Analyze Target Changes
+### Step 3: Analyze Target Changes
 
 1. For each commit/PR to cherry-pick:
    - Display commit hash, author, date
@@ -115,7 +60,7 @@ If no arguments provided, the command will guide you through identifying commits
 2. Identify potential conflicts by checking changed files
 3. **CONFIRMATION REQUIRED**: Proceed with these commits?
 
-### Step 5: Create Hotfix Branch
+### Step 4: Create Hotfix Branch
 
 1. Checkout the core branch (e.g., `core/1.23`)
 2. Pull latest changes: `git pull origin core/X.Y`
@@ -124,7 +69,7 @@ If no arguments provided, the command will guide you through identifying commits
    - Example: `hotfix/1.23.4-20241120`
 5. **CONFIRMATION REQUIRED**: Created branch correctly?
 
-### Step 6: Cherry-pick Changes
+### Step 5: Cherry-pick Changes
 
 For each commit:
 1. Attempt cherry-pick: `git cherry-pick <commit>`
@@ -138,7 +83,7 @@ For each commit:
    - Run validation: `pnpm typecheck && pnpm lint`
 4. **CONFIRMATION REQUIRED**: Cherry-pick successful and valid?
 
-### Step 7: Create PR to Core Branch
+### Step 6: Create PR to Core Branch
 
 1. Push the hotfix branch: `git push origin hotfix/<version>-<timestamp>`
 2. Create PR using gh CLI:
@@ -155,7 +100,7 @@ For each commit:
    - Impact assessment
 5. **CONFIRMATION REQUIRED**: PR created correctly?
 
-### Step 8: Wait for Tests
+### Step 7: Wait for Tests
 
 1. Monitor PR checks: `gh pr checks`
 2. Display test results as they complete
@@ -166,7 +111,7 @@ For each commit:
 4. Wait for all required checks to pass
 5. **CONFIRMATION REQUIRED**: All tests passing?
 
-### Step 9: Merge Hotfix PR
+### Step 8: Merge Hotfix PR
 
 1. Verify all checks have passed
 2. Check for required approvals
@@ -174,7 +119,7 @@ For each commit:
 4. Delete the hotfix branch
 5. **CONFIRMATION REQUIRED**: PR merged successfully?
 
-### Step 10: Create Version Bump
+### Step 9: Create Version Bump
 
 1. Checkout the core branch: `git checkout core/X.Y`
 2. Pull latest changes: `git pull origin core/X.Y`
@@ -186,7 +131,7 @@ For each commit:
 7. Commit: `git commit -m "[release] Bump version to 1.23.5"`
 8. **CONFIRMATION REQUIRED**: Version bump correct?
 
-### Step 11: Create Release PR
+### Step 10: Create Release PR
 
 1. Push release branch: `git push origin release/1.23.5`
 2. Create PR with Release label:
@@ -239,7 +184,7 @@ For each commit:
      ```
 5. **CONFIRMATION REQUIRED**: Release PR has "Release" label?
 
-### Step 12: Monitor Release Process
+### Step 11: Monitor Release Process
 
 1. Wait for PR checks to pass
 2. **FINAL CONFIRMATION**: Ready to trigger release by merging?
@@ -254,102 +199,7 @@ For each commit:
    - PyPI upload
    - pnpm types publication
 
-### Step 13: Manually Publish Draft Release
-
-**CRITICAL**: The release workflow creates a DRAFT release. You must manually publish it:
-
-1. **Go to GitHub Releases:** https://github.com/Comfy-Org/ComfyUI_frontend/releases
-2. **Find the DRAFT release** (e.g., "v1.23.5 Draft")
-3. **Click "Edit release"**
-4. **UNCHECK "Set as the latest release"** ⚠️ **CRITICAL**
-   - This prevents the hotfix from showing as "latest" 
-   - Main branch should always be "latest release"
-5. **Click "Publish release"**
-6. **CONFIRMATION REQUIRED**: Draft release published with "latest" unchecked?
-
-### Step 14: Create ComfyUI Requirements.txt Update PR
-
-**IMPORTANT**: Create PR to update ComfyUI's requirements.txt via fork:
-
-1. **Setup fork (if needed):**
-   ```bash
-   # Check if fork already exists
-   if gh repo view ComfyUI --json owner | jq -r '.owner.login' | grep -q "$(gh api user --jq .login)"; then
-     echo "Fork already exists"
-   else
-     # Fork the ComfyUI repository
-     gh repo fork comfyanonymous/ComfyUI --clone=false
-     echo "Created fork of ComfyUI"
-   fi
-   ```
-
-2. **Clone fork and create branch:**
-   ```bash
-   # Clone your fork (or use existing clone)
-   GITHUB_USER=$(gh api user --jq .login)
-   if [ ! -d "ComfyUI-fork" ]; then
-     gh repo clone ${GITHUB_USER}/ComfyUI ComfyUI-fork
-   fi
-   
-   cd ComfyUI-fork
-   git checkout master
-   git pull origin master
-   
-   # Create update branch
-   BRANCH_NAME="update-frontend-${NEW_VERSION}"
-   git checkout -b ${BRANCH_NAME}
-   ```
-
-3. **Update requirements.txt:**
-   ```bash
-   # Update the version in requirements.txt
-   sed -i "s/comfyui-frontend-package==[0-9].*$/comfyui-frontend-package==${NEW_VERSION}/" requirements.txt
-   
-   # Verify the change
-   grep "comfyui-frontend-package" requirements.txt
-   
-   # Commit the change
-   git add requirements.txt
-   git commit -m "Bump frontend to ${NEW_VERSION}"
-   git push origin ${BRANCH_NAME}
-   ```
-
-4. **Create PR from fork:**
-   ```bash
-   # Create PR using gh CLI from fork
-   gh pr create \
-     --repo comfyanonymous/ComfyUI \
-     --title "Bump frontend to ${NEW_VERSION}" \
-     --body "$(cat <<EOF
-Bump frontend to ${NEW_VERSION}
-
-\`\`\`
-python main.py --front-end-version Comfy-Org/ComfyUI_frontend@${NEW_VERSION}
-\`\`\`
-
-- Diff: [Comfy-Org/ComfyUI_frontend: v${OLD_VERSION}...v${NEW_VERSION}](https://github.com/Comfy-Org/ComfyUI_frontend/compare/v${OLD_VERSION}...v${NEW_VERSION})
-- PyPI Package: https://pypi.org/project/comfyui-frontend-package/${NEW_VERSION}/
-- npm Types: https://www.npmjs.com/package/@comfyorg/comfyui-frontend-types/v/${NEW_VERSION}
-
-## Changes
-
-- Fix: [Brief description of hotfixes included]
-EOF
-)"
-   ```
-
-5. **Clean up:**
-   ```bash
-   # Return to original directory
-   cd ..
-   
-   # Keep fork directory for future updates
-   echo "Fork directory 'ComfyUI-fork' kept for future use"
-   ```
-
-6. **CONFIRMATION REQUIRED**: ComfyUI requirements.txt PR created from fork?
-
-### Step 15: Post-Release Verification
+### Step 12: Post-Release Verification
 
 1. Verify GitHub release:
    ```bash
@@ -363,14 +213,12 @@ EOF
    ```bash
    pnpm view @comfyorg/comfyui-frontend-types@1.23.5
    ```
-4. Monitor ComfyUI requirements.txt PR for approval/merge
-5. Generate release summary with:
+4. Generate release summary with:
    - Version released
    - Commits included
    - Issues fixed
    - Distribution status
-   - ComfyUI integration status
-6. **CONFIRMATION REQUIRED**: Hotfix release fully completed?
+5. **CONFIRMATION REQUIRED**: Release completed successfully?
 
 ## Safety Checks
 
@@ -392,28 +240,19 @@ If something goes wrong:
 
 ## Important Notes
 
-- **Always try automated backports first** - This command is for when automation fails
 - Core branch version will be behind main - this is expected
 - The "Release" label triggers the PyPI/npm publication
-- **CRITICAL**: Always uncheck "Set as latest release" for hotfix releases
-- **Must create ComfyUI requirements.txt PR** - Hotfix isn't complete without it
 - PR numbers must include the `#` prefix
 - Mixed commits/PRs are supported but review carefully
 - Always wait for full test suite before proceeding
 
-## Modern Workflow Context
+## Expected Timeline
 
-**Primary Backport Method:** Automated via `needs-backport` + `X.YY` labels
-**This Command Usage:** 
-- Smart path detection - skip to version bump if backports already merged
-- Fallback to manual cherry-picking only when automation fails/has conflicts
-**Complete Hotfix:** Includes GitHub release publishing + ComfyUI requirements.txt integration
+- Step 1-3: ~10 minutes (analysis)
+- Steps 4-6: ~15-30 minutes (cherry-picking)
+- Step 7: ~10-20 minutes (tests)
+- Steps 8-10: ~10 minutes (version bump)
+- Step 11-12: ~15-20 minutes (release)
+- Total: ~60-90 minutes
 
-## Workflow Paths
-
-- **Path A:** Backports already merged → Skip to Step 10 (Version Bump)
-- **Path B:** Backport PRs need merging → Merge them → Skip to Step 10 (Version Bump)  
-- **Path C:** No/failed backports → Manual cherry-picking (Steps 2-9) → Version Bump (Step 10)
-
-
-This process ensures a complete hotfix release with proper GitHub publishing, ComfyUI integration, and multiple safety checkpoints.
+This process ensures a safe, verified hotfix release with multiple confirmation points and clear tracking of what changes are being released.

.env_example

@@ -33,4 +33,3 @@ DISABLE_VUE_PLUGINS=false
 # Algolia credentials required for developing with the new custom node manager.
 ALGOLIA_APP_ID=4E0RO38HS8
 ALGOLIA_API_KEY=684d998c36b67a9a9fce8fc2d8860579
-

.git-blame-ignore-revs

@@ -4,6 +4,3 @@
 
 # npm run format on litegraph merge (10,672 insertions, 7,327 deletions across 129 files)
 c53f197de2a3e0fa66b16dedc65c131235c1c4b6
-
-# Reorganize renderer components into domain-driven folder structure
-c8a83a9caede7bdb5f8598c5492b07d08c339d49

.github/workflows/backport.yaml

@@ -2,7 +2,7 @@ name: Auto Backport
 
 on:
   pull_request_target:
-    types: [closed, labeled]
+    types: [closed]
     branches: [main]
 
 jobs:
@@ -25,27 +25,7 @@ jobs:
           git config user.name "github-actions[bot]"
           git config user.email "github-actions[bot]@users.noreply.github.com"
 
-      - name: Check if backports already exist
-        id: check-existing
-        env:
-          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          PR_NUMBER: ${{ github.event.pull_request.number }}
-        run: |
-          # Check for existing backport PRs for this PR number
-          EXISTING_BACKPORTS=$(gh pr list --state all --search "backport-${PR_NUMBER}-to" --json title,headRefName,baseRefName | jq -r '.[].headRefName')
-          
-          if [ -z "$EXISTING_BACKPORTS" ]; then
-            echo "skip=false" >> $GITHUB_OUTPUT
-            exit 0
-          fi
-          
-          echo "Found existing backport PRs:"
-          echo "$EXISTING_BACKPORTS"
-          echo "skip=true" >> $GITHUB_OUTPUT
-          echo "::warning::Backport PRs already exist for PR #${PR_NUMBER}, skipping to avoid duplicates"
-
       - name: Extract version labels
-        if: steps.check-existing.outputs.skip != 'true'
         id: versions
         run: |
           # Extract version labels (e.g., "1.24", "1.22")
@@ -72,7 +52,6 @@ jobs:
           echo "Found version labels: ${VERSIONS}"
 
       - name: Backport commits
-        if: steps.check-existing.outputs.skip != 'true'
         id: backport
         env:
           PR_NUMBER: ${{ github.event.pull_request.number }}
@@ -130,7 +109,7 @@ jobs:
           fi
 
       - name: Create PR for each successful backport
-        if: steps.check-existing.outputs.skip != 'true' && steps.backport.outputs.success
+        if: steps.backport.outputs.success
         env:
           GH_TOKEN: ${{ secrets.PR_GH_TOKEN }}
         run: |
@@ -162,7 +141,7 @@ jobs:
           done
 
       - name: Comment on failures
-        if: steps.check-existing.outputs.skip != 'true' && failure() && steps.backport.outputs.failed
+        if: failure() && steps.backport.outputs.failed
         env:
           GH_TOKEN: ${{ github.token }}
         run: |

.github/workflows/claude-pr-review.yml

@@ -47,7 +47,6 @@ jobs:
     needs: wait-for-ci
     if: needs.wait-for-ci.outputs.should-proceed == 'true'
     runs-on: ubuntu-latest
-    timeout-minutes: 30
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
@@ -70,17 +69,19 @@ jobs:
           pnpm install -g typescript @vue/compiler-sfc
 
       - name: Run Claude PR Review
-        uses: anthropics/claude-code-action@v1.0.6
+        uses: anthropics/claude-code-action@main
         with:
           label_trigger: "claude-review"
-          prompt: |
+          direct_prompt: |
             Read the file .claude/commands/comprehensive-pr-review.md and follow ALL the instructions exactly. 
             
             CRITICAL: You must post individual inline comments using the gh api commands shown in the file. 
             DO NOT create a summary comment. 
             Each issue must be posted as a separate inline comment on the specific line of code.
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-          claude_args: "--max-turns 256 --allowedTools 'Bash(git:*),Bash(gh api:*),Bash(gh pr:*),Bash(gh repo:*),Bash(jq:*),Bash(echo:*),Read,Write,Edit,Glob,Grep,WebFetch'"
+          max_turns: 256
+          timeout_minutes: 30
+          allowed_tools: "Bash(git:*),Bash(gh api:*),Bash(gh pr:*),Bash(gh repo:*),Bash(jq:*),Bash(echo:*),Read,Write,Edit,Glob,Grep,WebFetch"
         env:
           PR_NUMBER: ${{ github.event.pull_request.number }}
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/create-release-candidate-branch.yaml

@@ -128,6 +128,45 @@ jobs:
           echo "- Critical security patches"
           echo "- Documentation updates"
 
+      - name: Create branch protection rules
+        if: steps.check_version.outputs.is_minor_bump == 'true' && env.branch_exists != 'true'
+        env:
+          GITHUB_TOKEN: ${{ secrets.PR_GH_TOKEN || secrets.GITHUB_TOKEN }}
+        run: |
+          BRANCH_NAME="${{ steps.check_version.outputs.branch_name }}"
+
+          # Create branch protection using GitHub API
+          echo "Setting up branch protection for $BRANCH_NAME..."
+
+          RESPONSE=$(curl -s -w "\n%{http_code}" -X PUT \
+            -H "Authorization: token $GITHUB_TOKEN" \
+            -H "Accept: application/vnd.github.v3+json" \
+            "https://api.github.com/repos/${{ github.repository }}/branches/$BRANCH_NAME/protection" \
+            -d '{
+              "required_status_checks": {
+                "strict": true,
+                "contexts": ["lint-and-format", "test", "playwright-tests"]
+              },
+              "enforce_admins": false,
+              "required_pull_request_reviews": {
+                "required_approving_review_count": 1,
+                "dismiss_stale_reviews": true
+              },
+              "restrictions": null,
+              "allow_force_pushes": false,
+              "allow_deletions": false
+            }')
+
+          HTTP_CODE=$(echo "$RESPONSE" | tail -n 1)
+          BODY=$(echo "$RESPONSE" | sed '$d')
+
+          if [[ "$HTTP_CODE" -eq 200 ]] || [[ "$HTTP_CODE" -eq 201 ]]; then
+            echo "✅ Branch protection successfully applied"
+          else
+            echo "⚠️  Failed to apply branch protection (HTTP $HTTP_CODE)"
+            echo "Response: $BODY"
+            # Don't fail the workflow, just warn
+          fi
 
       - name: Post summary
         if: steps.check_version.outputs.is_minor_bump == 'true'

.github/workflows/pr-playwright-deploy.yaml

@@ -1,4 +1,4 @@
-name: PR Playwright Deploy (Forks)
+name: PR Playwright Deploy and Comment
 
 on:
   workflow_run:
@@ -9,84 +9,293 @@ env:
   DATE_FORMAT: '+%m/%d/%Y, %I:%M:%S %p'
 
 jobs:
-  deploy-and-comment-forked-pr:
+  deploy-reports:
     runs-on: ubuntu-latest
-    if: |
-      github.repository == 'Comfy-Org/ComfyUI_frontend' && 
-      github.event.workflow_run.event == 'pull_request' &&
-      github.event.workflow_run.head_repository != null &&
-      github.event.workflow_run.repository != null &&
-      github.event.workflow_run.head_repository.full_name != github.event.workflow_run.repository.full_name
+    if: github.repository == 'Comfy-Org/ComfyUI_frontend' && github.event.workflow_run.event == 'pull_request' && github.event.action == 'completed'
     permissions:
-      pull-requests: write
       actions: read
+    strategy:
+      fail-fast: false
+      matrix:
+        browser: [chromium, chromium-2x, chromium-0.5x, mobile-chrome]
     steps:
-      - name: Log workflow trigger info
-        run: |
-          echo "Repository: ${{ github.repository }}"
-          echo "Event: ${{ github.event.workflow_run.event }}"
-          echo "Head repo: ${{ github.event.workflow_run.head_repository.full_name || 'null' }}"
-          echo "Base repo: ${{ github.event.workflow_run.repository.full_name || 'null' }}"
-          echo "Is forked: ${{ github.event.workflow_run.head_repository.full_name != github.event.workflow_run.repository.full_name }}"
-
-      - name: Checkout repository
-        uses: actions/checkout@v4
-
-      - name: Get PR Number
-        id: pr
+      - name: Get PR info
+        id: pr-info
         uses: actions/github-script@v7
         with:
           script: |
-            const { data: prs } = await github.rest.pulls.list({
+            const { data: pullRequests } = await github.rest.pulls.list({
               owner: context.repo.owner,
               repo: context.repo.repo,
               state: 'open',
+              head: `${context.repo.owner}:${context.payload.workflow_run.head_branch}`,
             });
             
-            const pr = prs.find(p => p.head.sha === context.payload.workflow_run.head_sha);
-            
-            if (!pr) {
-              console.log('No PR found for SHA:', context.payload.workflow_run.head_sha);
-              return null;
+            if (pullRequests.length === 0) {
+              console.log('No open PR found for this branch');
+              return { number: null, sanitized_branch: null };
             }
             
-            console.log(`Found PR #${pr.number} from fork: ${context.payload.workflow_run.head_repository.full_name}`);
-            return pr.number;
+            const pr = pullRequests[0];
+            const branchName = context.payload.workflow_run.head_branch;
+            const sanitizedBranch = branchName.toLowerCase().replace(/[^a-z0-9-]/g, '-').replace(/--+/g, '-').replace(/^-|-$/g, '');
+            
+            return {
+              number: pr.number,
+              sanitized_branch: sanitizedBranch
+            };
 
-      - name: Handle Test Start
-        if: steps.pr.outputs.result != 'null' && github.event.action == 'requested'
-        env:
-          GITHUB_TOKEN: ${{ github.token }}
+      - name: Set project name
+        if: fromJSON(steps.pr-info.outputs.result).number != null
+        id: project-name
         run: |
-          chmod +x scripts/cicd/pr-playwright-deploy-and-comment.sh
-          ./scripts/cicd/pr-playwright-deploy-and-comment.sh \
-            "${{ steps.pr.outputs.result }}" \
-            "${{ github.event.workflow_run.head_branch }}" \
-            "starting" \
-            "$(date -u '${{ env.DATE_FORMAT }}')"
+          if [ "${{ matrix.browser }}" = "chromium-0.5x" ]; then
+            echo "name=comfyui-playwright-chromium-0-5x" >> $GITHUB_OUTPUT
+          else
+            echo "name=comfyui-playwright-${{ matrix.browser }}" >> $GITHUB_OUTPUT
+          fi
+          echo "branch=${{ fromJSON(steps.pr-info.outputs.result).sanitized_branch }}" >> $GITHUB_OUTPUT
 
-      - name: Download and Deploy Reports
-        if: steps.pr.outputs.result != 'null' && github.event.action == 'completed'
+      - name: Download playwright report
+        if: fromJSON(steps.pr-info.outputs.result).number != null
         uses: actions/download-artifact@v4
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           run-id: ${{ github.event.workflow_run.id }}
-          pattern: playwright-report-*
-          path: reports
+          name: playwright-report-${{ matrix.browser }}
+          path: playwright-report
+
+      - name: Install Wrangler
+        if: fromJSON(steps.pr-info.outputs.result).number != null
+        run: npm install -g wrangler
+
+      - name: Deploy to Cloudflare Pages (${{ matrix.browser }})
+        if: fromJSON(steps.pr-info.outputs.result).number != null
+        id: cloudflare-deploy
+        continue-on-error: true
+        run: |
+          # Retry logic for wrangler deploy (3 attempts)
+          RETRY_COUNT=0
+          MAX_RETRIES=3
+          SUCCESS=false
           
-      - name: Handle Test Completion
-        if: steps.pr.outputs.result != 'null' && github.event.action == 'completed'
+          while [ $RETRY_COUNT -lt $MAX_RETRIES ] && [ $SUCCESS = false ]; do
+            RETRY_COUNT=$((RETRY_COUNT + 1))
+            echo "Deployment attempt $RETRY_COUNT of $MAX_RETRIES..."
+            
+            if npx wrangler pages deploy playwright-report --project-name=${{ steps.project-name.outputs.name }} --branch=${{ steps.project-name.outputs.branch }}; then
+              SUCCESS=true
+              echo "Deployment successful on attempt $RETRY_COUNT"
+            else
+              echo "Deployment failed on attempt $RETRY_COUNT"
+              if [ $RETRY_COUNT -lt $MAX_RETRIES ]; then
+                echo "Retrying in 10 seconds..."
+                sleep 10
+              fi
+            fi
+          done
+          
+          if [ $SUCCESS = false ]; then
+            echo "All deployment attempts failed"
+            exit 1
+          fi
         env:
           CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
           CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
-          GITHUB_TOKEN: ${{ github.token }}
+
+  comment-tests-starting:
+    runs-on: ubuntu-latest
+    if: github.repository == 'Comfy-Org/ComfyUI_frontend' && github.event.workflow_run.event == 'pull_request' && github.event.action == 'requested'
+    permissions:
+      pull-requests: write
+      actions: read
+    steps:
+      - name: Get PR number
+        id: pr
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const { data: pullRequests } = await github.rest.pulls.list({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              state: 'open',
+              head: `${context.repo.owner}:${context.payload.workflow_run.head_branch}`,
+            });
+
+            if (pullRequests.length === 0) {
+              console.log('No open PR found for this branch');
+              return null;
+            }
+
+            return pullRequests[0].number;
+
+      - name: Get completion time
+        id: completion-time
+        run: echo "time=$(date -u '${{ env.DATE_FORMAT }}')" >> $GITHUB_OUTPUT
+
+      - name: Generate comment body for start
+        if: steps.pr.outputs.result != 'null'
+        id: comment-body-start
         run: |
-          # Rename merged report if exists
-          [ -d "reports/playwright-report-chromium-merged" ] && \
-            mv reports/playwright-report-chromium-merged reports/playwright-report-chromium
-          
-          chmod +x scripts/cicd/pr-playwright-deploy-and-comment.sh
-          ./scripts/cicd/pr-playwright-deploy-and-comment.sh \
-            "${{ steps.pr.outputs.result }}" \
-            "${{ github.event.workflow_run.head_branch }}" \
-            "completed"
+          echo "<!-- PLAYWRIGHT_TEST_STATUS -->" > comment.md
+          echo "## 🎭 Playwright Test Results" >> comment.md
+          echo "" >> comment.md
+          echo "<img alt='comfy-loading-gif' src='https://github.com/user-attachments/assets/755c86ee-e445-4ea8-bc2c-cca85df48686' width='14px' height='14px' style='vertical-align: middle; margin-right: 4px;' /> **Tests are starting...** " >> comment.md
+          echo "" >> comment.md
+          echo "⏰ Started at: ${{ steps.completion-time.outputs.time }} UTC" >> comment.md
+          echo "" >> comment.md
+          echo "### 🚀 Running Tests" >> comment.md
+          echo "- 🧪 **chromium**: Running tests..." >> comment.md
+          echo "- 🧪 **chromium-0.5x**: Running tests..." >> comment.md
+          echo "- 🧪 **chromium-2x**: Running tests..." >> comment.md
+          echo "- 🧪 **mobile-chrome**: Running tests..." >> comment.md
+          echo "" >> comment.md
+          echo "---" >> comment.md
+          echo "⏱️ Please wait while tests are running across all browsers..." >> comment.md
+
+      - name: Comment PR - Tests Started
+        if: steps.pr.outputs.result != 'null'
+        uses: edumserrano/find-create-or-update-comment@82880b65c8a3a6e4c70aa05a204995b6c9696f53 # v3.0.0
+        with:
+          issue-number: ${{ steps.pr.outputs.result }}
+          body-includes: '<!-- PLAYWRIGHT_TEST_STATUS -->'
+          comment-author: 'github-actions[bot]'
+          edit-mode: replace
+          body-path: comment.md
+
+  comment-tests-completed:
+    runs-on: ubuntu-latest
+    needs: deploy-reports
+    if: github.repository == 'Comfy-Org/ComfyUI_frontend' && github.event.workflow_run.event == 'pull_request' && github.event.action == 'completed' && always()
+    permissions:
+      pull-requests: write
+      actions: read
+    steps:
+      - name: Get PR number
+        id: pr
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const { data: pullRequests } = await github.rest.pulls.list({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              state: 'open',
+              head: `${context.repo.owner}:${context.payload.workflow_run.head_branch}`,
+            });
+
+            if (pullRequests.length === 0) {
+              console.log('No open PR found for this branch');
+              return null;
+            }
+
+            return pullRequests[0].number;
+
+      - name: Download all deployment info
+        if: steps.pr.outputs.result != 'null'
+        uses: actions/download-artifact@v4
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          run-id: ${{ github.event.workflow_run.id }}
+          pattern: deployment-info-*
+          merge-multiple: true
+          path: deployment-info
+
+      - name: Get completion time
+        id: completion-time
+        run: echo "time=$(date -u '${{ env.DATE_FORMAT }}')" >> $GITHUB_OUTPUT
+
+      - name: Generate comment body for completion
+        if: steps.pr.outputs.result != 'null'
+        id: comment-body-completed
+        run: |
+          echo "<!-- PLAYWRIGHT_TEST_STATUS -->" > comment.md
+          echo "## 🎭 Playwright Test Results" >> comment.md
+          echo "" >> comment.md
+
+          # Check if we have any test results
+          FILES_FOUND=false
+          for file in deployment-info/*.txt; do
+            if [ -f "$file" ]; then
+              FILES_FOUND=true
+              break
+            fi
+          done
+
+          if [ "$FILES_FOUND" = "false" ]; then
+            # No test results found - tests were skipped or not run
+            echo "⏭️ **Tests were skipped or not run**" >> comment.md
+            echo "" >> comment.md
+            echo "⏰ Completed at: ${{ steps.completion-time.outputs.time }} UTC" >> comment.md
+            echo "" >> comment.md
+            echo "No test results available for this PR. This can happen when:" >> comment.md
+            echo "- Tests were skipped due to branch filters" >> comment.md
+            echo "- The test workflow didn't complete successfully" >> comment.md
+            echo "- Tests are still running" >> comment.md
+          else
+            # Check if all tests passed
+            ALL_PASSED=true
+            for file in deployment-info/*.txt; do
+              if [ -f "$file" ]; then
+                browser=$(basename "$file" .txt)
+                info=$(cat "$file")
+                exit_code=$(echo "$info" | cut -d'|' -f2)
+                if [ "$exit_code" != "0" ]; then
+                  ALL_PASSED=false
+                  break
+                fi
+              fi
+            done
+
+            if [ "$ALL_PASSED" = "true" ]; then
+              echo "✅ **All tests passed across all browsers!**" >> comment.md
+            else
+              echo "❌ **Some tests failed!**" >> comment.md
+            fi
+
+            echo "" >> comment.md
+            echo "⏰ Completed at: ${{ steps.completion-time.outputs.time }} UTC" >> comment.md
+            echo "" >> comment.md
+            echo "### 📊 Test Reports by Browser" >> comment.md
+
+            for file in deployment-info/*.txt; do
+              if [ -f "$file" ]; then
+                browser=$(basename "$file" .txt)
+                info=$(cat "$file")
+                exit_code=$(echo "$info" | cut -d'|' -f2)
+                url=$(echo "$info" | cut -d'|' -f3)
+                
+                # Validate URLs before using them in comments
+                sanitized_url=$(echo "$url" | grep -E '^https://[a-z0-9.-]+\.pages\.dev(/.*)?$' || echo "INVALID_URL")
+                if [ "$sanitized_url" = "INVALID_URL" ]; then
+                  echo "Invalid deployment URL detected: $url"
+                  url="#" # Use safe fallback
+                fi
+                
+                if [ "$exit_code" = "0" ]; then
+                  status="✅"
+                else
+                  status="❌"
+                fi
+                
+                echo "- $status **$browser**: [View Report]($url)" >> comment.md
+              fi
+            done
+
+            echo "" >> comment.md
+            echo "---" >> comment.md
+            if [ "$ALL_PASSED" = "true" ]; then
+              echo "🎉 Your tests are passing across all browsers!" >> comment.md
+            else
+              echo "⚠️ Please check the test reports for details on failures." >> comment.md
+            fi
+          fi
+
+      - name: Comment PR - Tests Complete
+        if: steps.pr.outputs.result != 'null'
+        uses: edumserrano/find-create-or-update-comment@82880b65c8a3a6e4c70aa05a204995b6c9696f53 # v3.0.0
+        with:
+          issue-number: ${{ steps.pr.outputs.result }}
+          body-includes: '<!-- PLAYWRIGHT_TEST_STATUS -->'
+          comment-author: 'github-actions[bot]'
+          edit-mode: replace
+          body-path: comment.md

.github/workflows/test-ui.yaml

@@ -239,6 +239,32 @@ jobs:
           path: ComfyUI_frontend/playwright-report/
           retention-days: 30
 
+      # Save test result for deployment info
+      - name: Save test result for ${{ matrix.browser }}
+        if: always()
+        run: |
+          # Use outcome to determine exit code (0 for success, 1 for failure/skipped)
+          if [ "${{ steps.playwright.outcome }}" = "success" ]; then
+            EXIT_CODE=0
+          else
+            EXIT_CODE=1
+          fi
+          # Generate project name based on browser
+          if [ "${{ matrix.browser }}" = "chromium-0.5x" ]; then
+            PROJECT_NAME="comfyui-playwright-chromium-0-5x"
+          else
+            PROJECT_NAME="comfyui-playwright-${{ matrix.browser }}"
+          fi
+          echo "${{ matrix.browser }}|${EXIT_CODE}|https://${PROJECT_NAME}.pages.dev" > deployment-info-${{ matrix.browser }}.txt
+        working-directory: ComfyUI_frontend
+
+      - uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: deployment-info-${{ matrix.browser }}
+          path: ComfyUI_frontend/deployment-info-${{ matrix.browser }}.txt
+          retention-days: 1
+
   # Merge sharded test reports
   merge-reports:
     needs: [playwright-tests-chromium-sharded]
@@ -285,64 +311,24 @@ jobs:
           path: ComfyUI_frontend/playwright-report/
           retention-days: 30
 
-  #### BEGIN Deployment and commenting (non-forked PRs only)
-  # when using pull_request event, we have permission to comment directly
-  # if its a forked repo, we need to use workflow_run event in a separate workflow (pr-playwright-deploy.yaml)
+      - name: Save deployment info for merged chromium report
+        if: always()
+        run: |
+          # Determine exit code based on the needs context
+          # The merge-reports job only runs if sharded tests complete (success, failure, or cancelled)
+          # We consider it successful only if all sharded tests succeeded
+          if [ "${{ needs.playwright-tests-chromium-sharded.result }}" = "success" ]; then
+            EXIT_CODE=0
+          else
+            EXIT_CODE=1
+          fi
+          
+          echo "chromium|${EXIT_CODE}|https://comfyui-playwright-chromium.pages.dev" > deployment-info-chromium.txt
+        working-directory: ComfyUI_frontend
 
-  # Post starting comment for non-forked PRs
-  comment-on-pr-start:
-    runs-on: ubuntu-latest
-    if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork == false
-    permissions:
-      pull-requests: write
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        
-      - name: Get start time
-        id: start-time
-        run: echo "time=$(date -u '+%m/%d/%Y, %I:%M:%S %p')" >> $GITHUB_OUTPUT
-        
-      - name: Post starting comment
-        env:
-          GITHUB_TOKEN: ${{ github.token }}
-        run: |
-          chmod +x scripts/cicd/pr-playwright-deploy-and-comment.sh
-          ./scripts/cicd/pr-playwright-deploy-and-comment.sh \
-            "${{ github.event.pull_request.number }}" \
-            "${{ github.head_ref }}" \
-            "starting" \
-            "${{ steps.start-time.outputs.time }}"
-          
-  # Deploy and comment for non-forked PRs only
-  deploy-and-comment:
-    needs: [playwright-tests, merge-reports]
-    runs-on: ubuntu-latest
-    if: always() && github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork == false
-    permissions:
-      pull-requests: write
-      contents: read
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        
-      - name: Download all playwright reports
-        uses: actions/download-artifact@v4
+      - uses: actions/upload-artifact@v4
+        if: always()
         with:
-          pattern: playwright-report-*
-          path: reports
-          
-      - name: Make deployment script executable
-        run: chmod +x scripts/cicd/pr-playwright-deploy-and-comment.sh
-        
-      - name: Deploy reports and comment on PR
-        env:
-          CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
-          CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
-          GITHUB_TOKEN: ${{ github.token }}
-        run: |
-          ./scripts/cicd/pr-playwright-deploy-and-comment.sh \
-            "${{ github.event.pull_request.number }}" \
-            "${{ github.head_ref }}" \
-            "completed"
-  #### END Deployment and commenting (non-forked PRs only)
+          name: deployment-info-chromium
+          path: ComfyUI_frontend/deployment-info-chromium.txt
+          retention-days: 1

.husky/pre-commit

@@ -1,4 +1,4 @@
 #!/usr/bin/env bash
 
-pnpm exec lint-staged
-pnpm exec tsx scripts/check-unused-i18n-keys.ts
+npx lint-staged
+npx tsx scripts/check-unused-i18n-keys.ts

.husky/pre-push

@@ -1,5 +0,0 @@
-#!/usr/bin/env bash
-
-# Run Knip with cache via package script
-pnpm knip
-

.mcp.json

@@ -0,0 +1,12 @@
+{
+  "mcpServers": {
+    "playwright": {
+      "command": "npx",
+      "args": ["-y", "@executeautomation/playwright-mcp-server"]
+    },
+    "context7": {
+      "command": "npx",
+      "args": ["-y", "@upstash/context7-mcp"]
+    }
+  }
+}

browser_tests/fixtures/ComfyMouse.ts

@@ -10,7 +10,7 @@ import type { Position } from './types'
  * - {@link Mouse.move}
  * - {@link Mouse.up}
  */
-interface DragOptions {
+export interface DragOptions {
   button?: 'left' | 'right' | 'middle'
   clickCount?: number
   steps?: number

browser_tests/fixtures/ComfyPage.ts

@@ -453,32 +453,6 @@ export class ComfyPage {
     await workflowsTab.close()
   }
 
-  /**
-   * Attach a screenshot to the test report.
-   * By default, screenshots are only taken in non-CI environments.
-   * @param name - Name for the screenshot attachment
-   * @param options - Optional configuration
-   * @param options.runInCI - Whether to take screenshot in CI (default: false)
-   * @param options.fullPage - Whether to capture full page (default: false)
-   */
-  async attachScreenshot(
-    name: string,
-    options: { runInCI?: boolean; fullPage?: boolean } = {}
-  ) {
-    const { runInCI = false, fullPage = false } = options
-
-    // Skip in CI unless explicitly requested
-    if (process.env.CI && !runInCI) {
-      return
-    }
-
-    const testInfo = comfyPageFixture.info()
-    await testInfo.attach(name, {
-      body: await this.page.screenshot({ fullPage }),
-      contentType: 'image/png'
-    })
-  }
-
   async resetView() {
     if (await this.resetViewButton.isVisible()) {
       await this.resetViewButton.click()

browser_tests/fixtures/components/Topbar.ts

@@ -1,13 +1,7 @@
-import { Locator, Page, expect } from '@playwright/test'
+import { Locator, Page } from '@playwright/test'
 
 export class Topbar {
-  private readonly menuLocator: Locator
-  private readonly menuTrigger: Locator
-
-  constructor(public readonly page: Page) {
-    this.menuLocator = page.locator('.comfy-command-menu')
-    this.menuTrigger = page.locator('.comfyui-logo-wrapper')
-  }
+  constructor(public readonly page: Page) {}
 
   async getTabNames(): Promise<string[]> {
     return await this.page
@@ -21,33 +15,10 @@ export class Topbar {
       .innerText()
   }
 
-  /**
-   * Get a menu item by its label, optionally within a specific parent container
-   */
-  getMenuItem(itemLabel: string, parent?: Locator): Locator {
-    if (parent) {
-      return parent.locator(`.p-tieredmenu-item:has-text("${itemLabel}")`)
-    }
-
+  getMenuItem(itemLabel: string): Locator {
     return this.page.locator(`.p-menubar-item-label:text-is("${itemLabel}")`)
   }
 
-  /**
-   * Get the visible submenu (last visible submenu in case of nested menus)
-   */
-  getVisibleSubmenu(): Locator {
-    return this.page.locator('.p-tieredmenu-submenu:visible').last()
-  }
-
-  /**
-   * Check if a menu item has an active checkmark
-   */
-  async isMenuItemActive(menuItem: Locator): Promise<boolean> {
-    const checkmark = menuItem.locator('.pi-check')
-    const classes = await checkmark.getAttribute('class')
-    return classes ? !classes.includes('invisible') : false
-  }
-
   getWorkflowTab(tabName: string): Locator {
     return this.page
       .locator(`.workflow-tabs .workflow-label:has-text("${tabName}")`)
@@ -95,50 +66,10 @@ export class Topbar {
 
   async openTopbarMenu() {
     await this.page.waitForTimeout(1000)
-    await this.menuTrigger.click()
-    await this.menuLocator.waitFor({ state: 'visible' })
-    return this.menuLocator
-  }
-
-  /**
-   * Close the topbar menu by clicking outside
-   */
-  async closeTopbarMenu() {
-    await this.page.locator('body').click({ position: { x: 10, y: 10 } })
-    await expect(this.menuLocator).not.toBeVisible()
-  }
-
-  /**
-   * Navigate to a submenu by hovering over a menu item
-   */
-  async openSubmenu(menuItemLabel: string): Promise<Locator> {
-    const menuItem = this.getMenuItem(menuItemLabel)
-    await menuItem.hover()
-    const submenu = this.getVisibleSubmenu()
-    await submenu.waitFor({ state: 'visible' })
-    return submenu
-  }
-
-  /**
-   * Get theme menu items and interact with theme switching
-   */
-  async getThemeMenuItems() {
-    const themeSubmenu = await this.openSubmenu('Theme')
-    return {
-      submenu: themeSubmenu,
-      darkTheme: this.getMenuItem('Dark (Default)', themeSubmenu),
-      lightTheme: this.getMenuItem('Light', themeSubmenu)
-    }
-  }
-
-  /**
-   * Switch to a specific theme
-   */
-  async switchTheme(theme: 'dark' | 'light') {
-    const { darkTheme, lightTheme } = await this.getThemeMenuItems()
-    const themeItem = theme === 'dark' ? darkTheme : lightTheme
-    const themeLabel = themeItem.locator('.p-menubar-item-label')
-    await themeLabel.click()
+    await this.page.locator('.comfyui-logo-wrapper').click()
+    const menu = this.page.locator('.comfy-command-menu')
+    await menu.waitFor({ state: 'visible' })
+    return menu
   }
 
   async triggerTopbarCommand(path: string[]) {
@@ -148,7 +79,9 @@ export class Topbar {
 
     const menu = await this.openTopbarMenu()
     const tabName = path[0]
-    const topLevelMenuItem = this.getMenuItem(tabName)
+    const topLevelMenuItem = this.page.locator(
+      `.p-menubar-item-label:text-is("${tabName}")`
+    )
     const topLevelMenu = menu
       .locator('.p-tieredmenu-item')
       .filter({ has: topLevelMenuItem })

browser_tests/fixtures/utils/litegraphUtils.ts

@@ -134,7 +134,7 @@ export class SubgraphSlotReference {
   }
 }
 
-class NodeSlotReference {
+export class NodeSlotReference {
   constructor(
     readonly type: 'input' | 'output',
     readonly index: number,
@@ -201,7 +201,7 @@ class NodeSlotReference {
   }
 }
 
-class NodeWidgetReference {
+export class NodeWidgetReference {
   constructor(
     readonly index: number,
     readonly node: NodeReference