Filter BOT/AI Pull Request Reviews #635

Activity

1. samad-yar-khan

   added

   enhancementNew feature or requestNew feature or request

   on Mar 17, 2025
2. jayantbh

   changed the title [-]Filter BOT/AI PUll Request Reviews[/-] [+]Filter BOT/AI Pull Request Reviews[/+] on Mar 17, 2025
3. thesujai commented on Mar 18, 2025

   

   thesujai

   on Mar 18, 2025

   Contributor

   More actions

   Hi @samad-yar-khan Hope you are well:)
   For identifying bots, a new field called actor_type (which indicates either "bot" or "user") should be added to the PullRequestEvent model.
   How will we know whether the review is from a bot or not?
   For GitHub, we can leverage the github.PullRequest.get_reviews() API, where each review's user object also includes a type indicating whether it is a bot or a user. So, just after this line, we will retrieve the actor type as:

   actor_type=actor.get("type", "")
   

   Then, we add it to the PullRequestEvent.

   I'm not sure how GitLab processes this, but the second part will be the same for both platforms:

   The second part is straightforward: simply filter out bot events before any operation in this method. At the beginning of the function, the code will look like:

   pr_events = [
               e
               for e in pr_events
               if not e.actor_type.lower() == "bot" 
           ]
   
4. Ayyanaruto commented on Apr 2, 2025

   

   Ayyanaruto

   on Apr 2, 2025

   Contributor

   More actions

   Can I work on this issue
5. samad-yar-khan commented on Apr 6, 2025

   

   samad-yar-khan

   on Apr 6, 2025

   Last edited by samad-yar-khan

   ContributorAuthor

   More actions

   @Ayyanaruto sure, give it a go. Lets discuss how you are planning to solve this ?
6. jayantbh

   added

   mediumModerate complexity issueModerate complexity issue

   on Apr 8, 2025
7. Ayyanaruto commented on Apr 8, 2025

   

   Ayyanaruto

   on Apr 8, 2025

   Last edited by Ayyanaruto

   Contributor

   More actions

   Hi @samad-yar-khan sorry for late reply as per my deduction the cycle time is calculated here backend/analytics_server/mhq/service/code/sync/etl_code_analytics.py In the Class etl_code_analytics I will check in the api response includes bot in username or if type is bot or author_association=BOT and filter out if present

   👍React with 👍1Reacted by Samad Yar Khan
8. Ayyanaruto commented on Apr 13, 2025

   

   Ayyanaruto

   on Apr 13, 2025

   Contributor

   More actions

   Hi @samad-yar-khan,
   I’ve created the PR for this issue. Sorry about the earlier PR — I had pushed the branch with the wrong logic and wasn’t aware of it. All the tests were failing, and I got a bit confused.
   I’ve closed that PR and created a fresh one. When you’re free, please take a look!
9. jayantbh

   closed this as completedon May 9, 2025