feat: PR summary generation with categorized changes and titles

Author: chvvkumar

.github/workflows/arduino-compile.yml

@@ -714,7 +714,7 @@ jobs:
           
           # Build the prompt
           read -r -d '' PROMPT << 'EOF'
-          You are a code review assistant. Analyze the following pull request and provide a comprehensive summary.
+          You are a code review assistant. Analyze the following pull request and provide a categorized summary of changes.
 
           **IMPORTANT:** Focus ONLY on the actual code changes listed below. Do NOT mention or summarize any GitHub Actions workflow files (.github/workflows/) as those have been filtered out.
 
@@ -729,10 +729,7 @@ jobs:
           %FILES%
 
           ## Your Task
-          Provide a well-structured summary with these sections:
-
-          ### 📋 Overview
-          Write 2-3 sentences describing what this PR accomplishes overall.
+          Provide ONLY the following section in clean markdown format:
 
           ### 🔄 Changes by Category
           Group the changes into relevant categories such as:
@@ -745,20 +742,7 @@ jobs:
           - **Configuration**: Config file changes
 
           Only include categories that apply. Use bullet points for each change.
-
-          ### ⚠️ Notable Details
-          Highlight any:
-          - Breaking changes
-          - Security considerations
-          - Performance implications
-          - Migration requirements
-
-          If none, write "No notable concerns identified."
-
-          ### 👀 Suggested Review Focus
-          List 2-4 specific areas or files reviewers should pay close attention to.
-
-          Format everything in clean markdown.
+          Do NOT include any other sections, headers, or conclusions.
           EOF
           
           # Replace placeholders (remove jq quotes)
@@ -828,62 +812,35 @@ jobs:
               pull_number: context.issue.number,
             });
             
-            // Create enhanced PR body with AI summary
-            const marker = '<!-- pr-commit-summary-bot -->';
-            const summarySection = `${marker}
-            
-            ${summary}
+            // Create PR body with only Changes by Category section
+            const newBody = `${summary}
             
             ---
-            <sub>📊 Analyzed **${commitCount}** commit(s) | 🕐 Updated: ${timestamp} | Generated by GitHub Actions</sub>
+            <sub>📊 Analyzed **${commitCount}** commit(s) | 🕐 Updated: ${timestamp} | Generated by GitHub Actions</sub>`;
             
-            ---
-            `;
+            // Generate title with emoji prefix (replace existing)
+            const titleLower = pr.title.toLowerCase();
+            let emoji = '✨'; // Default: sparkles for feature
             
-            // Check if the PR body already has our summary marker
-            let newBody;
-            if (pr.body && pr.body.includes(marker)) {
-              // Replace existing summary section
-              const markerIndex = pr.body.indexOf(marker);
-              const endMarkerIndex = pr.body.indexOf('---\n\n', markerIndex + summarySection.length);
-              
-              if (endMarkerIndex !== -1) {
-                // Replace everything from marker to the end of summary section
-                newBody = pr.body.substring(0, markerIndex) + summarySection + pr.body.substring(endMarkerIndex + 5);
-              } else {
-                // Fallback: append new summary
-                newBody = summarySection + (pr.body || '');
-              }
-            } else {
-              // Prepend summary to existing body
-              newBody = summarySection + (pr.body || '');
+            if (titleLower.includes('fix') || titleLower.includes('bug')) {
+              emoji = '🐛';
+            } else if (titleLower.includes('doc')) {
+              emoji = '📚';
+            } else if (titleLower.includes('refactor') || titleLower.includes('cleanup')) {
+              emoji = '♻️';
+            } else if (titleLower.includes('test')) {
+              emoji = '✅';
+            } else if (titleLower.includes('config') || titleLower.includes('setting')) {
+              emoji = '⚙️';
+            } else if (titleLower.includes('security')) {
+              emoji = '🔒';
+            } else if (titleLower.includes('performance') || titleLower.includes('optim')) {
+              emoji = '⚡';
             }
             
-            // Generate enhanced title with emoji prefix if not already present
-            let newTitle = pr.title;
-            if (!pr.title.match(/^[\u{1F300}-\u{1F9FF}]/u)) {
-              // Add a relevant emoji based on keywords in the title
-              const titleLower = pr.title.toLowerCase();
-              let emoji = '✨'; // Default: sparkles for feature
-              
-              if (titleLower.includes('fix') || titleLower.includes('bug')) {
-                emoji = '🐛';
-              } else if (titleLower.includes('doc')) {
-                emoji = '📚';
-              } else if (titleLower.includes('refactor') || titleLower.includes('cleanup')) {
-                emoji = '♻️';
-              } else if (titleLower.includes('test')) {
-                emoji = '✅';
-              } else if (titleLower.includes('config') || titleLower.includes('setting')) {
-                emoji = '⚙️';
-              } else if (titleLower.includes('security')) {
-                emoji = '🔒';
-              } else if (titleLower.includes('performance') || titleLower.includes('optim')) {
-                emoji = '⚡';
-              }
-              
-              newTitle = `${emoji} ${pr.title}`;
-            }
+            // Remove any existing emoji prefix and add new one
+            const cleanTitle = pr.title.replace(/^[\u{1F300}-\u{1F9FF}]\s*/u, '');
+            const newTitle = `${emoji} ${cleanTitle}`;
             
             // Update the PR
             await github.rest.pulls.update({