Merge branch 'prototype'

littlealien095 · littlealien095 · commit bb34987888fd · 2025-02-19T00:28:35.000Z
diff --git a/app/controllers/simscore/analyze_controller.rb b/app/controllers/simscore/analyze_controller.rb
@@ -5,28 +5,63 @@ class AnalyzeController < ::ApplicationController
     before_action :ensure_api_key_configured
 
     def create
-      Rails.logger.info("SimScore: Starting analysis for topic #{params[:topic_id]}")
+      start_time = Time.now
+      Rails.logger.info(
+        "SimScore: Analysis started",
+        {
+          topic_id: params[:topic_id],
+          user_id: current_user.id,
+          username: current_user.username,
+          timestamp: Time.now,
+        }.to_json,
+      )
 
       topic = Topic.find(params[:topic_id])
       guardian.ensure_can_see!(topic)
 
       # Get posts with their users
       posts = topic.posts.includes(:user).order(:post_number)
-      Rails.logger.info("SimScore: Found #{posts.count} posts to analyze")
+      Rails.logger.info(
+        "SimScore: Posts loaded",
+        {
+          topic_id: topic.id,
+          post_count: posts.count,
+          topic_title: topic.title,
+          duration_ms: ((Time.now - start_time) * 1000).to_i,
+        }.to_json,
+      )
 
       begin
         # Call SimScore API
-        Rails.logger.info("SimScore: Calling API service")
+        Rails.logger.info(
+          "SimScore: Calling API service",
+          { topic_id: topic.id, endpoint: SiteSetting.simscore_api_endpoint }.to_json,
+        )
+
         results = ApiService.analyze_posts(posts)
-        Rails.logger.info("SimScore: API analysis complete")
+
+        Rails.logger.info(
+          "SimScore: API analysis complete",
+          {
+            topic_id: topic.id,
+            duration_ms: ((Time.now - start_time) * 1000).to_i,
+            cluster_count: results["cluster_names"]&.size,
+          }.to_json,
+        )
 
         # Format results into markdown
         formatted_results = ApiService.format_results(results)
-        Rails.logger.info("SimScore: Results formatted")
+        Rails.logger.info(
+          "SimScore: Results formatted",
+          { topic_id: topic.id, result_length: formatted_results.length }.to_json,
+        )
 
         # Try to use SimScore user if available, otherwise use current user
         posting_user = User.find_by(username: "simscore") || current_user
-        Rails.logger.info("SimScore: Posting as user #{posting_user.username}")
+        Rails.logger.info(
+          "SimScore: Selected posting user",
+          { topic_id: topic.id, username: posting_user.username, user_id: posting_user.id }.to_json,
+        )
 
         # Create a new post with the results
         creator =
@@ -40,15 +75,35 @@ def create
         post = creator.create
 
         if post.present?
-          Rails.logger.info("SimScore: Successfully created post #{post.id}")
+          Rails.logger.info(
+            "SimScore: Post created successfully",
+            {
+              topic_id: topic.id,
+              post_id: post.id,
+              duration_ms: ((Time.now - start_time) * 1000).to_i,
+            }.to_json,
+          )
           render json: { success: true, post_id: post.id }
         else
-          Rails.logger.error("SimScore: Failed to create post: #{creator.errors.full_messages}")
+          Rails.logger.error(
+            "SimScore: Post creation failed",
+            {
+              topic_id: topic.id,
+              errors: creator.errors.full_messages,
+              duration_ms: ((Time.now - start_time) * 1000).to_i,
+            }.to_json,
+          )
           render json: { success: false, errors: creator.errors.full_messages }, status: 422
         end
       rescue StandardError => e
         Rails.logger.error(
-          "SimScore: Error during analysis: #{e.message}\n#{e.backtrace.join("\n")}",
+          "SimScore: Error during analysis",
+          {
+            topic_id: topic.id,
+            error: e.message,
+            backtrace: e.backtrace[0..5],
+            duration_ms: ((Time.now - start_time) * 1000).to_i,
+          }.to_json,
         )
         render json: { success: false, errors: [e.message] }, status: 422
       end
@@ -58,7 +113,10 @@ def create
 
     def ensure_api_key_configured
       unless SiteSetting.simscore_api_key.present?
-        Rails.logger.error("SimScore: API key not configured")
+        Rails.logger.error(
+          "SimScore: API key missing",
+          { user_id: current_user.id, username: current_user.username }.to_json,
+        )
         render json: { success: false, errors: ["SimScore API key not configured"] }, status: 422
       end
     end
diff --git a/assets/javascripts/discourse/components/simscore-button.gjs b/assets/javascripts/discourse/components/simscore-button.gjs
@@ -1,51 +1,163 @@
 import Component from "@glimmer/component";
 import { tracked } from "@glimmer/tracking";
-import { getOwner } from "@ember/application";
 import { action } from "@ember/object";
 import { service } from "@ember/service";
 import DButton from "discourse/components/d-button";
 import { ajax } from "discourse/lib/ajax";
 import { popupAjaxError } from "discourse/lib/ajax-error";
-import { i18n } from "discourse-i18n";
 
 export default class SimscoreButton extends Component {
   @service currentUser;
+  @service siteSettings;
+  @service dialog;
   @tracked loading = false;
-
-  get logger() {
-    return getOwner(this).lookup("service:logger");
-  }
+  @tracked ideas = [];
 
   @action
   async calculateSimScore() {
+    // eslint-disable-next-line no-console
+    console.info("Debug: args received:", {
+      args: this.args,
+      outletArgs: this.args?.outletArgs,
+      topic: this.args?.topic,
+    });
+
+    // eslint-disable-next-line no-console
+    console.info("Debug: raw args:", this.args);
+
     try {
-      this.logger.debug(
-        "SimScore: Starting calculation for topic",
-        this.args.topic.id
-      );
-      this.loading = true;
-      const response = await ajax(`/simscore/analyze/${this.args.topic.id}`, {
-        type: "POST",
+      if (!this.args?.topic?.id) {
+        // eslint-disable-next-line no-console
+        console.error("❌ SimScore: No topic ID found");
+        this.dialog.alert({
+          message: "Error: Cannot find topic ID",
+        });
+        return;
+      }
+
+      const topicId = this.args.topic.id;
+      const username = this.currentUser?.username;
+
+      // eslint-disable-next-line no-console
+      console.info("SimScore: Button pressed", {
+        topicId,
+        username,
+        timestamp: new Date().toISOString(),
+        topicTitle: this.args.topic?.title,
+        postCount: this.args.topic?.posts_count,
+      });
+
+      // Show a notification to the user
+      this.dialog.alert({
+        message: `Starting SimScore analysis for topic ${this.args.topic.title}...`,
       });
 
-      this.logger.debug("SimScore: Received response", response);
-
-      if (response.success) {
-        this.logger.debug(
-          "SimScore: Calculation successful, post created with ID:",
-          response.post_id
-        );
-        this.args.showSuccess?.(i18n("simscore.success"));
-      } else {
-        this.logger.error("SimScore: API returned error:", response.errors);
-        this.args.showError?.(i18n("simscore.error"));
+      this.loading = true;
+      // eslint-disable-next-line no-console
+      console.info("🔵 SimScore: Analysis started...");
+
+      // Ensure we have at least 4 ideas as per API requirements
+      // if (this.ideas.length < 4) {
+      //   this.dialog.alert({
+      //     message: "At least 4 ideas are required for analysis",
+      //   });
+      //   this.loading = false;
+      //   return;
+      // }
+
+      const formattedIdeas = [
+        ...this.ideas.map((idea, index) => ({
+          id: String(index + 1),
+          idea,
+        })),
+        {
+          id: this.ideas.length + 1,
+          idea: "Improve user engagement through gamification",
+        },
+        {
+          id: this.ideas.length + 2,
+          idea: "Implement a reward system for active contributors",
+        },
+        {
+          id: this.ideas.length + 3,
+          idea: "Add social sharing features",
+        },
+        {
+          id: this.ideas.length + 4,
+          idea: "Create a mobile-friendly interface",
+        },
+      ];
+
+      const payload = {
+        ideas: formattedIdeas,
+        advanced_features: {
+          relationship_graph: false,
+          cluster_names: false,
+          pairwise_similarity_matrix: false,
+        },
+      };
+
+      try {
+        // eslint-disable-next-line no-console
+        console.info("🔵 Debug payload:", payload);
+        const apiUrl = "https://simscore-api-dev.fly.dev/v1/rank_ideas";
+        // eslint-disable-next-line no-console
+        console.info("🔵 Making API request to:", apiUrl);
+
+        const response = await ajax(apiUrl, {
+          type: "POST",
+          contentType: "application/json",
+          dataType: "json",
+          crossDomain: true,
+          xhrFields: {
+            withCredentials: true,
+          },
+          headers: {
+            Accept: "application/json",
+            "Access-Control-Allow-Origin": "*",
+            Authorization:
+              "Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VyX2lkIjoiNWNhZWU2MTQtNTEzZS00NTljLTllN2UtNDM0MzYyMGFlMjlhIiwiZW1haWwiOiJtaWtoYWlsb3YuYW50QGdtYWlsLmNvbSIsImlzX2d1ZXN0IjpmYWxzZSwia2V5X2lkIjoiNWQ2YTVhYjEtNjI2Zi00MTFmLWE2ZWYtYTg3NjBjNTVlYTI5IiwidG9rZW5fdHlwZSI6ImFwaV9rZXkifQ.k3_qSLw1NbtT65S1yC1FQbVfMm8sRgL8x4FQnDkB8n0",
+          },
+          data: JSON.stringify(payload),
+        });
+
+        // eslint-disable-next-line no-console
+        console.info("🟢 API Response:", response);
+
+        if (response.ranked_ideas) {
+          const score = response.ranked_ideas[0].similarity_score;
+          // eslint-disable-next-line no-console
+          console.info("✅ SimScore: Analysis complete!", score);
+          this.dialog.alert({
+            message: `SimScore analysis completed! Score: ${(
+              score * 100
+            ).toFixed(1)}%`,
+          });
+        } else {
+          throw new Error("Invalid response format");
+        }
+      } catch (error) {
+        // eslint-disable-next-line no-console
+        console.error("❌ API Error:", error);
+        // eslint-disable-next-line no-console
+        console.error("❌ Error details:", {
+          message: error.message,
+          status: error.status,
+          stack: error.stack,
+        });
+        throw error;
+      } finally {
+        // eslint-disable-next-line no-console
+        console.info("🔵 SimScore: Operation completed");
+        this.loading = false;
       }
     } catch (error) {
-      this.logger.error("SimScore: Failed to calculate:", error);
+      // eslint-disable-next-line no-console
+      console.error("❌ SimScore: Exception:", error.message, error.stack);
+      this.dialog.alert({
+        message: `Error: ${error.message}`,
+      });
       popupAjaxError(error);
-      this.args.showError?.(i18n("simscore.error"));
-    } finally {
-      this.loading = false;
     }
   }
 
@@ -54,7 +166,8 @@ export default class SimscoreButton extends Component {
       class="btn-primary simscore-button"
       @action={{this.calculateSimScore}}
       @disabled={{this.loading}}
-      @label="simscore.calculate"
+      @icon="calculator"
+      @translatedLabel="Calculate SimScore"
     />
   </template>
 }
diff --git a/assets/javascripts/discourse/connectors/topic-navigation-bottom/simscore-button.gjs b/assets/javascripts/discourse/connectors/topic-navigation-bottom/simscore-button.gjs
@@ -1,8 +1,5 @@
-import Component from "@glimmer/component";
 import SimscoreButton from "../../components/simscore-button";
 
-export default class SimscoreButtonConnector extends Component {
-  <template>
-      <SimscoreButton @topic={{this.args.outletArgs.topic}} />
-  </template>
-} 
+<template>
+  <SimscoreButton @topic={{@outletArgs.model}} />
+</template>
