fix: remove /data/music default - require explicit configuration

Previously, the code defaulted to /data/music if no music library path
was configured. This caused issues when:
1. The path didn't exist
2. Multiple paths appeared to be valid due to symlinks/mounts
3. Scans started without proper user configuration

Now there is NO default path - users must explicitly configure their
music library via the admin UI (/admin) or MUSIC_LIBRARY_PATH env var.
The app will show a clear warning if no path is configured.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: jeffcrouse

backend/app/config.py

@@ -16,7 +16,8 @@ class Settings(BaseSettings):
     redis_url: str = "redis://localhost:6379/0"
 
     # Music library (comma-separated paths supported)
-    music_library_path: str = "/data/music"
+    # NOTE: No default - must be configured via admin UI or MUSIC_LIBRARY_PATH env var
+    music_library_path: str = ""
 
     @property
     def music_library_paths(self) -> list[Path]:
@@ -25,7 +26,10 @@ def music_library_paths(self) -> list[Path]:
         Priority:
         1. AppSettings (settings.json) if configured via admin UI
         2. Environment variable MUSIC_LIBRARY_PATH (comma-separated)
-        3. Default /data/music
+        3. Empty list (user must configure via /admin)
+
+        There is intentionally NO default path - the user must explicitly
+        configure their music library location via the admin UI.
         """
         # Check AppSettings first (configured via admin UI)
         from app.services.app_settings import get_app_settings_service
@@ -35,7 +39,7 @@ def music_library_paths(self) -> list[Path]:
             return [Path(p) for p in app_settings.music_library_paths if p]
 
         # Fall back to environment variable (for backwards compatibility)
-        if self.music_library_path and self.music_library_path != "/data/music":
+        if self.music_library_path:
             paths = []
             for p in self.music_library_path.split(","):
                 p = p.strip()
@@ -44,8 +48,8 @@ def music_library_paths(self) -> list[Path]:
             if paths:
                 return paths
 
-        # Default
-        return [Path("/data/music")]
+        # No default - user must configure via admin UI
+        return []
 
     # Data paths
     art_path: Path = Path("data/art")

backend/app/main.py

@@ -90,8 +90,8 @@ def migrate_env_to_settings() -> None:
 
     # Check for MUSIC_LIBRARY_PATH environment variable
     env_path = app_config.music_library_path
-    if env_path and env_path != "/data/music":
-        # User has a custom path configured via env var - migrate it
+    if env_path:
+        # User has a path configured via env var - migrate it to settings
         paths = [p.strip() for p in env_path.split(",") if p.strip()]
         if paths:
             service.update(music_library_paths=paths)

backend/tests/test_scanner.py

@@ -261,7 +261,7 @@ async def test_hash_matching_with_different_mount_paths(self, clean_db):
 
         This simulates the real-world scenario where:
         - Files exist at /srv/dev-disk-by-uuid-.../music/...
-        - Same files are also accessible at /data/music/...
+        - Same files are also accessible via a symlink or different mount
         - Scanner should recognize them as the same files by hash
         """
         with tempfile.TemporaryDirectory() as tmpdir: