DEV: Add structure for errors in spam

This PR adds some structure for handling errors in the spam config while also handling a specific error related to the spam scanning user not being an admin account.

Author: keegangeorge

app/controllers/discourse_ai/admin/ai_spam_controller.rb

@@ -86,6 +86,14 @@ def test
         render json: result
       end
 
+      def fix_errors
+        if params[:error] == "spam_scanner_not_admin"
+          DiscourseAi::AiModeration::SpamScanner.fix_spam_scanner_not_admin
+        end
+
+        render json: success_json
+      end
+
       private
 
       def allowed_params

app/serializers/ai_spam_serializer.rb

@@ -7,7 +7,8 @@ class AiSpamSerializer < ApplicationSerializer
              :available_llms,
              :stats,
              :flagging_username,
-             :spam_score_type
+             :spam_score_type,
+             :spam_scanning_user
 
   def is_enabled
     object[:enabled]
@@ -47,4 +48,10 @@ def stats
   def settings
     object[:settings]
   end
+
+  def spam_scanning_user
+    user = DiscourseAi::AiModeration::SpamScanner.flagging_user
+
+    user.serializable_hash(only: %i[id username name admin]) if user.present?
+  end
 end

assets/javascripts/discourse/components/ai-spam.gjs

@@ -13,6 +13,7 @@ import DTooltip from "discourse/components/d-tooltip";
 import withEventValue from "discourse/helpers/with-event-value";
 import { ajax } from "discourse/lib/ajax";
 import { popupAjaxError } from "discourse/lib/ajax-error";
+import dIcon from "discourse-common/helpers/d-icon";
 import i18n from "discourse-common/helpers/i18n";
 import getURL from "discourse-common/lib/get-url";
 import AdminConfigAreaCard from "admin/components/admin-config-area-card";
@@ -35,10 +36,51 @@ export default class AiSpam extends Component {
   @tracked isEnabled = false;
   @tracked selectedLLM = null;
   @tracked customInstructions = "";
+  @tracked errors = [];
 
   constructor() {
     super(...arguments);
     this.initializeFromModel();
+
+    if (this.args.model?.spam_scanning_user?.admin === false) {
+      this.errors.push({
+        message: i18n("discourse_ai.spam.errors.scan_not_admin.message"),
+        button: {
+          label: i18n("discourse_ai.spam.errors.scan_not_admin.action"),
+          action: this.fixScanUserNotAdmin,
+        },
+      });
+    }
+  }
+
+  @action
+  async fixScanUserNotAdmin() {
+    const spamScanningUser = this.args.model.spam_scanning_user;
+    if (!spamScanningUser || spamScanningUser.admin) {
+      return;
+    }
+    try {
+      const response = await ajax(
+        `/admin/plugins/discourse-ai/ai-spam/fix-errors`,
+        {
+          type: "POST",
+          data: {
+            error: "spam_scanner_not_admin",
+          },
+        }
+      );
+
+      if (response.success) {
+        this.toasts.success({
+          data: { message: i18n("discourse_ai.spam.errors.resolved") },
+          duration: 2000,
+        });
+      }
+    } catch (error) {
+      popupAjaxError(error);
+    } finally {
+      window.location.reload();
+    }
   }
 
   @action
@@ -165,11 +207,22 @@ export default class AiSpam extends Component {
   <template>
     <div class="ai-spam">
       <section class="ai-spam__settings">
+        <div class="ai-spam__errors">
+          {{#each this.errors as |e|}}
+            <div class="alert alert-error">
+              {{dIcon "triangle-exclamation"}}
+              <p>{{e.message}}</p>
+              <DButton
+                @action={{e.button.action}}
+                @translatedLabel={{e.button.label}}
+              />
+            </div>
+          {{/each}}
+        </div>
         <DPageSubheader
           @titleLabel={{i18n "discourse_ai.spam.title"}}
           @descriptionLabel={{i18n "discourse_ai.spam.spam_description"}}
         />
-
         <div class="control-group ai-spam__enabled">
           <DToggleSwitch
             class="ai-spam__toggle"

assets/stylesheets/modules/llms/common/spam.scss

@@ -45,6 +45,18 @@
   &__stats {
     margin-top: 2em;
   }
+
+  &__errors {
+    .alert {
+      display: flex;
+      align-items: center;
+      gap: 0.5rem;
+
+      .btn {
+        margin-left: auto;
+      }
+    }
+  }
 }
 
 .spam-test-modal {

config/locales/client.en.yml

@@ -160,6 +160,11 @@ en:
         stat_tooltips:
           incorrectly_flagged: "Items that the AI bot flagged as spam where moderators disagreed"
           missed_spam: "Items flagged by the community as spam that were not detected by the AI bot, which moderators agreed with"
+        errors:
+          scan_not_admin:
+            message: "Warning: spam scanning will not work correctly because the spam scan account is not an admin"
+            action: "Fix"
+          resolved: "The error has been resolved!"
 
       usage:
         short_title: "Usage"

config/routes.rb

@@ -83,6 +83,7 @@
     get "/ai-spam", to: "discourse_ai/admin/ai_spam#show"
     put "/ai-spam", to: "discourse_ai/admin/ai_spam#update"
     post "/ai-spam/test", to: "discourse_ai/admin/ai_spam#test"
+    post "/ai-spam/fix-errors", to: "discourse_ai/admin/ai_spam#fix_errors"
 
     resources :ai_llms,
               only: %i[index new create edit update destroy],

lib/ai_moderation/spam_scanner.rb

@@ -241,6 +241,11 @@ def self.perform_scan(post)
         end
       end
 
+      def self.fix_spam_scanner_not_admin
+        user = DiscourseAi::AiModeration::SpamScanner.flagging_user
+        user.update!(admin: true) if user.present?
+      end
+
       private
 
       def self.check_if_spam(result)