search

Codium-ai / pr-agent

🚀CodiumAI PR-Agent: An AI-Powered 🤖 Tool for Automated Pull Request Analysis, Feedback, Suggestions and More! 💻🔍
Apache License 2.0
6.03k stars 588 forks source link

[Feature Request] Replying to repetitive committable code suggestions using threads #920

Closed kkc-tonywu closed 4 months ago

kkc-tonywu commented 5 months ago

Hello everyone,

First of all, I'd like to express my gratitude to all those involved in integrating this tool with GitLab. So far, it has been working very well.

Whenever the PR-Agent conducts code reviews with commitable_code_suggestions enabled, it generates multiple suggestions. However, upon running the PR-Agent again, there is a chance of regenerating the same suggestions. Is it possible to respond to these repetitive suggestions through a threaded approach?

mrT23 commented 5 months ago

1) i am not sure what is a "thread" in github (or in gitlab ? bitbucket ?). is that something that is actually supported?

2) our recommended, and default way, for code suggestions is as a single comment. we introduced a lot of improvements there, including noise reduction, and are planning to keep working on that mode. commitable code suggestions are far too noisy for most users, and that's why we are not prioritizing them. Still, a contribution from the community for committable code suggestions will be welcomed

image

kkc-tonywu commented 5 months ago

I am referring to the GitLab platform. (Related API: Add note to existing merge request thread)

Because I noticed duplicated and similar code suggestions when using committable code suggestions, we would like to consolidate them under the same thread. This approach will help reduce clutter and streamline categorization.

image

image

commitable code suggestions are far too noisy for most users

I think there are two ways to solve this:

  1. [GITLAB] Aggregate review comments and shoot in 1 as generally the MR review works in Gitlab · Issue #307 · Codium-ai/pr-agent
  2. The notification settings can be adjusted and controlled through the configuration file, designed to be as flexible as possible.
mrT23 commented 5 months ago

you are welcome to open a PR