feat(aci): Add ability to filter workflows by connected detectors

Author: malwilley

src/sentry/workflow_engine/endpoints/organization_workflow_index.py

@@ -121,6 +121,16 @@ def filter_workflows(self, request: Request, organization: Organization) -> Quer
             # If specific IDs are provided, skip query and project filtering
             return queryset
 
+        if raw_detectorlist := request.GET.getlist("detector"):
+            try:
+                detector_ids = [int(id) for id in raw_detectorlist]
+            except ValueError:
+                raise ValidationError({"detector": ["Invalid detector ID format"]})
+            queryset = queryset.filter(detectorworkflow__detector_id__in=detector_ids).distinct()
+
+            # If detector IDs are provided, skip query and project filtering
+            return queryset
+
         if raw_query := request.GET.get("query"):
             for filter in parse_workflow_query(raw_query):
                 assert isinstance(filter, SearchFilter)

tests/sentry/workflow_engine/endpoints/test_organization_workflow_index.py

@@ -346,6 +346,53 @@ def test_query_filter_by_action(self) -> None:
         assert len(response3.data) == 2
         assert {self.workflow.name, self.workflow_two.name} == {w["name"] for w in response3.data}
 
+    def test_filter_by_detector(self) -> None:
+        project_1 = self.create_project(organization=self.organization)
+        project_2 = self.create_project(organization=self.organization)
+        project_3 = self.create_project(organization=self.organization)
+
+        detector_1 = self.create_detector(project=project_1, name="Detector 1")
+        detector_2 = self.create_detector(project=project_2, name="Detector 2")
+        detector_3 = self.create_detector(project=project_3, name="Detector 3")
+
+        self.create_detector_workflow(workflow=self.workflow, detector=detector_1)
+        self.create_detector_workflow(workflow=self.workflow_two, detector=detector_2)
+        self.create_detector_workflow(workflow=self.workflow_three, detector=detector_3)
+
+        # Filter by single detector
+        response = self.get_success_response(
+            self.organization.slug,
+            qs_params={"detector": str(detector_1.id)},
+        )
+        assert len(response.data) == 1
+        assert response.data[0]["name"] == self.workflow.name
+
+        # Filter by multiple detectors
+        response2 = self.get_success_response(
+            self.organization.slug,
+            qs_params=[
+                ("detector", str(detector_1.id)),
+                ("detector", str(detector_2.id)),
+            ],
+        )
+        assert len(response2.data) == 2
+        assert {w["name"] for w in response2.data} == {self.workflow.name, self.workflow_two.name}
+
+        # Filter by non-existent detector ID returns no results
+        response3 = self.get_success_response(
+            self.organization.slug,
+            qs_params={"detector": "999999"},
+        )
+        assert len(response3.data) == 0
+
+        # Invalid detector ID format returns error
+        response4 = self.get_error_response(
+            self.organization.slug,
+            qs_params={"detector": "not-an-id"},
+            status_code=400,
+        )
+        assert response4.data == {"detector": ["Invalid detector ID format"]}
+
     def test_compound_query(self) -> None:
         self.create_detector_workflow(
             workflow=self.workflow, detector=self.create_detector(project=self.project)