Skip to content

fix: Inaccurate Summary When Youtube Videos Have No Transcript Enabled#148

Merged
steipete merged 2 commits intosteipete:mainfrom
bytrangle:fix_no-summary-when-yt-videos-have-no-transcript
Mar 12, 2026
Merged

fix: Inaccurate Summary When Youtube Videos Have No Transcript Enabled#148
steipete merged 2 commits intosteipete:mainfrom
bytrangle:fix_no-summary-when-yt-videos-have-no-transcript

Conversation

@bytrangle
Copy link
Contributor

#fixes #140

Problem Description

When given a Youtube video without CC (Closed Caption) enabled, this app will try to use yt-dlp to download the video, then use a model to transcribe and summarize it. However, if the user specifies Gemini as the model, yt-dlp and transcription are skipped and the app defaults to a summary based on the HTML page description.

Root Cause

geminiApiKey wasn't passed to transcription object when calling createLinkPreviewClient in runUrlFlow. This caused resolveTranscriptionConfig to resolve geminiApiKey as null, making hasAnyProvider false and therefore canRunYtDlp false. So yt-dlp is never started.

Test the new changes

  1. Clear all the transcription and summary cache
rm -f ~/.summarize/cache.sqlite
rm -rf ~/.summarize/cache
  1. At the root directory, run build script with pnpm build
  2. Test a Youtube video without CC, bypassing summary cache and media cache
pnpm run summarize "https://www.youtube.com/watch?v=hhAbp3iQA44" --no-cache --no-media-cache

@steipete steipete force-pushed the fix_no-summary-when-yt-videos-have-no-transcript branch from 2d3c6c4 to 8994739 Compare March 12, 2026 23:38
@steipete steipete merged commit fc13e2c into steipete:main Mar 12, 2026
1 check passed
@steipete
Copy link
Owner

Landed via temp rebase onto main.

  • Gate: pnpm -s check
  • Land commit: 8994739986dd9caaddc9be4884a9d8be156e1b4c
  • Merge commit: fc13e2c9b87c7cb7851701018d36a156c0b151f5

Thanks @bytrangle!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Summarization Is Incomplete for Youtube Videos Without CC Enabled

2 participants