From ba65d42940afc7a7e3b35a5fb5b310bd5bfe8810 Mon Sep 17 00:00:00 2001
From: Roman Rizzi <roman@discourse.org>
Date: Thu, 18 May 2023 12:34:49 -0300
Subject: [PATCH] FIX: Hide summarization button from anons (#70)

---
 assets/javascripts/initializers/topic-summary.js   | 14 +++++++++-----
 .../summarization/common/summarization.scss        |  4 ----
 .../summarization/desktop/summarization.scss       |  3 +++
 .../summarization/mobile/summarization.scss        |  9 +++++++++
 plugin.rb                                          |  4 ++++
 5 files changed, 25 insertions(+), 9 deletions(-)
 create mode 100644 assets/stylesheets/modules/summarization/desktop/summarization.scss
 create mode 100644 assets/stylesheets/modules/summarization/mobile/summarization.scss

diff --git a/assets/javascripts/initializers/topic-summary.js b/assets/javascripts/initializers/topic-summary.js
index d14a6792..cc3174d5 100644
--- a/assets/javascripts/initializers/topic-summary.js
+++ b/assets/javascripts/initializers/topic-summary.js
@@ -33,12 +33,16 @@ export default {
   name: "discourse-ai-topic-summary",
 
   initialize(container) {
-    const settings = container.lookup("service:site-settings");
-    const summarizationEnabled =
-      settings.discourse_ai_enabled && settings.ai_summarization_enabled;
+    const user = container.lookup("service:current-user");
 
-    if (summarizationEnabled) {
-      withPluginApi("1.6.0", initializeTopicSummary);
+    if (user) {
+      const settings = container.lookup("service:site-settings");
+      const summarizationEnabled =
+        settings.discourse_ai_enabled && settings.ai_summarization_enabled;
+
+      if (summarizationEnabled) {
+        withPluginApi("1.6.0", initializeTopicSummary);
+      }
     }
   },
 };
diff --git a/assets/stylesheets/modules/summarization/common/summarization.scss b/assets/stylesheets/modules/summarization/common/summarization.scss
index b34ee0f2..d1bfce31 100644
--- a/assets/stylesheets/modules/summarization/common/summarization.scss
+++ b/assets/stylesheets/modules/summarization/common/summarization.scss
@@ -8,7 +8,3 @@
     min-height: 200px;
   }
 }
-
-.topic-ai-summarization {
-  margin-left: 10px;
-}
diff --git a/assets/stylesheets/modules/summarization/desktop/summarization.scss b/assets/stylesheets/modules/summarization/desktop/summarization.scss
new file mode 100644
index 00000000..8d9a425c
--- /dev/null
+++ b/assets/stylesheets/modules/summarization/desktop/summarization.scss
@@ -0,0 +1,3 @@
+.topic-ai-summarization {
+  margin-left: 10px;
+}
diff --git a/assets/stylesheets/modules/summarization/mobile/summarization.scss b/assets/stylesheets/modules/summarization/mobile/summarization.scss
new file mode 100644
index 00000000..e40c1126
--- /dev/null
+++ b/assets/stylesheets/modules/summarization/mobile/summarization.scss
@@ -0,0 +1,9 @@
+.toggle-summary {
+  button {
+    width: 100%;
+  }
+
+  .topic-ai-summarization {
+    margin-top: 10px;
+  }
+}
diff --git a/plugin.rb b/plugin.rb
index aa1ea37e..6d6bb82f 100644
--- a/plugin.rb
+++ b/plugin.rb
@@ -13,7 +13,11 @@ gem "tiktoken_ruby", "0.0.5", platform: RUBY_PLATFORM
 enabled_site_setting :discourse_ai_enabled
 
 register_asset "stylesheets/modules/ai-helper/common/ai-helper.scss"
+
 register_asset "stylesheets/modules/summarization/common/summarization.scss"
+register_asset "stylesheets/modules/summarization/desktop/summarization.scss", :desktop
+register_asset "stylesheets/modules/summarization/mobile/summarization.scss", :mobile
+
 register_asset "stylesheets/modules/ai-bot/common/bot-replies.scss"
 
 module ::DiscourseAi