Merge pull request #1023 from Hy4ri/fix/issue-1020-memory-reset

fix: memory screen reset actions serialization error (fixes #1020)

app/src/main/java/com/m57/hermescontrol/data/model/MemoryResponse.kt

@@ -13,3 +13,14 @@ data class BuiltinFileSizes(
     val memory: Long? = null,
     val user: Long? = null,
 )
+
+@Serializable
+data class MemoryResetRequest(
+    val target: String = "all",
+)
+
+@Serializable
+data class MemoryResetResponse(
+    val ok: Boolean = false,
+    val deleted: List<String> = emptyList(),
+)

app/src/main/java/com/m57/hermescontrol/data/remote/HermesApiService.kt

@@ -58,6 +58,8 @@ import com.m57.hermescontrol.data.model.MemoryProviderConfigUpdateRequest
 import com.m57.hermescontrol.data.model.MemoryProviderConfigUpdateResponse
 import com.m57.hermescontrol.data.model.MemoryProviderSetupRequest
 import com.m57.hermescontrol.data.model.MemoryProviderSetupResponse
+import com.m57.hermescontrol.data.model.MemoryResetRequest
+import com.m57.hermescontrol.data.model.MemoryResetResponse
 import com.m57.hermescontrol.data.model.MemoryResponse
 import com.m57.hermescontrol.data.model.MessagingPlatformResponse
 import com.m57.hermescontrol.data.model.MessagingPlatformTestResult
@@ -851,8 +853,8 @@ interface HermesApiService {
 
     @POST("api/memory/reset")
     suspend fun resetMemory(
-        @Body body: Map<String, String>,
+        @Body body: MemoryResetRequest,
-    ): Response<Map<String, Any>>
+    ): Response<MemoryResetResponse>
 
     // ── Admin: Credential pool ────────────────────────────────────────
     @GET("api/credentials/pool")

app/src/main/java/com/m57/hermescontrol/ui/memory/MemoryViewModel.kt

@@ -3,6 +3,7 @@ package com.m57.hermescontrol.ui.memory
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.m57.hermescontrol.data.model.LearningGraphResponse
+import com.m57.hermescontrol.data.model.MemoryResetRequest
 import com.m57.hermescontrol.data.model.MemoryResponse
 import com.m57.hermescontrol.data.remote.ApiClient
 import com.m57.hermescontrol.data.remote.NetworkResult
@@ -67,7 +68,7 @@ class MemoryViewModel : ViewModel(), ToastHost {
         if (_uiState.value.resetting != null) return
         _uiState.update { it.copy(resetting = target) }
         viewModelScope.launch {
-            val result = safeApiCall { ApiClient.hermesApi.resetMemory(mapOf("target" to target)) }
+            val result = safeApiCall { ApiClient.hermesApi.resetMemory(MemoryResetRequest(target = target)) }
             when (result) {
                 is NetworkResult.Success ->
                     _uiState.update {

app/src/test/java/com/m57/hermescontrol/ui/memory/MemoryViewModelTest.kt

@@ -1,6 +1,8 @@
 package com.m57.hermescontrol.ui.memory
 
 import com.m57.hermescontrol.data.model.MemoryProviderStatusRow
+import com.m57.hermescontrol.data.model.MemoryResetRequest
+import com.m57.hermescontrol.data.model.MemoryResetResponse
 import com.m57.hermescontrol.data.model.MemoryResponse
 import com.m57.hermescontrol.data.remote.ApiClient
 import com.m57.hermescontrol.data.remote.HermesApiService
@@ -103,15 +105,15 @@ class MemoryViewModelTest {
 
     @Test
     fun `resetMemory posts target and reloads`() {
-        coEvery { mockApi.resetMemory(mapOf("target" to "all")) } returns
+        coEvery { mockApi.resetMemory(MemoryResetRequest(target = "all")) } returns
-            Response.success(emptyMap<String, String>())
+            Response.success(MemoryResetResponse(ok = true))
         val vm = MemoryViewModel()
         vm.load()
         testDispatcher.scheduler.advanceUntilIdle()
         vm.resetMemory("all")
         testDispatcher.scheduler.advanceUntilIdle()
 
-        coVerify { mockApi.resetMemory(mapOf("target" to "all")) }
+        coVerify { mockApi.resetMemory(MemoryResetRequest(target = "all")) }
         assertEquals("Memory (all) reset successfully", vm.uiState.value.toastMessage)
         // One load + one reload after reset.
         coVerify(exactly = 2) { mockApi.getMemory() }
@@ -119,7 +121,7 @@ class MemoryViewModelTest {
 
     @Test
     fun `resetMemory failure shows error toast`() {
-        coEvery { mockApi.resetMemory(mapOf("target" to "memory")) } returns
+        coEvery { mockApi.resetMemory(MemoryResetRequest(target = "memory")) } returns
             Response.error(500, "boom".toResponseBody())
         val vm = MemoryViewModel()
         vm.resetMemory("memory")