[V2-SPEC] model_exit_signal 보조/직접 트리거 사양 불일치 #369

New Issue

agentson · 2026-03-02T01:01:37+09:00

agentson commented

2026-03-02 01:01:37 +09:00

Summary

v2 원문 계획은 모델 확률 신호를 trailing/리스크의 보조 트리거로 명시하지만,
현재 구현은 model_exit_signal이 EXIT 직접 조건으로 동작한다.

Evidence

계획 문구: docs/ouroboros/source/ouroboros_plan_v2.txt:75
EXIT 우선조건에 직접 포함: src/strategy/position_state_machine.py:43
reason 분기에서도 단독 청산 사유로 처리: src/strategy/exit_rules.py:88-89

Problem

문서 의도와 런타임 동작이 달라 청산 빈도/성능 해석이 왜곡될 수 있다.

Scope

src/strategy/position_state_machine.py
src/strategy/exit_rules.py
tests/test_strategy_exit_rules.py
docs/ouroboros/80_implementation_audit.md

Acceptance Criteria

정책을 "보조 트리거"로 유지할지 "직접 트리거"로 확정하고 문서/코드 중 하나를 정합화한다.
선택한 정책에 맞는 회귀 테스트를 추가한다.
구현 감사 문서의 REQ-V2-004 상태와 설명을 동기화한다.

## Summary v2 원문 계획은 모델 확률 신호를 trailing/리스크의 보조 트리거로 명시하지만, 현재 구현은 `model_exit_signal`이 EXIT 직접 조건으로 동작한다. ## Evidence - 계획 문구: `docs/ouroboros/source/ouroboros_plan_v2.txt:75` - EXIT 우선조건에 직접 포함: `src/strategy/position_state_machine.py:43` - reason 분기에서도 단독 청산 사유로 처리: `src/strategy/exit_rules.py:88-89` ## Problem 문서 의도와 런타임 동작이 달라 청산 빈도/성능 해석이 왜곡될 수 있다. ## Scope - `src/strategy/position_state_machine.py` - `src/strategy/exit_rules.py` - `tests/test_strategy_exit_rules.py` - `docs/ouroboros/80_implementation_audit.md` ## Acceptance Criteria - 정책을 "보조 트리거"로 유지할지 "직접 트리거"로 확정하고 문서/코드 중 하나를 정합화한다. - 선택한 정책에 맞는 회귀 테스트를 추가한다. - 구현 감사 문서의 REQ-V2-004 상태와 설명을 동기화한다.

agentson commented

2026-03-02 01:12:47 +09:00

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

등록 적합성: ✅ 적합

사양(문서)과 구현(코드) 간 불일치를 정확히 지목하고 있습니다. Acceptance Criteria에서 '보조 트리거 유지 vs 직접 트리거 확정' 중 하나를 선택해 정합화하도록 명시한 점이 좋습니다. 사양 해석 충돌 이슈는 코드 수정 전 정책 결정이 선행되어야 하는 만큼, 담당자가 먼저 정책 방향을 결정 후 작업하는 것을 권장합니다.

## 이슈 검토 결과 (Claude Code 자동 리뷰) ### 등록 적합성: ✅ 적합 사양(문서)과 구현(코드) 간 불일치를 정확히 지목하고 있습니다. Acceptance Criteria에서 '보조 트리거 유지 vs 직접 트리거 확정' 중 하나를 선택해 정합화하도록 명시한 점이 좋습니다. 사양 해석 충돌 이슈는 코드 수정 전 정책 결정이 선행되어야 하는 만큼, 담당자가 먼저 정책 방향을 결정 후 작업하는 것을 권장합니다.

agentson referenced this issue

2026-03-02 01:57:21 +09:00

docs: resync implementation audit status with actual code gaps (#373) #380

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#369