[fix] use relative import path in subgraph browser tests

When a subgraph node is renamed, the node library sidebar now immediately reflects the updated display name. This ensures consistency between the canvas, breadcrumb navigation, and node library.

Key changes:
- Add updateNodeDefDisplayName method to nodeDefStore with reference preservation
- Update TitleEditor to call nodeDefStore for subgraph title changes
- Organize subgraph browser tests into dedicated folder
- Add comprehensive unit test coverage for nodeDefStore

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

@@ -0,0 +1,708 @@
+# Comprehensive PR Review for ComfyUI Frontend
+
+<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.
+
+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>
+
+Arguments: PR number passed via PR_NUMBER environment variable
+
+## Phase 0: Initialize Variables and Helper Functions
+
+```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
+
+# 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
+
+# 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
+
+```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"
+}
+
+# Run pre-flight checks
+check_prerequisites
+
+echo "Starting comprehensive review of PR #$PR_NUMBER"
+
+# 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
+
+# Extract branch names
+BASE_BRANCH=$(jq -r '.baseRefName' < pr_info.json)
+HEAD_BRANCH=$(jq -r '.headRefName' < pr_info.json)
+
+# 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"
+
+# Get changed files using git locally (much faster)
+git diff --name-only "origin/$BASE_BRANCH" > changed_files.txt
+
+# Get the diff using git locally
+git diff "origin/$BASE_BRANCH" > pr_diff.txt
+
+# Get detailed file changes with line numbers
+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
+
+# Setup caching directory
+CACHE_DIR=".claude-review-cache"
+mkdir -p "$CACHE_DIR"
+
+# 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
+}
+
+# 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
+
+```bash
+# Don't create knowledge directory until we know we need it
+KNOWLEDGE_FOUND=false
+
+# Use local cache for knowledge base to avoid repeated downloads
+KNOWLEDGE_CACHE_DIR=".claude-knowledge-cache"
+mkdir -p "$KNOWLEDGE_CACHE_DIR"
+
+# 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
+
+# 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
+}
+
+# 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
+
+# 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
+
+# 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
+
+# 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 a comprehensive analysis covering these areas:
+
+### 3.1 Architectural Analysis
+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
+- Cyclomatic complexity and cognitive load
+- SOLID principles adherence
+- DRY violations that aren't caught by simple duplication checks
+- Proper abstraction levels
+- Interface design and API clarity
+- No leftover debug code (console.log, commented code, TODO comments)
+
+### 3.3 Library Usage Enforcement
+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
+Beyond obvious vulnerabilities:
+- Authentication/authorization implications
+- Data validation completeness
+- State management security
+- Cross-origin concerns
+- Extension security boundaries
+
+### 3.5 Performance Analysis
+- Render performance implications
+- Layout thrashing prevention
+- Memory leak potential
+- Network request optimization
+- State management efficiency
+
+### 3.6 Integration Concerns
+- Breaking changes to internal APIs
+- Extension compatibility
+- Backward compatibility
+- Migration requirements
+
+## Phase 4: Create Detailed Review Comments
+
+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:
+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
+# 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"
+}
+```
+
+## Phase 5: Validation Rules Application
+
+Apply ALL validation rules from the loaded knowledge, but focus on the changed lines:
+
+### From Frontend Standards
+- Vue 3 Composition API patterns
+- Component communication patterns
+- Proper use of composables
+- TypeScript strict mode compliance
+- Bundle optimization
+
+### From Security Audit
+- Input validation
+- XSS prevention
+- CSRF protection
+- Secure state management
+- API security
+
+### From Performance Check
+- Render optimization
+- Memory management
+- Network efficiency
+- Bundle size impact
+
+## Phase 6: Contextual Review Based on PR Type
+
+Analyze the PR description and changes to determine the type:
+
+```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, create a detailed summary:
+
+```bash
+# Initialize metrics tracking
+REVIEW_START_TIME=$(date +%s)
+
+# 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**: $ADDITIONS additions, $DELETIONS deletions across $FILE_COUNT files
+
+### Issue Distribution
+- 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
+
+## Key Findings
+
+### Architecture & Design
+[Detailed architectural analysis based on repository patterns]
+
+### Security Considerations
+[Security implications beyond basic vulnerabilities]
+
+### Performance Impact
+[Performance analysis including bundle size, render impact]
+
+### Integration Points
+[How this affects other systems, extensions, etc.]
+
+## Positive Observations
+[What was done well, good patterns followed]
+
+## References
+- [Repository Architecture Guide](https://github.com/Comfy-Org/comfy-claude-prompt-library/blob/master/project-summaries-for-agents/ComfyUI_frontend/REPOSITORY_GUIDE.md)
+- [Frontend Standards](https://github.com/Comfy-Org/comfy-claude-prompt-library/blob/master/.claude/commands/validation/frontend-code-standards.md)
+- [Security Guidelines](https://github.com/Comfy-Org/comfy-claude-prompt-library/blob/master/.claude/commands/validation/security-audit.md)
+
+## Next Steps
+1. Address critical issues before merge
+2. Consider architectural feedback for long-term maintainability
+3. Add tests for uncovered scenarios
+4. Update documentation if needed
+
+---
+*This is a comprehensive automated review. For architectural decisions requiring human judgment, please request additional manual review.*"
+```
+
+## Important: Think Deeply
+
+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.
+
+## Phase 8: Track Review Metrics
+
+After completing the review, save metrics for analysis:
+
+```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.

.github/CLAUDE.md

@@ -0,0 +1,36 @@
+# ComfyUI Frontend - Claude Review Context
+
+This file provides additional context for the automated PR review system.
+
+## Quick Reference
+
+### PrimeVue Component Migrations
+
+When reviewing, flag these deprecated components:
+- `Dropdown` → Use `Select` from 'primevue/select'
+- `OverlayPanel` → Use `Popover` from 'primevue/popover'
+- `Calendar` → Use `DatePicker` from 'primevue/datepicker'
+- `InputSwitch` → Use `ToggleSwitch` from 'primevue/toggleswitch'
+- `Sidebar` → Use `Drawer` from 'primevue/drawer'
+- `Chips` → Use `AutoComplete` with multiple enabled and typeahead disabled
+- `TabMenu` → Use `Tabs` without panels
+- `Steps` → Use `Stepper` without panels
+- `InlineMessage` → Use `Message` component
+
+### API Utilities Reference
+
+- `api.apiURL()` - Backend API calls (/prompt, /queue, /view, etc.)
+- `api.fileURL()` - Static file access (templates, extensions)
+- `$t()` / `i18n.global.t()` - Internationalization
+- `DOMPurify.sanitize()` - HTML sanitization
+
+## Review Scope
+
+This automated review performs comprehensive analysis including:
+- Architecture and design patterns
+- Security vulnerabilities
+- Performance implications
+- Code quality and maintainability
+- Integration concerns
+
+For implementation details, see `.claude/commands/comprehensive-pr-review.md`.

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

@@ -0,0 +1,75 @@
+name: Claude PR Review
+
+permissions:
+  contents: read
+  pull-requests: write
+  issues: write
+
+on:
+  pull_request:
+    types: [labeled]
+
+jobs:
+  wait-for-ci:
+    runs-on: ubuntu-latest
+    if: github.event.label.name == 'claude-review'
+    outputs:
+      should-proceed: ${{ steps.check-status.outputs.proceed }}
+    steps:
+      - name: Wait for other CI checks
+        uses: lewagon/wait-on-check-action@v1.3.1
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+          check-regexp: '^(ESLint|Prettier Check|Tests CI|Vitest Tests)'
+          wait-interval: 30
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+      
+      - name: Check if we should proceed
+        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 Check|Tests CI|Vitest Tests")) | {name, conclusion}')
+          
+          # Check if any required checks failed
+          if echo "$CHECK_RUNS" | grep -q '"conclusion": "failure"'; then
+            echo "Some CI checks failed - skipping Claude review"
+            echo "proceed=false" >> $GITHUB_OUTPUT
+          else
+            echo "All CI checks passed - proceeding with Claude review"
+            echo "proceed=true" >> $GITHUB_OUTPUT
+          fi
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+  claude-review:
+    needs: wait-for-ci
+    if: needs.wait-for-ci.outputs.should-proceed == 'true'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+
+      - name: Install dependencies for analysis tools
+        run: |
+          npm install -g typescript @vue/compiler-sfc
+
+      - name: Run Claude PR Review
+        uses: anthropics/claude-code-action@main
+        with:
+          prompt_file: .claude/commands/comprehensive-pr-review.md
+          anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
+          max_turns: 1
+          timeout_minutes: 30
+        env:
+          PR_NUMBER: ${{ github.event.pull_request.number }}
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          COMMIT_SHA: ${{ github.event.pull_request.head.sha }}
+          BASE_SHA: ${{ github.event.pull_request.base.sha }}
+          REPOSITORY: ${{ github.repository }}

.github/workflows/danger.yaml

@@ -0,0 +1,27 @@
+name: Danger PR Review
+on:
+  pull_request:
+    types: [opened, edited, synchronize]
+
+jobs:
+  danger:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 'lts/*'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run Danger
+        run: npx danger ci
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}

browser_tests/assets/nested-subgraph.json

@@ -0,0 +1,716 @@
+{
+  "id": "976d6e9a-927d-42db-abd4-96bfc0ecf8d9",
+  "revision": 0,
+  "last_node_id": 10,
+  "last_link_id": 11,
+  "nodes": [
+    {
+      "id": 10,
+      "type": "8beb610f-ddd1-4489-ae0d-2f732a4042ae",
+      "pos": [
+        532,
+        412.5
+      ],
+      "size": [
+        140,
+        46
+      ],
+      "flags": {},
+      "order": 0,
+      "mode": 0,
+      "inputs": [],
+      "outputs": [
+        {
+          "name": "LATENT",
+          "type": "LATENT",
+          "links": [
+            10
+          ]
+        },
+        {
+          "name": "VAE",
+          "type": "VAE",
+          "links": [
+            11
+          ]
+        }
+      ],
+      "title": "subgraph 2",
+      "properties": {},
+      "widgets_values": []
+    },
+    {
+      "id": 8,
+      "type": "VAEDecode",
+      "pos": [
+        758.2109985351562,
+        398.3681335449219
+      ],
+      "size": [
+        210,
+        46
+      ],
+      "flags": {},
+      "order": 1,
+      "mode": 0,
+      "inputs": [
+        {
+          "name": "samples",
+          "type": "LATENT",
+          "link": 10
+        },
+        {
+          "name": "vae",
+          "type": "VAE",
+          "link": 11
+        }
+      ],
+      "outputs": [
+        {
+          "name": "IMAGE",
+          "type": "IMAGE",
+          "slot_index": 0,
+          "links": [
+            9
+          ]
+        }
+      ],
+      "properties": {
+        "Node name for S&R": "VAEDecode"
+      },
+      "widgets_values": []
+    },
+    {
+      "id": 9,
+      "type": "SaveImage",
+      "pos": [
+        1028.9615478515625,
+        381.83746337890625
+      ],
+      "size": [
+        210,
+        270
+      ],
+      "flags": {},
+      "order": 2,
+      "mode": 0,
+      "inputs": [
+        {
+          "name": "images",
+          "type": "IMAGE",
+          "link": 9
+        }
+      ],
+      "outputs": [],
+      "properties": {},
+      "widgets_values": [
+        "ComfyUI"
+      ]
+    }
+  ],
+  "links": [
+    [
+      9,
+      8,
+      0,
+      9,
+      0,
+      "IMAGE"
+    ],
+    [
+      10,
+      10,
+      0,
+      8,
+      0,
+      "LATENT"
+    ],
+    [
+      11,
+      10,
+      1,
+      8,
+      1,
+      "VAE"
+    ]
+  ],
+  "groups": [],
+  "definitions": {
+    "subgraphs": [
+      {
+        "id": "8beb610f-ddd1-4489-ae0d-2f732a4042ae",
+        "version": 1,
+        "state": {
+          "lastGroupId": 0,
+          "lastNodeId": 10,
+          "lastLinkId": 14,
+          "lastRerouteId": 0
+        },
+        "revision": 0,
+        "config": {},
+        "name": "subgraph 2",
+        "inputNode": {
+          "id": -10,
+          "bounding": [
+            -154,
+            415.5,
+            120,
+            40
+          ]
+        },
+        "outputNode": {
+          "id": -20,
+          "bounding": [
+            1238,
+            395.5,
+            120,
+            80
+          ]
+        },
+        "inputs": [],
+        "outputs": [
+          {
+            "id": "4d6c7e4e-971e-4f78-9218-9a604db53a4b",
+            "name": "LATENT",
+            "type": "LATENT",
+            "linkIds": [
+              7
+            ],
+            "localized_name": "LATENT",
+            "pos": {
+              "0": 1258,
+              "1": 415.5
+            }
+          },
+          {
+            "id": "f8201d4f-7fc6-4a1b-b8c9-9f0716d9c09a",
+            "name": "VAE",
+            "type": "VAE",
+            "linkIds": [
+              14
+            ],
+            "localized_name": "VAE",
+            "pos": {
+              "0": 1258,
+              "1": 435.5
+            }
+          }
+        ],
+        "widgets": [],
+        "nodes": [
+          {
+            "id": 6,
+            "type": "CLIPTextEncode",
+            "pos": [
+              415,
+              186
+            ],
+            "size": [
+              422.84503173828125,
+              164.31304931640625
+            ],
+            "flags": {},
+            "order": 0,
+            "mode": 0,
+            "inputs": [
+              {
+                "localized_name": "clip",
+                "name": "clip",
+                "type": "CLIP",
+                "link": 13
+              }
+            ],
+            "outputs": [
+              {
+                "localized_name": "CONDITIONING",
+                "name": "CONDITIONING",
+                "type": "CONDITIONING",
+                "slot_index": 0,
+                "links": [
+                  4
+                ]
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "CLIPTextEncode"
+            },
+            "widgets_values": [
+              "beautiful scenery nature glass bottle landscape, , purple galaxy bottle,"
+            ]
+          },
+          {
+            "id": 3,
+            "type": "KSampler",
+            "pos": [
+              863,
+              186
+            ],
+            "size": [
+              315,
+              262
+            ],
+            "flags": {},
+            "order": 2,
+            "mode": 0,
+            "inputs": [
+              {
+                "localized_name": "model",
+                "name": "model",
+                "type": "MODEL",
+                "link": 12
+              },
+              {
+                "localized_name": "positive",
+                "name": "positive",
+                "type": "CONDITIONING",
+                "link": 4
+              },
+              {
+                "localized_name": "negative",
+                "name": "negative",
+                "type": "CONDITIONING",
+                "link": 10
+              },
+              {
+                "localized_name": "latent_image",
+                "name": "latent_image",
+                "type": "LATENT",
+                "link": 11
+              }
+            ],
+            "outputs": [
+              {
+                "localized_name": "LATENT",
+                "name": "LATENT",
+                "type": "LATENT",
+                "slot_index": 0,
+                "links": [
+                  7
+                ]
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "KSampler"
+            },
+            "widgets_values": [
+              32115495257102,
+              "randomize",
+              20,
+              8,
+              "euler",
+              "normal",
+              1
+            ]
+          },
+          {
+            "id": 10,
+            "type": "dbe5763f-440b-47b4-82ac-454f1f98b0e3",
+            "pos": [
+              194.13900756835938,
+              657.3333740234375
+            ],
+            "size": [
+              140,
+              106
+            ],
+            "flags": {},
+            "order": 1,
+            "mode": 0,
+            "inputs": [],
+            "outputs": [
+              {
+                "localized_name": "CONDITIONING",
+                "name": "CONDITIONING",
+                "type": "CONDITIONING",
+                "links": [
+                  10
+                ]
+              },
+              {
+                "localized_name": "LATENT",
+                "name": "LATENT",
+                "type": "LATENT",
+                "links": [
+                  11
+                ]
+              },
+              {
+                "localized_name": "MODEL",
+                "name": "MODEL",
+                "type": "MODEL",
+                "links": [
+                  12
+                ]
+              },
+              {
+                "localized_name": "CLIP",
+                "name": "CLIP",
+                "type": "CLIP",
+                "links": [
+                  13
+                ]
+              },
+              {
+                "localized_name": "VAE",
+                "name": "VAE",
+                "type": "VAE",
+                "links": [
+                  14
+                ]
+              }
+            ],
+            "title": "subgraph 3",
+            "properties": {},
+            "widgets_values": []
+          }
+        ],
+        "groups": [],
+        "links": [
+          {
+            "id": 4,
+            "origin_id": 6,
+            "origin_slot": 0,
+            "target_id": 3,
+            "target_slot": 1,
+            "type": "CONDITIONING"
+          },
+          {
+            "id": 7,
+            "origin_id": 3,
+            "origin_slot": 0,
+            "target_id": -20,
+            "target_slot": 0,
+            "type": "LATENT"
+          },
+          {
+            "id": 10,
+            "origin_id": 10,
+            "origin_slot": 0,
+            "target_id": 3,
+            "target_slot": 2,
+            "type": "CONDITIONING"
+          },
+          {
+            "id": 11,
+            "origin_id": 10,
+            "origin_slot": 1,
+            "target_id": 3,
+            "target_slot": 3,
+            "type": "LATENT"
+          },
+          {
+            "id": 12,
+            "origin_id": 10,
+            "origin_slot": 2,
+            "target_id": 3,
+            "target_slot": 0,
+            "type": "MODEL"
+          },
+          {
+            "id": 13,
+            "origin_id": 10,
+            "origin_slot": 3,
+            "target_id": 6,
+            "target_slot": 0,
+            "type": "CLIP"
+          },
+          {
+            "id": 14,
+            "origin_id": 10,
+            "origin_slot": 4,
+            "target_id": -20,
+            "target_slot": 1,
+            "type": "VAE"
+          }
+        ],
+        "extra": {}
+      },
+      {
+        "id": "dbe5763f-440b-47b4-82ac-454f1f98b0e3",
+        "version": 1,
+        "state": {
+          "lastGroupId": 0,
+          "lastNodeId": 9,
+          "lastLinkId": 9,
+          "lastRerouteId": 0
+        },
+        "revision": 0,
+        "config": {},
+        "name": "subgraph 3",
+        "inputNode": {
+          "id": -10,
+          "bounding": [
+            -154,
+            517,
+            120,
+            40
+          ]
+        },
+        "outputNode": {
+          "id": -20,
+          "bounding": [
+            898.2780151367188,
+            467,
+            128.6640625,
+            140
+          ]
+        },
+        "inputs": [],
+        "outputs": [
+          {
+            "id": "b4882169-329b-43f6-a373-81abfbdea55b",
+            "name": "CONDITIONING",
+            "type": "CONDITIONING",
+            "linkIds": [
+              6
+            ],
+            "localized_name": "CONDITIONING",
+            "pos": {
+              "0": 918.2780151367188,
+              "1": 487
+            }
+          },
+          {
+            "id": "01f51f96-a741-428e-8772-9557ee50b609",
+            "name": "LATENT",
+            "type": "LATENT",
+            "linkIds": [
+              2
+            ],
+            "localized_name": "LATENT",
+            "pos": {
+              "0": 918.2780151367188,
+              "1": 507
+            }
+          },
+          {
+            "id": "47fa906e-d80b-45c3-a596-211a0e59d4a1",
+            "name": "MODEL",
+            "type": "MODEL",
+            "linkIds": [
+              1
+            ],
+            "localized_name": "MODEL",
+            "pos": {
+              "0": 918.2780151367188,
+              "1": 527
+            }
+          },
+          {
+            "id": "f03dccd7-10e8-4513-9994-15854a92d192",
+            "name": "CLIP",
+            "type": "CLIP",
+            "linkIds": [
+              3
+            ],
+            "localized_name": "CLIP",
+            "pos": {
+              "0": 918.2780151367188,
+              "1": 547
+            }
+          },
+          {
+            "id": "a666877f-e34f-49bc-8a78-b26156656b83",
+            "name": "VAE",
+            "type": "VAE",
+            "linkIds": [
+              8
+            ],
+            "localized_name": "VAE",
+            "pos": {
+              "0": 918.2780151367188,
+              "1": 567
+            }
+          }
+        ],
+        "widgets": [],
+        "nodes": [
+          {
+            "id": 7,
+            "type": "CLIPTextEncode",
+            "pos": [
+              413,
+              389
+            ],
+            "size": [
+              425.27801513671875,
+              180.6060791015625
+            ],
+            "flags": {},
+            "order": 2,
+            "mode": 0,
+            "inputs": [
+              {
+                "localized_name": "clip",
+                "name": "clip",
+                "type": "CLIP",
+                "link": 5
+              }
+            ],
+            "outputs": [
+              {
+                "localized_name": "CONDITIONING",
+                "name": "CONDITIONING",
+                "type": "CONDITIONING",
+                "slot_index": 0,
+                "links": [
+                  6
+                ]
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "CLIPTextEncode"
+            },
+            "widgets_values": [
+              "text, watermark"
+            ]
+          },
+          {
+            "id": 5,
+            "type": "EmptyLatentImage",
+            "pos": [
+              473,
+              609
+            ],
+            "size": [
+              315,
+              106
+            ],
+            "flags": {},
+            "order": 0,
+            "mode": 0,
+            "inputs": [],
+            "outputs": [
+              {
+                "localized_name": "LATENT",
+                "name": "LATENT",
+                "type": "LATENT",
+                "slot_index": 0,
+                "links": [
+                  2
+                ]
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "EmptyLatentImage"
+            },
+            "widgets_values": [
+              512,
+              512,
+              1
+            ]
+          },
+          {
+            "id": 4,
+            "type": "CheckpointLoaderSimple",
+            "pos": [
+              26,
+              474
+            ],
+            "size": [
+              315,
+              98
+            ],
+            "flags": {},
+            "order": 1,
+            "mode": 0,
+            "inputs": [],
+            "outputs": [
+              {
+                "localized_name": "MODEL",
+                "name": "MODEL",
+                "type": "MODEL",
+                "slot_index": 0,
+                "links": [
+                  1
+                ]
+              },
+              {
+                "localized_name": "CLIP",
+                "name": "CLIP",
+                "type": "CLIP",
+                "slot_index": 1,
+                "links": [
+                  3,
+                  5
+                ]
+              },
+              {
+                "localized_name": "VAE",
+                "name": "VAE",
+                "type": "VAE",
+                "slot_index": 2,
+                "links": [
+                  8
+                ]
+              }
+            ],
+            "properties": {
+              "Node name for S&R": "CheckpointLoaderSimple"
+            },
+            "widgets_values": [
+              "v1-5-pruned-emaonly-fp16.safetensors"
+            ]
+          }
+        ],
+        "groups": [],
+        "links": [
+          {
+            "id": 5,
+            "origin_id": 4,
+            "origin_slot": 1,
+            "target_id": 7,
+            "target_slot": 0,
+            "type": "CLIP"
+          },
+          {
+            "id": 6,
+            "origin_id": 7,
+            "origin_slot": 0,
+            "target_id": -20,
+            "target_slot": 0,
+            "type": "CONDITIONING"
+          },
+          {
+            "id": 2,
+            "origin_id": 5,
+            "origin_slot": 0,
+            "target_id": -20,
+            "target_slot": 1,
+            "type": "LATENT"
+          },
+          {
+            "id": 1,
+            "origin_id": 4,
+            "origin_slot": 0,
+            "target_id": -20,
+            "target_slot": 2,
+            "type": "MODEL"
+          },
+          {
+            "id": 3,
+            "origin_id": 4,
+            "origin_slot": 1,
+            "target_id": -20,
+            "target_slot": 3,
+            "type": "CLIP"
+          },
+          {
+            "id": 8,
+            "origin_id": 4,
+            "origin_slot": 2,
+            "target_id": -20,
+            "target_slot": 4,
+            "type": "VAE"
+          }
+        ],
+        "extra": {}
+      }
+    ]
+  },
+  "config": {},
+  "extra": {
+    "frontendVersion": "1.24.0-1"
+  },
+  "version": 0.4
+}

browser_tests/tests/sidebar/nodeLibrary.spec.ts

@@ -336,4 +336,54 @@ test.describe('Node library sidebar', () => {
     await comfyPage.page.waitForTimeout(1000)
     expect(await tab.getNode('KSampler (Advanced)').count()).toBe(2)
   })
+
+  test('Subgraph node display name updates in library when renamed', async ({
+    comfyPage
+  }) => {
+    // Load a workflow with subgraphs to populate the node library
+    await comfyPage.loadWorkflow('nested-subgraph')
+    await comfyPage.nextFrame()
+
+    const tab = comfyPage.menu.nodeLibraryTab
+
+    // Navigate to subgraph folder in node library
+    await tab.getFolder('subgraph').click()
+    await comfyPage.nextFrame()
+
+    // Get initial subgraph node name in the library
+    const initialSubgraphNodeCount = await tab.getNode('Subgraph Node').count()
+    expect(initialSubgraphNodeCount).toBeGreaterThan(0)
+
+    // Get the subgraph node by ID (node 10 is the subgraph)
+    const subgraphNode = await comfyPage.getNodeRefById('10')
+    const nodePos = await subgraphNode.getPosition()
+    const nodeSize = await subgraphNode.getSize()
+
+    // Double-click on the title area of the subgraph node to edit
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: nodePos.x + nodeSize.width / 2,
+        y: nodePos.y - 10 // Title area is above the node body
+      },
+      delay: 5
+    })
+
+    // Wait for title editor to appear
+    await expect(comfyPage.page.locator('.node-title-editor')).toBeVisible()
+
+    // Clear existing text and type new title
+    await comfyPage.page.keyboard.press('Control+a')
+    const newTitle = 'Renamed Subgraph Node'
+    await comfyPage.page.keyboard.type(newTitle)
+    await comfyPage.page.keyboard.press('Enter')
+
+    // Wait a frame for the update to complete
+    await comfyPage.nextFrame()
+
+    // Verify the node library shows the updated title
+    expect(await tab.getNode(newTitle).count()).toBeGreaterThan(0)
+
+    // Verify the old node name is no longer in the library
+    expect(await tab.getNode('Subgraph Node').count()).toBe(0)
+  })
 })

browser_tests/tests/subgraph/subgraphBreadcrumb.spec.ts

@@ -0,0 +1,82 @@
+import { expect } from '@playwright/test'
+
+import { comfyPageFixture as test } from '../fixtures/ComfyPage'
+
+test.describe('Subgraph Breadcrumb Title Sync', () => {
+  test.beforeEach(async ({ comfyPage }) => {
+    await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
+  })
+
+  test('Breadcrumb updates when subgraph node title is changed', async ({
+    comfyPage
+  }) => {
+    // Load a workflow with subgraphs
+    await comfyPage.loadWorkflow('nested-subgraph')
+    await comfyPage.nextFrame()
+
+    // Get the subgraph node by ID (node 10 is the subgraph)
+    const subgraphNode = await comfyPage.getNodeRefById('10')
+
+    // Get node position and double-click on it to enter the subgraph
+    const nodePos = await subgraphNode.getPosition()
+    const nodeSize = await subgraphNode.getSize()
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: nodePos.x + nodeSize.width / 2,
+        y: nodePos.y + nodeSize.height / 2 + 10
+      }
+    })
+    await comfyPage.nextFrame()
+
+    // Wait for breadcrumb to appear
+    await comfyPage.page.waitForSelector('.subgraph-breadcrumb', {
+      state: 'visible',
+      timeout: 20000
+    })
+
+    // Get initial breadcrumb text
+    const breadcrumb = comfyPage.page.locator('.subgraph-breadcrumb')
+    const initialBreadcrumbText = await breadcrumb.textContent()
+
+    // Go back to main graph
+    await comfyPage.page.keyboard.press('Escape')
+
+    // Double-click on the title area of the subgraph node to edit
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: nodePos.x + nodeSize.width / 2,
+        y: nodePos.y - 10 // Title area is above the node body
+      },
+      delay: 5
+    })
+
+    // Wait for title editor to appear
+    await expect(comfyPage.page.locator('.node-title-editor')).toBeVisible()
+
+    // Clear existing text and type new title
+    await comfyPage.page.keyboard.press('Control+a')
+    const newTitle = 'Updated Subgraph Title'
+    await comfyPage.page.keyboard.type(newTitle)
+    await comfyPage.page.keyboard.press('Enter')
+
+    // Wait a frame for the update to complete
+    await comfyPage.nextFrame()
+
+    // Enter the subgraph again
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: nodePos.x + nodeSize.width / 2,
+        y: nodePos.y + nodeSize.height / 2
+      },
+      delay: 5
+    })
+
+    // Wait for breadcrumb
+    await comfyPage.page.waitForSelector('.subgraph-breadcrumb')
+
+    // Check that breadcrumb now shows the new title
+    const updatedBreadcrumbText = await breadcrumb.textContent()
+    expect(updatedBreadcrumbText).toContain(newTitle)
+    expect(updatedBreadcrumbText).not.toBe(initialBreadcrumbText)
+  })
+})

browser_tests/tests/subgraph/subgraphPersistence.spec.ts

@@ -0,0 +1,55 @@
+import { expect } from '@playwright/test'
+
+import { comfyPageFixture as test } from '../fixtures/ComfyPage'
+
+test.describe('Subgraph Persistence', () => {
+  test('Node library updates when subgraph title changes', async ({
+    comfyPage
+  }) => {
+    // Load a workflow with subgraphs to populate the node library
+    await comfyPage.loadWorkflow('nested-subgraph')
+    await comfyPage.nextFrame()
+
+    const tab = comfyPage.menu.nodeLibraryTab
+
+    // Navigate to subgraph folder in node library
+    await tab.getFolder('subgraph').click()
+    await comfyPage.nextFrame()
+
+    // Get initial subgraph node name in the library
+    const initialSubgraphNodeCount = await tab.getNode('Subgraph Node').count()
+    expect(initialSubgraphNodeCount).toBeGreaterThan(0)
+
+    // Get the subgraph node by ID (node 10 is the subgraph)
+    const subgraphNode = await comfyPage.getNodeRefById('10')
+    const nodePos = await subgraphNode.getPosition()
+    const nodeSize = await subgraphNode.getSize()
+
+    // Double-click on the title area of the subgraph node to edit
+    await comfyPage.canvas.dblclick({
+      position: {
+        x: nodePos.x + nodeSize.width / 2,
+        y: nodePos.y - 10 // Title area is above the node body
+      },
+      delay: 5
+    })
+
+    // Wait for title editor to appear
+    await expect(comfyPage.page.locator('.node-title-editor')).toBeVisible()
+
+    // Clear existing text and type new title
+    await comfyPage.page.keyboard.press('Control+a')
+    const newTitle = 'Renamed Subgraph Node'
+    await comfyPage.page.keyboard.type(newTitle)
+    await comfyPage.page.keyboard.press('Enter')
+
+    // Wait a frame for the update to complete
+    await comfyPage.nextFrame()
+
+    // Verify the node library shows the updated title
+    expect(await tab.getNode(newTitle).count()).toBeGreaterThan(0)
+
+    // Verify the old node name is no longer in the library
+    expect(await tab.getNode('Subgraph Node').count()).toBe(0)
+  })
+})

dangerfile.ts

@@ -0,0 +1,72 @@
+import { danger, fail } from 'danger'
+
+// Check if we should run the checks
+const shouldRunChecks = async () => {
+  const allChangedFiles = [
+    ...danger.git.modified_files,
+    ...danger.git.created_files
+  ]
+  const srcChanges = allChangedFiles.filter((file) => file.startsWith('src/'))
+
+  if (srcChanges.length === 0) {
+    return false
+  }
+
+  // Check total lines changed in src files
+  let totalLinesChanged = 0
+  for (const file of srcChanges) {
+    const diff = await danger.git.diffForFile(file)
+    if (diff) {
+      // Count only lines with actual content (non-empty after trimming whitespace)
+      // This excludes empty lines and lines containing only spaces/tabs
+      const additions =
+        diff.added?.split('\n').filter((line) => line.trim()).length || 0
+      const deletions =
+        diff.removed?.split('\n').filter((line) => line.trim()).length || 0
+      totalLinesChanged += additions + deletions
+    }
+  }
+
+  return totalLinesChanged > 3
+}
+
+// Check if browser tests were updated
+const checkBrowserTestCoverage = () => {
+  const allChangedFiles = [
+    ...danger.git.modified_files,
+    ...danger.git.created_files
+  ]
+  const hasBrowserTestChanges = allChangedFiles.some(
+    (file) => file.startsWith('browser_tests/') && file.endsWith('.ts')
+  )
+
+  if (!hasBrowserTestChanges) {
+    fail(`🧪 **E2E Test Coverage Missing**
+
+All changes should be covered under E2E testing. Please add or update browser tests.`)
+  }
+}
+
+// Check for screen recording in PR description
+const checkScreenRecording = () => {
+  const description = danger.github.pr.body || ''
+  const hasRecording =
+    /github\.com\/user-attachments\/assets\/[a-f0-9-]+/i.test(description) ||
+    /youtube\.com\/watch|youtu\.be\//i.test(description)
+
+  if (!hasRecording) {
+    fail(`📹 **Visual Documentation Missing**
+
+Please add a screen recording or screenshot:
+- GitHub: Drag & drop media to PR description  
+- YouTube: Add YouTube link`)
+  }
+}
+
+// Run the checks only if conditions are met
+shouldRunChecks().then((shouldRun) => {
+  if (shouldRun) {
+    checkBrowserTestCoverage()
+    checkScreenRecording()
+  }
+})

package.json

@@ -45,6 +45,7 @@
     "@vue/test-utils": "^2.4.6",
     "autoprefixer": "^10.4.19",
     "chalk": "^5.3.0",
+    "danger": "^13.0.4",
     "eslint": "^9.12.0",
     "eslint-config-prettier": "^10.1.2",
     "eslint-plugin-prettier": "^5.2.6",

src/components/graph/TitleEditor.vue

@@ -22,9 +22,11 @@ import EditableText from '@/components/common/EditableText.vue'
 import { useAbsolutePosition } from '@/composables/element/useAbsolutePosition'
 import { app } from '@/scripts/app'
 import { useCanvasStore, useTitleEditorStore } from '@/stores/graphStore'
+import { useNodeDefStore } from '@/stores/nodeDefStore'
 import { useSettingStore } from '@/stores/settingStore'
 
 const settingStore = useSettingStore()
+const nodeDefStore = useNodeDefStore()
 
 const showInput = ref(false)
 const editedTitle = ref('')
@@ -41,7 +43,18 @@ const previousCanvasDraggable = ref(true)
 
 const onEdit = (newValue: string) => {
   if (titleEditorStore.titleEditorTarget && newValue.trim() !== '') {
-    titleEditorStore.titleEditorTarget.title = newValue.trim()
+    const trimmedTitle = newValue.trim()
+    titleEditorStore.titleEditorTarget.title = trimmedTitle
+
+    // If this is a subgraph node, sync the runtime subgraph name for breadcrumb reactivity
+    const target = titleEditorStore.titleEditorTarget
+    if (target instanceof LGraphNode && target.isSubgraphNode?.()) {
+      target.subgraph.name = trimmedTitle
+
+      // Also update the node definition display name in the store
+      nodeDefStore.updateNodeDefDisplayName(target.type, trimmedTitle)
+    }
+
     app.graph.setDirtyCanvas(true, true)
   }
   showInput.value = false

src/composables/useLitegraphSettings.ts

@@ -124,12 +124,9 @@ export const useLitegraphSettings = () => {
   })
 
   watchEffect(() => {
-    const navigationMode = settingStore.get('Comfy.Canvas.NavigationMode') as
-      | 'standard'
-      | 'legacy'
-
-    LiteGraph.canvasNavigationMode = navigationMode
-    LiteGraph.macTrackpadGestures = navigationMode === 'standard'
+    LiteGraph.macTrackpadGestures = settingStore.get(
+      'LiteGraph.Pointer.TrackpadGestures'
+    )
   })
 
   watchEffect(() => {

src/constants/coreSettings.ts

@@ -785,21 +785,6 @@ export const CORE_SETTINGS: SettingParams[] = [
     defaultValue: 0.6,
     versionAdded: '1.9.1'
   },
-  {
-    id: 'Comfy.Canvas.NavigationMode',
-    category: ['LiteGraph', 'Canvas', 'CanvasNavigationMode'],
-    name: 'Canvas Navigation Mode',
-    defaultValue: 'legacy',
-    type: 'combo',
-    options: [
-      { value: 'standard', text: 'Standard (New)' },
-      { value: 'legacy', text: 'Left-Click Pan (Legacy)' }
-    ],
-    versionAdded: '1.24.0',
-    defaultsByInstallVersion: {
-      '1.24.0': 'standard'
-    }
-  },
   {
     id: 'Comfy.Canvas.SelectionToolbox',
     category: ['LiteGraph', 'Canvas', 'SelectionToolbox'],
@@ -870,6 +855,17 @@ export const CORE_SETTINGS: SettingParams[] = [
     versionAdded: '1.20.4',
     versionModified: '1.20.5'
   },
+  {
+    id: 'LiteGraph.Pointer.TrackpadGestures',
+    category: ['LiteGraph', 'Pointer', 'Trackpad Gestures'],
+    experimental: true,
+    name: 'Enable trackpad gestures',
+    tooltip:
+      'This setting enables trackpad mode for the canvas, allowing pinch-to-zoom and panning with two fingers.',
+    type: 'boolean',
+    defaultValue: false,
+    versionAdded: '1.19.1'
+  },
   // Release data stored in settings
   {
     id: 'Comfy.Release.Version',

src/schemas/apiSchema.ts

@@ -454,7 +454,6 @@ const zSettings = z.object({
   'Comfy.TutorialCompleted': z.boolean(),
   'Comfy.InstalledVersion': z.string().nullable(),
   'Comfy.Node.AllowImageSizeDraw': z.boolean(),
-  'Comfy.Canvas.NavigationMode': z.string(),
   'Comfy-Desktop.AutoUpdate': z.boolean(),
   'Comfy-Desktop.SendStatistics': z.boolean(),
   'Comfy-Desktop.WindowStyle': z.string(),

src/stores/nodeDefStore.ts

@@ -1,4 +1,4 @@
-import type { LGraphNode } from '@comfyorg/litegraph'
+import { type LGraphNode, LiteGraph } from '@comfyorg/litegraph'
 import axios from 'axios'
 import _ from 'lodash'
 import { defineStore } from 'pinia'
@@ -304,6 +304,28 @@ export const useNodeDefStore = defineStore('nodeDef', () => {
     nodeDefsByName.value[nodeDef.name] = nodeDefImpl
     nodeDefsByDisplayName.value[nodeDef.display_name] = nodeDefImpl
   }
+  function updateNodeDefDisplayName(nodeType: string, newDisplayName: string) {
+    const existingNodeDef = nodeDefsByName.value[nodeType]
+    if (!existingNodeDef) return
+
+    // Remove old display name mapping
+    delete nodeDefsByDisplayName.value[existingNodeDef.display_name]
+
+    // Clone and mutate like bookmark store does to preserve references
+    const clonedNodeDef = _.clone(existingNodeDef) as any
+    clonedNodeDef.display_name = newDisplayName
+
+    // Update both mappings
+    nodeDefsByName.value[nodeType] = clonedNodeDef
+    nodeDefsByDisplayName.value[newDisplayName] = clonedNodeDef
+
+    // Also update the LiteGraph registered node type's static title
+    // This is necessary for new instances to use the updated display name
+    const RegisteredNodeClass = LiteGraph.registered_node_types[nodeType]
+    if (RegisteredNodeClass) {
+      RegisteredNodeClass.title = newDisplayName
+    }
+  }
   function fromLGraphNode(node: LGraphNode): ComfyNodeDefImpl | null {
     // Frontend-only nodes don't have nodeDef
     // @ts-expect-error Optional chaining used in index
@@ -324,6 +346,7 @@ export const useNodeDefStore = defineStore('nodeDef', () => {
 
     updateNodeDefs,
     addNodeDef,
+    updateNodeDefDisplayName,
     fromLGraphNode
   }
 })

tests-ui/tests/store/nodeDefStore.test.ts

@@ -0,0 +1,142 @@
+import { LiteGraph } from '@comfyorg/litegraph'
+import { createPinia, setActivePinia } from 'pinia'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+
+import type { ComfyNodeDef as ComfyNodeDefV1 } from '@/schemas/nodeDefSchema'
+import { ComfyNodeDefImpl, useNodeDefStore } from '@/stores/nodeDefStore'
+
+// Mock LiteGraph
+vi.mock('@comfyorg/litegraph', async () => {
+  const actual = await vi.importActual('@comfyorg/litegraph')
+  return {
+    ...actual,
+    LiteGraph: {
+      registered_node_types: {}
+    }
+  }
+})
+
+describe('useNodeDefStore', () => {
+  let store: ReturnType<typeof useNodeDefStore>
+
+  beforeEach(() => {
+    // Set up Pinia
+    setActivePinia(createPinia())
+    // Create fresh store instance
+    store = useNodeDefStore()
+    // Clear registered node types
+    LiteGraph.registered_node_types = {}
+  })
+
+  describe('updateNodeDefDisplayName', () => {
+    const createTestNodeDef = (
+      name: string,
+      displayName: string
+    ): ComfyNodeDefV1 => ({
+      name,
+      display_name: displayName,
+      category: 'test',
+      python_module: 'test_module',
+      description: 'Test node',
+      input: { required: {}, optional: {} },
+      output: [],
+      output_node: false
+    })
+
+    it('should update display name in nodeDefsByName mapping', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      store.addNodeDef(nodeDef)
+
+      store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+
+      const updatedNodeDef = store.nodeDefsByName['TestNode']
+      expect(updatedNodeDef.display_name).toBe('New Display Name')
+    })
+
+    it('should update nodeDefsByDisplayName mapping', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      store.addNodeDef(nodeDef)
+
+      store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+
+      // Old display name should be removed
+      expect(
+        store.nodeDefsByDisplayName['Original Display Name']
+      ).toBeUndefined()
+      // New display name should be added
+      expect(store.nodeDefsByDisplayName['New Display Name']).toBeDefined()
+      expect(store.nodeDefsByDisplayName['New Display Name'].name).toBe(
+        'TestNode'
+      )
+    })
+
+    it('should update the object while preserving clone pattern', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      store.addNodeDef(nodeDef)
+
+      const originalRef = store.nodeDefsByName['TestNode']
+      store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+      const updatedRef = store.nodeDefsByName['TestNode']
+
+      // Clone creates a new reference but preserves properties
+      expect(updatedRef).not.toBe(originalRef)
+      expect(updatedRef.display_name).toBe('New Display Name')
+      expect(updatedRef.name).toBe('TestNode')
+      expect(updatedRef.category).toBe('test')
+    })
+
+    it('should update LiteGraph registered node type title', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      store.addNodeDef(nodeDef)
+
+      // Simulate a registered LiteGraph node type
+      const mockNodeClass = { title: 'Original Display Name' }
+      LiteGraph.registered_node_types['TestNode'] = mockNodeClass as any
+
+      store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+
+      expect(mockNodeClass.title).toBe('New Display Name')
+    })
+
+    it('should handle non-existent node type gracefully', () => {
+      // Should not throw when updating non-existent node
+      expect(() => {
+        store.updateNodeDefDisplayName('NonExistentNode', 'New Display Name')
+      }).not.toThrow()
+    })
+
+    it('should handle missing LiteGraph registered type gracefully', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      store.addNodeDef(nodeDef)
+
+      // No registered LiteGraph type
+      expect(LiteGraph.registered_node_types['TestNode']).toBeUndefined()
+
+      // Should not throw
+      expect(() => {
+        store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+      }).not.toThrow()
+
+      // Store should still be updated
+      expect(store.nodeDefsByName['TestNode'].display_name).toBe(
+        'New Display Name'
+      )
+    })
+
+    it('should work with ComfyNodeDefImpl instances', () => {
+      const nodeDef = createTestNodeDef('TestNode', 'Original Display Name')
+      const nodeDefImpl = new ComfyNodeDefImpl(nodeDef)
+
+      // Manually add to store mappings
+      store.nodeDefsByName['TestNode'] = nodeDefImpl
+      store.nodeDefsByDisplayName['Original Display Name'] = nodeDefImpl
+
+      store.updateNodeDefDisplayName('TestNode', 'New Display Name')
+
+      expect(store.nodeDefsByName['TestNode'].display_name).toBe(
+        'New Display Name'
+      )
+      expect(store.nodeDefsByDisplayName['New Display Name']).toBeDefined()
+    })
+  })
+})