Fix: gemini_client.decide() ignores prompt_override (#143) #144

agentson · 2026-02-18T02:02:34+09:00

agentson commented

2026-02-18 02:02:34 +09:00

Summary

decide() 메서드가 market_data["prompt_override"]를 무시하고 항상 일반 트레이드 결정 프롬프트를 생성하던 버그 수정
pre_market_planner가 Playbook 전용 프롬프트를 전달해도 Gemini가 일반 결정을 반환하여 JSONDecodeError 발생 → 모든 시장에서 defensive playbook 폴백

Changes

src/brain/gemini_client.py: prompt_override 키가 있으면 해당 프롬프트를 직접 사용 (1줄 추가)
tests/test_brain.py: 3개 테스트 추가 (override 전달, optimization 우회, 기존 동작 유지)
docs/requirements-log.md: 요구사항 기록 추가

Test Plan

27 brain tests 통과 (기존 24 + 신규 3)
전체 597 tests 통과
실제 run_overnight 실행으로 랭킹 API 동작 확인 (50 candidates per exchange)

Closes #143

## Summary - `decide()` 메서드가 `market_data["prompt_override"]`를 무시하고 항상 일반 트레이드 결정 프롬프트를 생성하던 버그 수정 - `pre_market_planner`가 Playbook 전용 프롬프트를 전달해도 Gemini가 일반 결정을 반환하여 `JSONDecodeError` 발생 → 모든 시장에서 defensive playbook 폴백 ## Changes - `src/brain/gemini_client.py`: `prompt_override` 키가 있으면 해당 프롬프트를 직접 사용 (1줄 추가) - `tests/test_brain.py`: 3개 테스트 추가 (override 전달, optimization 우회, 기존 동작 유지) - `docs/requirements-log.md`: 요구사항 기록 추가 ## Test Plan - [x] 27 brain tests 통과 (기존 24 + 신규 3) - [x] 전체 597 tests 통과 - [x] 실제 `run_overnight` 실행으로 랭킹 API 동작 확인 (50 candidates per exchange) Closes #143

agentson added 1 commit 2026-02-18 02:02:35 +09:00

fix: use prompt_override in gemini_client.decide() for playbook generation

CI / test (pull_request) Has been cancelled

Details

f7289606fc

decide() ignored market_data["prompt_override"], always building a generic
trade-decision prompt. This caused pre_market_planner playbook generation
to fail with JSONDecodeError on every market, falling back to defensive
playbooks. Now prompt_override takes priority over both optimization and
standard prompt building.

Closes #143

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

jihoson merged commit c5a8982122 into main

2026-02-18 02:05:51 +09:00

jihoson referenced this issue from a commit

2026-02-18 02:05:53 +09:00

Merge pull request 'Fix: gemini_client.decide() ignores prompt_override (#143)' (#144) from feature/issue-143-fix-prompt-override 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#144