feat: enhance review handling by adding actor URL support across multiple components

crates/adapters/postgres/src/diary.rs

@@ -322,10 +322,11 @@ impl DiaryRepository for PostgresDiaryRepository {
                     to_char(r.watched_at AT TIME ZONE 'UTC', 'YYYY-MM-DD HH24:MI:SS') AS watched_at,
                     to_char(r.created_at AT TIME ZONE 'UTC', 'YYYY-MM-DD HH24:MI:SS') AS created_at,
                     r.remote_actor_url,
-                    COALESCE(u.email, r.remote_actor_url) AS user_email
+                    COALESCE(u.email, a.handle, r.remote_actor_url) AS user_email
              FROM reviews r
              INNER JOIN movies m ON m.id = r.movie_id
              LEFT JOIN users u ON u.id = r.user_id
+             LEFT JOIN ap_remote_actors a ON a.url = r.remote_actor_url
              WHERE {}
              ORDER BY {}
              LIMIT {} OFFSET {}",

crates/adapters/sqlite/src/diary.rs

@@ -291,10 +291,11 @@ impl DiaryRepository for SqliteDiaryRepository {
             "SELECT m.id, m.external_metadata_id, m.title, m.release_year, m.director, m.poster_path,
                     r.id AS review_id, r.movie_id, r.user_id, r.rating, r.comment,
                     r.watched_at, r.created_at, r.remote_actor_url,
-                    COALESCE(u.email, r.remote_actor_url) AS user_email
+                    COALESCE(u.email, a.handle, r.remote_actor_url) AS user_email
              FROM reviews r
              INNER JOIN movies m ON m.id = r.movie_id
              LEFT JOIN users u ON u.id = r.user_id
+             LEFT JOIN ap_remote_actors a ON a.url = r.remote_actor_url
              WHERE {}
              ORDER BY {}
              LIMIT ? OFFSET ?",

crates/api-types/src/diary.rs

@@ -55,9 +55,10 @@ pub struct FeedEntryDto {
     pub movie: MovieDto,
     pub review: ReviewDto,
     pub user_id: Uuid,
-    pub user_email: String,
     pub user_display_name: String,
     pub is_federated: bool,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub actor_url: Option<String>,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize, utoipa::ToSchema)]

crates/presentation/src/mappers/diary.rs

@@ -4,12 +4,17 @@ use domain::models::FeedEntry;
 use super::movies::{movie_to_dto, review_to_dto};
 
 pub fn feed_entry_to_dto(e: &FeedEntry) -> FeedEntryDto {
+    use domain::models::ReviewSource;
+    let actor_url = match e.review().source() {
+        ReviewSource::Remote { actor_url } => Some(actor_url.clone()),
+        ReviewSource::Local => None,
+    };
     FeedEntryDto {
         movie: movie_to_dto(e.movie()),
         review: review_to_dto(e.review()),
         user_id: e.review().user_id().value(),
-        user_email: e.user_email().to_string(),
         user_display_name: e.user_display_name().to_string(),
         is_federated: e.review().is_remote(),
+        actor_url,
     }
 }

spa/src/components/review-card.tsx

@@ -12,9 +12,10 @@ type ReviewCardProps = {
   userName?: string
   userId?: string
   isFederated?: boolean
+  actorUrl?: string
 }
 
-export function ReviewCard({ movie, review, userName, userId, isFederated }: ReviewCardProps) {
+export function ReviewCard({ movie, review, userName, userId, isFederated, actorUrl }: ReviewCardProps) {
   return (
     <Card size="sm">
       <CardContent className="flex gap-3">
@@ -28,6 +29,10 @@ export function ReviewCard({ movie, review, userName, userId, isFederated }: Rev
                 <Link to="/users/$id" params={{ id: userId }} className="relative z-10 font-semibold text-primary">
                   {userName}
                 </Link>
+              ) : actorUrl ? (
+                <a href={actorUrl} target="_blank" rel="noopener noreferrer" className="relative z-10 font-semibold text-primary">
+                  {userName}
+                </a>
               ) : (
                 <span>{userName}</span>
               )}

spa/src/lib/api/diary.ts

@@ -29,9 +29,9 @@ export const feedEntryDtoSchema = z.object({
   movie: movieDtoSchema,
   review: reviewDtoSchema,
   user_id: z.string().uuid(),
-  user_email: z.string(),
   user_display_name: z.string(),
   is_federated: z.boolean(),
+  actor_url: z.string().optional(),
 })
 export type FeedEntryDto = z.infer<typeof feedEntryDtoSchema>
 

spa/src/routes/_app/index.tsx

@@ -115,8 +115,9 @@ function FeedTab() {
                 movie={entry.movie}
                 review={entry.review}
                 userName={entry.user_display_name}
-                userId={entry.user_id}
+                userId={entry.is_federated ? undefined : entry.user_id}
                 isFederated={entry.is_federated}
+                actorUrl={entry.actor_url}
               />
             )
             return entry.user_id === auth?.user_id ? (