chore: remove graduated user-feedback AI feature flags

wedamija · claude · wedamija · commit d7f82d904cd1 · 2026-02-23T14:53:15.000-08:00
Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/src/sentry/features/temporary.py b/src/sentry/features/temporary.py
@@ -463,16 +463,6 @@ def register_temporary_features(manager: FeatureManager) -> None:
     manager.add("organizations:uptime-response-capture", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=False, default=True)
     # Enable runtime assertions for uptime monitors
     manager.add("organizations:uptime-runtime-assertions", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
-    # Enable endpoint and frontend for AI-powered UF summaries
-    manager.add("organizations:user-feedback-ai-summaries", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
-    # Enable org-level caching for user feedback summaries and categorization
-    manager.add("organizations:user-feedback-ai-summaries-cache", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE)
-    # Enable label generation at ingest time for user feedbacks
-    manager.add("organizations:user-feedback-ai-categorization", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=False)
-    # Enable the backend endpoint and frontend for categorizing user feedbacks
-    manager.add("organizations:user-feedback-ai-categorization-features", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
-    # Enable AI-powered title generation for user feedback
-    manager.add("organizations:user-feedback-ai-titles", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
     # Enable auto spam classification at User Feedback ingest time
     manager.add("organizations:user-feedback-spam-ingest", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
     # Enable view hierarchies options
diff --git a/src/sentry/feedback/endpoints/organization_feedback_categories.py b/src/sentry/feedback/endpoints/organization_feedback_categories.py
@@ -7,7 +7,6 @@
 from rest_framework.response import Response
 from urllib3 import Retry
 
-from sentry import features
 from sentry.api.api_owners import ApiOwner
 from sentry.api.api_publish_status import ApiPublishStatus
 from sentry.api.base import region_silo_endpoint
@@ -117,11 +116,7 @@ def get(self, request: Request, organization: Organization) -> Response:
         :auth: required
         """
 
-        if not features.has(
-            "organizations:user-feedback-ai-categorization-features",
-            organization,
-            actor=request.user,
-        ) or not has_seer_access(organization, actor=request.user):
+        if not has_seer_access(organization, actor=request.user):
             return Response(
                 {"detail": "AI categorization is not available for this organization."}, status=403
             )
@@ -149,20 +144,15 @@ def get(self, request: Request, organization: Organization) -> Response:
             # Day granularity date range. Date range is long enough that the categories won't change much (as long as the same day is selected)
             categorization_cache_key = f"feedback_categorization:{organization.id}:{start.strftime('%Y-%m-%d')}:{end.strftime('%Y-%m-%d')}:{hashed_project_ids}"
 
-        has_cache = features.has(
-            "organizations:user-feedback-ai-summaries-cache", organization, actor=request.user
-        )
-
-        if has_cache:
-            cache_entry = cache.get(categorization_cache_key)
-            if cache_entry:
-                return Response(
-                    {
-                        "categories": cache_entry["categories"],
-                        "success": True,
-                        "numFeedbacksContext": cache_entry["numFeedbacksContext"],
-                    }
-                )
+        cache_entry = cache.get(categorization_cache_key)
+        if cache_entry:
+            return Response(
+                {
+                    "categories": cache_entry["categories"],
+                    "success": True,
+                    "numFeedbacksContext": cache_entry["numFeedbacksContext"],
+                }
+            )
 
         recent_feedbacks = query_recent_feedbacks_with_ai_labels(
             organization_id=organization.id,
@@ -327,12 +317,11 @@ def get(self, request: Request, organization: Organization) -> Response:
         categories.sort(key=lambda x: x["feedbackCount"], reverse=True)
         categories = categories[:MAX_RETURN_CATEGORIES]
 
-        if has_cache:
-            cache.set(
-                categorization_cache_key,
-                {"categories": categories, "numFeedbacksContext": len(context_feedbacks)},
-                timeout=CATEGORIES_CACHE_TIMEOUT,
-            )
+        cache.set(
+            categorization_cache_key,
+            {"categories": categories, "numFeedbacksContext": len(context_feedbacks)},
+            timeout=CATEGORIES_CACHE_TIMEOUT,
+        )
 
         return Response(
             {
diff --git a/src/sentry/feedback/endpoints/organization_feedback_summary.py b/src/sentry/feedback/endpoints/organization_feedback_summary.py
@@ -6,7 +6,6 @@
 from rest_framework.response import Response
 from urllib3 import Retry
 
-from sentry import features
 from sentry.api.api_owners import ApiOwner
 from sentry.api.api_publish_status import ApiPublishStatus
 from sentry.api.base import region_silo_endpoint
@@ -90,9 +89,7 @@ def get(self, request: Request, organization: Organization) -> Response:
         :auth: required
         """
 
-        if not features.has(
-            "organizations:user-feedback-ai-summaries", organization, actor=request.user
-        ) or not has_seer_access(organization, actor=request.user):
+        if not has_seer_access(organization, actor=request.user):
             return Response(
                 {"detail": "AI summaries are not available for this organization."}, status=403
             )
@@ -113,22 +110,18 @@ def get(self, request: Request, organization: Organization) -> Response:
         project_ids = [str(project_id) for project_id in numeric_project_ids]
         hashed_project_ids = hash_from_values(project_ids)
 
-        has_cache = features.has(
-            "organizations:user-feedback-ai-summaries-cache", organization, actor=request.user
-        )
         summary_cache_key = f"feedback_summary:{organization.id}:{start.strftime('%Y-%m-%d-%H')}:{end.strftime('%Y-%m-%d-%H')}:{hashed_project_ids}"
 
-        if has_cache:
-            # Hour granularity date range.
-            summary_cache = cache.get(summary_cache_key)
-            if summary_cache:
-                return Response(
-                    {
-                        "summary": summary_cache["summary"],
-                        "success": True,
-                        "numFeedbacksUsed": summary_cache["numFeedbacksUsed"],
-                    }
-                )
+        # Hour granularity date range.
+        summary_cache = cache.get(summary_cache_key)
+        if summary_cache:
+            return Response(
+                {
+                    "summary": summary_cache["summary"],
+                    "success": True,
+                    "numFeedbacksUsed": summary_cache["numFeedbacksUsed"],
+                }
+            )
 
         filters = {
             "type": FeedbackGroup.type_id,
@@ -172,12 +165,11 @@ def get(self, request: Request, organization: Organization) -> Response:
                 {"detail": "Failed to generate a summary for a list of feedbacks"}, status=500
             )
 
-        if has_cache:
-            cache.set(
-                summary_cache_key,
-                {"summary": summary, "numFeedbacksUsed": len(feedback_msgs)},
-                timeout=SUMMARY_CACHE_TIMEOUT,
-            )
+        cache.set(
+            summary_cache_key,
+            {"summary": summary, "numFeedbacksUsed": len(feedback_msgs)},
+            timeout=SUMMARY_CACHE_TIMEOUT,
+        )
 
         return Response(
             {
diff --git a/src/sentry/feedback/usecases/ingest/create_feedback.py b/src/sentry/feedback/usecases/ingest/create_feedback.py
@@ -8,7 +8,7 @@
 
 import jsonschema
 
-from sentry import features, options
+from sentry import options
 from sentry.constants import DataCategory
 from sentry.feedback.lib.utils import UNREAL_FEEDBACK_UNATTENDED_MESSAGE, FeedbackCreationSource
 from sentry.feedback.usecases.label_generation import (
@@ -298,9 +298,7 @@ def create_feedback_issue(
     )
     issue_fingerprint = [uuid4().hex]
 
-    use_ai_title = should_query_seer and features.has(
-        "organizations:user-feedback-ai-titles", project.organization
-    )
+    use_ai_title = should_query_seer
     title = truncate_feedback_title(
         get_feedback_title(feedback_message, project.organization_id, use_ai_title)
     )
@@ -335,9 +333,7 @@ def create_feedback_issue(
     )
 
     # Generating labels using Seer, which will later be used to categorize feedbacks
-    if should_query_seer and features.has(
-        "organizations:user-feedback-ai-categorization", project.organization
-    ):
+    if should_query_seer:
         try:
             labels = generate_labels(feedback_message, project.organization_id)
             # This will rarely happen unless the user writes a really long feedback message
diff --git a/static/app/components/feedback/feedbackItem/feedbackItemUsername.spec.tsx b/static/app/components/feedback/feedbackItem/feedbackItemUsername.spec.tsx
@@ -148,7 +148,7 @@ describe('FeedbackItemUsername', () => {
 
       render(<FeedbackItemUsername feedbackIssue={issue} />, {
         organization: {
-          features: ['user-feedback-ai-titles', 'gen-ai-features'],
+          features: ['gen-ai-features'],
         },
       });
 
@@ -168,17 +168,12 @@ describe('FeedbackItemUsername', () => {
     it.each([
       {
         description: 'AI features are disabled',
-        features: ['user-feedback-ai-titles'],
-        summary: 'Login issue with payment flow',
-      },
-      {
-        description: 'user feedback AI titles are disabled',
-        features: ['gen-ai-features'],
+        features: [] as string[],
         summary: 'Login issue with payment flow',
       },
       {
         description: 'AI features enabled but summary is null',
-        features: ['user-feedback-ai-titles', 'gen-ai-features'],
+        features: ['gen-ai-features'],
         summary: null,
       },
     ])(
diff --git a/static/app/components/feedback/feedbackItem/feedbackItemUsername.tsx b/static/app/components/feedback/feedbackItem/feedbackItemUsername.tsx
@@ -32,8 +32,7 @@ export default function FeedbackItemUsername({className, feedbackIssue, style}:
   const user = name && email && !isSameNameAndEmail ? `${name} <${email}>` : nameOrEmail;
 
   const summary = feedbackIssue.metadata.summary;
-  const isAiSummaryEnabled =
-    areAiFeaturesAllowed && organization.features.includes('user-feedback-ai-titles');
+  const isAiSummaryEnabled = areAiFeaturesAllowed;
 
   const userNodeId = useId();
 
diff --git a/static/app/components/feedback/summaryCategories/feedbackSummaryCategories.tsx b/static/app/components/feedback/summaryCategories/feedbackSummaryCategories.tsx
@@ -10,25 +10,17 @@ import FeedbackSummary from 'sentry/components/feedback/summaryCategories/feedba
 import FeedbackButton from 'sentry/components/feedbackButton/feedbackButton';
 import {IconThumb} from 'sentry/icons';
 import {t} from 'sentry/locale';
-import useOrganization from 'sentry/utils/useOrganization';
 import {useSyncedLocalStorageState} from 'sentry/utils/useSyncedLocalStorageState';
 
 export default function FeedbackSummaryCategories() {
-  const organization = useOrganization();
-
   const {areAiFeaturesAllowed} = useOrganizationSeerSetup();
 
   const [isExpanded, setIsExpanded] = useSyncedLocalStorageState(
     'user-feedback-ai-summary-categories-expanded',
     true
   );
 
-  const showSummaryCategories =
-    (organization.features.includes('user-feedback-ai-summaries') ||
-      organization.features.includes('user-feedback-ai-categorization-features')) &&
-    areAiFeaturesAllowed;
-
-  if (!showSummaryCategories) {
+  if (!areAiFeaturesAllowed) {
     return null;
   }
 
@@ -78,12 +70,8 @@ export default function FeedbackSummaryCategories() {
         </Disclosure.Title>
         <Disclosure.Content>
           <Stack gap="md" width="100%">
-            {organization.features.includes('user-feedback-ai-summaries') && (
-              <FeedbackSummary />
-            )}
-            {organization.features.includes(
-              'user-feedback-ai-categorization-features'
-            ) && <FeedbackCategories />}
+            <FeedbackSummary />
+            <FeedbackCategories />
           </Stack>
         </Disclosure.Content>
       </Disclosure>
diff --git a/tests/sentry/feedback/endpoints/test_organization_feedback_categories.py b/tests/sentry/feedback/endpoints/test_organization_feedback_categories.py
@@ -22,9 +22,6 @@ def setUp(self) -> None:
         self.org = self.organization
         self.project1 = self.project
         self.project2 = self.create_project(teams=[self.team])
-        self.features = {
-            "organizations:user-feedback-ai-categorization-features": True,
-        }
         self.url = reverse(
             self.endpoint,
             kwargs={"organization_id_or_slug": self.org.slug},
@@ -74,15 +71,10 @@ def _create_feedback(
         )
         create_feedback_issue(event, project, FeedbackCreationSource.NEW_FEEDBACK_ENVELOPE)
 
-    def test_get_feedback_categories_without_feature_flag(self) -> None:
-        response = self.get_error_response(self.org.slug)
-        assert response.status_code == 403
-
     def test_get_feedback_categories_without_seer_access(self) -> None:
         self.mock_has_seer_access.return_value = False
-        with self.feature(self.features):
-            response = self.get_error_response(self.org.slug)
-            assert response.status_code == 403
+        response = self.get_error_response(self.org.slug)
+        assert response.status_code == 403
 
     def test_get_feedback_categories_basic(self) -> None:
         self._create_feedback("a", ["User Interface", "Speed"], self.project1)
@@ -105,8 +97,7 @@ def test_get_feedback_categories_basic(self) -> None:
             },
         )
 
-        with self.feature(self.features):
-            response = self.get_success_response(self.org.slug)
+        response = self.get_success_response(self.org.slug)
 
         assert response.data["success"] is True
         assert response.data["numFeedbacksContext"] == 4
@@ -148,8 +139,7 @@ def test_get_feedback_categories_with_project_filter(self) -> None:
             },
         )
 
-        with self.feature(self.features):
-            response = self.get_success_response(self.org.slug, project=[self.project1.id])
+        response = self.get_success_response(self.org.slug, project=[self.project1.id])
 
         assert response.data["success"] is True
         assert response.data["numFeedbacksContext"] == 2
@@ -189,8 +179,7 @@ def test_max_group_labels_limit(self) -> None:
             },
         )
 
-        with self.feature(self.features):
-            response = self.get_success_response(self.org.slug)
+        response = self.get_success_response(self.org.slug)
 
         assert response.data["success"] is True
         categories = response.data["categories"]
@@ -221,8 +210,7 @@ def test_filter_invalid_associated_labels_by_count_ratio(self) -> None:
             },
         )
 
-        with self.feature(self.features):
-            response = self.get_success_response(self.org.slug)
+        response = self.get_success_response(self.org.slug)
 
         assert response.data["success"] is True
         categories = response.data["categories"]
@@ -235,8 +223,7 @@ def test_seer_request_error(self) -> None:
         self._create_feedback("a", ["User Interface", "Issues UI"], self.project1)
         self.mock_make_signed_seer_api_request.side_effect = Exception("seer failed")
 
-        with self.feature(self.features):
-            response = self.get_error_response(self.org.slug)
+        response = self.get_error_response(self.org.slug)
 
         assert response.status_code == 500
         assert response.data["detail"] == "Failed to generate user feedback label groups"
@@ -248,8 +235,7 @@ def test_seer_http_errors(self) -> None:
                 status=status, json_data={"detail": "seer failed"}
             )
 
-            with self.feature(self.features):
-                response = self.get_error_response(self.org.slug)
+            response = self.get_error_response(self.org.slug)
 
             assert response.status_code == 500
             assert response.data["detail"] == "Failed to generate user feedback label groups"
@@ -263,8 +249,7 @@ def test_fallback_to_primary_labels_when_below_threshold(self) -> None:
         ):
             self._create_feedback("a", ["User Interface", "Usability"], self.project1)
 
-            with self.feature(self.features):
-                response = self.get_success_response(self.org.slug)
+            response = self.get_success_response(self.org.slug)
 
             assert self.mock_make_signed_seer_api_request.call_count == 0
 
diff --git a/tests/sentry/feedback/endpoints/test_organization_feedback_summary.py b/tests/sentry/feedback/endpoints/test_organization_feedback_summary.py
diff --git a/tests/sentry/feedback/usecases/ingest/test_create_feedback.py b/tests/sentry/feedback/usecases/ingest/test_create_feedback.py
