Replace try catch

Author: ben-kaufman

app/src/main/java/to/bitkit/services/MigrationService.kt

@@ -1035,12 +1035,12 @@ class MigrationService @Inject constructor(
     suspend fun restoreFromRNRemoteBackup() {
         setRestoringFromRNRemoteBackup(true)
 
-        try {
+        runCatching {
             fetchRNRemoteLdkData()
             val bitkitFiles = rnBackupClient.listFiles(fileGroup = "bitkit")?.list ?: emptyList()
             retrieveAndApplyBitkitBackups(bitkitFiles)
             markMigrationCompleted()
-        } catch (e: Exception) {
+        }.onFailure { e ->
             Logger.error("RN remote backup restore failed", e, context = TAG)
             throw e
         }

app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt

@@ -390,7 +390,7 @@ class AppViewModel @Inject constructor(
         if (isCompletingMigration) return
         isCompletingMigration = true
 
-        try {
+        runCatching {
             lightningRepo.getPayments().onSuccess { payments ->
                 activityRepo.syncLdkNodePayments(payments)
             }.onFailure { e ->
@@ -406,10 +406,10 @@ class AppViewModel @Inject constructor(
                     Logger.warn("Sync failed during migration: $e", e, context = TAG)
                     finishMigrationWithFallbackSync()
                 }
-        } catch (e: Exception) {
+        }.onFailure { e ->
             Logger.error("Migration completion error: $e", e, context = TAG)
             finishMigrationWithError()
-        } finally {
+        }.also {
             isCompletingMigration = false
         }
     }

app/src/main/java/to/bitkit/viewmodels/WalletViewModel.kt

@@ -116,7 +116,7 @@ class WalletViewModel @Inject constructor(
 
             migrationService.setShowingMigrationLoading(true)
 
-            try {
+            runCatching {
                 migrationService.migrateFromReactNative()
                 walletRepo.setWalletExistsState()
                 walletExists = walletRepo.walletExists()
@@ -127,7 +127,7 @@ class WalletViewModel @Inject constructor(
                 } else {
                     migrationService.setShowingMigrationLoading(false)
                 }
-            } catch (e: Exception) {
+            }.onFailure { e ->
                 Logger.error("RN migration failed: $e", e, context = "WalletViewModel")
                 migrationService.markMigrationChecked()
                 migrationService.setShowingMigrationLoading(false)
@@ -183,13 +183,12 @@ class WalletViewModel @Inject constructor(
 
     private suspend fun restoreFromBackup() {
         _restoreState.update { RestoreState.InProgress.Metadata }
-        try {
+        runCatching {
             restoreFromMostRecentBackup()
-        } catch (e: Exception) {
+        }.onFailure { e ->
             Logger.error("Restore from backup failed", e, context = TAG)
-        } finally {
-            _restoreState.update { RestoreState.Completed }
         }
+        _restoreState.update { RestoreState.Completed }
     }
 
     private suspend fun restoreFromMostRecentBackup() {