fix negative relative mouse moves

[rgarcia]

fixes this error:

Internal_error: {"message":"failed during drag movement: mousemove_relative: unrecognized option '-60'\nUsage:
  mousemove_relative [options] \u003cx\u003e \u003cy\u003e\n-c, --clearmodifiers - reset active modifiers (alt, etc)
  while typing\n-p, --polar - Use polar coordinates. X as an angle, Y as distance\n--sync - only exit once the mouse
  has moved\n\nUsing polar coordinate mode makes 'x' the angle (in degrees) and\n'y' the distance.\n\nIf you want to
  use negative numbers for a coordinate, you'll need to\ninvoke it this way (with the '--'):\n mousemove_relative -- -
  20 -15\notherwise, normal usage looks like this:\n mousemove_relative 100 140\n"} .

tested manually: https://screen.studio/share/lM76bBHd

---

Note

Ensure negative relative mouse moves use the "--" separator when building xdotool drag steps to avoid option parsing errors.

^{Written by Cursor Bugbot for commit 8d5ee87. This will update automatically on new commits. Configure here.}

[mesa-dot-dev]

Mesa Description

Problem

The mousemove_relative command fails when using negative coordinates, as they are incorrectly interpreted as command-line options instead of positional arguments. This causes an "unrecognized option" error during relative mouse movements, such as dragging up or to the left.

fixes this error:

Internal_error: {"message":"failed during drag movement: mousemove_relative: unrecognized option '-60'\nUsage:
  mousemove_relative [options] <x> <y>\n...
If you want to
  use negative numbers for a coordinate, you'll need to\ninvoke it this way (with the '--'):\n mousemove_relative -- -
  20 -15\n...
"} .

Solution

The fix implements the recommended usage from the tool's help text by adding a -- separator before the x and y coordinates. This ensures that negative values are correctly parsed as arguments, resolving the error and allowing for proper relative mouse movement in all directions.

Validation

tested manually: https://screen.studio/share/lM76bBHd

^{Description generated by Mesa. Update settings}

[mesa-dot-dev]

Performed full review of 145ef64...8d5ee87

Analysis

1. Limited scope of testing - The PR fixes the negative coordinate issue but the analysis doesn't mention comprehensive testing across different platforms or edge cases.
2. Potential for breaking existing scripts or automation that may have worked around the original bug with custom handling.
3. The fix uses command-line argument conventions (--) which might not be universally supported across all environments where this software is deployed.

Tip

⚡ Quick Actions

This review was generated by Mesa.

Actions:

• Configure your agents

Slash Commands:

• /review - Request a full code review
• /review latest - Review only changes since the last review
• /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
• /help - Get help with Mesa commands and configuration options

^{1 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings}