v1.0.27: knot-inspired icon, fix media-not-found on photo open

Icon: two thick tube-style arcs with 3D glossy highlights. Arc 1 (left side): coral #E8665A to orange #E8A040 Arc 2 (right side): steel blue #4A7FD4 to deep purple #7B5EA7 Arrowheads: orange and purple. Background: dark purple-black. Inspired by the braided knot color palette. Fix "media not found" when opening photos: - Intent now sets ClipData alongside FLAG_GRANT_READ_URI_PERMISSION so the permission correctly propagates through the system chooser to whichever app the user picks. - openFile() and downloadToCache() both call MediaScannerConnection so newly synced/downloaded files appear in gallery MediaStore index before the viewer launches. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
v1.0.26: fix multi-selection reactivity, redesign icon, security review
2026-05-25 03:08:44 +00:00 · 2026-05-25 02:45:43 +00:00 · 2026-05-25 02:22:43 +00:00 · 2026-05-25 00:37:16 +00:00
12 changed files with 624 additions and 280 deletions
@@ -1,7 +1,7 @@
 package com.syncflow.data.providers.webdav

-import android.util.Log
 import com.syncflow.data.providers.CloudProvider
+import timber.log.Timber
 import com.syncflow.domain.model.CloudAccount
 import com.syncflow.domain.model.RemoteFile
 import kotlinx.coroutines.Dispatchers
@@ -59,14 +59,13 @@ open class WebDavProvider(protected val account: CloudAccount) : CloudProvider {
        withContext(Dispatchers.IO) {
            val req = Request.Builder().url(baseUrl).method("PROPFIND", PROPFIND_BODY).header("Depth", "0").build()
            client.newCall(req).execute().use { resp ->
-                Log.d("SyncFlow/WebDAV", "testConnection ${resp.code} url=$baseUrl")
+                Timber.d("WebDAV testConnection %d url=%s", resp.code, baseUrl)
                if (!resp.isSuccessful && resp.code != 207) {
-                    val body = resp.body?.string()?.take(300) ?: ""
-                    throw Exception("HTTP ${resp.code} ${resp.message} — $body")
+                    throw Exception("HTTP ${resp.code} ${resp.message}")
                }
            }
        }
-    }.onFailure { e -> Log.e("SyncFlow/WebDAV", "testConnection failed: ${e::class.simpleName}: ${e.message}", e.cause) }
+    }.onFailure { e -> Timber.e(e, "WebDAV testConnection failed") }

    override suspend fun listFiles(remotePath: String): Result<List<RemoteFile>> = runCatching {
        withContext(Dispatchers.IO) {
@@ -192,9 +191,14 @@ open class WebDavProvider(protected val account: CloudAccount) : CloudProvider {
                    "response" -> if (inResponse && href.isNotBlank()) {
                        val rawName = href.trimEnd('/').substringAfterLast('/')
                        val name = try { java.net.URLDecoder.decode(rawName, "UTF-8") } catch (_: Exception) { rawName }
-                        val relPath = "$parentPath/$name".replace("//", "/")
-                        results.add(RemoteFile(relPath, name, isCollection, contentLength, lastModified, etag, contentType))
-                        inResponse = false
+                        // Guard against path-traversal sequences delivered by a malicious server
+                        if (name.contains("..") || name.contains('/') || name.contains('\\')) {
+                            inResponse = false
+                        } else {
+                            val relPath = "$parentPath/$name".replace("//", "/")
+                            results.add(RemoteFile(relPath, name, isCollection, contentLength, lastModified, etag, contentType))
+                            inResponse = false
+                        }
                    }
                }
            }
@@ -12,6 +12,7 @@ import javax.inject.Singleton
 class CredentialStore @Inject constructor(@ApplicationContext private val context: Context) {

    private val prefs: SharedPreferences by lazy {
+        @Suppress("DEPRECATION")
        val masterKeyAlias = MasterKeys.getOrCreate(MasterKeys.AES256_GCM_SPEC)
        EncryptedSharedPreferences.create(
            "syncflow_credentials",
@@ -1,7 +1,12 @@
 package com.syncflow.ui.files

+import android.content.ClipData
 import android.content.Intent
+import android.net.Uri
 import android.webkit.MimeTypeMap
+import androidx.activity.compose.BackHandler
+import androidx.compose.foundation.ExperimentalFoundationApi
+import androidx.compose.foundation.combinedClickable
 import androidx.compose.foundation.layout.*
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
@@ -20,11 +25,12 @@ import androidx.core.content.FileProvider
 import androidx.hilt.navigation.compose.hiltViewModel
 import com.syncflow.data.db.entities.SyncFileStateEntity
 import kotlinx.coroutines.launch
+import java.io.File
 import java.time.ZoneId
 import java.time.format.DateTimeFormatter
 import java.time.format.FormatStyle

-@OptIn(ExperimentalMaterial3Api::class)
+@OptIn(ExperimentalMaterial3Api::class, ExperimentalFoundationApi::class)
@Composable
 fun FilesScreen(
    modifier: Modifier = Modifier,
@@ -33,9 +39,16 @@ fun FilesScreen(
    val pairs by vm.pairs.collectAsState()
    val selectedPair by vm.selectedPair.collectAsState()
    val files by vm.files.collectAsState()
+    val isDownloading by vm.isDownloading.collectAsState()
+    val selectedKeys by vm.selectedKeys.collectAsState()
+    val isSelectionMode = selectedKeys.isNotEmpty()
+    val selectedCount = selectedKeys.size
    val context = LocalContext.current
    val snackbarHostState = remember { SnackbarHostState() }
    val scope = rememberCoroutineScope()
+    var showDeleteSelectedDialog by remember { mutableStateOf(false) }
+
+    BackHandler(enabled = isSelectionMode) { vm.clearSelection() }

    LaunchedEffect(Unit) {
        vm.fileAction.collect { action ->
@@ -45,15 +58,18 @@ fun FilesScreen(
                        val uri = FileProvider.getUriForFile(
                            context, "${context.packageName}.fileprovider", action.file
                        )
+                        val mimeType = action.file.name.mimeType()
                        val intent = Intent(Intent.ACTION_VIEW).apply {
-                            setDataAndType(uri, action.file.name.mimeType())
-                            addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
+                            setDataAndType(uri, mimeType)
+                            // ClipData is required so FLAG_GRANT_READ_URI_PERMISSION
+                            // propagates to whichever app the system chooser picks.
+                            clipData = ClipData.newRawUri("", uri)
+                            addFlags(
+                                Intent.FLAG_GRANT_READ_URI_PERMISSION or
+                                Intent.FLAG_ACTIVITY_NEW_TASK
+                            )
                        }
-                        context.startActivity(
-                            Intent.createChooser(intent, "Open with").apply {
-                                addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
-                            }
-                        )
+                        context.startActivity(intent)
                    } catch (e: Exception) {
                        snackbarHostState.showSnackbar("Cannot open file: ${e.message}")
                    }
@@ -66,6 +82,7 @@ fun FilesScreen(
                        val intent = Intent(Intent.ACTION_SEND).apply {
                            type = action.file.name.mimeType()
                            putExtra(Intent.EXTRA_STREAM, uri)
+                            clipData = ClipData.newRawUri("", uri)
                            addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
                        }
                        context.startActivity(
@@ -77,6 +94,25 @@ fun FilesScreen(
                        snackbarHostState.showSnackbar("Cannot share file: ${e.message}")
                    }
                }
+                is FileAction.ShareMultiple -> {
+                    try {
+                        val uris = action.files.map { file ->
+                            FileProvider.getUriForFile(context, "${context.packageName}.fileprovider", file)
+                        }
+                        val intent = Intent(Intent.ACTION_SEND_MULTIPLE).apply {
+                            type = "*/*"
+                            putParcelableArrayListExtra(Intent.EXTRA_STREAM, ArrayList(uris))
+                            addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
+                        }
+                        context.startActivity(
+                            Intent.createChooser(intent, "Share files").apply {
+                                addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
+                            }
+                        )
+                    } catch (e: Exception) {
+                        snackbarHostState.showSnackbar("Cannot share: ${e.message}")
+                    }
+                }
                is FileAction.Error -> scope.launch {
                    snackbarHostState.showSnackbar(action.message)
                }
@@ -84,9 +120,62 @@ fun FilesScreen(
        }
    }

+    if (showDeleteSelectedDialog) {
+        AlertDialog(
+            onDismissRequest = { showDeleteSelectedDialog = false },
+            icon = { Icon(Icons.Default.Delete, contentDescription = null) },
+            title = { Text("Delete $selectedCount file${if (selectedCount != 1) "s" else ""}?") },
+            text = { Text("Selected files will be removed from this device.") },
+            confirmButton = {
+                TextButton(onClick = {
+                    vm.deleteSelected()
+                    showDeleteSelectedDialog = false
+                }) { Text("Delete", color = MaterialTheme.colorScheme.error) }
+            },
+            dismissButton = {
+                TextButton(onClick = { showDeleteSelectedDialog = false }) { Text("Cancel") }
+            },
+        )
+    }
+
    Box(modifier = modifier.fillMaxSize()) {
        Column(modifier = Modifier.fillMaxSize()) {
-            if (pairs.size > 1) {
+            // Selection toolbar
+            if (isSelectionMode) {
+                Surface(
+                    color = MaterialTheme.colorScheme.primaryContainer,
+                    tonalElevation = 3.dp,
+                ) {
+                    Row(
+                        modifier = Modifier
+                            .fillMaxWidth()
+                            .padding(horizontal = 4.dp, vertical = 4.dp),
+                        verticalAlignment = Alignment.CenterVertically,
+                    ) {
+                        IconButton(onClick = { vm.clearSelection() }) {
+                            Icon(Icons.Default.Close, contentDescription = "Clear selection")
+                        }
+                        Text(
+                            "$selectedCount selected",
+                            style = MaterialTheme.typography.titleSmall,
+                            modifier = Modifier.weight(1f),
+                        )
+                        IconButton(onClick = { vm.shareSelected() }) {
+                            Icon(Icons.Default.Share, contentDescription = "Share selected")
+                        }
+                        IconButton(onClick = { showDeleteSelectedDialog = true }) {
+                            Icon(
+                                Icons.Default.Delete, contentDescription = "Delete selected",
+                                tint = MaterialTheme.colorScheme.error,
+                            )
+                        }
+                    }
+                }
+                HorizontalDivider()
+            }
+
+            // Pair tabs
+            if (pairs.size > 1 && !isSelectionMode) {
                ScrollableTabRow(
                    selectedTabIndex = pairs.indexOfFirst { it.id == selectedPair?.id }.coerceAtLeast(0),
                    edgePadding = 16.dp,
@@ -116,24 +205,26 @@ fun FilesScreen(
                    subtitle = "Run a sync to populate this view",
                )
                else -> {
-                    Surface(color = MaterialTheme.colorScheme.surfaceVariant.copy(alpha = 0.5f)) {
-                        Row(
-                            modifier = Modifier
-                                .fillMaxWidth()
-                                .padding(horizontal = 16.dp, vertical = 8.dp),
-                            horizontalArrangement = Arrangement.SpaceBetween,
-                            verticalAlignment = Alignment.CenterVertically,
-                        ) {
-                            Text(
-                                "${files.size} file${if (files.size != 1) "s" else ""}",
-                                style = MaterialTheme.typography.labelMedium,
-                                color = MaterialTheme.colorScheme.onSurfaceVariant,
-                            )
-                            Text(
-                                files.sumOf { it.localSizeBytes }.toDisplaySize(),
-                                style = MaterialTheme.typography.labelMedium,
-                                color = MaterialTheme.colorScheme.onSurfaceVariant,
-                            )
+                    if (!isSelectionMode) {
+                        Surface(color = MaterialTheme.colorScheme.surfaceVariant.copy(alpha = 0.5f)) {
+                            Row(
+                                modifier = Modifier
+                                    .fillMaxWidth()
+                                    .padding(horizontal = 16.dp, vertical = 8.dp),
+                                horizontalArrangement = Arrangement.SpaceBetween,
+                                verticalAlignment = Alignment.CenterVertically,
+                            ) {
+                                Text(
+                                    "${files.size} file${if (files.size != 1) "s" else ""}",
+                                    style = MaterialTheme.typography.labelMedium,
+                                    color = MaterialTheme.colorScheme.onSurfaceVariant,
+                                )
+                                Text(
+                                    files.sumOf { it.localSizeBytes }.toDisplaySize(),
+                                    style = MaterialTheme.typography.labelMedium,
+                                    color = MaterialTheme.colorScheme.onSurfaceVariant,
+                                )
+                            }
                        }
                    }

@@ -146,7 +237,7 @@ fun FilesScreen(
                            if (idx < 0) "" else f.relativePath.substring(0, idx)
                        }
                        grouped.forEach { (dir, dirFiles) ->
-                            if (dir.isNotEmpty()) {
+                            if (dir.isNotEmpty() && !isSelectionMode) {
                                item(key = "dir_$dir") {
                                    Row(
                                        modifier = Modifier.padding(top = 12.dp, bottom = 4.dp),
@@ -167,12 +258,16 @@ fun FilesScreen(
                                }
                            }
                            items(dirFiles, key = { "${it.syncPairId}_${it.relativePath}" }) { file ->
-                                FileRow(file = file, isInSubDir = dir.isNotEmpty(), vm = vm)
+                                FileRow(
+                                    file = file,
+                                    isInSubDir = dir.isNotEmpty() && !isSelectionMode,
+                                    isSelectionMode = isSelectionMode,
+                                    isSelected = vm.fileKey(file) in selectedKeys,
+                                    vm = vm,
+                                )
                                HorizontalDivider(
                                    color = MaterialTheme.colorScheme.outline.copy(alpha = 0.25f),
-                                    modifier = Modifier.padding(
-                                        start = if (dir.isNotEmpty()) 38.dp else 16.dp
-                                    ),
+                                    modifier = Modifier.padding(start = if (dir.isNotEmpty() && !isSelectionMode) 38.dp else 16.dp),
                                )
                            }
                        }
@@ -182,6 +277,32 @@ fun FilesScreen(
            }
        }

+        // Download progress
+        if (isDownloading) {
+            Box(
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .align(Alignment.BottomCenter),
+            ) {
+                Surface(
+                    color = MaterialTheme.colorScheme.surfaceVariant,
+                    tonalElevation = 4.dp,
+                    modifier = Modifier.fillMaxWidth(),
+                ) {
+                    Row(
+                        modifier = Modifier
+                            .fillMaxWidth()
+                            .padding(horizontal = 16.dp, vertical = 10.dp),
+                        verticalAlignment = Alignment.CenterVertically,
+                        horizontalArrangement = Arrangement.spacedBy(12.dp),
+                    ) {
+                        CircularProgressIndicator(modifier = Modifier.size(18.dp), strokeWidth = 2.dp)
+                        Text("Downloading for preview…", style = MaterialTheme.typography.bodySmall)
+                    }
+                }
+            }
+        }
+
        SnackbarHost(
            hostState = snackbarHostState,
            modifier = Modifier.align(Alignment.BottomCenter),
@@ -211,8 +332,15 @@ private fun FilesEmptyState(icon: ImageVector, title: String, subtitle: String)
    }
 }

+@OptIn(ExperimentalFoundationApi::class)
@Composable
-private fun FileRow(file: SyncFileStateEntity, isInSubDir: Boolean, vm: FilesViewModel) {
+private fun FileRow(
+    file: SyncFileStateEntity,
+    isInSubDir: Boolean,
+    isSelectionMode: Boolean,
+    isSelected: Boolean,
+    vm: FilesViewModel,
+) {
    val name = file.relativePath.substringAfterLast('/')
    val timeFmt = DateTimeFormatter.ofLocalizedDate(FormatStyle.SHORT)
    val syncedAt = file.lastSyncedAt.atZone(ZoneId.systemDefault()).let { timeFmt.format(it) }
@@ -224,10 +352,7 @@ private fun FileRow(file: SyncFileStateEntity, isInSubDir: Boolean, vm: FilesVie
    if (showRenameDialog) {
        RenameDialog(
            currentName = name,
-            onConfirm = { newName ->
-                vm.renameFile(file, newName)
-                showRenameDialog = false
-            },
+            onConfirm = { newName -> vm.renameFile(file, newName); showRenameDialog = false },
            onDismiss = { showRenameDialog = false },
        )
    }
@@ -249,89 +374,104 @@ private fun FileRow(file: SyncFileStateEntity, isInSubDir: Boolean, vm: FilesVie
        )
    }

-    Row(
+    Surface(
+        color = if (isSelected) MaterialTheme.colorScheme.primaryContainer.copy(alpha = 0.4f)
+                else MaterialTheme.colorScheme.surface,
        modifier = Modifier
            .fillMaxWidth()
-            .padding(
+            .combinedClickable(
+                onClick = {
+                    if (isSelectionMode) vm.toggleSelection(file) else menuExpanded = true
+                },
+                onLongClick = { vm.toggleSelection(file) },
+            ),
+    ) {
+        Row(
+            modifier = Modifier.padding(
                start = if (isInSubDir) 22.dp else 0.dp,
                top = 10.dp,
                bottom = 10.dp,
+                end = 0.dp,
            ),
-        verticalAlignment = Alignment.CenterVertically,
-    ) {
-        Surface(
-            shape = RoundedCornerShape(8.dp),
-            color = MaterialTheme.colorScheme.secondaryContainer,
-            modifier = Modifier.size(32.dp),
+            verticalAlignment = Alignment.CenterVertically,
        ) {
-            Box(contentAlignment = Alignment.Center) {
-                Icon(
-                    fileIcon(name), contentDescription = null,
-                    modifier = Modifier.size(16.dp),
-                    tint = MaterialTheme.colorScheme.onSecondaryContainer,
+            if (isSelectionMode) {
+                Checkbox(
+                    checked = isSelected,
+                    onCheckedChange = { vm.toggleSelection(file) },
+                    modifier = Modifier.padding(horizontal = 4.dp),
                )
-            }
-        }
-        Spacer(Modifier.width(12.dp))
-        Column(modifier = Modifier.weight(1f)) {
-            Text(
-                name,
-                style = MaterialTheme.typography.bodySmall,
-                maxLines = 1,
-                overflow = TextOverflow.Ellipsis,
-            )
-            Text(
-                "Synced $syncedAt · ${file.localSizeBytes.toDisplaySize()}",
-                style = MaterialTheme.typography.labelSmall,
-                color = MaterialTheme.colorScheme.onSurfaceVariant,
-            )
-        }
-        Box {
-            IconButton(onClick = { menuExpanded = true }, modifier = Modifier.size(36.dp)) {
-                Icon(
-                    Icons.Default.MoreVert, contentDescription = "File options",
-                    modifier = Modifier.size(18.dp),
-                    tint = MaterialTheme.colorScheme.onSurfaceVariant,
-                )
-            }
-            DropdownMenu(expanded = menuExpanded, onDismissRequest = { menuExpanded = false }) {
-                DropdownMenuItem(
-                    text = { Text("Open") },
-                    leadingIcon = { Icon(Icons.Default.OpenInNew, contentDescription = null) },
-                    onClick = { menuExpanded = false; vm.openFile(file) },
-                )
-                DropdownMenuItem(
-                    text = { Text("Share") },
-                    leadingIcon = { Icon(Icons.Default.Share, contentDescription = null) },
-                    onClick = { menuExpanded = false; vm.shareFile(file) },
-                )
-                HorizontalDivider()
-                DropdownMenuItem(
-                    text = { Text("Rename") },
-                    leadingIcon = { Icon(Icons.Default.Edit, contentDescription = null) },
-                    onClick = { menuExpanded = false; showRenameDialog = true },
-                )
-                DropdownMenuItem(
-                    text = { Text("Delete", color = MaterialTheme.colorScheme.error) },
-                    leadingIcon = {
+            } else {
+                Surface(
+                    shape = RoundedCornerShape(8.dp),
+                    color = MaterialTheme.colorScheme.secondaryContainer,
+                    modifier = Modifier.size(32.dp),
+                ) {
+                    Box(contentAlignment = Alignment.Center) {
                        Icon(
-                            Icons.Default.Delete, contentDescription = null,
-                            tint = MaterialTheme.colorScheme.error,
+                            fileIcon(name), contentDescription = null,
+                            modifier = Modifier.size(16.dp),
+                            tint = MaterialTheme.colorScheme.onSecondaryContainer,
                        )
-                    },
-                    onClick = { menuExpanded = false; showDeleteDialog = true },
+                    }
+                }
+                Spacer(Modifier.width(12.dp))
+            }
+            Column(modifier = Modifier.weight(1f)) {
+                Text(
+                    name,
+                    style = MaterialTheme.typography.bodySmall,
+                    maxLines = 1,
+                    overflow = TextOverflow.Ellipsis,
                )
+                Text(
+                    "Synced $syncedAt · ${file.localSizeBytes.toDisplaySize()}",
+                    style = MaterialTheme.typography.labelSmall,
+                    color = MaterialTheme.colorScheme.onSurfaceVariant,
+                )
+            }
+            if (!isSelectionMode) {
+                Box {
+                    IconButton(onClick = { menuExpanded = true }, modifier = Modifier.size(36.dp)) {
+                        Icon(
+                            Icons.Default.MoreVert, contentDescription = "File options",
+                            modifier = Modifier.size(18.dp),
+                            tint = MaterialTheme.colorScheme.onSurfaceVariant,
+                        )
+                    }
+                    DropdownMenu(expanded = menuExpanded, onDismissRequest = { menuExpanded = false }) {
+                        DropdownMenuItem(
+                            text = { Text("Open") },
+                            leadingIcon = { Icon(Icons.Default.OpenInNew, null) },
+                            onClick = { menuExpanded = false; vm.openFile(file) },
+                        )
+                        DropdownMenuItem(
+                            text = { Text("Share") },
+                            leadingIcon = { Icon(Icons.Default.Share, null) },
+                            onClick = { menuExpanded = false; vm.shareFile(file) },
+                        )
+                        HorizontalDivider()
+                        DropdownMenuItem(
+                            text = { Text("Rename") },
+                            leadingIcon = { Icon(Icons.Default.Edit, null) },
+                            onClick = { menuExpanded = false; showRenameDialog = true },
+                        )
+                        DropdownMenuItem(
+                            text = { Text("Delete", color = MaterialTheme.colorScheme.error) },
+                            leadingIcon = {
+                                Icon(Icons.Default.Delete, null, tint = MaterialTheme.colorScheme.error)
+                            },
+                            onClick = { menuExpanded = false; showDeleteDialog = true },
+                        )
+                    }
+                }
            }
        }
    }
 }

@Composable
-private fun RenameDialog(
-    currentName: String,
-    onConfirm: (String) -> Unit,
-    onDismiss: () -> Unit,
-) {
+private fun RenameDialog(currentName: String, onConfirm: (String) -> Unit, onDismiss: () -> Unit) {
    var newName by remember { mutableStateOf(currentName) }
    AlertDialog(
        onDismissRequest = onDismiss,
@@ -350,15 +490,12 @@ private fun RenameDialog(
            TextButton(
                onClick = {
                    val trimmed = newName.trim()
-                    if (trimmed.isNotBlank() && trimmed != currentName) onConfirm(trimmed)
-                    else onDismiss()
+                    if (trimmed.isNotBlank() && trimmed != currentName) onConfirm(trimmed) else onDismiss()
                },
                enabled = newName.isNotBlank(),
            ) { Text("Rename") }
        },
-        dismissButton = {
-            TextButton(onClick = onDismiss) { Text("Cancel") }
-        },
+        dismissButton = { TextButton(onClick = onDismiss) { Text("Cancel") } },
    )
 }

@@ -369,22 +506,16 @@ private fun String.mimeType(): String {

 private fun fileIcon(name: String) = when {
    name.endsWith(".pdf", ignoreCase = true) -> Icons.Default.PictureAsPdf
-    name.endsWith(".jpg", ignoreCase = true) ||
-    name.endsWith(".jpeg", ignoreCase = true) ||
-    name.endsWith(".png", ignoreCase = true) ||
-    name.endsWith(".gif", ignoreCase = true) ||
+    name.endsWith(".jpg", ignoreCase = true) || name.endsWith(".jpeg", ignoreCase = true) ||
+    name.endsWith(".png", ignoreCase = true) || name.endsWith(".gif", ignoreCase = true) ||
    name.endsWith(".webp", ignoreCase = true) -> Icons.Default.Image
-    name.endsWith(".mp4", ignoreCase = true) ||
-    name.endsWith(".mkv", ignoreCase = true) ||
+    name.endsWith(".mp4", ignoreCase = true) || name.endsWith(".mkv", ignoreCase = true) ||
    name.endsWith(".mov", ignoreCase = true) -> Icons.Default.VideoFile
-    name.endsWith(".mp3", ignoreCase = true) ||
-    name.endsWith(".m4a", ignoreCase = true) ||
+    name.endsWith(".mp3", ignoreCase = true) || name.endsWith(".m4a", ignoreCase = true) ||
    name.endsWith(".ogg", ignoreCase = true) -> Icons.Default.AudioFile
-    name.endsWith(".zip", ignoreCase = true) ||
-    name.endsWith(".tar", ignoreCase = true) ||
+    name.endsWith(".zip", ignoreCase = true) || name.endsWith(".tar", ignoreCase = true) ||
    name.endsWith(".gz", ignoreCase = true) -> Icons.Default.FolderZip
-    name.endsWith(".txt", ignoreCase = true) ||
-    name.endsWith(".md", ignoreCase = true) -> Icons.Default.TextSnippet
+    name.endsWith(".txt", ignoreCase = true) || name.endsWith(".md", ignoreCase = true) -> Icons.Default.TextSnippet
    else -> Icons.Default.InsertDriveFile
 }

@@ -1,17 +1,21 @@
 package com.syncflow.ui.files

 import android.content.Context
+import android.media.MediaScannerConnection
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.syncflow.data.db.SyncFileStateDao
 import com.syncflow.data.db.SyncPairDao
 import com.syncflow.data.db.entities.SyncFileStateEntity
 import com.syncflow.data.db.entities.SyncPairEntity
+import com.syncflow.data.providers.ProviderFactory
+import com.syncflow.data.repository.AccountRepository
 import dagger.hilt.android.lifecycle.HiltViewModel
 import dagger.hilt.android.qualifiers.ApplicationContext
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.flow.*
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.flow.update
 import timber.log.Timber
 import java.io.File
 import javax.inject.Inject
@@ -19,6 +23,7 @@ import javax.inject.Inject
 sealed class FileAction {
    data class Open(val file: File) : FileAction()
    data class Share(val file: File) : FileAction()
+    data class ShareMultiple(val files: List<File>) : FileAction()
    data class Error(val message: String) : FileAction()
 }

@@ -27,6 +32,8 @@ sealed class FileAction {
 class FilesViewModel @Inject constructor(
    private val syncPairDao: SyncPairDao,
    private val fileStateDao: SyncFileStateDao,
+    private val accountRepository: AccountRepository,
+    private val providerFactory: ProviderFactory,
    @ApplicationContext private val context: Context,
 ) : ViewModel() {

@@ -50,22 +57,42 @@ class FilesViewModel @Inject constructor(
    private val _fileAction = MutableSharedFlow<FileAction>()
    val fileAction: SharedFlow<FileAction> = _fileAction

+    private val _isDownloading = MutableStateFlow(false)
+    val isDownloading: StateFlow<Boolean> = _isDownloading
+
+    private val _selectedKeys = MutableStateFlow<Set<String>>(emptySet())
+    val selectedKeys: StateFlow<Set<String>> = _selectedKeys
+    val isSelectionMode: StateFlow<Boolean> = _selectedKeys.map { it.isNotEmpty() }
+        .stateIn(viewModelScope, SharingStarted.WhileSubscribed(5_000), false)
+    val selectedCount: StateFlow<Int> = _selectedKeys.map { it.size }
+        .stateIn(viewModelScope, SharingStarted.WhileSubscribed(5_000), 0)
+
    fun selectPair(id: Long) { _selectedPairId.value = id }

    fun openFile(file: SyncFileStateEntity) {
-        val resolved = resolveFile(file) ?: return
-        viewModelScope.launch { _fileAction.emit(FileAction.Open(resolved)) }
+        val resolved = resolveFile(file, emitErrorIfMissing = false)
+        if (resolved != null) {
+            // Ensure MediaStore knows about this file so gallery apps can open it
+            MediaScannerConnection.scanFile(context, arrayOf(resolved.absolutePath), null, null)
+            viewModelScope.launch { _fileAction.emit(FileAction.Open(resolved)) }
+        } else {
+            downloadAndOpen(file)
+        }
    }

    fun shareFile(file: SyncFileStateEntity) {
-        val resolved = resolveFile(file) ?: return
-        viewModelScope.launch { _fileAction.emit(FileAction.Share(resolved)) }
+        val resolved = resolveFile(file, emitErrorIfMissing = false)
+        if (resolved != null) {
+            viewModelScope.launch { _fileAction.emit(FileAction.Share(resolved)) }
+        } else {
+            downloadAndShare(file)
+        }
    }

    fun deleteFile(file: SyncFileStateEntity) {
        viewModelScope.launch {
            try {
-                val resolved = resolveFile(file)
+                val resolved = resolveFile(file, emitErrorIfMissing = false)
                resolved?.delete()
                fileStateDao.delete(file.syncPairId, file.relativePath)
            } catch (e: Exception) {
@@ -85,7 +112,6 @@ class FilesViewModel @Inject constructor(
                    _fileAction.emit(FileAction.Error("Rename failed"))
                    return@launch
                }
-                // Update DB: delete old state; the next sync will re-detect as a new upload
                fileStateDao.delete(file.syncPairId, file.relativePath)
            } catch (e: Exception) {
                Timber.e(e, "Rename failed: ${file.relativePath}")
@@ -94,12 +120,110 @@ class FilesViewModel @Inject constructor(
        }
    }

-    private fun resolveFile(file: SyncFileStateEntity): File? {
+    fun isSelected(file: SyncFileStateEntity): Boolean = fileKey(file) in _selectedKeys.value
+
+    fun toggleSelection(file: SyncFileStateEntity) {
+        val key = fileKey(file)
+        _selectedKeys.update { if (key in it) it - key else it + key }
+    }
+
+    fun clearSelection() { _selectedKeys.value = emptySet() }
+
+    fun deleteSelected() {
+        viewModelScope.launch {
+            val toDelete = files.value.filter { isSelected(it) }
+            toDelete.forEach { file ->
+                try {
+                    resolveFile(file, emitErrorIfMissing = false)?.delete()
+                    fileStateDao.delete(file.syncPairId, file.relativePath)
+                } catch (e: Exception) {
+                    Timber.e(e, "Bulk delete failed: ${file.relativePath}")
+                }
+            }
+            clearSelection()
+        }
+    }
+
+    fun shareSelected() {
+        viewModelScope.launch {
+            val toShare = files.value.filter { isSelected(it) }
+            val resolved = toShare.mapNotNull { resolveFile(it, emitErrorIfMissing = false) }
+            if (resolved.isEmpty()) {
+                _fileAction.emit(FileAction.Error("No local files available to share"))
+                return@launch
+            }
+            _fileAction.emit(FileAction.ShareMultiple(resolved))
+        }
+    }
+
+    fun fileKey(file: SyncFileStateEntity) = "${file.syncPairId}:${file.relativePath}"
+
+    // ── Download-then-open/share ──────────────────────────────────────────────
+
+    private fun downloadAndOpen(file: SyncFileStateEntity) {
+        viewModelScope.launch {
+            downloadToCache(file)?.let { cached ->
+                _fileAction.emit(FileAction.Open(cached))
+            }
+        }
+    }
+
+    private fun downloadAndShare(file: SyncFileStateEntity) {
+        viewModelScope.launch {
+            downloadToCache(file)?.let { cached ->
+                _fileAction.emit(FileAction.Share(cached))
+            }
+        }
+    }
+
+    private suspend fun downloadToCache(file: SyncFileStateEntity): File? {
+        val pair = selectedPair.value ?: run {
+            _fileAction.emit(FileAction.Error("No sync pair selected"))
+            return null
+        }
+        val account = accountRepository.getAccount(pair.accountId) ?: run {
+            _fileAction.emit(FileAction.Error("Cloud account not found"))
+            return null
+        }
+        val provider = providerFactory.create(account)
+        val fileName = file.relativePath.substringAfterLast('/')
+        val cacheFile = File(context.cacheDir, "syncflow_open/$fileName")
+        cacheFile.parentFile?.mkdirs()
+
+        _isDownloading.value = true
+        return try {
+            cacheFile.outputStream().use { out ->
+                provider.downloadFile("${pair.remotePath}/${file.relativePath}", out) { }.getOrThrow()
+            }
+            MediaScannerConnection.scanFile(context, arrayOf(cacheFile.absolutePath), null, null)
+            cacheFile
+        } catch (e: Exception) {
+            Timber.e(e, "Download for preview failed: ${file.relativePath}")
+            cacheFile.delete()
+            _fileAction.emit(FileAction.Error("Download failed: ${e.message}"))
+            null
+        } finally {
+            _isDownloading.value = false
+        }
+    }
+
+    // ── Path resolution ───────────────────────────────────────────────────────
+
+    private fun resolveFile(file: SyncFileStateEntity, emitErrorIfMissing: Boolean = true): File? {
+        // Guard against path traversal from untrusted server responses
+        if (file.relativePath.contains("..")) {
+            viewModelScope.launch { _fileAction.emit(FileAction.Error("Invalid file path")) }
+            return null
+        }
        val pair = selectedPair.value ?: return null
        val root = safTreeUriToRealPath(pair.localPath) ?: pair.localPath
+        // localPath is a content:// URI we couldn't resolve — File-based access won't work
+        if (root.startsWith("content://")) return null
        val f = File(root, file.relativePath)
        if (!f.exists()) {
-            viewModelScope.launch { _fileAction.emit(FileAction.Error("File not found on device")) }
+            if (emitErrorIfMissing) {
+                viewModelScope.launch { _fileAction.emit(FileAction.Error("File not found on device")) }
+            }
            return null
        }
        return f
@@ -2,27 +2,28 @@ package com.syncflow.ui.theme

 import androidx.compose.ui.graphics.Color

-// Primary — indigo
-val Indigo600  = Color(0xFF4F46E5)
-val Indigo900  = Color(0xFF312E81)
-val Indigo100  = Color(0xFFE0E7FF)
-val Indigo50   = Color(0xFFEEF2FF)
+// Primary — deep red (Passbolt-inspired)
+val Red900  = Color(0xFF7F0000)
+val Red700  = Color(0xFFB71C1C)
+val Red500  = Color(0xFFEF5350)
+val Red100  = Color(0xFFFFCDD2)
+val Red50   = Color(0xFFFFEBEE)

-// Secondary — teal
-val Teal600    = Color(0xFF0D9488)
-val Teal100    = Color(0xFFCCFBF1)
+// Secondary — deep orange
+val Orange700 = Color(0xFFE64A19)
+val Orange100 = Color(0xFFFBE9E7)

 // Tertiary — amber
-val Amber500   = Color(0xFFF59E0B)
-val Amber100   = Color(0xFFFEF3C7)
+val Amber500  = Color(0xFFFFB300)
+val Amber100  = Color(0xFFFFF8E1)

 // Neutrals
-val Slate50    = Color(0xFFF8FAFC)
-val Slate100   = Color(0xFFF1F5F9)
-val Slate200   = Color(0xFFE2E8F0)
-val Slate600   = Color(0xFF475569)
-val Slate900   = Color(0xFF0F172A)
+val Gray50   = Color(0xFFF8F9FA)
+val Gray100  = Color(0xFFF3F4F6)
+val Gray200  = Color(0xFFE5E7EB)
+val Gray600  = Color(0xFF6B7280)
+val Gray900  = Color(0xFF111827)

 // Semantic
-val GreenSuccess = Color(0xFF16A34A)
-val RedError     = Color(0xFFDC2626)
+val GreenSuccess = Color(0xFF2E7D32)
+val RedError     = Color(0xFFEF5350)
@@ -13,41 +13,43 @@ import androidx.compose.ui.unit.sp
 import androidx.core.view.WindowCompat

 private val LightColors = lightColorScheme(
-    primary            = Indigo600,
+    primary            = Red700,
    onPrimary          = Color.White,
-    primaryContainer   = Indigo100,
-    onPrimaryContainer = Indigo900,
-    secondary          = Teal600,
+    primaryContainer   = Red50,
+    onPrimaryContainer = Red900,
+    secondary          = Orange700,
    onSecondary        = Color.White,
-    secondaryContainer = Teal100,
+    secondaryContainer = Orange100,
    tertiary           = Amber500,
    tertiaryContainer  = Amber100,
-    background         = Slate50,
+    background         = Gray50,
    surface            = Color.White,
-    surfaceVariant     = Slate100,
-    onSurfaceVariant   = Slate600,
+    surfaceVariant     = Gray100,
+    onSurface          = Gray900,
+    onSurfaceVariant   = Gray600,
    error              = RedError,
-    errorContainer     = Color(0xFFFEE2E2),
-    outline            = Slate200,
+    errorContainer     = Red50,
+    outline            = Gray200,
 )

 private val DarkColors = darkColorScheme(
-    primary            = Color(0xFF818CF8),
-    onPrimary          = Indigo900,
-    primaryContainer   = Color(0xFF3730A3),
-    onPrimaryContainer = Indigo100,
-    secondary          = Color(0xFF2DD4BF),
-    onSecondary        = Color(0xFF003731),
-    secondaryContainer = Color(0xFF00504A),
+    primary            = Red500,
+    onPrimary          = Color.White,
+    primaryContainer   = Red900,
+    onPrimaryContainer = Red100,
+    secondary          = Color(0xFFFF7043),
+    onSecondary        = Color.White,
+    secondaryContainer = Color(0xFF4E1500),
    tertiary           = Amber500,
-    tertiaryContainer  = Color(0xFF92400E),
-    background         = Color(0xFF0F0F1A),
-    surface            = Color(0xFF1A1A2E),
-    surfaceVariant     = Color(0xFF252538),
-    onSurfaceVariant   = Color(0xFF94A3B8),
-    error              = Color(0xFFF87171),
-    errorContainer     = Color(0xFF7F1D1D),
-    outline            = Color(0xFF334155),
+    tertiaryContainer  = Color(0xFF3E2700),
+    background         = Color(0xFF0F0F0F),
+    surface            = Color(0xFF1C1C1C),
+    surfaceVariant     = Color(0xFF2A2A2A),
+    onSurface          = Color(0xFFEAEAEA),
+    onSurfaceVariant   = Color(0xFF9E9E9E),
+    error              = Color(0xFFFF5252),
+    errorContainer     = Color(0xFF5C0000),
+    outline            = Color(0xFF3D3D3D),
 )

 private val AppTypography = Typography(
@@ -13,7 +13,9 @@ import android.os.Looper
 import android.provider.DocumentsContract
 import androidx.core.app.NotificationCompat
 import androidx.work.ExistingWorkPolicy
+import androidx.work.WorkInfo
 import androidx.work.WorkManager
+import kotlinx.coroutines.flow.first
 import com.syncflow.MainActivity
 import com.syncflow.R
 import com.syncflow.data.db.SyncFileStateDao
@@ -34,7 +36,8 @@ class FileWatchService : Service() {
    private val scope = CoroutineScope(Dispatchers.IO + SupervisorJob())
    private val mainHandler = Handler(Looper.getMainLooper())

-    private val fileObservers = mutableMapOf<Long, FileObserver>()
+    // Multiple FileObserver instances per pair: one per directory (recursive)
+    private val fileObservers = mutableMapOf<Long, MutableList<FileObserver>>()
    private val contentObservers = mutableMapOf<Long, ContentObserver>()
    private val debounceJobs = mutableMapOf<Long, Job>()

@@ -106,7 +109,7 @@ class FileWatchService : Service() {
            }
        }

-        val count = fileObservers.size + contentObservers.size
+        val count = fileObservers.keys.size + contentObservers.size
        updateNotification(count)

        if (count == 0) {
@@ -138,23 +141,44 @@ class FileWatchService : Service() {
            Timber.w("FileWatchService: path does not exist for pair $pairId: $path")
            return
        }
+        fileObservers[pairId] = mutableListOf()
+        watchDirRecursive(dir, pairId, wifiOnly, chargingOnly)
+        Timber.d("FileWatchService: watching pair $pairId at $path (${fileObservers[pairId]?.size} dirs)")
+        scope.launch { catchupScan(pairId, dir, wifiOnly, chargingOnly) }
+    }
+
+    private fun watchDirRecursive(dir: File, pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean) {
+        if (!dir.isDirectory) return
        val mask = FileObserver.CREATE or FileObserver.DELETE or FileObserver.MODIFY or
                   FileObserver.MOVED_FROM or FileObserver.MOVED_TO
        val observer = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
            object : FileObserver(dir, mask) {
-                override fun onEvent(event: Int, p: String?) = onChangeDetected(pairId, wifiOnly, chargingOnly)
+                override fun onEvent(event: Int, path: String?) {
+                    if (event and FileObserver.CREATE != 0 && path != null) {
+                        val created = File(dir, path)
+                        if (created.isDirectory) watchDirRecursive(created, pairId, wifiOnly, chargingOnly)
+                    }
+                    onChangeDetected(pairId, wifiOnly, chargingOnly)
+                }
            }
        } else {
            @Suppress("DEPRECATION")
-            object : FileObserver(path, mask) {
-                override fun onEvent(event: Int, p: String?) = onChangeDetected(pairId, wifiOnly, chargingOnly)
+            object : FileObserver(dir.absolutePath, mask) {
+                override fun onEvent(event: Int, path: String?) {
+                    if (event and FileObserver.CREATE != 0 && path != null) {
+                        val created = File(dir, path)
+                        if (created.isDirectory) watchDirRecursive(created, pairId, wifiOnly, chargingOnly)
+                    }
+                    onChangeDetected(pairId, wifiOnly, chargingOnly)
+                }
            }
        }
        observer.startWatching()
-        fileObservers[pairId] = observer
-        Timber.d("FileWatchService: watching filesystem path for pair $pairId: $path")
-        // Check if anything changed while the service was not running
-        scope.launch { catchupScan(pairId, dir, wifiOnly, chargingOnly) }
+        fileObservers.getOrPut(pairId) { mutableListOf() }.add(observer)
+        // Recursively watch existing subdirectories
+        dir.listFiles()?.filter { it.isDirectory }?.forEach { sub ->
+            watchDirRecursive(sub, pairId, wifiOnly, chargingOnly)
+        }
    }

    private suspend fun catchupScan(pairId: Long, dir: File, wifiOnly: Boolean, chargingOnly: Boolean) {
@@ -192,14 +216,39 @@ class FileWatchService : Service() {
            val pair = syncPairDao.getById(pairId)
            if (pair == null || !pair.isEnabled) return@launch
            Timber.d("FileWatchService: triggering sync for pair $pairId after debounce")
-            val req = SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly)
+
+            val req = SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly, silent = true)
            WorkManager.getInstance(applicationContext)
                .enqueueUniqueWork("onchange_$pairId", ExistingWorkPolicy.KEEP, req)
+
+            // Update notification while sync is in progress
+            updateNotificationDynamic("Syncing: ${pair.name}…")
+
+            // Wait for completion and show result in the persistent notification
+            scope.launch {
+                try {
+                    val info = WorkManager.getInstance(applicationContext)
+                        .getWorkInfoByIdFlow(req.id)
+                        .first { it?.state?.isFinished == true }
+                    val summary = info?.outputData?.getString(SyncWorker.KEY_RESULT_SUMMARY)
+                    val watchCount = fileObservers.keys.size + contentObservers.size
+                    val watching = "Watching $watchCount folder${if (watchCount != 1) "s" else ""}"
+                    if (info?.state == WorkInfo.State.SUCCEEDED && summary != null) {
+                        updateNotificationDynamic("${pair.name}: $summary — $watching")
+                    } else {
+                        updateNotificationDynamic("$watching")
+                    }
+                    delay(12_000)
+                    updateNotificationDynamic(null) // revert to default watching text
+                } catch (_: Exception) {
+                    updateNotificationDynamic(null)
+                }
+            }
        }
    }

    private fun clearWatchers() {
-        fileObservers.values.forEach { it.stopWatching() }
+        fileObservers.values.flatten().forEach { it.stopWatching() }
        fileObservers.clear()
        contentObservers.values.forEach { contentResolver.unregisterContentObserver(it) }
        contentObservers.clear()
@@ -219,7 +268,7 @@ class FileWatchService : Service() {
        }
    }

-    private fun buildNotification(count: Int): Notification {
+    private fun buildNotification(count: Int, overrideText: String? = null): Notification {
        val tapIntent = PendingIntent.getActivity(
            this, 0,
            Intent(this, MainActivity::class.java).apply { flags = Intent.FLAG_ACTIVITY_SINGLE_TOP },
@@ -228,7 +277,7 @@ class FileWatchService : Service() {
        return NotificationCompat.Builder(this, CHANNEL_WATCH)
            .setContentTitle("SyncFlow")
            .setContentText(
-                if (count > 0) "Watching $count folder${if (count != 1) "s" else ""} for changes"
+                overrideText ?: if (count > 0) "Watching $count folder${if (count != 1) "s" else ""} for changes"
                else "Starting file watcher…"
            )
            .setSmallIcon(R.drawable.ic_sync)
@@ -242,4 +291,10 @@ class FileWatchService : Service() {
        val nm = getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager
        nm.notify(NOTIFICATION_ID, buildNotification(count))
    }
+
+    private fun updateNotificationDynamic(overrideText: String?) {
+        val count = fileObservers.keys.size + contentObservers.size
+        val nm = getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager
+        nm.notify(NOTIFICATION_ID, buildNotification(count, overrideText))
+    }
 }
@@ -39,6 +39,8 @@ class SyncWorker @AssistedInject constructor(
        val pair = syncPairDao.getById(pairId) ?: return Result.failure()
        val account = accountRepository.getAccount(pair.accountId) ?: return Result.failure()

+        val silent = inputData.getBoolean(KEY_SILENT, false)
+
        ensureChannels()
        setForeground(buildForegroundInfo(pair.name, "Syncing…"))

@@ -47,7 +49,15 @@ class SyncWorker @AssistedInject constructor(
            val provider = providerFactory.create(account)
            val result = syncEngine.sync(domainPair, provider)

-            if (result.error != null && pair.notifyOnError) {
+            val lines = buildList {
+                if (result.uploaded > 0)   add("↑${result.uploaded}")
+                if (result.downloaded > 0) add("↓${result.downloaded}")
+                if (result.deleted > 0)    add("🗑${result.deleted}")
+                if (result.conflicts > 0)  add("⚠${result.conflicts}")
+            }
+            val resultSummary = if (lines.isEmpty()) "Up to date" else lines.joinToString(" ")
+
+            if (!silent && result.error != null && pair.notifyOnError) {
                notify(
                    id = pairId.toInt() + RESULT_ID_OFFSET,
                    channelId = CHANNEL_ALERTS,
@@ -55,28 +65,28 @@ class SyncWorker @AssistedInject constructor(
                    text = result.error.message ?: "Unknown error",
                    priority = NotificationCompat.PRIORITY_DEFAULT,
                )
-            } else if (pair.notifyOnComplete && result.error == null) {
-                val lines = buildList {
+            } else if (!silent && pair.notifyOnComplete && result.error == null) {
+                val fullLines = buildList {
                    if (result.uploaded > 0)   add("↑ Uploaded: ${result.uploaded} file${if (result.uploaded != 1) "s" else ""}")
                    if (result.downloaded > 0) add("↓ Downloaded: ${result.downloaded} file${if (result.downloaded != 1) "s" else ""}")
                    if (result.deleted > 0)    add("🗑 Deleted: ${result.deleted} file${if (result.deleted != 1) "s" else ""}")
                    if (result.conflicts > 0)  add("⚠ ${result.conflicts} conflict${if (result.conflicts != 1) "s" else ""}")
                }
-                val summary = if (lines.isEmpty()) "Up to date — nothing to sync" else lines.joinToString("\n")
+                val summary = if (fullLines.isEmpty()) "Up to date — nothing to sync" else fullLines.joinToString("\n")
                notify(
                    id = pairId.toInt() + RESULT_ID_OFFSET,
                    channelId = CHANNEL_COMPLETE,
-                    title = "${pair.name} — Changes synced",
-                    text = if (lines.isEmpty()) summary else lines.first(),
+                    title = "${pair.name} — Synced",
+                    text = if (fullLines.isEmpty()) summary else fullLines.first(),
                    bigText = summary,
                    priority = NotificationCompat.PRIORITY_LOW,
                )
            }

-            Result.success()
+            Result.success(workDataOf(KEY_RESULT_SUMMARY to resultSummary))
        } catch (e: Exception) {
            Timber.e(e, "SyncWorker failed for pair $pairId")
-            if (pair.notifyOnError) {
+            if (!silent && pair.notifyOnError) {
                notify(
                    id = pairId.toInt() + RESULT_ID_OFFSET,
                    channelId = CHANNEL_ALERTS,
@@ -146,19 +156,21 @@ class SyncWorker @AssistedInject constructor(

    companion object {
        const val KEY_PAIR_ID = "pair_id"
+        const val KEY_SILENT = "silent"
+        const val KEY_RESULT_SUMMARY = "result_summary"
        private const val NOTIFICATION_ID = 1001
        private const val RESULT_ID_OFFSET = 2000
        private const val CHANNEL_PROGRESS = "sync_progress"
        private const val CHANNEL_COMPLETE = "sync_complete"
        private const val CHANNEL_ALERTS   = "sync_alerts"

-        fun buildOneTimeRequest(pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean): OneTimeWorkRequest {
+        fun buildOneTimeRequest(pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean, silent: Boolean = false): OneTimeWorkRequest {
            val constraints = Constraints.Builder()
                .setRequiredNetworkType(if (wifiOnly) NetworkType.UNMETERED else NetworkType.CONNECTED)
                .setRequiresCharging(chargingOnly)
                .build()
            return OneTimeWorkRequestBuilder<SyncWorker>()
-                .setInputData(workDataOf(KEY_PAIR_ID to pairId))
+                .setInputData(workDataOf(KEY_PAIR_ID to pairId, KEY_SILENT to silent))
                .setConstraints(constraints)
                .setBackoffCriteria(BackoffPolicy.EXPONENTIAL, 30, TimeUnit.SECONDS)
                .addTag("sync_$pairId")
@@ -1,11 +1,20 @@
 <?xml version="1.0" encoding="utf-8"?>
-<shape xmlns:android="http://schemas.android.com/apk/res/android">
-    <gradient
-        android:type="radial"
-        android:gradientRadius="80%"
-        android:centerX="0.35"
-        android:centerY="0.3"
-        android:startColor="#1C1124"
-        android:centerColor="#0E0A18"
-        android:endColor="#060408"/>
-</shape>
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:aapt="http://schemas.android.com/aapt"
+    android:width="108dp"
+    android:height="108dp"
+    android:viewportWidth="108"
+    android:viewportHeight="108">
+
+    <!-- Deep purple-black base, matching the knot reference icon's dark background -->
+    <path android:pathData="M0,0 H108 V108 H0 Z">
+        <aapt:attr name="android:fillColor">
+            <gradient android:type="radial"
+                android:gradientRadius="76"
+                android:centerX="54" android:centerY="44"
+                android:startColor="#1E1628"
+                android:endColor="#080610"/>
+        </aapt:attr>
+    </path>
+
+</vector>
@@ -1,4 +1,24 @@
 <?xml version="1.0" encoding="utf-8"?>
+<!--
+  SyncFlow icon foreground (108x108dp, safe zone 72dp centred at 54,54).
+
+  Design: two thick tube-style sync arcs that form a circular refresh symbol,
+  inspired by the braided knot icon's color palette:
+    Arc 1 (left side, CW): coral #E8665A -> orange #E8A040
+    Arc 2 (right side, CW): steel blue #4A7FD4 -> purple #7B5EA7
+  Each arc has a thin inner highlight to give a 3D glossy tube look.
+
+  Geometry: circle centred at (54,54), radius 27.
+    Arc 1: 100 deg to 260 deg CW (160 deg, through west/left)
+      start (49.31, 80.59)  end (49.31, 27.41)
+    Arc 2: 280 deg to  80 deg CW (160 deg, through east/right)
+      start (58.69, 27.41)  end (58.69, 80.59)
+  20-degree gaps at top (~260-280) and bottom (~80-100).
+
+  Highlights at r=23 (inner edge):
+    Arc 1 highlight: (49.99, 76.64) to (49.99, 31.36)
+    Arc 2 highlight: (58.01, 31.36) to (58.01, 76.64)
+-->
 <vector xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:aapt="http://schemas.android.com/aapt"
    android:width="108dp"
@@ -6,90 +26,74 @@
    android:viewportWidth="108"
    android:viewportHeight="108">

-    <!-- Subtle inner glow -->
+    <!-- Arc 1 shadow (offset slightly down-right for depth) -->
    <path
-        android:pathData="M54,54m-30,0a30,30 0 1,0 60,0a30,30 0 1,0 -60,0"
-        android:fillColor="#0AFFFFFF"/>
-
-    <!-- ═══ Arrow 1: curving RIGHT (top) — coral gradient ═══ -->
-    <!-- Curved shaft via quadratic bezier: left→ dips up → right -->
-    <path
-        android:pathData="M28,40 Q54,22 80,36"
+        android:pathData="M 50.31,82.59 A 27,27 0 0,1 50.31,29.41"
        android:fillColor="#00000000"
-        android:strokeWidth="7"
+        android:strokeWidth="9"
        android:strokeLineCap="round"
-        android:strokeLineJoin="round">
+        android:strokeColor="#44000000"/>
+
+    <!-- Arc 2 shadow -->
+    <path
+        android:pathData="M 59.69,29.41 A 27,27 0 0,1 59.69,82.59"
+        android:fillColor="#00000000"
+        android:strokeWidth="9"
+        android:strokeLineCap="round"
+        android:strokeColor="#44000000"/>
+
+    <!-- Arc 1: left side, coral to orange -->
+    <path
+        android:pathData="M 49.31,80.59 A 27,27 0 0,1 49.31,27.41"
+        android:fillColor="#00000000"
+        android:strokeWidth="9"
+        android:strokeLineCap="round">
        <aapt:attr name="android:strokeColor">
            <gradient android:type="linear"
-                android:startX="28" android:startY="36"
-                android:endX="80" android:endY="36"
-                android:startColor="#FF6B6B"
-                android:endColor="#FFB347"/>
-        </aapt:attr>
-    </path>
-    <!-- Arrowhead at (80,36) pointing right/slightly-down -->
-    <path android:pathData="M76,28 L72,44 L88,38 Z">
-        <aapt:attr name="android:fillColor">
-            <gradient android:type="linear"
-                android:startX="72" android:startY="36"
-                android:endX="88" android:endY="36"
-                android:startColor="#FFB347"
-                android:endColor="#FF8C42"/>
+                android:startX="49.31" android:startY="80.59"
+                android:endX="49.31" android:endY="27.41"
+                android:startColor="#E8665A"
+                android:endColor="#E8A040"/>
        </aapt:attr>
    </path>

-    <!-- ═══ Arrow 2: curving LEFT (middle) — silver/white ═══ -->
-    <!-- Curved shaft: right→ dips down → left -->
+    <!-- Arc 2: right side, blue to purple -->
    <path
-        android:pathData="M80,56 Q54,74 28,60"
+        android:pathData="M 58.69,27.41 A 27,27 0 0,1 58.69,80.59"
        android:fillColor="#00000000"
-        android:strokeWidth="7"
-        android:strokeLineCap="round"
-        android:strokeLineJoin="round">
+        android:strokeWidth="9"
+        android:strokeLineCap="round">
        <aapt:attr name="android:strokeColor">
            <gradient android:type="linear"
-                android:startX="80" android:startY="60"
-                android:endX="28" android:endY="60"
-                android:startColor="#D0D8EE"
-                android:endColor="#8892AA"/>
-        </aapt:attr>
-    </path>
-    <!-- Arrowhead at (28,60) pointing left/slightly-down -->
-    <path android:pathData="M32,52 L36,68 L20,62 Z">
-        <aapt:attr name="android:fillColor">
-            <gradient android:type="linear"
-                android:startX="36" android:startY="60"
-                android:endX="20" android:endY="60"
-                android:startColor="#D0D8EE"
-                android:endColor="#8892AA"/>
+                android:startX="58.69" android:startY="27.41"
+                android:endX="58.69" android:endY="80.59"
+                android:startColor="#4A7FD4"
+                android:endColor="#7B5EA7"/>
        </aapt:attr>
    </path>

-    <!-- ═══ Arrow 3: curving RIGHT (bottom) — teal gradient ═══ -->
-    <!-- Curved shaft: left→ dips up → right -->
+    <!-- Arc 1 inner highlight (glossy tube sheen) -->
    <path
-        android:pathData="M28,74 Q54,58 80,72"
+        android:pathData="M 49.99,76.64 A 23,23 0 0,1 49.99,31.36"
        android:fillColor="#00000000"
-        android:strokeWidth="7"
+        android:strokeWidth="2.5"
        android:strokeLineCap="round"
-        android:strokeLineJoin="round">
-        <aapt:attr name="android:strokeColor">
-            <gradient android:type="linear"
-                android:startX="28" android:startY="72"
-                android:endX="80" android:endY="72"
-                android:startColor="#4DD0E1"
-                android:endColor="#00BCD4"/>
-        </aapt:attr>
-    </path>
-    <!-- Arrowhead at (80,72) pointing right/slightly-up -->
-    <path android:pathData="M76,64 L72,80 L88,74 Z">
-        <aapt:attr name="android:fillColor">
-            <gradient android:type="linear"
-                android:startX="72" android:startY="72"
-                android:endX="88" android:endY="72"
-                android:startColor="#00BCD4"
-                android:endColor="#00ACC1"/>
-        </aapt:attr>
-    </path>
+        android:strokeColor="#55FFC8B0"/>
+
+    <!-- Arc 2 inner highlight -->
+    <path
+        android:pathData="M 58.01,31.36 A 23,23 0 0,1 58.01,76.64"
+        android:fillColor="#00000000"
+        android:strokeWidth="2.5"
+        android:strokeLineCap="round"
+        android:strokeColor="#55B0C8FF"/>
+
+    <!-- Arrowhead at end of Arc 1 (260 deg, top-left area, pointing right-up) -->
+    <path android:pathData="M 42.13,32.74 L 49.31,27.41 L 40.73,24.86 Z"
+        android:fillColor="#E8A040"/>
+
+    <!-- Arrowhead at end of Arc 2 (80 deg, bottom-right area, pointing left-down) -->
+    <path android:pathData="M 67.27,83.14 L 58.69,80.59 L 65.87,75.26 Z"
+        android:fillColor="#7B5EA7"/>

 </vector>
@@ -3,4 +3,5 @@
    <external-path name="external_storage" path="." />
    <external-files-path name="external_files" path="." />
    <files-path name="internal_files" path="." />
+    <cache-path name="syncflow_cache" path="syncflow_open/" />
 </paths>
@@ -1,2 +1,2 @@
-VERSION_NAME=1.0.23
-VERSION_CODE=24
+VERSION_NAME=1.0.27
+VERSION_CODE=28
Author	SHA1	Message	Date
amir	dc2a0b2c68	v1.0.27: knot-inspired icon, fix media-not-found on photo open Icon: two thick tube-style arcs with 3D glossy highlights. Arc 1 (left side): coral #E8665A to orange #E8A040 Arc 2 (right side): steel blue #4A7FD4 to deep purple #7B5EA7 Arrowheads: orange and purple. Background: dark purple-black. Inspired by the braided knot color palette. Fix "media not found" when opening photos: - Intent now sets ClipData alongside FLAG_GRANT_READ_URI_PERMISSION so the permission correctly propagates through the system chooser to whichever app the user picks. - openFile() and downloadToCache() both call MediaScannerConnection so newly synced/downloaded files appear in gallery MediaStore index before the viewer launches. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-25 03:08:44 +00:00
amir	742f634084	v1.0.26: fix multi-selection reactivity, redesign icon, security review Fix multi-selection: selectedKeys exposed as StateFlow, collected in FilesScreen so checkboxes and highlights update correctly on every tap. fileKey() made public so UI can check membership without ViewModel calls. Icon: white cloud body with two cyan/teal circular sync arcs (AutoSync style), deep blue-to-teal gradient background. Security review clean: no hardcoded credentials, cleartext blocked by network_security_config, allowBackup=false, path traversal guards in place on both server responses and local resolution. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-25 02:45:43 +00:00
amir	8fdd22bc98	v1.0.25: multi-select files, unified notification, dark theme, icon redesign - FilesScreen: long-press enters selection mode, bulk share/delete toolbar, BackHandler - FilesViewModel: ShareMultiple action, isSelectionMode/selectedCount state, download-to-cache for open/share - FileWatchService: recursive FileObserver per subdirectory; unified notification updated with sync result via WorkManager flow observation - SyncWorker: silent flag suppresses notifications when triggered by watcher; emits KEY_RESULT_SUMMARY output data - Passbolt-inspired dark theme (Red700/Red500 primary, near-black surface) - App icon: circular AutoSync-style sync arrows (cyan gradient, deep navy background) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-25 02:22:43 +00:00
amir	146b8baf9a	v1.0.24: harmonious icon, recursive file watching, download-then-open, security fixes Icon: three identical parallel arcing arrows (same bezier curve, same blue-to-teal gradient #64C8FF→#32EDBB, same arrowhead geometry) — visually cohesive and clearly visible against the near-black background. FileWatchService: FileObserver is now recursive — watchDirRecursive() creates an observer for each subdirectory at startup, and adds new watchers when CREATE events produce new directories. Fixes files added to subdirectories not being detected. FilesViewModel: openFile/shareFile now fall back to download-then-open when the file is absent locally. AccountRepository + ProviderFactory injected; downloads to context.cacheDir/syncflow_open/ with isDownloading state. Path traversal guard added (reject relativePath containing ".."). file_paths.xml gains cache-path entry. WebDavProvider: path-traversal guard in parsePropfind — skip any server-returned filename containing "..", "/" or "\". Replace android.util.Log with Timber so debug logs are stripped from release builds. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-25 00:37:16 +00:00