fix: add legacy API key field names to Settings

Add ALPHA_VANTAGE_API_KEY and NEWSAPI_KEY for backward compatibility with existing .env configurations. Fixes test failures in test_volatility.py where Settings validation rejected extra fields from environment variables. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Merge pull request 'feat: implement Data Driven - External data integration (issue #22 )' (#29 ) from feature/issue-22-data-driven into main
2026-02-04 19:01:45 +09:00 · 2026-02-04 18:57:43 +09:00
1 changed files with 4 additions and 0 deletions
--- a/src/config.py
+++ b/src/config.py
@@ -24,6 +24,10 @@ class Settings(BaseSettings):
    NEWS_API_PROVIDER: str = "alphavantage"  # "alphavantage" or "newsapi"
    MARKET_DATA_API_KEY: str | None = None

+    # Legacy field names (for backward compatibility)
+    ALPHA_VANTAGE_API_KEY: str | None = None
+    NEWSAPI_KEY: str | None = None
+
    # Risk Management
    CIRCUIT_BREAKER_PCT: float = Field(default=-3.0, le=0.0)
    FAT_FINGER_PCT: float = Field(default=30.0, gt=0.0, le=100.0)
Author	SHA1	Message	Date
agentson	87556b145e	fix: add legacy API key field names to Settings Some checks failed CI / test (push) Has been cancelled Details Add ALPHA_VANTAGE_API_KEY and NEWSAPI_KEY for backward compatibility with existing .env configurations. Fixes test failures in test_volatility.py where Settings validation rejected extra fields from environment variables. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2026-02-04 19:01:45 +09:00
jihoson	645c761238	Merge pull request 'feat: implement Data Driven - External data integration (issue #22 )' (#29 ) from feature/issue-22-data-driven into main Some checks failed CI / test (push) Has been cancelled Details Reviewed-on: #29	2026-02-04 18:57:43 +09:00