Add Regex for scrobbling EPs and Singles that does not affect normal album titles

[RegorForgot]

Preflight Checklist

• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for a bug report that matches the one I want to file, without success.
• I have checked that the correct behavior occurs on Apple Music Web (if applicable).
• I have verified that my issue is with a supported version and that it is with the desktop app (Support for other areas can be found in our Discord / on the appropriate repo).

How did you download Cider?

itch.io

Operating System

Windows 11

Commit Hash

No response

Build Timestamp

No response

Describe the Bug

The scrobbling album extension remover removes items after a " - ", including if they are part of the actual album name. Please note that this only occurs on Apple Music if the thing being played is an EP or a Single. Hence a Regex can be used to detect these specific cases that occur only at the end of the EP/Single name, something similar to the below:

- (Single|EP)$

This will only return a match if the above items are found within the album name.

Steps to Reproduce

1. Enable Remove Album Extensions in the Connectivity tab under Last.fm settings
2. Attempt to listen to an album which has a dash as part of its normal name - for example "Aril Brikha - Deeparture in Time - Revisited". Cider will attempt to scrobble this album without the " - Revisited" at the end of the name (which is definitely part of the album name).

Anything else?

No response

[cider-collective]

Hello @RegorForgot 👋

We really appreciate you taking the time to report this issue, please double check your issue to make sure you are clear and concise in your description. This will help us to help you as quickly as possible.

We'd like to ask you be patient and we will get back to you as soon as we can.

In the meantime, take a look at our FAQs which may answer your question.

Thanks again and we hope you enjoy the app! 😃

[cryptofyre]

I’ll look into it.

[coredev-uk]

We do already operate this way. Our current replace method uses .replace(/ - Single| - EP/g, '') Which operates in the same way, so not sure what you're on about.

[coredev-uk]

nvm, we had that in one place, but the main one used a wildcard regex.

Fixed with https://github.com/ciderapp/citadel/commit/0f6dd4ac7f4f17bcbb97999533802a68edc382a6