Skip to content

Conversation

@MattiaPun
Copy link
Contributor

This change updates the artist separator in metadata tags from "," to ";". Using a comma for joined artist names results in the full string being treated as one literal artist in many media players and library systems.

This change brings metadata output in line with widely-used tagging conventions (Subsconic/Navidrome, Jellyfin, Plex, etc.)

itisFarzin added a commit to itisFarzin/streamrip that referenced this pull request Nov 13, 2025
Inspired by [1]

[1] - nathom#903

Signed-off-by: Farzin Kazemzadeh <[email protected]>
@itisFarzin
Copy link
Contributor

itisFarzin commented Nov 13, 2025

Not limiting this change to only Tidal, and album artists is a better idea: itisFarzin@a0ec8b7

@nathom nathom merged commit b63b957 into nathom:dev Nov 13, 2025
nathom added a commit that referenced this pull request Nov 13, 2025
nathom added a commit that referenced this pull request Nov 13, 2025
@nathom
Copy link
Owner

nathom commented Nov 13, 2025

Yeah we should actually do this for all sources

@itisFarzin
Copy link
Contributor

Screenshot_20251113-233720_Fennec Screenshot_20251113-233722_Fennec

This didn't work as it supposed to, at least in Jellyfin.

@MattiaPun
Copy link
Contributor Author

For Navidrome, it worked perfectly. Did you check if you have a custom seperator set for your music library?
image

@RyanThaDude
Copy link

Some softwares treat separate entries on the same tag differently. What it displays doesn't necessarily mean files tagged outside of that software will treat delimiter the same. Most of the popular ones actually use separate tags with the same tag name. For example, GENRE, may have multiple tags. Instead of a single tag with a delimiter, each genre will be in its own GENRE tag. If you use Mp3Tag it's easy to see when you view all the tags. You can see this with Python's Mutagen library as well.

@itisFarzin
Copy link
Contributor

@MattiaPun Thanks for the notice, it was a problem on my end. I didn't have the custom tag delimiter enabled.

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.

4 participants