From 03eb1b8c878ceacc1b7ccffba704a10b968e8fba Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Mon, 3 Mar 2025 09:37:58 -0800
Subject: [PATCH 01/15] DEV: Conditionally show sentiment reports

---
 app/models/classification_result.rb           |  4 ++
 .../initializers/ai-sentiment-admin-nav.js    | 38 +++++++++++--------
 config/settings.yml                           |  3 ++
 lib/sentiment/entry_point.rb                  | 18 +++++++--
 4 files changed, 43 insertions(+), 20 deletions(-)

diff --git a/app/models/classification_result.rb b/app/models/classification_result.rb
index 8da1c16d6..1d3f7b12c 100644
--- a/app/models/classification_result.rb
+++ b/app/models/classification_result.rb
@@ -2,6 +2,10 @@
 
 class ClassificationResult < ActiveRecord::Base
   belongs_to :target, polymorphic: true
+
+  def self.has_sentiment_classification?
+    where(classification_type: "sentiment").exists?
+  end
 end
 
 # == Schema Information
diff --git a/assets/javascripts/initializers/ai-sentiment-admin-nav.js b/assets/javascripts/initializers/ai-sentiment-admin-nav.js
index 5b485e4e7..c73b1add6 100644
--- a/assets/javascripts/initializers/ai-sentiment-admin-nav.js
+++ b/assets/javascripts/initializers/ai-sentiment-admin-nav.js
@@ -1,21 +1,27 @@
 import { apiInitializer } from "discourse/lib/api";
 
-export default apiInitializer("1.15.0", (api) => {
-  const settings = api.container.lookup("service:site-settings");
+export default apiInitializer("1.15.0", async (api) => {
+  const currentUser = api.getCurrentUser();
 
-  if (settings.ai_sentiment_enabled) {
-    api.addAdminSidebarSectionLink("reports", {
-      name: "sentiment_overview",
-      route: "admin.dashboardSentiment",
-      label: "discourse_ai.sentiments.sidebar.overview",
-      icon: "chart-column",
-    });
-    api.addAdminSidebarSectionLink("reports", {
-      name: "sentiment_analysis",
-      route: "adminReports.show",
-      routeModels: ["sentiment_analysis"],
-      label: "discourse_ai.sentiments.sidebar.analysis",
-      icon: "chart-pie",
-    });
+  if (
+    !currentUser ||
+    !currentUser.admin ||
+    !currentUser.can_see_sentiment_reports
+  ) {
+    return;
   }
+
+  api.addAdminSidebarSectionLink("reports", {
+    name: "sentiment_overview",
+    route: "admin.dashboardSentiment",
+    label: "discourse_ai.sentiments.sidebar.overview",
+    icon: "chart-column",
+  });
+  api.addAdminSidebarSectionLink("reports", {
+    name: "sentiment_analysis",
+    route: "adminReports.show",
+    routeModels: ["sentiment_analysis"],
+    label: "discourse_ai.sentiments.sidebar.analysis",
+    icon: "chart-pie",
+  });
 });
diff --git a/config/settings.yml b/config/settings.yml
index 665c2c83a..2246fed7e 100644
--- a/config/settings.yml
+++ b/config/settings.yml
@@ -25,6 +25,9 @@ discourse_ai:
   ai_sentiment_backfill_post_max_age_days:
     default: 60
     hidden: true
+  ai_sentiment_reports_enabled:
+    default: true
+    client: true
 
   ai_openai_dall_e_3_url: "https://api.openai.com/v1/images/generations"
   ai_openai_embeddings_url:
diff --git a/lib/sentiment/entry_point.rb b/lib/sentiment/entry_point.rb
index 2325f81ee..be0ecdf8d 100644
--- a/lib/sentiment/entry_point.rb
+++ b/lib/sentiment/entry_point.rb
@@ -14,10 +14,20 @@ def inject_into(plugin)
         plugin.on(:post_created, &sentiment_analysis_cb)
         plugin.on(:post_edited, &sentiment_analysis_cb)
 
-        EmotionFilterOrder.register!(plugin)
-        EmotionDashboardReport.register!(plugin)
-        SentimentDashboardReport.register!(plugin)
-        SentimentAnalysisReport.register!(plugin)
+        plugin.add_to_serializer(
+          :current_user,
+          :can_see_sentiment_reports,
+          include_condition: -> do
+            SiteSetting.ai_sentiment_enabled && SiteSetting.ai_sentiment_reports_enabled
+          end,
+        ) { ClassificationResult.has_sentiment_classification? }
+
+        if ClassificationResult.has_sentiment_classification?
+          EmotionFilterOrder.register!(plugin)
+          EmotionDashboardReport.register!(plugin)
+          SentimentDashboardReport.register!(plugin)
+          SentimentAnalysisReport.register!(plugin)
+        end
       end
     end
   end

From 10c229e97a47b27d24611729fa4213d7aefe490a Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Mon, 3 Mar 2025 10:13:32 -0800
Subject: [PATCH 02/15] FIX

---
 lib/sentiment/entry_point.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/sentiment/entry_point.rb b/lib/sentiment/entry_point.rb
index be0ecdf8d..f70d8c6a5 100644
--- a/lib/sentiment/entry_point.rb
+++ b/lib/sentiment/entry_point.rb
@@ -22,7 +22,8 @@ def inject_into(plugin)
           end,
         ) { ClassificationResult.has_sentiment_classification? }
 
-        if ClassificationResult.has_sentiment_classification?
+        if ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled &&
+             SiteSetting.ai_sentiment_reports_enabled
           EmotionFilterOrder.register!(plugin)
           EmotionDashboardReport.register!(plugin)
           SentimentDashboardReport.register!(plugin)

From fb6d722c73edfb6b0583ea8ac211422d2c308e14 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Mon, 3 Mar 2025 15:38:01 -0800
Subject: [PATCH 03/15] FIX: infinite scroll not working

---
 .../admin-report-sentiment-analysis.gjs       | 22 +++++--------------
 1 file changed, 6 insertions(+), 16 deletions(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index a1ca1e6f9..e3817afe7 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -18,7 +18,7 @@ import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/d
 
 export default class AdminReportSentimentAnalysis extends Component {
   @tracked selectedChart = null;
-  @tracked posts = null;
+  @tracked posts = [];
   @tracked hasMorePosts = false;
   @tracked nextOffset = 0;
   @tracked showingSelectedChart = false;
@@ -71,20 +71,6 @@ export default class AdminReportSentimentAnalysis extends Component {
     }
   }
 
-  doughnutTitle(data) {
-    const MAX_TITLE_LENGTH = 18;
-    const title = data?.title || "";
-    const score = data?.total_score ? ` (${data.total_score})` : "";
-
-    if (title.length + score.length > MAX_TITLE_LENGTH) {
-      return (
-        title.substring(0, MAX_TITLE_LENGTH - score.length) + "..." + score
-      );
-    }
-
-    return title + score;
-  }
-
   async postRequest() {
     return await ajax("/discourse-ai/sentiment/posts", {
       data: {
@@ -217,7 +203,10 @@ export default class AdminReportSentimentAnalysis extends Component {
 
       this.hasMorePosts = response.has_more;
       this.nextOffset = response.next_offset;
-      return response.posts.map((post) => Post.create(post));
+
+      const mappedPosts = response.posts.map((post) => Post.create(post));
+      this.posts.pushObjects(mappedPosts);
+      return mappedPosts;
     } catch (e) {
       popupAjaxError(e);
     }
@@ -228,6 +217,7 @@ export default class AdminReportSentimentAnalysis extends Component {
     this.showingSelectedChart = false;
     this.selectedChart = null;
     this.activeFilter = "all";
+    this.posts = [];
   }
 
   <template>

From c53749b3fcdaf7ce79cd6443052fe7a05e9ed69b Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Mon, 3 Mar 2025 17:59:28 -0800
Subject: [PATCH 04/15] DEV: Selected charts have unique URL

---
 .../admin-report-sentiment-analysis.gjs       | 46 +++++++++++++++++++
 .../javascripts/initializers/admin-reports.js |  7 +++
 2 files changed, 53 insertions(+)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index e3817afe7..ea83168f1 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -3,6 +3,7 @@ import { tracked } from "@glimmer/tracking";
 import { fn, hash } from "@ember/helper";
 import { on } from "@ember/modifier";
 import { action } from "@ember/object";
+import { service } from "@ember/service";
 import { modifier } from "ember-modifier";
 import { and } from "truth-helpers";
 import DButton from "discourse/components/d-button";
@@ -15,8 +16,11 @@ import Post from "discourse/models/post";
 import closeOnClickOutside from "discourse/modifiers/close-on-click-outside";
 import { i18n } from "discourse-i18n";
 import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
+import didInsert from "@ember/render-modifiers/modifiers/did-insert";
 
 export default class AdminReportSentimentAnalysis extends Component {
+  @service router;
+
   @tracked selectedChart = null;
   @tracked posts = [];
   @tracked hasMorePosts = false;
@@ -31,6 +35,30 @@ export default class AdminReportSentimentAnalysis extends Component {
       .classList.add("active");
   });
 
+  @action
+  async openToChart() {
+    const queryParams = this.router.currentRoute.queryParams;
+    if (queryParams.selectedChart) {
+      this.selectedChart = this.transformedData.find(
+        (data) => data.title === queryParams.selectedChart
+      );
+
+      if (!this.selectedChart) {
+        return;
+      }
+      this.showingSelectedChart = true;
+
+      try {
+        const response = await this.postRequest();
+        this.posts = response.posts.map((post) => Post.create(post));
+        this.hasMorePosts = response.has_more;
+        this.nextOffset = response.next_offset;
+      } catch (e) {
+        popupAjaxError(e);
+      }
+    }
+  }
+
   clearActiveFilters(element) {
     const filterButtons = element.querySelectorAll("li button");
     for (let button of filterButtons) {
@@ -179,6 +207,14 @@ export default class AdminReportSentimentAnalysis extends Component {
       return;
     }
 
+    const currentQueryParams = this.router.currentRoute.queryParams;
+    this.router.transitionTo(this.router.currentRoute.name, {
+      queryParams: {
+        ...currentQueryParams,
+        selectedChart: data.title,
+      },
+    });
+
     this.selectedChart = data;
     this.showingSelectedChart = true;
 
@@ -218,9 +254,19 @@ export default class AdminReportSentimentAnalysis extends Component {
     this.selectedChart = null;
     this.activeFilter = "all";
     this.posts = [];
+
+    const currentQueryParams = this.router.currentRoute.queryParams;
+    this.router.transitionTo(this.router.currentRoute.name, {
+      queryParams: {
+        ...currentQueryParams,
+        selectedChart: null,
+      },
+    });
   }
 
   <template>
+    <span {{didInsert this.openToChart}}></span>
+
     {{#unless this.showingSelectedChart}}
       <div class="admin-report-sentiment-analysis">
         {{#each this.transformedData as |data|}}
diff --git a/assets/javascripts/initializers/admin-reports.js b/assets/javascripts/initializers/admin-reports.js
index 1dd1a45e7..9fc3fc7c0 100644
--- a/assets/javascripts/initializers/admin-reports.js
+++ b/assets/javascripts/initializers/admin-reports.js
@@ -22,6 +22,13 @@ export default {
         "sentiment_analysis",
         AdminReportSentimentAnalysis
       );
+
+      api.registerValueTransformer(
+        "admin-reports-show-query-params",
+        ({ value }) => {
+          return [...value, "selectedChart"];
+        }
+      );
     });
   },
 };

From d895b9dfbfebda5dcbf506d2fe3231dacedc3b67 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Tue, 4 Mar 2025 08:39:34 -0800
Subject: [PATCH 05/15] DEV: Replace emoji in topic titles

---
 .../discourse_ai/sentiment/sentiment_controller.rb          | 2 +-
 .../components/admin-report-sentiment-analysis.gjs          | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/controllers/discourse_ai/sentiment/sentiment_controller.rb b/app/controllers/discourse_ai/sentiment/sentiment_controller.rb
index 57fa4d331..220488ec3 100644
--- a/app/controllers/discourse_ai/sentiment/sentiment_controller.rb
+++ b/app/controllers/discourse_ai/sentiment/sentiment_controller.rb
@@ -37,7 +37,7 @@ def posts
           SELECT
             p.id AS post_id,
             p.topic_id,
-            t.title AS topic_title,
+            t.fancy_title AS topic_title,
             p.cooked as post_cooked,
             p.user_id,
             p.post_number,
diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index ea83168f1..37014f4c2 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -3,6 +3,7 @@ import { tracked } from "@glimmer/tracking";
 import { fn, hash } from "@ember/helper";
 import { on } from "@ember/modifier";
 import { action } from "@ember/object";
+import didInsert from "@ember/render-modifiers/modifiers/did-insert";
 import { service } from "@ember/service";
 import { modifier } from "ember-modifier";
 import { and } from "truth-helpers";
@@ -16,7 +17,7 @@ import Post from "discourse/models/post";
 import closeOnClickOutside from "discourse/modifiers/close-on-click-outside";
 import { i18n } from "discourse-i18n";
 import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
-import didInsert from "@ember/render-modifiers/modifiers/did-insert";
+import replaceEmoji from "discourse/helpers/replace-emoji";
 
 export default class AdminReportSentimentAnalysis extends Component {
   @service router;
@@ -147,10 +148,11 @@ export default class AdminReportSentimentAnalysis extends Component {
     }
 
     return this.posts.filter((post) => {
+      post.topic_title = replaceEmoji(post.topic_title);
+
       if (this.activeFilter === "all") {
         return true;
       }
-
       return post.sentiment === this.activeFilter;
     });
   }

From e0f4c751fd47865bd42c04cc343a12a7abbbba4c Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 09:41:13 -0800
Subject: [PATCH 06/15] DEV: Hide CSV export button

---
 .../discourse/components/admin-report-sentiment-analysis.gjs  | 2 +-
 assets/stylesheets/modules/sentiment/common/dashboard.scss    | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index 37014f4c2..cc5884d9b 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -11,13 +11,13 @@ import DButton from "discourse/components/d-button";
 import HorizontalOverflowNav from "discourse/components/horizontal-overflow-nav";
 import PostList from "discourse/components/post-list";
 import dIcon from "discourse/helpers/d-icon";
+import replaceEmoji from "discourse/helpers/replace-emoji";
 import { ajax } from "discourse/lib/ajax";
 import { popupAjaxError } from "discourse/lib/ajax-error";
 import Post from "discourse/models/post";
 import closeOnClickOutside from "discourse/modifiers/close-on-click-outside";
 import { i18n } from "discourse-i18n";
 import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
-import replaceEmoji from "discourse/helpers/replace-emoji";
 
 export default class AdminReportSentimentAnalysis extends Component {
   @service router;
diff --git a/assets/stylesheets/modules/sentiment/common/dashboard.scss b/assets/stylesheets/modules/sentiment/common/dashboard.scss
index b3e2ef74a..ffcf1bae1 100644
--- a/assets/stylesheets/modules/sentiment/common/dashboard.scss
+++ b/assets/stylesheets/modules/sentiment/common/dashboard.scss
@@ -48,6 +48,10 @@
     .control:nth-of-type(6):nth-last-of-type(3) {
       display: none;
     }
+
+    .control:has(.export-csv-btn) {
+      display: none;
+    }
   }
 
   .main {

From 57b1c01946b2fe8ff2ab13420457b4f9e281a100 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 10:17:39 -0800
Subject: [PATCH 07/15] DEV: Varying sized doughnuts

---
 .../discourse/components/doughnut-chart.gjs   | 52 ++++++++++++++++++-
 .../modules/sentiment/common/dashboard.scss   | 30 +++++------
 2 files changed, 62 insertions(+), 20 deletions(-)

diff --git a/assets/javascripts/discourse/components/doughnut-chart.gjs b/assets/javascripts/discourse/components/doughnut-chart.gjs
index 79c160764..0bc0a462c 100644
--- a/assets/javascripts/discourse/components/doughnut-chart.gjs
+++ b/assets/javascripts/discourse/components/doughnut-chart.gjs
@@ -1,9 +1,33 @@
 import Component from "@glimmer/component";
+import { tracked } from "@glimmer/tracking";
 import Chart from "admin/components/chart";
 
 export default class DoughnutChart extends Component {
+  @tracked canvasSize = null;
+
+  calculateRadius(postCount) {
+    const minPosts = 0;
+    const maxPosts = 100;
+    const minRadius = 30;
+    const maxRadius = 100;
+    const clampedPostCount = Math.min(Math.max(postCount, minPosts), maxPosts);
+    return (
+      minRadius +
+      ((clampedPostCount - minPosts) / (maxPosts - minPosts)) *
+        (maxRadius - minRadius)
+    );
+  }
+
   get config() {
     const totalScore = this.args.totalScore || "";
+    // const radius = this.calculateRadius(this.args.totalScore)
+    // Temporary for tesitng:
+    const radius = this.calculateRadius(Math.floor(Math.random() * (100 + 1)));
+
+    const paddingTop = 30;
+    const paddingBottom = 0;
+    const canvasSize = 2 * radius + paddingTop + paddingBottom;
+    this.canvasSize = canvasSize;
 
     return {
       type: "doughnut",
@@ -13,14 +37,25 @@ export default class DoughnutChart extends Component {
           {
             data: this.args.data,
             backgroundColor: this.args.colors,
+            cutout: "70%",
+            radius,
           },
         ],
       },
       options: {
         responsive: true,
+        maintainAspectRatio: false,
+        layout: {
+          padding: {
+            top: paddingTop,
+            left: 0,
+            right: 0,
+            bottom: 0,
+          },
+        },
         plugins: {
           legend: {
-            position: this.args.legendPosition || "bottom",
+            display: false,
           },
         },
       },
@@ -55,13 +90,26 @@ export default class DoughnutChart extends Component {
             ctx.save();
           },
         },
+        {
+          id: "resizeCanvas",
+          afterDraw: function (chart) {
+            const size = `${canvasSize}px`;
+            chart.canvas.style.width = size;
+            chart.canvas.style.height = size;
+
+            chart.resize();
+          },
+        },
       ],
     };
   }
 
   <template>
     {{#if this.config}}
-      <h3 class="doughnut-chart-title">{{@doughnutTitle}}</h3>
+      <h3
+        class="doughnut-chart-title"
+        style="max-width: {{this.canvasSize}}px"
+      >{{@doughnutTitle}}</h3>
       <Chart @chartConfig={{this.config}} class="admin-report-doughnut" />
     {{/if}}
   </template>
diff --git a/assets/stylesheets/modules/sentiment/common/dashboard.scss b/assets/stylesheets/modules/sentiment/common/dashboard.scss
index ffcf1bae1..8021669d6 100644
--- a/assets/stylesheets/modules/sentiment/common/dashboard.scss
+++ b/assets/stylesheets/modules/sentiment/common/dashboard.scss
@@ -39,11 +39,6 @@
       flex: 1;
     }
 
-    .control:nth-of-type(n + 6) {
-      flex-basis: 49%;
-      align-self: flex-end;
-    }
-
     // Hides tag selector when showing subcategories selector
     .control:nth-of-type(6):nth-last-of-type(3) {
       display: none;
@@ -59,7 +54,6 @@
     display: flex;
     flex-flow: row wrap;
     order: 2;
-    gap: 1rem;
     align-items: flex-start;
     max-height: 100vh;
   }
@@ -70,29 +64,29 @@
   flex: 2;
   display: flex;
   flex-flow: row wrap;
-  gap: 3rem;
 
   .admin-report-doughnut {
-    max-width: 300px;
-    max-height: 300px;
     padding: 0.25rem;
   }
 
   &__chart-wrapper {
-    width: auto;
+    height: fit-content;
+    position: relative;
+    transition: transform 0.25s ease, box-shadow 0.25s ease;
+    border-radius: var(--d-border-radius);
+
     .doughnut-chart-title {
       @include ellipsis;
-      margin: 0 auto;
-      text-align: center;
-      margin-bottom: 1rem;
-      max-width: 300px;
+      position: absolute;
+      top: 3%;
+      left: 50%;
+      transform: translateX(-50%);
+      font-size: var(--font-0);
+      z-index: 999;
     }
-    transition: transform 0.25s ease, box-shadow 0.25s ease;
-    border-radius: var(--d-border-radius);
 
     &:hover {
       transform: translateY(-1rem);
-      box-shadow: var(--shadow-card);
       cursor: pointer;
     }
   }
@@ -125,7 +119,7 @@
   @include report-container-box();
   flex: 1 1 300px;
   min-width: 300px;
-
+  margin-left: 1rem;
   display: flex;
   flex-flow: column nowrap;
   overflow-y: auto;

From 8b0cdcbc7ad745a884766a0fa866eb933c2b37b5 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 11:54:20 -0800
Subject: [PATCH 08/15] DEV: Improvements

---
 .../admin-report-sentiment-analysis.gjs       | 51 ++++++++++++++++---
 .../discourse/components/doughnut-chart.gjs   | 19 +++++--
 .../modules/sentiment/common/dashboard.scss   | 14 ++++-
 config/locales/client.en.yml                  |  1 +
 lib/sentiment/entry_point.rb                  |  5 +-
 spec/reports/sentiment_analysis_spec.rb       |  5 +-
 6 files changed, 80 insertions(+), 15 deletions(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index cc5884d9b..042aa8409 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -14,10 +14,14 @@ import dIcon from "discourse/helpers/d-icon";
 import replaceEmoji from "discourse/helpers/replace-emoji";
 import { ajax } from "discourse/lib/ajax";
 import { popupAjaxError } from "discourse/lib/ajax-error";
+import { getAbsoluteURL } from "discourse/lib/get-url";
+import discourseLater from "discourse/lib/later";
+import { clipboardCopy } from "discourse/lib/utilities";
 import Post from "discourse/models/post";
 import closeOnClickOutside from "discourse/modifiers/close-on-click-outside";
 import { i18n } from "discourse-i18n";
 import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
+import DTooltip from "float-kit/components/d-tooltip";
 
 export default class AdminReportSentimentAnalysis extends Component {
   @service router;
@@ -28,6 +32,7 @@ export default class AdminReportSentimentAnalysis extends Component {
   @tracked nextOffset = 0;
   @tracked showingSelectedChart = false;
   @tracked activeFilter = "all";
+  @tracked shareIcon = "link";
 
   setActiveFilter = modifier((element) => {
     this.clearActiveFilters(element);
@@ -266,6 +271,21 @@ export default class AdminReportSentimentAnalysis extends Component {
     });
   }
 
+  @action
+  shareChart() {
+    const url = this.router.currentURL;
+    if (!url) {
+      return;
+    }
+
+    clipboardCopy(getAbsoluteURL(url));
+    this.shareIcon = "check";
+
+    discourseLater(() => {
+      this.shareIcon = "link";
+    }, 2000);
+  }
+
   <template>
     <span {{didInsert this.openToChart}}></span>
 
@@ -298,12 +318,29 @@ export default class AdminReportSentimentAnalysis extends Component {
 
     {{#if (and this.selectedChart this.showingSelectedChart)}}
       <div class="admin-report-sentiment-analysis__selected-chart">
-        <DButton
-          @label="back_button"
-          @icon="chevron-left"
-          class="btn-flat"
-          @action={{this.backToAllCharts}}
-        />
+        <div class="admin-report-sentiment-analysis__selected-chart-actions">
+          <DButton
+            @label="back_button"
+            @icon="chevron-left"
+            class="btn-flat"
+            @action={{this.backToAllCharts}}
+          />
+
+          {{!-- <DButton
+            @title="discourse_ai.sentiments.sentiment_analysis.share_chart"
+            @icon={{this.shareIcon}}
+            @action={{this.shareChart}}
+            class="share btn-flat"
+          /> --}}
+          <DTooltip
+            class="share btn-flat"
+            @icon={{this.shareIcon}}
+            {{on "click" this.shareChart}}
+            @content={{i18n
+              "discourse_ai.sentiments.sentiment_analysis.share_chart"
+            }}
+          />
+        </div>
 
         <DoughnutChart
           @labels={{@model.labels}}
@@ -311,7 +348,9 @@ export default class AdminReportSentimentAnalysis extends Component {
           @data={{this.selectedChart.scores}}
           @totalScore={{this.selectedChart.total_score}}
           @doughnutTitle={{this.selectedChart.title}}
+          @radius={{100}}
         />
+
       </div>
       <div class="admin-report-sentiment-analysis-details">
         <HorizontalOverflowNav
diff --git a/assets/javascripts/discourse/components/doughnut-chart.gjs b/assets/javascripts/discourse/components/doughnut-chart.gjs
index 0bc0a462c..ef9399015 100644
--- a/assets/javascripts/discourse/components/doughnut-chart.gjs
+++ b/assets/javascripts/discourse/components/doughnut-chart.gjs
@@ -1,5 +1,8 @@
 import Component from "@glimmer/component";
 import { tracked } from "@glimmer/tracking";
+import { concat } from "@ember/helper";
+import { htmlSafe } from "@ember/template";
+import { isDevelopment } from "discourse/lib/environment";
 import Chart from "admin/components/chart";
 
 export default class DoughnutChart extends Component {
@@ -18,11 +21,19 @@ export default class DoughnutChart extends Component {
     );
   }
 
+  getRadius() {
+    if (this.args.radius) {
+      return this.args.radius;
+    } else if (isDevelopment()) {
+      return this.calculateRadius(Math.floor(Math.random() * (100 + 1)));
+    } else {
+      return this.calculateRadius(this.args.totalScore);
+    }
+  }
+
   get config() {
     const totalScore = this.args.totalScore || "";
-    // const radius = this.calculateRadius(this.args.totalScore)
-    // Temporary for tesitng:
-    const radius = this.calculateRadius(Math.floor(Math.random() * (100 + 1)));
+    const radius = this.getRadius();
 
     const paddingTop = 30;
     const paddingBottom = 0;
@@ -108,7 +119,7 @@ export default class DoughnutChart extends Component {
     {{#if this.config}}
       <h3
         class="doughnut-chart-title"
-        style="max-width: {{this.canvasSize}}px"
+        style={{htmlSafe (concat "max-width: " this.canvasSize "px")}}
       >{{@doughnutTitle}}</h3>
       <Chart @chartConfig={{this.config}} class="admin-report-doughnut" />
     {{/if}}
diff --git a/assets/stylesheets/modules/sentiment/common/dashboard.scss b/assets/stylesheets/modules/sentiment/common/dashboard.scss
index 8021669d6..099c4aacd 100644
--- a/assets/stylesheets/modules/sentiment/common/dashboard.scss
+++ b/assets/stylesheets/modules/sentiment/common/dashboard.scss
@@ -100,13 +100,23 @@
       font-size: var(--font-up-2);
       margin: 0 auto;
       text-align: center;
-      margin-bottom: 1rem;
       margin-top: 0.3rem;
       padding-top: 2rem;
-      padding-bottom: 1rem;
       border-top: 1px solid var(--primary-low);
     }
   }
+
+  &__selected-chart-actions {
+    display: flex;
+    align-items: center;
+    .share {
+      margin-left: auto;
+
+      .d-icon-check {
+        color: var(--success);
+      }
+    }
+  }
 }
 
 :root {
diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml
index e204360ba..41f995578 100644
--- a/config/locales/client.en.yml
+++ b/config/locales/client.en.yml
@@ -656,6 +656,7 @@ en:
           overview: "Sentiment overview"
           analysis: "Sentiment analysis"
         sentiment_analysis:
+          share_chart: "Copy link to chart"
           filter_types:
             all: "All"
             positive: "Positive"
diff --git a/lib/sentiment/entry_point.rb b/lib/sentiment/entry_point.rb
index f70d8c6a5..a35bf1d7a 100644
--- a/lib/sentiment/entry_point.rb
+++ b/lib/sentiment/entry_point.rb
@@ -22,8 +22,9 @@ def inject_into(plugin)
           end,
         ) { ClassificationResult.has_sentiment_classification? }
 
-        if ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled &&
-             SiteSetting.ai_sentiment_reports_enabled
+        if Rails.env.test? ||
+             ClassificationResult.has_sentiment_classification? &&
+               SiteSetting.ai_sentiment_enabled && SiteSetting.ai_sentiment_reports_enabled
           EmotionFilterOrder.register!(plugin)
           EmotionDashboardReport.register!(plugin)
           SentimentDashboardReport.register!(plugin)
diff --git a/spec/reports/sentiment_analysis_spec.rb b/spec/reports/sentiment_analysis_spec.rb
index d251877c7..c5ea9464f 100644
--- a/spec/reports/sentiment_analysis_spec.rb
+++ b/spec/reports/sentiment_analysis_spec.rb
@@ -8,7 +8,10 @@
   fab!(:post_2) { Fabricate(:post, user: admin, topic: topic) }
   fab!(:classification_result) { Fabricate(:classification_result, target: post) }
 
-  before { SiteSetting.ai_sentiment_enabled = true }
+  before do
+    SiteSetting.ai_sentiment_reports_enabled = true
+    SiteSetting.ai_sentiment_enabled = true
+  end
 
   it "contains the correct filters" do
     report = Report.find("sentiment_analysis")

From 25f3cebff7e953571573c7677ceac7531bc750c5 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 12:06:03 -0800
Subject: [PATCH 09/15] DEV: fixes

---
 .../components/admin-report-sentiment-analysis.gjs     | 10 +++-------
 .../discourse/components/doughnut-chart.gjs            |  7 ++++++-
 .../modules/sentiment/common/dashboard.scss            |  4 +++-
 3 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index 042aa8409..7b9f168fa 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -20,8 +20,8 @@ import { clipboardCopy } from "discourse/lib/utilities";
 import Post from "discourse/models/post";
 import closeOnClickOutside from "discourse/modifiers/close-on-click-outside";
 import { i18n } from "discourse-i18n";
-import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
 import DTooltip from "float-kit/components/d-tooltip";
+import DoughnutChart from "discourse/plugins/discourse-ai/discourse/components/doughnut-chart";
 
 export default class AdminReportSentimentAnalysis extends Component {
   @service router;
@@ -326,12 +326,6 @@ export default class AdminReportSentimentAnalysis extends Component {
             @action={{this.backToAllCharts}}
           />
 
-          {{!-- <DButton
-            @title="discourse_ai.sentiments.sentiment_analysis.share_chart"
-            @icon={{this.shareIcon}}
-            @action={{this.shareChart}}
-            class="share btn-flat"
-          /> --}}
           <DTooltip
             class="share btn-flat"
             @icon={{this.shareIcon}}
@@ -348,6 +342,8 @@ export default class AdminReportSentimentAnalysis extends Component {
           @data={{this.selectedChart.scores}}
           @totalScore={{this.selectedChart.total_score}}
           @doughnutTitle={{this.selectedChart.title}}
+          @skipCanvasResize={{true}}
+          @displayLegend={{true}}
           @radius={{100}}
         />
 
diff --git a/assets/javascripts/discourse/components/doughnut-chart.gjs b/assets/javascripts/discourse/components/doughnut-chart.gjs
index ef9399015..2caf8d743 100644
--- a/assets/javascripts/discourse/components/doughnut-chart.gjs
+++ b/assets/javascripts/discourse/components/doughnut-chart.gjs
@@ -33,6 +33,7 @@ export default class DoughnutChart extends Component {
 
   get config() {
     const totalScore = this.args.totalScore || "";
+    const skipCanvasResize = this.args.skipCanvasResize || false;
     const radius = this.getRadius();
 
     const paddingTop = 30;
@@ -66,7 +67,8 @@ export default class DoughnutChart extends Component {
         },
         plugins: {
           legend: {
-            display: false,
+            display: this.args.displayLegend || false,
+            position: "bottom",
           },
         },
       },
@@ -104,6 +106,9 @@ export default class DoughnutChart extends Component {
         {
           id: "resizeCanvas",
           afterDraw: function (chart) {
+            if (skipCanvasResize) {
+              return;
+            }
             const size = `${canvasSize}px`;
             chart.canvas.style.width = size;
             chart.canvas.style.height = size;
diff --git a/assets/stylesheets/modules/sentiment/common/dashboard.scss b/assets/stylesheets/modules/sentiment/common/dashboard.scss
index 099c4aacd..ed98f6716 100644
--- a/assets/stylesheets/modules/sentiment/common/dashboard.scss
+++ b/assets/stylesheets/modules/sentiment/common/dashboard.scss
@@ -102,13 +102,15 @@
       text-align: center;
       margin-top: 0.3rem;
       padding-top: 2rem;
-      border-top: 1px solid var(--primary-low);
     }
   }
 
   &__selected-chart-actions {
     display: flex;
     align-items: center;
+    padding-bottom: 0.35rem;
+    border-bottom: 1px solid var(--primary-low);
+
     .share {
       margin-left: auto;
 

From 6f444a7762b279c8d322e38b5e997ec2dfb6f6e3 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 12:11:01 -0800
Subject: [PATCH 10/15] reorg

---
 .../admin-report-sentiment-analysis.gjs       | 72 +++++++++----------
 1 file changed, 36 insertions(+), 36 deletions(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index 7b9f168fa..e9bfd5920 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -41,30 +41,6 @@ export default class AdminReportSentimentAnalysis extends Component {
       .classList.add("active");
   });
 
-  @action
-  async openToChart() {
-    const queryParams = this.router.currentRoute.queryParams;
-    if (queryParams.selectedChart) {
-      this.selectedChart = this.transformedData.find(
-        (data) => data.title === queryParams.selectedChart
-      );
-
-      if (!this.selectedChart) {
-        return;
-      }
-      this.showingSelectedChart = true;
-
-      try {
-        const response = await this.postRequest();
-        this.posts = response.posts.map((post) => Post.create(post));
-        this.hasMorePosts = response.has_more;
-        this.nextOffset = response.next_offset;
-      } catch (e) {
-        popupAjaxError(e);
-      }
-    }
-  }
-
   clearActiveFilters(element) {
     const filterButtons = element.querySelectorAll("li button");
     for (let button of filterButtons) {
@@ -105,18 +81,6 @@ export default class AdminReportSentimentAnalysis extends Component {
     }
   }
 
-  async postRequest() {
-    return await ajax("/discourse-ai/sentiment/posts", {
-      data: {
-        group_by: this.currentGroupFilter,
-        group_value: this.selectedChart?.title,
-        start_date: this.args.model.start_date,
-        end_date: this.args.model.end_date,
-        offset: this.nextOffset,
-      },
-    });
-  }
-
   get colors() {
     return ["#2ecc71", "#95a5a6", "#e74c3c"];
   }
@@ -207,6 +171,42 @@ export default class AdminReportSentimentAnalysis extends Component {
     ];
   }
 
+  async postRequest() {
+    return await ajax("/discourse-ai/sentiment/posts", {
+      data: {
+        group_by: this.currentGroupFilter,
+        group_value: this.selectedChart?.title,
+        start_date: this.args.model.start_date,
+        end_date: this.args.model.end_date,
+        offset: this.nextOffset,
+      },
+    });
+  }
+
+  @action
+  async openToChart() {
+    const queryParams = this.router.currentRoute.queryParams;
+    if (queryParams.selectedChart) {
+      this.selectedChart = this.transformedData.find(
+        (data) => data.title === queryParams.selectedChart
+      );
+
+      if (!this.selectedChart) {
+        return;
+      }
+      this.showingSelectedChart = true;
+
+      try {
+        const response = await this.postRequest();
+        this.posts = response.posts.map((post) => Post.create(post));
+        this.hasMorePosts = response.has_more;
+        this.nextOffset = response.next_offset;
+      } catch (e) {
+        popupAjaxError(e);
+      }
+    }
+  }
+
   @action
   async showDetails(data) {
     if (this.selectedChart === data) {

From e14b159154024016c5a47221c396c242c6861605 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 12:21:58 -0800
Subject: [PATCH 11/15] DEV: no need `async`

---
 assets/javascripts/initializers/ai-sentiment-admin-nav.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assets/javascripts/initializers/ai-sentiment-admin-nav.js b/assets/javascripts/initializers/ai-sentiment-admin-nav.js
index c73b1add6..9bf45633e 100644
--- a/assets/javascripts/initializers/ai-sentiment-admin-nav.js
+++ b/assets/javascripts/initializers/ai-sentiment-admin-nav.js
@@ -1,6 +1,6 @@
 import { apiInitializer } from "discourse/lib/api";
 
-export default apiInitializer("1.15.0", async (api) => {
+export default apiInitializer("1.15.0", (api) => {
   const currentUser = api.getCurrentUser();
 
   if (

From 1fd3f7eedcd5f401685e64378088252a91c39645 Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 12:33:31 -0800
Subject: [PATCH 12/15] fix

---
 lib/sentiment/entry_point.rb | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/lib/sentiment/entry_point.rb b/lib/sentiment/entry_point.rb
index a35bf1d7a..76302ebe3 100644
--- a/lib/sentiment/entry_point.rb
+++ b/lib/sentiment/entry_point.rb
@@ -14,13 +14,10 @@ def inject_into(plugin)
         plugin.on(:post_created, &sentiment_analysis_cb)
         plugin.on(:post_edited, &sentiment_analysis_cb)
 
-        plugin.add_to_serializer(
-          :current_user,
-          :can_see_sentiment_reports,
-          include_condition: -> do
-            SiteSetting.ai_sentiment_enabled && SiteSetting.ai_sentiment_reports_enabled
-          end,
-        ) { ClassificationResult.has_sentiment_classification? }
+        plugin.add_to_serializer(:current_user, :can_see_sentiment_reports) do
+          ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled &&
+            SiteSetting.ai_sentiment_reports_enabled
+        end
 
         if Rails.env.test? ||
              ClassificationResult.has_sentiment_classification? &&

From 7c4ccfe0e8e8243bc2680564c5bdb030d6aece5f Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 13:01:29 -0800
Subject: [PATCH 13/15] DEV: Revert sizing and make header sticky

---
 .../admin-report-sentiment-analysis.gjs       |  3 +-
 .../discourse/components/doughnut-chart.gjs   | 60 +------------------
 .../modules/sentiment/common/dashboard.scss   | 24 ++++++--
 3 files changed, 22 insertions(+), 65 deletions(-)

diff --git a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
index e9bfd5920..f49745eb8 100644
--- a/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
+++ b/assets/javascripts/discourse/components/admin-report-sentiment-analysis.gjs
@@ -310,6 +310,7 @@ export default class AdminReportSentimentAnalysis extends Component {
               @data={{data.scores}}
               @totalScore={{data.total_score}}
               @doughnutTitle={{data.title}}
+              @displayLegend={{true}}
             />
           </div>
         {{/each}}
@@ -342,9 +343,7 @@ export default class AdminReportSentimentAnalysis extends Component {
           @data={{this.selectedChart.scores}}
           @totalScore={{this.selectedChart.total_score}}
           @doughnutTitle={{this.selectedChart.title}}
-          @skipCanvasResize={{true}}
           @displayLegend={{true}}
-          @radius={{100}}
         />
 
       </div>
diff --git a/assets/javascripts/discourse/components/doughnut-chart.gjs b/assets/javascripts/discourse/components/doughnut-chart.gjs
index 2caf8d743..939a1e316 100644
--- a/assets/javascripts/discourse/components/doughnut-chart.gjs
+++ b/assets/javascripts/discourse/components/doughnut-chart.gjs
@@ -8,38 +8,8 @@ import Chart from "admin/components/chart";
 export default class DoughnutChart extends Component {
   @tracked canvasSize = null;
 
-  calculateRadius(postCount) {
-    const minPosts = 0;
-    const maxPosts = 100;
-    const minRadius = 30;
-    const maxRadius = 100;
-    const clampedPostCount = Math.min(Math.max(postCount, minPosts), maxPosts);
-    return (
-      minRadius +
-      ((clampedPostCount - minPosts) / (maxPosts - minPosts)) *
-        (maxRadius - minRadius)
-    );
-  }
-
-  getRadius() {
-    if (this.args.radius) {
-      return this.args.radius;
-    } else if (isDevelopment()) {
-      return this.calculateRadius(Math.floor(Math.random() * (100 + 1)));
-    } else {
-      return this.calculateRadius(this.args.totalScore);
-    }
-  }
-
   get config() {
     const totalScore = this.args.totalScore || "";
-    const skipCanvasResize = this.args.skipCanvasResize || false;
-    const radius = this.getRadius();
-
-    const paddingTop = 30;
-    const paddingBottom = 0;
-    const canvasSize = 2 * radius + paddingTop + paddingBottom;
-    this.canvasSize = canvasSize;
 
     return {
       type: "doughnut",
@@ -49,22 +19,14 @@ export default class DoughnutChart extends Component {
           {
             data: this.args.data,
             backgroundColor: this.args.colors,
-            cutout: "70%",
-            radius,
+            cutout: "50%",
+            radius: 100,
           },
         ],
       },
       options: {
         responsive: true,
         maintainAspectRatio: false,
-        layout: {
-          padding: {
-            top: paddingTop,
-            left: 0,
-            right: 0,
-            bottom: 0,
-          },
-        },
         plugins: {
           legend: {
             display: this.args.displayLegend || false,
@@ -103,29 +65,13 @@ export default class DoughnutChart extends Component {
             ctx.save();
           },
         },
-        {
-          id: "resizeCanvas",
-          afterDraw: function (chart) {
-            if (skipCanvasResize) {
-              return;
-            }
-            const size = `${canvasSize}px`;
-            chart.canvas.style.width = size;
-            chart.canvas.style.height = size;
-
-            chart.resize();
-          },
-        },
       ],
     };
   }
 
   <template>
     {{#if this.config}}
-      <h3
-        class="doughnut-chart-title"
-        style={{htmlSafe (concat "max-width: " this.canvasSize "px")}}
-      >{{@doughnutTitle}}</h3>
+      <h3 class="doughnut-chart-title">{{@doughnutTitle}}</h3>
       <Chart @chartConfig={{this.config}} class="admin-report-doughnut" />
     {{/if}}
   </template>
diff --git a/assets/stylesheets/modules/sentiment/common/dashboard.scss b/assets/stylesheets/modules/sentiment/common/dashboard.scss
index ed98f6716..a38f2f84e 100644
--- a/assets/stylesheets/modules/sentiment/common/dashboard.scss
+++ b/assets/stylesheets/modules/sentiment/common/dashboard.scss
@@ -63,6 +63,9 @@
   @include report-container-box();
   flex: 2;
   display: flex;
+  gap: 1rem;
+  justify-content: space-around;
+  align-items: center;
   flex-flow: row wrap;
 
   .admin-report-doughnut {
@@ -77,15 +80,13 @@
 
     .doughnut-chart-title {
       @include ellipsis;
-      position: absolute;
-      top: 3%;
-      left: 50%;
-      transform: translateX(-50%);
-      font-size: var(--font-0);
-      z-index: 999;
+      margin: 0 auto;
+      margin-top: 1rem;
+      text-align: center;
     }
 
     &:hover {
+      box-shadow: var(--shadow-card);
       transform: translateY(-1rem);
       cursor: pointer;
     }
@@ -136,6 +137,7 @@
   flex-flow: column nowrap;
   overflow-y: auto;
   height: 100%;
+  padding-top: 0;
 
   &__filters {
     border-bottom: 1px solid var(--primary-low);
@@ -210,3 +212,13 @@
     }
   }
 }
+
+.admin-reports.admin-contents .sentiment-analysis {
+  .horizontal-overflow-nav {
+    background: var(--secondary);
+    position: sticky;
+    top: 0;
+    padding-top: 1rem;
+    z-index: z("header");
+  }
+}

From 3936b6460f29404041ce0ab9021384a2fdb51c0c Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 13:04:15 -0800
Subject: [PATCH 14/15] FIX

---
 assets/javascripts/discourse/components/doughnut-chart.gjs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/assets/javascripts/discourse/components/doughnut-chart.gjs b/assets/javascripts/discourse/components/doughnut-chart.gjs
index 939a1e316..a417de303 100644
--- a/assets/javascripts/discourse/components/doughnut-chart.gjs
+++ b/assets/javascripts/discourse/components/doughnut-chart.gjs
@@ -1,8 +1,5 @@
 import Component from "@glimmer/component";
 import { tracked } from "@glimmer/tracking";
-import { concat } from "@ember/helper";
-import { htmlSafe } from "@ember/template";
-import { isDevelopment } from "discourse/lib/environment";
 import Chart from "admin/components/chart";
 
 export default class DoughnutChart extends Component {

From 9e8270843d5bd745b524685c6b81be12fe50294f Mon Sep 17 00:00:00 2001
From: Keegan George <kgeorge13@gmail.com>
Date: Wed, 5 Mar 2025 13:46:16 -0800
Subject: [PATCH 15/15] DEV: remove sentiment setting

---
 config/settings.yml                     | 3 ---
 lib/sentiment/entry_point.rb            | 6 ++----
 spec/reports/sentiment_analysis_spec.rb | 5 +----
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/config/settings.yml b/config/settings.yml
index 2246fed7e..665c2c83a 100644
--- a/config/settings.yml
+++ b/config/settings.yml
@@ -25,9 +25,6 @@ discourse_ai:
   ai_sentiment_backfill_post_max_age_days:
     default: 60
     hidden: true
-  ai_sentiment_reports_enabled:
-    default: true
-    client: true
 
   ai_openai_dall_e_3_url: "https://api.openai.com/v1/images/generations"
   ai_openai_embeddings_url:
diff --git a/lib/sentiment/entry_point.rb b/lib/sentiment/entry_point.rb
index 76302ebe3..3c66985bc 100644
--- a/lib/sentiment/entry_point.rb
+++ b/lib/sentiment/entry_point.rb
@@ -15,13 +15,11 @@ def inject_into(plugin)
         plugin.on(:post_edited, &sentiment_analysis_cb)
 
         plugin.add_to_serializer(:current_user, :can_see_sentiment_reports) do
-          ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled &&
-            SiteSetting.ai_sentiment_reports_enabled
+          ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled
         end
 
         if Rails.env.test? ||
-             ClassificationResult.has_sentiment_classification? &&
-               SiteSetting.ai_sentiment_enabled && SiteSetting.ai_sentiment_reports_enabled
+             ClassificationResult.has_sentiment_classification? && SiteSetting.ai_sentiment_enabled
           EmotionFilterOrder.register!(plugin)
           EmotionDashboardReport.register!(plugin)
           SentimentDashboardReport.register!(plugin)
diff --git a/spec/reports/sentiment_analysis_spec.rb b/spec/reports/sentiment_analysis_spec.rb
index c5ea9464f..d251877c7 100644
--- a/spec/reports/sentiment_analysis_spec.rb
+++ b/spec/reports/sentiment_analysis_spec.rb
@@ -8,10 +8,7 @@
   fab!(:post_2) { Fabricate(:post, user: admin, topic: topic) }
   fab!(:classification_result) { Fabricate(:classification_result, target: post) }
 
-  before do
-    SiteSetting.ai_sentiment_reports_enabled = true
-    SiteSetting.ai_sentiment_enabled = true
-  end
+  before { SiteSetting.ai_sentiment_enabled = true }
 
   it "contains the correct filters" do
     report = Report.find("sentiment_analysis")