Conversation
…#24) - Add VitePress with docs:dev/build/preview scripts - Create full documentation site: guide, security, advanced, cli, tools - Migrate all README sections to dedicated pages - Add GitHub Actions workflow for docs deployment to Pages - Add MCP one-click install badges (VS Code, VS Code Insiders) - Create CONTRIBUTING.md with dev setup, testing, and PR guidelines - Reduce README from 1125 to 129 lines (links to docs site) - Fix outdated tool count: 62 → 47 (post-CQRS consolidation) - Remove outdated per-tool listings (replaced by auto-generated TOOLS.md)
Test Coverage ReportOverall Coverage: 91.92%
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Remove duplicate author attribution, streamline README structure, and fix VS Code one-click install deeplinks to use correct URL-encoded JSON format.
There was a problem hiding this comment.
Pull request overview
Migrates the project documentation from a large README into a VitePress site, adds MCP one-click install badges, and updates CI workflows to deploy docs (and publish coverage into the docs site on releases).
Changes:
- Introduced a VitePress documentation site under
docs/with guides, security docs, advanced configuration, and a tool reference. - Added VitePress scripts/dependency and updated
.gitignorefor VitePress build artifacts. - Refactored GitHub Actions: new docs deploy workflow, and simplified coverage workflow to PR-only commenting.
Reviewed changes
Copilot reviewed 22 out of 24 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Adds VitePress (and transitive deps) lockfile entries. |
| package.json | Adds vitepress devDependency and docs:* scripts. |
| docs/.vitepress/config.mts | New VitePress config (nav/sidebar/search/edit links). |
| docs/index.md | New docs homepage with hero and MCP install badges. |
| docs/guide/index.md | Introduction and feature overview moved to docs. |
| docs/guide/quick-start.md | Quick start guide for configuring MCP clients. |
| docs/guide/installation/npm.md | npm/npx and yarn dlx installation docs. |
| docs/guide/installation/docker.md | Docker installation docs for stdio + HTTP modes. |
| docs/guide/installation/vscode.md | VS Code setup + one-click install instructions. |
| docs/guide/installation/codex.md | Codex configuration instructions. |
| docs/guide/configuration.md | Environment variable reference moved to docs. |
| docs/guide/auto-discovery.md | Auto-discovery documentation added. |
| docs/guide/transport.md | Transport mode selection docs added. |
| docs/security/oauth.md | OAuth setup + endpoints + troubleshooting docs added. |
| docs/security/read-only.md | Read-only mode behavior docs added. |
| docs/advanced/tls.md | TLS / reverse-proxy deployment docs added. |
| docs/advanced/customization.md | Action filtering/description override/schema mode docs added. |
| docs/tools/index.md | Tool reference overview updated to 47 tools. |
| docs/cli/list-tools.md | list-tools CLI usage and doc generation instructions added. |
| README.md | Reduced README with links to docs and install badges. |
| CONTRIBUTING.md | Added contribution guide covering setup/testing/standards. |
| .gitignore | Ignores VitePress dist/cache outputs. |
| .github/workflows/docs.yml | New docs build/deploy workflow; integrates coverage on releases only. |
| .github/workflows/coverage-pages.yml | Refactored to PR-only coverage run + PR comment. |
The docs deploy workflow uses `yarn install --immutable` which depends on yarn.lock. Include it in path filters so lockfile changes trigger a rebuild.
- Note manage_context exception in read-only mode docs - Add 'opt-in' label to disabled-by-default feature flags - Add section header explaining opt-in groups reduce tool count
- All feature flags default to true (code uses !== "false" pattern) - Add /register (Dynamic Client Registration) to OAuth endpoints - Set DOCS_BASE=/ in local VitePress scripts for correct local preview
- Add USE_RELEASES, USE_REFS, USE_MEMBERS, USE_SEARCH to all docs - Move gated tools out of "Core (Always Available)" into own sections - Fix Docker example to forward all USE_* flags - Fix npm example with all 15 feature flags
|
🎉 This PR is included in version 6.31.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
VitePress Documentation Site
docs/.vitepress/config.mts— Site config with nav, sidebar, searchdocs/index.md— Home page with hero, features, install badgesdocs/guide/— Introduction, Quick Start, Installation (npm, Docker, VS Code, Codex), Configuration, Auto-Discovery, Transportdocs/security/— OAuth Authentication, Read-Only Modedocs/advanced/— TLS/HTTPS, Customization (descriptions, action filtering, schema mode)docs/tools/— Tool Reference (47 tools, correct count)docs/cli/— list-tools CLI documentationCI/CD
.github/workflows/docs.yml— VitePress build + deploy on docs changes and releases; coverage integrated on release only.github/workflows/coverage-pages.yml— Refactored to PR-only (coverage comments)Other
CONTRIBUTING.md— Dev setup, testing, architecture, PR guidelinesREADME.md— Reduced to minimal with links to docs sitepackage.json— Addeddocs:dev,docs:build,docs:previewscripts.gitignore— VitePress dist/cacheTest plan
yarn lint— 0 errorsyarn test— 3478 tests passedyarn build— successfulyarn docs:build— VitePress builds in ~2sCloses #24