From 8ada2e3f514d51b32de4c5e795e8b07061bf0216 Mon Sep 17 00:00:00 2001
From: BeardedBear <germain.poirrier@gmail.com>
Date: Mon, 23 Feb 2026 12:45:30 +0100
Subject: [PATCH 1/4] fix(artist): Prefer name search, handle homonyms

searchMusicBrainzArtistId now returns all matching artists instead of a
single result or null. ArtistStore now performs a name-based search
first
and resolves ambiguous results by using a Spotify URL lookup when a
Spotify ID is available. This improves accuracy for artists with the
same
name.

BREAKING CHANGE: searchMusicBrainzArtistId signature changed from
Promise<MusicBrainzArtist | null> to Promise<MusicBrainzArtist[]>;
update any
callers accordingly
---
 src/helpers/musicbrainz.ts      | 11 +++++------
 src/views/artist/ArtistStore.ts | 14 +++++++++-----
 2 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/helpers/musicbrainz.ts b/src/helpers/musicbrainz.ts
index d6c7d605..bab0423a 100644
--- a/src/helpers/musicbrainz.ts
+++ b/src/helpers/musicbrainz.ts
@@ -168,13 +168,13 @@ export async function getIdsFromMusicBrainz(
 }
 
 /**
- * Search for an artist by name in MusicBrainz (fallback when Spotify URL lookup fails)
+ * Search for artists by name in MusicBrainz
  * @param artistName - The name of the artist to search for
- * @returns Promise resolving to the first matching MusicBrainzArtist or null
+ * @returns Promise resolving to all matching MusicBrainzArtist with the same name, or empty array
  */
 export async function searchMusicBrainzArtistId(
   artistName: string,
-): Promise<MusicBrainzArtist | null> {
+): Promise<MusicBrainzArtist[]> {
   const data = await fetchFromMusicBrainz<MusicBrainzArtistSearch>("artist", {
     limit: 10,
     query: `artist:"${artistName}"`,
@@ -182,13 +182,12 @@ export async function searchMusicBrainzArtistId(
 
   if (data && data.artists && data.artists.length > 0) {
     const normalizedSearchName = normalizeName(artistName);
-    const filtered = data.artists.filter(
+    return data.artists.filter(
       (artist) => normalizeName(artist.name) === normalizedSearchName,
     );
-    return filtered[0];
   }
 
-  return null;
+  return [];
 }
 
 /**
diff --git a/src/views/artist/ArtistStore.ts b/src/views/artist/ArtistStore.ts
index 06443b2f..01aa1595 100644
--- a/src/views/artist/ArtistStore.ts
+++ b/src/views/artist/ArtistStore.ts
@@ -139,11 +139,15 @@ export const useArtist = defineStore("artist", {
       this.wikidataArtist = null;
       this.wikipediaExtract = null;
       try {
-        // Try exact match via Spotify URL, fallback to name search
-        const artist = spotifyId
-          ? ((await searchMusicBrainzBySpotifyId(spotifyId))
-            ?? (await searchMusicBrainzArtistId(artistName)))
-          : await searchMusicBrainzArtistId(artistName);
+        // Search by name first; if multiple homonyms found and Spotify ID available,
+        // use Spotify URL lookup for exact match
+        const nameResults = await searchMusicBrainzArtistId(artistName);
+        const artist
+          = nameResults.length === 1
+            ? nameResults[0]
+            : nameResults.length > 1 && spotifyId
+              ? ((await searchMusicBrainzBySpotifyId(spotifyId)) ?? nameResults[0])
+              : nameResults[0] ?? null;
 
         if (!artist?.id) return;
 

From 39ef51e915f28df3cfc3f6e59567e0fccd962a17 Mon Sep 17 00:00:00 2001
From: BeardedBear <germain.poirrier@gmail.com>
Date: Mon, 23 Feb 2026 13:04:06 +0100
Subject: [PATCH 2/4] style(artist): Tidy template formatting

---
 src/components/artist/ArtistProfile.vue | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/components/artist/ArtistProfile.vue b/src/components/artist/ArtistProfile.vue
index 350bddf1..7194dda8 100644
--- a/src/components/artist/ArtistProfile.vue
+++ b/src/components/artist/ArtistProfile.vue
@@ -10,11 +10,8 @@
       <template v-if="artistMetas?.country">
         <span class="dot desktop-only">·</span>
         <Tooltip :text="getCountry" placement="bottom">
-          <img
-            :src="getCountryFlagUrl(artistMetas.country)"
-            :alt="artistMetas?.area?.name || artistMetas?.country"
-            class="country-flag"
-          />
+          <img :src="getCountryFlagUrl(artistMetas.country)" :alt="artistMetas?.area?.name || artistMetas?.country"
+            class="country-flag" />
           <strong>
             {{ artistMetas?.["begin-area"]?.name || artistMetas?.area?.name }}
           </strong>
@@ -22,7 +19,7 @@
       </template>
       <template v-else>
         <span class="dot desktop-only">·</span>
-        {{ artistMetas?.["begin-area"]?.name ||artistMetas?.["begin-area"]?.id || artistMetas?.area?.name }}
+        {{ artistMetas?.["begin-area"]?.name || artistMetas?.["begin-area"]?.id || artistMetas?.area?.name }}
       </template>
       <template v-if="artistMetas?.['life-span']?.begin">
         <span class="dot">·</span>

From 4ca5a530c9571e38f7c0f8f85d8c308b5d898df6 Mon Sep 17 00:00:00 2001
From: BeardedBear <germain.poirrier@gmail.com>
Date: Mon, 23 Feb 2026 13:06:56 +0100
Subject: [PATCH 3/4] fix(artist): Display disambiguation and area name

---
 src/components/artist/ArtistProfile.vue | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/components/artist/ArtistProfile.vue b/src/components/artist/ArtistProfile.vue
index 7194dda8..c975bc2d 100644
--- a/src/components/artist/ArtistProfile.vue
+++ b/src/components/artist/ArtistProfile.vue
@@ -1,12 +1,16 @@
 <template>
   <div class="profile-container">
     <div class="profile-wrapper" :class="{ visible: artistMetas }">
-      <span v-if="artistTags && artistTags.length > 0" class="tag-list">
-        <span v-for="(value, index) in artistTags.slice(0, 5)" :key="index" class="tag">
-          {{ typeof value === "string" ? value : value.name }}
+      <template v-if="artistTags && artistTags.length > 0">
+        <span class="tag-list">
+          <span v-for="(value, index) in artistTags.slice(0, 5)" :key="index" class="tag">
+            {{ typeof value === "string" ? value : value.name }}
+          </span>
         </span>
-      </span>
-      <!-- {{ artistMetas }} -->
+      </template>
+      <template v-else-if="artistMetas?.disambiguation">
+        <span class="disambiguation">{{ artistMetas.disambiguation }}</span>
+      </template>
       <template v-if="artistMetas?.country">
         <span class="dot desktop-only">·</span>
         <Tooltip :text="getCountry" placement="bottom">
@@ -17,9 +21,9 @@
           </strong>
         </Tooltip>
       </template>
-      <template v-else>
+      <template v-else-if="artistMetas?.['begin-area']?.name || artistMetas?.area?.name">
         <span class="dot desktop-only">·</span>
-        {{ artistMetas?.["begin-area"]?.name || artistMetas?.["begin-area"]?.id || artistMetas?.area?.name }}
+        {{ artistMetas?.["begin-area"]?.name || artistMetas?.area?.name }}
       </template>
       <template v-if="artistMetas?.['life-span']?.begin">
         <span class="dot">·</span>
@@ -98,6 +102,10 @@ $transition-duration: 0.2s;
   }
 }
 
+.disambiguation {
+  opacity: 0.6;
+}
+
 .dot {
   margin: 0 1rem;
 

From 4e9fc848db18742aa492f7ff0a104a2f305a16fc Mon Sep 17 00:00:00 2001
From: BeardedBear <germain.poirrier@gmail.com>
Date: Mon, 23 Feb 2026 13:12:46 +0100
Subject: [PATCH 4/4] fix(artist): Hide separator dot when no tags or
 disambiguation

---
 src/components/artist/ArtistProfile.vue | 35 ++++++++++++++++++-------
 1 file changed, 26 insertions(+), 9 deletions(-)

diff --git a/src/components/artist/ArtistProfile.vue b/src/components/artist/ArtistProfile.vue
index c975bc2d..0c4bd75d 100644
--- a/src/components/artist/ArtistProfile.vue
+++ b/src/components/artist/ArtistProfile.vue
@@ -3,7 +3,11 @@
     <div class="profile-wrapper" :class="{ visible: artistMetas }">
       <template v-if="artistTags && artistTags.length > 0">
         <span class="tag-list">
-          <span v-for="(value, index) in artistTags.slice(0, 5)" :key="index" class="tag">
+          <span
+            v-for="(value, index) in artistTags.slice(0, 5)"
+            :key="index"
+            class="tag"
+          >
             {{ typeof value === "string" ? value : value.name }}
           </span>
         </span>
@@ -12,16 +16,25 @@
         <span class="disambiguation">{{ artistMetas.disambiguation }}</span>
       </template>
       <template v-if="artistMetas?.country">
-        <span class="dot desktop-only">·</span>
+        <span
+          v-if="artistTags.length > 0 || artistMetas?.disambiguation"
+          class="dot desktop-only"
+          >·
+        </span>
         <Tooltip :text="getCountry" placement="bottom">
-          <img :src="getCountryFlagUrl(artistMetas.country)" :alt="artistMetas?.area?.name || artistMetas?.country"
-            class="country-flag" />
+          <img
+            :src="getCountryFlagUrl(artistMetas.country)"
+            :alt="artistMetas?.area?.name || artistMetas?.country"
+            class="country-flag"
+          />
           <strong>
             {{ artistMetas?.["begin-area"]?.name || artistMetas?.area?.name }}
           </strong>
         </Tooltip>
       </template>
-      <template v-else-if="artistMetas?.['begin-area']?.name || artistMetas?.area?.name">
+      <template
+        v-else-if="artistMetas?.['begin-area']?.name || artistMetas?.area?.name"
+      >
         <span class="dot desktop-only">·</span>
         {{ artistMetas?.["begin-area"]?.name || artistMetas?.area?.name }}
       </template>
@@ -35,7 +48,9 @@
           <template v-else>
             <span>{{ artistMetas?.["life-span"]?.begin.split("-")[0] }}</span>
             <span v-if="!artistMetas['life-span'].end">
-              {{ artistMetas?.["life-span"]?.ended ? " / Inactive" : " / Active" }}
+              {{
+                artistMetas?.["life-span"]?.ended ? " / Inactive" : " / Active"
+              }}
             </span>
             <span v-else>&nbsp;/ {{ artistMetas["life-span"].end?.split("-")[0] }}</span>
           </template>
@@ -55,13 +70,15 @@ import { useArtist } from "@/views/artist/ArtistStore";
 
 const artistStore = useArtist();
 const artistMetas = computed(() => artistStore.musicbrainzArtist);
-const artistTags = computed(() => artistMetas.value?.tags || artistStore.artist.genres);
+const artistTags = computed(
+  () => artistMetas.value?.tags || artistStore.artist.genres,
+);
 const getCountry = computed(() => {
   const countryName = getCountryName(artistMetas.value?.country);
-  const artistCountry = artistMetas.value?.["begin-area"]?.name || artistMetas.value?.area?.name;
+  const artistCountry
+    = artistMetas.value?.["begin-area"]?.name || artistMetas.value?.area?.name;
   return countryName !== artistCountry ? countryName : "";
 });
-
 </script>
 
 <style lang="scss" scoped>