feat: add --duration to io longpress

cli/io.go

@@ -53,6 +53,8 @@ var ioTapCmd = &cobra.Command{
 	},
 }
 
+var longPressDuration int
+
 var ioLongPressCmd = &cobra.Command{
 	Use:   "longpress [x,y]",
 	Short: "Long press on a device screen at the given coordinates",
@@ -80,6 +82,7 @@ var ioLongPressCmd = &cobra.Command{
 			DeviceID: deviceId,
 			X:        x,
 			Y:        y,
+			Duration: longPressDuration,
 		}
 
 		response := commands.LongPressCommand(req)
@@ -186,6 +189,7 @@ func init() {
 	// io command flags
 	ioTapCmd.Flags().StringVar(&deviceId, "device", "", "ID of the device to tap on")
 	ioLongPressCmd.Flags().StringVar(&deviceId, "device", "", "ID of the device to long press on")
+	ioLongPressCmd.Flags().IntVar(&longPressDuration, "duration", 500, "duration of the long press in milliseconds")
 	ioButtonCmd.Flags().StringVar(&deviceId, "device", "", "ID of the device to press button on")
 	ioTextCmd.Flags().StringVar(&deviceId, "device", "", "ID of the device to send keys to")
 	ioSwipeCmd.Flags().StringVar(&deviceId, "device", "", "ID of the device to swipe on")

commands/input.go

@@ -20,6 +20,7 @@ type LongPressRequest struct {
 	DeviceID string `json:"deviceId"`
 	X        int    `json:"x"`
 	Y        int    `json:"y"`
+	Duration int    `json:"duration"`
 }
 
 // TextRequest represents the parameters for a text input command
@@ -91,13 +92,13 @@ func LongPressCommand(req LongPressRequest) *CommandResponse {
 		return NewErrorResponse(fmt.Errorf("failed to start agent on device %s: %v", targetDevice.ID(), err))
 	}
 
-	err = targetDevice.LongPress(req.X, req.Y)
+	err = targetDevice.LongPress(req.X, req.Y, req.Duration)
 	if err != nil {
 		return NewErrorResponse(fmt.Errorf("failed to long press on device %s: %v", targetDevice.ID(), err))
 	}
 
 	return NewSuccessResponse(map[string]interface{}{
-		"message": fmt.Sprintf("Long pressed on device %s at (%d,%d)", targetDevice.ID(), req.X, req.Y),
+		"message": fmt.Sprintf("Long pressed on device %s at (%d,%d) for %dms", targetDevice.ID(), req.X, req.Y, req.Duration),
 	})
 }
 

devices/android.go

@@ -140,7 +140,7 @@
 func (d *AndroidDevice) runAdbCommand(args ...string) ([]byte, error) {
 	deviceID := d.getAdbIdentifier()
 	cmdArgs := append([]string{"-s", deviceID}, args...)
 	cmd := exec.Command(getAdbPath(), cmdArgs...)
 	return cmd.CombinedOutput()
 }
 
@@ -319,8 +319,8 @@
 }
 
 // LongPress simulates a long press at (x, y) on the Android device.
-func (d *AndroidDevice) LongPress(x, y int) error {
-	_, err := d.runAdbCommand("shell", "input", "swipe", fmt.Sprintf("%d", x), fmt.Sprintf("%d", y), fmt.Sprintf("%d", x), fmt.Sprintf("%d", y), "500")
+func (d *AndroidDevice) LongPress(x, y, duration int) error {
+	_, err := d.runAdbCommand("shell", "input", "swipe", fmt.Sprintf("%d", x), fmt.Sprintf("%d", y), fmt.Sprintf("%d", x), fmt.Sprintf("%d", y), fmt.Sprintf("%d", duration))
 	if err != nil {
 		return err
 	}
@@ -411,7 +411,7 @@
 
 // getAVDName returns the AVD name for an emulator, or empty string if not an emulator
 func getAVDName(transportID string) string {
 	avdCmd := exec.Command(getAdbPath(), "-s", transportID, "shell", "getprop", "ro.boot.qemu.avd_name")
 	avdOutput, err := avdCmd.CombinedOutput()
 	if err == nil && len(avdOutput) > 0 {
 		avdName := strings.TrimSpace(string(avdOutput))
@@ -428,7 +428,7 @@
 	}
 
 	// fall back to product model
 	modelCmd := exec.Command(getAdbPath(), "-s", deviceID, "shell", "getprop", "ro.product.model")
 	modelOutput, err := modelCmd.CombinedOutput()
 	if err == nil && len(modelOutput) > 0 {
 		return strings.TrimSpace(string(modelOutput))
@@ -438,7 +438,7 @@
 }
 
 func getAndroidDeviceVersion(deviceID string) string {
 	versionCmd := exec.Command(getAdbPath(), "-s", deviceID, "shell", "getprop", "ro.build.version.release")
 	versionOutput, err := versionCmd.CombinedOutput()
 	if err == nil && len(versionOutput) > 0 {
 		return strings.TrimSpace(string(versionOutput))
@@ -449,7 +449,7 @@
 
 // GetAndroidDevices retrieves a list of connected Android devices
 func GetAndroidDevices() ([]ControllableDevice, error) {
 	command := exec.Command(getAdbPath(), "devices")
 	output, err := command.CombinedOutput()
 	if err != nil {
 		status := command.ProcessState.ExitCode()
@@ -533,7 +533,7 @@
 	defer cancel()
 
 	// launch emulator in background without context (so it persists after function returns)
 	cmd := exec.Command(getEmulatorPath(), "-netdelay", "none", "-netspeed", "full", "-avd", d.id, "-qt-hide-window")
 	err := cmd.Start()
 	if err != nil {
 		return fmt.Errorf("failed to start emulator: %w", err)
@@ -570,7 +570,7 @@
 
 // checkBootComplete checks if an emulator has finished booting
 func (d *AndroidDevice) checkBootComplete(deviceID string) (bool, error) {
 	cmd := exec.Command(getAdbPath(), "-s", deviceID, "shell", "getprop", "sys.boot_completed")
 	output, err := cmd.CombinedOutput()
 	if err != nil {
 		return false, err
@@ -818,7 +818,7 @@
 	utils.Verbose("Starting %s with app path: %s", serverClass, appPath)
 	cmdArgs := append([]string{"-s", d.getAdbIdentifier()}, "exec-out", fmt.Sprintf("CLASSPATH=%s", appPath), "app_process", "/system/bin", serverClass, "--quality", fmt.Sprintf("%d", config.Quality), "--scale", fmt.Sprintf("%.2f", config.Scale), "--fps", fmt.Sprintf("%d", config.FPS))
 	utils.Verbose("Running command: %s %s", getAdbPath(), strings.Join(cmdArgs, " "))
 	cmd := exec.Command(getAdbPath(), cmdArgs...)
 
 	stdout, err := cmd.StdoutPipe()
 	if err != nil {

devices/common.go

@@ -51,7 +51,7 @@ type ControllableDevice interface {
 	Boot() error     // boot simulator/emulator
 	Shutdown() error // shutdown simulator/emulator
 	Tap(x, y int) error
-	LongPress(x, y int) error
+	LongPress(x, y, duration int) error
 	Swipe(x1, y1, x2, y2 int) error
 	Gesture(actions []wda.TapAction) error
 	StartAgent(config StartAgentConfig) error

devices/ios.go

@@ -139,8 +139,8 @@ func (d IOSDevice) Tap(x, y int) error {
 	return d.wdaClient.Tap(x, y)
 }
 
-func (d IOSDevice) LongPress(x, y int) error {
-	return d.wdaClient.LongPress(x, y)
+func (d IOSDevice) LongPress(x, y, duration int) error {
+	return d.wdaClient.LongPress(x, y, duration)
 }
 
 func (d IOSDevice) Swipe(x1, y1, x2, y2 int) error {

devices/simulator.go

@@ -111,7 +111,7 @@
 // runSimctl executes xcrun simctl with the provided arguments
 func runSimctl(args ...string) ([]byte, error) {
 	fullArgs := append([]string{"simctl"}, args...)
 	cmd := exec.Command("xcrun", fullArgs...)
 	output, err := cmd.CombinedOutput()
 	if err != nil {
 		return nil, fmt.Errorf("failed to execute xcrun simctl command: %w", err)
@@ -635,8 +635,8 @@
 	return s.wdaClient.Tap(x, y)
 }
 
-func (s SimulatorDevice) LongPress(x, y int) error {
-	return s.wdaClient.LongPress(x, y)
+func (s SimulatorDevice) LongPress(x, y, duration int) error {
+	return s.wdaClient.LongPress(x, y, duration)
 }
 
 func (s SimulatorDevice) Swipe(x1, y1, x2, y2 int) error {

devices/wda/longpress.go

@@ -4,7 +4,7 @@ import (
 	"fmt"
 )
 
-func (c *WdaClient) LongPress(x, y int) error {
+func (c *WdaClient) LongPress(x, y, duration int) error {
 
 	sessionId, err := c.GetOrCreateSession()
 	if err != nil {
@@ -22,7 +22,7 @@ func (c *WdaClient) LongPress(x, y int) error {
 				Actions: []TapAction{
 					{Type: "pointerMove", Duration: 0, X: x, Y: y},
 					{Type: "pointerDown", Button: 0},
-					{Type: "pause", Duration: 500},
+					{Type: "pause", Duration: duration},
 					{Type: "pointerUp", Button: 0},
 				},
 			},

server/server.go

@@ -278,6 +278,7 @@ type IoLongPressParams struct {
 	DeviceID string `json:"deviceId"`
 	X        int    `json:"x"`
 	Y        int    `json:"y"`
+	Duration int    `json:"duration"`
 }
 
 type IoSwipeParams struct {
@@ -322,10 +323,17 @@ func handleIoLongPress(params json.RawMessage) (interface{}, error) {
 		return nil, fmt.Errorf("invalid parameters: %v. Expected fields: deviceId, x, y", err)
 	}
 
+	// default duration to 500ms if not provided
+	duration := ioLongPressParams.Duration
+	if duration == 0 {
+		duration = 500
+	}
+
 	req := commands.LongPressRequest{
 		DeviceID: ioLongPressParams.DeviceID,
 		X:        ioLongPressParams.X,
 		Y:        ioLongPressParams.Y,
+		Duration: duration,
 	}
 
 	response := commands.LongPressCommand(req)