From d445a251f7307733ab0dbbbf6889dbb06a907f0e Mon Sep 17 00:00:00 2001 From: joshavant <830519+joshavant@users.noreply.github.com> Date: Mon, 20 Jul 2026 23:04:53 -0500 Subject: [PATCH] fix(android): remove retired voice release screenshot --- .../src/main/java/ai/openclaw/app/AndroidScreenshotMode.kt | 1 - .../test/java/ai/openclaw/app/AndroidScreenshotModeTest.kt | 5 ++--- scripts/android-screenshots.sh | 3 +-- test/scripts/android-screenshots.test.ts | 4 +++- 4 files changed, 6 insertions(+), 7 deletions(-) diff --git a/apps/android/app/src/main/java/ai/openclaw/app/AndroidScreenshotMode.kt b/apps/android/app/src/main/java/ai/openclaw/app/AndroidScreenshotMode.kt index 26a37cb5e4f4..2ee3987755ad 100644 --- a/apps/android/app/src/main/java/ai/openclaw/app/AndroidScreenshotMode.kt +++ b/apps/android/app/src/main/java/ai/openclaw/app/AndroidScreenshotMode.kt @@ -13,7 +13,6 @@ enum class AndroidScreenshotScene( ) { Home("home", HomeDestination.Connect), Chat("chat", HomeDestination.Chat), - Voice("voice", HomeDestination.Voice), Settings("settings", HomeDestination.Settings), Gateway("gateway", HomeDestination.Settings, SettingsRoute.Gateway), VoiceWake("voice-wake", HomeDestination.Settings, SettingsRoute.Voice), diff --git a/apps/android/app/src/test/java/ai/openclaw/app/AndroidScreenshotModeTest.kt b/apps/android/app/src/test/java/ai/openclaw/app/AndroidScreenshotModeTest.kt index 97e91d30e7b0..66fcb4aa348c 100644 --- a/apps/android/app/src/test/java/ai/openclaw/app/AndroidScreenshotModeTest.kt +++ b/apps/android/app/src/test/java/ai/openclaw/app/AndroidScreenshotModeTest.kt @@ -23,10 +23,10 @@ class AndroidScreenshotModeTest { parseAndroidScreenshotModeIntent( Intent(Intent.ACTION_MAIN) .putExtra(extraAndroidScreenshotMode, true) - .putExtra(extraAndroidScreenshotScene, "voice"), + .putExtra(extraAndroidScreenshotScene, "chat"), ) - assertEquals(AndroidScreenshotScene.Voice, parsed) + assertEquals(AndroidScreenshotScene.Chat, parsed) } @Test @@ -45,7 +45,6 @@ class AndroidScreenshotModeTest { fun mapsScenesToProductionShellDestinations() { assertEquals(HomeDestination.Connect, AndroidScreenshotScene.Home.homeDestination) assertEquals(HomeDestination.Chat, AndroidScreenshotScene.Chat.homeDestination) - assertEquals(HomeDestination.Voice, AndroidScreenshotScene.Voice.homeDestination) assertEquals(HomeDestination.Settings, AndroidScreenshotScene.Settings.homeDestination) assertEquals(HomeDestination.Settings, AndroidScreenshotScene.VoiceWake.homeDestination) } diff --git a/scripts/android-screenshots.sh b/scripts/android-screenshots.sh index 3afb8410fd10..34bec109405d 100644 --- a/scripts/android-screenshots.sh +++ b/scripts/android-screenshots.sh @@ -37,7 +37,7 @@ KEEP_EMULATOR="${ANDROID_SCREENSHOT_KEEP_EMULATOR:-0}" SKIP_BUILD=0 SKIP_INSTALL=0 DRY_RUN=0 -SCENES=(home chat voice settings gateway voice-wake) +SCENES=(home chat settings gateway voice-wake) EMULATOR_PID="" EMULATOR_LOG="" STARTED_EMULATOR=0 @@ -484,7 +484,6 @@ scene_ready_text() { # The screenshot fixture seeds chat history and restores at the latest user # turn, so wait for that visible anchor instead of empty-chat copy. chat) printf '%s\n' "Draft a short status update for the team." ;; - voice) printf '%s\n' "Ready to talk" ;; settings) printf '%s\n' "OpenClaw mobile" ;; voice-wake) printf '%s\n' "Wake listener" ;; # Connected fixtures can push Add Gateway below the composed viewport, so diff --git a/test/scripts/android-screenshots.test.ts b/test/scripts/android-screenshots.test.ts index 2c7452f6fc59..be2f0d7ed071 100644 --- a/test/scripts/android-screenshots.test.ts +++ b/test/scripts/android-screenshots.test.ts @@ -25,7 +25,7 @@ describe("android screenshots script", () => { expect(result.stdout).toContain("Android screenshot size: 1440x2560"); expect(result.stdout).toContain("Screenshot AVD: OpenClaw_Screenshots_API36"); expect(result.stdout).toContain("Screenshot device profile: pixel_2"); - expect(result.stdout).toContain("Scenes: home chat voice settings gateway"); + expect(result.stdout).toContain("Scenes: home chat settings gateway voice-wake"); expect(result.stdout).not.toContain("connect chat voice screen settings"); expect(result.stdout).toContain("Dry run complete."); }); @@ -46,6 +46,8 @@ describe("android screenshots script", () => { expect(script).toContain("chat) printf '%s\\n' \"Draft a short status update for the team.\""); expect(script).not.toContain("chat) printf '%s\\n' \"Ready when you are\""); + expect(script).not.toContain("voice) printf"); + expect(script).not.toContain("Ready to talk"); expect(fixture).toContain('"Draft a short status update for the team."'); expect(script).toContain("settings) printf '%s\\n' \"OpenClaw mobile\""); expect(script).not.toContain("settings) printf '%s\\n' \"Settings\"");