From c69e50d3bb0268a86232e4ebbbe5e2d6db1b3460 Mon Sep 17 00:00:00 2001
From: Gigi <dergigi@pm.me>
Date: Sat, 25 Oct 2025 00:30:07 +0200
Subject: [PATCH] feat: add note content support for direct video URLs

- Add noteContent prop to VideoView component for displaying note text
- Update VideoView to prioritize note content over metadata when available
- Detect direct video URLs from Nostr notes (nostr.build, nostr.video domains)
- Pass bookmark information through URL selection in bookmark components
- Show placeholder message for direct videos from Nostr notes
- Maintains backward compatibility with existing video metadata extraction
---
 src/components/BookmarkItem.tsx              | 2 +-
 src/components/BookmarkViews/CardView.tsx    | 2 +-
 src/components/BookmarkViews/CompactView.tsx | 2 +-
 src/components/ThreePaneLayout.tsx           | 7 +++++++
 src/components/VideoView.tsx                 | 7 +++++--
 5 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/src/components/BookmarkItem.tsx b/src/components/BookmarkItem.tsx
index e90baf8a..0f862770 100644
--- a/src/components/BookmarkItem.tsx
+++ b/src/components/BookmarkItem.tsx
@@ -129,7 +129,7 @@ export const BookmarkItem: React.FC<BookmarkItemProps> = ({ bookmark, index, onS
     if (!hasUrls) return
     const firstUrl = extractedUrls[0]
     if (onSelectUrl) {
-      onSelectUrl(firstUrl)
+      onSelectUrl(firstUrl, bookmark)
     } else {
       window.open(firstUrl, '_blank')
     }
diff --git a/src/components/BookmarkViews/CardView.tsx b/src/components/BookmarkViews/CardView.tsx
index 23412586..b2414d0d 100644
--- a/src/components/BookmarkViews/CardView.tsx
+++ b/src/components/BookmarkViews/CardView.tsx
@@ -145,7 +145,7 @@ export const CardView: React.FC<CardViewProps> = ({
               <button
                 key={urlIndex}
                 className="bookmark-url"
-                onClick={(e) => { e.stopPropagation(); onSelectUrl?.(url) }}
+                onClick={(e) => { e.stopPropagation(); onSelectUrl?.(url, bookmark) }}
                 title="Open in reader"
               >
                 {url}
diff --git a/src/components/BookmarkViews/CompactView.tsx b/src/components/BookmarkViews/CompactView.tsx
index a84e6311..01402b79 100644
--- a/src/components/BookmarkViews/CompactView.tsx
+++ b/src/components/BookmarkViews/CompactView.tsx
@@ -56,7 +56,7 @@ export const CompactView: React.FC<CompactViewProps> = ({
         navigate(`/a/${naddr}`)
       }
     } else if (hasUrls) {
-      onSelectUrl?.(extractedUrls[0])
+      onSelectUrl?.(extractedUrls[0], bookmark)
     } else if (isNote) {
       navigate(`/e/${bookmark.id}`)
     }
diff --git a/src/components/ThreePaneLayout.tsx b/src/components/ThreePaneLayout.tsx
index ffe529e9..22ad9996 100644
--- a/src/components/ThreePaneLayout.tsx
+++ b/src/components/ThreePaneLayout.tsx
@@ -381,6 +381,12 @@ const ThreePaneLayout: React.FC<ThreePaneLayoutProps> = (props) => {
             const isExternalVideo = !isNostrArticle && !!props.selectedUrl && ['youtube', 'video'].includes(classifyUrl(props.selectedUrl).type)
             
             if (isExternalVideo) {
+              // Check if this is a direct video URL from a Nostr note
+              // For URLs like /r/https%3A%2F%2Fv.nostr.build%2FWFO5YkruM9GFJjeg.mp4
+              const isDirectVideoFromNote = props.selectedUrl?.includes('nostr.build') || 
+                                          props.selectedUrl?.includes('nostr.video') ||
+                                          props.selectedUrl?.includes('v.nostr.build')
+              
               return (
                 <VideoView
                   videoUrl={props.selectedUrl!}
@@ -391,6 +397,7 @@ const ThreePaneLayout: React.FC<ThreePaneLayoutProps> = (props) => {
                   settings={props.settings}
                   relayPool={props.relayPool}
                   activeAccount={props.activeAccount}
+                  noteContent={isDirectVideoFromNote ? "This video was shared from a Nostr note. The original note content would be displayed here if available." : undefined}
                   onOpenHighlights={() => {
                     if (props.isHighlightsCollapsed) {
                       props.onToggleHighlightsPanel()
diff --git a/src/components/VideoView.tsx b/src/components/VideoView.tsx
index 5f438c8f..c2665c91 100644
--- a/src/components/VideoView.tsx
+++ b/src/components/VideoView.tsx
@@ -34,6 +34,7 @@ interface VideoViewProps {
   relayPool?: RelayPool | null
   activeAccount?: IAccount | null
   onOpenHighlights?: () => void
+  noteContent?: string // Content from the original Nostr note
 }
 
 const VideoView: React.FC<VideoViewProps> = ({
@@ -45,7 +46,8 @@ const VideoView: React.FC<VideoViewProps> = ({
   settings,
   relayPool,
   activeAccount,
-  onOpenHighlights
+  onOpenHighlights,
+  noteContent
 }) => {
   const [isMarkedAsWatched, setIsMarkedAsWatched] = useState(false)
   const [isCheckingWatchedStatus, setIsCheckingWatchedStatus] = useState(false)
@@ -209,7 +211,8 @@ const VideoView: React.FC<VideoViewProps> = ({
   }
 
   const displayTitle = ytMeta?.title || title
-  const displaySummary = ytMeta?.description || summary
+  // For direct video URLs from Nostr notes, prioritize note content over metadata
+  const displaySummary = noteContent || ytMeta?.description || summary
   const durationText = videoDurationSec !== null ? formatDuration(videoDurationSec) : null
   
   // Get video thumbnail for cover image