chore(bug-prediction): Require repo owner and name (#107919)

[seer PR](https://github.com/getsentry/seer/pull/4817) which sends these
was deployed

Author: kddubey

src/sentry/seer/fetch_issues/by_error_type.py

@@ -45,10 +45,10 @@ def fetch_issues(
     provider: str,
     external_id: str,
     exception_type: str,
+    owner: str,
+    name: str,
     max_num_issues: int = utils.MAX_NUM_ISSUES_DEFAULT,
     num_days_ago: int = utils.MAX_NUM_DAYS_AGO_DEFAULT,
-    owner: str | None = None,
-    name: str | None = None,
     run_id: int | None = None,
 ) -> utils.SeerResponse | utils.SeerResponseError:
     repo_projects = utils.get_repo_and_projects(

src/sentry/seer/fetch_issues/by_function_name.py

@@ -267,9 +267,9 @@ def fetch_issues(
     external_id: str,
     filename: str,
     function_name: str,
+    owner: str,
+    name: str,
     max_num_issues_per_file: int = MAX_NUM_ISSUES_PER_FILE_DEFAULT,
-    owner: str | None = None,
-    name: str | None = None,
     run_id: int | None = None,
 ) -> utils.SeerResponse | utils.SeerResponseError:
     """

src/sentry/seer/fetch_issues/by_text_query.py

@@ -48,11 +48,11 @@ def fetch_issues(
     provider: str,
     external_id: str,
     query: str,
+    owner: str,
+    name: str,
     sort_by: str = SORT_BY_DEFAULT,
     limit: int = utils.MAX_NUM_ISSUES_DEFAULT,
     max_num_days_ago: int = utils.MAX_NUM_DAYS_AGO_DEFAULT,
-    owner: str | None = None,
-    name: str | None = None,
     run_id: int | None = None,
 ) -> utils.SeerResponse | utils.SeerResponseError:
     """

src/sentry/seer/fetch_issues/utils.py

@@ -5,11 +5,9 @@
 from typing import Any, TypedDict
 
 import sentry_sdk
-from django.db.models import Q
 
 from sentry.api.serializers import serialize
 from sentry.api.serializers.models.event import EventSerializer
-from sentry.constants import ObjectStatus
 from sentry.integrations.models.repository_project_path_config import RepositoryProjectPathConfig
 from sentry.models.group import Group
 from sentry.models.project import Project
@@ -65,8 +63,8 @@ def get_repo_and_projects(
     organization_id: int,
     provider: str,
     external_id: str,
-    owner: str | None = None,
-    name: str | None = None,
+    owner: str,
+    name: str,
     run_id: int | None = None,
 ) -> RepoProjects:
     """
@@ -81,15 +79,7 @@ def get_repo_and_projects(
             "run_id": run_id,
         }
     )
-    if owner and name:
-        repo = filter_repo_by_provider(organization_id, provider, external_id, owner, name).first()
-    else:  # TODO(kddubey): remove this branch once seer supplies owner and name
-        repo = Repository.objects.filter(
-            Q(provider=provider) | Q(provider=f"integrations:{provider}"),
-            organization_id=organization_id,
-            external_id=external_id,
-            status=ObjectStatus.ACTIVE,
-        ).first()
+    repo = filter_repo_by_provider(organization_id, provider, external_id, owner, name).first()
     if repo is None:
         raise Repository.DoesNotExist
     repo_configs = list(

tests/sentry/seer/fetch_issues/test_by_error_type.py

@@ -46,6 +46,8 @@ def test_simple(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="KeyError",
         )
         assert "error" not in seer_response
@@ -66,6 +68,8 @@ def test_simple(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="ValueError",
         )
         assert seer_response == {"issues": [], "issues_full": []}
@@ -158,6 +162,8 @@ def test_multiple_projects(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="KeyError",
         )
         assert "error" not in seer_response
@@ -208,6 +214,8 @@ def test_last_seen_filter(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="KeyError",
         )
         assert "error" not in seer_response
@@ -220,6 +228,8 @@ def test_last_seen_filter(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="KeyError",
             num_days_ago=9,
         )
@@ -284,6 +294,8 @@ def test_multiple_exception_types(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="KeyError",
         )
         assert "error" not in seer_response
@@ -297,6 +309,8 @@ def test_multiple_exception_types(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="ValueError",
         )
         assert "error" not in seer_response
@@ -335,7 +349,11 @@ def test_fetch_issues_from_repo_projects_returns_groups(self) -> None:
 
         # Get repo projects
         repo_projects = get_repo_and_projects(
-            organization_id=self.organization.id, provider="integrations:github", external_id="1"
+            organization_id=self.organization.id,
+            provider="integrations:github",
+            external_id="1",
+            owner="getsentry",
+            name="sentryA",
         )
 
         # Test the internal function directly
@@ -361,7 +379,11 @@ def test_fetch_issues_from_repo_projects_empty_result(self) -> None:
 
         # Get repo projects but don't create any matching events
         repo_projects = get_repo_and_projects(
-            organization_id=self.organization.id, provider="integrations:github", external_id="1"
+            organization_id=self.organization.id,
+            provider="integrations:github",
+            external_id="1",
+            owner="getsentry",
+            name="sentryA",
         )
 
         # Test the internal function directly
@@ -412,6 +434,8 @@ def _assert_exception_type_matches(
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type=search_exception_type,
         )
         assert "error" not in seer_response
@@ -511,6 +535,8 @@ def test_normalized_matching_multiple_groups(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="valueerror",
         )
         assert "error" not in seer_response
@@ -522,6 +548,8 @@ def test_normalized_matching_multiple_groups(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="type error",
         )
         assert "error" not in seer_response
@@ -533,6 +561,8 @@ def test_normalized_matching_multiple_groups(self) -> None:
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="1",
+            owner="getsentry",
+            name="sentryA",
             exception_type="runtimeerror",
         )
         assert seer_response == {"issues": [], "issues_full": []}

tests/sentry/seer/fetch_issues/test_by_function_name.py

@@ -305,6 +305,8 @@ def test_successful_fetch(self):
                     organization_id=self.organization.id,
                     provider="integrations:github",
                     external_id=self.gh_repo.external_id,
+                    owner="getsentry",
+                    name="sentry",
                     filename="test.py",
                     function_name="target_function",
                 )
@@ -338,6 +340,8 @@ def test_custom_max_num_issues(self):
                     organization_id=self.organization.id,
                     provider="integrations:github",
                     external_id=self.gh_repo.external_id,
+                    owner="getsentry",
+                    name="sentry",
                     filename="test.py",
                     function_name="target_function",
                     max_num_issues_per_file=10,
@@ -367,6 +371,8 @@ def test_with_run_id(self):
                     organization_id=self.organization.id,
                     provider="integrations:github",
                     external_id=self.gh_repo.external_id,
+                    owner="getsentry",
+                    name="sentry",
                     filename="test.py",
                     function_name="target_function",
                     run_id=12345,
@@ -393,6 +399,8 @@ def test_fetch_issues_end_to_end_with_metadata_and_message(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             filename="test.py",
             function_name="target_function",
         )
@@ -515,6 +523,8 @@ def test_fetch_issues_from_repo_projects_returns_groups(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
         )
 
         # Test the internal function directly with real search behavior
@@ -539,6 +549,8 @@ def test_fetch_issues_from_repo_projects_empty_result(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
         )
 
         # Test the internal function with non-matching criteria

tests/sentry/seer/fetch_issues/test_by_text_query.py

@@ -94,6 +94,8 @@ def test_fetch_issues_message_substring_search(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             query="hello",
         )
         assert "error" not in seer_response
@@ -106,6 +108,8 @@ def test_fetch_issues_message_substring_search(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             query="auth",
         )
         assert "error" not in seer_response
@@ -144,6 +148,8 @@ def test_fetch_issues_no_match(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             query="nonexistent_keyword_xyz123",
         )
 
@@ -165,6 +171,8 @@ def test_fetch_issues_culprit_search(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             query="database conn",
         )
 
@@ -189,6 +197,8 @@ def test_fetch_issues_limit_parameter(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
             query="common.py",
             limit=limit,
         )
@@ -213,6 +223,8 @@ def test_fetch_issues_from_repo_projects_returns_groups(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
         )
 
         # Test the internal function directly with a query that should match the created event
@@ -236,6 +248,8 @@ def test_fetch_issues_from_repo_projects_empty_result(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id=self.gh_repo.external_id,
+            owner="getsentry",
+            name="sentry",
         )
 
         # Test the internal function with a query that won't match anything

tests/sentry/seer/fetch_issues/test_utils.py

@@ -29,6 +29,8 @@ def test_get_repo_and_projects_success(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="123",
+            owner="getsentry",
+            name="sentry",
         )
 
         assert isinstance(result, RepoProjects)
@@ -56,6 +58,8 @@ def test_get_repo_and_projects_multiple_projects(self):
             organization_id=self.organization.id,
             provider="integrations:github",
             external_id="123",
+            owner="getsentry",
+            name="sentry",
         )
 
         assert len(result.repo_configs) == 2
@@ -76,6 +80,8 @@ def test_get_repo_and_projects_no_configs(self):
                 organization_id=self.organization.id,
                 provider="integrations:github",
                 external_id="123",
+                owner="getsentry",
+                name="sentry",
             )
 
     def test_get_repo_and_projects_unprefixed_provider(self):
@@ -87,24 +93,6 @@ def test_get_repo_and_projects_unprefixed_provider(self):
         )
         self.create_code_mapping(project=self.project, repo=repo)
 
-        result = get_repo_and_projects(
-            organization_id=self.organization.id,
-            provider="github",
-            external_id="123",
-        )
-
-        assert result.repo == repo
-        assert len(result.projects) == 1
-
-    def test_get_repo_and_projects_with_owner_and_name(self):
-        repo = self.create_repo(
-            project=self.project,
-            name="getsentry/sentry",
-            provider="integrations:github",
-            external_id="123",
-        )
-        self.create_code_mapping(project=self.project, repo=repo)
-
         result = get_repo_and_projects(
             organization_id=self.organization.id,
             provider="github",
@@ -144,6 +132,8 @@ def test_get_repo_and_projects_repo_not_found(self):
                 organization_id=self.organization.id,
                 provider="integrations:github",
                 external_id="nonexistent",
+                owner="getsentry",
+                name="sentry",
             )