Move all BLE logic in FlipperServiceApi (#83)

**Background**

Sometime we want execute command longer than application exist

**Changes**

- Refactor FlipperBleManager
- Move all logic to connect in FlipperServiceApi

.github/workflows/internal.yml

@@ -6,7 +6,7 @@ on:
       - 'dev'
 
 env:
-  VERSION: 0.2
+  VERSION: 0.3
 
 jobs:
   build_internal_release:

buildSrc/src/main/java/Dependencies.kt

@@ -63,6 +63,10 @@ object Libs {
         "androidx.lifecycle:lifecycle-viewmodel-ktx:${Versions.ANDROID_LIFECYCLE}"
     const val LIFECYCLE_RUNTIME_KTX =
         "androidx.lifecycle:lifecycle-runtime-ktx:${Versions.ANDROID_LIFECYCLE}"
+    const val LIFECYCLE_SERVICE =
+        "androidx.lifecycle:lifecycle-service:${Versions.ANDROID_LIFECYCLE}"
+    const val LIFECYCLE_KAPT =
+        "androidx.lifecycle:lifecycle-compiler:${Versions.ANDROID_LIFECYCLE}"
 
     const val ANNOTATIONS = "androidx.annotation:annotation:${Versions.ANDROID_ANNOTATIONS}"
     const val APPCOMPAT = "androidx.appcompat:appcompat:${Versions.ANDROIDX_APPCOMPAT}"

buildSrc/src/main/java/com/flipperdevices/gradle/AppExtensionConfiguration.kt

@@ -56,4 +56,4 @@ private fun AppExtension.configureBuildFeatures() {
 private fun AppExtension.configureCompileOptions() {
     compileOptions.sourceCompatibility = JavaVersion.VERSION_1_8
     compileOptions.targetCompatibility = JavaVersion.VERSION_1_8
-}
+}

components/bottombar/src/main/java/com/flipperdevices/bottombar/navigate/ScreenTabProviderImpl.kt

@@ -5,7 +5,6 @@ import com.flipperdevices.bottombar.model.FlipperBottomTab
 import com.flipperdevices.core.di.AppGraph
 import com.flipperdevices.core.navigation.screen.InfoScreenProvider
 import com.flipperdevices.filemanager.api.navigation.FileManagerScreenProvider
-import com.flipperdevices.pair.api.PairComponentApi
 import com.github.terrakok.cicerone.Screen
 import com.github.terrakok.cicerone.androidx.FragmentScreen
 import com.squareup.anvil.annotations.ContributesBinding
@@ -14,17 +13,12 @@ import javax.inject.Inject
 @ContributesBinding(AppGraph::class)
 class ScreenTabProviderImpl @Inject constructor(
     private val infoScreenProvider: InfoScreenProvider,
-    private val fileManagerScreenProvider: FileManagerScreenProvider,
-    private val pairComponentApi: PairComponentApi
+    private val fileManagerScreenProvider: FileManagerScreenProvider
 ) : ScreenTabProvider {
     override fun getScreen(tab: FlipperBottomTab): Screen {
         return when (tab) {
-            FlipperBottomTab.DEVICE -> infoScreenProvider.deviceInformationScreen(
-                pairComponentApi.getPairedDevice()
-            )
-            FlipperBottomTab.STORAGE -> fileManagerScreenProvider.fileManager(
-                pairComponentApi.getPairedDevice()
-            )
+            FlipperBottomTab.DEVICE -> infoScreenProvider.deviceInformationScreen()
+            FlipperBottomTab.STORAGE -> fileManagerScreenProvider.fileManager("/")
             else -> FragmentScreen { TestFragment() }
         }
     }

components/bridge/api/src/main/java/com/flipperdevices/bridge/api/di/FlipperBleComponentInterface.kt

@@ -1,9 +1,9 @@
 package com.flipperdevices.bridge.api.di
 
-import com.flipperdevices.bridge.api.pair.FlipperPairApi
+import android.bluetooth.BluetoothAdapter
 import com.flipperdevices.bridge.api.scanner.FlipperScanner
 
 interface FlipperBleComponentInterface {
     val flipperScanner: FlipperScanner
-    val flipperPairApi: FlipperPairApi
+    val bluetoothAdapter: BluetoothAdapter
 }

components/bridge/api/src/main/java/com/flipperdevices/bridge/api/manager/FlipperBleManager.kt

@@ -1,15 +1,33 @@
 package com.flipperdevices.bridge.api.manager
 
 import android.bluetooth.BluetoothDevice
-import com.flipperdevices.bridge.api.model.FlipperGATTInformation
-import kotlinx.coroutines.flow.StateFlow
-import no.nordicsemi.android.ble.ktx.state.ConnectionState
+import com.flipperdevices.bridge.api.manager.delegates.FlipperConnectionInformationApi
+import com.flipperdevices.bridge.api.manager.service.FlipperInformationApi
+import com.flipperdevices.bridge.api.manager.service.FlipperSerialApi
 
-interface FlipperBleManager : FlipperSerialApi {
-    val isDeviceConnected: Boolean
+interface FlipperBleManager {
+    // Manager delegates
+    val connectionInformationApi: FlipperConnectionInformationApi
+
+    // This section provide access to device apis
+    val informationApi: FlipperInformationApi
     val flipperRequestApi: FlipperRequestApi
-    fun getInformationStateFlow(): StateFlow<FlipperGATTInformation>
-    fun getConnectionStateFlow(): StateFlow<ConnectionState>
-    fun connectToDevice(device: BluetoothDevice)
-    fun disconnectDevice()
+    val serialApi: FlipperSerialApi
+
+    /**
+     * Connect to device {@param device}
+     * Await while disconnect process is not finish
+     */
+    suspend fun connectToDevice(device: BluetoothDevice)
+
+    /**
+     * Disconnect from current device
+     * Await while disconnect process is not finish
+     */
+    suspend fun disconnectDevice()
+
+    /**
+     * Close manager, unregister receivers
+     */
+    fun close()
 }

components/bridge/api/src/main/java/com/flipperdevices/bridge/api/manager/delegates/FlipperConnectionInformationApi.kt

@@ -0,0 +1,9 @@
+package com.flipperdevices.bridge.api.manager.delegates
+
+import kotlinx.coroutines.flow.StateFlow
+import no.nordicsemi.android.ble.ktx.state.ConnectionState
+
+interface FlipperConnectionInformationApi {
+    fun isDeviceConnected(): Boolean
+    fun getConnectionStateFlow(): StateFlow<ConnectionState>
+}

components/bridge/api/src/main/java/com/flipperdevices/bridge/api/manager/service/FlipperInformationApi.kt

@@ -0,0 +1,8 @@
+package com.flipperdevices.bridge.api.manager.service
+
+import com.flipperdevices.bridge.api.model.FlipperGATTInformation
+import kotlinx.coroutines.flow.StateFlow
+
+interface FlipperInformationApi {
+    fun getInformationFlow(): StateFlow<FlipperGATTInformation>
+}

components/bridge/api/src/main/java/com/flipperdevices/bridge/api/manager/FlipperSerialApi.kt → components/bridge/api/src/main/java/com/flipperdevices/bridge/api/manager/service/FlipperSerialApi.kt

@@ -1,4 +1,4 @@
-package com.flipperdevices.bridge.api.manager
+package com.flipperdevices.bridge.api.manager.service
 
 import kotlinx.coroutines.flow.Flow
 

components/bridge/impl/src/main/java/com/flipperdevices/bridge/impl/di/FlipperBleModule.kt

@@ -1,8 +1,6 @@
 package com.flipperdevices.bridge.impl.di
 
-import com.flipperdevices.bridge.api.pair.FlipperPairApi
 import com.flipperdevices.bridge.api.scanner.FlipperScanner
-import com.flipperdevices.bridge.impl.pair.FlipperPairApiImpl
 import com.flipperdevices.bridge.impl.scanner.FlipperScannerImpl
 import dagger.Binds
 import dagger.Module
@@ -13,8 +11,4 @@ abstract class FlipperBleModule {
     @Binds
     @Singleton
     abstract fun provideFlipperScanner(flipperScanner: FlipperScannerImpl): FlipperScanner
-
-    @Binds
-    @Singleton
-    abstract fun provideFlipperPairApi(flipperPairApi: FlipperPairApiImpl): FlipperPairApi
 }