Render with bubbles working

utils/AvatarRender.js

@@ -2,6 +2,7 @@ import { spawn } from 'child_process';
 import os from 'os';
 import path from 'path';
 import fs from 'fs';
+import Bubble from './bubble/Bubble.js';
 
 const ffmpegLocation = os.platform() === 'win32' ? 'ffmpeg.exe' : 'ffmpeg';
 
@@ -61,6 +62,7 @@ export class AvatarRenderer {
 
       // Burn captions for this section
       const captionFile = section.audioCaptionFile;
+      let sourceForBubbles;
       if (captionFile && captionFile.endsWith('.ass')) {
         const subbedFile = path.join(outDir, `${jobId}_section${i}_subbed.mp4`);
         const esc = captionFile.replace(/\\/g, '/').replace(/:/g, '\\:');
@@ -73,9 +75,25 @@ export class AvatarRenderer {
         onLog && onLog(`Burning captions for section ${i}...\n`);
         await this.runFFmpeg(subArgs, controller, onLog);
         tempFiles.push(subbedFile);
-        sectionCaptionedVideos.push(subbedFile);
+        sourceForBubbles = subbedFile;
       } else {
-        sectionCaptionedVideos.push(sectionVideo);
+        sourceForBubbles = sectionVideo;
+      }
+
+      // Apply bubbles if present on this section
+      if (section.bubbles && Array.isArray(section.bubbles) && section.bubbles.length) {
+        const bubbledFile = path.join(outDir, `${jobId}_section${i}_bubbled.mp4`);
+        try {
+          onLog && onLog(`Applying ${section.bubbles.length} bubble(s) to section ${i}...\n`);
+          await Bubble.makeBubble(sourceForBubbles, section.bubbles, bubbledFile, onLog);
+          tempFiles.push(bubbledFile);
+          sectionCaptionedVideos.push(bubbledFile);
+        } catch (e) {
+          onLog && onLog(`Bubble application failed for section ${i}, using original: ${e}\n`);
+          sectionCaptionedVideos.push(sourceForBubbles);
+        }
+      } else {
+        sectionCaptionedVideos.push(sourceForBubbles);
       }
     }
 
@@ -128,9 +146,21 @@ export class AvatarRenderer {
       }
     }
 
-    // Cleanup intermediate files
+    // Cleanup intermediate files — keep only the final resultFile and the manuscript JSON
+    const manuscriptPath = path.join(process.cwd(), 'public', 'original_manuscript.json');
     for (const f of tempFiles) {
-      // try { if (fs.existsSync(f)) fs.unlinkSync(f); } catch (_) { }
+      if (!f) continue;
+      try {
+        const abs = path.resolve(f);
+        // never delete the final result file or the manuscript file
+        if (abs === path.resolve(resultFile)) continue;
+        if (abs === path.resolve(manuscriptPath)) continue;
+        if (fs.existsSync(abs)) {
+          try { fs.unlinkSync(abs); onLog && onLog(`Deleted temp file: ${abs}\n`); } catch (e) { onLog && onLog(`Failed to delete temp file ${abs}: ${e}\n`); }
+        }
+      } catch (e) {
+        onLog && onLog(`Error while cleaning temp file ${f}: ${e}\n`);
+      }
     }
 
     return resultFile;

utils/bubble/Bubble.js

@@ -553,9 +553,9 @@ export async function test() {
     toSec: 5.0
   };
 
-  // Additional sample: simple-top-center-image uses an image or video asset centered at top with text
+  // Additional sample: simple-top-center-media uses an image or video asset centered at top with text
   const imageSample = {
-    templateName: 'simple-top-center-image',
+    templateName: 'simple-top-center-media',
     // provide mediaAbsPath (what makeBubble expects) pointing to an image
     mediaAbsPath: path.join(cwd, 'public', 'media2.png'),
     bubbleText: { text: 'IMAGE ABOVE, TEXT CENTERED' },
@@ -614,4 +614,4 @@ export async function test() {
   }
 }
 
-test()
+// test()

utils/bubble/bubble-templates.js

@@ -26,7 +26,7 @@ export const BaseBubbleTemplates = {
   // image-only variant: the caller must still provide the path when using this
   // template. animation defaults are pulled up to the top level so they survive
   // the case where the user specifies `mediaAbsPath` as a simple string.
-  'simple-top-center-image': {
+  'simple-top-center-media': {
     borderRadius: 20,
     normalizeAudio: true,
     audioEffectFile: 'woosh',