Skip to content

Conversation

@archandatta
Copy link
Contributor

@archandatta archandatta commented Feb 2, 2026

Note

Medium Risk
Introduces a new host-input API surface that shells out to xdotool; main risk is runtime fragility across environments and contention with other input operations despite mutex serialization.

Overview
Adds a new Computer API endpoint POST /computer/get_mouse_position that returns the current cursor coordinates as {x,y}.

Implements the handler by running xdotool getmouselocation --shell, parsing the output via a new parseMousePosition helper (with input locking and error reporting), and updates generated OAPI client/server types plus unit tests for the parser.

Written by Cursor Bugbot for commit a372758. This will update automatically on new commits. Configure here.

@archandatta archandatta marked this pull request as ready for review February 2, 2026 18:23
@archandatta archandatta requested review from Sayan- and rgarcia February 2, 2026 19:24
Copy link
Contributor

@Sayan- Sayan- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@archandatta archandatta force-pushed the archand/kernel-847/api-get-cursor-position branch from bc07f25 to a372758 Compare February 3, 2026 12:11
@archandatta archandatta merged commit 4198c2d into main Feb 3, 2026
5 checks passed
@archandatta archandatta deleted the archand/kernel-847/api-get-cursor-position branch February 3, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants