fix: reduce cyclomatic complexity by extracting helper functions

Signed-off-by: Mounil <[email protected]>

Author: Mounil2005

.github/scripts/coderabbit_plan_trigger.js

@@ -26,6 +26,44 @@ async function triggerCodeRabbitPlan(github, owner, repo, issue, marker, dryRun)
   }
 }
 
+function hasIntermediateOrAdvancedLabel(issue, label) {
+  // Check if issue has intermediate or advanced label (case-insensitive)
+  const hasIntermediateLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'intermediate');
+  const hasAdvancedLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'advanced');
+  
+  // Also check if the newly added label is intermediate/advanced
+  const isNewLabelIntermediate = label?.name?.toLowerCase() === 'intermediate';
+  const isNewLabelAdvanced = label?.name?.toLowerCase() === 'advanced';
+  
+  return hasIntermediateLabel || hasAdvancedLabel || isNewLabelIntermediate || isNewLabelAdvanced;
+}
+
+async function hasExistingCodeRabbitPlan(github, owner, repo, issueNumber) {
+  // Check for existing CodeRabbit plan comment (limited to first 500 comments)
+  const comments = [];
+  const iterator = github.paginate.iterator(github.rest.issues.listComments, {
+    owner, repo, issue_number: issueNumber, per_page: 100
+  });
+  
+  let count = 0;
+  for await (const { data: page } of iterator) {
+    comments.push(...page);
+    count += page.length;
+    if (count >= 500) break; // Hard upper bound to prevent unbounded pagination
+  }
+  
+  return comments.some(c => c.body?.includes('@coderabbitai plan'));
+}
+
+function logSummary(owner, repo, issue, dryRun) {
+  console.log('=== Summary ===');
+  console.log(`Repository: ${owner}/${repo}`);
+  console.log(`Issue Number: ${issue.number}`);
+  console.log(`Issue Title: ${issue.title || '(no title)'}`);
+  console.log(`Labels: ${issue.labels?.map(l => l.name).join(', ') || 'none'}`);
+  console.log(`Dry Run: ${dryRun}`);
+}
+
 module.exports = async ({ github, context }) => {
   try {
     const { owner, repo } = context.repo;
@@ -35,44 +73,18 @@ module.exports = async ({ github, context }) => {
     // Validations
     if (!issue?.number) return console.log('No issue in payload');
 
-    // Check if issue has intermediate or advanced label (case-insensitive)
-    const hasIntermediateLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'intermediate');
-    const hasAdvancedLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'advanced');
-    
-    // Also check if the newly added label is intermediate/advanced
-    const isNewLabelIntermediate = label?.name?.toLowerCase() === 'intermediate';
-    const isNewLabelAdvanced = label?.name?.toLowerCase() === 'advanced';
-    
-    if (!hasIntermediateLabel && !hasAdvancedLabel && !isNewLabelIntermediate && !isNewLabelAdvanced) {
+    if (!hasIntermediateOrAdvancedLabel(issue, label)) {
       return console.log('Issue does not have intermediate or advanced label');
     }
 
-    // Check for existing CodeRabbit plan comment (limited to first 500 comments)
-    const comments = [];
-    const iterator = github.paginate.iterator(github.rest.issues.listComments, {
-      owner, repo, issue_number: issue.number, per_page: 100
-    });
-    
-    let count = 0;
-    for await (const { data: page } of iterator) {
-      comments.push(...page);
-      count += page.length;
-      if (count >= 500) break; // Hard upper bound to prevent unbounded pagination
-    }
-    
-    if (comments.some(c => c.body?.includes('@coderabbitai plan'))) {
+    if (await hasExistingCodeRabbitPlan(github, owner, repo, issue.number)) {
       return console.log(`CodeRabbit plan already triggered for #${issue.number}`);
     }
 
     // Post CodeRabbit plan trigger
     await triggerCodeRabbitPlan(github, owner, repo, issue, marker, dryRun);
 
-    console.log('=== Summary ===');
-    console.log(`Repository: ${owner}/${repo}`);
-    console.log(`Issue Number: ${issue.number}`);
-    console.log(`Issue Title: ${issue.title || '(no title)'}`);
-    console.log(`Labels: ${issue.labels?.map(l => l.name).join(', ') || 'none'}`);
-    console.log(`Dry Run: ${dryRun}`);
+    logSummary(owner, repo, issue, dryRun);
   } catch (err) {
     console.log('❌ Error:', err.message);
   }