Repository navigation
feat(auth): add JEV_API_KEY and JEVWISE_MCP_TOKEN credential names - #86
Merged
Merged
Conversation
Introduce canonical environment variable names for upstream API access and local MCP agent authentication, keeping the previous names as ordered fallbacks so existing setups continue to work. **Credential resolution:** * API key lookup now checks `JEV_API_KEY`, then `JEVWISE_API_KEY`, `TS_JEV_API_KEY` and `TYPESAFE_API_KEY`, skipping blank values. * MCP bearer token lookup prefers `JEVWISE_MCP_TOKEN`, then compatibility aliases `JEVWISE_MCP_AUTH_TOKEN` and `JEV_MCP_TOKEN`; a nonempty invalid higher-priority value fails without falling back. * The missing/invalid token error now points to `JEVWISE_MCP_TOKEN` or `--token-file`, and flag help and the `--token-file` override reference the new name. **Documentation and tests:** * Added an Authentication section to the README and expanded configuration, MCP and usage guides to describe the separate upstream and agent credentials and their fallback order. * Updated agent onboarding and smoke-test fixtures to export `JEVWISE_MCP_TOKEN` and `JEV_API_KEY`, and added env-name coverage tests.
benbenbang
enabled auto-merge (rebase)
October 11, 2026 03:09
github-actions
Bot
added
documentation
|
🎉 This PR is included in version 1.5.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Introduce canonical environment variable names for upstream API access and
local MCP agent authentication, keeping the previous names as ordered
fallbacks so existing setups continue to work.
Credential resolution:
JEV_API_KEY, thenJEVWISE_API_KEY,TS_JEV_API_KEYandTYPESAFE_API_KEY, skipping blank values.JEVWISE_MCP_TOKEN, then compatibilityaliases
JEVWISE_MCP_AUTH_TOKENandJEV_MCP_TOKEN; a nonempty invalidhigher-priority value fails without falling back.
JEVWISE_MCP_TOKENor--token-file, and flag help and the--token-fileoverride reference thenew name.
Documentation and tests:
MCP and usage guides to describe the separate upstream and agent
credentials and their fallback order.
JEVWISE_MCP_TOKENandJEV_API_KEY, and added env-name coverage tests.