ref(autopilot): Limit missing integration detector to JS and python (#107398)

For other platforms, we cannot rely on the docs yet to give us the
information we need.
Fixed docs URL for python.

Author: ArthurKnaus

src/sentry/autopilot/tasks.py

@@ -5,6 +5,7 @@
 from itertools import chain, groupby
 from typing import Any
 
+from django.db.models import Q
 from django.utils import timezone
 from packaging import version
 from pydantic import BaseModel
@@ -30,6 +31,12 @@
 logger = logging.getLogger(__name__)
 
 
+class SupportedPlatformPrefix(StrEnum):
+    JAVASCRIPT = "javascript"
+    NODE = "node"
+    PYTHON = "python"
+
+
 class AutopilotDetectorName(StrEnum):
     SDK_UPDATE = "sdk-update"
     MISSING_SDK_INTEGRATION = "missing-sdk-integration"
@@ -266,32 +273,30 @@ def run_missing_sdk_integration_detector() -> None:
 
 
 def run_missing_sdk_integration_detector_for_organization(organization: Organization) -> None:
-    projects = Project.objects.filter(organization=organization).all()
-
-    if len(projects) == 0:
-        return
-
-    for project in projects:
-        # Get the repository mapped to this project via RepositoryProjectPathConfig
-        repo_config = (
-            RepositoryProjectPathConfig.objects.filter(
-                project=project,
-                repository__status=ObjectStatus.ACTIVE,
-            )
-            .select_related("repository")
-            .first()
+    platform_filter = Q()
+    for prefix in SupportedPlatformPrefix:
+        platform_filter |= Q(project__platform__startswith=prefix)
+
+    repo_configs = (
+        RepositoryProjectPathConfig.objects.filter(
+            platform_filter,
+            project__organization=organization,
+            repository__status=ObjectStatus.ACTIVE,
         )
+        .select_related("repository")
+        .values("project_id", "repository__name", "source_root")
+    )
 
-        if repo_config:
-            run_missing_sdk_integration_detector_for_project_task.apply_async(
-                args=(
-                    organization.id,
-                    project.id,
-                    repo_config.repository.name,
-                    repo_config.source_root,
-                ),
-                headers={"sentry-propagate-traces": False},
-            )
+    for config in repo_configs:
+        run_missing_sdk_integration_detector_for_project_task.apply_async(
+            args=(
+                organization.id,
+                config["project_id"],
+                config["repository__name"],
+                config["source_root"],
+            ),
+            headers={"sentry-propagate-traces": False},
+        )
 
 
 @instrumented_task(
@@ -344,11 +349,12 @@ def run_missing_sdk_integration_detector_for_project_task(
     # Get docs URL from platform data
     platform_data = INTEGRATION_ID_TO_PLATFORM_DATA.get(project.platform or "", {})
     docs_url = platform_data.get("link", None)
-    integration_docs_url = (
-        f"{docs_url}configuration/integrations/"
-        if docs_url
-        else "https://docs.sentry.io/platforms/"
-    )
+    if project.platform and project.platform.startswith(SupportedPlatformPrefix.PYTHON):
+        integration_docs_url = "https://docs.sentry.io/platforms/python/integrations/"
+    elif docs_url:
+        integration_docs_url = f"{docs_url}configuration/integrations/"
+    else:
+        integration_docs_url = "https://docs.sentry.io/platforms/"
 
     prompt = f"""# Objective
 Find missing Sentry SDK integrations for the project `{project.slug}` in repository `{repo_name}`.
@@ -420,7 +426,7 @@ def run_missing_sdk_integration_detector_for_project_task(
         )
         with metrics.timer(
             "autopilot.missing_sdk_integration_detector.run_duration",
-            tags={"organization_id": organization.id, "project_slug": project.slug},
+            tags={"project_slug": project.slug},
             sample_rate=1.0,
         ):
             state = client.get_run(run_id, blocking=True, poll_timeout=240.0, poll_interval=5.0)

tests/sentry/autopilot/test_tasks.py

@@ -294,6 +294,8 @@ def _create_code_mapping(self, project, repo_name: str):
     def test_skips_project_without_repository_mapping(
         self, mock_apply_async: mock.MagicMock
     ) -> None:
+        self.project.platform = "python"
+        self.project.save()
         run_missing_sdk_integration_detector_for_organization(self.organization)
         # No task should be spawned for projects without code mappings
         assert not mock_apply_async.called
@@ -303,6 +305,8 @@ def test_skips_project_without_repository_mapping(
         "sentry.autopilot.tasks.run_missing_sdk_integration_detector_for_project_task.apply_async"
     )
     def test_skips_inactive_repositories(self, mock_apply_async: mock.MagicMock) -> None:
+        self.project.platform = "python"
+        self.project.save()
         # Create a code mapping with an inactive repository
         code_mapping = self._create_code_mapping(self.project, "inactive-repo")
         code_mapping.repository.status = ObjectStatus.PENDING_DELETION
@@ -318,6 +322,8 @@ def test_skips_inactive_repositories(self, mock_apply_async: mock.MagicMock) ->
         "sentry.autopilot.tasks.run_missing_sdk_integration_detector_for_project_task.apply_async"
     )
     def test_spawns_task_for_project_with_mapping(self, mock_apply_async: mock.MagicMock) -> None:
+        self.project.platform = "python"
+        self.project.save()
         self._create_code_mapping(self.project, "test-repo")
 
         run_missing_sdk_integration_detector_for_organization(self.organization)
@@ -328,6 +334,62 @@ def test_spawns_task_for_project_with_mapping(self, mock_apply_async: mock.Magic
             headers={"sentry-propagate-traces": False},
         )
 
+    @pytest.mark.django_db
+    @mock.patch(
+        "sentry.autopilot.tasks.run_missing_sdk_integration_detector_for_project_task.apply_async"
+    )
+    def test_only_processes_supported_platforms(self, mock_apply_async: mock.MagicMock) -> None:
+        # Create projects with supported platforms
+        python_project = self.create_project(organization=self.organization, platform="python")
+        node_project = self.create_project(organization=self.organization, platform="node")
+        js_project = self.create_project(organization=self.organization, platform="javascript")
+        js_react_project = self.create_project(
+            organization=self.organization, platform="javascript-react"
+        )
+        python_django_project = self.create_project(
+            organization=self.organization, platform="python-django"
+        )
+        node_express_project = self.create_project(
+            organization=self.organization, platform="node-express"
+        )
+
+        # Create projects with unsupported platforms
+        go_project = self.create_project(organization=self.organization, platform="go")
+        ruby_project = self.create_project(organization=self.organization, platform="ruby")
+        java_project = self.create_project(organization=self.organization, platform="java")
+
+        # Create code mappings for all projects
+        self._create_code_mapping(python_project, "python-repo")
+        self._create_code_mapping(node_project, "node-repo")
+        self._create_code_mapping(js_project, "js-repo")
+        self._create_code_mapping(js_react_project, "js-react-repo")
+        self._create_code_mapping(python_django_project, "python-django-repo")
+        self._create_code_mapping(node_express_project, "node-express-repo")
+        self._create_code_mapping(go_project, "go-repo")
+        self._create_code_mapping(ruby_project, "ruby-repo")
+        self._create_code_mapping(java_project, "java-repo")
+
+        run_missing_sdk_integration_detector_for_organization(self.organization)
+
+        # Only supported platforms should have tasks spawned
+        assert mock_apply_async.call_count == 6
+
+        # Collect all project IDs that had tasks spawned
+        spawned_project_ids = {call[1]["args"][1] for call in mock_apply_async.call_args_list}
+
+        # Supported platforms should be included
+        assert python_project.id in spawned_project_ids
+        assert node_project.id in spawned_project_ids
+        assert js_project.id in spawned_project_ids
+        assert js_react_project.id in spawned_project_ids
+        assert python_django_project.id in spawned_project_ids
+        assert node_express_project.id in spawned_project_ids
+
+        # Unsupported platforms should NOT be included
+        assert go_project.id not in spawned_project_ids
+        assert ruby_project.id not in spawned_project_ids
+        assert java_project.id not in spawned_project_ids
+
 
 class TestRunMissingSdkIntegrationDetectorForProject(TestCase):
     @pytest.mark.django_db