chore(llm-detection): remove category/subcategory (#112531)

we do not use category or subcategory anymore for llm detected issues.

Author: shayna-ch

src/sentry/seer/issue_detection.py

@@ -42,8 +42,6 @@ def create_issue_occurrence(
             "title": issue.title,
             "trace_id": issue.trace_id,
             "project_id": effective_project_id,
-            "category": issue.category,
-            "subcategory": issue.subcategory,
         },
     )
 

src/sentry/tasks/llm_issue_detection/detection.py

@@ -84,8 +84,6 @@ class DetectedIssue(BaseModel):
     missing_telemetry: str | None = Field(None, max_length=MAX_LLM_FIELD_LENGTH)
     offender_span_ids: list[str]
     transaction_name: str = Field(..., max_length=MAX_LLM_FIELD_LENGTH)
-    category: str = Field(..., max_length=MAX_LLM_FIELD_LENGTH)
-    subcategory: str = Field(..., max_length=MAX_LLM_FIELD_LENGTH)
     verification_reason: str = Field(..., max_length=MAX_LLM_FIELD_LENGTH)
     group_for_fingerprint: str = Field(..., max_length=MAX_LLM_FIELD_LENGTH)
     project_id: int | None = None

tests/sentry/seer/test_issue_detection.py

@@ -24,8 +24,6 @@ def test_create_issue_occurrence_success(self, mock_create: Any) -> None:
             "missing_telemetry": None,
             "offender_span_ids": ["span1", "span2"],
             "title": "Test Issue",
-            "subcategory": "test",
-            "category": "performance",
             "verification_reason": "test reason",
             "trace_id": "abc123",
             "transaction_name": "/api/test",
@@ -62,8 +60,6 @@ def test_create_issue_occurrence_project_not_in_organization(self) -> None:
             "missing_telemetry": None,
             "offender_span_ids": ["span1", "span2"],
             "title": "Test Issue",
-            "subcategory": "test",
-            "category": "performance",
             "verification_reason": "test reason",
             "trace_id": "abc123",
             "transaction_name": "/api/test",
@@ -88,8 +84,6 @@ def test_create_issue_occurrence_uses_issue_project_id(self, mock_create: Any) -
             "missing_telemetry": None,
             "offender_span_ids": ["span1", "span2"],
             "title": "Test Issue",
-            "subcategory": "test",
-            "category": "performance",
             "verification_reason": "test reason",
             "trace_id": "abc123",
             "transaction_name": "/api/test",
@@ -120,8 +114,6 @@ def test_create_issue_occurrence_creation_fails(
             "missing_telemetry": None,
             "offender_span_ids": ["span1", "span2"],
             "title": "Test Issue",
-            "subcategory": "test",
-            "category": "performance",
             "verification_reason": "test reason",
             "trace_id": "abc123",
             "transaction_name": "/api/test",

tests/sentry/tasks/test_llm_issue_detection.py

@@ -89,8 +89,6 @@ def test_create_issue_occurrence_from_detection(self, mock_produce_occurrence):
             offender_span_ids=["span_1", "span_2"],
             trace_id="abc123xyz",
             transaction_name="test_transaction",
-            subcategory="Connection Pool Exhaustion",
-            category="Database",
             verification_reason="Problem is correctly identified",
             group_for_fingerprint="Slow Database Query",
         )
@@ -156,8 +154,6 @@ def test_create_issue_occurrence_uses_group_for_fingerprint_when_set(
             offender_span_ids=[],
             trace_id="trace456",
             transaction_name="GET /api",
-            subcategory="N+1",
-            category="Performance",
             verification_reason="Verified",
             group_for_fingerprint="N+1 Database Queries",
         )