mirror of
https://github.com/qodo-ai/pr-agent.git
synced 2025-07-02 03:40:38 +08:00
144 lines
6.5 KiB
Python
144 lines
6.5 KiB
Python
import copy
|
|
import json
|
|
import logging
|
|
|
|
from jinja2 import Environment, StrictUndefined
|
|
|
|
from pr_agent.algo.ai_handler import AiHandler
|
|
from pr_agent.algo.pr_processing import get_pr_diff
|
|
from pr_agent.algo.token_handler import TokenHandler
|
|
from pr_agent.algo.utils import convert_to_markdown, try_fix_json
|
|
from pr_agent.config_loader import settings
|
|
from pr_agent.git_providers import get_git_provider
|
|
from pr_agent.git_providers.git_provider import get_main_pr_language
|
|
|
|
|
|
class PRReviewer:
|
|
def __init__(self, pr_url: str, cli_mode=False):
|
|
|
|
self.git_provider = get_git_provider()(pr_url)
|
|
self.main_language = get_main_pr_language(
|
|
self.git_provider.get_languages(), self.git_provider.get_files()
|
|
)
|
|
self.ai_handler = AiHandler()
|
|
self.patches_diff = None
|
|
self.prediction = None
|
|
self.cli_mode = cli_mode
|
|
self.vars = {
|
|
"title": self.git_provider.pr.title,
|
|
"branch": self.git_provider.get_pr_branch(),
|
|
"description": self.git_provider.get_pr_description(),
|
|
"language": self.main_language,
|
|
"diff": "", # empty diff for initial calculation
|
|
"require_tests": settings.pr_reviewer.require_tests_review,
|
|
"require_security": settings.pr_reviewer.require_security_review,
|
|
"require_focused": settings.pr_reviewer.require_focused_review,
|
|
'num_code_suggestions': settings.pr_reviewer.num_code_suggestions,
|
|
}
|
|
self.token_handler = TokenHandler(self.git_provider.pr,
|
|
self.vars,
|
|
settings.pr_review_prompt.system,
|
|
settings.pr_review_prompt.user)
|
|
|
|
async def review(self):
|
|
logging.info('Reviewing PR...')
|
|
if settings.config.publish_review:
|
|
self.git_provider.publish_comment("Preparing review...", is_temporary=True)
|
|
logging.info('Getting PR diff...')
|
|
self.patches_diff = get_pr_diff(self.git_provider, self.token_handler)
|
|
logging.info('Getting AI prediction...')
|
|
self.prediction = await self._get_prediction()
|
|
logging.info('Preparing PR review...')
|
|
pr_comment = self._prepare_pr_review()
|
|
if settings.config.publish_review:
|
|
logging.info('Pushing PR review...')
|
|
self.git_provider.publish_comment(pr_comment)
|
|
self.git_provider.remove_initial_comment()
|
|
if settings.pr_reviewer.inline_code_comments:
|
|
logging.info('Pushing inline code comments...')
|
|
self._publish_inline_code_comments()
|
|
return ""
|
|
|
|
async def _get_prediction(self):
|
|
variables = copy.deepcopy(self.vars)
|
|
variables["diff"] = self.patches_diff # update diff
|
|
environment = Environment(undefined=StrictUndefined)
|
|
system_prompt = environment.from_string(settings.pr_review_prompt.system).render(variables)
|
|
user_prompt = environment.from_string(settings.pr_review_prompt.user).render(variables)
|
|
if settings.config.verbosity_level >= 2:
|
|
logging.info(f"\nSystem prompt:\n{system_prompt}")
|
|
logging.info(f"\nUser prompt:\n{user_prompt}")
|
|
model = settings.config.model
|
|
response, finish_reason = await self.ai_handler.chat_completion(model=model, temperature=0.2,
|
|
system=system_prompt, user=user_prompt)
|
|
|
|
return response
|
|
|
|
def _prepare_pr_review(self) -> str:
|
|
review = self.prediction.strip()
|
|
try:
|
|
data = json.loads(review)
|
|
except json.decoder.JSONDecodeError:
|
|
data = try_fix_json(review)
|
|
|
|
# reordering for nicer display
|
|
if 'PR Feedback' in data:
|
|
if 'Security concerns' in data['PR Feedback']:
|
|
val = data['PR Feedback']['Security concerns']
|
|
del data['PR Feedback']['Security concerns']
|
|
data['PR Analysis']['Security concerns'] = val
|
|
|
|
if settings.config.git_provider == 'github' and settings.pr_reviewer.inline_code_comments:
|
|
del data['PR Feedback']['Code suggestions']
|
|
|
|
markdown_text = convert_to_markdown(data)
|
|
user = self.git_provider.get_user_id()
|
|
|
|
if not self.cli_mode:
|
|
markdown_text += "\n### How to use\n"
|
|
if user and '[bot]' not in user:
|
|
markdown_text += f"> Tag me in a comment '@{user}' to ask for a new review after you update the PR.\n"
|
|
markdown_text += "> You can also tag me and ask any question, " \
|
|
f"for example '@{user} is the PR ready for merge?'"
|
|
else:
|
|
markdown_text += "> Add a comment that says 'review' to ask for a new review " \
|
|
"after you update the PR.\n"
|
|
markdown_text += "> You can also add a comment that says 'answer QUESTION', " \
|
|
"for example 'answer is the PR ready for merge?'"
|
|
|
|
if settings.config.verbosity_level >= 2:
|
|
logging.info(f"Markdown response:\n{markdown_text}")
|
|
return markdown_text
|
|
|
|
def _publish_inline_code_comments(self):
|
|
if settings.config.git_provider != 'github': # inline comments are currently only supported for github
|
|
return
|
|
|
|
review = self.prediction.strip()
|
|
try:
|
|
data = json.loads(review)
|
|
except json.decoder.JSONDecodeError:
|
|
data = try_fix_json(review)
|
|
|
|
pr = self.git_provider.pr
|
|
last_commit_id = list(pr.get_commits())[-1]
|
|
files = list(self.git_provider.get_diff_files())
|
|
|
|
for d in data['PR Feedback']['Code suggestions']:
|
|
relevant_file = d['relevant file'].strip()
|
|
relevant_line_in_file = d['relevant line in file'].strip()
|
|
content = d['suggestion content']
|
|
position = -1
|
|
for file in files:
|
|
if file.filename.strip() == relevant_file:
|
|
patch = file.patch
|
|
patch_lines = patch.splitlines()
|
|
for i, line in enumerate(patch_lines):
|
|
if relevant_line_in_file in line:
|
|
position = i
|
|
if position == -1:
|
|
logging.info(f"Could not find position for {relevant_file} {relevant_line_in_file}")
|
|
else:
|
|
body = content
|
|
path = relevant_file.strip()
|
|
pr.create_review_comment(body=body, commit_id=last_commit_id, path=path, position=position) |