hermes - 💡(How to fix) Fix YouTube Shorts filter not working in youtube-content skill

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Error Message

  1. Processing continues (no early exit or error for Shorts) Shorts URLs should be detected and rejected early with a clear error message (e.g. "Shorts are not supported in this app").
  • If the original input URL contains "/shorts/", raise an error / return early.

Code Example

patterns = [
    r'(?:v=|youtu\.be/|shorts/|embed/|live/)([a-zA-Z0-9_-]{11})',
    r'^([a-zA-Z0-9_-]{11})$',
]
RAW_BUFFERClick to expand / collapse

Bug Description

The youtube-content skill should exclude YouTube Shorts, but the filter is not implemented / not working.

Code Location

skills/media/youtube-content/scripts/fetch_transcript.py

Relevant code in extract_video_id():

patterns = [
    r'(?:v=|youtu\.be/|shorts/|embed/|live/)([a-zA-Z0-9_-]{11})',
    r'^([a-zA-Z0-9_-]{11})$',
]

The regex explicitly matches /shorts/ paths and accepts them the same as normal videos. There is no rejection logic for Shorts.

Steps to Reproduce

  1. Call the transcript fetcher with a Shorts URL (e.g. https://youtube.com/shorts/VIDEO_ID)
  2. extract_video_id successfully returns the ID
  3. Processing continues (no early exit or error for Shorts)

Expected Behavior

Shorts URLs should be detected and rejected early with a clear error message (e.g. "Shorts are not supported in this app").

Actual Behavior

Shorts are processed identically to regular videos.

Suggested Fix

After extracting the video ID, add a check:

  • If the original input URL contains "/shorts/", raise an error / return early.
  • Consider removing "shorts/" from the supported patterns entirely.

Environment

  • Hermes Agent (default profile on VPS)
  • youtube-content skill
  • NoteOps Telegram bot usage

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING