From 0f26bab8bce412354a1d2531784d4e5688daaa15 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 20 Mar 2026 10:16:19 +0000 Subject: [PATCH 1/2] Do not re-add bunker request on failed response send When sending the bunker response fails, only show the error message instead of re-adding the request back to the pending requests list. https://claude.ai/code/session_015N9LewondeT2cgh3Mv25cR --- .../com/greenart7c3/nostrsigner/service/BunkerRequestUtils.kt | 2 -- 1 file changed, 2 deletions(-) diff --git a/app/src/main/java/com/greenart7c3/nostrsigner/service/BunkerRequestUtils.kt b/app/src/main/java/com/greenart7c3/nostrsigner/service/BunkerRequestUtils.kt index da376d50..85dd709a 100644 --- a/app/src/main/java/com/greenart7c3/nostrsigner/service/BunkerRequestUtils.kt +++ b/app/src/main/java/com/greenart7c3/nostrsigner/service/BunkerRequestUtils.kt @@ -354,7 +354,6 @@ object BunkerRequestUtils { } } - val localBunkerRequest = bunkerRequest.copy() clearRequests() // assume that everything worked and try to revert it if it fails @@ -421,7 +420,6 @@ object BunkerRequestUtils { } onLoading(false) - addRequest(localBunkerRequest) } } }, From f7a56d882b490565978063dce3c031d7bb493955 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 20 Mar 2026 10:20:46 +0000 Subject: [PATCH 2/2] Do not re-add bunker requests on failed response in multi-event screen When sending a bunker response fails in BunkerMultiEventHomeScreen, only show the error message instead of re-adding the request back to the pending requests list. https://claude.ai/code/session_015N9LewondeT2cgh3Mv25cR --- .../components/BunkerMultiEventHomeScreen.kt | 30 ++++--------------- 1 file changed, 5 insertions(+), 25 deletions(-) diff --git a/app/src/main/java/com/greenart7c3/nostrsigner/ui/components/BunkerMultiEventHomeScreen.kt b/app/src/main/java/com/greenart7c3/nostrsigner/ui/components/BunkerMultiEventHomeScreen.kt index 0a79e4a5..f7cf3594 100644 --- a/app/src/main/java/com/greenart7c3/nostrsigner/ui/components/BunkerMultiEventHomeScreen.kt +++ b/app/src/main/java/com/greenart7c3/nostrsigner/ui/components/BunkerMultiEventHomeScreen.kt @@ -393,7 +393,6 @@ fun BunkerMultiEventHomeScreen( thisAccount.npub, ) - val localBunkerRequest = request.copy() BunkerRequestUtils.remove(request.request.id) if (request.checked.value) { @@ -404,11 +403,7 @@ fun BunkerMultiEventHomeScreen( BunkerResponse(request.request.id, localEvent.toJson(), null), application.application.relays, onLoading = {}, - onDone = { - if (!it) { - BunkerRequestUtils.addRequest(localBunkerRequest) - } - }, + onDone = {}, ) } else { AmberUtils.sendBunkerError( @@ -448,8 +443,7 @@ fun BunkerMultiEventHomeScreen( ) val signedMessage = thisAccount.signString(request.request.params.first()) - val localBunkerRequest = request.copy() - BunkerRequestUtils.remove(localBunkerRequest.request.id) + BunkerRequestUtils.remove(request.request.id) if (request.checked.value) { BunkerRequestUtils.sendBunkerResponse( @@ -459,11 +453,7 @@ fun BunkerMultiEventHomeScreen( BunkerResponse(request.request.id, signedMessage, null), application.application.relays, onLoading = {}, - onDone = { - if (!it) { - BunkerRequestUtils.addRequest(localBunkerRequest) - } - }, + onDone = {}, ) } else { AmberUtils.sendBunkerError( @@ -492,7 +482,6 @@ fun BunkerMultiEventHomeScreen( thisAccount.npub, ) - val localBunkerRequest = request.copy() BunkerRequestUtils.remove(request.request.id) if (request.checked.value) { BunkerRequestUtils.sendBunkerResponse( @@ -502,11 +491,7 @@ fun BunkerMultiEventHomeScreen( BunkerResponse(request.request.id, "", null), application.application.relays, onLoading = {}, - onDone = { - if (!it) { - BunkerRequestUtils.addRequest(localBunkerRequest) - } - }, + onDone = {}, ) } else { AmberUtils.sendBunkerError( @@ -553,7 +538,6 @@ fun BunkerMultiEventHomeScreen( ) val signature = request.encryptedData?.result ?: continue - val localBunkerRequest = request.copy() BunkerRequestUtils.remove(request.request.id) if (request.checked.value) { BunkerRequestUtils.sendBunkerResponse( @@ -563,11 +547,7 @@ fun BunkerMultiEventHomeScreen( BunkerResponse(request.request.id, signature, null), application.application.relays, onLoading = {}, - onDone = { - if (!it) { - BunkerRequestUtils.addRequest(localBunkerRequest) - } - }, + onDone = {}, ) } else { AmberUtils.sendBunkerError(