Filter out comments containing `Codecov Report` for triggering, and previous comment summarization purposes

[abuzarmahmood]

[abuzarmahmood]

Summary of Suggestions and Changes

1. Identifying Relevant Files:

   • The issue involves filtering out comments containing "Codecov Report" for triggering and summarization purposes. The relevant files identified for modification include:
     • triggers.py: Contains functions to check conditions related to GitHub issues.
     • agents.py: Responsible for agent creation and configuration.
     • response_agent.py: Handles generating responses to GitHub issues.
     • git_utils.py: Includes utility functions for interacting with the GitHub API.
     • test_triggers.py: Contains tests for the trigger functions.

2. Plan to Address the Issue:

   • Modify the logic in the relevant files to filter out comments containing "Codecov Report".
   • Update functions that retrieve and process comments to exclude those with the specified content.
   • Update tests to ensure the new filtering logic is correctly implemented.

3. Specific Changes Suggested:

   • Update Comment Retrieval Logic:

     • File: git_utils.py
     • Line: 87-110
     • Description: Modify the get_issue_comments function to filter out comments containing "Codecov Report".
     • Code Snippet:

       def get_issue_comments(issue: Issue) -> List[IssueComment]:
           """Get all comments for a specific issue or pull request, ignoring Graphite-related comments"""
           ignore_texts = ["app.graphite.dev", "Codecov Report"]
       
           if isinstance(issue, PullRequest):
               comments = issue.get_issue_comments()
           else:
               comments = issue.get_comments()
       
           comment_bodies = [" ".join(x.body.split()) for x in comments]
       
           # Filter out comments containing any of the ignore_texts
           filtered_comments = [
               comment for comment, comment_body in zip(comments, comment_bodies)
               if not any(ignore_text in comment_body for ignore_text in ignore_texts)
           ]
       
           return list(filtered_comments)

   • Update Trigger Functions to Use Filtered Comments:

     • File: triggers.py
     • Line: 33, 47, 61, 83, 99, 115, 137, 159, 187
     • Description: Ensure that trigger functions use the updated get_issue_comments function to retrieve filtered comments.
     • Code Snippet:

       comments = get_issue_comments(issue)

   • Update Tests for Comment Filtering:

     • File: test_triggers.py
     • Line: 47-255
     • Description: Add test cases to ensure comments containing "Codecov Report" are filtered out.
     • Code Snippet:

       @patch('src.triggers.get_issue_comments')
       def test_comment_filtering(self, mock_get_comments):
           # Test with Codecov Report comment
           mock_codecov_comment = Mock()
           mock_codecov_comment.body = "Codecov Report"
           mock_get_comments.return_value = [mock_codecov_comment]
       
           self.assertFalse(has_generate_edit_command_trigger(Mock()))
           self.assertFalse(has_bot_response(Mock()))
           self.assertFalse(has_user_feedback(Mock()))
           self.assertFalse(has_develop_issue_trigger(Mock()))
           self.assertFalse(has_pull_request_trigger(Mock()))
           self.assertFalse(has_pr_creation_comment(Mock())[0])
           self.assertFalse(has_error_comment(Mock()))
           self.assertFalse(has_user_comment_on_pr(Mock()))

These changes aim to ensure that comments containing "Codecov Report" are effectively filtered out, preventing them from triggering actions or being included in comment summarizations.

---

This response was automatically generated by blech_bot using model gpt-4o

[abuzarmahmood]

[ develop_issue ]

[abuzarmahmood]

Created pull request: #182
Continue discussion there.

---

This response was automatically generated by blech_bot using model gpt-4o