Detect ASINs in file path

[xuvatilavv]

Short-circuit album search if an ASIN is detected after a # in the album's file path. Made this for my own library but this might close #19 if I understand it right.

Could be good to also override the artist search if it's possible to get the right data using the album ASIN, but that's beyond my needs at the moment.

[djdembeck]

This is a great start to detecting ASIN in file path. That being said, it is a workaround and not a solution. I think the best way to accomplish this would be related to #19. This would look like:

• Search process finds out there's an asin to use (either in manual search or in file name)
• Search process skips actually searching, and directly tells the agent to update to this ASIN, not even searching it.

This would remove an entire API call, thus speeding up the process by about 100-400ms.

[djdembeck]

I've made some changes to the develop branch that you can check out here: 3cca5b6

I still need to test it on file names

[xuvatilavv]

Ah, my mistake, didn't notice the separate search queries 😅 your changes are much better, but I would suggest still requiring the ASIN be marked or have some other restriction, since media.filename is the (url encoded) absolute path to the file. The current regex could easily collide with user path structure (e.g. AUDIOBOOKS/...), and prevent any matches from being made.

I'd considered checking for something like \[[A-Z0-9]{10}\] since that's what Libation puts out by default, but that has the unfortunate collision of [UNABRIDGED], which I've seen on files in the wild a fair bit.

[djdembeck]

I believe I've found a better regex to eliminate your edge cases: (?=.\\d)[A-Z\\d]{10} This will verify there MUST be at least 1 digit in the 10 letter string. In my testing, Audible always has at least 1 letter. Typically something like 0062872389 (all digits) or B0731R4RFX (starts with B0)

[djdembeck]

#49