fix(desktop): YouTube ringtone の yt-dlp/ffmpeg を絶対パスで解決#260
Conversation
GUI 起動の Electron プロセスでは login shell PATH の取り込みが 不安定で、ffmpeg/ffprobe がインストール済みでも見つからないこ とがあった。 - 必要なバイナリを env.PATH と /opt/homebrew/bin 等のフォール バックディレクトリから絶対パスで解決 - spawn には絶対パスを渡し、yt-dlp には --ffmpeg-location を 明示 - 解決失敗時のエラーメッセージに「ログインシェル PATH を確認」 と追記
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 18 minutes and 12 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
概要
PR #259 でマージした YouTube 通知音機能で、Homebrew でインストール済みの
ffmpeg/ffprobeが「Missing required tool(s)」と判定されてしまう問題を修正します。原因
GUI から起動した Electron プロセスでは login-shell PATH の取り込みが不安定で、
spawnに渡した env 経由の PATH 解決でバイナリを見つけられないケースがありました。修正
yt-dlp/ffmpeg/ffprobe) を env.PATH と/opt/homebrew/bin等のフォールバックディレクトリから絶対パスで解決spawnには絶対パスを渡し、yt-dlp には--ffmpeg-locationを明示テスト計画
yt-dlp/ffmpegが GUI 起動時にも検出されるFrom YouTubeから通知音をクリップ生成できるRefs #258