Skip to content

feat: add take_screenshot tool using Godot native viewport capture#62

Open
tylerhaar7 wants to merge 1 commit into
Coding-Solo:mainfrom
tylerhaar7:main
Open

feat: add take_screenshot tool using Godot native viewport capture#62
tylerhaar7 wants to merge 1 commit into
Coding-Solo:mainfrom
tylerhaar7:main

Conversation

@tylerhaar7
Copy link
Copy Markdown

Add a new MCP tool that captures screenshots from running Godot projects using Godot's native viewport capture API instead of external screen capture libraries.

Features:

  • Uses file-based IPC to communicate with running Godot project
  • Cross-platform support (Windows, macOS, Linux)
  • Returns screenshot as base64 image or saves to specified path
  • Requires ScreenshotManager autoload in the Godot project

The implementation works by:

  1. Creating a trigger file in Godot's user:// directory
  2. ScreenshotManager autoload detects the file and captures viewport
  3. Screenshot is saved and read back by the MCP server

Add a new MCP tool that captures screenshots from running Godot projects
using Godot's native viewport capture API instead of external screen
capture libraries.

Features:
- Uses file-based IPC to communicate with running Godot project
- Cross-platform support (Windows, macOS, Linux)
- Returns screenshot as base64 image or saves to specified path
- Requires ScreenshotManager autoload in the Godot project

The implementation works by:
1. Creating a trigger file in Godot's user:// directory
2. ScreenshotManager autoload detects the file and captures viewport
3. Screenshot is saved and read back by the MCP server

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@sintetico82
Copy link
Copy Markdown

hello, can we merge this?

@tylerhaar7
Copy link
Copy Markdown
Author

hello, can we merge this?

Yeah

@akgrenier
Copy link
Copy Markdown

Thanks for the nudge. I checked the PR and it isn't mergeable right now because it has conflicts with current main (GitHub is showing it as conflicting/dirty). If you still want this merged, please rebase or merge the latest main into your branch and push an update; once the conflicts are resolved I can take another look.

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