Skip to content

docs(readme): rewrite to lead with user value and free-tier MCP#85

Merged
brainsparker merged 5 commits into
mainfrom
claude/readme-user-first-rewrite-I3bGz
May 26, 2026
Merged

docs(readme): rewrite to lead with user value and free-tier MCP#85
brainsparker merged 5 commits into
mainfrom
claude/readme-user-first-rewrite-I3bGz

Conversation

@brainsparker
Copy link
Copy Markdown
Contributor

Rationale

The previous README sold the repo to contributors (Bun setup, monorepo commands, quality checks) before it sold the toolkit to anyone evaluating whether to use You.com in their agent stack. Two strong distribution plays were buried: the free-tier remote MCP server at https://api.you.com/mcp?profile=free (no API key, no signup) and the official Anthropic MCP registry listing as io.github.youdotcom-oss/mcp. This rewrite restructures the README so the first screen is a copy-paste MCP config for the free tier, followed by per-client setup, packages, and three use-case snippets. Bun/monorepo content moves to links to AGENTS.md and CONTRIBUTING.md.

What changed

  • New hero: tagline, one-liner, free-tier MCP config block above the fold.
  • "Why You.com" — 5 one-line bullets including the registry listing.
  • Per-client quick start: Claude Desktop, Claude Code, Cursor, Windsurf, VS Code, and remote MCP — free-tier first, authenticated second.
  • Packages table with the four actual packages (mcp, ai-sdk-plugin, langchain, cli) and links to their READMEs.
  • Three short use-case snippets: MCP in Claude, Vercel AI SDK, LangChain.
  • Compact links section. Contributor section is one paragraph linking to AGENTS.md and CONTRIBUTING.md instead of inlining Bun commands.

Notes on accuracy

  • Verified package list against packages/ — only the four exist; no teams-anthropic, no api, no plugins/ dir.
  • Free-tier scope (you-search only) matches packages/mcp/README.md and packages/mcp/src/stdio-bridge.ts.
  • Registry name matches packages/mcp/server.json and is published from .github/workflows/publish-mcp.yml.

Opened as draft for a review pass before marking ready.


Generated by Claude Code

Restructures the root README so the first screen targets developers
evaluating You.com for their agent rather than contributors. Leads with
a copy-paste MCP config for the free tier, then per-client setup,
packages, and three use-case snippets. Bun/monorepo content moves to
links to AGENTS.md and CONTRIBUTING.md.
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 25, 2026

Droid encountered an error —— View job


Droid is reviewing code and running a security check…

@brainsparker brainsparker marked this pull request as ready for review May 25, 2026 13:06
@brainsparker brainsparker requested a review from a team as a code owner May 25, 2026 13:06
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 25, 2026

Droid finished @brainsparker's task —— View job


Droid review complete.

Summary: The README rewrite successfully restructures the document from contributor-focused to user-focused, with accurate package references and consistent code snippets verified against source. One bug: both claude mcp add commands place the -e flag after the -- separator, which passes it as an argument to npx instead of setting the environment variable in the MCP config — the free-tier quick start will not work as written.

Comment thread README.md Outdated
Comment on lines +67 to +70
claude mcp add you -- npx @youdotcom-oss/mcp -e YDC_PROFILE=free

# Authenticated
claude mcp add you -- npx @youdotcom-oss/mcp -e YDC_API_KEY=<your-key>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[P1] claude mcp add -e flag placed after -- separator

The -e flag must come before the server name, not after the -- separator. After --, it becomes an argument to npx rather than an environment variable setting for claude mcp add. The official CLI help shows: claude mcp add -e API_KEY=xxx my-server -- npx my-mcp-server. Users who copy these commands will register the MCP server without the YDC_PROFILE or YDC_API_KEY environment variables, so the free-tier command will fail (no key, no profile) and the authenticated command will run without credentials.

Suggested change
claude mcp add you -- npx @youdotcom-oss/mcp -e YDC_PROFILE=free
# Authenticated
claude mcp add you -- npx @youdotcom-oss/mcp -e YDC_API_KEY=<your-key>
claude mcp add -e YDC_PROFILE=free you -- npx @youdotcom-oss/mcp
# Authenticated
claude mcp add -e YDC_API_KEY=<your-key> you -- npx @youdotcom-oss/mcp

The -e flag must precede the -- separator; otherwise it is forwarded to
npx as an argument instead of stored as an env var on the MCP server.
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 25, 2026

Droid finished @brainsparker's task —— View job


Droid review complete.

Summary: The README rewrite is clean and accurate. The previous P1 (-e flag after --) was fixed in ea0b11f. All code snippets verified against package source: createYouClient APIs, langchain exports, and ai SDK exports are correct. MCP client config formats match platform conventions. No high-confidence bugs remain.

Copy link
Copy Markdown
Contributor

@factory-droid factory-droid Bot left a comment

Choose a reason for hiding this comment

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

✅ No actionable issues found.

@brainsparker brainsparker requested a review from EdwardIrby May 25, 2026 13:27
Comment thread README.md Outdated

- **Real web index** — backed by You.com's production search infrastructure, not a scraper.
- **Free tier, no signup** — `?profile=free` exposes `you-search` to any MCP client with zero auth.
- **Listed in the official Anthropic MCP registry** as [`io.github.youdotcom-oss/mcp`](https://github.com/modelcontextprotocol/registry).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good call — switched to the filtered registry listing in a057b1a. Also dropped "Anthropic" from the description since the registry is run by the MCP project itself.

Copy link
Copy Markdown
Contributor

@tyler5673 tyler5673 left a comment

Choose a reason for hiding this comment

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

looks good! one comment

…l.io

Tyler's PR feedback: the registry.modelcontextprotocol.io site is the
actual registry surface; the github.com/modelcontextprotocol/registry
repo is just the server source. Link directly to the filtered listing
for io.github.youdotcom-oss/mcp.

Also drop "Anthropic" from the description — the MCP registry is run by
the MCP project, not Anthropic specifically.
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 26, 2026

Droid encountered an error —— View job


Droid is reviewing code and running a security check…

Mirror the fix from a057b1a: point the Links-section "MCP registry
listing" entry at the filtered registry.modelcontextprotocol.io view
instead of the registry server repo, so both references to the listing
resolve to the same place.
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 26, 2026

Droid finished @brainsparker's task —— View job


Droid review complete.

Summary: One factual inconsistency found: the Quick Start section claims you-finance is unlocked by setting YDC_API_KEY, but the Packages section and source code confirm it requires an explicit ?tools=you-finance parameter or YDC_ALLOWED_TOOLS env var. Otherwise the README rewrite is accurate and well-structured.

Comment thread README.md Outdated
- `ydc tools`
- `ydc schema <tool> [input|output]`
- `ydc <tool> <json>`
Every snippet below works against the free tier. To unlock `you-research`, `you-contents`, and `you-finance`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[P1] Quick start overpromises you-finance as unlocked by API key

Line 31 says setting YDC_API_KEY unlocks you-research, you-contents, and you-finance, but line 145 states that you-finance "is available on request via ?tools=you-finance" and the authenticated default only exposes you-search, you-research, and you-contents. A user who sets only YDC_API_KEY will not see you-finance listed, contradicting the quick-start promise. The quick-start sentence should match the packages-section description.

Suggested change
Every snippet below works against the free tier. To unlock `you-research`, `you-contents`, and `you-finance`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys).
Every snippet below works against the free tier. To unlock `you-research` and `you-contents`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys). For `you-finance`, pass `?tools=you-finance` or set `YDC_ALLOWED_TOOLS=you-finance`.

The quick-start blurb listed you-finance as unlocked by YDC_API_KEY,
but the authenticated default only exposes you-search, you-research,
and you-contents. you-finance requires an explicit ?tools=you-finance
query param or YDC_ALLOWED_TOOLS=you-finance, matching the packages
section further down.
@factory-droid
Copy link
Copy Markdown
Contributor

factory-droid Bot commented May 26, 2026

Droid finished @brainsparker's task —— View job


Droid review complete.

Summary: One factual inaccuracy remains: the Quick start and Packages sections imply ?tools=you-finance adds you-finance to the default tool set, but ?tools= scopes (replaces) the visible tools — a user following the guidance would lose access to you-search, you-research, and you-contents. No security findings for this docs-only change.

Comment thread README.md
- `ydc tools`
- `ydc schema <tool> [input|output]`
- `ydc <tool> <json>`
Every snippet below works against the free tier. To unlock `you-research` and `you-contents`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys). `you-finance` is opt-in — pass `?tools=you-finance` on the URL or set `YDC_ALLOWED_TOOLS=you-finance`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[P1] ?tools=you-finance scopes to finance-only, losing default tools

The Quick start says "you-finance is opt-in — pass ?tools=you-finance on the URL or set YDC_ALLOWED_TOOLS=you-finance", implying these add you-finance to the defaults. In reality, ?tools= scopes (replaces) the visible tool set — a user who sets YDC_ALLOWED_TOOLS=you-finance will only see you-finance and lose you-search, you-research, and you-contents. The mcp package's own README confirms this with explicit examples: "Finance only: https://api.you.com/mcp?tools=you-finance" vs "Search plus finance: https://api.you.com/mcp?tools=you-search,you-finance". The same misconception appears at line 145. The guidance should show comma-separated lists including desired defaults.

Suggested change
Every snippet below works against the free tier. To unlock `you-research` and `you-contents`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys). `you-finance` is opt-in — pass `?tools=you-finance` on the URL or set `YDC_ALLOWED_TOOLS=you-finance`.
Every snippet below works against the free tier. To unlock `you-research` and `you-contents`, drop the `YDC_PROFILE` line and set `YDC_API_KEY` to a key from [you.com/platform/api-keys](https://you.com/platform/api-keys). `you-finance` is opt-in — `?tools=` scopes the visible tool set, so include every tool you need, e.g. `?tools=you-search,you-finance` or `YDC_ALLOWED_TOOLS=you-search,you-finance`.

@brainsparker brainsparker merged commit 16ae4f8 into main May 26, 2026
5 checks passed
@brainsparker brainsparker deleted the claude/readme-user-first-rewrite-I3bGz branch May 26, 2026 19:27
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.

3 participants