fix: resolve issue with Xpert unit summary toggle not updating state

Author: Alam-2U

CHANGELOG.rst

@@ -14,6 +14,11 @@ Change Log
 Unreleased
 **********
 
+3.8.6 - 2026-03-24
+**********************************************
+* Fixed is_summary_enabled() to properly check course-level settings when no unit record exists
+* This ensures course-level toggle correctly controls summary button visibility
+
 3.8.5 - 2026-02-10
 **********************************************
 * Added client ID support for unit summary feature

ai_aside/__init__.py

@@ -2,6 +2,6 @@
 A plugin containing xblocks and apps supporting GPT and other LLM use on edX.
 """
 
-__version__ = '3.8.5'
+__version__ = '3.8.6'
 
 default_app_config = "ai_aside.apps.AiAsideConfig"

ai_aside/config_api/api.py

@@ -146,21 +146,23 @@ def is_summary_enabled(course_key, unit_key=None):
 
     enabled_by_default = django_settings.SUMMARY_ENABLED_BY_DEFAULT is True
 
+    # Check unit-level override if unit_key provided
     if unit_key is not None:
         try:
             unit = _get_unit(course_key, unit_key)
-
             if unit is not None:
                 return unit.enabled
         except AiAsideNotFoundException:
-            return enabled_by_default
+            # No unit-level setting, fall through to check course-level
+            pass
 
+    # Check course-level setting
     try:
         course = _get_course(course_key)
+        if course is not None:
+            return course.enabled
     except AiAsideNotFoundException:
-        return enabled_by_default
-
-    if course is not None:
-        return course.enabled
+        # No course-level setting, use default
+        pass
 
     return enabled_by_default

tests/api/test_api.py

@@ -289,7 +289,7 @@ def test_is_summary_enabled_fallback(self, mock_enabled):
             enabled=False,
         )
 
-        self.assertFalse(is_summary_enabled(course_key_true, unit_key_non_existent))
+        self.assertTrue(is_summary_enabled(course_key_true, unit_key_non_existent))
         self.assertFalse(is_summary_enabled(course_key_false, unit_key_non_existent))
         self.assertFalse(is_summary_enabled(course_key_non_existent, unit_key_non_existent))
 
@@ -337,7 +337,7 @@ def test_is_summary_enabled_disabled_feature_flag_default_true(self, mock_enable
         self.assertTrue(is_summary_enabled(course_key_true))
         self.assertFalse(is_summary_enabled(course_key_false))
         self.assertTrue(is_summary_enabled(course_key_true, unit_key_non_existent))
-        self.assertTrue(is_summary_enabled(course_key_false, unit_key_non_existent))
+        self.assertFalse(is_summary_enabled(course_key_false, unit_key_non_existent))
         self.assertTrue(is_summary_enabled(course_key_non_existent, unit_key_non_existent))
 
     def test_reset_course_unit_settings(self):