-
Notifications
You must be signed in to change notification settings - Fork 5.3k
fix: fetching transcripts to summarize #24902
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: fetching transcripts to summarize #24902
Conversation
|
Thank you for the update! 🎉 We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days. |
Greptile OverviewGreptile SummaryThis PR fixes YouTube transcript fetching by replacing the brittle web scraping approach with YouTube's ANDROID client API. The previous implementation scraped the video page HTML and parsed complex JSON structures, which broke when YouTube changed their internal structure. The new approach directly calls the YouTube API endpoint with proper client identification. Key changes:
Minor issue:
Confidence Score: 4/5
Important Files Changed
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 1 comment
| "node-fetch": "^3.3.2", | ||
| "openai": "^5.1.0", | ||
| "openai": "^6.16.0", | ||
| "youtube-transcript": "^1.2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
youtube-transcript package is added but not imported anywhere in the code
| "youtube-transcript": "^1.2.1", | |
| "ytdl-core": "^4.11.5" |
0xdhrv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, approved 🔥
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
Description
fetching transcripts was broken due to changes on youtubes end. I added a test file to quickly check and work on fixes, as well as actually fixing the issue.
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare located outside the metadata folder if they were not generated with our metadata tool