mobile-debug-mcp 0.24.0 → 0.24.1

package/dist/manage/android.js

@@ -6,6 +6,9 @@ import { execAdb, spawnAdb, getAndroidDeviceMetadata, getDeviceInfo, findApk } f
 import { execAdbWithDiagnostics } from '../utils/diagnostics.js';
 import { detectJavaHome } from '../utils/java.js';
 export class AndroidManage {
+    isTestOnlyInstallFailure(output) {
+        return typeof output === 'string' && output.includes('INSTALL_FAILED_TEST_ONLY');
+    }
     async build(projectPath, _variant) {
         void _variant;
         try {
@@ -93,6 +96,13 @@ export class AndroidManage {
                 if (res.code === 0) {
                     return { device: deviceInfo, installed: true, output: res.stdout };
                 }
+                const installOutput = `${res.stdout}\n${res.stderr}`.trim();
+                if (this.isTestOnlyInstallFailure(installOutput)) {
+                    const retryRes = await spawnAdb(['install', '-r', '-t', apkToInstall], deviceId);
+                    if (retryRes.code === 0) {
+                        return { device: deviceInfo, installed: true, output: retryRes.stdout };
+                    }
+                }
             }
             catch (e) {
                 console.debug('[android-run] adb install failed, attempting push+pm fallback:', e instanceof Error ? e.message : String(e));
@@ -100,12 +110,25 @@ export class AndroidManage {
             const basename = path.basename(apkToInstall);
             const remotePath = `/data/local/tmp/${basename}`;
             await execAdb(['push', apkToInstall, remotePath], deviceId);
-            const pmOut = await execAdb(['shell', 'pm', 'install', '-r', remotePath], deviceId);
+            let finalPmRes = await spawnAdb(['shell', 'pm', 'install', '-r', remotePath], deviceId);
             try {
-                await execAdb(['shell', 'rm', remotePath], deviceId);
+                if (finalPmRes.code === 0) {
+                    return { device: deviceInfo, installed: true, output: finalPmRes.stdout };
+                }
+                if (this.isTestOnlyInstallFailure(`${finalPmRes.stdout}\n${finalPmRes.stderr}`)) {
+                    finalPmRes = await spawnAdb(['shell', 'pm', 'install', '-r', '-t', remotePath], deviceId);
+                    if (finalPmRes.code === 0) {
+                        return { device: deviceInfo, installed: true, output: finalPmRes.stdout };
+                    }
+                }
+                throw new Error(finalPmRes.stderr || finalPmRes.stdout || 'pm install failed');
+            }
+            finally {
+                try {
+                    await execAdb(['shell', 'rm', remotePath], deviceId);
+                }
+                catch { }
             }
-            catch { }
-            return { device: deviceInfo, installed: true, output: pmOut };
         }
         catch (e) {
             // gather diagnostics for attempted adb operations

package/docs/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 All notable changes to the **Mobile Debug MCP** project will be documented in this file.
 
+## [0.24.1]
+- Fixed Android install issue
+
 ## [0.24.0]
 - Improved execution loop
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mobile-debug-mcp",
-  "version": "0.24.0",
+  "version": "0.24.1",
   "description": "MCP server for mobile app debugging (Android + iOS), with focus on security and reliability",
   "type": "module",
   "bin": {

package/src/manage/android.ts

@@ -8,6 +8,10 @@ import { detectJavaHome } from '../utils/java.js'
 import { InstallAppResponse, StartAppResponse, TerminateAppResponse, RestartAppResponse, ResetAppDataResponse } from '../types.js'
 
 export class AndroidManage {
+  private isTestOnlyInstallFailure(output: string | undefined): boolean {
+    return typeof output === 'string' && output.includes('INSTALL_FAILED_TEST_ONLY')
+  }
+
   async build(projectPath: string, _variant?: string): Promise<{ artifactPath: string, output?: string } | { error: string }> {
     void _variant
     try {
@@ -92,6 +96,14 @@ export class AndroidManage {
         if (res.code === 0) {
           return { device: deviceInfo, installed: true, output: res.stdout }
         }
+
+        const installOutput = `${res.stdout}\n${res.stderr}`.trim()
+        if (this.isTestOnlyInstallFailure(installOutput)) {
+          const retryRes = await spawnAdb(['install', '-r', '-t', apkToInstall], deviceId)
+          if (retryRes.code === 0) {
+            return { device: deviceInfo, installed: true, output: retryRes.stdout }
+          }
+        }
       } catch (e) {
         console.debug('[android-run] adb install failed, attempting push+pm fallback:', e instanceof Error ? e.message : String(e))
       }
@@ -99,9 +111,21 @@ export class AndroidManage {
       const basename = path.basename(apkToInstall)
       const remotePath = `/data/local/tmp/${basename}`
       await execAdb(['push', apkToInstall, remotePath], deviceId)
-      const pmOut = await execAdb(['shell', 'pm', 'install', '-r', remotePath], deviceId)
-      try { await execAdb(['shell', 'rm', remotePath], deviceId) } catch {}
-      return { device: deviceInfo, installed: true, output: pmOut }
+      let finalPmRes = await spawnAdb(['shell', 'pm', 'install', '-r', remotePath], deviceId)
+      try {
+        if (finalPmRes.code === 0) {
+          return { device: deviceInfo, installed: true, output: finalPmRes.stdout }
+        }
+        if (this.isTestOnlyInstallFailure(`${finalPmRes.stdout}\n${finalPmRes.stderr}`)) {
+          finalPmRes = await spawnAdb(['shell', 'pm', 'install', '-r', '-t', remotePath], deviceId)
+          if (finalPmRes.code === 0) {
+            return { device: deviceInfo, installed: true, output: finalPmRes.stdout }
+          }
+        }
+        throw new Error(finalPmRes.stderr || finalPmRes.stdout || 'pm install failed')
+      } finally {
+        try { await execAdb(['shell', 'rm', remotePath], deviceId) } catch {}
+      }
     } catch (e) {
       // gather diagnostics for attempted adb operations
       const basename = path.basename(apkToInstall)

package/test/unit/manage/install.test.ts

@@ -69,8 +69,101 @@ exit 0
       assert.ok(res2.output && typeof res2.output === 'string', 'Project dir install succeeded with output')
     }
 
+    const testOnlyAdbPath = path.join(binDir, 'adb-test-only')
+    const testOnlyAdbScript = `#!/bin/sh
+if [ "$1" = "-s" ]; then
+  shift 2
+fi
+
+if [ "$1" = "shell" ] && [ "$2" = "getprop" ]; then
+  case "$3" in
+    ro.build.version.release) echo '16' ;;
+    ro.product.model) echo 'sdk_gphone64_arm64' ;;
+    ro.kernel.qemu) echo '1' ;;
+  esac
+  exit 0
+fi
+
+if [ "$1" = "install" ]; then
+  if [ "$2" = "-r" ] && [ "$3" = "-t" ]; then
+    echo 'Performing Streamed Install'
+    echo 'Success'
+    exit 0
+  fi
+  echo 'Performing Streamed Install'
+  echo 'adb: failed to install test.apk: Failure [INSTALL_FAILED_TEST_ONLY: Failed to install test-only apk. Did you forget to add -t?]' 1>&2
+  exit 1
+fi
+
+echo 'Success'
+exit 0
+`
+    await fs.writeFile(testOnlyAdbPath, testOnlyAdbScript, { mode: 0o755 })
+    process.env.ADB_PATH = testOnlyAdbPath
+
+    const { dir: d2, file: testOnlyApk } = await makeTempFile('.apk')
+    const testOnlyRes = await ai.installApp(testOnlyApk, 'emulator-5554')
+    console.log('testOnlyRes', testOnlyRes)
+    assert.strictEqual(testOnlyRes.installed, true, 'Test-only APK should retry with -t and install successfully')
+
+    const cleanupLog = path.join(binDir, 'pm-cleanup.log')
+    const pmFallbackAdbPath = path.join(binDir, 'adb-pm-fallback')
+    const pmFallbackAdbScript = `#!/bin/sh
+if [ "$1" = "-s" ]; then
+  shift 2
+fi
+
+if [ "$1" = "shell" ] && [ "$2" = "getprop" ]; then
+  case "$3" in
+    ro.build.version.release) echo '16' ;;
+    ro.product.model) echo 'sdk_gphone64_arm64' ;;
+    ro.kernel.qemu) echo '1' ;;
+  esac
+  exit 0
+fi
+
+if [ "$1" = "install" ]; then
+  echo 'adb install failed' 1>&2
+  exit 1
+fi
+
+if [ "$1" = "push" ]; then
+  echo 'pushed'
+  exit 0
+fi
+
+if [ "$1" = "shell" ] && [ "$2" = "pm" ] && [ "$3" = "install" ]; then
+  if [ "$4" = "-r" ] && [ "$5" = "-t" ]; then
+    echo 'Failure [INSTALL_FAILED_VERSION_DOWNGRADE]'
+    exit 1
+  fi
+  echo 'Failure [INSTALL_FAILED_TEST_ONLY: Failed to install test-only apk. Did you forget to add -t?]'
+  exit 1
+fi
+
+if [ "$1" = "shell" ] && [ "$2" = "rm" ]; then
+  echo cleanup >> "${cleanupLog}"
+  exit 0
+fi
+
+echo 'unexpected args:' "$@" 1>&2
+exit 1
+`
+    await fs.writeFile(pmFallbackAdbPath, pmFallbackAdbScript, { mode: 0o755 })
+    process.env.ADB_PATH = pmFallbackAdbPath
+
+    const { dir: d3, file: pmFallbackApk } = await makeTempFile('.apk')
+    const pmFallbackRes = await ai.installApp(pmFallbackApk, 'emulator-5554')
+    console.log('pmFallbackRes', pmFallbackRes)
+    assert.strictEqual(pmFallbackRes.installed, false, 'Failed pm fallback should surface as install failure')
+    assert.match(pmFallbackRes.error || '', /INSTALL_FAILED_VERSION_DOWNGRADE/, 'Final pm retry failure should be reported')
+    const cleanupCount = (await fs.readFile(cleanupLog, 'utf8')).trim().split('\n').filter(Boolean).length
+    assert.strictEqual(cleanupCount, 1, 'pm fallback cleanup should run once')
+
     // cleanup
     await fs.rm(d1, { recursive: true, force: true }).catch(() => {})
+    await fs.rm(d2, { recursive: true, force: true }).catch(() => {})
+    await fs.rm(d3, { recursive: true, force: true }).catch(() => {})
     await fs.rm(dirGradle, { recursive: true, force: true }).catch(() => {})
 
     // restore PATH and ADB_PATH
@@ -87,4 +180,4 @@ exit 0
   }
 }
 
-run().catch((e) => { console.error(e); process.exit(1) })
+run().catch((e) => { console.error(e); process.exit(1) })