mirror of
https://github.com/qodo-ai/pr-agent.git
synced 2025-07-10 15:50:37 +08:00
refactor: remove count info from todo summary
This commit is contained in:
@ -109,18 +109,15 @@ 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.")
|
||||
todos_summary: str = Field(description="When writing TODO section summaries, use this format:
|
||||
[count] TODOs found about [functional area based on TODO content]
|
||||
|
||||
- The [count] is the number of TODO items, equal to the length of the `todo_sections` list.
|
||||
- Functional areas describe what the TODOs are about:
|
||||
testing, error handling, validation, documentation, performance,
|
||||
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:
|
||||
3 TODOs found about input validation and error handling
|
||||
|
||||
Return 'No' (as a string) if there are no TODO comments.")
|
||||
- Example:
|
||||
TODO related to input validation and error handling
|
||||
|
||||
- Return 'No' (as a string) if there are no TODO comments.")
|
||||
{%- 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.")
|
||||
@ -170,7 +167,7 @@ review:
|
||||
No
|
||||
todo_sections: |
|
||||
No
|
||||
todos_summary: |
|
||||
todo_summary: |
|
||||
No
|
||||
{%- if require_can_be_split_review %}
|
||||
can_be_split:
|
||||
@ -292,7 +289,7 @@ review:
|
||||
No
|
||||
todo_sections: |
|
||||
No
|
||||
todos_summary: |
|
||||
todo_summary: |
|
||||
No
|
||||
{%- if require_can_be_split_review %}
|
||||
can_be_split:
|
||||
|
Reference in New Issue
Block a user