From e351428848aeea107d766d9e61ae692d74506627 Mon Sep 17 00:00:00 2001
From: mrT23 <tal.r@codium.ai>
Date: Thu, 7 Dec 2023 10:24:36 +0200
Subject: [PATCH] s

---
 pr_agent/tools/pr_description.py | 56 ++++----------------------------
 1 file changed, 7 insertions(+), 49 deletions(-)

diff --git a/pr_agent/tools/pr_description.py b/pr_agent/tools/pr_description.py
index 88f5b69c..924eedbe 100644
--- a/pr_agent/tools/pr_description.py
+++ b/pr_agent/tools/pr_description.py
@@ -269,7 +269,7 @@ class PRDescription:
         for idx, (key, value) in enumerate(self.data.items()):
             if key == 'pr_files':
                 value = self.file_label_dict
-                key_publish = "PR changes summary"
+                key_publish = "PR changes walkthrough"
             else:
                 key_publish = key.rstrip(':').replace("_", " ").capitalize()
             pr_body += f"## {key_publish}\n"
@@ -318,9 +318,9 @@ class PRDescription:
 
         try:
             pr_body += "<table>"
-            header = f"Relevant Files"
+            header = f"Relevant files"
             delta = 65
-            header +="&nbsp; " * delta
+            header += "&nbsp; " * delta
             pr_body += f"""<thead><tr><th></th><th>{header}</th></tr></thead>"""
             pr_body += """<tbody>"""
             for semantic_label in value.keys():
@@ -332,15 +332,15 @@ class PRDescription:
                     filename = filename.replace("'", "`")
                     filename_publish = filename.split("/")[-1]
                     filename_publish = f"{filename_publish}"
-                    if len(filename_publish)< (delta-5):
-                        filename_publish += "&nbsp; " * ((delta-5) - len(filename_publish))
+                    if len(filename_publish) < (delta - 5):
+                        filename_publish += "&nbsp; " * ((delta - 5) - len(filename_publish))
                     diff_plus_minus = ""
                     diff_files = self.git_provider.diff_files
                     for f in diff_files:
                         if f.filename.lower() == filename.lower():
                             num_plus_lines = f.num_plus_lines
                             num_minus_lines = f.num_minus_lines
-                            diff_plus_minus += f" (+{num_plus_lines}/-{num_minus_lines})"
+                            diff_plus_minus += f"+{num_plus_lines}/-{num_minus_lines}"
                             break
 
                     # try to add line numbers link to code suggestions
@@ -349,7 +349,7 @@ class PRDescription:
                         filename = filename.strip()
                         link = self.git_provider.get_line_link(filename, relevant_line_start=-1)
 
-                    file_change_description = self._insert_br_after_x_chars(file_change_description, x=(delta-5))
+                    file_change_description = self._insert_br_after_x_chars(file_change_description, x=(delta - 5))
                     pr_body += f"""
 <tr>
   <td>
@@ -368,48 +368,6 @@ class PRDescription:
                 pr_body += """</table></details></td></tr>"""
             pr_body += """</tr></tbody></table>"""
 
-        #     pr_body += """\n| | Relevant Files """
-        #     pr_body += "&nbsp; " * 70
-        #     pr_body += """|\n|-----------|-------------|\n"""
-        #     for semantic_label in value.keys():
-        #         s_label = semantic_label.strip("'").strip('"')
-        #         if self.git_provider.is_supported("gfm_markdown"):
-        #             # pr_body += f"<details> <summary>{semantic_label['label']}</summary>\n\n"
-        #             pr_body += f"| **{s_label}** | <details><summary>files:</summary><ul>"
-        #
-        #         list_tuples = value[semantic_label]
-        #         for filename, file_change_description in list_tuples:
-        #             filename = filename.replace("'", "`")
-        #             filename_publish = filename.split("/")[-1]
-        #             filename_publish = f"**{filename_publish}**"
-        #             diff_plus_minus = ""
-        #             diff_files = self.git_provider.diff_files
-        #             for f in diff_files:
-        #                 if f.filename.lower() == filename.lower():
-        #                     num_plus_lines = f.num_plus_lines
-        #                     num_minus_lines = f.num_minus_lines
-        #                     diff_plus_minus += f" ( +{num_plus_lines}/-{num_minus_lines} )"
-        #                     break
-        #
-        #             # try to add line numbers link to code suggestions
-        #             if hasattr(self.git_provider, 'get_line_link'):
-        #                 filename = filename.strip()
-        #                 link = self.git_provider.get_line_link(filename, relevant_line_start=-1)
-        #                 if link:
-        #                     diff_plus_minus = f"[{diff_plus_minus}]({link})"
-        #                     diff_plus_minus = f" <sup>{diff_plus_minus}</sup>"
-        #
-        #             if diff_plus_minus:
-        #                 filename_publish += diff_plus_minus
-        #             if self.git_provider.is_supported("gfm_markdown"):
-        #                 pr_body += f"<details><summary>{filename_publish}</summary>"
-        #                 file_change_description = self._insert_br_after_x_chars(file_change_description)
-        #                 if diff_plus_minus:
-        #                     pr_body += f"<ul>Changes summary:<br>**{file_change_description}**</ul></details>"
-        #                 else:
-        #                     pr_body += f"<ul>Changes summary:<br>**{file_change_description}**</ul></details>"
-        #         if self.git_provider.is_supported("gfm_markdown"):
-        #             pr_body += "</ul></details>|\n"
         except Exception as e:
             get_logger().error(f"Error processing pr files to markdown {self.pr_id}: {e}")
             pass