diff --git a/pr_agent/settings/pr_reviewer_prompts.toml b/pr_agent/settings/pr_reviewer_prompts.toml index 88a134eb..9c7626e8 100644 --- a/pr_agent/settings/pr_reviewer_prompts.toml +++ b/pr_agent/settings/pr_reviewer_prompts.toml @@ -109,15 +109,7 @@ class Review(BaseModel): {%- endif %} {%- if require_todo_scan %} todo_sections: Union[List[TodoSection], str] = Field(description="A list of TODO comments found in the code. Return 'No' (as a string) if there are no TODO comments or the list would be empty.") - todo_summary: str = Field(description="Summarize the functional areas of the TODO comments found in the code. - - Focus on describing the *functional areas* of the TODOs, such as: - testing, error handling, validation, documentation, performance, - security, logging, refactoring, API design, UI/UX - - - Example: - TODO related to input validation and error handling - - - Return 'No' (as a string) if there are no TODO comments.") + todo_summary: str = Field(description="Up to 6 words summarizing the functional areas of the TODO comments found in the code.") {%- endif %} {%- if require_can_be_split_review %} can_be_split: List[SubPR] = Field(min_items=0, max_items=3, description="Can this PR, which contains {{ num_pr_files }} changed files in total, be divided into smaller sub-PRs with distinct tasks that can be reviewed and merged independently, regardless of the order ? Make sure that the sub-PRs are indeed independent, with no code dependencies between them, and that each sub-PR represent a meaningful independent task. Output an empty list if the PR code does not need to be split.") @@ -168,7 +160,7 @@ review: todo_sections: | No todo_summary: | - No + ... {%- if require_can_be_split_review %} can_be_split: - relevant_files: @@ -290,7 +282,7 @@ review: todo_sections: | No todo_summary: | - No + ... {%- if require_can_be_split_review %} can_be_split: - relevant_files: