Update MCP server docs for directory submission#252
Conversation
Add Setup (multi-client), Authentication, Usage Examples, Troubleshooting, and Privacy & Support sections. Document 3 missing tools (upload_data, request_upload_url, balance) bringing total to 15. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Convert mcp-server.md to .mdx for HTML support. Add compressed setup walkthrough videos with poster thumbnails for Claude.ai/Desktop connector setup, upload domain whitelisting, and Claude Code HTTP MCP. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@claude code review |
|
Claude finished @RafaelPo's task in 3m 2s —— View job Code Review: Update MCP server docs for directory submission
Overall this is a solid documentation update that covers all the sections needed for the Anthropic MCP Directory submission. The MDX conversion works correctly with the existing Tool count mismatch with manifestThe docs state 15 tools, but
Meanwhile,
|
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add everyrow_browse_lists and everyrow_use_list tool documentation. Expand everyrow_forecast with required input columns and output fields. Update tool count from 15 to 17. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
| <div style={{display: 'grid', gridTemplateColumns: '1fr 1fr', gap: '1rem', marginTop: '1.5rem', marginBottom: '1.5rem'}}> | ||
| <div> | ||
| <p style={{fontWeight: 600, fontSize: '0.875rem', marginBottom: '0.5rem'}}>Adding the connector</p> |
There was a problem hiding this comment.
Bug: Video and poster paths in docs/mcp-server.mdx are incorrectly prefixed with /docs/. This will cause the assets to fail to load with a 404 error.
Severity: HIGH
Suggested Fix
Remove the /docs/ prefix from the src and poster attributes of the video elements in docs/mcp-server.mdx. For example, change src="/docs/videos/setup-claude-web.mp4" to src="/videos/setup-claude-web.mp4".
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: docs/mcp-server.mdx#L26-L28
Potential issue: The `docs/mcp-server.mdx` file references videos and poster images with
paths prefixed by `/docs/`, such as `src="/docs/videos/setup-claude-web.mp4"`. Because
the Next.js site is configured with `basePath: "/docs"`, static assets from the `public`
directory are served from the application root, not the base path. This means the
browser will attempt to load an incorrect URL, resulting in a 404 error for all videos
and poster images on the page, preventing users from viewing the instructional content.
Did we get this right? 👍 / 👎 to inform future reviews.
Summary
docs/mcp-serverwith all sections required by Anthropic's MCP Directory submission: Setup (multi-client), Authentication, Usage Examples, Troubleshooting, Privacy & Supporteveryrow_upload_data,everyrow_request_upload_url,everyrow_balance) — total now 15.mdxand add compressed setup walkthrough videos with poster thumbnails for Claude.ai/Desktop and Claude CodeTest plan
/docs/mcp-serveron the docs site🤖 Generated with Claude Code