refactor: simplify caption generation by directly creating captions from word groups, removing unnecessary context logic after feedback

Author: aman-imagekit

packages/video-player/javascript/modules/subtitles/subtitles.ts

@@ -645,56 +645,21 @@ async function generateCaptionsFromVTT(params: {
         const captions: CaptionData[] = [];
         const maxTimeGap = MAX_TIME_GAP_SECONDS;
 
-        // First, generate all word groups from all segments
-        const allWordGroups: Array<Array<TimedWord>> = [];
+        // Generate word groups from segments and create captions directly
         syntheticSegments.forEach((segment) => {
             const words = segment.words;
             const sentenceLength = segment.transcript.length;
             const wordGroups = groupWordsByChars(words, sentenceLength, maxChars, maxTimeGap);
 
+            // Create captions directly from word groups
             wordGroups.forEach(group => {
-                allWordGroups.push(group);
-            });
-        });
-
-        // Now process each word group with context from previous/next word groups
-        allWordGroups.forEach((currentGroup, groupIndex) => {
-            const currentText = currentGroup.map(w => w.word).join(' ');
-            
-            // Get previous word group text if available and within time gap
-            let previousText: string | null = null;
-            if (groupIndex > 0) {
-                const previousGroup = allWordGroups[groupIndex - 1];
-                const timeGap = currentGroup[0].start - previousGroup[previousGroup.length - 1].end;
-                if (timeGap < maxTimeGap) {
-                    previousText = previousGroup.map(w => w.word).join(' ');
-                }
-            }
-            
-            // Get next word group text if available and within time gap
-            let nextText: string | null = null;
-            if (groupIndex < allWordGroups.length - 1) {
-                const nextGroup = allWordGroups[groupIndex + 1];
-                const timeGap = nextGroup[0].start - currentGroup[currentGroup.length - 1].end;
-                if (timeGap < maxTimeGap) {
-                    nextText = nextGroup.map(w => w.word).join(' ');
+                if (group.length > 0) {
+                    captions.push({
+                        startTime: group[0].start,
+                        endTime: group[group.length - 1].end,
+                        text: group.map(w => w.word).join(' ')
+                    });
                 }
-            }
-            
-            // Build caption text with context
-            let text = currentText;
-            if (previousText || nextText) {
-                const contextLines: string[] = [];
-                // if (previousText) contextLines.push(previousText);
-                contextLines.push(currentText);
-                // if (nextText) contextLines.push(nextText);
-                text = contextLines.join('\n');
-            }
-            
-            captions.push({
-                startTime: currentGroup[0].start,
-                endTime: currentGroup[currentGroup.length - 1].end,
-                text
             });
         });