From c0496919e91f110aeed7bc47b24ebc3b8348ee81 Mon Sep 17 00:00:00 2001
From: TheSiahxyz <164138827+TheSiahxyz@users.noreply.github.com>
Date: Thu, 2 Apr 2026 16:02:59 +0900
Subject: fix: remove SecretStr field overrides in service subclass configs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

StrategyConfig and NewsCollectorConfig redefined anthropic_api_key and
finnhub_api_key as plain str, shadowing the parent SecretStr type. This
caused AttributeError at runtime when .get_secret_value() was called.
The fields are already inherited from Settings — no need to redeclare.
---
 services/strategy-engine/src/strategy_engine/config.py | 4 ----
 1 file changed, 4 deletions(-)

(limited to 'services/strategy-engine')

diff --git a/services/strategy-engine/src/strategy_engine/config.py b/services/strategy-engine/src/strategy_engine/config.py
index 15f8588..9fd9c49 100644
--- a/services/strategy-engine/src/strategy_engine/config.py
+++ b/services/strategy-engine/src/strategy_engine/config.py
@@ -7,7 +7,3 @@ class StrategyConfig(Settings):
     symbols: list[str] = ["AAPL", "MSFT", "GOOGL", "AMZN", "TSLA"]
     timeframes: list[str] = ["1m"]
     strategy_params: dict = {}
-    selector_final_time: str = "15:30"
-    selector_max_picks: int = 3
-    anthropic_api_key: str = ""
-    anthropic_model: str = "claude-sonnet-4-20250514"
-- 
cgit v1.2.3