Fix: Persist Debate vs Bot Outcomes Using Debate ID

backend/cmd/server/main.go

@@ -132,6 +132,9 @@ func setupRouter(cfg *config.Config) *gin.Engine {
 		// Set up transcript routes
 		routes.SetupTranscriptRoutes(auth)
 
+		// Set up assumption extraction routes
+		routes.SetupAssumptionRoutes(auth)
+
 		auth.GET("/coach/strengthen-argument/weak-statement", routes.GetWeakStatement)
 		auth.POST("/coach/strengthen-argument/evaluate", routes.EvaluateStrengthenedArgument)
 

backend/controllers/assumption_controller.go

@@ -0,0 +1,49 @@
+package controllers
+
+import (
+	"net/http"
+
+	"arguehub/services"
+
+	"github.com/gin-gonic/gin"
+)
+
+// GetDebateAssumptions retrieves or generates assumptions for a debate
+func GetDebateAssumptions(c *gin.Context) {
+	debateID := c.Param("debateId")
+
+	if debateID == "" {
+		c.JSON(http.StatusBadRequest, gin.H{
+			"error": "debateId is required",
+		})
+		return
+	}
+
+	// Extract assumptions (will use cache if available)
+	assumptions, err := services.ExtractAssumptions(debateID)
+	if err != nil {
+		// Determine appropriate error code
+		statusCode := http.StatusInternalServerError
+		errorMessage := err.Error()
+
+		// Provide user-friendly messages for common errors
+		if err.Error() == "AI service not available - Gemini client not initialized" {
+			statusCode = http.StatusServiceUnavailable
+			errorMessage = "AI analysis service is currently unavailable. Please ensure the Gemini API is configured."
+		} else if err.Error() == "no transcript found for this debate" {
+			statusCode = http.StatusNotFound
+			errorMessage = "No debate transcript found for this debate ID."
+		}
+
+		c.JSON(statusCode, gin.H{
+			"error": errorMessage,
+		})
+		return
+	}
+
+	// Return assumptions (even if empty)
+	c.JSON(http.StatusOK, gin.H{
+		"assumptions": assumptions,
+		"count":       len(assumptions),
+	})
+}

backend/controllers/debatevsbot_controller.go

@@ -34,7 +34,8 @@ type PhaseTiming struct {
 }
 
 type JudgeRequest struct {
-	History []models.Message `json:"history" binding:"required"`
+	DebateId string           `json:"debateId" binding:"required"`
+	History  []models.Message `json:"history" binding:"required"`
 }
 
 type DebateResponse struct {
@@ -207,8 +208,10 @@ func JudgeDebate(c *gin.Context) {
 	// Judge the debate
 	result := services.JudgeDebate(req.History)
 
-	// Update debate outcome
-	if err := db.UpdateDebateVsBotOutcome(email, result); err != nil {
+	// Update debate outcome using the provided debateId
+	if err := db.UpdateDebateVsBotOutcome(req.DebateId, result); err != nil {
+		log.Printf("Warning: Failed to update debate outcome for %s: %v", req.DebateId, err)
+		// We don't return here because the judging itself succeeded
 	}
 
 	// Get the latest debate information to extract proper details

backend/db/db.go

@@ -10,6 +10,7 @@ import (
 
 	"github.com/redis/go-redis/v9"
 	"go.mongodb.org/mongo-driver/bson"
+	"go.mongodb.org/mongo-driver/bson/primitive"
 	"go.mongodb.org/mongo-driver/mongo"
 	"go.mongodb.org/mongo-driver/mongo/options"
 )
@@ -69,14 +70,25 @@ func SaveDebateVsBot(debate models.DebateVsBot) error {
 	return nil
 }
 
-// UpdateDebateVsBotOutcome updates the outcome of the most recent bot debate for a user
-func UpdateDebateVsBotOutcome(userId, outcome string) error {
-	filter := bson.M{"userId": userId}
+// UpdateDebateVsBotOutcome updates the outcome of a specific bot debate by its ID
+func UpdateDebateVsBotOutcome(debateId, outcome string) error {
+	objID, err := primitive.ObjectIDFromHex(debateId)
+	if err != nil {
+		return fmt.Errorf("invalid debate ID: %w", err)
+	}
+
+	filter := bson.M{"_id": objID}
 	update := bson.M{"$set": bson.M{"outcome": outcome}}
-	_, err := DebateVsBotCollection.UpdateOne(context.Background(), filter, update, nil)
+
+	result, err := DebateVsBotCollection.UpdateOne(context.Background(), filter, update)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to update debate outcome: %w", err)
 	}
+
+	if result.MatchedCount == 0 {
+		return fmt.Errorf("no debate found with ID: %s", debateId)
+	}
+
 	return nil
 }
 
@@ -115,4 +127,4 @@ func ConnectRedis(addr, password string, db int) error {
 
 	log.Println("Connected to Redis")
 	return nil
-}
+}

backend/models/assumption.go

@@ -0,0 +1,33 @@
+package models
+
+import (
+	"encoding/json"
+	"time"
+
+	"go.mongodb.org/mongo-driver/bson/primitive"
+)
+
+// DebateAssumption represents implicit assumptions extracted from a debate
+type DebateAssumption struct {
+	ID               primitive.ObjectID `bson:"_id,omitempty" json:"id,omitempty"`
+	DebateID         string             `bson:"debateId" json:"debateId"`
+	ParticipantID    string             `bson:"participantId,omitempty" json:"participantId,omitempty"`
+	ParticipantEmail string             `bson:"participantEmail,omitempty" json:"participantEmail,omitempty"`
+	Side             string             `bson:"side" json:"side"` // "for" or "against"
+	Assumptions      []string           `bson:"assumptions" json:"assumptions"`
+	CreatedAt        time.Time          `bson:"createdAt" json:"createdAt"`
+}
+
+// MarshalJSON customizes the JSON serialization to convert ObjectID to string
+func (d DebateAssumption) MarshalJSON() ([]byte, error) {
+	type Alias DebateAssumption
+	a := Alias(d)
+	a.ID = primitive.NilObjectID
+	return json.Marshal(&struct {
+		ID string `json:"id"`
+		Alias
+	}{
+		ID:    d.ID.Hex(),
+		Alias: a,
+	})
+}

backend/routes/assumption_routes.go

@@ -0,0 +1,13 @@
+package routes
+
+import (
+	"arguehub/controllers"
+
+	"github.com/gin-gonic/gin"
+)
+
+// SetupAssumptionRoutes sets up routes for debate assumption extraction
+func SetupAssumptionRoutes(router *gin.RouterGroup) {
+	// Get assumptions for a specific debate
+	router.GET("/debates/:debateId/assumptions", controllers.GetDebateAssumptions)
+}