Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@enesgules
Copy link
Collaborator

@enesgules enesgules commented Oct 15, 2025

fixes ctx7-661

@linear
Copy link

linear bot commented Oct 15, 2025

"-y",
"@upstash/context7-mcp",
"--api-key",
"YOUR_API_KEY"
Copy link

Choose a reason for hiding this comment

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

I would recommend having the API key stored in a named environment variable and then instructing the user to set it in your README

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi @chrstnb, thanks for the review!
We already support an env variable called CONTEXT7_API_KEY
If a user has this in their env, would the mcp server be able to use it through the extension? If so, should I just remove this flag?

Copy link

Choose a reason for hiding this comment

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

I think the common pattern is to pass it in via the MCP definition (so like you're doing here but just with the CONTEXT7_API_KEY)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I couldn't understand exactly, would the following work?

"--api-key",
"${CONTEXT7_API_KEY}"

If not could you share the config that you meant?

Copy link

Choose a reason for hiding this comment

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

Yes it should! And then just include in the README that that env var needs to be set

@enesgules enesgules merged commit b9a0c6d into master Oct 17, 2025
2 checks passed
@enesgules enesgules deleted the feat/gemini-extenstion branch October 17, 2025 13: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