From 2235a1934521c40720eddb84fa3af1232b894145 Mon Sep 17 00:00:00 2001 From: mrT23 Date: Wed, 3 Jul 2024 20:53:15 +0300 Subject: [PATCH 1/2] increase margins --- pr_agent/algo/pr_processing.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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 From 422b4082b5f800a39117732f95e10f0b03f13856 Mon Sep 17 00:00:00 2001 From: mrT23 Date: Wed, 3 Jul 2024 20:58:25 +0300 Subject: [PATCH 2/2] No key issues to review --- pr_agent/algo/utils.py | 38 +++++++++++++++++++++++--------------- 1 file changed, 23 insertions(+), 15 deletions(-) diff --git a/pr_agent/algo/utils.py b/pr_agent/algo/utils.py index 4596223c..cf7a49e4 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""