1 files changed, 35 insertions, 4 deletions
diff --git a/packages/meshbay-hub/src/meshbay_hub/static/music-app.js b/packages/meshbay-hub/src/meshbay_hub/static/music-app.js
index e0308cb..1dbecbb 100644
--- a/packages/meshbay-hub/src/meshbay_hub/static/music-app.js
+++ b/packages/meshbay-hub/src/meshbay_hub/static/music-app.js
@@ -256,7 +256,13 @@ function AlbumCard({ album, transportRef, gekRef, musicbrainzEnabled, onOpen, on
   const repTrack = album.tracks.find((tr) => tr.thumb_hash) || album.tracks[0];
   const tRef = repTrack._tRef || transportRef;
   const gRef = repTrack._gRef || gekRef;
-  const needsLookup = musicbrainzEnabled && !repTrack.thumb_hash;
+  // Never for an album this file minted itself (`isUnknown`): the release
+  // name is one we wrote -- '<artist> - Various', or the untagged pile
+  // below -- so the lookup is a third-party request, on the operator's
+  // connection, that cannot match anything. Confirmed in a node's log:
+  // queries went out naming a placeholder as both the artist and the
+  // release, once per card.
+  const needsLookup = musicbrainzEnabled && !repTrack.thumb_hash && !album.isUnknown;
   const meta = useMusicMeta(tRef, repTrack.id, needsLookup);
   const coverHash = repTrack.thumb_hash || (meta && meta.cover_thumb_hash) || null;
 
@@ -287,7 +293,7 @@ function MusicDetailModal({ album, transportRef, gekRef, musicbrainzEnabled, onC
   const repTrack = album.tracks.find((tr) => tr.thumb_hash) || album.tracks[0];
   const tRef = repTrack._tRef || transportRef;
   const gRef = repTrack._gRef || gekRef;
-  const needsLookup = musicbrainzEnabled && !repTrack.thumb_hash;
+  const needsLookup = musicbrainzEnabled && !repTrack.thumb_hash && !album.isUnknown;
   const meta = useMusicMeta(tRef, repTrack.id, needsLookup);
   const coverHash = repTrack.thumb_hash || (meta && meta.cover_thumb_hash) || null;
 
@@ -672,17 +678,39 @@ function MusicApp({
   const filteredTracks = useMemo(() => (!needle ? tracks : tracks.filter(
     (tr) => (tr.display_title || tr.name).toLowerCase().includes(needle))), [tracks, needle]);
 
+  // A track whose artist tag is empty *and* whose folder gave nothing to fall
+  // back on. The flat list has always drawn these as its own top-level rows;
+  // the grid, whose unit is an album, drew them nowhere at all -- and `empty`
+  // below counts them, so it stayed false and no message appeared either. A
+  // library nothing has tagged therefore rendered a toolbar over a blank page,
+  // with every one of its tracks one mode-switch away and nothing saying so.
+  // Reported after a node restart, where the index is briefly served without
+  // the tags it re-reads at start-up, and true of a genuinely untagged library
+  // with no restart involved.
+  //
+  // One card, the same shape the singleton folding above already mints for an
+  // artist's leftovers: it names what it is, and the tracks are playable from
+  // it. Not a card each -- that is the wall of one-track tiles this file
+  // exists to avoid -- and not sorted in among the artists, because it is not
+  // a name anybody chose and the alphabet is no place for it.
+  const untagged = useMemo(() => (filteredTracks.length
+    ? { artist: t('music.unknown_artist'), album: t('music.unknown_album'),
+        isUnknown: true, tracks: filteredTracks }
+    : null), [filteredTracks]);
+
   // What this mode draws, in drawing order: albums under their artists in the
   // grid, and in the flat list loose tracks and artist folders sorted together.
   const units = useMemo(() => {
     if (mode === 'grid') {
-      return filteredArtists.flatMap((a) => a.albums.map((album) => ({ artist: a.artist, album })));
+      const byArtist = filteredArtists.flatMap(
+        (a) => a.albums.map((album) => ({ artist: a.artist, album })));
+      return untagged ? [...byArtist, { artist: untagged.artist, album: untagged }] : byArtist;
     }
     return [
       ...filteredTracks.map((tr) => ({ key: tr.display_title || tr.name, kind: 'track', track: tr })),
       ...filteredArtists.map((a) => ({ key: a.artist, kind: 'artist', artist: a })),
     ].sort((a, b) => a.key.localeCompare(b.key));
-  }, [mode, filteredArtists, filteredTracks]);
+  }, [mode, filteredArtists, filteredTracks, untagged]);
 
   const pager = usePager(units.length, pageSizeFrom(userPrefs),
     `${groupId}|${mode}|${needle}|${pageResetKey || ''}`);
@@ -690,6 +718,9 @@ function MusicApp({
     return units.slice(pager.start, pager.end);
   }, [units, pager.start, pager.end]);
 
+  // True exactly when the library has nothing, and — now that every track
+  // reaches a card — exactly when the grid has nothing to draw either. The two
+  // used to disagree, which is the whole of the defect above.
   const empty = albums.length === 0 && tracks.length === 0;
 
   return html`