diff --git a/pr_agent/git_providers/github_provider.py b/pr_agent/git_providers/github_provider.py index 1b48634d..b1000393 100644 --- a/pr_agent/git_providers/github_provider.py +++ b/pr_agent/git_providers/github_provider.py @@ -62,8 +62,14 @@ class GithubProvider: if relevant_line_in_file in line: position = i break + elif relevant_line_in_file[0] == '+' and relevant_line_in_file[1:] in line: + # The model often adds a '+' to the beginning of the relevant_line_in_file even if originally + # it's a context line + position = i + break if position == -1: - logging.info(f"Could not find position for {relevant_file} {relevant_line_in_file}") + if settings.config.verbosity_level >= 2: + logging.info(f"Could not find position for {relevant_file} {relevant_line_in_file}") else: path = relevant_file.strip() self.pr.create_review_comment(body=body, commit_id=self.last_commit_id, path=path, position=position) diff --git a/pr_agent/git_providers/gitlab_provider.py b/pr_agent/git_providers/gitlab_provider.py index edc70c27..c0152a8d 100644 --- a/pr_agent/git_providers/gitlab_provider.py +++ b/pr_agent/git_providers/gitlab_provider.py @@ -136,6 +136,8 @@ class GitLabProvider(GitProvider): edit_type = self.get_edit_type(line) break elif relevant_line_in_file[0] == '+' and relevant_line_in_file[1:] in line: + # The model often adds a '+' to the beginning of the relevant_line_in_file even if originally + # it's a context line found = True edit_type = self.get_edit_type(line) break