feat(settings): expose FRED/iWenCai API keys and visualize Feature Flags in Settings UI#300
Open
skloxo wants to merge 6 commits into
Open
feat(settings): expose FRED/iWenCai API keys and visualize Feature Flags in Settings UI#300skloxo wants to merge 6 commits into
skloxo wants to merge 6 commits into
Conversation
The bash/background_run tools were silently disabled when using SSE/API transport because include_shell_tools defaulted to False for non-stdio modes. This forced operators to discover and set VIBE_TRADING_ENABLE_SHELL_TOOLS=1 — an undocumented requirement. - api_server.py: trust authenticated requests (API_AUTH_KEY present) by default; explicit VIBE_TRADING_ENABLE_SHELL_TOOLS still overrides. - mcp_server.py: default to True for SSE when no explicit env var is set (MCP SSE is a local integration layer). Explicit env var still overrides for production hardening. Fixes the common deployment pitfall where agents lose bash capability after switching from stdio to SSE/API transport.
Collaborator
|
Thanks for the PR — we’ve seen it and will review as soon as we can. For settings changes, we’ll check that secrets stay local/config-driven, UI strings go through i18n, and feature-flag display does not expose anything unsafe or misleading. |
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
2 participants
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.
This PR exposes essential environment parameters (like macroeconomic data keys and semantic screening keys) to the frontend Settings page, and adds status indicator cards for core runtime components.
Key Changes
Verification