address pr comments

Author: Zylphrex

src/sentry/seer/autofix/on_completion_hook.py

@@ -502,15 +502,6 @@ def _trigger_coding_agent_handoff(
                     "failures": len(result.get("failures", [])),
                 },
             )
-        except Group.DoesNotExist:
-            logger.exception(
-                "autofix.on_completion_hook.coding_agent_handoff_group_not_found",
-                extra={
-                    "run_id": run_id,
-                    "organization_id": organization.id,
-                    "group_id": group.id,
-                },
-            )
         except Exception:
             logger.exception(
                 "autofix.on_completion_hook.coding_agent_handoff_failed",

src/sentry/seer/endpoints/group_ai_autofix.py

@@ -4,7 +4,7 @@
 from typing import Any
 
 from drf_spectacular.utils import extend_schema
-from rest_framework import serializers
+from rest_framework import serializers, status
 from rest_framework.exceptions import PermissionDenied
 from rest_framework.request import Request
 from rest_framework.response import Response
@@ -225,7 +225,7 @@ def _post_explorer(self, request: Request, group: Group) -> Response:
         """Handle POST for Explorer-based autofix."""
         serializer = ExplorerAutofixRequestSerializer(data=request.data)
         if not serializer.is_valid():
-            return Response(serializer.errors, status=400)
+            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
         data = serializer.validated_data
         step = data.get("step", "root_cause")
@@ -241,12 +241,12 @@ def _post_explorer(self, request: Request, group: Group) -> Response:
                     {
                         "detail": "run_id and either integration_id or provider are required for coding_agent_handoff"
                     },
-                    status=400,
+                    status=status.HTTP_400_BAD_REQUEST,
                 )
             if integration_id and provider:
                 return Response(
                     {"detail": "Cannot specify both integration_id and provider"},
-                    status=400,
+                    status=status.HTTP_400_BAD_REQUEST,
                 )
 
             result = trigger_coding_agent_handoff(
@@ -256,16 +256,18 @@ def _post_explorer(self, request: Request, group: Group) -> Response:
                 provider=provider,
                 user_id=request.user.id if request.user else None,
             )
-            return Response(result, status=202)
+            return Response(result, status=status.HTTP_202_ACCEPTED)
 
         if step == "open_pr":
             if not run_id:
-                return Response({"detail": "run_id is required for open_pr"}, status=400)
+                return Response(
+                    {"detail": "run_id is required for open_pr"}, status=status.HTTP_400_BAD_REQUEST
+                )
             try:
                 trigger_push_changes(group, run_id)
-            except SeerPermissionError as e:
-                raise PermissionDenied(str(e))
-            return Response({"run_id": run_id}, status=202)
+            except SeerPermissionError:
+                return Response(status=status.HTTP_404_NOT_FOUND)
+            return Response({"run_id": run_id}, status=status.HTTP_202_ACCEPTED)
 
         # Handle all built-in Seer steps
         try:
@@ -278,15 +280,15 @@ def _post_explorer(self, request: Request, group: Group) -> Response:
                 intelligence_level=data["intelligence_level"],
                 user_context=data.get("user_context"),
             )
-            return Response({"run_id": run_id}, status=202)
+            return Response({"run_id": run_id}, status=status.HTTP_202_ACCEPTED)
         except SeerPermissionError as e:
             raise PermissionDenied(str(e))
 
     def _post_legacy(self, request: Request, group: Group) -> Response:
         """Handle POST for legacy autofix."""
         serializer = AutofixRequestSerializer(data=request.data)
         if not serializer.is_valid():
-            return Response(serializer.errors, status=400)
+            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
         data = serializer.validated_data