diff --git a/pr_agent/algo/pr_processing.py b/pr_agent/algo/pr_processing.py index 96230d66..8065ea43 100644 --- a/pr_agent/algo/pr_processing.py +++ b/pr_agent/algo/pr_processing.py @@ -21,8 +21,8 @@ MORE_MODIFIED_FILES_ = "Additional modified files (insufficient token budget to ADDED_FILES_ = "Additional added files (insufficient token budget to process):\n" -OUTPUT_BUFFER_TOKENS_SOFT_THRESHOLD = 1000 -OUTPUT_BUFFER_TOKENS_HARD_THRESHOLD = 600 +OUTPUT_BUFFER_TOKENS_SOFT_THRESHOLD = 1500 +OUTPUT_BUFFER_TOKENS_HARD_THRESHOLD = 1000 diff --git a/pr_agent/algo/utils.py b/pr_agent/algo/utils.py index 69e52324..f38bb5e8 100644 --- a/pr_agent/algo/utils.py +++ b/pr_agent/algo/utils.py @@ -173,22 +173,30 @@ def convert_to_markdown_v2(output_data: dict, gfm_supported: bool = True, increm markdown_text += f"\n" elif 'key issues to review' in key_nice.lower(): value = value.strip() - issues = value.split('\n- ') - for i, _ in enumerate(issues): - issues[i] = issues[i].strip().strip('-').strip() - issues = unique_strings(issues) # remove duplicates - if gfm_supported: - markdown_text += f"" - markdown_text += f"{emoji} {key_nice}

\n\n" + if is_value_no(value): + if gfm_supported: + markdown_text += f"" + markdown_text += f"{emoji} No key issues to review" + markdown_text += f"\n" + else: + markdown_text += f"### {emoji} No key issues to review\n\n" else: - markdown_text += f"### {emoji} Key issues to review:\n\n" - for i, issue in enumerate(issues): - if not issue: - continue - issue = emphasize_header(issue, only_markdown=True) - markdown_text += f"{issue}\n\n" - if gfm_supported: - markdown_text += f"\n" + issues = value.split('\n- ') + for i, _ in enumerate(issues): + issues[i] = issues[i].strip().strip('-').strip() + issues = unique_strings(issues) # remove duplicates + if gfm_supported: + markdown_text += f"" + markdown_text += f"{emoji} {key_nice}

\n\n" + else: + markdown_text += f"### {emoji} Key issues to review:\n\n" + for i, issue in enumerate(issues): + if not issue: + continue + issue = emphasize_header(issue, only_markdown=True) + markdown_text += f"{issue}\n\n" + if gfm_supported: + markdown_text += f"\n" else: if gfm_supported: markdown_text += f""