Skip to content

fix(video): improve local YouTube workflow - #74

Merged
caiopizzol merged 11 commits into
mainfrom
fix/transcript-guided-frames
Aug 12, 2026
Merged

fix(video): improve local YouTube workflow#74
caiopizzol merged 11 commits into
mainfrom
fix/transcript-guided-frames

Conversation

@caiopizzol

@caiopizzol caiopizzol commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • download YouTube videos through the user's signed-in browser
  • return one concrete sign-in action when authenticated access is unavailable
  • transcribe locally with Whisper before selecting transcript-relevant frames
  • disable carried Whisper context to prevent loops in long recordings
  • add audio-only, frame-only, and exact frame-time video preparation
  • inspect five transcript-targeted frames, or five evenly spaced frames without usable transcript cues

Validation

  • bun run check
  • yt-dlp --cookies-from-browser chrome retrieved metadata for 7DRO4rEIHDk
  • full local Whisper transcription with large-v3-turbo
  • transcript-guided inspection of three exact frames
  • context-free forward test from an isolated skill installation: authenticated download, 660 timestamped
    segments through 50:45.8, no repeated-segment pattern, and three transcript-targeted frames
  • full-transcript forward test: 659 segments reviewed, five targeted frames inspected, one missed transition
    replaced
  • no-audio forward test: five evenly spaced frames inspected at 0, 1.475, 2.95, 4.425, and 5.9

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 9 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread skills/files/read-video/SKILL.md Outdated
Comment thread apps/video-tools-cli/src/arguments.ts
Comment thread apps/video-tools-cli/src/prepare-video.ts

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 8 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread skills/files/transcribe-audio/scripts/transcribe-groq.ts Outdated
Comment thread skills/files/transcribe-audio/scripts/transcribe-groq.ts Outdated
Comment thread scripts/tests/transcribe-audio.test.ts Outdated
Comment thread skills/files/transcribe-audio/scripts/transcribe-groq.ts Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 10 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.
Tip: cubic used a learning from your PR history. Let your coding agent read cubic learnings directly with the cubic MCP.

Re-trigger cubic

Comment thread skills/files/transcribe-audio/scripts/transcribe-whisper.ts Outdated
Comment thread skills/files/transcribe-audio/scripts/transcribe-whisper.ts Outdated
Comment thread skills/files/transcribe-audio/scripts/transcribe-whisper.ts
Comment thread skills/files/transcribe-audio/scripts/transcribe-whisper.ts Outdated
Comment thread skills/files/transcribe-audio/SKILL.md Outdated
@caiopizzol caiopizzol changed the title fix(video): target frames from transcripts fix(video): improve local YouTube workflow Aug 12, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 2 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread skills/media/download-youtube-video/references/browser-access.md Outdated
@caiopizzol
caiopizzol merged commit 7d741e2 into main Aug 12, 2026
2 checks passed
@caiopizzol
caiopizzol deleted the fix/transcript-guided-frames branch August 12, 2026 18:07
This was referenced Aug 10, 2026
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.

1 participant