From 224d6ce3582ed46e1f432e392f8e6504a543a37d Mon Sep 17 00:00:00 2001
From: Florent Chehab <flo@nextmile.fr>
Date: Mon, 6 Jul 2026 21:44:34 +0200
Subject: [PATCH] =?UTF-8?q?=E2=9C=A8(backend)=20use=20feature=20flag=20in?=
 =?UTF-8?q?=20call=20to=20summary-v2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Summary enabled is now infered directly from
the feature flag analytics backend.
---
 src/backend/core/recording/event/notification.py |  6 ++++--
 .../tests/recording/event/test_notification.py   | 16 +++++++++++++---
 2 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/src/backend/core/recording/event/notification.py b/src/backend/core/recording/event/notification.py
index cdbc253d..46bd1dc4 100644
--- a/src/backend/core/recording/event/notification.py
+++ b/src/backend/core/recording/event/notification.py
@@ -18,6 +18,7 @@ from asgiref.sync import async_to_sync
 from livekit import api as livekit_api
 
 from core import models, utils
+from core.analytics import UserFeatureFlag, is_user_feature_flag_enabled
 from core.utils import generate_download_s3_url
 
 logger = logging.getLogger(__name__)
@@ -371,8 +372,9 @@ class NotificationService:
                 ),
                 "download_link": f"{get_recording_download_base_url()}/{recording.id}",
                 "form_link": form_link,
-                # For now the feature flag logic is handled on summary side
-                "auto_create_summary": True,
+                "auto_create_summary": is_user_feature_flag_enabled(
+                    owner_access.user, UserFeatureFlag.TRANSCRIPT_SUMMARY_ENABLED
+                ),
             },
             "metadata": metadata_payload,
         }
diff --git a/src/backend/core/tests/recording/event/test_notification.py b/src/backend/core/tests/recording/event/test_notification.py
index 1d50a4f1..625c6da1 100644
--- a/src/backend/core/tests/recording/event/test_notification.py
+++ b/src/backend/core/tests/recording/event/test_notification.py
@@ -13,6 +13,7 @@ from django.contrib.sites.models import Site
 import pytest
 
 from core import factories, models
+from core.analytics import UserFeatureFlag
 from core.recording.event.notification import NotificationService, notification_service
 
 pytestmark = pytest.mark.django_db
@@ -312,7 +313,7 @@ def test_notify_summary_service_post_args_with_metadata(
             "user_email": owner.email,
             "title": 'Réunion "Engineering Sync" du 2026-01-02 à 11:30',
             "download_link": f"{settings.RECORDING_DOWNLOAD_BASE_URL}/{recording.id}",
-            "auto_create_summary": True,
+            "auto_create_summary": False,
             "form_link": None,
         },
         "metadata": {
@@ -343,10 +344,12 @@ def test_notify_summary_service_post_args_with_metadata(
 @mock.patch.object(
     NotificationService, "_get_recording_timestamps", new_callable=mock.AsyncMock
 )
+@pytest.mark.parametrize("auto_create_summary_enabled", [False, True])
 def test_notify_summary_service_post_args_without_metadata(
     mock_get_recording_timestamps,
     mock_generate_download_s3_url,
     mock_post,
+    auto_create_summary_enabled,
     settings,
 ):
     """Test summary notification computed request args when metadata is not available."""
@@ -376,7 +379,11 @@ def test_notify_summary_service_post_args_without_metadata(
     mock_response.json.return_value = {"job_id": "job-51"}
     mock_post.return_value = mock_response
 
-    result = NotificationService._notify_summary_service(recording)
+    with mock.patch(
+        "core.recording.event.notification.is_user_feature_flag_enabled",
+        return_value=auto_create_summary_enabled,
+    ) as mock_is_feature_flag_enabled:
+        result = NotificationService._notify_summary_service(recording)
 
     assert result is True
     expected_payload = {
@@ -389,7 +396,7 @@ def test_notify_summary_service_post_args_without_metadata(
             "user_email": owner.email,
             "title": "Transcription",
             "download_link": f"{settings.RECORDING_DOWNLOAD_BASE_URL}/{recording.id}",
-            "auto_create_summary": True,
+            "auto_create_summary": auto_create_summary_enabled,
             "form_link": None,
         },
         "metadata": None,
@@ -408,3 +415,6 @@ def test_notify_summary_service_post_args_without_metadata(
         recording.key, expires_in=60 * 60 * 24, override_domain=False
     )
     mock_get_recording_timestamps.assert_awaited_once_with(recording.worker_id)
+    mock_is_feature_flag_enabled.assert_called_once_with(
+        owner, UserFeatureFlag.TRANSCRIPT_SUMMARY_ENABLED
+    )