From d4a52ffc935ec165dbca75319a1cf9ccf42e038b Mon Sep 17 00:00:00 2001 From: Judy Date: Wed, 18 Jun 2025 18:30:24 +0900 Subject: [PATCH] refactor: clarify TODO field description and simplify 'No' return --- pr_agent/settings/pr_reviewer_prompts.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pr_agent/settings/pr_reviewer_prompts.toml b/pr_agent/settings/pr_reviewer_prompts.toml index 9c7626e8..157f4f73 100644 --- a/pr_agent/settings/pr_reviewer_prompts.toml +++ b/pr_agent/settings/pr_reviewer_prompts.toml @@ -108,7 +108,7 @@ class Review(BaseModel): security_concerns: str = Field(description="Does this PR code introduce possible vulnerabilities such as exposure of sensitive information (e.g., API keys, secrets, passwords), or security concerns like SQL injection, XSS, CSRF, and others ? Answer 'No' (without explaining why) if there are no possible issues. If there are security concerns or issues, start your answer with a short header, such as: 'Sensitive information exposure: ...', 'SQL injection: ...' etc. Explain your answer. Be specific and give examples if possible") {%- 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_sections: Union[List[TodoSection], str] = Field(description="A list of TODO comments found in the PR code. Return 'No' (as a string) if there are no TODO comments in the PR" 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 %}