feat(Sensors): Add search filter to sensor setting allow list dialog

app/src/main/kotlin/io/homeassistant/companion/android/settings/sensor/views/SensorDetailView.kt

@@ -28,6 +28,8 @@
 import androidx.compose.material.Card
 import androidx.compose.material.Checkbox
 import androidx.compose.material.CircularProgressIndicator
+import androidx.compose.material.Icon
+import androidx.compose.material.IconButton
 import androidx.compose.material.ContentAlpha
 import androidx.compose.material.Divider
 import androidx.compose.material.LocalContentAlpha
@@ -42,6 +44,9 @@
 import androidx.compose.material.Text
 import androidx.compose.material.TextField
 import androidx.compose.material.contentColorFor
+import androidx.compose.material.icons.Icons
+import androidx.compose.material.icons.filled.Clear
+import androidx.compose.material.icons.filled.Search
 import androidx.compose.material.rememberScaffoldState
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.CompositionLocalProvider
@@ -608,31 +613,44 @@
                     CircularProgressIndicator()
                 }
             } else if (listSettingDialog) {
-                LazyColumn {
-                    items(state.entries, key = { (id) -> id }) { (id, entry) ->
-                        SensorDetailSettingRow(
-                            label = entry,
-                            checked = if (state.setting.valueType ==
-                                SensorSettingType.LIST
-                            ) {
-                                inputValue.value == id
-                            } else {
-                                checkedValue.contains(id)
-                            },
-                            multiple = state.setting.valueType != SensorSettingType.LIST,
-                            onClick = { isChecked ->
-                                if (state.setting.valueType == SensorSettingType.LIST) {
-                                    inputValue.value = id
-                                    onSubmit(state.copy().apply { setting.value = inputValue.value })
+                var searchQuery by remember { mutableStateOf("") }
+                val filteredEntries = remember(state.entries, searchQuery) {
+                    filterSettingEntries(state.entries, searchQuery)
+                }
+                val showSearch = state.entries.size > 10
+                Column {
+                    if (showSearch) {
+                        SettingSearchField(
+                            query = searchQuery,
+                            onQueryChange = { searchQuery = it },
+                        )
+                    }
+                    LazyColumn(modifier = Modifier.weight(1f)) {
+                        items(filteredEntries, key = { (id) -> id }) { (id, entry) ->
+                            SensorDetailSettingRow(
+                                label = entry,
+                                checked = if (state.setting.valueType ==
+                                    SensorSettingType.LIST
+                                ) {
+                                    inputValue.value == id
                                 } else {
-                                    if (checkedValue.contains(id) && !isChecked) {
-                                        checkedValue.remove(id)
-                                    } else if (!checkedValue.contains(id) && isChecked) {
-                                        checkedValue.add(id)
+                                    checkedValue.contains(id)
+                                },
+                                multiple = state.setting.valueType != SensorSettingType.LIST,
+                                onClick = { isChecked ->
+                                    if (state.setting.valueType == SensorSettingType.LIST) {
+                                        inputValue.value = id
+                                        onSubmit(state.copy().apply { setting.value = inputValue.value })
+                                    } else {
+                                        if (checkedValue.contains(id) && !isChecked) {
+                                            checkedValue.remove(id)
+                                        } else if (!checkedValue.contains(id) && isChecked) {
+                                            checkedValue.add(id)
+                                        }
                                     }
-                                }
-                            },
-                        )
+                                },
+                            )
+                        }
                     }
                 }
             } else {
@@ -734,6 +752,47 @@
     )
 }
 
+@Composable
+private fun SettingSearchField(
+    query: String,
+    onQueryChange: (String) -> Unit,
+    modifier: Modifier = Modifier,
+) {
+    TextField(
+        value = query,
+        onValueChange = onQueryChange,
+        modifier = modifier
+            .fillMaxWidth()
+            .padding(horizontal = 16.dp),
+        singleLine = true,
+        label = { Text(stringResource(commonR.string.search)) },
+        leadingIcon = { Icon(Icons.Filled.Search, contentDescription = null) },
+        trailingIcon = if (query.isNotBlank()) {
+            {
+                IconButton(onClick = { onQueryChange("") }) {
+                    Icon(
+                        Icons.Filled.Clear,
+                        contentDescription = stringResource(commonR.string.clear_search),
+                    )
+                }
+            }
+        } else {
+            null
+        },
+    )
+}
+
+/**
+ * Filters setting entries by matching the query against entry labels (case-insensitive).
+ * Returns all entries when the query is blank.
+ */
+internal fun filterSettingEntries(
+    entries: List<Pair<String, String>>,
+    query: String,
+): List<Pair<String, String>> =
+    if (query.isBlank()) entries
+    else entries.filter { (_, label) -> label.contains(query.trim(), ignoreCase = true) }
+
 @Composable
 fun SensorDetailSettingRow(
     label: String,

app/src/test/kotlin/io/homeassistant/companion/android/settings/sensor/views/SensorDetailSettingDialogFilterTest.kt

@@ -0,0 +1,62 @@
+package io.homeassistant.companion.android.settings.sensor.views
+
+import org.junit.jupiter.api.Assertions.assertEquals
+import org.junit.jupiter.api.Test
+
+class SensorDetailSettingDialogFilterTest {
+
+    private val entries = listOf(
+        "com.google.chrome" to "Chrome\n(com.google.chrome)",
+        "org.mozilla.firefox" to "Firefox\n(org.mozilla.firefox)",
+        "com.example.app" to "Example App\n(com.example.app)",
+    )
+
+    @Test
+    fun `Given empty query when filtering then return all entries`() {
+        val result = filterSettingEntries(entries, query = "")
+
+        assertEquals(entries, result)
+    }
+
+    @Test
+    fun `Given blank query when filtering then return all entries`() {
+        val result = filterSettingEntries(entries, query = "   ")
+
+        assertEquals(entries, result)
+    }
+
+    @Test
+    fun `Given query matching app name when filtering then return matching entries`() {
+        val result = filterSettingEntries(entries, query = "Chrome")
+
+        assertEquals(listOf(entries[0]), result)
+    }
+
+    @Test
+    fun `Given query matching package name in label when filtering then return matching entries`() {
+        val result = filterSettingEntries(entries, query = "com.google")
+
+        assertEquals(listOf(entries[0]), result)
+    }
+
+    @Test
+    fun `Given case-insensitive query when filtering then return matches`() {
+        val result = filterSettingEntries(entries, query = "CHROME")
+
+        assertEquals(listOf(entries[0]), result)
+    }
+
+    @Test
+    fun `Given query matching no entries when filtering then return empty list`() {
+        val result = filterSettingEntries(entries, query = "nonexistent")
+
+        assertEquals(emptyList<Pair<String, String>>(), result)
+    }
+
+    @Test
+    fun `Given query with leading and trailing spaces when filtering then trim and match`() {
+        val result = filterSettingEntries(entries, query = " Chrome ")
+
+        assertEquals(listOf(entries[0]), result)
+    }
+}