Merge pull request #3242 from GCWing/wgq/mobile-account-signin

refactor(mobile): replace QR pairing with account sign-in

Author: wgqqqqq

scripts/check-harmonyos-architecture.mjs

@@ -127,12 +127,10 @@ const requiredPresentationFiles = [
   'components/AppRootOverlaySurfaces.ets',
   'components/ChatMessageChrome.ets',
   'components/RemoteSessionRow.ets',
-  'components/ConnectManualPairingOverlay.ets',
   'components/ConversationHeader.ets',
   'components/ConversationRouteSurface.ets',
   'components/ToolInteractionPanels.ets',
   'components/WideConversationHost.ets',
-  'components/platform/InlineQrScanner.ets',
   'components/remote/RemoteSurfaceHost.ets',
   'policy/ConversationHeaderPolicy.ets',
   'policy/ChatMessageStructurePolicy.ets',

src/apps/mobile/android/app/build.gradle.kts

@@ -77,11 +77,6 @@ dependencies {
     implementation(libs.androidx.activity.compose)
     implementation(libs.androidx.lifecycle.viewmodel.compose)
     implementation(libs.androidx.lifecycle.runtime.compose)
-    implementation("androidx.camera:camera-camera2:1.4.2")
-    implementation("androidx.camera:camera-lifecycle:1.4.2")
-    implementation("androidx.camera:camera-view:1.4.2")
-    // Bundle QR recognition so pairing also works without Play Services downloads.
-    implementation("com.google.mlkit:barcode-scanning:17.3.0")
     implementation(libs.androidx.window)
 
     testImplementation("junit:junit:4.13.2")

src/apps/mobile/android/app/src/androidTest/assets/pairing-qr.png

@@ -15,7 +15,6 @@ import androidx.compose.ui.test.onNodeWithText
 import androidx.compose.ui.test.performClick
 import com.openbitfun.mobile.app.ui.remote.CONNECT_ACCOUNT_DEVICE_REFRESH_TEST_TAG
 import com.openbitfun.mobile.app.ui.remote.CONNECT_ACCOUNT_DEVICE_ROW_TEST_TAG_PREFIX
-import com.openbitfun.mobile.app.ui.remote.CONNECT_ACCOUNT_DEVICE_SCAN_TEST_TAG
 import com.openbitfun.mobile.app.ui.remote.ConnectAccountDeviceScreen
 import com.openbitfun.mobile.app.ui.remote.AccountRemoteScreen
 import com.openbitfun.mobile.app.ui.theme.OpenBitFunTheme
@@ -109,10 +108,9 @@ class AccountRemoteScreenTest {
     }
 
     @Test
-    fun aSignedInAccountWithoutATargetCanRefreshSelectOrScan() {
+    fun aSignedInAccountWithoutATargetCanRefreshOrSelect() {
         var refreshes = 0
         var selected = ""
-        var scans = 0
         composeRule.setContent {
             OpenBitFunTheme(dark = false) {
                 ConnectAccountDeviceScreen(
@@ -129,7 +127,6 @@ class AccountRemoteScreenTest {
                     onBack = {},
                     onRefresh = { refreshes += 1 },
                     onSelect = { selected = it },
-                    onOpenScanner = { scans += 1 },
                     modifier = Modifier,
                 )
             }
@@ -139,11 +136,9 @@ class AccountRemoteScreenTest {
         composeRule.onNodeWithTag(CONNECT_ACCOUNT_DEVICE_REFRESH_TEST_TAG).performClick()
         composeRule.onNodeWithTag(CONNECT_ACCOUNT_DEVICE_ROW_TEST_TAG_PREFIX + "desk-1").performClick()
         composeRule.onNodeWithTag(CONNECT_ACCOUNT_DEVICE_ROW_TEST_TAG_PREFIX + "desk-2").performClick()
-        composeRule.onNodeWithTag(CONNECT_ACCOUNT_DEVICE_SCAN_TEST_TAG).performClick()
 
         assertEquals(1, refreshes)
         assertEquals("desk-1", selected)
-        assertEquals(1, scans)
     }
 
     @Test
@@ -165,7 +160,6 @@ class AccountRemoteScreenTest {
                     onBack = {},
                     onRefresh = {},
                     onSelect = { selected = it },
-                    onOpenScanner = {},
                     modifier = Modifier,
                 )
             }