Enhance workflow error reporting and validation processes

- Updated link-checker to provide clearer comments on broken links and how to fix them.
- Improved sync-custom-nodes workflow documentation.
- Enhanced template validation workflow to capture and comment on validation errors, including thumbnail issues, with specific guidance for resolution.

This update aims to improve user experience by ensuring all links and templates are properly validated before merging.

Author: comfyui-wiki

.github/workflows/link-checker.yml

@@ -198,67 +198,12 @@ jobs:
               lycheeReport = 'Lychee reported an error (non-zero exit) but no broken-link rows were parsed. Please check artifacts for details.';
             }
 
-            const comment = `## 🔗 Broken Links Found in Workflow Files
-
-            The link checker found broken or unreachable links in your workflow JSON files.
-
-            **Why this matters:** Broken model download links will prevent users from using the workflows properly. All URLs in \`properties.models[].url\` fields and MarkdownNote/Note widgets must be valid and accessible.
-
-            ### ❌ Failed Links:
+            const comment = `## ❌ Broken Links Found
 
             ${lycheeReport}
 
-            ${excludedReport ? '---\n\n' + excludedReport + '\n\n**Note:** These links are intentionally excluded from checking. If you need to modify the exclusion rules, edit \`.lycheeignore\`.\n\n' : ''}
-
-            ### Common Issues:
-
-            1. **404 Not Found**: The file or page doesn't exist
-               - Verify the URL is correct
-               - Check if the file was moved or renamed
-               - Update to the correct URL
-
-            2. **Timeout**: Server didn't respond in time
-               - Verify the URL is correct
-               - Check if the server is temporarily down
-               - Try accessing the URL manually
-
-            3. **Connection Failed**: Unable to reach the server
-               - Verify the domain exists
-               - Check if the repository/file was deleted
-
-            ### How to Fix:
-
-            1. **For model URLs in \`properties.models[].url\`:**
-               - Update the URL to a valid download link
-               - Ensure the model file is still available
-               - Test the URL manually before committing
-
-            2. **For links in MarkdownNote/Note widgets:**
-               - Update or remove broken documentation links
-               - Ensure all referenced resources are accessible
-
-            ### Testing Links Locally:
-
-            \`\`\`bash
-            # Extract and check links
-            python3 scripts/check_links.py extract
-            lychee links_to_check.txt
-
-            # View detailed report
-            python3 scripts/check_links.py report
-
-            # View excluded links
-            python3 scripts/check_links.py report-excluded
-            \`\`\`
-
-            ### 📦 Detailed Reports
-
-            For a complete list of all links and their sources, download the artifacts from this workflow run:
-            - \`link_report_detailed.txt\` - Full detailed report showing all links and which files contain them
-            - \`excluded_links.txt\` - Complete list of excluded links
-            - \`lychee-report.md\` - Full lychee check report
-
-            **Note**: All links in workflow files must be valid and accessible before merging to ensure users can download required models and access documentation.`;
+            ${excludedReport ? '**Excluded:**\n' + excludedReport + '\n\n' : ''}
+            **Fix:** Update or remove broken URLs in workflow files. Check artifacts for full report.`;
 
             github.rest.issues.createComment({
               issue_number: context.issue.number,

.github/workflows/sync-custom-nodes.yml

@@ -1,9 +1,8 @@
 name: Sync Custom Nodes Requirements
 
-# ⚠️ TEMPORARILY DISABLED - Remove the `if: false` condition below to re-enable
 # This workflow automatically syncs requiresCustomNodes field in all index files
 # when templates are changed in PRs.
-# 
+#
 # Process:
 #   1. Scans all template JSON files (excluding index files) for cnr_id values
 #   2. Extracts all non-comfy-core cnr_ids for each template

.github/workflows/validate-templates.yml

@@ -31,8 +31,37 @@ jobs:
         pip install jsonschema
     
     - name: Run validation script
-      run: python scripts/validate_templates.py
-    
+      id: validation
+      run: |
+        python scripts/validate_templates.py 2>&1 | tee validation_output.txt
+        echo "exit_code=${PIPESTATUS[0]}" >> $GITHUB_OUTPUT
+      continue-on-error: true
+
+    - name: Comment on PR with validation errors
+      if: steps.validation.outputs.exit_code != '0' && github.event_name == 'pull_request'
+      uses: actions/github-script@v7
+      with:
+        github-token: ${{ secrets.GITHUB_TOKEN }}
+        script: |
+          const fs = require('fs');
+          let output = '';
+          try {
+            output = fs.readFileSync('validation_output.txt', 'utf8');
+          } catch (error) {
+            output = 'Unable to read validation output';
+          }
+
+          github.rest.issues.createComment({
+            issue_number: context.issue.number,
+            owner: context.repo.owner,
+            repo: context.repo.repo,
+            body: `## ❌ Template Validation Failed
+
+\`\`\`
+${output}
+\`\`\``
+          });
+
     - name: Check JSON syntax
       run: |
         echo "Checking JSON syntax..."
@@ -94,28 +123,26 @@ jobs:
             report = 'Unable to read thumbnail validation report';
           }
           
-          const comment = `## ❌ Thumbnail Validation Issues Found
-          
-          Your template has thumbnail configuration issues that need to be resolved.
-          
-          **Why this matters:** Templates need proper thumbnails to provide users with visual previews. Templates with special thumbnail variants like "compareSlider" or "hoverDissolve" require specific thumbnail files to work correctly.
-          
-          ### Validation Report:
-          \`\`\`
-          ${report}
-          \`\`\`
-          
-          ### How to Fix:
-          1. **Missing thumbnails**: Ensure each template has at least one thumbnail file (e.g., \`template-1.webp\`)
-          2. **Incomplete dual thumbnails**: Templates with \`thumbnailVariant: "compareSlider"\` or \`"hoverDissolve"\` need both \`-1\` and \`-2\` thumbnail files
-          3. **Check file naming**: Thumbnail files should be named \`{template_name}-{number}.{extension}\`
-          4. **Verify file format**: Use the correct file extension (\`.webp\`, \`.mp3\`, etc.) as specified in the template's \`mediaSubtype\`
-          
-          **Note**: All templates must have proper thumbnails configured to ensure the best user experience when browsing templates.`;
+          const comment = `## ❌ Thumbnail Validation Failed
+
+\`\`\`
+${report}
+\`\`\`
+
+**Fix:**
+- Missing: Add \`template-1.webp\`
+- Dual thumbnails (\`compareSlider\`/\`hoverDissolve\`): Need both \`-1\` and \`-2\` files
+- Check naming: \`{name}-{number}.{ext}\``;
 
           github.rest.issues.createComment({
             issue_number: context.issue.number,
             owner: context.repo.owner,
             repo: context.repo.repo,
             body: comment
-          });
+          });
+
+    - name: Fail workflow if validation failed
+      if: steps.validation.outputs.exit_code != '0' || steps.thumbnail_check.outputs.thumbnail_issues == 'true'
+      run: |
+        echo "❌ Validation failed"
+        exit 1