Skip to content

feat: add per-segment auto-skip for intro/recap/outro#1255

Open
siriusvoid wants to merge 4 commits into
NuvioMedia:cmp-rewritefrom
siriusvoid:feat/auto-skip-intro-outro
Open

feat: add per-segment auto-skip for intro/recap/outro#1255
siriusvoid wants to merge 4 commits into
NuvioMedia:cmp-rewritefrom
siriusvoid:feat/auto-skip-intro-outro

Conversation

@siriusvoid

@siriusvoid siriusvoid commented Jun 4, 2026

Copy link
Copy Markdown

Summary

Adds automatic skipping for detected segments, matching the TV app. Each segment type
(intro, recap, outro) gets its own toggle, so you can leave it on the manual skip
button or have it skip on its own. Everything's off by default.

PR type

  • Reproducible bug fix
  • UI glitch/bug fix
  • Behavior bug/regression fix
  • Small maintenance only, with no UI or behavior change
  • Docs accuracy fix
  • Translation/localization only
  • Approved larger or directional change

Why

Skipping intro, recap, and outro currently means pressing the skip button every time,
which gets tedious over a long binge. This adds a per segment choice so users can keep
skipping manually or skip automatically.

Issue or approval

Fixes #951

UI / behavior impact

  • No UI change
  • No behavior change
  • UI changed only to fix a documented glitch/bug
  • Behavior changed only to fix a documented bug/regression
  • UI change has explicit maintainer approval
  • Behavior change has explicit maintainer approval

Policy check

  • I have read and understood CONTRIBUTING.md.
  • This PR is small, focused, and limited to one problem.
  • This PR is not cosmetic-only.
  • Any UI change fixes a linked glitch/bug and includes visual proof, or this PR has no UI change.
  • Any behavior change fixes a linked bug/regression or has explicit approval, or this PR has no behavior change.
  • This PR does not bundle unrelated refactors, cleanups, formatting, or drive-by changes.
  • This PR does not add dependencies, architecture changes, migrations, or product-direction changes without explicit approval.
  • I listed the testing performed below.

Scope boundaries

  • Auto skip only adds a single seek past the active segment, driven by playback position, when its toggle is on; segment detection and skip interval data are unchanged
  • Each segment is skipped at most once per playback, only after the initial seek and with a small end margin so it can't overshoot at startup; suppressed while the paused overlay is shown
  • When a segment is set to skip automatically, its skip button is hidden to avoid a flash before the seek; otherwise the skip button behaves as before
  • Toggles are gated on the existing Skip Intro setting and default to off
  • Next episode auto play logic is untouched

Testing

  • Auto Skip Intro on -> intro segment skipped automatically
  • Auto Skip Recap on -> recap segment skipped automatically
  • Auto Skip Outro on -> outro segment skipped automatically
  • All toggles off -> nothing auto-skips; manual skip button shown as before
  • No timecodes -> plays normally
  • Tested on iOS emulator via Xcode and on my Android phone with a debug build

Screenshots / Video (UI changes only)

Simulator Screenshot - iPhone 17 Pro - 2026-06-04 at 18 54 15

Breaking changes

None

Linked issues

Fixes #951

@siriusvoid siriusvoid force-pushed the feat/auto-skip-intro-outro branch 2 times, most recently from 3c25b48 to 3a4fad3 Compare June 8, 2026 03:23
@siriusvoid siriusvoid force-pushed the feat/auto-skip-intro-outro branch from 0ef8e78 to 692fec1 Compare June 8, 2026 03:51
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.

[Feature]: Auto Skip Intro/Outro

1 participant