From 872954d5ced555094998b851f67fb4f387421022 Mon Sep 17 00:00:00 2001
From: padreug <padreug@aiolabs.dev>
Date: Sat, 20 Sep 2025 15:40:43 +0200
Subject: [PATCH] Implement centralized collapse state management in NostrFeed
 component

- Introduced a centralized mechanism for managing the collapse state of posts, allowing for automatic collapsing of posts with more than 2 direct replies.
- Enhanced the ThreadedPost component to utilize the centralized collapse state, improving the visibility and interaction of nested replies.
- Added cascading collapse functionality to ensure that all descendant posts are collapsed when a parent post is collapsed.

These changes contribute to a more organized and user-friendly experience within the NostrFeed module.
---
 .../nostr-feed/components/NostrFeed.vue       | 69 ++++++++++++++++++-
 .../nostr-feed/components/ThreadedPost.vue    | 25 +++++--
 2 files changed, 86 insertions(+), 8 deletions(-)

diff --git a/src/modules/nostr-feed/components/NostrFeed.vue b/src/modules/nostr-feed/components/NostrFeed.vue
index b4a5fa6..7dc799f 100644
--- a/src/modules/nostr-feed/components/NostrFeed.vue
+++ b/src/modules/nostr-feed/components/NostrFeed.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { computed, watch } from 'vue'
+import { computed, watch, ref } from 'vue'
 import { Button } from '@/components/ui/button'
 import { Megaphone, RefreshCw, AlertCircle } from 'lucide-vue-next'
 import { useFeed } from '../composables/useFeed'
@@ -34,6 +34,32 @@ const { posts: notes, threadedPosts, isLoading, error, refreshFeed } = useFeed({
   contentFilters: props.contentFilters
 })
 
+// Centralized collapse state management
+const collapsedPosts = ref(new Set<string>())
+
+// Initialize collapsed state for posts with many replies
+watch(threadedPosts, (newPosts) => {
+  if (newPosts.length > 0) {
+    const newCollapsed = new Set(collapsedPosts.value)
+
+    // Auto-collapse posts with more than 2 direct replies
+    const addCollapsedPosts = (posts: any[]) => {
+      posts.forEach(post => {
+        if ((post.replies?.length || 0) > 2) {
+          newCollapsed.add(post.id)
+        }
+        // Recursively check nested replies
+        if (post.replies?.length > 0) {
+          addCollapsedPosts(post.replies)
+        }
+      })
+    }
+
+    addCollapsedPosts(newPosts)
+    collapsedPosts.value = newCollapsed
+  }
+}, { immediate: true })
+
 // Use profiles service for display names
 const { getDisplayName, fetchProfiles } = useProfiles()
 
@@ -103,6 +129,44 @@ async function onToggleLike(note: FeedPost) {
     console.error('Failed to toggle like:', error)
   }
 }
+
+// Handle collapse toggle with cascading behavior
+function onToggleCollapse(postId: string) {
+  const newCollapsed = new Set(collapsedPosts.value)
+
+  if (newCollapsed.has(postId)) {
+    // Expand this post (remove from collapsed set)
+    newCollapsed.delete(postId)
+  } else {
+    // Collapse this post (add to collapsed set)
+    newCollapsed.add(postId)
+
+    // Find all descendant posts and collapse them too (cascading collapse)
+    const collapseDescendants = (posts: any[], targetId: string) => {
+      posts.forEach(post => {
+        if (post.id === targetId && post.replies) {
+          // Found the target post, collapse all its descendants
+          const addAllDescendants = (replies: any[]) => {
+            replies.forEach(reply => {
+              newCollapsed.add(reply.id)
+              if (reply.replies?.length > 0) {
+                addAllDescendants(reply.replies)
+              }
+            })
+          }
+          addAllDescendants(post.replies)
+        } else if (post.replies?.length > 0) {
+          // Keep searching in nested replies
+          collapseDescendants(post.replies, targetId)
+        }
+      })
+    }
+
+    collapseDescendants(threadedPosts.value, postId)
+  }
+
+  collapsedPosts.value = newCollapsed
+}
 </script>
 
 <template>
@@ -181,8 +245,11 @@ async function onToggleLike(note: FeedPost) {
             :get-display-name="getDisplayName"
             :get-event-reactions="getEventReactions"
             :depth="0"
+            :parent-collapsed="false"
+            :collapsed-posts="collapsedPosts"
             @reply-to-note="onReplyToNote"
             @toggle-like="onToggleLike"
+            @toggle-collapse="onToggleCollapse"
           />
         </div>
       </div>
diff --git a/src/modules/nostr-feed/components/ThreadedPost.vue b/src/modules/nostr-feed/components/ThreadedPost.vue
index de31096..4cb340e 100644
--- a/src/modules/nostr-feed/components/ThreadedPost.vue
+++ b/src/modules/nostr-feed/components/ThreadedPost.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { ref, computed } from 'vue'
+import { computed } from 'vue'
 import { Badge } from '@/components/ui/badge'
 import { Button } from '@/components/ui/button'
 import { formatDistanceToNow } from 'date-fns'
@@ -12,22 +12,30 @@ interface Props {
   getDisplayName: (pubkey: string) => string
   getEventReactions: (eventId: string) => { likes: number; userHasLiked: boolean }
   depth?: number
+  parentCollapsed?: boolean
+  collapsedPosts?: Set<string>
 }
 
 interface Emits {
   (e: 'reply-to-note', note: { id: string; content: string; pubkey: string }): void
   (e: 'toggle-like', note: FeedPost): void
+  (e: 'toggle-collapse', postId: string): void
 }
 
 const props = withDefaults(defineProps<Props>(), {
   adminPubkeys: () => [],
-  depth: 0
+  depth: 0,
+  parentCollapsed: false,
+  collapsedPosts: () => new Set<string>()
 })
 
 const emit = defineEmits<Emits>()
 
-// Collapse state for this post's replies - auto-collapse if more than 2 direct replies
-const isCollapsed = ref((props.post.replies?.length || 0) > 2)
+// Check if this post is collapsed based on centralized state
+const isCollapsed = computed(() => props.collapsedPosts?.has(props.post.id) || false)
+
+// Check if this post should be visible (not hidden by parent collapse)
+const isVisible = computed(() => !props.parentCollapsed)
 
 // Check if this is an admin post
 const isAdminPost = computed(() => props.adminPubkeys.includes(props.post.pubkey))
@@ -47,9 +55,9 @@ const replyCount = computed(() => {
   return countReplies(props.post)
 })
 
-// Toggle collapse state
+// Toggle collapse state - emit to parent for centralized management
 function toggleCollapse() {
-  isCollapsed.value = !isCollapsed.value
+  emit('toggle-collapse', props.post.id)
 }
 
 // Handle reply action
@@ -101,7 +109,7 @@ function getRideshareType(post: FeedPost): string {
 </script>
 
 <template>
-  <div class="relative">
+  <div v-if="isVisible" class="relative">
     <!-- Vertical line connecting to parent (for nested replies) -->
     <div
       v-if="depth > 0"
@@ -241,8 +249,11 @@ function getRideshareType(post: FeedPost): string {
           :get-display-name="getDisplayName"
           :get-event-reactions="getEventReactions"
           :depth="depth + 1"
+          :parent-collapsed="isCollapsed"
+          :collapsed-posts="collapsedPosts"
           @reply-to-note="$emit('reply-to-note', $event)"
           @toggle-like="$emit('toggle-like', $event)"
+          @toggle-collapse="$emit('toggle-collapse', $event)"
         />
 
         <!-- Show "Load more replies" button when collapsed and there are more than 2 replies -->