refactor(mcp): centralize app loading timeout constants in shared package

Extract Constants:
- Move defaultAppLoadingTimeoutMs (10000) to @midscene/shared/mcp/types
- Move defaultAppLoadingCheckIntervalMs (2000) to @midscene/shared/mcp/types
- Remove duplicate constant definitions from android-mcp and ios-mcp

Type Safety:
- Add BaseAgent import to android-tools.ts
- Use double type assertion (as unknown as) for agent type conversion
- Fix spelling: temp-for-actionspace → temp-for-action-space

This follows DRY principle by maintaining constants in a single location
and improves type safety with proper imports and assertions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: quanru

packages/android-mcp/src/android-tools.ts

@@ -1,14 +1,16 @@
 import { type AndroidAgent, agentFromAdbDevice } from '@midscene/android';
 import { z } from '@midscene/core';
 import { getDebug } from '@midscene/shared/logger';
-import { BaseMidsceneTools, type ToolDefinition } from '@midscene/shared/mcp';
+import {
+  type BaseAgent,
+  BaseMidsceneTools,
+  type ToolDefinition,
+  defaultAppLoadingCheckIntervalMs,
+  defaultAppLoadingTimeoutMs,
+} from '@midscene/shared/mcp';
 
 const debug = getDebug('mcp:android-tools');
 
-// Default timeout for app loading verification
-const defaultAppLoadingTimeoutMs = 10000;
-const defaultAppLoadingCheckIntervalMs = 2000;
-
 /**
  * Android-specific tools manager
  * Extends BaseMidsceneTools to provide Android ADB device connection tools
@@ -19,28 +21,29 @@ export class AndroidMidsceneTools extends BaseMidsceneTools {
     const { AndroidDevice } = require('@midscene/android');
     // Create minimal temporary instance without connecting to device
     // The constructor doesn't establish ADB connection
-    return new AndroidDevice('temp-for-actionspace', {});
+    return new AndroidDevice('temp-for-action-space', {});
   }
 
   protected async ensureAgent(deviceId?: string): Promise<AndroidAgent> {
     if (this.agent && deviceId) {
       // If a specific deviceId is requested and we have an agent,
       // destroy it to create a new one with the new device
       try {
-        await this.agent.destroy();
+        await this.agent.destroy?.();
       } catch (error) {
         debug('Failed to destroy agent during cleanup:', error);
       }
       this.agent = undefined;
     }
 
     if (this.agent) {
-      return this.agent;
+      return this.agent as unknown as AndroidAgent;
     }
 
     debug('Creating Android agent with deviceId:', deviceId || 'auto-detect');
-    this.agent = await agentFromAdbDevice(deviceId);
-    return this.agent;
+    const agent = await agentFromAdbDevice(deviceId);
+    this.agent = agent as unknown as BaseAgent;
+    return agent;
   }
 
   /**

packages/ios-mcp/src/ios-tools.ts

@@ -1,14 +1,15 @@
 import { z } from '@midscene/core';
 import { type IOSAgent, agentFromWebDriverAgent } from '@midscene/ios';
 import { getDebug } from '@midscene/shared/logger';
-import { BaseMidsceneTools, type ToolDefinition } from '@midscene/shared/mcp';
+import {
+  BaseMidsceneTools,
+  type ToolDefinition,
+  defaultAppLoadingCheckIntervalMs,
+  defaultAppLoadingTimeoutMs,
+} from '@midscene/shared/mcp';
 
 const debug = getDebug('mcp:ios-tools');
 
-// Default timeout for app loading verification
-const defaultAppLoadingTimeoutMs = 10000;
-const defaultAppLoadingCheckIntervalMs = 2000;
-
 /**
  * iOS-specific tools manager
  * Extends BaseMidsceneTools to provide iOS WebDriverAgent connection tools

packages/shared/src/mcp/types.ts

@@ -4,6 +4,12 @@ import type { z } from 'zod';
 // Avoid circular dependency: don't import from @midscene/core
 // Instead, use generic types that will be provided by implementation
 
+/**
+ * Default timeout constants for app loading verification
+ */
+export const defaultAppLoadingTimeoutMs = 10000;
+export const defaultAppLoadingCheckIntervalMs = 2000;
+
 /**
  * Content item types for tool results (MCP compatible)
  */