Update update-contributors.yml

Author: RiddleTime

.github/workflows/update-contributors.yml

@@ -25,45 +25,48 @@ jobs:
           echo "{}" > contributors.json
           commit_count=0
           # Switch to dev branch
-          git checkout dev
+          git checkout dev || { echo "Failed to checkout dev branch"; exit 1; }
           # Get all commits in dev branch
-          commits=$(git log --pretty=format:%H)
-          echo "Total commits to process: $(echo "$commits" | wc -l)"
+          commits=$(git log --pretty=format:%H || { echo "Failed to run git log"; exit 1; })
+          total_commits=$(echo "$commits" | wc -l | tr -d ' ')
+          echo "Total commits to process: $total_commits"
           for commit in $commits; do
             ((commit_count++))
-            echo "Processing commit $commit ($commit_count)"
-            author_name=$(git show -s --format=%an $commit)
+            echo "Processing commit $commit ($commit_count/$total_commits)"
+            author_name=$(git show -s --format=%an "$commit" 2>/dev/null || echo "")
             if [ -z "$author_name" ]; then
               echo "Skipping commit $commit with no author name"
               continue
             fi
-            # Map author name to GitHub username using author_to_github.json
-            login=$(jq -r --arg name "$author_name" '.[$name] // $name' author_to_github.json)
-            if [ "$login" = "$author_name" ]; then
-              echo "Warning: No GitHub username mapping for author '$author_name' in commit $commit"
-            else
-              echo "Mapped author '$author_name' to GitHub username '$login'"
+            # Use author name as GitHub username, sanitize for URL
+            login=$(echo "$author_name" | tr -s ' ' '-' | tr -dc '[:alnum:]-')
+            if echo "$author_name" | grep -q ' '; then
+              echo "Warning: Author '$author_name' may not be a valid GitHub username (contains spaces, sanitized to '$login')"
             fi
-            stats=$(git show --shortstat --format= $commit | grep -E '[0-9]+ file(s)? changed' | awk '{add=$4; del=$6} END {print (add ? add : 0) - (del ? del : 0)}')
+            echo "Processing commit $commit by $login"
+            # Get commit stats
+            stats=$(git show --shortstat --format= "$commit" 2>/dev/null | grep -E '[0-9]+ file(s)? changed' | awk '{add=$4; del=$6} END {print (add ? add : 0) - (del ? del : 0)}' || echo "0")
             net_lines=${stats:-0}
             echo "Commit $commit by $login: $net_lines net lines"
             # Update JSON with contributor stats
             jq --arg login "$login" --arg net_lines "$net_lines" \
                '.[$login] |= (. // {netLines: 0, commits: 0, login: $login} | .netLines += ($net_lines | tonumber) | .commits += 1)' \
-               contributors.json > tmp.json && mv tmp.json contributors.json
+               contributors.json > tmp.json 2>/dev/null || { echo "Error: jq failed to update stats for $login in commit $commit"; exit 1; }
+            mv tmp.json contributors.json || { echo "Error: Failed to move tmp.json"; exit 1; }
           done
           echo "Total commits processed: $commit_count"
           # Format table
-          table=$(jq -r 'to_entries | sort_by(.value.netLines) | reverse | .[] | select(.value.netLines > 0 or .value.commits > 0) | "| [\(.key)](https://github.com/\(.key)) | \(.value.netLines) | \(.value.commits) |"' contributors.json)
+          table=$(jq -r 'to_entries | sort_by(.value.netLines) | reverse | .[] | select(.value.netLines > 0 or .value.commits > 0) | "| [\(.key)](https://github.com/\(.key)) | \(.value.netLines) | \(.value.commits) |"' contributors.json 2>/dev/null || { echo "Error: jq failed to format table"; exit 1; })
           if [ -z "$table" ]; then
             table="No contributors with commits or net lines added."
+            echo "Warning: No valid contributors found"
           else
             table="| Contributor | Net Lines Added | Commits |\n|-------------|-----------------|---------|\n$table"
           fi
-          echo "Total contributors: $(jq 'length' contributors.json)"
+          echo "Total contributors: $(jq 'length' contributors.json 2>/dev/null || echo "Error: jq failed to count contributors")"
           echo "Generated table:"
           echo "$table"
-          echo "$table" > contributors.txt
+          echo "$table" > contributors.txt || { echo "Error: Failed to write contributors.txt"; exit 1; }
         shell: bash
 
       - name: Update README