fix: skip cr task if there is no error

[xingwanying]

close #602

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
petercat	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 30, 2024 10:03am

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
server/agent/tools/pull_request.py	0.00%	10 Missing ⚠️
server/agent/base.py	0.00%	3 Missing ⚠️

Files with missing lines	Coverage Δ
server/agent/prompts/pull_request.py	66.66% <ø> (ø)
server/agent/base.py	24.29% <0.00%> (ø)
server/agent/tools/pull_request.py	15.55% <0.00%> (ø)

[petercat-assistant]

Walkthrough

This pull request addresses issue #602 by modifying the behavior of the code to skip certain tasks if no errors are encountered. The changes involve restructuring error handling and refining the logic for invoking agent executors.

Changes

File	Summary
server/agent/base.py	Refactored error handling in run_chat method and improved the logic for invoking agent executors.
server/agent/prompts/pull_request.py	Updated instructions for code review tasks, focusing on critical issues only.
server/agent/tools/pull_request.py	Refactored the get_file_content and create_review_comment functions for better readability and maintainability.

[petercat-assistant]

The error logging message has been improved to provide more context. Ensure that sensitive information is not logged inadvertently.