Merge pull request #95 from Codium-ai/tr/disable_review_suggestiosn

disable code suggestions by default for review

pr_agent/algo/utils.py

@@ -30,7 +30,8 @@ def convert_to_markdown(output_data: dict) -> str:
         elif isinstance(value, list):
             if key.lower() == 'code suggestions':
                 markdown_text += "\n"  # just looks nicer with additional line breaks
-            emoji = emojis.get(key, "‣")  # Use a dash if no emoji is found for the key
+            # emoji = emojis.get(key, "‣")  # Use a dash if no emoji is found for the key
+            emoji = ""
             markdown_text += f"- {emoji} **{key}:**\n\n"
             for item in value:
                 if isinstance(item, dict) and key.lower() == 'code suggestions':
@@ -38,7 +39,8 @@ def convert_to_markdown(output_data: dict) -> str:
                 elif item:
                     markdown_text += f"  - {item}\n"
         elif value != 'n/a':
-            emoji = emojis.get(key, "‣")  # Use a dash if no emoji is found for the key
+            # emoji = emojis.get(key, "‣")  # Use a dash if no emoji is found for the key
+            emoji = ""
             markdown_text += f"- {emoji} **{key}:** {value}\n"
     return markdown_text
 

pr_agent/settings/configuration.toml

@@ -8,7 +8,7 @@ verbosity_level=0 # 0,1,2
 require_focused_review=true
 require_tests_review=true
 require_security_review=true
-num_code_suggestions=3
+num_code_suggestions=0
 inline_code_comments = true
 ask_and_reflect=false
 

pr_agent/settings/pr_reviewer_prompts.toml

@@ -103,7 +103,7 @@ Example output:
             ...
         ]
 {%- endif %}
-{%- if require_security %},
+{%- if require_security %}
         "Security concerns": "No, because ..."
 {%- endif %}
     }