[V3-GAP] strategy_pnl/fx_pnl 런타임 분리 계산 미적용 #370

Open
opened 2026-03-02 01:02:08 +09:00 by agentson · 1 comment
Collaborator

Summary

REQ-V3-007("전략 손익 vs 환율 손익 분리")의 스키마는 존재하지만,
실행 경로에서 strategy_pnl/fx_pnl이 계산/전달되지 않아 사실상 미활성 상태다.

Evidence

  • 기본값 경로(strategy_pnl=pnl, fx_pnl=0.0): src/db.py:255-257
  • 주요 체결 로그 호출에서 분리 인자 미전달: src/main.py:2133, src/main.py:3276

Problem

운영/분석에서 환율 손익 분해가 불가능해 성과 해석이 왜곡된다.

Scope

  • src/main.py (SELL/BUY 실거래 로그 경로)
  • src/db.py (기본값 폴백 유지 여부)
  • tests/test_main.py, tests/test_db.py
  • docs/ouroboros/80_implementation_audit.md

Acceptance Criteria

  • 실거래 경로에서 strategy_pnlfx_pnl을 분리 계산해 저장한다.
  • 분리 손익 합이 pnl과 일치하는 무결성 테스트를 추가한다.
  • 감사 문서 REQ-V3-007 상태를 코드 실체 기준으로 갱신한다.
## Summary REQ-V3-007("전략 손익 vs 환율 손익 분리")의 스키마는 존재하지만, 실행 경로에서 `strategy_pnl/fx_pnl`이 계산/전달되지 않아 사실상 미활성 상태다. ## Evidence - 기본값 경로(`strategy_pnl=pnl`, `fx_pnl=0.0`): `src/db.py:255-257` - 주요 체결 로그 호출에서 분리 인자 미전달: `src/main.py:2133`, `src/main.py:3276` ## Problem 운영/분석에서 환율 손익 분해가 불가능해 성과 해석이 왜곡된다. ## Scope - `src/main.py` (SELL/BUY 실거래 로그 경로) - `src/db.py` (기본값 폴백 유지 여부) - `tests/test_main.py`, `tests/test_db.py` - `docs/ouroboros/80_implementation_audit.md` ## Acceptance Criteria - 실거래 경로에서 `strategy_pnl`과 `fx_pnl`을 분리 계산해 저장한다. - 분리 손익 합이 `pnl`과 일치하는 무결성 테스트를 추가한다. - 감사 문서 REQ-V3-007 상태를 코드 실체 기준으로 갱신한다.
Author
Collaborator

이슈 검토 결과 (Claude Code 자동 리뷰)

등록 적합성: 적합

Evidence 코드 검증 완료:

  • db.py:255-257: strategy_pnl = pnl, fx_pnl = 0.0 기본값 폴백 확인
  • main.py:2133, main.py:3276: log_trade() 호출에 strategy_pnl/fx_pnl 분리 인자 미전달 확인

스키마는 존재하지만 실행 경로에서 분리 계산이 전달되지 않는 전형적인 '미활성 기능' 패턴입니다. 이슈 구조 적합합니다.

## 이슈 검토 결과 (Claude Code 자동 리뷰) ### 등록 적합성: ✅ 적합 **Evidence 코드 검증 완료:** - `db.py:255-257`: `strategy_pnl = pnl, fx_pnl = 0.0` 기본값 폴백 확인 - `main.py:2133`, `main.py:3276`: `log_trade()` 호출에 strategy_pnl/fx_pnl 분리 인자 미전달 확인 스키마는 존재하지만 실행 경로에서 분리 계산이 전달되지 않는 전형적인 '미활성 기능' 패턴입니다. 이슈 구조 적합합니다.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#370