Update some basic workflows and add model links. (#247)

Author: comfyui-wiki

.github/workflows/check_input_assets.yml

@@ -30,11 +30,11 @@ jobs:
       - name: Run input assets validation
         id: validation
         run: |
-          python scripts/check_input_assets.py
+          python scripts/check_input_assets.py || echo "validation_failed=true" >> $GITHUB_OUTPUT
         continue-on-error: true
 
-      - name: Comment on PR with validation results
-        if: always()
+      - name: Comment on PR with validation results (failures only)
+        if: steps.validation.outputs.validation_failed == 'true'
         uses: actions/github-script@v7
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
@@ -86,6 +86,36 @@ jobs:
               console.log('Created new comment');
             }
       
+      - name: Remove or update comment on successful validation
+        if: steps.validation.outputs.validation_failed != 'true'
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            // Find existing comment
+            const { data: comments } = await github.rest.issues.listComments({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: context.issue.number,
+            });
+            
+            const botComment = comments.find(comment => 
+              comment.user.type === 'Bot' && 
+              comment.body.includes('Input Assets Validation Report')
+            );
+            
+            if (botComment) {
+              // Delete the existing comment since validation passed
+              await github.rest.issues.deleteComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: botComment.id
+              });
+              console.log('Deleted previous validation failure comment - validation now passes');
+            } else {
+              console.log('No previous comment found - validation passed cleanly');
+            }
+      
       - name: Upload validation report as artifact
         if: always()
         uses: actions/upload-artifact@v4

.github/workflows/model-analysis.yml

@@ -60,7 +60,7 @@ jobs:
               report = 'Unable to read model analysis report';
             }
             
-            const comment = `## ❌ Model Analysis Check Failed
+            const commentBody = `## ❌ Model Analysis Check Failed
             
             Model configuration issues were found in your template files that need to be fixed.
             
@@ -83,19 +83,48 @@ jobs:
             
             3. **Markdown link errors**:
                - Ensure \`[filename.safetensors](url)\` links have consistent filenames between text and URL
+               - Note: Civitai links are automatically whitelisted as they use model IDs instead of filenames
             
             4. **Subgraph nodes**:
                - Note: Subgraph nodes (UUID-type) are automatically skipped from model validation
                - Their model configurations are handled within the subgraph definition
             
-            Please fix these issues and resubmit.`;
+            Please fix these issues and resubmit.
             
-            github.rest.issues.createComment({
-              issue_number: context.issue.number,
+            ---
+            *This comment is automatically updated by the Model Analysis workflow.*`;
+            
+            // Find existing comment
+            const { data: comments } = await github.rest.issues.listComments({
               owner: context.repo.owner,
               repo: context.repo.repo,
-              body: comment
+              issue_number: context.issue.number,
             });
+            
+            const botComment = comments.find(comment => 
+              comment.user.type === 'Bot' && 
+              comment.body.includes('Model Analysis Check')
+            );
+            
+            if (botComment) {
+              // Update existing comment
+              await github.rest.issues.updateComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: botComment.id,
+                body: commentBody
+              });
+              console.log('Updated existing comment');
+            } else {
+              // Create new comment
+              await github.rest.issues.createComment({
+                issue_number: context.issue.number,
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                body: commentBody
+              });
+              console.log('Created new comment');
+            }
     
       - name: Fail job if model analysis issues found
         if: steps.model_analysis.outputs.model_analysis_failed == 'true'

scripts/analyze_models.py

@@ -149,10 +149,24 @@ def find_markdown_links(obj):
     return result
 
 def analyze_markdown_links(result: Dict):
-    """Check if markdown safetensors links are consistent (text matches filename in URL)."""
+    """Check if markdown safetensors links are consistent (text matches filename in URL).
+    
+    Whitelisted URLs (skipped from validation):
+    - Civitai URLs: Use model IDs instead of filenames in paths
+    """
+    # Whitelist patterns for URLs that don't need filename validation
+    whitelist_patterns = [
+        r'civitai\.com',  # Civitai uses model IDs, not filenames
+    ]
+    
     for link in result['markdown_links']:
         text_name = link['text']
         url = link['url']
+        
+        # Skip validation for whitelisted URLs
+        if any(re.search(pattern, url, re.IGNORECASE) for pattern in whitelist_patterns):
+            continue
+        
         # Extract filename from URL path (ignore query string)
         m = re.search(r'/([^/?]+\.safetensors)(?:[?]|$)', url)
         if m: