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,79 +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 for dependency version changes:**
-   ```bash
-   # Compare package.json between versions to detect dependency updates
-   PREV_PACKAGE_JSON=$(git show ${BASE_TAG}:package.json 2>/dev/null || echo '{}')
-   CURRENT_PACKAGE_JSON=$(cat package.json)
-   
-   # Extract litegraph versions
-   PREV_LITEGRAPH=$(echo "$PREV_PACKAGE_JSON" | grep -o '"@comfyorg/litegraph": "[^"]*"' | grep -o '[0-9][^"]*' || echo "not found")
-   CURRENT_LITEGRAPH=$(echo "$CURRENT_PACKAGE_JSON" | grep -o '"@comfyorg/litegraph": "[^"]*"' | grep -o '[0-9][^"]*' || echo "not found")
-   
-   echo "Litegraph version change: ${PREV_LITEGRAPH} → ${CURRENT_LITEGRAPH}"
-   ```
-
-2. **Generate litegraph changelog if version changed:**
-   ```bash
-   if [ "$PREV_LITEGRAPH" != "$CURRENT_LITEGRAPH" ] && [ "$PREV_LITEGRAPH" != "not found" ]; then
-     echo "📦 Fetching litegraph changes between v${PREV_LITEGRAPH} and v${CURRENT_LITEGRAPH}..."
-     
-     # Clone or update litegraph repo for changelog analysis
-     if [ ! -d ".temp-litegraph" ]; then
-       git clone https://github.com/comfyanonymous/litegraph.js.git .temp-litegraph
-     else
-       cd .temp-litegraph && git fetch --all && cd ..
-     fi
-     
-     # Get litegraph changelog between versions
-     LITEGRAPH_CHANGES=$(cd .temp-litegraph && git log v${PREV_LITEGRAPH}..v${CURRENT_LITEGRAPH} --oneline --no-merges 2>/dev/null || \
-       git log --oneline --no-merges --since="$(git log -1 --format=%ci ${BASE_TAG})" --until="$(git log -1 --format=%ci HEAD)" 2>/dev/null || \
-       echo "Unable to fetch litegraph changes")
-     
-     # Categorize litegraph changes
-     LITEGRAPH_FEATURES=$(echo "$LITEGRAPH_CHANGES" | grep -iE "(feat|feature|add)" || echo "")
-     LITEGRAPH_FIXES=$(echo "$LITEGRAPH_CHANGES" | grep -iE "(fix|bug)" || echo "")
-     LITEGRAPH_BREAKING=$(echo "$LITEGRAPH_CHANGES" | grep -iE "(break|breaking)" || echo "")
-     LITEGRAPH_OTHER=$(echo "$LITEGRAPH_CHANGES" | grep -viE "(feat|feature|add|fix|bug|break|breaking)" || echo "")
-     
-     # Clean up temp directory
-     rm -rf .temp-litegraph
-     
-     echo "✅ Litegraph changelog extracted"
-   else
-     echo "ℹ️  No litegraph version change detected"
-     LITEGRAPH_CHANGES=""
-   fi
-   ```
-
-3. **Check other 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
@@ -256,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
 
@@ -316,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
@@ -348,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
@@ -374,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
@@ -382,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:
@@ -407,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
@@ -420,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
@@ -435,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
@@ -473,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
@@ -541,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
    ```
@@ -592,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
@@ -650,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
@@ -679,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
 

.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/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/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/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:

README.md

@@ -686,12 +686,6 @@ 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
 
 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.

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/fixtures/ComfyPage.ts

@@ -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/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()
-  })
-})

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@comfyorg/comfyui-frontend",
   "private": true,
-  "version": "1.25.2",
+  "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,7 +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.20",
+    "@comfyorg/litegraph": "^0.16.16",
     "@primevue/forms": "^4.2.5",
     "@primevue/themes": "^4.2.5",
     "@sentry/vue": "^8.48.0",
@@ -97,8 +96,6 @@
     "axios": "^1.8.2",
     "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,7 +105,6 @@
     "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",

src/App.vue

@@ -15,14 +15,12 @@ import ProgressSpinner from 'primevue/progressspinner'
 import { computed, onMounted } from 'vue'
 
 import GlobalDialog from '@/components/dialog/GlobalDialog.vue'
-import { useConflictDetection } from '@/composables/useConflictDetection'
 import config from '@/config'
 import { useWorkspaceStore } from '@/stores/workspaceStore'
 
 import { electronAPI, isElectron } from './utils/envUtil'
 
 const workspaceStore = useWorkspaceStore()
-const conflictDetection = useConflictDetection()
 const isLoading = computed<boolean>(() => workspaceStore.spinner)
 const handleKey = (e: KeyboardEvent) => {
   workspaceStore.shiftDown = e.shiftKey
@@ -49,9 +47,5 @@ onMounted(() => {
   if (isElectron()) {
     document.addEventListener('contextmenu', showContextMenu)
   }
-
-  // Initialize conflict detection in background
-  // This runs async and doesn't block UI setup
-  void conflictDetection.initializeConflictDetection()
 })
 </script>

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/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/DotSpinner.vue

@@ -1,130 +0,0 @@
-<template>
-  <div
-    class="inline-flex items-center justify-center"
-    :style="{ width: size + 'px', height: size + 'px' }"
-  >
-    <svg
-      xmlns="http://www.w3.org/2000/svg"
-      :width="size"
-      :height="size"
-      viewBox="0 0 14 14"
-      fill="none"
-      class="animate-spin"
-      :style="{ animationDuration: duration }"
-    >
-      <g clip-path="url(#clip0_776_9582)">
-        <!-- Top dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M7 2.21053C7.61042 2.21053 8.10526 1.71568 8.10526 1.10526C8.10526 0.494843 7.61042 0 7 0C6.38958 0 5.89474 0.494843 5.89474 1.10526C5.89474 1.71568 6.38958 2.21053 7 2.21053Z"
-          :fill="color"
-        />
-        <!-- Left dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.25s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M2.21053 7C2.21053 7.61042 1.71568 8.10526 1.10526 8.10526C0.494843 8.10526 0 7.61042 0 7C0 6.38958 0.494843 5.89474 1.10526 5.89474C1.71568 5.89474 2.21053 6.38958 2.21053 7Z"
-          :fill="color"
-        />
-        <!-- Right dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.5s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M14 7C14 7.61042 13.5052 8.10526 12.8947 8.10526C12.2843 8.10526 11.7895 7.61042 11.7895 7C11.7895 6.38958 12.2843 5.89474 12.8947 5.89474C13.5052 5.89474 14 6.38958 14 7Z"
-          :fill="color"
-        />
-        <!-- Bottom dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.75s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M8.10526 12.8947C8.10526 13.5052 7.61041 14 6.99999 14C6.38957 14 5.89473 13.5052 5.89473 12.8947C5.89473 12.2843 6.38957 11.7895 6.99999 11.7895C7.61041 11.7895 8.10526 12.2843 8.10526 12.8947Z"
-          :fill="color"
-        />
-        <!-- Top-left dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.125s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M2.05039 3.61349C2.48203 4.04513 3.18184 4.04513 3.61347 3.61349C4.0451 3.18186 4.0451 2.48205 3.61347 2.05042C3.18184 1.61878 2.48203 1.61878 2.05039 2.05042C1.61876 2.48205 1.61876 3.18186 2.05039 3.61349Z"
-          :fill="color"
-        />
-        <!-- Bottom-right dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.625s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M11.9496 11.9496C11.518 12.3812 10.8182 12.3812 10.3865 11.9496C9.9549 11.5179 9.9549 10.8181 10.3865 10.3865C10.8182 9.95485 11.518 9.95485 11.9496 10.3865C12.3812 10.8181 12.3812 11.5179 11.9496 11.9496Z"
-          :fill="color"
-        />
-        <!-- Bottom-left dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.875s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M2.05039 11.9496C2.48203 12.3812 3.18184 12.3812 3.61347 11.9496C4.0451 11.5179 4.0451 10.8181 3.61347 10.3865C3.18184 9.95485 2.48203 9.95485 2.05039 10.3865C1.61876 10.8181 1.61876 11.5179 2.05039 11.9496Z"
-          :fill="color"
-        />
-        <!-- Top-right dot -->
-        <path
-          class="dot-animation"
-          style="animation-delay: 0.375s"
-          fill-rule="evenodd"
-          clip-rule="evenodd"
-          d="M11.9496 3.61349C11.518 4.04513 10.8182 4.04513 10.3865 3.61349C9.9549 3.18186 9.9549 2.48205 10.3865 2.05042C10.8182 1.61878 11.518 1.61878 11.9496 2.05042C12.3812 2.48205 12.3812 3.18186 11.9496 3.61349Z"
-          :fill="color"
-        />
-      </g>
-      <defs>
-        <clipPath id="clip0_776_9582">
-          <rect width="14" height="14" fill="white" />
-        </clipPath>
-      </defs>
-    </svg>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-import { useColorPaletteStore } from '@/stores/workspace/colorPaletteStore'
-
-const { size = 24, duration = '2s' } = defineProps<{
-  size?: number
-  duration?: string
-}>()
-
-const colorPaletteStore = useColorPaletteStore()
-
-const color = computed(() =>
-  colorPaletteStore.completedActivePalette.light_theme ? '#2C2B30' : '#D4D4D4'
-)
-</script>
-
-<style scoped>
-.dot-animation {
-  animation: dot-pulse 1s ease-in-out infinite;
-}
-
-@keyframes dot-pulse {
-  0%,
-  80%,
-  100% {
-    opacity: 0.3;
-  }
-  40% {
-    opacity: 1;
-  }
-}
-</style>

src/components/dialog/GlobalDialog.vue

@@ -27,7 +27,7 @@
     />
 
     <template v-if="item.footerComponent" #footer>
-      <component :is="item.footerComponent" v-bind="item.footerProps" />
+      <component :is="item.footerComponent" />
     </template>
   </Dialog>
 </template>

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" />
@@ -31,7 +31,7 @@
       </div>
     </template>
   </ListBox>
-  <div v-if="!isLegacyManager" class="flex justify-end py-3">
+  <div v-if="isManagerInstalled" class="flex justify-end py-3">
     <PackInstallButton
       :disabled="isLoading || !!error || missingNodePacks.length === 0"
       :node-packs="missingNodePacks"
@@ -45,14 +45,14 @@
 <script setup lang="ts">
 import Button from 'primevue/button'
 import ListBox from 'primevue/listbox'
-import { computed, onMounted, ref } from 'vue'
+import { computed } from 'vue'
 
 import NoResultsPlaceholder from '@/components/common/NoResultsPlaceholder.vue'
 import MissingCoreNodesMessage from '@/components/dialog/content/MissingCoreNodesMessage.vue'
 import PackInstallButton from '@/components/dialog/content/manager/button/PackInstallButton.vue'
 import { useMissingNodes } from '@/composables/nodePack/useMissingNodes'
-import { useComfyManagerService } from '@/services/comfyManagerService'
 import { useDialogService } from '@/services/dialogService'
+import { useAboutPanelStore } from '@/stores/aboutPanelStore'
 import type { MissingNodeType } from '@/types/comfy'
 import { ManagerTab } from '@/types/comfyManagerTypes'
 
@@ -60,11 +60,22 @@ const props = defineProps<{
   missingNodeTypes: MissingNodeType[]
 }>()
 
+const aboutPanelStore = useAboutPanelStore()
+
 // Get missing node packs from workflow with loading and error states
 const { missingNodePacks, isLoading, error, missingCoreNodes } =
   useMissingNodes()
 
-const isLegacyManager = ref(false)
+// Determines if ComfyUI-Manager is installed by checking for its badge in the about panel
+// This allows us to conditionally show the Manager button only when the extension is available
+// TODO: Remove this check when Manager functionality is fully migrated into core
+const isManagerInstalled = computed(() => {
+  return aboutPanelStore.badges.some(
+    (badge) =>
+      badge.label.includes('ComfyUI-Manager') ||
+      badge.url.includes('ComfyUI-Manager')
+  )
+})
 
 const uniqueNodes = computed(() => {
   const seenTypes = new Set()
@@ -92,13 +103,6 @@ const openManager = () => {
     initialTab: ManagerTab.Missing
   })
 }
-
-onMounted(async () => {
-  const isLegacyResponse = await useComfyManagerService().isLegacyManagerUI()
-  if (isLegacyResponse?.is_legacy_manager_ui) {
-    isLegacyManager.value = true
-  }
-})
 </script>
 
 <style scoped>

src/components/dialog/content/ManagerProgressDialogContent.test.ts

@@ -30,20 +30,11 @@ const defaultMockTaskLogs = [
 
 vi.mock('@/stores/comfyManagerStore', () => ({
   useComfyManagerStore: vi.fn(() => ({
-    taskLogs: [...defaultMockTaskLogs],
-    succeededTasksLogs: [...defaultMockTaskLogs],
-    failedTasksLogs: [...defaultMockTaskLogs],
-    managerQueue: { historyCount: 2 },
-    isLoading: false
+    taskLogs: [...defaultMockTaskLogs]
   })),
   useManagerProgressDialogStore: vi.fn(() => ({
     isExpanded: true,
-    activeTabIndex: 0,
-    getActiveTabIndex: vi.fn(() => 0),
-    setActiveTabIndex: vi.fn(),
-    toggle: vi.fn(),
-    collapse: mockCollapse,
-    expand: vi.fn()
+    collapse: mockCollapse
   }))
 }))
 

src/components/dialog/content/ManagerProgressDialogContent.vue

@@ -18,7 +18,7 @@
         'max-h-0': !isExpanded
       }"
     >
-      <div v-for="(log, index) in focusedLogs" :key="index">
+      <div v-for="(panel, index) in taskPanels" :key="index">
         <Panel
           :expanded="collapsedPanels[index] || false"
           toggleable
@@ -27,7 +27,7 @@
           <template #header>
             <div class="flex items-center justify-between w-full py-2">
               <div class="flex flex-col text-sm font-medium leading-normal">
-                <span>{{ log.taskName }}</span>
+                <span>{{ panel.taskName }}</span>
                 <span class="text-muted">
                   {{
                     isInProgress(index)
@@ -52,24 +52,24 @@
           </template>
           <div
             :ref="
-              index === focusedLogs.length - 1
+              index === taskPanels.length - 1
                 ? (el) => (lastPanelRef = el as HTMLElement)
                 : undefined
             "
             class="overflow-y-auto h-64 rounded-lg bg-black"
             :class="{
-              'h-64': index !== focusedLogs.length - 1,
-              'flex-grow': index === focusedLogs.length - 1
+              'h-64': index !== taskPanels.length - 1,
+              'flex-grow': index === taskPanels.length - 1
             }"
             @scroll="handleScroll"
           >
             <div class="h-full">
               <div
-                v-for="(logLine, logIndex) in log.logs"
+                v-for="(log, logIndex) in panel.logs"
                 :key="logIndex"
                 class="text-neutral-400 dark-theme:text-muted"
               >
-                <pre class="whitespace-pre-wrap break-words">{{ logLine }}</pre>
+                <pre class="whitespace-pre-wrap break-words">{{ log }}</pre>
               </div>
             </div>
           </div>
@@ -90,23 +90,17 @@ import {
   useManagerProgressDialogStore
 } from '@/stores/comfyManagerStore'
 
-const comfyManagerStore = useComfyManagerStore()
+const { taskLogs } = useComfyManagerStore()
 const progressDialogContent = useManagerProgressDialogStore()
+const managerStore = useComfyManagerStore()
 
 const isInProgress = (index: number) =>
-  index === comfyManagerStore.managerQueue.historyCount - 1 &&
-  comfyManagerStore.isLoading
+  index === taskPanels.value.length - 1 && managerStore.uncompletedCount > 0
 
+const taskPanels = computed(() => taskLogs)
 const isExpanded = computed(() => progressDialogContent.isExpanded)
 const isCollapsed = computed(() => !isExpanded.value)
 
-const focusedLogs = computed(() => {
-  if (progressDialogContent.getActiveTabIndex() === 0) {
-    return comfyManagerStore.succeededTasksLogs
-  }
-  return comfyManagerStore.failedTasksLogs
-})
-
 const collapsedPanels = ref<Record<number, boolean>>({})
 const togglePanel = (index: number) => {
   collapsedPanels.value[index] = !collapsedPanels.value[index]
@@ -121,7 +115,7 @@ const { y: scrollY } = useScroll(sectionsContainerRef, {
 
 const lastPanelRef = ref<HTMLElement | null>(null)
 const isUserScrolling = ref(false)
-const lastPanelLogs = computed(() => focusedLogs.value?.at(-1)?.logs)
+const lastPanelLogs = computed(() => taskPanels.value?.at(-1)?.logs)
 
 const isAtBottom = (el: HTMLElement | null) => {
   if (!el) return false

src/components/dialog/content/manager/ManagerDialogContent.vue

@@ -26,35 +26,6 @@
         }"
       >
         <div class="px-6 flex flex-col h-full">
-          <!-- Conflict Warning Banner -->
-          <div
-            v-if="shouldShowManagerBanner"
-            class="bg-yellow-600 bg-opacity-20 border border-yellow-400 rounded-lg p-4 mt-3 mb-4 flex items-center gap-6 relative"
-          >
-            <i class="pi pi-exclamation-triangle text-yellow-600 text-lg"></i>
-            <div class="flex flex-col gap-2 flex-1">
-              <p class="text-sm font-bold m-0">
-                {{ $t('manager.conflicts.warningBanner.title') }}
-              </p>
-              <p class="text-xs m-0">
-                {{ $t('manager.conflicts.warningBanner.message') }}
-              </p>
-              <p
-                class="text-sm font-bold m-0 cursor-pointer"
-                @click="onClickWarningLink"
-              >
-                {{ $t('manager.conflicts.warningBanner.button') }}
-              </p>
-            </div>
-            <button
-              type="button"
-              class="absolute top-2 right-2 w-6 h-6 border-none outline-none bg-transparent flex items-center justify-center text-yellow-600 rounded transition-colors"
-              :aria-label="$t('g.close')"
-              @click="dismissWarningBanner"
-            >
-              <i class="pi pi-times text-sm"></i>
-            </button>
-          </div>
           <RegistrySearchBar
             v-model:searchQuery="searchQuery"
             v-model:searchMode="searchMode"
@@ -63,7 +34,6 @@
             :suggestions="suggestions"
             :is-missing-tab="isMissingTab"
             :sort-options="sortOptions"
-            :is-update-available-tab="isUpdateAvailableTab"
           />
           <div class="flex-1 overflow-auto">
             <div
@@ -99,9 +69,7 @@
                     :is-selected="
                       selectedNodePacks.some((pack) => pack.id === item.id)
                     "
-                    @click.stop="
-                      (event: MouseEvent) => selectNodePack(item, event)
-                    "
+                    @click.stop="(event) => selectNodePack(item, event)"
                   />
                 </template>
               </VirtualGrid>
@@ -133,8 +101,7 @@ import {
   onMounted,
   onUnmounted,
   ref,
-  watch,
-  watchEffect
+  watch
 } from 'vue'
 import { useI18n } from 'vue-i18n'
 
@@ -152,7 +119,6 @@ import { useManagerStatePersistence } from '@/composables/manager/useManagerStat
 import { useInstalledPacks } from '@/composables/nodePack/useInstalledPacks'
 import { usePackUpdateStatus } from '@/composables/nodePack/usePackUpdateStatus'
 import { useWorkflowPacks } from '@/composables/nodePack/useWorkflowPacks'
-import { useConflictAcknowledgment } from '@/composables/useConflictAcknowledgment'
 import { useRegistrySearch } from '@/composables/useRegistrySearch'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
 import { useComfyRegistryStore } from '@/stores/comfyRegistryStore'
@@ -167,7 +133,6 @@ const { initialTab } = defineProps<{
 const { t } = useI18n()
 const comfyManagerStore = useComfyManagerStore()
 const { getPackById } = useComfyRegistryStore()
-const conflictAcknowledgment = useConflictAcknowledgment()
 const persistedState = useManagerStatePersistence()
 const initialState = persistedState.loadStoredState()
 
@@ -184,13 +149,6 @@ const {
   toggle: toggleSideNav
 } = useResponsiveCollapse()
 
-// Use conflict acknowledgment state from composable
-const {
-  shouldShowManagerBanner,
-  dismissWarningBanner,
-  dismissRedDotNotification
-} = conflictAcknowledgment
-
 const tabs = ref<TabItem[]>([
   { id: ManagerTab.All, label: t('g.all'), icon: 'pi-list' },
   { id: ManagerTab.Installed, label: t('g.installed'), icon: 'pi-box' },
@@ -354,13 +312,6 @@ watch([isAllTab, searchResults], () => {
   displayPacks.value = searchResults.value
 })
 
-const onClickWarningLink = () => {
-  window.open(
-    'https://docs.comfy.org/troubleshooting/custom-node-issues',
-    '_blank'
-  )
-}
-
 const onResultsChange = () => {
   switch (selectedTab.value?.id) {
     case ManagerTab.Installed:
@@ -489,7 +440,6 @@ whenever(selectedNodePack, async () => {
   if (hasMultipleSelections.value) return
   // Only fetch if we haven't already for this pack
   if (lastFetchedPackId.value === pack.id) return
-
   const data = await getPackById.call(pack.id)
   // If selected node hasn't changed since request, merge registry & Algolia data
   if (data?.id === pack.id) {
@@ -522,10 +472,6 @@ watch([searchQuery, selectedTab], () => {
   }
 })
 
-watchEffect(() => {
-  dismissRedDotNotification()
-})
-
 onBeforeUnmount(() => {
   persistedState.persistState({
     selectedTabId: selectedTab.value?.id,

src/components/dialog/content/manager/ManagerHeader.test.ts

@@ -1,82 +0,0 @@
-import { mount } from '@vue/test-utils'
-import { createPinia } from 'pinia'
-import PrimeVue from 'primevue/config'
-import Tag from 'primevue/tag'
-import Tooltip from 'primevue/tooltip'
-import { describe, expect, it } from 'vitest'
-import { createI18n } from 'vue-i18n'
-
-import enMessages from '@/locales/en/main.json'
-
-import ManagerHeader from './ManagerHeader.vue'
-
-const i18n = createI18n({
-  legacy: false,
-  locale: 'en',
-  messages: {
-    en: enMessages
-  }
-})
-
-describe('ManagerHeader', () => {
-  const createWrapper = () => {
-    return mount(ManagerHeader, {
-      global: {
-        plugins: [createPinia(), PrimeVue, i18n],
-        directives: {
-          tooltip: Tooltip
-        },
-        components: {
-          Tag
-        }
-      }
-    })
-  }
-
-  it('renders the component title', () => {
-    const wrapper = createWrapper()
-
-    expect(wrapper.find('h2').text()).toBe(
-      enMessages.manager.discoverCommunityContent
-    )
-  })
-
-  it('displays the legacy manager UI tag', () => {
-    const wrapper = createWrapper()
-
-    const tag = wrapper.find('[data-pc-name="tag"]')
-    expect(tag.exists()).toBe(true)
-    expect(tag.text()).toContain(enMessages.manager.legacyManagerUI)
-  })
-
-  it('applies info severity to the tag', () => {
-    const wrapper = createWrapper()
-
-    const tag = wrapper.find('[data-pc-name="tag"]')
-    expect(tag.classes()).toContain('p-tag-info')
-  })
-
-  it('displays info icon in the tag', () => {
-    const wrapper = createWrapper()
-
-    const icon = wrapper.find('.pi-info-circle')
-    expect(icon.exists()).toBe(true)
-  })
-
-  it('has cursor-help class on the tag', () => {
-    const wrapper = createWrapper()
-
-    const tag = wrapper.find('[data-pc-name="tag"]')
-    expect(tag.classes()).toContain('cursor-help')
-  })
-
-  it('has proper structure with flex container', () => {
-    const wrapper = createWrapper()
-
-    const flexContainer = wrapper.find('.flex.justify-end.ml-auto.pr-4')
-    expect(flexContainer.exists()).toBe(true)
-
-    const tag = flexContainer.find('[data-pc-name="tag"]')
-    expect(tag.exists()).toBe(true)
-  })
-})

src/components/dialog/content/manager/ManagerHeader.vue

@@ -4,22 +4,6 @@
       <h2 class="text-lg font-normal text-left">
         {{ $t('manager.discoverCommunityContent') }}
       </h2>
-      <div class="flex justify-end ml-auto pr-4">
-        <Tag
-          v-tooltip.left="$t('manager.legacyManagerUIDescription')"
-          severity="info"
-          icon="pi pi-info-circle"
-          :value="$t('manager.legacyManagerUI')"
-          class="cursor-help"
-          :pt="{
-            root: { class: 'text-xs' }
-          }"
-        />
-      </div>
     </div>
   </div>
 </template>
-
-<script setup lang="ts">
-import Tag from 'primevue/tag'
-</script>

src/components/dialog/content/manager/NodeConflictDialogContent.vue

@@ -1,231 +0,0 @@
-<template>
-  <div class="w-[552px] flex flex-col">
-    <ContentDivider :width="1" />
-    <div class="px-4 py-6 w-full h-full flex flex-col gap-2">
-      <!-- Description -->
-      <div v-if="showAfterWhatsNew">
-        <p
-          class="text-sm leading-4 text-neutral-800 dark-theme:text-white m-0 mb-4"
-        >
-          {{ $t('manager.conflicts.description') }}
-          <br /><br />
-          {{ $t('manager.conflicts.info') }}
-        </p>
-      </div>
-
-      <!-- Import Failed List Wrapper -->
-      <div
-        v-if="importFailedConflicts.length > 0"
-        class="w-full flex flex-col bg-neutral-200 dark-theme:bg-black min-h-8 rounded-lg"
-      >
-        <div
-          class="w-full h-8 flex items-center justify-between gap-2 pl-4"
-          @click="toggleImportFailedPanel"
-        >
-          <div class="flex-1 flex">
-            <span
-              class="text-xs font-bold text-yellow-600 dark-theme:text-yellow-400 mr-2"
-              >{{ importFailedConflicts.length }}</span
-            >
-            <span
-              class="text-xs font-bold text-neutral-600 dark-theme:text-white"
-              >{{ $t('manager.conflicts.importFailedExtensions') }}</span
-            >
-          </div>
-          <div>
-            <Button
-              :icon="
-                importFailedExpanded
-                  ? 'pi pi-chevron-down text-xs'
-                  : 'pi pi-chevron-right text-xs'
-              "
-              text
-              class="text-neutral-600 dark-theme:text-neutral-300 !bg-transparent"
-            />
-          </div>
-        </div>
-        <!-- Import failed list -->
-        <div
-          v-if="importFailedExpanded"
-          class="py-2 px-4 flex flex-col gap-2.5 max-h-[142px] overflow-y-auto scrollbar-hide"
-        >
-          <div
-            v-for="(packageName, i) in importFailedConflicts"
-            :key="i"
-            class="flex items-center justify-between h-6 px-4 flex-shrink-0 conflict-list-item"
-          >
-            <span class="text-xs text-neutral-600 dark-theme:text-neutral-300">
-              {{ packageName }}
-            </span>
-            <span class="pi pi-info-circle text-sm"></span>
-          </div>
-        </div>
-      </div>
-      <!-- Conflict List Wrapper -->
-      <div
-        class="w-full flex flex-col bg-neutral-200 dark-theme:bg-black min-h-8 rounded-lg"
-      >
-        <div
-          class="w-full h-8 flex items-center justify-between gap-2 pl-4"
-          @click="toggleConflictsPanel"
-        >
-          <div class="flex-1 flex">
-            <span
-              class="text-xs font-bold text-yellow-600 dark-theme:text-yellow-400 mr-2"
-              >{{ allConflictDetails.length }}</span
-            >
-            <span
-              class="text-xs font-bold text-neutral-600 dark-theme:text-white"
-              >{{ $t('manager.conflicts.conflicts') }}</span
-            >
-          </div>
-          <div>
-            <Button
-              :icon="
-                conflictsExpanded
-                  ? 'pi pi-chevron-down text-xs'
-                  : 'pi pi-chevron-right text-xs'
-              "
-              text
-              class="text-neutral-600 dark-theme:text-neutral-300 !bg-transparent"
-            />
-          </div>
-        </div>
-        <!-- Conflicts list -->
-        <div
-          v-if="conflictsExpanded"
-          class="py-2 px-4 flex flex-col gap-2.5 max-h-[142px] overflow-y-auto scrollbar-hide"
-        >
-          <div
-            v-for="(conflict, i) in allConflictDetails"
-            :key="i"
-            class="flex items-center justify-between h-6 px-4 flex-shrink-0 conflict-list-item"
-          >
-            <span
-              class="text-xs text-neutral-600 dark-theme:text-neutral-300"
-              >{{ getConflictMessage(conflict, t) }}</span
-            >
-            <span class="pi pi-info-circle text-sm"></span>
-          </div>
-        </div>
-      </div>
-      <!-- Extension List Wrapper -->
-      <div
-        class="w-full flex flex-col bg-neutral-200 dark-theme:bg-black min-h-8 rounded-lg"
-      >
-        <div
-          class="w-full h-8 flex items-center justify-between gap-2 pl-4"
-          @click="toggleExtensionsPanel"
-        >
-          <div class="flex-1 flex">
-            <span
-              class="text-xs font-bold text-yellow-600 dark-theme:text-yellow-400 mr-2"
-              >{{ conflictData.length }}</span
-            >
-            <span
-              class="text-xs font-bold text-neutral-600 dark-theme:text-white"
-              >{{ $t('manager.conflicts.extensionAtRisk') }}</span
-            >
-          </div>
-          <div>
-            <Button
-              :icon="
-                extensionsExpanded
-                  ? 'pi pi-chevron-down text-xs'
-                  : 'pi pi-chevron-right text-xs'
-              "
-              text
-              class="text-neutral-600 dark-theme:text-neutral-300 !bg-transparent"
-            />
-          </div>
-        </div>
-        <!-- Extension list -->
-        <div
-          v-if="extensionsExpanded"
-          class="py-2 px-4 flex flex-col gap-2.5 max-h-[142px] overflow-y-auto scrollbar-hide"
-        >
-          <div
-            v-for="conflictResult in conflictData"
-            :key="conflictResult.package_id"
-            class="flex items-center justify-between h-6 px-4 flex-shrink-0 conflict-list-item"
-          >
-            <span class="text-xs text-neutral-600 dark-theme:text-neutral-300">
-              {{ conflictResult.package_name }}
-            </span>
-            <span class="pi pi-info-circle text-sm"></span>
-          </div>
-        </div>
-      </div>
-    </div>
-    <ContentDivider :width="1" />
-  </div>
-</template>
-
-<script setup lang="ts">
-import { filter, flatMap, map, some } from 'lodash'
-import Button from 'primevue/button'
-import { computed, ref } from 'vue'
-import { useI18n } from 'vue-i18n'
-
-import ContentDivider from '@/components/common/ContentDivider.vue'
-import { useConflictDetection } from '@/composables/useConflictDetection'
-import { getConflictMessage } from '@/utils/conflictMessageUtil'
-
-interface Props {
-  showAfterWhatsNew?: boolean
-}
-
-const { showAfterWhatsNew } = withDefaults(defineProps<Props>(), {
-  showAfterWhatsNew: false
-})
-
-const { t } = useI18n()
-const { conflictedPackages } = useConflictDetection()
-
-const conflictsExpanded = ref<boolean>(false)
-const extensionsExpanded = ref<boolean>(false)
-const importFailedExpanded = ref<boolean>(false)
-
-const conflictData = computed(() => conflictedPackages.value)
-
-const allConflictDetails = computed(() => {
-  const allConflicts = flatMap(conflictData.value, (result) => result.conflicts)
-  return filter(allConflicts, (conflict) => conflict.type !== 'import_failed')
-})
-
-const packagesWithImportFailed = computed(() => {
-  return filter(conflictData.value, (result) =>
-    some(result.conflicts, (conflict) => conflict.type === 'import_failed')
-  )
-})
-
-const importFailedConflicts = computed(() => {
-  return map(
-    packagesWithImportFailed.value,
-    (result) => result.package_name || result.package_id
-  )
-})
-
-const toggleImportFailedPanel = () => {
-  importFailedExpanded.value = !importFailedExpanded.value
-  conflictsExpanded.value = false
-  extensionsExpanded.value = false
-}
-
-const toggleConflictsPanel = () => {
-  conflictsExpanded.value = !conflictsExpanded.value
-  extensionsExpanded.value = false
-  importFailedExpanded.value = false
-}
-
-const toggleExtensionsPanel = () => {
-  extensionsExpanded.value = !extensionsExpanded.value
-  conflictsExpanded.value = false
-  importFailedExpanded.value = false
-}
-</script>
-<style scoped>
-.conflict-list-item:hover {
-  background-color: rgba(0, 122, 255, 0.2);
-}
-</style>

src/components/dialog/content/manager/NodeConflictFooter.vue

@@ -1,59 +0,0 @@
-<template>
-  <div class="flex items-center justify-between w-full px-3 py-4">
-    <div class="w-full flex items-center justify-between gap-2 pr-1">
-      <Button
-        :label="$t('manager.conflicts.conflictInfoTitle')"
-        text
-        severity="secondary"
-        size="small"
-        icon="pi pi-info-circle"
-        :pt="{
-          label: { class: 'text-sm' }
-        }"
-        @click="handleConflictInfoClick"
-      />
-      <Button
-        v-if="props.buttonText"
-        :label="props.buttonText"
-        severity="secondary"
-        size="small"
-        @click="handleButtonClick"
-      />
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import Button from 'primevue/button'
-
-import { useDialogStore } from '@/stores/dialogStore'
-
-interface Props {
-  buttonText?: string
-  onButtonClick?: () => void
-}
-
-const props = withDefaults(defineProps<Props>(), {
-  buttonText: undefined,
-  onButtonClick: undefined
-})
-
-const dialogStore = useDialogStore()
-
-const handleConflictInfoClick = () => {
-  window.open(
-    'https://docs.comfy.org/troubleshooting/custom-node-issues',
-    '_blank'
-  )
-}
-
-const handleButtonClick = () => {
-  // Close the conflict dialog
-  dialogStore.closeDialog({ key: 'global-node-conflict' })
-
-  // Execute the custom button action if provided
-  if (props.onButtonClick) {
-    props.onButtonClick()
-  }
-}
-</script>

src/components/dialog/content/manager/NodeConflictHeader.vue

@@ -1,12 +0,0 @@
-<template>
-  <div class="h-12 flex items-center justify-between w-full pl-6">
-    <div class="flex items-center gap-2">
-      <!-- Warning Icon -->
-      <i class="pi pi-exclamation-triangle text-lg"></i>
-      <!-- Title -->
-      <p class="text-base font-bold">
-        {{ $t('manager.conflicts.title') }}
-      </p>
-    </div>
-  </div>
-</template>

src/components/dialog/content/manager/PackStatusMessage.vue

@@ -17,10 +17,9 @@
 
 <script setup lang="ts">
 import Message from 'primevue/message'
-import { computed, inject } from 'vue'
+import { computed } from 'vue'
 
 import { components } from '@/types/comfyRegistryTypes'
-import { ImportFailedKey } from '@/types/importFailedTypes'
 
 type PackVersionStatus = components['schemas']['NodeVersionStatus']
 type PackStatus = components['schemas']['NodeStatus']
@@ -33,15 +32,10 @@ type StatusProps = {
   severity: MessageSeverity
 }
 
-const { statusType, hasCompatibilityIssues } = defineProps<{
+const { statusType } = defineProps<{
   statusType: Status
-  hasCompatibilityIssues?: boolean
 }>()
 
-// Inject import failed context from parent
-const importFailedContext = inject(ImportFailedKey)
-const importFailed = importFailedContext?.importFailed
-
 const statusPropsMap: Record<Status, StatusProps> = {
   NodeStatusActive: {
     label: 'active',
@@ -77,14 +71,10 @@ const statusPropsMap: Record<Status, StatusProps> = {
   }
 }
 
-const statusLabel = computed(() => {
-  if (importFailed?.value) return 'importFailed'
-  if (hasCompatibilityIssues) return 'conflicting'
-  return statusPropsMap[statusType]?.label || 'unknown'
-})
-
-const statusSeverity = computed(() => {
-  if (hasCompatibilityIssues || importFailed?.value) return 'error'
-  return statusPropsMap[statusType]?.severity || 'secondary'
-})
+const statusLabel = computed(
+  () => statusPropsMap[statusType]?.label || 'unknown'
+)
+const statusSeverity = computed(
+  () => statusPropsMap[statusType]?.severity || 'secondary'
+)
 </script>

src/components/dialog/content/manager/PackVersionBadge.test.ts

@@ -6,18 +6,11 @@ import { nextTick } from 'vue'
 import { createI18n } from 'vue-i18n'
 
 import enMessages from '@/locales/en/main.json'
+import { SelectedVersion } from '@/types/comfyManagerTypes'
 
 import PackVersionBadge from './PackVersionBadge.vue'
 import PackVersionSelectorPopover from './PackVersionSelectorPopover.vue'
 
-// Mock config to prevent __COMFYUI_FRONTEND_VERSION__ error
-vi.mock('@/config', () => ({
-  default: {
-    app_title: 'ComfyUI',
-    app_version: '1.0.0'
-  }
-}))
-
 const mockNodePack = {
   id: 'test-pack',
   name: 'Test Pack',
@@ -127,7 +120,7 @@ describe('PackVersionBadge', () => {
 
     const badge = wrapper.find('[role="button"]')
     expect(badge.exists()).toBe(true)
-    expect(badge.find('span').text()).toBe('nightly')
+    expect(badge.find('span').text()).toBe(SelectedVersion.NIGHTLY)
   })
 
   it('falls back to NIGHTLY when nodePack.id is missing', () => {
@@ -141,7 +134,7 @@ describe('PackVersionBadge', () => {
 
     const badge = wrapper.find('[role="button"]')
     expect(badge.exists()).toBe(true)
-    expect(badge.find('span').text()).toBe('nightly')
+    expect(badge.find('span').text()).toBe(SelectedVersion.NIGHTLY)
   })
 
   it('toggles the popover when button is clicked', async () => {

src/components/dialog/content/manager/PackVersionBadge.vue

@@ -1,8 +1,8 @@
 <template>
   <div>
     <div
-      class="inline-flex items-center gap-1 rounded-2xl text-xs cursor-pointer py-1"
-      :class="{ 'bg-gray-100 dark-theme:bg-neutral-700 px-1.5': fill }"
+      class="inline-flex items-center gap-1 rounded-2xl text-xs cursor-pointer px-2 py-1"
+      :class="{ 'bg-gray-100 dark-theme:bg-neutral-700': fill }"
       aria-haspopup="true"
       role="button"
       tabindex="0"
@@ -12,7 +12,8 @@
     >
       <i
         v-if="isUpdateAvailable"
-        class="pi pi-arrow-circle-up text-blue-600 text-xs"
+        class="pi pi-arrow-circle-up text-blue-600"
+        style="font-size: 8px"
       />
       <span>{{ installedVersion }}</span>
       <i class="pi pi-chevron-right" style="font-size: 8px" />
@@ -21,7 +22,7 @@
     <Popover
       ref="popoverRef"
       :pt="{
-        content: { class: 'p-0 shadow-lg' }
+        content: { class: 'px-0' }
       }"
     >
       <PackVersionSelectorPopover
@@ -41,7 +42,8 @@ import { computed, ref, watch } from 'vue'
 import PackVersionSelectorPopover from '@/components/dialog/content/manager/PackVersionSelectorPopover.vue'
 import { usePackUpdateStatus } from '@/composables/nodePack/usePackUpdateStatus'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import type { components } from '@/types/comfyRegistryTypes'
+import { SelectedVersion } from '@/types/comfyManagerTypes'
+import { components } from '@/types/comfyRegistryTypes'
 import { isSemVer } from '@/utils/formatUtil'
 
 const TRUNCATED_HASH_LENGTH = 7
@@ -62,11 +64,11 @@ const popoverRef = ref()
 const managerStore = useComfyManagerStore()
 
 const installedVersion = computed(() => {
-  if (!nodePack.id) return 'nightly'
+  if (!nodePack.id) return SelectedVersion.NIGHTLY
   const version =
     managerStore.installedPacks[nodePack.id]?.ver ??
     nodePack.latest_version?.version ??
-    'nightly'
+    SelectedVersion.NIGHTLY
 
   // If Git hash, truncate to 7 characters
   return isSemVer(version) ? version : version.slice(0, TRUNCATED_HASH_LENGTH)

src/components/dialog/content/manager/PackVersionSelectorPopover.test.ts

@@ -3,32 +3,18 @@ import { createPinia } from 'pinia'
 import Button from 'primevue/button'
 import PrimeVue from 'primevue/config'
 import Listbox from 'primevue/listbox'
-import Select from 'primevue/select'
-import Tooltip from 'primevue/tooltip'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { nextTick } from 'vue'
 import { createI18n } from 'vue-i18n'
 
-import VerifiedIcon from '@/components/icons/VerifiedIcon.vue'
 import enMessages from '@/locales/en/main.json'
-
-// SelectedVersion is now using direct strings instead of enum
+import { SelectedVersion } from '@/types/comfyManagerTypes'
 
 import PackVersionSelectorPopover from './PackVersionSelectorPopover.vue'
 
 // Default mock versions for reference
 const defaultMockVersions = [
-  {
-    version: '1.0.0',
-    createdAt: '2023-01-01',
-    supported_os: ['windows', 'linux'],
-    supported_accelerators: ['CPU'],
-    supported_comfyui_version: '>=0.1.0',
-    supported_comfyui_frontend_version: '>=1.0.0',
-    supported_python_version: '>=3.8',
-    is_banned: false,
-    has_registry_data: true
-  },
+  { version: '1.0.0', createdAt: '2023-01-01' },
   { version: '0.9.0', createdAt: '2022-12-01' },
   { version: '0.8.0', createdAt: '2022-11-01' }
 ]
@@ -36,24 +22,13 @@ const defaultMockVersions = [
 const mockNodePack = {
   id: 'test-pack',
   name: 'Test Pack',
-  latest_version: {
-    version: '1.0.0',
-    supported_os: ['windows', 'linux'],
-    supported_accelerators: ['CPU'],
-    supported_comfyui_version: '>=0.1.0',
-    supported_comfyui_frontend_version: '>=1.0.0',
-    supported_python_version: '>=3.8',
-    is_banned: false,
-    has_registry_data: true
-  },
-  repository: 'https://github.com/user/repo',
-  has_registry_data: true
+  latest_version: { version: '1.0.0' },
+  repository: 'https://github.com/user/repo'
 }
 
 // Create mock functions
 const mockGetPackVersions = vi.fn()
 const mockInstallPack = vi.fn().mockResolvedValue(undefined)
-const mockCheckNodeCompatibility = vi.fn()
 
 // Mock the registry service
 vi.mock('@/services/comfyRegistryService', () => ({
@@ -74,13 +49,6 @@ vi.mock('@/stores/comfyManagerStore', () => ({
   }))
 }))
 
-// Mock the conflict detection composable
-vi.mock('@/composables/useConflictDetection', () => ({
-  useConflictDetection: vi.fn(() => ({
-    checkNodeCompatibility: mockCheckNodeCompatibility
-  }))
-}))
-
 const waitForPromises = async () => {
   await new Promise((resolve) => setTimeout(resolve, 16))
   await nextTick()
@@ -91,9 +59,6 @@ describe('PackVersionSelectorPopover', () => {
     vi.clearAllMocks()
     mockGetPackVersions.mockReset()
     mockInstallPack.mockReset().mockResolvedValue(undefined)
-    mockCheckNodeCompatibility
-      .mockReset()
-      .mockReturnValue({ hasConflict: false, conflicts: [] })
   })
 
   const mountComponent = ({
@@ -113,12 +78,7 @@ describe('PackVersionSelectorPopover', () => {
       global: {
         plugins: [PrimeVue, createPinia(), i18n],
         components: {
-          Listbox,
-          VerifiedIcon,
-          Select
-        },
-        directives: {
-          tooltip: Tooltip
+          Listbox
         }
       }
     })
@@ -160,15 +120,14 @@ describe('PackVersionSelectorPopover', () => {
 
     const options = listbox.props('options')!
     // Check that we have both special options and version options
-    // Latest version (1.0.0) should be excluded from the version list to avoid duplication
-    expect(options.length).toBe(defaultMockVersions.length + 1) // 2 special options + version options minus 1 duplicate
+    expect(options.length).toBe(defaultMockVersions.length + 2) // 2 special options + version options
 
     // Check that special options exist
-    expect(options.some((o) => o.value === 'nightly')).toBe(true)
-    expect(options.some((o) => o.value === 'latest')).toBe(true)
+    expect(options.some((o) => o.value === SelectedVersion.NIGHTLY)).toBe(true)
+    expect(options.some((o) => o.value === SelectedVersion.LATEST)).toBe(true)
 
-    // Check that version options exist (excluding latest version 1.0.0)
-    expect(options.some((o) => o.value === '1.0.0')).toBe(false) // Should be excluded as it's the latest
+    // Check that version options exist
+    expect(options.some((o) => o.value === '1.0.0')).toBe(true)
     expect(options.some((o) => o.value === '0.9.0')).toBe(true)
     expect(options.some((o) => o.value === '0.8.0')).toBe(true)
   })
@@ -345,7 +304,7 @@ describe('PackVersionSelectorPopover', () => {
       await waitForPromises()
       const listbox = wrapper.findComponent(Listbox)
       expect(listbox.exists()).toBe(true)
-      expect(listbox.props('modelValue')).toBe('nightly')
+      expect(listbox.props('modelValue')).toBe(SelectedVersion.NIGHTLY)
     })
 
     it('defaults to nightly when publisher name is "Unclaimed"', async () => {
@@ -366,343 +325,7 @@ describe('PackVersionSelectorPopover', () => {
       await waitForPromises()
       const listbox = wrapper.findComponent(Listbox)
       expect(listbox.exists()).toBe(true)
-      expect(listbox.props('modelValue')).toBe('nightly')
-    })
-  })
-
-  describe('version compatibility checking', () => {
-    it('shows warning icon for incompatible versions', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      // Mock compatibility check to return conflict for specific version
-      mockCheckNodeCompatibility.mockImplementation((versionData) => {
-        if (versionData.supported_os?.includes('linux')) {
-          return {
-            hasConflict: true,
-            conflicts: [
-              {
-                type: 'os',
-                current_value: 'windows',
-                required_value: 'linux'
-              }
-            ]
-          }
-        }
-        return { hasConflict: false, conflicts: [] }
-      })
-
-      const nodePackWithCompatibility = {
-        ...mockNodePack,
-        supported_os: ['linux'],
-        supported_accelerators: ['CUDA']
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: nodePackWithCompatibility }
-      })
-      await waitForPromises()
-
-      // Check that compatibility checking function was called
-      expect(mockCheckNodeCompatibility).toHaveBeenCalled()
-
-      // The warning icon should be shown for incompatible versions
-      const warningIcons = wrapper.findAll('.pi-exclamation-triangle')
-      expect(warningIcons.length).toBeGreaterThan(0)
-    })
-
-    it('shows verified icon for compatible versions', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      // Mock compatibility check to return no conflicts
-      mockCheckNodeCompatibility.mockReturnValue({
-        hasConflict: false,
-        conflicts: []
-      })
-
-      const wrapper = mountComponent()
-      await waitForPromises()
-
-      // Check that compatibility checking function was called
-      expect(mockCheckNodeCompatibility).toHaveBeenCalled()
-
-      // The verified icon should be shown for compatible versions
-      // Look for the VerifiedIcon component or SVG elements
-      const verifiedIcons = wrapper.findAll('svg')
-      expect(verifiedIcons.length).toBeGreaterThan(0)
-    })
-
-    it('calls checkVersionCompatibility with correct version data', async () => {
-      // Set up the mock for versions with specific supported data
-      const versionsWithCompatibility = [
-        {
-          version: '1.0.0',
-          supported_os: ['windows', 'linux'],
-          supported_accelerators: ['CUDA', 'CPU'],
-          supported_comfyui_version: '>=0.1.0',
-          supported_comfyui_frontend_version: '>=1.0.0'
-        }
-      ]
-      mockGetPackVersions.mockResolvedValueOnce(versionsWithCompatibility)
-
-      const nodePackWithCompatibility = {
-        ...mockNodePack,
-        supported_os: ['windows'],
-        supported_accelerators: ['CPU'],
-        supported_comfyui_version: '>=0.1.0',
-        supported_comfyui_frontend_version: '>=1.0.0',
-        latest_version: {
-          version: '1.0.0',
-          supported_os: ['windows', 'linux'],
-          supported_accelerators: ['CPU'], // latest_version data takes precedence
-          supported_comfyui_version: '>=0.1.0',
-          supported_comfyui_frontend_version: '>=1.0.0',
-          supported_python_version: '>=3.8',
-          is_banned: false,
-          has_registry_data: true
-        }
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: nodePackWithCompatibility }
-      })
-      await waitForPromises()
-
-      // Clear previous calls from component mounting/rendering
-      mockCheckNodeCompatibility.mockClear()
-
-      // Trigger compatibility check by accessing getVersionCompatibility
-      const vm = wrapper.vm as any
-      vm.getVersionCompatibility('1.0.0')
-
-      // Verify that checkNodeCompatibility was called with correct data
-      // Since 1.0.0 is the latest version, it should use latest_version data
-      expect(mockCheckNodeCompatibility).toHaveBeenCalledWith({
-        supported_os: ['windows', 'linux'],
-        supported_accelerators: ['CPU'], // latest_version data takes precedence
-        supported_comfyui_version: '>=0.1.0',
-        supported_comfyui_frontend_version: '>=1.0.0',
-        supported_python_version: '>=3.8',
-        is_banned: false,
-        has_registry_data: true,
-        version: '1.0.0'
-      })
-    })
-
-    it('shows version conflict warnings for ComfyUI and frontend versions', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      // Mock compatibility check to return version conflicts
-      mockCheckNodeCompatibility.mockImplementation((versionData) => {
-        const conflicts = []
-        if (versionData.supported_comfyui_version) {
-          conflicts.push({
-            type: 'comfyui_version',
-            current_value: '0.5.0',
-            required_value: versionData.supported_comfyui_version
-          })
-        }
-        if (versionData.supported_comfyui_frontend_version) {
-          conflicts.push({
-            type: 'frontend_version',
-            current_value: '1.0.0',
-            required_value: versionData.supported_comfyui_frontend_version
-          })
-        }
-        return {
-          hasConflict: conflicts.length > 0,
-          conflicts
-        }
-      })
-
-      const nodePackWithVersionRequirements = {
-        ...mockNodePack,
-        supported_comfyui_version: '>=1.0.0',
-        supported_comfyui_frontend_version: '>=2.0.0'
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: nodePackWithVersionRequirements }
-      })
-      await waitForPromises()
-
-      // Check that compatibility checking function was called
-      expect(mockCheckNodeCompatibility).toHaveBeenCalled()
-
-      // The warning icon should be shown for version incompatible packages
-      const warningIcons = wrapper.findAll('.pi-exclamation-triangle')
-      expect(warningIcons.length).toBeGreaterThan(0)
-    })
-
-    it('handles latest and nightly versions using nodePack data', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      const nodePackWithCompatibility = {
-        ...mockNodePack,
-        supported_os: ['windows'],
-        supported_accelerators: ['CPU'],
-        supported_comfyui_version: '>=0.1.0',
-        supported_comfyui_frontend_version: '>=1.0.0',
-        latest_version: {
-          ...mockNodePack.latest_version,
-          supported_os: ['windows'], // Match nodePack data for test consistency
-          supported_accelerators: ['CPU'], // Match nodePack data for test consistency
-          supported_python_version: '>=3.8',
-          is_banned: false,
-          has_registry_data: true
-        }
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: nodePackWithCompatibility }
-      })
-      await waitForPromises()
-
-      const vm = wrapper.vm as any
-
-      // Clear previous calls from component mounting/rendering
-      mockCheckNodeCompatibility.mockClear()
-
-      // Test latest version
-      vm.getVersionCompatibility('latest')
-      expect(mockCheckNodeCompatibility).toHaveBeenCalledWith({
-        supported_os: ['windows'],
-        supported_accelerators: ['CPU'],
-        supported_comfyui_version: '>=0.1.0',
-        supported_comfyui_frontend_version: '>=1.0.0',
-        supported_python_version: '>=3.8',
-        is_banned: false,
-        has_registry_data: true,
-        version: '1.0.0'
-      })
-
-      // Clear for next test call
-      mockCheckNodeCompatibility.mockClear()
-
-      // Test nightly version
-      vm.getVersionCompatibility('nightly')
-      expect(mockCheckNodeCompatibility).toHaveBeenCalledWith({
-        id: 'test-pack',
-        name: 'Test Pack',
-        supported_os: ['windows'],
-        supported_accelerators: ['CPU'],
-        supported_comfyui_version: '>=0.1.0',
-        supported_comfyui_frontend_version: '>=1.0.0',
-        repository: 'https://github.com/user/repo',
-        has_registry_data: true,
-        latest_version: {
-          supported_os: ['windows'],
-          supported_accelerators: ['CPU'],
-          supported_python_version: '>=3.8',
-          is_banned: false,
-          has_registry_data: true,
-          version: '1.0.0',
-          supported_comfyui_version: '>=0.1.0',
-          supported_comfyui_frontend_version: '>=1.0.0'
-        }
-      })
-    })
-
-    it('shows banned package warnings', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      // Mock compatibility check to return banned conflicts
-      mockCheckNodeCompatibility.mockImplementation((versionData) => {
-        if (versionData.is_banned === true) {
-          return {
-            hasConflict: true,
-            conflicts: [
-              {
-                type: 'banned',
-                current_value: 'installed',
-                required_value: 'not_banned'
-              }
-            ]
-          }
-        }
-        return { hasConflict: false, conflicts: [] }
-      })
-
-      const bannedNodePack = {
-        ...mockNodePack,
-        is_banned: true,
-        latest_version: {
-          ...mockNodePack.latest_version,
-          is_banned: true
-        }
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: bannedNodePack }
-      })
-      await waitForPromises()
-
-      // Check that compatibility checking function was called
-      expect(mockCheckNodeCompatibility).toHaveBeenCalled()
-
-      // Open the dropdown to see the options
-      const select = wrapper.find('.p-select')
-      if (!select.exists()) {
-        // Try alternative selector
-        const selectButton = wrapper.find('[aria-haspopup="listbox"]')
-        if (selectButton.exists()) {
-          await selectButton.trigger('click')
-        }
-      } else {
-        await select.trigger('click')
-      }
-      await wrapper.vm.$nextTick()
-
-      // The warning icon should be shown for banned packages in the dropdown options
-      const warningIcons = wrapper.findAll('.pi-exclamation-triangle')
-      expect(warningIcons.length).toBeGreaterThan(0)
-    })
-
-    it('shows security pending warnings', async () => {
-      // Set up the mock for versions
-      mockGetPackVersions.mockResolvedValueOnce(defaultMockVersions)
-
-      // Mock compatibility check to return security pending conflicts
-      mockCheckNodeCompatibility.mockImplementation((versionData) => {
-        if (versionData.has_registry_data === false) {
-          return {
-            hasConflict: true,
-            conflicts: [
-              {
-                type: 'pending',
-                current_value: 'no_registry_data',
-                required_value: 'registry_data_available'
-              }
-            ]
-          }
-        }
-        return { hasConflict: false, conflicts: [] }
-      })
-
-      const securityPendingNodePack = {
-        ...mockNodePack,
-        has_registry_data: false,
-        latest_version: {
-          ...mockNodePack.latest_version,
-          has_registry_data: false
-        }
-      }
-
-      const wrapper = mountComponent({
-        props: { nodePack: securityPendingNodePack }
-      })
-      await waitForPromises()
-
-      // Check that compatibility checking function was called
-      expect(mockCheckNodeCompatibility).toHaveBeenCalled()
-
-      // The warning icon should be shown for security pending packages
-      const warningIcons = wrapper.findAll('.pi-exclamation-triangle')
-      expect(warningIcons.length).toBeGreaterThan(0)
+      expect(listbox.props('modelValue')).toBe(SelectedVersion.NIGHTLY)
     })
   })
 })

src/components/dialog/content/manager/PackVersionSelectorPopover.vue

@@ -1,10 +1,8 @@
 <template>
-  <div class="w-64 pt-1">
-    <div class="py-2">
-      <span class="pl-3 text-md font-semibold text-neutral-500">
-        {{ $t('manager.selectVersion') }}
-      </span>
-    </div>
+  <div class="w-64 mt-2">
+    <span class="pl-3 text-muted text-md font-semibold opacity-70">
+      {{ $t('manager.selectVersion') }}
+    </span>
     <div
       v-if="isLoadingVersions || isQueueing"
       class="text-center text-muted py-4 flex flex-col items-center"
@@ -25,44 +23,24 @@
       v-model="selectedVersion"
       option-label="label"
       option-value="value"
-      :options="processedVersionOptions"
+      :options="versionOptions"
       :highlight-on-select="false"
-      class="w-full max-h-[50vh] border-none shadow-none rounded-md"
-      :pt="{
-        listContainer: { class: 'scrollbar-hide' }
-      }"
+      class="my-3 w-full max-h-[50vh] border-none shadow-none"
     >
       <template #option="slotProps">
         <div class="flex justify-between items-center w-full p-1">
-          <div class="flex items-center gap-2">
-            <template v-if="slotProps.option.value === 'nightly'">
-              <div class="w-4"></div>
-            </template>
-            <template v-else>
-              <i
-                v-if="slotProps.option.hasConflict"
-                v-tooltip="{
-                  value: slotProps.option.conflictMessage,
-                  showDelay: 300
-                }"
-                class="pi pi-exclamation-triangle text-yellow-500"
-              />
-              <VerifiedIcon v-else :size="20" class="relative right-0.5" />
-            </template>
-            <span>{{ slotProps.option.label }}</span>
-          </div>
+          <span>{{ slotProps.option.label }}</span>
           <i
-            v-if="slotProps.option.isSelected"
+            v-if="selectedVersion === slotProps.option.value"
             class="pi pi-check text-highlight"
           />
         </div>
       </template>
     </Listbox>
     <ContentDivider class="my-2" />
-    <div class="flex justify-end gap-2 py-1 px-3">
+    <div class="flex justify-end gap-2 p-1 px-3">
       <Button
         text
-        class="text-sm"
         severity="secondary"
         :label="$t('g.cancel')"
         :disabled="isQueueing"
@@ -71,7 +49,7 @@
       <Button
         severity="secondary"
         :label="$t('g.install')"
-        class="py-2.5 px-4 text-sm dark-theme:bg-unset bg-black/80 dark-theme:text-unset text-neutral-100 rounded-lg"
+        class="py-3 px-4 dark-theme:bg-unset bg-black/80 dark-theme:text-unset text-neutral-100 rounded-lg"
         :disabled="isQueueing"
         @click="handleSubmit"
       />
@@ -84,18 +62,19 @@ import { whenever } from '@vueuse/core'
 import Button from 'primevue/button'
 import Listbox from 'primevue/listbox'
 import ProgressSpinner from 'primevue/progressspinner'
-import { computed, onMounted, ref } from 'vue'
+import { onMounted, ref } from 'vue'
 import { useI18n } from 'vue-i18n'
 
 import ContentDivider from '@/components/common/ContentDivider.vue'
 import NoResultsPlaceholder from '@/components/common/NoResultsPlaceholder.vue'
-import VerifiedIcon from '@/components/icons/VerifiedIcon.vue'
-import { useConflictDetection } from '@/composables/useConflictDetection'
 import { useComfyRegistryService } from '@/services/comfyRegistryService'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import type { components } from '@/types/comfyRegistryTypes'
-import { components as ManagerComponents } from '@/types/generatedManagerTypes'
-import { getJoinedConflictMessages } from '@/utils/conflictMessageUtil'
+import {
+  ManagerChannel,
+  ManagerDatabaseSource,
+  SelectedVersion
+} from '@/types/comfyManagerTypes'
+import { components } from '@/types/comfyRegistryTypes'
 import { isSemVer } from '@/utils/formatUtil'
 
 const { nodePack } = defineProps<{
@@ -110,30 +89,27 @@ const emit = defineEmits<{
 const { t } = useI18n()
 const registryService = useComfyRegistryService()
 const managerStore = useComfyManagerStore()
-const { checkNodeCompatibility } = useConflictDetection()
 
 const isQueueing = ref(false)
 
-const selectedVersion = ref<string>('latest')
-
+const selectedVersion = ref<string>(SelectedVersion.LATEST)
 onMounted(() => {
-  const initialVersion = getInitialSelectedVersion() ?? 'latest'
+  const initialVersion = getInitialSelectedVersion() ?? SelectedVersion.LATEST
   selectedVersion.value =
     // Use NIGHTLY when version is a Git hash
-    isSemVer(initialVersion) ? initialVersion : 'nightly'
+    isSemVer(initialVersion) ? initialVersion : SelectedVersion.NIGHTLY
 })
 
 const getInitialSelectedVersion = () => {
   if (!nodePack.id) return
 
+  // If unclaimed, set selected version to nightly
+  if (nodePack.publisher?.name === 'Unclaimed') return SelectedVersion.NIGHTLY
+
   // If node pack is installed, set selected version to the installed version
   if (managerStore.isPackInstalled(nodePack.id))
     return managerStore.getInstalledPackVersion(nodePack.id)
 
-  // If unclaimed, set selected version to nightly
-  if (nodePack.publisher?.name === 'Unclaimed')
-    return 'nightly' as ManagerComponents['schemas']['SelectedVersion']
-
   // If node pack is not installed, set selected version to latest
   return nodePack.latest_version?.version
 }
@@ -150,9 +126,6 @@ const versionOptions = ref<
   }[]
 >([])
 
-// Store fetched versions with their full data
-const fetchedVersions = ref<components['schemas']['NodeVersion'][]>([])
-
 const isLoadingVersions = ref(false)
 
 const onNodePackChange = async () => {
@@ -160,35 +133,26 @@ const onNodePackChange = async () => {
 
   // Fetch versions from the registry
   const versions = await fetchVersions()
-  fetchedVersions.value = versions
-
-  // Get latest version number to exclude from the list
-  const latestVersionNumber = nodePack.latest_version?.version
-
   const availableVersionOptions = versions
     .map((version) => ({
       value: version.version ?? '',
       label: version.version ?? ''
     }))
-    .filter((option) => option.value && option.value !== latestVersionNumber) // Exclude latest version from the list
-
-  // Add Latest option with actual version number
-  const latestLabel = latestVersionNumber
-    ? `${t('manager.latestVersion')} (${latestVersionNumber})`
-    : t('manager.latestVersion')
+    .filter((option) => option.value)
 
+  // Add Latest option
   const defaultVersions = [
     {
-      value: 'latest' as ManagerComponents['schemas']['SelectedVersion'],
-      label: latestLabel
+      value: SelectedVersion.LATEST,
+      label: t('manager.latestVersion')
     }
   ]
 
   // Add Nightly option if there is a non-empty `repository` field
   if (nodePack.repository?.length) {
     defaultVersions.push({
-      value: 'nightly' as ManagerComponents['schemas']['SelectedVersion'],
-      label: t('manager.nightlyVersion') // Keep as just "nightly" - no version number
+      value: SelectedVersion.NIGHTLY,
+      label: t('manager.nightlyVersion')
     })
   }
 
@@ -208,95 +172,16 @@ whenever(
 
 const handleSubmit = async () => {
   isQueueing.value = true
-  if (!nodePack.id) {
-    throw new Error('Node ID is required for installation')
-  }
-
-  // Convert 'latest' to actual version number for installation
-  const actualVersion =
-    selectedVersion.value === 'latest'
-      ? nodePack.latest_version?.version ?? 'latest'
-      : selectedVersion.value
-
   await managerStore.installPack.call({
     id: nodePack.id,
-    version: actualVersion,
     repository: nodePack.repository ?? '',
-    channel: 'default' as ManagerComponents['schemas']['ManagerChannel'],
-    mode: 'cache' as ManagerComponents['schemas']['ManagerDatabaseSource'],
-    selected_version: actualVersion
+    channel: ManagerChannel.DEFAULT,
+    mode: ManagerDatabaseSource.CACHE,
+    version: selectedVersion.value,
+    selected_version: selectedVersion.value
   })
 
   isQueueing.value = false
   emit('submit')
 }
-
-const getVersionData = (version: string) => {
-  const latestVersionNumber = nodePack.latest_version?.version
-  const useLatestVersionData =
-    version === 'latest' || version === latestVersionNumber
-
-  if (useLatestVersionData) {
-    const latestVersionData = nodePack.latest_version
-    return {
-      ...latestVersionData
-    }
-  }
-
-  const versionData = fetchedVersions.value.find((v) => v.version === version)
-  if (versionData) {
-    return {
-      ...versionData
-    }
-  }
-
-  // Fallback to nodePack data
-  return {
-    ...nodePack
-  }
-}
-
-// Main function to get version compatibility info
-const getVersionCompatibility = (version: string) => {
-  const versionData = getVersionData(version)
-  const compatibility = checkNodeCompatibility(versionData)
-
-  const conflictMessage = compatibility.hasConflict
-    ? getJoinedConflictMessages(compatibility.conflicts, t)
-    : ''
-
-  return {
-    hasConflict: compatibility.hasConflict,
-    conflictMessage
-  }
-}
-
-// Helper to determine if an option is selected.
-const isOptionSelected = (optionValue: string) => {
-  if (selectedVersion.value === optionValue) {
-    return true
-  }
-  if (
-    optionValue === 'latest' &&
-    selectedVersion.value === nodePack.latest_version?.version
-  ) {
-    return true
-  }
-  return false
-}
-
-// Checks if an option is selected, treating 'latest' as an alias for the actual latest version number.
-const processedVersionOptions = computed(() => {
-  return versionOptions.value.map((option) => {
-    const compatibility = getVersionCompatibility(option.value)
-    const isSelected = isOptionSelected(option.value)
-
-    return {
-      ...option,
-      hasConflict: compatibility.hasConflict,
-      conflictMessage: compatibility.conflictMessage,
-      isSelected: isSelected
-    }
-  })
-})
 </script>

src/components/dialog/content/manager/button/PackActionButton.vue

@@ -1,24 +1,18 @@
 <template>
   <Button
     outlined
-    class="!m-0 p-0 max-w-[120px] rounded-lg text-gray-900 dark-theme:text-gray-50"
+    class="!m-0 p-0 rounded-lg text-gray-900 dark-theme:text-gray-50"
     :class="[
       variant === 'black'
         ? 'bg-neutral-900 text-white border-neutral-900'
-        : variant === 'red'
-          ? 'border-red-500'
-          : 'border-neutral-700',
+        : 'border-neutral-700',
       fullWidth ? 'w-full' : 'w-min-content'
     ]"
     :disabled="loading"
     v-bind="$attrs"
     @click="onClick"
   >
-    <span class="py-1.5 px-3 whitespace-nowrap text-xs flex items-center gap-2">
-      <i
-        v-if="hasWarning && !loading"
-        class="pi pi-exclamation-triangle text-yellow-500"
-      ></i>
+    <span class="py-2 px-3 whitespace-nowrap">
       <template v-if="loading">
         {{ loadingMessage ?? $t('g.loading') }}
       </template>
@@ -34,18 +28,15 @@ import Button from 'primevue/button'
 
 const {
   label,
-  loading = false,
   loadingMessage,
   fullWidth = false,
-  variant = 'default',
-  hasWarning = false
+  variant = 'default'
 } = defineProps<{
   label: string
   loading?: boolean
   loadingMessage?: string
   fullWidth?: boolean
-  variant?: 'default' | 'black' | 'red'
-  hasWarning?: boolean
+  variant?: 'default' | 'black'
 }>()
 
 const emit = defineEmits<{

src/components/dialog/content/manager/button/PackEnableToggle.test.ts

@@ -11,22 +11,9 @@ import { useComfyManagerStore } from '@/stores/comfyManagerStore'
 
 import PackEnableToggle from './PackEnableToggle.vue'
 
-// Mock lodash functions used throughout the app
-vi.mock('lodash', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('lodash')>()
-  return {
-    ...actual,
-    debounce: <T extends (...args: any[]) => any>(fn: T) => fn,
-    memoize: <T extends (...args: any[]) => any>(fn: T) => fn
-  }
-})
-
-// Mock config to prevent __COMFYUI_FRONTEND_VERSION__ error
-vi.mock('@/config', () => ({
-  default: {
-    app_title: 'ComfyUI',
-    app_version: '1.0.0'
-  }
+// Mock debounce to execute immediately
+vi.mock('lodash', () => ({
+  debounce: <T extends (...args: any[]) => any>(fn: T) => fn
 }))
 
 const mockNodePack = {

src/components/dialog/content/manager/button/PackEnableToggle.vue

@@ -1,26 +1,6 @@
 <template>
-  <div class="flex items-center gap-2">
-    <div
-      v-if="hasConflict"
-      v-tooltip="{
-        value: $t('manager.conflicts.warningTooltip'),
-        showDelay: 300
-      }"
-      class="flex items-center justify-center w-6 h-6 cursor-pointer"
-      @click="showConflictModal(true)"
-    >
-      <i class="pi pi-exclamation-triangle text-yellow-500 text-xl"></i>
-    </div>
+  <div class="flex items-center">
     <ToggleSwitch
-      v-if="!canToggleDirectly"
-      :model-value="isEnabled"
-      :disabled="isLoading"
-      :readonly="!canToggleDirectly"
-      aria-label="Enable or disable pack"
-      @focus="handleToggleInteraction"
-    />
-    <ToggleSwitch
-      v-else
       :model-value="isEnabled"
       :disabled="isLoading"
       aria-label="Enable or disable pack"
@@ -33,96 +13,52 @@
 import { debounce } from 'lodash'
 import ToggleSwitch from 'primevue/toggleswitch'
 import { computed, ref } from 'vue'
-import { useI18n } from 'vue-i18n'
 
-import { useConflictAcknowledgment } from '@/composables/useConflictAcknowledgment'
-import { useDialogService } from '@/services/dialogService'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import { useConflictDetectionStore } from '@/stores/conflictDetectionStore'
+import {
+  InstallPackParams,
+  ManagerChannel,
+  SelectedVersion
+} from '@/types/comfyManagerTypes'
 import type { components } from '@/types/comfyRegistryTypes'
-import { components as ManagerComponents } from '@/types/generatedManagerTypes'
 
 const TOGGLE_DEBOUNCE_MS = 256
 
-const { nodePack, hasConflict } = defineProps<{
+const { nodePack } = defineProps<{
   nodePack: components['schemas']['Node']
-  hasConflict?: boolean
 }>()
 
-const { t } = useI18n()
-const { isPackEnabled, enablePack, disablePack } = useComfyManagerStore()
-const { getConflictsForPackageByID } = useConflictDetectionStore()
-const { showNodeConflictDialog } = useDialogService()
-const { acknowledgmentState, markConflictsAsSeen } = useConflictAcknowledgment()
+const { isPackEnabled, enablePack, disablePack, installedPacks } =
+  useComfyManagerStore()
 
 const isLoading = ref(false)
 
 const isEnabled = computed(() => isPackEnabled(nodePack.id))
-const packageConflict = computed(() =>
-  getConflictsForPackageByID(nodePack.id || '')
-)
-
-const canToggleDirectly = computed(() => {
-  return !(
-    hasConflict &&
-    !acknowledgmentState.value.modal_dismissed &&
-    packageConflict.value
+const version = computed(() => {
+  const id = nodePack.id
+  if (!id) return SelectedVersion.NIGHTLY
+  return (
+    installedPacks[id]?.ver ??
+    nodePack.latest_version?.version ??
+    SelectedVersion.NIGHTLY
   )
 })
 
-const showConflictModal = (skipModalDismissed: boolean) => {
-  let modal_dismissed = acknowledgmentState.value.modal_dismissed
-  if (skipModalDismissed) modal_dismissed = false
-  if (packageConflict.value && !modal_dismissed) {
-    showNodeConflictDialog({
-      conflictedPackages: [packageConflict.value],
-      buttonText: !isEnabled.value
-        ? t('manager.conflicts.enableAnyway')
-        : t('manager.conflicts.understood'),
-      onButtonClick: async () => {
-        if (!isEnabled.value) {
-          await handleEnable()
-        }
-      },
-      dialogComponentProps: {
-        onClose: () => {
-          markConflictsAsSeen()
-        }
-      }
-    })
-  }
-}
-
-const handleEnable = () => {
-  if (!nodePack.id) {
-    throw new Error('Node ID is required for enabling')
-  }
-  return enablePack.call({
+const handleEnable = () =>
+  enablePack.call({
     id: nodePack.id,
-    version:
-      nodePack.latest_version?.version ??
-      ('latest' as ManagerComponents['schemas']['SelectedVersion']),
-    selected_version:
-      nodePack.latest_version?.version ??
-      ('latest' as ManagerComponents['schemas']['SelectedVersion']),
+    version: version.value,
+    selected_version: version.value,
     repository: nodePack.repository ?? '',
-    channel: 'default' as ManagerComponents['schemas']['ManagerChannel'],
-    mode: 'cache' as ManagerComponents['schemas']['ManagerDatabaseSource'],
-    skip_post_install: false
+    channel: ManagerChannel.DEFAULT,
+    mode: 'default' as InstallPackParams['mode']
   })
-}
 
-const handleDisable = () => {
-  if (!nodePack.id) {
-    throw new Error('Node ID is required for disabling')
-  }
-  return disablePack({
+const handleDisable = () =>
+  disablePack({
     id: nodePack.id,
-    version:
-      nodePack.latest_version?.version ??
-      ('latest' as ManagerComponents['schemas']['SelectedVersion'])
+    version: version.value
   })
-}
 
 const handleToggle = async (enable: boolean) => {
   if (isLoading.value) return
@@ -131,23 +67,10 @@ const handleToggle = async (enable: boolean) => {
   if (enable) {
     await handleEnable()
   } else {
-    await handleDisable()
+    handleDisable()
   }
   isLoading.value = false
 }
 
-const onToggle = debounce(
-  (enable: boolean) => {
-    void handleToggle(enable)
-  },
-  TOGGLE_DEBOUNCE_MS,
-  { trailing: true }
-)
-
-const handleToggleInteraction = async (event: Event) => {
-  if (!canToggleDirectly.value) {
-    event.preventDefault()
-    showConflictModal(false)
-  }
-}
+const onToggle = debounce(handleToggle, TOGGLE_DEBOUNCE_MS, { trailing: true })
 </script>

src/components/dialog/content/manager/button/PackInstallButton.vue

@@ -9,8 +9,8 @@
     :variant="variant"
     :loading="isInstalling"
     :loading-message="$t('g.installing')"
-    :has-warning="hasConflict"
     @action="installAllPacks"
+    @click="onClick"
   />
 </template>
 
@@ -18,51 +18,44 @@
 import { inject, ref } from 'vue'
 
 import PackActionButton from '@/components/dialog/content/manager/button/PackActionButton.vue'
-import { t } from '@/i18n'
-import { useDialogService } from '@/services/dialogService'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import { IsInstallingKey } from '@/types/comfyManagerTypes'
-import type { components } from '@/types/comfyRegistryTypes'
 import {
-  type ConflictDetail,
-  type ConflictDetectionResult
-} from '@/types/conflictDetectionTypes'
-import { components as ManagerComponents } from '@/types/generatedManagerTypes'
+  IsInstallingKey,
+  ManagerChannel,
+  ManagerDatabaseSource,
+  SelectedVersion
+} from '@/types/comfyManagerTypes'
+import type { components } from '@/types/comfyRegistryTypes'
 
 type NodePack = components['schemas']['Node']
 
-const { nodePacks, variant, label, hasConflict, conflictInfo } = defineProps<{
+const { nodePacks, variant, label } = defineProps<{
   nodePacks: NodePack[]
   variant?: 'default' | 'black'
   label?: string
-  hasConflict?: boolean
-  conflictInfo?: ConflictDetail[]
 }>()
 
 const isInstalling = inject(IsInstallingKey, ref(false))
+
+const onClick = (): void => {
+  isInstalling.value = true
+}
+
 const managerStore = useComfyManagerStore()
-const { showNodeConflictDialog } = useDialogService()
-
-const createPayload = (
-  installItem: NodePack
-): ManagerComponents['schemas']['InstallPackParams'] => {
-  if (!installItem.id) {
-    throw new Error('Node ID is required for installation')
-  }
 
+const createPayload = (installItem: NodePack) => {
   const isUnclaimedPack = installItem.publisher?.name === 'Unclaimed'
   const versionToInstall = isUnclaimedPack
-    ? ('nightly' as ManagerComponents['schemas']['SelectedVersion'])
-    : installItem.latest_version?.version ??
-      ('latest' as ManagerComponents['schemas']['SelectedVersion'])
+    ? SelectedVersion.NIGHTLY
+    : installItem.latest_version?.version ?? SelectedVersion.LATEST
 
   return {
     id: installItem.id,
-    version: versionToInstall,
     repository: installItem.repository ?? '',
-    channel: 'dev' as ManagerComponents['schemas']['ManagerChannel'],
-    mode: 'cache' as ManagerComponents['schemas']['ManagerDatabaseSource'],
-    selected_version: versionToInstall
+    channel: ManagerChannel.DEV,
+    mode: ManagerDatabaseSource.CACHE,
+    selected_version: versionToInstall,
+    version: versionToInstall
   }
 }
 
@@ -72,35 +65,14 @@ const installPack = (item: NodePack) =>
 const installAllPacks = async () => {
   if (!nodePacks?.length) return
 
-  if (hasConflict && conflictInfo) {
-    const conflictedPackages: ConflictDetectionResult[] = nodePacks.map(
-      (pack) => ({
-        package_id: pack.id || '',
-        package_name: pack.name || '',
-        has_conflict: true,
-        conflicts: conflictInfo || [],
-        is_compatible: false
-      })
-    )
-
-    showNodeConflictDialog({
-      conflictedPackages,
-      buttonText: t('manager.conflicts.installAnyway'),
-      onButtonClick: async () => {
-        // Proceed with installation
-        isInstalling.value = true
-        await performInstallation(nodePacks)
-      }
-    })
-    return
-  }
-  // No conflicts or conflicts acknowledged - proceed with installation
   isInstalling.value = true
-  await performInstallation(nodePacks)
-}
 
-const performInstallation = async (packs: NodePack[]) => {
-  await Promise.all(packs.map(installPack))
+  const uninstalledPacks = nodePacks.filter(
+    (pack) => !managerStore.isPackInstalled(pack.id)
+  )
+  if (!uninstalledPacks.length) return
+
+  await Promise.all(uninstalledPacks.map(installPack))
   managerStore.installPack.clear()
 }
 </script>

src/components/dialog/content/manager/button/PackUninstallButton.vue

@@ -6,7 +6,7 @@
         ? $t('manager.uninstallSelected')
         : $t('manager.uninstall')
     "
-    variant="red"
+    severity="danger"
     :loading-message="$t('manager.uninstalling')"
     @action="uninstallItems"
   />
@@ -15,6 +15,7 @@
 <script setup lang="ts">
 import PackActionButton from '@/components/dialog/content/manager/button/PackActionButton.vue'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
+import type { ManagerPackInfo } from '@/types/comfyManagerTypes'
 import type { components } from '@/types/comfyRegistryTypes'
 
 type NodePack = components['schemas']['Node']
@@ -25,16 +26,16 @@ const { nodePacks } = defineProps<{
 
 const managerStore = useComfyManagerStore()
 
-const uninstallPack = (item: NodePack) => {
-  if (!item.id) {
-    throw new Error('Node ID is required for uninstallation')
+const createPayload = (uninstallItem: NodePack): ManagerPackInfo => {
+  return {
+    id: uninstallItem.id,
+    version: uninstallItem.latest_version?.version
   }
-  return managerStore.uninstallPack({
-    id: item.id,
-    version: item.latest_version?.version ?? ''
-  })
 }
 
+const uninstallPack = (item: NodePack) =>
+  managerStore.uninstallPack(createPayload(item))
+
 const uninstallItems = async () => {
   if (!nodePacks?.length) return
   await Promise.all(nodePacks.map(uninstallPack))

src/components/dialog/content/manager/button/PackUpdateButton.vue

@@ -1,78 +0,0 @@
-<template>
-  <PackActionButton
-    v-bind="$attrs"
-    variant="black"
-    :label="$t('manager.updateAll')"
-    :loading="isUpdating"
-    :loading-message="$t('g.updating')"
-    @action="updateAllPacks"
-  />
-</template>
-
-<script setup lang="ts">
-import { ref } from 'vue'
-
-import PackActionButton from '@/components/dialog/content/manager/button/PackActionButton.vue'
-import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import type { components } from '@/types/comfyRegistryTypes'
-
-type NodePack = components['schemas']['Node']
-
-const { nodePacks } = defineProps<{
-  nodePacks: NodePack[]
-}>()
-
-const isUpdating = ref<boolean>(false)
-
-const managerStore = useComfyManagerStore()
-
-const createPayload = (updateItem: NodePack) => {
-  return {
-    id: updateItem.id!,
-    version: updateItem.latest_version!.version!
-  }
-}
-
-const updatePack = async (item: NodePack) => {
-  if (!item.id || !item.latest_version?.version) {
-    console.warn('Pack missing required id or version:', item)
-    return
-  }
-  await managerStore.updatePack.call(createPayload(item))
-}
-
-const updateAllPacks = async () => {
-  if (!nodePacks?.length) {
-    console.warn('No packs provided for update')
-    return
-  }
-
-  isUpdating.value = true
-
-  const updatablePacks = nodePacks.filter((pack) =>
-    managerStore.isPackInstalled(pack.id)
-  )
-
-  if (!updatablePacks.length) {
-    console.info('No installed packs available for update')
-    isUpdating.value = false
-    return
-  }
-
-  console.info(`Starting update of ${updatablePacks.length} packs`)
-
-  try {
-    await Promise.all(updatablePacks.map(updatePack))
-    managerStore.updatePack.clear()
-    console.info('All packs updated successfully')
-  } catch (error) {
-    console.error('Pack update failed:', error)
-    console.error(
-      'Failed packs info:',
-      updatablePacks.map((p) => p.id)
-    )
-  } finally {
-    isUpdating.value = false
-  }
-}
-</script>

src/components/dialog/content/manager/infoPanel/InfoPanel.vue

@@ -2,26 +2,20 @@
   <template v-if="nodePack">
     <div class="flex flex-col h-full z-40 overflow-hidden relative">
       <div class="top-0 z-10 px-6 pt-6 w-full">
-        <InfoPanelHeader
-          :node-packs="[nodePack]"
-          :has-conflict="hasCompatibilityIssues"
-        />
+        <InfoPanelHeader :node-packs="[nodePack]" />
       </div>
       <div
         ref="scrollContainer"
-        class="p-6 pt-2 overflow-y-auto flex-1 text-sm scrollbar-hide"
+        class="p-6 pt-2 overflow-y-auto flex-1 text-sm hidden-scrollbar"
       >
         <div class="mb-6">
           <MetadataRow
-            v-if="!importFailed && isPackInstalled(nodePack.id)"
+            v-if="isPackInstalled(nodePack.id)"
             :label="t('manager.filter.enabled')"
             class="flex"
             style="align-items: center"
           >
-            <PackEnableToggle
-              :node-pack="nodePack"
-              :has-conflict="hasCompatibilityIssues"
-            />
+            <PackEnableToggle :node-pack="nodePack" />
           </MetadataRow>
           <MetadataRow
             v-for="item in infoItems"
@@ -35,7 +29,6 @@
               :status-type="
                 nodePack.status as components['schemas']['NodeVersionStatus']
               "
-              :has-compatibility-issues="hasCompatibilityIssues"
             />
           </MetadataRow>
           <MetadataRow :label="t('manager.version')">
@@ -43,11 +36,7 @@
           </MetadataRow>
         </div>
         <div class="mb-6 overflow-hidden">
-          <InfoTabs
-            :node-pack="nodePack"
-            :has-compatibility-issues="hasCompatibilityIssues"
-            :conflict-result="conflictResult"
-          />
+          <InfoTabs :node-pack="nodePack" />
         </div>
       </div>
     </div>
@@ -70,14 +59,9 @@ import PackEnableToggle from '@/components/dialog/content/manager/button/PackEna
 import InfoPanelHeader from '@/components/dialog/content/manager/infoPanel/InfoPanelHeader.vue'
 import InfoTabs from '@/components/dialog/content/manager/infoPanel/InfoTabs.vue'
 import MetadataRow from '@/components/dialog/content/manager/infoPanel/MetadataRow.vue'
-import { useConflictDetection } from '@/composables/useConflictDetection'
-import { useImportFailedDetection } from '@/composables/useImportFailedDetection'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import { useConflictDetectionStore } from '@/stores/conflictDetectionStore'
 import { IsInstallingKey } from '@/types/comfyManagerTypes'
 import { components } from '@/types/comfyRegistryTypes'
-import type { ConflictDetectionResult } from '@/types/conflictDetectionTypes'
-import { ImportFailedKey } from '@/types/importFailedTypes'
 
 interface InfoItem {
   key: string
@@ -91,55 +75,18 @@ const { nodePack } = defineProps<{
 
 const scrollContainer = ref<HTMLElement | null>(null)
 
-const { isPackInstalled } = useComfyManagerStore()
-const isInstalled = computed(() => isPackInstalled(nodePack.id))
+const managerStore = useComfyManagerStore()
+const isInstalled = computed(() => managerStore.isPackInstalled(nodePack.id))
 const isInstalling = ref(false)
 provide(IsInstallingKey, isInstalling)
 whenever(isInstalled, () => {
   isInstalling.value = false
 })
 
-const { checkNodeCompatibility } = useConflictDetection()
-const { getConflictsForPackageByID } = useConflictDetectionStore()
+const { isPackInstalled } = useComfyManagerStore()
 
 const { t, d, n } = useI18n()
 
-// Check compatibility once and pass to children
-const conflictResult = computed((): ConflictDetectionResult | null => {
-  // For installed packages, use stored conflict data
-  if (isInstalled.value && nodePack.id) {
-    return getConflictsForPackageByID(nodePack.id) || null
-  }
-
-  // For non-installed packages, perform compatibility check
-  const compatibility = checkNodeCompatibility(nodePack)
-
-  if (compatibility.hasConflict) {
-    return {
-      package_id: nodePack.id || '',
-      package_name: nodePack.name || '',
-      has_conflict: true,
-      conflicts: compatibility.conflicts,
-      is_compatible: false
-    }
-  }
-
-  return null
-})
-
-const hasCompatibilityIssues = computed(() => {
-  return conflictResult.value?.has_conflict
-})
-
-const packageId = computed(() => nodePack.id || '')
-const { importFailed, showImportFailedDialog } =
-  useImportFailedDetection(packageId)
-
-provide(ImportFailedKey, {
-  importFailed,
-  showImportFailedDialog
-})
-
 const infoItems = computed<InfoItem[]>(() => [
   {
     key: 'publisher',
@@ -181,3 +128,17 @@ whenever(
   { immediate: true }
 )
 </script>
+<style scoped>
+.hidden-scrollbar {
+  /* Firefox */
+  scrollbar-width: none;
+
+  &::-webkit-scrollbar {
+    width: 1px;
+  }
+
+  &::-webkit-scrollbar-thumb {
+    background-color: transparent;
+  }
+}
+</style>

src/components/dialog/content/manager/infoPanel/InfoPanelHeader.vue

@@ -1,36 +1,28 @@
 <template>
-  <div v-if="nodePacks?.length" class="flex flex-col items-center">
+  <div v-if="nodePacks?.length" class="flex flex-col items-center mb-6">
     <slot name="thumbnail">
-      <PackIcon :node-pack="nodePacks[0]" width="204" height="106" />
+      <PackIcon :node-pack="nodePacks[0]" width="24" height="24" />
     </slot>
     <h2
       class="text-2xl font-bold text-center mt-4 mb-2"
       style="word-break: break-all"
     >
       <slot name="title">
-        <span class="inline-block text-base">{{ nodePacks[0].name }}</span>
+        {{ nodePacks[0].name }}
       </slot>
     </h2>
-    <div
-      v-if="!importFailed"
-      class="mt-2 mb-4 w-full max-w-xs flex justify-center"
-    >
+    <div class="mt-2 mb-4 w-full max-w-xs flex justify-center">
       <slot name="install-button">
         <PackUninstallButton
           v-if="isAllInstalled"
           v-bind="$attrs"
           :node-packs="nodePacks"
         />
-        <PackInstallButton
-          v-else
-          v-bind="$attrs"
-          :node-packs="nodePacks"
-          :has-conflict="hasConflict"
-        />
+        <PackInstallButton v-else v-bind="$attrs" :node-packs="nodePacks" />
       </slot>
     </div>
   </div>
-  <div v-else class="flex flex-col items-center">
+  <div v-else class="flex flex-col items-center mb-6">
     <NoResultsPlaceholder
       :message="$t('manager.status.unknown')"
       :title="$t('manager.tryAgainLater')"
@@ -39,7 +31,7 @@
 </template>
 
 <script setup lang="ts">
-import { inject, ref, watch } from 'vue'
+import { ref, watch } from 'vue'
 
 import NoResultsPlaceholder from '@/components/common/NoResultsPlaceholder.vue'
 import PackInstallButton from '@/components/dialog/content/manager/button/PackInstallButton.vue'
@@ -47,19 +39,13 @@ import PackUninstallButton from '@/components/dialog/content/manager/button/Pack
 import PackIcon from '@/components/dialog/content/manager/packIcon/PackIcon.vue'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
 import { components } from '@/types/comfyRegistryTypes'
-import { ImportFailedKey } from '@/types/importFailedTypes'
 
-const { nodePacks, hasConflict } = defineProps<{
+const { nodePacks } = defineProps<{
   nodePacks: components['schemas']['Node'][]
-  hasConflict?: boolean
 }>()
 
 const managerStore = useComfyManagerStore()
 
-// Inject import failed context from parent
-const importFailedContext = inject(ImportFailedKey)
-const importFailed = importFailedContext?.importFailed
-
 const isAllInstalled = ref(false)
 watch(
   [() => nodePacks, () => managerStore.installedPacks],

src/components/dialog/content/manager/infoPanel/InfoPanelMultiItem.vue

@@ -6,12 +6,8 @@
           <PackIconStacked :node-packs="nodePacks" />
         </template>
         <template #title>
-          <div class="mt-5">
-            <span class="inline-block mr-2 text-blue-500 text-base">{{
-              nodePacks.length
-            }}</span>
-            <span class="text-base">{{ $t('manager.packsSelected') }}</span>
-          </div>
+          {{ nodePacks.length }}
+          {{ $t('manager.packsSelected') }}
         </template>
         <template #install-button>
           <PackInstallButton :full-width="true" :node-packs="nodePacks" />

src/components/dialog/content/manager/infoPanel/InfoTabs.vue

@@ -1,31 +1,15 @@
 <template>
   <div class="overflow-hidden">
     <Tabs :value="activeTab">
-      <TabList class="overflow-x-auto scrollbar-hide">
-        <Tab v-if="hasCompatibilityIssues" value="warning" class="p-2 mr-6">
-          <div class="flex items-center gap-1">
-            <span>⚠️</span>
-            {{ importFailed ? $t('g.error') : $t('g.warning') }}
-          </div>
-        </Tab>
-        <Tab value="description" class="p-2 mr-6">
+      <TabList>
+        <Tab value="description">
           {{ $t('g.description') }}
         </Tab>
-        <Tab value="nodes" class="p-2">
+        <Tab value="nodes">
           {{ $t('g.nodes') }}
         </Tab>
       </TabList>
-      <TabPanels class="overflow-auto py-4 px-2">
-        <TabPanel
-          v-if="hasCompatibilityIssues"
-          value="warning"
-          class="bg-transparent"
-        >
-          <WarningTabPanel
-            :node-pack="nodePack"
-            :conflict-result="conflictResult"
-          />
-        </TabPanel>
+      <TabPanels class="overflow-auto">
         <TabPanel value="description">
           <DescriptionTabPanel :node-pack="nodePack" />
         </TabPanel>
@@ -43,25 +27,16 @@ import TabList from 'primevue/tablist'
 import TabPanel from 'primevue/tabpanel'
 import TabPanels from 'primevue/tabpanels'
 import Tabs from 'primevue/tabs'
-import { computed, inject, ref, watchEffect } from 'vue'
+import { computed, ref } from 'vue'
 
 import DescriptionTabPanel from '@/components/dialog/content/manager/infoPanel/tabs/DescriptionTabPanel.vue'
 import NodesTabPanel from '@/components/dialog/content/manager/infoPanel/tabs/NodesTabPanel.vue'
-import WarningTabPanel from '@/components/dialog/content/manager/infoPanel/tabs/WarningTabPanel.vue'
 import { components } from '@/types/comfyRegistryTypes'
-import type { ConflictDetectionResult } from '@/types/conflictDetectionTypes'
-import { ImportFailedKey } from '@/types/importFailedTypes'
 
-const { nodePack, hasCompatibilityIssues, conflictResult } = defineProps<{
+const { nodePack } = defineProps<{
   nodePack: components['schemas']['Node']
-  hasCompatibilityIssues?: boolean
-  conflictResult?: ConflictDetectionResult | null
 }>()
 
-// Inject import failed context from parent
-const importFailedContext = inject(ImportFailedKey)
-const importFailed = importFailedContext?.importFailed
-
 const nodeNames = computed(() => {
   // @ts-expect-error comfy_nodes is an Algolia-specific field
   const { comfy_nodes } = nodePack
@@ -69,17 +44,4 @@ const nodeNames = computed(() => {
 })
 
 const activeTab = ref('description')
-
-// Watch for compatibility issues and automatically switch to warning tab
-watchEffect(
-  () => {
-    if (hasCompatibilityIssues) {
-      activeTab.value = 'warning'
-    } else if (activeTab.value === 'warning') {
-      // If currently on warning tab but no issues, switch to description
-      activeTab.value = 'description'
-    }
-  },
-  { flush: 'post' }
-)
 </script>

src/components/dialog/content/manager/infoPanel/InfoTextSection.vue

@@ -1,7 +1,7 @@
 <template>
   <div class="flex flex-col gap-4 text-sm">
     <div v-for="(section, index) in sections" :key="index" class="mb-4">
-      <div class="mb-3">
+      <div class="mb-1">
         {{ section.title }}
       </div>
       <div class="text-muted break-words">

src/components/dialog/content/manager/infoPanel/MetadataRow.vue

@@ -1,6 +1,6 @@
 <template>
   <div class="flex py-1.5 text-xs">
-    <div class="w-1/3 truncate pr-2 text-muted">{{ label }}</div>
+    <div class="w-1/3 truncate pr-2 text-muted">{{ label }}:</div>
     <div class="w-2/3">
       <slot>{{ value }}</slot>
     </div>

src/components/dialog/content/manager/infoPanel/tabs/DescriptionTabPanel.vue

@@ -1,5 +1,5 @@
 <template>
-  <div class="overflow-hidden">
+  <div class="mt-4 overflow-hidden">
     <InfoTextSection
       v-if="nodePack?.description"
       :sections="descriptionSections"

src/components/dialog/content/manager/infoPanel/tabs/NodesTabPanel.vue

@@ -1,5 +1,5 @@
 <template>
-  <div class="flex flex-col gap-4 text-sm">
+  <div class="flex flex-col gap-4 mt-4 text-sm">
     <template v-if="mappedNodeDefs?.length">
       <div
         v-for="nodeDef in mappedNodeDefs"

src/components/dialog/content/manager/infoPanel/tabs/WarningTabPanel.vue

@@ -1,44 +0,0 @@
-<template>
-  <div class="flex flex-col gap-3">
-    <button
-      v-if="importFailedInfo"
-      class="cursor-pointer outline-none border-none inline-flex items-center justify-end bg-transparent gap-1"
-      @click="showImportFailedDialog"
-    >
-      <i class="pi pi-code text-base"></i>
-      <span class="dark-theme:text-white text-sm">{{
-        t('serverStart.openLogs')
-      }}</span>
-    </button>
-    <div
-      v-for="(conflict, index) in conflictResult?.conflicts || []"
-      :key="index"
-      class="p-3 bg-yellow-800/20 rounded-md"
-    >
-      <div class="flex justify-between items-center">
-        <div class="text-sm break-words flex-1">
-          {{ getConflictMessage(conflict, $t) }}
-        </div>
-      </div>
-    </div>
-  </div>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-import { useImportFailedDetection } from '@/composables/useImportFailedDetection'
-import { t } from '@/i18n'
-import { components } from '@/types/comfyRegistryTypes'
-import { ConflictDetectionResult } from '@/types/conflictDetectionTypes'
-import { getConflictMessage } from '@/utils/conflictMessageUtil'
-
-const { nodePack, conflictResult } = defineProps<{
-  nodePack: components['schemas']['Node']
-  conflictResult: ConflictDetectionResult | null | undefined
-}>()
-
-const packageId = computed(() => nodePack?.id || '')
-const { importFailedInfo, showImportFailedDialog } =
-  useImportFailedDetection(packageId)
-</script>

src/components/dialog/content/manager/packCard/PackCard.vue

@@ -43,7 +43,7 @@
             </span>
             <p
               v-if="nodePack.description"
-              class="flex-1 text-muted text-xs font-medium break-words overflow-hidden max-h-12 line-clamp-3 my-0 leading-4 mb-2 overflow-hidden"
+              class="flex-1 text-muted text-xs font-medium break-words overflow-hidden min-h-12 line-clamp-3 my-0 leading-4 mb-1 overflow-hidden"
             >
               {{ nodePack.description }}
             </p>
@@ -119,21 +119,13 @@ provide(IsInstallingKey, isInstalling)
 
 const { isPackInstalled, isPackEnabled } = useComfyManagerStore()
 
-const isDisabled = ref(false)
-const managerStore = useComfyManagerStore()
-
-// Watch the installedPacks object directly (which gets updated from WebSocket)
-whenever(
-  () => managerStore.installedPacksIds,
-  () => {
-    const isInstalled = isPackInstalled(nodePack?.id)
-    isDisabled.value = isInstalled && !isPackEnabled(nodePack?.id)
-
-    // Update isInstalling state after installation is complete
-    if (isInstalling.value && isInstalled) isInstalling.value = false
-  }
+const isInstalled = computed(() => isPackInstalled(nodePack?.id))
+const isDisabled = computed(
+  () => isInstalled.value && !isPackEnabled(nodePack?.id)
 )
 
+whenever(isInstalled, () => (isInstalling.value = false))
+
 const nodesCount = computed(() =>
   isMergedNodePack(nodePack) ? nodePack.comfy_nodes?.length : undefined
 )

src/components/dialog/content/manager/packCard/PackCardFooter.vue

@@ -1,39 +1,13 @@
 <template>
   <div
-    class="h-12 flex justify-between items-center px-4 text-xs text-muted font-medium leading-3"
+    class="min-h-12 flex justify-between items-center px-4 py-2 text-xs text-muted font-medium leading-3"
   >
     <div v-if="nodePack.downloads" class="flex items-center gap-1.5">
       <i class="pi pi-download text-muted"></i>
       <span>{{ formattedDownloads }}</span>
     </div>
-    <div class="flex justify-end items-center gap-2">
-      <template v-if="importFailed">
-        <div
-          class="flex justify-center items-center gap-2 cursor-pointer"
-          @click="showImportFailedDialog"
-        >
-          <i class="pi pi-exclamation-triangle text-red-500 text-sm"></i>
-          <span class="text-red-500 text-xs pt-0.5">{{
-            t('manager.failedToInstall')
-          }}</span>
-        </div>
-      </template>
-      <template v-else>
-        <template v-if="!isInstalled">
-          <PackInstallButton
-            :node-packs="[nodePack]"
-            :has-conflict="uninstalledPackConflict.hasConflict"
-            :conflict-info="uninstalledPackConflict.conflicts"
-          />
-        </template>
-        <template v-else>
-          <PackEnableToggle
-            :node-pack="nodePack"
-            :has-conflict="installedPackHasConflict"
-          />
-        </template>
-      </template>
-    </div>
+    <PackInstallButton v-if="!isInstalled" :node-packs="[nodePack]" />
+    <PackEnableToggle v-else :node-pack="nodePack" />
   </div>
 </template>
 
@@ -43,10 +17,7 @@ import { useI18n } from 'vue-i18n'
 
 import PackEnableToggle from '@/components/dialog/content/manager/button/PackEnableToggle.vue'
 import PackInstallButton from '@/components/dialog/content/manager/button/PackInstallButton.vue'
-import { useConflictDetection } from '@/composables/useConflictDetection'
-import { useImportFailedDetection } from '@/composables/useImportFailedDetection'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import { useConflictDetectionStore } from '@/stores/conflictDetectionStore'
 import type { components } from '@/types/comfyRegistryTypes'
 
 const { nodePack } = defineProps<{
@@ -56,29 +27,9 @@ const { nodePack } = defineProps<{
 const { isPackInstalled } = useComfyManagerStore()
 const isInstalled = computed(() => isPackInstalled(nodePack?.id))
 
-const { n, t } = useI18n()
+const { n } = useI18n()
 
 const formattedDownloads = computed(() =>
   nodePack.downloads ? n(nodePack.downloads) : ''
 )
-
-const { getConflictsForPackageByID } = useConflictDetectionStore()
-const { checkNodeCompatibility } = useConflictDetection()
-
-const { importFailed, showImportFailedDialog } = useImportFailedDetection(
-  nodePack.id
-)
-
-const conflicts = computed(
-  () => getConflictsForPackageByID(nodePack.id!) || null
-)
-
-const installedPackHasConflict = computed(() => {
-  if (!nodePack.id) return false
-  return !!conflicts.value
-})
-
-const uninstalledPackConflict = computed(() => {
-  return checkNodeCompatibility(nodePack)
-})
 </script>

src/components/dialog/content/manager/packCard/PackCardHeader.vue

@@ -20,10 +20,7 @@
         {{ $n(nodePack.downloads) }}
       </div>
       <template v-if="isInstalled">
-        <PackEnableToggle
-          :node-pack="nodePack"
-          :has-conflict="!!packageConflicts"
-        />
+        <PackEnableToggle :node-pack="nodePack" />
       </template>
       <template v-else>
         <PackInstallButton :node-packs="[nodePack]" />
@@ -38,7 +35,6 @@ import { computed } from 'vue'
 import PackEnableToggle from '@/components/dialog/content/manager/button/PackEnableToggle.vue'
 import PackInstallButton from '@/components/dialog/content/manager/button/PackInstallButton.vue'
 import { useComfyManagerStore } from '@/stores/comfyManagerStore'
-import { useConflictDetectionStore } from '@/stores/conflictDetectionStore'
 import type { components } from '@/types/comfyRegistryTypes'
 
 const { nodePack } = defineProps<{
@@ -46,13 +42,5 @@ const { nodePack } = defineProps<{
 }>()
 
 const { isPackInstalled } = useComfyManagerStore()
-const { getConflictsForPackageByID } = useConflictDetectionStore()
 const isInstalled = computed(() => isPackInstalled(nodePack?.id))
-
-const packageConflicts = computed(() => {
-  if (!nodePack.id || !isInstalled.value) return null
-
-  // For installed packages, check conflicts from store
-  return getConflictsForPackageByID(nodePack.id)
-})
 </script>

src/components/dialog/content/manager/packIcon/PackIcon.vue

@@ -1,37 +1,11 @@
 <template>
-  <div class="w-full max-w-[204] aspect-[2/1] rounded-lg overflow-hidden">
-    <!-- default banner show -->
-    <div v-if="showDefaultBanner" class="w-full h-full">
-      <img
-        :src="DEFAULT_BANNER"
-        alt="default banner"
-        class="w-full h-full object-cover"
-      />
-    </div>
-    <!-- banner_url or icon show -->
-    <div v-else class="relative w-full h-full">
-      <!-- blur background -->
-      <div
-        v-if="imgSrc"
-        class="absolute inset-0 bg-cover bg-center bg-no-repeat opacity-30"
-        :style="{
-          backgroundImage: `url(${imgSrc})`,
-          filter: 'blur(10px)'
-        }"
-      ></div>
-      <!-- image -->
-      <img
-        :src="isImageError ? DEFAULT_BANNER : imgSrc"
-        :alt="nodePack.name + ' banner'"
-        :class="
-          isImageError
-            ? 'relative w-full h-full object-cover z-10'
-            : 'relative w-full h-full object-contain z-10'
-        "
-        @error="isImageError = true"
-      />
-    </div>
-  </div>
+  <img
+    :src="isImageError ? DEFAULT_ICON : imgSrc"
+    :alt="nodePack.name + ' icon'"
+    class="object-contain rounded-lg max-h-72 max-w-72"
+    :style="{ width: cssWidth, height: cssHeight }"
+    @error="isImageError = true"
+  />
 </template>
 
 <script setup lang="ts">
@@ -39,14 +13,29 @@ import { computed, ref } from 'vue'
 
 import { components } from '@/types/comfyRegistryTypes'
 
-const DEFAULT_BANNER = '/assets/images/fallback-gradient-avatar.svg'
+const DEFAULT_ICON = '/assets/images/fallback-gradient-avatar.svg'
 
-const { nodePack } = defineProps<{
+const {
+  nodePack,
+  width = '4.5rem',
+  height = '4.5rem'
+} = defineProps<{
   nodePack: components['schemas']['Node']
+  width?: string
+  height?: string
 }>()
 
 const isImageError = ref(false)
+const shouldShowFallback = computed(
+  () => !nodePack.icon || nodePack.icon.trim() === '' || isImageError.value
+)
+const imgSrc = computed(() =>
+  shouldShowFallback.value ? DEFAULT_ICON : nodePack.icon
+)
 
-const showDefaultBanner = computed(() => !nodePack.banner_url && !nodePack.icon)
-const imgSrc = computed(() => nodePack.banner_url || nodePack.icon)
+const convertToCssValue = (value: string | number) =>
+  typeof value === 'number' ? `${value}rem` : value
+
+const cssWidth = computed(() => convertToCssValue(width))
+const cssHeight = computed(() => convertToCssValue(height))
 </script>

src/components/dialog/content/manager/packIcon/PackIconStacked.vue

@@ -1,19 +1,25 @@
 <template>
-  <div class="relative w-[224px] h-[104px] shadow-xl">
+  <div class="relative w-24 h-24">
     <div
       v-for="(pack, index) in nodePacks.slice(0, maxVisible)"
       :key="pack.id"
-      class="absolute w-[210px] h-[90px]"
+      class="absolute"
       :style="{
         bottom: `${index * offset}px`,
         right: `${index * offset}px`,
         zIndex: maxVisible - index
       }"
     >
-      <div class="border rounded-lg shadow-lg p-0.5">
-        <PackIcon :node-pack="pack" />
+      <div class="border rounded-lg p-0.5">
+        <PackIcon :node-pack="pack" width="4.5rem" height="4.5rem" />
       </div>
     </div>
+    <div
+      v-if="nodePacks.length > maxVisible"
+      class="absolute -top-2 -right-2 bg-primary rounded-full w-7 h-7 flex items-center justify-center text-xs font-bold shadow-md z-10"
+    >
+      +{{ nodePacks.length - maxVisible }}
+    </div>
   </div>
 </template>
 

src/components/dialog/content/manager/registrySearchBar/RegistrySearchBar.vue

@@ -33,10 +33,6 @@
         :node-packs="missingNodePacks"
         :label="$t('manager.installAllMissingNodes')"
       />
-      <PackUpdateButton
-        v-if="isUpdateAvailableTab && hasUpdateAvailable"
-        :node-packs="updateAvailableNodePacks"
-      />
     </div>
     <div class="flex mt-3 text-sm">
       <div class="flex gap-6 ml-1">
@@ -69,10 +65,8 @@ import { computed } from 'vue'
 import { useI18n } from 'vue-i18n'
 
 import PackInstallButton from '@/components/dialog/content/manager/button/PackInstallButton.vue'
-import PackUpdateButton from '@/components/dialog/content/manager/button/PackUpdateButton.vue'
 import SearchFilterDropdown from '@/components/dialog/content/manager/registrySearchBar/SearchFilterDropdown.vue'
 import { useMissingNodes } from '@/composables/nodePack/useMissingNodes'
-import { useUpdateAvailableNodes } from '@/composables/nodePack/useUpdateAvailableNodes'
 import {
   type SearchOption,
   SortableAlgoliaField
@@ -89,7 +83,6 @@ const { searchResults, sortOptions } = defineProps<{
   suggestions?: QuerySuggestion[]
   sortOptions?: SortableField[]
   isMissingTab?: boolean
-  isUpdateAvailableTab?: boolean
 }>()
 
 const searchQuery = defineModel<string>('searchQuery')
@@ -103,10 +96,6 @@ const { t } = useI18n()
 // Get missing node packs from workflow with loading and error states
 const { missingNodePacks, isLoading, error } = useMissingNodes()
 
-// Use the composable to get update available nodes
-const { hasUpdateAvailable, updateAvailableNodePacks } =
-  useUpdateAvailableNodes()
-
 const hasResults = computed(
   () => searchQuery.value?.trim() && searchResults?.length
 )

src/components/dialog/footer/ManagerProgressFooter.vue

@@ -1,44 +1,41 @@
 <template>
   <div
-    class="w-full px-6 py-2 shadow-lg flex items-center justify-between"
+    class="w-full px-6 py-4 shadow-lg flex items-center justify-between"
     :class="{
       'rounded-t-none': progressDialogContent.isExpanded,
       'rounded-lg': !progressDialogContent.isExpanded
     }"
   >
-    <div class="flex items-center text-base leading-none">
+    <div class="justify-center text-sm font-bold leading-none">
       <div class="flex items-center">
         <template v-if="isInProgress">
-          <DotSpinner duration="1s" class="mr-2" />
-          <span>{{ currentTaskName }}</span>
-        </template>
-        <template v-else-if="isRestartCompleted">
-          <span class="mr-2">🎉</span>
+          <i class="pi pi-spin pi-spinner mr-2 text-3xl" />
           <span>{{ currentTaskName }}</span>
         </template>
         <template v-else>
-          <span class="mr-2">✅</span>
-          <span>{{ $t('manager.restartToApplyChanges') }}</span>
+          <i class="pi pi-check-circle mr-2 text-green-500" />
+          <span class="leading-none">{{
+            $t('manager.restartToApplyChanges')
+          }}</span>
         </template>
       </div>
     </div>
     <div class="flex items-center gap-4">
-      <span v-if="isInProgress" class="text-sm text-neutral-700">
-        {{ completedTasksCount }} {{ $t('g.progressCountOf') }}
+      <span v-if="isInProgress" class="text-xs font-bold text-neutral-600">
+        {{ comfyManagerStore.uncompletedCount }} {{ $t('g.progressCountOf') }}
         {{ comfyManagerStore.taskLogs.length }}
       </span>
       <div class="flex items-center">
         <Button
-          v-if="!isInProgress && !isRestartCompleted"
+          v-if="!isInProgress"
           rounded
           outlined
-          class="mr-4 rounded-md border-2 px-3 text-neutral-600 border-neutral-900 hover:bg-neutral-100 !dark-theme:bg-transparent dark-theme:text-white dark-theme:border-white dark-theme:hover:bg-neutral-800"
+          class="px-4 py-2 rounded-md mr-4"
           @click="handleRestart"
         >
-          {{ $t('manager.applyChanges') }}
+          {{ $t('g.restart') }}
         </Button>
         <Button
-          v-else-if="!isRestartCompleted"
           :icon="
             progressDialogContent.isExpanded
               ? 'pi pi-chevron-up'
@@ -47,7 +44,6 @@
           text
           rounded
           size="small"
-          class="font-bold"
           severity="secondary"
           :aria-label="progressDialogContent.isExpanded ? 'Collapse' : 'Expand'"
           @click.stop="progressDialogContent.toggle"
@@ -57,7 +53,6 @@
           text
           rounded
           size="small"
-          class="font-bold"
           severity="secondary"
           aria-label="Close"
           @click.stop="closeDialog"
@@ -70,10 +65,9 @@
 <script setup lang="ts">
 import { useEventListener } from '@vueuse/core'
 import Button from 'primevue/button'
-import { computed, ref } from 'vue'
+import { computed } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import DotSpinner from '@/components/common/DotSpinner.vue'
 import { api } from '@/scripts/api'
 import { useComfyManagerService } from '@/services/comfyManagerService'
 import { useWorkflowService } from '@/services/workflowService'
@@ -83,93 +77,41 @@ import {
 } from '@/stores/comfyManagerStore'
 import { useCommandStore } from '@/stores/commandStore'
 import { useDialogStore } from '@/stores/dialogStore'
-import { useSettingStore } from '@/stores/settingStore'
 
 const { t } = useI18n()
 const dialogStore = useDialogStore()
 const progressDialogContent = useManagerProgressDialogStore()
 const comfyManagerStore = useComfyManagerStore()
-const settingStore = useSettingStore()
 
-// State management for restart process
-const isRestarting = ref<boolean>(false)
-const isRestartCompleted = ref<boolean>(false)
-
-const isInProgress = computed(
-  () => comfyManagerStore.isProcessingTasks || isRestarting.value
-)
-
-const completedTasksCount = computed(() => {
-  return (
-    comfyManagerStore.succeededTasksIds.length +
-    comfyManagerStore.failedTasksIds.length
-  )
-})
+const isInProgress = computed(() => comfyManagerStore.uncompletedCount > 0)
 
 const closeDialog = () => {
   dialogStore.closeDialog({ key: 'global-manager-progress-dialog' })
 }
 
-const fallbackTaskName = t('manager.installingDependencies')
+const fallbackTaskName = t('g.installing')
 const currentTaskName = computed(() => {
-  if (isRestarting.value) {
-    return t('manager.restartingBackend')
-  }
-  if (isRestartCompleted.value) {
-    return t('manager.extensionsSuccessfullyInstalled')
-  }
   if (!comfyManagerStore.taskLogs.length) return fallbackTaskName
   const task = comfyManagerStore.taskLogs.at(-1)
   return task?.taskName ?? fallbackTaskName
 })
 
 const handleRestart = async () => {
-  // Store original toast setting value
-  const originalToastSetting = settingStore.get(
-    'Comfy.Toast.DisableReconnectingToast'
-  )
+  const onReconnect = async () => {
+    // Refresh manager state
 
-  try {
-    await settingStore.set('Comfy.Toast.DisableReconnectingToast', true)
+    comfyManagerStore.clearLogs()
+    comfyManagerStore.setStale()
 
-    isRestarting.value = true
+    // Refresh node definitions
+    await useCommandStore().execute('Comfy.RefreshNodeDefinitions')
 
-    const onReconnect = async () => {
-      try {
-        comfyManagerStore.setStale()
-
-        await useCommandStore().execute('Comfy.RefreshNodeDefinitions')
-
-        await useWorkflowService().reloadCurrentWorkflow()
-      } finally {
-        await settingStore.set(
-          'Comfy.Toast.DisableReconnectingToast',
-          originalToastSetting
-        )
-
-        isRestarting.value = false
-        isRestartCompleted.value = true
-
-        setTimeout(() => {
-          closeDialog()
-          comfyManagerStore.resetTaskState()
-        }, 3000)
-      }
-    }
-
-    useEventListener(api, 'reconnected', onReconnect, { once: true })
-
-    await useComfyManagerService().rebootComfyUI()
-  } catch (error) {
-    // If restart fails, restore settings and reset state
-    await settingStore.set(
-      'Comfy.Toast.DisableReconnectingToast',
-      originalToastSetting
-    )
-    isRestarting.value = false
-    isRestartCompleted.value = false
-    closeDialog() // Close dialog on error
-    throw error
+    // Reload workflow
+    await useWorkflowService().reloadCurrentWorkflow()
   }
+  useEventListener(api, 'reconnected', onReconnect, { once: true })
+
+  await useComfyManagerService().rebootComfyUI()
+  closeDialog()
 }
 </script>

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" />
@@ -55,9 +50,9 @@ 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'
@@ -72,7 +67,6 @@ import { useContextMenuTranslation } from '@/composables/useContextMenuTranslati
 import { useCopy } from '@/composables/useCopy'
 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'
@@ -119,10 +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()
-
 watchEffect(() => {
   nodeDefStore.showDeprecated = settingStore.get('Comfy.Node.ShowDeprecated')
 })
@@ -358,13 +348,6 @@ onMounted(async () => {
     }
   )
 
-  whenever(
-    () => minimapRef.value,
-    (ref) => {
-      minimap.setMinimapRef(ref)
-    }
-  )
-
   whenever(
     () => useCanvasStore().canvas,
     (canvas) => {

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,15 +56,6 @@
       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>
 
@@ -76,7 +66,6 @@ import ButtonGroup from 'primevue/buttongroup'
 import { computed } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import { useCanvasInteractions } from '@/composables/graph/useCanvasInteractions'
 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/SelectionToolbox.vue

@@ -5,7 +5,6 @@
       header: 'hidden',
       content: 'p-0 flex flex-row'
     }"
-    @wheel="canvasInteractions.handleWheel"
   >
     <ExecuteButton />
     <ColorPickerButton />
@@ -40,7 +39,6 @@ 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 RefreshButton from '@/components/graph/selectionToolbox/RefreshButton.vue'
-import { useCanvasInteractions } from '@/composables/graph/useCanvasInteractions'
 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/selectionToolbox/ConvertToSubgraphButton.vue

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

src/components/helpcenter/HelpCenterMenuContent.vue

@@ -14,17 +14,7 @@
         @mouseenter="onMenuItemHover(menuItem.key, $event)"
         @mouseleave="onMenuItemLeave(menuItem.key)"
       >
-        <div class="help-menu-icon-container">
-          <div class="help-menu-icon">
-            <component
-              :is="menuItem.icon"
-              v-if="typeof menuItem.icon === 'object'"
-              :size="16"
-            />
-            <i v-else :class="menuItem.icon" />
-          </div>
-          <div v-if="menuItem.showRedDot" class="menu-red-dot" />
-        </div>
+        <i :class="menuItem.icon" class="help-menu-icon" />
         <span class="menu-label">{{ menuItem.label }}</span>
         <i v-if="menuItem.key === 'more'" class="pi pi-chevron-right" />
       </button>
@@ -129,21 +119,10 @@
 </template>
 
 <script setup lang="ts">
-// No need to import useStorage anymore - it's in the composable
 import Button from 'primevue/button'
-import {
-  type CSSProperties,
-  type Component,
-  computed,
-  nextTick,
-  onMounted,
-  ref
-} from 'vue'
+import { type CSSProperties, computed, nextTick, onMounted, ref } from 'vue'
 import { useI18n } from 'vue-i18n'
 
-import PuzzleIcon from '@/components/icons/PuzzleIcon.vue'
-import { useConflictAcknowledgment } from '@/composables/useConflictAcknowledgment'
-import { useDialogService } from '@/services/dialogService'
 import { type ReleaseNote } from '@/services/releaseService'
 import { useCommandStore } from '@/stores/commandStore'
 import { useReleaseStore } from '@/stores/releaseStore'
@@ -154,13 +133,12 @@ import { formatVersionAnchor } from '@/utils/formatUtil'
 // Types
 interface MenuItem {
   key: string
-  icon?: string | Component
+  icon?: string
   label?: string
   action?: () => void
   visible?: boolean
   type?: 'item' | 'divider'
   items?: MenuItem[]
-  showRedDot?: boolean
 }
 
 // Constants
@@ -192,7 +170,6 @@ const { t, locale } = useI18n()
 const releaseStore = useReleaseStore()
 const commandStore = useCommandStore()
 const settingStore = useSettingStore()
-const dialogService = useDialogService()
 
 // Emits
 const emit = defineEmits<{
@@ -215,10 +192,6 @@ const moreMenuItem = computed(() =>
   menuItems.value.find((item) => item.key === 'more')
 )
 
-// Use conflict acknowledgment state from composable
-const { shouldShowRedDot: shouldShowManagerRedDot } =
-  useConflictAcknowledgment()
-
 const menuItems = computed<MenuItem[]>(() => {
   const moreItems: MenuItem[] = [
     {
@@ -298,17 +271,6 @@ const menuItems = computed<MenuItem[]>(() => {
         emit('close')
       }
     },
-    {
-      key: 'manager',
-      type: 'item',
-      icon: PuzzleIcon,
-      label: t('helpCenter.managerExtension'),
-      showRedDot: shouldShowManagerRedDot.value,
-      action: () => {
-        dialogService.showManagerDialog()
-        emit('close')
-      }
-    },
     {
       key: 'more',
       type: 'item',
@@ -543,39 +505,16 @@ onMounted(async () => {
   box-shadow: none;
 }
 
-.help-menu-icon-container {
-  position: relative;
-  margin-right: 0.75rem;
-  width: 16px;
-  flex-shrink: 0;
-}
-
 .help-menu-icon {
+  margin-right: 0.75rem;
   font-size: 1rem;
   color: var(--p-text-muted-color);
   width: 16px;
   display: flex;
   justify-content: center;
-  align-items: center;
   flex-shrink: 0;
 }
 
-.help-menu-icon svg {
-  color: var(--p-text-muted-color);
-}
-
-.menu-red-dot {
-  position: absolute;
-  top: -2px;
-  right: -2px;
-  width: 8px;
-  height: 8px;
-  background: #ff3b30;
-  border-radius: 50%;
-  border: 1.5px solid var(--p-content-background);
-  z-index: 1;
-}
-
 .menu-label {
   flex: 1;
 }

src/components/helpcenter/WhatsNewPopup.vue

@@ -75,11 +75,6 @@ import { formatVersionAnchor } from '@/utils/formatUtil'
 const { locale, t } = useI18n()
 const releaseStore = useReleaseStore()
 
-// Emit event for parent component
-const emit = defineEmits<{
-  'whats-new-dismissed': []
-}>()
-
 // Local state for dismissed status
 const isDismissed = ref(false)
 
@@ -139,10 +134,6 @@ const closePopup = async () => {
     await releaseStore.handleWhatsNewSeen(latestRelease.value.version)
   }
   hide()
-
-  // Emit event to notify parent that What's New was dismissed
-  // Parent can then check if conflict modal should be shown
-  emit('whats-new-dismissed')
 }
 
 // Learn more handled by anchor href

src/components/icons/PuzzleIcon.vue

@@ -1,43 +0,0 @@
-<template>
-  <svg
-    xmlns="http://www.w3.org/2000/svg"
-    :width="size"
-    :height="size"
-    viewBox="0 0 16 16"
-    fill="none"
-    :class="iconClass"
-  >
-    <g clip-path="url(#clip0_1099_16244)">
-      <path
-        d="M4.99992 3.00016C4.99992 2.07969 5.74611 1.3335 6.66658 1.3335C7.58706 1.3335 8.33325 2.07969 8.33325 3.00016V4.00016H8.99992C9.9318 4.00016 10.3977 4.00016 10.7653 4.1524C11.2553 4.35539 11.6447 4.74474 11.8477 5.2348C11.9999 5.60234 11.9999 6.06828 11.9999 7.00016H12.9999C13.9204 7.00016 14.6666 7.74635 14.6666 8.66683C14.6666 9.5873 13.9204 10.3335 12.9999 10.3335H11.9999V11.4668C11.9999 12.5869 11.9999 13.147 11.7819 13.5748C11.5902 13.9511 11.2842 14.2571 10.9079 14.4488C10.4801 14.6668 9.92002 14.6668 8.79992 14.6668H8.33325V13.5002C8.33325 12.6717 7.66168 12.0002 6.83325 12.0002C6.00482 12.0002 5.33325 12.6717 5.33325 13.5002V14.6668H4.53325C3.41315 14.6668 2.85309 14.6668 2.42527 14.4488C2.04895 14.2571 1.74299 13.9511 1.55124 13.5748C1.33325 13.147 1.33325 12.5869 1.33325 11.4668V10.3335H2.33325C3.25373 10.3335 3.99992 9.5873 3.99992 8.66683C3.99992 7.74635 3.25373 7.00016 2.33325 7.00016H1.33325C1.33325 6.06828 1.33325 5.60234 1.48549 5.2348C1.68848 4.74474 2.07783 4.35539 2.56789 4.1524C2.93543 4.00016 3.40137 4.00016 4.33325 4.00016H4.99992V3.00016Z"
-        :stroke="color"
-        stroke-width="1.2"
-        stroke-linecap="round"
-        stroke-linejoin="round"
-      />
-    </g>
-    <defs>
-      <clipPath id="clip0_1099_16244">
-        <rect width="16" height="16" fill="white" />
-      </clipPath>
-    </defs>
-  </svg>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-interface Props {
-  size?: number | string
-  color?: string
-  class?: string
-}
-
-const {
-  size = 16,
-  color = 'currentColor',
-  class: className
-} = defineProps<Props>()
-
-const iconClass = computed(() => className || '')
-</script>

src/components/icons/VerifiedIcon.vue

@@ -1,29 +0,0 @@
-<template>
-  <svg
-    xmlns="http://www.w3.org/2000/svg"
-    :width="size"
-    :height="size"
-    viewBox="0 0 16 16"
-    fill="none"
-    :class="iconClass"
-  >
-    <path
-      d="M8.00049 1.3335C8.73661 1.33367 9.33332 1.93038 9.3335 2.6665V2.83447C9.82278 2.96041 10.2851 3.15405 10.7095 3.40479L10.8286 3.28564C11.3493 2.76525 12.1937 2.76519 12.7144 3.28564C13.235 3.80626 13.2348 4.65067 12.7144 5.17139L12.5952 5.29053C12.846 5.71486 13.0396 6.17725 13.1655 6.6665H13.3335C14.0699 6.6665 14.6665 7.26411 14.6665 8.00049C14.6663 8.73672 14.0698 9.3335 13.3335 9.3335H13.1655C13.0396 9.82284 12.846 10.2851 12.5952 10.7095L12.7144 10.8286C13.235 11.3493 13.235 12.1937 12.7144 12.7144C12.1937 13.235 11.3493 13.235 10.8286 12.7144L10.7095 12.5952C10.2851 12.846 9.82284 13.0396 9.3335 13.1655V13.3335C9.3335 14.0698 8.73672 14.6663 8.00049 14.6665C7.26411 14.6665 6.6665 14.0699 6.6665 13.3335V13.1655C6.17725 13.0396 5.71486 12.846 5.29053 12.5952L5.17139 12.7144C4.65067 13.2348 3.80626 13.235 3.28564 12.7144C2.76519 12.1937 2.76525 11.3493 3.28564 10.8286L3.40479 10.7095C3.15405 10.2851 2.96041 9.82278 2.83447 9.3335H2.6665C1.93038 9.33332 1.33367 8.73661 1.3335 8.00049C1.3335 7.26422 1.93027 6.66668 2.6665 6.6665H2.83447C2.96043 6.17722 3.15403 5.71488 3.40479 5.29053L3.28564 5.17139C2.76536 4.65065 2.76508 3.80621 3.28564 3.28564C3.80621 2.76508 4.65065 2.76536 5.17139 3.28564L5.29053 3.40479C5.71488 3.15403 6.17722 2.96043 6.6665 2.83447V2.6665C6.66668 1.93027 7.26422 1.3335 8.00049 1.3335ZM7.3335 8.00049L6.00049 6.6665L4.6665 8.00049L7.3335 10.6665L11.3335 6.6665L10.0005 5.3335L7.3335 8.00049Z"
-      :fill="color"
-    />
-  </svg>
-</template>
-
-<script setup lang="ts">
-import { computed } from 'vue'
-
-interface Props {
-  size?: number | string
-  color?: string
-  class?: string
-}
-
-const { size = 16, color = '#60A5FA', class: className } = defineProps<Props>()
-
-const iconClass = computed(() => className || '')
-</script>

src/components/sidebar/SideToolbar.vue

@@ -14,8 +14,9 @@
       />
       <div class="side-tool-bar-end">
         <SidebarLogoutIcon v-if="userStore.isMultiUserServer" />
+        <SidebarThemeToggleIcon />
         <SidebarHelpCenterIcon />
-        <SidebarBottomPanelToggleButton />
+        <SidebarSettingsToggleIcon />
       </div>
     </nav>
   </teleport>
@@ -31,7 +32,6 @@
 import { computed } from 'vue'
 
 import ExtensionSlot from '@/components/common/ExtensionSlot.vue'
-import SidebarBottomPanelToggleButton from '@/components/sidebar/SidebarBottomPanelToggleButton.vue'
 import { useKeybindingStore } from '@/stores/keybindingStore'
 import { useSettingStore } from '@/stores/settingStore'
 import { useUserStore } from '@/stores/userStore'
@@ -41,6 +41,8 @@ import type { SidebarTabExtension } from '@/types/extensionTypes'
 import SidebarHelpCenterIcon from './SidebarHelpCenterIcon.vue'
 import SidebarIcon from './SidebarIcon.vue'
 import SidebarLogoutIcon from './SidebarLogoutIcon.vue'
+import SidebarSettingsToggleIcon from './SidebarSettingsToggleIcon.vue'
+import SidebarThemeToggleIcon from './SidebarThemeToggleIcon.vue'
 
 const workspaceStore = useWorkspaceStore()
 const settingStore = useSettingStore()

src/components/sidebar/SidebarBottomPanelToggleButton.vue

@@ -1,19 +0,0 @@
-<template>
-  <SidebarIcon
-    :tooltip="$t('menu.toggleBottomPanel')"
-    :selected="bottomPanelStore.bottomPanelVisible"
-    @click="bottomPanelStore.toggleBottomPanel"
-  >
-    <template #icon>
-      <i-ph:terminal-bold />
-    </template>
-  </SidebarIcon>
-</template>
-
-<script setup lang="ts">
-import { useBottomPanelStore } from '@/stores/workspace/bottomPanelStore'
-
-import SidebarIcon from './SidebarIcon.vue'
-
-const bottomPanelStore = useBottomPanelStore()
-</script>