Return action review author names

src/server.ts

@@ -297,17 +297,29 @@ app.get("/api/sessions/:sessionId/reviews", async (req, res, next) => {
     if (!actionIds.length) return res.json({ reviews: {} });
 
     const [{ data: annotations, error: annotationsError }, { data: notes, error: notesError }] = await Promise.all([
-      supabaseAdmin.from("annotations").select("action_id,score").in("action_id", actionIds),
+      supabaseAdmin.from("annotations").select("action_id,score,user_id").in("action_id", actionIds),
       supabaseAdmin.from("annotations_notes").select("action_id,note").in("action_id", actionIds),
     ]);
     if (annotationsError) throw new Error(`Unable to load action reviews: ${annotationsError.message}`);
     if (notesError) throw new Error(`Unable to load action notes: ${notesError.message}`);
 
-    const reviews: Record<string, { coachScore: number | null; feedback: string[] }> = Object.fromEntries(
-      actionIds.map((actionId) => [actionId, { coachScore: null, feedback: [] }]),
+    const userIds = [...new Set((annotations ?? []).map((annotation) => annotation.user_id).filter((id): id is string => Boolean(id)))];
+    const { data: users, error: usersError } = userIds.length
+      ? await supabaseAdmin.from("users").select("user_id,fname,lname").in("user_id", userIds)
+      : { data: [], error: null };
+    if (usersError) throw new Error(`Unable to load review authors: ${usersError.message}`);
+    const authorNames = new Map((users ?? []).map((user) => [user.user_id, [user.fname, user.lname].filter(Boolean).join(" ").trim()]));
+
+    const reviews: Record<string, { coachScore: number | null; feedback: string[]; feedbackAuthorName: string }> = Object.fromEntries(
+      actionIds.map((actionId) => [actionId, { coachScore: null, feedback: [], feedbackAuthorName: "Unknown user" }]),
     );
     for (const annotation of annotations ?? []) {
-      if (reviews[annotation.action_id]) reviews[annotation.action_id].coachScore = annotation.score == null ? null : Number(annotation.score);
+      if (reviews[annotation.action_id]) {
+        reviews[annotation.action_id].coachScore = annotation.score == null ? null : Number(annotation.score);
+        reviews[annotation.action_id].feedbackAuthorName = annotation.user_id
+          ? authorNames.get(annotation.user_id) || "Former user"
+          : "Former user";
+      }
     }
     for (const note of notes ?? []) reviews[note.action_id]?.feedback.push(note.note);
     return res.json({ reviews });
@@ -342,7 +354,11 @@ app.put("/api/sessions/:sessionId/actions/:actionId/review", async (req, res, ne
       p_notes: [...new Set(input.data.feedback)],
     });
     if (error) throw new Error(`Unable to save action review: ${error.message}`);
-    return res.json({ ok: true });
+    const { data: author, error: authorError } = await supabaseAdmin.from("users")
+      .select("fname,lname").eq("user_id", current.user.id).maybeSingle();
+    if (authorError) throw new Error(`Unable to load review author: ${authorError.message}`);
+    const authorName = [author?.fname, author?.lname].filter(Boolean).join(" ").trim() || "Unknown user";
+    return res.json({ ok: true, authorName });
   } catch (error) { next(error); }
 });