Consolidated community contributions: AI architecture overhaul, tablet optimization, localization (ja/ar/pt-BR/fr), customizable library tabs, playlist concurrency fix, UI improvements, security hardening, and code quality pass

.github/workflows/codeql.yml

@@ -21,7 +21,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v7
 
       - name: Set up JDK 21
         uses: actions/setup-java@v5
@@ -47,9 +47,10 @@ jobs:
         with:
           languages: java-kotlin
           build-mode: manual
+          cache: true
 
       - name: Build with Gradle
-        run: ${{ github.workspace }}/gradlew :app:assembleDebug :wear:assembleDebug --no-daemon
+        run: ${{ github.workspace }}/gradlew :app:assembleDebug :wear:assembleDebug --no-daemon --no-build-cache
 
       - name: Perform CodeQL Analysis
         uses: github/codeql-action/analyze@v4

.github/workflows/nightly-apk.yml

@@ -85,7 +85,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v7
 
       - name: Set up JDK 21
         uses: actions/setup-java@v5

.github/workflows/phone-debug.yml

@@ -18,7 +18,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v7
 
       - name: Set up JDK 21
         uses: actions/setup-java@v5

.github/workflows/phone-release.yml

@@ -18,7 +18,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v7
 
       - name: Set up JDK 21
         uses: actions/setup-java@v5

.github/workflows/wearos-apk.yml

@@ -18,7 +18,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v7
 
       - name: Set up JDK 21
         uses: actions/setup-java@v5

app/build.gradle.kts

@@ -247,6 +247,8 @@ dependencies {
     implementation(libs.androidx.media3.exoplayer.ffmpeg)
     implementation(libs.androidx.media3.exoplayer.midi)
     implementation(libs.androidx.media3.transformer)
+    implementation(libs.androidx.media3.common)
+    implementation(libs.androidx.media3.common.ktx)
     implementation(libs.androidx.mediarouter)
     implementation(libs.androidx.media)
     implementation(libs.coil.compose)
@@ -298,6 +300,8 @@ dependencies {
         exclude(group = "androidx.compose.runtime")
         exclude(group = "androidx.compose.ui")
     }
+    implementation(libs.haze)
+    implementation(libs.haze.materials)
 
     // Projects
     implementation(project(":shared"))

app/src/androidTest/java/com/theveloper/pixelplay/data/repository/PlaylistSongCountTest.kt

@@ -0,0 +1,136 @@
+package com.theveloper.pixelplay.data.repository
+
+import android.content.Context
+import androidx.datastore.core.DataStore
+import androidx.datastore.preferences.core.Preferences
+import androidx.datastore.preferences.core.PreferenceDataStoreFactory
+import androidx.datastore.preferences.preferencesDataStoreFile
+import androidx.room.Room
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import com.theveloper.pixelplay.data.database.LocalPlaylistDao
+import com.theveloper.pixelplay.data.database.PixelPlayDatabase
+import com.theveloper.pixelplay.data.preferences.PlaylistPreferencesRepository
+import com.theveloper.pixelplay.data.preferences.UserPreferencesRepository
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.coroutineScope
+import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.runBlocking
+import kotlinx.coroutines.test.runTest
+import kotlinx.serialization.json.Json
+import org.junit.After
+import org.junit.Assert.assertEquals
+import org.junit.Before
+import org.junit.Test
+import org.junit.runner.RunWith
+
+/**
+ * Regression test for issue #2391:
+ * "Playlist song count doesn't update when removing songs — only when adding."
+ *
+ * Exercises the real PlaylistPreferencesRepository against an in-memory Room DB to
+ * verify that the song count exposed by userPlaylistsFlow (used by the Playlists menu)
+ * reflects removals as well as additions.
+ */
+@RunWith(AndroidJUnit4::class)
+class PlaylistSongCountTest {
+
+    private lateinit var db: PixelPlayDatabase
+    private lateinit var dao: LocalPlaylistDao
+    private lateinit var dataStore: DataStore<Preferences>
+    private lateinit var repo: PlaylistPreferencesRepository
+
+    @Before
+    fun setup() {
+        val context = ApplicationProvider.getApplicationContext<Context>()
+        db = Room.inMemoryDatabaseBuilder(context, PixelPlayDatabase::class.java)
+            .addCallback(PixelPlayDatabase.createRuntimeArtifactsCallback())
+            .allowMainThreadQueries()
+            .build()
+        dao = db.localPlaylistDao()
+        dataStore = PreferenceDataStoreFactory.create {
+            context.preferencesDataStoreFile("test_settings_${System.nanoTime()}")
+        }
+        val userPrefs = UserPreferencesRepository(dataStore, Json { ignoreUnknownKeys = true })
+        repo = PlaylistPreferencesRepository(dao, userPrefs)
+    }
+
+    @After
+    fun teardown() {
+        db.close()
+    }
+
+    private suspend fun countFor(playlistId: String): Int =
+        repo.userPlaylistsFlow.first().first { it.id == playlistId }.songIds.size
+
+    @Test
+    fun menuSongCount_reflectsAddAndRemove() = runTest {
+        val playlist = repo.createPlaylist(name = "J-Pop", songIds = listOf("10", "20", "30"))
+        assertEquals("initial count", 3, countFor(playlist.id))
+
+        // Remove a song — the bug report says this does NOT update the count.
+        repo.removeSongFromPlaylist(playlist.id, "20")
+        assertEquals("after removing one song", 2, countFor(playlist.id))
+
+        // Remove another.
+        repo.removeSongFromPlaylist(playlist.id, "30")
+        assertEquals("after removing a second song", 1, countFor(playlist.id))
+
+        // Adding works per the report — verify it still does.
+        repo.addSongsToPlaylist(playlist.id, listOf("40"))
+        assertEquals("after adding one song", 2, countFor(playlist.id))
+    }
+
+    /**
+     * Reproduces the real-world trigger for issue #2391: removing several songs in
+     * quick succession. Each edit does an unsynchronized read-modify-write
+     * (userPlaylistsFlow.first() -> modify -> updatePlaylist), so concurrent removals
+     * all read the same original list and the last writer wins, silently dropping the
+     * other removals. The Playlists-menu count (songIds.size) then stays stuck high.
+     */
+    @Test
+    fun concurrentRemovals_doNotLoseUpdates() = runBlocking {
+        val playlist = repo.createPlaylist(
+            name = "Race",
+            songIds = listOf("1", "2", "3", "4", "5")
+        )
+        assertEquals(5, countFor(playlist.id))
+
+        // Remove four songs concurrently — "remove one or two of them", fast.
+        coroutineScope {
+            listOf("1", "2", "3", "4").forEach { id ->
+                launch(Dispatchers.IO) { repo.removeSongFromPlaylist(playlist.id, id) }
+            }
+        }
+
+        assertEquals("All concurrent removals must persist", 1, countFor(playlist.id))
+    }
+
+    /**
+     * Walks the exact reproduction from issue #2391, asserting the fixed behaviour:
+     * the song count stays accurate after a quick removal of "one or two" songs, and
+     * a later addition does not preserve a phantom difference.
+     */
+    @Test
+    fun issue2391_quickRemoveThenAdd_keepsCountAccurate() = runBlocking {
+        // Steps 2-3: create a playlist and add a few songs.
+        val playlist = repo.createPlaylist(
+            name = "J-Pop",
+            songIds = listOf("1", "2", "3", "4", "5", "6")
+        )
+        assertEquals(6, countFor(playlist.id))
+
+        // Step 4: remove one or two of them — quickly, as fast taps do.
+        coroutineScope {
+            launch(Dispatchers.IO) { repo.removeSongFromPlaylist(playlist.id, "2") }
+            launch(Dispatchers.IO) { repo.removeSongFromPlaylist(playlist.id, "4") }
+        }
+        // Step 5: the menu count must reflect BOTH removals (the bug left it stuck high).
+        assertEquals("count after removing two songs", 4, countFor(playlist.id))
+
+        // Steps 6-7: adding more must not carry over a phantom difference.
+        repo.addSongsToPlaylist(playlist.id, listOf("7", "8"))
+        assertEquals("count after adding two songs", 6, countFor(playlist.id))
+    }
+}

app/src/debug/res/values-de/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">Pausieren, wenn Lautstärke null erreicht</string>
+    <string name="settings_pause_on_volume_zero_desc">Wiedergabe automatisch pausieren, wenn die Lautstärke auf 0 gesetzt wird</string>
+    <string name="settings_volume_section">Lautstärke</string>
 </resources>

app/src/debug/res/values-fr/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">Mettre en pause quand le volume atteint zéro</string>
+    <string name="settings_pause_on_volume_zero_desc">Mettre automatiquement en pause la lecture lorsque le volume est à 0</string>
+    <string name="settings_volume_section">Volume</string>
 </resources>

app/src/debug/res/values-ko/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">볼륨이 0이 되면 일시정지</string>
+    <string name="settings_pause_on_volume_zero_desc">볼륨이 0으로 설정되면 자동으로 재생을 일시정지합니다</string>
+    <string name="settings_volume_section">볼륨</string>
 </resources>

app/src/debug/res/values-nb/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">Sett på pause når volumet er null</string>
+    <string name="settings_pause_on_volume_zero_desc">Sett automatisk avspillingen på pause når volumet settes til 0</string>
+    <string name="settings_volume_section">Volum</string>
 </resources>

app/src/debug/res/values-ru/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">Пауза при нулевой громкости</string>
+    <string name="settings_pause_on_volume_zero_desc">Автоматически приостанавливать воспроизведение, когда громкость равна 0</string>
+    <string name="settings_volume_section">Громкость</string>
 </resources>

app/src/debug/res/values/strings.xml

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="app_name">PixelPlayer [D]</string>
+    <string name="settings_pause_on_volume_zero">Pause when volume reaches zero</string>
+    <string name="settings_pause_on_volume_zero_desc">Automatically pause playback when the volume is set to 0</string>
+    <string name="settings_volume_section">Volume</string>
 </resources>