Fix all pre-commit issues: YAML syntax and formatting

- Simplified Python script to single-line to avoid YAML parsing issues
- Fixed trailing whitespace in workflow files
- All pre-commit checks now passing locally

Signed-off-by: Dan Gil <[email protected]>

Author: dagil-nvidia

.github/workflows/dependency-extraction-nightly.yml

@@ -28,82 +28,64 @@ permissions:
 jobs:
   extract-dependencies:
     runs-on: ubuntu-latest
-    
+
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
           fetch-depth: 0 # Need history for comparison
-      
+
       - name: Set up Python
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
-      
+
       - name: Install dependencies
         run: pip install pyyaml
-      
+
       - name: Run dependency extraction
         run: |
           TIMESTAMP=$(date +%Y%m%d_%H%M)
-          
+
           # Generate timestamped version (for artifacts)
           python3 .github/workflows/extract_dependency_versions.py \
             --output .github/reports/dependency_versions_${TIMESTAMP}.csv \
             --report-unversioned \
             --report-removed .github/reports/removed_dependencies.json
-          
+
           # Copy to latest version (for repo tracking)
           mkdir -p .github/reports
           cp .github/reports/dependency_versions_${TIMESTAMP}.csv .github/reports/dependency_versions_latest.csv
-          
+
           # Copy unversioned report if it exists
           if [ -f "unversioned_dependencies_${TIMESTAMP}.csv" ]; then
             cp unversioned_dependencies_${TIMESTAMP}.csv .github/reports/unversioned_dependencies_latest.csv
           fi
-          
+
           echo "TIMESTAMP=${TIMESTAMP}" >> $GITHUB_ENV
-      
+
       - name: Check for changes
         id: check_changes
         run: |
           if [[ -n $(git status --porcelain .github/reports/*_latest.csv) ]]; then
             echo "has_changes=true" >> $GITHUB_OUTPUT
-            
+
             # Count dependencies by status from latest
             new_count=$(grep -c ",New," .github/reports/dependency_versions_latest.csv 2>/dev/null || echo "0")
             changed_count=$(grep -c ",Changed," .github/reports/dependency_versions_latest.csv 2>/dev/null || echo "0")
             unchanged_count=$(grep -c ",Unchanged," .github/reports/dependency_versions_latest.csv 2>/dev/null || echo "0")
-            
+
             # Parse removed dependencies from JSON
             if [ -f ".github/reports/removed_dependencies.json" ]; then
               removed_count=$(python3 -c "import json; print(json.load(open('.github/reports/removed_dependencies.json'))['count'])" 2>/dev/null || echo "0")
-              
-              # Format removed dependencies list for PR body (limit to first 10)
-              removed_list=$(python3 << 'PYTHON_SCRIPT'
-import json
-try:
-    data = json.load(open('.github/reports/removed_dependencies.json'))
-    removed = data['removed'][:10]
-    lines = []
-    for dep in removed:
-        critical = ' **[CRITICAL]**' if dep.get('Critical') == 'Yes' else ''
-        lines.append(f"    • **{dep['Dependency Name']}** (was: `{dep['Version']}`){critical}")
-        lines.append(f"      _from {dep['Source File']}_")
-    
-    if data['count'] > 10:
-        lines.append(f"    _... and {data['count'] - 10} more (see CSV for full list)_")
-    
-    print('\n'.join(lines))
-except:
-    print('    _No removed dependencies_')
-PYTHON_SCRIPT
-)
+
+              # Simple formatting - just list names and versions
+              removed_list=$(python3 -c "import json; data=json.load(open('.github/reports/removed_dependencies.json')); removed=data['removed'][:10]; lines=[]; [lines.extend([f\"    • **{d['Dependency Name']}** (was: \\\`{d['Version']}\\\`){' **[CRITICAL]**' if d.get('Critical')=='Yes' else ''}\", f\"      _from {d['Source File']}_\"]) for d in removed]; (lines.append(f\"    _... and {data['count']-10} more_\") if data['count']>10 else None); print('\\n'.join(lines))" 2>/dev/null || echo "    _Unable to parse removed dependencies_")
             else
               removed_count="0"
               removed_list="    _No removed dependencies_"
             fi
-            
+
             echo "new_deps=$new_count" >> $GITHUB_OUTPUT
             echo "changed_deps=$changed_count" >> $GITHUB_OUTPUT
             echo "unchanged_deps=$unchanged_count" >> $GITHUB_OUTPUT
@@ -114,7 +96,7 @@ PYTHON_SCRIPT
           else
             echo "has_changes=false" >> $GITHUB_OUTPUT
           fi
-      
+
       - name: Create Pull Request
         if: steps.check_changes.outputs.has_changes == 'true'
         uses: peter-evans/create-pull-request@v6
@@ -124,36 +106,36 @@ PYTHON_SCRIPT
           title: '[Automated] Nightly Dependency Version Update - $(date +%Y-%m-%d)'
           body: |
             ## 🤖 Automated Dependency Version Update
-            
+
             This PR contains the nightly dependency extraction results.
-            
+
             ### 📊 Summary
             - **New Dependencies:** ${{ steps.check_changes.outputs.new_deps }}
             - **Changed Versions:** ${{ steps.check_changes.outputs.changed_deps }}
             - **Removed Dependencies:** ${{ steps.check_changes.outputs.removed_deps }}
             - **Unchanged:** ${{ steps.check_changes.outputs.unchanged_deps }}
-            
+
             ### 🗑️ Removed Dependencies
             ${{ steps.check_changes.outputs.removed_list }}
-            
+
             ### 📋 Files Updated
             - ✅ `.github/reports/dependency_versions_latest.csv` - Latest dependency snapshot
             - ✅ `.github/reports/unversioned_dependencies_latest.csv` - Unversioned deps report (if applicable)
-            
+
             > **Note:** Timestamped versions are stored in GitHub Artifacts (90-day retention) to avoid repo clutter.
-            
+
             ### ✔️ Review Checklist
             - [ ] Review new dependencies for security/licensing concerns
             - [ ] Check version changes for breaking updates
             - [ ] Review removed dependencies (intentional?)
             - [ ] Verify unversioned dependencies report
             - [ ] Update baseline count if increase is expected
-            
+
             ---
-            
+
             🔗 **Documentation:** [Dependency Extraction Guide](../docs/dependency_extraction.md)
             📦 **Artifacts:** Download timestamped CSVs from workflow run
-            
+
             _Generated by nightly dependency extraction workflow_
             _Timestamp: ${{ env.TIMESTAMP }}_
           branch: automated/dependency-extraction-${{ github.run_number }}
@@ -162,7 +144,7 @@ PYTHON_SCRIPT
             automated
             dependencies
             documentation
-      
+
       - name: Upload artifacts
         if: always()
         uses: actions/upload-artifact@v4
@@ -172,7 +154,7 @@ PYTHON_SCRIPT
             .github/reports/dependency_versions_*.csv
             .github/reports/unversioned_dependencies_*.csv
           retention-days: 90
-      
+
       - name: Summary
         if: always()
         run: |

.github/workflows/dependency-extraction-release.yml

@@ -33,21 +33,21 @@ permissions:
 jobs:
   snapshot-dependencies:
     runs-on: ubuntu-latest
-    
+
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
           fetch-depth: 0
-      
+
       - name: Set up Python
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
-      
+
       - name: Install dependencies
         run: pip install pyyaml
-      
+
       - name: Extract version from branch or input
         id: version
         run: |
@@ -57,32 +57,32 @@ jobs:
             # Extract from branch name: release/1.2.3 -> 1.2.3
             VERSION=$(echo "${{ github.ref_name }}" | sed 's/release\///')
           fi
-          
+
           # Validate version format (X.Y.Z)
           if [[ ! $VERSION =~ ^[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
             echo "Error: Invalid version format '$VERSION'. Expected X.Y.Z"
             exit 1
           fi
-          
+
           echo "version=$VERSION" >> $GITHUB_OUTPUT
           echo "📦 Creating dependency snapshot for version: v$VERSION"
-      
+
       - name: Run dependency extraction
         run: |
           VERSION="${{ steps.version.outputs.version }}"
-          
+
           # Create versioned snapshot
           mkdir -p .github/reports/releases
           python3 .github/workflows/extract_dependency_versions.py \
             --output .github/reports/releases/dependency_versions_v${VERSION}.csv
-          
+
           echo "VERSION=${VERSION}" >> $GITHUB_ENV
-      
+
       - name: Check if snapshot already exists
         id: check_exists
         run: |
           VERSION="${{ steps.version.outputs.version }}"
-          
+
           # Check if this version snapshot already exists in git
           if git ls-files --error-unmatch ".github/reports/releases/dependency_versions_v${VERSION}.csv" 2>/dev/null; then
             echo "exists=true" >> $GITHUB_OUTPUT
@@ -91,7 +91,7 @@ jobs:
             echo "exists=false" >> $GITHUB_OUTPUT
             echo "✅ Creating new snapshot for v${VERSION}"
           fi
-      
+
       - name: Create Pull Request
         if: steps.check_exists.outputs.exists == 'false'
         uses: peter-evans/create-pull-request@v6
@@ -101,53 +101,53 @@ jobs:
           title: '[Release] Dependency Snapshot v${{ steps.version.outputs.version }}'
           body: |
             ## 📸 Release Dependency Snapshot
-            
+
             This PR adds a permanent dependency snapshot for **release v${{ steps.version.outputs.version }}**.
-            
+
             ### 📋 Files Added
             - `.github/reports/releases/dependency_versions_v${{ steps.version.outputs.version }}.csv`
-            
+
             ### 📊 Purpose
             This snapshot captures the exact dependency versions used in this release for:
             - 🔍 Historical tracking and auditing
             - 🐛 Debugging version-specific issues
             - 📈 Comparing dependency evolution across releases
             - 🔒 Compliance and security reviews
-            
+
             ### ✔️ Review Checklist
             - [ ] Verify this is the correct release version
             - [ ] Check that snapshot doesn't already exist
             - [ ] Review any new or changed dependencies
-            
+
             ---
-            
+
             🔗 **Release Branch:** `${{ github.ref_name }}`
             📦 **Version:** v${{ steps.version.outputs.version }}
-            
+
             _Generated by release dependency snapshot workflow_
           branch: release-snapshot/v${{ steps.version.outputs.version }}
           delete-branch: true
           labels: |
             release
             dependencies
             documentation
-      
+
       - name: Upload snapshot artifact
         if: always()
         uses: actions/upload-artifact@v4
         with:
           name: dependency-snapshot-v${{ steps.version.outputs.version }}
           path: .github/reports/releases/dependency_versions_v${{ steps.version.outputs.version }}.csv
           retention-days: 365  # Keep release snapshots for 1 year
-      
+
       - name: Summary
         if: always()
         run: |
           VERSION="${{ steps.version.outputs.version }}"
-          
+
           echo "## Release Dependency Snapshot" >> $GITHUB_STEP_SUMMARY
           echo "" >> $GITHUB_STEP_SUMMARY
-          
+
           if [[ "${{ steps.check_exists.outputs.exists }}" == "true" ]]; then
             echo "ℹ️ **Snapshot Already Exists**" >> $GITHUB_STEP_SUMMARY
             echo "" >> $GITHUB_STEP_SUMMARY