Skip to content

Conversation

@torresmateo
Copy link
Collaborator

@torresmateo torresmateo commented Oct 17, 2025

@vercel
Copy link

vercel bot commented Oct 17, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Oct 24, 2025 3:05am

@torresmateo torresmateo marked this pull request as ready for review October 17, 2025 20:47
@cursor
Copy link

cursor bot commented Oct 17, 2025

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on November 13.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@nearestnabors
Copy link
Contributor

I feel like this one buries the lede.

Who will be visiting this page? People who have a server running great (possibly in other clients), wandered away from the quickstarts, and they just want to set it up with (other) clients.

It's too short to be a tutorial—if it stays step by step, it should probably have a pre-reqs/you will learn block at the top:
image
But if you did that, the steps would be knocked down to 2!

It could probably make a really good guide if you focused just on this part:

image

And mention starting and running server with a link to a quickstart in the preamble.

Anyway, that's not blocking shipping!

@torresmateo
Copy link
Collaborator Author

@EricGustin I added a guide on how to use arcade configure to this page, which I think is now in much better shape (thanks for the suggesitons @nearestnabors!)

Changes from the latest review:

  • Uses the "Outcomes panel" insted of a 2 step "tutorial"
  • 2 sections. arcade configure and manually configuring MCP servers
  • defaults to stdio

Copy link
Member

@EricGustin EricGustin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks really good! A few comments that you may or may not want to address

Copy link
Contributor

@nearestnabors nearestnabors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there an http + Claude configuration? Oops!
Screenshot 2025-10-23 at 11 00 36 AM

torresmateo and others added 2 commits October 23, 2025 23:51
Co-authored-by: RL "Nearest" Nabors <236306+nearestnabors@users.noreply.github.com>
@torresmateo torresmateo merged commit ca5d698 into main Oct 24, 2025
4 checks passed
@torresmateo torresmateo deleted the mateo/local-clients branch October 24, 2025 14:41
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.

4 participants