fix: market_data에 unrealized_pnl_pct/holding_days 추가하여 SELL 시나리오 정상화 (#259) #263

agentson · 2026-02-26T00:23:38+09:00

agentson commented

2026-02-26 00:23:38 +09:00

Summary

get_open_position()에 timestamp 컬럼 추가
trading_cycle() market_data 구성 시 open_position 조회 후 보유 종목 정보 enrichment:
- unrealized_pnl_pct: (현재가 - 매수가) / 매수가 × 100
- holding_days: 매수일로부터 경과 일수

영향

기존: NVDA 38주 보유 중이지만 Condition requires 'unrealized_pnl_pct' but key missing 경고와 함께 모든 시나리오 평가 실패 → confidence=0 HOLD 고착

수정 후: take-profit(unrealized_pnl_pct > 4%), stop-loss(RSI<45 AND pnl<-2.5%) 시나리오 정상 평가

Test plan

전체 테스트 888개 통과

Closes #259

## Summary - `get_open_position()`에 `timestamp` 컬럼 추가 - `trading_cycle()` market_data 구성 시 open_position 조회 후 보유 종목 정보 enrichment: - `unrealized_pnl_pct`: (현재가 - 매수가) / 매수가 × 100 - `holding_days`: 매수일로부터 경과 일수 ## 영향 기존: NVDA 38주 보유 중이지만 `Condition requires 'unrealized_pnl_pct' but key missing` 경고와 함께 모든 시나리오 평가 실패 → confidence=0 HOLD 고착 수정 후: take-profit(unrealized_pnl_pct > 4%), stop-loss(RSI<45 AND pnl<-2.5%) 시나리오 정상 평가 ## Test plan - [x] 전체 테스트 888개 통과 Closes #259

agentson added 2 commits 2026-02-26 00:23:38 +09:00

fix: mock_settings에 MODE='paper' 명시하여 paper 모드 테스트 실패 수정 (#261 ) 3e777a5ab8

mock_settings fixture에 MODE 미지정 시 .env의 MODE=live가 적용되어
paper TR_ID를 검증하는 테스트 3개가 실패.

- test_buy_market_order: VTTT1002U 기대 → TTTT1002U 실제
- test_sell_limit_order: VTTT1001U 기대 → TTTT1006U 실제
- test_us_paper_uses_vttt1004u: VTTT1004U 기대 → TTTT1004U 실제

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: market_data에 unrealized_pnl_pct/holding_days 추가하여 SELL 시나리오 정상화 (#259 )

CI / test (pull_request) Has been cancelled

Details

7aa5fedc12

trading_cycle()의 market_data에 보유 포지션 정보가 없어
Condition requires 'unrealized_pnl_pct' but key missing from market_data 경고 발생.
보유 종목(NVDA 등)의 take-profit/stop-loss 시나리오가 평가 불가하여 HOLD(confidence=0) 고착.

- get_open_position()에 timestamp 컬럼 추가
- market_data 구성 시 open_position 조회 후 아래 키 추가:
  - unrealized_pnl_pct: (current_price - entry_price) / entry_price * 100
  - holding_days: 매수일로부터 경과 일수

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

jihoson merged commit 6a6d3bd631 into main

2026-02-26 00:23:56 +09:00

jihoson referenced this issue from a commit

2026-02-26 00:23:56 +09:00

Merge pull request 'fix: market_data에 unrealized_pnl_pct/holding_days 추가하여 SELL 시나리오 정상화 (#259)' (#263) from feature/issue-259-market-data-pnl-holding-days into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#263