refactor(code-review): Renames and moving files around

Renames the "code-review-local" things from "cli-bug-prediction" or "cli-pr-review"

Author: giovanni-guidini

src/sentry/api/urls.py

@@ -11,7 +11,6 @@
     OrganizationAuthTokenDetailsEndpoint,
 )
 from sentry.api.endpoints.organization_auth_tokens import OrganizationAuthTokensEndpoint
-from sentry.api.endpoints.organization_cli_bug_prediction import OrganizationCodeReviewLocalEndpoint
 from sentry.api.endpoints.organization_events_root_cause_analysis import (
     OrganizationEventsRootCauseAnalysisEndpoint,
 )
@@ -526,6 +525,7 @@
     ProjectRuleGroupHistoryIndexEndpoint,
 )
 from sentry.rules.history.endpoints.project_rule_stats import ProjectRuleStatsIndexEndpoint
+from sentry.seer.code_review.endpoints.code_review_local import OrganizationCodeReviewLocalEndpoint
 from sentry.seer.endpoints.group_ai_autofix import GroupAutofixEndpoint
 from sentry.seer.endpoints.group_ai_summary import GroupAiSummaryEndpoint
 from sentry.seer.endpoints.group_autofix_setup_check import GroupAutofixSetupCheck

…oints/organization_cli_bug_prediction.py …de_review/endpoints/code_review_local.pysrc/sentry/api/endpoints/organization_cli_bug_prediction.py renamed to src/sentry/seer/code_review/endpoints/code_review_local.py src/sentry/api/endpoints/organization_cli_bug_prediction.py renamed to src/sentry/seer/code_review/endpoints/code_review_local.py

@@ -12,12 +12,12 @@
 from sentry.api.api_publish_status import ApiPublishStatus
 from sentry.api.base import region_silo_endpoint
 from sentry.api.bases.organization import OrganizationEndpoint, OrganizationIntegrationsPermission
-from sentry.api.serializers.rest_framework.cli_bug_prediction import (
-    CliBugPredictionRequestSerializer,
-)
 from sentry.models.organization import Organization
 from sentry.models.repository import Repository
-from sentry.seer.cli_bug_prediction import get_cli_bug_prediction_status, trigger_cli_bug_prediction
+from sentry.seer.code_review.endpoints.serializers.code_review_local import (
+    CodeReviewLocalRequestSerializer,
+)
+from sentry.seer.code_review_local import get_code_review_local_status, trigger_code_review_local
 from sentry.utils import metrics
 
 logger = logging.getLogger(__name__)
@@ -90,7 +90,7 @@ def post(self, request: Request, organization: Organization) -> Response:
             )
 
         # Validate request
-        serializer = CliBugPredictionRequestSerializer(data=request.data)
+        serializer = CodeReviewLocalRequestSerializer(data=request.data)
         if not serializer.is_valid():
             return Response({"detail": serializer.errors}, status=400)
 
@@ -137,7 +137,7 @@ def post(self, request: Request, organization: Organization) -> Response:
         user_name = request.user.username or getattr(request.user, "email", None) or str(user_id)
 
         try:
-            trigger_response = trigger_cli_bug_prediction(
+            trigger_response = trigger_code_review_local(
                 repo_provider=repo_data["provider"],
                 repo_owner=repo_data["owner"],
                 repo_name=repo_data["name"],
@@ -291,7 +291,9 @@ def _resolve_repository(
             provider_variants.append(f"integrations:{repo_provider}")
 
         return Repository.objects.get(
-            organization_id=organization.id, name=repo_name, provider__in=provider_variants
+            organization_id=organization.id,
+            name=repo_name,
+            provider__in=provider_variants,
         )
 
     def _poll_seer_for_results(
@@ -331,7 +333,7 @@ def _poll_seer_for_results(
             )
 
             try:
-                response = get_cli_bug_prediction_status(run_id)
+                response = get_code_review_local_status(run_id)
             except (UrllibTimeoutError, MaxRetryError):
                 # If status check times out, wait and retry
                 logger.warning(

…ers/rest_framework/cli_bug_prediction.py …dpoints/serializers/code_review_local.pysrc/sentry/api/serializers/rest_framework/cli_bug_prediction.py renamed to src/sentry/seer/code_review/endpoints/serializers/code_review_local.py src/sentry/api/serializers/rest_framework/cli_bug_prediction.py renamed to src/sentry/seer/code_review/endpoints/serializers/code_review_local.py

@@ -16,7 +16,7 @@ def validate_base_commit_sha(self, value):
         return value
 
 
-class CliBugPredictionRequestSerializer(serializers.Serializer):
+class CodeReviewLocalRequestSerializer(serializers.Serializer):
     repository = RepositoryInfoSerializer(required=True)
     diff = serializers.CharField(required=True, max_length=500_000)
     current_branch = serializers.CharField(required=False, max_length=255)

src/sentry/seer/cli_bug_prediction.py src/sentry/seer/code_review_local.pysrc/sentry/seer/cli_bug_prediction.py renamed to src/sentry/seer/code_review_local.py

@@ -18,7 +18,7 @@
 )
 
 
-def trigger_cli_bug_prediction(
+def trigger_code_review_local(
     repo_provider: str,
     repo_owner: str,
     repo_name: str,
@@ -156,12 +156,12 @@ def trigger_cli_bug_prediction(
     return response_data
 
 
-def get_cli_bug_prediction_status(run_id: int) -> dict[str, Any]:
+def get_code_review_local_status(run_id: int) -> dict[str, Any]:
     """
     Get the status of a CLI bug prediction run.
 
     Args:
-        run_id: The Seer run ID from trigger_cli_bug_prediction
+        run_id: The Seer run ID from trigger_code_review_local
 
     Returns:
         dict with "status" key and optionally "predictions" and "diagnostics"

…/test_organization_cli_bug_prediction.py …view/endpoints/test_code_review_local.pytests/sentry/api/endpoints/test_organization_cli_bug_prediction.py renamed to tests/sentry/seer/code_review/endpoints/test_code_review_local.py tests/sentry/api/endpoints/test_organization_cli_bug_prediction.py renamed to tests/sentry/seer/code_review/endpoints/test_code_review_local.py

@@ -8,7 +8,7 @@
 
 
 @pytest.mark.django_db
-class OrganizationCliBugPredictionTest(APITestCase):
+class OrganizationCodeReviewLocalTest(APITestCase):
     endpoint = "sentry-api-0-organization-code-review-local"
     method = "post"
 
@@ -18,7 +18,7 @@ def setUp(self):
         self.project = self.create_project(organization=self.organization)
         self.repository = self.create_repo(
             project=self.project,
-            name="test-repo",
+            name="getsentry/test-repo",
             provider="github",
             external_id="12345",
         )
@@ -36,8 +36,8 @@ def setUp(self):
         self.login_as(user=self.user)
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_happy_path(self, mock_status, mock_trigger):
         """Test successful prediction request"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -176,7 +176,7 @@ def test_repository_not_found(self):
         assert "not found" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
     def test_seer_trigger_timeout(self, mock_trigger):
         """Test handling of Seer trigger timeout"""
         from urllib3.exceptions import TimeoutError
@@ -192,7 +192,7 @@ def test_seer_trigger_timeout(self, mock_trigger):
         assert "unavailable" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
     def test_seer_trigger_error(self, mock_trigger):
         """Test handling of Seer trigger error"""
         mock_trigger.side_effect = ValueError("Seer error")
@@ -203,12 +203,12 @@ def test_seer_trigger_error(self, mock_trigger):
             status_code=502,
         )
 
-        assert "Failed" in response.data["detail"]
+        assert "error" in response.data["detail"].lower()
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.time")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.time")
     def test_seer_polling_timeout(self, mock_time_module, mock_status, mock_trigger):
         """Test handling of polling timeout"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -234,8 +234,8 @@ def fake_time():
         assert "exceeded maximum processing time" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_seer_error_base_commit_not_found(self, mock_status, mock_trigger):
         """Test mapping of base commit not found error"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -253,8 +253,8 @@ def test_seer_error_base_commit_not_found(self, mock_status, mock_trigger):
         assert "pushed to the remote" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_seer_error_diff_too_large(self, mock_status, mock_trigger):
         """Test mapping of diff too large error"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -272,8 +272,8 @@ def test_seer_error_diff_too_large(self, mock_status, mock_trigger):
         assert "500KB" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_seer_error_too_many_files(self, mock_status, mock_trigger):
         """Test mapping of too many files error"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -291,8 +291,8 @@ def test_seer_error_too_many_files(self, mock_status, mock_trigger):
         assert "50 files" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_seer_error_clone_failed(self, mock_status, mock_trigger):
         """Test mapping of repository clone failed error"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -310,8 +310,8 @@ def test_seer_error_clone_failed(self, mock_status, mock_trigger):
         assert "permissions" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_seer_error_unknown(self, mock_status, mock_trigger):
         """Test mapping of unknown Seer error"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -358,8 +358,8 @@ def test_rate_limit_org(self, mock_is_limited):
         assert "Organization rate limit exceeded" in response.data["detail"]
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_optional_fields(self, mock_status, mock_trigger):
         """Test that optional fields are not required"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}

…i/test_cli_bug_prediction_integration.py …ew/test_code_review_local_integration.pytests/sentry/api/test_cli_bug_prediction_integration.py renamed to tests/sentry/seer/code_review/test_code_review_local_integration.py tests/sentry/api/test_cli_bug_prediction_integration.py renamed to tests/sentry/seer/code_review/test_code_review_local_integration.py

@@ -7,7 +7,7 @@
 
 
 @pytest.mark.django_db
-class CliBugPredictionIntegrationTest(APITestCase):
+class CodeReviewLocalIntegrationTest(APITestCase):
     """
     Integration tests for CLI bug prediction end-to-end flow.
 
@@ -23,7 +23,7 @@ def setUp(self):
         self.project = self.create_project(organization=self.organization)
         self.repository = self.create_repo(
             project=self.project,
-            name="test-repo",
+            name="getsentry/test-repo",
             provider="github",
             external_id="12345",
         )
@@ -39,8 +39,8 @@ def setUp(self):
         self.login_as(user=self.user)
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_end_to_end_single_poll(self, mock_status, mock_trigger):
         """Test end-to-end flow with immediate completion"""
         mock_trigger.return_value = {"run_id": 123, "status": "pending"}
@@ -75,8 +75,8 @@ def test_end_to_end_single_poll(self, mock_status, mock_trigger):
         assert len(response.data["predictions"]) == 1
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     @patch("time.sleep")
     def test_end_to_end_multiple_polls(self, mock_sleep, mock_status, mock_trigger):
         """Test end-to-end flow with multiple polling cycles"""
@@ -120,8 +120,8 @@ def test_end_to_end_multiple_polls(self, mock_sleep, mock_status, mock_trigger):
         assert response.data["predictions"][0]["severity"] == "high"
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     @patch("time.sleep")
     def test_status_check_network_error_recovery(self, mock_sleep, mock_status, mock_trigger):
         """Test that network errors during status check are retried"""
@@ -152,8 +152,8 @@ def test_status_check_network_error_recovery(self, mock_sleep, mock_status, mock
         assert response.data["status"] == "completed"
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_empty_predictions_response(self, mock_status, mock_trigger):
         """Test handling of completed status with no predictions"""
         mock_trigger.return_value = {"run_id": 999, "status": "pending"}
@@ -175,8 +175,8 @@ def test_empty_predictions_response(self, mock_status, mock_trigger):
         assert response.data["diagnostics"]["files_analyzed"] == 5
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     @patch("time.sleep")
     def test_multiple_predictions(self, mock_sleep, mock_status, mock_trigger):
         """Test handling of multiple predictions in response"""
@@ -222,8 +222,8 @@ def test_multiple_predictions(self, mock_sleep, mock_status, mock_trigger):
         assert response.data["predictions"][2]["severity"] == "low"
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     @patch("time.sleep")
     def test_seer_state_transition_pending_to_completed(
         self, mock_sleep, mock_status, mock_trigger
@@ -250,8 +250,8 @@ def test_seer_state_transition_pending_to_completed(
         assert response.data["status"] == "completed"
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     @patch("time.sleep")
     def test_seer_state_transition_with_in_progress(self, mock_sleep, mock_status, mock_trigger):
         """Test state transition: pending -> in_progress -> completed"""
@@ -277,8 +277,8 @@ def test_seer_state_transition_with_in_progress(self, mock_sleep, mock_status, m
         assert response.data["status"] == "completed"
 
     @with_feature("organizations:code-review-local")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.trigger_cli_bug_prediction")
-    @patch("sentry.api.endpoints.organization_cli_bug_prediction.get_cli_bug_prediction_status")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.trigger_code_review_local")
+    @patch("sentry.seer.code_review.endpoints.code_review_local.get_code_review_local_status")
     def test_diagnostics_included_in_response(self, mock_status, mock_trigger):
         """Test that diagnostics are properly included in response"""
         mock_trigger.return_value = {"run_id": 444, "status": "pending"}