From c4ccfd865dc5d9ff4c84a5eb1ad5649918460319 Mon Sep 17 00:00:00 2001 From: dst03106 Date: Wed, 18 Jun 2025 17:29:21 +0900 Subject: [PATCH] fix: shorten todo summary prompt for balance related comment: https://github.com/qodo-ai/pr-agent/pull/1869#discussion_r2151443243 --- pr_agent/settings/pr_reviewer_prompts.toml | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) 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: