Merge pull request #6 from daniel-montalvo/accname-core-aam

Try preview link updates

Author: daniel-montalvo

.github/workflows/update-preview-links.yml

@@ -17,27 +17,17 @@ jobs:
       - name: Set up Node.js
         uses: actions/setup-node@v3
         with:
-          node-version: '16'
+          node-version: 'latest'
 
       # Step 3: Install dependencies
       - name: Install dependencies
         run: npm install axios yargs
 
-      # Step 4: Run the script
-      - name: Generate preview links
+      # Step 4: Run the script and update PR
+      - name: Generate preview links and update PR
         run: |
-          node common/script/updatePreviewLinks.js \
+          node common/script/updatePreviewLinks.mjs \
             --repo ${{ github.repository }} \
             --pull_request_number ${{ github.event.pull_request.number }} \
-            --token ${{ secrets.GITHUB_TOKEN }}
-
-      # Step 5: Update the pull request body
-      - name: Update pull request body
-        uses: peter-evans/create-or-update-comment@v4
-        with:
-          issue-number: ${{ github.event.pull_request.number }}
-          body: |
-            🚀 **Netlify Preview**:
-            🔄 **Changed Pages**:
-            $(cat .github/scripts/changed_files.md)
-          edit-mode: replace
+            --token ${{ secrets.GITHUB_TOKEN }} \
+            --update_pr

common/script/updatePreviewLinks.js

@@ -0,0 +1,151 @@
+import axios from "axios";
+import fs from "fs";
+import path from "path";
+import yargs from "yargs";
+import { hideBin } from "yargs/helpers";
+import { fileURLToPath } from "url";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+// Parse command-line arguments
+const args = yargs(hideBin(process.argv))
+  .option("repo", {
+    alias: "r",
+    type: "string",
+    description: "GitHub repository in the format owner/repo",
+    demandOption: true,
+  })
+  .option("pull_request_number", {
+    alias: "pr",
+    type: "number",
+    description: "Pull request number",
+    demandOption: true,
+  })
+  .option("token", {
+    alias: "t",
+    type: "string",
+    description: "GitHub personal access token",
+    demandOption: true,
+  })
+  .option("update_pr", {
+    alias: "u",
+    type: "boolean",
+    description: "Update the PR description with preview links",
+    default: false,
+  })
+  .help().argv;
+
+const { repo, pull_request_number, token, update_pr } = args;
+
+async function updatePRDescription(markdownContent) {
+  if (!update_pr) return;
+  
+  try {
+    // Get current PR description
+    const prResponse = await axios.get(`https://api.github.com/repos/${repo}/pulls/${pull_request_number}`, {
+      headers: {
+        Authorization: `token ${token}`,
+        Accept: 'application/vnd.github.v3+json'
+      }
+    });
+    
+    const currentBody = prResponse.data.body || '';
+    
+    // Remove any existing preview section
+    const cleanedBody = currentBody.replace(/🚀 \*\*Netlify Preview\*\*:[\s\S]*?(?=\n\n|\n$|$)/g, '').trim();
+    
+    // Create new body with preview links prepended
+    const newBody = `🚀 **Netlify Preview**:
+🔄 **Changed Pages**:
+${markdownContent}
+
+${cleanedBody}`.trim();
+    
+    // Update PR description
+    await axios.patch(`https://api.github.com/repos/${repo}/pulls/${pull_request_number}`, {
+      body: newBody
+    }, {
+      headers: {
+        Authorization: `token ${token}`,
+        Accept: 'application/vnd.github.v3+json'
+      }
+    });
+    
+    console.log('PR description updated successfully');
+  } catch (error) {
+    console.error('Error updating PR description:', error.message);
+  }
+}
+
+// Define the base URLs
+const previewBaseURL = `https://deploy-preview--${pull_request_number}--wai-aria-.netlify.app`;
+const EDBaseURL = `https://w3c.github.io`;
+
+async function getChangedFiles() {
+  try {
+    // Build headers conditionally - only include Authorization if token is provided
+    const headers = {};
+    if (token) {
+      headers.Authorization = `token ${token}`;
+    }
+
+    // Fetch the list of changed files from the GitHub API
+    const response = await axios.get(`https://api.github.com/repos/${repo}/pulls/${pull_request_number}/files`, {
+      headers,
+    });
+
+    const files = response.data.map((file) => file.filename);
+
+    // Filter to only include index.html files
+    const specSources = files.filter(file => 
+      file === 'index.html' || file.endsWith('/index.html')
+    );
+
+    // Build the Markdown list with preview URLs and diff links
+    const markdownList = specSources.map((file) => {
+      const previewUrl = `${previewBaseURL}/${file}`;
+      
+      // Build ED URL based on file path
+      let EDUrl;
+      if (file === 'index.html') {
+        EDUrl = `${EDBaseURL}/aria/`;
+      } else if (file.endsWith('/index.html')) {
+        // Extract directory name for subdirectory index.html files
+        const dirName = file.split('/').slice(-2, -1)[0];
+        EDUrl = `${EDBaseURL}/${dirName}/`;
+      }
+      
+      const diffUrl = `https://services.w3.org/htmldiff?doc1=${encodeURIComponent(EDUrl)}&doc2=${encodeURIComponent(previewUrl)}`;
+      
+      // Generate spec name based on file path
+      let specName;
+      if (file === 'index.html') {
+        specName = 'ARIA';
+      } else if (file.endsWith('/index.html')) {
+        // Extract directory name for subdirectory index.html files
+        const dirName = file.split('/').slice(-2, -1)[0];
+        specName = dirName;
+      }
+      
+      return `- [${specName} preview](${previewUrl}) ([diff](${diffUrl}))`;
+    }).join("\n");
+
+    // Output the Markdown list
+    console.log(markdownList);
+
+    // Optionally, write the output to a file
+    const outputPath = path.join(__dirname, "changed_files.md");
+    fs.writeFileSync(outputPath, markdownList, "utf8");
+    console.log(`Markdown list written to ${outputPath}`);
+    
+    // Update PR description if requested
+    await updatePRDescription(markdownList);
+  } catch (error) {
+    console.error("Error fetching changed files:", error.message);
+    process.exit(1);
+  }
+}
+
+// Run the script
+getChangedFiles();