Skip to content

Comments

@W-20598954 MCP Vuepress docs#172

Open
patricksullivansf wants to merge 5 commits intomainfrom
psullivan.W-20598954.mcpVueDocs
Open

@W-20598954 MCP Vuepress docs#172
patricksullivansf wants to merge 5 commits intomainfrom
psullivan.W-20598954.mcpVueDocs

Conversation

@patricksullivansf
Copy link
Contributor

@patricksullivansf patricksullivansf commented Feb 19, 2026

Summary

Add MCP to Documentation site.

Screenshot 2026-02-20 at 9 08 42 AM Screenshot 2026-02-20 at 9 05 56 AM Screenshot 2026-02-20 at 9 06 21 AM Screenshot 2026-02-20 at 9 06 54 AM Screenshot 2026-02-20 at 9 07 33 AM

Testing

Pull PR and test doc site locally with pnpm run docs:dev

Dependencies

  • No net-new third-party dependencies were added
  • If net-new third-party dependencies were added, rationale/discussion is included and 3pl-approved is set by a maintainer

  • Tests pass (pnpm test)
  • Code is formatted (pnpm run format)

@clavery
Copy link
Collaborator

clavery commented Feb 19, 2026

MCP Server should come before API reference in the header (we can also remove the latter from the homepage, it doesn't need to take up that space).

When I talked to Yuming I was wondering if we should try to "merge" the MCP in with the guides docs because there is so much overlap and then have sidebar sections specific to the CLI and MCP. Although maybe the sidebar would grow too large at that point.

We should make sure we're referencing back to the Authentication guide for detailed setup guides in AM.

@patricksullivansf
Copy link
Contributor Author

patricksullivansf commented Feb 20, 2026

MCP Server should come before API reference in the header (we can also remove the latter from the homepage, it doesn't need to take up that space).

Done, but I left API ref in the header. Seems useful and isn't crowded on my browser. YMMV.

When I talked to Yuming I was wondering if we should try to "merge" the MCP in with the guides docs because there is so much overlap and then have sidebar sections specific to the CLI and MCP. Although maybe the sidebar would grow too large at that point.

I can see the draw, but I believe the use cases and setup steps are different enough that it might cause confusion. Happy to be wrong though...

Done. (sort of). the MCP is partially integrated into the Guide: it shares the same nav menu and quick start includes the MCP.

We should make sure we're referencing back to the Authentication guide for detailed setup guides in AM.

Done. Linked the various auth references back to the configuration, and from there to the Auth guide. Also removed as much redundancy as I thought reasonable to balance comprehension vs maintainability. I'm sure more could be done here.

@patricksullivansf patricksullivansf force-pushed the psullivan.W-20598954.mcpVueDocs branch from d201b54 to a0b4d31 Compare February 20, 2026 21:09
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