Skip to content

docs: add optional HOL Guard protection for Cursor MCP calls - #389

Open
kantorcodes wants to merge 1 commit into
CursorTouch:mainfrom
kantorcodes:docs/hol-guard-protected-windows-mcp
Open

docs: add optional HOL Guard protection for Cursor MCP calls#389
kantorcodes wants to merge 1 commit into
CursorTouch:mainfrom
kantorcodes:docs/hol-guard-protected-windows-mcp

Conversation

@kantorcodes

Copy link
Copy Markdown

Description

Documents an optional HOL Guard pre-execution protection path for Windows-MCP when it is used from Cursor.

The SECURITY.md addition:

  • installs HOL Guard with pipx install hol-guard;
  • uses Windows-MCP's existing uvx windows-mcp serve configuration in .cursor/mcp.json;
  • enables Guard's Cursor integration with hol-guard install cursor and verifies it with hol-guard doctor cursor;
  • explains that a Guard deny on Cursor's beforeMCPExecution path blocks the MCP invocation before the corresponding Windows-MCP tool call executes;
  • explicitly limits the claim to supported Cursor agent-hook paths and keeps Windows-MCP's existing isolation/least-privilege guidance intact.

Motivation

Windows-MCP's security policy correctly emphasizes that its tools perform real OS actions and are not sandboxed. This adds a concrete, optional defense-in-depth setup for users who want local pre-execution policy on MCP calls without changing Windows-MCP itself.

Testing

Documentation-only change. The branch is based directly on current upstream main (c3b59aeaf7fce6ef1793fec936e2713a70249272) and differs by one file, +36/-0. The Windows-MCP command/config matches the current README, and the HOL Guard commands and beforeMCPExecution behavior were cross-checked against HOL Guard's current Cursor protection contract and setup guide.

No runtime code is changed.

@kantorcodes
kantorcodes force-pushed the docs/hol-guard-protected-windows-mcp branch from 8b78ecc to 3fc491a Compare August 27, 2026 13:09
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