Merge pull request #3020 from wgqqqqq/wgq/mobile-harmony-parity

feat(mobile): align native controller features with HarmonyOS

Author: wgqqqqq

src/apps/mobile/AGENTS.md

@@ -25,6 +25,7 @@ Native mobile applications are product entrypoints under `src/apps/mobile`.
 | `ios/` | iOS app, resources, lifecycle, and adapters |
 | `harmonyos/` | HarmonyOS app, resources, lifecycle, and adapters |
 | `shared/` | Kotlin Multiplatform core: protocol, crypto, transport, persistence, domain, feature stores |
+| `miniapps/` | Build-time packaging and isolated document bridge for product-owned offline tools; native storage and WebView lifecycle stay in each app |
 | `design-system/` | HarmonyOS-derived mobile tokens, component contracts, deterministic preview scenarios, and the desktop comparison surface |
 
 ## Native UI Contract
@@ -90,3 +91,13 @@ UiState or an Intent declared there, and no module above it is visible to them.
   from `shared/`, with an emulator or handset attached. Those suites compile the
   same `commonTest` sources onto ART; they are not in CI, so run them by hand
   when touching either module.
+
+## Offline Mini Apps
+
+Android preBuild and the iOS resource phase run `miniapps/generate.cjs` (Node.js
+must be on PATH). Generated native HTML is ignored; update product-owned built-in
+sources or the packaging owner, never the generated resources. Harmony retains
+its existing native host and invokes the same generator without native wrappers.
+Run `node --test src/apps/mobile/miniapps/*.test.cjs src/apps/mobile/harmonyos/miniapps/*.test.cjs`
+from the repository root after bundle/bridge changes. Storage keys remain scoped
+to an allowlisted app; retain unreadable records and surface the error.

src/apps/mobile/README.md

@@ -21,6 +21,12 @@ interfaces.
 All three apps can send images with or without text. Camera photos are decoded
 on the phone and converted to a supported format before upload. Failed sends
 retain the draft and images; acknowledgement removes only the submitted content.
+Android supports selecting several photos at once and retains prepared attachments
+across Activity recreation and process restarts. Android stores prepared image drafts
+in app-private files excluded from backups, scoped by account endpoint, account,
+device and session. Failed saves and unreadable records offer retry without erasing
+the stored draft. An in-progress photo conversion still needs to be retried if the
+process stops before preparation and saving finish.
 
 Model selection belongs to the connected host. A primary model that supports
 images receives their pixels directly. For a text-only primary model, select an
@@ -57,6 +63,10 @@ metadata, execution-mode IDs, speech draft merging, and completion observation i
 | Capability | Android and iOS behavior | Compatibility and verification boundary |
 |---|---|---|
 | Execution modes | Minimal / Standard / Ultimate, with the HarmonyOS density glyph; workspace menus also retain Cowork | Enabled only by live `get_workspace_info.capabilities` containing `harness_profiles_v1`; older hosts keep Code / Cowork. Capability absence in old payloads is covered by a round-trip test. |
+| Running-turn input | A nonempty remote draft offers Send; an empty composer retains Stop | Hosts advertising `dialog_steer_v1` receive `steer_turn` with the active turn ID. Older hosts retain `send_message` queueing; failed sends retain the draft and attachments. |
+| Plans | Legacy CreatePlan, structured plans and `.plan.md` writes show a plan card with file preview and Build | Build requires `plan_build_v1`, a completed plan tool and an idle connected session. Paths remain remote; the mobile app never runs the plan locally. |
+| Offline Mini Apps | Gomoku, Regex Playground and Daily Divination open from welcome and sidebar without login | Product-owned sources are bundled by `miniapps/generate.cjs`; native WebView hosts expose only per-app allowlisted storage and clipboard. No network, Node or shell bridge. |
+| Code preview | Native lexical colors, line numbers and referenced-line backgrounds | Both apps use the core-feature highlighter with native theme tokens; large files retain its bounded plain-text fallback. |
 | Account profile | Public GitHub login and avatar replace the numeric-ID placeholder | The immutable Relay user ID still authorizes devices. Display metadata uses a separate encrypted 24-hour cache; offline refreshes preserve credentials and cached display. |
 | Task completion | Notify for a previously observed successful remote turn while backgrounded | Identity includes target, session and turn. Replayed, failed and cancelled turns do not notify. These are local notifications, not server push. iOS observes within its OS background-task allowance; Android observes while the controller process remains alive. Neither promises notification after process termination. |
 | Speech input | Continue an existing draft without trimming its whitespace or inserting spaces into Chinese text | iOS owns Speech/AVAudio lifecycle in a platform adapter and cancels on route, target or scene changes. Android uses the system recognition activity. |
@@ -81,3 +91,10 @@ or opens system notification settings for an existing decision.
 Camera and microphone access stays contextual to scanning and voice input.
 Notification authorization does not extend the platform background-execution
 limits described above.
+
+## Connection recovery
+
+Native iOS and Android controllers probe idle session lists while the app is in the
+foreground. An open transcript uses its existing session poll for recovery instead
+of duplicating the health request. Temporary transport failures keep the displayed
+list or transcript; a successful response restores the connected state.

src/apps/mobile/android/AGENTS.md

@@ -49,6 +49,9 @@ JAVA_HOME='/Applications/Android Studio.app/Contents/jbr/Contents/Home' ./gradle
 # Verify negotiated execution modes and legacy fallback on an emulator.
 JAVA_HOME='/Applications/Android Studio.app/Contents/jbr/Contents/Home' ./gradlew :app:connectedDebugAndroidTest -Pandroid.testInstrumentationRunnerArguments.class=com.openbitfun.mobile.app.HarnessProfileMenuTest
 
+# Verify running-input, plan gating, offline tools, and attachment retention.
+./gradlew :app:connectedDebugAndroidTest -Pandroid.testInstrumentationRunnerArguments.class=com.openbitfun.mobile.app.MobileParityTest
+
 # Run shared JVM tests after core-feature changes.
 cd ../shared
 JAVA_HOME='/Applications/Android Studio.app/Contents/jbr/Contents/Home' ./gradlew jvmTest

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

@@ -95,3 +95,10 @@ dependencies {
     // release explicitly until the BOM updates its transitive constraint.
     androidTestImplementation(libs.androidx.test.espresso)
 }
+
+// Project product-owned offline tools into Android assets before packaging.
+val generateMiniApps by tasks.registering(Exec::class) {
+    workingDir(rootProject.file("../miniapps"))
+    commandLine("node", "generate.cjs", "android")
+}
+tasks.named("preBuild").configure { dependsOn(generateMiniApps) }

src/apps/mobile/android/app/src/androidTest/kotlin/com/openbitfun/mobile/app/AccountRemoteScreenTest.kt

@@ -68,7 +68,7 @@ class AccountRemoteScreenTest {
             }
         }
 
-        composeRule.onAllNodesWithText("Connect to a desktop").assertCountEquals(0)
+        composeRule.onAllNodesWithText(testString(R.string.pairing_title)).assertCountEquals(0)
     }
 
     @Test
@@ -98,7 +98,7 @@ class AccountRemoteScreenTest {
             }
         }
 
-        composeRule.onNodeWithText("Choose a desktop").assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.connect_account_devices_title)).assertIsDisplayed()
         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()

src/apps/mobile/android/app/src/androidTest/kotlin/com/openbitfun/mobile/app/ChatMessageBubbleTest.kt

@@ -55,7 +55,7 @@ class ChatMessageBubbleTest {
         }
 
         val first = composeRule.onNodeWithText("Checking the manifest.").getUnclippedBoundsInRoot()
-        val tool = composeRule.onNodeWithText("Running \"AndroidManifest.xml\"").getUnclippedBoundsInRoot()
+        val tool = composeRule.onNodeWithText(testString(R.string.tool_line_running, "AndroidManifest.xml")).getUnclippedBoundsInRoot()
         val second = composeRule.onNodeWithText("It targets API 35.").getUnclippedBoundsInRoot()
 
         assertTrue(first.top < tool.top)
@@ -175,8 +175,8 @@ class ChatMessageBubbleTest {
         }
 
         // The agent's reply started and stopped; it was never "not delivered".
-        composeRule.onNodeWithText("Reply interrupted.").assertIsDisplayed()
-        composeRule.onNodeWithText("Retry").assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.chat_reply_interrupted)).assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.account_devices_retry)).assertIsDisplayed()
     }
 
     @Test

src/apps/mobile/android/app/src/androidTest/kotlin/com/openbitfun/mobile/app/CreateSessionScreenTest.kt

@@ -69,7 +69,7 @@ class CreateSessionScreenTest {
             )
         }
 
-        composeRule.onNodeWithText("Chat").assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.navigation_general_chat)).assertIsDisplayed()
         composeRule.onNodeWithTag(COMPOSER_INPUT_TEST_TAG).performTextInput("review the parser")
         composeRule.onNodeWithTag(COMPOSER_SEND_TEST_TAG).performClick()
 
@@ -102,7 +102,7 @@ class CreateSessionScreenTest {
             )
         }
 
-        composeRule.onNodeWithText("No desktop is connected.").assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.create_no_device)).assertIsDisplayed()
         // The draft survives — it is worth keeping until a desktop comes back —
         // but the send is the same dimmed control a dropout produces.
         composeRule.onNodeWithTag(COMPOSER_INPUT_TEST_TAG).performTextInput("review the parser")
@@ -162,10 +162,10 @@ class CreateSessionScreenTest {
 
         composeRule.onNodeWithTag(CREATE_SESSION_WORKSPACE_TEST_TAG).performClick()
 
-        composeRule.onNodeWithText("Choose a workspace").assertIsDisplayed()
+        composeRule.onNodeWithText(testString(R.string.create_workspace_picker)).assertIsDisplayed()
         // A desktop with no recent projects is not a broken picker, and saying
         // where to fix it is the only useful thing the sheet can offer.
-        composeRule.onNodeWithText("No recent workspaces. Open one on the desktop first.")
+        composeRule.onNodeWithText(testString(R.string.create_no_workspaces))
             .assertIsDisplayed()
     }
 

src/apps/mobile/android/app/src/androidTest/kotlin/com/openbitfun/mobile/app/FileReferenceCardsTest.kt

@@ -118,7 +118,7 @@ class FileReferenceCardsTest {
             )
         }
 
-        composeRule.onNodeWithContentDescription("Download").assertIsDisplayed().performClick()
+        composeRule.onNodeWithContentDescription(testString(R.string.file_download)).assertIsDisplayed().performClick()
         composeRule.onNodeWithTag(FILE_DOWNLOAD_ACTION_TEST_TAG)
             .assertWidthIsEqualTo(44.dp)
             .assertHeightIsEqualTo(44.dp)

src/apps/mobile/android/app/src/androidTest/kotlin/com/openbitfun/mobile/app/MobileDesignPreviewVisualTest.kt

@@ -14,6 +14,7 @@ import androidx.compose.ui.graphics.toArgb
 import androidx.compose.ui.graphics.toPixelMap
 import androidx.compose.ui.platform.testTag
 import androidx.compose.ui.test.ExperimentalTestApi
+import androidx.compose.ui.test.assertIsEnabled
 import androidx.compose.ui.test.assertIsDisplayed
 import androidx.compose.ui.test.captureToImage
 import androidx.compose.ui.test.junit4.v2.createComposeRule
@@ -121,8 +122,8 @@ class MobileDesignPreviewVisualTest {
             composeRule.onNodeWithText(scenario.composerDraft).assertIsDisplayed()
         }
         if (scenario == MobilePreviewScenarios.StreamingDark) {
-            composeRule.onNodeWithContentDescription(targetContext.getString(R.string.message_stop))
-                .assertIsDisplayed()
+            composeRule.onNodeWithContentDescription(targetContext.getString(R.string.message_send))
+                .assertIsDisplayed().assertIsEnabled()
         }
     }