-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Update summarize-youtube-video-with-ai extension #24952
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
Update summarize-youtube-video-with-ai extension #24952
Conversation
- fix transcription, remove deprecated command - fixes - Pull contributions - fix youtube transcripts
|
Thank you for your contribution! 🎉 🔔 @iKasch @bl1nk you might want to have a look. You can use this guide to learn how to check out the Pull Request locally in order to test it. 📋 Quick checkout commandsBRANCH="ext/summarize-youtube-video-with-ai"
FORK_URL="https://github.com/mateusbadalotti/raycast-extensions.git"
EXTENSION_NAME="summarize-youtube-video-with-ai"
REPO_NAME="raycast-extensions"
git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run devWe'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 a critical bug where YouTube transcripts were returning empty due to incorrect XML parsing for YouTube's srv3 format. The previous regex only matched direct content in The PR also removes the deprecated unified "Summarize YouTube Video" command and its associated view component, completing the migration to provider-specific commands that was introduced earlier.
Confidence Score: 5/5
Important Files Changed
|
|
@mateusbadalotti looks good to me, thank you! |
|
Sure! done @iKasch |
pernielsentikaer
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: |
|
🎉 🎉 🎉 Such a great contribution deserves a reward, but unfortunately we couldn't find your Raycast account based on your GitHub username (@mateusbadalotti). Please link your GitHub account to your Raycast account to receive your credits and soon be able to exchange them for some swag. |
Description
Fixes #23718
srv3format by extracting text from<s>segments instead of<p>tagsYouTube’s
timedtextAPI returns captions in srv3 XML format, where text is nested inside<s>(segment) tags within<p>(paragraph) tags:The previous regex only matched direct content in
tags, resulting in empty transcripts. This fix:
<s>tags (primary)<text>tags for older formatsAlso removed the deprecated unified command since provider-specific commands are now the recommended approach.
Screencast
Extensions works properly again:
image.mp4
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder