bitcoindevkit / bitcoindevkit/BDKSwiftExampleWallet

Make transaction sending fully async so the UI only reports success after broadcast completes

Ouverte
#360 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
Langage dominant
Swift
Étoiles
31
Forks
16
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

## Summary

The current send flow can report success before the transaction has actually been broadcast.

`BDKClient.send` is exposed as a synchronous throwing API, but the live implementation immediately starts an internal `Task` and returns without waiting for the async broadcast to finish. Because of that, the UI continues as if the send succeeded even though the real network operation is still in progress.

## Why This Matters

This creates a user-facing correctness bug:

- The app can show a success state before the transaction is actually broadcast.
- Broadcast failures may be surfaced too late, or not in the right screen context.
- The user can be navigated away from the send screen even if the transaction was rejected.
- A wallet app should never optimistically confirm a send before the backend confirms success.

## Current Behavior

- `BuildTransactionViewModel.send()` calls `bdkClient.send(...)`.
- The live `BDKClient.send` implementation starts a background `Task` and returns immediately.
- `BuildTransactionViewModel` then posts `TransactionSent` right away.
- `BuildTransactionView` waits one second and assumes success if no error has appeared yet.

This means success is based on timing, not on the actual result of the broadcast.

## Expected Behavior

The send flow should only report success after signing and broadcasting have completed successfully.

## Proposed Fix

- Change the send API from synchronous to `async throws` across the stack.
- Make `BDKClient.send` await `BDKService.send(...)` directly instead of wrapping it in a detached `Task`.
- Update `BuildTransactionViewModel.send()` to be `async`.
- In `BuildTransactionView`, await the send call and only show the success UI after it returns successfully.
- Remove the one-second timer-based success heuristic.
- Post wallet update notifications only after the async send has actually succeeded.

## Suggested Acceptance Criteria

- A failed broadcast does not show the success checkmark.
- The user remains on the send screen when send fails.
- A successful broadcast shows success only after the async operation completes.
- No send-related state depends on fixed delays or race-prone background fire-and-forget tasks.

## References

- `BDKSwiftExampleWallet/Service/BDK Service/BDKService.swift`
- `BDKSwiftExampleWallet/View Model/Send/BuildTransactionViewModel.swift`
- `BDKSwiftExampleWallet/View/Send/BuildTransactionView.swift`

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Start with BDKSwiftExampleWallet/Service/BDK Service/BDKService.swift and trace the send flow through BuildTransactionViewModel.swift and BuildTransactionView.swift. Update the flow so signing and broadcasting complete before success is reported, remove the fixed-delay heuristic, and verify that failures stay on the send screen while successful broadcasts show success afterward.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
swift
Domaine
mobile
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.