docs: update mcp.md #1583
docs: update mcp.md #1583
Conversation
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
jirispilka
left a comment
jirispilka
left a comment
There was a problem hiding this comment.
I did a quick pass only. I'll have more comments once I read it properly.
It is really text heavy and does not look nice. We need to add separation and play with the content so that people won't run away.
We should add an image, whenever possible.
For example this:
We should also add a section about SSE
There is no information how you can connect over it.
Let's add a paragraph about mcp.apify.com too. It will be released soon.
|
The point about images is good, I concur. Or maybe even mermaid diagrams. |
MQ37
left a comment
MQ37
left a comment
There was a problem hiding this comment.
Agree with @jirispilka, this is too overwhelming. I think most users want to just find out how to connect and not read too much about what is Apify or MCP (there are other and better sources for this). Maybe we can keep this overwhelming version somewhere hidden, use something like HTML details>summary (https://www.w3schools.com/tags/tag_summary.asp) so tech users and LLM scraping the site can access detailed information.
Co-authored-by: Michał Olender <[email protected]>
- Deleted use cases - not relevant for docs - Simplified introduction - Deleted "Why use MCP with Apify" paragraph (not relevant) - Added section "How to use"
jirispilka
left a comment
jirispilka
left a comment
There was a problem hiding this comment.
I believe it’s still pretty long... it provides unnecessary boilerplate information while missing important setup details — like how to configure Claude, for example.
Fixed paragraph for long guidelines
- fixed info about Claude config
Deleted redundant paragraphs
indent
Kuba's suggestions for Quickstart section
|
Added your suggestions and made it more concise. @jirispilka not sure what you mean by "how to configure Claude". I would say it's pretty clear right now. I will create design task for the new image but I wouldn't wait for it, we could update it when it's done. So let's not block it. |
There was a problem hiding this comment.
Thank you for the changes 👍 I would just improve the Quickstart so TLDR people have all the info needed there (see #1583 (comment)) and then fix the deprecated arg and mention the CLI option.
jirispilka
left a comment
jirispilka
left a comment
There was a problem hiding this comment.
This is much better. Thanks! We are almost there, I just left a couple of comments.
Improved Quickstart section
Fixed enableAddingActors
Jirka feedback
indent
Fixed Actor Tooling section
Fix final issues with SSE
jirispilka
left a comment
jirispilka
left a comment
There was a problem hiding this comment.
Thanks, I did a quick pass and it looks good.
Let's go out with this and we can minor changes afterward. This conversation is already quite loong.
|
I'll give it a final pass from TW POV today |
TC-MO
left a comment
TC-MO
left a comment
There was a problem hiding this comment.
These should be last changes needed
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
New version of MCP docs
curlnonsense