fix(stt): add fallback for ffmpeg

[waleedlatif1]

Summary

• add fallback for ffmpeg

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Nov 20, 2025 5:45am

[greptile-apps]

Greptile Summary

• Enhanced ffmpeg initialization with binary verification using fsSync.accessSync before falling back to system ffmpeg
• Changed error handling to throw module-level errors instead of silent warnings when no ffmpeg binary is found
• Added improved logging with specific installation instructions for Alpine Linux

Confidence Score: 3/5

• This PR improves ffmpeg detection but introduces a Windows compatibility issue that will cause failures
• The enhanced fallback logic is good, but using which ffmpeg breaks Windows compatibility. The module-level error throwing is also more strict but appropriate for catching configuration issues early
• Pay close attention to apps/sim/lib/audio/extractor.ts for the Windows compatibility fix

Important Files Changed

Filename	Overview
apps/sim/lib/audio/extractor.ts	Improved ffmpeg fallback logic with verification, but has Windows compatibility issue and throws at module load time if no binary found

Sequence Diagram

sequenceDiagram
    participant User
    participant API as "STT Proxy API"
    participant Extractor as "Audio Extractor"
    participant FFmpeg as "FFmpeg Binary"
    participant System as "System FFmpeg"

    User->>API: POST /api/proxy/stt (video file)
    API->>Extractor: extractAudioFromVideo(buffer, mimeType)
    
    Note over Extractor: Module initialization
    Extractor->>FFmpeg: Check ffmpeg-static binary exists
    alt ffmpeg-static exists and executable
        Extractor->>Extractor: Use ffmpeg-static path
    else ffmpeg-static not found
        Extractor->>System: execSync("which ffmpeg")
        alt system ffmpeg found
            Extractor->>Extractor: Use system ffmpeg path
        else no ffmpeg found
            Extractor-->>API: throw Error("No FFmpeg binary found")
            API-->>User: 500 error
        end
    end
    
    Extractor->>FFmpeg: ffmpeg.setFfmpegPath(path)
    Extractor->>FFmpeg: Convert video to audio
    FFmpeg-->>Extractor: Audio buffer
    Extractor-->>API: AudioExtractionResult
    API-->>User: Transcription response

Loading

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format}