Skip to content

Clarify non-disruptive operation rules#43

Open
matt-goldman wants to merge 1 commit intoRedth:mainfrom
matt-goldman:clarify-non-disruptive-operations-rules
Open

Clarify non-disruptive operation rules#43
matt-goldman wants to merge 1 commit intoRedth:mainfrom
matt-goldman:clarify-non-disruptive-operations-rules

Conversation

@matt-goldman
Copy link
Copy Markdown

I was adding Maui Dev Flow to a repo and ported in the skill. During code review Copilot flagged this.

In the Skill file you have this (line 507:

## ⚠️ Non-Disruptive Operation

**CRITICAL:** Never run commands that steal focus, move windows, simulate mouse/keyboard input,
or otherwise disrupt the user's desktop. The user is likely working on the same computer.

**Never use:**
- `osascript` to focus/activate windows, click UI elements, or send keystrokes
- `screencapture` interactively (the MauiDevFlow screenshot command captures in-process instead)
- `xdotool` focus/activate/key commands that affect the active window
- Any command that moves the mouse cursor or simulates input at the OS level
- `open -a` to bring apps to the foreground (use `open` only to launch, not to focus)

**Instead:** All inspection and interaction goes through `maui-devflow` CLI commands, which
communicate with the in-app agent over HTTP — no foreground focus required. If you need
something that would require OS-level control (e.g., dismissing a system dialog outside the
app), **ask the user** to do it manually rather than attempting automation that would hijack
their input.

But the ios/macOS file specifically says to use osascript in the troubleshooting section (line 201 onward), and the Linux file specifically says to use xdotool (line 82 onward) for key simulation.

I've added clarifications based on my interpretation - hopefully this is useful! (and hopefully I've understood the intent correctly)

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.

1 participant