fix: limit lines

Author: robaone-redshelf

.github/actions/auto-pr-description/filter_diff.js

@@ -4,10 +4,10 @@ const fs = require('fs');
 
 /**
  * Filters out specified files from a git diff
- * Usage: node filter_diff.js <diff_file> <ignore_files_comma_separated>
+ * Usage: node filter_diff.js <diff_file> <ignore_files_comma_separated> [max_lines]
  */
 
-function filterDiff(diffContent, ignoreFiles) {
+function filterDiff(diffContent, ignoreFiles, maxLines = null) {
   if (!ignoreFiles || ignoreFiles.trim() === '') {
     return diffContent;
   }
@@ -55,15 +55,24 @@ function filterDiff(diffContent, ignoreFiles) {
     }
   }
 
-  return filteredLines.join('\n');
+  let result = filteredLines.join('\n');
+  
+  // Apply line limit if specified
+  if (maxLines && filteredLines.length > maxLines) {
+    const truncatedLines = filteredLines.slice(0, maxLines);
+    truncatedLines.push('', '# ... (diff truncated due to size limits) ...', '');
+    result = truncatedLines.join('\n');
+  }
+  
+  return result;
 }
 
 // Main execution
 if (require.main === module) {
-  const [, , diffFile, ignoreFiles] = process.argv;
+  const [, , diffFile, ignoreFiles, maxLines] = process.argv;
 
   if (!diffFile) {
-    console.error('Usage: filter_diff.js <diff_file> <ignore_files_comma_separated>');
+    console.error('Usage: filter_diff.js <diff_file> <ignore_files_comma_separated> [max_lines]');
     process.exit(1);
   }
 
@@ -74,7 +83,8 @@ if (require.main === module) {
 
   try {
     const diffContent = fs.readFileSync(diffFile, 'utf8');
-    const filteredDiff = filterDiff(diffContent, ignoreFiles || '');
+    const maxLinesInt = maxLines ? parseInt(maxLines, 10) : null;
+    const filteredDiff = filterDiff(diffContent, ignoreFiles || '', maxLinesInt);
 
     // Write filtered diff back to the same file
     fs.writeFileSync(diffFile, filteredDiff, 'utf8');
@@ -88,6 +98,9 @@ if (require.main === module) {
     if (ignoreFiles) {
       console.error(`Ignored files: ${ignoreFiles}`);
     }
+    if (maxLinesInt && originalLines > maxLinesInt) {
+      console.error(`Applied line limit: ${maxLinesInt}`);
+    }
   } catch (error) {
     console.error(`Error filtering diff: ${error.message}`);
     process.exit(1);

.github/actions/auto-pr-description/generate_pr_description.js

@@ -22,22 +22,22 @@ const path = require('path');
   }
 
   // Create prompt for PR description generation
-  const promptTemplate = `You are an expert programmer who is tasked with writing a pull request description.
-You will be given the git diff of the changes and you must write a markdown pull request description.
-Do not include the git diff in the pull request description. Do not include any other text in the pull request description.
-The pull request description should follow the following format:
+  const promptTemplate = `Write a concise pull request description based on the git diff. Use this exact format:
 
 ## Description
-A short description of the changes.
+Brief summary of changes (1-2 sentences max).
 
 ## Changes
-- [ ] Change 1
-- [ ] Change 2
+- [ ] Key change 1
+- [ ] Key change 2
+- [ ] Key change 3 (max 5 items)
 
 ## Verification
-- [ ] Verification step 1
-- [ ] Verification step 2
-`;
+- [ ] Test step 1
+- [ ] Test step 2
+- [ ] Test step 3 (max 3 items)
+
+Keep it concise and focused on the most important changes.`;
 
   const diffContent = fs.readFileSync(diffFile, 'utf8');
   const combinedPrompt = `${promptTemplate}\n\nHere is the git diff:\n\n${diffContent}`;
@@ -56,7 +56,7 @@ A short description of the changes.
           temperature: 0.7,
           topK: 40,
           topP: 0.95,
-          maxOutputTokens: 2048,
+          maxOutputTokens: 1024,
         }
       })
     });
@@ -70,12 +70,24 @@ A short description of the changes.
 
     const json = await response.json();
 
-    if (!json.candidates || !json.candidates[0] || !json.candidates[0].content) {
+    if (!json.candidates || !json.candidates[0]) {
       console.error('Error: Invalid response from Gemini API');
       console.error(JSON.stringify(json, null, 2));
       process.exit(1);
     }
 
+    // Check if response was truncated due to max tokens
+    if (json.candidates[0].finishReason === 'MAX_TOKENS') {
+      console.error('Warning: Response was truncated due to token limit. Consider reducing diff size or using more specific ignore-files.');
+      // Continue processing the partial response
+    }
+
+    if (!json.candidates[0].content) {
+      console.error('Error: No content in API response');
+      console.error(JSON.stringify(json, null, 2));
+      process.exit(1);
+    }
+
     if (!json.candidates[0].content.parts || !json.candidates[0].content.parts[0] || !json.candidates[0].content.parts[0].text) {
       console.error('Error: Invalid response structure from Gemini API - missing parts or text');
       console.error(JSON.stringify(json, null, 2));

.github/actions/auto-release-description/filter_diff.js

@@ -4,10 +4,10 @@ const fs = require('fs');
 
 /**
  * Filters out specified files from a git diff
- * Usage: node filter_diff.js <diff_file> <ignore_files_comma_separated>
+ * Usage: node filter_diff.js <diff_file> <ignore_files_comma_separated> [max_lines]
  */
 
-function filterDiff(diffContent, ignoreFiles) {
+function filterDiff(diffContent, ignoreFiles, maxLines = null) {
   if (!ignoreFiles || ignoreFiles.trim() === '') {
     return diffContent;
   }
@@ -55,15 +55,24 @@ function filterDiff(diffContent, ignoreFiles) {
     }
   }
 
-  return filteredLines.join('\n');
+  let result = filteredLines.join('\n');
+  
+  // Apply line limit if specified
+  if (maxLines && filteredLines.length > maxLines) {
+    const truncatedLines = filteredLines.slice(0, maxLines);
+    truncatedLines.push('', '# ... (diff truncated due to size limits) ...', '');
+    result = truncatedLines.join('\n');
+  }
+  
+  return result;
 }
 
 // Main execution
 if (require.main === module) {
-  const [, , diffFile, ignoreFiles] = process.argv;
+  const [, , diffFile, ignoreFiles, maxLines] = process.argv;
 
   if (!diffFile) {
-    console.error('Usage: filter_diff.js <diff_file> <ignore_files_comma_separated>');
+    console.error('Usage: filter_diff.js <diff_file> <ignore_files_comma_separated> [max_lines]');
     process.exit(1);
   }
 
@@ -74,7 +83,8 @@ if (require.main === module) {
 
   try {
     const diffContent = fs.readFileSync(diffFile, 'utf8');
-    const filteredDiff = filterDiff(diffContent, ignoreFiles || '');
+    const maxLinesInt = maxLines ? parseInt(maxLines, 10) : null;
+    const filteredDiff = filterDiff(diffContent, ignoreFiles || '', maxLinesInt);
 
     // Write filtered diff back to the same file
     fs.writeFileSync(diffFile, filteredDiff, 'utf8');
@@ -88,6 +98,9 @@ if (require.main === module) {
     if (ignoreFiles) {
       console.error(`Ignored files: ${ignoreFiles}`);
     }
+    if (maxLinesInt && originalLines > maxLinesInt) {
+      console.error(`Applied line limit: ${maxLinesInt}`);
+    }
   } catch (error) {
     console.error(`Error filtering diff: ${error.message}`);
     process.exit(1);

.github/actions/auto-release-description/generate_pr_description.js

@@ -22,22 +22,22 @@ const path = require('path');
   }
 
   // Create prompt for PR description generation
-  const promptTemplate = `You are an expert programmer who is tasked with writing a pull request description.
-You will be given the git diff of the changes and you must write a markdown pull request description.
-Do not include the git diff in the pull request description. Do not include any other text in the pull request description.
-The pull request description should follow the following format:
+  const promptTemplate = `Write a concise pull request description based on the git diff. Use this exact format:
 
 ## Description
-A short description of the changes.
+Brief summary of changes (1-2 sentences max).
 
 ## Changes
-- [ ] Change 1
-- [ ] Change 2
+- [ ] Key change 1
+- [ ] Key change 2
+- [ ] Key change 3 (max 5 items)
 
 ## Verification
-- [ ] Verification step 1
-- [ ] Verification step 2
-`;
+- [ ] Test step 1
+- [ ] Test step 2
+- [ ] Test step 3 (max 3 items)
+
+Keep it concise and focused on the most important changes.`;
 
   const diffContent = fs.readFileSync(diffFile, 'utf8');
   const combinedPrompt = `${promptTemplate}\n\nHere is the git diff:\n\n${diffContent}`;
@@ -56,7 +56,7 @@ A short description of the changes.
           temperature: 0.7,
           topK: 40,
           topP: 0.95,
-          maxOutputTokens: 2048,
+          maxOutputTokens: 1024,
         }
       })
     });
@@ -70,12 +70,24 @@ A short description of the changes.
 
     const json = await response.json();
 
-    if (!json.candidates || !json.candidates[0] || !json.candidates[0].content) {
+    if (!json.candidates || !json.candidates[0]) {
       console.error('Error: Invalid response from Gemini API');
       console.error(JSON.stringify(json, null, 2));
       process.exit(1);
     }
 
+    // Check if response was truncated due to max tokens
+    if (json.candidates[0].finishReason === 'MAX_TOKENS') {
+      console.error('Warning: Response was truncated due to token limit. Consider reducing diff size or using more specific ignore-files.');
+      // Continue processing the partial response
+    }
+
+    if (!json.candidates[0].content) {
+      console.error('Error: No content in API response');
+      console.error(JSON.stringify(json, null, 2));
+      process.exit(1);
+    }
+
     if (!json.candidates[0].content.parts || !json.candidates[0].content.parts[0] || !json.candidates[0].content.parts[0].text) {
       console.error('Error: Invalid response structure from Gemini API - missing parts or text');
       console.error(JSON.stringify(json, null, 2));