test

Author: norareidy

.github/workflows/add-redirects.yml

@@ -1,63 +1,143 @@
-name: Add Redirects
+name: Verify Redirects
+
 on:
   pull_request_target:
+
 jobs:
-  add-redirects:
-    name: Get Redirects for Modified Files
+  verify-redirects:
+    name: Verifying Redirects
     runs-on: ubuntu-latest
+    env:
+      REDIRECTS_FILE: "pr/config/redirects"
     permissions:
       pull-requests: write
     steps:
-      - uses: actions/checkout@v4
+      - name: Check Out Base Branch
+        uses: actions/checkout@v4
+      
+      - name: Checkout PR Head Branch  
+        uses: actions/checkout@v4  
+        with:  
+          token: ${{ secrets.GITHUB_TOKEN }}  
+          ref: ${{ github.event.pull_request.head.ref }}  
+          repository: ${{ github.event.pull_request.head.repo.full_name }}
+          path: pr
+      
       - name: Get Changed Files
         id: changed-files
         uses: tj-actions/changed-files@c65cd883420fd2eb864698a825fc4162dd94482c
         with:
           include_all_old_new_renamed_files: true
-      - name: Make Redirects for Renamed Files
+
+      - name: Find Missing Redirects for Renamed Files
         id: renamed-files
         if: steps.changed-files.outputs.renamed_files_count > 0
         env:
           RENAMED_FILES: ${{ steps.changed-files.outputs.all_old_new_renamed_files }}
         run: |
           renamed_redirects=""
           for file in $RENAMED_FILES; do
+
+            # only run for .txt files
             if [[ ! "$file" == *.txt ]]; then
               continue
             fi
+
+            # format old and new URLs
             old=$(echo "$file" | cut -d',' -f1)
             old="${old#source}"
             old="${old%.txt}"
             new=$(echo "$file" | cut -d',' -f2)
             new="${new#source}"
             new="${new%.txt}"
-            #single quotes = ${var} rendered literally; double quotes = $var interpreted
-            renamed_redirects+='<li>[&lt;v&gt;-*]: ${prefix}/${version}'"$old"'/ -> ${base}/${version}'"$new"'/</li>'
+            
+            # single quotes = ${var} rendered literally; double quotes = $var interpreted
+            redirect='${prefix}/${version}'"$old"'/ -> ${base}/${version}'"$new"'/'
+
+            # if redirect not already in file, add to string to add to PR description
+            if ! grep -q "$redirect" $REDIRECTS_FILE; then
+              renamed_redirects+="$redirect,"
+            fi
           done
-          echo "redirects=${renamed_redirects}" >> "$GITHUB_OUTPUT"
-      - name: Make Redirects for Deleted Files
+
+          # cut trailing comma
+          renamed_redirects="${renamed_redirects%,}"
+
+          echo "redirects=${renamed_redirects}" >> "$GITHUB_OUTPUT"  
+
+      - name: Find Missing Redirects for Deleted Files
         id: deleted-files
         if: steps.changed-files.outputs.deleted_files_count > 0
         env:
           DELETED_FILES: ${{ steps.changed-files.outputs.deleted_files }}
         run: |
           deleted_redirects=""
           for file in $DELETED_FILES; do
+
+            # only run for .txt files
             if [[ ! "$file" == *.txt ]]; then
               continue
             fi
+            
+            # format old URL
             old=$(echo "$file" | cut -d',' -f1)
             old="${old#source}"
             old="${old%.txt}"
-            deleted_redirects+='<li>[&lt;v&gt;-*]: ${prefix}/${version}'"$old"'/ -> ${base}/${version}/</li>'
+            
+            # single quotes = ${var} rendered literally; double quotes = $var interpreted
+            redirect='${prefix}/${version}'"$old"'/ -> ${base}/${version}/'
+                       
+            # if redirect not already in file, add to string to add to PR description
+            if ! grep -q "$redirect" $REDIRECTS_FILE; then 
+              deleted_redirects+="$redirect,"
+            fi
           done
-          echo "redirects=${deleted_redirects}" >> "$GITHUB_OUTPUT"
+          
+          # cut trailing comma
+          deleted_redirects="${deleted_redirects%,}"
+
+          echo "redirects=${deleted_redirects}" >> "$GITHUB_OUTPUT"  
+
+      - name: Build Redirect HTML
+        id: build-redirect-html
+        env:
+          RENAMED_REDIRECTS: ${{ steps.renamed-files.outputs.redirects }}
+          DELETED_REDIRECTS: ${{ steps.deleted-files.outputs.redirects }}
+        run: |
+          # empty string that stays empty if there are no redirects to suggest
+          redirect_html=""
+          
+          combined_redirect_string="${RENAMED_REDIRECTS},${DELETED_REDIRECTS}"
+
+          for redirect in $combined_redirect_string; do
+            if [[ ! -z "$redirect" ]]; then
+              if [[ redirect_html="" ]]; then
+                redirect_html="*Suggested redirects for moved/deleted files (replace &lt;v&gt; with earliest backport target version):*"
+              fi
+              redirect_html+="<li>[&lt;v&gt;-*]: $redirect</li>"  
+            fi  
+          done
+          
+          echo "redirect_html=${redirect_html}" >> "$GITHUB_OUTPUT"
+
       - name: Update PR Description
-        if: steps.changed-files.outputs.renamed_files_count > 0 || steps.changed-files.outputs.deleted_files_count > 0
         uses: MongoCaleb/pr-description-action@4bdfe35b98f64532b419ad20b350a92546cd3aa1
         with:
           regex: "- \\[ \\] Did you add redirects\\?.*"
           appendContentOnMatchOnly: false
           regexFlags: is
-          content: "- [ ] Did you add redirects?\n *Suggested redirects for moved/deleted files (replace &lt;v&gt; with earliest backport target version):*\n${{ steps.renamed-files.outputs.redirects }}${{ steps.deleted-files.outputs.redirects }}\n"
-          token: ${{ secrets.GITHUB_TOKEN }}
+          content: "- [ ] Did you add redirects?\n ${{ steps.build-redirect-html.outputs.redirect_html }}"
+          token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Check for duplicates in redirects file
+        run: |
+          if [[ -f "$REDIRECTS_FILE" ]]; then  
+            duplicates=$(sort "$REDIRECTS_FILE" | uniq -d)  
+            if [[ -n "$duplicates" ]]; then  
+              echo "Duplicate lines found in $REDIRECTS_FILE:" 
+              echo "$duplicates"  
+              exit 1 # error
+            fi
+          else  
+            echo "Redirects file doesn't exist. Skipping duplicate check."  
+          fi