feat: US minimum price entry filter (#320) #340

agentson · 2026-02-28T14:40:26+09:00

agentson commented

2026-02-28 14:40:26 +09:00

Summary

Add US_MIN_PRICE setting (default 5.0)
Suppress BUY decisions for US markets when current price is below/equal to threshold
Apply the same filter in both realtime and daily execution paths

Validation

pytest -q tests/test_main.py -k "bearish_outlook_allows_high_confidence_buy or bullish_outlook_lowers_buy_confidence_threshold or order_policy_rejection_skips_order_execution"

## Summary - Add `US_MIN_PRICE` setting (default 5.0) - Suppress BUY decisions for US markets when current price is below/equal to threshold - Apply the same filter in both realtime and daily execution paths ## Validation - pytest -q tests/test_main.py -k "bearish_outlook_allows_high_confidence_buy or bullish_outlook_lowers_buy_confidence_threshold or order_policy_rejection_skips_order_execution"

agentson added 1 commit 2026-02-28 14:40:26 +09:00

feat: block US BUY entries below minimum price threshold (#320 )

Gitea CI / test (pull_request) Waiting to run

Details

Gitea CI / test (push) Has been cancelled

Details

08607eaa56

agentson commented

2026-02-28 16:12:00 +09:00

Ticket Maturity Update (2026-02-28)

Implemented: done
Integrated: done
Observed: done (targeted test commands executed and passing)
Accepted: pending (reviewer + runtime verifier approval required)

Coverage Matrix:

Stage	Status
Implemented	PASS
Integrated	PASS
Observed	PASS
Accepted	NOT_OBSERVED

## Ticket Maturity Update (2026-02-28) - Implemented: done - Integrated: done - Observed: done (targeted test commands executed and passing) - Accepted: pending (reviewer + runtime verifier approval required) Coverage Matrix: | Stage | Status | | --- | --- | | Implemented | PASS | | Integrated | PASS | | Observed | PASS | | Accepted | NOT_OBSERVED |

agentson reviewed 2026-02-28 16:16:07 +09:00

agentson left a comment

Review: PR #340 — feat: US minimum price entry filter (#320)

플랜 일치: ✅ ACT-03 정확히 일치

US_MIN_PRICE 설정(기본 5.0), US 시장 BUY 시 가격 이하 차단, KR 미적용

코드 품질: ✅ 양호

config.py: US_MIN_PRICE: float = Field(default=5.0, ge=0.0) — pydantic 검증 포함
main.py: market.code.startswith("US") 로 US 시장만 필터링
실시간 + 일간 2곳 모두 적용
로그: suppression 사유와 가격 정보 기록

테스트: ❌ 없음 — BLOCKING

ACT-03 테스트 계획에 명시된 케이스 모두 누락:
- ❌ 가격별 필터 동작 테스트 (경계값: .99, .00, .01)
- ❌ KR 시장 미적용 확인 테스트
프로젝트 정책: "All changes require tests" + "coverage >= 80%"

우려사항

current_price <= min_price — <= 사용으로 정확히 .00도 차단됨. ACT-03 계획의 " 이하 차단"과 일치하므로 의도된 동작이나, 경계값 테스트로 확인 필요

결론: Changes Requested — 테스트 추가 필수

## Review: PR #340 — feat: US minimum price entry filter (#320) ### 플랜 일치: ✅ ACT-03 정확히 일치 - US_MIN_PRICE 설정(기본 5.0), US 시장 BUY 시 가격 이하 차단, KR 미적용 ### 코드 품질: ✅ 양호 - **config.py**: `US_MIN_PRICE: float = Field(default=5.0, ge=0.0)` — pydantic 검증 포함 - **main.py**: `market.code.startswith("US")` 로 US 시장만 필터링 - **실시간 + 일간 2곳 모두 적용** - **로그**: suppression 사유와 가격 정보 기록 ### 테스트: ❌ 없음 — **BLOCKING** - ACT-03 테스트 계획에 명시된 케이스 모두 누락: - ❌ 가격별 필터 동작 테스트 (경계값: .99, .00, .01) - ❌ KR 시장 미적용 확인 테스트 - 프로젝트 정책: "All changes require tests" + "coverage >= 80%" ### 우려사항 - `current_price <= min_price` — `<=` 사용으로 정확히 .00도 차단됨. ACT-03 계획의 " 이하 차단"과 일치하므로 의도된 동작이나, 경계값 테스트로 확인 필요 ### 결론: **Changes Requested** — 테스트 추가 필수

agentson added 1 commit 2026-02-28 17:15:12 +09:00

test: add US minimum price boundary and KR-scope coverage (#320 )

Gitea CI / test (push) Waiting to run

Details

Gitea CI / test (pull_request) Waiting to run

Details

9267f1fb77

agentson commented

2026-02-28 17:15:17 +09:00

리뷰 요청 반영 완료했습니다.

추가 테스트:

US 경계값 테스트: 4.99 / 5.00 / 5.01 (5.00 이하는 차단, 5.01은 허용)
KR 시장 비적용 테스트: 가격 5.0 이하여도 KR BUY는 차단되지 않음

검증:

pytest -q tests/test_main.py -k "us_min_price_filter_boundary or us_min_price_filter_not_applied_to_kr_market"
결과: 4 passed

리뷰 요청 반영 완료했습니다. 추가 테스트: - US 경계값 테스트: 4.99 / 5.00 / 5.01 (5.00 이하는 차단, 5.01은 허용) - KR 시장 비적용 테스트: 가격 5.0 이하여도 KR BUY는 차단되지 않음 검증: - `pytest -q tests/test_main.py -k "us_min_price_filter_boundary or us_min_price_filter_not_applied_to_kr_market"` - 결과: 4 passed

agentson reviewed 2026-02-28 17:42:12 +09:00

agentson left a comment

Re-review: PR #340 — feat: US minimum price entry filter (#320)

이전 리뷰 지적사항 반영 확인

경계값 테스트: ✅ 추가됨
- @pytest.mark.parametrize 로 $4.99(차단), $5.00(차단), $5.01(허용) 경계값 테스트
- should_block 파라미터로 차단/허용 양방향 검증
KR 시장 미적용 테스트: ✅ 추가됨
- test_us_min_price_filter_not_applied_to_kr_market: KR 시장에서 price=4.0이어도 주문 실행 확인

코드 품질

구현 코드는 이전과 동일 (변경 없음) — 올바른 접근
테스트가 실시간 경로(trading_cycle)를 통한 E2E 테스트로 작성됨 — 적절한 수준

결론: LGTM ✅

## Re-review: PR #340 — feat: US minimum price entry filter (#320) ### 이전 리뷰 지적사항 반영 확인 1. **경계값 테스트**: ✅ 추가됨 - `@pytest.mark.parametrize` 로 $4.99(차단), $5.00(차단), $5.01(허용) 경계값 테스트 - should_block 파라미터로 차단/허용 양방향 검증 2. **KR 시장 미적용 테스트**: ✅ 추가됨 - `test_us_min_price_filter_not_applied_to_kr_market`: KR 시장에서 price=4.0이어도 주문 실행 확인 ### 코드 품질 - 구현 코드는 이전과 동일 (변경 없음) — 올바른 접근 - 테스트가 실시간 경로(trading_cycle)를 통한 E2E 테스트로 작성됨 — 적절한 수준 ### 결론: **LGTM** ✅

jihoson merged commit 9456d66de4 into feature/v3-session-policy-stream

2026-02-28 18:22:28 +09:00

jihoson deleted branch feature/issue-320-us-min-price-filter

2026-02-28 18:22:28 +09:00

jihoson referenced this issue from a commit

2026-02-28 18:22:29 +09:00

Merge pull request 'feat: US minimum price entry filter (#320)' (#340) from feature/issue-320-us-min-price-filter into feature/v3-session-policy-stream

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#340