Skip to content

Update MCP server docs for directory submission#252

Merged
RafaelPo merged 5 commits intomainfrom
feat/mcp-docs-update
Feb 27, 2026
Merged

Update MCP server docs for directory submission#252
RafaelPo merged 5 commits intomainfrom
feat/mcp-docs-update

Conversation

@RafaelPo
Copy link
Contributor

Summary

  • Update docs/mcp-server with all sections required by Anthropic's MCP Directory submission: Setup (multi-client), Authentication, Usage Examples, Troubleshooting, Privacy & Support
  • Document 3 missing tools (everyrow_upload_data, everyrow_request_upload_url, everyrow_balance) — total now 15
  • Convert to .mdx and add compressed setup walkthrough videos with poster thumbnails for Claude.ai/Desktop and Claude Code

Test plan

  • Verify page renders at /docs/mcp-server on the docs site
  • Confirm all 15 tools are documented
  • Check videos play correctly and poster images display
  • Verify no broken links or missing sections

🤖 Generated with Claude Code

RafaelPo and others added 2 commits February 27, 2026 21:36
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>
@RafaelPo
Copy link
Contributor Author

@claude code review

@github-actions
Copy link
Contributor

github-actions bot commented Feb 27, 2026

Claude finished @RafaelPo's task in 3m 2s —— View job


Code Review: Update MCP server docs for directory submission

  • Read repo conventions and docs-site architecture
  • Read the main changed file (docs/mcp-server.mdx)
  • Check video/image assets and path conventions
  • Cross-reference documented tools against actual MCP tool implementations
  • Post review feedback

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 MDXRemote setup, and the video embeds are well-structured. A few items worth addressing:

Tool count mismatch with manifest

The docs state 15 tools, but everyrow-mcp/manifest.json lists 16 tools. Two tools present in the manifest and registered in the code are not documented:

  • everyrow_browse_lists — registered at tools.py:111
  • everyrow_use_list — registered at tools.py:174

Meanwhile, everyrow_request_upload_url is documented but absent from the manifest (it's dynamically registered only in HTTP mode via uploads.py:46). If the intent is to document only the remote MCP surface (which includes request_upload_url), the "15 tools" claim is accurate for the HTTP server but not for the stdio plugin. Consider either:

  • Documenting browse_lists/use_list and updating the count to 17, or
  • Adding a note that some tools are mode-specific

Fix this →

everyrow_forecast docs are sparse

The everyrow_forecast section (line 144-152) only documents the context parameter. The tool's actual docstring in tools.py:764-779 specifies important requirements:

"The CSV should contain at minimum a question column. Recommended additional columns: resolution_criteria, resolution_date, background."

And it documents output columns: rationale (str) and probability (int, 0-100). This is useful information for users and the directory listing.

Fix this →

Minor observations (non-blocking)

  • Video file sizes are reasonable — 297-375 KB for MP4s and 74-141 KB for poster JPGs. Good compression.
  • Asset paths are correct/docs/videos/... matches the basePath: "/docs" in next.config.ts with files in docs-site/public/videos/. Static export should serve these correctly.
  • LFS tracking — all 6 media files are properly tracked via Git LFS.
  • Inline JSX styles in the video grid (lines 26-39, 62-67) — works fine with MDXRemote, though extracting to a reusable component could be nice if more video sections are added in the future. Not needed now.
  • Privacy links point to futuresearch.ai while the product is everyrow.io — this appears intentional (parent company) but worth confirming for the directory submission that Anthropic won't flag the domain mismatch.

| Branch: feat/mcp-docs-update

RafaelPo and others added 2 commits February 27, 2026 22:29
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>
Comment on lines +26 to +28
<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>
Copy link

Choose a reason for hiding this comment

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

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.

@RafaelPo RafaelPo merged commit 4e4ac8b into main Feb 27, 2026
7 checks passed
@RafaelPo RafaelPo deleted the feat/mcp-docs-update branch February 27, 2026 22:52
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.

1 participant