Added deno for YTMusic Provider #2636
Merged
+2,490
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Could you remove the entry of uv.lock in the gitignore? This contains a snapshot of the dependencies we use, so it makes sense it changes if you add deno. |
|
I've closed my MR In favor of this. Tested and works fine: |
Contributor
Author
|
Thanks for the guidance Marvin and thanks for testing the fix Romulo! I have made the uv.lock commit so hopefully that wraps this up nicely. |
15 tasks
MarvinSchenkel
approved these changes
Nov 13, 2025
Contributor
|
Thanks @Dnny44 |
MarvinSchenkel
pushed a commit
that referenced
this pull request
Nov 14, 2025
* Added deno for YTMusic Provider * Commiting uv lock
|
not sure if I'm doing something wrong. I just downloaded the stable MA add-on update 2.6.3 through the HA updater I tried removing and re-installing the YT Music PO Token Generator add-on |
Contributor
|
UPDATE REGARDING YT MUSIC: We just released 2.6.3 with the fixes for YT Music. However, the required JS runtime that is used to solve JS challenges for YTM does NOT work with the architecture we use for stable. As of 2.7.0, Music Assistant will use a new base image architecture that does support this. This unfortuantely means that YTM cannot be fixed in the stable channel for now and will only work again starting 2.7.0. YT Music currently only works in the beta releases. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A JavaScript runtime is now required for the Youtube Music provider to work, deno is the recommended and the default setting on yt-dlp: https://github.com/yt-dlp/yt-dlp/wiki/EJS
This fixes music-assistant/support#4348
There is another Pull Request [https://github.com//pull/2634] for this as well but based on @MarvinSchenkel commit history I think this is the preferred method of adding the dependency.