Conversation
Add a new skill for installing and managing the Figma MCP server, which provides tools to interact with Figma designs directly from OpenHands using OAuth authentication. Co-authored-by: openhands <openhands@all-hands.dev>
The URL was incorrectly pointing to https://figma.com/mcp/sse but the official Figma documentation specifies https://mcp.figma.com/mcp Reference: https://developers.figma.com/docs/figma-mcp-server/remote-server-installation/ Co-authored-by: openhands <openhands@all-hands.dev>
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🔴 Needs improvement: Critical inconsistency between installation commands and architectural issues with skill structure. See inline comments for details.
Co-authored-by: openhands <openhands@all-hands.dev>
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🔴 Needs improvement: Installation commands contradict each other and will break for users. The previous "fix" did not resolve this.
- Fix URL inconsistency: use official https://mcp.figma.com/mcp URL in both files - Update description to clarify read-only access - Apply progressive disclosure: slim SKILL.md to ~15 lines, move details to README.md - Remove content duplication between files - Add Usage and Troubleshooting sections to README.md Co-authored-by: openhands <openhands@all-hands.dev>
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🔴 Needs improvement: Same data consistency issue persists - installation commands don't match across the PR.
- Updated PR description to use correct installation command - Clarified in README.md that OAuth is handled automatically by the server at runtime, not via an installation flag - All installation commands now consistent across PR description, SKILL.md, and README.md Co-authored-by: openhands <openhands@all-hands.dev>
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟡 Acceptable - Structure is clean and follows conventions. Core functionality looks solid, but documentation could be more precise about what actually happens when using this skill.
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟡 Acceptable - Structure is clean and follows conventions. Core functionality looks solid, but documentation could be more precise about what actually happens when using this skill.
|
This should not be marked ready yet because it requires testing with oauth, which a human needs to do. |
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟡 Acceptable - Clean structure, follows conventions. Two minor clarifications would improve user experience.
- Clarify why restart is required (MCP servers initialized at session startup) - Fix OAuth troubleshooting guidance (browser session, not re-running mcp add) Co-authored-by: openhands <openhands@all-hands.dev>
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟡 Acceptable - Functional skill that follows repository conventions, but documentation has clarity gaps that will cause user confusion. The installation works, but users will struggle to understand the actual workflow.
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟡 Acceptable - Clean structure that solves a real problem. Documentation over-emphasizes code generation when the tool has broader capabilities. See inline comments for specific improvements.
Co-authored-by: openhands <openhands@all-hands.dev>
|
Follow-up context-layer investigation for this PR:
The first small slice I tackled is tightening the context-layer's Figma OAuth path by adding provider-specific I didn't open a context-layer PR here because the request was to investigate and start follow-up work, not to open one automatically. |
4 participants
Summary
Adds a Figma MCP skill plus companion README documentation for installing the remote Figma MCP server and using it from OpenHands.
Fixes #81
Details
SKILL.mdshort and decision-oriented for progressive disclosureREADME.mdwith concrete installation, usage, and troubleshooting guidancemarketplaces/default.jsonTesting
Documentation/skill change - no automated tests were added.
Validation completed:
Claude Code CompatibilityGitHub Action: ✅ passedTestsGitHub Action: ✅ passeduv sync --group test && uv run pytest -q tests/test_skills_have_readme.py tests/test_workflow_sync.py tests/test_sdk_loading.pyEvidence
Documentation and validation evidence:
Live Figma/OAuth evidence is still pending manual verification.
Full end-to-end proof still requires a Figma account plus a browser session that can complete the Figma OAuth flow, which is not available in this environment. The PR should stay in draft until that verification is collected.
Manual verification steps:
openhands mcp add figma --transport http https://mcp.figma.com/mcpGenerate a React component from this Figma frame: https://www.figma.com/file/ABC123/MyDesign?node-id=1-2Checklist