fix(ci): safely sync issue metadata to PR and fix JSON parsing

- Stop direct interpolation of workflow outputs into github-script to avoid malformed JSON errors.
- Pass issues and PR number via environment variables and parse from process.env.
- Add defensive JSON parsing with cleaning for extra quotes/newlines.
- Fix typo in PR comment: use issue.milestone correctly.
- Use addLabels() instead of setLabels() to prevent race conditions and preserve existing PR labels.
- Sync milestones and add a comment summarizing the synchronized metadata.

Author: sheikhlimon

.github/workflows/autolabel-pr-issue.yml

@@ -30,6 +30,7 @@ jobs:
             const prTitle = context.payload.pull_request.title || '';
             const prBody = context.payload.pull_request.body || '';
 
+            // Regex patterns to find linked issues
             const patterns = [
               /(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)\s+#(\d+)/gi,
               /#(\d+)/g
@@ -51,21 +52,33 @@ jobs:
       - name: Sync Issue Metadata to PR
         if: steps.extract-issues.outputs.issues != '' && steps.extract-issues.outputs.issues != '[]'
         uses: actions/github-script@v7
+        env:
+          ISSUES_JSON: ${{ steps.extract-issues.outputs.issues }}
+          PR_NUMBER: ${{ steps.extract-issues.outputs.pr }}
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           script: |
-            const issuesOutput = '${{ steps.extract-issues.outputs.issues }}';
-            const prNumber = parseInt('${{ steps.extract-issues.outputs.pr }}');
+            // Read inputs from environment variables to avoid YAML quoting issues
+            const issuesOutput = process.env.ISSUES_JSON || '[]';
+            const prNumber = parseInt(process.env.PR_NUMBER, 10);
 
             let issues = [];
             try {
               issues = JSON.parse(issuesOutput);
             } catch (e) {
-              console.error("Failed to parse issues:", issuesOutput);
+              // Clean extra quotes/newlines
+              const cleaned = issuesOutput.replace(/^\s*['"]?/, '').replace(/['"]?\s*$/, '');
+              try {
+                issues = JSON.parse(cleaned);
+              } catch (err) {
+                console.error('Failed to parse issues output:', err.message);
+                console.error('Raw issues output:', JSON.stringify(issuesOutput));
+                return;
+              }
             }
 
-            if (!issues || issues.length === 0) {
-              console.log("No linked issues found");
+            if (!Array.isArray(issues) || issues.length === 0) {
+              console.log('No linked issues found');
               return;
             }
 
@@ -75,13 +88,13 @@ jobs:
                 const { data: issue } = await github.rest.issues.get({
                   owner: context.repo.owner,
                   repo: context.repo.repo,
-                  issue_number: parseInt(issueNumber)
+                  issue_number: parseInt(issueNumber, 10)
                 });
 
                 console.log(`Syncing metadata from Issue #${issueNumber} to PR #${prNumber}`);
 
                 // --- Sync Labels safely using addLabels ---
-                const issueLabels = issue.labels.map(l => l.name);
+                const issueLabels = Array.isArray(issue.labels) ? issue.labels.map(l => l.name) : [];
                 if (issueLabels.length > 0) {
                   await github.rest.issues.addLabels({
                     owner: context.repo.owner,
@@ -112,6 +125,6 @@ jobs:
                 });
 
               } catch (error) {
-                console.error(`Error syncing issue #${issueNumber} to PR #${prNumber}:`, error.message);
+                console.error(`Error syncing issue #${issueNumber} to PR #${prNumber}:`, error);
               }
             }