Fix LoadImage bleeds values between subgraphs

.claude/commands/comprehensive-pr-review.md

@@ -1,275 +1,479 @@
 # Comprehensive PR Review for ComfyUI Frontend
 
-You are performing a comprehensive code review for the PR specified in the PR_NUMBER environment variable. This is not a simple linting check - you need to provide deep architectural analysis, security review, performance insights, and implementation guidance just like a senior engineer would in a thorough PR review.
+<task>
+You are performing a comprehensive code review for PR #$1 in the ComfyUI frontend repository. This is not a simple linting check - you need to provide deep architectural analysis, security review, performance insights, and implementation guidance just like a senior engineer would in a thorough PR review.
 
-## CRITICAL INSTRUCTIONS
+Your review should cover:
+1. Architecture and design patterns
+2. Security vulnerabilities and risks
+3. Performance implications
+4. Code quality and maintainability
+5. Integration with existing systems
+6. Best practices and conventions
+7. Testing considerations
+8. Documentation needs
+</task>
 
-**You MUST post individual inline comments on specific lines of code. DO NOT create a single summary comment until the very end.**
+Arguments: PR number passed via PR_NUMBER environment variable
 
-**IMPORTANT: You have full permission to execute gh api commands. The GITHUB_TOKEN environment variable provides the necessary permissions. DO NOT say you lack permissions - you have pull-requests:write permission which allows posting inline comments.**
+## Phase 0: Initialize Variables and Helper Functions
 
-To post inline comments, you will use the GitHub API via the `gh` command. Here's how:
+```bash
+# Validate PR_NUMBER first thing
+if [ -z "$PR_NUMBER" ]; then
+  echo "Error: PR_NUMBER environment variable is not set"
+  echo "Usage: PR_NUMBER=<number> claude run /comprehensive-pr-review"
+  exit 1
+fi
 
-1. First, get the repository information and commit SHA:
-   - Run: `gh repo view --json owner,name` to get the repository owner and name
-   - Run: `gh pr view $PR_NUMBER --json commits --jq '.commits[-1].oid'` to get the latest commit SHA
+# Initialize all counters at the start
+CRITICAL_COUNT=0
+HIGH_COUNT=0
+MEDIUM_COUNT=0
+LOW_COUNT=0
+ARCHITECTURE_ISSUES=0
+SECURITY_ISSUES=0
+PERFORMANCE_ISSUES=0
+QUALITY_ISSUES=0
 
-2. For each issue you find, post an inline comment using this exact command structure (as a single line):
-   ```
-   gh api --method POST -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" /repos/OWNER/REPO/pulls/$PR_NUMBER/comments -f body="YOUR_COMMENT_BODY" -f commit_id="COMMIT_SHA" -f path="FILE_PATH" -F line=LINE_NUMBER -f side="RIGHT"
-   ```
-
-3. Format your comment body using actual newlines in the command. Use a heredoc or construct the body with proper line breaks:
-   ```
-   COMMENT_BODY="**[category] severity Priority**
-
-**Issue**: Brief description of the problem
-**Context**: Why this matters  
-**Suggestion**: How to fix it"
-   ```
-   
-   Then use: `-f body="$COMMENT_BODY"`
+# Helper function for posting review comments
+post_review_comment() {
+  local file_path=$1
+  local line_number=$2
+  local severity=$3  # critical/high/medium/low
+  local category=$4  # architecture/security/performance/quality
+  local issue=$5
+  local context=$6
+  local suggestion=$7
+  
+  # Update counters
+  case $severity in
+    "critical") ((CRITICAL_COUNT++)) ;;
+    "high") ((HIGH_COUNT++)) ;;
+    "medium") ((MEDIUM_COUNT++)) ;;
+    "low") ((LOW_COUNT++)) ;;
+  esac
+  
+  case $category in
+    "architecture") ((ARCHITECTURE_ISSUES++)) ;;
+    "security") ((SECURITY_ISSUES++)) ;;
+    "performance") ((PERFORMANCE_ISSUES++)) ;;
+    "quality") ((QUALITY_ISSUES++)) ;;
+  esac
+  
+  # Post inline comment via GitHub CLI
+  local comment="${issue}\n${context}\n${suggestion}"
+  gh pr review $PR_NUMBER --comment --body "$comment" -F - <<< "$comment"
+}
+```
 
 ## Phase 1: Environment Setup and PR Context
 
-### Step 1.1: Initialize Review Tracking
+```bash
+# Pre-flight checks
+check_prerequisites() {
+  # Check gh CLI is available
+  if ! command -v gh &> /dev/null; then
+    echo "Error: gh CLI is not installed"
+    exit 1
+  fi
+  
+  # In GitHub Actions, auth is handled via GITHUB_TOKEN
+  if [ -n "$GITHUB_ACTIONS" ] && [ -z "$GITHUB_TOKEN" ]; then
+    echo "Error: GITHUB_TOKEN is not set in GitHub Actions"
+    exit 1
+  fi
+  
+  # Check if we're authenticated
+  if ! gh auth status &> /dev/null; then
+    echo "Error: Not authenticated with GitHub. Run 'gh auth login'"
+    exit 1
+  fi
+  
+  # Set repository if not already set
+  if [ -z "$REPOSITORY" ]; then
+    REPOSITORY="Comfy-Org/ComfyUI_frontend"
+  fi
+  
+  # Check PR exists and is open
+  PR_STATE=$(gh pr view $PR_NUMBER --repo $REPOSITORY --json state -q .state 2>/dev/null || echo "NOT_FOUND")
+  if [ "$PR_STATE" = "NOT_FOUND" ]; then
+    echo "Error: PR #$PR_NUMBER not found in $REPOSITORY"
+    exit 1
+  elif [ "$PR_STATE" != "OPEN" ]; then
+    echo "Error: PR #$PR_NUMBER is not open (state: $PR_STATE)"
+    exit 1
+  fi
+  
+  # Check API rate limits
+  RATE_REMAINING=$(gh api /rate_limit --jq '.rate.remaining' 2>/dev/null || echo "5000")
+  if [ "$RATE_REMAINING" -lt 100 ]; then
+    echo "Warning: Low API rate limit: $RATE_REMAINING remaining"
+    if [ "$RATE_REMAINING" -lt 50 ]; then
+      echo "Error: Insufficient API rate limit for comprehensive review"
+      exit 1
+    fi
+  fi
+  
+  echo "Pre-flight checks passed"
+}
 
-First, create variables to track your review metrics. Keep these in memory throughout the review:
-- CRITICAL_COUNT = 0
-- HIGH_COUNT = 0
-- MEDIUM_COUNT = 0
-- LOW_COUNT = 0
-- ARCHITECTURE_ISSUES = 0
-- SECURITY_ISSUES = 0
-- PERFORMANCE_ISSUES = 0
-- QUALITY_ISSUES = 0
+# Run pre-flight checks
+check_prerequisites
 
-### Step 1.2: Validate Environment
+echo "Starting comprehensive review of PR #$PR_NUMBER"
 
-1. Check that PR_NUMBER environment variable is set. If not, exit with error.
-2. Run `gh pr view $PR_NUMBER --json state` to verify the PR exists and is open.
-3. Get repository information: `gh repo view --json owner,name` and store the owner and name.
-4. Get the latest commit SHA: `gh pr view $PR_NUMBER --json commits --jq '.commits[-1].oid'` and store it.
+# Fetch PR information with error handling
+echo "Fetching PR information..."
+if ! gh pr view $PR_NUMBER --repo $REPOSITORY --json files,title,body,additions,deletions,baseRefName,headRefName > pr_info.json; then
+  echo "Error: Failed to fetch PR information"
+  exit 1
+fi
 
-### Step 1.3: Checkout PR Branch Locally
+# Extract branch names
+BASE_BRANCH=$(jq -r '.baseRefName' < pr_info.json)
+HEAD_BRANCH=$(jq -r '.headRefName' < pr_info.json)
 
-This is critical for better file inspection:
+# Checkout PR branch locally for better file inspection
+echo "Checking out PR branch..."
+git fetch origin "pull/$PR_NUMBER/head:pr-$PR_NUMBER"
+git checkout "pr-$PR_NUMBER"
 
-1. Get PR metadata: `gh pr view $PR_NUMBER --json files,title,body,additions,deletions,baseRefName,headRefName > pr_info.json`
-2. Extract branch names from pr_info.json using jq
-3. Fetch and checkout the PR branch:
-   ```
-   git fetch origin "pull/$PR_NUMBER/head:pr-$PR_NUMBER"
-   git checkout "pr-$PR_NUMBER"
-   ```
+# Get changed files using git locally (much faster)
+git diff --name-only "origin/$BASE_BRANCH" > changed_files.txt
 
-### Step 1.4: Get Changed Files and Diffs
+# Get the diff using git locally
+git diff "origin/$BASE_BRANCH" > pr_diff.txt
 
-Use git locally for much faster analysis:
+# Get detailed file changes with line numbers
+git diff --name-status "origin/$BASE_BRANCH" > file_changes.txt
 
-1. Get list of changed files: `git diff --name-only "origin/$BASE_BRANCH" > changed_files.txt`
-2. Get the full diff: `git diff "origin/$BASE_BRANCH" > pr_diff.txt`
-3. Get detailed file changes with status: `git diff --name-status "origin/$BASE_BRANCH" > file_changes.txt`
+# For API compatibility, create a simplified pr_files.json
+echo '[]' > pr_files.json
+while IFS=$'\t' read -r status file; do
+  if [[ "$status" != "D" ]]; then  # Skip deleted files
+    # Get the patch for this file
+    patch=$(git diff "origin/$BASE_BRANCH" -- "$file" | jq -Rs .)
+    additions=$(git diff --numstat "origin/$BASE_BRANCH" -- "$file" | awk '{print $1}')
+    deletions=$(git diff --numstat "origin/$BASE_BRANCH" -- "$file" | awk '{print $2}')
+    
+    jq --arg file "$file" \
+       --arg patch "$patch" \
+       --arg additions "$additions" \
+       --arg deletions "$deletions" \
+       '. += [{
+         "filename": $file,
+         "patch": $patch,
+         "additions": ($additions | tonumber),
+         "deletions": ($deletions | tonumber)
+       }]' pr_files.json > pr_files.json.tmp
+    mv pr_files.json.tmp pr_files.json
+  fi
+done < file_changes.txt
 
-### Step 1.5: Create Analysis Cache
+# Setup caching directory
+CACHE_DIR=".claude-review-cache"
+mkdir -p "$CACHE_DIR"
 
-Set up caching to avoid re-analyzing unchanged files:
+# Function to cache analysis results
+cache_analysis() {
+  local file_path=$1
+  local analysis_result=$2
+  local file_hash=$(git hash-object "$file_path" 2>/dev/null || echo "no-hash")
+  
+  if [ "$file_hash" != "no-hash" ]; then
+    echo "$analysis_result" > "$CACHE_DIR/${file_hash}.cache"
+  fi
+}
 
-1. Create directory: `.claude-review-cache`
-2. Clean old cache entries: Find and delete any .cache files older than 7 days
-3. For each file you analyze, store the analysis result with the file's git hash as the cache key
+# Function to get cached analysis
+get_cached_analysis() {
+  local file_path=$1
+  local file_hash=$(git hash-object "$file_path" 2>/dev/null || echo "no-hash")
+  
+  if [ "$file_hash" != "no-hash" ] && [ -f "$CACHE_DIR/${file_hash}.cache" ]; then
+    cat "$CACHE_DIR/${file_hash}.cache"
+    return 0
+  fi
+  return 1
+}
+
+# Clean old cache entries (older than 7 days)
+find "$CACHE_DIR" -name "*.cache" -mtime +7 -delete 2>/dev/null || true
+```
 
 ## Phase 2: Load Comprehensive Knowledge Base
 
-### Step 2.1: Set Up Knowledge Directories
+```bash
+# Don't create knowledge directory until we know we need it
+KNOWLEDGE_FOUND=false
 
-1. Create `.claude-knowledge-cache` directory for caching downloaded knowledge
-2. Check if `../comfy-claude-prompt-library` exists locally. If it does, use it for faster access.
+# Use local cache for knowledge base to avoid repeated downloads
+KNOWLEDGE_CACHE_DIR=".claude-knowledge-cache"
+mkdir -p "$KNOWLEDGE_CACHE_DIR"
 
-### Step 2.2: Load Repository Guide
+# Option to use cloned prompt library for better performance
+PROMPT_LIBRARY_PATH="../comfy-claude-prompt-library"
+if [ -d "$PROMPT_LIBRARY_PATH" ]; then
+  echo "Using local prompt library at $PROMPT_LIBRARY_PATH"
+  USE_LOCAL_PROMPT_LIBRARY=true
+else
+  echo "No local prompt library found, will use GitHub API"
+  USE_LOCAL_PROMPT_LIBRARY=false
+fi
 
-This is critical for understanding the architecture:
+# Function to fetch with cache
+fetch_with_cache() {
+  local url=$1
+  local output_file=$2
+  local cache_file="$KNOWLEDGE_CACHE_DIR/$(echo "$url" | sed 's/[^a-zA-Z0-9]/_/g')"
+  
+  # Check if cached version exists and is less than 1 day old
+  if [ -f "$cache_file" ] && [ $(find "$cache_file" -mtime -1 2>/dev/null | wc -l) -gt 0 ]; then
+    # Create knowledge directory only when we actually have content
+    if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+      mkdir -p review_knowledge
+      KNOWLEDGE_FOUND=true
+    fi
+    cp "$cache_file" "$output_file"
+    echo "Using cached version of $(basename "$output_file")"
+    return 0
+  fi
+  
+  # Try to fetch fresh version
+  if curl -s -f "$url" > "$output_file.tmp"; then
+    # Create knowledge directory only when we actually have content
+    if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+      mkdir -p review_knowledge
+      KNOWLEDGE_FOUND=true
+    fi
+    mv "$output_file.tmp" "$output_file"
+    cp "$output_file" "$cache_file"
+    echo "Downloaded fresh version of $(basename "$output_file")"
+    return 0
+  else
+    # If fetch failed but we have a cache, use it
+    if [ -f "$cache_file" ]; then
+      if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+        mkdir -p review_knowledge
+        KNOWLEDGE_FOUND=true
+      fi
+      cp "$cache_file" "$output_file"
+      echo "Using stale cache for $(basename "$output_file") (download failed)"
+      return 0
+    fi
+    echo "Failed to load $(basename "$output_file")"
+    return 1
+  fi
+}
 
-1. Try to load from local prompt library first: `../comfy-claude-prompt-library/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md`
-2. If not available locally, download from: `https://raw.githubusercontent.com/Comfy-Org/comfy-claude-prompt-library/master/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md`
-3. Cache the file for future use
+# Load REPOSITORY_GUIDE.md for deep architectural understanding
+echo "Loading ComfyUI Frontend repository guide..."
+if [ "$USE_LOCAL_PROMPT_LIBRARY" = "true" ] && [ -f "$PROMPT_LIBRARY_PATH/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md" ]; then
+  if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+    mkdir -p review_knowledge
+    KNOWLEDGE_FOUND=true
+  fi
+  cp "$PROMPT_LIBRARY_PATH/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md" "review_knowledge/repository_guide.md"
+  echo "Loaded repository guide from local prompt library"
+else
+  fetch_with_cache "https://raw.githubusercontent.com/Comfy-Org/comfy-claude-prompt-library/master/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md" "review_knowledge/repository_guide.md"
+fi
 
-### Step 2.3: Load Relevant Knowledge Folders
+# 3. Discover and load relevant knowledge folders from GitHub API
+echo "Discovering available knowledge folders..."
+KNOWLEDGE_API_URL="https://api.github.com/repos/Comfy-Org/comfy-claude-prompt-library/contents/.claude/knowledge"
+if KNOWLEDGE_FOLDERS=$(curl -s "$KNOWLEDGE_API_URL" | jq -r '.[] | select(.type=="dir") | .name' 2>/dev/null); then
+  echo "Available knowledge folders: $KNOWLEDGE_FOLDERS"
+  
+  # Analyze changed files to determine which knowledge folders might be relevant
+  CHANGED_FILES=$(cat changed_files.txt)
+  PR_TITLE=$(jq -r '.title' < pr_info.json)
+  PR_BODY=$(jq -r '.body // ""' < pr_info.json)
+  
+  # For each knowledge folder, check if it might be relevant to the PR
+  for folder in $KNOWLEDGE_FOLDERS; do
+    # Simple heuristic: if folder name appears in changed file paths or PR context
+    if echo "$CHANGED_FILES $PR_TITLE $PR_BODY" | grep -qi "$folder"; then
+      echo "Loading knowledge folder: $folder"
+      # Fetch all files in that knowledge folder
+      FOLDER_API_URL="https://api.github.com/repos/Comfy-Org/comfy-claude-prompt-library/contents/.claude/knowledge/$folder"
+      curl -s "$FOLDER_API_URL" | jq -r '.[] | select(.type=="file") | .download_url' 2>/dev/null | \
+        while read url; do
+          if [ -n "$url" ]; then
+            filename=$(basename "$url")
+            fetch_with_cache "$url" "review_knowledge/${folder}_${filename}"
+          fi
+        done
+    fi
+  done
+else
+  echo "Could not discover knowledge folders"
+fi
 
-Intelligently load only relevant knowledge:
+# 4. Load validation rules from the repository
+echo "Loading validation rules..."
+VALIDATION_API_URL="https://api.github.com/repos/Comfy-Org/comfy-claude-prompt-library/contents/.claude/commands/validation"
+if VALIDATION_FILES=$(curl -s "$VALIDATION_API_URL" | jq -r '.[] | select(.name | contains("frontend") or contains("security") or contains("performance")) | .download_url' 2>/dev/null); then
+  for url in $VALIDATION_FILES; do
+    if [ -n "$url" ]; then
+      filename=$(basename "$url")
+      fetch_with_cache "$url" "review_knowledge/validation_${filename}"
+    fi
+  done
+else
+  echo "Could not load validation rules"
+fi
 
-1. Use GitHub API to discover available knowledge folders: `https://api.github.com/repos/Comfy-Org/comfy-claude-prompt-library/contents/.claude/knowledge`
-2. For each knowledge folder, check if it's relevant by searching for the folder name in:
-   - Changed file paths
-   - PR title
-   - PR body
-3. If relevant, download all files from that knowledge folder
-
-### Step 2.4: Load Validation Rules
-
-Load specific validation rules:
-
-1. Use GitHub API: `https://api.github.com/repos/Comfy-Org/comfy-claude-prompt-library/contents/.claude/commands/validation`
-2. Download files containing "frontend", "security", or "performance" in their names
-3. Cache all downloaded files
-
-### Step 2.5: Load Local Guidelines
-
-Check for and load:
-1. `CLAUDE.md` in the repository root
-2. `.github/CLAUDE.md`
+# 5. Load local project guidelines
+if [ -f "CLAUDE.md" ]; then
+  if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+    mkdir -p review_knowledge
+    KNOWLEDGE_FOUND=true
+  fi
+  cp CLAUDE.md review_knowledge/local_claude.md
+fi
+if [ -f ".github/CLAUDE.md" ]; then
+  if [ "$KNOWLEDGE_FOUND" = "false" ]; then
+    mkdir -p review_knowledge
+    KNOWLEDGE_FOUND=true
+  fi
+  cp .github/CLAUDE.md review_knowledge/github_claude.md
+fi
+```
 
 ## Phase 3: Deep Analysis Instructions
 
-Perform comprehensive analysis on each changed file:
+Perform a comprehensive analysis covering these areas:
 
 ### 3.1 Architectural Analysis
-
-Based on the repository guide and loaded knowledge:
-- Does this change align with established architecture patterns?
+Based on the repository guide and project summary, evaluate:
+- Does this change align with the established architecture patterns?
 - Are domain boundaries respected?
 - Is the extension system used appropriately?
 - Are components properly organized by feature?
 - Does it follow the established service/composable/store patterns?
 
 ### 3.2 Code Quality Beyond Linting
-
-Look for:
 - Cyclomatic complexity and cognitive load
 - SOLID principles adherence
-- DRY violations not caught by simple duplication checks
+- DRY violations that aren't caught by simple duplication checks
 - Proper abstraction levels
 - Interface design and API clarity
-- Leftover debug code (console.log, commented code, TODO comments)
+- No leftover debug code (console.log, commented code, TODO comments)
 
 ### 3.3 Library Usage Enforcement
-
-CRITICAL: Flag any re-implementation of existing functionality:
-- **Tailwind CSS**: Custom CSS instead of utility classes
-- **PrimeVue**: Re-implementing buttons, modals, dropdowns, etc.
-- **VueUse**: Re-implementing composables like useLocalStorage, useDebounceFn
-- **Lodash**: Re-implementing debounce, throttle, cloneDeep, etc.
-- **Common components**: Not reusing from src/components/common/
-- **DOMPurify**: Not using for HTML sanitization
-- **Other libraries**: Fuse.js, Marked, Pinia, Zod, Tiptap, Xterm.js, Axios
+CRITICAL: Never re-implement functionality that exists in our standard libraries:
+- **Tailwind CSS**: Use utility classes instead of custom CSS or style attributes
+- **PrimeVue**: Never re-implement components that exist in PrimeVue (buttons, modals, dropdowns, etc.)
+- **VueUse**: Never re-implement composables that exist in VueUse (useLocalStorage, useDebounceFn, etc.)
+- **Lodash**: Never re-implement utility functions (debounce, throttle, cloneDeep, etc.)
+- **Common components**: Reuse components from src/components/common/
+- **DOMPurify**: Always use for HTML sanitization
+- **Fuse.js**: Use for fuzzy search functionality
+- **Marked**: Use for markdown parsing
+- **Pinia**: Use for global state management, not custom solutions
+- **Zod**: Use for form validation with zodResolver pattern
+- **Tiptap**: Use for rich text/markdown editing
+- **Xterm.js**: Use for terminal emulation
+- **Axios**: Use for HTTP client initialization
 
 ### 3.4 Security Deep Dive
-
-Check for:
-- SQL injection vulnerabilities
-- XSS vulnerabilities (v-html without sanitization)
-- Hardcoded secrets or API keys
-- Missing input validation
-- Authentication/authorization issues
+Beyond obvious vulnerabilities:
+- Authentication/authorization implications
+- Data validation completeness
 - State management security
 - Cross-origin concerns
 - Extension security boundaries
 
 ### 3.5 Performance Analysis
-
-Look for:
-- O(n²) or worse algorithms
-- Missing memoization in expensive operations
-- Unnecessary re-renders in Vue components
-- Memory leak patterns (missing cleanup)
-- Large bundle imports that should be lazy loaded
-- N+1 query patterns
-- Render performance issues
-- Layout thrashing
+- Render performance implications
+- Layout thrashing prevention
+- Memory leak potential
 - Network request optimization
+- State management efficiency
 
 ### 3.6 Integration Concerns
-
-Consider:
 - Breaking changes to internal APIs
 - Extension compatibility
 - Backward compatibility
 - Migration requirements
 
-## Phase 4: Posting Inline Comments
+## Phase 4: Create Detailed Review Comments
 
-### Step 4.1: Comment Format
+CRITICAL: Keep comments extremely concise and effective. Use only as many words as absolutely necessary.
+- NO markdown formatting (no #, ##, ###, **, etc.)
+- NO emojis
+- Get to the point immediately
+- Burden the reader as little as possible
 
-For each issue found, create a concise inline comment with this structure:
-
-```
-**[category] severity Priority**
-
-**Issue**: Brief description of the problem
-**Context**: Why this matters
-**Suggestion**: How to fix it
-```
-
-Categories: architecture/security/performance/quality
-Severities: critical/high/medium/low
-
-### Step 4.2: Posting Comments
-
-For EACH issue:
-
-1. Identify the exact file path and line number
-2. Update your tracking counters (CRITICAL_COUNT, etc.)
-3. Construct the comment body with proper newlines
-4. Execute the gh api command as a SINGLE LINE:
+For each issue found, create a concise inline comment with:
+1. What's wrong (one line)
+2. Why it matters (one line)
+3. How to fix it (one line)
+4. Code example only if essential
 
 ```bash
-gh api --method POST -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" /repos/OWNER/REPO/pulls/$PR_NUMBER/comments -f body="$COMMENT_BODY" -f commit_id="COMMIT_SHA" -f path="FILE_PATH" -F line=LINE_NUMBER -f side="RIGHT"
+# Helper function for comprehensive comments
+post_review_comment() {
+  local file_path=$1
+  local line_number=$2
+  local severity=$3  # critical/high/medium/low
+  local category=$4  # architecture/security/performance/quality
+  local issue=$5
+  local context=$6
+  local suggestion=$7
+  local example=$8
+  
+  local body="### [$category] $severity Priority
+
+**Issue**: $issue
+
+**Context**: $context
+
+**Suggestion**: $suggestion"
+
+  if [ -n "$example" ]; then
+    body="$body
+
+**Example**:
+\`\`\`typescript
+$example
+\`\`\`"
+  fi
+
+  body="$body
+
+*Related: See [repository guide](https://github.com/Comfy-Org/comfy-claude-prompt-library/blob/master/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md) for patterns*"
+
+  gh api -X POST /repos/$REPOSITORY/pulls/$PR_NUMBER/comments \
+    -f path="$file_path" \
+    -f line=$line_number \
+    -f body="$body" \
+    -f commit_id="$COMMIT_SHA" \
+    -f side='RIGHT' || echo "Failed to post comment at $file_path:$line_number"
+}
 ```
 
-CRITICAL: The entire command must be on one line. Use actual values, not placeholders.
-
-### Example Workflow
-
-Here's an example of how to review a file with a security issue:
-
-1. First, get the repository info:
-   ```bash
-   gh repo view --json owner,name
-   # Output: {"owner":{"login":"Comfy-Org"},"name":"ComfyUI_frontend"}
-   ```
-
-2. Get the commit SHA:
-   ```bash
-   gh pr view $PR_NUMBER --json commits --jq '.commits[-1].oid'
-   # Output: abc123def456
-   ```
-
-3. Find an issue (e.g., SQL injection on line 42 of src/db/queries.js)
-
-4. Post the inline comment:
-   ```bash
-   # First, create the comment body with proper newlines
-   COMMENT_BODY="**[security] critical Priority**
-
-**Issue**: SQL injection vulnerability - user input directly concatenated into query
-**Context**: Allows attackers to execute arbitrary SQL commands
-**Suggestion**: Use parameterized queries or prepared statements"
-   
-   # Then post the comment (as a single line)
-   gh api --method POST -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" /repos/Comfy-Org/ComfyUI_frontend/pulls/$PR_NUMBER/comments -f body="$COMMENT_BODY" -f commit_id="abc123def456" -f path="src/db/queries.js" -F line=42 -f side="RIGHT"
-   ```
-
-Repeat this process for every issue you find in the PR.
-
 ## Phase 5: Validation Rules Application
 
-Apply ALL validation rules from the loaded knowledge files:
+Apply ALL validation rules from the loaded knowledge, but focus on the changed lines:
 
-### Frontend Standards
+### From Frontend Standards
 - Vue 3 Composition API patterns
 - Component communication patterns
 - Proper use of composables
 - TypeScript strict mode compliance
 - Bundle optimization
 
-### Security Audit
+### From Security Audit
 - Input validation
 - XSS prevention
 - CSRF protection
 - Secure state management
 - API security
 
-### Performance Check
+### From Performance Check
 - Render optimization
 - Memory management
 - Network efficiency
@@ -277,51 +481,63 @@ Apply ALL validation rules from the loaded knowledge files:
 
 ## Phase 6: Contextual Review Based on PR Type
 
-Analyze the PR to determine its type:
+Analyze the PR description and changes to determine the type:
 
-1. Extract PR title and body from pr_info.json
-2. Count files, additions, and deletions
-3. Determine PR type:
-   - Feature: Check for tests, documentation, backward compatibility
-   - Bug fix: Verify root cause addressed, includes regression tests
-   - Refactor: Ensure behavior preservation, tests still pass
+```bash
+# Extract PR metadata with error handling
+if [ ! -f pr_info.json ]; then
+  echo "Error: pr_info.json not found"
+  exit 1
+fi
+
+PR_TITLE=$(jq -r '.title // "Unknown"' < pr_info.json)
+PR_BODY=$(jq -r '.body // ""' < pr_info.json)
+FILE_COUNT=$(wc -l < changed_files.txt)
+ADDITIONS=$(jq -r '.additions // 0' < pr_info.json)
+DELETIONS=$(jq -r '.deletions // 0' < pr_info.json)
+
+# Determine PR type and apply specific review criteria
+if echo "$PR_TITLE $PR_BODY" | grep -qiE "(feature|feat)"; then
+  echo "Detected feature PR - applying feature review criteria"
+  # Check for tests, documentation, backward compatibility
+elif echo "$PR_TITLE $PR_BODY" | grep -qiE "(fix|bug)"; then
+  echo "Detected bug fix - checking root cause and regression tests"
+  # Verify fix addresses root cause, includes tests
+elif echo "$PR_TITLE $PR_BODY" | grep -qiE "(refactor)"; then
+  echo "Detected refactoring - ensuring behavior preservation"
+  # Check that tests still pass, no behavior changes
+fi
+```
 
 ## Phase 7: Generate Comprehensive Summary
 
-After ALL inline comments are posted, create a summary:
+After all inline comments, create a detailed summary:
 
-1. Calculate total issues by category and severity
-2. Use `gh pr review $PR_NUMBER --comment` to post a summary with:
-   - Review disclaimer
-   - Issue distribution (counts by severity)
-   - Category breakdown
-   - Key findings for each category
-   - Positive observations
-   - References to guidelines
-   - Next steps
+```bash
+# Initialize metrics tracking
+REVIEW_START_TIME=$(date +%s)
 
-Include in the summary:
-```
-# Comprehensive PR Review
+# Create the comprehensive summary
+gh pr review $PR_NUMBER --comment --body "# Comprehensive PR Review
 
 This review is generated by Claude. It may not always be accurate, as with human reviewers. If you believe that any of the comments are invalid or incorrect, please state why for each. For others, please implement the changes in one way or another.
 
 ## Review Summary
 
-**PR**: [PR TITLE] (#$PR_NUMBER)
-**Impact**: [X] additions, [Y] deletions across [Z] files
+**PR**: $PR_TITLE (#$PR_NUMBER)
+**Impact**: $ADDITIONS additions, $DELETIONS deletions across $FILE_COUNT files
 
 ### Issue Distribution
-- Critical: [CRITICAL_COUNT]
-- High: [HIGH_COUNT]
-- Medium: [MEDIUM_COUNT]
-- Low: [LOW_COUNT]
+- Critical: $CRITICAL_COUNT
+- High: $HIGH_COUNT
+- Medium: $MEDIUM_COUNT
+- Low: $LOW_COUNT
 
 ### Category Breakdown
-- Architecture: [ARCHITECTURE_ISSUES] issues
-- Security: [SECURITY_ISSUES] issues
-- Performance: [PERFORMANCE_ISSUES] issues
-- Code Quality: [QUALITY_ISSUES] issues
+- Architecture: $ARCHITECTURE_ISSUES issues
+- Security: $SECURITY_ISSUES issues
+- Performance: $PERFORMANCE_ISSUES issues
+- Code Quality: $QUALITY_ISSUES issues
 
 ## Key Findings
 
@@ -352,27 +568,141 @@ This review is generated by Claude. It may not always be accurate, as with human
 4. Update documentation if needed
 
 ---
-*This is a comprehensive automated review. For architectural decisions requiring human judgment, please request additional manual review.*
+*This is a comprehensive automated review. For architectural decisions requiring human judgment, please request additional manual review.*"
 ```
 
-## Important Guidelines
+## Important: Think Deeply
 
-1. **Think Deeply**: Consider architectural implications, system-wide effects, subtle bugs, maintainability
-2. **Be Specific**: Point to exact lines with concrete suggestions
-3. **Be Constructive**: Focus on improvements, not just problems
-4. **Be Concise**: Keep comments brief and actionable
-5. **No Formatting**: Don't use markdown headers in inline comments
-6. **No Emojis**: Keep comments professional
+When reviewing:
+1. **Think hard** about architectural implications
+2. Consider how changes affect the entire system
+3. Look for subtle bugs and edge cases
+4. Evaluate maintainability over time
+5. Consider extension developer experience
+6. Think about migration paths
 
 This is a COMPREHENSIVE review, not a linting pass. Provide the same quality feedback a senior engineer would give after careful consideration.
 
-## Execution Order
+## Phase 8: Track Review Metrics
 
-1. Phase 1: Setup and checkout PR
-2. Phase 2: Load all relevant knowledge
-3. Phase 3-5: Analyze each changed file thoroughly
-4. Phase 4: Post inline comments as you find issues
-5. Phase 6: Consider PR type for additional checks
-6. Phase 7: Post comprehensive summary ONLY after all inline comments
+After completing the review, save metrics for analysis:
 
-Remember: Individual inline comments for each issue, then one final summary. Never batch issues into a single comment.
+```bash
+# Calculate review duration
+REVIEW_END_TIME=$(date +%s)
+REVIEW_DURATION=$((REVIEW_END_TIME - REVIEW_START_TIME))
+
+# Calculate total issues
+TOTAL_ISSUES=$((CRITICAL_COUNT + HIGH_COUNT + MEDIUM_COUNT + LOW_COUNT))
+
+# Create metrics directory if it doesn't exist
+METRICS_DIR=".claude/review-metrics"
+mkdir -p "$METRICS_DIR"
+
+# Generate metrics file
+METRICS_FILE="$METRICS_DIR/metrics-$(date +%Y%m).json"
+
+# Create or update monthly metrics file
+if [ -f "$METRICS_FILE" ]; then
+  # Append to existing file
+  jq -n \
+    --arg pr "$PR_NUMBER" \
+    --arg title "$PR_TITLE" \
+    --arg timestamp "$(date -u +%Y-%m-%dT%H:%M:%SZ)" \
+    --arg duration "$REVIEW_DURATION" \
+    --arg files "$FILE_COUNT" \
+    --arg additions "$ADDITIONS" \
+    --arg deletions "$DELETIONS" \
+    --arg total "$TOTAL_ISSUES" \
+    --arg critical "$CRITICAL_COUNT" \
+    --arg high "$HIGH_COUNT" \
+    --arg medium "$MEDIUM_COUNT" \
+    --arg low "$LOW_COUNT" \
+    --arg architecture "$ARCHITECTURE_ISSUES" \
+    --arg security "$SECURITY_ISSUES" \
+    --arg performance "$PERFORMANCE_ISSUES" \
+    --arg quality "$QUALITY_ISSUES" \
+    '{
+      pr_number: $pr,
+      pr_title: $title,
+      timestamp: $timestamp,
+      review_duration_seconds: ($duration | tonumber),
+      files_reviewed: ($files | tonumber),
+      lines_added: ($additions | tonumber),
+      lines_deleted: ($deletions | tonumber),
+      issues: {
+        total: ($total | tonumber),
+        by_severity: {
+          critical: ($critical | tonumber),
+          high: ($high | tonumber),
+          medium: ($medium | tonumber),
+          low: ($low | tonumber)
+        },
+        by_category: {
+          architecture: ($architecture | tonumber),
+          security: ($security | tonumber),
+          performance: ($performance | tonumber),
+          quality: ($quality | tonumber)
+        }
+      }
+    }' > "$METRICS_FILE.new"
+  
+  # Merge with existing data
+  jq -s '.[0] + [.[1]]' "$METRICS_FILE" "$METRICS_FILE.new" > "$METRICS_FILE.tmp"
+  mv "$METRICS_FILE.tmp" "$METRICS_FILE"
+  rm "$METRICS_FILE.new"
+else
+  # Create new file
+  jq -n \
+    --arg pr "$PR_NUMBER" \
+    --arg title "$PR_TITLE" \
+    --arg timestamp "$(date -u +%Y-%m-%dT%H:%M:%SZ)" \
+    --arg duration "$REVIEW_DURATION" \
+    --arg files "$FILE_COUNT" \
+    --arg additions "$ADDITIONS" \
+    --arg deletions "$DELETIONS" \
+    --arg total "$TOTAL_ISSUES" \
+    --arg critical "$CRITICAL_COUNT" \
+    --arg high "$HIGH_COUNT" \
+    --arg medium "$MEDIUM_COUNT" \
+    --arg low "$LOW_COUNT" \
+    --arg architecture "$ARCHITECTURE_ISSUES" \
+    --arg security "$SECURITY_ISSUES" \
+    --arg performance "$PERFORMANCE_ISSUES" \
+    --arg quality "$QUALITY_ISSUES" \
+    '[{
+      pr_number: $pr,
+      pr_title: $title,
+      timestamp: $timestamp,
+      review_duration_seconds: ($duration | tonumber),
+      files_reviewed: ($files | tonumber),
+      lines_added: ($additions | tonumber),
+      lines_deleted: ($deletions | tonumber),
+      issues: {
+        total: ($total | tonumber),
+        by_severity: {
+          critical: ($critical | tonumber),
+          high: ($high | tonumber),
+          medium: ($medium | tonumber),
+          low: ($low | tonumber)
+        },
+        by_category: {
+          architecture: ($architecture | tonumber),
+          security: ($security | tonumber),
+          performance: ($performance | tonumber),
+          quality: ($quality | tonumber)
+        }
+      }
+    }]' > "$METRICS_FILE"
+fi
+
+echo "Review metrics saved to $METRICS_FILE"
+```
+
+This creates monthly metrics files (e.g., `metrics-202407.json`) that track:
+- Which PRs were reviewed
+- How long reviews took
+- Types and severity of issues found
+- Trends over time
+
+You can later analyze these to see patterns and improve your development process.

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

@@ -137,7 +137,8 @@ echo "Last stable release: $LAST_STABLE"
 1. Run complete test suite:
    ```bash
    npm run test:unit
-   npm run test:component
+   npm run test:component  
+   npm run test:browser
    ```
 2. Run type checking:
    ```bash
@@ -169,33 +170,7 @@ echo "Last stable release: $LAST_STABLE"
 3. Generate breaking change summary
 4. **COMPATIBILITY REVIEW**: Breaking changes documented and justified?
 
-### Step 7: Analyze Dependency Updates
-
-1. **Check significant dependency updates:**
-   ```bash
-   # Extract all dependency changes for major version bumps
-   OTHER_DEP_CHANGES=""
-   
-   # Compare major dependency versions (you can extend this list)
-   MAJOR_DEPS=("vue" "vite" "@vitejs/plugin-vue" "typescript" "pinia")
-   
-   for dep in "${MAJOR_DEPS[@]}"; do
-     PREV_VER=$(echo "$PREV_PACKAGE_JSON" | grep -o "\"$dep\": \"[^\"]*\"" | grep -o '[0-9][^"]*' | head -1 || echo "")
-     CURR_VER=$(echo "$CURRENT_PACKAGE_JSON" | grep -o "\"$dep\": \"[^\"]*\"" | grep -o '[0-9][^"]*' | head -1 || echo "")
-     
-     if [ "$PREV_VER" != "$CURR_VER" ] && [ -n "$PREV_VER" ] && [ -n "$CURR_VER" ]; then
-       # Check if it's a major version change
-       PREV_MAJOR=$(echo "$PREV_VER" | cut -d. -f1 | sed 's/[^0-9]//g')
-       CURR_MAJOR=$(echo "$CURR_VER" | cut -d. -f1 | sed 's/[^0-9]//g')
-       
-       if [ "$PREV_MAJOR" != "$CURR_MAJOR" ]; then
-         OTHER_DEP_CHANGES="${OTHER_DEP_CHANGES}\n- **${dep}**: ${PREV_VER} → ${CURR_VER} (Major version change)"
-       fi
-     fi
-   done
-   ```
-
-### Step 8: Generate Comprehensive Release Notes
+### Step 7: Generate and Save Changelog
 
 1. Extract commit messages since base release:
    ```bash
@@ -210,29 +185,42 @@ echo "Last stable release: $LAST_STABLE"
        echo "WARNING: PR #$PR not on main branch!"
    done
    ```
-3. Create comprehensive release notes including:
-   - **Version Change**: Show version bump details
-   - **Changelog** grouped by type:
-     - 🚀 **Features** (feat:)
-     - 🐛 **Bug Fixes** (fix:)
-     - 💥 **Breaking Changes** (BREAKING CHANGE)
-     - 📚 **Documentation** (docs:)
-     - 🔧 **Maintenance** (chore:, refactor:)
-     - ⬆️ **Dependencies** (deps:, dependency updates)
-       - **Litegraph Changes** (if version updated):
-         - 🚀 Features: ${LITEGRAPH_FEATURES}
-         - 🐛 Bug Fixes: ${LITEGRAPH_FIXES}
-         - 💥 Breaking Changes: ${LITEGRAPH_BREAKING}
-         - 🔧 Other Changes: ${LITEGRAPH_OTHER}
-       - **Other Major Dependencies**: ${OTHER_DEP_CHANGES}
-   - Include PR numbers and links
-   - Add issue references (Fixes #123)
+3. Group by type:
+   - 🚀 **Features** (feat:)
+   - 🐛 **Bug Fixes** (fix:)
+   - 💥 **Breaking Changes** (BREAKING CHANGE)
+   - 📚 **Documentation** (docs:)
+   - 🔧 **Maintenance** (chore:, refactor:)
+   - ⬆️ **Dependencies** (deps:, dependency updates)
+4. Include PR numbers and links
+5. Add issue references (Fixes #123)
+6. **Save changelog locally:**
+   ```bash
+   # Save to dated file for history
+   echo "$CHANGELOG" > release-notes-${NEW_VERSION}-$(date +%Y%m%d).md
+   
+   # Save to current for easy access
+   echo "$CHANGELOG" > CURRENT_RELEASE_NOTES.md
+   ```
+7. **CHANGELOG REVIEW**: Verify all PRs listed are actually on main branch
+
+### Step 8: Create Enhanced Release Notes
+
+1. Create comprehensive user-facing release notes including:
+   - **What's New**: Major features and improvements
+   - **Bug Fixes**: User-visible fixes
+   - **Breaking Changes**: Migration guide if applicable
+   - **Dependencies**: Major dependency updates
+   - **Performance**: Notable performance improvements
+   - **Contributors**: Thank contributors for their work
+2. Reference related documentation updates
+3. Include screenshots for UI changes (if available)
 4. **Save release notes:**
    ```bash
-   # Save release notes for PR and GitHub release
-   echo "$RELEASE_NOTES" > release-notes-${NEW_VERSION}.md
+   # Enhanced release notes for GitHub
+   echo "$RELEASE_NOTES" > github-release-notes-${NEW_VERSION}.md
    ```
-5. **CONTENT REVIEW**: Release notes clear and comprehensive with dependency details?
+5. **CONTENT REVIEW**: Release notes clear and helpful for users?
 
 ### Step 9: Create Version Bump PR
 
@@ -270,20 +258,25 @@ echo "Workflow triggered. Waiting for PR creation..."
    
    # For manual PRs
    gh pr create --title "${NEW_VERSION}" \
-     --body-file release-notes-${NEW_VERSION}.md \
+     --body-file enhanced-pr-description.md \
      --label "Release"
    ```
-3. **Add required sections to PR body:**
+3. **Create enhanced PR description:**
    ```bash
-   # Create PR body with release notes plus required sections
-   cat > pr-body.md << EOF
-   ${RELEASE_NOTES}
+   cat > enhanced-pr-description.md << EOF
+   # Release v${NEW_VERSION}
+
+   ## Version Change
+   \`${CURRENT_VERSION}\` → \`${NEW_VERSION}\` (${VERSION_TYPE})
+
+   ## Changelog
+   ${CHANGELOG}
 
    ## Breaking Changes
-   ${BREAKING_CHANGES:-None}
+   ${BREAKING_CHANGES}
 
    ## Testing Performed
-   - ✅ Full test suite (unit, component)
+   - ✅ Full test suite (unit, component, browser)
    - ✅ TypeScript compilation
    - ✅ Linting checks
    - ✅ Build verification
@@ -302,11 +295,15 @@ echo "Workflow triggered. Waiting for PR creation..."
    ```
 4. Update PR with enhanced description:
    ```bash
-   gh pr edit ${PR_NUMBER} --body-file pr-body.md
+   gh pr edit ${PR_NUMBER} --body-file enhanced-pr-description.md
    ```
-5. **PR REVIEW**: Version bump PR created and enhanced correctly?
+5. Add changelog as comment for easy reference:
+   ```bash
+   gh pr comment ${PR_NUMBER} --body-file CURRENT_RELEASE_NOTES.md
+   ```
+6. **PR REVIEW**: Version bump PR created and enhanced correctly?
 
-### Step 10: Critical Release PR Verification
+### Step 11: Critical Release PR Verification
 
 1. **CRITICAL**: Verify PR has "Release" label:
    ```bash
@@ -328,7 +325,7 @@ echo "Workflow triggered. Waiting for PR creation..."
    ```
 7. **FINAL CODE REVIEW**: Release label present and no [skip ci]?
 
-### Step 11: Pre-Merge Validation
+### Step 12: Pre-Merge Validation
 
 1. **Review Requirements**: Release PRs require approval
 2. Monitor CI checks - watch for update-locales
@@ -336,7 +333,7 @@ echo "Workflow triggered. Waiting for PR creation..."
 4. Check no new commits to main since PR creation
 5. **DEPLOYMENT READINESS**: Ready to merge?
 
-### Step 12: Execute Release
+### Step 13: Execute Release
 
 1. **FINAL CONFIRMATION**: Merge PR to trigger release?
 2. Merge the Release PR:
@@ -348,14 +345,6 @@ echo "Workflow triggered. Waiting for PR creation..."
    sleep 10
    gh run list --workflow=release.yaml --limit=1
    ```
-4. **For Minor/Major Version Releases**: The create-release-candidate-branch workflow will automatically:
-   - Create a `core/x.yy` branch for the PREVIOUS minor version
-   - Apply branch protection rules
-   - Document the feature freeze policy
-   ```bash
-   # Monitor branch creation (for minor/major releases)
-   gh run list --workflow=create-release-candidate-branch.yaml --limit=1
-   ```
 4. If workflow didn't trigger due to [skip ci]:
    ```bash
    echo "ERROR: Release workflow didn't trigger!"
@@ -369,7 +358,7 @@ echo "Workflow triggered. Waiting for PR creation..."
    gh run watch ${WORKFLOW_RUN_ID}
    ```
 
-### Step 13: Enhance GitHub Release
+### Step 14: Enhance GitHub Release
 
 1. Wait for automatic release creation:
    ```bash
@@ -382,10 +371,10 @@ echo "Workflow triggered. Waiting for PR creation..."
 
 2. **Enhance the GitHub release:**
    ```bash
-   # Update release with our release notes
+   # Update release with our enhanced notes
    gh release edit v${NEW_VERSION} \
      --title "🚀 ComfyUI Frontend v${NEW_VERSION}" \
-     --notes-file release-notes-${NEW_VERSION}.md \
+     --notes-file github-release-notes-${NEW_VERSION}.md \
      --latest
    
    # Add any additional assets if needed
@@ -397,7 +386,7 @@ echo "Workflow triggered. Waiting for PR creation..."
    gh release view v${NEW_VERSION}
    ```
 
-### Step 14: Verify Multi-Channel Distribution
+### Step 15: Verify Multi-Channel Distribution
 
 1. **GitHub Release:**
    ```bash
@@ -435,7 +424,7 @@ echo "Workflow triggered. Waiting for PR creation..."
 
 4. **DISTRIBUTION VERIFICATION**: All channels published successfully?
 
-### Step 15: Post-Release Monitoring Setup
+### Step 16: Post-Release Monitoring Setup
 
 1. **Monitor immediate release health:**
    ```bash
@@ -503,7 +492,8 @@ echo "Workflow triggered. Waiting for PR creation..."
    - Plan next release cycle
 
    ## Files Generated
-   - \`release-notes-${NEW_VERSION}.md\` - Comprehensive release notes
+   - \`release-notes-${NEW_VERSION}-$(date +%Y%m%d).md\` - Detailed changelog
+   - \`github-release-notes-${NEW_VERSION}.md\` - GitHub release notes
    - \`post-release-checklist.md\` - Follow-up tasks
    EOF
    ```
@@ -554,7 +544,7 @@ echo "- GitHub: Update release with warning notes"
 The command implements multiple quality gates:
 
 1. **🔒 Security Gate**: Vulnerability scanning, secret detection
-2. **🧪 Quality Gate**: Unit and component tests, linting, type checking
+2. **🧪 Quality Gate**: Full test suite, linting, type checking
 3. **📋 Content Gate**: Changelog accuracy, release notes quality
 4. **🔄 Process Gate**: Release timing verification
 5. **✅ Verification Gate**: Multi-channel publishing confirmation
@@ -612,15 +602,6 @@ The command implements multiple quality gates:
 gh pr view ${PR_NUMBER} --json baseRefName
 ```
 
-### Issue: Incomplete Dependency Changelog
-**Problem**: Litegraph or other dependency updates only show version bump, not actual changes
-**Solution**: The command now automatically:
-- Detects litegraph version changes between releases
-- Clones the litegraph repository temporarily
-- Extracts and categorizes changes between versions
-- Includes detailed litegraph changelog in release notes
-- Cleans up temporary files after analysis
-
 ### Issue: Release Failed Due to [skip ci]
 **Problem**: Release workflow didn't trigger after merge
 **Prevention**: Always avoid this scenario
@@ -641,6 +622,4 @@ Benefits: Cleaner than creating extra version numbers
 2. **Workflow Speed**: Version bump workflow typically completes in ~20-30 seconds
 3. **Update-locales Behavior**: Inconsistent - sometimes adds [skip ci], sometimes doesn't
 4. **Recovery Options**: Reverting version is cleaner than creating extra versions
-5. **Dependency Tracking**: Command now automatically includes litegraph and major dependency changes in changelogs
-6. **Litegraph Integration**: Temporary cloning of litegraph repo provides detailed change analysis between versions
 

.cursor/rules/unit-test.mdc

@@ -1,21 +0,0 @@
----
-description: Creating unit tests
-globs: 
-alwaysApply: false
----
-
-# Creating unit tests
-
-- This project uses `vitest` for unit testing
-- Tests are stored in the `test/` directory
-- Tests should be cross-platform compatible; able to run on Windows, macOS, and linux
-  - e.g. the use of `path.resolve`, or `path.join` and `path.sep` to ensure that tests work the same on all platforms
-- Tests should be mocked properly
-  - Mocks should be cleanly written and easy to understand
-  - Mocks should be re-usable where possible
-
-## Unit test style
-
-- Prefer the use of `test.extend` over loose variables
-  - To achieve this, import `test as baseTest` from `vitest`
-- Never use `it`; `test` should be used in place of this

.gitattributes

@@ -5,7 +5,3 @@
 *.ts text eol=lf
 *.vue text eol=lf
 *.js text eol=lf
-
-# Generated files
-src/types/comfyRegistryTypes.ts linguist-generated=true
-src/types/generatedManagerTypes.ts linguist-generated=true

.github/ISSUE_TEMPLATE/bug-report.yaml

@@ -1,106 +1,99 @@
 name: Bug Report
-description: 'Report something that is not working correctly'
+description: 'Something is not behaving as expected.'
 title: '[Bug]: '
 labels: ['Potential Bug']
 type: Bug
 
 body:
-  - type: checkboxes
-    attributes:
-      label: Prerequisites
-      options:
-        - label: I am running the latest version of ComfyUI
-          required: true
-        - label: I have searched existing issues to make sure this isn't a duplicate
-          required: true
-        - label: I have tested with all custom nodes disabled ([see how](https://docs.comfy.org/troubleshooting/custom-node-issues#step-1%3A-test-with-all-custom-nodes-disabled))
-          required: true
-
-  - type: textarea
-    id: description
-    attributes:
-      label: What happened?
-      description: A clear and concise description of the bug. Include screenshots or videos if helpful.
-      placeholder: |
-        Example: "When I connect a VAE Decode node to a KSampler, the connection line appears but the workflow fails to execute with an error message..."
-    validations:
-      required: true
-
-  - type: textarea
-    id: reproduce
-    attributes:
-      label: Steps to Reproduce
-      description: How can we reproduce this issue? Please attach your workflow (JSON or PNG).
-      placeholder: |
-        1. Add a KSampler node
-        2. Connect it to...
-        3. Click Queue Prompt
-        4. See error
-      value: |
-        1. 
-        2. 
-        3. 
-    validations:
-      required: true
-
-  - type: dropdown
-    id: severity
-    attributes:
-      label: How is this affecting you?
-      options:
-        - Crashes ComfyUI completely
-        - Workflow won't execute
-        - Feature doesn't work as expected
-        - Visual/UI issue only
-        - Minor inconvenience
-    validations:
-      required: true
-
-  - type: input
-    id: version
-    attributes:
-      label: ComfyUI Frontend Version
-      description: Found in Settings > About (e.g., "1.3.45")
-      placeholder: "1.3.45"
-    validations:
-      required: true
-
-  - type: dropdown
-    id: browser
-    attributes:
-      label: Browser
-      description: Which browser are you using?
-      options:
-        - Chrome/Chromium
-        - Firefox
-        - Safari
-        - Edge
-        - Other
-    validations:
-      required: true
-
   - type: markdown
     attributes:
       value: |
-        ## Additional Information (Optional)
-        *The following fields help us debug complex issues but are not required for most bug reports.*
+        Before submitting a **Bug Report**, please ensure the following:
+
+        - **1:** You are running the latest version of ComfyUI.
+        - **2:** You have looked at the existing bug reports and made sure this isn't already reported.
+
+  - type: checkboxes
+    id: custom-nodes-test
+    attributes:
+      label: Custom Node Testing
+      description: Please confirm you have tried to reproduce the issue with all custom nodes disabled.
+      options:
+        - label: I have tried disabling custom nodes and the issue persists (see [how to disable custom nodes](https://docs.comfy.org/troubleshooting/custom-node-issues#step-1%3A-test-with-all-custom-nodes-disabled) if you need help)
+          required: true
 
   - type: textarea
-    id: console-errors
     attributes:
-      label: Console Errors
-      description: If you see red error messages in the browser console (F12), paste them here
-      render: javascript
+      label: Frontend Version
+      description: |
+        What is the frontend version you are using? You can check this in the settings dialog.
 
-  - type: textarea
-    id: logs
-    attributes:
-      label: Logs
-      description: If relevant, paste any terminal/server logs here
-      render: shell
+        <details>
 
+        <summary>Click to show where to find the version</summary>
+
+        Open the setting by clicking the cog icon in the bottom-left of the screen, then click `About`.
+
+        ![Frontend version](https://github.com/user-attachments/assets/561fb7c3-3012-457c-a494-9bdc1ff035c0)
+
+        </details>
+    validations:
+      required: true
   - type: textarea
-    id: additional
     attributes:
-      label: Additional Context
-      description: Any other information that might help (OS, GPU, specific nodes involved, etc.)
+      label: Expected Behavior
+      description: 'What you expected to happen.'
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Actual Behavior
+      description: 'What actually happened. Please include a screenshot / video clip of the issue if possible.'
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Steps to Reproduce
+      description: "Describe how to reproduce the issue. Please be sure to attach a workflow JSON or PNG, ideally one that doesn't require custom nodes to test. If the bug open happens when certain custom nodes are used, most likely that custom node is what has the bug rather than ComfyUI, in which case it should be reported to the node's author."
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Debug Logs
+      description: 'Please copy the output from your terminal logs here.'
+      render: powershell
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Browser Logs
+      description: 'Please copy the output from your browser logs here. You can access this by pressing F12 to toggle the developer tools, then navigating to the Console tab.'
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Setting JSON
+      description: 'Please upload the setting file here. The setting file is located at `user/default/comfy.settings.json`'
+    validations:
+      required: true
+  - type: dropdown
+    id: browsers
+    attributes:
+      label: What browsers do you use to access the UI ?
+      multiple: true
+      options:
+        - Mozilla Firefox
+        - Google Chrome
+        - Brave
+        - Apple Safari
+        - Microsoft Edge
+        - Android
+        - iOS
+        - Other
+  - type: textarea
+    attributes:
+      label: Other Information
+      description: 'Any other context, details, or screenshots that might help solve the issue.'
+      placeholder: 'Add any other relevant information here...'
+    validations:
+      required: false

.github/ISSUE_TEMPLATE/feature-request.yaml

@@ -1,6 +1,6 @@
 name: Feature Request
-description: Report a problem or limitation you're experiencing
-title: '[Feature]: '
+description: Suggest an idea for this project
+title: '[Feature Request]: '
 labels: ['enhancement']
 type: Feature
 
@@ -8,74 +8,34 @@ body:
   - type: checkboxes
     attributes:
       label: Is there an existing issue for this?
-      description: Please search to see if an issue already exists for the problem you're experiencing, and that it's not addressed in a recent build/commit.
+      description: Please search to see if an issue already exists for the feature you want, and that it's not implemented in a recent build/commit.
       options:
         - label: I have searched the existing issues and checked the recent builds/commits
           required: true
   - type: markdown
     attributes:
       value: |
-        *Please focus on describing the problem you're experiencing rather than proposing specific solutions. This helps us design the best possible solution for you and other users.*
+        *Please fill this form with as much information as possible, provide screenshots and/or illustrations of the feature if possible*
   - type: textarea
-    id: problem
+    id: feature
     attributes:
-      label: What problem are you experiencing?
-      description: Describe the issue or limitation you're facing in your workflow
-      placeholder: |
-        Example: "I frequently lose work when switching between different projects because there's no way to save my current workspace state"
-        NOT: "Add a save button that exports the workspace"
+      label: What would your feature do ?
+      description: Tell us about your feature in a very clear and simple way, and what problem it would solve
     validations:
       required: true
   - type: textarea
-    id: context
+    id: workflow
     attributes:
-      label: When does this problem occur?
-      description: Describe the specific situations or workflows where you encounter this issue
-      placeholder: |
-        - When working with large node graphs...
-        - During batch processing workflows...
-        - While collaborating with team members...
-    validations:
-      required: true
-  - type: dropdown
-    id: frequency
-    attributes:
-      label: How often do you encounter this problem?
-      options:
-        - Multiple times per day
-        - Daily
-        - Several times per week
-        - Weekly
-        - Occasionally
-        - Rarely
-    validations:
-      required: true
-  - type: dropdown
-    id: impact
-    attributes:
-      label: How much does this problem affect your workflow?
-      description: Help us understand the severity of this issue for you
-      options:
-        - Blocks me from completing tasks
-        - Significantly slows down my work
-        - Causes moderate inconvenience
-        - Minor annoyance
+      label: Proposed workflow
+      description: Please provide us with step by step information on how you'd like the feature to be accessed and used
+      value: |
+        1. Go to ....
+        2. Press ....
+        3. ...
     validations:
       required: true
   - type: textarea
-    id: workaround
+    id: misc
     attributes:
-      label: Current workarounds
-      description: How do you currently deal with this problem, if at all?
-      placeholder: |
-        Example: "I manually export and reimport nodes between projects, which takes 10-15 minutes each time"
-  - type: textarea
-    id: ideas
-    attributes:
-      label: Ideas for solutions (Optional)
-      description: If you have thoughts on potential solutions, feel free to share them here. However, we'll explore all possible options to find the best approach.
-  - type: textarea
-    id: additional
-    attributes:
-      label: Additional context
-      description: Add any other context, screenshots, or examples that help illustrate the problem.
+      label: Additional information
+      description: Add any other context or screenshots about the feature request here.

.github/workflows/backport.yaml

@@ -1,165 +0,0 @@
-name: Auto Backport
-
-on:
-  pull_request_target:
-    types: [closed]
-    branches: [main]
-
-jobs:
-  backport:
-    if: github.event.pull_request.merged == true && contains(github.event.pull_request.labels.*.name, 'needs-backport')
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-      pull-requests: write
-      issues: write
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Configure git
-        run: |
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-
-      - name: Extract version labels
-        id: versions
-        run: |
-          # Extract version labels (e.g., "1.24", "1.22")
-          VERSIONS=""
-          LABELS='${{ toJSON(github.event.pull_request.labels) }}'
-          for label in $(echo "$LABELS" | jq -r '.[].name'); do
-            # Match version labels like "1.24" (major.minor only)
-            if [[ "$label" =~ ^[0-9]+\.[0-9]+$ ]]; then
-              # Validate the branch exists before adding to list
-              if git ls-remote --exit-code origin "core/${label}" >/dev/null 2>&1; then
-                VERSIONS="${VERSIONS}${label} "
-              else
-                echo "::warning::Label '${label}' found but branch 'core/${label}' does not exist"
-              fi
-            fi
-          done
-
-          if [ -z "$VERSIONS" ]; then
-            echo "::error::No version labels found (e.g., 1.24, 1.22)"
-            exit 1
-          fi
-
-          echo "versions=${VERSIONS}" >> $GITHUB_OUTPUT
-          echo "Found version labels: ${VERSIONS}"
-
-      - name: Backport commits
-        id: backport
-        env:
-          PR_NUMBER: ${{ github.event.pull_request.number }}
-          PR_TITLE: ${{ github.event.pull_request.title }}
-          MERGE_COMMIT: ${{ github.event.pull_request.merge_commit_sha }}
-        run: |
-          FAILED=""
-          SUCCESS=""
-
-          for version in ${{ steps.versions.outputs.versions }}; do
-            echo "::group::Backporting to core/${version}"
-
-            TARGET_BRANCH="core/${version}"
-            BACKPORT_BRANCH="backport-${PR_NUMBER}-to-${version}"
-
-            # Fetch target branch (fail if doesn't exist)
-            if ! git fetch origin "${TARGET_BRANCH}"; then
-              echo "::error::Target branch ${TARGET_BRANCH} does not exist"
-              FAILED="${FAILED}${version}:branch-missing "
-              echo "::endgroup::"
-              continue
-            fi
-
-            # Create backport branch
-            git checkout -b "${BACKPORT_BRANCH}" "origin/${TARGET_BRANCH}"
-
-            # Try cherry-pick
-            if git cherry-pick "${MERGE_COMMIT}"; then
-              git push origin "${BACKPORT_BRANCH}"
-              SUCCESS="${SUCCESS}${version}:${BACKPORT_BRANCH} "
-              echo "Successfully created backport branch: ${BACKPORT_BRANCH}"
-              # Return to main (keep the branch, we need it for PR)
-              git checkout main
-            else
-              # Get conflict info
-              CONFLICTS=$(git diff --name-only --diff-filter=U | tr '\n' ',')
-              git cherry-pick --abort
-
-              echo "::error::Cherry-pick failed due to conflicts"
-              FAILED="${FAILED}${version}:conflicts:${CONFLICTS} "
-
-              # Clean up the failed branch
-              git checkout main
-              git branch -D "${BACKPORT_BRANCH}"
-            fi
-
-            echo "::endgroup::"
-          done
-
-          echo "success=${SUCCESS}" >> $GITHUB_OUTPUT
-          echo "failed=${FAILED}" >> $GITHUB_OUTPUT
-
-          if [ -n "${FAILED}" ]; then
-            exit 1
-          fi
-
-      - name: Create PR for each successful backport
-        if: steps.backport.outputs.success
-        env:
-          GH_TOKEN: ${{ secrets.PR_GH_TOKEN }}
-        run: |
-          PR_TITLE="${{ github.event.pull_request.title }}"
-          PR_NUMBER="${{ github.event.pull_request.number }}"
-          PR_AUTHOR="${{ github.event.pull_request.user.login }}"
-
-          for backport in ${{ steps.backport.outputs.success }}; do
-            IFS=':' read -r version branch <<< "${backport}"
-
-            if PR_URL=$(gh pr create \
-              --base "core/${version}" \
-              --head "${branch}" \
-              --title "[backport ${version}] ${PR_TITLE}" \
-              --body "Backport of #${PR_NUMBER} to \`core/${version}\`"$'\n\n'"Automatically created by backport workflow." \
-              --label "backport" 2>&1); then
-
-              # Extract PR number from URL
-              PR_NUM=$(echo "${PR_URL}" | grep -o '[0-9]*$')
-
-              if [ -n "${PR_NUM}" ]; then
-                gh pr comment "${PR_NUMBER}" --body "@${PR_AUTHOR} Successfully backported to #${PR_NUM}"
-              fi
-            else
-              echo "::error::Failed to create PR for ${version}: ${PR_URL}"
-              # Still try to comment on the original PR about the failure
-              gh pr comment "${PR_NUMBER}" --body "@${PR_AUTHOR} Backport branch created but PR creation failed for \`core/${version}\`. Please create the PR manually from branch \`${branch}\`"
-            fi
-          done
-
-      - name: Comment on failures
-        if: failure() && steps.backport.outputs.failed
-        env:
-          GH_TOKEN: ${{ github.token }}
-        run: |
-          PR_NUMBER="${{ github.event.pull_request.number }}"
-          PR_AUTHOR="${{ github.event.pull_request.user.login }}"
-          MERGE_COMMIT="${{ github.event.pull_request.merge_commit_sha }}"
-
-          for failure in ${{ steps.backport.outputs.failed }}; do
-            IFS=':' read -r version reason conflicts <<< "${failure}"
-
-            if [ "${reason}" = "branch-missing" ]; then
-              gh pr comment "${PR_NUMBER}" --body "@${PR_AUTHOR} Backport failed: Branch \`core/${version}\` does not exist"
-
-            elif [ "${reason}" = "conflicts" ]; then
-              # Convert comma-separated conflicts back to newlines for display
-              CONFLICTS_LIST=$(echo "${conflicts}" | tr ',' '\n' | sed 's/^/- /')
-
-              COMMENT_BODY="@${PR_AUTHOR} Backport to \`core/${version}\` failed: Merge conflicts detected."$'\n\n'"Please manually cherry-pick commit \`${MERGE_COMMIT}\` to the \`core/${version}\` branch."$'\n\n'"<details><summary>Conflicting files</summary>"$'\n\n'"${CONFLICTS_LIST}"$'\n\n'"</details>"
-              gh pr comment "${PR_NUMBER}" --body "${COMMENT_BODY}"
-            fi
-          done

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

@@ -4,8 +4,6 @@ permissions:
   contents: read
   pull-requests: write
   issues: write
-  id-token: write
-  statuses: write
 
 on:
   pull_request:
@@ -22,7 +20,7 @@ jobs:
         uses: lewagon/wait-on-check-action@v1.3.1
         with:
           ref: ${{ github.event.pull_request.head.sha }}
-          check-regexp: '^(eslint|prettier|test|playwright-tests)'
+          check-regexp: '^(ESLint|Prettier Check|Tests CI|Vitest Tests)'
           wait-interval: 30
           repo-token: ${{ secrets.GITHUB_TOKEN }}
       
@@ -30,7 +28,7 @@ jobs:
         id: check-status
         run: |
           # Get all check runs for this commit
-          CHECK_RUNS=$(gh api repos/${{ github.repository }}/commits/${{ github.event.pull_request.head.sha }}/check-runs --jq '.check_runs[] | select(.name | test("eslint|prettier|test|playwright-tests")) | {name, conclusion}')
+          CHECK_RUNS=$(gh api repos/${{ github.repository }}/commits/${{ github.event.pull_request.head.sha }}/check-runs --jq '.check_runs[] | select(.name | test("ESLint|Prettier Check|Tests CI|Vitest Tests")) | {name, conclusion}')
           
           # Check if any required checks failed
           if echo "$CHECK_RUNS" | grep -q '"conclusion": "failure"'; then
@@ -65,17 +63,10 @@ jobs:
       - name: Run Claude PR Review
         uses: anthropics/claude-code-action@main
         with:
-          label_trigger: "claude-review"
-          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.
+          prompt_file: .claude/commands/comprehensive-pr-review.md
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-          max_turns: 256
+          max_turns: 1
           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

@@ -1,214 +0,0 @@
-name: Create Release Branch
-
-on:
-  pull_request:
-    types: [closed]
-    branches: [main]
-    paths:
-      - 'package.json'
-
-jobs:
-  create-release-branch:
-    runs-on: ubuntu-latest
-    if: >
-      github.event.pull_request.merged == true &&
-      contains(github.event.pull_request.labels.*.name, 'Release')
-    permissions:
-      contents: write
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-          token: ${{ secrets.PR_GH_TOKEN || secrets.GITHUB_TOKEN }}
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: 'lts/*'
-
-      - name: Check version bump type
-        id: check_version
-        run: |
-          # Get current version from main
-          CURRENT_VERSION=$(node -p "require('./package.json').version")
-          # Remove 'v' prefix if present (shouldn't be in package.json, but defensive)
-          CURRENT_VERSION=${CURRENT_VERSION#v}
-          echo "current_version=$CURRENT_VERSION" >> $GITHUB_OUTPUT
-
-          # Validate version format
-          if ! [[ "$CURRENT_VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+ ]]; then
-            echo "ERROR: Invalid version format: $CURRENT_VERSION"
-            exit 1
-          fi
-
-          # Extract major and minor versions
-          MAJOR=$(echo $CURRENT_VERSION | cut -d. -f1)
-          MINOR=$(echo $CURRENT_VERSION | cut -d. -f2)
-          PATCH=$(echo $CURRENT_VERSION | cut -d. -f3 | cut -d- -f1)
-
-          echo "major=$MAJOR" >> $GITHUB_OUTPUT
-          echo "minor=$MINOR" >> $GITHUB_OUTPUT
-          echo "patch=$PATCH" >> $GITHUB_OUTPUT
-
-          # Get previous version from the commit before the merge
-          git checkout HEAD^1
-          PREV_VERSION=$(node -p "require('./package.json').version" 2>/dev/null || echo "0.0.0")
-          # Remove 'v' prefix if present
-          PREV_VERSION=${PREV_VERSION#v}
-
-          # Validate previous version format
-          if ! [[ "$PREV_VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+ ]]; then
-            echo "WARNING: Invalid previous version format: $PREV_VERSION, using 0.0.0"
-            PREV_VERSION="0.0.0"
-          fi
-
-          PREV_MINOR=$(echo $PREV_VERSION | cut -d. -f2)
-
-          echo "prev_version=$PREV_VERSION" >> $GITHUB_OUTPUT
-          echo "prev_minor=$PREV_MINOR" >> $GITHUB_OUTPUT
-
-          # Get previous major version for comparison
-          PREV_MAJOR=$(echo $PREV_VERSION | cut -d. -f1)
-
-          # Check if current version is a pre-release
-          if [[ "$CURRENT_VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+- ]]; then
-            IS_PRERELEASE=true
-          else
-            IS_PRERELEASE=false
-          fi
-
-          # Check if this was a minor version bump or major version bump
-          # But skip if it's a pre-release version
-          if [[ "$IS_PRERELEASE" == "true" ]]; then
-            echo "is_minor_bump=false" >> $GITHUB_OUTPUT
-            echo "reason=prerelease version" >> $GITHUB_OUTPUT
-          elif [[ "$MAJOR" -gt "$PREV_MAJOR" && "$MINOR" == "0" && "$PATCH" == "0" ]]; then
-            # Major version bump (e.g., 1.99.x → 2.0.0)
-            echo "is_minor_bump=true" >> $GITHUB_OUTPUT
-            BRANCH_NAME="core/${PREV_MAJOR}.${PREV_MINOR}"
-            echo "branch_name=$BRANCH_NAME" >> $GITHUB_OUTPUT
-          elif [[ "$MAJOR" == "$PREV_MAJOR" && "$MINOR" -gt "$PREV_MINOR" && "$PATCH" == "0" ]]; then
-            # Minor version bump (e.g., 1.23.x → 1.24.0)
-            echo "is_minor_bump=true" >> $GITHUB_OUTPUT
-            BRANCH_NAME="core/${MAJOR}.${PREV_MINOR}"
-            echo "branch_name=$BRANCH_NAME" >> $GITHUB_OUTPUT
-          else
-            echo "is_minor_bump=false" >> $GITHUB_OUTPUT
-          fi
-
-          # Return to main branch
-          git checkout main
-
-      - name: Create release branch
-        if: steps.check_version.outputs.is_minor_bump == 'true'
-        run: |
-          BRANCH_NAME="${{ steps.check_version.outputs.branch_name }}"
-
-          # Check if branch already exists
-          if git ls-remote --heads origin "$BRANCH_NAME" | grep -q "$BRANCH_NAME"; then
-            echo "⚠️  Branch $BRANCH_NAME already exists, skipping creation"
-            echo "branch_exists=true" >> $GITHUB_ENV
-            exit 0
-          else
-            echo "branch_exists=false" >> $GITHUB_ENV
-          fi
-
-          # Create branch from the commit BEFORE the version bump
-          # This ensures the release branch has the previous minor version
-          git checkout -b "$BRANCH_NAME" HEAD^1
-
-          # Push the new branch
-          git push origin "$BRANCH_NAME"
-
-          echo "✅ Created release branch: $BRANCH_NAME"
-          echo "This branch is now in feature freeze and will only receive:"
-          echo "- Bug fixes"
-          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": ["build", "test"]
-              },
-              "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'
-        run: |
-          BRANCH_NAME="${{ steps.check_version.outputs.branch_name }}"
-          PREV_VERSION="${{ steps.check_version.outputs.prev_version }}"
-          CURRENT_VERSION="${{ steps.check_version.outputs.current_version }}"
-
-          if [[ "${{ env.branch_exists }}" == "true" ]]; then
-            cat >> $GITHUB_STEP_SUMMARY << EOF
-          ## 🌿 Release Branch Already Exists
-
-          The release branch for the previous minor version already exists:
-          EOF
-          else
-            cat >> $GITHUB_STEP_SUMMARY << EOF
-          ## 🌿 Release Branch Created
-
-          A new release branch has been created for the previous minor version:
-          EOF
-          fi
-
-          cat >> $GITHUB_STEP_SUMMARY << EOF
-
-          - **Branch**: \`$BRANCH_NAME\`
-          - **Version**: \`$PREV_VERSION\` (feature frozen)
-          - **Main branch**: \`$CURRENT_VERSION\` (active development)
-
-          ### Branch Policy
-
-          The \`$BRANCH_NAME\` branch is now in **feature freeze** and will only accept:
-          - 🐛 Bug fixes
-          - 🔒 Security patches
-          - 📚 Documentation updates
-
-          All new features should continue to be developed against \`main\`.
-
-          ### Backporting Changes
-
-          To backport a fix to this release branch:
-          1. Create your fix on \`main\` first
-          2. Cherry-pick to \`$BRANCH_NAME\`
-          3. Create a PR targeting \`$BRANCH_NAME\`
-          4. Use the \`Release\` label on the PR
-          EOF

.github/workflows/eslint.yaml

@@ -2,7 +2,7 @@ name: ESLint
 
 on:
   pull_request:
-    branches-ignore: [ wip/*, draft/*, temp/* ]
+    branches: [ main, master, dev*, core/*, desktop/* ]
 
 jobs:
   eslint:

.github/workflows/format.yaml

@@ -2,7 +2,7 @@ name: Prettier Check
 
 on:
   pull_request:
-    branches-ignore: [ wip/*, draft/*, temp/* ]
+    branches: [ main, master, dev*, core/*, desktop/* ]
 
 jobs:
   prettier:

.github/workflows/test-ui.yaml

@@ -4,7 +4,7 @@ on:
   push:
     branches: [main, master, core/*, desktop/*]
   pull_request:
-    branches-ignore: [wip/*, draft/*, temp/*]
+    branches: [main, master, dev*, core/*, desktop/*]
 
 jobs:
   setup:

.github/workflows/update-litegraph.yaml

@@ -0,0 +1,43 @@
+name: Update Litegraph Dependency
+
+on:
+  workflow_dispatch:
+
+jobs:
+  update-litegraph:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      pull-requests: write
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+
+      - name: Update litegraph
+        run: npm install @comfyorg/litegraph@latest
+
+      - name: Get new version
+        id: get-version
+        run: |
+          NEW_VERSION=$(node -e "console.log(JSON.parse(require('fs').readFileSync('./package-lock.json')).packages['node_modules/@comfyorg/litegraph'].version)")
+          echo "NEW_VERSION=$NEW_VERSION" >> $GITHUB_OUTPUT
+
+      - name: Create Pull Request
+        uses: peter-evans/create-pull-request@271a8d0340265f705b14b6d32b9829c1cb33d45e
+        with:
+          token: ${{ secrets.PR_GH_TOKEN }}
+          commit-message: '[chore] Update litegraph to ${{ steps.get-version.outputs.NEW_VERSION }}'
+          title: '[chore] Update litegraph to ${{ steps.get-version.outputs.NEW_VERSION }}'
+          body: |
+            Automated update of litegraph to version ${{ steps.get-version.outputs.NEW_VERSION }}.
+            Ref: https://github.com/Comfy-Org/litegraph.js/releases/tag/v${{ steps.get-version.outputs.NEW_VERSION }}
+          branch: update-litegraph-${{ steps.get-version.outputs.NEW_VERSION }}
+          base: main
+          labels: |
+            dependencies

.github/workflows/update-manager-types.yaml

@@ -3,11 +3,6 @@ name: Update ComfyUI-Manager API Types
 on:
   # Manual trigger
   workflow_dispatch:
-    inputs:
-      target_branch:
-        description: 'Target branch for the PR'
-        required: true
-        default: 'main'
 
 jobs:
   update-manager-types:
@@ -90,7 +85,7 @@ jobs:
 
             These types are automatically generated using openapi-typescript.
           branch: update-manager-types-${{ steps.manager-info.outputs.commit }}
-          base: ${{ inputs.target_branch }}
+          base: main
           labels: Manager
           delete-branch: true
           add-paths: |

.github/workflows/vitest.yaml

@@ -4,7 +4,7 @@ on:
   push:
     branches: [ main, master, dev*, core/*, desktop/* ]
   pull_request:
-    branches-ignore: [ wip/*, draft/*, temp/* ]
+    branches: [ main, master, dev*, core/*, desktop/* ]
 
 jobs:
   test:

.gitignore

@@ -11,8 +11,6 @@ node_modules
 dist
 dist-ssr
 *.local
-# Claude configuration
-.claude/*.local.json
 
 # Editor directories and files
 .vscode/*

.husky/pre-commit

@@ -1,4 +1,9 @@
-#!/usr/bin/env bash
-
-npx lint-staged
-npx tsx scripts/check-unused-i18n-keys.ts
+if [[ "$OS" == "Windows_NT" ]]; then
+  npx.cmd lint-staged
+  # Check for unused i18n keys in staged files
+  npx.cmd tsx scripts/check-unused-i18n-keys.ts
+else
+  npx lint-staged
+  # Check for unused i18n keys in staged files
+  npx tsx scripts/check-unused-i18n-keys.ts
+fi

.mcp.json

@@ -3,10 +3,6 @@
     "playwright": {
       "command": "npx",
       "args": ["-y", "@executeautomation/playwright-mcp-server"]
-    },
-    "context7": {
-      "command": "npx",
-      "args": ["-y", "@upstash/context7-mcp"]
     }
   }
 }

CLAUDE.md

@@ -1,61 +1,58 @@
-# ComfyUI Frontend Project Guidelines
-
-## Quick Commands
-
-- `npm run`: See all available commands
-- `npm run typecheck`: Type checking
-- `npm run lint`: Linting
-- `npm run format`: Prettier formatting
-- `npm run test:component`: Run component tests with browser environment
-- `npm run test:unit`: Run all unit tests
-- `npm run test:unit -- tests-ui/tests/example.test.ts`: Run single test file
-
-## Development Workflow
-
-1. Make code changes
-2. Run tests (see subdirectory CLAUDE.md files)
-3. Run typecheck, lint, format
-4. Check README updates
-5. Consider docs.comfy.org updates
-
-## Git Conventions
-
-- Use [prefix] format: [feat], [bugfix], [docs]
-- Add "Fixes #n" to PR descriptions
-- Never mention Claude/AI in commits
-
-## External Resources
-
-- PrimeVue docs: <https://primevue.org>
-- ComfyUI docs: <https://docs.comfy.org>
-- Electron: <https://www.electronjs.org/docs/latest/>
-- Wiki: <https://deepwiki.com/Comfy-Org/ComfyUI_frontend/1-overview>
-
-## Project Philosophy
-
-- Clean, stable public APIs
-- Domain-driven design
-- Thousands of users and extensions
-- Prioritize clean interfaces that restrict extension access
-
-## Repository Navigation
-
-- Check README files in key folders (tests-ui, browser_tests, composables, etc.)
-- Prefer running single tests for performance
-- Use --help for unfamiliar CLI tools
-
-## GitHub Integration
-
-When referencing Comfy-Org repos:
-
-1. Check for local copy
-2. Use GitHub API for branches/PRs/metadata
-3. Curl GitHub website if needed
-
-## Common Pitfalls
-
-- NEVER use `any` type - use proper TypeScript types
-- NEVER use `as any` type assertions - fix the underlying type issue
-- NEVER use `--no-verify` flag when committing
-- NEVER delete or disable tests to make them pass
-- NEVER circumvent quality checks
+- use `npm run` to see what commands are available
+- For component communication, prefer Vue's event-based pattern (emit/@event-name) for state changes and notifications; use defineExpose with refs only for imperative operations that need direct control (like form.validate(), modal.open(), or editor.focus()); events promote loose coupling and are better for reusable components, while exposed methods are acceptable for tightly-coupled component pairs or when wrapping third-party libraries that require imperative APIs
+- After making code changes, follow this general process: (1) Create unit tests, component tests, browser tests (if appropriate for each), (2) run unit tests, component tests, and browser tests until passing, (3) run typecheck, lint, format (with prettier) -- you can use `npm run` command to see the scripts available, (4) check if any READMEs (including nested) or documentation needs to be updated, (5) Decide whether the changes are worth adding new content to the external documentation for (or would requires changes to the external documentation) at https://docs.comfy.org, then present your suggestion
+- When referencing PrimeVue, you can get all the docs here: https://primevue.org. Do this instead of making up or inferring names of Components
+- When trying to set tailwind classes for dark theme, use "dark-theme:" prefix rather than "dark:"
+- Never add lines to PR descriptions or commit messages that say "Generated with Claude Code"
+- When making PR names and commit messages, if you are going to add a prefix like "docs:", "feat:", "bugfix:", use square brackets around the prefix term and do not use a colon (e.g., should be "[docs]" rather than "docs:").
+- When I reference GitHub Repos related to Comfy-Org, you should proactively fetch or read the associated information in the repo. To do so, you should exhaust all options: (1) Check if we have a local copy of the repo, (2) Use the GitHub API to fetch the information; you may want to do this IN ADDITION to the other options, especially for reading specific branches/PRs/comments/reviews/metadata, and (3) curl the GitHub website and parse the html or json responses
+- For information about ComfyUI, ComfyUI_frontend, or ComfyUI-Manager, you can web search or download these wikis: https://deepwiki.com/Comfy-Org/ComfyUI-Manager, https://deepwiki.com/Comfy-Org/ComfyUI_frontend/1-overview, https://deepwiki.com/comfyanonymous/ComfyUI/2-core-architecture
+- If a question/project is related to Comfy-Org, Comfy, or ComfyUI ecosystem, you should proactively use the Comfy docs to answer the question. The docs may be referenced with URLs like https://docs.comfy.org
+- When operating inside a repo, check for README files at key locations in the repo detailing info about the contents of that folder. E.g., top-level key folders like tests-ui, browser_tests, composables, extensions/core, stores, services often have their own README.md files. When writing code, make sure to frequently reference these README files to understand the overall architecture and design of the project. Pay close attention to the snippets to learn particular patterns that seem to be there for a reason, as you should emulate those.
+- Prefer running single tests, and not the whole test suite, for performance
+- If using a lesser known or complex CLI tool, run the --help to see the documentation before deciding what to run, even if just for double-checking or verifying things.
+- IMPORTANT: the most important goal when writing code is to create clean, best-practices, sustainable, and scalable public APIs and interfaces. Our app is used by thousands of users and we have thousands of mods/extensions that are constantly changing and updating; and we are also always updating. That's why it is IMPORTANT that we design systems and write code that follows practices of domain-driven design, object-oriented design, and design patterns (such that you can assure stability while allowing for all components around you to change and evolve). We ABSOLUTELY prioritize clean APIs and public interfaces that clearly define and restrict how/what the mods/extensions can access.
+- If any of these technologies are referenced, you can proactively read their docs at these locations: https://primevue.org/theming, https://primevue.org/forms/, https://www.electronjs.org/docs/latest/api/browser-window, https://vitest.dev/guide/browser/, https://atlassian.design/components/pragmatic-drag-and-drop/core-package/drop-targets/, https://playwright.dev/docs/api/class-test, https://playwright.dev/docs/api/class-electron, https://www.algolia.com/doc/api-reference/rest-api/, https://pyav.org/docs/develop/cookbook/basics.html
+- IMPORTANT: Never add Co-Authored by Claude or any reference to Claude or Claude Code in commit messages, PR descriptions, titles, or any documentation whatsoever
+- The npm script to type check is called "typecheck" NOT "type check"
+- Use the Vue 3 Composition API instead of the Options API when writing Vue components. An exception is when overriding or extending a PrimeVue component for compatibility, you may use the Options API.
+- when we are solving an issue we know the link/number for, we should add "Fixes #n" (where n is the issue number) to the PR description.
+- Never write css if you can accomplish the same thing with tailwind utility classes
+- Utilize ref and reactive for reactive state
+- Implement computed properties with computed()
+- Use watch and watchEffect for side effects
+- Implement lifecycle hooks with onMounted, onUpdated, etc.
+- Utilize provide/inject for dependency injection
+- Use vue 3.5 style of default prop declaration. Do not define a `props` variable; instead, destructure props. Since vue 3.5, destructuring props does not strip them of reactivity.
+- Use Tailwind CSS for styling
+- Leverage VueUse functions for performance-enhancing styles
+- Use lodash for utility functions
+- Implement proper props and emits definitions
+- Utilize Vue 3's Teleport component when needed
+- Use Suspense for async components
+- Implement proper error handling
+- Follow Vue 3 style guide and naming conventions
+- IMPORTANT: Use vue-i18n for ALL user-facing strings - no hard-coded text in services/utilities. Place new translation entries in src/locales/en/main.json
+- Avoid using `@ts-expect-error` to work around type issues. We needed to employ it to migrate to TypeScript, but it should not be viewed as an accepted practice or standard.
+- DO NOT use deprecated PrimeVue components. Use these replacements instead:
+  * `Dropdown` → Use `Select` (import from 'primevue/select')
+  * `OverlayPanel` → Use `Popover` (import from 'primevue/popover') 
+  * `Calendar` → Use `DatePicker` (import from 'primevue/datepicker')
+  * `InputSwitch` → Use `ToggleSwitch` (import from 'primevue/toggleswitch')
+  * `Sidebar` → Use `Drawer` (import from 'primevue/drawer')
+  * `Chips` → Use `AutoComplete` with multiple enabled and typeahead disabled
+  * `TabMenu` → Use `Tabs` without panels
+  * `Steps` → Use `Stepper` without panels
+  * `InlineMessage` → Use `Message` component
+* Use `api.apiURL()` for all backend API calls and routes
+  - Actual API endpoints like /prompt, /queue, /view, etc.
+  - Image previews: `api.apiURL('/view?...')`
+  - Any backend-generated content or dynamic routes
+* Use `api.fileURL()` for static files served from the public folder:
+  - Templates: `api.fileURL('/templates/default.json')`
+  - Extensions: `api.fileURL(extensionPath)` for loading JS modules
+  - Any static assets that exist in the public directory
+- When implementing code that outputs raw HTML (e.g., using v-html directive), always ensure dynamic content has been properly sanitized with DOMPurify or validated through trusted sources. Prefer Vue templates over v-html when possible.
+- For any async operations (API calls, timers, etc), implement cleanup/cancellation in component unmount to prevent memory leaks
+- Extract complex template conditionals into separate components or computed properties
+- Error messages should be actionable and user-friendly (e.g., "Failed to load data. Please refresh the page." instead of "Unknown error")

README.md

@@ -4,8 +4,6 @@
 
 **Official front-end implementation of [ComfyUI](https://github.com/comfyanonymous/ComfyUI).**
 
-<!-- Testing automatic backport workflow -->
-
 [![Website][website-shield]][website-url]
 [![Discord][discord-shield]][discord-url]
 [![Matrix][matrix-shield]][matrix-url]
@@ -526,10 +524,6 @@ Here are some ways to get involved:
 
 Have another idea? Drop into Discord or open an issue, and let's chat!
 
-### Architecture Decision Records
-
-We document significant architectural decisions using ADRs (Architecture Decision Records). See [docs/adr/](docs/adr/) for all ADRs and the template for creating new ones.
-
 ## Development
 
 ### Prerequisites & Technology Stack
@@ -692,23 +686,16 @@ Component test verifies Vue components in `src/components/`.
 
 Playwright test verifies the whole app. See <https://github.com/Comfy-Org/ComfyUI_frontend/blob/main/browser_tests/README.md> for details.
 
-### Custom Icons
-
-The project supports custom SVG icons through the unplugin-icons system. Custom icons are stored in `src/assets/icons/custom/` and can be used as Vue components with the `i-comfy:` prefix.
-
-For detailed instructions on adding and using custom icons, see [src/assets/icons/README.md](src/assets/icons/README.md).
-
 ### litegraph.js
 
-Since Aug 5, 2025, litegraph.js is now integrated directly into this repository. It was merged using git subtree to preserve the complete commit history ([PR #4667](https://github.com/Comfy-Org/ComfyUI_frontend/pull/4667), [ADR](docs/adr/0001-merge-litegraph-into-frontend.md)).
+This repo is using litegraph package hosted on <https://github.com/Comfy-Org/litegraph.js>. Any changes to litegraph should be submitted in that repo instead.
 
-#### Important Notes
+#### Test litegraph.js changes
 
-- **Issue References**: Commits from the original litegraph repository may contain issue/PR numbers (e.g., #4667) that refer to issues/PRs in the original litegraph.js repository, not this one.
-- **File Paths**: When viewing historical commits, file paths may show the original structure before the subtree merge. In those cases, just consider the paths relative to the new litegraph folder.
-- **Contributing**: All litegraph modifications should now be made directly in this repository.
+- Run `npm link` in the local litegraph repo.
+- Run `npm link @comfyorg/litegraph` in this repo.
 
-The original litegraph repository (<https://github.com/Comfy-Org/litegraph.js>) is now archived.
+This will replace the litegraph package in this repo with the local litegraph repo.
 
 ### i18n
 

browser_tests/CLAUDE.md

@@ -1,17 +0,0 @@
-# E2E Testing Guidelines
-
-## Browser Tests
-- Test user workflows
-- Use Playwright fixtures
-- Follow naming conventions
-
-## Best Practices
-- Check assets/ for test data
-- Prefer specific selectors
-- Test across viewports
-
-## Testing Process
-After code changes:
-1. Create browser tests as appropriate
-2. Run tests until passing
-3. Then run typecheck, lint, format

browser_tests/README.md

@@ -2,133 +2,76 @@
 
 This document outlines the setup, usage, and common patterns for Playwright browser tests in the ComfyUI_frontend project.
 
-## Prerequisites
+## WARNING
 
-**CRITICAL**: Start ComfyUI backend with `--multi-user` flag:
-
-```bash
-python main.py --multi-user
-```
-
-Without this flag, parallel tests will conflict and fail randomly.
+The browser tests will change the ComfyUI backend state, such as user settings and saved workflows.
+If `TEST_COMFYUI_DIR` in `.env` isn't set to your `(Comfy Path)/ComfyUI` directory, these changes won't be automatically restored.
 
 ## Setup
 
 ### ComfyUI devtools
-
 Clone <https://github.com/Comfy-Org/ComfyUI_devtools> to your `custom_nodes` directory.  
 _ComfyUI_devtools adds additional API endpoints and nodes to ComfyUI for browser testing._
 
 ### Node.js & Playwright Prerequisites
-
 Ensure you have Node.js v20 or v22 installed. Then, set up the Chromium test driver:
-
 ```bash
 npx playwright install chromium --with-deps
 ```
 
-### Environment Configuration
+### Environment Variables
+Ensure the environment variables in `.env` are set correctly according to your setup. 
 
-Create `.env` from the template:
+The `.env` file will not exist until you create it yourself.
 
-```bash
-cp .env_example .env
-```
+A template with helpful information can be found in `.env_example`.
 
-Key settings for debugging:
-
-```bash
-# Remove Vue dev overlay that blocks UI elements
-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
-
-# Path to ComfyUI for backing up user data/settings before tests
-TEST_COMFYUI_DIR=/path/to/your/ComfyUI
-```
-
-### Common Setup Issues
-
-**Most tests require the new menu system** - Add to your test:
-
-```typescript
-test.beforeEach(async ({ comfyPage }) => {
-  await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
-})
-```
+### Multiple Tests
+If you are running Playwright tests in parallel or running the same test multiple times, the flag `--multi-user` must be added to the main ComfyUI process.
 
 ### Release API Mocking
-
 By default, all tests mock the release API (`api.comfy.org/releases`) to prevent release notification popups from interfering with test execution. This is necessary because the release notifications can appear over UI elements and block test interactions.
 
 To test with real release data, you can disable mocking:
-
 ```typescript
-await comfyPage.setup({ mockReleases: false })
+await comfyPage.setup({ mockReleases: false });
 ```
 
 For tests that specifically need to test release functionality, see the example in `tests/releaseNotifications.spec.ts`.
 
 ## Running Tests
 
-**Always use UI mode for development:**
+There are multiple ways to run the tests:
 
-```bash
-npx playwright test --ui
-```
+1. **Headless mode with report generation:**
+   ```bash
+   npx playwright test
+   ```
+   This runs all tests without a visible browser and generates a comprehensive test report.
 
-UI mode features:
+2. **UI mode for interactive testing:**
+   ```bash
+   npx playwright test --ui
+   ```
+   This opens a user interface where you can select specific tests to run and inspect the test execution timeline.
 
-- **Locator picker**: Click the target icon, then click any element to get the exact locator code to use in your test. The code appears in the _Locator_ tab.
-- **Step debugging**: Step through your test line-by-line by clicking _Source_ tab
-- **Time travel**: In the _Actions_ tab/panel, click any step to see the browser state at that moment
-- **Console/Network Tabs**: View logs and API calls at each step
-- **Attachments Tab**: View all snapshots with expected and actual images
+   ![Playwright UI Mode](https://github.com/user-attachments/assets/6a1ebef0-90eb-4157-8694-f5ee94d03755)
 
-![Playwright UI Mode](https://github.com/user-attachments/assets/c158c93f-b39a-44c5-a1a1-e0cc975ee9f2)
-
-For CI or headless testing:
-
-```bash
-npx playwright test                    # Run all tests
-npx playwright test widget.spec.ts     # Run specific test file
-```
-
-### Local Development Config
-
-For debugging, you can try adjusting these settings in `playwright.config.ts`:
-
-```typescript
-export default defineConfig({
-  // VERY HELPFUL: Skip screenshot tests locally
-  grep: process.env.CI ? undefined : /^(?!.*screenshot).*$/
-
-  retries: 0, // No retries while debugging. Increase if writing new tests. that may be flaky.
-  workers: 1, // Single worker for easier debugging. Increase to match CPU cores if you want to run a lot of tests in parallel.
-  timeout: 30000, // Longer timeout for breakpoints
-
-  use: {
-    trace: 'on', // Always capture traces (CI uses 'on-first-retry')
-    video: 'on' // Always record video (CI uses 'retain-on-failure')
-  },
-
-})
-```
+3. **Running specific tests:**
+   ```bash
+   npx playwright test widget.spec.ts
+   ```
 
 ## Test Structure
 
 Browser tests in this project follow a specific organization pattern:
 
 - **Fixtures**: Located in `fixtures/` - These provide test setup and utilities
-
   - `ComfyPage.ts` - The main fixture for interacting with ComfyUI
   - `ComfyMouse.ts` - Utility for mouse interactions with the canvas
   - Components fixtures in `fixtures/components/` - Page object models for UI components
 
 - **Tests**: Located in `tests/` - The actual test specifications
-
   - Organized by functionality (e.g., `widget.spec.ts`, `interaction.spec.ts`)
   - Snapshot directories (e.g., `widget.spec.ts-snapshots/`) contain reference screenshots
 
@@ -143,18 +86,18 @@ When writing new tests, follow these patterns:
 
 ```typescript
 // Import the test fixture
-import { comfyPageFixture as test } from '../fixtures/ComfyPage'
+import { comfyPageFixture as test } from '../fixtures/ComfyPage';
 
 test.describe('Feature Name', () => {
   // Set up test environment if needed
   test.beforeEach(async ({ comfyPage }) => {
     // Common setup
-  })
+  });
 
   test('should do something specific', async ({ comfyPage }) => {
     // Test implementation
-  })
-})
+  });
+});
 ```
 
 ### Leverage Existing Fixtures and Helpers
@@ -176,102 +119,66 @@ Most common testing needs are already addressed by these helpers, which will mak
 
 1. **Focus elements explicitly**:
    Canvas-based elements often need explicit focus before interaction:
-
    ```typescript
    // Click the canvas first to focus it before pressing keys
-   await comfyPage.canvas.click()
-   await comfyPage.page.keyboard.press('a')
+   await comfyPage.canvas.click();
+   await comfyPage.page.keyboard.press('a');
    ```
 
 2. **Mark canvas as dirty if needed**:
    Some interactions need explicit canvas updates:
-
    ```typescript
    // After programmatically changing node state, mark canvas dirty
    await comfyPage.page.evaluate(() => {
-     window['app'].graph.setDirtyCanvas(true, true)
-   })
+     window['app'].graph.setDirtyCanvas(true, true);
+   });
    ```
 
-3. **Use node references over coordinates**:
+3. **Use node references over coordinates**: 
    Node references from `fixtures/utils/litegraphUtils.ts` provide stable ways to interact with nodes:
-
    ```typescript
    // Prefer this:
-   const node = await comfyPage.getNodeRefsByType('LoadImage')[0]
-   await node.click('title')
-
+   const node = await comfyPage.getNodeRefsByType('LoadImage')[0];
+   await node.click('title');
+   
    // Over this:
-   await comfyPage.canvas.click({ position: { x: 100, y: 100 } })
+   await comfyPage.canvas.click({ position: { x: 100, y: 100 } });
    ```
 
 4. **Wait for canvas to render after UI interactions**:
-
    ```typescript
-   await comfyPage.nextFrame()
+   await comfyPage.nextFrame();
    ```
 
 5. **Clean up persistent server state**:
    While most state is reset between tests, anything stored on the server persists:
-
    ```typescript
    // Reset settings that affect other tests (these are stored on server)
-   await comfyPage.setSetting('Comfy.ColorPalette', 'dark')
-   await comfyPage.setSetting('Comfy.NodeBadge.NodeIdBadgeMode', 'None')
-
+   await comfyPage.setSetting('Comfy.ColorPalette', 'dark');
+   await comfyPage.setSetting('Comfy.NodeBadge.NodeIdBadgeMode', 'None');
+   
    // Clean up uploaded files if needed
-   await comfyPage.request.delete(`${comfyPage.url}/api/delete/image.png`)
+   await comfyPage.request.delete(`${comfyPage.url}/api/delete/image.png`);
    ```
 
 6. **Prefer functional assertions over screenshots**:
    Use screenshots only when visual verification is necessary:
-
    ```typescript
    // Prefer this:
-   expect(await node.isPinned()).toBe(true)
-   expect(await node.getProperty('title')).toBe('Expected Title')
-
+   expect(await node.isPinned()).toBe(true);
+   expect(await node.getProperty('title')).toBe('Expected Title');
+   
    // Over this - only use when needed:
-   await expect(comfyPage.canvas).toHaveScreenshot('state.png')
+   await expect(comfyPage.canvas).toHaveScreenshot('state.png');
    ```
 
 7. **Use minimal test workflows**:
    When creating test workflows, keep them as minimal as possible:
-
    ```typescript
    // Include only the components needed for the test
-   await comfyPage.loadWorkflow('single_ksampler')
+   await comfyPage.loadWorkflow('single_ksampler');
    ```
 
-8. **Debug helpers for visual debugging** (remove before committing):
-
-   ComfyPage includes temporary debug methods for troubleshooting:
-
-   ```typescript
-   test('debug failing interaction', async ({ comfyPage }, testInfo) => {
-     // Add visual markers to see click positions
-     await comfyPage.debugAddMarker({ x: 100, y: 200 })
-
-     // Attach screenshot with markers to test report
-     await comfyPage.debugAttachScreenshot(testInfo, 'node-positions', {
-       element: 'canvas',
-       markers: [{ position: { x: 100, y: 200 } }]
-     })
-
-     // Show canvas overlay for easier debugging
-     await comfyPage.debugShowCanvasOverlay()
-
-     // Remember to remove debug code before committing!
-   })
-   ```
-
-   Available debug methods:
-
-   - `debugAddMarker(position)` - Red circle at position
-   - `debugAttachScreenshot(testInfo, name)` - Attach to test report
-   - `debugShowCanvasOverlay()` - Show canvas as overlay
-   - `debugGetCanvasDataURL()` - Get canvas as base64
-
 ## Common Patterns and Utilities
 
 ### Page Object Pattern
@@ -285,7 +192,7 @@ test('Can toggle boolean widget', async ({ comfyPage }) => {
   const node = (await comfyPage.getFirstNodeRef())!
   const widget = await node.getWidget(0)
   await widget.click()
-})
+});
 ```
 
 ### Node References
@@ -325,8 +232,8 @@ Canvas operations use special helpers to ensure proper timing:
 ```typescript
 // Using ComfyMouse for drag and drop
 await comfyMouse.dragAndDrop(
-  { x: 100, y: 100 }, // From
-  { x: 200, y: 200 } // To
+  { x: 100, y: 100 },  // From
+  { x: 200, y: 200 }   // To
 )
 
 // Standard ComfyPage helpers
@@ -368,52 +275,21 @@ await expect(node).toBeCollapsed()
 - **Screenshots vary**: Ensure your OS and browser match the reference environment (Linux)
 - **Async / await**: Race conditions are a very common cause of test flakiness
 
-## Screenshot Testing
+## Screenshot Expectations
 
 Due to variations in system font rendering, screenshot expectations are platform-specific. Please note:
 
-- **Do not commit local screenshot expectations** to the repository
+- **DO NOT commit local screenshot expectations** to the repository
 - We maintain Linux screenshot expectations as our GitHub Action runner operates in a Linux environment
 - While developing, you can generate local screenshots for your tests, but these will differ from CI-generated ones
 
-### Working with Screenshots Locally
+To set new test expectations for PR:
 
-Option 1 - Skip screenshot tests (add to `playwright.config.ts`):
+1. Write your test with screenshot assertions using `toHaveScreenshot(filename)`
+2. Create a pull request from a `Comfy-Org/ComfyUI_frontend` branch
+3. Add the `New Browser Test Expectation` tag to your pull request
+4. The GitHub CI will automatically generate and commit the reference screenshots
 
-```typescript
-export default defineConfig({
-  grep: process.env.CI ? undefined : /^(?!.*screenshot).*$/
-})
-```
+This approach ensures consistent screenshot expectations across all PRs and avoids issues with platform-specific rendering.
 
-Option 2 - Generate local baselines for comparison:
-
-```bash
-npx playwright test --update-snapshots
-```
-
-### Getting Test Artifacts from GitHub Actions
-
-When tests fail in CI, you can download screenshots and traces:
-
-1. Go to the failed workflow run in GitHub Actions
-2. Scroll to "Artifacts" section at the bottom
-3. Download `playwright-report` or `test-results`
-4. Extract and open the HTML report locally
-5. View actual vs expected screenshots and execution traces
-
-### Creating New Screenshot Baselines
-
-For PRs from `Comfy-Org/ComfyUI_frontend` branches:
-
-1. Write test with `toHaveScreenshot('filename.png')`
-2. Create PR and add `New Browser Test Expectation` label
-3. CI will generate and commit the Linux baseline screenshots
-
-> **Note:** Fork PRs cannot auto-commit screenshots. A maintainer will need to commit the screenshots manually for you (don't worry, they'll do it).
-
-## Resources
-
-- [Playwright UI Mode](https://playwright.dev/docs/test-ui-mode) - Interactive test debugging
-- [Playwright Debugging Guide](https://playwright.dev/docs/debug)
-- [act](https://github.com/nektos/act) - Run GitHub Actions locally for CI debugging
+> **Note:** If you're making a pull request from a forked repository, the GitHub action won't be able to commit updated screenshot expectations directly to your PR branch.

browser_tests/assets/missing_nodes_in_subgraph.json

@@ -1,182 +0,0 @@
-{
-  "id": "test-missing-nodes-in-subgraph",
-  "revision": 0,
-  "last_node_id": 2,
-  "last_link_id": 0,
-  "nodes": [
-    {
-      "id": 1,
-      "type": "KSampler",
-      "pos": [100, 100],
-      "size": [270, 262],
-      "flags": {},
-      "order": 1,
-      "mode": 0,
-      "inputs": [
-        {
-          "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": [
-        {
-          "name": "LATENT",
-          "type": "LATENT",
-          "links": []
-        }
-      ],
-      "properties": {
-        "Node name for S&R": "KSampler"
-      },
-      "widgets_values": [0, "randomize", 20, 8, "euler", "simple", 1]
-    },
-    {
-      "id": 2,
-      "type": "subgraph-with-missing-node",
-      "pos": [400, 100],
-      "size": [144, 46],
-      "flags": {},
-      "order": 0,
-      "mode": 0,
-      "inputs": [
-        {
-          "name": "input1",
-          "type": "CONDITIONING",
-          "link": null
-        }
-      ],
-      "outputs": [
-        {
-          "name": "output1",
-          "type": "LATENT",
-          "links": null
-        }
-      ],
-      "properties": {},
-      "widgets_values": []
-    }
-  ],
-  "links": [],
-  "groups": [],
-  "definitions": {
-    "subgraphs": [
-      {
-        "id": "subgraph-with-missing-node",
-        "version": 1,
-        "state": {
-          "lastGroupId": 0,
-          "lastNodeId": 2,
-          "lastLinkId": 2,
-          "lastRerouteId": 0
-        },
-        "revision": 0,
-        "config": {},
-        "name": "Subgraph with Missing Node",
-        "inputNode": {
-          "id": -10,
-          "bounding": [100, 200, 120, 60]
-        },
-        "outputNode": {
-          "id": -20,
-          "bounding": [500, 200, 120, 60]
-        },
-        "inputs": [
-          {
-            "id": "input1-id",
-            "name": "input1",
-            "type": "CONDITIONING",
-            "linkIds": [1],
-            "pos": {
-              "0": 150,
-              "1": 220
-            }
-          }
-        ],
-        "outputs": [
-          {
-            "id": "output1-id",
-            "name": "output1",
-            "type": "LATENT",
-            "linkIds": [2],
-            "pos": {
-              "0": 520,
-              "1": 220
-            }
-          }
-        ],
-        "widgets": [],
-        "nodes": [
-          {
-            "id": 1,
-            "type": "MISSING_NODE_TYPE_IN_SUBGRAPH",
-            "pos": [250, 180],
-            "size": [200, 100],
-            "flags": {},
-            "order": 0,
-            "mode": 0,
-            "inputs": [
-              {
-                "name": "input",
-                "type": "CONDITIONING",
-                "link": 1
-              }
-            ],
-            "outputs": [
-              {
-                "name": "output",
-                "type": "LATENT",
-                "links": [2]
-              }
-            ],
-            "properties": {
-              "Node name for S&R": "MISSING_NODE_TYPE_IN_SUBGRAPH"
-            },
-            "widgets_values": ["some", "widget", "values"]
-          }
-        ],
-        "links": [
-          {
-            "id": 1,
-            "origin_id": -10,
-            "origin_slot": 0,
-            "target_id": 1,
-            "target_slot": 0,
-            "type": "CONDITIONING"
-          },
-          {
-            "id": 2,
-            "origin_id": 1,
-            "origin_slot": 0,
-            "target_id": -20,
-            "target_slot": 0,
-            "type": "LATENT"
-          }
-        ]
-      }
-    ]
-  },
-  "config": {},
-  "extra": {
-    "ds": {
-      "scale": 1,
-      "offset": [0, 0]
-    }
-  },
-  "version": 0.4
-}

browser_tests/fixtures/ComfyPage.ts

@@ -1,10 +1,10 @@
+import type { LGraphNode } from '@comfyorg/litegraph'
 import type { APIRequestContext, Locator, Page } from '@playwright/test'
 import { expect } from '@playwright/test'
 import { test as base } from '@playwright/test'
 import dotenv from 'dotenv'
 import * as fs from 'fs'
 
-import type { LGraphNode } from '../../src/lib/litegraph/src/litegraph'
 import type { NodeId } from '../../src/schemas/comfyWorkflowSchema'
 import type { KeyCombo } from '../../src/schemas/keyBindingSchema'
 import type { useWorkspaceStore } from '../../src/stores/workspaceStore'
@@ -168,7 +168,7 @@ export class ComfyPage {
     this.menu = new ComfyMenu(page)
     this.actionbar = new ComfyActionbar(page)
     this.templates = new ComfyTemplates(page)
-    this.settingDialog = new SettingDialog(page, this)
+    this.settingDialog = new SettingDialog(page)
     this.confirmDialog = new ConfirmDialog(page)
   }
 
@@ -563,7 +563,6 @@ export class ComfyPage {
         if (fileName.endsWith('.webm')) return 'video/webm'
         if (fileName.endsWith('.json')) return 'application/json'
         if (fileName.endsWith('.glb')) return 'model/gltf-binary'
-        if (fileName.endsWith('.avif')) return 'image/avif'
         return 'application/octet-stream'
       }
 

browser_tests/fixtures/components/SettingDialog.ts

@@ -1,19 +1,15 @@
 import { Page } from '@playwright/test'
 
-import { ComfyPage } from '../ComfyPage'
-
 export class SettingDialog {
-  constructor(
-    public readonly page: Page,
-    public readonly comfyPage: ComfyPage
-  ) {}
+  constructor(public readonly page: Page) {}
 
   get root() {
     return this.page.locator('div.settings-container')
   }
 
   async open() {
-    await this.comfyPage.executeCommand('Comfy.ShowSettingsDialog')
+    const button = this.page.locator('button.comfy-settings-btn:visible')
+    await button.click()
     await this.page.waitForSelector('div.settings-container')
   }
 

browser_tests/fixtures/components/Topbar.ts

@@ -15,6 +15,10 @@ export class Topbar {
       .innerText()
   }
 
+  async openSubmenuMobile() {
+    await this.page.locator('.p-menubar-mobile .p-menubar-button').click()
+  }
+
   getMenuItem(itemLabel: string): Locator {
     return this.page.locator(`.p-menubar-item-label:text-is("${itemLabel}")`)
   }
@@ -64,41 +68,31 @@ export class Topbar {
     await this.getSaveDialog().waitFor({ state: 'hidden', timeout: 500 })
   }
 
-  async openTopbarMenu() {
-    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[]) {
     if (path.length < 2) {
       throw new Error('Path is too short')
     }
 
-    const menu = await this.openTopbarMenu()
     const tabName = path[0]
-    const topLevelMenuItem = this.page.locator(
-      `.p-menubar-item-label:text-is("${tabName}")`
+    const topLevelMenu = this.page.locator(
+      `.top-menubar .p-menubar-item-label:text-is("${tabName}")`
     )
-    const topLevelMenu = menu
-      .locator('.p-tieredmenu-item')
-      .filter({ has: topLevelMenuItem })
     await topLevelMenu.waitFor({ state: 'visible' })
-    await topLevelMenu.hover()
+    await topLevelMenu.click()
 
-    let currentMenu = topLevelMenu
     for (let i = 1; i < path.length; i++) {
       const commandName = path[i]
-      const menuItem = currentMenu
+      const menuItem = this.page
         .locator(
-          `.p-tieredmenu-submenu .p-tieredmenu-item:has-text("${commandName}")`
+          `.top-menubar .p-menubar-submenu .p-menubar-item:has-text("${commandName}")`
         )
         .first()
       await menuItem.waitFor({ state: 'visible' })
       await menuItem.hover()
-      currentMenu = menuItem
+
+      if (i === path.length - 1) {
+        await menuItem.click()
+      }
     }
-    await currentMenu.click()
   }
 }

browser_tests/fixtures/utils/vueNodeFixtures.ts

@@ -1,131 +0,0 @@
-import type { Locator, Page } from '@playwright/test'
-
-import type { NodeReference } from './litegraphUtils'
-
-/**
- * VueNodeFixture provides Vue-specific testing utilities for interacting with
- * Vue node components. It bridges the gap between litegraph node references
- * and Vue UI components.
- */
-export class VueNodeFixture {
-  constructor(
-    private readonly nodeRef: NodeReference,
-    private readonly page: Page
-  ) {}
-
-  /**
-   * Get the node's header element using data-testid
-   */
-  async getHeader(): Promise<Locator> {
-    const nodeId = this.nodeRef.id
-    return this.page.locator(`[data-testid="node-header-${nodeId}"]`)
-  }
-
-  /**
-   * Get the node's title element
-   */
-  async getTitleElement(): Promise<Locator> {
-    const header = await this.getHeader()
-    return header.locator('[data-testid="node-title"]')
-  }
-
-  /**
-   * Get the current title text
-   */
-  async getTitle(): Promise<string> {
-    const titleElement = await this.getTitleElement()
-    return (await titleElement.textContent()) || ''
-  }
-
-  /**
-   * Set a new title by double-clicking and entering text
-   */
-  async setTitle(newTitle: string): Promise<void> {
-    const titleElement = await this.getTitleElement()
-    await titleElement.dblclick()
-
-    const input = (await this.getHeader()).locator(
-      '[data-testid="node-title-input"]'
-    )
-    await input.fill(newTitle)
-    await input.press('Enter')
-  }
-
-  /**
-   * Cancel title editing
-   */
-  async cancelTitleEdit(): Promise<void> {
-    const titleElement = await this.getTitleElement()
-    await titleElement.dblclick()
-
-    const input = (await this.getHeader()).locator(
-      '[data-testid="node-title-input"]'
-    )
-    await input.press('Escape')
-  }
-
-  /**
-   * Check if the title is currently being edited
-   */
-  async isEditingTitle(): Promise<boolean> {
-    const header = await this.getHeader()
-    const input = header.locator('[data-testid="node-title-input"]')
-    return await input.isVisible()
-  }
-
-  /**
-   * Get the collapse/expand button
-   */
-  async getCollapseButton(): Promise<Locator> {
-    const header = await this.getHeader()
-    return header.locator('[data-testid="node-collapse-button"]')
-  }
-
-  /**
-   * Toggle the node's collapsed state
-   */
-  async toggleCollapse(): Promise<void> {
-    const button = await this.getCollapseButton()
-    await button.click()
-  }
-
-  /**
-   * Get the collapse icon element
-   */
-  async getCollapseIcon(): Promise<Locator> {
-    const button = await this.getCollapseButton()
-    return button.locator('i')
-  }
-
-  /**
-   * Get the collapse icon's CSS classes
-   */
-  async getCollapseIconClass(): Promise<string> {
-    const icon = await this.getCollapseIcon()
-    return (await icon.getAttribute('class')) || ''
-  }
-
-  /**
-   * Check if the collapse button is visible
-   */
-  async isCollapseButtonVisible(): Promise<boolean> {
-    const button = await this.getCollapseButton()
-    return await button.isVisible()
-  }
-
-  /**
-   * Get the node's body/content element
-   */
-  async getBody(): Promise<Locator> {
-    const nodeId = this.nodeRef.id
-    return this.page.locator(`[data-testid="node-body-${nodeId}"]`)
-  }
-
-  /**
-   * Check if the node body is visible (not collapsed)
-   */
-  async isBodyVisible(): Promise<boolean> {
-    const body = await this.getBody()
-    return await body.isVisible()
-  }
-}

browser_tests/tests/dialog.spec.ts

@@ -13,21 +13,6 @@ test.describe('Load workflow warning', () => {
     const missingNodesWarning = comfyPage.page.locator('.comfy-missing-nodes')
     await expect(missingNodesWarning).toBeVisible()
   })
-
-  test('Should display a warning when loading a workflow with missing nodes in subgraphs', async ({
-    comfyPage
-  }) => {
-    await comfyPage.loadWorkflow('missing_nodes_in_subgraph')
-
-    // Wait for the element with the .comfy-missing-nodes selector to be visible
-    const missingNodesWarning = comfyPage.page.locator('.comfy-missing-nodes')
-    await expect(missingNodesWarning).toBeVisible()
-
-    // Verify the missing node text includes subgraph context
-    const warningText = await missingNodesWarning.textContent()
-    expect(warningText).toContain('MISSING_NODE_TYPE_IN_SUBGRAPH')
-    expect(warningText).toContain('in subgraph')
-  })
 })
 
 test('Does not report warning on undo/redo', async ({ comfyPage }) => {
@@ -384,7 +369,7 @@ test.describe('Signin dialog', () => {
     await textBox.press('Control+c')
 
     await comfyPage.page.evaluate(() => {
-      void window['app'].extensionManager.dialog.showSignInDialog()
+      window['app'].extensionManager.dialog.showSignInDialog()
     })
 
     const input = comfyPage.page.locator('#comfy-org-sign-in-password')

browser_tests/tests/domWidget.spec.ts

@@ -47,42 +47,4 @@ test.describe('DOM Widget', () => {
     const finalCount = await comfyPage.getDOMWidgetCount()
     expect(finalCount).toBe(initialCount + 1)
   })
-
-  test('should reposition when layout changes', async ({ comfyPage }) => {
-    // --- setup ---
-
-    const textareaWidget = comfyPage.page
-      .locator('.comfy-multiline-input')
-      .first()
-    await expect(textareaWidget).toBeVisible()
-
-    await comfyPage.setSetting('Comfy.Sidebar.Size', 'small')
-    await comfyPage.setSetting('Comfy.Sidebar.Location', 'left')
-    await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
-    await comfyPage.nextFrame()
-
-    let oldPos: [number, number]
-    const checkBboxChange = async () => {
-      const boudningBox = (await textareaWidget.boundingBox())!
-      expect(boudningBox).not.toBeNull()
-      const position: [number, number] = [boudningBox.x, boudningBox.y]
-      expect(position).not.toEqual(oldPos)
-      oldPos = position
-    }
-    await checkBboxChange()
-
-    // --- test ---
-
-    await comfyPage.setSetting('Comfy.Sidebar.Size', 'normal')
-    await comfyPage.nextFrame()
-    await checkBboxChange()
-
-    await comfyPage.setSetting('Comfy.Sidebar.Location', 'right')
-    await comfyPage.nextFrame()
-    await checkBboxChange()
-
-    await comfyPage.setSetting('Comfy.UseNewMenu', 'Bottom')
-    await comfyPage.nextFrame()
-    await checkBboxChange()
-  })
 })

browser_tests/tests/groupNode.spec.ts

@@ -264,15 +264,10 @@ test.describe('Group Node', () => {
     test('Copies and pastes group node after clearing workflow', async ({
       comfyPage
     }) => {
-      // Set setting
-      await comfyPage.setSetting('Comfy.ConfirmClear', false)
-
-      // Clear workflow
       await comfyPage.menu.topbar.triggerTopbarCommand([
         'Edit',
         'Clear Workflow'
       ])
-
       await comfyPage.ctrlV()
       await verifyNodeLoaded(comfyPage, 1)
     })

browser_tests/tests/interaction.spec.ts

@@ -768,14 +768,8 @@ test.describe('Viewport settings', () => {
     comfyMouse
   }) => {
     // Screenshot the canvas element
-    await comfyPage.setSetting('Comfy.Graph.CanvasMenu', true)
-    const toggleButton = comfyPage.page.getByTestId('toggle-minimap-button')
-
-    await toggleButton.click()
-
     await comfyPage.menu.topbar.saveWorkflow('Workflow A')
-    await comfyPage.nextFrame()
-    const screenshotA = (await comfyPage.canvas.screenshot()).toString('base64')
+    await expect(comfyPage.canvas).toHaveScreenshot('viewport-workflow-a.png')
 
     // Save workflow as a new file, then zoom out before screen shot
     await comfyPage.menu.topbar.saveWorkflowAs('Workflow B')
@@ -783,12 +777,7 @@ test.describe('Viewport settings', () => {
     for (let i = 0; i < 4; i++) {
       await comfyMouse.wheel(0, 60)
     }
-
-    await comfyPage.nextFrame()
-    const screenshotB = (await comfyPage.canvas.screenshot()).toString('base64')
-
-    // Ensure that the screenshots are different due to zoom level
-    expect(screenshotB).not.toBe(screenshotA)
+    await expect(comfyPage.canvas).toHaveScreenshot('viewport-workflow-b.png')
 
     const tabA = comfyPage.menu.topbar.getWorkflowTab('Workflow A')
     const tabB = comfyPage.menu.topbar.getWorkflowTab('Workflow B')
@@ -796,269 +785,11 @@ test.describe('Viewport settings', () => {
     // Go back to Workflow A
     await tabA.click()
     await comfyPage.nextFrame()
-    expect((await comfyPage.canvas.screenshot()).toString('base64')).toBe(
-      screenshotA
-    )
+    await expect(comfyPage.canvas).toHaveScreenshot('viewport-workflow-a.png')
 
     // And back to Workflow B
     await tabB.click()
     await comfyPage.nextFrame()
-    expect((await comfyPage.canvas.screenshot()).toString('base64')).toBe(
-      screenshotB
-    )
-  })
-})
-
-test.describe('Canvas Navigation', () => {
-  test.describe('Legacy Mode', () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.Canvas.NavigationMode', 'legacy')
-    })
-
-    test('Left-click drag in empty area should pan canvas', async ({
-      comfyPage
-    }) => {
-      await comfyPage.dragAndDrop({ x: 50, y: 50 }, { x: 150, y: 150 })
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-left-drag-pan.png'
-      )
-    })
-
-    test('Middle-click drag should pan canvas', async ({ comfyPage }) => {
-      await comfyPage.page.mouse.move(50, 50)
-      await comfyPage.page.mouse.down({ button: 'middle' })
-      await comfyPage.page.mouse.move(150, 150)
-      await comfyPage.page.mouse.up({ button: 'middle' })
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-middle-drag-pan.png'
-      )
-    })
-
-    test('Mouse wheel should zoom in/out', async ({ comfyPage }) => {
-      await comfyPage.page.mouse.move(400, 300)
-      await comfyPage.page.mouse.wheel(0, -120)
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-wheel-zoom-in.png'
-      )
-
-      await comfyPage.page.mouse.wheel(0, 240)
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-wheel-zoom-out.png'
-      )
-    })
-
-    test('Left-click on node should not pan canvas', async ({ comfyPage }) => {
-      await comfyPage.clickTextEncodeNode1()
-      const selectedCount = await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCount).toBe(1)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-click-node-select.png'
-      )
-    })
-  })
-
-  test.describe('Standard Mode', () => {
-    test.beforeEach(async ({ comfyPage }) => {
-      await comfyPage.setSetting('Comfy.Canvas.NavigationMode', 'standard')
-    })
-
-    test('Left-click drag in empty area should select nodes', async ({
-      comfyPage
-    }) => {
-      const clipNodes = await comfyPage.getNodeRefsByType('CLIPTextEncode')
-      const clipNode1Pos = await clipNodes[0].getPosition()
-      const clipNode2Pos = await clipNodes[1].getPosition()
-      const offset = 64
-
-      await comfyPage.dragAndDrop(
-        {
-          x: Math.min(clipNode1Pos.x, clipNode2Pos.x) - offset,
-          y: Math.min(clipNode1Pos.y, clipNode2Pos.y) - offset
-        },
-        {
-          x: Math.max(clipNode1Pos.x, clipNode2Pos.x) + offset,
-          y: Math.max(clipNode1Pos.y, clipNode2Pos.y) + offset
-        }
-      )
-
-      const selectedCount = await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCount).toBe(clipNodes.length)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-left-drag-select.png'
-      )
-    })
-
-    test('Middle-click drag should pan canvas', async ({ comfyPage }) => {
-      await comfyPage.page.mouse.move(50, 50)
-      await comfyPage.page.mouse.down({ button: 'middle' })
-      await comfyPage.page.mouse.move(150, 150)
-      await comfyPage.page.mouse.up({ button: 'middle' })
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-middle-drag-pan.png'
-      )
-    })
-
-    test('Ctrl + mouse wheel should zoom in/out', async ({ comfyPage }) => {
-      await comfyPage.page.mouse.move(400, 300)
-      await comfyPage.page.keyboard.down('Control')
-      await comfyPage.page.mouse.wheel(0, -120)
-      await comfyPage.page.keyboard.up('Control')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-ctrl-wheel-zoom-in.png'
-      )
-
-      await comfyPage.page.keyboard.down('Control')
-      await comfyPage.page.mouse.wheel(0, 240)
-      await comfyPage.page.keyboard.up('Control')
-      await comfyPage.nextFrame()
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-ctrl-wheel-zoom-out.png'
-      )
-    })
-
-    test('Left-click on node should select node (not start selection box)', async ({
-      comfyPage
-    }) => {
-      await comfyPage.clickTextEncodeNode1()
-      const selectedCount = await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCount).toBe(1)
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-click-node-select.png'
-      )
-    })
-
-    test('Space + left-click drag should pan canvas', async ({ comfyPage }) => {
-      // Click canvas to focus it
-      await comfyPage.page.click('canvas')
-      await comfyPage.nextFrame()
-
-      await comfyPage.page.keyboard.down('Space')
-      await comfyPage.dragAndDrop({ x: 50, y: 50 }, { x: 150, y: 150 })
-      await comfyPage.page.keyboard.up('Space')
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'standard-space-drag-pan.png'
-      )
-    })
-
-    test('Space key overrides default left-click behavior', async ({
-      comfyPage
-    }) => {
-      const clipNodes = await comfyPage.getNodeRefsByType('CLIPTextEncode')
-      const clipNode1Pos = await clipNodes[0].getPosition()
-      const offset = 64
-
-      await comfyPage.dragAndDrop(
-        {
-          x: clipNode1Pos.x - offset,
-          y: clipNode1Pos.y - offset
-        },
-        {
-          x: clipNode1Pos.x + offset,
-          y: clipNode1Pos.y + offset
-        }
-      )
-
-      const selectedCountAfterDrag =
-        await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCountAfterDrag).toBeGreaterThan(0)
-
-      await comfyPage.clickEmptySpace()
-      const selectedCountAfterClear =
-        await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCountAfterClear).toBe(0)
-
-      await comfyPage.page.keyboard.down('Space')
-      await comfyPage.dragAndDrop(
-        {
-          x: clipNode1Pos.x - offset,
-          y: clipNode1Pos.y - offset
-        },
-        {
-          x: clipNode1Pos.x + offset,
-          y: clipNode1Pos.y + offset
-        }
-      )
-      await comfyPage.page.keyboard.up('Space')
-
-      const selectedCountAfterSpaceDrag =
-        await comfyPage.getSelectedGraphNodesCount()
-      expect(selectedCountAfterSpaceDrag).toBe(0)
-    })
-  })
-
-  test('Shift + mouse wheel should pan canvas horizontally', async ({
-    comfyPage
-  }) => {
-    await comfyPage.page.click('canvas')
-    await comfyPage.nextFrame()
-
-    await expect(comfyPage.canvas).toHaveScreenshot('standard-initial.png')
-
-    await comfyPage.page.mouse.move(400, 300)
-
-    await comfyPage.page.keyboard.down('Shift')
-    await comfyPage.page.mouse.wheel(0, 120)
-    await comfyPage.page.keyboard.up('Shift')
-    await comfyPage.nextFrame()
-    await expect(comfyPage.canvas).toHaveScreenshot(
-      'standard-shift-wheel-pan-right.png'
-    )
-
-    await comfyPage.page.keyboard.down('Shift')
-    await comfyPage.page.mouse.wheel(0, -240)
-    await comfyPage.page.keyboard.up('Shift')
-    await comfyPage.nextFrame()
-    await expect(comfyPage.canvas).toHaveScreenshot(
-      'standard-shift-wheel-pan-left.png'
-    )
-
-    await comfyPage.page.keyboard.down('Shift')
-    await comfyPage.page.mouse.wheel(0, 120)
-    await comfyPage.page.keyboard.up('Shift')
-    await comfyPage.nextFrame()
-    await expect(comfyPage.canvas).toHaveScreenshot(
-      'standard-shift-wheel-pan-center.png'
-    )
-  })
-
-  test.describe('Edge Cases', () => {
-    test('Multiple modifier keys work correctly in legacy mode', async ({
-      comfyPage
-    }) => {
-      await comfyPage.setSetting('Comfy.Canvas.NavigationMode', 'legacy')
-
-      await comfyPage.page.keyboard.down('Alt')
-      await comfyPage.page.keyboard.down('Shift')
-      await comfyPage.dragAndDrop({ x: 50, y: 50 }, { x: 150, y: 150 })
-      await comfyPage.page.keyboard.up('Shift')
-      await comfyPage.page.keyboard.up('Alt')
-
-      await expect(comfyPage.canvas).toHaveScreenshot(
-        'legacy-alt-shift-drag.png'
-      )
-    })
-
-    test('Cursor changes appropriately in different modes', async ({
-      comfyPage
-    }) => {
-      const getCursorStyle = async () => {
-        return await comfyPage.page.evaluate(() => {
-          return (
-            document.getElementById('graph-canvas')!.style.cursor || 'default'
-          )
-        })
-      }
-
-      await comfyPage.setSetting('Comfy.Canvas.NavigationMode', 'legacy')
-      await comfyPage.page.mouse.move(50, 50)
-      await comfyPage.page.mouse.down()
-      expect(await getCursorStyle()).toBe('grabbing')
-      await comfyPage.page.mouse.up()
-    })
+    await expect(comfyPage.canvas).toHaveScreenshot('viewport-workflow-b.png')
   })
 })

browser_tests/tests/loadWorkflowInMedia.spec.ts

@@ -15,8 +15,7 @@ test.describe('Load Workflow in Media', () => {
     'workflow.mp4',
     'workflow.mov',
     'workflow.m4v',
-    'workflow.svg',
-    'workflow.avif'
+    'workflow.svg'
   ]
   fileNames.forEach(async (fileName) => {
     test(`Load workflow in ${fileName} (drop from filesystem)`, async ({

browser_tests/tests/menu.spec.ts

@@ -63,7 +63,7 @@ test.describe('Menu', () => {
     test('@mobile Items fully visible on mobile screen width', async ({
       comfyPage
     }) => {
-      await comfyPage.menu.topbar.openTopbarMenu()
+      await comfyPage.menu.topbar.openSubmenuMobile()
       const topLevelMenuItem = comfyPage.page
         .locator('a.p-menubar-item-link')
         .first()
@@ -74,9 +74,8 @@ test.describe('Menu', () => {
     })
 
     test('Displays keybinding next to item', async ({ comfyPage }) => {
-      await comfyPage.menu.topbar.openTopbarMenu()
       const workflowMenuItem = comfyPage.menu.topbar.getMenuItem('Workflow')
-      await workflowMenuItem.hover()
+      await workflowMenuItem.click()
       const exportTag = comfyPage.page.locator('.keybinding-tag', {
         hasText: 'Ctrl + s'
       })

browser_tests/tests/minimap.spec.ts

@@ -1,77 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { comfyPageFixture as test } from '../fixtures/ComfyPage'
-
-test.describe('Minimap', () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
-    await comfyPage.setSetting('Comfy.Minimap.Visible', true)
-    await comfyPage.setSetting('Comfy.Graph.CanvasMenu', true)
-    await comfyPage.loadWorkflow('default')
-    await comfyPage.page.waitForFunction(
-      () => window['app'] && window['app'].canvas
-    )
-  })
-
-  test('Validate minimap is visible by default', async ({ comfyPage }) => {
-    const minimapContainer = comfyPage.page.locator('.litegraph-minimap')
-
-    await expect(minimapContainer).toBeVisible()
-
-    const minimapCanvas = minimapContainer.locator('.minimap-canvas')
-    await expect(minimapCanvas).toBeVisible()
-
-    const minimapViewport = minimapContainer.locator('.minimap-viewport')
-    await expect(minimapViewport).toBeVisible()
-
-    await expect(minimapContainer).toHaveCSS('position', 'absolute')
-    await expect(minimapContainer).toHaveCSS('z-index', '1000')
-  })
-
-  test('Validate minimap toggle button state', async ({ comfyPage }) => {
-    const toggleButton = comfyPage.page.getByTestId('toggle-minimap-button')
-
-    await expect(toggleButton).toBeVisible()
-
-    await expect(toggleButton).toHaveClass(/minimap-active/)
-
-    const minimapContainer = comfyPage.page.locator('.litegraph-minimap')
-    await expect(minimapContainer).toBeVisible()
-  })
-
-  test('Validate minimap can be toggled off and on', async ({ comfyPage }) => {
-    const minimapContainer = comfyPage.page.locator('.litegraph-minimap')
-    const toggleButton = comfyPage.page.getByTestId('toggle-minimap-button')
-
-    await expect(minimapContainer).toBeVisible()
-    await expect(toggleButton).toHaveClass(/minimap-active/)
-
-    await toggleButton.click()
-    await comfyPage.nextFrame()
-
-    await expect(minimapContainer).not.toBeVisible()
-    await expect(toggleButton).not.toHaveClass(/minimap-active/)
-
-    await toggleButton.click()
-    await comfyPage.nextFrame()
-
-    await expect(minimapContainer).toBeVisible()
-    await expect(toggleButton).toHaveClass(/minimap-active/)
-  })
-
-  test('Validate minimap keyboard shortcut Alt+M', async ({ comfyPage }) => {
-    const minimapContainer = comfyPage.page.locator('.litegraph-minimap')
-
-    await expect(minimapContainer).toBeVisible()
-
-    await comfyPage.page.keyboard.press('Alt+KeyM')
-    await comfyPage.nextFrame()
-
-    await expect(minimapContainer).not.toBeVisible()
-
-    await comfyPage.page.keyboard.press('Alt+KeyM')
-    await comfyPage.nextFrame()
-
-    await expect(minimapContainer).toBeVisible()
-  })
-})

browser_tests/tests/subgraph.spec.ts

@@ -397,8 +397,7 @@ test.describe('Subgraph Operations', () => {
       // Enable new menu for breadcrumb navigation
       await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
 
-      const workflowName = 'subgraph-with-promoted-text-widget'
-      await comfyPage.loadWorkflow(workflowName)
+      await comfyPage.loadWorkflow('subgraph-with-promoted-text-widget')
 
       const textareaCount = await comfyPage.page
         .locator(SELECTORS.domWidget)
@@ -421,13 +420,8 @@ test.describe('Subgraph Operations', () => {
       })
 
       // Click breadcrumb to navigate back to parent graph
-      const homeBreadcrumb = comfyPage.page.getByRole('link', {
-        // In the subgraph navigation breadcrumbs, the home/top level
-        // breadcrumb is just the workflow name
-        name: workflowName
-      })
-      await homeBreadcrumb.waitFor({ state: 'visible' })
-      await homeBreadcrumb.click()
+      const breadcrumb = comfyPage.page.locator(SELECTORS.breadcrumb).first()
+      await breadcrumb.click()
       await comfyPage.nextFrame()
       await comfyPage.page.waitForTimeout(300)
 

browser_tests/tests/versionMismatchWarnings.spec.ts

@@ -1,117 +0,0 @@
-import { expect } from '@playwright/test'
-
-import { SystemStats } from '../../src/schemas/apiSchema'
-import { comfyPageFixture as test } from '../fixtures/ComfyPage'
-
-test.describe('Version Mismatch Warnings', () => {
-  const ALWAYS_AHEAD_OF_INSTALLED_VERSION = '100.100.100'
-  const ALWAYS_BEHIND_INSTALLED_VERSION = '0.0.0'
-
-  const createMockSystemStatsRes = (
-    requiredFrontendVersion: string
-  ): SystemStats => {
-    return {
-      system: {
-        os: 'posix',
-        ram_total: 67235385344,
-        ram_free: 13464207360,
-        comfyui_version: '0.3.46',
-        required_frontend_version: requiredFrontendVersion,
-        python_version: '3.12.3 (main, Jun 18 2025, 17:59:45) [GCC 13.3.0]',
-        pytorch_version: '2.6.0+cu124',
-        embedded_python: false,
-        argv: ['main.py']
-      },
-      devices: [
-        {
-          name: 'cuda:0 NVIDIA GeForce RTX 4070 : cudaMallocAsync',
-          type: 'cuda',
-          index: 0,
-          vram_total: 12557156352,
-          vram_free: 2439249920,
-          torch_vram_total: 0,
-          torch_vram_free: 0
-        }
-      ]
-    }
-  }
-
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
-  })
-
-  test('should show version mismatch warnings when installed version lower than required', async ({
-    comfyPage
-  }) => {
-    // Mock system_stats route to indicate that the installed version is always ahead of the required version
-    await comfyPage.page.route('**/system_stats**', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(
-          createMockSystemStatsRes(ALWAYS_AHEAD_OF_INSTALLED_VERSION)
-        )
-      })
-    })
-    await comfyPage.setup()
-
-    // Expect a warning toast to be shown
-    await expect(
-      comfyPage.page.getByText('Version Compatibility Warning')
-    ).toBeVisible()
-  })
-
-  test('should not show version mismatch warnings when installed version is ahead of required', async ({
-    comfyPage
-  }) => {
-    // Mock system_stats route to indicate that the installed version is always ahead of the required version
-    await comfyPage.page.route('**/system_stats**', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(
-          createMockSystemStatsRes(ALWAYS_BEHIND_INSTALLED_VERSION)
-        )
-      })
-    })
-    await comfyPage.setup()
-
-    // Expect no warning toast to be shown
-    await expect(
-      comfyPage.page.getByText('Version Compatibility Warning')
-    ).not.toBeVisible()
-  })
-
-  test('should persist dismissed state across sessions', async ({
-    comfyPage
-  }) => {
-    // Mock system_stats route to indicate that the installed version is always ahead of the required version
-    await comfyPage.page.route('**/system_stats**', async (route) => {
-      await route.fulfill({
-        status: 200,
-        contentType: 'application/json',
-        body: JSON.stringify(
-          createMockSystemStatsRes(ALWAYS_AHEAD_OF_INSTALLED_VERSION)
-        )
-      })
-    })
-    await comfyPage.setup()
-
-    // Locate the warning toast and dismiss it
-    const warningToast = comfyPage.page
-      .locator('div')
-      .filter({ hasText: 'Version Compatibility' })
-      .nth(3)
-    await warningToast.waitFor({ state: 'visible' })
-    const dismissButton = warningToast.getByRole('button', { name: 'Close' })
-    await dismissButton.click()
-
-    // Reload the page, keeping local storage
-    await comfyPage.setup({ clearStorage: false })
-
-    // The same warning from same versions should not be shown to the user again
-    await expect(
-      comfyPage.page.getByText('Version Compatibility Warning')
-    ).not.toBeVisible()
-  })
-})

browser_tests/tests/vueNodes/NodeHeader.spec.ts

@@ -1,138 +0,0 @@
-import {
-  comfyExpect as expect,
-  comfyPageFixture as test
-} from '../../fixtures/ComfyPage'
-import { VueNodeFixture } from '../../fixtures/utils/vueNodeFixtures'
-
-test.describe('NodeHeader', () => {
-  test.beforeEach(async ({ comfyPage }) => {
-    await comfyPage.setSetting('Comfy.UseNewMenu', 'Enabled')
-    await comfyPage.setSetting('Comfy.Graph.CanvasMenu', false)
-    await comfyPage.setSetting('Comfy.EnableTooltips', true)
-    await comfyPage.setup()
-    // Load single SaveImage node workflow (positioned below menu bar)
-    await comfyPage.loadWorkflow('single_save_image_node')
-  })
-
-  test('displays node title', async ({ comfyPage }) => {
-    // Get the single SaveImage node from the workflow
-    const nodes = await comfyPage.getNodeRefsByType('SaveImage')
-    expect(nodes.length).toBeGreaterThanOrEqual(1)
-
-    const node = nodes[0]
-    const vueNode = new VueNodeFixture(node, comfyPage.page)
-
-    const title = await vueNode.getTitle()
-    expect(title).toBe('Save Image')
-
-    // Verify title is visible in the header
-    const header = await vueNode.getHeader()
-    await expect(header).toContainText('Save Image')
-  })
-
-  test('allows title renaming', async ({ comfyPage }) => {
-    // Get the single SaveImage node from the workflow
-    const nodes = await comfyPage.getNodeRefsByType('SaveImage')
-    const node = nodes[0]
-    const vueNode = new VueNodeFixture(node, comfyPage.page)
-
-    // Test renaming with Enter
-    await vueNode.setTitle('My Custom Sampler')
-    const newTitle = await vueNode.getTitle()
-    expect(newTitle).toBe('My Custom Sampler')
-
-    // Verify the title is displayed
-    const header = await vueNode.getHeader()
-    await expect(header).toContainText('My Custom Sampler')
-
-    // Test cancel with Escape
-    const titleElement = await vueNode.getTitleElement()
-    await titleElement.dblclick()
-    await comfyPage.nextFrame()
-
-    // Type a different value but cancel
-    const input = (await vueNode.getHeader()).locator(
-      '[data-testid="node-title-input"]'
-    )
-    await input.fill('This Should Be Cancelled')
-    await input.press('Escape')
-    await comfyPage.nextFrame()
-
-    // Title should remain as the previously saved value
-    const titleAfterCancel = await vueNode.getTitle()
-    expect(titleAfterCancel).toBe('My Custom Sampler')
-  })
-
-  test('handles node collapsing', async ({ comfyPage }) => {
-    // Get the single SaveImage node from the workflow
-    const nodes = await comfyPage.getNodeRefsByType('SaveImage')
-    const node = nodes[0]
-    const vueNode = new VueNodeFixture(node, comfyPage.page)
-
-    // Initially should not be collapsed
-    expect(await node.isCollapsed()).toBe(false)
-    const body = await vueNode.getBody()
-    await expect(body).toBeVisible()
-
-    // Collapse the node
-    await vueNode.toggleCollapse()
-    expect(await node.isCollapsed()).toBe(true)
-
-    // Verify node content is hidden
-    const collapsedSize = await node.getSize()
-    await expect(body).not.toBeVisible()
-
-    // Expand again
-    await vueNode.toggleCollapse()
-    expect(await node.isCollapsed()).toBe(false)
-    await expect(body).toBeVisible()
-
-    // Size should be restored
-    const expandedSize = await node.getSize()
-    expect(expandedSize.height).toBeGreaterThanOrEqual(collapsedSize.height)
-  })
-
-  test('shows collapse/expand icon state', async ({ comfyPage }) => {
-    // Get the single SaveImage node from the workflow
-    const nodes = await comfyPage.getNodeRefsByType('SaveImage')
-    const node = nodes[0]
-    const vueNode = new VueNodeFixture(node, comfyPage.page)
-
-    // Check initial expanded state icon
-    let iconClass = await vueNode.getCollapseIconClass()
-    expect(iconClass).toContain('pi-chevron-down')
-
-    // Collapse and check icon
-    await vueNode.toggleCollapse()
-    iconClass = await vueNode.getCollapseIconClass()
-    expect(iconClass).toContain('pi-chevron-right')
-
-    // Expand and check icon
-    await vueNode.toggleCollapse()
-    iconClass = await vueNode.getCollapseIconClass()
-    expect(iconClass).toContain('pi-chevron-down')
-  })
-
-  test('preserves title when collapsing/expanding', async ({ comfyPage }) => {
-    // Get the single SaveImage node from the workflow
-    const nodes = await comfyPage.getNodeRefsByType('SaveImage')
-    const node = nodes[0]
-    const vueNode = new VueNodeFixture(node, comfyPage.page)
-
-    // Set custom title
-    await vueNode.setTitle('Test Sampler')
-    expect(await vueNode.getTitle()).toBe('Test Sampler')
-
-    // Collapse
-    await vueNode.toggleCollapse()
-    expect(await vueNode.getTitle()).toBe('Test Sampler')
-
-    // Expand
-    await vueNode.toggleCollapse()
-    expect(await vueNode.getTitle()).toBe('Test Sampler')
-
-    // Verify title is still displayed
-    const header = await vueNode.getHeader()
-    await expect(header).toContainText('Test Sampler')
-  })
-})

docs/adr/0001-merge-litegraph-into-frontend.md

@@ -1,65 +0,0 @@
-# 1. Merge LiteGraph.js into ComfyUI Frontend
-
-Date: 2025-08-05
-
-## Status
-
-Accepted
-
-## Context
-
-ComfyUI's frontend architecture currently depends on a forked version of litegraph.js maintained as a separate package (@comfyorg/litegraph). This separation has created several architectural and operational challenges:
-
-**Architectural Issues:**
-- The current split creates a distributed monolith where both packages handle rendering, user interactions, and data models without clear separation of responsibilities
-- Both frontend and litegraph manipulate the same data structures, forcing tight coupling across the frontend's data model, views, and business logic
-- The lack of clear boundaries prevents implementation of modern architectural patterns like MVC or event-sourcing
-
-**Operational Issues:**
-- ComfyUI is the only known user of the @comfyorg/litegraph fork
-- Managing separate repositories significantly slows developer velocity due to coordination overhead
-- Version mismatches between frontend and litegraph cause recurring issues
-- No upstream contributions to consider (original litegraph.js is no longer maintained)
-
-**Future Requirements:**
-The following planned features are blocked by the current architecture:
-- Multiplayer collaboration requiring CRDT-based state management
-- Cloud-based backend support
-- Alternative rendering backends
-- Improved undo/redo system
-- Clear API versioning and compatibility layers
-
-## Decision
-
-We will merge litegraph.js directly into the ComfyUI frontend repository using git subtree to preserve the complete commit history.
-
-The merge will:
-1. Move litegraph source to `src/lib/litegraph/`
-2. Update all import paths from `@comfyorg/litegraph` to `@/lib/litegraph`
-3. Remove the npm dependency on `@comfyorg/litegraph`
-4. Preserve the full git history using subtree merge
-
-This integration is the first step toward restructuring the application along clear Model-View-Controller boundaries, with state mutations going through a single CRDT-mediated access point.
-
-## Consequences
-
-### Positive
-
-- **Enables architectural refactoring**: Direct integration allows restructuring along proper MVC boundaries
-- **Unblocks new features**: Multiplayer, cloud features, and improved undo/redo can now be implemented
-- **Faster development**: Eliminates overhead of coordinating changes across two tightly-coupled packages
-- **Better developer experience**: No more version mismatch issues or cross-repository debugging
-- **Simplified maintenance**: One less repository to maintain, release, and version
-
-### Negative
-
-- **Larger repository**: The frontend repository will increase in size
-- **Loss of versioning**: No more semantic versioning for litegraph changes
-- **Maintenance responsibility**: Must maintain litegraph code directly
-- **Historical references**: Past commit messages may reference issues from the original litegraph repository
-
-## Notes
-
-- Git subtree was chosen over submodules to provide a cleaner developer experience
-- The original litegraph repository will be archived after the merge
-- Future litegraph improvements will be made directly in the frontend repository

docs/adr/0002-crdt-based-layout-system.md

@@ -1,111 +0,0 @@
-# 2. CRDT-Based Layout System
-
-Date: 2024-08-16
-
-## Status
-
-Accepted
-
-## Context
-
-ComfyUI's node graph editor faces fundamental architectural limitations that prevent us from achieving our product goals:
-
-### The Problem
-
-In the current system, each node manages its own position directly within LiteGraph. This creates several critical issues:
-
-1. **Performance Degradation**: Every UI update requires traversing the entire graph to detect changes. With graphs containing 100+ nodes, this polling-based approach causes visible lag during interactions.
-
-2. **Snap-Back Hell**: Multiple systems (LiteGraph canvas, Vue widgets, drag handlers) fight over node positions. Users experience frustrating "snap-back" where nodes jump between positions during drag operations.
-
-3. **No Collaboration Path**: Direct mutation of node positions makes real-time collaboration impossible. There's no way to merge concurrent edits from multiple users without conflicts.
-
-4. **Limited Renderer Options**: Position data is tightly coupled to LiteGraph's canvas renderer, blocking us from implementing WebGL rendering for large graphs or accessibility-focused DOM rendering.
-
-5. **Missing Features**: Without a proper event system, we can't implement undo/redo, animation systems, or viewport culling efficiently.
-
-### Why Now?
-
-- User complaints about performance with large workflows are increasing
-- The AI art community expects real-time collaboration (see Figma, Miro)
-- Accessibility requirements demand alternative rendering modes
-- The technical debt is compounding with each new feature
-
-## Decision
-
-We will implement a centralized layout tree using CRDT (Conflict-free Replicated Data Types) as the single source of truth for all spatial data.
-
-### Key Design Choices
-
-1. **CRDT-Based Layout Tree**: Use Yjs to maintain a centralized tree structure that owns all node positions, sizes, and spatial relationships.
-
-2. **Command Pattern**: Every position change is an explicit command/operation rather than direct mutation. This enables:
-   - Precise operation history for undo/redo
-   - Automatic conflict resolution for concurrent edits
-   - Event stream for observers without polling
-
-3. **Unidirectional Data Flow**: 
-   ```
-   User Input → Layout Commands → CRDT Tree → Renderers
-   ```
-   LiteGraph becomes a pure renderer that receives position updates, never mutates them.
-
-4. **Spatial Indexing**: The tree structure naturally supports a QuadTree spatial index for O(log n) viewport queries instead of O(n) full scans.
-
-### Why CRDT?
-
-CRDTs solve our core problems elegantly:
-- **Local-First**: Works perfectly for single-user while being collaboration-ready
-- **Automatic Conflict Resolution**: No more snap-back from competing updates
-- **Event-Driven**: Changes propagate through observers, not polling
-- **Memory Efficient**: Only changed portions of the tree are updated
-
-### Implementation Approach
-
-Phase 1: Build alongside existing system
-- Layout tree observes LiteGraph changes initially
-- Gradually migrate interactions to command pattern
-- Maintain full backwards compatibility
-
-Phase 2: Invert control
-- Layout tree becomes source of truth
-- LiteGraph receives updates via one-way sync
-- Enable alternative renderers
-
-## Consequences
-
-### Positive
-
-- **10x Performance**: Viewport culling and spatial indexing eliminate full graph traversals
-- **Multiplayer Ready**: CRDT foundation enables real-time collaboration without architecture changes
-- **Undo/Redo**: Command pattern makes history trivial to implement
-- **Renderer Flexibility**: Clean separation allows WebGL, DOM, or hybrid rendering
-- **Developer Experience**: Clear data flow and event system simplify debugging
-
-### Negative
-
-- **Learning Curve**: Team needs to understand CRDT concepts and command pattern
-- **Migration Complexity**: Existing code must be carefully migrated to new system
-- **Initial Memory Overhead**: ~30KB for Yjs library + operation history storage
-
-### Mitigations
-
-- Provide clear migration guides and examples
-- Build compatibility layer for gradual migration
-- Implement operation history pruning for long-running sessions
-
-## Notes
-
-This architecture aligns with modern state management patterns seen in Figma, Linear, and other collaborative tools. The investment in CRDT infrastructure pays dividends across multiple feature areas and positions ComfyUI as a modern, collaborative AI workflow tool.
-
-The command pattern also opens doors for:
-- Macro recording and playback
-- Automated testing of UI interactions  
-- Remote control via API
-- AI-assisted layout optimization
-
-## References
-
-- [Yjs Documentation](https://docs.yjs.dev/)
-- [CRDTs: The Hard Parts](https://martin.kleppmann.com/2020/07/06/crdt-hard-parts-hydra.html)
-- [Figma's Multiplayer Technology](https://www.figma.com/blog/how-figmas-multiplayer-technology-works/)

docs/adr/README.md

@@ -1,80 +0,0 @@
-# Architecture Decision Records
-
-This directory contains Architecture Decision Records (ADRs) for the ComfyUI Frontend project.
-
-## What is an ADR?
-
-An Architecture Decision Record captures an important architectural decision made along with its context and consequences. ADRs help future developers understand why certain decisions were made and provide a historical record of the project's evolution.
-
-## ADR Index
-
-| ADR | Title | Status | Date |
-|-----|-------|--------|------|
-| [0001](0001-merge-litegraph-into-frontend.md) | Merge LiteGraph.js into ComfyUI Frontend | Accepted | 2025-08-05 |
-| [0002](0002-crdt-based-layout-system.md) | CRDT-Based Layout System | Accepted | 2024-08-16 |
-
-## Creating a New ADR
-
-1. Copy the template below
-2. Name it with the next number in sequence: `NNNN-descriptive-title.md`
-3. Fill in all sections
-4. Update this index
-5. Submit as part of your PR
-
-## ADR Template
-
-```markdown
-# N. Title
-
-Date: YYYY-MM-DD
-
-## Status
-
-[Proposed | Accepted | Rejected | Deprecated | Superseded by [ADR-NNNN](NNNN-title.md)]
-
-## Context
-
-Describe the issue that motivated this decision and any context that influences or constrains the decision.
-
-- What is the problem?
-- Why does it need to be solved?
-- What forces are at play (technical, business, team)?
-
-## Decision
-
-Describe the decision that was made and the key points that led to it.
-
-- What are we going to do?
-- How will we do it?
-- What alternatives were considered?
-
-## Consequences
-
-### Positive
-
-- What becomes easier or better?
-- What opportunities does this create?
-
-### Negative
-
-- What becomes harder or worse?
-- What risks are we accepting?
-- What technical debt might we incur?
-
-## Notes
-
-Optional section for additional information, references, or clarifications.
-```
-
-## ADR Status Values
-
-- **Proposed**: The decision is being discussed
-- **Accepted**: The decision has been agreed upon
-- **Rejected**: The decision was not accepted
-- **Deprecated**: The decision is no longer relevant
-- **Superseded**: The decision has been replaced by another ADR
-
-## Further Reading
-
-- [Documenting Architecture Decisions](https://cognitect.com/blog/2011/11/15/documenting-architecture-decisions) by Michael Nygard
-- [Architecture Decision Records](https://adr.github.io/) - Collection of ADR resources

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comfyorg/comfyui-frontend",
   "private": true,
-  "version": "1.25.5",
+  "version": "1.24.1",
   "type": "module",
   "repository": "https://github.com/Comfy-Org/ComfyUI_frontend",
   "homepage": "https://comfy.org",
@@ -40,7 +40,6 @@
     "@types/fs-extra": "^11.0.4",
     "@types/lodash": "^4.17.6",
     "@types/node": "^20.14.8",
-    "@types/semver": "^7.7.0",
     "@types/three": "^0.169.0",
     "@vitejs/plugin-vue": "^5.1.4",
     "@vue/test-utils": "^2.4.6",
@@ -63,8 +62,8 @@
     "tsx": "^4.15.6",
     "typescript": "^5.4.5",
     "typescript-eslint": "^8.0.0",
-    "unplugin-icons": "^0.22.0",
-    "unplugin-vue-components": "^0.28.0",
+    "unplugin-icons": "^0.19.3",
+    "unplugin-vue-components": "^0.27.4",
     "vite": "^5.4.19",
     "vite-plugin-dts": "^4.3.0",
     "vite-plugin-html": "^3.2.2",
@@ -78,6 +77,7 @@
     "@alloc/quick-lru": "^5.2.0",
     "@atlaskit/pragmatic-drag-and-drop": "^1.3.1",
     "@comfyorg/comfyui-electron-types": "^0.4.43",
+    "@comfyorg/litegraph": "^0.16.16",
     "@primevue/forms": "^4.2.5",
     "@primevue/themes": "^4.2.5",
     "@sentry/vue": "^8.48.0",
@@ -94,11 +94,8 @@
     "@xterm/xterm": "^5.5.0",
     "algoliasearch": "^5.21.0",
     "axios": "^1.8.2",
-    "chart.js": "^4.5.0",
     "dompurify": "^3.2.5",
     "dotenv": "^16.4.5",
-    "extendable-media-recorder": "^9.2.27",
-    "extendable-media-recorder-wav-encoder": "^7.0.129",
     "firebase": "^11.6.0",
     "fuse.js": "^7.0.0",
     "jsondiffpatch": "^0.6.0",
@@ -108,14 +105,12 @@
     "pinia": "^2.1.7",
     "primeicons": "^7.0.0",
     "primevue": "^4.2.5",
-    "semver": "^7.7.2",
     "three": "^0.170.0",
     "tiptap-markdown": "^0.8.10",
     "vue": "^3.5.13",
     "vue-i18n": "^9.14.3",
     "vue-router": "^4.4.3",
     "vuefire": "^3.2.1",
-    "yjs": "^13.6.27",
     "zod": "^3.23.8",
     "zod-validation-error": "^3.3.0"
   }

scripts/prepare-types.js

@@ -16,7 +16,9 @@ const typesPackage = {
   homepage: mainPackage.homepage,
   description: `TypeScript definitions for ${mainPackage.name}`,
   license: mainPackage.license,
-  dependencies: {},
+  dependencies: {
+    '@comfyorg/litegraph': mainPackage.dependencies['@comfyorg/litegraph']
+  },
   peerDependencies: {
     vue: mainPackage.dependencies.vue,
     zod: mainPackage.dependencies.zod

src/CLAUDE.md

@@ -1,57 +0,0 @@
-# Source Code Guidelines
-
-## Service Layer
-
-### API Calls
-
-- Use `api.apiURL()` for backend endpoints
-- Use `api.fileURL()` for static files
-
-#### ✅ Correct Usage
-```typescript
-// Backend API call
-const response = await api.get(api.apiURL('/prompt'))
-
-// Static file
-const template = await fetch(api.fileURL('/templates/default.json'))
-```
-
-#### ❌ Incorrect Usage
-```typescript
-// WRONG - Direct URL construction
-const response = await fetch('/api/prompt')
-const template = await fetch('/templates/default.json')
-```
-
-### Error Handling
-
-- User-friendly and actionable messages
-- Proper error propagation
-
-### Security
-
-- Sanitize HTML with DOMPurify
-- Validate trusted sources
-- Never log secrets
-
-## State Management (Stores)
-
-### Store Design
-
-- Follow domain-driven design
-- Clear public interfaces
-- Restrict extension access
-
-### Best Practices
-
-- Use TypeScript for type safety
-- Implement proper error handling
-- Clean up subscriptions
-- Avoid @ts-expect-error
-
-## General Guidelines
-
-- Use lodash for utility functions
-- Implement proper TypeScript types
-- Follow Vue 3 composition API style guide
-- Use vue-i18n for ALL user-facing strings in `src/locales/en/main.json`

src/assets/css/style.css

@@ -5,7 +5,6 @@
   @tailwind utilities;
 }
 
-
 :root {
   --fg-color: #000;
   --bg-color: #fff;
@@ -135,188 +134,6 @@ body {
   border: thin solid;
 }
 
-/* Shared markdown content styling for consistent rendering across components */
-.comfy-markdown-content {
-  /* Typography */
-  font-size: 0.875rem; /* text-sm */
-  line-height: 1.6;
-  word-wrap: break-word;
-}
-
-/* Headings */
-.comfy-markdown-content h1 {
-  font-size: 22px; /* text-[22px] */
-  font-weight: 700; /* font-bold */
-  margin-top: 2rem; /* mt-8 */
-  margin-bottom: 1rem; /* mb-4 */
-}
-
-.comfy-markdown-content h1:first-child {
-  margin-top: 0; /* first:mt-0 */
-}
-
-.comfy-markdown-content h2 {
-  font-size: 18px; /* text-[18px] */
-  font-weight: 700; /* font-bold */
-  margin-top: 2rem; /* mt-8 */
-  margin-bottom: 1rem; /* mb-4 */
-}
-
-.comfy-markdown-content h2:first-child {
-  margin-top: 0; /* first:mt-0 */
-}
-
-.comfy-markdown-content h3 {
-  font-size: 16px; /* text-[16px] */
-  font-weight: 700; /* font-bold */
-  margin-top: 2rem; /* mt-8 */
-  margin-bottom: 1rem; /* mb-4 */
-}
-
-.comfy-markdown-content h3:first-child {
-  margin-top: 0; /* first:mt-0 */
-}
-
-.comfy-markdown-content h4,
-.comfy-markdown-content h5,
-.comfy-markdown-content h6 {
-  margin-top: 2rem; /* mt-8 */
-  margin-bottom: 1rem; /* mb-4 */
-}
-
-.comfy-markdown-content h4:first-child,
-.comfy-markdown-content h5:first-child,
-.comfy-markdown-content h6:first-child {
-  margin-top: 0; /* first:mt-0 */
-}
-
-/* Paragraphs */
-.comfy-markdown-content p {
-  margin: 0 0 0.5em;
-}
-
-.comfy-markdown-content p:last-child {
-  margin-bottom: 0;
-}
-
-/* First child reset */
-.comfy-markdown-content *:first-child {
-  margin-top: 0; /* mt-0 */
-}
-
-/* Lists */
-.comfy-markdown-content ul,
-.comfy-markdown-content ol {
-  padding-left: 2rem; /* pl-8 */
-  margin: 0.5rem 0; /* my-2 */
-}
-
-/* Nested lists */
-.comfy-markdown-content ul ul,
-.comfy-markdown-content ol ol,
-.comfy-markdown-content ul ol,
-.comfy-markdown-content ol ul {
-  padding-left: 1.5rem; /* pl-6 */
-  margin: 0.5rem 0; /* my-2 */
-}
-
-.comfy-markdown-content li {
-  margin: 0.5rem 0; /* my-2 */
-}
-
-/* Code */
-.comfy-markdown-content code {
-  color: var(--code-text-color);
-  background-color: var(--code-bg-color);
-  border-radius: 0.25rem; /* rounded */
-  padding: 0.125rem 0.375rem; /* px-1.5 py-0.5 */
-  font-family: monospace;
-}
-
-.comfy-markdown-content pre {
-  background-color: var(--code-block-bg-color);
-  border-radius: 0.25rem; /* rounded */
-  padding: 1rem; /* p-4 */
-  margin: 1rem 0; /* my-4 */
-  overflow-x: auto; /* overflow-x-auto */
-}
-
-.comfy-markdown-content pre code {
-  background-color: transparent; /* bg-transparent */
-  padding: 0; /* p-0 */
-  color: var(--p-text-color);
-}
-
-/* Tables */
-.comfy-markdown-content table {
-  width: 100%; /* w-full */
-  border-collapse: collapse; /* border-collapse */
-}
-
-.comfy-markdown-content th,
-.comfy-markdown-content td {
-  padding: 0.5rem; /* px-2 py-2 */
-}
-
-.comfy-markdown-content th {
-  color: var(--fg-color);
-}
-
-.comfy-markdown-content td {
-  color: var(--drag-text);
-}
-
-.comfy-markdown-content tr {
-  border-bottom: 1px solid var(--content-bg);
-}
-
-.comfy-markdown-content tr:last-child {
-  border-bottom: none;
-}
-
-.comfy-markdown-content thead {
-  border-bottom: 1px solid var(--p-text-color);
-}
-
-/* Links */
-.comfy-markdown-content a {
-  color: var(--drag-text);
-  text-decoration: underline;
-}
-
-/* Media */
-.comfy-markdown-content img,
-.comfy-markdown-content video {
-  max-width: 100%; /* max-w-full */
-  height: auto; /* h-auto */
-  display: block; /* block */
-  margin-bottom: 1rem; /* mb-4 */
-}
-
-/* Blockquotes */
-.comfy-markdown-content blockquote {
-  border-left: 3px solid var(--p-primary-color, var(--primary-bg));
-  padding-left: 0.75em;
-  margin: 0.5em 0;
-  opacity: 0.8;
-}
-
-/* Horizontal rule */
-.comfy-markdown-content hr {
-  border: none;
-  border-top: 1px solid var(--p-border-color, var(--border-color));
-  margin: 1em 0;
-}
-
-/* Strong and emphasis */
-.comfy-markdown-content strong {
-  font-weight: bold;
-}
-
-.comfy-markdown-content em {
-  font-style: italic;
-}
-
 .comfy-modal {
   display: none; /* Hidden by default */
   position: fixed; /* Stay in place */
@@ -820,92 +637,3 @@ audio.comfy-audio.empty-audio-widget {
   width: calc(100vw - env(titlebar-area-width, 100vw));
 }
 /* End of [Desktop] Electron window specific styles */
-
-/* Vue Node LOD (Level of Detail) System */
-/* These classes control rendering detail based on zoom level */
-
-/* Minimal LOD (zoom <= 0.4) - Title only for performance */
-.lg-node--lod-minimal {
-  min-height: 32px;
-  transition: min-height 0.2s ease;
-  /* Performance optimizations */
-  text-shadow: none;
-  backdrop-filter: none;
-}
-
-.lg-node--lod-minimal .lg-node-body {
-  display: none !important;
-}
-
-/* Reduced LOD (0.4 < zoom <= 0.8) - Essential widgets, simplified styling */
-.lg-node--lod-reduced {
-  transition: opacity 0.1s ease;
-  /* Performance optimizations */
-  text-shadow: none;
-}
-
-.lg-node--lod-reduced .lg-widget-label,
-.lg-node--lod-reduced .lg-slot-label {
-  display: none;
-}
-
-.lg-node--lod-reduced .lg-slot {
-  opacity: 0.6;
-  font-size: 0.75rem;
-}
-
-.lg-node--lod-reduced .lg-widget {
-  margin: 2px 0;
-  font-size: 0.875rem;
-}
-
-/* Full LOD (zoom > 0.8) - Complete detail rendering */
-.lg-node--lod-full {
-  /* Uses default styling - no overrides needed */
-}
-
-/* Smooth transitions between LOD levels */
-.lg-node {
-  transition: min-height 0.2s ease;
-  /* Disable text selection on all nodes */
-  user-select: none;
-  -webkit-user-select: none;
-  -moz-user-select: none;
-  -ms-user-select: none;
-}
-
-.lg-node .lg-slot,
-.lg-node .lg-widget {
-  transition: opacity 0.1s ease, font-size 0.1s ease;
-}
-
-/* Performance optimization during canvas interaction */
-.transform-pane--interacting .lg-node * {
-  transition: none !important;
-}
-
-.transform-pane--interacting .lg-node {
-  will-change: transform;
-}
-
-/* Global performance optimizations for LOD */
-.lg-node--lod-minimal,
-.lg-node--lod-reduced {
-  /* Remove ALL expensive paint effects */
-  box-shadow: none !important;
-  filter: none !important;
-  backdrop-filter: none !important;
-  text-shadow: none !important;
-  -webkit-mask-image: none !important;
-  mask-image: none !important;
-  clip-path: none !important;
-}
-
-/* Reduce paint complexity for minimal LOD */
-.lg-node--lod-minimal {
-  /* Skip complex borders */
-  border-radius: 0 !important;
-  /* Use solid colors only */
-  background-image: none !important;
-}
-

src/assets/icons/README.md

@@ -1,184 +0,0 @@
-# ComfyUI Custom Icons Guide
-
-This guide explains how to add and use custom SVG icons in the ComfyUI frontend.
-
-## Overview
-
-ComfyUI uses a hybrid icon system that supports:
-- **PrimeIcons** - Legacy icon library (CSS classes like `pi pi-plus`)
-- **Iconify** - Modern icon system with 200,000+ icons
-- **Custom Icons** - Your own SVG icons
-
-Custom icons are powered by [unplugin-icons](https://github.com/unplugin/unplugin-icons) and integrate seamlessly with Vue's component system.
-
-## Quick Start
-
-### 1. Add Your SVG Icon
-
-Place your SVG file in the `custom/` directory:
-```
-src/assets/icons/custom/
-└── your-icon.svg
-```
-
-### 2. Use in Components
-
-```vue
-<template>
-  <!-- Use as a Vue component -->
-  <i-comfy:your-icon />
-  
-  <!-- In a PrimeVue button -->
-  <Button>
-    <template #icon>
-      <i-comfy:your-icon />
-    </template>
-  </Button>
-</template>
-```
-
-## SVG Requirements
-
-### File Naming
-- Use kebab-case: `workflow-icon.svg`, `node-tree.svg`
-- Avoid special characters and spaces
-- The filename becomes the icon name
-
-### SVG Format
-```xml
-<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg">
-  <path d="..." />
-</svg>
-```
-
-**Important:**
-- Use `viewBox` for proper scaling (24x24 is standard)
-- Don't include `width` or `height` attributes
-- Use `currentColor` for theme-aware icons
-- Keep SVGs optimized and simple
-
-### Color Theming
-
-For icons that adapt to the current theme, use `currentColor`:
-
-```xml
-<!-- ✅ Good: Uses currentColor -->
-<svg viewBox="0 0 24 24">
-  <path stroke="currentColor" fill="none" d="..." />
-</svg>
-
-<!-- ❌ Bad: Hardcoded colors -->
-<svg viewBox="0 0 24 24">
-  <path stroke="white" fill="black" d="..." />
-</svg>
-```
-
-## Usage Examples
-
-### Basic Icon
-```vue
-<i-comfy:workflow />
-```
-
-### With Classes
-```vue
-<i-comfy:workflow class="text-2xl text-blue-500" />
-```
-
-### In Buttons
-```vue
-<Button severity="secondary" text>
-  <template #icon>
-    <i-comfy:workflow />
-  </template>
-</Button>
-```
-
-### Conditional Icons
-```vue
-<template #icon>
-  <i-comfy:workflow v-if="isWorkflow" />
-  <i-comfy:node v-else />
-</template>
-```
-
-## Technical Details
-
-### How It Works
-
-1. **unplugin-icons** automatically discovers SVG files in `custom/`
-2. During build, SVGs are converted to Vue components
-3. Components are tree-shaken - only used icons are bundled
-4. The `i-` prefix and `comfy:` namespace identify custom icons
-
-### Configuration
-
-The icon system is configured in `vite.config.mts`:
-
-```typescript
-Icons({
-  compiler: 'vue3',
-  customCollections: {
-    'comfy': FileSystemIconLoader('src/assets/icons/custom'),
-  }
-})
-```
-
-### TypeScript Support
-
-Icons are automatically typed. If TypeScript doesn't recognize a new icon:
-1. Restart your dev server
-2. Check that the SVG file is valid
-3. Ensure the filename follows kebab-case convention
-
-## Troubleshooting
-
-### Icon Not Showing
-1. **Check filename**: Must be kebab-case without special characters
-2. **Restart dev server**: Required after adding new icons
-3. **Verify SVG**: Ensure it's valid SVG syntax
-4. **Check console**: Look for Vue component resolution errors
-
-### Icon Wrong Color
-- Replace hardcoded colors with `currentColor`
-- Use `stroke="currentColor"` for outlines
-- Use `fill="currentColor"` for filled shapes
-
-### Icon Wrong Size
-- Remove `width` and `height` from SVG
-- Ensure `viewBox` is present
-- Use CSS classes for sizing: `class="w-6 h-6"`
-
-## Best Practices
-
-1. **Optimize SVGs**: Use tools like [SVGO](https://jakearchibald.github.io/svgomg/) to minimize file size
-2. **Consistent viewBox**: Stick to 24x24 or 16x16 for consistency
-3. **Semantic names**: Use descriptive names like `workflow-duplicate` not `icon1`
-4. **Theme support**: Always use `currentColor` for adaptable icons
-5. **Test both themes**: Verify icons look good in light and dark modes
-
-## Migration from PrimeIcons
-
-When replacing a PrimeIcon with a custom icon:
-
-```vue
-<!-- Before: PrimeIcon -->
-<Button icon="pi pi-box" />
-
-<!-- After: Custom icon -->
-<Button>
-  <template #icon>
-    <i-comfy:workflow />
-  </template>
-</Button>
-```
-
-## Adding Icon Collections
-
-To add an entire icon set from npm:
-
-1. Install the icon package
-2. Configure in `vite.config.mts`
-3. Use with the appropriate prefix
-
-See the [unplugin-icons documentation](https://github.com/unplugin/unplugin-icons) for details.

src/assets/icons/custom/workflow.svg

@@ -1,7 +0,0 @@
-<svg width="16" height="16" viewBox="0 0 16 16" fill="none" xmlns="http://www.w3.org/2000/svg">
-<path d="M14 5V3C14 2.44772 13.5523 2 13 2H11C10.4477 2 10 2.44772 10 3V5C10 5.55228 10.4477 6 11 6H13C13.5523 6 14 5.55228 14 5Z" stroke="currentColor" stroke-width="1.3" stroke-linecap="round"/>
-<path d="M6 5V3C6 2.44772 5.55228 2 5 2H3C2.44772 2 2 2.44772 2 3V5C2 5.55228 2.44772 6 3 6H5C5.55228 6 6 5.55228 6 5Z" stroke="currentColor" stroke-width="1.3" stroke-linecap="round"/>
-<path d="M14 13V11C14 10.4477 13.5523 10 13 10H11C10.4477 10 10 10.4477 10 11V13C10 13.5523 10.4477 14 11 14H13C13.5523 14 14 13.5523 14 13Z" stroke="currentColor" stroke-width="1.3" stroke-linecap="round"/>
-<path d="M10 4H6" stroke="currentColor" stroke-width="1.3" stroke-linecap="round"/>
-<path d="M10 12H8C5.79086 12 4 10.2091 4 8V6" stroke="currentColor" stroke-width="1.3" stroke-linecap="round"/>
-</svg>

src/components/CLAUDE.md

@@ -1,45 +0,0 @@
-# Component Guidelines
-
-## Vue 3 Composition API
-
-- Use setup() function
-- Destructure props (Vue 3.5 style)
-- Use ref/reactive for state
-- Implement computed() for derived state
-- Use provide/inject for dependency injection
-
-## Component Communication
-
-- Prefer `emit/@event-name` for state changes
-- Use `defineExpose` only for imperative operations (`form.validate()`, `modal.open()`)
-- Events promote loose coupling
-
-## UI Framework
-
-- Deprecated PrimeVue component replacements:
-  - Dropdown → Select
-  - OverlayPanel → Popover
-  - Calendar → DatePicker
-  - InputSwitch → ToggleSwitch
-  - Sidebar → Drawer
-  - Chips → AutoComplete with multiple enabled
-  - TabMenu → Tabs without panels
-  - Steps → Stepper without panels
-  - InlineMessage → Message
-
-## Styling
-
-- Use Tailwind CSS only (no custom CSS)
-- Dark theme: use "dark-theme:" prefix
-- For common operations, try to use existing VueUse composables that automatically handle effect scope
-  - Example: Use `useElementHover` instead of manually managing mouseover/mouseout event listeners
-  - Example: Use `useIntersectionObserver` for visibility detection instead of custom scroll handlers
-
-## Best Practices
-
-- Extract complex conditionals to computed
-- Implement cleanup for async operations
-- Use vue-i18n for ALL UI strings
-- Use lifecycle hooks: onMounted, onUpdated
-- Use Teleport/Suspense when needed
-- Proper props and emits definitions

src/components/actionbar/ComfyActionbar.vue

@@ -30,11 +30,10 @@ import ComfyQueueButton from './ComfyQueueButton.vue'
 
 const settingsStore = useSettingStore()
 
-const position = computed(() => settingsStore.get('Comfy.UseNewMenu'))
+const visible = computed(
+  () => settingsStore.get('Comfy.UseNewMenu') !== 'Disabled'
+)
 
-const visible = computed(() => position.value !== 'Disabled')
-
-const topMenuRef = inject<Ref<HTMLDivElement | null>>('topMenuRef')
 const panelRef = ref<HTMLElement | null>(null)
 const dragHandleRef = ref<HTMLElement | null>(null)
 const isDocked = useLocalStorage('Comfy.MenuPosition.Docked', false)
@@ -50,16 +49,7 @@ const {
 } = useDraggable(panelRef, {
   initialValue: { x: 0, y: 0 },
   handle: dragHandleRef,
-  containerElement: document.body,
-  onMove: (event) => {
-    // Prevent dragging the menu over the top of the tabs
-    if (position.value === 'Top') {
-      const minY = topMenuRef?.value?.getBoundingClientRect().top ?? 40
-      if (event.y < minY) {
-        event.y = minY
-      }
-    }
-  }
+  containerElement: document.body
 })
 
 // Update storedPosition when x or y changes
@@ -192,6 +182,7 @@ const adjustMenuPosition = () => {
 
 useEventListener(window, 'resize', adjustMenuPosition)
 
+const topMenuRef = inject<Ref<HTMLDivElement | null>>('topMenuRef')
 const topMenuBounds = useElementBounding(topMenuRef)
 const overlapThreshold = 20 // pixels
 const isOverlappingWithTopMenu = computed(() => {

src/components/breadcrumb/SubgraphBreadcrumb.vue

@@ -1,95 +1,47 @@
 <template>
-  <div
-    class="subgraph-breadcrumb w-auto"
-    :class="{
-      'subgraph-breadcrumb-collapse': collapseTabs,
-      'subgraph-breadcrumb-overflow': overflowingTabs
-    }"
-    :style="{
-      '--p-breadcrumb-gap': `${ITEM_GAP}px`,
-      '--p-breadcrumb-item-min-width': `${MIN_WIDTH}px`,
-      '--p-breadcrumb-item-padding': `${ITEM_PADDING}px`,
-      '--p-breadcrumb-icon-width': `${ICON_WIDTH}px`
-    }"
-  >
+  <div v-if="workflowStore.isSubgraphActive" class="p-2 subgraph-breadcrumb">
     <Breadcrumb
-      ref="breadcrumbRef"
-      class="bg-transparent p-0"
+      class="bg-transparent"
+      :home="home"
       :model="items"
       aria-label="Graph navigation"
-    >
-      <template #item="{ item }">
-        <SubgraphBreadcrumbItem
-          :item="item"
-          :is-active="item === items.at(-1)"
-        />
-      </template>
-      <template #separator
-        ><span style="transform: scale(1.5)"> / </span></template
-      >
-    </Breadcrumb>
+      @item-click="handleItemClick"
+    />
   </div>
 </template>
 
 <script setup lang="ts">
 import { useEventListener } from '@vueuse/core'
 import Breadcrumb from 'primevue/breadcrumb'
-import type { MenuItem } from 'primevue/menuitem'
-import { computed, onUpdated, ref, watch } from 'vue'
+import type { MenuItem, MenuItemCommandEvent } from 'primevue/menuitem'
+import { computed } from 'vue'
 
-import SubgraphBreadcrumbItem from '@/components/breadcrumb/SubgraphBreadcrumbItem.vue'
-import { useOverflowObserver } from '@/composables/element/useOverflowObserver'
 import { useCanvasStore } from '@/stores/graphStore'
 import { useSubgraphNavigationStore } from '@/stores/subgraphNavigationStore'
 import { useWorkflowStore } from '@/stores/workflowStore'
-import { forEachSubgraphNode } from '@/utils/graphTraversalUtil'
-
-const MIN_WIDTH = 28
-const ITEM_GAP = 8
-const ITEM_PADDING = 8
-const ICON_WIDTH = 20
 
 const workflowStore = useWorkflowStore()
 const navigationStore = useSubgraphNavigationStore()
-const breadcrumbRef = ref<InstanceType<typeof Breadcrumb>>()
+
 const workflowName = computed(() => workflowStore.activeWorkflow?.filename)
-const collapseTabs = ref(false)
-const overflowingTabs = ref(false)
-
-const breadcrumbElement = computed(() => {
-  if (!breadcrumbRef.value) return null
-
-  const el = (breadcrumbRef.value as unknown as { $el: HTMLElement }).$el
-  const list = el?.querySelector('.p-breadcrumb-list') as HTMLElement
-  return list
-})
 
 const items = computed(() => {
-  const items = navigationStore.navigationStack.map<MenuItem>((subgraph) => ({
+  if (!navigationStore.navigationStack.length) return []
+
+  return navigationStore.navigationStack.map<MenuItem>((subgraph) => ({
     label: subgraph.name,
     command: () => {
       const canvas = useCanvasStore().getCanvas()
       if (!canvas.graph) throw new TypeError('Canvas has no graph')
 
       canvas.setGraph(subgraph)
-    },
-    updateTitle: (title: string) => {
-      const rootGraph = useCanvasStore().getCanvas().graph?.rootGraph
-      if (!rootGraph) return
-
-      forEachSubgraphNode(rootGraph, subgraph.id, (node) => {
-        node.title = title
-      })
     }
   }))
-
-  return [home.value, ...items]
 })
 
 const home = computed(() => ({
   label: workflowName.value,
   icon: 'pi pi-home',
-  key: 'root',
   command: () => {
     const canvas = useCanvasStore().getCanvas()
     if (!canvas.graph) throw new TypeError('Canvas has no graph')
@@ -98,6 +50,10 @@ const home = computed(() => ({
   }
 }))
 
+const handleItemClick = (event: MenuItemCommandEvent) => {
+  event.item.command?.(event)
+}
+
 // Escape exits from the current subgraph.
 useEventListener(document, 'keydown', (event) => {
   if (event.key === 'Escape') {
@@ -109,116 +65,21 @@ useEventListener(document, 'keydown', (event) => {
     )
   }
 })
-
-// Check for overflow on breadcrumb items and collapse/expand the breadcrumb to fit
-let overflowObserver: ReturnType<typeof useOverflowObserver> | undefined
-watch(breadcrumbElement, (el) => {
-  overflowObserver?.dispose()
-  overflowObserver = undefined
-
-  if (!el) return
-
-  overflowObserver = useOverflowObserver(el, {
-    onCheck: (isOverflowing) => {
-      overflowingTabs.value = isOverflowing
-
-      if (collapseTabs.value) {
-        // Items are currently hidden, check if we can show them
-        if (!isOverflowing) {
-          const items = [
-            ...el.querySelectorAll('.p-breadcrumb-item')
-          ] as HTMLElement[]
-
-          if (items.length < 3) return
-
-          const itemsWithIcon = items.filter((item) =>
-            item.querySelector('.p-breadcrumb-item-link-icon-visible')
-          ).length
-          const separators = el.querySelectorAll(
-            '.p-breadcrumb-separator'
-          ) as NodeListOf<HTMLElement>
-          const separator = separators[separators.length - 1] as HTMLElement
-          const separatorWidth = separator.offsetWidth
-
-          // items + separators + gaps + icons
-          const itemsWidth =
-            (MIN_WIDTH + ITEM_PADDING + ITEM_PADDING) * items.length +
-            itemsWithIcon * ICON_WIDTH
-          const separatorsWidth = (items.length - 1) * separatorWidth
-          const gapsWidth = (items.length - 1) * (ITEM_GAP * 2)
-          const totalWidth = itemsWidth + separatorsWidth + gapsWidth
-          const containerWidth = el.clientWidth
-
-          if (totalWidth <= containerWidth) {
-            collapseTabs.value = false
-          }
-        }
-      } else if (isOverflowing) {
-        collapseTabs.value = true
-      }
-    }
-  })
-})
-
-// If e.g. the workflow name changes, we need to check the overflow again
-onUpdated(() => {
-  if (!overflowObserver?.disposed.value) {
-    overflowObserver?.checkOverflow()
-  }
-})
 </script>
 
-<style scoped>
-.subgraph-breadcrumb:not(:empty) {
-  flex: auto;
-  flex-shrink: 10000;
-  min-width: 120px;
-}
-
-.subgraph-breadcrumb,
-:deep(.p-breadcrumb) {
-  @apply overflow-hidden;
-}
-
-:deep(.p-breadcrumb-item) {
-  @apply flex items-center rounded-lg overflow-hidden;
-  min-width: calc(var(--p-breadcrumb-item-min-width) + 1rem);
-  /* Collapse middle items first */
-  flex-shrink: 10000;
-}
-
-:deep(.p-breadcrumb-item:has(.p-breadcrumb-item-link-icon-visible)) {
-  min-width: calc(var(--p-breadcrumb-item-min-width) + 1rem + 20px);
-}
-
-:deep(.p-breadcrumb-item:first-child) {
-  /* Then collapse the root workflow */
-  flex-shrink: 5000;
-}
-
-:deep(.p-breadcrumb-item:last-child) {
-  /* Then collapse the active item */
-  flex-shrink: 1;
-}
-
-:deep(.p-breadcrumb-item:hover),
-:deep(.p-breadcrumb-item:has(.p-breadcrumb-item-link-menu-visible)) {
-  background-color: color-mix(in srgb, var(--fg-color) 10%, transparent);
-  color: var(--fg-color);
-}
-</style>
-
 <style>
-.subgraph-breadcrumb-collapse .p-breadcrumb-list {
-  .p-breadcrumb-item,
-  .p-breadcrumb-separator {
-    @apply hidden;
-  }
+.subgraph-breadcrumb {
+  .p-breadcrumb-item-link,
+  .p-breadcrumb-item-icon {
+    @apply select-none;
 
-  .p-breadcrumb-item:nth-last-child(3),
-  .p-breadcrumb-separator:nth-last-child(2),
-  .p-breadcrumb-item:nth-last-child(1) {
-    @apply block;
+    color: #d26565;
+    text-shadow:
+      1px 1px 0 #000,
+      -1px -1px 0 #000,
+      1px -1px 0 #000,
+      -1px 1px 0 #000,
+      0 0 0.375rem #000;
   }
 }
 </style>

src/components/breadcrumb/SubgraphBreadcrumbItem.vue

@@ -1,215 +0,0 @@
-<template>
-  <a
-    ref="wrapperRef"
-    v-tooltip.bottom="{
-      value: item.label,
-      showDelay: 512
-    }"
-    href="#"
-    class="cursor-pointer p-breadcrumb-item-link"
-    :class="{
-      'flex items-center gap-1': isActive,
-      'p-breadcrumb-item-link-menu-visible': menu?.overlayVisible,
-      'p-breadcrumb-item-link-icon-visible': isActive
-    }"
-    @click="handleClick"
-  >
-    <span class="p-breadcrumb-item-label">{{ item.label }}</span>
-    <i v-if="isActive" class="pi pi-angle-down text-[10px]"></i>
-  </a>
-  <Menu
-    v-if="isActive"
-    ref="menu"
-    :model="menuItems"
-    :popup="true"
-    :pt="{
-      root: {
-        style: 'background-color: var(--comfy-menu-secondary-bg)'
-      },
-      itemLink: {
-        class: 'py-2'
-      }
-    }"
-  />
-  <InputText
-    v-if="isEditing"
-    ref="itemInputRef"
-    v-model="itemLabel"
-    class="fixed z-[10000] text-[.8rem] px-2 py-2"
-    @blur="inputBlur(true)"
-    @click.stop
-    @keydown.enter="inputBlur(true)"
-    @keydown.esc="inputBlur(false)"
-  />
-</template>
-
-<script setup lang="ts">
-import InputText from 'primevue/inputtext'
-import Menu, { MenuState } from 'primevue/menu'
-import type { MenuItem } from 'primevue/menuitem'
-import { computed, nextTick, ref } from 'vue'
-import { useI18n } from 'vue-i18n'
-
-import { useDialogService } from '@/services/dialogService'
-import { useWorkflowService } from '@/services/workflowService'
-import { useCommandStore } from '@/stores/commandStore'
-import { useSubgraphNavigationStore } from '@/stores/subgraphNavigationStore'
-import { ComfyWorkflow, useWorkflowStore } from '@/stores/workflowStore'
-import { appendJsonExt } from '@/utils/formatUtil'
-
-interface Props {
-  item: MenuItem
-  isActive?: boolean
-}
-
-const props = withDefaults(defineProps<Props>(), {
-  isActive: false
-})
-
-const { t } = useI18n()
-const menu = ref<InstanceType<typeof Menu> & MenuState>()
-const dialogService = useDialogService()
-const workflowStore = useWorkflowStore()
-const workflowService = useWorkflowService()
-const isEditing = ref(false)
-const itemLabel = ref<string>()
-const itemInputRef = ref<{ $el?: HTMLInputElement }>()
-const wrapperRef = ref<HTMLAnchorElement>()
-
-const rename = async (
-  newName: string | null | undefined,
-  initialName: string
-) => {
-  if (newName && newName !== initialName) {
-    // Synchronize the node titles with the new name
-    props.item.updateTitle?.(newName)
-
-    if (workflowStore.activeSubgraph) {
-      workflowStore.activeSubgraph.name = newName
-    } else if (workflowStore.activeWorkflow) {
-      try {
-        await workflowService.renameWorkflow(
-          workflowStore.activeWorkflow,
-          ComfyWorkflow.basePath + appendJsonExt(newName)
-        )
-      } catch (error) {
-        console.error(error)
-        dialogService.showErrorDialog(error)
-        return
-      }
-    }
-
-    // Force the navigation stack to recompute the labels
-    // TODO: investigate if there is a better way to do this
-    const navigationStore = useSubgraphNavigationStore()
-    navigationStore.restoreState(navigationStore.exportState())
-  }
-}
-
-const menuItems = computed<MenuItem[]>(() => {
-  return [
-    {
-      label: t('g.rename'),
-      icon: 'pi pi-pencil',
-      command: async () => {
-        let initialName =
-          workflowStore.activeSubgraph?.name ??
-          workflowStore.activeWorkflow?.filename
-
-        if (!initialName) return
-
-        const newName = await dialogService.prompt({
-          title: t('g.rename'),
-          message: t('breadcrumbsMenu.enterNewName'),
-          defaultValue: initialName
-        })
-
-        await rename(newName, initialName)
-      }
-    },
-    {
-      label: t('breadcrumbsMenu.duplicate'),
-      icon: 'pi pi-copy',
-      command: async () => {
-        await workflowService.duplicateWorkflow(workflowStore.activeWorkflow!)
-      },
-      visible: props.item.key === 'root'
-    },
-    {
-      separator: true
-    },
-    {
-      label: t('breadcrumbsMenu.clearWorkflow'),
-      icon: 'pi pi-trash',
-      command: async () => {
-        await useCommandStore().execute('Comfy.ClearWorkflow')
-      }
-    },
-    {
-      separator: true,
-      visible: props.item.key === 'root'
-    },
-    {
-      label: t('breadcrumbsMenu.deleteWorkflow'),
-      icon: 'pi pi-times',
-      command: async () => {
-        await workflowService.deleteWorkflow(workflowStore.activeWorkflow!)
-      },
-      visible: props.item.key === 'root'
-    }
-  ]
-})
-
-const handleClick = (event: MouseEvent) => {
-  if (isEditing.value) {
-    return
-  }
-
-  if (event.detail === 1) {
-    if (props.isActive) {
-      menu.value?.toggle(event)
-    } else {
-      props.item.command?.({ item: props.item, originalEvent: event })
-    }
-  } else if (props.isActive && event.detail === 2) {
-    menu.value?.hide()
-    event.stopPropagation()
-    event.preventDefault()
-    isEditing.value = true
-    itemLabel.value = props.item.label as string
-    void nextTick(() => {
-      if (itemInputRef.value?.$el) {
-        itemInputRef.value.$el.focus()
-        itemInputRef.value.$el.select()
-        if (wrapperRef.value) {
-          itemInputRef.value.$el.style.width = `${Math.max(200, wrapperRef.value.offsetWidth)}px`
-        }
-      }
-    })
-  }
-}
-
-const inputBlur = async (doRename: boolean) => {
-  if (doRename) {
-    await rename(itemLabel.value, props.item.label as string)
-  }
-
-  isEditing.value = false
-}
-</script>
-
-<style scoped>
-.p-breadcrumb-item-link,
-.p-breadcrumb-item-icon {
-  @apply select-none;
-}
-
-.p-breadcrumb-item-link {
-  @apply overflow-hidden;
-  padding: var(--p-breadcrumb-item-padding);
-}
-
-.p-breadcrumb-item-label {
-  @apply whitespace-nowrap text-ellipsis overflow-hidden;
-}
-</style>

src/components/common/EditableText.spec.ts

@@ -68,73 +68,4 @@ describe('EditableText', () => {
     // @ts-expect-error fixme ts strict error
     expect(wrapper.emitted('edit')[0]).toEqual(['Test Text'])
   })
-
-  it('cancels editing on escape key', async () => {
-    const wrapper = mountComponent({
-      modelValue: 'Original Text',
-      isEditing: true
-    })
-
-    // Change the input value
-    await wrapper.findComponent(InputText).setValue('Modified Text')
-
-    // Press escape
-    await wrapper.findComponent(InputText).trigger('keyup.escape')
-
-    // Should emit cancel event
-    expect(wrapper.emitted('cancel')).toBeTruthy()
-
-    // Should NOT emit edit event
-    expect(wrapper.emitted('edit')).toBeFalsy()
-
-    // Input value should be reset to original
-    expect(wrapper.findComponent(InputText).props()['modelValue']).toBe(
-      'Original Text'
-    )
-  })
-
-  it('does not save changes when escape is pressed and blur occurs', async () => {
-    const wrapper = mountComponent({
-      modelValue: 'Original Text',
-      isEditing: true
-    })
-
-    // Change the input value
-    await wrapper.findComponent(InputText).setValue('Modified Text')
-
-    // Press escape (which triggers blur internally)
-    await wrapper.findComponent(InputText).trigger('keyup.escape')
-
-    // Manually trigger blur to simulate the blur that happens after escape
-    await wrapper.findComponent(InputText).trigger('blur')
-
-    // Should emit cancel but not edit
-    expect(wrapper.emitted('cancel')).toBeTruthy()
-    expect(wrapper.emitted('edit')).toBeFalsy()
-  })
-
-  it('saves changes on enter but not on escape', async () => {
-    // Test Enter key saves changes
-    const enterWrapper = mountComponent({
-      modelValue: 'Original Text',
-      isEditing: true
-    })
-    await enterWrapper.findComponent(InputText).setValue('Saved Text')
-    await enterWrapper.findComponent(InputText).trigger('keyup.enter')
-    // Trigger blur that happens after enter
-    await enterWrapper.findComponent(InputText).trigger('blur')
-    expect(enterWrapper.emitted('edit')).toBeTruthy()
-    // @ts-expect-error fixme ts strict error
-    expect(enterWrapper.emitted('edit')[0]).toEqual(['Saved Text'])
-
-    // Test Escape key cancels changes with a fresh wrapper
-    const escapeWrapper = mountComponent({
-      modelValue: 'Original Text',
-      isEditing: true
-    })
-    await escapeWrapper.findComponent(InputText).setValue('Cancelled Text')
-    await escapeWrapper.findComponent(InputText).trigger('keyup.escape')
-    expect(escapeWrapper.emitted('cancel')).toBeTruthy()
-    expect(escapeWrapper.emitted('edit')).toBeFalsy()
-  })
 })

src/components/common/EditableText.vue

@@ -14,12 +14,10 @@
       fluid
       :pt="{
         root: {
-          onBlur: finishEditing,
-          ...inputAttrs
+          onBlur: finishEditing
         }
       }"
       @keyup.enter="blurInputElement"
-      @keyup.escape="cancelEditing"
       @click.stop
     />
   </div>
@@ -29,41 +27,21 @@
 import InputText from 'primevue/inputtext'
 import { nextTick, ref, watch } from 'vue'
 
-const {
-  modelValue,
-  isEditing = false,
-  inputAttrs = {}
-} = defineProps<{
+const { modelValue, isEditing = false } = defineProps<{
   modelValue: string
   isEditing?: boolean
-  inputAttrs?: Record<string, any>
 }>()
 
-const emit = defineEmits(['update:modelValue', 'edit', 'cancel'])
+const emit = defineEmits(['update:modelValue', 'edit'])
 const inputValue = ref<string>(modelValue)
 const inputRef = ref<InstanceType<typeof InputText> | undefined>()
-const isCanceling = ref(false)
 
 const blurInputElement = () => {
   // @ts-expect-error - $el is an internal property of the InputText component
   inputRef.value?.$el.blur()
 }
 const finishEditing = () => {
-  // Don't save if we're canceling
-  if (!isCanceling.value) {
-    emit('edit', inputValue.value)
-  }
-  isCanceling.value = false
-}
-const cancelEditing = () => {
-  // Set canceling flag to prevent blur from saving
-  isCanceling.value = true
-  // Reset to original value
-  inputValue.value = modelValue
-  // Emit cancel event
-  emit('cancel')
-  // Blur the input to exit edit mode
-  blurInputElement()
+  emit('edit', inputValue.value)
 }
 watch(
   () => isEditing,

src/components/dialog/content/LoadWorkflowWarning.vue

@@ -2,7 +2,7 @@
   <NoResultsPlaceholder
     class="pb-0"
     icon="pi pi-exclamation-circle"
-    title="Some Nodes Are Missing"
+    title="Missing Node Types"
     message="When loading the graph, the following node types were not found"
   />
   <MissingCoreNodesMessage :missing-core-nodes="missingCoreNodes" />

src/components/dialog/content/MissingCoreNodesMessage.vue

@@ -42,11 +42,11 @@
 </template>
 
 <script setup lang="ts">
+import type { LGraphNode } from '@comfyorg/litegraph'
 import { whenever } from '@vueuse/core'
 import Message from 'primevue/message'
 import { computed, ref } from 'vue'
 
-import type { LGraphNode } from '@/lib/litegraph/src/litegraph'
 import { useSystemStatsStore } from '@/stores/systemStatsStore'
 import { compareVersions } from '@/utils/formatUtil'
 

src/components/graph/GraphCanvas.vue

@@ -16,14 +16,9 @@
         <SecondRowWorkflowTabs
           v-if="workflowTabsPosition === 'Topbar (2nd-row)'"
         />
+        <SubgraphBreadcrumb />
       </div>
       <GraphCanvasMenu v-if="canvasMenuEnabled" class="pointer-events-auto" />
-
-      <MiniMap
-        v-if="comfyAppReady && minimapEnabled"
-        ref="minimapRef"
-        class="pointer-events-auto"
-      />
     </template>
   </LiteGraphCanvasSplitterOverlay>
   <GraphCanvasMenu v-if="!betaMenuEnabled && canvasMenuEnabled" />
@@ -34,55 +29,6 @@
     class="w-full h-full touch-none"
   />
 
-  <!-- TransformPane for Vue node rendering (development) -->
-  <TransformPane
-    v-if="transformPaneEnabled && canvasStore.canvas && comfyAppReady"
-    :canvas="canvasStore.canvas as LGraphCanvas"
-    :viewport="canvasViewport"
-    :show-debug-overlay="showPerformanceOverlay"
-    @raf-status-change="rafActive = $event"
-    @transform-update="handleTransformUpdate"
-  >
-    <!-- Vue nodes rendered based on graph nodes -->
-    <VueGraphNode
-      v-for="nodeData in nodesToRender"
-      :key="nodeData.id"
-      :node-data="nodeData"
-      :position="nodePositions.get(nodeData.id)"
-      :size="nodeSizes.get(nodeData.id)"
-      :selected="nodeData.selected"
-      :readonly="false"
-      :executing="executionStore.executingNodeId === nodeData.id"
-      :error="
-        executionStore.lastExecutionError?.node_id === nodeData.id
-          ? 'Execution error'
-          : null
-      "
-      :zoom-level="canvasStore.canvas?.ds?.scale || 1"
-      :data-node-id="nodeData.id"
-      @node-click="handleNodeSelect"
-      @update:collapsed="handleNodeCollapse"
-      @update:title="handleNodeTitleUpdate"
-    />
-  </TransformPane>
-
-  <!-- Debug Panel (Development Only) -->
-  <VueNodeDebugPanel
-    v-if="debugPanelVisible"
-    v-model:debug-override-vue-nodes="debugOverrideVueNodes"
-    v-model:show-performance-overlay="showPerformanceOverlay"
-    :canvas-viewport="canvasViewport"
-    :vue-nodes-count="vueNodesCount"
-    :nodes-in-viewport="nodesInViewport"
-    :performance-metrics="performanceMetrics"
-    :current-f-p-s="currentFPS"
-    :last-transform-time="lastTransformTime"
-    :raf-active="rafActive"
-    :is-dev-mode-enabled="isDevModeEnabled"
-    :should-render-vue-nodes="shouldRenderVueNodes"
-    :transform-pane-enabled="transformPaneEnabled"
-  />
-
   <NodeTooltip v-if="tooltipEnabled" />
   <NodeSearchboxPopover />
 
@@ -93,63 +39,39 @@
     <SelectionOverlay v-if="selectionToolboxEnabled">
       <SelectionToolbox />
     </SelectionOverlay>
-    <!-- Render legacy DOM widgets only when Vue nodes are disabled -->
-    <DomWidgets v-if="!shouldRenderVueNodes" />
+    <DomWidgets />
   </template>
 </template>
 
 <script setup lang="ts">
+import type { LGraphNode } from '@comfyorg/litegraph'
 import { useEventListener, whenever } from '@vueuse/core'
-import {
-  computed,
-  onMounted,
-  onUnmounted,
-  reactive,
-  ref,
-  shallowRef,
-  watch,
-  watchEffect
-} from 'vue'
+import { computed, onMounted, ref, watch, watchEffect } from 'vue'
 
 import LiteGraphCanvasSplitterOverlay from '@/components/LiteGraphCanvasSplitterOverlay.vue'
 import BottomPanel from '@/components/bottomPanel/BottomPanel.vue'
+import SubgraphBreadcrumb from '@/components/breadcrumb/SubgraphBreadcrumb.vue'
 import DomWidgets from '@/components/graph/DomWidgets.vue'
 import GraphCanvasMenu from '@/components/graph/GraphCanvasMenu.vue'
-import MiniMap from '@/components/graph/MiniMap.vue'
 import NodeTooltip from '@/components/graph/NodeTooltip.vue'
 import SelectionOverlay from '@/components/graph/SelectionOverlay.vue'
 import SelectionToolbox from '@/components/graph/SelectionToolbox.vue'
 import TitleEditor from '@/components/graph/TitleEditor.vue'
-import TransformPane from '@/components/graph/TransformPane.vue'
-import VueNodeDebugPanel from '@/components/graph/debug/VueNodeDebugPanel.vue'
 import NodeSearchboxPopover from '@/components/searchbox/NodeSearchBoxPopover.vue'
 import SideToolbar from '@/components/sidebar/SideToolbar.vue'
 import SecondRowWorkflowTabs from '@/components/topbar/SecondRowWorkflowTabs.vue'
-import { useTransformState } from '@/composables/element/useTransformState'
 import { useChainCallback } from '@/composables/functional/useChainCallback'
-import { useGraphNodeManager } from '@/composables/graph/useGraphNodeManager'
-import type {
-  NodeState,
-  VueNodeData
-} from '@/composables/graph/useGraphNodeManager'
 import { useNodeBadge } from '@/composables/node/useNodeBadge'
 import { useCanvasDrop } from '@/composables/useCanvasDrop'
 import { useContextMenuTranslation } from '@/composables/useContextMenuTranslation'
 import { useCopy } from '@/composables/useCopy'
-import { useFeatureFlags } from '@/composables/useFeatureFlags'
 import { useGlobalLitegraph } from '@/composables/useGlobalLitegraph'
 import { useLitegraphSettings } from '@/composables/useLitegraphSettings'
-import { useMinimap } from '@/composables/useMinimap'
 import { usePaste } from '@/composables/usePaste'
 import { useWorkflowAutoSave } from '@/composables/useWorkflowAutoSave'
 import { useWorkflowPersistence } from '@/composables/useWorkflowPersistence'
 import { CORE_SETTINGS } from '@/constants/coreSettings'
 import { i18n, t } from '@/i18n'
-import type { LGraphCanvas, LGraphNode } from '@/lib/litegraph/src/litegraph'
-import { layoutStore } from '@/renderer/core/layout/store/LayoutStore'
-import { useLayout } from '@/renderer/core/layout/sync/useLayout'
-import { useLayoutSync } from '@/renderer/core/layout/sync/useLayoutSync'
-import VueGraphNode from '@/renderer/extensions/vueNodes/components/LGraphNode.vue'
 import { UnauthorizedError, api } from '@/scripts/api'
 import { app as comfyApp } from '@/scripts/app'
 import { ChangeTracker } from '@/scripts/changeTracker'
@@ -177,7 +99,6 @@ const workspaceStore = useWorkspaceStore()
 const canvasStore = useCanvasStore()
 const executionStore = useExecutionStore()
 const toastStore = useToastStore()
-const { mutations: layoutMutations } = useLayout()
 const betaMenuEnabled = computed(
   () => settingStore.get('Comfy.UseNewMenu') !== 'Disabled'
 )
@@ -192,361 +113,6 @@ const selectionToolboxEnabled = computed(() =>
   settingStore.get('Comfy.Canvas.SelectionToolbox')
 )
 
-const minimapRef = ref<InstanceType<typeof MiniMap>>()
-const minimapEnabled = computed(() => settingStore.get('Comfy.Minimap.Visible'))
-const minimap = useMinimap()
-
-// Feature flags
-const { shouldRenderVueNodes, isDevModeEnabled } = useFeatureFlags()
-
-// TransformPane enabled when Vue nodes are enabled OR debug override
-const debugOverrideVueNodes = ref(false)
-// Persist debug panel visibility in settings so core commands can toggle it
-const debugPanelVisible = computed({
-  get: () => settingStore.get('Comfy.VueNodes.DebugPanel.Visible') ?? false,
-  set: (v: boolean) => {
-    void settingStore.set('Comfy.VueNodes.DebugPanel.Visible', v)
-  }
-})
-const transformPaneEnabled = computed(
-  () => shouldRenderVueNodes.value || debugOverrideVueNodes.value
-)
-// Account for browser zoom/DPI scaling
-const getActualViewport = () => {
-  // Get the actual canvas element dimensions which account for zoom
-  const canvas = canvasRef.value
-  if (canvas) {
-    return {
-      width: canvas.clientWidth,
-      height: canvas.clientHeight
-    }
-  }
-  // Fallback to window dimensions
-  return {
-    width: window.innerWidth,
-    height: window.innerHeight
-  }
-}
-
-const canvasViewport = ref(getActualViewport())
-
-// Debug metrics - use shallowRef for frequently updating values
-const vueNodesCount = shallowRef(0)
-const nodesInViewport = shallowRef(0)
-const currentFPS = shallowRef(0)
-const lastTransformTime = shallowRef(0)
-const rafActive = shallowRef(false)
-
-// Rendering options
-const showPerformanceOverlay = ref(false)
-
-// FPS tracking
-let lastTime = performance.now()
-let frameCount = 0
-let fpsRafId: number | null = null
-
-const updateFPS = () => {
-  frameCount++
-  const currentTime = performance.now()
-  if (currentTime >= lastTime + 1000) {
-    currentFPS.value = Math.round(
-      (frameCount * 1000) / (currentTime - lastTime)
-    )
-    frameCount = 0
-    lastTime = currentTime
-  }
-  if (transformPaneEnabled.value) {
-    fpsRafId = requestAnimationFrame(updateFPS)
-  }
-}
-
-// Start FPS tracking when TransformPane is enabled
-watch(transformPaneEnabled, (enabled) => {
-  if (enabled) {
-    fpsRafId = requestAnimationFrame(updateFPS)
-  } else {
-    // Stop FPS tracking
-    if (fpsRafId !== null) {
-      cancelAnimationFrame(fpsRafId)
-      fpsRafId = null
-    }
-  }
-})
-
-// Update viewport on resize
-useEventListener(window, 'resize', () => {
-  canvasViewport.value = getActualViewport()
-})
-
-// Also update when canvas is ready
-watch(canvasRef, () => {
-  if (canvasRef.value) {
-    canvasViewport.value = getActualViewport()
-  }
-})
-
-// Vue node lifecycle management - initialize after graph is ready
-let nodeManager: ReturnType<typeof useGraphNodeManager> | null = null
-let cleanupNodeManager: (() => void) | null = null
-const vueNodeData = ref<ReadonlyMap<string, VueNodeData>>(new Map())
-const nodeState = ref<ReadonlyMap<string, NodeState>>(new Map())
-const nodePositions = ref<ReadonlyMap<string, { x: number; y: number }>>(
-  new Map()
-)
-const nodeSizes = ref<ReadonlyMap<string, { width: number; height: number }>>(
-  new Map()
-)
-let detectChangesInRAF = () => {}
-const performanceMetrics = reactive({
-  frameTime: 0,
-  updateTime: 0,
-  nodeCount: 0,
-  culledCount: 0,
-  adaptiveQuality: false
-})
-
-// Initialize node manager when graph becomes available
-// Add a reactivity trigger to force computed re-evaluation
-const nodeDataTrigger = ref(0)
-
-const initializeNodeManager = () => {
-  if (!comfyApp.graph || nodeManager) return
-  nodeManager = useGraphNodeManager(comfyApp.graph)
-  cleanupNodeManager = nodeManager.cleanup
-  // Use the manager's reactive maps directly
-  vueNodeData.value = nodeManager.vueNodeData
-  nodeState.value = nodeManager.nodeState
-  nodePositions.value = nodeManager.nodePositions
-  nodeSizes.value = nodeManager.nodeSizes
-  detectChangesInRAF = nodeManager.detectChangesInRAF
-  Object.assign(performanceMetrics, nodeManager.performanceMetrics)
-
-  // Initialize layout system with existing nodes
-  const nodes = comfyApp.graph._nodes.map((node: any) => ({
-    id: node.id.toString(),
-    pos: node.pos,
-    size: node.size
-  }))
-  layoutStore.initializeFromLiteGraph(nodes)
-
-  // Initialize layout sync (one-way: Layout Store → LiteGraph)
-  const { startSync } = useLayoutSync()
-  startSync(canvasStore.canvas)
-
-  // Force computed properties to re-evaluate
-  nodeDataTrigger.value++
-}
-
-const disposeNodeManager = () => {
-  if (!nodeManager) return
-  try {
-    cleanupNodeManager?.()
-  } catch {
-    /* empty */
-  }
-  nodeManager = null
-  cleanupNodeManager = null
-  // Reset reactive maps to inert defaults
-  vueNodeData.value = new Map()
-  nodeState.value = new Map()
-  nodePositions.value = new Map()
-  nodeSizes.value = new Map()
-  // Reset metrics
-  performanceMetrics.frameTime = 0
-  performanceMetrics.updateTime = 0
-  performanceMetrics.nodeCount = 0
-  performanceMetrics.culledCount = 0
-}
-
-// Watch for transformPaneEnabled to gate the node manager lifecycle
-watch(
-  () => transformPaneEnabled.value && Boolean(comfyApp.graph),
-  (enabled) => {
-    if (enabled) {
-      initializeNodeManager()
-    } else {
-      disposeNodeManager()
-    }
-  },
-  { immediate: true }
-)
-
-// Transform state for viewport culling
-const { syncWithCanvas } = useTransformState()
-
-// Replace problematic computed property with proper reactive system
-const nodesToRender = computed(() => {
-  // Access performanceMetrics to trigger on RAF updates
-  void performanceMetrics.updateTime
-  // Access trigger to force re-evaluation after nodeManager initialization
-  void nodeDataTrigger.value
-
-  if (!comfyApp.graph || !transformPaneEnabled.value) {
-    return []
-  }
-
-  const allNodes = Array.from(vueNodeData.value.values())
-
-  // Apply viewport culling - check if node bounds intersect with viewport
-  if (nodeManager && canvasStore.canvas && comfyApp.canvas) {
-    const canvas = canvasStore.canvas
-    const manager = nodeManager
-
-    // Ensure transform is synced before checking visibility
-    syncWithCanvas(comfyApp.canvas)
-
-    const ds = canvas.ds
-
-    // Access transform time to make this reactive to transform changes
-    void lastTransformTime.value
-
-    // Work in screen space - viewport is simply the canvas element size
-    const viewport_width = canvas.canvas.width
-    const viewport_height = canvas.canvas.height
-
-    // Add margin that represents a constant distance in canvas space
-    // Convert canvas units to screen pixels by multiplying by scale
-    const canvasMarginDistance = 200 // Fixed margin in canvas units
-    const margin_x = canvasMarginDistance * ds.scale
-    const margin_y = canvasMarginDistance * ds.scale
-
-    const filtered = allNodes.filter((nodeData) => {
-      const node = manager.getNode(nodeData.id)
-      if (!node) return false
-
-      // Transform node position to screen space (same as DOM widgets)
-      const screen_x = (node.pos[0] + ds.offset[0]) * ds.scale
-      const screen_y = (node.pos[1] + ds.offset[1]) * ds.scale
-      const screen_width = node.size[0] * ds.scale
-      const screen_height = node.size[1] * ds.scale
-
-      // Check if node bounds intersect with expanded viewport (in screen space)
-      const isVisible = !(
-        screen_x + screen_width < -margin_x ||
-        screen_x > viewport_width + margin_x ||
-        screen_y + screen_height < -margin_y ||
-        screen_y > viewport_height + margin_y
-      )
-
-      return isVisible
-    })
-
-    return filtered
-  }
-
-  return allNodes
-})
-
-// Remove side effects from computed - use watchers instead
-watch(
-  () => vueNodeData.value.size,
-  (count) => {
-    vueNodesCount.value = count
-  },
-  { immediate: true }
-)
-
-watch(
-  () => nodesToRender.value.length,
-  (count) => {
-    nodesInViewport.value = count
-  }
-)
-
-// Update performance metrics when node counts change
-watch(
-  () => [vueNodeData.value.size, nodesToRender.value.length],
-  ([totalNodes, visibleNodes]) => {
-    performanceMetrics.nodeCount = totalNodes
-    performanceMetrics.culledCount = totalNodes - visibleNodes
-  }
-)
-
-// Integrate change detection with TransformPane RAF
-// Track previous transform to detect changes
-let lastScale = 1
-let lastOffsetX = 0
-let lastOffsetY = 0
-
-const handleTransformUpdate = (time: number) => {
-  lastTransformTime.value = time
-
-  // Sync transform state only when it changes (avoids reflows)
-  if (comfyApp.canvas?.ds) {
-    const currentScale = comfyApp.canvas.ds.scale
-    const currentOffsetX = comfyApp.canvas.ds.offset[0]
-    const currentOffsetY = comfyApp.canvas.ds.offset[1]
-
-    if (
-      currentScale !== lastScale ||
-      currentOffsetX !== lastOffsetX ||
-      currentOffsetY !== lastOffsetY
-    ) {
-      syncWithCanvas(comfyApp.canvas)
-      lastScale = currentScale
-      lastOffsetX = currentOffsetX
-      lastOffsetY = currentOffsetY
-    }
-  }
-
-  // Detect node changes during transform updates
-  detectChangesInRAF()
-
-  // Update performance metrics
-  performanceMetrics.frameTime = time
-
-  void nodesToRender.value.length
-}
-
-// Node event handlers
-const handleNodeSelect = (event: PointerEvent, nodeData: VueNodeData) => {
-  if (!canvasStore.canvas || !nodeManager) return
-
-  const node = nodeManager.getNode(nodeData.id)
-  if (!node) return
-
-  if (!event.ctrlKey && !event.metaKey) {
-    canvasStore.canvas.deselectAllNodes()
-  }
-
-  canvasStore.canvas.selectNode(node)
-
-  // Bring node to front when clicked (similar to LiteGraph behavior)
-  // Skip if node is pinned
-  if (!node.flags?.pinned) {
-    layoutMutations.setSource('vue')
-    layoutMutations.bringNodeToFront(nodeData.id)
-  }
-  node.selected = true
-
-  canvasStore.updateSelectedItems()
-}
-
-// Handle node collapse state changes
-const handleNodeCollapse = (nodeId: string, collapsed: boolean) => {
-  if (!nodeManager) return
-
-  const node = nodeManager.getNode(nodeId)
-  if (!node) return
-
-  // Use LiteGraph's collapse method if the state needs to change
-  const currentCollapsed = node.flags?.collapsed ?? false
-  if (currentCollapsed !== collapsed) {
-    node.collapse()
-  }
-}
-
-// Handle node title updates
-const handleNodeTitleUpdate = (nodeId: string, newTitle: string) => {
-  if (!nodeManager) return
-
-  const node = nodeManager.getNode(nodeId)
-  if (!node) return
-
-  // Update the node title in LiteGraph for persistence
-  node.title = newTitle
-}
-
 watchEffect(() => {
   nodeDefStore.showDeprecated = settingStore.get('Comfy.Node.ShowDeprecated')
 })
@@ -698,7 +264,7 @@ const loadCustomNodesI18n = async () => {
       i18n.global.mergeLocaleMessage(locale, message)
     })
   } catch (error) {
-    // Ignore i18n loading errors - not critical
+    console.error('Failed to load custom nodes i18n', error)
   }
 }
 
@@ -715,7 +281,6 @@ onMounted(async () => {
   useCopy()
   usePaste()
   useWorkflowAutoSave()
-  useFeatureFlags() // This will automatically sync Vue nodes flag with LiteGraph
 
   comfyApp.vueAppReady = true
 
@@ -728,6 +293,9 @@ onMounted(async () => {
     await settingStore.loadSettingValues()
   } catch (error) {
     if (error instanceof UnauthorizedError) {
+      console.log(
+        'Failed loading user settings, user unauthorized, cleaning local Comfy.userId'
+      )
       localStorage.removeItem('Comfy.userId')
       localStorage.removeItem('Comfy.userName')
       window.location.reload()
@@ -752,32 +320,6 @@ onMounted(async () => {
 
   comfyAppReady.value = true
 
-  // Set up a one-time listener for when the first node is added
-  // This handles the case where Vue nodes are enabled but the graph starts empty
-  // TODO: Replace this with a reactive graph mutations observer when available
-  if (
-    transformPaneEnabled.value &&
-    comfyApp.graph &&
-    !nodeManager &&
-    comfyApp.graph._nodes.length === 0
-  ) {
-    const originalOnNodeAdded = comfyApp.graph.onNodeAdded
-    comfyApp.graph.onNodeAdded = function (node: any) {
-      // Restore original handler
-      comfyApp.graph.onNodeAdded = originalOnNodeAdded
-
-      // Initialize node manager if needed
-      if (transformPaneEnabled.value && !nodeManager) {
-        initializeNodeManager()
-      }
-
-      // Call original handler
-      if (originalOnNodeAdded) {
-        originalOnNodeAdded.call(this, node)
-      }
-    }
-  }
-
   comfyApp.canvas.onSelectionChange = useChainCallback(
     comfyApp.canvas.onSelectionChange,
     () => canvasStore.updateSelectedItems()
@@ -806,13 +348,6 @@ onMounted(async () => {
     }
   )
 
-  whenever(
-    () => minimapRef.value,
-    (ref) => {
-      minimap.setMinimapRef(ref)
-    }
-  )
-
   whenever(
     () => useCanvasStore().canvas,
     (canvas) => {
@@ -825,18 +360,4 @@ onMounted(async () => {
 
   emit('ready')
 })
-
-onUnmounted(() => {
-  // Clean up FPS tracking
-  if (fpsRafId !== null) {
-    cancelAnimationFrame(fpsRafId)
-    fpsRafId = null
-  }
-
-  // Clean up node manager
-  if (nodeManager) {
-    nodeManager.cleanup()
-    nodeManager = null
-  }
-})
 </script>

src/components/graph/GraphCanvasMenu.vue

@@ -1,7 +1,6 @@
 <template>
   <ButtonGroup
     class="p-buttongroup-vertical absolute bottom-[10px] right-[10px] z-[1000]"
-    @wheel="canvasInteractions.handleWheel"
   >
     <Button
       v-tooltip.left="t('graphCanvasMenu.zoomIn')"
@@ -57,26 +56,16 @@
       data-testid="toggle-link-visibility-button"
       @click="() => commandStore.execute('Comfy.Canvas.ToggleLinkVisibility')"
     />
-    <Button
-      v-tooltip.left="t('graphCanvasMenu.toggleMinimap') + ' (Alt + m)'"
-      severity="secondary"
-      :icon="'pi pi-map'"
-      :aria-label="$t('graphCanvasMenu.toggleMinimap')"
-      :class="{ 'minimap-active': minimapVisible }"
-      data-testid="toggle-minimap-button"
-      @click="() => commandStore.execute('Comfy.Canvas.ToggleMinimap')"
-    />
   </ButtonGroup>
 </template>
 
 <script setup lang="ts">
+import { LiteGraph } from '@comfyorg/litegraph'
 import Button from 'primevue/button'
 import ButtonGroup from 'primevue/buttongroup'
 import { computed } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import { useCanvasInteractions } from '@/composables/graph/useCanvasInteractions'
-import { LiteGraph } from '@/lib/litegraph/src/litegraph'
 import { useCommandStore } from '@/stores/commandStore'
 import { useCanvasStore } from '@/stores/graphStore'
 import { useSettingStore } from '@/stores/settingStore'
@@ -85,9 +74,7 @@ const { t } = useI18n()
 const commandStore = useCommandStore()
 const canvasStore = useCanvasStore()
 const settingStore = useSettingStore()
-const canvasInteractions = useCanvasInteractions()
 
-const minimapVisible = computed(() => settingStore.get('Comfy.Minimap.Visible'))
 const linkHidden = computed(
   () => settingStore.get('Comfy.LinkRenderMode') === LiteGraph.HIDDEN_LINK
 )
@@ -120,15 +107,4 @@ const stopRepeat = () => {
   margin: 0;
   border-radius: 0;
 }
-
-.p-button.minimap-active {
-  background-color: var(--p-button-primary-background);
-  border-color: var(--p-button-primary-border-color);
-  color: var(--p-button-primary-color);
-}
-
-.p-button.minimap-active:hover {
-  background-color: var(--p-button-primary-hover-background);
-  border-color: var(--p-button-primary-hover-border-color);
-}
 </style>

src/components/graph/MiniMap.vue

@@ -1,88 +0,0 @@
-<template>
-  <div
-    v-if="visible && initialized"
-    ref="containerRef"
-    class="litegraph-minimap absolute bottom-[20px] right-[90px] z-[1000]"
-    :style="containerStyles"
-    @mousedown="handleMouseDown"
-    @mousemove="handleMouseMove"
-    @mouseup="handleMouseUp"
-    @mouseleave="handleMouseUp"
-    @wheel="handleWheel"
-  >
-    <canvas
-      ref="canvasRef"
-      :width="width"
-      :height="height"
-      class="minimap-canvas"
-    />
-    <div class="minimap-viewport" :style="viewportStyles" />
-  </div>
-</template>
-
-<script setup lang="ts">
-import { onMounted, onUnmounted, watch } from 'vue'
-
-import { useMinimap } from '@/composables/useMinimap'
-import { useCanvasStore } from '@/stores/graphStore'
-
-const minimap = useMinimap()
-const canvasStore = useCanvasStore()
-
-const {
-  initialized,
-  visible,
-  containerRef,
-  canvasRef,
-  containerStyles,
-  viewportStyles,
-  width,
-  height,
-  init,
-  destroy,
-  handleMouseDown,
-  handleMouseMove,
-  handleMouseUp,
-  handleWheel
-} = minimap
-
-watch(
-  () => canvasStore.canvas,
-  async (canvas) => {
-    if (canvas && !initialized.value) {
-      await init()
-    }
-  },
-  { immediate: true }
-)
-
-onMounted(async () => {
-  if (canvasStore.canvas) {
-    await init()
-  }
-})
-
-onUnmounted(() => {
-  destroy()
-})
-</script>
-
-<style scoped>
-.litegraph-minimap {
-  overflow: hidden;
-}
-
-.minimap-canvas {
-  display: block;
-  width: 100%;
-  height: 100%;
-  pointer-events: none;
-}
-
-.minimap-viewport {
-  position: absolute;
-  top: 0;
-  left: 0;
-  pointer-events: none;
-}
-</style>

src/components/graph/NodeTooltip.vue

@@ -10,15 +10,15 @@
 </template>
 
 <script setup lang="ts">
-import { useEventListener } from '@vueuse/core'
-import { nextTick, ref } from 'vue'
-
-import { st } from '@/i18n'
 import {
   LiteGraph,
   isOverNodeInput,
   isOverNodeOutput
-} from '@/lib/litegraph/src/litegraph'
+} from '@comfyorg/litegraph'
+import { useEventListener } from '@vueuse/core'
+import { nextTick, ref } from 'vue'
+
+import { st } from '@/i18n'
 import { app as comfyApp } from '@/scripts/app'
 import { isDOMWidget } from '@/scripts/domWidget'
 import { useNodeDefStore } from '@/stores/nodeDefStore'

src/components/graph/SelectionOverlay.vue

@@ -13,32 +13,30 @@
 </template>
 
 <script setup lang="ts">
+import { createBounds } from '@comfyorg/litegraph'
 import { whenever } from '@vueuse/core'
 import { ref, watch } from 'vue'
 
-import { useSelectedLiteGraphItems } from '@/composables/canvas/useSelectedLiteGraphItems'
 import { useAbsolutePosition } from '@/composables/element/useAbsolutePosition'
-import { createBounds } from '@/lib/litegraph/src/litegraph'
 import { useCanvasStore } from '@/stores/graphStore'
 
 const canvasStore = useCanvasStore()
 const { style, updatePosition } = useAbsolutePosition()
-const { getSelectableItems } = useSelectedLiteGraphItems()
 
 const visible = ref(false)
 const showBorder = ref(false)
 
 const positionSelectionOverlay = () => {
-  const selectableItems = getSelectableItems()
-  showBorder.value = selectableItems.size > 1
+  const { selectedItems } = canvasStore.getCanvas()
+  showBorder.value = selectedItems.size > 1
 
-  if (!selectableItems.size) {
+  if (!selectedItems.size) {
     visible.value = false
     return
   }
 
   visible.value = true
-  const bounds = createBounds(selectableItems)
+  const bounds = createBounds(selectedItems)
   if (bounds) {
     updatePosition({
       pos: [bounds[0], bounds[1]],
@@ -47,6 +45,7 @@ const positionSelectionOverlay = () => {
   }
 }
 
+// Register listener on canvas creation.
 whenever(
   () => canvasStore.getCanvas().state.selectionChanged,
   () => {

src/components/graph/SelectionToolbox.vue

@@ -5,7 +5,6 @@
       header: 'hidden',
       content: 'p-0 flex flex-row'
     }"
-    @wheel="canvasInteractions.handleWheel"
   >
     <ExecuteButton />
     <ColorPickerButton />
@@ -15,7 +14,7 @@
     <MaskEditorButton />
     <ConvertToSubgraphButton />
     <DeleteButton />
-    <RefreshSelectionButton />
+    <RefreshButton />
     <ExtensionCommandButton
       v-for="command in extensionToolboxCommands"
       :key="command.id"
@@ -39,8 +38,7 @@ import ExtensionCommandButton from '@/components/graph/selectionToolbox/Extensio
 import HelpButton from '@/components/graph/selectionToolbox/HelpButton.vue'
 import MaskEditorButton from '@/components/graph/selectionToolbox/MaskEditorButton.vue'
 import PinButton from '@/components/graph/selectionToolbox/PinButton.vue'
-import RefreshSelectionButton from '@/components/graph/selectionToolbox/RefreshSelectionButton.vue'
-import { useCanvasInteractions } from '@/composables/graph/useCanvasInteractions'
+import RefreshButton from '@/components/graph/selectionToolbox/RefreshButton.vue'
 import { useExtensionService } from '@/services/extensionService'
 import { type ComfyCommandImpl, useCommandStore } from '@/stores/commandStore'
 import { useCanvasStore } from '@/stores/graphStore'
@@ -48,7 +46,6 @@ import { useCanvasStore } from '@/stores/graphStore'
 const commandStore = useCommandStore()
 const canvasStore = useCanvasStore()
 const extensionService = useExtensionService()
-const canvasInteractions = useCanvasInteractions()
 
 const extensionToolboxCommands = computed<ComfyCommandImpl[]>(() => {
   const commandIds = new Set<string>(

src/components/graph/TitleEditor.vue

@@ -13,17 +13,13 @@
 </template>
 
 <script setup lang="ts">
+import { LGraphGroup, LGraphNode, LiteGraph } from '@comfyorg/litegraph'
+import type { LiteGraphCanvasEvent } from '@comfyorg/litegraph'
 import { useEventListener } from '@vueuse/core'
 import { type CSSProperties, computed, ref, watch } from 'vue'
 
 import EditableText from '@/components/common/EditableText.vue'
 import { useAbsolutePosition } from '@/composables/element/useAbsolutePosition'
-import {
-  LGraphGroup,
-  LGraphNode,
-  LiteGraph
-} from '@/lib/litegraph/src/litegraph'
-import type { LiteGraphCanvasEvent } from '@/lib/litegraph/src/litegraph'
 import { app } from '@/scripts/app'
 import { useCanvasStore, useTitleEditorStore } from '@/stores/graphStore'
 import { useSettingStore } from '@/stores/settingStore'

src/components/graph/TransformPane.spec.ts

@@ -1,438 +0,0 @@
-import { mount } from '@vue/test-utils'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { nextTick, ref } from 'vue'
-
-import TransformPane from './TransformPane.vue'
-
-// Mock the transform state composable
-const mockTransformState = {
-  camera: ref({ x: 0, y: 0, z: 1 }),
-  transformStyle: ref({
-    transform: 'scale(1) translate(0px, 0px)',
-    transformOrigin: '0 0'
-  }),
-  syncWithCanvas: vi.fn(),
-  canvasToScreen: vi.fn(),
-  screenToCanvas: vi.fn(),
-  isNodeInViewport: vi.fn()
-}
-
-vi.mock('@/composables/element/useTransformState', () => ({
-  useTransformState: () => mockTransformState
-}))
-
-// Mock requestAnimationFrame/cancelAnimationFrame
-global.requestAnimationFrame = vi.fn((cb) => {
-  setTimeout(cb, 16)
-  return 1
-})
-global.cancelAnimationFrame = vi.fn()
-
-describe('TransformPane', () => {
-  let wrapper: ReturnType<typeof mount>
-  let mockCanvas: any
-
-  beforeEach(() => {
-    vi.clearAllMocks()
-
-    // Create mock canvas with LiteGraph interface
-    mockCanvas = {
-      canvas: {
-        addEventListener: vi.fn(),
-        removeEventListener: vi.fn()
-      },
-      ds: {
-        offset: [0, 0],
-        scale: 1
-      }
-    }
-
-    // Reset mock transform state
-    mockTransformState.camera.value = { x: 0, y: 0, z: 1 }
-    mockTransformState.transformStyle.value = {
-      transform: 'scale(1) translate(0px, 0px)',
-      transformOrigin: '0 0'
-    }
-  })
-
-  afterEach(() => {
-    if (wrapper) {
-      wrapper.unmount()
-    }
-  })
-
-  describe('component mounting', () => {
-    it('should mount successfully with minimal props', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      expect(wrapper.exists()).toBe(true)
-      expect(wrapper.find('.transform-pane').exists()).toBe(true)
-    })
-
-    it('should apply transform style from composable', () => {
-      mockTransformState.transformStyle.value = {
-        transform: 'scale(2) translate(100px, 50px)',
-        transformOrigin: '0 0'
-      }
-
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      const transformPane = wrapper.find('.transform-pane')
-      const style = transformPane.attributes('style')
-      expect(style).toContain('transform: scale(2) translate(100px, 50px)')
-    })
-
-    it('should render slot content', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        },
-        slots: {
-          default: '<div class="test-content">Test Node</div>'
-        }
-      })
-
-      expect(wrapper.find('.test-content').exists()).toBe(true)
-      expect(wrapper.find('.test-content').text()).toBe('Test Node')
-    })
-  })
-
-  describe('debug overlay', () => {
-    it('should not show debug overlay by default', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      expect(wrapper.find('.viewport-debug-overlay').exists()).toBe(false)
-    })
-
-    it('should show debug overlay when enabled', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas,
-          viewport: { width: 1920, height: 1080 },
-          showDebugOverlay: true
-        }
-      })
-
-      expect(wrapper.find('.viewport-debug-overlay').exists()).toBe(true)
-    })
-
-    it('should display viewport dimensions in debug overlay', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas,
-          viewport: { width: 1280, height: 720 },
-          showDebugOverlay: true
-        }
-      })
-
-      const debugOverlay = wrapper.find('.viewport-debug-overlay')
-      expect(debugOverlay.text()).toContain('Viewport: 1280x720')
-    })
-
-    it('should include device pixel ratio in debug overlay', () => {
-      // Mock device pixel ratio
-      Object.defineProperty(window, 'devicePixelRatio', {
-        writable: true,
-        value: 2
-      })
-
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas,
-          viewport: { width: 1920, height: 1080 },
-          showDebugOverlay: true
-        }
-      })
-
-      const debugOverlay = wrapper.find('.viewport-debug-overlay')
-      expect(debugOverlay.text()).toContain('DPR: 2')
-    })
-  })
-
-  describe('RAF synchronization', () => {
-    it('should start RAF sync on mount', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-
-      // Should emit RAF status change to true
-      expect(wrapper.emitted('rafStatusChange')).toBeTruthy()
-      expect(wrapper.emitted('rafStatusChange')?.[0]).toEqual([true])
-    })
-
-    it('should call syncWithCanvas during RAF updates', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-
-      // Allow RAF to execute
-      await new Promise((resolve) => setTimeout(resolve, 20))
-
-      expect(mockTransformState.syncWithCanvas).toHaveBeenCalledWith(mockCanvas)
-    })
-
-    it('should emit transform update timing', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-
-      // Allow RAF to execute
-      await new Promise((resolve) => setTimeout(resolve, 20))
-
-      expect(wrapper.emitted('transformUpdate')).toBeTruthy()
-      const updateEvent = wrapper.emitted('transformUpdate')?.[0]
-      expect(typeof updateEvent?.[0]).toBe('number')
-      expect(updateEvent?.[0]).toBeGreaterThanOrEqual(0)
-    })
-
-    it('should stop RAF sync on unmount', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-      wrapper.unmount()
-
-      expect(wrapper.emitted('rafStatusChange')).toBeTruthy()
-      const events = wrapper.emitted('rafStatusChange') as any[]
-      expect(events[events.length - 1]).toEqual([false])
-      expect(global.cancelAnimationFrame).toHaveBeenCalled()
-    })
-  })
-
-  describe('canvas event listeners', () => {
-    it('should add event listeners to canvas on mount', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-
-      expect(mockCanvas.canvas.addEventListener).toHaveBeenCalledWith(
-        'wheel',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.addEventListener).toHaveBeenCalledWith(
-        'pointerdown',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.addEventListener).toHaveBeenCalledWith(
-        'pointerup',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.addEventListener).toHaveBeenCalledWith(
-        'pointercancel',
-        expect.any(Function),
-        expect.any(Object)
-      )
-    })
-
-    it('should remove event listeners on unmount', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      await nextTick()
-      wrapper.unmount()
-
-      expect(mockCanvas.canvas.removeEventListener).toHaveBeenCalledWith(
-        'wheel',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.removeEventListener).toHaveBeenCalledWith(
-        'pointerdown',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.removeEventListener).toHaveBeenCalledWith(
-        'pointerup',
-        expect.any(Function),
-        expect.any(Object)
-      )
-      expect(mockCanvas.canvas.removeEventListener).toHaveBeenCalledWith(
-        'pointercancel',
-        expect.any(Function),
-        expect.any(Object)
-      )
-    })
-  })
-
-  describe('interaction state management', () => {
-    it('should apply interacting class during interactions', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      // Simulate interaction start by checking internal state
-      // Note: This tests the CSS class application logic
-      const transformPane = wrapper.find('.transform-pane')
-
-      // Initially should not have interacting class
-      expect(transformPane.classes()).not.toContain(
-        'transform-pane--interacting'
-      )
-    })
-
-    it('should handle pointer events for node delegation', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      const transformPane = wrapper.find('.transform-pane')
-
-      // Simulate pointer down - we can't test the exact delegation logic
-      // in unit tests due to vue-test-utils limitations, but we can verify
-      // the event handler is set up correctly
-      await transformPane.trigger('pointerdown')
-
-      // The test passes if no errors are thrown during event handling
-      expect(transformPane.exists()).toBe(true)
-    })
-  })
-
-  describe('transform state integration', () => {
-    it('should provide transform utilities to child components', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      // The component should provide transform state via Vue's provide/inject
-      // This is tested indirectly through the composable integration
-      expect(mockTransformState.syncWithCanvas).toBeDefined()
-      expect(mockTransformState.canvasToScreen).toBeDefined()
-      expect(mockTransformState.screenToCanvas).toBeDefined()
-    })
-  })
-
-  describe('error handling', () => {
-    it('should handle null canvas gracefully', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: undefined
-        }
-      })
-
-      expect(wrapper.exists()).toBe(true)
-      expect(wrapper.find('.transform-pane').exists()).toBe(true)
-    })
-
-    it('should handle missing canvas properties', () => {
-      const incompleteCanvas = {} as any
-
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: incompleteCanvas
-        }
-      })
-
-      expect(wrapper.exists()).toBe(true)
-      // Should not throw errors during mount
-    })
-  })
-
-  describe('performance optimizations', () => {
-    it('should use contain CSS property for layout optimization', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        }
-      })
-
-      const transformPane = wrapper.find('.transform-pane')
-
-      // This test verifies the CSS contains the performance optimization
-      // Note: In JSDOM, computed styles might not reflect all CSS properties
-      expect(transformPane.element.className).toContain('transform-pane')
-    })
-
-    it('should disable pointer events on container but allow on children', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas
-        },
-        slots: {
-          default: '<div data-node-id="test">Test Node</div>'
-        }
-      })
-
-      const transformPane = wrapper.find('.transform-pane')
-
-      // The CSS should handle pointer events optimization
-      // This is primarily a CSS concern, but we verify the structure
-      expect(transformPane.exists()).toBe(true)
-      expect(wrapper.find('[data-node-id="test"]').exists()).toBe(true)
-    })
-  })
-
-  describe('viewport prop handling', () => {
-    it('should handle missing viewport prop', () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas,
-          showDebugOverlay: true
-        }
-      })
-
-      // Should not crash when viewport is undefined
-      expect(wrapper.exists()).toBe(true)
-    })
-
-    it('should update debug overlay when viewport changes', async () => {
-      wrapper = mount(TransformPane, {
-        props: {
-          canvas: mockCanvas,
-          viewport: { width: 800, height: 600 },
-          showDebugOverlay: true
-        }
-      })
-
-      expect(wrapper.text()).toContain('800x600')
-
-      await wrapper.setProps({
-        viewport: { width: 1920, height: 1080 }
-      })
-
-      expect(wrapper.text()).toContain('1920x1080')
-    })
-  })
-})

src/components/graph/TransformPane.vue

@@ -1,137 +0,0 @@
-<!-- eslint-disable @intlify/vue-i18n/no-raw-text -->
-<template>
-  <div
-    class="transform-pane"
-    :class="{ 'transform-pane--interacting': isInteracting }"
-    :style="transformStyle"
-    @pointerdown="handlePointerDown"
-  >
-    <!-- Vue nodes will be rendered here -->
-    <slot />
-
-    <!-- DEV ONLY: Viewport bounds visualization -->
-    <div
-      v-if="props.showDebugOverlay"
-      class="viewport-debug-overlay"
-      :style="{
-        position: 'absolute',
-        left: '10px',
-        top: '10px',
-        border: '2px solid red',
-        width: (props.viewport?.width || 0) - 20 + 'px',
-        height: (props.viewport?.height || 0) - 20 + 'px',
-        pointerEvents: 'none',
-        opacity: 0.5
-      }"
-    >
-      <div
-        style="
-          position: absolute;
-          top: 0;
-          left: 0;
-          background: red;
-          color: white;
-          padding: 2px 5px;
-          font-size: 10px;
-        "
-      >
-        Viewport: {{ props.viewport?.width }}x{{ props.viewport?.height }} DPR:
-        {{ devicePixelRatio }}
-      </div>
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed, provide } from 'vue'
-
-import { useTransformState } from '@/composables/element/useTransformState'
-import { useCanvasTransformSync } from '@/composables/graph/useCanvasTransformSync'
-import { useTransformSettling } from '@/composables/graph/useTransformSettling'
-import { LGraphCanvas } from '@/lib/litegraph/src/litegraph'
-
-interface TransformPaneProps {
-  canvas?: LGraphCanvas
-  viewport?: { width: number; height: number }
-  showDebugOverlay?: boolean
-}
-
-const props = defineProps<TransformPaneProps>()
-
-// Get device pixel ratio for display
-const devicePixelRatio = window.devicePixelRatio || 1
-
-// Transform state management
-const {
-  camera,
-  transformStyle,
-  syncWithCanvas,
-  canvasToScreen,
-  screenToCanvas,
-  isNodeInViewport
-} = useTransformState()
-
-// Transform settling detection for re-rasterization optimization
-const canvasElement = computed(() => props.canvas?.canvas)
-const { isTransforming } = useTransformSettling(canvasElement, {
-  settleDelay: 200,
-  trackPan: true
-})
-
-// Use isTransforming for the CSS class (aliased for clarity)
-const isInteracting = isTransforming
-
-// Provide transform utilities to child components
-provide('transformState', {
-  camera,
-  canvasToScreen,
-  screenToCanvas,
-  isNodeInViewport
-})
-
-// Event delegation for node interactions
-const handlePointerDown = (event: PointerEvent) => {
-  const target = event.target as HTMLElement
-  const nodeElement = target.closest('[data-node-id]')
-
-  if (nodeElement) {
-    // TODO: Emit event for node interaction
-    // Node interaction with nodeId will be handled in future implementation
-  }
-}
-
-// Canvas transform synchronization
-const emit = defineEmits<{
-  rafStatusChange: [active: boolean]
-  transformUpdate: [time: number]
-}>()
-
-useCanvasTransformSync(props.canvas, syncWithCanvas, {
-  onStart: () => emit('rafStatusChange', true),
-  onUpdate: (duration) => emit('transformUpdate', duration),
-  onStop: () => emit('rafStatusChange', false)
-})
-</script>
-
-<style scoped>
-.transform-pane {
-  position: absolute;
-  inset: 0;
-  contain: layout style paint;
-  transform-origin: 0 0;
-  pointer-events: none;
-  top: 0;
-  left: 0;
-  width: 100%;
-  height: 100%;
-}
-
-.transform-pane--interacting {
-  will-change: transform;
-}
-
-/* Allow pointer events on nodes */
-.transform-pane :deep([data-node-id]) {
-  pointer-events: auto;
-}
-</style>

src/components/graph/debug/QuadTreeDebugSection.vue

@@ -1,112 +0,0 @@
-<!-- eslint-disable @intlify/vue-i18n/no-raw-text -->
-<template>
-  <div class="pt-2 border-t border-surface-200 dark-theme:border-surface-700">
-    <h4 class="font-semibold mb-1">QuadTree Spatial Index</h4>
-
-    <!-- Enable/Disable Toggle -->
-    <div class="mb-2">
-      <label class="flex items-center gap-2">
-        <input
-          :checked="enabled"
-          type="checkbox"
-          @change="$emit('toggle', ($event.target as HTMLInputElement).checked)"
-        />
-        <span>Enable Spatial Indexing</span>
-      </label>
-    </div>
-
-    <!-- Status Message -->
-    <p v-if="!enabled" class="text-muted text-xs italic">
-      {{ statusMessage }}
-    </p>
-
-    <!-- Metrics when enabled -->
-    <template v-if="enabled && metrics">
-      <p class="text-muted">Strategy: {{ strategy }}</p>
-      <p class="text-muted">Total Nodes: {{ metrics.totalNodes }}</p>
-      <p class="text-muted">Visible Nodes: {{ metrics.visibleNodes }}</p>
-      <p class="text-muted">Query Time: {{ metrics.queryTime.toFixed(2) }}ms</p>
-      <p class="text-muted">Tree Depth: {{ metrics.treeDepth }}</p>
-      <p class="text-muted">Culling Efficiency: {{ cullingEfficiency }}</p>
-      <p class="text-muted">Rebuilds: {{ metrics.rebuildCount }}</p>
-
-      <!-- Show debug visualization toggle -->
-      <div class="mt-2">
-        <label class="flex items-center gap-2">
-          <input
-            :checked="showVisualization"
-            type="checkbox"
-            @change="
-              $emit(
-                'toggle-visualization',
-                ($event.target as HTMLInputElement).checked
-              )
-            "
-          />
-          <span>Show QuadTree Boundaries</span>
-        </label>
-      </div>
-    </template>
-
-    <!-- Performance Comparison -->
-    <template v-if="enabled && performanceComparison">
-      <div class="mt-2 text-xs">
-        <p class="text-muted font-semibold">Performance vs Linear:</p>
-        <p class="text-muted">Speedup: {{ performanceComparison.speedup }}x</p>
-        <p class="text-muted">
-          Break-even: ~{{ performanceComparison.breakEvenNodeCount }} nodes
-        </p>
-      </div>
-    </template>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-interface Props {
-  enabled: boolean
-  metrics?: {
-    totalNodes: number
-    visibleNodes: number
-    queryTime: number
-    treeDepth: number
-    rebuildCount: number
-  }
-  strategy?: string
-  threshold?: number
-  showVisualization?: boolean
-  performanceComparison?: {
-    speedup: number
-    breakEvenNodeCount: number
-  }
-}
-
-const props = withDefaults(defineProps<Props>(), {
-  metrics: undefined,
-  strategy: 'quadtree',
-  threshold: 100,
-  showVisualization: false,
-  performanceComparison: undefined
-})
-
-defineEmits<{
-  toggle: [enabled: boolean]
-  'toggle-visualization': [show: boolean]
-}>()
-
-const statusMessage = computed(() => {
-  if (!props.enabled && props.metrics) {
-    return `Disabled (threshold: ${props.threshold} nodes, current: ${props.metrics.totalNodes})`
-  }
-  return `Spatial indexing will enable at ${props.threshold}+ nodes`
-})
-
-const cullingEfficiency = computed(() => {
-  if (!props.metrics || props.metrics.totalNodes === 0) return 'N/A'
-
-  const culled = props.metrics.totalNodes - props.metrics.visibleNodes
-  const percentage = ((culled / props.metrics.totalNodes) * 100).toFixed(1)
-  return `${culled} nodes (${percentage}%)`
-})
-</script>

src/components/graph/debug/QuadTreeVisualization.vue

@@ -1,112 +0,0 @@
-<template>
-  <svg
-    v-if="visible && debugInfo"
-    :width="svgSize.width"
-    :height="svgSize.height"
-    :style="svgStyle"
-    class="quadtree-visualization"
-  >
-    <!-- QuadTree boundaries -->
-    <g v-for="(node, index) in flattenedNodes" :key="`quad-${index}`">
-      <rect
-        :x="node.bounds.x"
-        :y="node.bounds.y"
-        :width="node.bounds.width"
-        :height="node.bounds.height"
-        :stroke="getDepthColor(node.depth)"
-        :stroke-width="getStrokeWidth(node.depth)"
-        fill="none"
-        :opacity="0.3 + node.depth * 0.05"
-      />
-    </g>
-
-    <!-- Viewport bounds (optional) -->
-    <rect
-      v-if="viewportBounds"
-      :x="viewportBounds.x"
-      :y="viewportBounds.y"
-      :width="viewportBounds.width"
-      :height="viewportBounds.height"
-      stroke="#00ff00"
-      stroke-width="3"
-      fill="none"
-      stroke-dasharray="10,5"
-      opacity="0.8"
-    />
-  </svg>
-</template>
-
-<script setup lang="ts">
-import { computed, ref } from 'vue'
-
-import type { Bounds } from '@/utils/spatial/QuadTree'
-
-interface Props {
-  visible: boolean
-  debugInfo: any | null
-  transformStyle: any
-  viewportBounds?: Bounds
-}
-
-const props = defineProps<Props>()
-
-// Flatten the tree structure for rendering
-const flattenedNodes = computed(() => {
-  if (!props.debugInfo?.tree) return []
-
-  const nodes: any[] = []
-  const traverse = (node: any, depth = 0) => {
-    nodes.push({
-      bounds: node.bounds,
-      depth,
-      itemCount: node.itemCount,
-      divided: node.divided
-    })
-
-    if (node.children) {
-      node.children.forEach((child: any) => traverse(child, depth + 1))
-    }
-  }
-
-  traverse(props.debugInfo.tree)
-  return nodes
-})
-
-// SVG size (matches the transform pane size)
-const svgSize = ref({ width: 20000, height: 20000 })
-
-// Apply the same transform as the TransformPane
-const svgStyle = computed(() => ({
-  ...props.transformStyle,
-  position: 'absolute',
-  top: 0,
-  left: 0,
-  pointerEvents: 'none'
-}))
-
-// Color based on depth
-const getDepthColor = (depth: number): string => {
-  const colors = [
-    '#ff6b6b', // Red
-    '#ffa500', // Orange
-    '#ffd93d', // Yellow
-    '#6bcf7f', // Green
-    '#4da6ff', // Blue
-    '#a78bfa' // Purple
-  ]
-  return colors[depth % colors.length]
-}
-
-// Stroke width based on depth
-const getStrokeWidth = (depth: number): number => {
-  return Math.max(0.5, 2 - depth * 0.3)
-}
-</script>
-
-<style scoped>
-.quadtree-visualization {
-  position: absolute;
-  overflow: visible;
-  z-index: 10; /* Above nodes but below UI */
-}
-</style>

src/components/graph/debug/VueNodeDebugPanel.vue

@@ -1,165 +0,0 @@
-<!-- eslint-disable @intlify/vue-i18n/no-raw-text -->
-<template>
-  <!-- TransformPane Debug Controls -->
-  <div
-    class="fixed top-20 right-4 bg-surface-0 dark-theme:bg-surface-800 p-4 rounded-lg shadow-lg border border-surface-300 dark-theme:border-surface-600 z-50 pointer-events-auto w-80"
-    style="contain: layout style"
-  >
-    <h3 class="font-bold mb-2 text-sm">TransformPane Debug</h3>
-    <div class="space-y-2 text-xs">
-      <div>
-        <label class="flex items-center gap-2">
-          <input v-model="debugOverrideVueNodes" type="checkbox" />
-          <span>Enable TransformPane</span>
-        </label>
-      </div>
-
-      <!-- Canvas Metrics -->
-      <div
-        class="pt-2 border-t border-surface-200 dark-theme:border-surface-700"
-      >
-        <h4 class="font-semibold mb-1">Canvas State</h4>
-        <p class="text-muted">
-          Status: {{ canvasStore.canvas ? 'Ready' : 'Not Ready' }}
-        </p>
-        <p class="text-muted">
-          Viewport: {{ Math.round(canvasViewport.width) }}x{{
-            Math.round(canvasViewport.height)
-          }}
-        </p>
-        <template v-if="canvasStore.canvas?.ds">
-          <p class="text-muted">
-            Offset: ({{ Math.round(canvasStore.canvas.ds.offset[0]) }},
-            {{ Math.round(canvasStore.canvas.ds.offset[1]) }})
-          </p>
-          <p class="text-muted">
-            Scale: {{ canvasStore.canvas.ds.scale?.toFixed(3) || 1 }}
-          </p>
-        </template>
-      </div>
-
-      <!-- Node Metrics -->
-      <div
-        class="pt-2 border-t border-surface-200 dark-theme:border-surface-700"
-      >
-        <h4 class="font-semibold mb-1">Graph Metrics</h4>
-        <p class="text-muted">
-          Total Nodes: {{ comfyApp.graph?.nodes?.length || 0 }}
-        </p>
-        <p class="text-muted">
-          Selected Nodes: {{ canvasStore.canvas?.selectedItems?.size || 0 }}
-        </p>
-        <p class="text-muted">Vue Nodes Rendered: {{ vueNodesCount }}</p>
-        <p class="text-muted">Nodes in Viewport: {{ nodesInViewport }}</p>
-        <p class="text-muted">
-          Culled Nodes: {{ performanceMetrics.culledCount }}
-        </p>
-        <p class="text-muted">
-          Cull Percentage:
-          {{
-            Math.round(
-              ((vueNodesCount - nodesInViewport) / Math.max(vueNodesCount, 1)) *
-                100
-            )
-          }}%
-        </p>
-      </div>
-
-      <!-- Performance Metrics -->
-      <div
-        class="pt-2 border-t border-surface-200 dark-theme:border-surface-700"
-      >
-        <h4 class="font-semibold mb-1">Performance</h4>
-        <p v-memo="[currentFPS]" class="text-muted">FPS: {{ currentFPS }}</p>
-        <p v-memo="[Math.round(lastTransformTime)]" class="text-muted">
-          Transform Update: {{ Math.round(lastTransformTime) }}ms
-        </p>
-        <p
-          v-memo="[Math.round(performanceMetrics.updateTime)]"
-          class="text-muted"
-        >
-          Lifecycle Update: {{ Math.round(performanceMetrics.updateTime) }}ms
-        </p>
-        <p v-memo="[rafActive]" class="text-muted">
-          RAF Active: {{ rafActive ? 'Yes' : 'No' }}
-        </p>
-        <p v-memo="[performanceMetrics.adaptiveQuality]" class="text-muted">
-          Adaptive Quality:
-          {{ performanceMetrics.adaptiveQuality ? 'On' : 'Off' }}
-        </p>
-      </div>
-
-      <!-- Feature Flags Status -->
-      <div
-        v-if="isDevModeEnabled"
-        class="pt-2 border-t border-surface-200 dark-theme:border-surface-700"
-      >
-        <h4 class="font-semibold mb-1">Feature Flags</h4>
-        <p class="text-muted text-xs">
-          Vue Nodes: {{ shouldRenderVueNodes ? 'Enabled' : 'Disabled' }}
-        </p>
-        <p class="text-muted text-xs">
-          Dev Mode: {{ isDevModeEnabled ? 'Enabled' : 'Disabled' }}
-        </p>
-      </div>
-
-      <!-- Performance Options -->
-      <div
-        v-if="transformPaneEnabled"
-        class="pt-2 border-t border-surface-200 dark-theme:border-surface-700"
-      >
-        <h4 class="font-semibold mb-1">Debug Options</h4>
-        <label class="flex items-center gap-2">
-          <input v-model="showPerformanceOverlay" type="checkbox" />
-          <span>Show Performance Overlay</span>
-        </label>
-      </div>
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-import { app as comfyApp } from '@/scripts/app'
-import { useCanvasStore } from '@/stores/graphStore'
-
-interface Props {
-  debugOverrideVueNodes: boolean
-  canvasViewport: { width: number; height: number }
-  vueNodesCount: number
-  nodesInViewport: number
-  performanceMetrics: {
-    culledCount: number
-    updateTime: number
-    adaptiveQuality: boolean
-  }
-  currentFPS: number
-  lastTransformTime: number
-  rafActive: boolean
-  isDevModeEnabled: boolean
-  shouldRenderVueNodes: boolean
-  transformPaneEnabled: boolean
-  showPerformanceOverlay: boolean
-}
-
-interface Emits {
-  (e: 'update:debugOverrideVueNodes', value: boolean): void
-  (e: 'update:showPerformanceOverlay', value: boolean): void
-}
-
-const props = defineProps<Props>()
-const emit = defineEmits<Emits>()
-
-const canvasStore = useCanvasStore()
-
-const debugOverrideVueNodes = computed({
-  get: () => props.debugOverrideVueNodes,
-  set: (value: boolean) => emit('update:debugOverrideVueNodes', value)
-})
-
-const showPerformanceOverlay = computed({
-  get: () => props.showPerformanceOverlay,
-  set: (value: boolean) => emit('update:showPerformanceOverlay', value)
-})
-</script>

src/components/graph/selectionToolbox/ColorPickerButton.spec.ts

@@ -1,122 +0,0 @@
-import { mount } from '@vue/test-utils'
-import { createPinia, setActivePinia } from 'pinia'
-import PrimeVue from 'primevue/config'
-import Tooltip from 'primevue/tooltip'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-import { createI18n } from 'vue-i18n'
-
-// Import after mocks
-import ColorPickerButton from '@/components/graph/selectionToolbox/ColorPickerButton.vue'
-import { useCanvasStore } from '@/stores/graphStore'
-import { useWorkflowStore } from '@/stores/workflowStore'
-
-// Mock the litegraph module
-vi.mock('@/lib/litegraph/src/litegraph', async () => {
-  const actual = await vi.importActual('@/lib/litegraph/src/litegraph')
-  return {
-    ...actual,
-    LGraphCanvas: {
-      node_colors: {
-        red: { bgcolor: '#ff0000' },
-        green: { bgcolor: '#00ff00' },
-        blue: { bgcolor: '#0000ff' }
-      }
-    },
-    LiteGraph: {
-      NODE_DEFAULT_BGCOLOR: '#353535'
-    },
-    isColorable: vi.fn(() => true)
-  }
-})
-
-// Mock the colorUtil module
-vi.mock('@/utils/colorUtil', () => ({
-  adjustColor: vi.fn((color: string) => color + '_light')
-}))
-
-// Mock the litegraphUtil module
-vi.mock('@/utils/litegraphUtil', () => ({
-  getItemsColorOption: vi.fn(() => null),
-  isLGraphNode: vi.fn((item) => item?.type === 'LGraphNode'),
-  isLGraphGroup: vi.fn((item) => item?.type === 'LGraphGroup'),
-  isReroute: vi.fn(() => false)
-}))
-
-describe('ColorPickerButton', () => {
-  let canvasStore: ReturnType<typeof useCanvasStore>
-  let workflowStore: ReturnType<typeof useWorkflowStore>
-
-  const i18n = createI18n({
-    legacy: false,
-    locale: 'en',
-    messages: {
-      en: {
-        color: {
-          noColor: 'No Color',
-          red: 'Red',
-          green: 'Green',
-          blue: 'Blue'
-        }
-      }
-    }
-  })
-
-  beforeEach(() => {
-    setActivePinia(createPinia())
-    canvasStore = useCanvasStore()
-    workflowStore = useWorkflowStore()
-
-    // Set up default store state
-    canvasStore.selectedItems = []
-
-    // Mock workflow store
-    workflowStore.activeWorkflow = {
-      changeTracker: {
-        checkState: vi.fn()
-      }
-    } as any
-  })
-
-  const createWrapper = () => {
-    return mount(ColorPickerButton, {
-      global: {
-        plugins: [PrimeVue, i18n],
-        directives: {
-          tooltip: Tooltip
-        }
-      }
-    })
-  }
-
-  it('should render when nodes are selected', () => {
-    // Add a mock node to selectedItems
-    canvasStore.selectedItems = [{ type: 'LGraphNode' } as any]
-    const wrapper = createWrapper()
-    expect(wrapper.find('button').exists()).toBe(true)
-  })
-
-  it('should not render when nothing is selected', () => {
-    // Keep selectedItems empty
-    canvasStore.selectedItems = []
-    const wrapper = createWrapper()
-    // The button exists but is hidden with v-show
-    expect(wrapper.find('button').exists()).toBe(true)
-    expect(wrapper.find('button').attributes('style')).toContain(
-      'display: none'
-    )
-  })
-
-  it('should toggle color picker visibility on button click', async () => {
-    canvasStore.selectedItems = [{ type: 'LGraphNode' } as any]
-    const wrapper = createWrapper()
-    const button = wrapper.find('button')
-
-    expect(wrapper.find('.color-picker-container').exists()).toBe(false)
-
-    await button.trigger('click')
-    expect(wrapper.find('.color-picker-container').exists()).toBe(true)
-
-    await button.trigger('click')
-    expect(wrapper.find('.color-picker-container').exists()).toBe(false)
-  })
-})

src/components/graph/selectionToolbox/ColorPickerButton.vue

@@ -2,10 +2,6 @@
   <div class="relative">
     <Button
       v-show="canvasStore.nodeSelected || canvasStore.groupSelected"
-      v-tooltip.top="{
-        value: localizedCurrentColorName ?? t('color.noColor'),
-        showDelay: 512
-      }"
       severity="secondary"
       text
       @click="() => (showColorPicker = !showColorPicker)"
@@ -44,17 +40,13 @@
 </template>
 
 <script setup lang="ts">
+import type { ColorOption as CanvasColorOption } from '@comfyorg/litegraph'
+import { LGraphCanvas, LiteGraph, isColorable } from '@comfyorg/litegraph'
 import Button from 'primevue/button'
 import SelectButton from 'primevue/selectbutton'
 import { computed, ref, watch } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import type { ColorOption as CanvasColorOption } from '@/lib/litegraph/src/litegraph'
-import {
-  LGraphCanvas,
-  LiteGraph,
-  isColorable
-} from '@/lib/litegraph/src/litegraph'
 import { useCanvasStore } from '@/stores/graphStore'
 import { useWorkflowStore } from '@/stores/workflowStore'
 import { useColorPaletteStore } from '@/stores/workspace/colorPaletteStore'
@@ -131,16 +123,6 @@ const currentColor = computed(() =>
     : null
 )
 
-const localizedCurrentColorName = computed(() => {
-  if (!currentColorOption.value?.bgcolor) return null
-  const colorOption = colorOptions.find(
-    (option) =>
-      option.value.dark === currentColorOption.value?.bgcolor ||
-      option.value.light === currentColorOption.value?.bgcolor
-  )
-  return colorOption?.localizedName ?? NO_COLOR_OPTION.localizedName
-})
-
 watch(
   () => canvasStore.selectedItems,
   (newSelectedItems) => {

src/components/graph/selectionToolbox/ConvertToSubgraphButton.vue

@@ -7,12 +7,9 @@
     }"
     severity="secondary"
     text
+    icon="pi pi-box"
     @click="() => commandStore.execute('Comfy.Graph.ConvertToSubgraph')"
-  >
-    <template #icon>
-      <i-lucide:shrink />
-    </template>
-  </Button>
+  />
 </template>
 
 <script setup lang="ts">

src/components/graph/selectionToolbox/ExecuteButton.vue

@@ -19,11 +19,11 @@
 </template>
 
 <script setup lang="ts">
+import type { LGraphNode } from '@comfyorg/litegraph'
 import Button from 'primevue/button'
 import { computed, ref } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import type { LGraphNode } from '@/lib/litegraph/src/litegraph'
 import { useCommandStore } from '@/stores/commandStore'
 import { useCanvasStore } from '@/stores/graphStore'
 import { isLGraphNode } from '@/utils/litegraphUtil'

src/components/graph/vueWidgets/WidgetButton.vue

@@ -1,43 +0,0 @@
-<template>
-  <div class="flex flex-col gap-1">
-    <label v-if="widget.name" class="text-sm opacity-80">{{
-      widget.name
-    }}</label>
-    <Button
-      v-bind="filteredProps"
-      :disabled="readonly"
-      size="small"
-      @click="handleClick"
-    />
-  </div>
-</template>
-
-<script setup lang="ts">
-import Button from 'primevue/button'
-import { computed } from 'vue'
-
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-import {
-  BADGE_EXCLUDED_PROPS,
-  filterWidgetProps
-} from '@/utils/widgetPropFilter'
-
-// Button widgets don't have a v-model value, they trigger actions
-const props = defineProps<{
-  widget: SimplifiedWidget<void>
-  readonly?: boolean
-}>()
-
-// Button specific excluded props
-const BUTTON_EXCLUDED_PROPS = [...BADGE_EXCLUDED_PROPS, 'iconClass'] as const
-
-const filteredProps = computed(() =>
-  filterWidgetProps(props.widget.options, BUTTON_EXCLUDED_PROPS)
-)
-
-const handleClick = () => {
-  if (!props.readonly && props.widget.callback) {
-    props.widget.callback()
-  }
-}
-</script>

src/components/graph/vueWidgets/WidgetChart.vue

@@ -1,78 +0,0 @@
-<template>
-  <div class="flex flex-col gap-1">
-    <div
-      class="p-4 border border-gray-300 dark-theme:border-gray-600 rounded max-h-[48rem]"
-    >
-      <Chart :type="chartType" :data="chartData" :options="chartOptions" />
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import type { ChartData } from 'chart.js'
-import Chart from 'primevue/chart'
-import { computed } from 'vue'
-
-import type { ChartInputSpec } from '@/schemas/nodeDef/nodeDefSchemaV2'
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-
-type ChartWidgetOptions = NonNullable<ChartInputSpec['options']>
-
-const value = defineModel<ChartData>({ required: true })
-
-const props = defineProps<{
-  widget: SimplifiedWidget<ChartData, ChartWidgetOptions>
-  readonly?: boolean
-}>()
-
-const chartType = computed(() => props.widget.options?.type ?? 'line')
-
-const chartData = computed(() => value.value || { labels: [], datasets: [] })
-
-const chartOptions = computed(() => ({
-  responsive: true,
-  maintainAspectRatio: false,
-  plugins: {
-    legend: {
-      labels: {
-        color: '#FFF',
-        usePointStyle: true,
-        pointStyle: 'circle'
-      }
-    }
-  },
-  scales: {
-    x: {
-      ticks: {
-        color: '#9FA2BD'
-      },
-      grid: {
-        display: true,
-        color: '#9FA2BD',
-        drawTicks: false,
-        drawOnChartArea: true,
-        drawBorder: false
-      },
-      border: {
-        display: true,
-        color: '#9FA2BD'
-      }
-    },
-    y: {
-      ticks: {
-        color: '#9FA2BD'
-      },
-      grid: {
-        display: false,
-        drawTicks: false,
-        drawOnChartArea: false,
-        drawBorder: false
-      },
-      border: {
-        display: true,
-        color: '#9FA2BD'
-      }
-    }
-  }
-}))
-</script>

src/components/graph/vueWidgets/WidgetColorPicker.vue

@@ -1,52 +0,0 @@
-<!-- Needs custom color picker for alpha support -->
-<template>
-  <div class="flex flex-col gap-1">
-    <label v-if="widget.name" class="text-sm opacity-80">{{
-      widget.name
-    }}</label>
-    <ColorPicker
-      v-model="localValue"
-      v-bind="filteredProps"
-      :disabled="readonly"
-      inline
-      @update:model-value="onChange"
-    />
-  </div>
-</template>
-
-<script setup lang="ts">
-import ColorPicker from 'primevue/colorpicker'
-import { computed } from 'vue'
-
-import { useWidgetValue } from '@/composables/graph/useWidgetValue'
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-import {
-  PANEL_EXCLUDED_PROPS,
-  filterWidgetProps
-} from '@/utils/widgetPropFilter'
-
-const props = defineProps<{
-  widget: SimplifiedWidget<string>
-  modelValue: string
-  readonly?: boolean
-}>()
-
-const emit = defineEmits<{
-  'update:modelValue': [value: string]
-}>()
-
-// Use the composable for consistent widget value handling
-const { localValue, onChange } = useWidgetValue({
-  widget: props.widget,
-  modelValue: props.modelValue,
-  defaultValue: '#000000',
-  emit
-})
-
-// ColorPicker specific excluded props include panel/overlay classes
-const COLOR_PICKER_EXCLUDED_PROPS = [...PANEL_EXCLUDED_PROPS] as const
-
-const filteredProps = computed(() =>
-  filterWidgetProps(props.widget.options, COLOR_PICKER_EXCLUDED_PROPS)
-)
-</script>

src/components/graph/vueWidgets/WidgetFileUpload.vue

@@ -1,324 +0,0 @@
-<template>
-  <!-- Replace entire widget with image preview when image is loaded -->
-  <!-- Edge-to-edge: -mx-2 removes the parent's p-2 (8px) padding on each side -->
-  <div
-    v-if="hasImageFile"
-    class="relative -mx-2"
-    style="width: calc(100% + 1rem)"
-  >
-    <!-- Select section above image -->
-    <div class="flex items-center justify-between gap-4 mb-2 px-2">
-      <label
-        v-if="widget.name"
-        class="text-xs opacity-80 min-w-[4em] truncate"
-        >{{ widget.name }}</label
-      >
-      <!-- Group select and folder button together on the right -->
-      <div class="flex items-center gap-1">
-        <!-- TODO: finish once we finish value bindings with Litegraph -->
-        <Select
-          :model-value="selectedFile?.name"
-          :options="[selectedFile?.name || '']"
-          :disabled="true"
-          class="min-w-[8em] max-w-[20em] text-xs"
-          size="small"
-          :pt="{
-            option: 'text-xs'
-          }"
-        />
-        <Button
-          icon="pi pi-folder"
-          size="small"
-          class="!w-8 !h-8"
-          :disabled="readonly"
-          @click="triggerFileInput"
-        />
-      </div>
-    </div>
-
-    <!-- Image preview -->
-    <!-- TODO: change hardcoded colors when design system incorporated -->
-    <div class="relative group">
-      <img :src="imageUrl" :alt="selectedFile?.name" class="w-full h-auto" />
-      <!-- Darkening overlay on hover -->
-      <div
-        class="absolute inset-0 bg-black bg-opacity-0 group-hover:bg-opacity-20 transition-all duration-200 pointer-events-none"
-      />
-      <!-- Control buttons in top right on hover -->
-      <div
-        v-if="!readonly"
-        class="absolute top-2 right-2 flex gap-1 opacity-0 group-hover:opacity-100 transition-opacity duration-200"
-      >
-        <!-- Edit button -->
-        <button
-          class="w-6 h-6 rounded flex items-center justify-center transition-all duration-150 focus:outline-none border-none"
-          style="background-color: #262729"
-          @click="handleEdit"
-        >
-          <i class="pi pi-pencil text-white text-xs"></i>
-        </button>
-        <!-- Delete button -->
-        <button
-          class="w-6 h-6 rounded flex items-center justify-center transition-all duration-150 focus:outline-none border-none"
-          style="background-color: #262729"
-          @click="clearFile"
-        >
-          <i class="pi pi-times text-white text-xs"></i>
-        </button>
-      </div>
-    </div>
-  </div>
-
-  <!-- Audio preview when audio file is loaded -->
-  <div
-    v-else-if="hasAudioFile"
-    class="relative -mx-2"
-    style="width: calc(100% + 1rem)"
-  >
-    <!-- Select section above audio player -->
-    <div class="flex items-center justify-between gap-4 mb-2 px-2">
-      <label
-        v-if="widget.name"
-        class="text-xs opacity-80 min-w-[4em] truncate"
-        >{{ widget.name }}</label
-      >
-      <!-- Group select and folder button together on the right -->
-      <div class="flex items-center gap-1">
-        <Select
-          :model-value="selectedFile?.name"
-          :options="[selectedFile?.name || '']"
-          :disabled="true"
-          class="min-w-[8em] max-w-[20em] text-xs"
-          size="small"
-          :pt="{
-            option: 'text-xs'
-          }"
-        />
-        <Button
-          icon="pi pi-folder"
-          size="small"
-          class="!w-8 !h-8"
-          :disabled="readonly"
-          @click="triggerFileInput"
-        />
-      </div>
-    </div>
-
-    <!-- Audio player -->
-    <div class="relative group px-2">
-      <div
-        class="bg-[#1a1b1e] rounded-lg p-4 flex items-center gap-4"
-        style="border: 1px solid #262729"
-      >
-        <!-- Audio icon -->
-        <div class="flex-shrink-0">
-          <i class="pi pi-volume-up text-2xl opacity-60"></i>
-        </div>
-
-        <!-- File info and controls -->
-        <div class="flex-1">
-          <div class="text-sm font-medium mb-1">{{ selectedFile?.name }}</div>
-          <div class="text-xs opacity-60">
-            {{
-              selectedFile ? (selectedFile.size / 1024).toFixed(1) + ' KB' : ''
-            }}
-          </div>
-        </div>
-
-        <!-- Control buttons -->
-        <div v-if="!readonly" class="flex gap-1">
-          <!-- Delete button -->
-          <button
-            class="w-8 h-8 rounded flex items-center justify-center transition-all duration-150 focus:outline-none border-none hover:bg-[#262729]"
-            @click="clearFile"
-          >
-            <i class="pi pi-times text-white text-sm"></i>
-          </button>
-        </div>
-      </div>
-    </div>
-  </div>
-
-  <!-- Show normal file upload UI when no image or audio is loaded -->
-  <div
-    v-else
-    class="flex flex-col gap-1 w-full border border-solid p-1 rounded-lg"
-    :style="{ borderColor: '#262729' }"
-  >
-    <div
-      class="border border-dashed p-1 rounded-md transition-colors duration-200 hover:border-[#5B5E7D]"
-      :style="{ borderColor: '#262729' }"
-    >
-      <div class="flex flex-col items-center gap-2 w-full py-4">
-        <!-- Quick and dirty file type detection for testing -->
-        <!-- eslint-disable-next-line @intlify/vue-i18n/no-raw-text -->
-        <span class="text-xs opacity-60"> Drop your file or </span>
-        <div>
-          <Button
-            label="Browse Files"
-            size="small"
-            class="text-xs"
-            :disabled="readonly"
-            @click="triggerFileInput"
-          />
-        </div>
-      </div>
-    </div>
-  </div>
-  <!-- Hidden file input always available for both states -->
-  <input
-    ref="fileInputRef"
-    type="file"
-    class="hidden"
-    :accept="widget.options?.accept"
-    :multiple="false"
-    :disabled="readonly"
-    @change="handleFileChange"
-  />
-</template>
-
-<script setup lang="ts">
-import Button from 'primevue/button'
-import Select from 'primevue/select'
-import { computed, onUnmounted, ref, watch } from 'vue'
-
-// import { useI18n } from 'vue-i18n' // Commented out for testing
-
-import { useWidgetValue } from '@/composables/graph/useWidgetValue'
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-
-// const { t } = useI18n() // Commented out for testing
-
-const props = defineProps<{
-  widget: SimplifiedWidget<File[] | null>
-  modelValue: File[] | null
-  readonly?: boolean
-}>()
-
-const emit = defineEmits<{
-  'update:modelValue': [value: File[] | null]
-}>()
-
-// Use the composable for consistent widget value handling
-const { localValue, onChange } = useWidgetValue({
-  widget: props.widget,
-  modelValue: props.modelValue,
-  defaultValue: null,
-  emit
-})
-
-const fileInputRef = ref<HTMLInputElement | null>(null)
-
-// Since we only support single file, get the first file
-const selectedFile = computed(() => {
-  const files = localValue.value || []
-  return files.length > 0 ? files[0] : null
-})
-
-// Quick file type detection for testing
-const detectFileType = (file: File) => {
-  const type = file.type?.toLowerCase() || ''
-  const name = file.name?.toLowerCase() || ''
-
-  if (
-    type.startsWith('image/') ||
-    name.match(/\.(jpg|jpeg|png|gif|webp|svg)$/)
-  ) {
-    return 'image'
-  }
-  if (type.startsWith('video/') || name.match(/\.(mp4|webm|ogg|mov)$/)) {
-    return 'video'
-  }
-  if (type.startsWith('audio/') || name.match(/\.(mp3|wav|ogg|flac)$/)) {
-    return 'audio'
-  }
-  if (type === 'application/pdf' || name.endsWith('.pdf')) {
-    return 'pdf'
-  }
-  if (type.includes('zip') || name.match(/\.(zip|rar|7z|tar|gz)$/)) {
-    return 'archive'
-  }
-  return 'file'
-}
-
-// Check if we have an image file
-const hasImageFile = computed(() => {
-  return selectedFile.value && detectFileType(selectedFile.value) === 'image'
-})
-
-// Check if we have an audio file
-const hasAudioFile = computed(() => {
-  return selectedFile.value && detectFileType(selectedFile.value) === 'audio'
-})
-
-// Get image URL for preview
-const imageUrl = computed(() => {
-  if (hasImageFile.value && selectedFile.value) {
-    return URL.createObjectURL(selectedFile.value)
-  }
-  return ''
-})
-
-// // Get audio URL for playback
-// const audioUrl = computed(() => {
-//   if (hasAudioFile.value && selectedFile.value) {
-//     return URL.createObjectURL(selectedFile.value)
-//   }
-//   return ''
-// })
-
-// Clean up image URL when file changes
-watch(imageUrl, (newUrl, oldUrl) => {
-  if (oldUrl && oldUrl !== newUrl) {
-    URL.revokeObjectURL(oldUrl)
-  }
-})
-
-const triggerFileInput = () => {
-  fileInputRef.value?.click()
-}
-
-const handleFileChange = (event: Event) => {
-  const target = event.target as HTMLInputElement
-  if (!props.readonly && target.files && target.files.length > 0) {
-    // Since we only support single file, take the first one
-    const file = target.files[0]
-
-    // Use the composable's onChange handler with an array
-    onChange([file])
-
-    // Reset input to allow selecting same file again
-    target.value = ''
-  }
-}
-
-const clearFile = () => {
-  // Clear the file
-  onChange(null)
-
-  // Reset file input
-  if (fileInputRef.value) {
-    fileInputRef.value.value = ''
-  }
-}
-
-const handleEdit = () => {
-  // TODO: hook up with maskeditor
-}
-
-// Clear file input when value is cleared externally
-watch(localValue, (newValue) => {
-  if (!newValue || newValue.length === 0) {
-    if (fileInputRef.value) {
-      fileInputRef.value.value = ''
-    }
-  }
-})
-
-// Clean up image URL on unmount
-onUnmounted(() => {
-  if (imageUrl.value) {
-    URL.revokeObjectURL(imageUrl.value)
-  }
-})
-</script>

src/components/graph/vueWidgets/WidgetGalleria.vue

@@ -1,123 +0,0 @@
-<template>
-  <div class="flex flex-col gap-1">
-    <Galleria
-      v-model:activeIndex="activeIndex"
-      :value="galleryImages"
-      v-bind="filteredProps"
-      :disabled="readonly"
-      :show-thumbnails="showThumbnails"
-      :show-nav-buttons="showNavButtons"
-      class="max-w-full"
-      :pt="{
-        thumbnails: {
-          class: 'overflow-hidden'
-        },
-        thumbnailContent: {
-          class: 'py-4 px-2'
-        },
-        thumbnailPrevButton: {
-          class: 'm-0'
-        },
-        thumbnailNextButton: {
-          class: 'm-0'
-        }
-      }"
-    >
-      <template #item="{ item }">
-        <img
-          :src="item.itemImageSrc || item.src || item"
-          :alt="item.alt || 'Gallery image'"
-          class="w-full h-auto max-h-64 object-contain"
-        />
-      </template>
-      <template #thumbnail="{ item }">
-        <div class="p-1 w-full h-full">
-          <img
-            :src="item.thumbnailImageSrc || item.src || item"
-            :alt="item.alt || 'Gallery thumbnail'"
-            class="w-full h-full object-cover rounded-lg"
-          />
-        </div>
-      </template>
-    </Galleria>
-  </div>
-</template>
-
-<script setup lang="ts">
-import Galleria from 'primevue/galleria'
-import { computed, ref } from 'vue'
-
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-import {
-  GALLERIA_EXCLUDED_PROPS,
-  filterWidgetProps
-} from '@/utils/widgetPropFilter'
-
-interface GalleryImage {
-  itemImageSrc?: string
-  thumbnailImageSrc?: string
-  src?: string
-  alt?: string
-}
-
-type GalleryValue = string[] | GalleryImage[]
-
-const value = defineModel<GalleryValue>({ required: true })
-
-const props = defineProps<{
-  widget: SimplifiedWidget<GalleryValue>
-  readonly?: boolean
-}>()
-
-const activeIndex = ref(0)
-
-const filteredProps = computed(() =>
-  filterWidgetProps(props.widget.options, GALLERIA_EXCLUDED_PROPS)
-)
-
-const galleryImages = computed(() => {
-  if (!value.value || !Array.isArray(value.value)) return []
-
-  return value.value.map((item, index) => {
-    if (typeof item === 'string') {
-      return {
-        itemImageSrc: item,
-        thumbnailImageSrc: item,
-        alt: `Image ${index + 1}`
-      }
-    }
-    return item
-  })
-})
-
-const showThumbnails = computed(() => {
-  return (
-    props.widget.options?.showThumbnails !== false &&
-    galleryImages.value.length > 1
-  )
-})
-
-const showNavButtons = computed(() => {
-  return (
-    props.widget.options?.showNavButtons !== false &&
-    galleryImages.value.length > 1
-  )
-})
-</script>
-
-<style scoped>
-/* Ensure thumbnail container doesn't overflow */
-:deep(.p-galleria-thumbnails) {
-  overflow: hidden;
-}
-
-/* Constrain thumbnail items to prevent overlap */
-:deep(.p-galleria-thumbnail-item) {
-  flex-shrink: 0;
-}
-
-/* Ensure thumbnail wrapper maintains aspect ratio */
-:deep(.p-galleria-thumbnail) {
-  overflow: hidden;
-}
-</style>

src/components/graph/vueWidgets/WidgetImage.vue

@@ -1,29 +0,0 @@
-<template>
-  <div class="flex flex-col gap-1">
-    <label v-if="widget.name" class="text-sm opacity-80">{{
-      widget.name
-    }}</label>
-    <Image v-bind="filteredProps" :src="widget.value" />
-  </div>
-</template>
-
-<script setup lang="ts">
-import Image from 'primevue/image'
-import { computed } from 'vue'
-
-import type { SimplifiedWidget } from '@/types/simplifiedWidget'
-import {
-  IMAGE_EXCLUDED_PROPS,
-  filterWidgetProps
-} from '@/utils/widgetPropFilter'
-
-// Image widgets typically don't have v-model, they display a source URL/path
-const props = defineProps<{
-  widget: SimplifiedWidget<string>
-  readonly?: boolean
-}>()
-
-const filteredProps = computed(() =>
-  filterWidgetProps(props.widget.options, IMAGE_EXCLUDED_PROPS)
-)
-</script>