-
Notifications
You must be signed in to change notification settings - Fork 50
Added server.json for official MCP registry #276
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
We should prefer the remote streamable HTTP version. |
Agree, lets do remote one. I will change that. |
@MichalKalita I also added the remote stremable http server, I guess we can also keep the local stdio one. |
{ | ||
"name": "Authorization", | ||
"description": "Apify API token for authentication with Apify platform services. For example 'Bearer <apify-api-token>'", | ||
"is_required": true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should define OAuth and not require an Apify token
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, but I dont know how to explicitly configure OAuth there - they only mention headers in the MD docs on how to contribute.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess the release.yaml is in some temporary state? Let's review it again when it is ready.
"url": "https://github.com/apify/apify-mcp-server", | ||
"source": "github" | ||
}, | ||
"version": "0.4.10", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will need to update this value too? Is there some simple way?
If not, let's push it now and we will automate it later
"registry_type": "npm", | ||
"registry_base_url": "https://registry.npmjs.org", | ||
"identifier": "@apify/actors-mcp-server", | ||
"version": "0.4.10", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, we need to update version (can be automated later)
the version bump for |
Co-authored-by: Jiří Spilka <[email protected]>
Added server.json and update CI pipeline.
Based on this tutorial: https://github.com/modelcontextprotocol/registry/blob/main/docs/guides/publishing/publish-server.md