Skip to content

Conversation

@0237h
Copy link
Contributor

@0237h 0237h commented Nov 19, 2025

introduction.mdx

  • Adds the two types of MCP
  • Emphasizes REST easier to use for LLMs than SQL
  • Misc editing

claude.mdx

  • Adds REST MCP config
  • Updates config commands
  • Adds depreciation notice for --sse-url option
  • Improves troubleshooting section

cline.mdx
Same as Claude

cursor.mdx
Same as Claude

faq.mdx

  • Adds How do I choose between the REST-MCP and the SQL-MCP and which one should I use ?
  • Adds Will using the REST-MCP counts towards my Token API usage ?

- Adds the two types of MCP
- Misc editing
- Adds REST MCP config
- Updates config commands
- Adds depreciation notice for `--sse-url` option
- Improves troubleshooting section
- Adds `How do I choose between the REST-MCP and the SQL-MCP and which one should I use ?`
- Adds `Will using the REST-MCP counts towards my Token API usage ?`
@0237h
Copy link
Contributor Author

0237h commented Nov 19, 2025

Realized this one might be poorly formulated:

How do I choose between the REST-MCP and the SQL-MCP and which one should I use ?

I meant:

How do I toggle between the REST-MCP and the SQL-MCP and which one should I use ?

But maybe that's still not clear enough and can be separated into two. The idea is to ask about the configuration aspect of choosing REST/SQL (answer is, use the adequate URL) and to ask about the use cases for one or the other.

So maybe two questions:

Can I use both MCPs at the same time ?
No, you choose which MCP to use based on the remote URL, ...

and

Which MCP should I use ?
...

@matthewdarwin
Copy link

We should definately suggest to users to use the REST MCP as the SQL one is easy to get things wrong and generate inefffient queries. By default use that one unless there is some special need to use SQL.

In this PR we won't address pricing for SQL MCP, other than you need to be on PRO or ENTERPRISE plan. Pricing page can be updated in a future PR.

Emphasizes REST easier to use for LLMs than SQL
Adds new instructions
Adds new instructions
@0237h 0237h marked this pull request as ready for review November 20, 2025 14:25
@0237h 0237h requested a review from a team as a code owner November 20, 2025 14:25
Copy link
Contributor

@benface benface left a comment

Choose a reason for hiding this comment

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

Found some typos, but otherwise LGTM.

0237h and others added 7 commits November 20, 2025 15:30
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Co-authored-by: Benoît Rouleau <benoit.rouleau@icloud.com>
Copy link
Contributor

@benface benface left a comment

Choose a reason for hiding this comment

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

🙌

@benface benface merged commit 7db31d3 into main Nov 20, 2025
4 checks passed
@benface benface deleted the token-api-mcp-docs-refactor branch November 20, 2025 17:06
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.

4 participants