From ccb00ee77d97eaed8077ab621cf17e72ca157038 Mon Sep 17 00:00:00 2001 From: agentson Date: Thu, 26 Feb 2026 01:39:45 +0900 Subject: [PATCH] =?UTF-8?q?fix:=20=EB=A1=9C=EA=B7=B8=20WARNING=202?= =?UTF-8?q?=EC=A2=85=20=EC=88=98=EC=A0=95=20-=20scanner=20=EC=98=A4?= =?UTF-8?q?=ED=95=B4=20=EB=A9=94=EC=8B=9C=EC=A7=80=20=EB=B0=8F=20=ED=99=80?= =?UTF-8?q?=EB=94=A9=20=EC=A2=85=EB=AA=A9=20rsi=20=EB=88=84=EB=9D=BD=20(#2?= =?UTF-8?q?67)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 1. WARNING → DEBUG: fallback_stocks 없어도 overseas ranking API로 scanner 정상 동작하므로 오해를 주는 WARNING 레벨을 DEBUG로 낮춤 (2곳) 2. 홀딩 종목 market_data 보강: scanner를 통하지 않은 종목(NVDA 등)에 price_change_pct 기반 implied_rsi와 volume_ratio=1.0 기본값 설정, scenario_engine 조건 평가 완전화 3. test_main.py: 새로운 동작에 맞게 관련 테스트 2개 업데이트 Co-Authored-By: Claude Sonnet 4.6 --- src/main.py | 14 ++++++++++---- tests/test_main.py | 14 ++++++++------ 2 files changed, 18 insertions(+), 10 deletions(-) diff --git a/src/main.py b/src/main.py index 14c8e40..3b652bb 100644 --- a/src/main.py +++ b/src/main.py @@ -581,6 +581,11 @@ async def trading_cycle( if candidate: market_data["rsi"] = candidate.rsi market_data["volume_ratio"] = candidate.volume_ratio + else: + # Holding stocks not in scanner: derive implied RSI from price change, + # volume_ratio defaults to 1.0 (no surge data available). + market_data["rsi"] = max(0.0, min(100.0, 50.0 + price_change_pct * 2.0)) + market_data["volume_ratio"] = 1.0 # Enrich market_data with holding info for SELL/HOLD scenario conditions open_pos = get_open_position(db_conn, stock_code, market.code) @@ -1499,8 +1504,9 @@ async def run_daily_session( active_stocks={}, ) if not fallback_stocks: - logger.warning( - "No dynamic overseas symbol universe for %s; scanner cannot run", + logger.debug( + "No dynamic overseas symbol universe for %s;" + " scanner will use overseas ranking API", market.code, ) try: @@ -2812,9 +2818,9 @@ async def run(settings: Settings) -> None: active_stocks=active_stocks, ) if not fallback_stocks: - logger.warning( + logger.debug( "No dynamic overseas symbol universe for %s;" - " scanner cannot run", + " scanner will use overseas ranking API", market.code, ) diff --git a/tests/test_main.py b/tests/test_main.py index a8a0aa7..3f15a00 100644 --- a/tests/test_main.py +++ b/tests/test_main.py @@ -1668,10 +1668,10 @@ class TestScenarioEngineIntegration: scan_candidates={"US": {"005930": us_candidate}}, # Wrong market ) - # Should NOT have rsi/volume_ratio because candidate is under US, not KR + # Should NOT use US candidate's rsi (=15.0); fallback implied_rsi used instead market_data = engine.evaluate.call_args[0][2] - assert "rsi" not in market_data - assert "volume_ratio" not in market_data + assert market_data["rsi"] != 15.0 # US candidate's rsi must be ignored + assert market_data["volume_ratio"] == 1.0 # Fallback default @pytest.mark.asyncio async def test_scenario_engine_called_without_scanner_data( @@ -1702,11 +1702,13 @@ class TestScenarioEngineIntegration: scan_candidates={}, # No scanner data ) - # Should still work, just without rsi/volume_ratio + # Holding stocks without scanner data use implied_rsi (from price_change_pct) + # and volume_ratio=1.0 as fallback, so rsi/volume_ratio are always present. engine.evaluate.assert_called_once() market_data = engine.evaluate.call_args[0][2] - assert "rsi" not in market_data - assert "volume_ratio" not in market_data + assert "rsi" in market_data # Implied RSI from price_change_pct=2.5 → 55.0 + assert market_data["rsi"] == pytest.approx(55.0) + assert market_data["volume_ratio"] == 1.0 assert market_data["current_price"] == 50000.0 @pytest.mark.asyncio