feat: Decision outcome 업데이트 (issue #92) #120

agentson · 2026-02-14T22:38:11+09:00

agentson commented

2026-02-14 22:38:11 +09:00

Summary

trades 테이블에 decision_id 컬럼 추가 (CREATE + ALTER 마이그레이션)
log_trade()에 quantity, price, pnl, decision_id 파라미터 추가
SELL 실행 시 원본 BUY decision의 outcome(pnl, accuracy) 업데이트
get_latest_buy_trade() 헬퍼 함수 추가
realtime/daily 양쪽 모드에 동일 로직 적용

Test plan

BUY→SELL→outcome 업데이트 통합 테스트 추가
498/498 테스트 통과
ruff lint 클린

Closes #92

🤖 Generated with Claude Code

## Summary - trades 테이블에 `decision_id` 컬럼 추가 (CREATE + ALTER 마이그레이션) - `log_trade()`에 `quantity`, `price`, `pnl`, `decision_id` 파라미터 추가 - SELL 실행 시 원본 BUY decision의 outcome(pnl, accuracy) 업데이트 - `get_latest_buy_trade()` 헬퍼 함수 추가 - realtime/daily 양쪽 모드에 동일 로직 적용 ## Test plan - [x] BUY→SELL→outcome 업데이트 통합 테스트 추가 - [x] 498/498 테스트 통과 - [x] ruff lint 클린 Closes #92 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agentson added 1 commit 2026-02-14 22:38:11 +09:00

feat: link decision outcomes to trades via decision_id (issue #92 )

CI / test (pull_request) Has been cancelled

Details

a14f944fcc

Add decision_id column to trades table, capture log_decision() return
value, and update original BUY decision outcome on SELL execution.

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

jihoson merged commit bde54c7487 into main

2026-02-14 22:41:29 +09:00

jihoson referenced this issue from a commit

2026-02-14 22:41:31 +09:00

Merge pull request 'feat: Decision outcome 업데이트 (issue #92)' (#120) from feature/issue-92-decision-outcome 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#120