Format release notes with github script

The previous function was throwing errors in some cases
when the gh cli expected a local .git directory.

Instead we should avoid an interactive CLI for steps like this.

Change-type: patch
Signed-off-by: Kyle Harding <[email protected]>

Author: klutchell

.github/workflows/flowzone.yml

@@ -1420,7 +1420,7 @@ jobs:
 
       # Get Renovate release notes from PR body, massage a little and draft a comment.
       - name: Format release notes
-        id: format_release_notes
+        id: format_release_notes_old
         continue-on-error: true
         env:
           # We need pull_requests: read to get the PR title and body but that permission
@@ -1496,6 +1496,81 @@ jobs:
               fi
           fi
 
+      # Get Renovate release notes from PR body, massage a little and draft a comment.
+      - name: Format release notes
+        id: format_release_notes
+        continue-on-error: true
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+        with:
+          github-token: ${{ steps.gh_app_token.outputs.token || secrets.FLOWZONE_TOKEN }}
+          result-encoding: string
+          script: |
+            const { data: pr } = await github.rest.pulls.get({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              pull_number: context.issue.number
+            });
+
+            if (!pr.title) {
+              return;
+            }
+
+            let releaseNotes = '';
+            let releaseNotesComment = '';
+
+            // Check if PR is from Renovate
+            const isRenovate = pr.user.type === 'Bot' && pr.user.login.includes('renovate');
+            const hasReleaseNotes = pr.body.includes('### Release Notes');
+
+            if (isRenovate && hasReleaseNotes) {
+              // Extract release notes section
+              const releaseNotesMatch = pr.body.match(/### Release Notes\n([\s\S]*?)---/);
+              if (releaseNotesMatch) {
+                const prBody = releaseNotesMatch[1].trim();
+
+                // Extract notable changes
+                const notableChanges = prBody
+                  .split('\n')
+                  .filter(line => line.match(/^> -|^-|^<summary>/))
+                  .map(line => line.replace(/^<summary>(.*)<\/summary>/, '-$1'))
+                  .filter((line, index, self) => self.indexOf(line) === index)
+                  .slice(1);
+
+                // Format release notes
+                const releaseNotesChanges = notableChanges
+                  .map(line => line.replace(/^> -(.*)/, '  -$1'))
+                  .join('\n');
+
+                releaseNotes = `## ${pr.title}\n\n### Notable changes\n\n${releaseNotesChanges}\n\n${prBody}`;
+                core.setOutput('body', releaseNotes);
+
+                // Format comment
+                const notableChangesFormatted = notableChanges
+                  .map(line => `* ${line.replace(/^> -|^-/, '')}`)
+                  .join('\n');
+
+                releaseNotesComment = `#release-notes ${pr.title}\n\nNotable changes\n* [only keep the important and rephrase, leaving this in place will avoid posting release notes]\n${notableChangesFormatted}\n\n${prBody}`;
+                core.setOutput('comment', releaseNotesComment);
+
+                return releaseNotes;
+              }
+            }
+
+            // Handle non-Renovate cases
+            if (pr.body.match(/## Release Notes/i)) {
+
+              const releaseNotesMatch = pr.body.match(/## Release Notes\n([\s\S]*?)(?=## |$)/i);
+              if (releaseNotesMatch) {
+                const releaseNotesBody = releaseNotesMatch[1].trim();
+
+                releaseNotes = `## ${pr.title}\n${releaseNotesBody}`;
+                core.setOutput('body', releaseNotes);
+
+                return releaseNotes;
+              }
+            }
+
+
       # https://octokit.github.io/rest.js/v21/#repos-list-tags
       # https://docs.github.com/en/rest/git/refs
       # https://octokit.github.io/rest.js/v21/#repos-compare-commits-with-basehead