Skip to content

Conversation

EricGustin
Copy link
Member

@EricGustin EricGustin commented Oct 10, 2025

Name, title, version, etc. for an MCPApp were being overwritten by its internal MCPServer.

⚠️ this is blocking arcade deploy from working

Copy link

vercel bot commented Oct 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
arcade-ai-arcade-mcp-python Ready Ready Preview Comment Oct 11, 2025 0:07am

Copy link

codecov bot commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 45.83333% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
libs/arcade-mcp-server/arcade_mcp_server/worker.py 0.00% 13 Missing ⚠️

📢 Thoughts on this report? Let us know!

@EricGustin EricGustin marked this pull request as ready for review October 11, 2025 00:15
@EricGustin EricGustin requested review from a team and sdreyer October 13, 2025 18:39
@EricGustin EricGustin merged commit 83c0eea into main Oct 13, 2025
21 of 22 checks passed
@EricGustin EricGustin deleted the ericgustin/fix-mcp-name-bug branch October 13, 2025 20:04
EricGustin added a commit that referenced this pull request Oct 16, 2025
Blocked by #614 (and the
reason for failing tests)

# PR Description
`arcade deploy` will deploy your local MCP server to Arcade. `arcade
deploy` should be executed at the root of your MCP Server package.
Before deploying, the command runs your server locally to ensure your
project is setup correctly and the server runs properly. `arcade deploy`
assumes your entrypoint file will execute `MCPApp.run` when the file is
invoked directly. This means you must either have an `if __name__ ==
"__main__" block that contains `MCPApp.run`, or `MCPApp.run` should be
top-level code (unindented living directly in the body of the file).

<img width="3318" height="594" alt="image"
src="https://github.com/user-attachments/assets/8249843e-6f9d-4d01-854d-356b0aae5055"
/>

<img width="1662" height="1056" alt="image"
src="https://github.com/user-attachments/assets/f44951f2-2718-4799-aecc-0e22c1b951b8"
/>
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.

2 participants