Skip to content

feat: Add Celluloid support (with ani-skip) #1504

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Vvamp
Copy link

@Vvamp Vvamp commented Apr 15, 2025

Type of change

  • Bug fix
  • Feature
  • Documentation update

Description

Celluloid is a popular MPV frontend. Since celluloid can pass mpv arguments by using a 'mpv-' prefix in the command args, I figured it'd be easy to add. It just uses mpv on the backend after all.

I opted for not adding a new case in the switch, but instead adding the "celluloid" pattern in the mpv option. I then added an optional prefix, which is set if the player is celluloid. It could be done as a seperate option, but I wanted to prevent duplicate code.

I bumped the version to 4.10.0 as I noticed that is what you guys were aiming for in the next release. I checked 'range selection' as working, as you guys are already aware of the pre-existing bug with range selection and it's not related to this PR. The readme and help still mention that --skip only works with mpv, but since celluloid is still just mpv, I thought it was still correct.

Note: To make this work the user would have to copy ani-skip's skip.lua to the scripts folder under ~/.config/celluloid/scripts/ instead of ~/.config/mpv/scripts/.

Checklist

  • any anime playing
  • bumped version

  • next, prev and replay work
  • -c history and continue work
  • -d downloads work
  • -s syncplay works
  • -q quality works
  • -v vlc works
  • -e select episode works
  • -S select index works
  • -r range selection works
  • --skip ani-skip works
  • --skip-title ani-skip title argument works
  • --no-detach no detach works
  • --dub and regular (sub) mode both work
  • all providers return links (not necessarily on a single anime, use debug mode to confirm)

  • -h help info is up to date
  • Readme is up to date
  • Man page is up to date

@Vvamp Vvamp requested a review from Derisis13 as a code owner April 15, 2025 23:53
@Derisis13
Copy link
Collaborator

Thanks for the dettailed explanation and thoughtful implementation.
I'd like to see the readme and the manpage updated to reflect the new capability.
Also please iron out that one shellcheck complaint you seem to have missed.

Then I'll test it myself too, and see if it works. Merging may have to wait until 4.10 is merged too, but maybe we can include the feature in the release.

@Vvamp
Copy link
Author

Vvamp commented Apr 17, 2025

Thanks for the quick reply.
I added the capabilities to the README, help and manpage, as well as a small note to the README's ani-skip section to explain how to get ani-skip to work with celluloid. Let me know if you need me to make any other adjustments.

@Vvamp Vvamp force-pushed the feature/celluloid branch from f4a016d to cf48c97 Compare April 17, 2025 13:43
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.

2 participants