check if files need to be ignored in case they are listed in a discussion

Author: haesleinhuepf

src/git_bob/_ai_github_utilities.py

@@ -60,7 +60,8 @@ def comment_on_issue(repository, issue, prompt_function, **kwargs):
         file_changes = ""
         conversation_type = "issue"
 
-    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), prompt_visionlm=prompt_function)
+    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), 
+                                   prompt_visionlm=prompt_function, repository=repository, branch_name="main")
     print("Discussion:", discussion)
 
     all_files = "* " + "\n* ".join(Config.git_utilities.list_repository_files(repository))
@@ -84,6 +85,8 @@ def comment_on_issue(repository, issue, prompt_function, **kwargs):
 """)
     filenames = text_to_json(relevant_files)
 
+    filenames = [f for f in filenames if not is_ignored(f, repository, "main")]
+
     file_content_dict = Config.git_utilities.get_repository_file_contents(repository, "main", filenames)
 
     temp = []
@@ -148,7 +151,10 @@ def review_pull_request(repository, issue, prompt_function, **kwargs):
     if Config.pull_request is None: # it's not a PR
         return comment_on_issue(repository, issue, prompt_function)
 
-    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), prompt_visionlm=prompt_function)
+    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), 
+                                   prompt_visionlm=prompt_function, 
+                                   repository=repository, 
+                                   branch_name="main")
     print("Discussion:", discussion)
 
     file_changes = Config.git_utilities.get_diff_of_pull_request(repository, issue)
@@ -606,7 +612,11 @@ def solve_github_issue(repository, issue, prompt_function, base_branch=None):
 
     repo = Config.git_utilities.get_repository_handle(repository)
 
-    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), prompt_visionlm=prompt_function)
+    discussion = modify_discussion(Config.git_utilities.get_conversation_on_issue(repository, issue), 
+                                   prompt_visionlm=prompt_function,
+                                   repository=repository,
+                                   branch_name=base_branch)
+    repository=repository, branch_name=base_branch)
     print("Discussion:", discussion)
 
     all_files = "* " + "\n* ".join(Config.git_utilities.list_repository_files(repository, branch_name=base_branch))

src/git_bob/_utilities.py

@@ -261,9 +261,10 @@ def image_to_url(image):
     return img_str
 
 
-def modify_discussion(discussion, prompt_visionlm=prompt_openai):
+def modify_discussion(discussion, prompt_visionlm=prompt_openai, repository=None, branch_name=None):
     import re
     import docx2markdown
+    from ._ai_github_utilities import is_ignored
     #from ._github_utilities import get_conversation_on_issue, get_diff_of_pull_request, get_file_in_repository
 
     # Regex to find URLs in the discussion
@@ -273,6 +274,9 @@ def modify_discussion(discussion, prompt_visionlm=prompt_openai):
     # Placeholder for additional content extracted from URLs
     additional_content = {}
 
+    if branch_name is None:
+        branch_name = "main"
+
     # Process each URL based on its type
     for url in urls:
         if url.endswith(")"): # happens with ![](url) syntax
@@ -348,7 +352,7 @@ def modify_discussion(discussion, prompt_visionlm=prompt_openai):
         if len(potential_filename) < 4: # too short to be a filename
             continue
         # check if file exists
-        if os.path.exists(potential_filename):
+        if os.path.exists(potential_filename) and not is_ignored(potential_filename, repository, branch_name):
             if potential_filename.endswith('.ipynb'):
                 file_contents = read_text_file(potential_filename)
                 file_contents = erase_outputs_of_code_cells(file_contents)