[FX-ACCOUNTING] TKT-P1-006 전략/환율 PnL 분리 회계 #296

agentson · 2026-02-27T08:44:15+09:00

agentson commented

2026-02-27 08:44:15 +09:00

목적/범위

거래 로그에서 전략 손익(strategy_pnl)과 환율 손익(fx_pnl)을 분리 저장
기존 pnl 기반 로직은 후방 호환 유지

시나리오/분류

[FX-ACCOUNTING]

추적 ID 매핑

Type	IDs
REQ	`REQ-V3-007`
TASK	`TASK-V3-013`, `TASK-CODE-011`
TEST	`TEST-ACC-006`

구현 내용

src/db.py
- trades 스키마: strategy_pnl, fx_pnl 컬럼 추가
- 마이그레이션: 기존 DB에 신규 컬럼 자동 추가
- log_trade 확장: 분리값 저장 + legacy pnl 호출 호환
tests/test_db.py
- 스키마/마이그레이션 컬럼 검증
- 분리 PnL 저장 및 legacy 호환 동작 검증

검증 증적

pytest -q tests/test_db.py -> 12 passed
pytest -q -> 950 passed
timeout 10s python3 -m src.main --mode paper -> 기동/종료 정상

Closes #295

## 목적/범위 - 거래 로그에서 전략 손익(`strategy_pnl`)과 환율 손익(`fx_pnl`)을 분리 저장 - 기존 `pnl` 기반 로직은 후방 호환 유지 ## 시나리오/분류 - `[FX-ACCOUNTING]` ## 추적 ID 매핑 | Type | IDs | | --- | --- | | REQ | `REQ-V3-007` | | TASK | `TASK-V3-013`, `TASK-CODE-011` | | TEST | `TEST-ACC-006` | ## 구현 내용 - `src/db.py` - trades 스키마: `strategy_pnl`, `fx_pnl` 컬럼 추가 - 마이그레이션: 기존 DB에 신규 컬럼 자동 추가 - `log_trade` 확장: 분리값 저장 + legacy `pnl` 호출 호환 - `tests/test_db.py` - 스키마/마이그레이션 컬럼 검증 - 분리 PnL 저장 및 legacy 호환 동작 검증 ## 검증 증적 - `pytest -q tests/test_db.py` -> `12 passed` - `pytest -q` -> `950 passed` - `timeout 10s python3 -m src.main --mode paper` -> 기동/종료 정상 Closes #295

agentson added 1 commit 2026-02-27 08:44:15 +09:00

feat: separate strategy and fx pnl fields in trade logs (TASK-CODE-011) 7bc4e88335

agentson added 1 commit 2026-02-27 08:46:24 +09:00

fix: backfill split pnl migration and harden partial pnl inputs 34cf081c96

agentson commented

2026-02-27 08:46:51 +09:00

[Dev Agent] 의사결정 기록: 리뷰 지적(high/medium) 반영으로 (1) migration 시 legacy pnl->strategy_pnl backfill 추가 (2) partial 입력에서 비직관 음수 strategy_pnl 제거 (3) migration/partial-input 회귀 테스트 추가. 검증: test_db 13 passed, full pytest 951 passed.