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/news-collector/src/news_collector/config.py | 3 ---
 1 file changed, 3 deletions(-)

(limited to 'services/news-collector/src/news_collector')

diff --git a/services/news-collector/src/news_collector/config.py b/services/news-collector/src/news_collector/config.py
index 70d98f1..6e78eba 100644
--- a/services/news-collector/src/news_collector/config.py
+++ b/services/news-collector/src/news_collector/config.py
@@ -5,6 +5,3 @@ from shared.config import Settings
 
 class NewsCollectorConfig(Settings):
     health_port: int = 8084
-    finnhub_api_key: str = ""
-    news_poll_interval: int = 300
-    sentiment_aggregate_interval: int = 900
-- 
cgit v1.2.3